Push Read the Docs Generated Markdown to Github pages

Push Eric's read the docs stuff to github pages.

@bendhouseart Why is this Issue here?  I think the decision was to go with readthedocs instead, right?

You're right, closed.


Spacing for comments

[feature request]
It would be nice to have options to enforce the spacing of comments in code. E.g.,
var a = b; // does not stick to the code
var a = b;// sticks to the code
var a = function(b /* comment sticks right */, c /* comment sticks right */) {}
var a = function(b/*comment sticks left*/ , c/* comment sticks left*/ ) {}
var a = function(b/* comment sticks left & right */, c/* comment sticks left & right */) {}
var a = function(b /* comment doesn't stick */ , c /* comment doesn't stick */ ) {}
// newline comments are always valid on the left side
/* newline comments are always valid on the left side*/
var a = b; /* comments followed by a newline are always valid on the right side */

This request has two reasons:

nice, consistent styling
to fix issues with Illegal space before closing round bracket at input if "disallowSpacesInsideParentheses": true  is used, for code like function(el /* comment */, i /* comment */) {} or if (this /*strict mode unsupported*/) {}

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

This issue has been open more than a month without supported PR for it.
Unfortunately, we don't have the resources to implement rules that could not be included in any of the supported presets (if it could be, please point that out).
However, if you or anybody else submit PR for it, we will gladly consider it.


Muller's method

What kind of problems is it mostly used for? Please describe.
Determining the roots of univariate complex functions. Muller's method is commonly used to determine the (quasi-normal) oscillation modes of neutron stars [1, 2].
Note: Given that this method only works for univariate functions, it might be too restrictive for NonlinearSolve.jl, but I thought I would suggest it anyway, since I frequently use it. Also, the algorithm is very simple so it should be easy to implement.
Describe the algorithm you’d like
The algorithm is quite simple and described well in Press et al. [3]. (See also Wikipedia [4].)
To summarise: Muller's method is a generalisation of the secant method, with the key difference being that it uses quadratic interpolation across three points (as opposed to linear interpolation among two). Solving for the roots of a quadratic is trivial and allows the method to work for complex roots.
Other implementations to know about
There is an existing Julia implementation in Roots.jl [5] and a Fortran routine in IMSL [6].
References
[1] Kokkotas & Schutz (1992)
[2] Kruger, PhD Thesis (2016)
[3] Press et al. (2007), Sec. 9.5.2, p. 466
[4] Wikipedia, Muller's method
[5] Roots.muller
[6] IMSL Math/Library Users Manual, Zanly

Given that this method only works for univariate functions, it might be too restrictive for NonlinearSolve.jl

This would be great to have in SimpleNonlinearSolve

yes open the pr


chore: update commitlint rules to not choke on long, dependabot-generated PR titles

Sometimes dependabot will generate a long PR title that runs-afoul of commitlint's "header-max-length" rule which only allows 100-or-less chars (currently).

This issue is a request to update the commitlint config such that we can avoid having to manually update dependabot PR titles just to get them to pass CI title checks.

Possible options include:

configure commitlint with a longer header-max-length value, or
find a way to exclude just dependabot PR titles from the header-max-length rule (ala here).


[PE-D][Tester C] Edit Command detection

It seems that there is some issues in the command detection for edit command. This is similar to the previous bug.

Since this happens for some commands, I'm thinking its not specific for any command, but maybe the class or implementation that handles the command detection.

Labels: severity.Low type.FeatureFlaw
original: edures888/ped#6

Solution
Fix it


QGC Can not update airframe

system：window7
To Add New Airframe-“wingwingwingwing” to QGroundControl
I followed https://dev.px4.io/en/airframes/adding_a_new_frame.html
a new airframe was added to directory
D:\PX4\home\Firmware\ROMFS\px4fmu_common\init.d\airframes

Content is

And  “3039_wingwingwingwing” was added to CMakeLists.txt file

running make clean and then make px4fmu-v5_default
px4_fmu-v5_default.px4 was build successfully and
###\Firmware\build\px4_fmu-v5_default\ airframe.xml file was updated

But  when I upload the firmware to pixhawk4 ,(I do it twice --from QGC and from command )
nothing was happened

There is no new airframe”wingwingwingwing” added
I do not know What’s wrong
Need help!!!

I think I know why this happened
this issue can be closed
thanks!


Define an API for DCs

Currently, there is a callback for using DC: GetDelegatedCredential. This can be improved to:

[ ] Use the GetCertificate and GetCertificateClient callbacks to return the DCs as part of the Certificate structure
[ ] Allow the possibility of returning a list of DCs: the appropriate DC will be chosen by using the advertise signature algorithm on the ClientHello/Certificate Request extension.
[ ] Refactor the DC tests.

cc./ @armfazh @wbl

Related to #63

I like the idea of adding DCs to the crypto/tls.Certificate structure, since it allows us to expose DCs by whatever means is easiest to configure. Something like this?
type Certificate struct {
   ...
   DCs [][]byte
   DCPrivateKeys []crypto.PrivateKey
}

so that DCPrivateKeys[i] is the private key of DCs[i] for all 0 <= i <= len(DCs)?

@cjpatton check the latest pr: https://github.com/cloudflare/go/commit/dc791fdbeeacff13d3e1f076106a013fd56cd5f3

You're idea is much better than mine! I left some comments on the PR.

Merged now. Closing this.


Organize imports and formatters are conflicting due to a race condition

https://github.com/microsoft/vscode-python/issues/6933
This is not an extension issue though.

VSCode Version: 1.39.1
OS Version: Windows 10

Steps to Reproduce:

Have Black installed
Set config options as such:

{
  "editor.formatOnSave": true,
  "editor.codeActionsOnSave": {
    "source.organizeImports": true
  }
}

Create a python file with the following contents:

from something import (
    aaaaa,
    bbbbb,
    ccccc,
    ddddd,
    eeeee,
    fffff,
    ggggg,
    hhhhh,
    iiiii,
    jjjjj,
    kkkkk,
)

Save the file multiple times and watch it dance

Does this issue occur when all extensions are disabled?: No
The problem is that there's a race condition  since there are two formatters being used when saving changes to a file. The results vary based on which formatter completes first. I.e. if black formats first, then you see one result, then isort formats the imports and changes the imports and you see another set of changes.
As you keep hitting save both trip on one another.
The Python extension wants to organize formatters and isort to be ran consistently, but we don't have control over that as VS Code is doing the excutions.

I know this issue is closed but it is the closest I found to an issue I have with isort.
Did you have any setup.cfg files to configure isort?
I was struggling to get it to work with the setup.cfg files in my project directory.
isort -sp .\setup.cfg
Produces a different sorting than if I use the editor's on-save formatting.
Looking at your description of this issue it is possible that it is actually using setup.cfg but the race condition is messing up my imports.
In your journey with VSCode + isort did you find that it checked for and respected any setup.cfg file?
If not (or you are unsure) then at least I can open a new issue to be investigated.

@baolsen sorry I don't think I understand what you mean, this issue is open, not closed.

@baolsen – I don't think the race issue is related to VS Code not reading isort settings from setup.cfg.  At least for me, I can consistently get import sorting to work as expected if I specify the settings file in my VS Code settings:
  "python.sortImports.args": [
    "--settings-path=/path/to/my/setup.cfg"
  ],

Why the above doesn't happen automatically is unclear to me, but that is for a different Issue.

@pydolan's suggestion did seem to make the behavior consistently correct for me. I recommend using the ${workspaceFolder} var like so instead of hard coding a path.
"python.sortImports.args": [
    "--settings-path=${workspaceFolder}/setup.cfg"
]

As an aside, I also had to add "editor.codeActionsOnSaveTimeout": 2000 to my config in order to extend the timeout beyond the default 750ms. Otherwise, my file was showing as unsaved after isort had sorted the imports. Seems like isort is taking more than 750ms to complete. See issue #72496 for a better description of the problem. This problem may only occur on larger projects.

I really hope that this gets addressed soon. It keeps us from being able to sort imports on save.
This race condition leads to deleted or duplicated code. That feels quite significant to me for this issue to be fixed.
Code actions should run before the code formatter.
They should run strictly sequentially.

@mjbvz The comments made by @baolsen @pydolan and @reywood seem to address an issue unrelated to this issue. For clarity: Can these comments be removed from this issue?

Here's what I see debugging this setup for JS/TS:

Organize imports is called. It returns a code action with a command: https://github.com/microsoft/vscode/blob/c0c9e0043ce6133042268f22847021ad9a1ce410/extensions/typescript-language-features/src/features/organizeImports.ts#L76
That command is then invoked and edits the document (making sure to await the result in the command) : https://github.com/microsoft/vscode/blob/c0c9e0043ce6133042268f22847021ad9a1ce410/extensions/typescript-language-features/src/features/organizeImports.ts#L55

Then the formatter is called. At this point the formatter has the updated document that organize imports just changed

From this, it seems to me that VS Code calls things in the correct order and does correctly await the result of organize imports
@luabud  Can you please share a link to the python extension's implementation of organize imports?

I actually can't repro the race condition anymore, it's consistently organizing the imports first and then triggering the formatter. And it seems that if we wanted to change the order, we could do it in the extension side, so I'm closing this issue.
FWIW the code for sorting imports is here: https://github.com/microsoft/vscode-python/blob/master/src/client/providers/importSortProvider.ts

I can very much still reproduce this. I just used the settings.json from above:
{
  "editor.formatOnSave": true,
  "editor.codeActionsOnSave": {
    "source.organizeImports": true
  }
}

and opened a random python file. Directly on the first one it dances and deletes lines of code.
I extracted these lines of code into an empty python file ans pressed STRG + S multiple times (nothing else):

