install fails at trying to setup rabbitmq users

+ sudo rabbitmqctl delete_user zulip
Deleting user "zulip" ...
Error: unable to connect to node rabbit@e611e15f9c9d: nodedown

DIAGNOSTICS
===========

nodes in question: [rabbit@e611e15f9c9d]

hosts, their running nodes and ports:
- e611e15f9c9d: [{rabbitmqctl7195,42064}]

current node details:
- node name: rabbitmqctl7195@e611e15f9c9d
- home dir: /var/lib/rabbitmq
- cookie hash: xCFdbNsgD1SPwNlmRbq/ZA==

+ true
+ sudo rabbitmqctl delete_user guest
Deleting user "guest" ...
Error: unable to connect to node rabbit@e611e15f9c9d: nodedown

DIAGNOSTICS
===========

nodes in question: [rabbit@e611e15f9c9d]

hosts, their running nodes and ports:
- e611e15f9c9d: [{rabbitmqctl7240,40403}]

current node details:
- node name: rabbitmqctl7240@e611e15f9c9d
- home dir: /var/lib/rabbitmq
- cookie hash: xCFdbNsgD1SPwNlmRbq/ZA==

+ true
+ sudo rabbitmqctl add_user zulip 58cfc28bf881b28310986a8a5d1306c2fab8af5aff63562df93456b5cce3b0b7
Creating user "zulip" ...
Error: unable to connect to node rabbit@e611e15f9c9d: nodedown

DIAGNOSTICS
===========

nodes in question: [rabbit@e611e15f9c9d]

hosts, their running nodes and ports:
- e611e15f9c9d: [{rabbitmqctl7286,53989}]

current node details:
- node name: rabbitmqctl7286@e611e15f9c9d
- home dir: /var/lib/rabbitmq
- cookie hash: xCFdbNsgD1SPwNlmRbq/ZA==

This occurs with the following Dockerfile:
FROM ubuntu:trusty

ENV DEBIAN_FRONTEND noninteractive
# Bump when you want to re-run apt-get upgrade
ENV LAST_UPDATE 2

RUN apt-get update
RUN apt-get -y upgrade
RUN apt-get -y install wget

ENV ZULIP_VERSION "1.3.5"

RUN wget https://www.zulip.com/dist/releases/zulip-server-$ZULIP_VERSION.tar.gz
RUN tar -xvf zulip-server-$ZULIP_VERSION.tar.gz
RUN rm zulip-server-$ZULIP_VERSION.tar.gz
RUN mv zulip-server-$ZULIP_VERSION /root/zulip

WORKDIR /root/zulip
RUN ./scripts/setup/install

Looks like rabbitmq-server isn't started.
https://github.com/timabbott/zulip/commit/f2248d2ea6563d2e5f5953849b8c358c9a9a4e58 is what I needed to do in order to get Travis CI (based on docker) installing a development type environment, which may be helpful for debugging (e.g. I did have to run a rabbitmq restart in the middle of the install process there)

I tired the above fix and I am still having the install fail.

