text:
Users SBAT to test a transaction on the wallet without losing money

Expected Behavior
There should be a way for users to test a transaction on Alfajores on the wallet, and get the money back. This would look something like "Test out a transaction.". A toast would also have to be shown, something like "Note: You may lose some money in gas fees."
The implementation would involve using the current escrow contract to send a transaction and also reclaim it instantly, showcasing both a send and a received transaction in the activity feed, like so:
in a TS function,

create a new invite code/ temp wallet address
use escrow's transfer fn, with the expiry seconds field set to 1 second
reclaim the same payment using the invite code
src/invite/saga's sendInvite function and src/escrow/saga would be a great place to see how inviting and escrowed payments work.

Current Behavior
Currently the user would have to send a real payment transaction, meaning they would lose Testnet money.

"Celo Default Recipient" added in dev envs


text:
Finalize the automation of the smart contract upgrade process

[ ] Finalize #3893 and #4277 .
[ ] Script to verify a contracts directory has contracts that are deployed on-chain (deployed bytecode, modulo metadata, matches) (#4496)
[ ] Use the above in CI to verify changes against master.
[ ] Document and make scripts easily usable externally for third party due diligence.
[ ] Script to generate governance proposal data and deployment transactions from diff of contracts' directories.

Thinking about sequencing here, the following things need to happen before we can kick off the first contract upgrade:

Finalize automated tooling

Check build directory matches deployed contracts (@m-chrzan)
Check implied version deltas match actual (@asaj)
CI integration (???)
Automate governance proposal creation (???)

Finalize desired contract changes

DowntimeSlasher (@gastonponti)
Exchange (https://github.com/celo-org/celo-proposals/pull/26) (???)
Version numbers (@asaj)

Complete third party audit(s)

Anything else?

source: gharchive/issue

authors: asaj nambrot

repo: celo-org/celo-monorepo

license_type: permissive

license_source: github-api


text:
[WalletConnect] DApps should be able to submit multiple transactions at once

This should already be supported by the WalletConnect protocol. We just need to handle an array of actions in Valora.

closing since we decided it would only handle one transaction at a time

@aslawson this was around a DApp sending multiple TXs, then the user having to approve each one. What we want is the following flow

DApp sends 2 TXs
User approves both
DApp continues

We don't want

DApp sends a TX
User approves
DApp sends another TX
User Approves

This makes it extremely clunky to interact with DApps. The user will still only be able to see one action at a time.

Gotcha -- What do other wallets do? Or does wallet connect suggest any best practices here? I'm not finding much in the documentation.
I was under the impression most wallets perform the latter example ^  but it was up to the dapp developer to wrap everything into a single transaction (which we should encourage).
I definitely agree if the standard is to send them one at a time, the experience becomes clunkier with mobile apps.

This also now seems related to the discussion on accessing offchain data, no? How do they access multiple contact's photos/names at once?

The best example is the transferAndCall route. We approve some contract to take tokens from us, then do the action (e.g. Uniswap). Here we can't do a single TX I believe, but we should be able to approve these two actions in succession on Valora without having to wait for the first TX to be mined.
Re. accessing off chain data, imagine a dApp wants to read a users encrypted profile picture. To do this it

Requests the shared secret (1st request to Valora)
Computes the storage path and requests file
Decrypts key (2nd request to Valora) and then decrypts profile picture.

Currently this would look like 2 confirmable requests, if we wanted to request a users name as well, 4 requests. To display the names and pictures of all their friends, 4 * N requests. So maybe here we would want to not prompt on Valora for decrypt and computeSharedSecret calls? We'd put the onus on users to connect to DApps they trust.
We could also go the route of a celo_offchainRead, Valora would handle the above steps and just transmit back decrypted data. Not sure if this would work for binary payloads like pictures though, so maybe not worth thinking about.

Gotcha -- so transferAndCall use cases are really the only ones that couldn't be wrapped by another call function since it requires two block transactions right now.
Is this more a decision for your current implementation work on this or is this additional work to support multiple transaction prompts which we can have anyone else take on?

Pretty sure this works already

authors: AlexBHarley aslawson


text:
Update trail.gdshader

work around for the shader compiling error:
"non constant expression in initialization"

Looks like it works. Thanks!

source: gharchive/pull-request

authors: bitdream celyk

repo: celyk/GPUTrail

license: MIT


text:
Application vulnerable to XSS/injection of JavaScript

Use ngSanitize to help combat this

A parent could create a child with a fancy name like
<script>doBadStuff();</script>

and submit results on leaderboards, etc. Admin pages may be vulnerable if the views were to display usernames, etc.

authors: JulioChavez khaledhassan

license: mit

license_source: bigquery


text:
[Feature Request] zip raw report

original: https://github.com/cenfun/monocart-reporter/issues/149
The idea is
{
  reports: [
    ["raw", {
      zip: true
    }]
  ]
}

Hi @cenfun, I'm not sure that would help much. The original request was if the code coverage could be consolidated at the end of a run. So that instead of many raw code coverage files we could have one or a few. Would that be possible?
Zipping wouldn't help in my scenario, note that I already manually zip the final output from monocart reporter.  The place where I think compressing would help is on cenfun/monocart-reporter/issues/147, which if done would actually also include what you are proposing since it would zip the entire output from monocart-reporter (including raw code coverage files of course).

The original request was if the code coverage could be consolidated at the end of a run. So that instead of many raw code coverage files we could have one or a few. Would that be possible?

Note that if this is not possible, then it's ok, I have no idea how complex that would potentially be...

Thanks for the clarification.
I have implemented the capability of zip in this commit. Perhaps it is possible to implement merge, Let me do some research:
{
  reports: [
    ["raw", {
      merge: true,
      zip: true
    }]
  ]
}

The file is too large to upload here to GitHub so please download from this OneDrive link. This is a compressed file of the output of monocart-reporter for the first shard of one of my projects. It contains many raw code coverage files.
These are the second and third shard outputs (these ones don't have any code coverage on them, all of the code coverage is on the first shard which executes google chrome whilst the second shard is running firefox tests and the third is running webkit):
monocart-reporter-2.tar.gz
monocart-reporter-3.tar.gz
And this contains the merged report for the above:
playwright-test-reporters.tar.gz
In the merged report you can see that the code coverage from all the raw reports should end up being 89.55%. Is this enough? Do you need anything else?

Please try version 2.11.0

original size:  198M
after zip : 27M
after merge:  14M
after merge + zip:  2.6M

I can confirm your results and everything is working as expected. I tested both the merge and the zip options.
Is merge default value enabled or disabled? If it's not set to enable by default, is there any reason for it?
Also, I think perhaps you need to add a note on the monocart-reporter README under the Merge shard reports section to say that if the zip option is used then the user needs to extract the zip first before doing the merge or else it won't work. Perhaps you could add it near the existing note:

Note: The coverage reports will be merged automatically if we specify the raw report in coverage options:

Alternatively, you could add support for zipped code coverage reports to the merge API/CLI so that if it finds a zipped code coverage file it will unzip and just work.

Thanks for the comfirm that the options are working well.
One step, one issue.
First, we need to make monocart-coverage-reports works as expected, then next step, I will update monocart-reporter to apply the new options.
In fact, we should not manually unzip the raw.zip file, Alternatively, it should support loading zip file directly.

One step, one issue.

I agree, I'm closing this issue then. Would you like me to open a new issue for the zip support on the merge API/CLI or will you take care of that if you think it should be done?
Did you notice my question:

Is merge default value enabled or disabled? If it's not set to enable by default, is there any reason for it?

Would you like me to open a new issue for the zip support on the merge API/CLI or will you take care of that if you think it should be done?

I will open a new issue, thanks.

Is merge default value enabled or disabled? If it's not set to enable by default, is there any reason for it?

the merge option defaults to false (disabled), considering these reasons

Most of the time, raw files may not be numerous or large, so there is no need to merge or even compress them.
Merging coverage is a lossy operation and no longer the original raw data.
Merging coverage consumes some system resources, so we leave it to the user to decide.

Sounds good, thank  you for this work @cenfun . Feel free to close this issue whenever you want.

authors: cenfun edumserrano

repo: cenfun/monocart-coverage-reports


text:
Create a zipkin example

We already have a Jeager example of tracing. Could you write a Zipkin example, please?

Hello there @victormn, thank you for the request and welcome to OpenCensus!
So we have an example test in https://github.com/census-instrumentation/opencensus-go/blob/52b12e1eb8fdf769ad0bf5fd73a2af9484e6b196/exporter/zipkin/example_test.go#L15-L40
However, if you'd like to create a conventional example like Jaeger's in the example's  directory, you are more than welcome and I'll tag other teammates @ramonza @rakyll.

Hi, I was planning on adding this. Will get a PR in this weekend.

Perfect, thank you again @freddygv!

authors: freddygv odeke-em victormn

repo: census-instrumentation/opencensus-go


text:
ci: remove queue actions from non-merge rules in Mergify config

By the addition of the queue rules in the Mrgify configuration, all PRs
that require changes, or have been updated and review should be dropped,
are now added to the queue for merging. This is obviously not what we
want.
Fixes: 43fc945 ("ci: move from merge action to queue action")

Merging it as multi-arch build is not affected by this.

repo: ceph/ceph-csi


text:
Example from readme does not work with 2.0.0

Hello,
Using Cequel 2.0.0 I can no longer include Cequel::Record. I am using ruby 2.3.0
[ericu-destroyer-of-worlds] cequel_playground$ cat blog.rb
require 'cequel'
require 'cequel/version'
puts Cequel::VERSION

class Blog
  include Cequel::Record

  key :subdomain, :text
  column :name, :text
  column :description, :text
end
[ericu-destroyer-of-worlds] cequel_playground$ ruby ./blog.rb
2.0.0
/home/ericu/.rvm/gems/ruby-2.3.0@cequel/gems/cequel-2.0.0/lib/cequel/record.rb:97:in `block in <module:Record>': uninitialized constant ActiveModel::Serializers::Xml (NameError)
	from /home/ericu/.rvm/gems/ruby-2.3.0@cequel/gems/activesupport-5.0.0.1/lib/active_support/concern.rb:120:in `class_eval'
	from /home/ericu/.rvm/gems/ruby-2.3.0@cequel/gems/activesupport-5.0.0.1/lib/active_support/concern.rb:120:in `append_features'
	from ./blog.rb:6:in `include'
	from ./blog.rb:6:in `<class:Blog>'
	from ./blog.rb:5:in `<main>'

Here are the other gems in my gemset presently
[ericu-destroyer-of-worlds] cequel_playground$ gem install cequel
Fetching: i18n-0.7.0.gem (100%)
Successfully installed i18n-0.7.0
Fetching: thread_safe-0.3.5.gem (100%)
Successfully installed thread_safe-0.3.5
Fetching: tzinfo-1.2.2.gem (100%)
Successfully installed tzinfo-1.2.2
Fetching: concurrent-ruby-1.0.2.gem (100%)
Successfully installed concurrent-ruby-1.0.2
Fetching: activesupport-5.0.0.1.gem (100%)
Successfully installed activesupport-5.0.0.1
Fetching: activemodel-5.0.0.1.gem (100%)
Successfully installed activemodel-5.0.0.1
Fetching: ione-1.2.3.gem (100%)
Successfully installed ione-1.2.3
Fetching: cassandra-driver-3.0.3.gem (100%)
Building native extensions.  This could take a while...
Successfully installed cassandra-driver-3.0.3
Fetching: cequel-2.0.0.gem (100%)
Successfully installed cequel-2.0.0
Parsing documentation for i18n-0.7.0
Installing ri documentation for i18n-0.7.0
Parsing documentation for thread_safe-0.3.5
Installing ri documentation for thread_safe-0.3.5
Parsing documentation for tzinfo-1.2.2
Installing ri documentation for tzinfo-1.2.2
Parsing documentation for concurrent-ruby-1.0.2
Installing ri documentation for concurrent-ruby-1.0.2
Parsing documentation for activesupport-5.0.0.1
Installing ri documentation for activesupport-5.0.0.1
Parsing documentation for activemodel-5.0.0.1
Installing ri documentation for activemodel-5.0.0.1
Parsing documentation for ione-1.2.3
Installing ri documentation for ione-1.2.3
Parsing documentation for cassandra-driver-3.0.3
Installing ri documentation for cassandra-driver-3.0.3
Parsing documentation for cequel-2.0.0
Installing ri documentation for cequel-2.0.0
Done installing documentation for i18n, thread_safe, tzinfo, concurrent-ruby, activesupport, activemodel, ione, cassandra-driver, cequel after 11 seconds
9 gems installed

@hydrogen18, with rails 5 you need to add gem 'activemodel-serializers-xml' to your Gemfile. #329 removes the need to that but it has not been released yet.

OK, subscribed to the other issue.

For the record, this is fixed in version 2.0.1

repo: cequel/cequel


text:
Wrap raw string passed to pluck

fixes #1170
Newer versions of rails deprecate passing raw strings to query methods like pluck.

@carlthuringer Thanks!

authors: carlthuringer lgebhardt

repo: cerebris/jsonapi-resources


text:
Venafi Certificate Valid Date

Is your feature request related to a problem? Please describe.
I am trying to figure out if it was possible to request a certificate from Venafi with a specific End date, not use the CA defaults.
I browsed the code and here it seems the duration parameter is not really used: https://github.com/cert-manager/cert-manager/blob/d073db164834b772f54aeb7ff1539e1dac3ca992/pkg/issuer/venafi/client/request.go#L106
Where it should be added to the vclient request: https://github.com/Venafi/vcert/blob/039d25df38a3f6ca5b1addc934824c5df633ff35/pkg/certificate/request.go#L65
Describe the solution you'd like
Pass duration to the vclient Request (maybe https://github.com/cert-manager/cert-manager/blob/d073db164834b772f54aeb7ff1539e1dac3ca992/pkg/issuer/venafi/client/request.go#L186 ? )
Describe alternatives you've considered
None
Additional context
None
/kind feature

/remove-lifecycle rotten

A beta-release is now available which contains the fix for this issue. Please test and feedback if you have time.

https://github.com/cert-manager/cert-manager/releases/tag/v1.16.0-beta.0

A beta-release is now available which contains the fix for this issue. Please test and feedback if you have time.

https://github.com/cert-manager/cert-manager/releases/tag/v1.16.0-beta.0

I did a quick test with our Venafi installation and seems to work!

authors: Guitarkalle wallrj

repo: cert-manager/cert-manager


text:
Make warning about clashing ports on AWS more clear

Discussed briefly in slack; this is apparently a gotcha that keeps popping up
Signed-off-by: Ashley Davis ashley.davis@jetstack.io
/kind documentation

Adding lgtm with a hold- as I added an optional suggestion. Do unhold if you'd rather merge as is!
/hold
/lgtm

Thanks! This looks good to me 🥳
/hold cancel
/lgtm

authors: SgtCoDFish irbekrm

repo: cert-manager/website


text:
Erros Comuns em tag Graphic

Element graphic is not declared in sub list of possible chil - 195
Element graphic is not declared in ext-link list of possible - 147
Element graphic is not declared in xref list of possible chi - 73
Element graphic is not declared in italic list of possible c - 60
Element graphic is not declared in bold list of possible chi - 833

https://github.com/cesarbruschetta/document-store-migracao/commit/85b9e119cea5383a445e24eb97aa894b49dcb88c

repo: cesarbruschetta/document-store-migracao


text:
Use customizable FileReader strategy

For projects that require a different method for reading the import file (say, from an external URL) this allows for easier customization than MonkeyPatching the scan method

This has already been addressed on the original branch!


text:
Přidat skript pro export obsahu ze slackových kanálů

Objevil se nám požadavek na export obsahu z kanálu, můžeme na to přidat skript do bin/? Není to sice úplně agenda k webu, ale to je asi fuk.

Poznámky:

conversations.history

TODO:

[ ] Získat seznam zpráv do kanálu (bez odpovědí)
[ ] Získat počet odpovědí
[ ] Přeložit ID autora na něco smysluplného
[ ] Přeložit timestamp zprávy na něco smysluplného
[ ] Přeložit zdroják zprávy do něčeho příčetného
[ ] Exportovat do CSV

repo: cesko-digital/web


text:
Inline preview button is broken

The Preview button actually performs the refactoring.

The preview was actually never supported, but the button is here by default, so I disabled it. I might take the time to add support for previews in a future release.

authors: bjansen gavinking

repo: ceylon/ceylon-ide-intellij


text:
NPE in typechecker using native interface

shared native interface PublicKey;
shared native ("jvm") interface PublicKey {
}

source/x/toto.ceylon:2: error: the '' caused an exception visiting a 'InterfaceDefinition' node: '"java.lang.UnsupportedOperationException"' at 'java.util.AbstractList.add(AbstractList.java:148)'
shared native ("jvm") interface PublicKey {
^
source/x/toto.ceylon:2: error: the 'AnnotationVisitor' caused an exception visiting a 'InterfaceDefinition' node: '"java.lang.NullPointerException"' at 'com.redhat.ceylon.compiler.typechecker.analyzer.AnnotationVisitor.visit(AnnotationVisitor.java:312)'
shared native ("jvm") interface PublicKey {
^
source/x/toto.ceylon:2: error: the 'WarningSuppressionVisitor' caused an exception visiting a 'InterfaceDefinition' node: '"java.lang.NullPointerException"' at 'com.redhat.ceylon.model.typechecker.model.ModelUtil.isForBackend(ModelUtil.java:2654)'
shared native ("jvm") interface PublicKey {
^
source/x/toto.ceylon:2: error: the 'ExpressionVisitor' caused an exception visiting a 'Annotation' node: '"java.lang.NullPointerException"' at 'com.redhat.ceylon.compiler.typechecker.analyzer.AnalyzerUtil.getTypedDeclaration(AnalyzerUtil.java:98)'
shared native ("jvm") interface PublicKey {
^
source/x/toto.ceylon:2: error: the 'ExpressionVisitor' caused an exception visiting a 'BaseMemberExpression' node: '"java.lang.NullPointerException"' at 'com.redhat.ceylon.compiler.typechecker.analyzer.AnalyzerUtil.getTypedDeclaration(AnalyzerUtil.java:98)'
shared native ("jvm") interface PublicKey {
^
source/x/toto.ceylon:2: error: the 'ExpressionVisitor' caused an exception visiting a 'Annotation' node: '"java.lang.NullPointerException"' at 'com.redhat.ceylon.compiler.typechecker.analyzer.AnalyzerUtil.getTypedDeclaration(AnalyzerUtil.java:98)'
shared native ("jvm") interface PublicKey {
       ^
source/x/toto.ceylon:2: error: the 'ExpressionVisitor' caused an exception visiting a 'BaseMemberExpression' node: '"java.lang.NullPointerException"' at 'com.redhat.ceylon.compiler.typechecker.analyzer.AnalyzerUtil.getTypedDeclaration(AnalyzerUtil.java:98)'
shared native ("jvm") interface PublicKey {
       ^
source/x/toto.ceylon:2: error: native declarations not of same type: 'PublicKey'
shared native ("jvm") interface PublicKey {
                                ^
source/x/toto.ceylon:1: error: compiler bug: null at com.redhat.ceylon.compiler.java.codegen.AbstractTransformer.serialiseTypeSignature(AbstractTransformer.java:3602)
shared native interface PublicKey;
^
9 errors

The native header doesn't have a body. Try with shared native interface PublicKey {}

ok, that removes the last error, but the previous 8 remain

@quintesse if you don't support native interfaces, please do the appropriate validation.

@quintesse could we please get this one fixed for 1.2.2?

So I fixed this by implementing support for native interfaces in the JVM backend.
Could you do so as well @chochos ? Like always it'll probably way easier for you ;)

you mean this breaks in js?

shit, it does!

@chochos I guess you have to use that ModelUtil.mergeStatements() function for Tree.Interface declarations as well. That's what I had to do at least. Hopefully on the JS side it's no more work than that.

except now I have an InterfaceDeclaration and an InterfaceDefinition

But I'm getting a typechecker error: native classes do not extend the same type: 'PublicKey' for the InterfaceDefinition

I think you only need to do this for InterfaceDefiniton, I think InterfaceDeclaration is for aliases.

HAve you updated to the latest code? I only pushed this half an hour ago.

Ah no, you're right, weird. Lemme see.

There's also a different case where you can have a native InterfaceDefinition header with just formal members and then another InterfaceDefinition for the current backend with the implementations.

It's because you cant do shared native interface PublicKey;you *must* doshared native interface PublicKey {}`

I'm guessing the problem is because InterfaceDefinitions automagically satisfy Identifiable (or was it Basic?) and the native InterfaceDeclaration doesn't satisfy anything probably.

There's also a different case

You can't do that, all member declarations have to be exactly the same, you're not implementing anything, so there's no formal vs actual

OK so the original code was invalid anyway. But now we need to implement support for native interfaces anyway.

Yes, that's what I did for the JVM

Do you add an error to native interface declarations?

Not yet, I'm working on that now

It's going to be at the typechecker level, right?

yup

Ok, added the error.

I'm off to bed now, you can close this issue when you're done with your part, okay @chochos ! Night!

Done, closing.

authors: chochos gavinking marvelous quintesse

repo: ceylon/ceylon


text:
Find a Housing Counselor: Remove sidebar breakout and content-l column classes post-covid sidebar breakout removal

Follow-on to https://github.com/cfpb/consumerfinance.gov/pull/8421
With the removal of the covid breakout sidebar, we need to also delete some corresponding column classes that expect a second column.
Removals

Find a Housing Counselor: Removes heading area content-l columns and o-sidebar-breakout

How to test this PR

Visit http://localhost:8000/find-a-housing-counselor/ and resize to mobile and see that intro content extends full width.

Screenshots
Before:

After:

The only uncertainty I have about this is that it appears that the right-hand white space matches what the page looked like pre-COVID sidebar: https://web.archive.org/web/20200105183404/https://www.consumerfinance.gov/find-a-housing-counselor/

The only uncertainty I have about this is that it appears that the right-hand white space matches what the page looked like pre-COVID sidebar: https://web.archive.org/web/20200105183404/https://www.consumerfinance.gov/find-a-housing-counselor/

@csebianlander This PR matches. The heading area has a max-width to accommodate that, however, the problem was with it thinking there's a second column, it gets too narrow at mobile size.
Desktop in this PR

Mobile before this PR:

authors: anselmbradford csebianlander


text:
Update usage headings

Update headings for variation pages sub sections such as Variations, Usage, etc to match the new hierarchy
Also styled the names of the variations (aka "Primary buttons") as h5 like in the designs.
Additions

Removals

sticky header CSS that was doing the wrong thing anyway and was geting in way of styling h3s properly

Changes

Testing

Screenshots
Notes

Todos

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the CFPB development guidelines
[ ] Passes all existing automated tests
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future todos are captured in comments
[ ] Visually tested in supported browsers and devices (see checklist below :point_down:)
[ ] Project documentation has been updated
[ ] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Browsers

[ ] Chrome on desktop
[ ] Firefox
[ ] Safari on macOS
[ ] Edge
[ ] Internet Explorer 9, 10, and 11
[ ] Safari on iOS
[ ] Chrome on Android

Accessibility

[ ] Keyboard friendly
[ ] Screen reader friendly

Other

[ ] Is useable without CSS
[ ] Is useable without JS
[ ] Flexible from small to large screens
[ ] No linting errors or warnings
[ ] JavaScript tests are passing

@designlanguage let's chat about these changes

Took a look at this again and it looks good @cfarm !

authors: cfarm designlanguage

repo: cfpb/design-system


text:
Update online file installation instructions in using-dumpling.md

As stated in the title

thank you c:

authors: cabrerahector emiyl


text:
Refactoring to use common constant for ft per meter and usage of StringUtils

Just some simple changes to cleanup the code. No functional changes.

I'll merge this because that refactoring is great, but I will remove the "Constants" file afterwards, because that itself is conceptually wrong. All entries in the Constants.java should be defined where they are used. If there is suddenly a need to make those constants publicly available, then the encapsulation of the code needing them is not correct.

authors: Bananeweizen RoadRunner-

repo: cgeo/c-geo-opensource


text:
Flaky test: tests/chainer_tests/links_tests/connection_tests/test_deconvolution_2d.py

https://jenkins.preferred.jp/job/chainer/job/chainer_pr/1497/TEST=chainer-py2,label=mn1-p100/testReport/junit/tests.chainer_tests.links_tests.connection_tests.test_deconvolution_2d/TestDeconvolution2DParameterShapePlaceholder_param_16_{use_cudnn='always', nobias=True, deconv_args=((None, 2, 3, 2, 1), {})%7D/test_backward_gpu/

This seems related to #7695 so I will take over it.

Update CuDNN to 5.1 for python2 jenkins tests should solve this issue.
@kmaehashi volunteered to do it.

Fix in https://github.com/chainer/chainer-test/pull/514

Reoccurred in convolution_nd. Most likely not fixed yet.
https://github.com/chainer/chainer/issues/7695#issuecomment-517100236

Reoccurred https://jenkins.preferred.jp/job/chainer/job/chainer_pr/1930/
It's the functions and not the link test though.

Let me reopen this ticker.

Pending the backport, master already merged.

authors: emcastillo hvy kmaehashi niboshi

repo: chainer/chainer


text:
Improve docs of n_step_lstm

This is PR for improving docs of functions/links. Related issue: #2182
Modified functions/links:

functions/connection/n_step_lstm.py

@niboshi Thank you for the review! I followed your review.

@niboshi I have just fixed them!

Thank you for a quick fix!

jenkins, test this please

LGTM. Thanks!

authors: keisuke-umezawa niboshi


text:
Verbose error in gradient check

Made two error messages related to gradient check errors more verbose.
The messages now include actual/expected values. Simply because when actual values don't match expected values in assertions, we often want to know in what sense they differ (w/o their internal variable name).

LGTM!

authors: hvy niboshi


text:
support for fp16 in naive comm

This PR adds the support for fp16 in naive comm.
Related to https://github.com/chainer/chainer/issues/6330
ToDo:

[x] support for fp16 in allreduce
[x] support for fp16 in bcast

Jenkins, test this please.

Jenkins CI test (for commit 7d39fe3f5c2fd2e766d0b2db8526c300112997ff, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

👀

Jenkins, test this please.

Jenkins CI test (for commit 7d39fe3f5c2fd2e766d0b2db8526c300112997ff, target branch master) succeeded!

Jenkins CI test (for commit 7d39fe3f5c2fd2e766d0b2db8526c300112997ff, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

authors: chainer-ci kuenishi shu65


text:
Enable label support

Right now, when running docker inspect on an apko generated image, the Config.Labels. field is set to null.
It would be nice to add support similar to what exists for annotations (config file/ --annotations support), but for labels instead.
Context
On annotations vs labels, here's my understanding (please chime in if I'm missing anything):
There's (3) major OCI config/manifest files.

OCI Image index - Points to os/arch specific manifest files for an image
Image manifest - os/arch specific, layers etc
OCI Image Configuration file - This is what you see when you e.g. docker inspect and has the Config.Labels field

Notably (1) and (2) support annotations (3) supports labels (again - What you can view with docker inspect).
You can view annotations on (1) and (2) via docker buildx imagetools inspect ..., but annotations != labels. They kind of match in spec (?), but my understanding is the distinction is where they show up.
APKO supports annotations with (1) and (2).
It does not support (3).
Question
I see this PR open here that would merge annotations and labels - Anything provided as an annotations would become a label in the OCI image configuration file (shows up w/ docker inspect).
I wanted to ask so we can contribute either of them / merge the above PR / etc to see labels functionality -
Is the goal to:
A - Just allow users to provide annotations, and put them in all (3) locations?
B - Allow users to provide annotations that go into (1) and (2), but also provide labels that only go to (3)?
C - Something else?
Seeking clarification on how labels vs annotations ought to behave in apko and so it can be contributed.
Thank you!

Looking at this comment here:
https://github.com/chainguard-dev/apko/issues/692#issuecomment-1559402957
@kaniini would you be able to chime in on preferred approach? We're eager to contribute it ASAP whichever direction you prefer. Alternatively, if there's a best person to raise this design question to, would appreciate it!!!
CC: @kaniini @amouat

Closed by https://github.com/chainguard-dev/apko/pull/1215

repo: chainguard-dev/apko


text:
keycloak: run keytool in container rather than requiring jvm on host.

This test just assumed that 'keytool' was available (and at the right version) on the host.  Instead of that, use keytool provided by the jvm that is inside the image we're testing.
New Image Pull Request Template

Image Size

[ ] The Image is smaller in size than its common public counterpart.
[ ] The Image is larger in size than its common public counterpart (please explain in the notes).

Notes:
Image Vulnerabilities

[ ] The Grype vulnerability scan returned 0 CVE(s).
[ ] The Grype vulnerability scan returned > 0 CVE(s) (please explain in the notes).

Notes:
Image Tagging

[ ] The image is not tagged with version tags.
[ ] The image is tagged with :latest
[ ] The image is not tagged with :latest (please explain in the notes).

Notes:
Basic Testing - K8s cluster

[ ] The container image was successfully loaded into a kind cluster.
[ ] The container image could not be loaded into a kind cluster (please explain in the notes).

Notes:
Basic Testing - Package/Application

[ ] The application is accessible to the user/cluster/etc. after start-up.
[ ] The application is not accessible to the user/cluster/etc. after start-up. (please explain in the notes).

Notes:
Helm

[ ] A Helm chart has been provided and the container image can be used with the chart.  If needed, please add a -compat package to close any gaps with the public helm chart.
[ ] A Helm chart has been provided and the container image is not working with the chart (please explain in the notes).
[ ] A Helm chart was not provided.

Notes:
Processor Architectures

[ ] The image was built and tested for x86_64.
[ ] The image could not be built for x86_64 (please explain in the notes).
[ ] The image was built and tested for aarch64.
[ ] The image could not be built for aarch64. (please explain in the notes).

Notes:
Functional Testing + Documentation

[ ] Functional tests have been included and the tests are passing.  All tests have been documnted in the notes section.

Notes:
Environment Testing + Documentation

[ ] There has not been a request and/or there is no indication that this image needs tested on a public cloud provider.
[ ] The container image has been tested successfully on a public cloud provider (AWS, GCP, Azure).
[ ] The container image has not been tested successfully on a public cloud provider (AWS, GCP, Azure) (please explain in the notes).

Notes:
Version

[ ] The package version is the latest version of the package.  The latest tag points to this version.
[ ] The package version is the not the latest version of the package (please explain in the notes).

Notes:
Dev Tag Availability

[ ] There is a dev tag available that includes a shell and apk tools (by depending on 'wolfi-base')
[ ] There is not a dev tag available that includes a shell and apk tools (by depending on 'wolfi-base') (please explain in the notes).

Notes:
Access Control + Authentication

[ ] The image runs as nonroot and GID/UID are set to 65532 or upstream default
[ ] Alternatively the username and GID/UID may be a commonly used one from the ecosystem e.g: postgres
[ ] The image requires a non-standard username or non-standard GID/UID (please explain in the notes).

ENTRYPOINT

[ ] applications/servers/utilities set to call main program with no arguments e.g. [redis-server]
[ ] applications/servers/utilities not set to call main program with no arguments e.g. [redis-server] (please explain in the notes)
[ ] base images leave empty.
[ ] base image and not empty (please explain in the notes).
[ ] dev variants is set to entrypoint script that falls back to system.
[ ] dev variants is not set to entrypoint script that falls back to system (please explain in the notes).

CMD

[ ] For server applications give arguments to start in daemon mode (may be empty)
[ ] For utilities/tooling bring up help e.g. –help
[ ] For base images with a shell, call it e.g. [/bin/sh]

Environment Variables

[ ] Environment variables added.
[ ] Environment variables not added and not required.

SIGTERM

[ ] The image responds to SIGTERM (e.g., docker kill $(docker run -d --rm cgr.dev/chainguard/nginx))

Logs

[ ] Error logs write to stderr and normal logs to stdout. Logs DO NOT write to file.

Documentation - README

[ ] A README file has been provided and it follows the README template.

As part of a Chainguard effort to reduce code duplication and provide higher quality images, we have stopped accepting pull requests for changes to images in this repository. Due to this, we are closing all pull requests today in order to start fresh.
If you think this pull request was closed in error, please feel free to re-open this pull request or open an issue and we will try to help you out. Thank you!

authors: jdolitsky smoser

repo: chainguard-images/images


text:
API: Add agencies, lines and services

Add those 3 data type to the API.

Implemented in #1174

authors: greenscientist tahini

repo: chairemobilite/transition


text:
Module '"@ark-ui/solid"' has no exported member 'Dialog'

Hi friends! Sorry if it's too early for stuff like this, but I'm not able to import any components for the Dialog. When looking at intellisense there's actually only a handful of things that I can import.

@ark-ui/solid version: "^0.0.0-rc-20221106094212"
solid-js version: 1.6.8
If there's anything else I can do to help let me know! Thanks!

I looked @ark-ui/solid@0.0.0-rc-20221106094212 dist file, but is no Dialog related code, you can upgrade the version to 0.0.0-rc-20230403193437

@ShumRain 0.0.0-rc-20221106094212 is the latest version according to npm. How can I upgrade it?

@ahhshm @mattwells19 you can do this in your package.json until Ark is in v1 "@ark-ui/solid": "rc", Rerun pnpm i
Let me know if that works

Yes, it works. Thank you

@ahhshm @mattwells19 you can do this in your package.json until Ark is in v1 "@ark-ui/solid": "rc", Rerun pnpm i
Let me know if that works

That worked for me as well. Thanks!

authors: ShumRain ahhshm lvl-mattwells mattwells19

repo: chakra-ui/ark


text:
Can't install version ~1.0

Hi,
I'm trying to install version ~1.0 from this https://github.com/chalasr/RCHJWTUserBundle/tree/1.x (it seems like this readme is for version 2.0) and I have the following error:
[Symfony\Component\Config\Definition\Exception\InvalidConfigurationException]
  Unrecognized options "user_class, user_identity_field, passphrase" under "rch_jwt_user"

Thanks for report this @cve I'll quickly look at this, the configuration should be valid for both 1.x and master. So, About what we discussed in #10, don't worry about the branch, work on master, I'll then apply the changes for both.
Again, thank you for your efforts!

authors: chalasr cve

repo: chalasr/RCHJWTUserBundle


text:
Outdated package.json since 4.1.1

4.1.1 was supposed to add ESM support and require NodeJS 12, but looking at the published package contents on NPM, this does not show up in code anymore:
https://www.npmjs.com/package/chalk/v/4.1.1
https://www.npmjs.com/package/chalk/v/4.1.2
See the explore tab or download the files from NPM. If you compare them, they are not the same as checked into this repo.

4.1.1 was supposed to add ESM support and require NodeJS 12

No it was not. What's in the main branch is supposed to do that, but it's not done yet. What's in the main branch is not necessarily always what's published. This is how most repos on GitHub works.

Actually cross-referenced release dates and main branch to make sure this could have been released yet. Which is how repos on GitHub usually work, but fine, if you have a different strategy.

You're mistaken. 4.1.1 was published from branch meta-tweaks which is 2 commits diverged from main. The common ancestral commit is the 4.1.0, release which sported this package.json which clearly targets Node >= 10.0 and is not ESM. 4.1.1 did not change that as can be seen here.
Dates mean nothing, look at the content please. Master/main are always to be considered bleeding edge. I'm not really sure where the confusion is - Chalk doesn't do anything weird or unusual here w.r.t. git repositories.
Nothing actionable and I see this conversation getting less civil moving forward so going to pre-emptively lock.

authors: Qix- alopix sindresorhus

repo: chalk/chalk


text:
🛑 Hotal The Nest is down

In 67a34ec, Hotal The Nest (https://hotelthenest.com/) was down:

HTTP code: 503
Response time: 891 ms

Resolved: Hotal The Nest is back up in f106b94 after 20 minutes.

repo: chamannarved/drabdigital-uptime-moniter


text:
IOS Error JavaScript execution returned a result of an unsupported type" UserInfo={NSLocalizedDescription=JavaScript execution returned a result of an unsupported type

Hi, I've installed this plugin and on Android, it's work well, but on IOS I receive 2 error:

After the build() method I receive this error:

Unhandled Exception: PlatformException(evaluateJavaScript_failed, Failed evaluating JavaScript, JavaScript string was: '$(".note-placeholder").html("");'
Error Domain=WKErrorDomain Code=5 "JavaScript execution returned a result of an unsupported type" UserInfo={NSLocalizedDescription=JavaScript execution returned a result of an unsupported type}, null)

When I try to get a text from _keyEditor.currentState.getText() I receive this expectation:

Unhandled Exception: PlatformException(evaluateJavaScript_failed, Failed evaluating JavaScript, JavaScript string was: 'GetTextSummernote.postMessage(document.getElementsByClassName('note-editable')[0].innerHTML);'
Error Domain=WKErrorDomain Code=5 "JavaScript execution returned a result of an unsupported type" UserInfo={NSLocalizedDescription=JavaScript execution returned a result of an unsupported type}, null)

I've followed the steps for IOS configuration updating Info.plist.
What I wrong?
My flutter doctor result is
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, 1.23.0-18.1.pre, on macOS 11.1 20C69 x86_64, locale it)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 12.2)
[✓] Chrome - develop for the web
[✓] Android Studio (version 3.6)
[✓] Connected device (3 available)
    ! Error: iPhone di Giuseppe is not connected. Xcode will continue when iPhone di Giuseppe is connected.
      (code -13)

• No issues found!

Have a look at the #16 :)

Have a look at the #16 :)


text:
Update "github" to 0.25, drop custom fork

We were using the a fork that added support for one endpoint we need. That has been upstreamed, and is included in the 0.25 release. So we can use the official 0.25 now.

@OpsBotPrime merge

Pull request approved by @ruuda, rebasing now.

Rebased as 8af2a569d02a28c29c6f726a9207ae5f2af716bc, waiting for CI …

authors: OpsBotPrime ruuda

repo: channable/hoff


text:
Handle older MRC files that erroneously wrote uint8 data instead of int8

The simplest way I found to determine whether a tomogram has uint8 data that gets read by the mrcfile library as int8 is to check the dmax header value. Basically if the max value written to that header is >127, we've likely overflowed some data. It's possible that some older versions of the mrc writing library didn't write this header correctly, or wrote wrong values, but that seems unlikely.
What happens in the case of these older tomograms is that uint data (range 0:255) gets read in as int data (byte-order 0:127, and then -128:-1). This meant that the uint values (127, 128, 129) which were supposed to be contiguous, get mapped to (127,  -128, -127), which makes them discontiguous. This patch re-maps all of the incorrectly-read data back into a contiguous range from -128:127.
I still need to write some utilities to identify & fix the offending tomograms, but this is a start.

The problem using the min/max values is that these are optional, so they might not be set or reflect the data contained. If this is not the case for any of our data we can use it of course, just something we should stay aware of as an edge case.

The problem using the min/max values is that these are optional, so they might not be set or reflect the data contained. If this is not the case for any of our data we can use it of course, just something we should stay aware of as an edge case.

Yeah, unfortunately there's not a lot of other hints that we have to determine whether data has overflowed - the headers for "good" and "bad" tomograms look virtually identical otherwise, so we'd have to rely on processing the image data itself, or use the (also optional, and imnsho even less reliable) date information in the label header to make decisions.

authors: jgadling uermel


text:
[staging] Pagination is not working when filters are applied

Steps to Reproduce

Locate filter panel on left side of page
Click on "Ground Truth Annotation" filter
Pay attention to Results

Actual Results
For this particular filter, there is only 7 results available and pagination is still displayed
Expected Results
Pagination should not be displayed and not functional
Evidence
https://github.com/chanzuckerberg/cryoet-data-portal/assets/63799148/9a99763d-17d7-4d9c-b304-ccb1db4d2789

bug is resolved

authors: junxini lauramarcos

repo: chanzuckerberg/cryoet-data-portal


text:
Update tooltip.

Summary:

What: Now that we're no longer using GISAID, update the public id tooltip for SC2 to mention genbank instead.  This PR renames the mpox publicId column to be generic and uses it in both tables.  This has the correct tooltip.
Ticket: None
Env: none

Demos:

Notes:
Checklist:

[ ] I merged latest <base branch>
[ ] I manually verified the change
[ ] I added labels to my PR
[ ] I tested in multiple browsers
[ ] I added relevant unit tests
[ ] I have notified others of changes they need to make locally (migrations, jobs, package updates, etc)

Closing - decided to just change sc2 tooltip rather than combining.

repo: chanzuckerberg/czgenepi


text:
Update FAQ's instructions for hiding a plugin

Addresses https://github.com/chanzuckerberg/napari-hub/issues/514

Thanks, @richaagarwal !

authors: kevinyamauchi richaagarwal

repo: chanzuckerberg/napari-hub


text:
[feature] Custom scan_each_shard batch size

This allows users to decide how many keys to fetch for each scan.
According to redis, the SCAN command

The default COUNT value is 10

This is not ideal for our use case -- we gonna end up with too many round trips. I updated the default to 1000 (the same default batch size as activerecord)
Test Plan

The changes are covered by spec
ci

Good thing to have, although it can't reduce more than 10% of current trips since 90% are finds that fetch 1 record

Good point! perhaps we can add a script to return full objects back.

rebase
:mag_right: Click to see changes since last update.

Codecov Report

Merging #84 (fc488cc) into master (596d425) will increase coverage by 0.23%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master      #84      +/-   ##
==========================================
+ Coverage   94.40%   94.64%   +0.23%
==========================================
  Files          32       32
  Lines        1680     1642      -38
==========================================
- Hits         1586     1554      -32
+ Misses         94       88       -6

Impacted Files
Coverage Δ

lib/redcord/redis.rb
91.42% <66.66%> (-0.58%)
:arrow_down:

lib/redcord/relation.rb
87.91% <0.00%> (-0.87%)
:arrow_down:

lib/redcord/tracer.rb
89.47% <0.00%> (-0.53%)
:arrow_down:

lib/redcord/attribute.rb
94.11% <0.00%> (-0.07%)
:arrow_down:

lib/redcord/redis_connection.rb
98.30% <0.00%> (-0.03%)
:arrow_down:

lib/redcord/logger.rb
100.00% <0.00%> (ø)

lib/redcord/serializer.rb
100.00% <0.00%> (ø)

spec/actions_spec.rb
86.52% <0.00%> (+0.44%)
:arrow_up:

spec/serializer_spec.rb
97.97% <0.00%> (+0.97%)
:arrow_up:

... and 3 more

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 596d425...fc488cc. Read the comment docs.

authors: codecov-io donaldong

repo: chanzuckerberg/redcord


text:
Cancelling an upload returns BAD REQUEST

Using the lovely notebooks:

upload_local_datafile(datafile_path, new_collection_id, identifier)
import time
time.sleep(60)
delete_dataset(new_collection_id, identifier)

BAD request is returned even when the deletion is delayed for +2 minutes.
The mitigation is to cancel the upload in the UX. This is not a primary scenario.

This was a bug in the notebook code. Re-tested with a copy of the notebooks from @danieljhegeman 's branch in progress. The problem has been corrected.

repo: chanzuckerberg/single-cell-data-portal


text:
feat(correlation id): Add correlation ids to logs to improve debugging

Need
Is your feature request related to a problem? Please describe.
Sifting through cloudwatch logs when you discover a problem is slow and tedious. Often you're given a mountain of log records and not enough keywords in the logs to sufficiently filter them. Another added complexity for the API requests is that they are run in parallel, so related logs are not always grouped together. We need a way to identify related logs easily to speed up the debugging process. It also becomes difficult to debug issues that span multiple system components. For example, finding the request that kicks off a specific processing job.
Approach

Generate a correlation_id (a unique identifier) for each set of related requests or operations.
Add a correlation_id to requests to the backend API. These Ids are returned to the user in the header and appear in all logs related to that request.
Pass the correlation_id to processes started due to an API request. Specifically, this would be the dataset processing pipeline should accept a correlation_id that it adds to all of its log messages.
Add a correlation_id to submission lambda and pass that id to the dataset processing pipeline
Optional: support request chaining to allow requests to contain a correlation_id in the header which can be used to link multiple requests within the logs.

Definition of Done

Making a request will return X-Request-Id as a header and will contain a unique Id that can be used to identify log messages in the backend generated by the request.
Logs for a request that starts a processing job, and the job will have the same correlation_id
Logs for a submission lambda that starts a processing job, and the job will have the same correlation_id

Generate a correlation_id (a unique identifier) for each set of related requests or operations.
Add a correlation_id to requests to the backend API. These Ids are returned to the user in the header and appear in all logs related to that request.
This work has been completed by #3380

This work has been completed by https://github.com/chanzuckerberg/single-cell-data-portal/pull/3380

@Bento007 : Can this be closed?

yes


text:
RTNETLINK answers: No such file or directory

apiVersion: chaosblade.io/v1alpha1
kind: ChaosBlade
metadata:
  name: loss-node-network-by-names
spec:
  experiments:
  - scope: node
    target: network
    action: loss
    desc: "node network loss"
    matchers:
    - name: names
      value: ["docker-desktop"]
    - name: percent
      value: ["60"]
    - name: interface
      value: ["eth0"]
    - name: local-port
      value: ["40690"]

"status": {
        "expStatuses": [
            {
                "action": "loss",
                "error": "see resStatus for the error details",
                "resStatuses": [
                    {
                        "error": "RTNETLINK answers: No such file or directory\n exit status 2 exit status 1",
                        "identifier": "/docker-desktop/",
                        "kind": "node",
                        "state": "Error",
                        "success": false
                    }
                ],
                "scope": "node",
                "state": "Error",
                "success": false,
                "target": "network"
            }
        ],
        "phase": "Error"

目标容器内是否必须有tc命令才行？

目标容器内是否必须有tc命令才行？

确认网卡 eth0 存在吗？

authors: chenzhenyang tiny-x

repo: chaosblade-io/chaosblade-operator


text:
CHAOSS Education Workshop Materials

CHAOSS Education is an ongoing project which will be delivered through short, informative videos and interactive workshops!
Contribute to Video Recordings:
Our video library is constantly growing! Become part of the process by tackling any of the interesting issues listed on this repo.
Join Our Workshop Development:
We're building a comprehensive workshop series on essential open-source topics.
Here are the topics:

Demystifying Open Source: Explore the world of open source and understand its importance.
Open Source Participation 101: Learn how open-source software gets built – both socially and technically. We'll delve into Pull Requests (PRs) and the power of Slack communication.
Finding Your Perfect Project: Discover an open-source project that aligns perfectly with your skillset.
Managing Expectations: Set realistic expectations for yourself and understand the open-source community culture.

If you've presented a talk on any of these subjects at CHAOSScon, Summits or any other conference, please share your talk or conference videos or Slide deck here. We're eager to learn from your experiences.
Add the link under this thread

I hope these help

https://opensource.guide
https://medium.com/shecodeafrica/how-to-overcome-the-challenges-of-open-source-contribution-ea333815ffdf
https://medium.com/@preciousdanabubakar/open-source-contribution-why-its-a-great-idea-553f3459e4bb

repo: chaoss/education


text:
Create metrics table

In an effort to make the metrics more consumable for tooling, I created this HTML table. If this is acceptable, I will create one for the remaining metrics pages.
I went with HTML on this because it allows for row spanning which I believe allows us to visualize the relationships between the lower level metrics and aggregate them into high-level metrics.

Hi @klumb ,
Thank you for this initiative. I think the table structure looks great but I am not a tooling person.
Maybe you could send a message to the mailing lists (metrics and software), include a link to this pull request, and ask for feedback from people who will actually consume the document.
Once the table is build out to your satisfaction, we can merge in the pull request.

I like the idea. But maybe it would be better to write the table in Markdown? (just for consistency with the rest of the document).

What are the options outside of HTML?

Github Flavoured Markdown supports tables, but not (AFAIK) merged cells, so it would look like this:
Metric | Signal | Action
--- | --- | ---
Value Accumulation | Bug Resolution | Tickets Closed = Time from bug opened to bug closed
Value Accumulation | Bug Resolution | Time to Close = Total # of tickets closed
Value Accumulation | Patch Speed | Time to Merge = Time from submission to merge or abandon
Value Accumulation | Patch Speed | Review Efficiency = # of merged patches / # of abandoned patches
Community Growth | # of new developers | New Contributors = # of first time contributors over time
Community Growth | # of new developers | ...

Metric
Signal
Action

Value Accumulation
Bug Resolution
Tickets Closed = Time from bug opened to bug closed

Value Accumulation
Bug Resolution
Time to Close = Total # of tickets closed

Value Accumulation
Patch Speed
Time to Merge = Time from submission to merge or abandon

Value Accumulation
Patch Speed
Review Efficiency = # of merged patches / # of abandoned patches

Community Growth
# of new developers
New Contributors = # of first time contributors over time

Community Growth
# of new developers
...

I am not following exactly what this is .... The term metric is now at a high level of abstraction, and signal is more granular. Action then sounds like a specifically measurable item that we had previously been calling a "metric."
I like the idea of separating layers, but I am concerned that right now this commits us to a language about what each "thing" means that has not yet been sorted out.
What I would like to see is a set of 3-7 specific, measurable items identified for a pilot cycle through the various software implementations.  From that I think we can start to figure out what the right words and layers are ... This is helpful, but I want to build something off of a list of "things to build" without making a commitment to what we call them just yet.
In this paper I describe how making commitments to what we call things too soon can cause a lot of confusion for along time on social computing projects ... https://goo.gl/info/ypuKzJ

Thanks, I do think markdown is the better way to go here, I only went with HTML because it allowed me to display the table the way I visualized it based on the current discussion in the markdown file.
@goggins, your comment is very helpful. What does tooling want to see here?

@klumb : see my comment above and let me know if it helps. I do not speak for the software committee, but only for my thoughts as a member of the software committee.

Reposting my response from the mailing list to keep the discussion together:
Quick feedback based on my undestanding:

Our framing to date has been:
Metric Category (Growth, Maturity & Decline, for example)
Metric (Tickets Closed, for example).

I always thought of "activity metric" as the counter part to (component of) what you call "metric category".

The proposed change to our framing looks sort of like this:
Metric Category

“Metric” (Value accumulation, for example)
“Signal” (Bug Resolution, for example)
“Action” (Tickets Closed, for example)

Here’s my concern, and its not really a concern that is conceptual, but instead procedural, or wishing to guide how we iterate on the things we encase in software construction. As principles:

I think we know that the lowest level of abstraction needs to be something implemented in code.

Agreed.
For this reason, I don't think the framing of metric - signal - action is appropriate for understanding these levels of abstractions.
I propose an understand the terms that is independent of metric aggregation:

metric is independent of level of abstraction (could be composit or raw activity metric) [example: average time to close issue is increasing month over month];
signal is merely how the metric can be interpreted [example: contributors are submitting issues faster than maintainers can close them];
action is the response to the (metric informed) signal [example: promote more contributors to maintainers to spread the load of responding to and closing issues].

With this understanding, we need to ignore signal and action for now and focus solely on metric - which is the only aspect relevant to implementation for now.

I think we know that the metric category is a bucket for collecting things implemented in code into groups that make sense to both the metrics and software committees.

Agreed.
I think Kevin was proposing a way to show these "buckets" and what they contain, including the "layers of subcategories".

Its also likely that there will be layers of subcategories and groupings in between the coarsest (metrics category) and most granular (discrete measurable items or “activity metrics” like ‘Tickets closed’).

I propose that;

We identify the layers in between through an actual implementation of 3-7 “activity metrics” or “measurable items” like ‘tickets closed’ in different software against possibly different repository types.
From that working iteration cycle, we then come back and try to identify how the “activity metrics” or “measurable items” might be grouped together to inform subcategories or subgroupings of “growth, maturity and decline” (a metrics category, for example.).

Sounds like a good approach.
Does this mean we go back to what we used to have on the wiki, only with fewer (3-7) activity metrics for each of the four highest-level metrics. During the implementation process, I would expect to learn whether these activity metrics are actually at the lowest level (measurable item?) or composite metrics (which introduces a layer).

@klumb said:

What does tooling want to see here?

I think, at least from the GrimoireLab side, we don't have a preference here. In our standard dashboard metrics are now grouped by data source, but other arrangements are possible, and one based on "what do metrics mean" seems quite reasonable to me.

In response to the table above and based on today's Google Hangout.

Remove two leftmost columns.
Rename Action column as Activity Metic

This will remove any predefined associations and hone in on 3-7 activity metrics that can be used to define specific metrics.

I have updated the table of Growth-Maturity-Decline and added tables for the remaining metrics. The activity metrics I have inserted into the tables are taken directly from the metrics discussions occurring in the markdown pages.

I'd say go ahead and merge this. We can start a new request to tweak the language.

Thanks for the review @germonprez

Just a quick comment (sorry I've been absent as I've been traveling....)
I like the simplified table we have with 2 columns.  At some point, would it make sense to add a third column to note if any of the metrics are already implemented in the software?  We could just say Y/N with a link to a repo if there's an implementation in the Software committee repo.

Good idea :)

authors: GeorgLink germonprez jgbarah klumb npch rpaik sgoggins

repo: chaoss/metrics


text:
cnvkit fails with "no such file or directory" when running Rscript

Apparently it tries to load a script that's no longer there:

[2016-04-13T06:40Z] uni03: No outlier bins
[2016-04-13T06:40Z] uni03: Traceback (most recent call last):
[2016-04-13T06:40Z] uni03:   File "/storage/rozo/bcbio/anaconda/bin/cnvkit.py", line 11, in <module>
[2016-04-13T06:40Z] uni03:     args.func(args)
[2016-04-13T06:40Z] uni03:   File "/storage/rozo/bcbio/anaconda/lib/python2.7/site-packages/cnvlib/commands.py", line 666, in _cmd_se
gment
[2016-04-13T06:40Z] uni03:     rlibpath=args.rlibpath)
[2016-04-13T06:40Z] uni03:   File "/storage/rozo/bcbio/anaconda/lib/python2.7/site-packages/cnvlib/segmentation/__init__.py", line 55
, in do_segmentation
[2016-04-13T06:40Z] uni03:     seg_out = ngfrills.call_quiet('Rscript', script_fname)
[2016-04-13T06:40Z] uni03:   File "/storage/rozo/bcbio/anaconda/lib/python2.7/site-packages/cnvlib/ngfrills/__init__.py", line 42, in
 call_quiet
[2016-04-13T06:40Z] uni03:     % (' '.join(args), err))
[2016-04-13T06:40Z] uni03: RuntimeError: Subprocess command failed:
[2016-04-13T06:40Z] uni03: $ Rscript /tmp/tmp5nsfEn
[2016-04-13T06:40Z] uni03:
[2016-04-13T06:40Z] uni03: Rscript execution error: No such file or directory
[2016-04-13T06:40Z] uni03:
[2016-04-13T06:40Z] uni03: Uncaught exception occurred
Traceback (most recent call last):
  File "/storage/rozo/bcbio/anaconda/lib/python2.7/site-packages/bcbio/provenance/do.py", line 21, in run
    _do_run(cmd, checks, log_stdout)
  File "/storage/rozo/bcbio/anaconda/lib/python2.7/site-packages/bcbio/provenance/do.py", line 95, in _do_run
    raise subprocess.CalledProcessError(exitcode, error_msg)
CalledProcessError: Command 'set -o pipefail; unset R_HOME && export PATH=/storage/rozo/bcbio/anaconda/bin:$PATH && /storage/rozo/bcb
io/anaconda/bin/cnvkit.py segment -o /storage/rozo/projects/OV-223/ov223/work/structural/LC0044-FAL/cnvkit/raw/tx/tmpgcblTH/LC0044-FA
L-sort.cns /storage/rozo/projects/OV-223/ov223/work/structural/LC0044-FAL/cnvkit/raw/LC0044-FAL-sort.cnr
No outlier bins
Traceback (most recent call last):
  File "/storage/rozo/bcbio/anaconda/bin/cnvkit.py", line 11, in <module>
    args.func(args)
  File "/storage/rozo/bcbio/anaconda/lib/python2.7/site-packages/cnvlib/commands.py", line 666, in _cmd_segment
    rlibpath=args.rlibpath)
  File "/storage/rozo/bcbio/anaconda/lib/python2.7/site-packages/cnvlib/segmentation/__init__.py", line 55, in do_segmentation
    seg_out = ngfrills.call_quiet('Rscript', script_fname)
  File "/storage/rozo/bcbio/anaconda/lib/python2.7/site-packages/cnvlib/ngfrills/__init__.py", line 42, in call_quiet
    % (' '.join(args), err))
RuntimeError: Subprocess command failed:
$ Rscript /tmp/tmp5nsfEn

Rscript execution error: No such file or directory

Latest master.

Launching the script manually causes the same error, so I wonder if it's a bug in cnvkit instead? It's the first time I use it though, so I can't tell.

It's a local installation error, seen after further investigation. Closing.

repo: chapmanb/bcbio-nextgen


text:
Mongoid - Activity base class is using STI

PublicActivity::Activity inherits from corresponding orm class, which is PublicActivity;:ORM::Mongoid::Activity.
This inheritance causes the presence of a "_type" field in every document, which is very redundant and useless and overloads database when using base model. Moreover it forces all queries to have a "_type" clause which degrades performance, and create an heavy index (non created by default...).
Since Activities can grow very fast (in my case, i have almost 10x more documents in activities than tracked models, and i don't track all actions yet).
Is there a way to avoid this inheritance ?
I would like to keep using this gem as long as possible before switching to a better solution that i don't have time to develop. Thank you :)

Actually, i just found a quick fix which seems to work with my project.
This changes pass tests with mongoid, but i did not tested with ActiveRecord.
Here is the quick (ugly) fix that i added to my initializer, just before adding some fields in class:
module PublicActivity
  Activity = inherit_orm("Activity")
end

This solution seems to work perfectly, i checked that kind_of? keeps working as expected, and it's possible that this change doesn't break depending applications. I'm running ruby 2.0.0 with rails 4.2.
You could maybe consider changing your Activity model definition to avoir STI, if it seems right.

Interesting, will definitely look into it! Had no idea Mongoid does it, as I'm not actively using any NoSQL databases.

I intend to get rid of anything other than ActiveRecord so closing this

authors: ArTiSTiX pokonski


text:
docs(SIM114): fix typo in example Python code

I think the original documentation here was probably intended to be more like pseudo-code. But I think it's nice if it is valid Python.

Ah no, definitely just a typo. Thank you!

authors: Jeremiah-England charliermarsh

repo: charliermarsh/ruff


text:
docs:  explain how to build cdn js file

I was trying to figure out how to re-build the same file that is served from CDNs, and I got confused and tripped over myself on the way.  through my own mistakes, I had actually convinced myself that the build script in this repo did not produce the file that the CDN was serving.   Nothing in the docs talked about it, so I just assumed it was some "special thing that exists somewhere else."
I am adding this to the docs now so that the next person reading through will be re-assured and hopefully find the right file.

Not sure if this would be valuable or just confuse people, because I'm not sure it is common to try to reproduce something in cdn. What about NPM etc.

Ok, then how about this? instead of

file you would get from a CDN will be located at ./dist/chart.umd.js

it could say

if you want to include the Chart.js javascript file that you just built directly on an HTML page instead of importing it as a module, use ... dist/chart.umd.js

I kind of doubt that I'm not the only chartjs user who doesn't know what umd means and doesn't know that I should look for "jsdelivr": "./dist/chart.umd.js",  in package.json to learn that  "chart.umd.js is the one you can just plop onto the page in a script tag"

authors: ForestJohnson kurkle


text:
Enable RTTI for non-Asm.js builds

By default -fno-rtti was being set on builds which broke exception handling.

Hey @nbeerbower,
Hopefully we'll be moving to CMake soon.
This should allow anyone using the library to enable/disable RTTI from where it's used.
We probably don't want to enable it for all users, because it does come with overheads.
Thanks for the contribution though!

repo: charto/nbind


text:
分开启动api服务时，在基于知识库问答时返回空答案，且后端有报错：httpx.RemoteProtocolError: peer closed connection without sending complete message body (incomplete chunked read)

问题描述 / Problem Description
用简洁明了的语言描述这个问题 / Describe the problem in a clear and concise manner.
复现问题的步骤 / Steps to Reproduce

执行 '...' / Run '...'
点击 '...' / Click '...'
滚动到 '...' / Scroll to '...'
问题出现 / Problem occurs

预期的结果 / Expected Result
描述应该出现的结果 / Describe the expected result.
实际结果 / Actual Result
描述实际发生的结果 / Describe the actual result.
环境信息 / Environment Information

langchain-ChatGLM 版本/commit 号：(例如：v2.0.1 或 commit 123456) / langchain-ChatGLM version/commit number: (e.g., v2.0.1 or commit 123456)
是否使用 Docker 部署（是/否）：是 / Is Docker deployment used (yes/no): yes
使用的模型（ChatGLM2-6B / Qwen-7B 等）：ChatGLM-6B / Model used (ChatGLM2-6B / Qwen-7B, etc.): ChatGLM2-6B
使用的 Embedding 模型（moka-ai/m3e-base 等）：moka-ai/m3e-base / Embedding model used (moka-ai/m3e-base, etc.): moka-ai/m3e-base
使用的向量库类型 (faiss / milvus / pg_vector 等)： faiss / Vector library used (faiss, milvus, pg_vector, etc.): faiss
操作系统及版本 / Operating system and version:
Python 版本 / Python version:
其他相关环境信息 / Other relevant environment information:

附加信息 / Additional Information
添加与问题相关的任何其他信息 / Add any other information related to the issue.

使用的是V0.2.2，且使用一键脚本无法启动成功，已经按照要求更新了requirements

我也出现了这个毛病，您那解决了吗

同样遇到这个问题

同样遇到这个问题

感觉应该是数据传输过程中的问题，但是不明白会出现这种问题

同样遇到这个问题

同样遇到这个问题，请问有人解决了吗？

同上，如何解决？

同上，如何解决？

我直接拉最新版本0.2.4的，重新部署就没有这个问题。

repo: chatchat-space/Langchain-Chatchat


text:
根据官方文档，添加对英文版的bge embedding的指示模板

按照实际需要，依据官方文档将bge英文版的查询提示模板添加到embeddings支持中，避免使用时异常

我们认为你的PR合理，决定合并到我们的代码中

authors: WilliamChen-luckbob zRzRzRzRzRzRzR


text:
手工部署项目后500错误

请问大佬，在docker下，分别在mongo、nodejs、nginx容器中，按照手工方法复制service文件夹，build打包前端，.env文件中VITE_GLOB_API_URL=/api未修改，运行后报500错，请问哪个地方设置错误了？

老哥 我也是相同问题，前端后端都没有任何报错，但是就是进不去

repo: chatgpt-web-dev/chatgpt-web


text:
refactor: remove unnecessary joins in getUserByFirebaseId method

Now, from synced develop branch:

These change tries to solve the issue #454

Refactored the getUserByFirebaseId function to only fetch the user data, removing unnecessary joins/relations.

Searched for any functions that rely on req['userEntity'] to ensure no existing functionality is broken. I found no instances of such functions, so it seems ok.

Ran all tests and confirmed they passed successfully

Perfect! Thanks @reidn3r

repo: chaynHQ/bloom-backend


text:
mempages_test & stats_test test failed for v7.1.0

I am doing test build which is falling How to solve ?
go test ./...
But getting failed some tests related to mempages_test & stats_test like this
$ go test ./...
?       github.com/checkpoint-restore/go-criu/v7        [no test files]
--- FAIL: TestNewMemoryReader (0.00s)
mempages_test.go:20: error opening binary file: open test-imgs/pstree.img: no such file or directory
--- FAIL: TestGetMemPages (0.00s)
mempages_test.go:85: error opening binary file: open test-imgs/pstree.img: no such file or directory
--- FAIL: TestGetPsArgsAndEnvVars (0.00s)
mempages_test.go:194: error opening binary file: open test-imgs/pstree.img: no such file or directory
--- FAIL: TestGetShmemSize (0.00s)
mempages_test.go:254: error opening binary file: open test-imgs/pstree.img: no such file or directory
--- FAIL: TestGetDumpStats (0.00s)
stats_test.go:8: Failed to get stats
stats_test.go:11: PagesWritten is 0
--- FAIL: TestGetRestoreStats (0.00s)
stats_test.go:18: Failed to get stats
stats_test.go:21: ForkingTime is 0
FAIL
FAIL    github.com/checkpoint-restore/go-criu/v7/crit   0.003s
?       github.com/checkpoint-restore/go-criu/v7/crit/cli       [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/cmd       [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/apparmor   [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/autofs     [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/binfmt-misc        [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/bpfmap-data        [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/bpfmap-file        [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/cgroup     [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/core-aarch64       [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/core-arm   [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/core-mips  [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/core-ppc64 [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/core-s390  [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/core-x86   [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/cpuinfo    [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/creds      [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/criu-core  [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/criu-sa    [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/eventfd    [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/eventpoll  [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/ext-file   [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/fdinfo     [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/fh [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/fifo       [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/file-lock  [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/fown       [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/fs [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/fsnotify   [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/ghost-file [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/img-streamer       [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/inventory  [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/ipc-desc   [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/ipc-msg    [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/ipc-sem    [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/ipc-shm    [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/ipc-var    [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/macvlan    [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/memfd      [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/mm [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/mnt        [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/netdev     [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/ns [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/opts       [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/packet-sock        [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/pagemap    [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/pidns      [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/pipe       [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/pipe-data  [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/pstree     [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/regfile    [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/remap-file-path    [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/rlimit     [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/rseq       [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/seccomp    [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/siginfo    [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/signalfd   [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/sit        [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/sk-inet    [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/sk-netlink [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/sk-opts    [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/sk-packet  [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/sk-unix    [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/stats      [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/sysctl     [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/tcp-stream [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/time       [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/timens     [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/timer      [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/timerfd    [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/tty        [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/tun        [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/userns     [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/utsns      [no test files]
?       github.com/checkpoint-restore/go-criu/v7/crit/images/vma        [no test files]
?       github.com/checkpoint-restore/go-criu/v7/magic  [no test files]
?       github.com/checkpoint-restore/go-criu/v7/phaul  [no test files]
?       github.com/checkpoint-restore/go-criu/v7/rpc    [no test files]
ok      github.com/checkpoint-restore/go-criu/v7/scripts/magic-gen      0.002s
?       github.com/checkpoint-restore/go-criu/v7/stats  [no test files]
?       github.com/checkpoint-restore/go-criu/v7/test   [no test files]
?       github.com/checkpoint-restore/go-criu/v7/test/crit      [no test files]
?       github.com/checkpoint-restore/go-criu/v7/test/phaul     [no test files]
?       github.com/checkpoint-restore/go-criu/v7/utils  [no test files]
FAIL
Please explain why this has failed and also let me know if this test is important or not.

Take a look how we run the tests in CI: https://github.com/checkpoint-restore/go-criu/blob/master/.github/workflows/main.yml
Basically you need to go through the makefile and not call go test directly.

As Adrian mentioned, the tests require checkpoint files that are generated by another Makefile target. You would need to invoke sudo make test to be able to run them successfully.

authors: Riteshkr22 adrianreber snprajwal

repo: checkpoint-restore/go-criu


text:
Bug: Full resolution image fails to appear

On Safari iOS, on almost 95% of the images I tap on, regardless of the image's instance, a low/normal appears for a split second, it tries to switch to the full resolution image but it ends up showing nothing. If I pinch to zoom on the blank view, the full-res image appears though. It feels like the full image is there, but the switch between them is not successful.
On Windows with Firefox, the switch between resolutions is also clearly visible, like an extremely fast, progressive load of the full-res image. It's still jarring but at least the switch is properly completed.
edit: here's a quick demo of the issue: https://share.icloud.com/photos/0cdL2MSn7OTFfiePpxAcFY4YQ

Should be fixed now on production (possibly need to reload), can check it out 🙏

I'm on production on Safari, doesn't seem to be fixed.  In fact I never started seeing it until last week.  Is there a way to do an extra-hard reload to make sure I have the latest phanpy?

I'm still seeing the issue, it's much rarer than before but when it happens, it's worse than before: I can't see the full image even if I zoom in/out. It looks like the the full image is loaded but is not visible and I can't get it to appear.

Still here, on almost every picture on Safari.  Working fine on Mac Chrome, though.  For complicated reasons I need to use Safari for most things so this is annoying.

@timbray is this happening for images in the timeline or in the full-page image modal (after tap on image)? Or both? Is it Safari desktop? 🙇♂

Only in the full page. Desktop Safari on M2.

It looks like it's working now? I haven't seen the issue since yesterday I think. I've also noticed the swap between low/hi resolution images in a couple of cases, it worked quickly and without any issues.

Agreed, latest fix seems to be working, THANKS!

authors: cheeaun lamnatos timbray

repo: cheeaun/phanpy


text:
timer needs to be started to work

totally missed this on the first pass-thru at fixing this, my apologies.
Description
activates the timer when timer is in use
Issues Resolved
n/a
Check List

[x] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[x] New functionality includes testing.
[x] New functionality has been documented in the README if applicable
[x] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

As Mr. systemd do you have any thoughts on these
https://github.com/chef-cookbooks/chef-client/pull/477
https://github.com/chef-cookbooks/chef-client/pull/483

yeah, both seem like good ideas. timeout should almost certainly be opt-in, though. #477 made me just realize that this PR should have also had a stop action added on the other side of the ternary when i added the start 😭... i'll go ahead and send another PR incoming since it looks like from the comments the other contributor is likely busy.

@nathwill I added the :stop in #483 on top

I see, #492 addresses that

authors: artem-sidorenko nathwill tas50

repo: chef-cookbooks/chef-client


text:
Remove unused components/config

This PR removes

opscode-expander-reindexer
rabbitmq job's queue

neither of which are used in Chef Server

:+1: :boom: cruft.

cc @chef/chef-server-maintainers

+1

authors: metadave stevendanna tylercloke

repo: chef/chef-server


text:
RemoteFile::SFTP Custom Port Implementation Broken

Description
RemoteFile::SFTP breaks when using custom ports
https://github.com/chef/chef/blob/9f6f9a9681611ce71fd6e541c88e228a0e095abc/lib/chef/provider/remote_file/sftp.rb#L62-L65
Net::SFTP does not support parsing the port from the host string. It needs passed in as a separate arg
https://github.com/net-ssh/net-sftp#label-SYNOPSIS-3A
Normally this wouldn't be an issue if you're just using port 22 but if you've got Just-In Time (port closed until requested open) enabled on a Linux SFTP server to protect SSH it makes sense to host SFTP on a separate port so SFTP is always available.
The following changes fix my issue (although full disclosure haven't tested the normal test case without specifying the port in the URI). Port method changed because in this scenario we need a fallback when port isn't specified in the URI. Same code can be used used to monkey patch with a chef library. Side note, I couldn't figure out how to use Refinements with Chef in case someone reading this knows, I'd be interested.
class Chef
  class Provider
    class RemoteFile
      class SFTP
        def port
          @uri.port || 22
        end

        def sftp
          @sftp ||= Net::SFTP.start(hostname, user, { password: pass, port: port })
        end
      end
    end
  end
end

Chef Version
Latest
Platform Version
Windows
Replication Case
Create SFTP URI per official docs using custom port (ie 2222)
Example: "sftp://user:pass@server:2222/some/path"
https://docs.chef.io/resources/remote_file/
Client Output

Chef-client produces the following error but I had to use pry to try replicating what was producing it. Port is not included with the hostname when default port is used which is the only reason code works.
# What I suspect the code is doing
pry(main)> Addrinfo.getaddrinfo("server:2222", 22)
SocketError: getaddrinfo: No such host is known

Suggested labels: Low Priority, Good First Issue

repo: chef/chef


text:
knife bootstrap - error installing git-daemon

Knife boostrap fails while installing git. Here is the error.
Appreciate any help in this regard.
Chef::Exceptions::Package
No version specified, and no candidate version available for git-daemon
Resource Declaration:
In /var/chef/cache/cookbooks/git/recipes/server.rb
package 'git-daemon'

@ctippur Apologies that you never received a response on this.  This appears to be a problem with a particular cookbook you are attempting to use rather than a bug in knife or chef-client.  If you are still experiencing this problem, many in the Chef community would be happy to help, particular in our IRC channel on freenode and the mailing list.  Is this still an issue for you?  If not, I'll close this bug out.

@ctippur As Steven noted above, this is more of an issue with the cookbook or recipe you are using and the Chef mailing list or IRC channel are more appropriate forums for help with that.

authors: ctippur smurawski stevendanna


text:
Directory resource fails due to Insufficient Permissions

ChefDK 0.4.0 with Chef Client 12.0.3
Directory resource fails to create directories in some folder paths - even though permissions do not appear different.
#Worked
chef-apply --% -e 'directory "c:/users/administrator/test"'
#Failed
chef-apply --% -e 'directory "c:/users/administrator/desktop/test'
#Worked
chef-apply --% -e 'directory "c:/test'
#Failed
chef-apply --% -e 'directory "c:/users/test'

Originally reported against ChefDK - https://github.com/chef/chef-dk/issues/340

Related to https://github.com/chef/chef/pull/2896 ?

Possibly.. Tracking it down now

I tested with
 chef-apply --% -e 'directory "c:/Users/Administrator/Desktop/test"'

against Server 2012 with
Chef Client 11.6.0 - FAILED
Chef Client 11.18.6 - FAILED
Chef Client 12.0.3 - FAILED
Chef Client 12.1.2 - SUCCESS
I'll do some more complete testing, but this only affects certain directories (principally, maybe only, in the local Administrator user profile).

@jdmundrawala Yeah, looks very much related to that PR, since it is resolved in 12.1.x

authors: jdmundrawala smurawski


text:
Problem with ACLs when a user and a client has the same name

Description
If you have in the system a user that has a name (for example: "charles_xavier") and you have a client with the same name ("charles_xavier"), when you try to add that user to the ACL of a node there is an error. I suppose that the cause if that the system doesn't now if the actor that you are trying to add to the ACL is the user or the client.
I think that it would be better if there were a way to specify whether the actor is a client or a user.
Chef Version
12.0.6
Platform Version
CentOS 6.
Replication Case
Create a file called "acltest.txt" with the following content:
{
  "create": {
    "actors": [
      "pivotal",
      "charles_xavier"
    ],
    "groups": [
      "clients",
      "users",
      "admins"
    ]
  }
}

Then execute the following commands:
chef-server-ctl  user-create charles_xavier Charles Xavier charles_xavier@xmen.com mypass
knife client create charles_xavier
knife node create mytestnode
knife raw /organizations/default/nodes/mytestnode/_acl/create -m PUT -i acltest.txt

The error message is:
ERROR: Server responded with error 400 "Bad Request"
ERROR: Error Body: {"error":["Invalid/missing actor in request body"]}

Hi, this is the bug tracker for the chef-client. For chef-server bugs, like this one, please file an issue at https://github.com/chef/chef-server/issues/new . I'll be closing this issue now.

Ok. Issue created: https://github.com/chef/chef/issues/6378


text:
print STDOUT from the powershell wrapper. fixes issue #3596

fixes issue #3596

This appears to be ok, but I'm not a powershell user (only confirmed usage via the docs). Anyone from @chef/maintainers able to lend a hand for review?

Same here, it looks fine but I'm not a subject matter expert

Paging @smurawski for :eyes:

/cc @chef/client-windows - this is related @adamedx 's fix.  Naively, this seems sensible but since we're using invokereturnasis, I don't know if that makes a difference how Write-Output would format it.

I've hacking with that powershell wrapper script. I don't like that the trap is making the use of $ErrorActionPreference impossible. @adamedx are you working on rewriting the powershell wrapper used by the powershell_script resource?

I have some concerns about using Write-Output -- does that write to the pipeline? If so, that would affect the output of the script. Seems like it's passing on Windows so maybe that's ok, but I would think something that generated output without affecting the pipeline would be preferable.

@alexpop my own involvement here is pretty light, but the maintainers may be able to help if you are running into some issues.

Yeah, I think Write-Output is the wrong choice, it should be Write-Host instead. Also, we're missing some test coverage since some test should break if our wrapper alters the script output:
% adamed@SEA-ADAMED1: 0x0100E0D0: C:\Users\edwardsa
2015.06.26 Fri 14:38:55.00-07:00> { }.invokereturnasis() -eq $null
True
% adamed@SEA-ADAMED1: 0x0100E0D0: C:\Users\edwardsa
2015.06.26 Fri 14:39:10.45-07:00> { write-output 'hi' }.invokereturnasis() -eq $null
False
% adamed@SEA-ADAMED1: 0x0100E0D0: C:\Users\edwardsa
2015.06.26 Fri 14:39:17.40-07:00> { write-host 'hi' }.invokereturnasis() -eq $null
hi
True
% adamed@SEA-ADAMED1: 0x0100E0D0: C:\Users\edwardsa
2015.06.26 Fri 14:39:22.63-07:00>

Thanks @adamedx. If I use Write-Host, I don't get the stdout of the powershell script from kitchen or from a direct chef-client run.
With Write-Output, I get the stdout when I run chef-client on the box using the same command as kitchen:
$env:TEMP='C:\Users\vagrant\appdata\local\Temp'
chef-client.bat --local-mode --config $env:TEMP\kitchen\client.rb --log_level info --force-formatter --no-color --json-attributes $env:TEMP\kitchen\dna.json --chef-zero-port 8889

With Write-Output, I don't get the stdout when I run kitchen converge from my laptop with info or debug log levels. This might be related to: https://github.com/test-kitchen/test-kitchen/issues/685

I gave this a quick test and it makes sense to me.  @adamedx - the write-output is after the invokereturnasis and is printing what powershell would otherwise have sent to stdout.  It was error case that I was worried about - I didn't know if printing the output there would cause any weirdness with the output happening after the script failed but I don't see anything particularly offensive about the current behavior.

:+1:

:+1:

Merged with https://github.com/chef/chef/pull/4087

authors: adamedx alexpop danielsdeleo jtimberman ksubrama lamont-granquist martinisoft smurawski


text:
golint fixes and pool_test additions

Hi,
I've fixed a number of golint complaints, and added a basic pool_test.go
Tested in linux & windows.
One controversial change might be in pool_x.go there was always a (needless?) bar.SetWidth which didn't seem to add anything in normal terminals, and broke the test cases.

I'll close the request then as I don't think you want to pull any of this...

repo: cheggaaa/pb


text:
Store additional data in the frames

Some formats defines more information than what is used inside chemfiles. We may want to keep this data around, in case someone would like to use it.
I propose to add a generic data storage inside a Frame, to be used for additional data from the file format (think to occupancy in PDB or CIF files, forces, ...). This storage should be able to store per atom and global data.
On the C++ side, data can be stored in a std::map<std::string, value>, where value is a tagged union of the allowed data types. I think the allowed data types should be scalars (as doubles), 3D vectors and strings.
On the C side, we should wrap the union into an opaque struct, and make it a bit more type safe
/// Setter functions
chfl_status chfl_frame_store_scalar(const CHFL_FRAME* frame, const char* name, double value);
chfl_status chfl_frame_store_string(const CHFL_FRAME* frame, const char* name, const char* value);
chfl_status chfl_frame_store_vector(const CHFL_FRAME* frame, const char* name, chfl_vector value);

enum chfl_data_type {
    CHFL_SCALAR,
    CHFL_STRING,
    CHFL_VECTOR,
}

/// Get the type of stored data, or an error
chfl_status chfl_frame_load_type(const CHFL_FRAME* frame, const char* name, chfl_data_type* type);

/// Extract the value by calling the right function
chfl_status chfl_frame_load_scalar(const CHFL_FRAME* frame, const char* name, double* value);
chfl_status chfl_frame_load_string(const CHFL_FRAME* frame, const char* name, const char** value);
chfl_status chfl_frame_load_vector(const CHFL_FRAME* frame, const char* name, chfl_vector* value);

Thinking a bit more about this, we should allow both scalar data on some basic data types (bool, string, double and Vector3D), and std::vector of these data types.
There will be some work to define a clean API on both the C++ and C sides, but I think it is worth it, among others to allow to support specific additional data for some formats.

This could also help with data preservation when doing a round trip with chemfiles: going PDB -> chemfiles -> PDB, or netCDF -> chemfiles -> netCDF would not loose extra data (beta factor, HETATM/ATOM, forces, ...). Data preservation is always nice to have, and could be useful in some cases.

repo: chemfiles/chemfiles


text:
Check to ensure end IDs are greater than start IDs in the PDB reader

Turns out the fix is really simple. The way to misrepresent PDB files will never cease to amaze me!

Thanks a lot! Travis failure is unrelated, I'll give it a look before merging this if I can.
I think we should try to read the full PDB archive to check for other issues like this. Is this something you did with lemon?

I resubmitted the failed Travis job.
This can be done, but Lemon does this using the MMTF format, which avoids these issues.

authors: Luthaf frodofine


text:
🛑 Homepage is down

In fa46182, Homepage (https://larrychen.tech) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Homepage is back up in d9852f5.

repo: chendachao/uptime


text:
🛑 Homepage is down

In 6afee1b, Homepage (https://larrychen.tech) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Homepage is back up in 1531098.


text:
可视化模型过程中的问题

您好，非常感谢您再STL模型方面做出的贡献，给我提供了很多帮助。
我想请教一下，从问题“kth数据集问题 https://github.com/chengtan9907/OpenSTL/issues/28”@JCChen778中发现了和我遇到类似的问题，但是我依然没有解决，希望得到您的指导。
我的问题：在进行可视化的时候遇到提示错误：
E:\Anaconda3\envs\pytorch-gpu-openSTL\python.exe H:/OpenSTL-master/tools/visualizations/vis_video.py -d mmnist -w work_dirs/Debug --index 0 --save_dirs fig_mmnist_vis
Traceback (most recent call last):
File "H:/OpenSTL-master/tools/visualizations/vis_video.py", line 136, in
main()
File "H:/OpenSTL-master/tools/visualizations/vis_video.py", line 45, in main
assert os.path.isdir(args.work_dirs)
AssertionError
Process finished with exit code 1
开始怀疑可能是work_dir的路径不正确，按照可视化文档（docs/en/visualization/video_visualization.md）中的指令进行修改后，
-d mmnist -w work_dirs/Debug --index 0 --save_dirs fig_mmnist_vis
仍然不能解决，以下是我从官网下载程序包并训练之后得到的文件的相关结构，是否与您的相同。谢谢。
OpenSTL-master
|--tools
| |--visualizations
| | |--vis_video.py
| |--work_dirs
| | |--Debug
| | | |--checkpoints
| | | | |--lastest.pth

你好，建议试试-d mmnist -w ../work_dirs/Debug --index 0 --save_dirs yoursavedirs --vis_channel -1

I will close this issue if there is no more question. Please feel free to open a new issue when you have new questions.


text:
🛑 tt is down

In 7ff6d1a, tt (https://telebot-app-serverless-i7z8-chenhaonanniubi.vercel.app/checkhealth) was down:

HTTP code: 504
Response time: 10067 ms

Resolved: tt is back up in 0f6b854 after 18 minutes.

repo: chenhaonanniubi/upptime


text:
🛑 tt is down

In ff9703e, tt (https://telebot-app-serverless-i7z8-chenhaonanniubi.vercel.app/checkhealth) was down:

HTTP code: 500
Response time: 2951 ms

Resolved: tt is back up in 914e971 after 22 minutes.


text:
目录不会自动刷新

在"全部文件"列表页面上传文件后，有一些文件可以实现只要上传好了，全部文件列表页面就立即出现了这个新上传的文件。
但是有一时候新上传的文件上传完毕之后这个全部文件页面不会自动刷新，看不到这个新上传的文件。
这是个BUG，需要修复。
非常感谢软件作者能开发这个好用的软件，希望能在下一个版本修复这个问题。
辛苦了。。。

并发刷新时可能会造成这个问题，已修复

并发刷新时可能会造成这个问题，已修复
确实已经解决该问题！完美！赞！！！


text:
流式测试的结果

作者您好！ 请问在使用麦克风接入时，正确的输出应该是怎样的，我的程序有如下警告，不知道是不是可以接入麦克风正常运行了？
./stream_kws_main 0 40 80 keyword-spot-dstcn-maxpooling-wenwen/onnx/keyword-spot-dstcn-maxpooling-wenwen.ort Kws Model Info: 	cache_dim: 256 	cache_len: 105 ALSA lib pcm_dsnoop.c:601:(snd_pcm_dsnoop_open) unable to open slave ALSA lib pcm_dmix.c:1032:(snd_pcm_dmix_open) unable to open slave Cannot connect to server socket err = No such file or directory Cannot connect to server request channel jack server is not running or cannot be started JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock Cannot connect to server socket err = No such file or directory Cannot connect to server request channel jack server is not running or cannot be started JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock ALSA lib pcm_oss.c:397:(_snd_pcm_oss_open) Cannot open device /dev/dsp ALSA lib pcm_oss.c:397:(_snd_pcm_oss_open) Cannot open device /dev/dsp ALSA lib confmisc.c:160:(snd_config_get_card) Invalid field card ALSA lib pcm_usb_stream.c:482:(_snd_pcm_usb_stream_open) Invalid card 'card' ALSA lib confmisc.c:160:(snd_config_get_card) Invalid field card ALSA lib pcm_usb_stream.c:482:(_snd_pcm_usb_stream_open) Invalid card 'card' ALSA lib pcm_dmix.c:1032:(snd_pcm_dmix_open) unable to open slave 0 12 INFO (main():/home/zhangyang/code/keyword-spot/onnxruntime/bin/stream_kws_main.cc:115) === Now recording!! Please speak into the microphone. ===
应该如何接入麦克风呢？

提示
“Now recording!! Please speak into the microphone”
表示接入了，开始录音了，你说对着麦克风说对应的唤醒词就可以了。

authors: chenyangMl yangzhangok

repo: chenyangMl/keyword-spot


text:
🛑 Chequeado is down

In a0d6e0b, Chequeado (https://chequeado.com) was down:

HTTP code: 502
Response time: 9058 ms

Resolved: Chequeado is back up in d41e477.

repo: chequeado/status


text:
Forslag til et enklere strategispill

Kan også spilles på sjakkbrett med 4 + 1 bonder på hvite felter

https://en.wikipedia.org/wiki/Catch_the_Hare

Takker! Godt at det finnes en wiki-side med regler. Spørs om jeg hadde klart å huske de ellers 😄

authors: teodoran vsandvold

repo: cheslie-team/cheslie-player