@Spenhouet Oh that's weird! I can't repro this behaviour with the sample code you proved. I tried it with black, autopep8 and yapf.
I opened an issue on the Python extension repo so we can investigate it  (https://github.com/microsoft/vscode-python/issues/10923), as it may not be due to a race condition.  It'd help immensely if you could fill up our bug report template there providing your environment information, the formatter you're using as well as the contents of the Python channel in the Output:

@luabud Sorry, I was mixing up issues. I was actually referring to this: https://github.com/microsoft/vscode/issues/90220

same issue here, the worst thing is that it even deletes some imports

@copdips do you see the import deletion behaviour consistently? I wasn't able to reproduce it.

@luabud
Yes, as per the GIF in https://github.com/microsoft/vscode/issues/83586#issuecomment-607497052, it deletes imports.
But I managed to find a workaroud from: https://github.com/microsoft/vscode/issues/90221#issuecomment-583664840 by using source.organizeImports.python instead of source.organizeImports

@luabud I can reproduce this again with the following import:
from transform_data import (preprocess_continous_image, transform_image_parallelized,
                            weighted_majority_resampling)

Maybe you could try if you can also reproduce it with that import.
Formatter is yapf.


"No internet" bug is reported by both tinycheck and seewho users

"No internet" bug is reported by both tinycheck and seewho users
Please add comments here with proof of links or screenshots where users report this problem so that @ppetru has more to go on in troubleshooting.

https://github.com/KasperskyLab/TinyCheck/issues/98


httpOnly property is missing in the type definition of 'CookieAttributes'

I believe the httpOnly property is missing in the type definition of 'CookieAttributes'.
I am getting this error when setting the httpOnly property.
Object literal may only specify known properties, and 'httpOnly' does not exist in type 'CookieAttributes'.

Nevermind, this has already been asked.


Update readme for version <IP_ADDRESS>

Hi @KarolKaczmarek, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Karol Kaczmarek (CHARLES)). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


[BEAM-1620] Run ValidatesRunner on Dataflow streaming in postcommit

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify.
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @davorbonaci @jasonkuster
This may take effort to get executing reliably - please give feedback on the approach to getting this execution included. Example alternative: a separate Jenkins build. This might be sensible for parallelism, but might be extraneous given other changes happening.

Coverage remained the same at 70.577% when pulling 85d7b214a27b26ffca6fe56d32e8736d275ef354 on kennknowles:Dataflow-streaming-ValidatesRunner into 53dd0a529a3570d4a6edef4f3e59938403817df4 on apache:master.

Given the option of using a regex (or other maven plugin) to munge the JSON versus using a side-channels system property, I chose the latter.

run dataflow validatesrunner

Looks like there's breakage on master.

run dataflow validatesrunner

The build that is ostensibly waiting is simply not queued. Rebasing and retriggering.

run dataflow validatesrunner

Coverage increased (+0.0006%) to 70.591% when pulling d8f4d6399a844f3ec4cbfe6b5b20f9f816e7ba4e on kennknowles:Dataflow-streaming-ValidatesRunner into 6a2586a4ee279f66fd4eae816e93fa8e397acf6b on apache:master.

Coverage increased (+0.008%) to 70.488% when pulling ba5067f39db2314d7fe3ffc7bf05cc56d658d522 on kennknowles:Dataflow-streaming-ValidatesRunner into e2be6961d07f1c7efe73dbc00280c8e5c9c94dfa on apache:master.

The failure looks to be a legitimate bug in TestDataflowRunner. Debugging should commence after #2988.

run dataflow validatesrunner

run dataflow validatesrunner

run dataflow validatesrunner

Precommit is now pushing 100 minutes, so it timed out after finishing Python but before finishing aggregated javadoc.

We've seen some streaming pipelines run and terminate correctly, so I'm optimistic about the config. It will blow our timeout on ValidatesRunner, since we already had to bump it from 100 to 120 just to finish the batch tests.

We've got good stuff in the works to make this work with Jenkins pipelining/parallel execution so until then we might find a way to make it easy to execute like this via unskipping something, but not built into the postcommit.

Our choices for this, as I understand them, are now:

Make two separate postcommits that run in parallel, hacking the pom in a way such that we can invoke them separately easily.
Run the executions simultaneously in a single postcommit (not sure if Maven can do this, or how to config it)
Bump the timeout to 3 or 4 hours. Probably a major pain for productivity.
Drop this PR and wait until pipelined jobs to revisit this. An unfortunate lack of coverage in the meantime.

(no plans to move on this until there is consensus)

@davorbonaci @jasonkuster I have now revised this again. To regain historical perspective:
Version 1: Two profiles (streaming/batch), two Jenkins jobs with different pipeline options to set --streaming=true. It was requested that there be a single profile with two executions.
Version 2: One profile, one Jenkins job, hence one set of pipeline options, two executions and a Java system property to force streaming mode since the pipeline options had to be the same.
Version 3 (current): One profile, two executions, each has the ability to be skipped, two Jenkins jobs that each select an execution.
I left it with one set of test pipeline options and the Java system property. We could probably go back to separate pipeline options and clean it up. LMK what you think.

Coverage increased (+0.02%) to 70.878% when pulling 52b14940bddf866030ea9c42162757978c49efe2 on kennknowles:Dataflow-streaming-ValidatesRunner into 90ba7536ffde38cc24a6570c81ef7d44f03ee7d3 on apache:master.

Coverage increased (+0.004%) to 69.538% when pulling b850ef6d72658021812d9a1fbfd11405a538b46b on kennknowles:Dataflow-streaming-ValidatesRunner into 8ff9c003f407a8db7941a071280e16c603732f92 on apache:master.

Closing this because we should add it via Gradle instead.
CC: @lukecwik


How to combine ComponentCreators when using 'styled' Components

We used styled-components before. Now we rewrite all our styled components using @material-ui/styles's styled function, which returns a ComponentCreator<Component>.
One of our styles gets reused in another style like this:
const TooltipText = styled.span`
   ...
`

const TooltipWrapper = styled.div`
  position: relative;
  &:hover ${TooltipText} {
    visibility: visible;
  }
`

Is there a way to do the same with @material-ui/styles's styled function?

It's a duplicate of #16113.
@adonig What's the motivation for moving away from styled-components? If I were you, it's only something I would consider doing to reduce bundle size, until we execute the styles engine adapter architecture.

@oliviertassinari We have a mix of components using styled-components and components using @material-ui/styles. This causes some circumstances with media queries, breakpoints, themes, etc. That's why we decided it might be worth to try to refactor the styled components to material-ui before our code base grows even larger.

This use case sounds valid. I agree that you should aim for minimizing the number of approaches you use to solve the same problems, or the same class of problems. It reduces entropy. I'm closing for the duplicate.


Include a link to the upgraded project in the PR

... in a perfect world to its release notes.

Good idea, @ennru. Do you have an idea how to obtain the URL of the website or the release notes from maven coordinates (groupId,artifactId, and a version) because currently that is the only thing the bot knows about a dependency. Any suggestions are welcome.

I think you'd have to use Scaladex as your information source, Scaladex knows this kind of thing

Do you have access to the pom.xml? If the SCM section points to Github, you could point to the Releases tab.

sbt-updates does not download the pom.xml but only the maven-metadata.xml which just contains the versions. But I could fetch the pom.xml with coursier via coursier fetch -A pom $groupId:$artifactId:$version. This should work for all public dependencies.

It would be great if scala-steward could include release notes in the description of the PR, similarly to how dependabot does it - see the screenshot below for a reference:

This data is fetched from Github releases, so we could use it for projects which use this feature and provide any release notes. This feature would be quite cool, I'd be happy to contribute this if that's something you would be keen on having in scala-steward, @fthomas.

That would be awesome, @mwz!


AWS Deployment Problem

