feat: make --gas-report JSON output compatible

Motivation

Closes: https://github.com/foundry-rs/foundry/issues/8789
Part of meta-ticket https://github.com/foundry-rs/foundry/issues/8794
Solution

Updates existing relevant tests to use Snapbox and adds additional tests for JSON output
Adds an additonal check on --gas-report when the --junit flag is specified as JUnit output is not supported
Adds a silent variable to mute unexpected output, this can be expanded in the future for other outputs that are mutually exclusive with the standard output
Each test suite ran with --json is written to an individual line to allow users to easily consume the output by parsing by line.
JSON formatted output looks like this:
{
   "gas":106715,
   "size":277,
   "functions":{
      "increment":{
         "increment()":{
            "calls":1,
            "min":43404,
            "mean":43404,
            "median":43404,
            "max":43404
         }
      },
      "number":{
         "number()":{
            "calls":257,
            "min":283,
            "mean":283,
            "median":283,
            "max":283
         }
      },
      "setNumber":{
         "setNumber(uint256)":{
            "calls":258,
            "min":23582,
            "mean":43384,
            "median":43542,
            "max":43866
         }
      }
   }
}

Compare to Markdown output:
| src/Counter.sol:Counter contract |                 |       |        |       |         |
|----------------------------------|-----------------|-------|--------|-------|---------|
| Deployment Cost                  | Deployment Size |       |        |       |         |
| 106715                           | 277             |       |        |       |         |
| Function Name                    | min             | avg   | median | max   | # calls |
| increment                        | 43404           | 43404 | 43404  | 43404 | 1       |
| number                           | 283             | 283   | 283    | 283   | 257     |
| setNumber                        | 23582           | 43224 | 43554  | 43866 | 258     |

ping @DaniPopes for final review

Hi @sakulstra thanks for your feedback, I've opened a new ticket here with your points as feature requests: https://github.com/foundry-rs/foundry/issues/9111 assigned to me


passenger: Error installing Phusion Passenger

https://gist.github.com/01ce5a4d4e81441219d0
it occurred during the installation of nginx with passenger using the following command
brew install nginx --with-passenger

/usr/bin/gcc-4.8  -Iext -Iext/common -fPIC -D_REENTRANT -I/usr/local/include -Wall -Wextra -Wno-unused-parameter -Wno-parentheses -Wpointer-arith -Wwrite-strings -Wno-long-long -Wno-missing-field-initializers -feliminate-unused-debug-symbols -feliminate-unused-debug-types -fvisibility=hidden -DVISIBILITY_ATTRIBUTE_SUPPORTED -Wno-attributes -ggdb -DHAS_ALLOCA_H -DHAVE_ACCEPT4 -DHAS_SFENCE -DHAS_LFENCE -DPASSENGER_DEBUG -DBOOST_DISABLE_ASSERTS -o buildout/apache2/mod_passenger.o -c ext/apache2/mod_passenger.c
ext/apache2/mod_passenger.c:25:19: fatal error: httpd.h: No such file or directory
 #include <httpd.h>
                   ^
compilation terminated.
rake aborted!

Run brew install homebrew/apache/httpd24

❯❯❯ brew install homebrew/apache/httpd24
/gsc/btl/linuxbrew/Cellar/apr/1.5.1/build-1/libtool --silent --mode=link /gsc/btl/linuxbrew/bin/gcc-4.9 -std=gnu99  -g -Wall -pthread  -Os -w -pipe -B/gsc/btl/linuxbrew/opt/glibc/lib -march=core2     -L/gsc/btl/linuxbrew/lib -Wl,--dynamic-linker=/gsc/btl/linuxbrew/opt/glibc/lib/ld-linux-x86-64.so.2 -Wl,-rpath,/gsc/btl/linuxbrew/lib    -o mod_authz_user.la -rpath NONE/libexec -module -avoid-version  mod_authz_user.lo
libtool: link: only absolute run-paths are allowed
/tmp/httpd24-s22575/httpd-2.4.12/modules/aaa/modules.mk:18: recipe for target 'mod_authz_user.la' failed
make[4]: *** [mod_authz_user.la] Error 1

Hey, how can I help with this?
I was running my install through Linuxbrew, not sure that they two platforms are apples to apples, but I may be able to find some insight into why it is not compiling properly

We need to find a package that provides httpd.h. The only one that I know of is apache.


Makyen: Watch shopdumps101.com

Makyen requests the watch of the watch_keyword shopdumps101\.com. See the Metasmoke search here and the Stack Exchange search here.

shopdumps101\.com has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Coverage remained the same at 65.091% when pulling d15c111c6ba662afddd7f6c73c8e9601a24ae474 on auto-blacklist-1524277548.7788475 into caa5f863ca666f887c6090b04f0bce243a4a96a7 on master.

!!/approve


fix(catch): fix catch typings

Description:
The catch operator is returning an observable with the type of the innerObservable (observable returned when error occur).
It should be a union of the outerObservable's type and the innerObservable's type :
function _catch<T, R>(this: Observable<T>, selector: (err: any, caught: Observable<T>) => ObservableInput<R>): Observable<T | R>

Coverage remained the same at 97.689% when pulling c6c80d904cdb3a4f4c7a9450a686ddfcd221a41b on ghetolay:catch_typings into 01e1343629c579b573f2a2e47c575bc4b0ae0877 on ReactiveX:master.

Would you mind share examples of It should be a union of the outerObservable's type?

Would be great to see the use case for this... catch allows you to handle an error and decide what to do.  There is nothing stopping you from doing something like...
o.catch((e, c) => c);

Which would make T -> R.  With this change then the output would always be a special union of two potentially different types.

Any example where you would return an observable with a different type on the catch would reveal the need of union but I think this apply for all case.
if we take the first example given on catch :
Observable.of(1, 2, 3, 4, 5)
   .map(n => {
 	   if (n == 4) {
 	     throw 'four!';
     }
	   return n;
   })
   .catch(err => Observable.of('I', 'II', 'III', 'IV', 'V'))
   .subscribe(x => console.log(x));

x should be of type number | string and not just number.
And in the case you use the same type like :
obsv$ = Observable.of(1)
  .catch( err => Observable.of(0) );

the type of obsv$ is Observable<number> but not because the catch returns that type, it's because the union number | number is equal to type number.

With this change then the output would always be a special union of two potentially different types.

@david-driscoll isn't it what exactly happens after a catch ? it's either the source observable of the one defined on catch, you can't know which one and thus can't know for sure the type of emitted item.

That is actually a fair point, I had forgotten about how that behavior would flow.  I have been corrected 👍


Add state delta to tx receipts

Tx index was added already in https://github.com/spacemeshos/api/commit/1b4ca35d24c0eb1aa1ae7980570080442e1bd9f0
This adds the second requirement, state delta
Closes #34

You should get @YaronWittenstein involved in this and in any API issues such as this which is SVM related - he's been working on svm data in receipts over the last few days...

What you propose is a list of decoded key-values. The approach I talked with Yaron and I think what he's building is having packed binary svm data in the receipts where it is the responsibility of clients to decode it (e.g. smapp and explore front-end) using the svm-codec lib. What you proposing will require the node to decode lots of svm data per receipt to create it at request time or at the time it is generated by the node.

We need to talk about the idea to log every variable change in svm. I think that the current thinking was to just include the updated storage root hash in the receipt and so clients can run the computation and compare storage hashes to validate vars change. It is not a bad idea to include (k,v) of every changed value in the receipt.

I think that GlobalState will do that without involving SVM.
This SMIP describes the interface between SVM and the Global State:
https://github.com/spacemeshos/SMIPS/issues/20

just include the updated storage root hash in the receipt and so clients can run the computation and compare storage hashes to validate vars change.

It's not reasonable to expect clients to "run the computation" - they don't have an SVM interpreter nor do they have all the context.

The STF component should be able to ask the GloblState for all the (k, v) changes and flush to disk at one call all the changes at once.

Great - then it shouldn't be too hard to include this in the tx receipt

just include the updated storage root hash in the receipt and so clients can run the computation and compare storage hashes to validate vars change.

It's not reasonable to expect clients to "run the computation" - they don't have an SVM interpreter nor do they have all the context.

@lrettig @avive I think I'm not following - I thought that the each Full-Node executes each transaction.
So the Receipt's outcode App State is only for validation.

The STF component should be able to ask the GloblState for all the (k, v) changes and flush to disk at one call all the changes at once.

Great - then it shouldn't be too hard to include this in the tx receipt

As I've written in the previous message here - Why do we need to save all the (k, v) changes as part of the Receipt?

just include the updated storage root hash in the receipt and so clients can run the computation and compare storage hashes to validate vars change.

It's not reasonable to expect clients to "run the computation" - they don't have an SVM interpreter nor do they have all the context.

@lrettig @avive I think I'm not following - I thought that the each Full-Node executes each transaction.
So the Receipt's outcode App State is only for validation.

By client I don't mean full node. I mean something like a web app.
Why include all state changes? Because even a full node can't calculate these for itself without storing all intermediate, historical state.

But the changes are sitting already in the Global State - why replicate that data to the Receipts?

Yes, the global state is updated as a side effect of executing a transaction, but the list of state changes from each tx is not stored by a full node (only maybe an archive node)

Closed due to inactivity


App Mesh A/B Testing

In order to support A/B Testing, App Mesh needs to implement routing based on headers and cookies similar to Flagger's Istio A/B Testing.
Depends on https://github.com/aws/aws-app-mesh-roadmap/issues/15 and https://github.com/aws/aws-app-mesh-roadmap/issues/14

@jaypipes Flagger interacts with App Mesh via the CRDs, the A/B Testing can be implemented in Flagger after https://github.com/aws/aws-app-mesh-controller-for-k8s/pull/56 is merged and released.


Add cipher, decipher, digest, etc. functions to command line tool

From @izaera on October 22, 2013 18:10
Copied from original issue: izaera/cipher#12

I think a command line tool would be best suited as a separate package depending on this library. It could very well be added to the Pointy Castle project of someone would be willing to contribute it :)

:+1:


[WIP] Fix leapp prints nothing when running with no args on python3.6.

A change introduced in the stdlib argparse module accidentally caused
all subparsers to become optional instead of required as in python 2.7.
This change therefore causes leapp to print nothing and exit with error
code 0 when being executed without any arguments.
This commit marks the subparser action as required, so that the
python3.6 module recognizes this situation and prints the usage message.
To provide a meaningful message on python3.6 the subparser action was
edited to use 'command' metavar and the 'title' option was dropped.
Tasks to be done:

[ ] verify that the usage message when using python2.7 did not become somehow unreadable due to the metavar/title changes performed.

Can one of the admins verify this patch?

The usage message running python2.7 without any arguments is changed by this PR to (notice the added metavar command):
usage: leapp [-h] [--version] command ...

Optional arguments:
  -h, --help  show this help message and exit
  --version   show program's version number and exit

Required arguments:
  command
    answer    Manage answerfile generation: register persistent user choices
              for specific dialog sections
    upgrade   Upgrade the current system to the next available major version.
    list-runs
              List previous Leapp upgrade executions
    preupgrade
              Generate preupgrade report
    rerun     Re-runs the upgrade from the given phase and using the
              information and progress from the last invocation of leapp
              upgrade.
error: too few arguments

Running leapp with no arguments on python3.6 outputs (with exit code 2):
usage: leapp [-h] [--version] command ...

Optional arguments:
  -h, --help  show this help message and exit
  --version   show program's version number and exit

Required arguments:
  command
    answer    Manage answerfile generation: register persistent user choices
              for specific dialog sections
    list-runs
              List previous Leapp upgrade executions
    preupgrade
              Generate preupgrade report
    rerun     Re-runs the upgrade from the given phase and using the
              information and progress from the last invocation of leapp
              upgrade.
    upgrade   Upgrade the current system to the next available major version.
error: the following arguments are required: command


Fixed some leftover capitalized api values

Also catch JSON.parse errors in DaemonAPI and renamed some package.json scripts

did you see my PR? I think this is in conflict with that one

@DavidVorick Yo dawg, I heard you like PR's. So I put a PR on your PR :P

ah whoops missed that


fix: trickle down s3_output in athena.to_iceberg

Feature or Bugfix

Bugfix

Detail

trickle down s3_output in athena.to_iceberg

Relates

#2766

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

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 14f03776eafce048436feb0b2c63b544dcd243ec
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 14f03776eafce048436feb0b2c63b544dcd243ec
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 3bec6ccadbc12010bc4b682168d31e3b16a5961f
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 3bec6ccadbc12010bc4b682168d31e3b16a5961f
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Effect of updated evaluation metrics

I found this commit,
Update evaluation metrics. Use GPT2 tokenizer for xlingual tokenizaion.
But I've done some experiments with the old one.

Well, I compared the results in README.md with your paper and it had an impact.


[Bug]: Coalesce doesn't work because it won't accept null values because of stringize...

What happened?
You can't actually pass the Coalesce function a null value, because of
TypeError  Tpetry\QueryExpressions\Function\Conditional\ManyArgumentsExpression::stringize():
Return value must be of type string|int|float, null returned.

The getValue of Coalesce calls getExpressions, which calls stringize, which doesn't accept or return nulls - to work around it not accepting nulls you can wrap the null in Expression, but then stringize crashes with the above noted error because it won't return a null.
How to reproduce the bug
Testing the intersection of 2 date ranges where either range could be open-ended ($end can be null and assignments.ends_at can be null)
$start = '2023-09-14';
$end = null;

Assignments::where($start, '<=', new \Tpetry\QueryExpressions\Function\Conditional\Coalesce(
[
    'assignments.ends_at',
    new \Illuminate\Database\Query\Expression($start)
]))
->where(new \Tpetry\QueryExpressions\Function\Conditional\Coalesce(
[
    new \Illuminate\Database\Query\Expression($end),
    'assignments.starts_at'
]), '<=', 'assignments.starts_at')->get();

No work arounds, you have to not use the package for this and revert to using DB::Raw();
Package Version
0.7.0
PHP Version
8.1
Laravel Version
10.19
Which operating systems does with happen with?
Windows
Notes
No response

Do you have an example why you want to pass NULL? It is automatically the result when not passed a non-null value. So passing NULL to it doesn‘t make sense for me.

To replicate:
// Intersection of 2 date ranges where either range could be open-ended
// (StartA <= Coalesce(EndB, StartA)) And (Coalesce(EndA, StartB) <= StartB)

Assignment::whereRaw('? <= COALESCE(ends_at, ?)', [$contract->starts_at, $contract->starts_at])
->whereRaw('COALESCE(?, starts_at) <= starts_at', [$contract->ends_at])
->get();

You've been using the expression wrong.
Any non-expression passed to an expression is used as a column name. So you can't pass in $contract->starts_at to the constructor as it would be recognized as a column name.
The correct approach is using the Value class which supports nullable values:
$query->where(new Coalesce([new Value($contract->starts_at), 'starts_at']), '<=', 'starts_at');

ooooh, that explains so much. Thank you.


BF: fix reading off-spec matlab logical sparse

See : https://github.com/scipy/scipy/issues/3539#issuecomment-68616473
MATLAB apparently likes to store sparse logical matrices with declared data
type miDOUBLE (float64) but actual data type is bytes.  Add special-case check
for this in the reading code and check reading an example matlab file.

Merged, thanks Matthew.


[Enum Request] Table 1523 "Workflow Step Argument" - "Approver Limit Type"

Hi
Could you please change this option to enum?
Of cource, same applies for tables:

"Workflow Step Argument Archive"
"Approval Entry"
"Posted Approval Entry"
"Overdue Approval Entry"

Thank you

Please be aware that fields Approver Limit Type exists in tables Workflow Step Argument and Workflow Step Argument Archive. This field has following set of options:  'Approver Chain,Direct Approver,First Qualified Approver,Specific Approver'.
Field Limit Type in table Approval entry contains different set of options 'Approval Limits,Credit Limits,Request Limits,No Limits'.
So these are two different enums.

Thank you for your feedback.
"Approver Limit Type" is what concerns me. "Limit type" is OK as it is.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


to do list

**In this PR I implemented the following: **

Implemented creating, updating and removing tasks in a todo list based on react tutorial

Good Job, I have reviewed your code for the to do list-React project, and I appreciate the effort you put into it. Overall, your code is well-structured and follows React best practices. it is time to merge your project.


iFrame does not work iPad iOS 8.3

Testing for issue https://github.com/phetsims/tasks/issues/380
iframe http://www.colorado.edu/physics/phet/dev/olsonjb/friction_en-iframe.html does not work on iPad iOS 8.3. Screen starts with the PhET loading then changes to a white screen that fills the web page.

It's my hypothesis that any sim with a 'DOM' element will probably fail this way, since we need to add a large container to the DOM temporarily.
The online recommendation (since iOS Safari seems to rescale the iframe to match the 65535x65535 dimensions) is to use scrolling="no" on the iframe, and this seems to resolve the issue for me.
@orejolaphet, are you able to test http://www.colorado.edu/physics/phet/dev/olsonjb/friction-iframe-test/iframe-test3.html to see if the same problem occurs? (It seems fixed on iOS 9 iPad 2 in the office).

Nice! http://www.colorado.edu/physics/phet/dev/olsonjb/friction-iframe-test/iframe-test3.html  Looks fixed on iOS 8.3 & iOS 9
Should note problem still occurs on  http://www.colorado.edu/physics/phet/dev/olsonjb/friction-iframe-test/iframe-test4.html

Thanks! Yes, iframe-test4.html does not include scrolling="no", but another method that I found didn't work.
Since this fix would require people using our "embed" iframe HTML with scrolling="no" (not just a plain vanilla iframe), I'd like to discuss it at the developer meeting to see whether it is acceptable. Notably, the iframe template now looks like:
<iframe src="PHET_SIM_URL" width="600" height="400" allowfullscreen scrolling="no"></iframe>

10/1/15 dev meeting:
• this is a problem with any sim that contains a DOM element
• this is specific to mobile Safari
• scrolling="no" seems to work
• we can specify what needs to be in the iframe tag
TODO:
• add to sim-iframe.html template (done by JO)
• test to see if this causes problems on other platforms (as part of https://github.com/phetsims/tasks/issues/375)
• change the embed snippet provided by the website (AD will do this after RC test passes)

Assigning to @aaronsamuel137. When this has been verified in the context of https://github.com/phetsims/tasks/issues/375, changed the "embed" HTML snippet on the website. See checklist above (https://github.com/phetsims/scenery/issues/472#issuecomment-144802216).

I've added this to the website, is there anywhere else that this needs to go? Assigning to @jonathanolson  for review.

Looks great, thanks!


Crash on alt-tab

I'm getting a crash consistently when alt-tabbing away from lite on Windows.
Error: C:\lite-dev\lite/data/core/init.lua:351: attempt to call field 'wait_event' (a nil value)
stack traceback:
C:\lite-dev\lite/data/core/init.lua:351: in function 'step'
C:\lite-dev\lite/data/core/init.lua:417: in function 'run'
[string "local core..."]:9: in function <[string "local core..."]:2>
[C]: in function 'xpcall'
[string "local core..."]:2: in main chunk

Sorry, never mind. My executable and Lua files were out of sync.

Not a problem! I've attached a windows build from the master branch if you want to continue running the master-branch lua without having to wait for the next release.
lite_master.zip

Thanks!


Android - any thoughts on iOS?

I am new to Cordova and was looking at building a Muse plugin for iOS.  Is this something that you are considering? Would be nice to enable both Android and iOS access, no?
Hope so... Greg

I would love to add iOS compatibility to this as well.  If you submit a pull request I will definitely review/merge it.

Matt,
Thanks for the response.  I am not the developer that you are but such is my goal.
I’ll let you know if I make any progress.
Greg

On Aug 29, 2015, at 6:59 AM, Matt Triff<EMAIL_ADDRESS>wrote:
I would love to add iOS compatibility to this as well. If you submit a pull request I will definitely review/merge it.
—
Reply to this email directly or view it on GitHub https://github.com/mtriff/cordova-muse/issues/1#issuecomment-135972281.

I'm willing to fund the development of an iOS port, albeit limited funds to get someone to assist me to do it. I will submit PR when done.

I know this is an old issue, but any progress on iOS integration?


Release 1.2.0

Codecov Report

Merging #531 (d1fb10b) into master (50cdb9d) will increase coverage by 0.61%.
The diff coverage is 28.00%.

:exclamation: Current head d1fb10b differs from pull request most recent head fa6437b.


Upgrade 'tar' package to avoid library vulnerability.

https://npmjs.com/advisories/803

Thanks. Fixed and published in v0.0.26


add and maintain a CHANGES file

...because it's hard to guess what's changed with every new release. :-)