Try starting epmd explicitly (one of rabbitmq's dependencies) and then starting rabbitmq -- another development mentioned that worked for them in an environment where they were getting this sort of failure...
(Also the .travis.yml is now merged into master)

Starting epmd with empd -daemon did not change the result.
Error log below:
Deleting user "guest" ...
Error: unable to connect to node rabbit@hook: nodedown

DIAGNOSTICS
===========

nodes in question: [rabbit@hook]

hosts, their running nodes and ports:
- hook: [{rabbitmqctl29496,33511}]

current node details:
- node name: rabbitmqctl29496@hook
- home dir: /var/lib/rabbitmq
- cookie hash: 3XsN7TD1p6mRQluJafLl+w==

+ true
+ sudo rabbitmqctl add_user zulip 2fb280f8f8c96133f48fcfac951b7fad567c92a15488246096f3ab74864de340
Creating user "zulip" ...
Error: unable to connect to node rabbit@hook: nodedown

DIAGNOSTICS
===========

nodes in question: [rabbit@hook]

hosts, their running nodes and ports:
- hook: [{rabbitmqctl29541,57316}]

current node details:
- node name: rabbitmqctl29541@hook
- home dir: /var/lib/rabbitmq
- cookie hash: 3XsN7TD1p6mRQluJafLl+w==

Update, purging and reinstall rabbitmq seems to "fix" the issue. Followed the steps outlined [here] (http://www.techsfo.com/blog/2013/06/rabbitmq-breaks-when-you-rename-hostname/). It seems to be an issue related to the install process not going through clean on the first try.

Cool, thanks for posting the fix.  I'm not sure I see an easy way to fix this for future users, but we should probably at least document this in a README.prod.md troubleshooting section...

An option would be to have start/finish detection for the install and if it gets messed up mid-way through to either roll back or purge and start fresh.

As well when using the default docker image for ubuntu you need to the modify the /usr/sbin/policy-rc.d from exit 101 to exit 0 as per directions here. Note I am not sure of the implications of making this change but it seems to achieve the desire goal.

we had an issue with rabbitmq not working because the hostname was not set in the /etc/hosts file.  After updating it, rabbitmq started up fine.

Just to verify you are setting it to your FQDN, yes?

I was just using whatever hostname and hostname -f provided, however, let me preface my input with the fact that I never got the complete zulip package to work.


addded pull request template

resolve issue #36

Hello @pinak22dhir,
Thank you for your pull request! I appreciate your effort in resolving issue #36 and adding the pull request template.
I will review the changes shortly and provide feedback. If everything looks good, we'll proceed with merging.
Once again, thank you for your contribution to our project.

Nicely done @pinak22dhir, ⭐
I'm merging the PR now. Thanks for the contribution.
Feel free to contribute/create more issues, if you've found any. ✌


Problem opening in mac

Hey Eric, I hope you are having a great day. When downloading the mac unbox download, i'm receiving this message (image below) is there anything i can do on my end to fix this

?

Same here, on OS X Big Sur.

Yoo sorry about this! This is something I’m planning on fixing soon, but in the meantime you can download the version I mention here: https://github.com/erikrichardlarson/unbox/issues/51#issuecomment-889644467

Hello Eric,
I'm also having trouble - Mac Mini M1 with MacOS Monterey 12.1


"map" validator gets confused when keys are duplicated

Consider schema and data below:
schema.yaml
maptest: map(bool())

test.yaml
maptest:
  key1: false
  key2: xpto
  key2: true

Expected behavior: (either one of)

If YAML grammar doesn't allow for duplicate keys on above situation (and this I honestly don't know) Yamale should complain and fail validation.
If duplicated keys are allowed, then Yamale should validate both, which means failing on the first "key2". What happens instead, is that it ignores the first "key2" and validates the second, resulting in a succesfsull validation (which is incorrect).

Teste on latest release, 3.0.2. Any more info, just ask.

Hi, thanks for using Yamale.
This behavior is dependent on the parser used; we don't take a stance on whether the file is syntactically valid YAML. The returned object from a yaml.load() is what Yamale validates. Here a discussion in the PyYAML project that covers this case: https://github.com/yaml/pyyaml/issues/165
The ruamel parser will fail to load a document with duplicate keys. To use ruamel with Yamale, you can use the --parser ruamel option on the command line: https://github.com/23andMe/Yamale#command-line

Thanks for explaining, I'll do as suggested.


Deal with partial fixity (only x / only y)

This pull request includes the attributes fix_xand fix_y to the __init__() of compas_ags.diagrams.formdiagram.
With these attributes, constraints can be set up based on them.
I also added one example file and .json with the geometry of the example.

Please @brgcode review this PR before #106 ;)


Fix Printer class member 'content_offset' naming

Replace "content_offset" by "drawing_area_offset".

Hi, and thanks for the work!
I'm not sure this renaming is necessary. In addition to the breaking change it would cause, it may not be much clearer.
First, let's summarize the various values. One of the big concepts is "real size" vs "virtual size", and it mostly matters when scrolling (or cropping) is involved.
A view can think it's given a large size, but only a subset of that will actually be visible at a given time. Most of the time, the view doesn't have to care what part is visible. It should mostly be used for optimizations (if it wants to avoid spending time on parts that will not be visible anyway).
Let's define a few values:

In the view coordinates, the "virtual area" allocated to the view is Rect::from_size((0,0), printer.size).
In the view coordinates, the subset that is actually visible in the end is Rect::from_size(printer.content_offset, printer.output_size)
In the terminal coordinates, the area that can be drawn on is Rect::from_size(printer.offset, printer.output_size)

From this, we can see that content_offset is the "offset into the content", used to only draw a subset of a view. If a view has a "size" of 10x10, but we only want to draw the bottom 2x2 corner (maybe because we are scrolling into this large view), the content offset will be 8x8. Calling it drawing_area_offset can be confusing as it's not really an offset into the drawing area.
For output_size, I agree it could be called drawing_area_size, since it's the size where actual drawing is allowed, but I'm not sure it justifies the breaking change. In addition, it could still be mis-interpreted as being the size given to draw a view.
I do agree that the naming are not ideal, but maybe what it needs is more examples, and/or viewport functions that return the Rect I defined earlier - these should help understand where each value fits in.


Fix typos in documentation files

Changes
packages/ui/CHANGELOG.md

Fixed typo in Vue type module reference:

Old: decleration
New: declaration
Reason: Corrected spelling of technical term "declaration" which is the proper spelling in programming context.

Fixed typo in architecture description:

Old: scalibity
New: scalability
Reason: Corrected spelling of technical term "scalability" which is the standard term used when discussing system architecture.

app/laboratory/tests/README.md

Fixed path formatting:

Old: <app/laboratory/tests/README.md>
New: app/laboratory/tests/README.md
Reason: Removed unnecessary angle brackets for consistent path formatting across documentation.

Thank you for your contribution. At this time, Reown does not generally accept these types of contributions for security reasons.


Warning: invalid local UTF-8 at the end of the meilisearch-setup script

Description
A Warning message appears at the end of the meilisearch-setup script.
The command gives by the warning or an export should fix the problem and should be added to the cloud-config.yaml.
Screenshots or Logs
_____________________________________________________________________
WARNING! Your environment specifies an invalid locale.
 The unknown environment variables are:
   LC_CTYPE=UTF-8 LC_ALL=
 This can affect your user experience significantly, including the
 ability to manage packages. You may install the locales by running:

   sudo apt-get install language-pack-UTF-8
     or
   sudo locale-gen UTF-8

To see all available language packs, run:
   apt-cache search "^language-pack-[a-z][a-z]$"
To disable this message for all users, run:
   sudo touch /var/lib/cloud/instance/locale-check.skip
_____________________________________________________________________

This issue doesn't appear anymore


fix(plugin-vue): allow overwrite esbuild config

fix(plugin-vue): Support decorators in .vue file.
issues: https://github.com/vitejs/vite-plugin-vue/issues/430
More thoughts:
      {
        target: 'esnext',
        // #430 Support decorators in .vue file. https://github.com/vitejs/vite-plugin-vue/issues/430
        // target can be overridden by esbuild config target
        ...options.devServer?.config.esbuild,
        loader: 'ts',
        sourcemap: options.sourceMap,
      },

in vite-plugin-vue，There is no merge from esbuildTransformOptions, so submit the modification and merge esbuildTransformOptions, that is, options.devServer?.config.esbuild in the development environment

in vite vite:esbuild，Merged from esbuildTransformOptions

Just for fault tolerance and compatibility considerations
vite is the basic component of vite-plugin-vue. When vite-plugin-vue calls the transformWithEsbuild function provided by vite:esbuild in the vite project, should it be consistent with the parameters of vite:esbuild calling transformWithEsbuild, that is, esbuildTransformOptions, that is, options.devServer?.config.esbuild in the development environment
Because the running results of vite in the development environment and the production environment should be consistent, it is also necessary to keep it consistent with the parameters passed in when vite:esbuild calls the transformWithEsbuild function
This code will only be triggered in the development environment, so it is correct to use options.devServer?.config.esbuild

When I used the decorator function of ts, I also encountered a problem
When I used the decorator in the dev environment of vite, an exception occurred, but it was normal in the build environment
So I checked the differences between the dev and build environments
I found that in the build environment, the target of esbuild can be replaced by esbuild:{target}, esbuildTransformOptions in vite.config.ts, while in the dev environment, it is hard-coded, target:esnext, and esbuildTransformOptions does not replace target:esnext
Logic executed when vite:esbuild builds
https://github.com/vitejs/vite/blob/main/packages/vite/src/node/plugins/esbuild.ts     (238)

vite-plugin-vue Logic executed in dev environment
https://github.com/vitejs/vite-plugin-vue/blob/main/packages/plugin-vue/src/main.ts   (253)

This leads to inconsistency between dev environment and build environment, which I find strange
vite is the basic component of vite-plugin-vue.
In the vite project, when vite-plugin-vue calls the transformWithEsbuild function provided by vite:esbuild, the parameters passed in should be consistent with the parameters of transformWithEsbuild called by vite:esbuild itself when building.
That is, esbuildTransformOptions should be used to replace and merge target:esnext
And this code will only be triggered in the development environment, so options.devServer?.config.esbuild can be used
This way, the results of dev environment and build environment can be consistent
So I submitted a PR https://github.com/vitejs/vite-plugin-vue/pull/444

https://github.com/vitejs/vite-plugin-vue/pull/222 This should be another solution, but it has not been accepted for a long time. Why is it?

Is it also related to this submission? Can it be closed? #13756

This fix allows overwrite esbuild config, so it fixes #278 , and #31 is repeated with 278. Can #31 be closed?
#278 , in dev mode, whether the configuration when calling esbuild build can be read synchronously from vite.config.js or support parameter passing duplication,


Document keyword arguments that matter

Document options_dict

Document cq_token

cz_radius and maybe also cz_rads and maybe rename @SalahedeenIssa

make cz_rads stripped_cz_rads


feat: allow templates in annotations

This allows templates in annotations for all resources. It tried to keep the pattern used in other places (i.e. extra-env for deployment) and use the same pattern for all annotations.
/fixes #254

I approved the tests; at least you can see which tests failed.

Also missing are the chart version bump and release notes for ArtifactHub.

I incremented the chart version and added the changes as annotations - hope I got the format right?
Sorry, I checked for a contributing instruction, but didn't find one...

@pierluigilenoci  The last merge introduced an extra newline and the linter complained about it. I removed it and pushed again. Could you please approve the workflow again?

Approved the run

I incremented the chart version and added the changes as annotations - hope I got the format right?

You did well 😄

Sorry, I checked for a contributing instruction, but didn't find one...

In reality, a contribution guide is missing.


Carbon\Carbon::rawCreateFromFormat(): Argument #2 ($time) must be of type string, int given

Package
filament/filament
Package Version
v3.2.50
Laravel Version
v11
Livewire Version
No response
PHP Version
8.3.3
Problem description
https://github.com/briannesbitt/Carbon/issues/2967
Related to this issue Filament DatePicker also throws an exception
Expected behavior
Should be working well.
Steps to reproduce
Forms\Components\DateTimePicker::make('start_date')
->required()
->label(__('admin/local.start_date')),
Reproduction repository
https://github.com/briannesbitt/Carbon/issues/2967
Relevant log output
No response

I should mention that start_date field is unix_timestamp from the DB

Hello @UtkuDalmaz please dump the stack trace, so I may spot the code that would need to be updated to work around this and propose a pull-request.
Basically Carbon::createFromFormat('U', $integerValue) should be replaced with:

Carbon::createFromTimestampUTC($integerValue) if the format is always 'U' (so the value always represent a timestamp)
Carbon::createFromFormat('U', (string) $integerValue) if 'U' is actually a dynamic value.

Here it is
TypeError: Carbon\Carbon::rawCreateFromFormat(): Argument #2 ($time) must be of type string, int given, called in /var/task/vendor/nesbot/carbon/src/Carbon/Traits/Creator.php on line 647
#116 /vendor/nesbot/carbon/src/Carbon/Traits/Creator.php(565): Carbon\Carbon::rawCreateFromFormat
#115 /vendor/nesbot/carbon/src/Carbon/Traits/Creator.php(647): Carbon\Carbon::createFromFormat
#114 /vendor/filament/forms/src/Components/DateTimePicker.php(86): Filament\Forms\Components\DateTimePicker::Filament\Forms\Components\{closure}
#113 /vendor/filament/support/src/Concerns/EvaluatesClosures.php(35): Filament\Support\Components\Component::evaluate
#112 /vendor/filament/forms/src/Components/Concerns/HasState.php(86): Filament\Forms\Components\Component::callAfterStateHydrated
#111 /vendor/filament/forms/src/Components/Concerns/HasState.php(223): Filament\Forms\Components\Component::hydrateState
#110 /vendor/filament/forms/src/Concerns/HasState.php(202): Filament\Forms\ComponentContainer::hydrateState
#109 /vendor/filament/forms/src/Concerns/HasState.php(187): Filament\Forms\ComponentContainer::fill
#108 /vendor/filament/filament/src/Resources/Pages/EditRecord.php(109): Filament\Resources\Pages\EditRecord::fillFormWithDataAndCallHooks
#107 /vendor/filament/filament/src/Resources/Pages/EditRecord.php(92): Filament\Resources\Pages\EditRecord::fillForm

Thanks, fix submitted #11890


Space in the end creates a problem [Mass Edit]

There's an issue in the program where it gives an error when there's space after the line typed.

Here, you can see it works fine

I was confused what the problem was, especially because dividing created a problem in the program like I previously reported.

Fixed in the next release.


Replace mount fork hack with CLONE_FS

This change spins up a new goroutine, locks it to a thread, then unshares CLONE_FS which allows us to Chdir from inside the thread without affecting the rest of the program.
The thread is no longer usable after unshare so it leaves the thread locked to prevent go from returning the thread to the thread pool. (see https://pkg.go.dev/runtime#UnlockOSThread)
Thanks @corhere for working on a bunch of this stuff in moby and pointing out that we should be able to do this instead.

Would it be possible to have some unit tests?

Would it be possible to have some unit tests?

Test added.

 We have found a Go bug :)

Windows failure looks unrelated.
The rockylinux test had an issue, the results don't seem to be showing up anymore but I recall it was something cgroup related which also seems unrelated... that said I've not noticed either of these failures before.

Should we get a separate PR for the usual "upgrade the world" to 1.19.3 versus just the one GH Actions CI setting? Should be able to get that in quickly and then rebase this

@estesp Good idea, https://github.com/containerd/containerd/pull/7620


Test flight latest iOS build (1077)

The iOS App 1.3.3 (build 1077) is processing on App Store Connect. @KatieWoe, @kathy-phet please prioritize testing of this build.
Issues to verify:
https://github.com/phetsims/phet-ios-app/issues/602
It would also be good to open a few translated sims to ensure the app updates translations, as expected. Something similar to this test procedure.

[x] Tester = BL, Platform = iPad , Time =1hr
[x] Tester = NS, Platform = iPad Air 2 iPadOS 15.0, Time =2 hr
[x] Tester = KW, Platform = iPad and iPhone, Time = 2 hr
[ ] Tester = , Platform = , Time =

Pasted from last (1075) test:
Tester = KW, Platform = iPad 6th Gen iPadOS 14.6, Time = 3 hrs + 2 hrs on (1075)
Tester = E2, Platform = iPad Pro 2nd Gen iPadOS 14.7, Time = 2.5 hrs
Tester = KW, Platform = iPhone 7, Time = .5 hr

Stale, closing.


Create Groups Package in Systems:

Create GroupSort extraction function
Create Tests for GroupSort

The package structure may not be easy to understand, because groups and systems might be seen in different ways. However, these group functions are designed to work with the measurement system. Both use the MutableList to improve flexibility and allow coroutines to run on smaller independent sections of the data. This is really about the Systems module as a whole. Perhaps a better name will come along, maybe one that better related to system information categorization and processing.


Is there any option for scrolling from component file.

I have used ng2-scroll-to plugin (https://www.npmjs.com/package/ng2-scroll-to) it is working fine with tags in html. But I need scrolling functionality from component methods.
Is there any way to achieve this scrolling functionality from components as well.

I would also love this! Looking at the code it looks like the only way to trigger it right now is from a click event.

Me too! I really need this!


Fixed compilation error

Thanks for pointing this out. I'll probably move all @synthesize elements to the top of files to prevent this particular error with people still on Xcode 4.2.


Go 1.23

Go 1.23 has been released (more information and release notes here) 🥳.
I assume you are already aware of the release and will probably draft a new release as time permits -- thank you for your open source commitment! Hence, I just created this issue for visibility.

Sure - it will happen as soon as I find the time in the next couple of days 👍

Packages for Go 1.23 are now available in the PPA. Please help testing. They will be promoted to default soon. Closing this here.

@longsleep confirmed works great for me; rebuilt our product using it, ran tests etc, no problems.
Many thanks for your continued work maintaining this resource!

Same for us; we already deployed the new package and everything works flawlessly.
Thank you for your open source commitment and all the great work you do!


feat: add install with go support, fix flags

Summary

feat: add install with go support, fix flags
Ticket Link

Fixes #45
Release Note

feat: add install with go support, fix flags

looks cool! can you add some doc in the readme?
@dentrax and @necatican can you please help me with this folks? ✌

looks cool! can you add some doc in the readme?

We(w/@dentrax @necatican)'ve completed the documentation of our new feature. Thank you @cpanato ✌


Upgrade to BeautifulSoup 4

This should fix problems with tricky/malformed HTML being served on the messages page ( and other tags inside an  tag). BS3 had problems parsing it and because of that the (around) three most recent messages in conversations which had more than three messages were not saved to disk properly.
If the import of BS4 fails, please delete the .okcmd-virtualenv/ folder to set up a new virtual environment.

This should fix #29, but please test it before merging to master.

The message output diff between with and without this BS4 patch is quite large -- but the problem seems to have been fixed. Seems conversations with more than a few message have been broken a while -- the question is for how long. Either way, if this patch tests well with you guys we should encourage people to re-download messages.

Tested by @falsifian in #29. Merging.


Better docs for ImageProvider

An ImageProvider has a "resolve" method which returns a ImageStream. An ImageStream can be listened to by registering a listener with addListener (and unregistering with removeListener). These methods take callbacks whose first argument is an ImageInfo, which is an object with the dart:ui Image object (which you can pass to drawImage) and a double that describes the expected scale (number of image pixels per logical pixel).

Also I should mention what the second argument to those callbacks is actually used for.


500 Internal Server Error

I tried the sample on my Windows 10 Pro(ver1803 build 17134.345) PC, but it does not work well.
I started the node sample as described in the tutorial.
>pwd
C:\Users\hogehoge\Desktop\code\nodejs-connect-rest-sample-master\starter-project
>node --version
v8.9.4
>npm start
><EMAIL_ADDRESS>start C:\Users\hogehoge\Desktop\code\nodejs-connect-rest-sample-master\starter-project
> mocha ./tests/confTest.js && node ./bin/www
  ADAL
    √ Checking clientID and clientSecret in config.js
  1 passing (5ms)

{"name":"AzureAD: OIDC Passport Strategy","hostname":"hoge123","pid":16716,"level":40,"msg":"Production environments should always validate the issuer.","time":"2018-10-27T14:59:19.256Z","v":0}
{"name":"AzureAD: OIDC Passport Strategy","hostname":"hoge123","pid":16716,"level":40,"msg":"Using http for redirectUrl is not recommended, please consider using https","time":"2018-10-27T14:59:19.258Z","v":0}

Listening on port 3000

I executed the SendMail button at http: // localhost: 3000 but Internal Server Error is displayed.

500 Internal Server Error
{
  "error": {
    "code": "InternalServerError",
    "message": "Unexpected end when reading token. Path 'Attachments[0]'.",
    "innerError": {
      "request-id": "b16c1817-72bc-4561-af57-8e466b335f27",
      "date": "2018-10-27T15:34:29"
    }
  }
}
Error: Internal Server Error
    at Request.callback (C:\Users\hogehoge\Desktop\code\nodejs-connect-rest-sample-master\starter-project\node_modules\superagent\lib\node\index.js:706:15)
    at parser (C:\Users\hogehoge\Desktop\code\nodejs-connect-rest-sample-master\starter-project\node_modules\superagent\lib\node\index.js:916:18)
    at IncomingMessage.res.on (C:\Users\hogehoge\Desktop\code\nodejs-connect-rest-sample-master\starter-project\node_modules\superagent\lib\node\parsers\json.js:19:7)
    at emitNone (events.js:111:20)
    at IncomingMessage.emit (events.js:208:7)
    at endReadableNT (_stream_readable.js:1055:12)
    at _combinedTickCallback (internal/process/next_tick.js:138:11)
    at process._tickCallback (internal/process/next_tick.js:180:9)

Am I something wrong?

@taketakekaho we are archiving this sample as it is no longer being maintained. We're replacing it with a new sample which you can find here: https://github.com/microsoftgraph/msgraph-training-nodeexpressapp


ZEPPELIN-3295. Remove build of profile spark-1.6 and scala-2.11

What is this PR for?
Trivial PR to remove the build of profile spark-1.6 and scala-2.11, The travis build of profile spark-1.6 and scala-2.11 only test the embedded mode of spark instead real spark binary distribution of spark 1.6 with scala-2.11, so IMHO it is safe for us to just remove that build and only keep spark 1.6 with scala-2.10
What type of PR is it?
[Improvement]
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-3295

How should this be tested?

Travis CI pass

Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

@Leemoonsoo @felixcheung Please help review it. Thanks

@zjffdu It looks like this commit broke the master build on Travis.

@SleepyThread How does it broke the master build ? There's some flaky test in zeppelin, it is normal is you see some failed test, community is working on that to fix these flaky test to make travis build stable.


feat: verify contract API implementation

What I did
fixes: #22
fixes: APE-35
Allows user to verify contracts.
Required PR(s):
https://github.com/ApeWorX/ape/pull/1026
https://github.com/ApeWorX/ape/pull/1008
How I did it
Carefully following their API, cross-checking with browser and Hardhat's implementation and Brownie until things worked.
Had to make changes in ape-solidity and ape to get it to work.
How to verify it
Checklist

[ ] Passes all linting checks (pre-commit and CI jobs)
[x] New test cases have been added and are passing
[ ] Documentation has been updated
[ ] PR title follows Conventional Commit standard (will be automatically included in the changelog)

see https://github.com/ApeWorX/ape-etherscan/pull/42


most of the stream submodule

The only big difference to SeqAn3 is include/bio/stream/compression.hpp .
And the stream implementations have been changed to accept threads and compression level as a parameter but are otherwise in their old (chaotic) state.


System.IO.FileStream

Issue Title
System.IO.FileStream - ctor with FileSystemRights is missing in  .net core
General
This topic is mainly about IKVM. IKVM works nicely converting java 8 jars to regular .net assemblies, but project stopped in 2015.  I was playing around with IKVM and .netcore (2.2) I was able to load those assemblies, and even instantiate some objects. When  trying somewhat more practical (web client/ db connection) I was getting  error about missing constructor for FileStream. After running analysis tool, it looks like this might be one of a few missing  parts.  Below are result

The jdbc assembly is not really needed even to connect to DB, the Core assembly could be 100% covered if that missing constructor would be added. Converted jars are 100% compatible too. IKVM.Runtime assembly has quite a few missing methods from System.Reflection.Emit, but probably those are not needed. So I'm really curios if adding missing constructor will enable IKVM for core/netstandard, at least partially. Anyway to test it?
Below are the missing methods for Core assembly, analysis tool says there is some workaround coming for the 2nd one

Target type
Target member

T:System.IO.FileStream
M:System.IO.FileStream.#ctor(System.String,System.IO.FileMode,System.Security.AccessControl.FileSystemRights,System.IO.FileShare,System.Int32,System.IO.FileOptions)

T:System.Diagnostics.StackTrace
M:System.Diagnostics.StackTrace.#ctor(System.Threading.Thread,System.Boolean)

Transferring to the corefx repo.


ci test

for detail, goto summary download Artifacts base-ylt-cov-report(base commit coverage report) and ylt-cov-report(current pull request coverage report)}


refactor bootstrap_config

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: nmittler
If they are not already assigned, you can assign the PR to them by writing /assign @nmittler in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @knightXun. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

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

/ok-to-test

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/retest

@knightXun: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
48f699448ac963b2f88c94b702b21bc7d71f104c
link
/test e2e-bookInfo-envoyv2-v1alpha3

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

@knightXun: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
48f699448ac963b2f88c94b702b21bc7d71f104c
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/e2e-mixer-no_auth-mcp.sh
48f699448ac963b2f88c94b702b21bc7d71f104c
link
/test e2e-mixer-no_auth-mcp

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

New changes are detected. LGTM label has been removed.

@ozevren I think this pr can be merge before we fix the lint test

/test e2e-pilot-auth-v1alpha3-v2

@knightXun: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
bb6ba62f0b31c83d0951880933459eb226dd6206
link
/test e2e-mixer-no_auth-mcp

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

/test e2e-mixer-no_auth-mcp

@ozevren hi, the e2e-pilot-auth-v1alpha3-v2 failed, how can i retigger it again?

You need to go to the CircleCI UI and click on "Rerun workflow" (or something similarly named) and select "rerun with ssh". You may need to do that after signing-in to CircleCI via your GitHub account.

@ozevren oh,thank you. I see the test is passed.

@ozevren hi, can this pr be merged? thank you.

@ozevren I alse found i have no authority to press Rerun workflow, maybe that's the members' right.

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Modules Refactoring

Modules can now be build on top of Jellyfish::Extension, they can require a minimum version of Jellyfish, can require a minimum version of other extensions.

Products belong to a base provisioner
Products are built by using custom classes for ProductType, and Service
Provisioners are registered with custom classes of RegisteredProvisioner
Modules can provide new UX (early work, still much to do)
Admin features have been broken into Manager and Administrator access
Settings ability to manage application settings inside of the app, including settings provided by modules
Multiple providers of each type can be created
Products can require user input at time of order
Cart removed, ordering happens directly from the marketplace.
UX code reduced (CSS reduced 24%, JS reduced 18%)
Standardized a lot of the CSS classes
Moved dashboard to '/' and login to '/login'
Fixed a long existing bug which caused API requests to stack up
UX updated to Angular 1.4.x
Moved many forms to use Angular Formly object definitions

And a mountain of other small changes.
@mafernando @AllenBW @sonejah21 @chriskacerguis

too much has changed for review.  so, I'll just give this a :+1:

Going to fix some CodeClimate issues before merging.


Upgrade deviceLocale on Android

Hi,
First off thanks for your module!
I just noticed that your last version now gets the device locale which is great. I wrote a module to do it myself but would be happy to use yours.
My problem is that locale.toString() is... weird on Android as it can return unpredictable results and even the correct result is different from the iOS one (en_US vs en-US). locale.toLanguageTag() seems more robust but only available on API 21 (Lollipop), but using it with a good (enough) fallback seems the good solution to me.
In reference here is the code I'm using now:
private String getCurrentLanguage() {
    Locale current = getReactApplicationContext().getResources().getConfiguration().locale;
    if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {
        return current.toLanguageTag();
    } else {
        StringBuilder builder = new StringBuilder();
        builder.append(current.getLanguage());
        if (current.getCountry() != null) {
            builder.append("-");
            builder.append(current.getCountry());
        }
        return builder.toString();
    }
}

I would be happy to look into that and submit a PR.

Thanks :) that looks like a good solution, if you want to submit a PR that would be great!


