text:
Update created field type to integer.

The created field in the send message response has been updated to
an integer rather than the current string, meaning unmarshalling the response
fails.
Unfortunately the official docs don't seem to reflect this
(here) but it is reproducible repeatedly
here.
I didn't add tests for this since it would require building a mock that wasn't
in place currently. I can do this if you'd like!

Thanks for catching this

Thanks for the library Mike :-), owe you a beverage next time I swing by Kirkland :-)

authors: mikehelmick robshakir

repo: mikehelmick/go-vestaboard


text:
Use FETCH NEXT over CTE ROW_NUMBER() when doing a BatchQuery.

Use this query instead.
SELECT *
FROM Table
ORDER BY Column1, Column2
OFFSET 13000000
ROWS FETCH NEXT 1000000 ROWS ONLY;

This must only affect the following classes.

https://github.com/mikependon/RepoDb/blob/master/RepoDb/RepoDb/_SqlServer/StatementBuilders/SqlServerStatementBuilder.cs
And all UnitTests

Note: Integration Tests must run as it is now.

Done in #1124

source: gharchive/issue

authors: mikependon ngardon

repo: mikependon/RepoDB

license_type: permissive

license_source: github-api


text:
How to get position in bindView(ViewHolder holder, List payloads)?

How to get position in bindView(ViewHolder holder, List payloads)?

https://developer.android.com/reference/android/support/v7/widget/RecyclerView.ViewHolder.html#getAdapterPosition()

repo: mikepenz/FastAdapter

license_source: bigquery


text:
Not working with swipe refresh

I have integrated ScrollListener without refreshing it works fine but when i try to swipe refresh its not loading more

@bb1690 please write a proper issue report.
What are you doing.
What is happening.
What do you expect to happen.
Which code are you using.
Which version of the library are you using.


text:
Proc.newにブロックをつけた

Ruby2.7からブロックなしのProcはdeprecatedになったため
( mikutter/twitter#1 )

{}をつけるだけだとだめっぽいことがわかったのでだめっぽいです。。

各メソッドの引数の最後に &proc とかそんな感じで追加して、Proc.new を &proc に置き換えるといいかんじになるかと

いい感じになりました！

LGTM

source: gharchive/pull-request

authors: Akkiesoft shibafu528 toshia

repo: mikutter/twitter

license: MIT


text:
add support for multiplying dataclass times scalar

e.g. support for:
model = model - 0.1 * dmodel

Using myia.hypermap.HyperMap instead of myia.composite.Elemwise (using the appropriate leaf function) would work as a quick-fix, but it would bypass Python's __add__/__radd__ protocol entirely, which may not be the right thing here.
Alternatively, we could implement Python's protocol, which is essentially this:
def add(x, y):
    if hasattr(x, '__add__'):
        z = x.__add__(y)
    else:
        z = NotImplemented
    if z is NotImplemented:
        z = y.__radd__(x)
    return z

We'd just need to add support for NotImplemented, hasattr and some way to check for NotImplemented.

authors: breuleux ethancaballero

repo: mila-iqia/myia


text:
Open at login

Added an option to open the app at login (using GBLaunchAtLogin: https://github.com/lmirosevic/GBLaunchAtLogin).
Updated build number to 5.
Added localization for the new menu option in English and Spanish (other localizations needed).

@milgra is this useful in any way?

thank you, merged!

German translation would be:
„Beim Start öffnen“
Maybe you can change that @milgra without the need of a seperate pull request :)

authors: alejandroivan maciboy milgra

repo: milgra/highsierramediakeyenabler

license: unlicense


text:
need to indent expressions surrounded by parenthesis

input and expected output:
;(
  foo ||
  (bar &&
     ipsum
  )
);

current output:
;(
foo ||
(bar &&
ipsum
)
);

we should probably do something similar to expressionParentheses.addSpaceInside and call it automatically for every single node during the indentation process

this is harder than expected.. already spent 30min on this and still no idea on how to solve this.. so many edge cases, we can't simply check if it's surrounded by parenthesis because:
foo(
  lorem &&
  ipsum
);

if (
  dolor ||
  amet
) {
  bar();
}

qwe
  .rty({
    asd: (
      'some long ' +
      'text'
    )
  });

I like to use arrow function implicit returns with parenthesis, I can't get esformatter to add the indent.
input and expected output:
const helloWorld = () => (
    'Hello' + ' ' + 'World'
)

current output:
const helloWorld = () => (
    'Hello' + ' ' + 'World'
)

This is a really nice pattern for React Stateless component. esformatter-jsx has this issue too.
const HelloWorld = () => (

Hello World

)

@DaleJefferson The arrow function is WAY easier to solve than this issue, so I created a new bug for it (#399). I won't have time to work on it this week, tho; PRs are highly appreciated!

authors: DaleJefferson millermedeiros

repo: millermedeiros/esformatter

license: mit


text:
Change diskann parameters (cached number nodes) to (search cache budget)

Signed-off-by: cqy123456 yaya645@126.com
issue: #253

/lgtm
/approve

repo: milvus-io/knowhere


text:
[ERROR]: Hardware requirements need to be updated

Is there an existing issue for this?

[X] I have searched the existing issues.

Issue
In this page: https://milvus.io/docs/v2.0.x/prerequisite-docker.md
In section "Hardware requirements", we say "Current version of Milvus does not support AMD and Apple M1 CPUs.", actually current milvus can support Apple M1 CPI, so we need to update the sentence to be "Current version of Milvus does not support AMD CPUs"
Suggestion
No response
Anything else?
No response

Closed with PR #1598

authors: YiyunNi yhmo

repo: milvus-io/milvus-docs


text:
Similarities files

Hi:
After downloading data, there are six .npy files in config_files/hpo_metab/similarities. I want to learn how to make them?

Hi @wangzeyu135798, I just added the code to precompute pairwise shortest paths, ego graphs, and degrees. See the README for details. The .npy files in the similarities folder will be generated while running SubGNN if they do not exist (see get_similarities function here). If they already exist, SubGNN will read them in from file. Hope this helps! Feel free to reopen the issue if you have additional questions.

repo: mims-harvard/SubGNN


text:
[Build] Update project build config and setup depend-a-bot

🚀 The feature
We should update the current build configuration to a more modern one:

instead of requirements files and a large setup.py use pyproject.toml instead
for some reasons i would suggest to setup depend-a-bot to tackle some dependency issues and keep it actual

@frgfm i would assign this one to you :)
Is that ok for you ?
Motivation, pitch
better dependencies handling
reduce the amount of configuration specific files
hold dependencies up to date
Alternatives
No response
Additional context
No response

Yes, sounds good! Thanks for opening this tracker :)
I'll deal with this shortly!

I will proceed in several parts:

[ ] moves existing configs to pyproject.toml for tools (not build) #966
[ ] add black formatting
[ ] moves requirements to pyproject.toml and make version specifier stricter
[ ] set up the bot

@frgfm do you think we can also change CI to circleci ?

I could, but why do you think it would be better?
In my opinion, considering the project is on Github, and the Github CI has a lot of useful features/actions, it's easier with that :)

Mh yes but i see as a big disadvantage that it is not possible to choose a job specific instance type (for example SAR and MASTER Onnx tests are not possible in both frameworks in fact that it kills the provided instance type by Github CI -> locally works) and i think this will not be the only case .. another think is that we would be much more flexible for example we could scale the CI to specific jobs computation need .. do you know what i mean ? :)

It is only a idea if you have another solution in mind would be also fine :) but we need anything to test also computation expensive workflows in our CI

Yes I get what you mean, but the CI is already cluttered with jobs either way :sweat_smile:
I'll have to take a look and reassess what needs testing and whether we could do it more efficiently! Just FYI, we have an unusually high amount of features being tested by the CI (in a good way)

Yes I get what you mean, but the CI is already cluttered with jobs either way 😅 I'll have to take a look and reassess what needs testing and whether we could do it more efficiently! Just FYI, we have an unusually high amount of features being tested by the CI (in a good way)

Yes I know it's more about the fact that we can't test more computationally intensive jobs (like Onnx for SAR/MASTER) due to Github CI's instance limitation :) Otherwise we could maybe annotate these jobs as being only tested locally (like huggingface slow tests)

Yes I get what you mean, but the CI is already cluttered with jobs either way 😅 I'll have to take a look and reassess what needs testing and whether we could do it more efficiently! Just FYI, we have an unusually high amount of features being tested by the CI (in a good way)

Yes I know it's more about the fact that we can't test more computationally intensive jobs (like Onnx for SAR/MASTER) due to Github CI's instance limitation :) Otherwise we could maybe annotate these jobs as being only tested locally (like huggingface slow tests)

Yes I know it's more about the fact that we can't test more computationally intensive jobs (like Onnx for SAR/MASTER) due to Github CI's instance limitation :) Otherwise we could maybe annotate these jobs as being only tested locally (like huggingface slow tests)

I would prefer the last option, we need to prioritize what needs testing in docTR :)

Huhu @frgfm 👋 do you think we could wrap this up maybe next week? I think updating the requirements will also solve the docker problem. Then we would be ready for the release 👍 😃

@felixdittrich92 sure :)

@frgfm any updates ? :)

repo: mindee/doctr


text:
upgrade packer to version 1.5.6

This PR bumps packer to version 1.5.6.
I feel that we should automate this process instead of manually bumping versions. Any idea how we can achieve that @mariux ?

This PR bumps packer to version 1.5.6.
I feel that we should automate this process instead of manually bumping versions. Any idea how we can achieve that @mariux ?

we could but we need to update unless something important is fixed or implemented... so we do not need to catch every version... this might even introduce new bugs...

authors: mariux soerenmartius

repo: mineiros-io/build-tools


text:
Support for 'published' tag in atom feed

There are atom feeds which use 'published' tag in entry instead of 'updated'. As described here 'published' is optional element. It could be used when 'updated' tag is not present for entry.

Please, can you update the unit tests as well? It should be pretty straightforward.
Thank you

No problem. Updated unit test.

authors: fguillot neepl

repo: miniflux/miniflux


text:
Question about PresignedPutObject API

Thanks for the prefect open source tools.
When I use minio SDK, I try to get a PresignedPutObject for others (not controlled by me) to put object into it.
I think I need some limits about the presigned url.
I found the header like 'x-amz-content-sha256' can do that. I can limit the user of the presigned put object url that they can only upload specific file. Of course, the object will not been shown before it has been already uploaded.
However, there are no chance in minio go sdk PresignedPutObject API for me to make it.
I think the feature may be very fantastic.
Thanks for any suggestions or answers.

https://pkg.go.dev/github.com/minio/minio-go/v7?utm_source=godoc#Client.Presign use this to construct whatever you like.

Thanks for your answer.
I tried this API.
However it can not add a header like 'x-amz-content-sha256'  and signature including this header.

I found that there is a PR #1449  like my question.
However it's open and can not make it because it doesn't add this feature to Presign API.

I found that there is a PR #1449 like my question.
However it's open and can not make it because it doesn't add this feature to Presign API.

you can add the x-amz-content-sha256 as part of the url.Values{} query param.

I tested and found that it just add a query kv into url. (Will this be inclued into signature?)
And I even can not use that url to put object. I am trying to solve this problem.
But I think query param isn't the best choice.

https://pkg.go.dev/github.com/minio/minio-go/v7#Client.PresignHeader - This the API that addresses x-amz-content-sha256 requirement.

authors: ABNER-1 harshavardhana minio-trusted

repo: minio/minio-go


text:
Update awscli tests to log error messages