Hello,
I have launched an ec2 instance on AWS.
I defined port according to AWS deployment guide., installed node.js and tried to run npm start command. But I got the below error.
Do you have any idea, why did I get this error.
/home/ubuntu/xxx/webpack/webpack.config.js:15
module.exports = (env = {}) => {
^
SyntaxError: Unexpected token =
at exports.runInThisContext (vm.js:53:16)
at Module._compile (module.js:374:25)
at Object.Module._extensions..js (module.js:417:10)
at Module.load (module.js:344:32)
at Function.Module._load (module.js:301:12)
at Module.require (module.js:354:17)
at require (internal/module.js:12:17)
at requireConfig (/home/ubuntu/xxx/node_modules/webpack/bin/convert-argv.js:96:18)
at /home/ubuntu/xxx/node_modules/webpack/bin/convert-argv.js:109:17
at Array.forEach (native)
FULL OUTPUT
$ ls
app           config    node_modules   package-lock.json  Roadmap.md
app.json      docs      nodemon.json   Procfile           server
Changelog.md  LICENSE   npm-debug.log  public_theme       webpack
common        __MACOSX  package.json   README.md
~/xxx$ npm build
~/xxx$ npm start
<EMAIL_ADDRESS>start /home/ubuntu/xxx
npm run build && cross-env NODE_ENV=production node compiled/server.js
<EMAIL_ADDRESS>build /home/ubuntu/xxx
npm run clean && cross-env NODE_ENV=production npm run webpack -- --env.browser && NODE_ENV=production npm run webpack -- --env.server
<EMAIL_ADDRESS>clean /home/ubuntu/xxx
rimraf public && rimraf compiled
<EMAIL_ADDRESS>webpack /home/ubuntu/xxx
webpack --colors --display-error-details --config ./webpack/webpack.config.js "--env.browser"
/home/ubuntu/xxx/webpack/webpack.config.js:15
module.exports = (env = {}) => {
^
SyntaxError: Unexpected token =
at exports.runInThisContext (vm.js:53:16)
at Module._compile (module.js:374:25)
at Object.Module._extensions..js (module.js:417:10)
at Module.load (module.js:344:32)
at Function.Module._load (module.js:301:12)
at Module.require (module.js:354:17)
at require (internal/module.js:12:17)
at requireConfig (/home/ubuntu/xxx/node_modules/webpack/bin/convert-argv.js:96:18)
at /home/ubuntu/xxx/node_modules/webpack/bin/convert-argv.js:109:17
at Array.forEach (native)
npm ERR! Linux 4.4.0-1022-aws
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "run" "webpack" "--" "--env.browser"
npm ERR! node v4.2.6
npm ERR! npm  v3.5.2
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>webpack: webpack --colors --display-error-details --config ./webpack/webpack.config.js "--env.browser"
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>webpack script 'webpack --colors --display-error-details --config ./webpack/webpack.config.js "--env.browser"'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the react-webpack-node package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     webpack --colors --display-error-details --config ./webpack/webpack.config.js "--env.browser"
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs react-webpack-node
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls react-webpack-node
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/ubuntu/xxx/npm-debug.log
npm ERR! Linux 4.4.0-1022-aws
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "run" "build"
npm ERR! node v4.2.6
npm ERR! npm  v3.5.2
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>build: npm run clean && cross-env NODE_ENV=production npm run webpack -- --env.browser && NODE_ENV=production npm run webpack -- --env.server
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script 'npm run clean && cross-env NODE_ENV=production npm run webpack -- --env.browser && NODE_ENV=production npm run webpack -- --env.server'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the react-webpack-node package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm run clean && cross-env NODE_ENV=production npm run webpack -- --env.browser && NODE_ENV=production npm run webpack -- --env.server
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs react-webpack-node
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls react-webpack-node
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/ubuntu/xxx/npm-debug.log
npm ERR! Linux 4.4.0-1022-aws
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "start"
npm ERR! node v4.2.6
npm ERR! npm  v3.5.2
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: npm run build && cross-env NODE_ENV=production node compiled/server.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'npm run build && cross-env NODE_ENV=production node compiled/server.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the react-webpack-node package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm run build && cross-env NODE_ENV=production node compiled/server.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs react-webpack-node
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls react-webpack-node
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/ubuntu/xxx/npm-debug.log
~/xxx$

I have solved the problem. I would like to share all commands here for those who will install it later.

Install nodejs
curl -sL https://deb.nodesource.com/setup_6.x | sudo -E bash -
Install nodejs
sudo apt-get install -y nodejs
Open the folder which includes the PEM file and run a terminal in this folder.
Run the below code to copy local file to remote folder.
scp -i pilaty.pem /Users/Archive.zip ubuntu@ec2-xx-xx-xxx-xx.eu-central-1.compute.amazonaws.com:/home/ubuntu/XXX/
Install UNZIP
sudo apt-get install unzip
Unzip file
unzip Archive.zip
Remove Archive.zip file
rm Archive.zip
If you open related port in the security group, now you can see your web site on the public link.
http://ec2-xx-xx-xxx-xx.eu-central-1.compute.amazonaws.com:3003/
Install NGINX
sudo apt-get install nginx
Delete default config file
sudo rm /etc/nginx/sites-enabled/default
Create a new one
sudo nano /etc/nginx/sites-available/node
Run the below code to activate
sudo ln -s /etc/nginx/sites-available/node /etc/nginx/sites-enabled/node
Install forever
sudo npm install forever -g
Start app to run forever
sudo forever start app.js
To see all forever task, run the below code
sudo forever list
To stop one of them, run the below code
sudo forever stop PID
Ngnix Restart
/etc/init.d/nginx restart

Ngnix My Config File
server {
listen 80;
server_name abc.com;
location / {
    proxy_set_header   X-Forwarded-For $remote_addr;
    proxy_set_header   Host $http_host;
    proxy_pass         "http://<IP_ADDRESS>:3003";
}

}
server {
listen 80;
server_name xyc.com;
location / {
    proxy_set_header   X-Forwarded-For $remote_addr;
    proxy_set_header   Host $http_host;
    proxy_pass         "http://<IP_ADDRESS>:3001";
}

}
server {
listen 80;
server_name www.xyc.com;
rewrite ^(.*) http://xyc.com$1 permanent;
}


Fix OpenAI Invalid value for "tool_choice"

This is to fix the following error, only pass tool_choice if tools are present
Langchain::LLM::ApiError: OpenAI API error: Invalid value for 'tool_choice': 'tool_choice' is only allowed when 'tools' are specified.

@mengqing How did you encounter this error?

If I run the Assistants example without specifying tools, eg
assistant = Langchain::Assistant.new(
  llm: llm,
  thread: thread,
  instructions: "You are a Meteorologist Assistant that is able to pull the weather for any location"
)
assistant.add_message_and_run content: "What's the weather in New York City?"

Then it returns the above error.
The culprit is here https://github.com/andreibondarev/langchainrb/blob/main/lib/langchain/assistants/assistant.rb#L131

@mengqing Do you think your fix is better than on that line doing:
tool_choice: tools.any? ? "auto" : "none"

The error still occurs even if setting tool_choice to none when tools is blank.
eg:
curl https://api.openai.com/v1/chat/completions \
  -H "Content-Type: application/json" \
  -H "Authorization: Bearer $OPENAI_API_KEY" \
  -d '{
    "model": "gpt-3.5-turbo",
    "messages": [
      {
        "role": "system",
        "content": "You are a helpful assistant."
      },
      {
        "role": "user",
        "content": "Hello!"
      }
    ],
    "tool_choice": "none"
  }'

@mengqing Then I'm thinking the proper thing to do would be to do this:
      params = {messages: thread.openai_messages}
      # TODO: Not sure that tool_choice should always be "auto"; Maybe we can let the user toggle it.
      params.merge!(tools: tools.map(&:to_openai_tool), tool_choice: "auto") if tools.any?

      llm.chat(**params)

What do you think?

@mengqing Then I'm thinking the proper thing to do would be to do this:
      params = {messages: thread.openai_messages}
      # TODO: Not sure that tool_choice should always be "auto"; Maybe we can let the user toggle it.
      params.merge!(tools: tools.map(&:to_openai_tool), tool_choice: "auto") if tools.any?

      llm.chat(**params)

What do you think?

I think because the issue comes from the underlying OpenAI's API, someone is still able to call the llm.chat with the following args and be faced with the same issue
llm.chat({..., tool_choice: "auto"})

However this isn't a big issue because the issue here can be avoided by validating the arguments passed to the method, where as it is not possible currently with the Assistant API.
So I think this method is also good

@mengqing I ended up fixing it slightly differently. I'll publish a new gem version in a few. Thank you for finding the bug!


Update for 0.12

Update nan and code for use with 0.12

@thomaschaaf are you interested in taking the ownership of node-ftdi?


Remove <:PState from Pattern

This will massively simplify the code, and allow to eliminate most of '.asInstanceOf' calls.

#59

merged to master


Upgrade Adafruit-DHT to 1.3.4

Description:
Related issue (if applicable): fixes #16321
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):
sensor:
  - platform: dht
    sensor: AM2302
    pin: 22
    monitored_conditions:
      - temperature
      - humidity

Checklist:

[X] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

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

[x] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

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

Starting home assistant after code change (sensor not connected, therfore not showing results):
2018-08-31 12:51:29 INFO (Thread-3) [homeassistant.util.package] Attempting install of Adafruit-DHT==1.3.4
2018-08-31 12:52:29 INFO (MainThread) [homeassistant.components.sensor] Setting up sensor.dht
2018-08-31 12:53:07 INFO (MainThread) [homeassistant.core] Bus:Handling <Event state_changed[L]: new_state=<state sensor.dht_sensor_humidity=unknown; friendly_name=DHT Sensor Humidity, unit_of_measurement=% @ 2018-08-31T12:53:07.663078+00:00>, old_state=None, entity_id=sensor.dht_sensor_humidity>

Hi @thomaslian,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

No discussion in PRs


nuget and msbuild errors when creating building the Dockerfile.

Expected Behavior

Build without any nuget or msbuild issue.
Actual Behavior

I get the following error
The 'Newtonsoft.Json 10.0.3' package requires NuGet client version '2.12' or above, but the current NuGet version is '2.8.60717.93'.
'QLNet' already has a dependency defined for 'System.Reflection.Emit.Lightweight'.
The 'System.Net.Http 4.3.3' package requires NuGet client version '2.12' or above, but the current NuGet version is '2.8.60717.93'.
The 'Microsoft.CSharp 4.6.0' package requires NuGet client version '2.12' or above, but the current NuGet version is '2.8.60717.93'.
The 'System.Reflection.Emit 4.3.0' package requires NuGet client version '2.12' or above, but the current NuGet version is '2.8.60717.93'.
The 'System.Reflection.Emit.Lightweight 4.3.0' package requires NuGet client version '2.12' or above, but the current NuGet version is '2.8.60717.93'.
The 'System.ValueTuple 4.4.0' package requires NuGet client version '2.12' or above, but the current NuGet version is '2.8.60717.93'.

After upgrading the nuget version, I get the following msbuild error
MSBUILD : error MSB1011: Specify which project or solution file to use because this folder contains more than one project or solution file.

Potential Solution

Downloading latest nuget and specif
Reproducing the Problem

Upgrade the Nuget version then specify the solutions file when calling msbuild.
System Information

Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues

[x] I have provided detailed steps to reproduce the issue

Which dockerfile -backed container did this error reproduce?
Running msbuild in latest foundation image reports
MSBuild auto-detection: using msbuild version '14.0' from '/usr/lib/mono/xbuild/14.0/bin'.

Apologies, I should have been more clear in my description. This error happens when yo un-comment Option 1 in the Dockerfile. I have provided a fix below but I am not sure why it's failing the travis-cl tests. I am not doing any modifications to the source code :(
https://github.com/QuantConnect/Lean/pull/3712/files


2012R2 labs and ISO spanish language

First of all, thanks for creating this amazing tool.
I have to test a migration from 2012R2 to 2016 so I need to build a 2012R2 lab. I see that it was listed in the ISO list but even in archive I don´t see any 2012R2 lab. Is there any?
Also I was wondering if I have to take into account anything in particular to create my own labs based on existing ones but using Spanish ISOs.
Thanks!

You could modify an existing configuration but it might be easier if I do it for you as I am in the midst of updating configurations. I could create a single 2012R2 server configuration. I'm not sure about the Spanish requirement. I'd have to see if that is a download option.

I added 2 Windows Server 2012 RS configurations in v4.9.0 which has been published to the PowerShell Gallery. Still looking into non-English images.

I run update-module psautolab and now I have two psautolab modules (version 4.8.0 and 4.9.0).
I don´t see those 2 Windows Server 2012R2 configurations.

After you update the module run Refresh-Host which will copy the updated configurations to your Autolab folder.

That worked. Thanks! Let me know if I can help about the Spanish ISO thing

I added 2 Windows Server 2012 RS configurations in v4.9.0 which has been published to the PowerShell Gallery. Still looking into non-English images.