AwaitableDisposable isn't portable between platforms with async support and without it

The problem is described in this StackOverflow question: http://stackoverflow.com/questions/28703336/nuget-pcl-library-conflict
To fix this issue, you would need to depend on the Microsoft.Bcl.Async package for all platforms (even those that natively support async), and return the TaskAwaiter from the Microsoft.Runtime.CompilerServices in all cases.

Has any progress been made or even accessed on this issue? I am running into this when I try to test my PCL with Nunit.

Not yet, sorry. There isn't currently a way to build AsyncEx, but building a build VM is on my todo list.

Should be fixed in 4.0.0-pre.


Script causes PCB to be renamed

After running the script with KICad 6.0.10 the PCB is renamed from filename to filename-rounded. This causes problems when attempting to later edit the PCB since it loads the original filename which does not contain any changes made since running the script. I did not see this behavior with earlier versions of KiCad.
Steps to reproduce:

Open design in PCBNew
Run rounded tracks operation
Notice in title that filename is now filename-rounded
(optional) perform any other changes to the PCB
Save the output (File->Save or ^S)
Quit PCB editor
Reopen PCB editor
All recent changes are not present

It seems that after running this that saving to the filename now saves to filename-rounded instead of filename.

Do you have "Create new file" checked? That is the expected behaviour when it's ticked. It is partly a remnant from earlier versions - before the undo history worked properly, before it was possible to apply to just the selection, and especially before native arcs were added.
You should still be able to open the rounded file output in pcbnew, just by clicking it in the project manager.
It's not impossible a recent update changed the checkbox behaviour, I can't test right now though.

