jsx-max-props-per-line: autofix

Hi this is my attempt at implementing #929. This aims to implement the fix such that a line with more than max props can be broken into more than 2 lines:
// Test case
<App a b c d e />

// [1]
<App a
b
c
d
e />

// [2]
<App a b
c d
e />

Would like an opinion on this if it is overkill. An alternate way of fixing would be under the simple fix commit (a simple addition of \n at a only one point).
Also, aware of #882 and hope to see some progress!

Rebased. Looks like the new options doesn't really affect how the autofix will work so should be fine.
Added fix outputs to all the new invalid tests cases.
A second opinion is always welcomed :)

I'd love for this to land soon! It will enable me to start using prettier (with prettier-eslint) and prevent it from messing with my max-props-per-line preference.

Rebased as a single atomic commit.
@lencioni @yannickcr

Also would be happy for this to be released!

I am just using @snowypowers's repo/branch until this lands. Working perfectly for my prettier-eslint setup, thanks @snowypowers!

rebased again due to merge conflicts (mostly minor due to string templating changes)

Can we get a confirmation that this fix command is working correctly?
If so, I would like to fulfill my agreement of the bounty of $50 to @snowypowers

Thank you @snowypowers
You should find a $50 contribution in your paypal :)

hi @revisionfour  got it! Thanks, this PR has been floating for quite long, good to see people using it and finally, getting merged! ^.^

Yeah, agreed. Getting it merged in is huge because thats what really matters to the community as a whole. Sometime you have to bang on the repo maintainers door until they take a look and finally merge it in.
I'm hoping that the community continues to add fix commands to rules that can have them applied. I see eslint as an analogy to grammar/spelling correction in word processors. Most people can't remember when there was no spell or grammar checker when they went to write a paper. Right now in my opinion, writing code is still in its infancy because we should have a best standards correction always on when we code. This is vital to getting government and large teams to write consistent and readable code, while reducing bugs and improving efficiency and meeting deadlines. Many programmers believe that using eslint is for amateurs and these programmers are highly misguided by their egos. I would ask these programmers to write me a 10 page paper about the future of coding from start to finish without spelling or grammar checking on and with no revisions. How many programmers could do this without 1 spelling or grammer error. None. eslint is vital to getting programming into the hands of the masses.

Not every eslint rule is programmatically fixable; many require human intervention and always will - but that's part of why it's so important to make all the ones that are fixable, fixable.
Thanks for the contribution!

@revisionfour, When will be next release going to happen?
CC: @snowypowers


METL-148 part1

These are the less risky improvements I've made to remove compiler warnings.

Those all look harmless.
On 25 Oct. 2016 10:08 am, "Doug Hagan"<EMAIL_ADDRESS>wrote:

These are the less risky improvements I've made to remove compiler
warnings.
You can view, comment on, or merge this pull request online at:
https://github.com/StackableRegiments/analyticalmetlx/pull/17
Commit Summary

METL-148: Explicitly specify exception type as Throwable when
intending to catch all Throwables.
METL-148: Topic.find returns Box[Topic] not Topic.
METL-148: Updated deprecated HTTP.UTF_8 to StandardCharsets.UTF_8.
METL-148: Updated deprecated Mongo save to save(true) to avoid
unsafe WriteConcern. This brings it in line with default MongoClient
behaviour and sets the WriteConcern to ACKNOWLEDGED.
METL-148: Updated deprecated MandatoryTypedField and
OptionallyTypedField "is" to "get".
METL-148: Updated deprecated Mongo save to save(true) to avoid
unsafe WriteConcern. This brings it in line with default MongoClient
behaviour and sets the WriteConcern to ACKNOWLEDGED.
METL-148: Updated deprecated CometActor.updateListeners(msg:Any) to
sendListenersMessage(msg:Any). This is not required for a no-args call to
updateListeners (eg in Notices.scala:28).
METL-148: Updated deprecated Mongo to MongoClient, and MongoOptions
to MongoClientOptions.
METL-148: Updated deprecated LiftRules.statelessDispatchTable to
statelessDispatch.
METL-148: Replaced deprecated IOUtils methods.
METL-148: Removed unreachable code: uri.getPort is of type int, so
other:Int will always match it.
METL-148: Removed unreachable code: h and w are of type Double, so
d:Double will always match.
METL-148: Updated deprecated CacheConfiguration.diskPersistent(false)
to CacheConfiguration.persistence(new PersistenceConfiguration().
strategy(PersistenceConfiguration.Strategy.NONE)).
METL-148: Updated deprecated RichException.getStackTraceString() to
ExceptionUtils.getStackTraceString(e) which uses e.printStackTrace.
METL-148: Updated deprecated Box(x) to Box.legacyNullTest(x).
(BoxTrait.apply is deprecated.)
METL-148: Updated deprecated Box.openTheBox to
Box.openOrThrowException which throws an explicit exception if the contents
are null.

File Changes

M src/main/scala/com/metl/comet/Bubbles.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-0
(88)
M src/main/scala/com/metl/comet/Notices.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-1
(2)
M src/main/scala/com/metl/comet/Reputation.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-2
(4)
M src/main/scala/com/metl/comet/Topics.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-3
(18)
M src/main/scala/com/metl/data/JsonSerializer.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-4
(5)
M src/main/scala/com/metl/model/BackendAdaptor.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-5
(28)
M src/main/scala/com/metl/model/D2LGroupsProvider.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-6
(10)
M src/main/scala/com/metl/model/DBFormats.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-7
(26)
M src/main/scala/com/metl/model/EmbeddedXmpp.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-8
(16)
M src/main/scala/com/metl/model/HttpResponder.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-9
(2)
M src/main/scala/com/metl/model/LTI.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-10
(5)
M src/main/scala/com/metl/model/Reputation.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-11
(14)
M src/main/scala/com/metl/model/SlideRenderer.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-12
(2)
M src/main/scala/com/metl/model/StackManager.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-13
(18)
M src/main/scala/com/metl/model/metl2011/
MeTL2011BackendAdaptor.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-14
(4)
M src/main/scala/com/metl/model/metl2011/MeTL2011Resources.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-15
(2)
M src/main/scala/com/metl/snippet/Summaries.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-16
(16)
M src/main/scala/com/metl/snippet/version.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-17
(4)
A src/main/scala/com/metl/utils/ExceptionUtils.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-18
(11)
M src/main/scala/com/metl/utils/Http.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-19
(10)
M src/main/scala/com/metl/utils/StronglyTypedJsonActor.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-20
(2)
M src/main/scala/com/metl/view/Endpoints.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-21
(4)
M src/main/scala/com/metl/view/StatelessHtml.scala
https://github.com/StackableRegiments/analyticalmetlx/pull/17/files#diff-22
(6)

Patch Links:

https://github.com/StackableRegiments/analyticalmetlx/pull/17.patch
https://github.com/StackableRegiments/analyticalmetlx/pull/17.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/StackableRegiments/analyticalmetlx/pull/17, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVeI4nD2jlLehWcHwM0u59IA1BMik6xmks5q3TpigaJpZM4KfYr7
.


Make the codebase cargo clippy compatible

run cargo clippy
make it happy!

I would like to work on this, will finish it by EOW.


Reddit Data csv

Is there any way I could get access to the Reddit data CSV files used here? I tried google big query but after three queries my quota limit exceeded and on top of that, google cloud is not accepting my card for 60 days trial.
It would really help me if you could upload the csv files to drive or somewhere and share them. It would mean a lot. Thanks a lot in advance...

I think pushio shares the archives for comments and threads. So you can potentially make csv from those. Also I think there was someone sharing the data somewhere but I don't remember the link.

Thanks a lot for sending me the files. I will be able to make it work now I guess XD

I would appreciate it if you can send me csv file too<EMAIL_ADDRESS>


python3Packages.pypandoc: 1.8.1 -> 1.10

https://github.com/JessicaTegner/pypandoc/releases/tag/v1.9 https://github.com/JessicaTegner/pypandoc/releases/tag/v1.10
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Tested apostrophe HTML/PDF export.

Result of nixpkgs-review pr 201517 run on x86_64-linux 1

4 packages built:

apostrophe
python310Packages.pypandoc
python39Packages.pypandoc
wayback-machine-archiver


Before-level cinematic includes an environment and background

We'll need to create some background assets -- trees, bushes, buildings, mailboxes, whatever. These cinematics need to happen somewhere.
Closing this might also close #77, #78, #84, #85 and #173.

I think we can close this if the cinematics specify where they happen. We'll add more environment detail over time, but saying stuff like "this cinematic happens inside" and "this cinematic happens outside" is enough to close this story.

I think we can close this if the cinematics specify where they happen. We'll add more environment detail over time, but saying stuff like "this cinematic happens inside" and "this cinematic happens outside" is enough to close this story.


fix(slick): disable slick greeter by default

Description of changes
We're disabling slick greeter by default.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@bobby285271 @water-sucks

I will later change this to a mkEnableOption in a later PR.


Unable to switch playground out of TSX mode

The playground has a toggle for choosing what --jsx is passed to the compiler, but setting it to None still leaves the playground's input file as input.tsx resulting in compilation errors when trying to use generic arrow functions.
Playground
const func = <T>(arr: T) => {
};

This is a regression - the old site allowed the above code since it was not contained in a TSX file.

@orta Is this back again?


Add a "user_data" mechanism

It will probably be needed in some complex setups and simplify a lot building abstractions.
Most likely, user_data() method for the Proxy and Resource traits, providing an UserDataGuard wrapping an interior MutexGuard, and deref-ing to a &mut Option<Box<Any+Send+'static>>.

Potential problem with this is, a Mutex might be expensive to create?
Most likely most proxies/resources won't ever need it, so we don't want to add this cost unconditionally.
Also, that means we add a threadsafe-synchronization cost unconditionally, even for single-threaded programs...

Done in #84


Hello World

My first batch change!
Created by Sourcegraph batch change alan.scherger/PC2v1.

Lol.

Pardon the spam i just picked 3 random repos!

@flyinprogrammer which ones?? :D


Automatically merge PR for .deployedprotoversion update

On branch versioning_workflow
Changes to be committed:
modified:   .github/workflows/deploy.yml

This PR attempts to further improve upon #870 by adding permissions so that the build-and-deploy workflow can automatically merge the PRs it creates for updated version files.


Google detect malware

https://www.virustotal.com/gui/file/8582bc3e8628523d08515356a7e9655e519e2872dbfdcc6f2925a80188ac5ac6

Thanks for reporting


Scene componentDidMount call when switching tabs

Currently componentDidMount is my scene called very time i switching, this will cause my view will add multiple time ( native module ). How to make Scene componentDidMount call once.

Please provide code