Fixes https://github.com/minio/mint/issues/150

update @harshavardhana

Can we remove the /tmp/multipart file once it is used?

Done

The buckets are not deleted after a failure in the code, since there is no exception, we should try to clean it up.

Added bucket removal logic to error logging cases.

Also, the error field should be used for stack trace alone, so, do you think we should change the error field here to message

Removed few of the cases. But in cases where awscli itself passes (i.e. returns exit code 0) and still the test case fails (say object is downloaded but hash verification fails) I dont see other option, as there is no stack to be published and yet we need to tell in the log what caused the test case to fail.

If AWS returns an error output, they need to be converted to a single line?

done

Should we log the entire awscli command here in the case of function ? Will it be useful to search if the function contains actual endpoint information, bucket names etc? Or can we simplify the function value to just what we are trying to do here, that way it is consistent between endpoint A and endpoint B.

I understand this will be taken care based on the discussion we had during today's call.

@balamurugana found one more issue with this log. The duration is a number and it should not have double quotes around the log.

The duration is a number and it should not have double quotes around the log.

Fixed

authors: kannappanr nitisht

repo: minio/mint


text:
Tag repository with minishift version

We should tag/branch this repository when a minishift version happens so we can keep compatibility between the addons here provided and the minishift versions.
At some point in time we might want to provide 2 different versions of an addon, to support old minishift versions (CDK) and new minishift versions that provide new features (e.g. multiline descriptions in 1.5.0 that do not work in previous releases).

This would not guarantee that an add-on will work. I hope the author will check their add-ons from time to time. But also, the app's here are outside of our support. They are meant as a way for the community to distribute their work.
I think this should actually the solved by https://github.com/minishift/minishift/issues/1313

A little of topic, but still on it: Couldn't the minishift team provide some infra to run tests which exist for addons?
I.e. minishift team provides infra, and the addons provide the tests for validation.

@fabiand please create an issue for this. But the problem is nested virtualization is mostly not available on public CI. Also, the time to run this would likely be very long. For each addon we would need to recreate the instance.

I've filed #27 - We - KubeVirt - might also rely on minishift for demo purpose. Eventually we can run some CI on our infra (which supports nesting and parallel jobs). But I need to check this.

@fabiand please comment this on #27 instead

@fabiand removed comments here...

Tagging alone won't indeed help I think. I think in this case we need to make compatibility more explicit. We already discussed this in the Minishift issue https://github.com/minishift/minishift/issues/1134, where we talked about introducing an additional optional meta data header to enforce a certain OpenShift version. The same could be done (if needed for Minsihift version). This way, add-on developers can gain some more control for add-ons where these things matters. Other add-ons I expect to stay stable, unless even the OpenShift API changes in a major way.

minishift/minishift#1134 and minishift/minishift#1313 together should be enough I think. We can keep this issue open just to track the two issues in Minishift repository .

authors: LalatenduMohanty fabiand gbraad hferentschik jorgemoralespou

repo: minishift/minishift-addons


text:
rename 'standard' to 'certificated' in cucumber tests

Cucumber tests refer to 'standard' checks when they mean 'certificated' - this converts these to use the better terminology

The goal of those step definitions was originally (prior to adding the controlled work flow) to mean "normal" assessments, i.e. default, vanilla, no special cases so that we didn't have to explicitly spell out dates of birth and proceeding types in every scenario when we didn't care about them.
The problem, we quickly found, is that the idea of "normal" goes stale very fast. E.g. CFE used to think of domestic abuse cases as "normal", because that's what Apply dealt with, but for CCQ, DA cases are a special case (and this difference in thinking was a contributory cause of last week's CFE bug).
I'm not sure that these tests actually do mean 'certificated' - but I think what they actually mean - "normal" - is not helpful. Which is a longwinded way of saying I approve of this change, but from a slightly different angle!

authors: patrick-laa starswan


text:
Minor application form fix

resolve display error when fee empty

Go!

authors: colinbruce dotemacs

repo: ministryofjustice/fr-staffapp


text:
Updates to guidance

For domestic violence & insolvency

@jitenp - are you able to review this and merge?

Looks good, Tests are green. Merging.

authors: jitenp leighmoney


text:
SAA-1159 split transactions within a potentially long-running job

Problem: The createScheduledInstances job was running as a single, long transaction, and immediately after a prison migration this is potentially a long job.  It was created sync events for uncommitted data, and the sync service was detecting these events and calling back for the data before it was committed.
This change alters the transaction boundaries such that each activity is treated as its own transaction.
It also fixes some issues with the job runner - this job was recorded two job results every time it ran, so this is also now resolved.

Codecov Report
Patch coverage is 92.85% of modified lines.

Files Changed
Coverage

...mentapi/service/ManageScheduledInstancesService.kt
92.85%

:loudspeaker: Thoughts on this report? Let us know!.

authors: codecov-commenter timharrison-moj


text:
822 - ACCT endpoint

Implement GET ACCT Alerts endpoint
Draft because there's an outstanding question on the Trello ticket around whether we should only return active alerts

Also, I think we have a decision on the active/historical question: we do want to show historical (no longer active) ACCT alerts.

authors: brownbearsoftware edavey


text:
Component Analysis, e.g. JFrog Artifactory

Someone (from HMPPS?) asked about this, at some point.
I haven't heard of any other requests for it since then, so I suggest we wait and see if anyone chases before we do this.

Taking SonarQube on Cloud Platform and offering it as a service to teams outside CP.

Stale. No user needs identified.

authors: AntonyBishop SteveMarshall digitalronin


text:
UML-807 - Remove unused certificates

Purpose
The environment build is looking for a certificate that is not required.
Fixes UML-807
Approach
Remove the data sources for the certificates in the Terraform environment configuration.
Remove the certificates from the Terraform account configuration
Checklist

[x] I have performed a self-review of my own code
[ ] I have added relevant logging with appropriate levels to my code
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] The product team have tested these changes

Coverage remained the same at 81.076% when pulling a2808a3c938186d70b964bb02feb3310f98296b1 on UML-807 into b3299bd9f573df4d03e11f9220a424a468e639a8 on master.

authors: andrewpearce-digital coveralls

repo: ministryofjustice/opg-use-an-lpa


text:
Prevent CSRFs and 404s after reporting a problem

Attempting to login with a google address that isn't in permitted domains will
lead to a CSRF error when redirecting back to the original page.  This PR allows
us to detect that, and redirect to the login page.

👍

authors: jsugarman stephenrichards

repo: ministryofjustice/peoplefinder


text:
Update Sudbury Christmas visit slots

Unbookable:

Christmas Day
Boxing Day
New Year's Day

Additional slots:

23rd December 0945-1145 and 1345-1545
24th December 0945-1145 and 1345-1545
28th December 0945-1145 and 1345-1545
29th December 0945-1145 and 1345-1545
30th December 0945-1145 and 1345-1545
31st December 0945-1145 and 1345-1545

Tested locally

authors: EGiataganas kyliemulholland

repo: ministryofjustice/prison-visits


text:
Improvement to sql generator, added validation

What does this pull request do?
adds validation for CSV files
What is the intent behind these changes?
to reduce risk of system template data being incorrectly configured

Had to fix some tests as they did not work on a FRIDAY because code does not return date for current date and day of week was Friday.

repo: ministryofjustice/visit-scheduler


text:
Initialization Cells in Jupyter Books

Is there a way when using Thebelab in a Jupyter Book to have certain hidden cells run first as initialization cells (e.g. cells that load dependencies). I usually like to keep these hidden in textbooks so this feature would be very useful!

Ah I just realized that I'd asked about this several months back already :-)
here's the relevant issue:
https://github.com/minrk/thebelab/issues/94
I think in Jupyter Book, we could play around with the fix suggested there (to add javascript that listens for the thebelab event, and when it hears it, runs any cell that matches some kind of tag query).
So I think this could be a JB fix and not require changes in thebelab...

Let's close this one since I think it's covered in #94 and will hopefully be closable soon!

authors: choldgraf chrispyles

repo: minrk/thebelab


text:
adding $moai token

excluding treasury, mutant supply and burn wallet from circulating supply.

gotta love windows making things harder.

repo: minswap/market-cap


text:
🛑 Matrix is down