My mistake. I had it set.

No worries


(Github Actions) Bump versions of actions/checkout in ci workflow and scan for GitHub Actions with dependabot

Description

This pull request will bump the versions of actions/checkout in the ci workflow and for dependabot adds a package ecosystem for scanning for github actions.

Rejecting this as it has already been tried before and failed so had to be reverted. See https://github.com/github/linguist/pull/5915
There is also no good reason to change this.


Fix for starting embark with a contract that has an error, then fixing, seems to still break the pipeline

Needed to reset the compileError flag once compilation succeeded.

I'm guessing the relevant commit is only the last one? Maybe cherry-pick that commit on top of a branch on par with develop


Release updated conda package for 16.35.1

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.

Hi! This is the friendly conda-forge automerge bot!
I considered the following status checks when analyzing this PR:

linter: passed
travis: failed
azure: passed

Thus the PR was not passing and not merged.

@corneliusroemer : Any idea what this error means?  I'm stumped.


Enable front-end build by default

Enable general front-end build by default for new projects

Would be nice to include https://github.com/adobe/aem-project-archetype/issues/236 as well

Going to look at this and #236 today


internal/minioclient: instrument nontrivial functions with monkit's Task

PR Checklist

[x] The PR title is formatted as follows: miniogw: add TestXYZ for XYZ

[x] The package name goes before the colon
[x] The part after the colon uses the verb tense + phrase that completes the blank in, "This change modifies Gateway-ST to ___________"
[x] Lowercase verb after the colon
[x] No trailing period
[x] Keep the title as short as possible. Ideally under 72 characters or shorter
[x] No Markdown

Commit Message Body

What, why, please describe the tests and the performance impact (if any)
No Markdown
The body is wrapped at 72 characters unless it's really needed (ASCII art, table, or long link)
If there is a corresponding issue, add either Closes #1234, Fixes #1234, Resolves #1234 or Updates #1234 (the latter if this is not a complete fix) to this comment
If referring to a repo other than storj/gateway-st, you can use the owner/repo#issue_number syntax: Fixes storj/common#1234
We do not use Signed-off-by lines. Please don't add them. Our Gerrit server & GitHub bots enforce CLA compliance instead
More: https://github.com/storj/docs/blob/main/code/Git.md
Delete these instructions once you have read and applied them

Related Issue

Code Review Checklist (to be filled out by reviewer)

[x] Does the PR describe what changes are being made?
[x] Does the PR describe why the changes are being made?
[x] Does the code follow our style guide?
[x] Does the code follow our testing guide?
[x] Is the PR appropriately sized? (If it could be broken into smaller PRs, it should be)
[x] Does the new code have enough tests? (every PR should have tests or justification otherwise. Bug-fix PRs especially)
[x] Does the new code have enough documentation that answers "how do I use it?" and "what does it do?"? (both source documentation and higher level, diagrams?)
[x] Does any documentation needs updating?
[x] Do the database access patterns make sense?
[x] Copy the Commit Message Body section contents into the submit prompt upon PR completion

Please review and let me know if there any changes to be done

Please review and merge, add hacktoberfest-accepted label, so that it can be part of hacktoberfest2022

please review and please tag this repo with hacktoberfest tag and merge thanks for the opportunity

@avinilcode could you please squash the commits?


Hyperscript doesn't work when bundled with Parcel

Hyperscript doesn't work when bundled with Parcel.
https://github.com/bigskysoftware/_hyperscript/issues/162

Hi, @Janaka-Steph
Fixed in v2.3.3. Please update CLI and re-check again.


DeBruijn: introduce a version of #_ that depends on no postulate

Per the discussion in issue #513, this patch introduces a formalisation of variables based on de Bruijn indices that relies on no postulates. The accompanying text is updated accordingly.

There appears to be a serious typo: No closing paren!
defined in Chapter [Decidable]({{ site.baseurl }}/Decidable/. The type

You are right. I just added the missing parenthesis.

Why did you close this? If you just push to the same branch, it'll automatically update the pull request.

Why did you close this? If you just push to the same branch, it'll automatically update the pull request.

My apologies, this was not my intent. I misinterpreted the user interface here in Github.
How do I mark @wadler 's request for changes as done?

Why did you close this? If you just push to the same branch, it'll automatically update the pull request.

My apologies, this was not my intent. I misinterpreted the user interface here in Github.
How do I mark @wadler 's request for changes as done?

You can't, you'll have to wait for them to review your code again.

Dear @wadler and @wenkokke , I just updated the pull request with your feedback from today. Hopefully it does what you had in mind. I linked to sections in Chapter Decidable instead of to the definitions themselves. If there is a way to link to the code blocks providing the definitions, please let me know and I will happily make another update to the pull request.

Great! All that remains to do is to delete "The type of function #_ asks for clarification." as per my previous message.

My apologies, I misunderstood the comment the first time. I hope I got it right this time!


Add Bang event for the class R_Knod

The event miss and cause the molette is selectable from outside not only we are on it.

fixed


feat: priority experiments

@mathuo Thanks for this experiment! It looks very promising, especially preferredSize. However panels rarely have square shaped content: the preferred width most of the time it does not equal with the preferred height.
Some examples:

List panel designed for displaying as a narrow vertical bar by default. It has a preferred width of 420px, but no preferred height if I put it above/below a panel (especially not the 420px specified for width).
Search box panel: it has a text field for searching. It has a preferred height of 120px and a preferred width of 450px.

This is really  important to address, while min/max are a good start, while using the dockview it is important to be able to set flexible sizes to each panel

Refer to https://github.com/mathuo/dockview/issues/641


Fixed Carthage not building

Not sure if this is a good fix, as I just used !, but there was an error in Carthage while building, requiring me to add it.

Are you using XCode 9?
This has to be done in a backwards compatible way because this change is only in Swift 3.2 onwards.
This has already been fixed on the master branch. I'll add that and a couple of other fixes to a new cocoapod release over the weeekend.

Thanks! :)


Provide email notification

As OpenPAI is an async platform. It's better to notify users when job started, completed or error. So that users don't need to visit web portal frequently.

@xudifsd It's about to provide notification for PAI users. When job started, completed, user can get notification. Is it implemented?

Oh, I misunderstood this issue. I thought it was about providing email notification about system performance and fault.

Thanks squirrelsc, @scarlett2018 could you please help to triage this?

it seems that this feature can be implemented in launcher, where job state can be tracked accurately? @yqwang-ms , what do you think?

I know launcher can track accurately, but it would be better to let high level service to send the email.
launcher would be better to keep pure as container orchestrator and others should leverage it to detect state change events.
https://github.com/Microsoft/pai/tree/master/subprojects/frameworklauncher

And our system is already level triggered instead of edge triggered, even for Launcher itself. So, miss event change event should be acceptable.

@yqwang-ms, is it possible for a job to know the state of itself? e.g., we provide an optional hook or callback for system call in case of job completion? this could provide enough flexibility for user to customize their own behavior (sending email, sms, or whatever he/she prefers.)

It is possible, but that introduces interop with a potential unreliable service. Such as, should launcher best effort call the callback or block on the success of the callback? We provide at least once call or exactly once or etc? So, for the simplest and reliable solution is the external service monitor the rest api, then take actions, this is aligned with k8s design principle.

The simliar offerings in k8s is https://github.com/bitnami-labs/kubewatch
It is also just a controller which monitors the api server, and it may miss events.

AM is a system components, it has admin privilege, we cannot directly execute the shell script (i.e. YARN cannot isolate everything). The shell script also make AM different from each other and hard to debug.
Compared with very long running time AI jobs, we can poll the frameworks api in every min, it should be fine and not heavy weight

After some discussion, this feature is better implemented outside of OpenPAI's core infrastructure code base.
OpenPAI provides REST APIs to query a job status. User can develop and deploy a watchdog to watch the jobs and define any actions in case of job status changes. The watchdog can be deployed to OpenPAI as a job. But the core PAI platform will not treat it as a system component, so no liability about its liveness, etc.
And the easiest way for a user to implement (a part of) this feature is to add some script at the end of the command line input. But this approach can only work when there is no exception raised in the user code. But I assume this would work for most cases.

do we need to support multiple language support (e.g. Chinese and English) in email text?

@xinase No, we current do not have such option, You can configure email text here by changing the summary of each alerting rule.