Why does it ask me to have automatedlab module installed?
PS C:\WINDOWS\system32> install-module psautolab
ADVERTENCIA: Unable to resolve package source 'C:\Program Files\WindowsPowerShell\Modules\AutomatedLab\5.17.0\'.

I don't know what that is or why it would show up. That is very strange PowerShell behavior.
If you had previously installed this module you should be able to run Update-Module PSAutolab -force. The other option, assuming you never installed this module is to try this: Find-Module psautolab | Install-module -force.

Thanks! It seems a cosmetic issue. Anyway, as you can see in the lability github forum I managed to install the 2012R2-GUI lab in Spanish. Is there any lab for a DC, CA and Exchange for 2012R2 that you can upload or any guidance of tweaking any existing one for a newer Windows Server version?

There is nothing for 2012R2 other than the single server configurations I created. You will need to setup your own environment. You can manually create additional virtual machines using the ISO files to install from. Once you have everything setup, be sure to checkpoint everything. You might even want to export or backup all of the VM so you can restore them later. Just know that there is a time limit on the evaluation ISOs.

There is nothing for 2012R2 other than the single server configurations I created. You will need to setup your own environment. You can manually create additional virtual machines using the ISO files to install from. Once you have everything setup, be sure to checkpoint everything. You might even want to export or backup all of the VM so you can restore them later. Just know that there is a time limit on the evaluation ISOs.

Any guidance or help you can provide for creating my own labs? Maybe I could use as a base any of the existing ones? Or things change a lot from 2012R2 to 2016? I don´t see any lab that deploys a CA

The PowerShellLab includes an Active Directory CA. although the configuration probably needs a few more manual steps. Unless you have solid DSC experience, your best bet is to manually install and configure what you need then checkpoint the VMs.  Again, what you are trying to do and your use case is beyond the scope of this module.

Closing this because the original issue has been resolved.


Make output_path available for perturbations and metrics.