In ba56820, Matrix (https://matrix.mint.lgbt:8448/_matrix/federation/v1/version) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Matrix is back up in ae2b6a9 after 2 hours, 29 minutes.

repo: mint-lgbt/status


text:
Logging interface tap_error_log prints the wrong source crate

When prints happen from tap_error_log, the source of the error is lost and the source is considered to be the utils crate itself. This is undesirable as it hides the source of problems. We should find a way to get around this.

Horrible hacks aside, I am afraid it probably needs to be implemented as a macro to support this.

Horrible hacks aside, I am afraid it probably needs to be implemented as a macro to support this.

Yep. I can see that's the only way. Though we have to at least acknowledge and study this because it's causing damage in debugging.
Another solution, btw, is code duplication... not the best either.

authors: TheQuantumPhysicist iljakuklic

repo: mintlayer/mintlayer-core


text:
Workspace dependencies

This is mostly for discussion. I think that it is convenient to specify the versions of widely used dependencies in the root toml file. It is even more important for things such as parity-scale-codec, even though we have a CI check.

Yes, it was added in the 1.64 version.
It is convenient to specify the version only once, but I'm not sure when exactly a dependency should be moved to the workspace file. For now I have moved the dependencies that are used in 4+ crates.


text:
Predictable Caching

I imagine there are caching strategies implemented in Irmin. But I can't find anything Googling this.
Where can I read more about it? Is it exposed to the users in some way? (Like, is it possible to disable it?)

caches are handled via https://mirage.github.io/irmin/irmin/Irmin/module-type-S/Tree/index.html You can clear tree caches by using Tree.clear (https://mirage.github.io/irmin/irmin/Irmin/module-type-S/Tree/index.html#caches) -- you can use it to clear the cache after every operation. Caches for content-adressable stores are tricky to get right (and you need them if you don't want your operations to be awfully slow). Happy to discuss further/improve that API via the irmin's Github issue tracker if you'd like.

In the context of blockchains, predictable access times (and thus predictable caching) and fast access times are comparably important.
The main reason is that blockchains have a rough computational model of user provided operations, in the form of gas. Mis-priced operations in terms of gas (as in, operations which gas cost don't reflect the actual underlying cost) could lead to DOS attacks. The typical way to do so would be to create a block that contains operations that are much more computationally expensive than expected, thus halting miners/bakers.
If everything fits in the cache right now, then it is not an immediate problem. But that won't scale as the state grows bigger, So I'm looking for ways to model this explicitly, and for an API that supports this.

Most of the functions in Tree now takes a cache argument to control caching since irmin 2.8.

authors: Galfour samoht

repo: mirage/irmin

license: isc


text:
Add benchmarks for trees

Not sure what are the best benchmarks for inodes, I'm introducing three modes here for now:

dune exec -- ./bench/irmin-pack/tree.exe --mode=tezos-log
consists of mimicking tezos tree operations: tezos.log contains the tree add and remove operations for the first 21480 commits of a bootstrap. Each commit starts with an empty tree.
Add large trees

 dune exec -- ./bench/irmin-pack/tree.exe --mode=large --width=1000000 --nlarge=1 --ncommits=2
 +1610285517us  application Large trees mode: 2 commits, each consisting of 1 large trees of 1000000 entries completed in 76.20s
 +1610285517us: total size of store : 237 M

Add long trees

dune exec -- ./bench/irmin-pack/tree.exe --mode=chains --depth=1000000 --nchain=3 --ncommits=2
+1610285741us  application Chain trees mode: 2 commits, each consisting of 3 chains of depth 1000000 completed in 146.80s
+1610285741us: total size of store : 437 M

The default configuration (dune exec ./bench/irmin-pack/tree.exe) produces the output:
Configuration:
 Tezos_log mode: 13315 commits (895827 adds; 54196 removes; 2950152 finds; 669692 mems; 439329 mem_tree; 59 copies)

Results for inodes [32, 256]:
 Total time: 65.475349
 Size on disk: 156 M

Configuration:
 Chain trees mode: 10 commits, each consisting of 2 chains of depth 1000

Results for inodes [32, 256]:
 Total time: 0.213851
 Size on disk: 0 M
Results for inodes [2, 5]:
 Total time: 0.207955
 Size on disk: 0 M

Configuration:
 Large trees mode: 10 commits, each consisting of 1 large trees of 1000000 entries

Results for inodes [32, 256]:
 Total time: 361.190747
 Size on disk: 1005 M
Results for inodes [2, 5]:
 Total time: 788.810630
 Size on disk: 1694 M

and takes around 32 minutes, but we can tweak this if we want faster benchmarks.
For the tezos-log benchmarks for now I’m only verifying the correctness of mem and find, and it’s failing at commit 13316. But I think we can start using these benchmarks in this state, and we can add more checks and a fixed trace later on.
The tezos traces are produced from  https://github.com/tarides/tezos/tree/logs.

The default configuration (dune exec ./bench/irmin-pack/tree.exe) produces the output:
Configuration:
 Tezos_log mode: 13315 commits (895827 adds; 54196 removes; 2950152 finds; 669692 mems; 439329 mem_tree; 59 copies)

Results for inodes [32, 256]:
 Total time: 65.475349
 Size on disk: 156 M

Configuration:
 Chain trees mode: 10 commits, each consisting of 2 chains of depth 1000

Results for inodes [32, 256]:
 Total time: 0.213851
 Size on disk: 0 M
Results for inodes [2, 5]:
 Total time: 0.207955
 Size on disk: 0 M

Configuration:
 Large trees mode: 10 commits, each consisting of 1 large trees of 1000000 entries

Results for inodes [32, 256]:
 Total time: 361.190747
 Size on disk: 1005 M
Results for inodes [2, 5]:
 Total time: 788.810630
 Size on disk: 1694 M

and takes around 32 minutes, but we can tweak this if we want faster benchmarks.
For the tezos-log benchmarks for now I’m only verifying the correctness of mem and find, and it’s failing at commit 13316. But I think we can start using these benchmarks in this state, and we can add more checks and a fixed trace later on.
The tezos traces are produced from  https://github.com/tarides/tezos/tree/logs.

I moved the trace file to a different repo (https://github.com/icristescu/dataset) which uses git-lfs.
This means that for opam pin or for installing the dataset we need git-lfs, so the tree benchmarks are not compiled by the CI (see commit).
Also if anyone wants to run them, they need  git-lfs. Let me know if you think this is ok.

I moved the trace file to a different repo (https://github.com/icristescu/dataset) which uses git-lfs.
This means that for opam pin or for installing the dataset we need git-lfs, so the tree benchmarks are not compiled by the CI (see commit).
Also if anyone wants to run them, they need  git-lfs. Let me know if you think this is ok.


text:
Update dependencies and Ember

📖 Description
This updates dependencies and Ember just so we’re ready to start a new project!
📝 Notes

Changes to the code were made to fix linting and formatting issues
We can finally replace our ember-cli-code-coverage fork with the official package because it now supports TypeScript correctly! 🎉

🦀 Dispatch

#dispatch/ember

🦀 Requesting reviewers for this pull request:

@loboulet (contributor with 0 commits in the last 90 days and 24 commits overall)
@ddallaire (reviewer for the ember stack)

🦀 Requesting reviewers for this pull request:

@loboulet (contributor with 0 commits in the last 90 days and 24 commits overall)
@ddallaire (reviewer for the ember stack)

authors: charlesdemers mirego-builds

repo: mirego/ember-boilerplate


text:
提交最新的修改，修复页面不显示问题，添加作者信息（Submit the latest changes, fix the page not displaying problem, add author information）

提交最新的修改，修复页面不显示问题，添加作者信息（Submit the latest changes, fix the page not displaying problem, add author information）
由于作者没有更新，我看到lssues中有问题提交，代替修复此扩展（Since the author has not updated, I see there are issues in lssues to commit instead of fixing this extension）

请求验证通过

repo: miru-project/repo


text:
Error when I click on shipping zones or create new shipping zone

What I'm trying to achieve
Create a shipping zone or edit one.
Steps to reproduce the problem

click on create shipping zone

What I expected to happen
I expect to create a new shipping zone
Screenshots

System information
Operating system: macOS Catalina
Browser: Chrome

@JuanTwoThree Hey! Thanks for reporting. We'll check it out.

authors: JuanTwoThree karolkielecki

repo: mirumee/saleor-dashboard


text:
Exact environment?

Hi Guys,
This is a super interesting work. Would you guys consider sharing the exact environment you used in this work? I'm getting a number of errors when trying to install it.
For example, after creating a clean environment, I get:
$ conda create -n synflownet
$ conda activate synflownet
(synflownet) $ conda list
(synflownet) # packages in environment at /blue/lic/seabra/.conda/envs/synflownet:
#
# Name                    Version                   Build  Channel
$ pip install -e . --find-links https://data.pyg.org/whl/torch-2.1.2+cu121.html
Defaulting to user installation because normal site-packages is not writeable
Looking in links: https://data.pyg.org/whl/torch-2.1.2+cu121.html
Obtaining file:///blue/lic/seabra/source/synflownet
  Installing build dependencies ... done
  Checking if build backend supports build_editable ... done
  Getting requirements to build editable ... done
  Preparing editable metadata (pyproject.toml) ... done

INFO: pip is looking at multiple versions of gflownet to determine which version is compatible with other requirements. This could take a while.
ERROR: Could not find a version that satisfies the requirement torch==2.1.2 (from gflownet) (from versions: 2.2.0, 2.2.1, 2.2.2, 2.3.0, 2.3.1, 2.4.0, 2.4.1)
ERROR: No matching distribution found for torch==2.1.2

As I suspect this is some package compatibility problem, would you consider sharing the precise environment (including Python, torch, and gflownet versions) you used to run SynFlowNet?
Thanks a lot!

I managed to install it cleanly by following:
$ conda create -n synflownet
$ conda activate synflownet
(synflownet) $ conda install python=3.10
(synflownet) $ pip install importlib-resources datasets==2.0.0 dill huggingface-hub==0.7.0 multiprocess xxhash accelerate huggingface-hub tokenizers transformers rich==10.11.0
(synflownet) $ pip install -e . --find-links https://data.pyg.org/whl/torch-2.1.2+cu121.html

Thanks!

repo: mirunacrt/synflownet


text:
404 chosen.css not found

followed your build instructions, and it don't work.  From the server I get:
Failed to load resource: server responded with a status of (404 Not Found): http://localhost:9000/bower_components/chosen/chosen.css
This problem seems to appears because you only included chosen.min.css in the package that bower installs.
Am I doing something wrong here?

Are you sure that the bower_compenents folder is setup to be served properly by your local host server?

Yes, I have other components under that folder and they are working well.  As I mentioned, the issue appears to the that there is no chosen.css under that folder (only a chosen.min.css)

Just had another look now, you shouldn't need to include chosen.css as the angular-timezone-selector.css file is built with all of the CSS from chosen that is needed.

This problem is actually happening because of - https://github.com/harvesthq/chosen/issues/2324 . If we are going with including the chosen.css in angular-timezone-selector.css, should we remove the dependency on chosen in our bower.json?

It needs to stay in bower.json so that it will be downloaded to be built into the CSS when a fresh install is done

authors: a-k-g ashokfernandez davehorton

repo: mishguruorg/angular-timezone-selector


text:
Buy URLs

Will look at namecheap. Thinking something along the lines of the name of the repo here.
Drop the ideas below of what's available.

Options:

mishfriendlyfood.com
mishfood.com
food.dev ($1K)
food.melbourne ($12K)
food.tech ($20K)

Looked at:

mishfriendly.food but .food is not a TDL

Bought mishfriendlyfood.com and mishfood.com
Purchased!

Bought mishfriendlyfood.com and mishfood.com

repo: mishmanners/mish-friendly-food


text:
🛑 La bonne alternance is down

In eb7897f, La bonne alternance (https://labonnealternance.apprentissage.beta.gouv.fr/) was down:

HTTP code: 503
Response time: 821 ms

Resolved: La bonne alternance is back up in a6b2331 after 8 minutes.

repo: mission-apprentissage/upptime


text:
Exception wrapping

I've been trying to improve the exception handling of MPF/MC as I work—whenever I get a crash that requires me to scroll through the callstack or look in the logs, I update the Exception chain to try and get the meaningful error into the final callstack message.
I think we should do a 50.4 release soon, so making a PR for the changes I've got so far.

Nice

repo: missionpinball/mpf-mc


text:
Question on the feedfoward torque and contact detection

Hi, I'm looking forward to estimating the foot contacts of MiniCheetah. I'm thinking about if it is possible to use the feedback term of motor positions to detect a contact, as shown in
   // estimate torque
    datas[leg].tauEstimate =
        legTorque +
        commands[leg].kpJoint * (commands[leg].qDes - datas[leg].q) +
        commands[leg].kdJoint * (commands[leg].qdDes - datas[leg].qd);

in  LegController::updateCommand.
My question is that, is the feedforward term, legTorque  always zero? If not, how is it computed? Does it have different values when the foot is in the air and the food is touching the ground?  The code related to legTorque is
    // tauFF
    Vec3<T> legTorque = commands[leg].tauFeedForward;

    // forceFF
    Vec3<T> footForce = commands[leg].forceFeedForward;

    // cartesian PD
    footForce +=
        commands[leg].kpCartesian * (commands[leg].pDes - datas[leg].p);
    footForce +=
        commands[leg].kdCartesian * (commands[leg].vDes - datas[leg].v);

    // Torque
    legTorque += datas[leg].J.transpose() * footForce;

is the feedforward term, legTorque always zero?

No. The legTorque contains the input joint torque for all the actuators of a given leg, so it cannot be zero.

If not, how is it computed?

The controller is responsible for generating the appropriate leg torques for a specified action. One simple example, you can checkout the FSM_Standup where the input torque is calculated based on desired foot position. Here, the desired foot positions is calculated based on the interpolation of pDes[2](Z-axis of foot). And based on the error between the current foot position and desired foot position the required footForce is calculated. Finally, a Jacobian transformation of footForce gives you the required joint torques of each actuator of a leg.

Does it have different values when the foot is in the air and the food is touching the ground?

As stated earlier, these are commands and it's values are determined by the controller. The foot being in air/ground doesn't directly affect it's value.

repo: mit-biomimetics/Cheetah-Software


text:
Sc/mpi4py

Branch for mpi4py

Preview link:
https://mit-orcd.github.io/orcd-docs-previews/PR/PR21/

repo: mit-orcd/orcd-docs-edit


text:
Expose mrb_iv_get/set, mrb_gv_get/set to Go in a consistent fashion.

Re-submission of #67 due to flawed pull request.
Added *MrbValue.GetInstanceVariable, *MrbValue.SetInstanceVariable, *Mrb.GetGlobalVariable, *Mrb.SetGlobalVariable and their underlying C methods to allow Go to pass variables to/from Ruby.
This is honestly a staple of ANY scripting implementation.

sorry I got back to this so late.
great tests; thanks for your contribution!

authors: erikh fabian-f

repo: mitchellh/go-mruby


text:
[FALSE-POSITIVE] cs-portalintranet.com

Domains or links
https://cs-portalintranet.com/
More Information
How did you discover your web site or domain was listed here?
Incorrectly marked as Phishing on virustotal.
Have you requested removal from other sources?
We've requested removals from a handful of other providers, but they don't provide links / references to the removal request.
Additional context
Please can you remove the cs-portalintranet.com site from your block list. This site is associated with the CybSafe PHISH product, which is a security training and awareness tool designed to simulate phishing scenarios for the purpose of training individuals to recognise and respond appropriately to phishing threats. We’d greatly appreciate it if you could remove it from your lists. If you require more information, then please feel free to get in touch with us by replying here or emailing support AT cybsafe.com.

Do not find you listed by us but you are by others
https://www.virustotal.com/gui/domain/cs-portalintranet.com

authors: andrew-cybsafe mitchellkrogza


text:
EVEN HERE, There seems to be no FREE LOCOMOTION ...

I am so tired of 'teleporters'...  They will be the death of VR yet.
There's a youtube video to 'turn the VR Template into Free Locomotion'   WHAT A JOKE,. THE TURNING TAKES YOU ON A BIG SWOOPING PUKE'Y ARC THAT EVEN MAKES ME SICK..  I SWEAR I ALMOST QUIT VR AFTER THAT 'TUTORIAL'  ....  No wonder there is really nothing (NOTHING) good in VR yet,.. Any Artist has zero hope of WRANGLING the beast that is UNREAL, the BASIC RESOURCES like a 'free VR locomotion template',
Does ANYBODY know of a simple VR Locomotion sample project or template with PROPER TURNING?  WHY HAS UNREAL NEVER ADDED A FREE LOCOMOTION VR TEMPLATE????
Thank you for any help,..  Not sure if I'm even posting in the exact right area, I think it is an 'issue' with the tool.  A huge omission .. It is not even mentioned...  it is certainly my 'ISSUE'  :)   It's limiting my releasing an alpha build\
Robert.

I'm not too sure if this is the right place, but I did write an instructor guide with Epic that has an implementation of free locomotion. It's an expansion on some of the systems of my VR Cookbook and might be of use to you.
You can get it here: https://www.unrealengine.com/en-US/blog/unreal-engine-4-instructor-guides-are-now-available?sessionInvalidated=true
I'm gonna close this issue for now as it doesn't really relate to this repo but feel free to reply if you have any issues with the Content Examples themselves.

authors: NextWorldVR mitchemmc

repo: mitchemmc/VRContentExamples


text:
Ensure that footer renders at the bottom of page

When there is little content in the body of the page, the footer is rendered directly beneath that content and leaves a significant space underneath. The footer should render at the bottom of the page when there is not enough content to fill the screen, but should not be pinned to the bottom regardless of scroll position

@gsidebo I'm assigning it to you since you have associated a PR that fixes this issue.

repo: mitodl/mitxonline


text:
Switch some settings to use urljoin

Pre-Flight checklist

[x] Testing

[x] Code is tested
[x] Changes have been manually tested

What are the relevant tickets?
N/A - this fixes a small avoidable issue w/ configuration
What's this PR do?
This fixes the refine dashboard settings to have correct urls regardless of SITE_BASE_URL having a trailing slash or not. I wasted a bit of time trying to debug oauth issues related to this.
How should this be manually tested?
You should be able to login to the refine dashboard having a base url with and without a trailing slash.

Codecov Report

Merging #802 (23a28f5) into main (08a3a66) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #802      +/-   ##
==========================================
- Coverage   85.42%   85.40%   -0.02%
==========================================
  Files         132      132
  Lines        5798     5798
  Branches      954      954
==========================================
- Hits         4953     4952       -1
- Misses        692      693       +1
  Partials      153      153

Impacted Files
Coverage Δ

main/settings.py
93.23% <ø> (ø)

ecommerce/discounts.py
78.94% <0.00%> (-1.76%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

authors: codecov-commenter rhysyngsun


text:
PyVim on Mitogen blacklist with vmware_guest module

It appeas that vmware_guest has a dependency that is in a blacklist within Mitogen. I'm unsure whether this is expected behavour (I've searched the codebase and can't find any references to pyVim).
Example snippet from my role:
120 - name: Create VM from template
121   vmware_guest:
122     validate_certs: False
123     hostname: "{{ vcenter_hostname }}"
124     username: "{{ vcenter_user }}"
125     password: "{{ vcenter_pass }}"
126     cluster: "LAB"
127     datacenter: Rack
128     name: "{{ vmname }}"
129     template: "{{ vmtemplate }}"
130     folder: /
131     state: poweredon
132     networks:
133       - name: "{{ vmnetwork }}"
134         start_connected: true
135         ip: "{{ vmipaddr }}"
136         netmask: "{{ vmnetmask }}"
137         gateway: "{{ vmgateway }}"
138         domain: "{{ domain_root }}"
139         dns_servers:
140           - "{{ dns_server_ip }}"
141         type: static
142     customization:
143       dns_servers:
144         - "{{ dns_server_ip }}"
145     hardware:
146       memory_mb: "{{ vmmem }}"
147       num_cpus: "{{ vmcpu }}"
148   register: newvm

The traceback:
The full traceback is:
Traceback (most recent call last):
  File "master:/usr/local/lib/python2.7/site-packages/ansible/module_utils/vmware.py", line 32, in <module>
    from pyVim import connect
  File "<stdin>", line 1323, in load_module
  File "<stdin>", line 1254, in _refuse_imports
ImportError: u'pyVim' is present in the Mitogen importer blacklist, therefore this context will not attempt to request it from the master, as the request will always be refused.

Ansible version:
➜  ansible ansible --version
ansible 2.8.5
  config file = /Users/dan/.ansible.cfg
  configured module search path = [u'/Users/dan/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/lib/python2.7/site-packages/ansible
  executable location = /usr/local/bin/ansible
  python version = 2.7.16 (default, Jun 19 2019, 07:40:37) [GCC 4.2.1 Compatible Apple LLVM 10.0.1 (clang-1001.0.46.4)]

Mitogen version:
➜  ansible pip show mitogen
 Name: mitogen
Version: 0.2.8
Summary: Library for writing distributed self-replicating programs.
Home-page: https://github.com/dw/mitogen/
Author: David Wilson
Author-email: UNKNOWN
License: New BSD
Location: /usr/local/lib/python2.7/site-packages
Requires:
Required-by:

Ansible config:
➜  ansible-config dump --only-changed`
ANSIBLE_PIPELINING(/Users/dan/.ansible.cfg) = True
ANSIBLE_SSH_ARGS(/Users/dan/.ansible.cfg) = -o ControlMaster=auto -o ControlPersist=60s
ANSIBLE_SSH_RETRIES(/Users/dan/.ansible.cfg) = 3
DEFAULT_FILTER_PLUGIN_PATH(/Users/dan/.ansible.cfg) = [u'/etc/ansible/plugins/filter']
DEFAULT_FORKS(/Users/dan/.ansible.cfg) = 30
DEFAULT_GATHERING(/Users/dan/.ansible.cfg) = smart
DEFAULT_INTERNAL_POLL_INTERVAL(/Users/dan/.ansible.cfg) = 0.001
DEFAULT_POLL_INTERVAL(/Users/dan/.ansible.cfg) = 1
DEFAULT_ROLES_PATH(/Users/dan/.ansible.cfg) = [u'/etc/ansible/roles', u'/usr/share/ansible/roles', u'/ansible/roles']
DEFAULT_STRATEGY(/Users/dan/.ansible.cfg) = mitogen_linear
DEFAULT_STRATEGY_PLUGIN_PATH(/Users/dan/.ansible.cfg) = [u'/Users/dan/.ansible/mitogen/ansible_mitogen/plugins/strategy']
HOST_KEY_CHECKING(/Users/dan/.ansible.cfg) = False
RETRY_FILES_ENABLED(/Users/dan/.ansible.cfg) = False

Are you able to advise a workaround, or whether this is unexpected at all?

This is still a problem with Mitogen 0.3.3. To work around it, delegate the task to localhost (that way Mitogen doesn't have to transfer the module) with delegate_to: localhost on the task or disable Mitogen for that task (requires putting it in a separate play, strategy plugins can't be specified on the role/task level) like this:
- hosts: localhost
  strategy: linear
  tasks:
    - <vmware module>

- hosts: all
  strategy: mitogen_linear # can be omitted if set in ansible.cfg
  <role/task list>

authors: dsgnr faivirol

repo: mitogen-hq/mitogen


text:
use getattr in jinja2.filters.make_attrgetter instead of getitem

IMO, jinja2.filters.make_attrgetter should use environment.getattr instead of environment.getitem, which tries Python's getattr first and then falls back to attempting to treat the object as a dict/list.
For context, we ran into an issue when trying to use the sort filter's attribute argument on a list of security-proxied objects; trying to use index notation on them raises an exception that is not caught in environment.getitem.
I've tested this change minor change locally and all of the tests pass.

That absolutely makes sense, in my opinion. The dot notation and the fact that filter like groupby and sort call the argument used for the lookup attribute implies that Environment.getattr() instead of Environment.getitem() is used. I am totally surprised that this is not the case, right now.

Unfortunately, I just discovered a problem with using getattr() instead of getitem(). If you use a value for the lookup, that isn't a string, a TypeError is raised and no item lookup is done. That's probably the reason why getitem() is used here.

authors: kilink wallunit


text:
rye init issue

When I type "rye init" command without define a project name, it automaticlly creates initial files in the local folder.
I think it's better to show the --help menu instead of just create the file directly.
Thanks.

This would feel unnatural to me since I interpret rye init as "init here" (rye init .). I don't interpret it as "tell me more about this command".

OK, I didn't realize rye init = rye init .  The dot is a default directory.
Since it is used for creating a new project, should it better to have user manually define the project name?
Or maybe specifly in documentation that rye init means created project in local folder?
Just some thoughts, thanks.

@yalunren let me know what you think about #401. Whenever I'm unsure of the usage for a command I'd use --help. So hopefully that PR at least makes it more clear.

@yalunren let me know what you think about #401. Whenever I'm unsure of the usage for a command I'd use --help. So hopefully that PR at least makes it more clear.

Got it. Thanks for the update!

authors: cnpryer yalunren

repo: mitsuhiko/rye


text:
fix: always use gcc_64 as arch_dir for linux

From Qt 6.7.0, arch keyword is changed to linux_gcc_64 but arch_dir should always be gcc_64.
The change fixes issue #769 Fails to install Qt6 6.7.0 on Linux/Ubuntu

FYI this is also fixed in https://github.com/miurahr/aqtinstall/pull/766/files#diff-b336135f575fd47d95d02ec92ce8e4fa51a5add90b3d2057af8ecad46b204891R388
That fix also accounts for the new gcc_arm64 arch dir.

authors: miurahr rectalogic

repo: miurahr/aqtinstall


text:
Unhandled Exception: RangeError (index)

RangeError encountered while running on IOS device.
Need to pass additional "country" parameter in _getIOSStoreVersionAndUrl, should be "bundleId" and "country" parameters.

Thank you for taking time to report.
"country" parameter has been added to ios/mac request.
0.2.2 has been released.

add country parameter when requesting ios/mac version info (default='us').
optional country argument added to VersionCheck constructor.


text:
Persisting tweaks

We've just integrated our first a/b tests and we noticed the values of the tweaks are not persisted. So if the user is in cohort B and then kill/restart the app, the value of the tweak will be re-fetched.

Is that on purpose?
If it is, is there any plan to add persistency?
Is there any harm if we persist it ourselves?

Hi @RomainPiel! I can definitely see the benefit of persisting tweaks. My concern is that you may load tweaks locally that were already removed remotely. That means that after a request, those tweaks will be removed and that could be a very odd user experience.
What is your user case and why do you need to persist them? I want to understand what you are going through :)

Ah right, I see how it could be problematic if the tweak is removed remotely. I guess instead of removing it we could return the initial default value from the server.
So my use case is that I have a A/B test on my very first screen. So it can happen and it happens quite frequently that the tweak value is initialised with some delay after the view is rendered. I don't really want to wait for it to come back so the UX is a bit odd.
Caching it fixes it for the subsequent launches (not really for the first one but that's acceptable).
Thanks for your help!

Setting the initial value would be an equivalent of removing/updating a tweak after another one is loaded.
For your use case, I think it'd be an improvement if we persist the tweaks. However, you'd face the same situation as soon as you remove them remotely. If that's something that does not concern you, I'd like to implement this on the SDK in such a way that loading persisted tweaks does not happen by default unless you specify so in an Activity or in the app.
Alternatively, on our latest SDK release we provide callbacks when tweaks are updated (see changelog and how to set it up https://github.com/mixpanel/mixpanel-android/releases/tag/v4.9.2). In your app, would that be a solution? I.e. render the UI after tweaks are updated from the server.

I agree, the best solution would be to handle it in the SDK because it has full context on wether the tweak still exists or not.
We have considered using addOnMixpanelTweaksUpdatedListener() but it wouldn't work for us, we don't want to block the UI until we get the experiments and we don't want to alter the UI after it has been rendered.
In any case, the problem of the tweak being removed on the server side is not a massive issue.
Assuming that version 1 has a 50/50 A/B test. Then eventually maybe B wins. We can set it to 100% B for a while. Then we roll out version 2 that only has feature B with no tweak check. After some time version 1 is not much used, we can drop the tweak remotely.
Obviously it would be awesome if the SDK was handling it, it would mean less code on our end and a cleaner implementation. But the remote deletion is not a biggie.

Something that I could do (and that the SDK could do) is to set an expiration date on that cached value. That's what Google is doing with firebase remote controls: https://firebase.google.com/docs/remote-config/android#caching

@RomainPiel Thanks for the link! I see a lot of benefits persisting the tweaks moving forward. I will keep this in my radar but i can't give you an ETA at this moment. In the mean time, feel free to implement it on your side and open a PR. I'll be happy to review it.
Thanks!

Cool, I'd be happy to help but I'm not very familiar with the code. So any quick walkthrough the potential changes would be greatly appreciated.
I've had a quick look at the code and it seems that there is some persistency happening during initialization in ViewCrawler. These mPersistentTweaks are applied to the shared tweaks. I'm wondering why these values are not reused after a re-launch.

Does mixpanel iOS sdk persists toggles? -- because I guess it does. if yes can we check how they its implemented in iOS and can get better insights?cc @asifmujteba

Any updates on this?

Starting on the next version (5.2.0) we will start persisting tweaks https://github.com/mixpanel/mixpanel-android/pull/492
Thanks for your feedback guys!

authors: AamirAbro RomainPiel athkalia patedit

repo: mixpanel/mixpanel-android


text:
how to disable install track

Hi, I am using a mixpanel for my app but I don't need to use installrefer for my case, however, if I remove
    implementation 'com.android.installreferrer:installreferrer:2.2' from my build.gradle,  I got errors when I try to call mixpanel API to just capture event. Is there a way to use mixpanel API without adding installreferrer:installreferrer dependency ?
I am tried 5.9 and 6.0-beta-1, all have same issue.
Thanks
Rejecting re-init on previously-failed class java.lang.Class<com.mixpanel.android.mpmetrics.InstallReferrerPlay>: java.lang.NoClassDefFoundError: Failed resolution of: Lcom/android/installreferrer/api/InstallReferrerStateListener;
    at com.mixpanel.android.mpmetrics.MixpanelAPI com.mixpanel.android.mpmetrics.MixpanelAPI.getInstance(android.content.Context, java.lang.String) (MixpanelAPI.java:253)
    at com.mixpanel.android.mpmetrics.MixpanelAPI ai.dongmi.barbell.manager.EventTrackingManager$mixpanel$2.invoke() (EventTrackingManager.kt:30)
    at java.lang.Object ai.dongmi.barbell.manager.EventTrackingManager$mixpanel$2.invoke() (EventTrackingManager.kt:29)
    at java.lang.Object kotlin.SynchronizedLazyImpl.getValue() (LazyJVM.kt:74)
    at com.mixpanel.android.mpmetrics.MixpanelAPI ai.dongmi.barbell.manager.EventTrackingManager.getMixpanel() (EventTrackingManager.kt:29)

Hi @yangcheng , we have released v6.0.0 which no longer contains this feature anymore so it should be good. I'm closing this one now.

thanks! ugprading now

authors: yangcheng zihejia


text:
Adding flags to disable notifications and surveys to avoid unneeded d…

…ecide requests

these config options will disable a/b tests as well right? why not have one configuration option to just turn off decide rather than multiple options where the end user will not know turning off one also turns off the other features?

@ryanseams nice catch and I agree with the idea of turning decide check of.

This is not a good solution in general. Besides what @ryanseams said, current implementation will disable everything (tweaks, notifications, surveys...) only by having notifications OR surveys off.

duplicate.. closing

authors: patedit ryanseams yinfeiru


text:
Python 3.4 stated, actually 3.5 needed

The README.md states that one needs at least Python 3.4 - running on 3.4.2 failed (default py version on Ubuntu 14.04, with 16.04 it's fine as > 3.5 is the default version here). There is an import for typings which seems to be introduced first with Python 3.5.

You're correct, Have you tried to install the typing package manually via pip3?

Nope, I simply upgraded my Docker image to 16.04 and therefore to Python
3.5.2. Just opened this issue for the docs, 14.04 was a bit old anyway ;)
On 5 Dec 2016 7:41 p.m., "Johannes Bechberger" notifications@github.com
wrote:

You're correct, Have you tried to install the typing package manually via
pip3?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mj3-16/mjtest/issues/57#issuecomment-264938550, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFVDXXGbx6ccmhN9K5k9sfDI14TIRIaaks5rFFrngaJpZM4LEQD-
.

authors: FlorianLoch parttimenerd


text:
mj-text does not inherit font-size from mj-container

<mjml>
  <mj-body>
    <mj-container font-size="32">
      <mj-section>
        <mj-column>
          <mj-text>Hello world!</mj-text>
        </mj-column>
      </mj-section>
    </mj-container>
  </mj-body>
</mjml>

Why font-size in mj-container does not affect to mj-text?

@liminspace I think you're missing the px. <mj-container font-size="32px">

it not helps

@liminspace it's always better to specify properties on each elements when coding a mail ! A bunch of desktop and web email clients need css on every single elements to render the whole thing good ;)

I think MJML can make layout more easy and DRY principe is possible.

Well I think we'll remove it in a near future, #79 will easily fix this behaviour and is more flexible

authors: iRyusa jarsbe liminspace pixelize

repo: mjmlio/mjml


text:
[SUGGESTION] Add package to packagist.org - update composer file

I cannot find the package in http://packagist.org, it should be added manually via the packagist website.
A sample composer.son file contents should be:
{
  "name": "mjolnic/bootstrap-colorpicker",
  "description": "A nice and customizable colorpicker plugin for Twitter Bootstrap 2 and 3",
  "license": "Licensed under the Apache License, Version 2.0",
  "authors": [
    {
      "name": "Javier Aguilar",
      "homepage": "http://www.mjolnic.com/"
    }
  ],
  "minimum-stability": "dev",
  "require": {
    "components/jquery" : ">=1.10",
    "twbs/bootstrap": "3.x"
  }
}

It turns out that tabs/bootstrap is a full package of the bootstrap and is not recommended to be used as a dependency. Common practice is to use components/bootstrap for a light version of the bootstrap. Sorry for the misleading.
https://packagist.org/packages/components/bootstrap

repo: mjolnic/bootstrap-colorpicker


text:
Use goroutines for each license and add features aggregate expiration

Main change was to execute each license collection in parallel using goroutines. Also add metric to indicate if failures occurred during collection.
Added metric flexlm_feature_aggregate_expiration_seconds that groups features for a license by expiration times. I use this in an alert so that if a license has 25 features expiring soon we only get 1 alert and not 25 alerts.
The diff is not as big if you use something like git diff -w to remove whitespace changes.
Alert we are using:
  rules:
  - record: flexlm_feature_aggregate_expiration_days
    expr: (flexlm_feature_aggregate_expiration_seconds - time()) / 86400
  - alert: LicenseExpiring
    expr: flexlm_feature_aggregate_expiration_days < 14
    for: 5m
    labels:
      notify: daily
      alertgroup: licenses
      severity: warning
      alsonotify: license-admins
    annotations:
      title: License {{ $labels.license }} expiring soon on {{ $labels.host }}
      description: License {{ $labels.license }} on {{ $labels.host }} has {{ $labels.features }} features ({{ $labels.licenses }} licenses) expiring in {{ $value }} days

Coverage decreased (-3.4%) to 33.615% when pulling 4e993ee6b4ada07a333cc45f2f5259410fbcafa2 on treydock:osc into 346cffd5a775daa48f36c498ba615ba6d43511b7 on mjtrangoni:master.

Hi @treydock,
Thanks for your PR! Could you please take a look at the linter errors? I will be checking your PR at weekend.

Fixed lint issues. Had to increase dupl to threshold of 200 because it kept saying 2 functions were identical when they were not identical.  If want to keep threshold at 100 I could add an ignore comment instead.

LGTM. Thanks again for your PR!
Could you please also add your additional rules for the Alert Manager here?

I may need to adjust the way aggregate labels are done. I ran into issue where the index label kept changing so alerts began to flap.
I'll update PR with a fix once I've verified it on my end and will also update README with alert examples

Don't know if the right solution but am sorting the aggregate expirations to hopefully avoid labels from switching back and forth. Ran into issue where expired license and new license kept flipping between different index numbers and this resulted in lots of alerts / resolutions.
Also added README example of alert I'm using for expiration.

This should be good to merge if everything looks good. I have been running this PR on our systems for a bit now and everything is working as expected.

@treydock thanks!

authors: coveralls mjtrangoni treydock


text:
Problems using custom dataset to train Adaface

I am trying to train AdaFace with my own dataset: a custom folder ('data'), a subfolder called 'imgs' that has a collection of other folders ('folder_001', 'folder_002', etc) with various photos of the same face inside each one ('f_001.jpg', 'f_002.jpg' etc). There are in total 30000 folders with images. I am using the following script to train (a small change from 'run_ir50_ms1mv2.sh'):
python main.py \
    --data_root /mnt/disk/data \
    --train_data_path imgs \
    --prefix ir50_ms1mv2_adaface \
    --gpus 1 \
    --use_16bit \
    --arch ir_50 \
    --batch_size 256 \
    --num_workers 8 \
    --epochs 26 \
    --lr_milestones 12,20,24 \
    --lr 0.1 \
    --head adaface \
    --m 0.4 \
    --h 0.333 \
    --custom_num_class 30000 \
    --low_res_augmentation_prob 0.2 \
    --crop_augmentation_prob 0.2 \
    --photometric_augmentation_prob 0.2

The result:
FileNotFoundError: [Errno 2] No such file or directory: '/mnt/disk/data/faces_emore/agedb_30/meta/sizes'
For some reason it keeps searching for validation dataset folders such as 'agedb_30', 'faces_emore' that don't exist in my project. Why are these datasets required? Do I need to set val_data_path the same as train_data_path? Am I missing some other parameter that would make this work?
I also tried to overcome this by following the README_TRAIN.md instructions closely and downloading the dataset 'faces_webface_112x112', preprocessing it with convert.py and then replacing the folder 'imgs' with my own 'imgs' folder. The result:
FileNotFoundError: Found no valid file for the classes agedb_30, faces_emore. Supported extensions are: .jpg, .jpeg, .png, .ppm, .bmp, .pgm, .tif, .tiff, .webp
Finally, it is not clear to me if I have to first convert my RGB image training dataset to BGR before training AdaFace and if I can use different resolution images to train Adaface (e.g. 224x224) than the standard 112x112

Hi trnikon.
The agedb30 is one of the 5 validation sets that are used for tracking the model performance during training. If you do not need these, you should change the validation dataset and data_loader with your own dataset.
For creating the agedb30, it will be created in this line of code. https://github.com/mk-minchul/AdaFace/blob/76f4ce203a9f768cf6c118c02124ddbae3c3dce9/convert.py#L89

Thanks by your updating  @mk-minchul
I got the same log: "No such file or directory: ... /agedb_30/meta/sizes", in my case I'm trying to use another folder with images (like in train folder: ) to validate the results. For example:
  -"train" folder:_
    -"1"..."62" folders:
      -images of each folder class (62)
 -"val" folder:
    -"1"..."62" folders not in train:
      -face images of each folder class (62).

In my case how can I use a different validation dataset and data_loader, taking base from my "val" folder? and do not use neither  agedb_30 or .rec file.
Regards

Have you managed to solve this?

I am trying to train AdaFace with my own dataset: a custom folder ('data'), a subfolder called 'imgs' that has a collection of other folders ('folder_001', 'folder_002', etc) with various photos of the same face inside each one ('f_001.jpg', 'f_002.jpg' etc). There are in total 30000 folders with images. I am using the following script to train (a small change from 'run_ir50_ms1mv2.sh'):
python main.py \
    --data_root /mnt/disk/data \
    --train_data_path imgs \
    --prefix ir50_ms1mv2_adaface \
    --gpus 1 \
    --use_16bit \
    --arch ir_50 \
    --batch_size 256 \
    --num_workers 8 \
    --epochs 26 \
    --lr_milestones 12,20,24 \
    --lr 0.1 \
    --head adaface \
    --m 0.4 \
    --h 0.333 \
    --custom_num_class 30000 \
    --low_res_augmentation_prob 0.2 \
    --crop_augmentation_prob 0.2 \
    --photometric_augmentation_prob 0.2

The result: FileNotFoundError: [Errno 2] No such file or directory: '/mnt/disk/data/faces_emore/agedb_30/meta/sizes'
For some reason it keeps searching for validation dataset folders such as 'agedb_30', 'faces_emore' that don't exist in my project. Why are these datasets required? Do I need to set val_data_path the same as train_data_path? Am I missing some other parameter that would make this work?
I also tried to overcome this by following the README_TRAIN.md instructions closely and downloading the dataset 'faces_webface_112x112', preprocessing it with convert.py and then replacing the folder 'imgs' with my own 'imgs' folder. The result: FileNotFoundError: Found no valid file for the classes agedb_30, faces_emore. Supported extensions are: .jpg, .jpeg, .png, .ppm, .bmp, .pgm, .tif, .tiff, .webp
Finally, it is not clear to me if I have to first convert my RGB image training dataset to BGR before training AdaFace and if I can use different resolution images to train Adaface (e.g. 224x224) than the standard 112x112

Have you solved this?

yes, use the rec file to create the folder to test data using the code.
python convert.py --rec_path <DATASET_ROOT>/<DATASET_NAME> --make_image_files --make_validation_memfiles
and put the path <DATASET_ROOT> in the comand to train

yes, use the rec file to create the folder to test data using the code.
python convert.py --rec_path <DATASET_ROOT>/<DATASET_NAME> --make_image_files --make_validation_memfiles
and put the path <DATASET_ROOT> in the comand to train

Hello,
I would like to use an other dataset for the validation (ylfw). Do you please have an idea,  how I should structure the dataset folder to get adaface run on it?

authors: ANDRESHZ martinenkoEduard mk-minchul trnikon vkouam

repo: mk-minchul/AdaFace


text:
[PSA] Server migration

A quick message to contributors: the Feedback Friends server has just been migrated, and the DNS now points to my personal VM. The dedicated VM will expire on 2016-10-18, until then you can still access it if there's anything you want to backup.
I'll keep the site up for an undetermined time, but unless the situation evolves so that LD37 needs it, or if there's popular demand I don't think it will be enabled/publicized in the future.
Thanks to all contributors, and @ttencate in particular for making the mini-site much better than initially planned! Hopefully FF will be inspire cool social features for the future Ludum Dare site.

And thank you for getting this ball rolling in the first place, and being so open and welcoming towards contributions! Here's to a fruitful collaboration :beers:

authors: mkalam-alami ttencate

repo: mkalam-alami/ludumdare-feedback-friends


text:
Sim2net python3

I keep running into a syntax error on the print statement after running the sim2net -i. I have realized that this is due to sim2net being written in python 2 and I am using python 3. Is there a python3 version of sim2net?

Back when I used sim2net i was using python 2.
I doubt there is a version for python 3.
On Sat, May 2, 2020 at 7:36 PM amitneil notifications@github.com wrote:

I keep running into a syntax error on the print statement after running
the sim2net -i. I have realized that this is due to sim2net being written
in python 2 and I am using python 3. Is there a python3 version of sim2net?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mkalewski/sim2net/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI2SVJ4MLPMNHA23PBU4AFDRPRDZ5ANCNFSM4MXX62GA
.

--
Fatima Al-Haj, BSc, MSc Information SystemsPhD Student @ Computer
Science University of Jordan, Amman


text:
Plugins?

First of all, thank you from the bottom of my heart for keeping this project going.
ConfuserEx supports plugins but I have yet to find any. My fault, no doubt.
Any leads would be most appreciated.

As far as I am aware, there aren't any plugins that are publicly available.
I know of a few cases, where there users of ConfuserEx created custom plugins for their specific projects, but those are all private developments.

Here is an example of a plugin: https://github.com/HoLLy-HaCKeR/Confuser.Protections.HoLLy
It cannot be loaded by the 2.0 branch because the plugin system has changed.

Here's some more.
https://github.com/xHeaven/ConfuserEx-Additions
https://github.com/xHeaven/KoiVM-modded
Cheers

Thank you, guys. Much appreciated.

authors: ElektroKill LaraSQP mkaring xHeaven

repo: mkaring/ConfuserEx


text:
Maskify extension fun

closes #12

ga bisa squash merge ya? prefer squash merge sih kalo gini

I will be check about squash merge, didn't notice so far.

ga bisa squash merge ya? prefer squash merge sih kalo gini

Squash merge uda di allow ya. silakan dicoba aja. thanks

authors: desdulianto mkdika

repo: mkdika/jeneric-kt


text:
🛑 FarmPi Mirror is down

In fe0ef3c, FarmPi Mirror (https://farmpi.octofarm.net/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: FarmPi Mirror is back up in 608deda after 8 minutes.


text:
MKJAILDATASET - mount point expectations

@feld I wanted to discuss MKJAILDATASET - it seems the mountpoint is always /mkjail.
The code does not assume to know the mountpoint and users can modify that as they wish. The code always extracts that mountpoint, e.g. SRCPATH in /usr/local/share/mkjail/getrelease.sh:
SRCPATH="$(zfs get -H mountpoint ${ZPOOL_MKJAIL_DB}/${MKJAILDATASET} | awk '{print $3}')/${VERSION}"
The reason for the discussion is to document our conclusions in mkjail.conf.sample, if only for my comfort because I can't always remember it. This came up today as I was creating patches for a new variable, ZPOOL_MKJAIL_DB (used above).

@feld Is this low risk enough?

repo: mkjail/mkjail


text:
Wiki documentation needs an update?

Environment

Python: 2.7.12
pyodbc: 4.0.24
OS: Mac OS X 10.12.6 (Sierra)
DB: SqlServer
driver: freetds 1.00.94 with unixodbc 2.3.7

Issue
After following the instructions at https://github.com/mkleehammer/pyodbc/wiki/Connecting-to-SQL-Server-from-Mac-OSX and banging my head against the wall for a few hours, I discovered I had two odbc.ini files and two odbcinst.ini files. One set in /usr/local/etc and another set in /usr/local/Cellar/unixodbc/2.3.7/.bottle/etc. I edited the unixodbc set and could not get my python connection to work. After editing the other set, it worked as expected.
Maybe a mention in the doc is warranted.

The Wiki page suggests checking odbcinst -j to determine the correct location of odbcinst.ini, was it pointing at the wrong one?

Hmmm. It was... but I also had a terrible time installing and a bit of a mess from a previous attempt to use pymssql. So maybe something was off. I ended up removing everything and starting over and it's now reporting the right location. Sorry for the false alarm.

authors: ruebrenda v-makouz

repo: mkleehammer/pyodbc


text:
feat: add optional custom footer text

Status
READY
Description
Adds the possibility to add a text to the footer.
Type of Change

[x] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[ ] 🗑 Chore

I see your idea, but it has some flaws:

It probably won't work properly on a split slide template.
I think it would make more sense to refactor the current socialHandle text in the footer and make it configurable. E.g. if the showSocialHandle is true, then show the handle as it is now. Then, if the footer text is provided, show it instead of the social handle.

What could be even better, instead of limiting to use only text, allow to provide any widget for the footer (e.g. if you want to show logo instead of a social handle). The logic could be similar - if you provide the widget for the footer configuration, use it instead of the social handle. Of course, it should be clearly defined in the documentation.
What do you think?

How do you like this new approach? Replace the String? text with a Widget? widget and the approriate logic for it.

Nice progress, you got the idea right, just some minor adjustments are still needed (check the comments). Also, it would be nice if you define this widget for the whole slide deck in the example and also update the README.md accordingly with the updated code snippet.

I wouldn't want to set this widget for the whole deck, since we only want to show the example on the footer slide. Or did I miss something here?

Nice progress, you got the idea right, just some minor adjustments are still needed (check the comments). Also, it would be nice if you define this widget for the whole slide deck in the example and also update the README.md accordingly with the updated code snippet.

I wouldn't want to set this widget for the whole deck, since we only want to show the example on the footer slide. Or did I miss something here?

My point is that it would be nice to show that it's possible to define this widget once for it to be rendered for the whole slide deck. Maybe we can do it in the README.md file, in the following section:
Also, you can define a global configuration for your slide deck:

```dart
FlutterDeckApp(
  configuration: const FlutterDeckConfiguration(
    background: FlutterDeckBackgroundConfiguration(
      light: FlutterDeckBackground.solid(Color(0xFFB5FFFC)),
      dark: FlutterDeckBackground.solid(Color(0xFF16222A)),
    ),
    controls: FlutterDeckControlsConfiguration(
      nextKey: LogicalKeyboardKey.arrowRight,
      previousKey: LogicalKeyboardKey.arrowLeft,
      openDrawerKey: LogicalKeyboardKey.period,
    ),
    footer: FlutterDeckFooterConfiguration(
      showSlideNumbers: true,
      showSocialHandle: true,
    ),
    header: FlutterDeckHeaderConfiguration(
      showHeader: false,
    ),
    showProgress: false,
    transition: FlutterDeckTransition.fade(),
  ),
  <...>
);

Nice progress, you got the idea right, just some minor adjustments are still needed (check the comments). Also, it would be nice if you define this widget for the whole slide deck in the example and also update the README.md accordingly with the updated code snippet.

I wouldn't want to set this widget for the whole deck, since we only want to show the example on the footer slide. Or did I miss something here?

My point is that it would be nice to show that it's possible to define this widget once for it to be rendered for the whole slide deck. Maybe we can do it in the README.md file, in the following section:
Also, you can define a global configuration for your slide deck:

```dart
FlutterDeckApp(
  configuration: const FlutterDeckConfiguration(
    background: FlutterDeckBackgroundConfiguration(
      light: FlutterDeckBackground.solid(Color(0xFFB5FFFC)),
      dark: FlutterDeckBackground.solid(Color(0xFF16222A)),
    ),
    controls: FlutterDeckControlsConfiguration(
      nextKey: LogicalKeyboardKey.arrowRight,
      previousKey: LogicalKeyboardKey.arrowLeft,
      openDrawerKey: LogicalKeyboardKey.period,
    ),
    footer: FlutterDeckFooterConfiguration(
      showSlideNumbers: true,
      showSocialHandle: true,
    ),
    header: FlutterDeckHeaderConfiguration(
      showHeader: false,
    ),
    showProgress: false,
    transition: FlutterDeckTransition.fade(),
  ),
  <...>
);

I already added
    footer: FlutterDeckFooterConfiguration(
      showSlideNumbers: true,
      showSocialHandle: true,
      widget: _MyCustomFooter(),
    ),
``` there, is it good enough?

Nice progress, you got the idea right, just some minor adjustments are still needed (check the comments). Also, it would be nice if you define this widget for the whole slide deck in the example and also update the README.md accordingly with the updated code snippet.

I wouldn't want to set this widget for the whole deck, since we only want to show the example on the footer slide. Or did I miss something here?

My point is that it would be nice to show that it's possible to define this widget once for it to be rendered for the whole slide deck. Maybe we can do it in the README.md file, in the following section:
Also, you can define a global configuration for your slide deck:

```dart
FlutterDeckApp(
  configuration: const FlutterDeckConfiguration(
    background: FlutterDeckBackgroundConfiguration(
      light: FlutterDeckBackground.solid(Color(0xFFB5FFFC)),
      dark: FlutterDeckBackground.solid(Color(0xFF16222A)),
    ),
    controls: FlutterDeckControlsConfiguration(
      nextKey: LogicalKeyboardKey.arrowRight,
      previousKey: LogicalKeyboardKey.arrowLeft,
      openDrawerKey: LogicalKeyboardKey.period,
    ),
    footer: FlutterDeckFooterConfiguration(
      showSlideNumbers: true,
      showSocialHandle: true,
    ),
    header: FlutterDeckHeaderConfiguration(
      showHeader: false,
    ),
    showProgress: false,
    transition: FlutterDeckTransition.fade(),
  ),
  <...>
);

I already added
    footer: FlutterDeckFooterConfiguration(
      showSlideNumbers: true,
      showSocialHandle: true,
      widget: _MyCustomFooter(),
    ),

there, is it good enough?

Ahh, true, I missed this somehow. Yup, all good.

authors: Goddchen mkobuolys


text:
Table headers block scrollbar when the scroll container has padding

When the scroll container has left and/or right padding the width of the floatThead-container is not calculated correctly and the table headers are positioned over the scrollbar which makes the top of the scrollbar not usable.
Steps to reproduce:

Add a div with left and right padding around the table and add the floatthead plugin on the table with the div as the scrollContainer.
Now try to use the top of the scrollbar by clicking on the top button of the scrollbar or dragging the scrollbar down from the top.

JSFiddle: https://jsfiddle.net/rickhoutman/ptsh2y9L/4/

Yeah that looks like a bug, here is a workaround:
https://jsfiddle.net/0fo643ry/1/

Yes, the workaround works, but also the scrollbar falls into the padding, so quite difficult to get styling correct this way.

Can you explain what you mean by that..

Ok, maybe I see what you mean. How about this:
https://jsfiddle.net/7hgmxp9j/1/

Yes, that is better. What I meant was that the scrollbar in my example was on the right side of the wrapper div and extended to the top and bottom of the wrapper and the table was padded into the wrapper.

What are the chances this would come up the exact day I have the same issue. Thank you both for this discussion. I was able to resolve my issues following along here.

I looked at all these fiddles and comments and I cannot see what the original issue was that were were talking about. Its possible that something was fixed in a browser, or maybe this was a windows scrollbar issue, and i am on a mac now.
If someone knows what this is all about, feel free to reopen

authors: burgoyne mkoryak rickhoutman

repo: mkoryak/floatThead


text:
Style Transfer with Webcam - Video - Not working or very slow

I've tried to use the "Style Transfer with Webcam" demo both online and offline with the source code, but it's not working on my computer. It  loads the model, but when I press "start", it just freeze the camera. The online version on Chrome (not on Firefox) worked after waiting a lot, but it was still very laggy. Is it possible to make it faster and make the source code work?

How many FPS you get largely depends on your computer specifications. What are your specifications?

Yes, I didn't considered my computer, because it usually works well (I have tried also on my phone and the computer of a friend of mine, with same results: webcam freezed).
I expected it to not be computationally intensive, since it was part of the tutorials.
Here my specifications:
System Information
Operating System: Windows 7 Ultimate 64-bit (6.1, Build 7601)
BIOS: Default System BIOS
Processor: Intel(R) Xeon(R) CPU X3470  @ 2.93GHz (8 CPUs), ~2.9GHz
Memory: 8192MB RAM
PS: I'd like to use ml5 also to make graphics for a musical project, so for me it would be important to be able to make the program run smoothly. Thanks for your work by the way, this project is really cool!

I'm not able to reproduce the problem. Any updates on your side?

yep, it is. Are you having the same issue?

@cvalenzuela, yes!
Actually I am running a 5 fps 100×100 video on a gaming laptop, then also I am facing the same issue as above.

I'm not able to reproduce the problem. Any updates on your side?

No, I still have the same problem. I think it's not a problem if it needs some requirements: this functionality has a lot of potential and can be worth buying some hardware, but I think it would be useful to quantify them, having some suggested minimum requirements to make it run and requirements to make it run smoothly.
I think it could be useful also to understand if it's a hardware or software problem, since it seems it could be related to the kind of browser/server.


text:
Pruning (ARGUMENT CAPTURE) can be safely moved to after the new ivars are added to arg_choices

Not doing until after camera ready since it could slightly change pruning orders which could confuse us, but it generally seems like it could be slightly better to do argument capture later like this! Wont change what is pruned overall but could prune branches a little earlier sometimes

furthermore should be fine to have it iterate over first_zid_of_ivar instead of arg_choices

done 8408fa1d27af64e5e49dc9afdf2997e9a7d4a278


text:
[Bug] _pickle.UnpicklingError: invalid load key, 'v' when compiling model

🐛 Bug
I am not sure if it's a bug, or if I am the one doing something wrong, but I basically followed the steps, installed mlc-llm as it was explained in docs, and it gives me an error.
To Reproduce
Follow steps from here.
lesiu@pop-os:~/coding/compile_models/redpanjama3b$ python3 -m mlc_llm.build --hf-path togethercomputer/RedPajama-INCITE-Chat-3B-v1 --target webgpu --quantization q4f16_1
Weights exist at dist/models/RedPajama-INCITE-Chat-3B-v1, skipping download.
Using path "dist/models/RedPajama-INCITE-Chat-3B-v1" for model "RedPajama-INCITE-Chat-3B-v1"
Target configured: webgpu -keys=webgpu,gpu -max_num_threads=256
Automatically using target for weight quantization: vulkan -keys=vulkan,gpu -max_num_threads=256 -max_shared_memory_per_block=65536 -max_threads_per_block=1024 -supports_16bit_buffer=1 -supports_float16=1 -supports_float32=1 -supports_int16=1 -supports_int32=1 -supports_int8=1 -thread_warp_size=1
Get old param:   0%|                               | 0/388 [00:00<?, ?tensors/sStart computing and quantizing weights... This may take a while.00<?, ?tensors/s]
Get old param:   0%|                       | 1/388 [00:00<02:09,  2.98tensors/s]Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/home/lesiu/.local/lib/python3.10/site-packages/mlc_llm/build.py", line 47, in <module>
    main()
  File "/home/lesiu/.local/lib/python3.10/site-packages/mlc_llm/build.py", line 43, in main
    core.build_model_from_args(parsed_args)
  File "/home/lesiu/.local/lib/python3.10/site-packages/mlc_llm/core.py", line 866, in build_model_from_args
    params = utils.convert_weights(mod_transform, param_manager, params, args)
  File "/home/lesiu/.local/lib/python3.10/site-packages/mlc_llm/utils.py", line 272, in convert_weights
    vm["transform_params"]()
  File "tvm/_ffi/_cython/./packed_func.pxi", line 332, in tvm._ffi._cy3.core.PackedFuncBase.__call__
  File "tvm/_ffi/_cython/./packed_func.pxi", line 263, in tvm._ffi._cy3.core.FuncCall
  File "tvm/_ffi/_cython/./packed_func.pxi", line 252, in tvm._ffi._cy3.core.FuncCall3
  File "tvm/_ffi/_cython/./base.pxi", line 182, in tvm._ffi._cy3.core.CHECK_CALL
  File "/home/lesiu/.local/lib/python3.10/site-packages/tvm/_ffi/base.py", line 481, in raise_last_ffi_error
    raise py_err
  File "tvm/_ffi/_cython/./packed_func.pxi", line 56, in tvm._ffi._cy3.core.tvm_callback
  File "/home/lesiu/.local/lib/python3.10/site-packages/mlc_llm/utils.py", line 37, in inner
    return func(*args, **kwargs)
  File "/home/lesiu/.local/lib/python3.10/site-packages/mlc_llm/relax_model/param_manager.py", line 599, in get_item
    load_torch_params_from_bin(torch_binname)
  File "/home/lesiu/.local/lib/python3.10/site-packages/mlc_llm/relax_model/param_manager.py", line 559, in load_torch_params_from_bin
    torch_params = torch.load(
  File "/home/lesiu/.local/lib/python3.10/site-packages/torch/serialization.py", line 1028, in load
    return _legacy_load(opened_file, map_location, pickle_module, **pickle_load_args)
  File "/home/lesiu/.local/lib/python3.10/site-packages/torch/serialization.py", line 1246, in _legacy_load
    magic_number = pickle_module.load(f, **pickle_load_args)
_pickle.UnpicklingError: invalid load key, 'v'.
lesiu@pop-os:~/coding/compile_models/redpanjama3b$

Expected behavior
I excepted the model to compile to my target (webgpu)
Environment
Python version: Python 3.10.12
OS: Pop!_OS 22.04

This error is thrown by PyTorch, and is likely because the weights in dist/models/RedPajama-INCITE-Chat-3B-v1 are non-existent or incomplete

Let me know if it's case

@junrushao
dist/models/RedPajama-INCITE-Chat-3B-v1$ ls
config.json             README.md                tokenizer.json
generation_config.json  special_tokens_map.json
pytorch_model.bin       tokenizer_config.json

It seems all the files were downloaded, but indeed they seem to be incomplete - in my case pytorch_model.bin is just 135 bytes, when it should be much more.

opened a tracking issue https://github.com/mlc-ai/mlc-llm/issues/1319 on better error message. in the meantime, @Dolidodzik can you try to redownload the files?

Well, sorry for that but for some reason i am getting another issue now completely, and I don't even know why. I am getting ValueError: Multiple weight shard files without json map is not supported, I tried adding --use-safetensors param but I still couldn't get it to work.
lesiu@pop-os:~/coding/compile_models$ python3 -m mlc_llm.build --hf-path togethercomputer/RedPajama-INCITE-Chat-3B-v1 --target webgpu --quantization q4f16_1 --use-safetensors
Weights exist at dist/models/RedPajama-INCITE-Chat-3B-v1, skipping download.
Using path "dist/models/RedPajama-INCITE-Chat-3B-v1" for model "RedPajama-INCITE-Chat-3B-v1"
Target configured: webgpu -keys=webgpu,gpu -max_num_threads=256
Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/home/lesiu/.local/lib/python3.10/site-packages/mlc_llm/build.py", line 47, in <module>
    main()
  File "/home/lesiu/.local/lib/python3.10/site-packages/mlc_llm/build.py", line 43, in main
    core.build_model_from_args(parsed_args)
  File "/home/lesiu/.local/lib/python3.10/site-packages/mlc_llm/core.py", line 833, in build_model_from_args
    param_manager.init_torch_pname_to_bin_name(args.use_safetensors)
  File "/home/lesiu/.local/lib/python3.10/site-packages/mlc_llm/relax_model/param_manager.py", line 292, in init_torch_pname_to_bin_name
    mapping = load_torch_pname2binname_map(
  File "/home/lesiu/.local/lib/python3.10/site-packages/mlc_llm/relax_model/param_manager.py", line 921, in load_torch_pname2binname_map
    raise ValueError("Multiple weight shard files without json map is not supported")
ValueError: Multiple weight shard files without json map is not supported

@tqchen @junrushao

Well the input format is not safetensor but plain pytorch. @MasterJH5574 could you take a look as the author of this piece of code?

@Dolidodzik Hi! Do you mind sharing the files you re-downloaded under dist/models/RedPajama-INCITE-Chat-3B-v1 again here? When you use --use-safetensors, the error happens when there are multiple files ended with .safetensor while the file model.safetensors.index.json does not exist. So I am wondering what the files under your directory are.

The thing is, weather or not I use --use-safetensors or not, the output is the same.
Well, actually new thing happened - when I was lsing directory you asked about, I realized that .bin file is missing, so I downloaded it manually from huggingface, and placed it there. And now I again get another issue... (Sorry for being so incompetent with that stuff, but this is my first dealing with such a thing, and it's pretty novel thing so there aren't many tutorials yet, thank you guys for doing great work on that)
ANYWAY, currently where i am at, my files:
dist/models/RedPajama-INCITE-Chat-3B-v1$ ls
 config.json             'pytorch_model(1).bin'   special_tokens_map.json   tokenizer.json
 generation_config.json   README.md               tokenizer_config.json

My output now after running command:

python3 -m mlc_llm.build --hf-path togethercomputer/RedPajama-INCITE-Chat-3B-v1 --target webgpu --quantization q4f16_1
Weights exist at dist/models/RedPajama-INCITE-Chat-3B-v1, skipping download.
Using path "dist/models/RedPajama-INCITE-Chat-3B-v1" for model "RedPajama-INCITE-Chat-3B-v1"
Target configured: webgpu -keys=webgpu,gpu -max_num_threads=256
Automatically using target for weight quantization: vulkan -keys=vulkan,gpu -max_num_threads=256 -max_shared_memory_per_block=65536 -max_threads_per_block=1024 -supports_16bit_buffer=1 -supports_float16=1 -supports_float32=1 -supports_int16=1 -supports_int32=1 -supports_int8=1 -thread_warp_size=1
Get old param:   0%|                               | 0/388 [00:00<?, ?tensors/sStart computing and quantizing weights... This may take a while.00<?, ?tensors/s]
Get old param:  98%|███████████████████▌| 380/388 [00:11<00:00, 153.15tensors/s]Finish computing and quantizing weights.████████████▍| 505/518 [00:11<00:00, 202.97tensors/s]
Total param size: 1.4645195007324219 GB
Start storing to cache dist/RedPajama-INCITE-Chat-3B-v1-q4f16_1/params
[0518/0518] saving param_517
All finished, 51 total shards committed, record saved to dist/RedPajama-INCITE-Chat-3B-v1-q4f16_1/params/ndarray-cache.json
Finish exporting chat config to dist/RedPajama-INCITE-Chat-3B-v1-q4f16_1/params/mlc-chat-config.json
Save a cached module to dist/RedPajama-INCITE-Chat-3B-v1-q4f16_1/mod_cache_before_build.pkl.
Get old param: 100%|████████████████████| 388/388 [00:29<00:00, 153.15tensors/s][22:33:53] /workspace/tvm/src/target/llvm/codegen_llvm.cc:185: Warning: Set native vector bits to be 128 for wasm32
Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/home/lesiu/.local/lib/python3.10/site-packages/mlc_llm/build.py", line 47, in <module>
    main()
  File "/home/lesiu/.local/lib/python3.10/site-packages/mlc_llm/build.py", line 43, in main
    core.build_model_from_args(parsed_args)
  File "/home/lesiu/.local/lib/python3.10/site-packages/mlc_llm/core.py", line 923, in build_model_from_args
    build(mod, args)
  File "/home/lesiu/.local/lib/python3.10/site-packages/mlc_llm/core.py", line 762, in build
    ex.export_library(args.lib_path, **args.export_kwargs)
  File "/home/lesiu/.local/lib/python3.10/site-packages/tvm/relax/vm_build.py", line 146, in export_library
    return self.mod.export_library(
  File "/home/lesiu/.local/lib/python3.10/site-packages/tvm/runtime/module.py", line 624, in export_library
    return fcompile(file_name, files, **kwargs)
  File "/home/lesiu/.local/lib/python3.10/site-packages/tvm/contrib/emcc.py", line 60, in create_tvmjs_wasm
    libs += [find_lib_path("wasm_runtime.bc")[0]]
  File "/home/lesiu/.local/lib/python3.10/site-packages/tvm/_ffi/libinfo.py", line 166, in find_lib_path
    raise RuntimeError(message)
RuntimeError: Cannot find libraries: wasm_runtime.bc
List of candidates:
/home/lesiu/.local/bin/wasm_runtime.bc
/usr/local/sbin/wasm_runtime.bc
/usr/local/bin/wasm_runtime.bc
/usr/sbin/wasm_runtime.bc
/usr/bin/wasm_runtime.bc
/usr/sbin/wasm_runtime.bc
/usr/bin/wasm_runtime.bc
/usr/games/wasm_runtime.bc
/usr/local/games/wasm_runtime.bc
/home/lesiu/.local/lib/python3.10/site-packages/tvm/wasm_runtime.bc
/home/lesiu/.local/lib/wasm_runtime.bc

Please follow steps here https://llm.mlc.ai/docs/install/emcc.html

Ok, so I tried to. I've setup $TVM_HOME, and installed emcc.

lesiu@pop-os:~/clones/emsdk/scripts$ echo $TVM_HOME
/home/lesiu/TVM_HOME/
lesiu@pop-os:~/clones/emsdk/scripts$ emcc --version
emcc (Emscripten gcc/clang-like replacement + linker emulating GNU ld) 3.1.49 (04a0cad4d4c9f5d62876821274d78cd0a52427af)
Copyright (C) 2014 the Emscripten authors (see AUTHORS.txt)
This is free and open source software under the MIT license.
There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

lesiu@pop-os:~/clones/emsdk/scripts$ emcc --check
emcc (Emscripten gcc/clang-like replacement + linker emulating GNU ld) 3.1.49 (04a0cad4d4c9f5d62876821274d78cd0a52427af)
shared:INFO: (Emscripten: Running sanity checks)

The problem is ./scripts/prep_emcc_deps.sh is missing in my install. I found issue discussing similar problem, but I couldn't make much out of it... I would highly appreciate any help here.
lesiu@pop-os:~/clones/emsdk/scripts$ ls
create_release.py          update_linux_arm64.sh  update_python.py
update_bazel_workspace.py  update_node.py

Okay, so that was the part I was missing. After successfully building TVM on my machine with that script, mlc_llm.build command seems to work. Thank you very much, I appreciate all the help. <3
$ python3 -m mlc_llm.build --hf-path togethercomputer/RedPajama-INCITE-Chat-3B-v1 --target webgpu --quantization q4f16_1
Weights exist at dist/models/RedPajama-INCITE-Chat-3B-v1, skipping download.
Using path "dist/models/RedPajama-INCITE-Chat-3B-v1" for model "RedPajama-INCITE-Chat-3B-v1"
Target configured: webgpu -keys=webgpu,gpu -max_num_threads=256
Load cached module from dist/RedPajama-INCITE-Chat-3B-v1-q4f16_1/mod_cache_before_build.pkl and skip tracing. You can use --use-cache=0 to retrace
[17:31:43] /workspace/tvm/src/target/llvm/codegen_llvm.cc:185: Warning: Set native vector bits to be 128 for wasm32
Finish exporting to dist/RedPajama-INCITE-Chat-3B-v1-q4f16_1/RedPajama-INCITE-Chat-3B-v1-q4f16_1-webgpu.wasm

Well, since that works...
May I ask if there's any documentation/example that shows how to use generated .wasm build in WebLLM? Are there any things that I need to keep in mind, or can I just get any model from huggingface, generate .wasm and provide it to webllm AppConfig?

As of now we have limited model familiy support, see https://llm.mlc.ai/docs/prebuilt_models.html

authors: Dolidodzik MasterJH5574 junrushao tqchen

repo: mlc-ai/mlc-llm


text:
[Serving] Server file movements

This PR moves the server-related files to the subdirectory server/. The previous server.py is now the __main__.py file under directory server/.
This PR also optimizes a bit on __init__.py under the serve namespace.

Merging this for now let us introduce subcommand as @junrushao suggested as a followup
related refs in https://github.com/mlc-ai/mlc-llm/blob/main/python/mlc_chat/main.py#L34


text:
style(librispeech): simplify masking + reshaping

It's not a significant change and doesn't impact performance or results. It simply minifies the code by using built-in functions.
If merged, it's probably best to squash the commits, as I've been editing one line at a time via GitHub's Web Editor.

Hi Luke, did you make any progress on this ? I'm facing a similar issue using flax default LSTM cells and run gets stuck at 75% WER so was curious if you solved the issue

Hi Sourabh, thanks for reaching out!
Unfortunately, I'm still facing the same instabilities as before:

I've only added the length-bucketing we discussed and started padding to multiples of 128 (+21% step/s vs. pad-to-8, +33% step/s vs. pad-to-power-of-2). With these changes, the model now runs 4.2x as fast, but, unfortunately, that's all that happened on my end.
Do you have any idea what we could change to improve stability?

@sourabh2k15 do we still need this with the new changes, or can we close it?

cleaning up old PRs, closing

authors: ClashLuke sourabh2k15 znado

repo: mlcommons/algorithmic-efficiency