Is it possible we expose job status as a metrics, and define an alert rule to be triggered when status changed? I suppose Prometheus alert server is able to send out this kind of email alerts to job owner's email. @xudifsd is it possible?

@sterowang Yes, it is possible to send to job owner via alert-manage as long as we enrolled the user's email info. We can also send alert email about low resource usage the same way.

Is it possible we expose job status as a metrics, and define an alert rule to be triggered when status changed? I suppose Prometheus alert server is able to send out this kind of email alerts to job owner's email. @xudifsd is it possible?

Good ones, we should combine these set of features with the "User Home Page" and "Job Metrics for User".

closed and track in https://github.com/microsoft/pai/issues/2235


Better std.algorithm.comparison.equal error messages

@CyberShadow with this change building Dustmite now fails with the following error message:
../dmd/generated/linux/release/64/dmd -I../dmd/druntime/import -I../phobos -L-L../phobos/generated/linux/release/64 -m64 -fPIC -preview=dip1000 -w -de -version=Dlang_Tools DustMite/dustmite.d DustMite/splitter.d DustMite/polyhash.d -ofgenerated/linux/64/dustmite
--
  | ../phobos/std/algorithm/comparison.d(976): Error: static assert:  All passed ranges must by InputRanges
  | DustMite/dustmite.d(521):        instantiated from here: `equal!(const(Address)*, const(Address)*)`
  | DustMite/dustmite.d(541):        instantiated from here: `addDependents!(MapResult!(__lambda10, EntityRef[]))`

Any ideas?
@burner you should be able to reproduce this locally like so:
cd $DLANG
git clone https://github.com/dlang/tools
make -C tools -f posix.mak RELEASE=1 --jobs=4

Yeah, it's picking up the wrong overload now. There is an equal declaration in dustmite.d, but now the compiler picks up the one in std.algorithm, because the constraints are gone.
I don't think this is going to work :(
Didn't we have proposals to add explanations to constraints so that we can have both cakes?

Didn't we have proposals to add explanations to constraints so that we can have both cakes?

Yes it was mine. I presume it must have been a DIP because it changed grammar, I remember I had trouble trying to implement it which I think was the reason it didn't go anywhere, will have a look for it tomorrow.

Why can't we just do
    if (// More than one range is required
		rs.length > 1 &&
		/// All passed ranges must be input ranges
        allSatisfy!(isInputRange, Ranges) &&
		/// None of the passed ranges may be an `InfiniteRange`
        !allSatisfy!(isInfinite, Ranges) &&
		/// The first and second range's elements must be comparable with `binaryFun!pred`
        is(typeof(binaryFun!pred(rs[0].front, rs[1].front))) &&
		/// All successive range's elements must be comparable with`binaryFun!pred`
        (rs.length == 2 || is(typeof(equal!pred(rs[1 .. $])) == bool)))

It is syntactically unambiguous, doesn't require any language changes, and doesn't need a DIP (because it doesn't change grammar and only affects compiler output). I.e. just print any doc node attached to the expression that failed.

To make the documentation and error messages more difficult to parse for muggles smile

That's the point of these PR's
Closing this for now

thank you for all the review work


Add Parametric Tan Hyperbolic Linear Unit Activation

Added new activation that improves the convergence and accuracy of some models.
[1] Parametric Tan Hyperbolic Linear Unit Activation for Deep Neural Networks
Related Issues
(None)
PR Overview
[ y ] This PR requires new unit tests [y/n] (make sure tests are included)
[ y ] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[ y ] This PR is backwards compatible [y/n]
[ n ] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

@gabrieldemarmiesse I resolved all conflicts. It seems only problems with encoding remain. Just resolve its towards master brunch please.

Github is still saying "this branch has conflicts that must be resolved".

@gabrieldemarmiesse  You have no permissions to resolve in direction of master tree?
As I see it just text encodind issue.

Reviewers should not modify pull requests directly, otherwise, they're not reviewer, they're authors ;) Generally though, in the OSS world, it's the job of the person submitting the pull request to resolve conflicts.

@gabrieldemarmiesse Then please clarify how can I resolve it. Do I need change encoding through notepad++? Which one I need? We have problems only with that files that strange...

@gabrieldemarmiesse I'm really can't understand this. May be bug with git. I check encoding of contrib_docs/structure.py as example and it exactly the same as in master brunch. What is this conflict about?

@gabrieldemarmiesse I found a way to do all in web, so no more issues whatever they are :) Please review.

It seems some visual studio files were added. Could you remove them?

Done


Merge 6.9.0 to main

➡ Forward port
Port ign-math6 as of 6.9.0 to main. Needed by https://github.com/ignitionrobotics/sdformat/pull/803 and https://github.com/ignition-tooling/release-tools/issues/574
It has compilation warnings that need fixing still
Branch comparison: https://github.com/ignitionrobotics/ign-math/compare/main...ignition-math6_6.9.0
Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

We need to update the python bindings to accommodate the deprecated functions so the warnings are fixed.