Point taken.  Its been in my head for a while, but I've honestly not been sure how to start.  I'll do a bit of reading this week and take a stab at putting something in place.  What version are you on currently?

@xaque208 I've upgraded to 3.0.1 just today. Personally I like the style of the puppetlabs CHANGELOGs (i.e. https://github.com/puppetlabs/puppetlabs-postgresql/blob/master/CHANGELOG.md).

Hows this for a start? https://github.com/xaque208/puppet-bacula/pull/64

Changelog has been merged.  Thank you for the suggestion.

@xaque208 Great, thank you!


Rule E3004 is not checking for DependsOn declarations?

cfn-lint version: 0.8.2
Description of issue.
The following template has a circular dependency:
Resources:
  Resource:
    Type: AWS::SNS::Topic
    DependsOn: Resource2
  Resource2:
    Type: AWS::SNS::Topic
    DependsOn: Resource

but cfn-lint doesn't report any problems with it.

Yup, code and description in this file would need to be updated:
https://github.com/aws-cloudformation/cfn-python-lint/blob/35a1fa54e828ce5497683551e9c43b4b210b4227/src/cfnlint/rules/resources/CircularDependency.py#L13-L16
Couple other rules have examples grabbing DependsOn attribute values:
https://github.com/aws-cloudformation/cfn-python-lint/blob/b788cc9bd3d49ed20d5f2e58602755a0ef37f52c/src/cfnlint/rules/resources/DependsOn.py#L40
https://github.com/aws-cloudformation/cfn-python-lint/blob/4e210a2325ca225ff8483ec66f0ca9096cd4f73f/src/cfnlint/rules/resources/DependsOnObsolete.py#L64


Ensure that Group Chat "invited" and "removed" system messages show correctly in the LHN

Problem
Coming from @s77rt's comment here. These messages look strange in the LHN.
Solution
Workspace rooms do not have this problem. So presumably there is some issue in the backend we need to fix, but it might be a frontend issue as well. Actually, @jasperhuangg probably knows why and can answer this question faster than us.

@marcaaron this is expected. We don't show the email address of users since for some reports, we don't know whether or not the users are allowed to view them (like in public rooms)
We could distinguish between the public/non public rooms, but this isn't something that needs to be addressed now.

If you look at the link the LHN messages are broken so I think we need to apply the same change we did for rooms to group chats or something. This is what they look like for Group Chats:

So, feels like we missed a detail somewhere - but I haven't looked into where yet.

Looking at this one today.

Waiting for deploy.

This is done now.


Suggestion/Request: Config to let the "Goat Horn" item serve as the whistle

Minecraft added Goat Horns in 1.19 and AFAIK they're just a fun item without any actual purpose.
Could you make it so the Goat Horn toggles your pets between follow and independent when you fully "sound it"? I don't know if this is possible to do without client-side code but I'm very hopeful so! Either instead of the commands /indypets whistle and /indypets unwhistle or in addition to them.
If it could be configurable to enable/disable that'd be even better :)
And thank you for the awesome mod, this is a feature of tamed pets/followers that I will never be able to play Minecraft without again!

Goat Horns do have a purpose: They can be heard by other nearby players, so they can be used for communication.
IndyPets' whistle sound effect on the other hand is client-side only, so it's different in that aspect.
It is certainly possible to use Goat Horns as a toggle without client modification, though it's not quite clear how exactly they should work, so here's a bunch of questions/thoughts:
Should all Goat Horns have that behavior, or only certain ones?
Should they all toggle, or should e.g. one be for whistle, another one for unwhistle?
I imagine it could be a little confusing if the same sound could mean "follow me" but also "dismissed" - which is why I added separate sounds for the whistle and unwhistle hotkeys.
Hm... It should be theoretically possible to configure each type of horn separately, maybe like
/indypets horn <horn_type> <option>
where <horn_type> is the concrete type like ponder_goat_horn, or * for all of them at once and <option> would be something like toggle / whistle / unwhistle / disabled (default).
So to make all horns toggle, one could do /indypets horn * toggle.
About the "toggling":
The default whistle keybind doesn't actually toggle each pet, but it alternates between whistling and unwhistling.
That is so that when you e.g. come home and dismiss your pets, it doesn't make your other pets follow you.
I'd make Goat Horns the same way, though there's one caveat:
When the whistle/horn is in the wrong state, with the hotkey you can just immediately whistle again to fix it, but Goat Horns have a cooldown you'd need to wait for.
(Note to self: I need to save that state in the config, so it doesn't get lost on relog.)
You also said horns should toggle "when you fully sound it" - do you mean it should wait until the end of the sound?
Because of sounds being client-side, I don't think that's possible server-side, though I could maybe add a like 3s delay to it...?
Not sure if that's a good idea though.
Another random idea to consider:
Sounding the horn while standing: follow
Sounding the horn while sneaking: dismiss
Might be better than a toggle because it gives more control, and has no "wrong state and having to wait" issue.
I also thought about changing the pitch of the "unwhistle" horn sound to distinguish the sounds a little - though the thing I don't like about that is that this is kinda a feature in itself since other players can hear them - like it's almost "too cool" to just be about controlling your pets.
(Same with the whistle sound effects - even if it were possible, I'm still not sure I'd make them not client-side only.)

Progress!

The command is only for vanilla players (same as with the /indypets config command).
I didn't manage to add the * option due to limitations of Mojang's Brigadier - it seems this is only solvable with client-side modification.
I put a test release for 1.21(.1) here if you wanna give it a try.
It should be fully functional, just missing some small bibs and bobs here and there - or as one says:
The last 10% of the work are 90% of the work.

Just released 1.4.1 with the Goat Horn option!
Compared to the test version, I added a Whistle / Sneak Unwhistle option as well but otherwise it's pretty much the same.
Let me know if there's any issues.


fix: FluentSelect selected item disappears after OnInitializedAsync completes.

🐛 Bug Report
Using FluentSelect in a Blazor WebAssembly application, if the OnInitializedAsync includes an await call, then after the await call completes the FluentSelect is set to blank instead of its correct Selected value.
Below is a simplified reproduction based on one of the FluentSelect examples.  If you comment out the  "await Task.Delay (1000)" it works fine.
🔦 Context
The actual use case that brought this to light was trying to initialise a FluentSelect using a set of option string values obtained from a Web API (via an awaited call in OnIntializedAsync), and then having it pre-select the user's previous last choice from a string value stored in Blazor.LocalStorage.
In that more complex example everything in the underlying data structures had the correct values, right down to the correct Options entry in the options list having its Selected attribute set, but still the dropdown initially fails to reflect the Selected value.  Thereafter it works fine (chaning selected value, switching to other tabs in the app and back), but refreshing the page resets the dropdown to the first option, regardless of the underlying data values.
I haven't tested if the behaviour is different for a Blazor Server application.
💻 Repro or Code Sample
@page "/TESTPAGE"
From a List of Option<string> items
Second item in the list is initially selected through the OptionSelected (Func delegate) parameter.
<FluentSelect Items=@stringOptions1
OptionText="@(i => i.Text)"
OptionValue="@(i => i.Value)"
OptionSelected="@(i => i.Selected)"
@bind-Value="@stringValue" />

    Selected Value: @stringValue

@code {
string? stringValue;
static List<Option> stringOptions1 = new()
{
{ new Option { Value = "9", Text = "Nine", Icon = (new Icons.Regular.Size24.AppsList(), Color.Neutral, "start" ) } },
{ new Option { Value = "8", Text = "Eight", Selected = true } },
{ new Option { Value = "7", Text = "Seven" } },
{ new Option { Value = "6", Text = "Six" } },
{ new Option { Value = "5", Text = "Five" } },
{ new Option { Value = "4", Text = "Four" } },
{ new Option { Value = "3", Text = "Three" } },
{ new Option { Value = "2", Text = "Two" } },
{ new Option { Value = "1", Text = "One" } }
};

protected override async Task OnInitializedAsync ()
{
await base.OnInitializedAsync ();
await Task.Delay (1000);
}
}
🤔 Expected Behavior
The dropdown should show item "Eight" selected after > 1 second has elapsed.
😯 Current Behavior
Initially the dropdown shows item "Eight" selected.  After 1 second the dropdown display goes blank.
Tested Firefox, Edge & Chrome - all have same behaviour.
🌍 Your Environment
Visual Studio 2022, Windows 11
NET 8 Blazor WebAssembly web application,
using Microsoft.FluentUi.AspNetCore.Components 4.10.2
Microsoft.AspNetCore.Components.WebAssembly 8.0.10

Markdown messed up the code sample, but I think you can still get something usable from it.

It happens on Server as well. We'll look into it.


Improve JvmObjectTracker for long running jobs

@qualiu is working on this

Fixed in https://github.com/Microsoft/Mobius/commit/7d6ea24a8d8b151692a33890820e33852bf52059.
@qintao1976 is working on additional improvements.


Add docker creds to release to S3 task

What is the context of this PR?
This PR is to add the docker credentials to the release to S3 job which was missed off the original PR
How to review
Release pipeline runs without error

 Dockerhub error authentication with new creds

These creds will now be removed as they arent correctly authenticating anyway. The plan now is to replace this pipeline too


Krybception

Too many kryb:

http://localhost:3456/dannet/data/synset-3472
http://localhost:3456/dannet/data/synset-77864
http://localhost:3456/dannet/data/synset-77865
http://localhost:3456/dannet/data/synset-77866

Nathalie found this issue. Not sure whether it is something I will fix or whether it will come in as part of the overall changeds the lexicographers are deciding on.

These all seem to be cases of #60, however they contain #59 so at the very least new senses must be constructed. This issue should be dealt with in #59 instead.


Add option to invert servo angles

If your glove construction has a counter-clockwise rotation, you might need to set this to true to change the direction your servos move when going from EXTENDED to RETRACTED

A bit of context: My team and I are working on a new reel-based design that requires the servos be put in backwards from how the LucidVR firmware expects. By adding this option, one can easily adjust the direction that the servos turn to compensate for this backwards orientation.

This is already included in the proto 5 branch, will be on it's way to main once proto 5 firmware is ready for merge :)


Support for Compact Identifiers in the webservice API

Duplicate of https://github.com/bridgedb/BridgeDb/issues/83


Returning {} for Merge commits

It is returning {} for merge commits.

Thanks for your issue. I answered in the PR (#2)


Test against stack traces on various browsers

We know at the very least Safari outputs a weirdly formatted stack trace. We should test some other formats too!

[ ] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge
[ ] Node.js
[ ] IE11 (?)
[ ] Deno (?)

Probably not gonna happen at this point


feature request: Alternate menu bar icon(s)

To improve the intuitive association of the app's function, I suggest something that resembles a clipboard (offered in both black and white). I've also seen similar apps offer various shapes and orientations of scissors, a gear, etc. If you'd like some properly prepared files I can likely provide them. A dropdown under Appearance would make the most sense to select one.

Unfortunately, I don't have the capacity to work on this now, but I am open to merging a pull request adding new icons and settings dropdowns allowing you to choose between them.

I'm open to work on this,
Can you provide me some guidance, and which Icon should I put in the dropdown?

@FabyoSK Thanks for your interest. Roughly, you need to do the following:

Add a new row with NSPopupButton in AppearancePreferenceViewController.xib under Show menu icon.
This popup list would allow selecting from 3 different icons. Obviously, there will be a default one, but I suggest we use extra icons from https://developer.apple.com/sf-symbols/ since we can export them as SVG. The following makes the most sense to me:

name
icon

list.clipboard

scissors.circle

Add the SVG version of icons to Assets.xcassets. There are some examples there. We'll also need PNG versions for the older versions of macOS, but I can help with this once it works.
Extend NSImage+Names.swift to load new images.
Add a new string preference called menuIcon to UserDefaults+Configuration.swift. This would default to maccy.
When the popup list is changed, change the UserDefaults.standard.menuIcon accordingly.
Add a new observer attached to  UserDefaults.standard.menuIcon in Maccy.swift that would update statusItem icon whenever the preference changes. See initializeObservers() for examples.

It's probably not detailed, but feel free to ask anything along the way :)

Thank you for your shortly reply 😁
I'll spend some time running the project locally today, If I found any blockers I'll let you know :)

@p0deje hello again
I'm facing some issues running the app, since it don't have any guide, for some reason I'm not able to fetch the dependencies

Before I became aware of SF Symbols, I had mocked up a few options as transparent PNG. The Symbols look like a good option, but I would vote for shapes that are easier to visually acquire at small sizes (have more visual weight) amongst the often cluttered menubar. My votes would go to:
clipboard.fill

scissors

I can also be of assistance preparing the SVG or PNG alternatives for MacOS < 11.7.4 (Big Sur) as those cannot use SF Symbols. Thank you both for your interest in working on this.

@FabyoSK Strange, there is no 5.7.0 version of LaunchAtLogin https://github.com/sindresorhus/LaunchAtLogin/releases/.  Can you try removing the dependency and adding it back?

@darienm Those would work well too. I don't have a strong preference, though I think rounded scissors might look better on a menubar. In any event, we'll need 5 different assets for each icon:

Light mode 16x16 PNG
Light mode 32x32 PNG
Dark mode 16x16 PNG
Dark mode 32x32 PNG
SVG

If you can prepare those, it would be amazing.

@p0deje After some debug, I found out that the issue was my XCode, I needed to update it to XCode 14 in order to download the dependecies
But now I'm having this issue

I can't found the root cause of it

@p0deje NVM, I figure out that was the Sparkle framework with signin issues
Now I have the project up and running

I've put together a clipboard and a scissors, in both light and dark modes, in both 16x16 and 32x32 sizes for consideration. All PNG files with transparent backgrounds. SVGs also provided. Attached as Round 01. Let me know your thoughts. If these are working, I am happy to prepare others.
maccy_alt_icons_round_1.zip

@darienm Thank you, they look quite nice. I'm only curious to see why clipboard.fill proportions look so different between 16 and 32? It looks like the smaller version of the icon is much wider.

I took a small liberty with the individual pixels at the smaller size to make the shape of a clipboard more recognizable. It was intentional and not an artifact of the scaling.

I would prefer the icons to be similar to each other for consistency, but frankly, I never use such a huge resolution where the smaller icon version is being used. @darienm Do you have a strong opinion on keeping them different?

I'm providing consistent, narrow versions of the 16px clipboards in this attachment. I can't tell if an effort is being made to vertically position the icons in the menubar based on the screenshot provided. clipboard.fill.narrow_16.zip

@darienm I think these look better, thanks!

Can you provide the Copy icon?
And also provide feedback on the design?
the dropdown should be below or on the side of Show menu icon?

A quick demo of it working:
I'll waiting here for feedbacks about it
Uploading output.mp4…

I didn't find an icon named "copy" in SF Symbols, but guessed you meant the one that had been previously shown in this thread. The doc.on.clipboard icon set is attached. I thought logically the icon selection would appear just above the "Popup at" option near the top. (First select the icon, then select the behavior.) But that's not my call.
doc.on.clipboard_set_round_01.zip

Side note: I don't know it's necessary to name the icons; rather let the user determine what's most appropriate for their needs.

Side note: I don't know it's necessary to name the icons; rather let the user determine what's most appropriate for their needs.

I think its not necessary, I put it for clarity
I can remove it if we agreed to

@FabyoSK Looks great IMHO. A couple of notes for feedback:

Let's keep the dropdown after the "Show menu icon" checkbox.
I'd recommend removing titles from icons. They are not particularly important, but if we keep the titles - they need to be localized so it's extra work.

Other than that, I think it looks ready for a pull request.

@p0deje Noted,
I'll make my PR today, I just need to clean up the code 😉

That satisfies my initial request beyond expectations. Thank you!

The PR is here: #523
Please check it and let me know if I can improve something :)