Closing because the library doesn't remount screens, so it's a bug in the user code, and the issue template wasn't filled.

Sorry for late reply, my code is:
<TabView
              navigationState={this.state}
              renderTabBar={this._renderTabBar}
              renderScene={
                SceneMap({
                  call: () => (
                    <CallScreen />
                  ),
                  chat: () => (
                    <ChatScreen messages={this.state.messages}/>
                  ),
                  problem: () => (
                    <ProblemScreen/>
                  )
                })
              }
              onIndexChange={index => this.setState({index})}/>

CallScreen.js
class CallScreen extends React.PureComponent {
  componentDidMount() {
      console.log("Mount)
   }

  componentWillUnmount() {
      console.log("Unmount")
   }
   render () {
      return (
        <View>... </View>
     )
  }
}

Everytime im switching to another tabs, componentDidMount and componentWillUnmount inside CallScreen got called. How to prevent this :(

Issue fixed, just read the following note

Always define your components elsewhere in the top level of the file. If you pass inline functions, it'll re-create the component every render, which will cause the entire route to unmount and remount every change. It's very bad for performance and will also cause any local state to be lost.

Thanks :D

Issue fixed, just read the following note. My fault.

Always define your components elsewhere in the top level of the file. If you pass inline functions, it'll re-create the component every render, which will cause the entire route to unmount and remount every change. It's very bad for performance and will also cause any local state to be lost.

Thanks :D

Hi, But if you want pass a params from the state, Who do it?
For Example
`
const Calls = (Calls=[])=>
const Chats = (Chats=[])=>
export default class PedidoNuevo extends Component {
state={
Calls:[1,2,3,4,5],
Chats:[43,67]
}
render(){
return (<TabView
navigationState={this.state}
renderTabBar={this._renderTabBar}
renderScene={
SceneMap({
call:Calls,
chat: Chats,
problem: () => (

)
})
}
onIndexChange={index => this.setState({index})}/>)
}
}
`
How to pass this params?
Thanks

Hi @JuanCamilo0809
If you need to pass additional props, use a custom renderScene function:
Ex:
renderScene = ({ route }) => {   switch (route.key) {     case 'first':       return <FirstRoute foo={this.props.foo} />;     case 'second':       return <SecondRoute />;     default:       return null;   } };


⚠ Bofrost.com has degraded performance

In 2593faa, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 479 ms

Resolved: Bofrost.com performance has improved in d917f58.


Release/fluffy bunny rabbit

Try out this version of the Hiro Wallet - download extension builds.

Releases:

send form loading UI updates
punycode support
client side logs

Todo:

[ ] QA on Chrome
[ ] QA on Firefox

Seems to work fine on Chrome, was unable to QA it on FF.


refactor: implement Configurable for blocks

Previously, blocks struct try to mimic the Configurable trait by having functions with the same name and similar signature. It cannot implement Configurable because it propagate an Error to caller in case of invalid values.
However, those invalid values should be handled by clap in the first place. So, we can implement Configurable for blocks with custom configure_from instead of the default one.

TODO

[x] Use cargo fmt
[ ] Add necessary tests
[ ] Add changelog entry
[ ] Update default config/theme in README (if applicable)
[ ] Update man page at lsd/doc/lsd.md (if applicable)

Codecov Report

Merging #711 (e44bf44) into master (b5ad6d0) will increase coverage by 0.00%.
The diff coverage is 88.23%.

@@           Coverage Diff           @@
##           master     #711   +/-   ##
=======================================
  Coverage   88.36%   88.37%
=======================================
  Files          41       41
  Lines        4546     4532   -14
=======================================
- Hits         4017     4005   -12
+ Misses        529      527    -2

Impacted Files
Coverage Δ

src/flags/blocks.rs
95.95% <88.05%> (+0.50%)
:arrow_up:

src/flags.rs
100.00% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


How to run server...

I'm trying to run server, but ngninx log says:
*1 connect() to unix:/var/run/gunicorn/pushjet-api.sock failed (111: Connection refused) while connecting to upstream,
and gunicorn's log says:
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/gunicorn/arbiter.py", line 502, in spawn_worker
worker.init_process()
File "/usr/lib/python2.7/dist-packages/gunicorn/workers/base.py", line 114, in init_process
self.wsgi = self.app.wsgi()
File "/usr/lib/python2.7/dist-packages/gunicorn/app/base.py", line 66, in wsgi
self.callable = self.load()
File "/usr/lib/python2.7/dist-packages/gunicorn/app/wsgiapp.py", line 65, in load
return self.load_wsgiapp()
File "/usr/lib/python2.7/dist-packages/gunicorn/app/wsgiapp.py", line 52, in load_wsgiapp
return util.import_app(self.app_uri)
File "/usr/lib/python2.7/dist-packages/gunicorn/util.py", line 365, in import_app
import(module)
ImportError: No module named wsgi

The wsgi.py is in the .gitignore, could you stage it for commit just once ?

Found solution ; gunicorn conf - the difference is last line: 'wsgi' -> 'application:app'
CONFIG = {
'mode': 'wsgi',
'working_dir': '/usr/src/Pushjet-Server-Api/',
'python': '/usr/bin/python2',
'args': (
'--bind=unix:/var/run/gunicorn/pushjet-api.sock',
'--workers=2',
'--timeout=30',
'application:app',
),
}

I'm having this same issue, and changing that line to application:app doesn't seem to fix it. Is that what it's supposed to be? It's strange, I'm not seeing any more errors in the gunicorn log, but it's clearly not starting. If I start it manually, it complains about there not being a module named wsgi,


Update the manifest to the latest nightly build

Fixes #
Proposed Changes

Release Note

How often do we have to update the manifest to the latest nightly?
Should we investigate into automation if this should be done frequently?

@k4leung4 I plan to do it on a weekly basis, but there are several issues:

Inconsistency of labels used in one nightly built yaml:
If we download the latest serving yaml here https://storage.googleapis.com/knative-nightly/serving/latest/serving.yaml. I see the deployment
name: networking-istio
has the label
serving.knative.dev/release: "v20200316-516fac9"
, but the deployment
name: autoscaler-hpa
has the label
serving.knative.dev/release: "v20200319-2f826455d"
Why do we have two different labels?
If we do it automatically, how to get the version number?

How to change the version number in version.go, if we do the update automatically?

the release labels are different because net-istio is built from knative.dev/net-istio and vendored, https://github.com/knative/serving/pull/7266/files

we only use release versions for releases right? so maybe have a daily job that updates with the nightly build. for releases, we manually do update with the release build, cut the release branch, and the daily job will update it back to the nightly build?

The issue is with the unit tests TestManifestVersionSame.
If the version does not match the version number in manifest, they fail.

/approve
/lgtm


az upgrade should only upgrade az,  "brew cleanup" should be skipped

This is autogenerated. Please review and update as needed.
Describe the bug
I ran "az upgrade" expecting that the az cli command would be upgraded.
The whole of homebrew was upgraded including python 3.8 and python 3.9.
From the output, only python 3.8 is a dependency of az, not python 3.9.   This may have been the result of an automatic  'brew cleanup' following on from the az update:
==> Summary
🍺<EMAIL_ADDRESS>4,498 files, 72.6MB
==> Installing azure-cli
==> Pouring azure-cli-2.18.0.catalina.bottle.tar.gz
==> Caveats
Bash completion has been installed to:
  /usr/local/etc/bash_completion.d
==> Summary
🍺  /usr/local/Cellar/azure-cli/2.18.0: 18,244 files, 295.2MB
==> `brew cleanup` has not been run in 30 days, running now...
...
runed 1 symbolic links and 2 directories from /usr/local
==> Upgrading 1 dependent:
<EMAIL_ADDRESS>3.9.0 -> 3.9.1_6
==> Upgrading<EMAIL_ADDRESS>3.9.0 -> 3.9.1_6
==> Downloading https://homebrew.bintray.com/bottles/tcl-tk-8.6.11.catalina.bottle.tar.gz

curl: (22) The requested URL returned error: 407
Error: Failed to download resource "tcl-tk"
Download failed: https://homebrew.bintray.com/bottles/tcl-tk-8.6.11.catalina.bottle.tar.gz
Warning: Bottle installation failed: building from source.
==> Downloading https://github.com/andreas-kupries/critcl/archive/<IP_ADDRESS>.tar.gz

curl: (22) The requested URL returned error: 407
Error: Failed to download resource "tcl-tk--critcl"
Download failed: https://github.com/andreas-kupries/critcl/archive/<IP_ADDRESS>.tar.gz
==> Checking for dependents of upgraded formulae...
==> No broken dependents found!

(Yes I am using a proxy, and it had worked for downloading the updates to az;  I'm unsure why the 407's started at this point)
Following on from the python 3.9, the failures around the tcl package left me highly confused and not confident in the state of my system.
Suggest that 'az upgrade' pass an option to brew to skip that cleanup, if possible.
Just upgrade do the minimum which concerns az.
Apparently this might be done by az setting HOMEBREW_NO_INSTALL_CLEANUP=1 before az calls into brew..
Command Name
az upgrade
Errors:
To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az upgrade

Expected Behavior
Minimum upgrade to perform az upgrade takes place.   Do not upgrade my entire homebrew system.
Environment Summary
macOS-10.15.7-x86_64-i386-64bit
Python 3.8.7
Installer: HOMEBREW

azure-cli 2.18.0

Additional Context

thanks for sharing the finding! @fengzhou-msft could you pls help to have a look? thanks

thanks for sharing the finding! @fengzhou-msft could you pls help to have a look? thanks

use #17150 to track az upgrade via brew.


No response when I open a compressed javascript file

VSCode Version: Code 1.10.1 (653f8733dd5a5c43d66d7168b4701f94d72b62e5, 2017-03-02T00:47:58.633Z)
OS Version: Linux x64 4.8.0-39-generic
Extensions:

Extension
Author
Version

beautify
HookyQR
0.8.8

crane
HvyIndustries
0.2.2

eval
Stormspirit
0.0.6

comment-snippets
Tobiah
0.2.0

vscode-translate
chun
0.0.2

gitignore
codezombiech
0.5.0

vscode-eslint
dbaeumer
1.2.7

githistory
donjayamanne
0.1.5

vscode-html-css
ecmel
0.0.33

vscode-great-icons
emmanuelbeziat
1.1.44

vscode-npm
fknop
3.3.0

vscode-coding-tracker
hangxingliu
0.3.2

Kotlin
mathiasfrohlich
1.2.0

vs-code-quick-require
milkmidi
0.1.0

vscode-apache
mrmlnc
1.1.0

cpptools
ms-vscode
0.10.2

csharp
ms-vscode
1.7.0

js-atom-grammar
ms-vscode
0.1.12

debugger-for-chrome
msjsdiag
2.5.5

node-modules-resolve
naumovs
1.0.2

sass-indented
robinbentley
1.3.0

code-spell-checker
streetsidesoftware
0.17.2

html-preview-vscode
tht13
0.1.1

Just use VSCode open this document: mdjs.min.js
Then VSCode will not response and system toast a "No response and would you want to restart" modal dialog

@hangxingliu I cannot reproduce, for me the file opens fine.
Does the issue still occur when you launch VS Code w/ extensions disabled? From the command line: code --disable-extensions

Thanks for your guide. After code --disable-extensions and code --extensions-dir=tmp, I found this is a bug in the extension js-atom-grammar.I think that I should create an issue in js-atom-grammar project

I found there are not js-atom-grammar repository. And it's author is "Microsoft". So I think that this bug should be processed by developer in Microsoft.

How to reproduce:

Installing extension js-atom-grammar (the version I am using is 0.1.12)
Then downloading and opening the file https://raw.githubusercontent.com/hangxingliu/mdjs/8815d3c0dedf312360618f1d31f1a23d532f10ca/mdjs.min.js (This is a javascript file compressed by uglify-js)
You will find your VSCode has not response and then crash.

And if you open the file: https://raw.githubusercontent.com/hangxingliu/mdjs/master/mdjs.min.js
You will find your VSCode has not response some seconds and then the highlight of this file be broken.

The Atom JavaScrip grammar comes from https://github.com/atom/language-javascript.
I created https://github.com/atom/language-javascript/issues/500 for you there.


Update Figma select component to provide icon option on select menu items.

Summary
Ticket #224 allows the addition of custom elements within a select option. To mirror this function in Figma, we need to update the component in Figma to include an icon.
💬 Description
Add ability to have left aligned icon alongside select items. This would allow a use case such as the selection of a country and the inclusion of a flag icon next to each option.
💰 Use value
Provides greater customization of components and mirrors changes in developed library.
Additional info
Tell us anything else useful to help us understand your suggestion.

Relates to devlopment ticket #224


[Bug] Sky Sweeper doesn't seem to work on sand underwater

Using a Ferricore shovel with Sky Sweeper enabled, sand above the mined block isn't mined when underwater. It works on above-water sand, as well as both above- and below-water gravel.

I just tested it, and it seemed to work ok for me.  Are you sure the tool wasn't disabled when you tried?

I'll check again, huh.

Okay, after checking again the bug persists; the tool is definitely enabled and I double-checked that it still works on land without changing anything. I'm using a ~fairly large custom modpack I just threw together, so it's probably some sort of conflict. I'm including my mod list just in case but I know these can be tricky and I wouldn't expect anybody else to try to replicate my environment. My apologies for not checking in a test environment before reporting.
Mod list: https://pastebin.com/aErPda2S
I might try a binary search later this week if I have time, see if I can track down the problem, but definitely not today and no promises. 💜

Can you confirm that when the mod is installed alone (By itself) the tool works as you'd expect, just as a confirmation that its definitely some kind of conflict? :)

Sure, I was able to do that real quick. Running the same mod version (0.9.7) and loader version (Neoforge 20.4.231) and no other mods, I was unable to replicate the bug.

Thanks I’ll close this. If you find out the cause let me know and I’ll see if I can work around it.


REJEKIBET APK 51.0 (Login) Unduh gratis untuk Android

REJEKIBET is a versatile online gaming platform that offers a variety of thrilling and entertaining games for players seeking an engaging experience. With its easy-to-use interface and a wide selection of games, REJEKIBET appeals to casual players and gaming enthusiasts alike. The platform ensures a vibrant and fun atmosphere, providing users with numerous options to explore and enjoy different genres and styles of play.
Unduh LINK: REJEKIBET APK

Closing the issue as it seems to be an advertisement.
/close not-planned


Study Definitions Repository Definition

From JIRA DDF-427
We lack a clear CDISC governed glossary definition for a 'Study Definition Repository (SDR)' as well as how we define 'Study Definition'.
On DDF GitHub we have:
"The Study Definitions Repository is a novel central component aimed at facilitating the exchange of structured study definitions across clinical systems using both technical and data standards."
In previous DDF material we had:
Repository that holds the specific study instances. Each study definition includes both the study structure (design) plus the applied standards (metadata that were selected in the study builder.
This will be versioned, and cover the process from study definition from planning, setup, execution, analysis, reporting to submission.
The Study Definition will reference the Industry Standards as well as the Sponsor Standards.
Some instances will be managed and defined via the connected Study Builders; other instances will be sourced in from other Study Definition Data Sources.

@EMuhlbradt @cupkes @dih-cdisc @jared-schreibman-cdisc @jowencdisc @drewcdisc
Erin and I are meeting Friday morning (19-May) to work out definitions for "Study Definition" and "Study Definition Repository". Erin found a couple of sources (CDISC DDF Project and Trancelerate), however it's still not clear what these really are and how they are intended to be used and appear to even contradict one another. One source implies these relate to the complete definition of a specific instance of a study (e.g., identifies all of the components, standards, design elements, and technical/metadata) whereas the other suggests that each study definition is a reusable snippit that can be used with other study definitions to completely describe a study. The first case suggest a 1:1 relationship between a Study and a Study Definition and the second case suggests a 1:many relationship.
It would be very helpful if each of you could provide a comment in this ticket with your own idea of what we mean by the term "Study Definition". Is this even the most accurate term? Your input will help us to synthesize thoughts into an accurate and somewhat precise glossary-type definition for this term. The "Study Definition Repository" is straightforward in that it will be a digitally stored collection of individual Study Definitions.
Thanks for your thoughts on this.

You are going to love this one :)
Study Definition: An imprecise label used to denote a set of metadata (defined with a version of the USDM) that defines some or all of a clinical [trial/study] design. This collection of metadata may well expand over time to encompass an entire clinical [trial/study] protocol.

Study Definitions Repository: A digital management and storage system for study definitions.
Accompanying Waffle:
NOTE: The study definitions repository:
...allows retrieval of study definitions for the creation of a study instance.
...facilitates the exchange of structured study definitions across clinical systems using both technical and data standards.
...will be versioned, and cover the process of study planning, setup, execution, analysis, reporting to submission.


implement simplified schema inference and io format

Fixes #208

Codecov Report

Merging #209 into master will increase coverage by 0.29%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #209      +/-   ##
==========================================
+ Coverage   96.42%   96.72%   +0.29%
==========================================
  Files          13       13
  Lines        1118     1159      +41
==========================================
+ Hits         1078     1121      +43
+ Misses         40       38       -2

Impacted Files
Coverage Δ

pandera/checks.py
98.71% <100.00%> (+<0.01%)
:arrow_up:

pandera/io.py
98.80% <100.00%> (+7.14%)
:arrow_up:

pandera/schema_components.py
98.42% <100.00%> (+0.02%)
:arrow_up:

pandera/schema_statistics.py
100.00% <100.00%> (ø)

pandera/schemas.py
95.54% <100.00%> (-0.52%)
:arrow_down:

pandera/dtypes.py
94.73% <0.00%> (+1.75%)
:arrow_up:

Continue to review full report at Codecov.

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


gentoo version

just wanted to let you know that on the gentoo packages site (https://packages.gentoo.org/packages/net-misc/youtube-viewer) it says that the package there is outdated. no clue if this is true but yeah just wanted to let you know

I will close this, as I'm not the maintainer of the Gentoo package.


fix(server): Add support for directories with multiple wildcard children

Updates the logic in module-tree.ts to support a path that has multiple children that are wildcards at the same segment position. E.g.:
/myPets/{petName}/health
/myPets/{petId}/records

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

36 of 43 (83.72%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 82.619%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/server/module-tree.ts
31
38
81.58%

Totals

Change from base Build<PHONE_NUMBER>9:
-0.2%

Covered Lines:
3284

Relevant Lines:
4066

💛  - Coveralls


feat: Control Teradata decimal format when cast to string

This PR adds a TO_CHAR format to Teradata NUMBERs/DECIMALs to work around two inconsistencies with other engines.

SELECT CAST(CAST(7 AS DECIMAL(5,2)) AS VARCHAR(64));
'7.00'
SELECT CAST(CAST(-0.7 AS DECIMAL(5,2)) AS VARCHAR(64));
'-.70'

After merging this PR we get these literals from the examples above:

'7'
'-0.7'

/gcbrun

/gcbrun


Half-Life 2 unplayable

Any updates in the last two days that would render half-life 2 unplayable?
This guy's half-life 2 is completely playable 2 days ago with only a couple rendering issues. Mine, not so much.
https://www.youtube.com/watch?v=HNpBncAs0Ek
https://i.imgur.com/9raWxLD.png - main menu.
Will happily provide more information if it is something on my end.

The problem is that d9vk currently does not support all the new and shiny feature levels of d3d9, which means you have to set some launch options. Those should be enough to make the game work:
-sw +mat_disable_d3d9ex 1 +mat_dxlevel 90 -dxlevel 90

That did the trick. Renders wonderfully, has issues where all map textures vanish depending on where you look. Pretty sure it's some kind of surface texture issue with vulkan.
https://i.imgur.com/EZ6Zgvt.jpg
Anyway, cool beans. Can't wait to see other games working with d9vk. Very curious on how dishonored will perform.

May I suggest you close this? (:


In Use in controller modal in the note change "links" to "external links"

https://issues.redhat.com/browse/AAH-1112
Problem Description: In Use in controller modal in the note change "links" to "external links"
QA Criteria: In Use in controller modal in the note has "external links" insteaad of "links"
Proposed Solution: In Use in controller modal in the note change "links" to "external links"
Reminder: Set the affected Component(s): [API, UI, Infrastructure, Importer, Installer, Sync, Docs, pulp_ansible, pulpcore]

Before:

After:


[Test Framework] Native Echo broken for WebSocket and GRPC

When using a sidecar, the native echo component fails for WebSocket and GRPC. Without a sidecar, it's fine.  I suspect it's related to the way we're filtering inbound resources.

Fixed by #14614


Add CUPY_INCLUDE_PATH and CUPY_LIBRARY_PATH env vars

This closes #5751.
(I'm going to use this one in v11 to support building NVTX1 on CUDA 12.0 / Windows)

Q: How does this change the build process for those who need to build from source?

Nothing will change if these envs are not set. By these env vars, users can just set a list of directories to discover, instead of using platform-dependent argument env vars (LDFLAGS=-L/path/to/cudnn/lib64 on Linux, LINK=/LIBPATH:C:\path\to\cudnn\lib on Windows).

As a next step I'm thinking of python cupyx/tools/install_library.py --action env generate outputs like
export CUPY_INCLUDE_PATH=~/.cupy/cuda_libs/11.8/cudnn/8.7.0/include
export CUPY_LIBRARY_PATH=~/.cupy/cuda_libs/11.8/cudnn/8.7.0/lib64

so that contributors can use tools managed by the installer for development (and also for wheel build).

/test mini

Rebased and fixed conflict!

/test mini

/test mini

/test mini

@emcastillo Failed to backport automatically.

Not cleanly cherry-picked

@emcastillo This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?


MG|GS /Doc

mg/doc is currently just a stub implementation to allow design of gs/doc .
mg/doc currently supports only the expression pkgname.Funcname but the design(general interaction) of the goto definition and doc hint functionality in gs/doc ignoring a bit of styling is complete (direct #21 and #31 here)

infer expression given only the values for row/col or offset into src

done

search all types and vars in all packages not just top-level functions in import packages

partially complete - types are not resolved

implement doc browser to quickly find all definitions in GOPATH and GOROOT

done

offer function parameter hint as you type - this would allow truncating the completion when it gets too long (ST2 truncates it in a weird way, but there's no way for me to know how wide the completion pop-up can be)

not currently, no. Only pkg.X is supported

Is this still the case?

Yeah, is this still the case? I'm hitting the same problems.
@tamird nice seeing you here too.

Thanks for all the work on this; definitely +1 to this feature.

@diogomonica the solution is to use something that uses godef/oracle for symbol lookup. I'm using https://github.com/ironcladlou/GoTools

I can right click on a method name and the mouse context menu shows "Goto Definition" as the first option.  But the "Cmd-. Cmd-g" (on Mac) does not work.  What is the recommended way to configure it or re-enable the shortcut which was so helpful?  Thanks!

F12


Change Prettier's bracket spacing configuration

Should we also look at other Prettier configuration?

Not sure I prefer this TBH. It forces you to not use spaces everywhere, when I want them sometimes (and yes, I know if we didnt use an automatic formatter that wouldnt be a tradeoff we'd need to discuss). I think in the previous commit they were desirable, but I think on the whole I prefer spaces (ie/ how the code looks before this change). @AustinXII  - what is your preference?

Should we also look at other Prettier configuration?

I want to turn on consistent quotes in object literals, but thats a different commit IMO

Okay, the other defaults make sense to me.

I want to turn on consistent quotes in object literals, but thats a different commit IMO.

I'd generally disagree, unless we're talking about different things?

I'd generally disagree, unless we're talking about different things?

Another 'it depends'. But for an object with like only 10 keys and 2 have to be quoted it bugs me that the rest arent consistent. Also, remember - this is automatically enforced so people editing the file can just forget quotes and then run npm run fix (or forget to and have to followup with shameful commits after the fact...)


[Feature] Configurable cookie auth duration (session/max age)

Summary
Currently the built-in authentication sets a cookie SESSION with "session" expiry. While I'm happy with the built-in auth I'd like to configure the cookie to instead be valid for e.g. a year to stay logged in. If such a configuration option is added the default can of course remain session expiry such that it's not a breaking change.
Motivation
I'm using "Digital signage" software (Xibo) by which I can push content to screens remotely. I'd also like to show Appsmith content but the issue is that every time the software re-opens the auth session is gone and it needs to be logged in again. These screens are mostly "passive" to look at as in there is no keyboard/mouse even attached to them to log in. If the auth cookie duration could be configurable this would solve my use case.
Additional Context
This is the cookie I'm referring to.

@zegerhoogeboom would making the application public solve this for you? You can click share and make the application public so that the appsmith content can always be shown.

Oh wow that's perfect actually, thanks a lot!

Always happy to help 😄


[T4A6][T09-A3] Tan Jian Sin

Ready for review

@BrandonTJS
Some comments added. Please ack & close the PR after reading comments.


WiFi authentication page doesn't work on recent Android

Using a freshly built Alles image, I get the message "Header fields are too long for server to interpret" after connecting to the WiFi from my Android phone:

Steps to reproduce:

Build Alles binary (using Docker image to ensure clean environment)
Load binary onto Alles, reboot
On phone, connect to the new wifi network
Wait until the 'sign into wifi network' prompt appears, then accept it
The phone shows the above page

If I manually configure the phone to use the wifi network without a connection, then open http://<IP_ADDRESS> in a browser, the configuration page is able to load.
I think the solution is simple- CONFIG_HTTPD_MAX_REQ_HDR_LEN needs to be increased:
https://github.com/bwhitman/alles/blob/main/sdkconfig#L375
For instance to 1024:
https://esp32.com/viewtopic.php?t=28702

Thanks matt! I haven't run into this so can't confirm it's a fix, but i did confirm it doesn't break my existing setup, so i've added it.


Create ci_compile.yml

1st step to CI
next steps: unittests, file checks, ....

now everything should work:


Update request: paperless-ngx 2.11.4 → 2.11.6

Package name: paperless-ngx
Latest released version: 2.11.6
Current version on the unstable channel: 2.11.4
Current version on the stable/release channel: 2.10.2
[x] Checked the nixpkgs pull requests

Notify maintainers
@lukegb @gador @erikarvstedt @leona-ya

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

See #336797


Redo API docs tags.

While reviewing #2054, I noticed the setup for tags and tag groups was more complicated than it needed to be and there was some unnecessary cruft present, mostly as a result of figuring out how tags worked across the proto/swagger/redoc interactions.
In this PR:

Remove unnecessary tag setup that we're not using anymore.
Move all the tag setup into one file.
Use the default tags in the proto to remove an unnecessary level of redirection.

undoing the tags stuff makes it so that all of the proto docs are visible. do we want to hide the ones we haven't yet actually documented?

It's my understanding that only things tagged with a tag in a tag group will show up in the UI, but I'll double check.

I just rebuilt the docs and only see the Reporting and Stats endpoints in the UI, so I think this is doing what's intended. If you're seeing other behavior please let me know!

oh interesting. i def saw them all this morning but i’ll pull things down and check again later!


add option --prefer-ipv6

[x] I'm reporting a feature request
[x] I've verified that I'm running youtube-dl version 2<IP_ADDRESS>
[x] I've searched the bugtracker for similar feature requests including closed ones

It's somewhat useful as some of the websites may not be available through IPv6.

Thinking about taking this one on.  Is this still a relevant task?


Memory consumption of create_function()

I have noticed a lot of memory leaks if using the short syntax like
from($array)->select("explode(', ', $v)")->toArray();

But it's ok with closures
from($array)->select(function($var) { return explode(', ', $var); })->toArray();

proceeding first loop at one runtime with a big amount of data will cause to overhead of memory.
Not sure if this bottleneck is reason of lib implementation, or PHP's create_function(), but if you suggest me, I'll write a test samples of this issue.

Should be fixed by commit 243caef2d0f5dd8e423569e13cec34109712bde6 (Added lambda cache to Utils::createLambdaFromString).  Not sure whether it will cause any issues, but seems to work correctly so far.


[Feature] Add Constraints to Seeds

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
As of DBT Core 1.8 Model contracts and constraints are not supported for seeds
As requested in another feature  requesting contracts for seeds there was no reason given other than setting data types for the columns which the column types config already provides.
Currently with seeds there are no ways to define constraints. Most data platforms do not ENFORCE constraints specified in the documentation. Like models, I believe seeds should be able to specify constraints such as primary & surrogate key and not null.
Adding constraints to a seed file could either follow the standard of model contracts
seeds:
  - name: seed_name
    config:
      schema: seed_schema
      tags: seed
      column_types:
        seed_id: int
      contract:
        enforced: true
    columns:
      - name: seed_id
        constraints:
         - type: primary_key

Or allow the configuration of constraints outside the declaration of a contract
seeds:
  - name: seed_name
    config:
      schema: seed_schema
      tags: seed
      column_types:
        seed_id: int
      constraints:
        seed_id: [primary_key, not_null]

This could apply at the table level, as well as the column level
Describe alternatives you've considered
No response
Who will this benefit?
This will benefit seeds by allowing defined constraints on individual columns or the table itself for data governance to determine relationships between primary, surrogate keys and not nullable columns because seeds are materialized as tables
Are you interested in contributing this feature?
No response
Anything else?
No response

Thanks for proposing this @cboethigtrellance!
We're planning to continue supporting constraints only for models at this time (in conjunction with model contracts) and not extend them to other resource types like seeds or snapshots. So I'm going to close this as "not planned". See below for a couple alternatives instead.
dbt_constraints package
If your database is Snowflake, PostgreSQL, Oracle, Redshift, or Vertica, the dbt_constraints would be something to take a look at. (If you are not using one of those databases, see here for notes how to add support for your database.)
Example:
packages.yml
packages:
  - package: Snowflake-Labs/dbt_constraints
    version: [">=1.0.0", "<1.1.0"]

seeds:
  - name: seed_name
    columns:
      - name: id
        tests:
          - not_null

Install the package and execute the seeds (which will automatically add constraint based off the not_null test above):
dbt deps
dbt seed

Post hook to add constraints
Alternatively, you can use a post-hook to add constraints as well.
Example:
dbt_project.yml
seeds:
  my_project:
    seed_name:
      +post-hook:
        - "alter table {{ this }} alter column id set not null"


IS015693 anticollision code for CLRC6330

Hi @iwanders could you provide the sample code for ISO15693 tags detection with anticollision

Hi @nsrinivas37 , I cannot, because I have not worked on ISO15693 myself.
Other people have been discussing ISO 15693 under issue #4. Please read that thread and comment there, I've reopened that issue for visibility. I'm closing this one to ensure we keep the discussion in one issue.


Learning rate and scheduler for stage 1 training.

Hi, @thuanz123
Thank you for reproducing and open-sourcing ViT-VQGAN using PyTorch. I would like to inquire, is the learning rate fixed at 4.5e-6 during stage 1 training in your code? I noticed that the paper mentions a base_lr of 1e-4, along with linear warm-up and cosine decay.

I tried the scheduler from the paper, turns out to be totally instable. I'd also be curious to know the reason. The architecture seems identical.


Excessive rate limit violations

Copilot.lua is responsible for a disproportionately large number of rate limited completions requests, with the culprit appearing to be copilot-cmp. Users of this combo are at risk of having their accounts flagged for abuse. Recommendations:

Use getCompletions, not getCompletionsCycling. Cycling completions are a resource intensive operation that should only be requested by explicit user action. Spamming it as the user types will exceed the rate limit.
Upgrade Copilot.lua to the latest language server. This will mitigate the damage by recognizing a rate limited server response and backing off for a bit. ✅

Can confirm it's still an issue.

@zbirenbaum Could you provide some assistance or feedback please?
I'd like to avoid breaching copilot terms: https://docs.github.com/en/site-policy/github-terms/github-terms-for-additional-products-and-features#github-copilot
I've read on reddit that some users of other neovim extensions which used copilot got warning emails due to excessive use.

Apologies for missing this, I was out of commission for a bit. Lets definitely get this addressed.
Thank you for the heads up @tpope and the patch @azdanov
I merged #118 please let me know if that wasn't enough to fix it and will address asap.
In the past I tried using getCompletions instead of getCompletionsCycling and ran into some strange behaviors with cmp due to latency I believe, so there may be some followup required for functionality, but keeping users in compliance takes priority for sure.

Thank you for your time! And thanks to @tris203 for the patch, I only posted in this thread :slightly_smiling_face:

Thanks @tris203!! Appreciate the contribution

That should do the trick.


feat: update paysWithFeeOrigin to support asset conversion tx payment pallet functionality [WIP]

TODO:

determine if ExtrinsicPayload can be constructed with MultiLocation assetId

Closing in favor of: https://github.com/paritytech/asset-transfer-api/pull/333


update garden to new nextjs layout

I've followed along with the steps listed here:
https://wiki.dendron.so/notes/FnK2ws6w1uaS1YzBUY3BR/.
Just wanting some verification that things are configured ok here.

I'm getting an error when I run the yarn command: An unexpected error occurred: "https://registry.yarnpkg.com/@dendronhq%2fdendron-11ty: Not found".

Alright, I've made the changes.


Set up your VS code for ES lint

see branch "linting"

Here's the docs with more details. We are using a prettier/ es lint repo from my favorite programmer. The one change from the docs is we want formatOnSave: true (i edited it above to what we want.)
https://github.com/wesbos/eslint-config-wesbos

Ended up doing it manually myself instead of webos. Installed es lint and prettier


HELP ME! eggs thingy

Collecting rainbowstream
Using cached rainbowstream-1.2.7.tar.gz
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 20, in
File "/tmp/pip-build-2b05p4jz/rainbowstream/setup.py", line 31, in
long_description=open("./README.rst", "r").read(),
File "/usr/lib/python3.4/encodings/ascii.py", line 26, in decode
return codecs.ascii_decode(input, self.errors)[0]
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 404: ordinal not in range(128)
----------------------------------------

Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-2b05p4jz/rainbowstream
OS - Arch Linux
Python version 3.4.3

use utf-8 plz.
sed -i "/ja_JP\.UTF-8/{s/^#\?//}" /etc/locale.gen
locale-gen
localectl set-locale LANG=ja_JP.utf8

maybe you need sudo.

For Arch I was able to get around this error by using python2.
pacman -S python2-pip
pip2 install rainbowstream


Function add_a_handset in DECTDevicesAPI class : line2MemberId facultative

Hello,
In class DECTDevicesAPI and function add_a_handset, there is an error.
The line2MemberId parameter is not mandatory in the API (https://developer.webex.com/docs/api/v1/dect-devices-settings/add-a-handset-to-a-dect-network).
Is it possible to make it facultative?
Previous:
    def add_a_handset(self, location_id: str, dect_network_id: str, line1_member_id: str, line2_member_id: str, custom_display_name: str, org_id: str = None):
        params = {}
        if org_id is not None:
            params['orgId'] = org_id
        body = dict()
        body['line1MemberId'] = line1_member_id
        body['line2MemberId'] = line2_member_id
        body['customDisplayName'] = custom_display_name
        url = self.ep(f'locations/{location_id}/dectNetworks/{dect_network_id}/handsets')
        super().post(url, params=params, json=body)

After:
    def add_a_handset(self, location_id: str, dect_network_id: str, line1_member_id: str, line2_member_id: str = None, custom_display_name: str, org_id: str = None):
        params = {}
        if org_id is not None:
            params['orgId'] = org_id
        body = dict()
        body['line1MemberId'] = line1_member_id
        if line2_member_id is not None:
            body['line2MemberId'] = line2_member_id
        body['customDisplayName'] = custom_display_name
        url = self.ep(f'locations/{location_id}/dectNetworks/{dect_network_id}/handsets')
        super().post(url, params=params, json=body)

My bad. Missed the fact that line2MemberId is indeed documented as optional. Made it optional now. To not break existing parameter order parameter custom_display_name had to be made optional as well although it actually is mandatory. If custom_display_name is not passed a ValueError is raised.


Add maskable icons to manifest

This PR adds missing maskable icons to the web app manifest as suggested by Lighthouse.
Issue: https://github.com/GoogleChrome/kino/issues/142

At least we control the padding ;)
See https://web.dev/maskable-icon/

The padding I understand, but why the white background?

Which other color would you use? White seems appropriate to me.
I don't see how no color would match the triangle shape of the icon as well.

My understanding is that the background should be completely filled or have none as there are different icon shapes like square, teardrop, round rectangle, and squircle that the icon can be shown in. If the background is a white circle but the theme color is dark wouldn't the icon be partially white and then outside of the circle it would be the theme color?

@derekherman The icon background is completely filled. DevTools only previews the icon as a circle, because that is the maximum amount of crop that can be applied.

FYI I used https://maskable.app/ to create this maskable icon for kinoweb.dev.

I see. Shouldn’t the background color be the theme color of the app then? It looks best on that dark background.

I'm not sure I like the dark background. Note that Google apps all use white background.

We can leave it with the white background.


Add support  for some new calls to the FossID API client

See individual commits for details.
Theses calls will be used in the future to add snippets information to ORT data model.
See also https://github.com/oss-review-toolkit/ort/issues/3265.

Can you please rebase this onto latest main once?


Buttons don't have a focus state.

I've noticed some elements, especially buttons, don't have a focus state styled. Buttons even have the default browser outline disabled, without anything in its place. Meaning users cannot navigate using the tab key.
Generally, it's not that big of a deal since it can be added easily, but I think some basic visual info should be added by default.
Anyway, thanks for an amazing elements kit!

I would like to work on this task, kindly assign it to me


Works on only some projects

I've several projects. They all use Stack LTS 9.10. Only 1 of them both brings up the Haskell IDE menu AND works.
The other projects only display Haskell IDE menu AFTER I open the <project>.cabal file.
I turned on DEBUG on haskell-ghc-mod and it prints lots of debug info for the one and only project that works.
It prints nothing in the other projects even after I open the <project>.cabal file and the Haskell IDE menu item is displayed.
With zero debug info, I'm at a loss. I'd really appreciate any help in debugging this further.

The other projects only display Haskell IDE menu AFTER I open the .cabal file.

Package activation is deferred until language-haskell grammar is used. So it should load when you open a Haskell source file.
General description of your issue sounds suspiciously similar to grammar conflict. Would you happen to have any Haskell grammar packages besides language-haskell?

I disabled all other Haskell packages except for ide-haskell, language-haskell, haskell-ghc-mod and autocomplete-haskell and it started working.
So next, I enabled one at a time and restarted Atom. And it continued to work even after I enabled the last of them.
Now, I'm even more confused. So, here's my list of packages. Maybe you can tell me which one MAY interfere:

ide-haskell
language-haskell
autocomplete-haskell
haskell-ghc-mod
haskell-grammar
haskell-hoogle
ide-haskell-cabal
ide-haskell-repl

After I posted that above comment, I opened another project and it did NOT work. So I took a different tact. I disabled one at a time and it appears that haskell-grammar is the culprit.
Thanks for the tip. I will close this later in the day after I've had some time to see that it's still working.

FWIW, conflict with haskell-grammar is documented in https://atom-haskell.github.io/core-packages/ide-haskell/#known-conflicts. Note that haskell-grammar is unmaintained since 2015.
Probably need to make it more visible.
Also while on topic, ide-haskell-hoogle might provide a better experience than haskell-hoogle. Disclaimer: I'm biased, since the former is Atom Haskell project, while the latter is not; in my defence, haskell-hoogle is also unmaintained since 2015

I suspect I installed haskell-grammar long before I started using ide-haskell. But I think having the warning about conflicts in other places would be helpful, e.g. in the package's property page in Atom.
I'll change to ide-haskell-hoogle. Thanks again for your help and for working on these great tools.


Flutter SDK not found. Define location with flutter.sdk in the local.properties file.

I get this error:
Flutter SDK not found. Define location with flutter.sdk in the local.properties file.

sdk not found


Embeddings for featureless data

I am trying to use a series of graph convolution layers on a custom dataset (say 400 nodes) for producing embeddings using modularity-based communities as in the Karate Club example. This is just an experiment to see if the graph convolution layers are indeed able to separate the communities. I plan to use some features in conjunction later on. I have encountered a specific error on setting featureless=True in the GCN layers.
Setup for the model:

Consists of 4 GCN layers, with the first having sparse_inputs=True. The activations are relu for the first three and identity activation for the last one. Dimensions for the same are 400 (input)-> 32 -> 16 -> 8 -> 2.
4 placeholders are used (support, features, dropout, num_features_nonzero), which are defined in the exact same way as in train.py.
The adjacency matrix is generated using the nx.adjacency_matrix() method. features is sp.identity(400), num_supports is equal to 1 and supports are calculated in the standard way as in train.py. Preprocessing is also standard.

The issue is that without setting featureless=True in all the 4 layers, this works (although the embeddings produced are not accurate like the Karate Club example), but as soon as this is set, I encounter the following error:
InvalidArgumentError (see above for traceback): Cannot multiply A and B because inner dimension does not match: 400 vs. 8.  Did you forget a transpose?  Dimensions of A: [400, 400).  Dimensions of B: [8,2]
	 [[Node: graphconvolution_39/SparseTensorDenseMatMul/SparseTensorDenseMatMul = SparseTensorDenseMatMul[T=DT_FLOAT, Tindices=DT_INT64, adjoint_a=false, adjoint_b=false, _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_Placeholder_62_0_4, _arg_Placeholder_61_0_3, _arg_Placeholder_60_0_2, graphconvolution_39_vars/weights_0/read)]]

The issue seems to be with the dot function for the 1st layer according to what I understand. Are the features supposed to be changed in some way for featureless=True?

Did you solve this? I'm now having the same error

I am also interested in accomplishing the featureless embedding. was this fixed?


Migrate playbook/validation/product-design-sprint/prototype

https://thoughtbot.com/playbook/validation/product-design-sprint/prototype

[x] Move content we want to preserve into new guide
[x] Add redirect to thoughtbot.com

There was no content that wasn't already migrated over that was worth preserving. The redirect is here https://github.com/thoughtbot/thoughtbot.com/pull/2674


ApplePay.swift 'requiredShippingContactFields' is only available in iOS 11.0 or newer

As you can see in the screenshot that my minimum deployment target is 12.4 which is greater than 11 but I am still receiving these field errors.
Can someone please help me to understand and fix this issue?
Thank-you

So I solved the issue by myself
You just have to upgrade the IOS Deployment Target for the react-native-apple-pay library
Pods => Scroll Down Targets  List => Select Library (i-e react-native-apple-pay) => Build Settings Tab and type target in search


Discussion: determining about what DANE implementation we care about.

Today, I was alerted by Daniel Micay that even though we list DANE as a security requirement, we listed some providers which do not support DANE.  After talking with him more,  he pointed out that he was talking about DANE for outgoing emails.
During our testing, we always used hardenize.com to test for DANE support, but missed the fact this can only test for incoming mails, not outgoing mails. for testing outgoing mail, we will be using havedane.net .
We now have three things to find out and discuss:

find out what provider supports what implementation of dane (dane for incoming, outgoing or both)

discuss whether we require dane on both incoming and outgoing, or only one of these two.

Decide how to proceed. we already know that we will need a PR to clarify in the requirements about which implementation of DANE we care about, but we should also reach out to the listed providers.

Another point of discussion is how we like to see non DANE domains handled (if I email a mail address on a domain without DANE (so not invalid , just absent), should the mail provider still deliver the email?)

This is a pretty inclusive list of what is supported by which providers:
https://dismail.de/serverlist.html

This is a pretty inclusive list of what is supported by which providers:
https://dismail.de/serverlist.html

interesting resource, but it does seem a bit out of date. (for example is shows that tutanota does not support dmarc)

Last update was 2020-01-02 and they allow you to send them corrections.

I'm thinking verification to invalid may not actually be that important to us.
We do require MTA-STS as well which is far more common these days. The most important part is incoming DANE verification (for servers which can't check MTA-STS policy), or for servers which haven't got a MTA-STS policy published, but do have DANE.
Using that havedane.net I was able to get these results
ProtonMail
Wed 13 May 2020 03:54:40 UTC
Results
✔ Email to non-DANE domain delivered.
✔ Email to DANE domain delivered.
❌ Email to domain with invalid DANE delivered.
Mailbox.org
Wed 13 May 2020 04:18:40 UTC
✔ Email to non-DANE domain delivered.
✔ Email to DANE domain delivered.
✔ Email to domain with invalid DANE not delivered.
Soverin
Wed 13 May 2020 04:36:00 UTC
✔ Email to non-DANE domain delivered.
✔ Email to DANE domain delivered.
❌ Email to domain with invalid DANE delivered.
Disroot
Wed 13 May 2020 04:30:50 UTC
✔ Email to non-DANE domain delivered.
✔ Email to DANE domain delivered.
❌ Email to domain with invalid DANE delivered.
Tutanota
Tue 12 May 2020 18:30:00 UTC
✔ Email to non-DANE domain delivered.
✔ Email to DANE domain delivered.
✔ Email to domain with invalid DANE not delivered.

Both MTA-STS and DANE are designed to be checked by the client i.e. the side that's sending mail. So, setting up DNS records, etc. for MTA-STS and DNSSEC + TLSA allows others to send mail to you securely but doesn't secure TLS when you're sending mail.
It's important to have both MTA-STS and DANE due to the fact that some providers only provide one of them. It's not enough to have only DANE because then you can't send and receive email with Gmail securely. It's not enough to only have MTA-STS because then you can't send and receive email with most EU services securely.
There's a very serious issue with most services in that they do not fully implement these features. It's common to only implement MTA-STS and DANE for incoming mail since that's what scans will check.
It's also common to only implement SPF + DKIM + DMARC for sending mail since that's what scans will check. It's very important for mail providers to enforce DMARC for incoming mail since otherwise mail can be spoofed... but many don't do that. DKIM without DMARC is pretty useless since it doesn't have a way to enforce that all mails are signed as you would expect. They can just be optionally signed and refer to the DNS record for the key...

Also important to note that havedane.net only verifies that checking TLSA records is enforced. It does not verify if they do proper local DNSSEC verification. I think it's a common issue to enable DANE verification without setting up a validating local resolver like unbound. It's typical to rely on remote recursive resolvers via the unencrypted/unauthenticated DNS protocol which is insecure but can't really be detected by a test. Could detect if there's no DNSSEC validation at all, but there probably is by an upstream server. It would be possible to make a test with a 2nd invalid test case where the DANE record is valid but DNSSEC is not valid, which could at least detect cases with no DNSSEC verification.

You would need a similar test case like havedane.net for verifying that MTA-STS is enforced for sending email. I'm curious how many providers other than Gmail actually enforce MTA-STS. Having the DANE / MTA-STS setup accomplishes nothing if the sender doesn't enforce them.

Thank you for your comments Daniel, they are very much appreciated. I agree personally that DANE for outgoing mail is important. especially since it will take a while before MTA-STS will become a more common feature like STARTTLS. Would you happen to know a way of detecting whether DANE is done with a local resolver instead of an remote one.
Also, would the problem be less severe if one were to use DoH or DoT to connect to their remote resolver?
About MTA-STS, I will do some digging today to find out what methods we can use to test MTA-STS enforcement.

Also, would the problem be less severe if one were to use DoH or DoT to connect to their remote resolver?

Yes, it still trusts the resolver, but it wouldn't be vulnerable to MITM. Ideally, mail servers have unbound to provide local DNSSEC verification and forwarding to a caching resolver via DoT to provide improved privacy. A sample unbound.conf:
server:
    qname-minimisation: yes
    auto-trust-anchor-file: "/var/lib/unbound/root.key"
    tls-cert-bundle: /etc/ssl/certs/ca-certificates.crt

forward-zone:
    name: "."
    forward-tls-upstream: yes
    forward-addr<EMAIL_ADDRESS>    forward-addr<EMAIL_ADDRESS>
The system needs to be set up to use <IP_ADDRESS> as the DNS resolver.

About MTA-STS, I will do some digging today to find out what methods we can use to test MTA-STS enforcement.

Host a mail server with an MTA-STS policy and a self-signed certificate. If a mail provider can send mail to it, they don't enforce MTA-STS. I expect that this is the case for most providers that have an MTA-STS policy for others to use. It's very common with DANE but I expect it's even more common with MTA-STS. For example, Postfix supports a configuration option for enforcing DANE itself with the help of a local DNSSEC validating resolver but MTA-STS has to be done with an external service and it's more painful. So, people using Postfix without DANE verification set up have little excuse but MTA-STS is a whole other service just like enforcing DMARC via opendmarc - need to set up a separate service, configure your MTA to use it and configure that service in a proper enforcing mode without bypasses. There are common issues with DMARC enforcement bypasses too...

So, people using Postfix without DANE verification set up have little excuse but MTA-STS is a whole other service just like enforcing DMARC via opendmarc

(they do have to set up DNSSEC but that's simpler and covers more than just DANE - MTA-STS could be bypassed on the first connection without DNSSEC too)


[Stepification] Research write up for cluster management once a cluster is up

I have no idea what this is about but it sounds old and imported from Jira.  Closing unless someone has a reason to do this.


Extra check for babel-loader

Ran into this issue: https://github.com/prateekbh/babel-esm-plugin/issues/28

@ForsakenHarmony or @prateekbh can this bugfix be published to NPM as well please? I'm using it in a project and I'm unable to deploy with the current version

will do today


options add default?

can we new a default options in options and use it like this
// New creates a new distributed locker on a given key.
func New(client RedisClient, key string, opts *Options) *Locker {
	var o Options
	if opts != nil {
		o = *opts
     	        o.normalize()
	} else {
                o.default() // defalult options is init and a singleton instance
        }

	return &Locker{client: client, key: key, opts: o}
}

I think it can save some memory and cpu time

@mofadeyunduo I don't see a difference to what normalize() does. It certainly doesn't add any more allocations or CPU time.


SetupDiGetDeviceInterfaceDetail() throws MashalDirectiveException "Pointers cannot reference marshaled structures"

Hi, I just ran into this issue. Let me know if any other info would be helpful!
Actual behavior
Calling SetupDiGetDeviceInterfaceDetail() in a net48 project results in:
Unhandled Exception: System.Runtime.InteropServices.MarshalDirectiveException: Cannot marshal 'parameter #3': Pointers cannot reference marshaled structures.  Use ByRef instead.
   at Windows.Win32.PInvoke.SetupDiGetDeviceInterfaceDetail(HDEVINFO DeviceInfoSet, SP_DEVICE_INTERFACE_DATA* DeviceInterfaceData, SP_DEVICE_INTERFACE_DETAIL_DATA_W* DeviceInterfaceDetailData, UInt32 DeviceInterfaceDetailDataSize, UInt32* RequiredSize, SP_DEVINFO_DATA* DeviceInfoData)

Expected behavior
It should be callable. It worked in version 0.3.49-beta.
Repro steps

NativeMethods.txt content:

SetupDiGetDeviceInterfaceDetail

NativeMethods.json: Not present

Any of your own code that should be shared?

Program.cs:
using System;
using Microsoft.Win32.SafeHandles;
using Windows.Win32.Devices.DeviceAndDriverInstallation;

namespace ConsoleApp2
{
    internal class Program
    {
        static void Main(string[] args)
        {
            unsafe
            {
                uint size = 0;
				// The argument values are not relevant here.
                Windows.Win32.PInvoke.SetupDiGetDeviceInterfaceDetail(new SafeFileHandle(IntPtr.Zero, false), new SP_DEVICE_INTERFACE_DATA(), null, 0, &size, null);
            }
        }
    }
}

Context

CsWin32 version: 0.3.106 (it worked in 0.3.49-beta)
Win32Metadata version: not set by project
Target Framework: net48
LangVersion 8.0

Thanks for reporting. The char type referenced in the DevicePath field is causing .NET Framework to consider SP_DEVICE_INTERFACE_DETAIL_DATA_W to be a managed type. Changing it to ushort gets it to work. I'll investigate what the best fix is for CsWin32 here.

I tested this and it appeared to work. I think it would be safe.
-internal global::Windows.Win32.VariableLengthInlineArray<char> DevicePath;
+internal global::Windows.Win32.VariableLengthInlineArray<char, ushort> DevicePath;

and
-    internal struct VariableLengthInlineArray<T>
+    internal struct VariableLengthInlineArray<T, T2>
 where T : unmanaged
+where T2 : unmanaged
     {
-        internal T e0;
+        internal T2 e0;

         internal unsafe ref T this[int index]
         {
             [UnscopedRef]
             [MethodImpl(MethodImplOptions.AggressiveInlining)]
-            get => ref Unsafe.Add(ref this.e0, index);
+            get => ref Unsafe.Add(ref Unsafe.AsRef<T>(Unsafe.AsPointer(ref this.e0)), index);
         }

This works by changing the field from char to ushort, while still presenting almost the same char based API. In fact we could probably change the field e0 itself to private and add a ref T property getter to even present char there.
@tannergooding @AaronRobinsonMSFT  do you have any ideas here?

The char type referenced in the DevicePath field is causing .NET Framework to consider SP_DEVICE_INTERFACE_DETAIL_DATA_W to be a managed type

A nit on this statement. The concept of "managed" or "unmanaged" is historically a C# ism and not defined by the runtime. The runtime deals with the terms blittable and non-blittable. In .NET the bool and char primitives are non-blittable but are considered "unmanaged" by C#. This is friction we are slowly trying to fix in .NET 7+, it will be a long process.

@tannergooding @AaronRobinsonMSFT do you have any ideas here?

Using short, as you discovered, is what I would recommend.

In .NET the bool and char primitives are non-blittable

Do you mean in .NET Framework? .NET 8 didn't throw an exception in the OP's repro -- only .NET Framework did. That seems to suggest that .NET 8 considers char to be blittable.

.NET 8 didn't throw an exception in the OP's repro -- only .NET Framework did. That seems to suggest that .NET 8 considers char to be blittable.

Hmmm. That is surprising. Are you using DisableRuntimeMarshalling?

No.

Hmmm. That is surprising. Are you using DisableRuntimeMarshalling?

I can confirm that it will not throw even the [assembly: DisableRuntimeMarshalling] present in .NET 8


[Storage] [Typing] [File Share] Typed _shared_access_signature.py

As usual, this typing PR includes:

Reordering of imports
Updated docstring type annotations to match current style
Typed function parameters and return types
Removed unnecessary # type: ignore statements

:shipit:

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-storage-file-share


Does datadog-agent support Kubernetes on containerd?

Describe what happened:
I tried to run the datadog-agent on GKE using cos_containerd but got the following error.
$ kubectl logs datadog-agent-l2ds5
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] 01-check-apikey.sh: executing...
foreground: warning: unable to spawn /var/run/s6/etc/cont-init.d/01-check-apikey.sh: Permission denied
foreground: fatal: unable to wait for /var/run/s6/etc/cont-init.d/01-check-apikey.sh: No child process
s6-svscan: warning: unable to exec finish script .s6-svscan/finish: Permission denied
s6-svscan: warning: executing into .s6-svscan/crash
s6-svscan: fatal: unable to exec .s6-svscan/crash: Permission denied

I believe this is because the yaml file I applied is from the document and it has
        volumeMounts:
          - name: dockersocket
            mountPath: /var/run/docker.sock

Of course there's no docker.sock on the containerd node so here's my question.
Is there any way to run the agent on containerd?
Describe what you expected:
The agent works on Kubernetes w/ containerd
Steps to reproduce the issue:
Make a kubernetes node with containerd and run an agent pod on it
Additional environment details (Operating System, Cloud provider, etc):

GKE

v1.13.5
COS with containerd

The Helm chart supports it, so I'd say yes: https://github.com/helm/charts/tree/master/stable/datadog#cri-integration

You can found more information about how to deploy the datadog-agent using helm here: https://docs.datadoghq.com/agent/kubernetes/helm

Based on what the helm chart and datadog-agent source code are doing, it seems that adding the following params should work for it.
      containers:
        env:
        - name: DD_CRI_SOCKET_PATH
          value: "/run/containerd/containerd.sock"
        volumeMounts:
        - name: runtimesocket
          mountPath: /run/containerd/containerd.sock
      volumes:
      - hostPath:
          path: /run/containerd/containerd.sock
        name: runtimesocket

@clamoriniere I am not using Helm for my environment and not planning so. But thanks!

@inductor not problem :)
What you proposed: DD_CRI_SOCKET_PATH  + volumePath should be enough to activate the containerd support.

This is my yaml but still gets the same issue D:
apiVersion: extensions/v1beta1
kind: DaemonSet
metadata:
  name: datadog-agent
spec:
  template:
    metadata:
      labels:
        app: datadog-agent
      name: datadog-agent
    spec:
      containers:
      - image: datadog/agent:latest
        imagePullPolicy: Always
        name: datadog-agent
        ports:
        - containerPort: 8125
          name: dogstatsdport
          protocol: UDP
        - containerPort: 8126
          hostPort: 8126
          name: traceport
          protocol: TCP
        env:
        - name: DD_API_KEY
          value: xxxxxxxxxxxxxxxxxxxxxxxxxxx
        - name: KUBERNETES
          value: "true"
        - name: DD_KUBERNETES_KUBELET_HOST
          valueFrom:
            fieldRef:
              fieldPath: status.hostIP
        - name: DD_LOGS_ENABLED
          value: "true"
        - name: DD_LOGS_CONFIG_CONTAINER_COLLECT_ALL
          value: "true"
        - name: DD_APM_ENABLED
          value: "true"
        - name: DD_APM_ANALYZED_SPANS
          value: "flask|flask.request=1"
        - name: DD_TAGS
          value: "<YOUR_DD_TAGS>"
        # Disable sending to datadog Infrastructure
        - name: DD_ENABLE_PAYLOADS_EVENTS
          value: "false"
        - name: DD_ENABLE_PAYLOADS_SERIES
          value: "false"
        - name: DD_ENABLE_PAYLOADS_SERVICE_CHECKS
          value: "false"
        - name: DD_ENABLE_PAYLOADS_SKETCHES
          value: "false"
        - name: DD_CRI_SOCKET_PATH
          value: "/run/containerd/containerd.sock"
        resources:
          requests:
            memory: "256Mi"
            cpu: "200m"
          limits:
            memory: "256Mi"
            cpu: "200m"
        volumeMounts:
        - name: runtimesocket
          mountPath: /run/containerd/containerd.sock
          readOnly: true
        - name: procdir
          mountPath: /host/proc
          readOnly: true
        - name: cgroups
          mountPath: /host/sys/fs/cgroup
          readOnly: true
        livenessProbe:
          exec:
            command:
            - ./probe.sh
          initialDelaySeconds: 15
          periodSeconds: 10
      volumes:
      - hostPath:
          path: /run/containerd/containerd.sock
        name: runtimesocket
      - hostPath:
          path: /proc
        name: procdir
      - hostPath:
          path: /sys/fs/cgroup
        name: cgroups

Hi @inductor
I had the same issue as you and I resolved it by mounting these to have the right permissions:
volumeMounts:
          - mountPath: /var/run/s6
            name: s6-run
          - mountPath: /host/var/run
            name: var-run
            readOnly: true

volumes:
        - hostPath:
            path: /var/run
            type: DirectoryOrCreate
          name: var-run
        - hostPath:
            path: /var/run/s6
          name: s6-run

Let me know if this works for you!
Simon

Thanks @Simwar , I'll try it out!

@inductor how did it go? All set?

@irabinovitch hi, everything works fine thanks l! I’m closing this issue

I am trying to achieve the same but I am getting the following error:
failed to dial "/var/run/containerd/containerd.sock": connection error: desc = "transport: error while dialing: dial unix /var/run/containerd/containerd.sock: connect: connection refused
Any inputs?
@inductor @Simwar

@deepaksrinath
Are you using containerd with eks?
If so, this may be helpful: https://github.com/DataDog/helm-charts/issues/338

@cw-sakamoto That link was helpful. Thank you!!!


when i was training,i met the opencv errors

Traceback (most recent call last):
File "/media/ysy/0ba6c187-2e6e-4b2e-b6af-f98d1ff3acee/ychengyu/anaconda3/envs/py36/lib/python3.6/site-packages/torch/multiprocessing/spawn.py", line 19, in _wrap
fn(i, *args)
File "/media/ysy/0ba6c187-2e6e-4b2e-b6af-f98d1ff3acee/ychengyu/code/Higher-HR/tools/dist_train.py", line 278, in main_worker
final_output_dir, tb_log_dir, writer_dict, fp16=cfg.FP16.ENABLED)
File "/media/ysy/0ba6c187-2e6e-4b2e-b6af-f98d1ff3acee/ychengyu/code/Higher-HR/tools/../lib/core/trainer.py", line 35, in do_train
for i, (images, heatmaps, masks, joints) in enumerate(data_loader):
File "/media/ysy/0ba6c187-2e6e-4b2e-b6af-f98d1ff3acee/ychengyu/anaconda3/envs/py36/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 637, in next
return self._process_next_batch(batch)
File "/media/ysy/0ba6c187-2e6e-4b2e-b6af-f98d1ff3acee/ychengyu/anaconda3/envs/py36/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 658, in _process_next_batch
raise batch.exc_type(batch.exc_msg)
cv2.error: Traceback (most recent call last):
File "/media/ysy/0ba6c187-2e6e-4b2e-b6af-f98d1ff3acee/ychengyu/anaconda3/envs/py36/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 138, in _worker_loop
samples = collate_fn([dataset[i] for i in batch_indices])
File "/media/ysy/0ba6c187-2e6e-4b2e-b6af-f98d1ff3acee/ychengyu/anaconda3/envs/py36/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 138, in
samples = collate_fn([dataset[i] for i in batch_indices])
File "/media/ysy/0ba6c187-2e6e-4b2e-b6af-f98d1ff3acee/ychengyu/code/Higher-HR/tools/../lib/dataset/COCOKeypoints.py", line 64, in getitem
img, anno = super().getitem(idx)
File "/media/ysy/0ba6c187-2e6e-4b2e-b6af-f98d1ff3acee/ychengyu/code/Higher-HR/tools/../lib/dataset/COCODataset.py", line 121, in getitem
img = cv2.cvtColor(img, cv2.COLOR_BGR2RGB)
cv2.error: OpenCV(3.4.1) /io/opencv/modules/imgproc/src/color.cpp:11115: error: (-215) scn == 3 || scn == 4 in function cvtColor
This is an error report. How can I solve it？thanks

Could you check the image path really contains image?

Could you check the image path really contains image?

Thank you for your reply. I'd checked  my data set which missed some of the pictures. Now the problem has been solved. Thanks

在運行python tools/valid.py
--cfg experiments/coco/higher_hrnet/w32_512_adam_lr1e-3.yaml
TEST.MODEL_FILE models/pytorch/pose_coco/pose_higher_hrnet_w32_512.pth
出現了[<EMAIL_ADDRESS>global loadsave.cpp:244 cv::findDecoder imread_('data/coco\images\val2017\000000397133.jpg'): can't open/read file: check file path/integrity
Traceback (most recent call last):
File "tools/valid.py", line 220, in
main()
File "tools/valid.py", line 153, in main
for i, (images, annos) in enumerate(data_loader):
File "D:\anaconda\envs\pointnet\lib\site-packages\torch\utils\data\dataloader.py", line 435, in next
data = self._next_data()
File "D:\anaconda\envs\pointnet\lib\site-packages\torch\utils\data\dataloader.py", line 475, in _next_data
data = self._dataset_fetcher.fetch(index)  # may raise StopIteration
File "D:\anaconda\envs\pointnet\lib\site-packages\torch\utils\data_utils\fetch.py", line 44, in fetch
data = [self.dataset[idx] for idx in possibly_batched_index]
File "D:\anaconda\envs\pointnet\lib\site-packages\torch\utils\data_utils\fetch.py", line 44, in
data = [self.dataset[idx] for idx in possibly_batched_index]
File "D:\kinect\HigherHRNet-Human-Pose-Estimation-master\HigherHRNet-Human-Pose-Estimation-master\tools..\lib\dataset\COCODataset.py", line 122, in getitem
img = cv2.cvtColor(img, cv2.COLOR_BGR2RGB)
cv2.error: OpenCV(4.7.0) D:\a\opencv-python\opencv-python\opencv\modules\imgproc\src\color.cpp:182: error: (-215:Assertion failed) !_src.empty() in function 'cv::cvtColor'


Handle stacked ensembling for partial dependence fast mode

Once #3779 is implemented, fast mode will not work in pipelines that use stacked ensembling.
This was not done due to the added complexity of handling it, but we should be able to handle stacked ensembling for partial dependence fast mode by recreating each individual inputted pipeline, getting their original X_t, transform the single column, and generate predictions, thus recreating the dataframe that gets passed into the stacked ensembler estimator.
This should be implemented by adding a handle_partial_dependence_fast_mode method in StackedEnsembleBase to handle the ensembler and change _can_be_used_for_fast_partial_dependence to True.
Note: I expect this to mean we have to change the parameters and return values of _handle_partial_dependence_fast_mode, because handling this component means we need to _get_cloned_feature_pipelines and _transform_single_feature for each sub-pipeline. We should make sure the implementation is still functional for future components that may need to use this method.

Can maybe use make_pipeline to recreate sub pipelines?


Improve JobArg/JobKwarg

Is your feature request related to a problem? Please describe.
Currently have 3 possible fields. Should have one field and be implemented.

v2022.3.0


[BUG] timeouts on imagepull of large container

Based on a customer call, no specific logs or errors, but the verbal indication is it takes a long time to pull and start large containers, probably due to timeouts within the pull process.
Might be worth increasing this parameter on kubelet:
--image-pull-progress-deadline duration
--
  | If no pulling progress is made before this deadline, the image pulling will be cancelled. This docker-specific flag only works when container-runtime is set to docker. (default 1m0s)

Reference: https://openai.com/blog/scaling-kubernetes-to-2500-nodes/ on pulling larger images.

Closed per #596


Add test cases for unsat dependencies in workspace members

Adding some test cases to help inform the work in #6066

More test case suggestions would be greatly appreciated.


implement actionTransformer

I sometimes use immtuables in my actions as well as my state, so I added an actionTransformer option that transforms the action before logging.

Hi, thanks for idea. Will release in 1.1.0 tomorrow!

@theaqua Whoa, I just realized that I accidentally made it print xAction to the console when I created the xAction variable to comply with the strict eslint config.


Investigate the ask price metric to ensure that it's not a client issue

Overview
During the Bittrex CELOEUR listing we had a period without any asks in the order book. This caused the graph to look like this:

We should make sure that we weren't reporting an average from the client and that we saw that because the dashboard was taking the average.

Is this just some error handling where if there are no asks, we take the maximum of the bid prices, and if there's no bids, we take the minimum of the asks?


Fixed some bugs

Fixed some bugs with having events that have the same date.
Known bug:

When user upload a csv using dates, then the program currently doesn't check whether there are any repeated dates.

I also fixed a bug that kept user from uploading a csv file after they had made changes and saved using excel.


Table GA3 does not exist

Hi again my friend!
I have managed to set up all the steps you indicate in the notebook but I am stuck in step 4: backfill GA3 data
My question is the following: is it necessary to have a table created in bigquery with the GA3 data version?
If yes, is it possible to export the GA3 data to a bigquery table? because I have only seen that it can be done with the GA4 version.
Thank so much :)
Best.
Rai

Hey @raismartup,
If I understand your question correctly: no, it's not necessary to have your GA3 data in BigQuery before running step 4. You do not need to manually create anything in BigQuery - by following the steps in the notebook, the tool will create the BigQuery table and fill it with data. The GA3 data will be downloaded from the API and merged with the the GA4 data.
Does that help you to move forward?

Hi!
thanks for your quick response :)
I just realized that's exactly what it does, but I didn't know exactly what to put in the ga3_view_id parameter and I know that you have to put the ID of the project you want to fill with the GA4 table.
Also, the goal_metric parameter to want to take all the metrics, is it enough to leave it like that? or you have to put another value that indicates that you want to bring you everything?
On the other hand (taking the opportunity to reply to this thread) there is a limit of history that you can join, isn't it? 100.000 rows for example??
Thanks :)

@raismartup Yes, ga3_view_id must be a valid ID from the rightmost (Views) column in the Google Analytics interface. You can see an example here:

The default goal_metric is ga:goalCompletionsAll. You can leave it as that to pull all of the goal completions as a single total.
I'm not aware of any history limit. You can set the pull_start_date to designate how many days you want to look back.

Thank so much for you help! :)


Enable switch event support

Allow obj.on('event1 event2', function(){ ... })

I think this would be nice if we would pass an event object to listeners so they could figure out what type of event is being fired.
The way the events work now I fear this feature could lead to confusion some events have similar params and others different ones. I'd rather keep it simple for now.
Thanks for the contribution and sorry for answering so late!


Niche incompatibility with axum + deadpool-diesel

I'm not sure if this belongs here but I have to start somewhere. I've never encountered an issue like this before in rust, but it seems I can work around it by not using this crate.
I have a minimal reproducible example here: https://github.com/manstie/axum-typetag
See the readme for error details.

Your issue is unrelated to this crate. You could have figured this out by removing all use of typetag from your program.
- let payload = serde_json::to_value(queueable)?;
+ drop(queueable);
+ let payload = serde_json::Value::Null;

(And removing typetag::serde attributes.) The error is the same.
You are missing a Send bound. Holding queueable across an await point requires it to be sendable across threads.
- let queueable: Box<dyn Queueable> = Box::new(example.clone());
+ let queueable: Box<dyn Queueable + Send> = Box::new(example.clone());


Release new version

Hi @nox ,
Is it possible to release new version of the crate to crates.io?
I would like to release few private crates to crates.io but they are depending on #69
Please let me know if I can help with it somehow.
Have a great week,
MaxV

I'll do it when I'll do it.


Split ips in x-forwarded-for

What
X-forwarded-for field in kibana is a string of comma separated ips. We want to be able to easily filter by ip. This PR converts the field into an array of ips
How to review

Code review
Check that you can filter this field by individual ips

After merging
Copy config/logit/output/generated_logit_filters.conf and paste in Logit --> configure logstash pipelines, replacing the one in the editor. Test configuration --> apply
Do this for prod and staging
This will restart Kibana, so let people know!

🚨⚠ Please do not merge this pull request via the GitHub UI ⚠🚨

I might be missing something, but what difference does this make? It seems to look quite a bit like before.
I think what we probably need is to generate a new field based on x_forwarded_for which only has the "real ip". Perhaps called real_ip or something. But there's a gotcha in that we should make sure we only do it when we trust the trailing IPs in the field.
I should possibly have put this in the story but I didn't want to be accused of "solutionizing".

(We had a call and I showed risicle that it did make a difference XD)

I think my remaining concern is that we'll end up with thousands of e.g. 10.*.*.* addresses (and our LB's IPs) in the dataset, drowning out the "real IPs". And I don't know how effectively we can "ignore" the intermediate IPs with the tools kibana gives us if we want to e.g. build a ~histogram~ rank-frequency plot of IPs - does "excluding" <IP_ADDRESS> actually result in all log messages that include that IP being thrown out? Because that's not we'd want - we'd want to just not include those IPs in the ~histogram~ rank-frequency plot.
Basically, this leaves a lot of the work up to kibana, and I don't know if kibana's powerful enough to manage that.

I think I see your point. However, in terms of the problem this is intended to solve, it is still a good solution, or at least good progress. We'll be able to identify any IPs with a "high" number of requests much quicker than we do right now

I think my remaining concern is that we'll end up with thousands of e.g. 10...* addresses (and our LB's IPs) in the dataset, drowning out the "real IPs"

We could use the Logstash CIDR filter to filter out <IP_ADDRESS>/8, and things like CloudFront IP addresses (which we could pull in when we compile the filters)

Instead of filtering out what we don't want, I've added a field client_ip that contains the first ip in the x_forwarded_for field.
I've just added the change to dev in case you want to have a look (gorouter.header.client_ip)
How does that sound, @AP-Hunt @risicle?

I think that's the approach I originally imagined, and it's what I was talking about when I mentioned "safe" ways of interpreting x-forwarded-for. If we simply use the leftmost value we open ourselves to spoofing: https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-Forwarded-For#parsing. The "full fidelity" approach described in that article might be a bit much to implement in logstash, but a slightly simpler approach of always picking the third-from-right (?) IP might work in lieu of that.

Actually come to think I'm not sure that would be flexible enough to deal with traffic that is/isn't coming via cloudfront. Which makes me think we might also have to exclude all cloudfront IPs from this field too.

Note there is actually a proper plugin for this https://github.com/IngaFeick/logstash-filter-forwarded but I don't think we're going to go about getting logit to install/maintain it for us.

Having looked deeper at how practical it is for us to do this "properly" in logit (it isn't, especially if we consider that tenants may be pointing their own CDN at us and expecting us to pass on their x-forwarded-for headers) and noticing that handling of the RTR logs also does basically exactly this (the most recent solution, see line 565 of the combined file onwards), I think we're going to have to go with this. I think I'd be happier if we called the resultant field likely_remote_addr, but :shrug: .
As long as we still have the split, multi-indexed x_forwarded_for field, it should still be possible to dig out a spoofing intermediary with a little effort.

For clarification, when I mentioned tenants' external CDNs it's because I was entertaining the idea that we should scrub all x-forwarded-for headers that came into the system and solve the problem once and for all, for our tenants too, but this would annoy tenants who are using foobarcdn and are expecting to get x-forwarded-for through it.

Pushed a version which is as much a copy of the RTR field's handling as possible, but fundamentally the same as Cat's most recent version.


Mass Connectivity Outreach

Related to work as part of a temporary committee/working group thing
What are other departments doing if a massive increase in remote connectivity is required?

Follow up with R&D