Done in c69f1a9a0594fa9254edfa8c6b142200b0c41d93. One of the tests fails for me locally before and after this PR though. I tried a few different things, and even if I don't use abs, the vector's value doesn't look right locally. Let's see what CI says.
ERROR: test_math (__main__.TestQuaternion)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "src/ign-math/src/python/Quaternion_TEST.py", line 191, in test_math
    self.assertAlmostEqual(
  File "/usr/lib/python3.8/unittest/case.py", line 943, in assertAlmostEqual
    diff = abs(first - second)
TypeError: bad operand type for abs(): 'Vector3d'

We need to update the python bindings to accommodate the deprecated functions so the warnings are fixed.

Done in c69f1a9. One of the tests fails for me locally before and after this PR though. I tried a few different things, and even if I don't use abs, the vector's value doesn't look right locally. Let's see what CI says.
ERROR: test_math (__main__.TestQuaternion)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "src/ign-math/src/python/Quaternion_TEST.py", line 191, in test_math
    self.assertAlmostEqual(
  File "/usr/lib/python3.8/unittest/case.py", line 943, in assertAlmostEqual
    diff = abs(first - second)
TypeError: bad operand type for abs(): 'Vector3d'

I fixed a few more instances of deprecated APIs being called in python tests and amended https://github.com/ignitionrobotics/ign-math/commit/c69f1a9a0594fa9254edfa8c6b142200b0c41d93 to https://github.com/ignitionrobotics/ign-math/commit/482381defbdc5224fbc938206a9dfc5098cbbaab and force-pushed

I fixed a few more instances of deprecated APIs being called in python tests and amended c69f1a9 to 482381d and force-pushed

It's still failing. I've been looking into it, I think it's because of some of the API changes on main for constructors and accessors. For example, this test failure:
  FAIL: test_pose (__main__.TestPose3)
  ----------------------------------------------------------------------
  Traceback (most recent call last):
    File "/github/workspace/src/python/Pose3_TEST.py", line 44, in test_pose
      self.assertAlmostEqual((B + A).pos().x(), 1.0 + 1.0/math.sqrt(2))
  AssertionError: 0.0 != 1.7071067811865475 within 7 places (1.7071067811865475 difference)

Is fixed if we use (B + A).x() instead of (B + A).pos().x().

Updated the last commit from 482381d to 84837c2. I changed some tests to workaround the failures, and commented out others. I still don't know what's the issue, but I suspect it's related to the Vector3 copy constructor. I tried changing that from default to an implementation though and that didn't help :woman_shrugging:


Don't spam journald upon every scrape GET request

Every time the Prometheus scraper makes an HTTP GET request I see this line on stdout:
<IP_ADDRESS> - - [22/Sep/2022 17:51:19] "GET /metrics HTTP/1.1" 200 8104

I run the program as a systemd service, as such I see 4 such messages per minute in my journald afterwards which is quiet the nuisance.
By subclassing the RequestHandler and overriding the log_message function those messages won't be printed.

Sorry, must have missed the mails from  both your PRs.
I'd rather not just discard the logs, but maybe add an option to write them to a file (eg --access-logs $path), which can still default to stdout, but can then also be set to a normal log file, or to /dev/null.


React Code fix for disabled button for a Tooltip

Console log:
index.js:1406 Material-UI: you are providing a disabled `button` child to the Tooltip component.
A disabled element does not fire events.
Tooltip needs to listen to the child element's events to display the title.

Add a simple wrapper element, such as a `span`.
    at Tooltip (http://localhost:3000/static/js/1.chunk.js:32659:28)
    at WithStyles (http://localhost:3000/static/js/1.chunk.js:185912:31)
    at div
    at RowMenu (http://localhost:3000/static/js/1.chunk.js:206785:23)
    at div
    at Waypoint (http://localhost:3000/static/js/1.chunk.js:335957:5)
    at RowHeader (http://localhost:3000/static/js/main.chunk.js:31918:3)
    at div
    at div
    at Cell (http://localhost:3000/static/js/1.chunk.js:207526:21)
    at div
    at td
    at TableCell (http://localhost:3000/static/js/1.chunk.js:29642:28)
    at WithStyles (http://localhost:3000/static/js/1.chunk.js:185912:31)
    at TableBodyCell (http://localhost:3000/static/js/1.chunk.js:268937:11)
    at tr
    at TableRow (http://localhost:3000/static/js/1.chunk.js:30572:23)
    at WithStyles (http://localhost:3000/static/js/1.chunk.js:185912:31)
    at t (http://localhost:3000/static/js/1.chunk.js:269010:12)
    at WithStyles (http://localhost:3000/static/js/1.chunk.js:185912:31)
    at tbody
    at TableBody (http://localhost:3000/static/js/1.chunk.js:29444:23)

@mandolyte why is this needed?

We are using React in a way that is permitted at the moment, but may stop working in the future. Near term, not too important; just annoying messages in the console log.


Is there support on Powershell v2.0

Affected Puppet, Ruby, OS and module versions/distributions

Puppet: 3.4.0
Ruby:
Distribution:
Module version:v2.0.0

How to reproduce (e.g Puppet code you use)
Using Windows 2008 R2, powershell 2.0.
windowsfeature.pp
windowsfeature { 'MSMQ':
ensure => present,
}
puppet apply windowsfeature.pp
What are you seeing
Error: Could not prefetch windowsfeature provider 'default': Execution of 'C:\Windows\sysnative\WindowsPowershell\v1.0\powershell.exe Get-WindowsFeature | ConvertTo-JSON' returned 1: The term 'ConvertTo-JSON' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a pathwas included, verify that the path is correct and try again.
At line:1 char:36

Get-WindowsFeature | ConvertTo-JSON <<<<
CategoryInfo          : ObjectNotFound: (ConvertTo-JSON:String) [], Comm
andNotFoundException
FullyQualifiedErrorId : CommandNotFoundException

What behaviour did you expect instead
Enable Windows feature
Output log
Any additional information you'd like to impart

Hi. I'm going to close this Issue. We deprecated the module and the new version is available here: https://forge.puppet.com/puppetlabs/iis


bugfix: remove extra shootName from external evaluation  targetURL

Description
Changes proposed in this pull request:

Fix issue in https://github.com/kyma-project/control-plane/pull/1605/files

Related issue(s)

/retest

/retest

/retest


Set default http proxy in settings.json

Description of the new feature/enhancement
You know, it's hard for Chinese users to link because of the GWF. So every time I run pip install xxx, I have to set up a proxy first.
set http_proxy=http://<IP_ADDRESS>:port
set https_proxy=http://<IP_ADDRESS>:port

Can you allow the user to set the default proxy in settings.json. That way I don't have to enter it once every time.
Proposed technical implementation details (optional)

Thanks for the suggestion! This is actually already being tracked by another issue on our repo - please refer to #2785 for more discussion.
/dup #2785

When can I support it


Assertion in dead member elimination with spec constants

Hi,
I have this compute shader:
#version 450 core

layout(constant_id = 0) const int c_0 = 1;

layout(local_size_x_id = 1, local_size_y = 1, local_size_z = 1) in;

void main()
{
}

Compiling it with glslang without opts will give this:
              OpCapability Shader
          %1 = OpExtInstImport "GLSL.std.450"
               OpMemoryModel Logical GLSL450
               OpEntryPoint GLCompute %main "main"
               OpExecutionMode %main LocalSize 1 1 1
               OpSource GLSL 450
               OpName %main "main"
               OpName %c_0 "c_0"
               OpDecorate %c_0 SpecId 0
               OpDecorate %9 SpecId 1
               OpDecorate %gl_WorkGroupSize BuiltIn WorkgroupSize
       %void = OpTypeVoid
          %3 = OpTypeFunction %void
        %int = OpTypeInt 32 1
        %c_0 = OpSpecConstant %int 1
       %uint = OpTypeInt 32 0
          %9 = OpSpecConstant %uint 1
     %uint_1 = OpConstant %uint 1
     %v3uint = OpTypeVector %uint 3
%gl_WorkGroupSize = OpSpecConstantComposite %v3uint %9 %uint_1 %uint_1
       %main = OpFunction %void None %3
          %5 = OpLabel
               OpReturn
               OpFunctionEnd

Using spirv-opt with -Os throws this assertion:
spirv-opt: /xxx/source/opt/eliminate_dead_members_pass.cpp:397: bool spvtools::opt::EliminateDeadMembersPass::UpdateConstantComposite(spvtools::opt::Instruction*): Assertion `inst->opcode() == SpvOpConstantComposite || inst->opcode() == SpvOpCompositeConstruct' failed.

Any idea what might be wrong?

May want to change title to:
Assertion in dead member elimination with spec constants

May want to change title to:
Assertion in dead member elimination with spec constants

Done. Thanks


{AKS} AKS to LA final region map update for Azure Monitor for Containers

Description

This PR is updating the AKS region to Log Analytics region mapping for the Container Insights service.
Both AKS and Log analytics have added new regions since the last update and this PR makes sure that we don't default to eastus for those new regions and actually map it to its proper location.
This is the final list that has been agreed upon by talking with the AKS, LA, Networking and other teams that use the default workspace naming convention.
Testing Guide

Scenario for testing:
Create an AKS cluster in one of the new regions with monitoring enabled (e.g. make sure the resource group is in a new region like: Norway East)
command ->
az aks create --resource-group myResourceGroup --name myAKSCluster --node-count 1 --enable-addons monitoring
Once this command finishes you should see that monitoring is enabled for this cluster and its sending data to the default norwayeast log analytics workspace.
You can check this by doing an 'az aks show --resource-group myResourceGroup --name myAKSCluster' and checking under the omsagent addon for the log analytics workspace id.
History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

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

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

@fengzhou-msft This PR is now ready for review. I've confirmed with the associated teams and the region map, short codes being used are final.

@fengzhou-msft This PR is now ready for review. I've confirmed with the associated teams and the region map, short codes being used are final.

@fengzhou-msft Have a look at this PR when you get a chance. Its ready to merge

@fengzhou-msft Have a look at this PR when you get a chance. Its ready to merge

@fengzhou-msft Have a look at this PR when you get a chance. Its ready to merge.

@fengzhou-msft Have a look at this PR when you get a chance. Its ready to merge.


bug: [Bug] App layout broken after closing the keyboard

Type: bug
Ionic Version: 2.x
Platform: android 5.1 webview
More info from Team Viewer Quick Support:

This issue is a weird one, one of our customers happened this while working with our app with Ionic 2-beta.10:

Here's the original layout, the image above in issue description is after i open the keyboard by clicking in an Input and then closing the keyboard by clicking away of Input:

I've tested in 2 different devices and it doesn't happen, so i think is an issue with that exact device, idk how to test if i don't own the device.

I'm finally able to replicate in my Galaxy tab A with Android 5.0.2 model SM-P555M, after testing this was the issue:
body {
  left: 50%;
  max-width: 1024px;
  transform: translateX(-50%);
}

For some reason the left: 50%; was breaking the responsiveness i have put to be able to use the app even in the browser, the fix? here:
body {
  top:  0;
  left: 50%;
  max-width: 1024px;
  transform: translateX(-50%);
}

Just adding the top propery with 0 as value


[CV] units and more detailed description of how to compute metrics

after going through some metrics in the current OBO, some descriptions are vague (at best) and/or ambiguous.
E.g. https://github.com/HUPO-PSI/mzQC/blob/master/cv/v0_1_0/qc-cv.obo#L194 says Log ratio of .... Log to that base? There is no universal agreement IMHO. Writing LogN or Log2 would be desirable and make QC values more comparable.
Also, most metrics could use a unit (e.g. for RT etc...). Is there a way to restrict (or even fix) the unit to be used?
Also, can we put restrictions on the values, e.g. fractions must be between 0-1. Reporting multiple fractions must sum to 1 etc...
The main idea/benefit of providing a tighter description is a) better interpretability of the data b) better comparability across tools which compute these metrics.

Whether or not to include units in the CV has been a topic of some discussion already. In the end we settled on having explicit units for the quality metrics in the mzQC files.
There should indeed be restrictions on some values. This is part of what the semantic validation of the official Python parser checks. So far we haven't made these restrictions explicit in the CV, but it might be relevant to consider that.

Overview of semantic validation in issue #50. Please add any checks that should still be included.

I agree, that many CVs need further description. It would be great, if the inventors of the metrics, or the persons who created them in their programs/workflows/pipelines could provide a better description.
The descriptions used by me so far where from some documents I found when I started to take over the CV or my own tries to interpret the CV.
The best way to get good descriptions for the items in the CV would be, if you could open an request-new-CV-entry-issue with the description etc.
(I know, I did not have much time in the last weeks, but should have some more to finish v1.0 now).

Probably the best way to get better descriptions is to ping the tool developers directly and explicitly mention the metrics that need to be improved. Otherwise I'm afraid not much will happen, as I doubt that many people will have seen this issue. :)

This will be handled by the "comment" information in the CV and the requirements for better descriptions in the future.


fix: typo 'the the'

Coverage increased (+0.002%) to 97.171% when pulling ddaa29b6c724570ee2fd639a605577a5c1e4182f on gonzalezger:patch-1 into 6e5ab83c57902ec8d258b4277d5d8a31feb41eaf on Vincit:master.

Thanks


feat: add "new" command and connect with Studio

Description
It adds the asyncapi new command and optionally connects it with a local instance of Studio. See #78 for more info.
Related issue(s)
#78

[x] Implement asyncapi start studio command.
[x] https://github.com/asyncapi/studio/pull/141

Just been through it, mostly LGTM @fmvilas
A couple of thoughts:

Maybe you should update the README with the new command here as part of this PR? so we document the feature and how to use it?

I guess when you opt into the studio in the CLI the websocket stuff will also happen behind the scenes. Wonder if this should be optional or not... the socket/live changes. Let's leave it for now maybe... but wonder if people would find value not having it synced.... dunno.

Maybe you should update the README with the new command here as part of this PR? so we document the feature and how to use it?

Oh yeah, definitely. Going to update it. Thanks.

I guess when you opt into the studio in the CLI the websocket stuff will also happen behind the scenes. Wonder if this should be optional or not... the socket/live changes. Let's leave it for now maybe... but wonder if people would find value not having it synced.... dunno.

Let's find out. If people ask for this feature (or lack) they can add it later.


Confirm 'gps_mapper' functionality and fix naming scheme across a few files.

Closes #83.

Note that this leaves the great-circle distance/haversine calculation in bearing.py incorrect (not dividing by 2 where it should be) so that @joshuamorton does not get conflicts during merge, which means that still needs to be fixed at some point by someone.

Nah, haversine works.


Add improved scene support to the light integration

Description:
Add improved scene support to the light integration.
Related issue (if applicable): fixes #26962
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[x] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[x] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

The failed tests also fail in the current dev branch.

These error occur here:

test_reproduce_turn_on(hass)
https://github.com/home-assistant/home-assistant/blob/8ba4ee1012fcdbfe7b238ef1ac4de34e09adf641/tests/helpers/test_state.py#L132

>       assert ["light.test"] == last_call.data.get("entity_id")
E       AssertionError: assert ['light.test'] == 'light.test'

test_reproduce_turn_off(hass)
https://github.com/home-assistant/home-assistant/blob/8ba4ee1012fcdbfe7b238ef1ac4de34e09adf641/tests/helpers/test_state.py#L149

>       assert ["light.test"] == last_call.data.get("entity_id")
E       AssertionError: assert ['light.test'] == 'light.test'

test_reproduce_complex_data(hass)
https://github.com/home-assistant/home-assistant/blob/8ba4ee1012fcdbfe7b238ef1ac4de34e09adf641/tests/helpers/test_state.py#L170

>       assert complex_data == last_call.data.get("complex")
E       AssertionError: assert ['hello', {'11': '22'}] == None

Unfortunately I've no idea why these tests fail.

Retroactively adding the breaking change label.  This was painful.

@Santobert could you please edit the PR and add instructions for people who previously used light scenes with transition, brightness_pct, and other "non standard" attributes?  What do they need to do to achieve similar functionality?

@andrewsayre I'm not sure anybody's ever gonna read this, but here it is. Did I miss anything?

Thanks—people will read it because the PR is linked to the issues.  :(


Remove 2d_coords feature branch

WHEN :  internal testing environment is removed, following internal release of next Iris version (2.2).

👀


Nlog support for MassTransit + log4net logging docs

When you merge my semver pull request you'll have to fix the versioning in the added nuspec also, to correspond to the semver branch'es.

Think I have this done.


invalid literal for int() with base 10: b''

Hello!
When I try to test the new version of instaloader (3.3.1) I've got this error:
$ python3 instaloader.py  mag_bush
Traceback (most recent call last):
File "instaloader.py", line 1754, in
main()
File "instaloader.py", line 1748, in main
filter_str=args.only_if)
File "instaloader.py", line 1574, in main
filter_func=filter_func)
File "instaloader.py", line 1434, in download_profile
name_updated, profile_id = self.check_profile_id(name, profile_metadata)
File "instaloader.py", line 1349, in check_profile_id
profile_id = int(id_file.read())
ValueError: invalid literal for int() with base 10: b''
$ python --version
Python 3.6.4
I've also noticed that the directory "bush_mag" was created with the file called "id" in it. But the file is empty.
Best regards

I've just noticed that the directory "bush_mag" was created with the file called "id" in it. But the file is empty.

I've also noticed that the directory "bush_mag" was created with the file called "id" in it. But the file is empty.

Could you delete that id file and then try again? This file probably has been created while Instaloader was broken with bug #81, and now it should work fine.

Oh,  now it works perfectly! )
Thank you very much!


Add livenessProbe and readinessProbe

Adds default livenessProbe and readinessProbes, along with customLivenessProbe and customReadinessProbe to the Trino chart for both the coordinator and workers.

I'll take a look this weekend. sorry about the delay.

LGTM and let's reword commit to Allow customising liveness and readiness probes since the probes already existed.


🚀 Filter our deprecated APIs

Is your feature request related to a problem? Please describe.
Some of the APIs stop working over time either if they are stopped being worked or not maintained for X period of time or what-so-ever be the reason.
We need to filter out such APIs from the list.
Describe the solution you'd like
A simple solution would be to add a simple key in the feature list of the API. The key can either be deprecated or broken with boolean value true for any such APIs which are not working.
Describe alternatives you've considered
N/A (Feedbacks are welcome)
Additional context
Anyone working on this will have to proofread all the existing APIs to see if they are working. If not, add the key to that specific API already.
Also the frontend has to be prepared accordingly by adding a tag in the card to properly portray this information.

Fixed by 54a2163f935078f4f365ac6b40b3404280b435c9


feat: cli support display more meaningful fields while list

Problem:
Before cli only show name, create time, id while list
Solution:
Support more fields while list/get
Related Issue:
#1170
#1582

How does resource list look like now?

How does resource list look like now?

Can we show type(like the UI) and endpoints for resources?


RDK-32110 : DIAL Dynamic App Config

Reason for change:
DIAL Dynamic App Config
rdkservice changes.
Retry logic fix.
Test Procedure: None
Risks: Low
Change-Id: I75837653dd52f165ddf559c7f36c49592af766f1
Signed-off-by:Anooj Cheriyan<EMAIL_ADDRESS>(cherry picked from commit 4f530691ae075ecf22ac535d771f8bbfd4f43fdb)

New one created: https://github.com/rdkcentral/rdkservices/pull/1482


Why sif is converted to a sandbox?

Hi,
I built a singularity box (.sif). Why is it converting to a sandbox before executing a command? This takes a long time. Sometimes it fails due to memory issues. Is there a shorter way to run qsiprep in an HPC system. In any case, singularity box is too big!
...@head8:~/qsiprep$ singularity shell qsiprep-0.13.0RC1.sif
INFO:    Convert SIF file to sandbox...

Never mind! This only happens when I run it under conda. Solution is to deacitave conda/conda's singularity and use the system version of  singularity.


✨(domains) allow creation of "pending" mailboxes

Purpose
Previously, mailbox creation was restricted to "active" domains. We now allow users to create mailboxes on pending/failed/disabled domains, and mailboxes thus created have the "pending" mailboxes status. They are sent to dimail upon domain status switching to "active".
Proposal
Description...

[] item 1...
[] item 2...

Should we allow mailbox creation for all three types ? Pending / Failed / Disabled domains ?
@sdemagny @Morendil

@mjeammet I'd say yes, always create.
My current working metaphor (open to being replaced by a better one) is "moving into a new home". There is no heating/electricity/water yet (the domain is not connected) but as long as you are in the building, it is comforting and keeps the project moving if you are able to set up the furniture the way you want (creating groups and mailboxes).
I'm not sure exactly what domain state transitions we are effecting automatically, but assuming I understand the intentions:
Pending has the meaning of "there is no electricity yet". Cool, while I wait for that to happen I can keep setting up.
Failed has the meaning of "we tried setting you up with the electricity provider but something went wrong". Well, I'll have to look into what went wrong, but perhaps I don't know what to do and have to leave a message. Meanwhile, I can hope things will get fixed, and keep setting up.
Perhaps Disabled is different.
Disabled maybe has the meaning of "you said you didn't want to live here any more so we're cutting off electricity", and in that case there's no reason to keep rearranging furniture. But perhaps it has another meaning like "this domain was an alternate address for the same building" - if we support domain aliasing. We don't yet know how we're going to handle domain aliases.
But my heuristic in these cases would be to implemented the simplest thing, and the simplest thing is to not pay attention to domain status at all.

Dans les deux commits à squasher avec le reste si ça te va Marie :

[x] gestion de l'interdiction de créer une boite mail lorsque le domaine a été désactivée
[x] ajout des trads manquantes (il en manque plein mais j'ai juste traduit celles qu'on utilise dans notre PR)
[x] ajout du status au serializer de la mailbox (le front pourra l'afficher dès maintenant comme ça)


DetectBestOrientation with tesseract 4 api

Hello,
do you plan to re-add the function DetectBestOrientation? In the current version it into comments.
This works like normal:
Interop.TessApi.Native.TessBaseAPIDetectOrientationScript(Engine.Handle, out, deg, out ori_cnf, scr_nm out, out scr_cnf)
p.s.: sorry for my english

I can not finded autorotate in last changes, so:
I added next code on "Page.cs":
    public void DetectBestOrientation(out int deg, out float ori_cnf, out string scr_nm, out float scr_cnf)
    {
        if (Interop.TessApi.Native.TessBaseAPIDetectOrientationScript(Engine.Handle, out deg, out ori_cnf, out scr_nm, out scr_cnf) != 0)
        {
        }
        else
        {
            throw new TesseractException("Failed to detect image orientation.");
        }
    }

And use it in my vb.net programm:
Dim page2 As Page
Dim img_ts2 As Pix
img_ts2 = Pix.LoadFromFile(new_img_name)
Dim engine2 As TesseractEngine
engine2 = New TesseractEngine(F_auth.TESPATH, res_lang2, T4_engine_mode)
page2 = engine2.Process(img_ts2)
page2.DetectBestOrientation(deg, ori_cnf, scr_nm, scr_cnf)
In "deg" I get correct degree of rotation.
But... If deg=0, I can't save image, because it used. And if deg<>0, image rotates and saves, but it drops on line: GC.WaitForPendingFinalizers().
Is that way of correct search best page orientaion?

Pix and Page inherit from IDisposable.  Maybe you need to wrap them in using statements?

Hello tdhintz,
I rewrite code:
Using img_ts2 As Pix = Pix.LoadFromFile(new_img_name)
If res_lang2.Length > 0 Then
Dim engine2 As TesseractEngine
res_lang2 = Mid(res_lang2, 1, res_lang2.Length - 1)
engine2 = New TesseractEngine(F_auth.TESPATH, res_lang2, T4_engine_mode)
For j = 0 To vars.Item(0).Count - 1
engine2.SetVariable(vars.Item(0).Item(j).Item(0), vars.Item(0).Item(j).Item(1))
Next
Using page2 As Page = engine2.Process(img_ts2)
page2.DetectBestOrientation(deg, ori_cnf, scr_nm, scr_cnf)
End Using
End If
Dim t_f As Tesseract.ImageFormat
Select Case img_format.ToString
Case Imaging.ImageFormat.Jpeg.ToString
t_f = Tesseract.ImageFormat.JfifJpeg
Case Imaging.ImageFormat.Tiff.ToString
t_f = Tesseract.ImageFormat.Tiff
Case Imaging.ImageFormat.Png.ToString
t_f = Tesseract.ImageFormat.Png
End Select
Select Case deg
Case 270
img_ts2.Rotate90(1).Save(new_img_name, CType(t_f, Tesseract.ImageFormat))
Case 180
img_ts2.Rotate90(1).Rotate90(1).Save(new_img_name, CType(t_f, Tesseract.ImageFormat))
Case 90
img_ts2.Rotate90(-1).Save(new_img_name, CType(t_f, Tesseract.ImageFormat))
Case Else
img_ts2.Save(new_img_name, CType(t_f, Tesseract.ImageFormat))
End Select
End Using
There still proplems with threads(?).
If I used this line:
page2.DetectBestOrientation(deg, ori_cnf, scr_nm, scr_cnf)
then procedure:
Public Sub MEMORY_CLEAR()
GC.Collect()
GC.WaitForPendingFinalizers()
GC.RemoveMemoryPressure(GC.GetTotalMemory(True))
EmptyWorkingSet(System.Diagnostics.Process.GetCurrentProcess.Handle)
SetProcessWorkingSetSize(System.Diagnostics.Process.GetCurrentProcess.Handle, -1, -1)
End Sub
was dropped without message on line:
GC.WaitForPendingFinalizers()
DetectBestOrientation create new thread, maybe problem in it...
DetectBestOrientation on tesseract 3 version worked without this problem.

I have a few observations.  None of these identify the problem but they might be clues.

DetectBestOrientation is not a class or a managed object, so it has no direct finalizer.
There is no try/catch in MEMORY_CLEAR as stated above.  So, what do you mean by the word "dropped"?  An exception?  A hang or freeze?  An uncatchable process crash?
You note the wrapper for DetectBestOrientation has out parameters (I didn't audit the code myself), but the example shows no out parameters.  That doesn't seem possible.

I have added try-catch in MEMORY_CLEAR. It can't catch this error:

next step:

Interesting, that application breaks in second call MEMORY_CLEAR after DetectBestOrientation.
MEMORY_CLEAR used many times in application: before and after recognize for example, and this problem never happened.
Then I uncomment "Exit Sub" line, and once it breaks on procedure:
    Shared Function LoadImage(ByVal imagePath As String) As Image
        Dim img As Image = Nothing
        Try
            Using mstream As New IO.MemoryStream(IO.File.ReadAllBytes(imagePath))
                img = Image.FromStream(mstream)
            End Using
        Catch
            img = Nothing
        End Try
        Return img
    End Function

and sometimes it works without breaks,
and sometimes it breaks in other places, which difficult to find, because try-catch do not help.
If I push "debug", I will get following message:

It seems that the problem in libtesseract400.dll
Maybe it need call some procedure like "correct_finalize_orientation_script" or "free_resources" or something like that after TessBaseAPIDetectOrientationScript
About out parameters: I do not understand.
There is a line in "BaseApi.cs":
int TessBaseAPIDetectOrientationScript(HandleRef handle, out int orient_deg, out float orient_conf, out string script_name, out float script_conf);
so in DetectBestOrientation I use same parametrs:
     public void DetectBestOrientation(out int deg, out float ori_cnf, out string scr_nm, out float scr_cnf)
        {
            if (Interop.TessApi.Native.TessBaseAPIDetectOrientationScript(Engine.Handle, out deg, out ori_cnf, out scr_nm, out scr_cnf) != 0)
            {
            }
            else
            {
                throw new TesseractException("Failed to detect image orientation.");
            }
        }

And in my application:
                    Dim deg As Integer
                    Dim ori_cnf As Single
                    Dim scr_nm As String = ""
                    Dim scr_cnf As Single
...
page2.DetectBestOrientation(deg, ori_cnf, scr_nm, scr_cnf)

In tesseract sources I find:
bool TessBaseAPI::DetectOrientationScript(int* orient_deg, float* orient_conf, const char** script_name, float* script_conf)
which use
bool TessBaseAPI::DetectOS(OSResults* osr)
which use
int orientation_and_script_detection(STRING& filename, OSResults* osr, tesseract::Tesseract* tess)
which use
int os_detect(TO_BLOCK_LIST* port_blocks, OSResults* osr, tesseract::Tesseract* tess)
which use
int os_detect_blobs(const GenericVector<int>* allowed_scripts, BLOBNBOX_CLIST* blob_list, OSResults* osr, tesseract::Tesseract* tess)
which use
bool os_detect_blob(BLOBNBOX* bbox, OrientationDetector* o, ScriptDetector* s, OSResults* osr, tesseract::Tesseract* tess)
which use
bool OrientationDetector::detect_blob(BLOB_CHOICE_LIST* scores)
But this information does not help me...

Update:
As sayed on this page:
https://github.com/charlesw/tesseract/issues/340
TessBaseAPIDetectOrientationScript is on 3.05 too.
And on this page:
https://github.com/charlesw/tesseract/blob/develop/src/Tesseract/Page.cs
there is method to use it.
So I copyed it to current sources. And it seems to works.


[Gradle] generateContractTest not copying contracts specified in contractDepencency due to wrong pattern to pick contracts

This issue is extremely similar to https://github.com/spring-cloud/spring-cloud-contract/issues/495
I've created a minimal reproduction here. This is what I was trying to accomplish.
I have a project, called contracts that I want to include my API and my contracts (written in groovy in this case). The contracts project is published to (my local) artifactory on build. I have another project called tests. The tests project needs the contracts defined in the contracts project to generate the tests and stubs.  The tests project uses the contractDepenency setting to accomplish this:
contracts {
    contractDependency {
        stringNotation = 'com.test:contracts:+:stubs'
    }
    testMode = 'MockMvc'
    contractsWorkOffline = true
}

Unfortunately no tests or stubs are generated when building the tests project. The folder generated-test-sources stays empty and the stubs folder contains com.test/tests/1.0/contracts but the contracts folder is empty.
I've debugged this issue by running gradlew generateContractTests -d > tmp.tmp. The most interesting part is:
Will download contracts for [com.test:contracts:+:stubs]
Desired version is [+] - will try to resolve the latest version
Resolved version range is [{1.0=C:\Users\rtriesscheijn\.m2\repository (enhanced)}]
Will download the stub for version [1.0]
Resolved version is [1.0]
Resolving artifact [com.test:contracts:jar:stubs:1.0] using remote repositories []
Resolved artifact [com.test:contracts:jar:stubs:1.0] to C:\Users\rtriesscheijn\.m2\repository\com\test\contracts\1.0\contracts-1.0-stubs.jar
Unpacking stub from JAR [URI: file:/C:/Users/rtriesscheijn/.m2/repository/com/test/contracts/1.0/contracts-1.0-stubs.jar]
Unpacked file to [C:\Users\RTRIES~1\AppData\Local\Temp\contracts3545973372033847890]
Will pick a pattern from group id and artifact id
Pattern to pick contracts equals [^C:\\Users\\RTRIES~1\\AppData\\Local\\Temp\\contracts3545973372033847890(\\)?.*com\\test\\tests\\.*$]
Ant Pattern to pick files equals [**/com/test/tests/**/]
[C:\Users\RTRIES~1\AppData\Local\Temp\contracts3545973372033847890] to [C:\Projects\tmp\spring-cloud-contracts\tests\build\stubs\META-INF\com.test\tests\1.0\contracts]. The inclusion ant pattern is [**/com/test/tests/**/*.*]

As you can see the jar file with the contracts is successfully downloaded and if I look inside the temporary folder the contract is present:
contracts3545973372033847890\META-INF\com.test\contracts\1.0\contracts\someContract.groovy

The pattern to pick the contract is definitely wrong in multiple ways:

It uses the artifact name tests instead of contracts
It uses the package name com\test instead of com.test

Am I doing something very wrong, or is this a bug in one of the gradle functions?

Something that seems to work is changing the contracts section in the gradle file of the tests project to this. Does this accidentally fix it, and is it still a bug? Because I don't see anything about this in the docs (or did I miss it?).
contracts {
    contractDependency {
        stringNotation = 'com.test:contracts:+:stubs'
    }
    contractsPath = '/'  // <---- using '' doesn't work but using '/' seems to fix it.
    testMode = 'MockMvc'
    contractsWorkOffline = true
}

Issue is in the slashSeparatedGroupId() (as pointed out in ticket#495): https://github.com/spring-cloud/spring-cloud-contract/blob/c354bffafbd37ad1da4d80294521445420634bf4/spring-cloud-contract-stub-runner/src/main/java/org/springframework/cloud/contract/stubrunner/ContractDownloader.java#L65
Workaround is to set the contractsPath to some text to not end up in L63-L65.

So maybe you want to create a test and file a PR? :D

I would make a PR, but I'm not sure what the expected logic should be in L63-L65 of ContractDownloader. I'm afraid I would make a fix that just works for me.

Sure, so maybe start with a test that is failing for you. That will be a good starting point :)

I think you've made a mistake and I've made a bug :)
THE BUG
The bug is related to the fact that initially, we were supporting slash-separated paths to projects. E.g. if the group id was a.b.c then we did checked for a/b/c folder. At certain point we've started building stubs with the groupid folder as such e.g. a.b.c/artifact-id. This is where the bug comes in. For Maven and Gradle we're using ant patterns for copying files. We're only providing the slash-separated ant pattern. With this fix, we're backward compatible (we're leaving the slash-separated one) and we're adding support for dot-delimited group id.
THE MISTAKE
├── build.gradle
├── settings.gradle
└── src
    └── test
        └── resources
            └── contracts
                └── someContract.groovy

If the contracts folder was to contain the contracts for all other apps then your structure is wrong and you're building the jar in the wrong way. First of all you're building the contract jar as if contracts was an app. So after building the jar your stub structure looks like this:
├── build
│   ├── stubs
│   │   └── META-INF
│   │       └── com.test
│   │           └── contracts
│   │               └── 1.0
│   │                   ├── contracts
│   │                   │   └── someContract.groovy
│   │                   └── mappings
│   │                       └── someContract.json

contracts is the name of the artifact id of the contracts project. That makes absolutely perfect sense. If you wanted contracts to contain contracts that all other projects were to use you should structure it like here https://cloud.spring.io/spring-cloud-static/Edgware.SR2/multi/multi__spring_cloud_contract_faq.html#_common_repo_with_contracts and package it differently. So in your case you should either NOT use the contractDependency closure, and pass the path to the other project as the source of contracts, or use the contractDependency and modify the contractsPath (as you already have).


Oveflow and paranoid checks should be enabled by default for Rust contracts

Rust supports overflow checks for integer arithmetics, however they are disabled by default for optimised builds.
These checks are important because of an attack vector when dealing with user balances.
Checks should be enabled in the generated Cargo.toml by default
https://github.com/near/create-near-app/blob/master/common/contracts/rust/Cargo.toml
More information here:
https://stackoverflow.com/q/64129432/315168

@chadoh Could you, please, triage this issue?

Is this sufficient, @frol? https://github.com/near/create-near-app/pull/502

@chadoh Is it the only place where the Cargo.toml needs to be updated to reflect all the examples? I think we also need to go over our example contracts in Rust and update those.

Yes, good call, I'll apply the same update added in #502 to all Rust examples

Updated all examples and merged #502; thanks for the ideas @miohtama and the help @frol


analysis server doesn't always recognize when pub get has been performed

From https://github.com/flutter/flutter/issues/4031, we've seen cases where the analysis server doesn't recognize that the .packages file has changed on disk. A reanalyze sources will fix the issue. Perhaps related to file watching code?

Possibly. The two most likely scenarios are that (a) server is not being notified that the file has changed or (b) for some reason it's deciding that nothing has changed so nothing needs to be done. I don't remember, though, having any code to try to optimize the case where a .packages file contains the same map as before, so (a) seems more likely.

Do we have a way to reproduce this behavior?

From reading the original bug, it looks like it happened right after project creation, at a kiosk at a trade show; @sethladd might have more details.

Does pub run update the .packages file or should this be pub get or pub upgrade?

pub get; good catch :)

closing as assumed stale


[#153872395] Update io-ts package

see https://github.com/gcanti/io-ts/issues/8

awesome

LGTM, please update yarn.lock as well, then merge