@FabyoSK Thank you for the PR, it looks and works wonderfully.
@darienm Thank you for the designs, the icons look great.

Implemented in 93ddbc60446830b2120fa653132b29d2f1865b5b, thanks again folks, great job!


Diff Models

It would be really useful if swagger-diff would report changes within the models themselves, such as the removal of a field or making a field required.

Thank you for your feedback! Unfortunately, we're juggling a few different priorities at this time, and I don't know that we'll get around to adding this functionality. If you're interested, we'd welcome a PR adding this feature. I imagine this would entail adding a new command-line argument, a path through the code that starts with the definitions and disables recursion, new output functionality, and corresponding test coverage. We'd be happy to review a PR, and appreciate the contribution!


Request: Official stylus port

Would love to see a regularly updated official stylus port.  I'm unfortunately not in a position to adopt gulp and think it would be nice if I could simply define some variables and import the semantic ui stylesheets I need from my modules as part of my webpack build flow.

I'd still love to see an official stylus port done!  The current build flows for stylus are just way too much to take on in my projects!


Implement RawSocket revision

WAMP-over-RawSocket is implemented in AutobahnCpp, but needs adjustment to the revised section in the spec: https://github.com/tavendo/WAMP/blob/master/spec/advanced.md#rawsocket-transport
See also: https://github.com/tavendo/AutobahnPython/issues/291

implemented now


Themed app icon for a13?

really like this app but cant put it on my homescreen 😕

Tested on Google Pixel 6 - Stock Pixel Launcher
The app already is providing a monochromatic icon for android 13.
Make sure your launcher is not the issue first.


Switch to sjsonnet