Purpose
The information retrieval metrics require access to a qrels file downloaded as part of the caller scenario (discussed in #473). This value is currently hard coded since there doesn't seem to be a way to get this path in run_specs.py. We need to make this path available to the metrics that require it.
Another alternative is for the information retrieval metric to retrieve the exact same qrels file, but this introduces quite a few more dependencies.

I added get_benchmark_output_path() in #2109, which returns the value set through the flag.
The remaining work would be find all instances where "benchmark_output" is hardcoded in metrics and perturbations, and replace them with calling this method.

Related: #1918

Working on this!

@AnshKhurana Are you still working on this?


Config using default terminal colors

I'm working on showing the running of a project where it is important that the colors displayed on the terminal are accurate to the default terminal color settings (ie. i dont want a fancy config color theme). Is there a way to specify not to use a theme and rather use the default terminal settings?

Unfortunately it is not possible. I suggest to find a way to export the colors scheme of the terminal and find an online website to convert these colors into hexa colors  format

Alright thanks!


Show pip info level logging with verbose

Closes #1434.
Changes:

-v now always allowed as shorthand for --verbose
added callback that sets pip.logger's log level to INFO if --verbose specified.

Open questions:

Does using callback with --verbose look like a reasonable way to do this? I like that it is simple and combines log setting/verbosity into one place (but I don't know if it's "click-ish")
Level of log output look okay? (success example +  see example from "bad" install at end).  Perhaps only pip.utils should get this log level applied with verbose? (other stuff is so noisy it's a bit of a bummer)
Should I attempt to colorize output with log handler?

Failing build dumps pip log info - yay!
Pipfile.lock not found, creating…
Locking [dev-packages] dependencies…
INFO:pip._vendor.requests.packages.urllib3.connectionpool:Starting new HTTPS connection (1): pypi.counsyl.com
INFO:pip.req.req_set:Collecting ipython==6.2.1
INFO:pip.download:File was already downloaded /Users/jtratner/Library/Caches/pipenv/pkgs/ipython-6.2.1.tar.gz
INFO:pip.utils:Complete output from command python setup.py egg_info:
INFO:pip.utils:
IPython 6.0+ does not support Python 2.6, 2.7, 3.0, 3.1, or 3.2.
When using Python 2.7, please install IPython 5.x LTS Long Term Support version.
Beginning with IPython 6.0, Python 3.3 and above is required.

See IPython `README.rst` file for more information:

    https://github.com/ipython/ipython/blob/master/README.rst

Python sys.version_info(major=2, minor=7, micro=14, releaselevel='final', serial=0) detected.

----------------------------------------

Builds failing because of #1437

this looks good to me!

Thanks for reviewing!
On Sat, Feb 17, 2018 at 11:01 AM Kenneth Reitz<EMAIL_ADDRESS>wrote:

Merged #1438 https://github.com/pypa/pipenv/pull/1438.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pypa/pipenv/pull/1438#event-1479240482, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ABhjq9OXUWEkIML285eYyv0q1i78FiiMks5tVyIagaJpZM4SJXK4
.


Integration with flock messenger

Hi,
This PR adds Flock messenger support.

@aananthraj LGTM. Thanks for contributing!

@tylerauerbeck thanks. 👍


NotRegistered error but the message is successfully sent

{
  "multicast_id":<PHONE_NUMBER>051704056,
  "success": 0,
  "failure": 1,
  "canonical_ids": 0,
  "results": [
    {
      "error": "NotRegistered"
    }
  ]
}

That just means the fcmId you provided was not registered to your target device.
Have you implemented the client side FCM?
Check this out https://firebase.google.com/docs/cloud-messaging/android/client

The doc says

If it is NotRegistered, you should remove the registration ID from your server database because the application was uninstalled from the device, or the client app isn't configured to receive messages.

But the app wasn't uninstalled or anything, and actually freshly generated the token :/

I've had an issue where when I send two notifications within a few seconds of each other to different token IDs they share the same response.
I have tested with a token I know is correct as well as a fake one e.g. "123". The device receives the notification but both responses from FCM say "InvalidRegistration".
If I had in a 5 second timeout I get two different responses (one fail, one success) from FCM.

weird, did you let the firebase team know?

Yes I have. More info about the error here with a script example: https://github.com/Saber-Tecnologias/fcm-node/issues/3

Fixed on my last PR, you could check

i ma getting same error, i minuts ago notification was working perfact
but now canonical_ids":0,"results":[{"error":"NotRegistered"}]} shwing this type of error

i too have the sa,me issue


gapi.client.init never never returns

If I leave the tab before the request is made or I am never on the tab when the request is made then this happens.
Upon calling gapi.client.init it never returns success or error. Chrome will continue to say "Processing request...".
If I stay on the page for the entirety of the call, it works fine.
The last network request is:
Request URL:ws://localhost:4200/sockjs-node/981/21nsl0uu/websocket
Request Method:GET
Status Code:101 Switching Protocols

and the last console log is:
Navigated to https://accounts.google.com/o/oauth2/postmessageRelay?parent=http%3A%2F%2Flocalhost%3A4200&jsh=m%3B%2F_%2Fscs%2Fapps-static%2F_%2Fjs%2Fk%3Doz.gapi.en.6_Oh42KNmBQ.O%2Fm%3D__features__%2Fam%3DAQE%2Frt%3Dj%2Fd%3D1%2Frs%3DAGLTcCOLDThr61T0lUfV81vOOCnO7rxu2w

@theblindprophet could you provide a snippet of code for the config that you pass to gapi.client.init?

This is in an Angular 5 project (Typescript):
declare let gapi: any;
...
constructor(private as: ApiService) {
    this.signedInObserve = new Subject();

    this.handleClientLoad();
}

/**
 * Load gapi
 */
public handleClientLoad(): void {
    // Load the API's client and auth2 modules.
    // Call the initClient function after the modules load.
    gapi.load('client:auth2', () => {
        this.init();
    });
}

/**
 * Init gapi
 */
public init(): void {
    let scopes = [
        'https://www.googleapis.com/auth/userinfo.email'
    ];
    gapi.client.init({
        'apiKey': '<apiKey>',
        'clientId': '<clientId>',
        'scope': scopes.join(' ')
    }).then(() => {
        console.log("Init complete");
        this.loaded = true;
        this.initClient();
    }).catch(err => {
        console.log(err);
    });
}

@theblindprophet thanks for the further information. When you come back to the tab, does it finish initializing? If not, could you look at HTTP requests in the Network tab of the Developer Console for any requests to "google" endpoints. Is there any 404, pending requests or other errors? If so, please mention the URLs here.

Right now it does finish... when I had my double monitor setup and would hide the entire window with Chrome it would hang up.
For example now it hangs up only until I return:

I am almost certain when it would hang up the last request would be:
https://accounts.google.com/o/oauth2/postmessageRelay...

It's OK to have this as the last request. Is there any failed request?

I am having this same issue.  Get a 404 on:
https://googleapis.com/static/proxy.html?usegapi=1&jsh=m%3B%2F_%2Fscs%2Fapps-static%2F_%2Fjs%2Fk%3Doz.gapi.en.iUW0BsJjLUw.O%2Fam%3DQQ%2Frt%3Dj%2Fd%3D1%2Frs%3DAGLTcCNo55M6gTQ-sZjYPaMWWtlsJ4Y0tw%2Fm%3D__features__
when calling:
const discoveryDocs = [
'https://people.googleapis.com/$discovery/rest?version=v1'
];
gapi.client.load('oauth2', 'v2', initClient);
function initClient() {
console.log('oauth2 loaded, initClient \nApiKey', apiKey, '\nDiscoverDocs',
discoveryDocs, '\nclientId:', clientId, '\nscopes:', scopes);
gapi.client.init({
'apiKey': apiKey,
'discoveryDocs': discoveryDocs,
'clientId': clientId,
'scope': 'profile'
})
.then(() => {
console.log('gapi.client.init complete');
// Listen for sign-in state changes.
gapi.auth2.getAuthInstance().isSignedIn.listen(updateSigninStatus);
// Handle the initial sign-in state.
updateSigninStatus(gapi.auth2.getAuthInstance().isSignedIn.get());
//       authorizeButton.onclick = this.handleAuthClick;
//       signoutButton.onclick = this.handleSignoutClick;
},
() => {
console.log('gapi.client.init failed');
});


Use of rxjs throwError fails test

I'm using jest 22.4.4
I'm trying to get coverage for an rxjs catchError code path.
My expect passed, but then the test fails with the error that was returned by the mock.
import { BehaviorSubject } from 'rxjs/internal/BehaviorSubject';

import { async, TestBed, ComponentFixture } from '@angular/core/testing';
import { NO_ERRORS_SCHEMA } from '@angular/core';

import { ReactiveFormsModule, FormArray } from '@angular/forms';
import { MatAutocompleteModule } from '../../../node_modules/@angular/material';
import { AuthComponent } from './auth.component';
import { AuthService } from '../services/auth.service';
import { mock, instance, when } from '../../../node_modules/ts-mockito'
import { Observable, throwError } from '../../../node_modules/rxjs';

describe('AuthSpecComponent', () => {
    let fixture: ComponentFixture<AuthComponent>;
    let mockService = mock(AuthService);
    let component: AuthComponent;

    // const bar:Bar = mock(Bar);
    // const tested:TestedClass = new TestedClass(instance(bar));

    beforeEach(async(() => {
        TestBed.configureTestingModule({
            imports: [
                ReactiveFormsModule,
                MatAutocompleteModule,
            ],
            declarations: [
                AuthComponent,
            ],
            providers: [
                { provide: AuthService, useValue: instance(mockService) },
            ],
            schemas: [NO_ERRORS_SCHEMA]
        }).compileComponents();
        fixture = TestBed.createComponent(AuthComponent);
        component = fixture.debugElement.componentInstance;
        component.ngOnInit();
    }))

    afterEach(async(() => {
        component.ngOnDestroy();
    }))

    it("GIVEN 400 WHEN feedback THEN Failure", async(() => {
        let value = { 'status': 400 }
        let obs = new BehaviorSubject<any>(value);
        when(mockService.post('foo', 'bar', 'baz')).thenReturn(throwError(value));
        component.authForm.patchValue({
            tenantId: 'foo',
            clientId: 'bar',
            secret: 'baz'
        })
        component.onSubmit();
        expect(component.feedback).toBe(`{\n  "status": 400\n}`);
    }));
});

The test fails and the reason is the Error from throwError.

So the issue was the onSubmit contained a pipe that also called throwError. I worked around it by returning an Observable instead of returning throwError inside catchError.


Unable to read Enum values from dependencies

Spring native is not adding Enum getter setter methods to reflect-config.json when referred from a dependency project unless you specify a hint in the main project.
Attaching a zip to replicate the issue. In this example my demo project gets the model from the dependency project and when i try to save a User object which has Enum Model, i am getting the below exception
Resolved [org.springframework.http.converter.HttpMessageNotReadableException: JSON parse error: Cannot deserialize value of type com.example.dependency.model.EmployeeDetails$ModelEnum from String "wfh": not one of the values accepted for Enum class: [WFH, OFFICE]; nested exception is com.fasterxml.jackson.databind.exc.InvalidFormatException: Cannot deserialize value of type com.example.dependency.model.EmployeeDetails$ModelEnum from String "wfh": not one of the values accepted for Enum class: [WFH, OFFICE] at [Source: (PushbackInputStream); line: 3, column: 13] (through reference chain: com.example.demo.model.User["model"])]
Example request: http://localhost:8080/addUser
{
"id" : "3",
"model":"workFromHome"
}
My enum
public static enum ModelEnum {
WFH("wfh"),
OFFICE("office");
private String value;

private ModelEnum(String value) {
  this.value = value;
}

@JsonValue
public String getValue() {
  return this.value;
}

public String toString() {
  return String.valueOf(this.value);
}

@JsonCreator
public static EmployeeDetails.ModelEnum fromValue(String value) {
  EmployeeDetails.ModelEnum[] var1 = values();
  int var2 = var1.length;

  for(int var3 = 0; var3 < var2; ++var3) {
    EmployeeDetails.ModelEnum b = var1[var3];
    if (b.value.equals(value)) {
      return b;
    }
  }

  throw new IllegalArgumentException("Unexpected value '" + value + "'");
}

Demo :
IdeaProjects.zip
Note: Project works as expected when i run as a regular Spring boot application

Hi, Could anyone please look into this.

While we can add reflection hints automatically by looking at controller handlers and other Spring API, this is way more involved: a Controller handler returns a List<User>, with User being an entity that extends EmployeeDetails, and EmployeeDetails (itself defined in a dependency) having an enum attribute which is itself annotated with a @JsonCreator method.
That's quite removed from the original intent, and using @TypeHint here would make perfect sense. I'll close this issue in favor of #1152, where we might figure out new ways to address this.

Thanks @bclozel


New fasta file parser

A new fasta file parser should be created that does not cut input sequence names as SeqIO from Biopython does. SeqIO should be maintained and used just to validate input file format.

Solved


[STORM-1344] Replace stormbot_say with hubot.post_message

using the hubot pack instead of stormbot_say

LGTM, +1

This is a simple enough change, but here is a concern of mine.
What if I don't have Hubot? What if my ChatOps adapter is something different? I've been trying to think about an adapter of convention for things like this that don't lock us to something specific. I think this is going to come up often.

What if I don't have Hubot?

The odd part here is that we are trying to make rules generally available. Rules are often infrastructure/install base specific so this rule should probably not even exist and shared publicly with an expectation that it would actually work.
To answer you original question I think an adapter approach is required. We could expect that users would setup chat.post_message action and back it with their preferred impl. Similarly, we could provide other canonical actions that can come with implementations that are infra specific e.g. server.create, ticket.create or page.acknowledge with each of them being backed by a specific action.
Anyway, for now I think I will merge this change as stormbot_say has been removed and to maintain this rule we should use the hubot pack.


Decide format of emails to send to committees and legislators.

This is the placeholder for now.

this email is generated from this modal

I was thinking this:

Many congresspeople will probably just briefly look at the subject line or skim, so I think it's important to get the support/oppose/info into the subject line.

Sounds good.  @mvictor55 thoughts?

That's an interesting point. If there were a couple hundred emails, and they didn't indicate in the subject line, would the staff read through them? Read through some? I'm not sure how they'd react.

Would a user send an email to their Senator about a House bill, or to their representative about a Senate bill?

@jkoren yes, it is typical to CC both your Senator and Rep.

mock up


New SIP and SRC process

Good job. Please review my suggestions and replace really all EIPs by SIPs

done


[Tour of Beam] [Task] Update Playground CI/CD for ToB learning materials

What needs to happen?
Update Playground CI/CD for ToB learning materials
add following options to ci/cd commands:
--directories for designating folders with content
-- origin for designating learning material origin
Issue Priority
Priority: 3

.take-issue


Woudn't be nice if this lib is wrapped as a shell tool, and can be installed with i.e. brew ? (Like gum)

Woudn't be nice if this lib is wrapped as a shell tool, and can be installed with i.e. brew ? (Like gum)

Hey @dzonekl, that's a great idea! We (@aymanbagabas) just added log to gum and it will be available in the next release.
https://github.com/charmbracelet/gum#log
If you want to try it out now, you can install gum on the latest main:
go install github.com/charmbracelet/gum@main

Great ! thanks guys !


[WIP] First implementation of the ownCloud XMPP Backend

I temporary forked the jsxc/jsxc.owncloud repo to keep track of my changes. This PR and fork will be removed as soon as the repo is moved to the ownCloud repo.

[x] TODO add some abstraction to stanza handlers
[x] TODO use DB mappers which implement te serializable interface
[ ]  TODO move checking for two polling request to SESSION
[x] TOOD fix Strophe handling messages twice when it's only received ones
[ ]  Presence
[x] Remove use of \OCP\User
[ ] check behaviour with multiple trusted_domains
[ ] add checkbox in admin section to use internal chat

cc @sualko @jancborchardt (-plus)

@sualko are you okay with adding the sabre/xmp dependency? This makes the XML handling more easy, as you may know at the conf I spend a lot of time parsing the XML via simplemxl and domdocument....

Sabre/xml is totally fine. The simplexml seams to be very buggy.
I will add you know to the repositories, because I wait until deepdiver is back from holidays.
I also will publish a bug fix release maybe on Monday. This is only to inform you.

@sualko can you make it possible to open issues here btw? Or should I open them in another repository?

Please open them in jsxc/jsxc. Maybe we should discuss this practice, but in my current point of view this is the easiest way to collect all issues for jsxc at one place.

@sualko can you have a look at making a connection to our backend? I always get erros like jid is null or something similar.

I will check it.

Great! The best I have is:
/**
 * This function is a hack. Please FIXME!
 */
jsxc.options.favicon.enable = false;
jsxc.options.otr.enable = false;
Strophe.getNodeFromJid = function () {
	return OC.currentUser;
};
Strophe.getDomainFromJid = function () {
	return '33.33';
};
function ocInit(){
	localStorage.clear();
	jsxc.storage.setItem('debug', true);
	jsxc.options.set('xmpp', {'url': '/index.php/apps/ojsxc/http-bind'});
	jsxc.xmpp.login(OC.currentUser +<EMAIL_ADDRESS>3434, 3443243); // TODO remove the hardcoded values
	jsxc.xmpp.connected();
	jsxc.storage.setItem('debug', true);
}

$(document).ready(function () {
	ocInit();
});

But now the Strophe isn't polling...

Actually it is long-polling but it doesn’t handle the messages.

Normally it should be enough, if you execute this commands once and than reload.
jsxc.options.set('otr', {enable: false});
jsxc.options.set('xmpp', {'url': '/index.php/apps/ojsxc/http-bind'});
jsxc.storage.setItem('debug', true);
jsxc.storage.setItem('sid', '7862');
jsxc.storage.setItem('rid', '897878733');

Thanks, but I can't get it to work. There are simply no requests send to '/http-bind', and nothing is logged in the browser console.

Sorry, try this one:
localStorage.clear()
jsxc.options.set('otr', {enable: false});
jsxc.options.set('xmpp', {'url': '/index.php/apps/ojsxc/http-bind'});
jsxc.storage.setItem('debug', true);
jsxc.storage.setItem('sid', '7862');
jsxc.storage.setItem('rid', '897878733');
jsxc.storage.setItem('lastActivity', (new Date()).getTime());
jsxc.storage.setItem('jid', OC.currentUser + '@33.3');

No problem! I'm very happy that you are willing to help :smiley:
The client new successfully connects to the backend! However chat messages aren't received. This is a correct response for sending a new message, right?
<message<EMAIL_ADDRESS><EMAIL_ADDRESS>type="chat">af</message>

Mmm strophe.js/jquery logs the received stanza as an empty body:

Chrome console displays the XML stanza however....

I think it has something to do with the presence. Maybe StropheJS is thinking he shouldn't receive the message because he isn't online?

Incoming messages looks like this:

  <message xmlns="jabber:client" from="georg@localhost/11395954601441475563228758" to="klaus@localhost/10705725741441475541495758" type="chat" id="1441475584282-msg">
    Hello
  </message>

An empty response from the server (on the end of a long poll) looks like this:

Maybe StropheJS is thinking he shouldn't receive the message because he isn't online?

This should be no problem for strophe. I think you only missed the namespace. I will test your implementation tomorrow.

Yes! It's working! Thanks @sualko

Congratulations. Great work.

:fireworks:

How are you doing?
As you can see, we moved to the owncloud orga. :+1:

@LEDfan @sualko @DeepDiver1975 Nice! So should we rename this to »chat« and move the current app to »chat-legacy« or something like that?

i don't think we necessarily have to rename the repo as long as the app has a nice user facing name.

For me we could rename both. It would be nice if there would occur jsxc in it, but this is not necessary.
Suggestions?

Let's keep the repo name of the other chat app as it is. No need to rename stuff around. This repository can be renamed. Something like 'jsxc.chat' maybe? Or only 'jsxc'? The 'owncloud' is redundant here

I prefer "jsxc.chat", because otherwise the risk of confusion to the core repo is to high in my point of view.

O.K. renamed!

Very nice :+1: Like I said at the conf,my academic year has just started, so I afraid I won't have much time to work on this....  I hope I still can make it happen though

@LEDfan can you move the branch to this repo, so maybe others can continue your work.

@sualko yes of course. I'll do this now.


Easy to follow article.

Exactly what I was looking for.  Thanks!

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 97cbbb3e-3916-4340-4ddc-67bf22159bf3
Version Independent ID: 5b620628-1c10-b0bb-bd4d-1fd229260ea1
Content: Using report tooltip pages in Power BI - Power BI
Content Source: powerbi-docs/desktop-tooltips.md
Service: powerbi
Sub-service: powerbi-service
GitHub Login: @davidiseminger
Microsoft Alias: davidi

@cal172  - thank you! It's great to hear it's what you were looking for, and thanks for taking the time to share that you liked it. Much appreciated!


propagate the config for logging to batch jobs

propagate the config for logging to batch jobs

loggers cannot be pickled in python 2.7 or 3.6
loggers can be pickled in python 3.7

working on multiprocessing first

using the threading version in the example at https://docs.python.org/3/library/logging.handlers.html#queuehandler

tried the multiprocessing version as well. decided not to use for now because handlers need to be configured in the other process. copy an example code here for the record.

class LoggingListener(multiprocessing.Process):
    def __init__(self, queue):
        multiprocessing.Process.__init__(self)
        self.queue = queue

    def run(self):
        while True:
            record = self.queue.get()
            if record is None:
                break
            logger = logging.getLogger(record.name)
            logger.handle(record)

the logger in the main process won't received the record.

idea for the future

pass record to the progress bar so that the progress bar can control both progress bars and logging messages on stdout.

useful links

http://plumberjack.blogspot.co.uk/2010/10/logutils-using-recent-logging-features.html
http://plumberjack.blogspot.co.uk/2010/09/using-logging-with-multiprocessing.html
https://docs.python.org/3/library/logging.handlers.html#queuehandler


window manager exits without errors in ubuntu 18.04

Hi, I set up everything and wasn't able to start sessions via mstsc.
Here is my log.
[20181129-11:29:40] [INFO ] A connection received from ::1 port 49560
[20181129-11:29:40] [INFO ] ++ created session (access granted): username lin, ip ::ffff:<IP_ADDRESS>:58704 - socket: 12
[20181129-11:29:40] [INFO ] starting Xorg session...
[20181129-11:29:40] [DEBUG] Closed socket 9 (AF_INET6 :: port 5910)
[20181129-11:29:40] [DEBUG] Closed socket 9 (AF_INET6 :: port 6010)
[20181129-11:29:40] [DEBUG] Closed socket 9 (AF_INET6 :: port 6210)
[20181129-11:29:40] [DEBUG] Closed socket 8 (AF_INET6 ::1 port 3350)
[20181129-11:29:40] [INFO ] calling auth_start_session from pid 39307
[20181129-11:29:40] [DEBUG] Closed socket 7 (AF_INET6 ::1 port 3350)
[20181129-11:29:40] [DEBUG] Closed socket 8 (AF_INET6 ::1 port 3350)
[20181129-11:29:40] [INFO ] /usr/lib/xorg/Xorg :10 -auth .Xauthority -config xrdp/xorg.conf -noreset -nolisten tcp -logfile .xorgxrdp.%s.log
[20181129-11:29:41] [CORE ] waiting for window manager (pid 39308) to exit
[20181129-11:29:41] [CORE ] window manager (pid 39308) did exit, cleaning up session
[20181129-11:29:41] [INFO ] calling auth_stop_session and auth_end from pid 39307
[20181129-11:29:41] [DEBUG] cleanup_sockets:
[20181129-11:29:41] [DEBUG] cleanup_sockets: deleting /var/run/xrdp/sockdir/xrdp_chansrv_socket_10
[20181129-11:29:41] [DEBUG] cleanup_sockets: deleting /var/run/xrdp/sockdir/xrdpapi_10
[20181129-11:29:41] [DEBUG] cleanup_sockets: failed to delete /var/run/xrdp/sockdir/xrdpapi_10
[20181129-11:29:41] [INFO ] ++ terminated session:  username lin, display :10.0, session_pid 39307, ip ::ffff:<IP_ADDRESS>:58704 - socket: 12

The connection log from mstsc window shows
connection problem, giving up
some problem

I hardly get any useful infromation from the log... Anyone can help? Thank you!

Same here... things i tried (without success - none of these solutions worked):
# install xrdp and related packages
sudo apt install -y xrdp gnome-tweak-tool

# replace allowed users in Xwraper.config
sudo sed -i 's/allowed_users=console/allowed_users=anybody/' /etc/X11/Xwrapper.config

# add policies
sudo bash -c "cat >/etc/polkit-1/localauthority/50-local.d/45-allow.colord.pkla" <<EOF
[Allow Colord all Users]
Identity=unix-user:*
Action=org.freedesktop.color-manager.create-device;org.freedesktop.color-manager.create-profile;org.freedesktop.color-manager.delete-device;org.freedesktop.color-manager.delete-profile;org.freedesktop.color-manager.modify-device;org.freedesktop.color-manager.modify-profile
ResultAny=no
ResultInactive=no
ResultActive=yes
EOF

# enable extensions
gnome-shell-extension-tool -e<EMAIL_ADDRESS>gnome-shell-extension-tool -e<EMAIL_ADDRESS>

# try some other things
sudo apt install -y vino
gsettings set org.gnome.Vino require-encryption false

# increase MaxSessions
sudo vi /etc/xrdp/sesman.ini
# set MaxSessions=500

# reinstall libglib (see https://dynamic.reauktion.de/flugphase/2018/04/24/ubuntu-xrdp-server-black-screen-on-login/)

links:

https://c-nergy.be/blog/?p=11868
https://www.lifewire.com/setup-ubuntu-remote-desktop-4129666
https://sourceforge.net/p/xrdp/discussion/389417/thread/1af7d057/?limit=25
https://askubuntu.com/questions/1031519/xrdp-on-ubuntu-18-04lts
https://dynamic.reauktion.de/flugphase/2018/04/24/ubuntu-xrdp-server-black-screen-on-login/

In my case it now works, after i installed xorgxrdp manually (there were problems with dependencies xorg-video-abi-23 and xserver-xorg-core).
After that, i used http://c-nergy.be/blog/?p=12043 to get rid of color profile warnings.
sudo apt install -y xrdp xorgxrdp xorg-video-abi-23 xserver-xorg-core
cat <<EOF > /tmp/45-allow-colord.pkla
[Allow Colord all Users]
Identity=unix-user:*
Action=org.freedesktop.color-manager.create-device;org.freedesktop.color-manager.create-profile;org.freedesktop.color-manager.delete-device;org.freedesktop.color-manager.delete-profile;org.freedesktop.color-manager.modify-device;org.freedesktop.color-manager.modify-profile
ResultAny=no
ResultInactive=no
ResultActive=yes
EOF
sudo mv /tmp/45-allow-colord.pkla /etc/polkit-1/localauthority/50-local.d/
sudo rm /var/crash/*
reboot

Now its working for me.

In my case it now works, after i installed xorgxrdp manually (there were problems with dependencies xorg-video-abi-23 and xserver-xorg-core).
After that, i used http://c-nergy.be/blog/?p=12043 to get rid of color profile warnings.
sudo apt install -y xrdp xorgxrdp xorg-video-abi-23 xserver-xorg-core
cat <<EOF > /tmp/45-allow-colord.pkla
[Allow Colord all Users]
Identity=unix-user:*
Action=org.freedesktop.color-manager.create-device;org.freedesktop.color-manager.create-profile;org.freedesktop.color-manager.delete-device;org.freedesktop.color-manager.delete-profile;org.freedesktop.color-manager.modify-device;org.freedesktop.color-manager.modify-profile
ResultAny=no
ResultInactive=no
ResultActive=yes
EOF
sudo mv /tmp/45-allow-colord.pkla /etc/polkit-1/localauthority/50-local.d/
sudo rm /var/crash/*
reboot

Now its working for me.
edit: Since the dependencies of xorgxrdp contain xserver-xorg-core, but in my case xserver-xorg-core-hwe is installed, that seems to be the problem with the dependencies. So to get the more recent release of xrdp and xorgxrdp you could also use the files from Ubuntu 19.04, which worked nicely in my case:
cd /tmp
wget http://archive.ubuntu.com/ubuntu/pool/universe/x/xrdp/xrdp_0.9.9-1_amd64.deb
sudo dpkg -i xrdp_0.9.9-1_amd64.deb
wget http://archive.ubuntu.com/ubuntu/pool/universe/x/xorgxrdp/xorgxrdp_0.2.9-1_amd64.deb
sudo dpkg -i --ignore-depends=xserver-xorg-core xorgxrdp_0.2.9-1_amd64.deb

# get rid of the color profile messages
cat <<EOF > /tmp/45-allow-colord.pkla
[Allow Colord all Users]
Identity=unix-user:*
Action=org.freedesktop.color-manager.create-device;org.freedesktop.color-manager.create-profile;org.freedesktop.color-manager.delete-device;org.freedesktop.color-manager.delete-profile;org.freedesktop.color-manager.modify-device;org.freedesktop.color-manager.modify-profile
ResultAny=no
ResultInactive=no
ResultActive=yes
EOF
sudo mv /tmp/45-allow-colord.pkla /etc/polkit-1/localauthority/50-local.d/
sudo rm /var/crash/*
reboot

This solution fixed my problem too. Do you know the detailed meaning of the configuration file or where can I find such info. Thanks.

wbuser@ubuntu:~$ sudo apt install -y xrdp xorgxrdp xorg-video-abi-23 xserver-xorg-core
Package xorg-video-abi-23 is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source

E: Package 'xorg-video-abi-23' has no installation candidate

This package is not available now
(I am on Ubuntu 20.04)


Configure the keepList in the API

Problem
From the user perspective, a user would want to configure how many and which versions they want their helpers to maintain. The KeepList is present in protobufs, but there is no way to configure it in the API. There is a statement in protocol.md that states that the sharer must delete old versions stored with their helpers using the KeepList feature, so maybe this is not needed.
Solution
Add a method in DeRecSecret called updateKeepList()

This issue is not a necessary change.


Visual Bug

Having a visual bug with Storyteller, where the opening page will only partly be visible. Bug disappears briefly after selecting one of the pages or hovering over the tab in the top right hand corner which begins the turning animation.

Foundry Version
V10 Stable Version build 290
Module Version
Version 1.0.3

It looks like you're missing a header from the magazines. Maybe another mod removes it?
I couldn't reproduce the problem :(


Document all remaining undocumented symbols added since 1.1.1

All remaining symbols from #13192 are documented.
Fixes #13192.

All nits fixed in the latest fixup. Please take another look.

Fixups pushed addressing feedback from @paulidale.
@t8m - Please can you reconfirm?

Maybe close and open this PR again.. The fix has been merged.

If it was all green before the NIT changes should be ok.

New commit addressing latest feedback. Please reconfirm again everyone!

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Pushed. Thanks!


Fix issues around incomparable structs

With thanks to @amomchilov for the suggestions on addressing the correctness of the <=> operator: https://github.com/bellroy/sorbet-struct-comparable/pull/3
This PR adds additional tests to help cover <, > and the matter of incomparable structs or incomparable attributes of comparable structs and then with @amomchilov's suggestions gets these tests passing.

@amomchilov Yeah I had a look around and this pattern seemed to be the common way people are doing testing against proper implementation of Comparable through directly poking at <=>.
On re-reading https://ruby-doc.org/core-3.0.1/Object.html#method-i-3C-3D-3E and https://docs.ruby-lang.org/en/3.0.0/Comparable.html its obvious it was doing the incomparable bit incorrectly.


Refactoring sometimes replaces data in the wrong file

Well, it is actually not even working right now... :-/


[Hotfix][Connector-V2][File] Fix ParquetReadStrategy get NPE

Purpose of this pull request

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

@TyrantLucifer please help to review


Update README.md

Added a school on privacy-preserving ML in Warsaw, Poland

Thanks for updating the file. One correction: the fee is approximately 40
euro (not 400). Could you please change it?
Best,
Stefan
On Thu, Jun 1, 2023 at 12:08 PM Gabriele Berton @.***>
wrote:

Closed #154 https://github.com/sshkhr/awesome-mlss/pull/154.
—
Reply to this email directly, view it on GitHub
https://github.com/sshkhr/awesome-mlss/pull/154#event-9401164188, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AE7EGBWNEZDUXHDVTWLUTQLXJBSXHANCNFSM6AAAAAAYVZLSPM
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Stefan Dziembowski
Institute of Informatics, University of Warsaw
www.crypto.edu.pl/Dziembowski


Improve wording in documentation

Remove redundant wording form documentation.

The CI failures seem unrelated (eg: "pip: command not found").

Thanks! I will merge this even if CI is broken.
This repo is more or less deprecated now. Dev is continuing in Tower repo. I would <3 a PR there too https://github.com/tower-rs/tower/blob/master/tower-service/src/lib.rs#L89


Update prisma peer dependency version to 5.0.0

Need to test what was changed and what we need to test. Anyone open to start a PR?

Maybe it helps, but I started getting these errors when running prisma generate after upgrading to v5.0.0:
✘ Type OfferWhereInput.shareClasses is not supported.
✘ Type OfferWhereInput.shareholderAgreementFiles is not supported.
✘ Type OfferScalarWhereWithAggregatesInput.shareClasses is not supported.

All of them are related to the annotation of the Json columns. For example given the following model:
model Offer {
  id                        String                    @id @default(cuid())
  investorId                String
  investor                  Investor                  @relation(fields: [investorId], references: [id], onDelete: Cascade)
  companyId                 String
  company                   Company                   @relation(fields: [companyId], references: [id], onDelete: Cascade)
  /// [ShareClasses]
  shareClasses              Json?

  @@map("offers")
}

If I remove the /// [ShareClasses], the first line of the error disappears. However, the types seem to be working.

very urgency.

+1 ✌

We're also running into this with our Json annotations.

Same here. Would love a pointer and I could work on a PR.

Just to +1, seems like we're hitting the same issue when running prisma generate as a post install step during deployment

Even though there were some warnings in the logs when using prisma-json-types-generator with Prisma 5.0.0, it still worked fine and all types we used were modified properly. However, after upgrading to Prisma 5.1.0, our custom types are no longer being applied. See https://github.com/arthurfiorette/prisma-json-types-generator/issues/122

I took a look at the failing tests and created a PR that makes them pass: https://github.com/arthurfiorette/prisma-json-types-generator/pull/128 (see with already run tests at https://github.com/prisma/prisma-json-types-generator/pull/2). See the PR text for more information.

Hey folks, sorry for the update delay, I'll be working on this later today/tomorrow. I'll close this issue in favor of #122.


Update README.md

Summary of your changes
Description
Add article 'DynamoDB — Everything You Need To Know About Single Table Design'
Checklist

[x] My change follows the Contributing Guidelines
[x] I have added only one new link to the list.
[x] I have checked that the link that I added does NOT exist in the project already.
[x] I have sorted the link alphabetically under the related section.

Happy Hacktoberfest @sdmg15 :)


442 solution

Fixes #350

My code sample is in :

[ ] CPP
[x] Java

Screenshot of working code sample  :

Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My code sample is formatted and works correctly

LGTM


Server crash after recently creating custom portal; projectile

I'll get straight to the point - server crash because of NullPointerException
java.lang.NullPointerException: Cannot invoke "net.minecraft.class_1297.method_36456(float)" because "entity" is null
	at net.kyrptonaught.customportalapi.util.CustomTeleporter.TPToDim(CustomTeleporter.java:46)
	at net.kyrptonaught.customportalapi.CustomPortalBlock.onEntityCollision(CustomPortalBlock.java:116)
	at net.minecraft.block.AbstractBlock$AbstractBlockState.onEntityCollision(AbstractBlock:926)
	at net.minecraft.entity.Entity.checkBlockCollision(Entity:1035)
	at net.minecraft.entity.projectile.PersistentProjectileEntity.tick(PersistentProjectileEntity:280)
	at mod.azure.doom.entity.projectiles.ChaingunBulletEntity.tick(ChaingunBulletEntity.java:126)
	at net.minecraft.server.world.ServerWorld.tickEntity(ServerWorld:655)
[at ...]

I don't know if this was a problem with the portal generation making a mess or the projectile entering a portal weirdly but a crash is a crash that could be avoided - hopefully accounting for null parameters.
Modpack used: Another Quality Modpack 2 v2.7.0c
Mod used: Custom Portals by palyon_1 v2.1.0
Custom Portal API version 0.0.1-beta41-1.17
Full log and crash report (scroll down to the end for the latter): https://gist.github.com/ThatScar/10f77bcf5c299ec6acb698e08287b1c5
Full description of what happened: A player entered a custom portal to the nether. I don't know if the portal has been entered before by other players but I am fairly sure this act of entrance created a new portal on the nether side. The server immediately crashed when a player teleported using it. Upon reloading the server, it crashed again when the player connected. Later, a different player ThatScar (me) connected and there was no crash; portal was destroyed by hand and the issue stopped happening.
A screenshot of the portal in question: https://imgur.com/4soprGQ
Hope this helps somebody,
ThatScar

Thank you for the report, I will investigate the crash shortly.
Also worth mentioning that CustomPortalapi is not affiliated with the CustomPortals mod you linked. They just have similar names serving a similar function. I had created this api before I knew of its existence

Is the portal shown in that screenshot man made?

Ah just saw you were using beta-41, this was already fixed in a later beta. Beta-44 is the latest release as of writing this

Thank you for replying.
P.S. It seems the portal was actually ghast-made!


Rewamp object factory

Closes https://github.com/cbeust/testng/pull/2472

@juherr - I have done the following with respect to your PR https://github.com/cbeust/testng/pull/2472

Squashed the last 2 commits that are related to ITestObjectFactory
Fixed the test failures as an additional commit.

Please help take a look at this PR.

Thanks for the test fixes!! 👍


errorOnUnhandledRejections does not work as advertised

I'm submitting a ...

[x] bug report
[ ] feature request
[ ] other

Current behavior:
$qProvider.errorOnUnhandledRejections(false) suppresses all errors not just unhandled rejections.
Expected / new behavior:
$qProvider.errorOnUnhandledRejections(false) should only suppress unhandled rejection errors.
Minimal reproduction of the problem with instructions:
http://plnkr.co/edit/9lpc6KBgtSqOmxzn8ej4?p=preview
AngularJS version: 1.7.5
Browser: all
Anything else:
The issue in it's current form makes it extremely hard to catch REAL errors and send them to our error logging platform (Sentry)

errorOnUnhandledRejections(false) results in no real errors at all being observed in console or  $exceptionHandler
errorOnUnhandledRejections(true) results in tens of thousands of "Possibly unhandled rejection" errors being sent to Sentry, even though there are no real issues.

It's worth adding that we intentionally do not catch quite a lot of our promises as we expect them to result in no more actions.

Thanks for the report.
To be precise, errorOnUnhandledRejections(false) doesn't silence all errors but Promise rejections. I added a synchronous error to your Plunker and it shows up:
http://plnkr.co/edit/JErDPnK7w7d9WO9mmYRz?p=preview
AngularJS Promises are compatible with the Promises/A+ standard that many other promise library implements. Standard ES6 promises also adhere to those rules. And that standard states errors thrown in promise handlers should be treated in the same way as returning a rejected promise.
See the motivations explained at https://blog.domenic.me/youre-missing-the-point-of-promises/ (note that the header is not up to date as promises in jQuery 3.0+ are Promises/A+-compatible). Now, to be clear, I don't want to restart the discussion about the merits of this approach here. Those discussions have happened in the past, some people supported that, some opposed but the decision was taken and the whole ecosystem settled on implementing the Promises/A+ standard. And as long as AngularJS implements it as well - and since it's now in LTS mode no major changes are planned - the current behavior is required and we cannot change it.
In light of the above, I'm closing the issue. Feel free to post additional comments here if something needs further clarification.


Compiler error handling - syntax of actions

fixes #44
Signed-off-by: Bonczidai Levente<EMAIL_ADDRESS>

@oritstone @orius123  - need your opinion
One issue remains:
Python_ScriptTransformer and Java_ActionTransformer have ACTION scope - OK
ActionTransformer and WorkFlowTransformer also have ACTION scope - why?
Implications:
TransformersHandler accepts such situations:
action:
    action:

action:
    workflow:

These cases are checked in later stage in ExecutionStepFactory and fail with Invalid action data  message.
What to do (?) :

leave it like this.. fail with this message .. don't change transformers
remove ACTION scope from ActionTransformer and WorkFlowTransformer.. fail in early stage in TransformersHandler with this message: Error compiling source: operation_action_invalid_type.yaml. Property: action at: action data is illegal

I vote for option 2

@oritstone or @orius123 please review , so we can merge this...

All new yaml file names should have .sl extension
Operations names should be identical to the file names

@oritstone  - please take a look at the previous commit..
I moved the "validation that there is always either java_action or python_script and not both" in the TransformersHandler.validateKeyWords() method

@Bonczidai please pull from master, to resolve conflicts...

@oritstone can we merge this?

Missing copyrights section on top of the new slang files

updated..


Admin Category Management - Level 2 - Bug

When click "Category Level 2
ADD:

NAME: Enable SYMBOLS & NUMBERS, currently if we add, throwing error.if we mention special symbols , (.@, it does not accept .

Edit
User should be able to edit everything except ID.

ok sir


Add: Add a string type to know that our function will return a string

Description
Add a type for the function to know that it will return a string
Issue: #1369

Do we actually need this? It seems like typescript can always infer that this returns a string, so I'm not sure what this does. That being said, I'm still new to typescript, so maybe I'm missing something.
If anything, it seems like we should remove the static typing from the 2nd method in this file, assetSrcSet. It's currently declared like this:
export function assetSrcSet(srcset: string): string {

and returns like this:
  return constructed.join(",");

so we know (and so does typescript) it always returns a string.

@deer It's more of a cosmetic change, since like you said TS can infer those types. It would help in cases where changes suddenly change the inferred return type though. But that said in all the years working with TS my experience is that this is more of a theoretical risk that I haven't encountered in practice. But I don't mind if somebody wants to add these return types to ramp up on the code base. So I'm happy to merge such PRs when CI passes.

It's just like @marvinhagemeister said. It's more about a theoretical risk rather than an actual risk and also for me is to continue with the typing just like i've seen in other functions just like the next one which has a :string as a return.
Thank you both!


Extension installed detection [bug]

The extension version v3.1.0 is now public and live: https://chromewebstore.google.com/detail/penumbra-wallet/lkpmkhpnhknhmibgnmmhdhgdilepfghe
However, it does not look like that the web app can detect the extension. It's kinda strange because this works perfectly fine in development. It's specifically an issue in our production deployed app.
Perhaps we can test locally, not with pnpm dev, but with pnpm build to replicate.

Oh wait, I'm mistaken. v3.0.0 is published. We are still waiting on v3.1.0. Given the recent routing changes, this is probably an expected regression. Going to close for now. Will re-open with the new extension is published and we can re-test.


adjust carina-demo archetype based on 7.0.1 changes

need port all changes and release 7.0.0 carina archetype to maven central

done.

7.0.1 specified as carina-core build
1.1.3 as zebrunnr testng agent
removed android and ios xml suites from full_regression cron

to test:

generate new project using these steps: http://qaprosoft.github.io/carina/getting_started/
verify that above changes exist
run via command line api test

mvn -U -Dsuite=api clean test

-> no compilation error detected, all tests pass.
Note: publishing to maven central repo might be slow so better to test in 3-4 hours from now.


fix(FocusManager): refactor to extend Base

Refactors FocusManager, Row, and Column to extend Base.

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


Derive(MemDbg) makes trait non-object safe

MemDbg:mem_dbg{,_depth}_on functions take a &mut impl Write, which prevent doing &dyn MyTrait when MyTrait: MemDbg.
Not quite sure what a good fix is though. Clearly you want to provide the custom writer functionality.

I think the easiest solution is to pass around the writer as owned instead of taking the reference

You could also take a &mut dyn Write, but that could be considered ugly in cases where it's not necessary.

Oh probably it would make more sense for the default mem_dbg function to write to stderr rather than stdout.


Rules contradicting C# naming conventions

More than a few violations contradict official Microsoft recommendations and coding conventions and I was wondering why.
Few examples:

SA1200 contradicts Microsoft C# Coding Conventions
SA1309 contradicts Microsoft C# Naming Conventions, specifically "Private instance fields start with an underscore (_)."
SA1101 contradicts .Net Runtime Coding Guidelines, specifically "4. We avoid this. unless absolutely necessary".

And the list goes on...

The StyleCop rules predate the pages you linked to. We try to avoid making changes to the default behavior of the rules so existing users don't experience large numbers of breaking changes during updates.

@sharwell
Understood. However, the opposite is also true - for example, when coding in C# I tend to follow Microsoft's guidelines. I added StyleCop to a solution with several projects and got 1000s of warnings, right off the bat. Perhaps it would be beneficial to add configuration "Presets"? The default one could stay the same as today and a "Microsoft-guidelines" preset would reflect the more recent guidelines?
I would be happy to contribute.
cheers,
Uriel

Note that the pages you listed above are specific to individual repositories (one being the samples for the documentation site). Another rule listed on the page for the documentation site is:

Limit lines to 65 characters to enhance code readability on docs, especially on mobile screens.

We don't currently have one single set of rules that applies to all repositories. The stylecop.json file allows many items to be customized to fit different rules. It would be nice to futher consolidate SA1101/SX1101, and SA1309/SX1309/SX1309S by adding options, but we never ended up doing that.


First Invasive check through external K8s Job - dcgmi

This PR creates a singleton clientset that we can use anywhere in the code  to get access to kube api object.
It might be an overkill, but for now we can keep this.
An example of usage
cset := utils.GetClientsetInstance()
pods, err := cset.Cset.CoreV1().Pods("autopilot").List(context.TODO(), metav1.ListOptions{})

I also updated golang version, it seemed like a good moment to do that.

PR is updated with core for creating an external Job running dcgmi diag -r 3 IFF GPUs are idle.
Each Autopilot daemon pod is responsible for checking ONLY the resources on the node they belong to.
Each dcgmi Job:

Is a standalone pod assigned directly to the relevant node through NodeName.
It is NOT created if GPUs are currently used.
It requests all 8 GPUs on the node, so no other workload can take that node.
Is launched every 4h by default. The value can be customized through Helm. When set to 0, the feature is deactivated
Will self delete after 4h, made possible by TTLSecondsAfterFinished
Will not restart in any case, including failure (i.e., return -1 kind of scenario). Made possible by setting BackoffLimit  to 0
Will label the relevant node with key autopilot/dcgm.level.3 and PASS or ERR value, with a UTC timestamp, failure type (GPU Memory) and the affected GPU IDs. For instance autopilot/dcgm.level.3=PASS_2024-03-15_15.11.57UTC or autopilot/dcgm.level.3=ERR_2024-03-15_15.11.57UTC_GPU_Memory.0.1.2
Periodic checks and invasive checks are mutually exclusive. A mutex prevents autopilot from running both at the same time.

The PR also contains a bunch of refactoring and bug fixes and improvements

ping test, wrong unreachable node count
ping test, improved output with net iface id assigned to the the unreachable IP
ping test, if there is a discrepancy between autopilot pod's multi-nic annotation and the actual ifaces (listed with ifconfig, for instance), ping won't be run. This is just local to the pod, i.e., there's no distributed info exchange about this. A log entry is printed (this could become a prometheus metrics, like a flag saying the pod is healthy or not), the pod should be restarted but best thing would be to restart multi-nic controller
pciebw test, improved error handling


Commit17 BX Bitphoenix X

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

It is mergeable but why does not merge to main net on solona

Hallo Sir I am wating for PR merge
Please merge it , why it shows automerge but does not work actully for me

**
this the last commit to Update BX Bitphoenix X.
**
thank you

this is he last commit to Update BX Bitphoenix X.

**
thank you

Please read the SPL-token README, and ensure your SPL-token info uses the correct Uniswap format.

i try for days why it does not merge what is the problem
Please help someone

i need your help , how should i merge it ?!!!!

a little change

it

please approve as soon as posbl.
thank you

if this time does not work , cann i completly delete the token-list and start from Beginn to merge

what is the problem this time: the error shows > unsupported file modified: .github/PULL_REQUEST_TEMPLATE.md
i have not changed it