sjsonnet (https://github.com/databricks/sjsonnet) is an order of magnitude faster than the Go version.
On my local machine, building my service with the go version takes about 3 minutes but it takes less than 3 seconds with the scala version

I would be hesitant in fully switching to sjsonnet over the supported go-jsonnet. There are a number of language features that aren't implemented in sjsonnet currently: https://github.com/databricks/sjsonnet/issues/74, https://github.com/databricks/sjsonnet/issues/126, https://github.com/databricks/sjsonnet/issues/91
But maybe this is something that could be available behind a separate tag?


topojson.merge topojson.mesh

My cartogram shows regional data, but I am would like to overlay it with the country borders. For my regular choropleth, I do this using topojson.merge (to merge regions belonging to the same country into a single shape) and topojson.mesh (to extract the borders between countries)... but I have been struggling for a while to do this with the cartogram shapes. I guess one of the reasons is that there is no GeometryCollection to work with?
Displaying country borders would aid a lot to recognise what regions/country one is looking at in the distorted map.

Great, i got it working.
first defining, almost exactly as you suggested
collection = {type: "FeatureCollection", features: thecarto.features};
topology = topojson.topology({collection: collection});
then drawing/coloring the regions
    mapg.selectAll(".subunit2").data(thecarto.features).enter().append("path") 				.attr("class","subunit2")                                 .attr("d", carto.path) 				.attr('fill',function(d){return getcolor(coldata[d.id]);});
then i loop over all countries, merging the regions, and drawing the country borders
// countrylist Array [ "AT", "BE", "BG", "CY", "CZ", "DE", "DK", "EE", "ES", "FI", 18 more… ]
countrylist.forEach(function(d) {
var ctr = topojson.mergeArcs(topology,topology.objects.collection.geometries.filter(function(z) {return (z.id.substring(0,2) == d);}));
mapg.append("path").datum(topojson.mesh(topology,ctr)).attr("d", carto.path).attr("class", "cartoctrborder").style("stroke", "black").style("stroke-width", bordersize_ctr).style("fill", "none");
});
thanks a bunch!


fix: rename private DNS zone privatelink.dp.kubernetesconfiguration.azure.com

Overview/Summary
Regarding the documentation, the name of the private DNS zone is wrong:
https://learn.microsoft.com/en-us/azure/private-link/private-endpoint-dns#hybrid--multicloud
The documentation was wrong too and has been fixed Sep 2023:
https://github.com/MicrosoftDocs/azure-docs/issues/115266#issuecomment-1739208876

Related Issues/Work Items

This PR fixes/adds/changes/removes

Insert dp. in all occurances of privatelink.kubernetesconfiguration.azure.com. This results in privatelink.dp.kubernetesconfiguration.azure.com.
I changed all occurances except the ones in generateddocs. I hope this is correct.

Breaking Changes
The private DNS zone has to be recreated.
Testing Evidence

As part of this Pull Request I have

[ ] Read the Contribution Guide and ensured this PR is compliant with the guide
[x] Ensured the resource API versions in .bicep file/s I am adding/editing are using the latest API version possible
[x] Checked for duplicate Pull Requests
[x] Associated it with relevant GitHub Issues
[ ] (ALZ Bicep Core Team Only) Associated it with relevant ADO Items
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[x] Performed testing and provided evidence.
[ ] Updated one or more of the following tests (if required)

Unit
Linting
E2E (End-To-End)
ValidateAzCloud (Base validation in Azure Cloud)
ValidateMcCloud (Base validation in Azure China Cloud)

[ ] Updated relevant and associated documentation (e.g. Contribution Guide, Module READMEs, Wiki Docs etc.)
[ ] If relevant, created or updated Code Tours here

azp run /validateazcloud

/azp run validateazcloud


getting different result from terminal and with shelljs

Node version (or tell us if you're using electron or some other framework):
v6.11.3
ShellJS version (the most recent version/Github branch you see the bug on):
0.7.8
Operating system:
Ubuntu 16.04 (AWS instance)
Description of the bug:
I'm trying to execute a shell script from node js file. When I'm executing this script from terminal I'm getting different result from what it returns when executed from nodejs using shelljs
Example ShellJS command to reproduce the error:
var shell = require('shelljs');
shell.exec('./alphanet.sh status', function(code, stdout, stderr) {
  console.log('Exit code:', code);
  console.log('Program output:', stdout);
  console.log('Program stderr:', stderr);
});

here's the link for the script https://github.com/tezos/tezos/blob/alphanet/scripts/alphanet.sh

@anupamvijayvergia is this still an issue?

@nfischer it was related to this https://github.com/shelljs/shelljs/issues/680 thanks for checking in!


Prisma 5.0.0 requires write permissions to its directory unless PRISMA_MIGRATION_ENGINE_BINARY is set

When all environment variables for custom engines are set (including PRISMA_SCHEMA_ENGINE_BINARY), Prisma still requires write permissions to the directory where the engines would have been downloaded unless deprecated PRISMA_MIGRATION_ENGINE_BINARY is set as well:
Error: Can't write to /nix/store/1q4ky0y9n112qrj4apn15x3p229f1fsy-prisma-5.0.0/lib/node_modules/prisma/node_modules/@prisma/engines please make sure you install "prisma" with the right permissions.

Discovered when updating the Nix package: https://github.com/NixOS/nixpkgs/pull/243100#issuecomment-1637878415

I cannot reproduce 🤔
export PRISMA_SCHEMA_ENGINE_BINARY=~/.cache/prisma/master/ecb5e33339ea427464f728202402a5a832eec339/darwin-arm64/schema-engine
export PRISMA_QUERY_ENGINE_BINARY=~/.cache/prisma/master/ecb5e33339ea427464f728202402a5a832eec339/darwin-arm64/query-engine
export PRISMA_QUERY_ENGINE_LIBRARY=~/.cache/prisma/master/ecb5e33339ea427464f728202402a5a832eec339/darwin-arm64/libquery-engine.node

➜  EngineEnvVars rm -R node_modules
➜  EngineEnvVars yarn
yarn install v1.22.19
[1/4] 🔍  Resolving packages...
[2/4] 🚚  Fetching packages...
[3/4] 🔗  Linking dependencies...
[4/4] 🔨  Building fresh packages...
✨  Done in 0.98s.
➜  EngineEnvVars npx prisma -v
Environment variables loaded from prisma/.env
prisma                  : 5.1.0-dev.66
@prisma/client          : 5.1.0-dev.66
Current platform        : darwin-arm64
Query Engine (Node-API) : libquery-engine ecb5e33339ea427464f728202402a5a832eec339 (at ../../.cache/prisma/master/ecb5e33339ea427464f728202402a5a832eec339/darwin-arm64/libquery-engine.node, resolved by PRISMA_QUERY_ENGINE_LIBRARY)
Schema Engine           : schema-engine-cli ecb5e33339ea427464f728202402a5a832eec339 (at ../../.cache/prisma/master/ecb5e33339ea427464f728202402a5a832eec339/darwin-arm64/schema-engine, resolved by PRISMA_SCHEMA_ENGINE_BINARY)
Schema Wasm             : @prisma/prisma-schema-wasm 5.1.0-19.ecb5e33339ea427464f728202402a5a832eec339
Default Engines Hash    : ecb5e33339ea427464f728202402a5a832eec339
Studio                  : 0.487.0

The directory of @prisma/engines does not contain any binaries like expected
➜  EngineEnvVars ls node_modules/@prisma/engines
LICENSE      README.md    dist         package.json scripts
➜  EngineEnvVars ls -la node_modules/@prisma/engines
total 40
drwxr-xr-x  7 j42  staff    224 Jul 25 14:37 .
drwxr-xr-x  5 j42  staff    160 Jul 25 14:37 ..
-rw-r--r--  1 j42  staff  11357 Jul 25 14:32 LICENSE
-rw-r--r--  1 j42  staff    740 Jul 25 14:32 README.md
drwxr-xr-x  5 j42  staff    160 Jul 25 14:37 dist
-rw-r--r--  1 j42  staff   1128 Jul 25 14:32 package.json
drwxr-xr-x  3 j42  staff     96 Jul 25 14:37 scripts
➜  EngineEnvVars ls -la node_modules/@prisma/engines/dist
total 3088
drwxr-xr-x  5 j42  staff      160 Jul 25 14:37 .
drwxr-xr-x  7 j42  staff      224 Jul 25 14:37 ..
-rw-r--r--  1 j42  staff      644 Jul 25 14:32 index.d.ts
-rw-r--r--  1 j42  staff  1576750 Jul 25 14:32 index.js
drwxr-xr-x  6 j42  staff      192 Jul 25 14:37 scripts

@aqrln Could you try again maybe and if that happens again we could do some pairing?

I checked with @aqrln, and we think we can close this as the code looks good and we can't reproduce this.


Loop thought jSON results

Hi, how can i do it using DOM7? Is it possible? I want remove the jQuery library from my app, using only DOM7.
$.each([ 52, 97 ], function( index, value ) {
alert( index + ": " + value );
});
Thanks in advance!!!

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/forEach


Improve customizability of PTKView

This pull request cleans up PTKView and adds the following features:

Size-independent layout. You should be able to specify any reasonably-sized frame and PTKView will attempt to lay out in that area, rather than having hardcoded sizes everywhere with no layout code.
Configurable font and font colors.
Ability to override backgroundImageView.
Ability to supply a custom separatorView (e.g. for a vertical line instead of a gradient).
Ability to supply custom padding via contentInsets.

The defaults should be the same, but please let me know if you notice any differences.
This work was done because we had a design for which PTKView was a great fit, however I was forced with the choices of: forking off our own and customizing it heavily; writing our own replacement; or updating PTKView as in this PR to allow customization hooks in the hopes that others will find it useful.

Any chance of getting this pr merged in or has anyone found a way around changing placeholder image size or font colors without this pr?

I'd love to see this get merged!

Hey all,
As part of a large refactor, this code has moved to our iOS SDK at https://github.com/stripe/stripe-ios, which now has a class called STPPaymentCardTextField with similar behavior to PTKView. That class exposes a lot of appearance-customizing properties, so I'm going to close this PR in favor of that. I'd also encourage you to migrate your apps to use our SDK instead of this library (it should take just a few minutes, and we've written a brief guide at https://github.com/stripe/paymentkit#migration). Of course, if that doesn't work for you, or you have any trouble migrating, reply here and I'll be happy to help. Thank you for opening this PR, though, and I'm sorry we won't be able to get it merged.
Cheers!


The bound in VI is concave

[Version: 2023-04-01]
In page 451, just before Algorithm 10.4:

Convergence is guaranteed since the bound is convex wrt each of the factors $q_i$ [Bis06, p. 466].

The bound (ELBO) should be concave wrt each of the factor $q_i$.
Thanks.

[Version: 2023-04-01]
In the bibliography, reference : [Gre98] P. Green. “ Reversible Jump Markov Chain Monte Carlo computation and Bayesian model determination”. In: Biometrika 82 (1998), pp. 711–732.
The year of publication should be 1995.
Link to the paper
Thanks.

fixed


feat: auto reload ssl cert

What problem does this PR solve?
Issue Number: close #5503
Problem Summary:
What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[x] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/run-unit-tests

/run-integration-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/hold

/run-all-tests

/unhold

/run-build-release

/run-build-release

/run-build-release

/run-all-tests

/merge

/unhold


Fixing GOC notifier

Current coverage is 37.63%

Merging #50 into master will increase coverage by +0.11% as of ebaf2a5

@@            master     #50   diff @@
======================================
  Files          206     206
  Stmts        11100   11101     +1
  Branches      1950    1950
  Methods          0       0
======================================
+ Hit           4165    4178    +13
+ Partial        543     542     -1
+ Missed        6392    6381    -11

Review entire Coverage Diff as of ebaf2a5

Powered by Codecov. Updated on successful CI builds.


w-100 randomly adds way too much padding and a scroll bar on a responsive table

Windows 10.0.18363.592
Google Chrome | 79.0.3945.130
Bootstrap 4 appears to be adding a ton of padding to columns without any way to control it when using w-100 on a responsive table. It is also adding a horizontal scroll bar that is not required for the size of the screen. On a page with multiple tables using w-100 looks the best because then all of the tables are the same size so it would be great if you guys could fix this problem.
here is an image showing the issue

Not sure what you're after exactly, but .w-100 only has a width property on it. It doesn't add padding or anything else.

This is caused by the borders (left and right) on your custom <tbody class="product-table-body">. Not Bootstrap fault.

I reported three different issues and your response didn't apply to any of them.

Not sure what you're after exactly, but .w-100 only has a width property on it. It doesn't add padding or anything else.

What I was after was trying to figure out why bootstrap was adding so much horizontal spacing between the first column and the second column without being told to do so.

The table cell has extra top-bottom padding added to it make it the same height as the rest of the cells in the row -- one of the other cells must have taller content (I would guess one of the cells with the icons). See https://jsfiddle.net/19twx54L/ and look at the padding that's automatically added to the second cell.
As for the "Column being pushed too far out," I'll defer to someone else to explain that quirk. Suffice to say, I don't believe Bootstrap is causing it. I would guess the scrollbar is appearing due to your choice of which element you added the class .w-100 to (table.table vs. div.table-responsive) but without a reduced test case (as recommended by the bug report template) it's hard to know.

w-100 does nothing when being added to the div.table-responsive so I don't really think that you are correct.

text-center on the table is adding the padding to the columns that is making everything look wrong.


Improve the UI

Improve the UI in some way?
Make the results more prominent, improve the 'show results' overlay, better font?

Hello,
May I handle this issue?
Nelaven

If this is still available i would love to improve the UI :)

@meyfleur feel free to add any UI improvements - I have just merged a PR by @nelavensubas which adds a font style


not supporting rtl

when view changed from ltr to rtl  getting mess
Can you fix this as soon as possible?
it is a great package and i wanna use it in my app but my app represent multiple language so  this package  must support  rtl
please fix this issue

Yeap. This library not support ltr right now

@AhmetToma now, this lib supported rtl


How to deploy it at Cloudflare pages?

I tried many ways, but they weren't ok
some useful links:
https://developers.cloudflare.com/pages/framework-guides/deploy-a-hugo-site/
https://developers.cloudflare.com/pages/

I tried many ways, but they weren't ok

At least tell us what did you try, and what errors did you encounter.

Build command: hugo
Build directory: public
result: 404
2.
Build command: amazon-linux-extras install golang1.11 && hugo --gc --minify
Build directory: public
result: 404

may cannot install go at Cloudflare pages.

Just tried using default configuration + environment variable to set Hugo version, and there's no issue.

it works. thanks


New ApiCenter events

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure:azure-messaging-eventgrid


InfluxDB Templates: The option section of Tasks should be treated like any other options section

This is an example of a task that has been exported as a Template:
apiVersion: influxdata.com/v2alpha1
kind: Task
metadata:
  name: task-uuid
spec:
  name: task-0
  description: desc_0
  every: 10m
  offset: 15s
  query:  >
    from(bucket: "rucket_1")
      |> range(start: -5d, stop: -1h)
      |> filter(fn: (r) => r._measurement == "cpu")
      |> filter(fn: (r) => r._field == "usage_idle")
      |> aggregateWindow(every: 1m, fn: mean)
      |> yield(name: "mean")
  status: inactive
  associations:
    - kind: Label
      name: label-1

Notice that in this Template, the name, description, and every are separated from the logic of the task itself.
The actual syntax for a task in our system includes the option task = { } section describing the task. This was originally done so that Tasks could be completely self contained as a query.
There is a proposal to allow InfluxDB Templates to become configurable on install by leveraging a option param object at the beginning of the script. See https://github.com/influxdata/influxdb/pull/19233/files#diff-7b41afbbe926a875639d639cebeb40e2R24 for an example.
We should adjust the format of the exported task resource to match this general pattern, so that tasks are still self describing while allowing the user to customize the frequency of them.
I propose doing something like this:
apiVersion: influxdata.com/v2alpha1
kind: Task
metadata:
  name: task-uuid
spec:
  task:
    - key: name
      default: "bar"
      type: string
    - key: every
      default: 1m
      type: duration
  query:  >
    option task = { name: "bar", every: 1m }
    from(bucket: "rucket_1")
      |> range(start: -5d, stop: -1h)
      |> filter(fn: (r) => r._measurement == "cpu")
      |> filter(fn: (r) => r._field == "usage_idle")
      |> aggregateWindow(every: 1m, fn: mean)
      |> yield(name: "mean")
  status: inactive
  associations:
    - kind: Label
      name: label-1

And if you included params, it could also look like this:
apiVersion: influxdata.com/v2alpha1
kind: Task
metadata:
  name: task-uuid
spec:
  task:
    - key: name
      default: "bar"
      type: string
    - key: every
      default: 1m
      type: duration
  params:
    - key: custom
      default: "bar"
      type: string
  query:  >
    option task = { name: "bar", every: 1m }
    option params = { custom: "param" }
    from(bucket: "rucket_1")
      |> range(start: -5d, stop: -1h)
      |> filter(fn: (r) => r._measurement == "cpu")
      |> filter(fn: (r) => r._field == "usage_idle")
      |> aggregateWindow(every: 1m, fn: mean)
      |> yield(name: "mean")
  status: inactive
  associations:
    - kind: Label
      name: label-1

I dig it 👍. @glinton / @desa  this would be a really excellent first issue to bring in for onboarding templates.


Problem executing IOS

Since I executed in my local machine the command
sudo xcode-select -switch /Library/Developer/CommandLineTools (in order to have git functions) I have the following error
~/r/M/f/mobile_app_react_native (local-storage)> npx react-native run-ios
warn Package react-native-sqlite-storage contains invalid configuration: "dependency.platforms.ios.project" is not allowed. Please verify it's properly linked using "react-native config" command and contact the package maintainers about this.
info Found Xcode workspace "mobile_app_react_native_android.xcworkspace"
xcrun: error: unable to find utility "simctl", not a developer tool or in PATH
error Could not get the simulator list from Xcode. Please open Xcode and try running project directly from there to resolve the remaining issues.
Error: Command failed: xcrun simctl list --json devices
xcrun: error: unable to find utility "simctl", not a developer tool or in PATH

    at checkExecSyncError (node:child_process:851:11)
    at Object.execFileSync (node:child_process:887:15)
    at runOnSimulator (/Users/blancacanocamarero/repositorios/Memories/frontend/mobile_app_react_native/node_modules/@react-native-community/cli-platform-ios/build/commands/runIOS/index.js:152:54)
    at Object.runIOS [as func] (/Users/blancacanocamarero/repositorios/Memories/frontend/mobile_app_react_native/node_modules/@react-native-community/cli-platform-ios/build/commands/runIOS/index.js:106:12)
    at Command.handleAction (/Users/blancacanocamarero/repositorios/Memories/frontend/mobile_app_react_native/node_modules/@react-native-community/cli/build/index.js:192:23)
    at Command.listener (/Users/blancacanocamarero/repositorios/Memories/frontend/mobile_app_react_native/node_modules/commander/index.js:315:8)
    at Command.emit (node:events:513:28)
    at Command.parseArgs (/Users/blancacanocamarero/repositorios/Memories/frontend/mobile_app_react_native/node_modules/commander/index.js:651:12)
    at Command.parse (/Users/blancacanocamarero/repositorios/Memories/frontend/mobile_app_react_native/node_modules/commander/index.js:474:21)
info Run CLI with --verbose flag for more details.

After typing sudo xcode-select -s /Applications/Xcode.app/Contents/Developer on terminal the error is now
(sources:

stack overflow solution
first point

 ~/r/M/f/mobile_app_react_native (execution-problem)> npx react-native run-ios
warn Package react-native-sqlite-storage contains invalid configuration: "dependency.platforms.ios.project" is not allowed. Please verify it's properly linked using "react-native config" command and contact the package maintainers about this.
info Found Xcode workspace "mobile_app_react_native_android.xcworkspace"
error No simulator available with name "iPhone 13".
info Run CLI with --verbose flag for more details.

Fixing selecting the device npx react-native run-ios --simulator="iPhone 13"
(Open Xcode > windows (options)  > device an simulators ) or stack overflow comment


Update Pitch Deck Slides

Update Pitch Deck Slides per recommendations

Updated my slides, rethemed the presentation, started some extra slides in the back
[t:25]

Added visual for enterprise control panel
[t:25]

Updated my slides, added revenue slide
[t:30]

Updated my slides
[t:15]

[t:10] Updated slides

updated slides
[t:10]

Implemented mentor suggestions
[t:40]

Added competitor prices slide and updated next steps slide
[t:45]

[t:25] Got a call back from a rep with Allied Solutions and updated interview results to pitch deck slide

Reformated Market Opportunity slide based on meeting with mentor
[t:20]

Updated slides.
Added API info continuation slide to update with Google AppSheet info.
[t:25]

Wrote script (30)
Implemented slide and script changes suggested during practice pitches (30)
[t:60]


[SPARK-14480][SQL] Simplify CSV parsing process with a better performance

What changes were proposed in this pull request?
https://issues.apache.org/jira/browse/SPARK-14480
In CSVParser.scala, there is an Reader wrapping Iterator but there are two problems by this.
Firstly, it was actually not faster than processing line by line with Iterator due to additional logics to wrap Iterator to Reader.
Secondly, this brought a bit of complexity because it needs additional logics to allow every line to be read bytes by bytes. So, it was pretty difficult to figure out issues about parsing, (eg. SPARK-14103).
To cut in short, this PR includes two changes,

Parse CSV data with Iterator not Reader.
Refactor CSV data source to be consistent with JSON data source.

This PR removes classes CSVParser and introduces new classes UnivocityParser, UnivocityGenerator and CSVUtils to be consistent with JSON data source (JacksonParser, JacksonGenerator and JacksonUtils). Also, DefaultSource moves to CSVRelation just like JSONRelation.
How was this patch tested?
Existing tests should cover this.

I have two things to note.

It looks buildInternalScan() is not called but just remaining. Just in case I did not remove this and tested this.
It looks the changes are a lot but the logics and filtering are not changed much except that it uses Iterator instead of Reader.
Performance was tested with this patch. This can be found in https://issues.apache.org/jira/browse/SPARK-14480.

cc @rxin

Test build #55415 has started for PR 12268 at commit b5a9669.

Test build #55415 has finished for PR 12268 at commit b5a9669.

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

Merged build finished. Test FAILed.

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

retest this please

Test build #55420 has started for PR 12268 at commit b5a9669.

Test build #55420 has finished for PR 12268 at commit b5a9669.

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

Merged build finished. Test FAILed.

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

retest this please

Test build #55428 has started for PR 12268 at commit b5a9669.

Test build #55428 has finished for PR 12268 at commit b5a9669.

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

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

Merged build finished. Test PASSed.

Test build #55457 has started for PR 12268 at commit f65592b.

Test build #55459 has started for PR 12268 at commit 55596e1.

Test build #55457 has finished for PR 12268 at commit f65592b.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Could I maybe cc @liancheng  and @cloud-fan to review? This resembles JSON data source structure. So, the class structures and input/output in methods are consistent with JSON data source one.

Test build #55461 has started for PR 12268 at commit 262f346.

Test build #55459 has finished for PR 12268 at commit 55596e1.

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

Merged build finished. Test PASSed.

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

Test build #55461 has finished for PR 12268 at commit 262f346.

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

Merged build finished. Test PASSed.

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

Test build #55597 has started for PR 12268 at commit 9015317.

Test build #55599 has started for PR 12268 at commit 5a95276.

Test build #55597 has finished for PR 12268 at commit 9015317.

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

Merged build finished. Test PASSed.

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

Test build #55599 has finished for PR 12268 at commit 5a95276.

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

Merged build finished. Test PASSed.

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

@yhuai Could you please review this? I don't want to keep resolving conflicts and I am pretty sure that this is a sensible PR.
This PR touches pretty a lot of files so this causes many conflicts. I would appreciate that if this is reviewed quicker.

cc @rxin , do you know who is the original author of the CSV part?

Could I please cc @falaki who I believe is the original author?

Please excuse my pings, @cloud-fan , @rxin , @falaki , @yhuai

will try to take a look in the next few days.

Test build #56309 has started for PR 12268 at commit d9ea3cb.

Test build #56309 has finished for PR 12268 at commit d9ea3cb.

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

Merged build finished. Test PASSed.

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

@rxin Could you please review this?

Test build #56768 has started for PR 12268 at commit 92f8f38.

Test build #56768 has finished for PR 12268 at commit 92f8f38.

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

Merged build finished. Test PASSed.

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

ping @rxin

Test build #56955 has started for PR 12268 at commit ad21b8e.

Test build #56959 has started for PR 12268 at commit d59c7e9.

Test build #56959 has finished for PR 12268 at commit d59c7e9.

This patch fails to build.
This patch merges cleanly.
This patch adds the following public classes (experimental):
sealed abstract class LDAModel protected[ml] (
class LocalLDAModel protected[ml] (
class DistributedLDAModel protected[ml] (
class ContinuousQueryManager(sparkSession: SparkSession)
class DataFrameReader protected[sql](sparkSession: SparkSession) extends Logging
class Dataset[T] protected[sql](
class QueryExecution(val sparkSession: SparkSession, val logical: LogicalPlan)
class FileStreamSinkLog(sparkSession: SparkSession, path: String)
class HDFSMetadataLog[T: ClassTag](sparkSession: SparkSession, path: String)
class StreamFileCatalog(sparkSession: SparkSession, path: Path) extends FileCatalog with Logging
case class PlanSubqueries(sparkSession: SparkSession) extends Rule[SparkPlan]

Merged build finished. Test FAILed.

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

retest this please

Test build #56961 has started for PR 12268 at commit d59c7e9.

Test build #56961 has finished for PR 12268 at commit d59c7e9.

This patch fails to build.
This patch merges cleanly.
This patch adds the following public classes (experimental):
sealed abstract class LDAModel protected[ml] (
class LocalLDAModel protected[ml] (
class DistributedLDAModel protected[ml] (
class ContinuousQueryManager(sparkSession: SparkSession)
class DataFrameReader protected[sql](sparkSession: SparkSession) extends Logging
class Dataset[T] protected[sql](
class QueryExecution(val sparkSession: SparkSession, val logical: LogicalPlan)
class FileStreamSinkLog(sparkSession: SparkSession, path: String)
class HDFSMetadataLog[T: ClassTag](sparkSession: SparkSession, path: String)
class StreamFileCatalog(sparkSession: SparkSession, path: Path) extends FileCatalog with Logging
case class PlanSubqueries(sparkSession: SparkSession) extends Rule[SparkPlan]

Merged build finished. Test FAILed.

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

This was due to https://github.com/apache/spark/commit/d2614eaadb93a48fba27fe7de64aff942e345f8e

Test build #56963 has started for PR 12268 at commit f62755e.

Test build #56963 has finished for PR 12268 at commit f62755e.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

@rxin It looks this is still failing, https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56962
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56963

Fixed in https://github.com/apache/spark/commit/f8709218115f6c7aa4fb321865cdef8ceb443bd1

Test build #56965 has started for PR 12268 at commit fe63ba2.

Test build #56955 has finished for PR 12268 at commit ad21b8e.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds no public classes.

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

Build finished. Test PASSed.

Test build #56965 has finished for PR 12268 at commit fe63ba2.

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

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

Merged build finished. Test PASSed.

cc @hvanhovell would you have some time to take a look at this?
@HyukjinKwon most of us are very busy trying to get things out for 2.0 so this one will very likely slip.

@rxin No problem. Let me just rebase it if it has conflicts anyway. It is easier to track the changes.

Test build #57058 has started for PR 12268 at commit ee71064.

Test build #57058 has finished for PR 12268 at commit ee71064.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class MultivariateGaussian(
class DecisionTreeClassifier @Since(\"1.4.0\") (
class GBTClassifier @Since(\"1.4.0\") (
class RandomForestClassifier @Since(\"1.4.0\") (
  class AFTSurvivalRegressionWrapperWriter(instance: AFTSurvivalRegressionWrapper)
 class AFTSurvivalRegressionWrapperReader extends MLReader[AFTSurvivalRegressionWrapper]
class DecisionTreeRegressor @Since(\"1.4.0\") (@Since(\"1.4.0\") override val uid: String)
class GBTRegressor @Since(\"1.4.0\") (@Since(\"1.4.0\") override val uid: String)
class RandomForestRegressor @Since(\"1.4.0\") (@Since(\"1.4.0\") override val uid: String)
case class CartesianProductExec(

Merged build finished. Test PASSed.

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

@HyukjinKwon I have taken a pass. The PR looks pretty solid. I do think we can make it a bit more concise in some places and I do think we can make a bit faster as well. Let me know what you think?

@hvanhovell Thank you for a close look! I think I need to change this title of this issue and JIRA because "better performance" might be too broad.
The main purpose of this PR was,

Refactoring this to be consistent with JSON data source
Remove unnecessary conversion from Iterator to Reader.

Could I please make some JIRAs and PRs for this in separate PRs or follow-ups if it makes sense?

@hvanhovell If you think it makes sense I will change the title of this PR and JIRA, and will add some more commits to deal with minor things (code style and etc.).

@rxin, @hvanhovell Do you mind if I ask your thoughts on this please?

Since this is almost a complete rewrite, I think we should only consider it early in the release cycle, i.e. for 2.1, not for 2.0 when we are so close.

Test build #57496 has started for PR 12268 at commit bd510c2.

Test build #57498 has started for PR 12268 at commit 8e1bdf7.

Test build #57496 has finished for PR 12268 at commit bd510c2.

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

Merged build finished. Test FAILed.

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

Test build #57498 has finished for PR 12268 at commit 8e1bdf7.

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

Merged build finished. Test PASSed.

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

Test build #57834 has started for PR 12268 at commit a0aed27.

Test build #57834 has finished for PR 12268 at commit a0aed27.

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

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

Merged build finished. Test PASSed.

Test build #58046 has started for PR 12268 at commit f2234e3.

Test build #58046 has finished for PR 12268 at commit f2234e3.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class NoSuchPermanentFunctionException(db: String, func: String)
class NoSuchFunctionException(db: String, func: String)
case class GetExternalRowField(

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

Merged build finished. Test PASSed.

Test build #58538 has started for PR 12268 at commit cbb1674.

Test build #58538 has finished for PR 12268 at commit cbb1674.

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

Merged build finished. Test PASSed.

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

Test build #59174 has started for PR 12268 at commit d1f616e.

Test build #59175 has started for PR 12268 at commit 66b1757.

Test build #59174 has finished for PR 12268 at commit d1f616e.

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

Test build #59175 has finished for PR 12268 at commit 66b1757.

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

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

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

Test build #59846 has started for PR 12268 at commit 7abdfc1.

Test build #59846 has finished for PR 12268 at commit 7abdfc1.

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

Merged build finished. Test PASSed.

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

Test build #60751 has started for PR 12268 at commit 7abdfc1.

Test build #60751 has finished for PR 12268 at commit 7abdfc1.

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

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

Merged build finished. Test PASSed.

@HyukjinKwon i just took another look at this. Can you separate this into multiple, smaller pull requests?
IIUC, the main change you want to do is to get rid of the StringIteratorReader. Why don't you just submit a patch for only that? It'd make it much easier to review and merge. The problem with mixing multiple things into a single pull request is that it significantly increases the overhead to review, and makes it almost impossible to merge.
We can start with this one change, and then do other refactorings later.
Also it would be great to show some benchmark numbers since it is a performance improvement. The other thing I'd like to see (in a separate pull request) is to try use the Record API from univocity and compare the performance with our current data cast API.

@rxin I see. Actually, I added a benchmark for this change in the JIRA.
So.. would this be okay if I do as below:

Get rid of StringIteratorReader
Refactoring (maybe with a follow-up for the tests for.. JacksonParser, JacksonGenerator, UnivocityParser and UnivocityGenerator)
Use Record API from univocity and compare the performance (with a proper benchmark)
Maybe other fixes addressing @hvanhovell's comments (maybe with a benchmark)

If it makes sense, I would appreciate that If I can create a umbrella for those four as sub-tasts and be assigned to them.

Sounds good. Please be surgical in each pr.
Would also be great to include benchmark results in pr description. Thanks.

Maybe close this one for now?

Sure.


BI-2029 (Experiment Preview and Visual Progress Indicators Broken in v0.90

Description
BI-2029 Experiment Preview and Visual Progress Indicators Broken in v0.9
In ImportExperiment.vue,the method createObservationIndexMap() was setting values in this.observationIndexMap that cause an out-of-bounds error in cellClassIfExisting.  This modifies createObservationIndexMap() so that does not happen.
Dependencies
bi-api: development branch
Testing

import an experiment with 2 or more observation variables.
EXPECTED RESULT
Experiment preview is displayed.

Checklist:

[x] I have performed a self-review of my own code
[x] I have tested my code and ensured it meets the acceptance criteria of the story
[ ] I have create/modified unit tests to cover this change
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to documentation
[ ] I have run TAF: <link to TAF run>

Can you make sure this gets merged into the release/0.9 branch as well develop


use callback info rather than updating entire usb list

@bfjelds,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


feat(dbm-services): dbha global monitor

关联研发信息
dev_branch: feat/dbha-global-monitor/#4048
target_branch: v1.3.0


Added image, model and dataset file

Stock prediction of the bank negara.

Follow the pull request template and re-open a fresh PR.
@pawaspy

Where I could get the pr template?!


[ISSUE #7669] map variable delayLevelTable changed to ConcurrentSkipL…

…istMap from ConcurrentHashMap

Which Issue(s) This PR Fixes

Fixes #7669
Brief Description
[Enhancement] Iterating map variable "delayLevelTable" in order should be JDK-independent , map variable delayLevelTable changed to ConcurrentSkipListMap

How Did You Test This Change?
using mvn test

@lizhanhui  can you verify this pr

LGTM


Revise and verify commands for PowerShell 7

At some point, PowerShell 7 will need to be supported as a desktop client on Windows 10. Need to verify this module and configurations will work on PowerShell 7 and Windows PowerShell 5.1.
This issue can wait until PowerShell 7 is officially released.

This might be more difficult than expected. PS7 Ships with version 2.0.5 of the PSDesiredStateConfiguration module which is not distributed through the PowerShell gallery.

Desired State Configuration is being revised for PowerShell 7 so this module will never be compatible.


test: Use built-in mock with cast functions instead of manual casting.

This should / may resolve #603.
Signed-off-by: Philip Tricca<EMAIL_ADDRESS>

Any chance you can test this on your system with GCC 5.x @liuqun? I only have 6 and 7 installed on my dev system currently.

Thanks flihp, now gcc-5.4 does not complain about [-Werror=int-to-pointer-cast] any more.
One more thing: tabrmd also needs to be fixed. I've posted an issue at https://github.com/intel/tpm2-abrmd/issues/239

Thanks for closing this out @liuqun. I saw the issue in tpm2-abrmd but wanted to make sure this was the right fix first. I'll take care of that one next.

this needs to be cherry-picked into the 1.x release branch too.


Feature Added - UI of more detailed view of a book

I added a new feature of more detailed view of a book when the user clicks on a particular book from the book slider. For now all the books opens the sample component as the route is same for all for now. Later when the backend will be integrated, the routes be be different according to the particular book id and also the respective component can be rendered. Below is the work that I had done:
https://drive.google.com/file/d/17ggkU3BOZxyFTOqt1PWdJI-DB6wx38Rf/view?usp=sharing

@laxanarana1513 There is a deployment failure. Check this link for error logs: https://ctxt.io/2/AACQnyTcEw

I added a new feature of more detailed view of a book when the user clicks on a particular book from the book slider. For now all the books opens the sample component as the route is same for all for now. Later when the backend will be integrated, the routes be be different according to the particular book id and also the respective component can be rendered. Below is the work that I had done: https://drive.google.com/file/d/17ggkU3BOZxyFTOqt1PWdJI-DB6wx38Rf/view?usp=sharing

@laxanarana1513 Great job! There's just a small change needed. Please update the button content from "Start your free 30 days" to "Start Reading".

@laxanarana1513 can u just do "git pull" and commit again?

I added a new feature of more detailed view of a book when the user clicks on a particular book from the book slider. For now all the books opens the sample component as the route is same for all for now. Later when the backend will be integrated, the routes be be different according to the particular book id and also the respective component can be rendered. Below is the work that I had done: https://drive.google.com/file/d/17ggkU3BOZxyFTOqt1PWdJI-DB6wx38Rf/view?usp=sharing

@laxanarana1513 Great job! There's just a small change needed. Please update the button content from "Start your free 30 days" to "Start Reading".

Okk I do that

@laxanarana1513 deployment issue is still there : https://ctxt.io/2/AABQyFC3Ew

@laxanarana1513 deployment issue is still there : https://ctxt.io/2/AABQyFC3Ew

Idk why its failing... Can't understand... What I can do?

@laxanarana1513 can u just do "git pull" and commit again?

have u tried this?

@rohansx  It is causing problem coz in server, mongodb creadientials are provided.. should I comment that??

@rohansx It is causing problem coz in server, mongodb creadientials are provided.. should I comment that??

deployment failure is not due to credentials. The error says there is something wrong with package.json file

@laxanarana1513 deployment issue is still there : https://ctxt.io/2/AABQyFC3Ew

Idk why its failing... Can't understand... What I can do?

is it running properly in ur system?

@laxanarana1513 yup it is working, what was the issue?

@laxanarana1513 yup it is working, what was the issue?

Some packages were not installed properly

@laxanarana1513 ok, thank you for your contribution


rename repo

We need to look into possibly rename of the repo so that it specifically indicates it is kokkos (sub)project of the Kokkos project.
See most recent Kokkos release briefing.

Do you mean moving it to the kokkos project specifically (e.g., https://github.com/kokkos).

Otherwise, sure, we could rename the repo xstack-klokkos to something else, which would only require permission from the sandialabs admin.

Do you mean moving it to the kokkos project specifically (e.g., https://github.com/kokkos)?

No, I don't mean this, for now.

Otherwise, sure, we could rename the repo xstack-klokkos to something else, which would only require permission from the sandialabs admin.

Yes, the above is all I want to do. That sounds less painful than what I thought. Do you think I/we can get it done in the next week or two?

Update: this is OK for the time being, until around October.


404 at /toolbox/toolbox_install_winows

I found a broken link : https://docs.docker.com/toolbox/toolbox_install_winows

Hi @spiderH, thank you for sharing this issue. Can you provide more details on how you arrived to this link?

Thanks for sharing the issue. We are unable to reproduce the issue and are closing it due to insufficient information. If you still encounter the issue, please respond with the requested details.


[Date/Time Picker] Touch scroll doesn't snap to value

Describe the bug
After #7834, scrolling via touch can lead to an invalid state as no scroll snapping is implemented
See comment: https://github.com/AvaloniaUI/Avalonia/pull/7834#issuecomment-1072861675
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. Windows, Mac, Linux (State distribution)]
Version [e.g. 0.10.0-rc1 or 0.9.12]

Additional context
Add any other context about the problem here.

After attempting to fix this, I'm not so sure this has an easy fix. I've gotten snapping to work (quick tested by re-enabling the scroll bar and dragging it). But when I tested with my Surface, it doesn't work with the scroll gesture. If you swipe up/down slowly, it will jump several items and then stop there until you start a new gesture. Swiping quickly will get a better response, but still isn't fluid.
One thing I've noticed too is the gesture scroll delta is large (not sure if this play a part). I'd expect it to follow your finger, (e.g., if you tap on 12 in the day column of the DatePicker, 12 should stay under your finger - instead it will reach the end of the viewport before your finger). I looked at the handling logic in ScrollContentPresenter but I can't figure out how its all wired together - and it may be that this logic doesn't work well with modifying the offset in the middle of a scroll gesture?
I would suggest porting WinUI's scroll viewer which has better touch interaction and snapping built-in, but considering the ScrollViewer code is probably larger than the entire Avalonia codebase, that's clearly not an option.

I would suggest porting WinUI's scroll viewer which has better touch interaction and snapping built-in

That isn't open sourced yet, correct? I don't see it in WinUI 2. I also heard they were planning some changes for WinUI 3 but I doubt they will ever get around to it.

but considering the ScrollViewer code is probably larger than the entire Avalonia codebase

I heard that said about DataGrid some years ago. I can't think that is true for a ScrollViewer implementation though! :) It should be doable if the code exists somewhere.
A lot of Avalonia controls have reduced functionality compared to WPF or WinUI. The reason the WPF controls are larger is they support more features and edge cases can pile on code quickly. Avalonia code-base is going to have to grow without fear though and start pulling in more features. Pulling code from WPF or even WinUI shouldn't be feared as much as it is -- it actually will help everyone.

Technically no, the ScrollViewer isn't open sourced (yet), but it's likely successor is: see WinUI /dev/ScrollView & /dev/ScrollPresenter. They're still in "preview" but it's essentially everything that the regular ScrollViewer is with some API enhancements.

I heard that said about DataGrid some years ago. I can't think that is true for a ScrollViewer implementation though! :) It should be doable if the code exists somewhere.

I was exaggerating a little there, but ScrollView & ScrollPresenter combined is approx ~10,000 lines of c++.

but it's likely successor is: see WinUI /dev/ScrollView & /dev/ScrollPresenter.

Oh ok, these are the newer controls I heard about but haven't really paid attention to them. I'll have to browse through when I get a chance though. I probably missed this some time ago.

ScrollView & ScrollPresenter combined is approx ~10,000 lines of c++.

10,000 lines of C++ becomes around, what, ~7500 lines of C#? That is around three normal controls in my experience (up to ~3000 LOC for a control). Avalonia Calendar: 2125k LOC for example. Avalonia has gotten lucky (and used clever design/architecture changes) to keep extremely minimal control sizes to date but the luck is going to run out for certain controls :)

Could we use the ScrollGestureEndedEvent to detect when scrolling has finished and snap to the item there?

Could we use the ScrollGestureEndedEvent to detect when scrolling has finished and snap to the item there?

I want to say I tried that and it didn't work, but I may have messed something up. I'll try taking another look at this.

I think I've got a fix for this.

Hopefully fixed by https://github.com/AvaloniaUI/Avalonia/pull/7964


Support for custom pretrained models

Hi
Is it possible to use pretrained models other than trained by inplace_abn? Like SAM?
Can you please suggest how to use them?

In short, is it mandatory to use Inplace_ABN pretrained models? Can we use other pretrained models trained in a different way or not use pretrained model at all?

The code has been designed to use the deeplab architecture. However, loss functions and the framework could also be employed using any other architecture, given it outputs a final class-probability mask (take a look at CoMFormer here, where I adapted MiB to Mask2Former).
Anyway, adapting the code to very different architectures (e.g. Mask2Former) may require substantial changes.

It is not mandatory to use Inplace_ABN models or use any pretraining (indeed, performance may be very different).


1.1.5 has breaking change - fails when no ignored devices defined

Describe The Bug:
Upgrading to 1.1.5 broke the plugin, causing a TypeError
To Reproduce:
Install 1.1.5
Configure with no ignored devices.
Expected behavior:
Expected the plugin to continue to work correctly,
Logs:
TypeError: Cannot read properties of undefined (reading 'includes')
    at GoveeHomebridgePlatform.goveeDiscoveredReading (/usr/local/lib/node_modules/homebridge-plugin-govee/dist/platform.js:76:32)

Plugin Config:
{
            "name": "Govee",
            "batteryThreshold": 25,
            "debug": false,
            "humidityOffset": 0,
            "platform": "GoveeHomebridgePlugin"
        },

Environment:

Plugin Version: 1.1.5
Homebridge Version: 1.4.0
Node.js Version: v16.14.1
NPM Version: v8.5.0
Operating System: Raspbian / hb-service

I found a workaround to just add random text to the 'ignored devices' part of the configuration

I will look at it on Monday.
Sorry for that, did not test it with no entries.
You workaround seems reasonable to have at least something in the list. As it is a simple string match that was added, it will just not find the string and add all other devices.

Already have a tested a fix (developed only my iPad) 😀
Will put it into code on Monday. List will now be checked if empty and check for ignored devices skipped.

Added a pull request for the project owner.
Hope i did it right, as GitHub is not really my "home" ;-)

@MrFly72 Thank you for a quick turn around on the fix.
I published version 1.1.6 that fixes the issue. Sorry for the delay, it took me some time to fix my setup to test everything before publishing.