text:
cce runtime error:errno=145 segmentation fault

Hi,everyone.Now I run my demo code in cce target,But an error  occurred like this:
[ERROR] RUNTIME(4472)kernal task happen error, error code=0x26, [aicore exception].
[ERROR] RUNTIME(4472)aicore kernel execute failed, device_id=0, stream_id=1, task_id=0, fault kernel_name=myfunc_kernel0, func_name=myfunc_kernel0
[ERROR] AKG:2021-05-05-12:21:33.569.843 [cce_module.cc:232] [cce] Check failed: e == RT_ERROR_NONE: Cce runtime error: errno=145, info=Unknow cce error code
Stack trace:
  [bt] (0) /home/HwHiAiUser/akg/build/libakg.so(dmlc::LogMessageFatal::~LogMessageFatal()+0x58) [0xfffef59e1cb4]
  [bt] (1) /home/HwHiAiUser/akg/build/libakg.so(air::runtime::CceWrappedFunc::operator()(air::runtime::TVMArgs, air::runtime::TVMRetValue*, void**, long*, unsigned long) const+0x610) [0xfffef6f061f4]
  [bt] (2) /home/HwHiAiUser/akg/build/libakg.so(air::runtime::detail::PackFuncVoidAddr_<4, air::runtime::CceWrappedFunc>(air::runtime::CceWrappedFunc, std::vector<air::runtime::detail::ArgConvertCode, std::allocator<air::runtime::detail::ArgConvertCode> > const&, int)::{lambda(air::runtime::TVMArgs, air::runtime::TVMRetValue*)#1}::operator()(air::runtime::TVMArgs, air::runtime::TVMRetValue*) const+0x2d0) [0xfffef6f0885c]
  [bt] (3) /home/HwHiAiUser/akg/build/libakg.so(std::_Function_handler<void (air::runtime::TVMArgs, air::runtime::TVMRetValue*), air::runtime::detail::PackFuncVoidAddr_<4, air::runtime::CceWrappedFunc>(air::runtime::CceWrappedFunc, std::vector<air::runtime::detail::ArgConvertCode, std::allocator<air::runtime::detail::ArgConvertCode> > const&, int)::{lambda(air::runtime::TVMArgs, air::runtime::TVMRetValue*)#1}>::_M_invoke(std::_Any_data const&, air::runtime::TVMArgs&&, air::runtime::TVMRetValue*&&)+0x7c) [0xfffef6f0d8bc]
  [bt] (4) /home/HwHiAiUser/akg/build/libakg.so(std::function<void (air::runtime::TVMArgs, air::runtime::TVMRetValue*)>::operator()(air::runtime::TVMArgs, air::runtime::TVMRetValue*) const+0x78) [0xfffef5a2e6b8]
  [bt] (5) /home/HwHiAiUser/akg/build/libakg.so(air::runtime::PackedFunc::CallPacked(air::runtime::TVMArgs, air::runtime::TVMRetValue*) const+0x5c) [0xfffef5af9200]
  [bt] (6) /home/HwHiAiUser/akg/build/libakg.so(air::runtime::StackVM::Run(air::runtime::StackVM::State*) const+0x14d0) [0xfffef7727b30]
  [bt] (7) /home/HwHiAiUser/akg/build/libakg.so(air::runtime::StackVM::Run(air::runtime::TVMArgs const&, air::runtime::ModuleNode*) const+0x108) [0xfffef7726168]
  [bt] (8) /home/HwHiAiUser/akg/build/libakg.so(air::runtime::StackVMModuleNode::GetFunction(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, air::runtime::ObjectPtr<air::runtime::Object> const&)::{lambda(air::runtime::TVMArgs, air::runtime::TVMRetValue*)#1}::operator()(air::runtime::TVMArgs, air::runtime::TVMRetValue*) const+0x48) [0xfffef772c1b4]

Traceback (most recent call last):

  File "vector_add.py", line 30, in <module>
    mod(a, b, c)

  File "/home/HwHiAiUser/akg/third_party/incubator-tvm/python/tvm/_ffi/function.py", line 144, in __call__
    return f(*args)

  File "/home/HwHiAiUser/akg/third_party/incubator-tvm/python/tvm/_ffi/_ctypes/function.py", line 207, in __call__
    raise get_last_ffi_error()

tvm._ffi.base.TVMError: Traceback (most recent call last):
  [bt] (8) /home/HwHiAiUser/akg/build/libakg.so(air::runtime::StackVMModuleNode::GetFunction(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, air::runtime::ObjectPtr<air::runtime::Object> const&)::{lambda(air::runtime::TVMArgs, air::runtime::TVMRetValue*)#1}::operator()(air::runtime::TVMArgs, air::runtime::TVMRetValue*) const+0x48) [0xfffef772c1b4]
  [bt] (7) /home/HwHiAiUser/akg/build/libakg.so(air::runtime::StackVM::Run(air::runtime::TVMArgs const&, air::runtime::ModuleNode*) const+0x108) [0xfffef7726168]
  [bt] (6) /home/HwHiAiUser/akg/build/libakg.so(air::runtime::StackVM::Run(air::runtime::StackVM::State*) const+0x14d0) [0xfffef7727b30]
  [bt] (5) /home/HwHiAiUser/akg/build/libakg.so(air::runtime::PackedFunc::CallPacked(air::runtime::TVMArgs, air::runtime::TVMRetValue*) const+0x5c) [0xfffef5af9200]
  [bt] (4) /home/HwHiAiUser/akg/build/libakg.so(std::function<void (air::runtime::TVMArgs, air::runtime::TVMRetValue*)>::operator()(air::runtime::TVMArgs, air::runtime::TVMRetValue*) const+0x78) [0xfffef5a2e6b8]
  [bt] (3) /home/HwHiAiUser/akg/build/libakg.so(std::_Function_handler<void (air::runtime::TVMArgs, air::runtime::TVMRetValue*), air::runtime::detail::PackFuncVoidAddr_<4, air::runtime::CceWrappedFunc>(air::runtime::CceWrappedFunc, std::vector<air::runtime::detail::ArgConvertCode, std::allocator<air::runtime::detail::ArgConvertCode> > const&, int)::{lambda(air::runtime::TVMArgs, air::runtime::TVMRetValue*)#1}>::_M_invoke(std::_Any_data const&, air::runtime::TVMArgs&&, air::runtime::TVMRetValue*&&)+0x7c) [0xfffef6f0d8bc]
  [bt] (2) /home/HwHiAiUser/akg/build/libakg.so(air::runtime::detail::PackFuncVoidAddr_<4, air::runtime::CceWrappedFunc>(air::runtime::CceWrappedFunc, std::vector<air::runtime::detail::ArgConvertCode, std::allocator<air::runtime::detail::ArgConvertCode> > const&, int)::{lambda(air::runtime::TVMArgs, air::runtime::TVMRetValue*)#1}::operator()(air::runtime::TVMArgs, air::runtime::TVMRetValue*) const+0x2d0) [0xfffef6f0885c]
  [bt] (1) /home/HwHiAiUser/akg/build/libakg.so(air::runtime::CceWrappedFunc::operator()(air::runtime::TVMArgs, air::runtime::TVMRetValue*, void**, long*, unsigned long) const+0x610) [0xfffef6f061f4]
  [bt] (0) /home/HwHiAiUser/akg/build/libakg.so(dmlc::LogMessageFatal::~LogMessageFatal()+0x58) [0xfffef59e1cb4]
  File "/home/HwHiAiUser/akg/third_party/incubator-tvm/src/runtime/cce/cce_module.cc", line 232
TVMError: Check failed: e == RT_ERROR_NONE: Cce runtime error: errno=145, info=Unknow cce error code

Segmentation fault

Here is demo snippet:
import akg
from akg import tvm
import numpy as np

n = 5

a = tvm.placeholder([n], name='a')
b = tvm.placeholder([n], name='b')
c = tvm.compute([n], lambda i: a + b[n - i - 1])

s = tvm.create_schedule(c.op)

mod = akg.build(s, (a, b, c), 'cce', [], name='myfunc', attrs={}, polyhedral=True, binds=None)

print(mod.imported_modules[0].get_source())

a_np = np.random.random([n]).astype(a.dtype)
b_np = np.random.random([n]).astype(b.dtype)

print(a_np, b_np)
ctx = tvm.context('cce')
a = tvm.nd.array(a_np, ctx)
b = tvm.nd.array(b_np, ctx)
c = tvm.nd.array(np.zeros([n], dtype=a_np.dtype), ctx)
mod(a, b, c)
ctx.sync()
print(c)

I'm confusing about that, it's a simple operator,it makes no sense akg cannot run it.
Thanks!

Hi @MingliSun , thanks for your issue. First of all, I cannot reproduce the error in your description:  TVMError: Check failed: e == RT_ERROR_NONE: Cce runtime error: errno=145, info=Unknow cce error code.
Can you please check your compile option? I am using cmake .. and when I ran your script, I got this error
TVMError: Check failed: ObjectTypeChecker<TObjectRef>: :Check(ptr): Expected type List[Expr] but get Array. So I am trying to resolve this error.
The problem in your code is that you pass a into tvm.compute in this line
-> c = tvm.compute([n], lambda i: a + b[n - i - 1])

Instead, you can try to modify it to
c = tvm.compute([n], lambda i: a[i] + b[n - i - 1])

and the code runs normally. I use index i to get element in array a and I guess this is what you want to compute. If not, you can change it according to your compute logic and remember to retrieve elements in tvm.placeholder using index. And the output is as follows:
#ifdef __CCE_KT_TEST__
#define __aicore__
#else
#define __aicore__ [aicore]
#endif

extern "C"  __global__ __aicore__ void myfunc_kernel0(__gm__ float* __restrict__ a, __gm__ float* __restrict__ b, __gm__ float* __restrict__ compute) {
  __ubuf__  float* b_local_UB = (__ubuf__  float *)(0);
  __ubuf__  float* compute_0_local_UB = (__ubuf__  float *)(32);
  __ubuf__  float* compute_local_UB = (__ubuf__  float *)(32);
  __ubuf__  float* a_local_UB = (__ubuf__  float *)(64);
  copy_gm_to_ubuf(((__ubuf__ float *)b_local_UB + 0), ((__gm__ float *)b + 0), 0, 1, 1, 0, 0);
  set_flag(PIPE_MTE2, PIPE_S, EVENT_ID0);
  wait_flag(PIPE_MTE2, PIPE_S, EVENT_ID0);
  for (int32_t cc1 = 0; cc1 < 5; ++cc1) {
    compute_0_local_UB[cc1] = b_local_UB[(4 - cc1)];
  }
  set_flag(PIPE_S, PIPE_V, EVENT_ID0);
  copy_gm_to_ubuf(((__ubuf__ float *)a_local_UB + 0), ((__gm__ float *)a + 0), 0, 1, 1, 0, 0);
  set_flag(PIPE_MTE2, PIPE_V, EVENT_ID0);
  set_vector_mask((uint64_t)0llu, (uint64_t)255llu);
  wait_flag(PIPE_S, PIPE_V, EVENT_ID0);
  wait_flag(PIPE_MTE2, PIPE_V, EVENT_ID0);
  vadd(((__ubuf__ float *)compute_local_UB + 0), ((__ubuf__ float *)a_local_UB + 0), ((__ubuf__ float *)compute_0_local_UB + 0), 1, 1, 1, 1, 0, 0, 0);
  set_flag(PIPE_V, PIPE_MTE3, EVENT_ID0);
  wait_flag(PIPE_V, PIPE_MTE3, EVENT_ID0);
  copy_ubuf_to_gm(((__gm__ float *)compute + 0), ((__ubuf__ float *)compute_local_UB + 0), 0, 1, 1, 0, 0);
  pipe_barrier(PIPE_ALL);

  uint64_t *statusInScalarBuffer = (uint64_t *) 0x40000;
  uint64_t status0 = get_status();
  uint64_t overflowStatus = status0 & 0x00000000000005B8;
  if (overflowStatus > 0) {
    *statusInScalarBuffer = 1;
  }
}

[0.2562516  0.64928097 0.10287885 0.85447043 0.14970173] [0.6659899  0.09130476 0.17711475 0.17782658 0.22820798]
[0. 0. 0. 0. 0.]

Hope this is helpful.

@MingliSun  There is some difference of the instruction set between Ascend910 and Ascend310, and we now only support Ascend910.

@ckeyever do all versions of akg only support Ascend910 not Ascend310 even the initial version? if so , I will change my target into  Ascend910.Thanks for your reply.

@ckeyever do all versions of akg only support Ascend910 not Ascend310 even the initial version? if so , I will change my target into Ascend910.Thanks for your reply.

Yes, Ascend310 is not uspport at the begining.

Issue will be closed if there is no other question.

authors: MingliSun SijiaYang ckeyever

repo: mindspore-ai/akg


text:
update Go version and use net.ErrClosing

We can now use the net.ErrClosing error instead of error string checking since Go 1.16.
https://github.com/golang/go/issues/4373#issuecomment-769145698

Since 2ae5c408

source: gharchive/issue

repo: minekube/gate

license_type: permissive

license_source: github-api


text:
Add FontAwesome icons

Hello,
This is pull request to add FontAwesome Icons to website. Team is free to change them how they want.
A preview:

@xdev31 this wuld Look amazing!

@tobyplow Thanks for Feedback

In #50 CDNs were unwanted, and without one this will make the website substantially heavier.

@ShadowNinja good point.

@ShadowNinja Thanks for your feedback

Mhh, just visited the font awesome website, and saw the fonts load very slowly, looking at weird replacement symbols instead. I think we don't need this!
Therefore, :-1:

:( I'm very sorry, if you don't like it. I tried to improve website @est31

@xdev31 you don't need to be sorry. Its always good if someone tries to improve the website, even if its rejected. Maybe, if other devs agree it will be even merged against my :-1: ...

Icons would be nice, but I don't like these particular ones. So :-1: for this PR (but another with nicer icons may get my approval)

source: gharchive/pull-request

license: mit

license_source: bigquery


text:
🛑 MineVN Maven & Resource Pack is down

In 3645896, MineVN Maven & Resource Pack (http://app.minevn.net/pack) was down:

HTTP code: 0
Response time: 0 ms

Resolved: MineVN Maven & Resource Pack is back up in 07c17ec.

license: MIT


text:
🛑 MineVN Maven & Resource Pack is down

In ddfcc85, MineVN Maven & Resource Pack (http://app.minevn.net/pack) was down:

HTTP code: 0
Response time: 0 ms

Resolved: MineVN Maven & Resource Pack is back up in 0f41fa0.


text:
SaveAsByTemplate - Excel Dimension Xml is null

Excel Type

[x] XLSX
[ ] XLSM
[ ] CSV
[ ] OTHER

MiniExcel Version: 1.30.0
Description:
I used the same example from the documentation, however the described ( Excel Dimension Xml is null )error occurs.
The .net core version is 6.0

I have the same error "Excel Dimension Xml is null"

authors: BacelarIvanildo laboua

repo: mini-software/MiniExcel


text:
Unicode chars which change length when lowercased

These two characters change byte length when lowercased which is quite
unique behavior. This can potentially cause issues where assumptions
about input length == length after processing.
Credit: https://twitter.com/jifa/status/625776454479970304

It may also be useful to add their lowercased versions, which one would presume (but one does not know) would shrink when uppercased?

Agreed. What if some kind of parsing code reacts much more poorly to skipping a character than duplicating a character and it took an index into the string before lower/uppercasing it on the assumption that the length would remain unchanged?

authors: ansell davbo ssokolow

repo: minimaxir/big-list-of-naughty-strings


text:
Use provided sha256 hasher

Use client provided hasher for SHA256 signatures.
By default this is "minio/sha256-simd" or "crypto/sha256" for FIPS.
Bonus: Skip tests if env var is missing.

@klauspost looks like this needs one more fix

authors: klauspost vadmeste

repo: minio/minio-go


text:
Create booking

Implement create booking endpoint + tests

 Add API endpoints for bookings

repo: ministryofjustice/approved-premises-api


text:
Create and call function createHash in namespace creation

This function will create a sha1 hash of all directories in the newly
created namespace tree. The sha1 hash and the namespace name are written
to a hidden file called .autoApprove.

This is a nice approach. I wonder if it would make sense to post the hash somewhere else, along with a reference to the PR/namespace ("PR" would mean the CLI tool has to actually raise the PR, or at least create a commit), so that the user doesn't have an obvious local artefact they could (potentially) mess with.
I know I'm the one who said, "let's trust the users" but that seems like a relatively cheap way to gain a little bit more security.

Okay interesting - so if I understand it correctly, the local binary would create and commit/PR the auto-approve in the users branch. Would this not add more confusion than an artefact? I suppose both could be explained with a document.
As an alternative, we could remove the artefact altogether and perform the hash and diff at our end (as a GitHub action).

Refactored the hashing dir function as was getting inconsistent results. I will close this PR and create a new one at a later date.

repo: ministryofjustice/cloud-platform-cli


text:
892 - OAsys offence-analysis endpoint

We will need to merge https://github.com/ministryofjustice/hmpps-approved-premises-tools/pull/13/files before this is usable in the local environment.

 Implement OASys "offence analysis" endpoint


text:
Enable RSpec/ExampleLength cop

What
Enable RSpec/ExampleLength
Do we want to enable this cop to

prevent further high block lengths
enable us to whittle some down

Have configured to allow certain structures over multiple
lines and increased default of 5 lines to 10 lines. This
reduces violations from 384 to 73.
We can exclude for now and work on them via the usual rubocop_todo.yml
but wanted to show them failing to start with.
https://docs.rubocop.org/rubocop-rspec/cops_rspec.html#rspecexamplelength
Checklist
Before you ask people to review this PR:

Tests and rubocop should be passing: bundle exec rake
Github should not be reporting conflicts; you should have recently run git rebase main.
There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
The PR description should say what you changed and why, with a link to the JIRA story.
You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
You should have checked that the commit messages say why the change was made.

What problem are we trying to solve by adding a rule like this?
I'm struggling to understand how this could be helpful, so I hope this discussion shines some light on what we hope to gain from this!
There are two main reasons I think this is something we should not be doing:

It will negatively impact how easy it is to understand our tests.
Lines of code as a metric is unhelpful and often counter-productive.

Let’s explore the first point.
Understandability is always preferable to brevity —and this forces us to potentially compromise everything else in the pursuit of... example blocks less than 10 lines?
Generally speaking we try to make our code as readable and maintainable as possible.
That can mean lots of things, but in application code it usually means shorter methods, defined in smaller objects, with fewer responsibilities.
When we’re writing tests, though, that isn’t always the case.
The rule dictates that

A long example is usually more difficult to understand

But I don’t think this holds true.
That premise applies the way we think about code to tests, which would be fine —except tests are fundamentally different.
For starters, I would argue the main objective of a test (besides correctness and repeatability) is to tell a story.
This means writing tests as documentation.
When we dive into a before unseen test, we need to familiarise ourselves quickly with our surroundings.
If that test tells a clear story, it will be easier to understand.
If the test cannot tell a story on its own, but necessitates us grokking the entire test file to grasp what it is doing, then it will be much harder to understand and make changes to.
That concept is diametrically opposed to the idea we should be writing as few lines of code as possible in our tests.
If we think about the second point, I don’t think lines of code is a helpful or useful metric at all in any context.
When we try to make our code take up as few lines as possible, it typically leads to us writing more complex and less readable code.
That doesn't seem like a worthwhile endeavour to me.
We don't enforce any other arbitrary stylistic cops, and I don’t think we should start now.
Cops like this are entirely subjective, and they are almost always a complete waste of time and energy.

authors: cpjmcquillan jsugarman


text:
Update schema to reflect migrations

These colums need to be nullable and migration reflects this but
schema does not currently
To fix problems locally you can run this
rails db:migrate:down VERSION=20220708123323

rails db:migrate:up VERSION=20220708123323

What
Link to story
Describe what you did and why.
Checklist
Before you ask people to review this PR:

Tests and rubocop should be passing: bundle exec rake
Github should not be reporting conflicts; you should have recently run git rebase main.
There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
The PR description should say what you changed and why, with a link to the JIRA story.
You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
You should have checked that the commit messages say why the change was made.

@cpjmcquillan  @RoseSAK this should fix your local machines if you get errors saving proceedings via UI


text:
Add help target to Makefile

When you run make without a specific target, it runs the first command in the Makefile. CodeQL does this, and it causes the verification to take ~15mins since the current first target (all) builds everything and runs the end-to-end tests.
This PR adds a help target at the top of the file so that that is what's run by CodeQL.
It's hopefully also useful.
#patch

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:loudspeaker: Thoughts on this report? Let us know!.

authors: codecov-commenter gregtyler


text:
UML-962 - check state of user in login on actor.

Purpose
To Fix UML-962
Approach

Actor Triage page now detects if logged in and redirects to Lpa dashboard or lpa add accordingly.
follows the same approach as login code.
viewer home page has not changed

Learning
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.663% when pulling 7a16d7bbd338257c3221c75f877fb9c6941401f4 on UML-962-fix-loggedin-issue into 832e2b50f5aa8da79863d53b4fe873bfb33323a6 on master.

authors: coveralls williamfalconeruk

repo: ministryofjustice/opg-use-an-lpa


text:
Simplify the trusted host setting

Context
This PR is based on https://github.com/ministryofjustice/prisoner-content-hub-backend/pull/315
Intent
This PR removes helper functions prisoner-content-hub-backend.trustedHosts and prisoner-content-hub-backend.trustedHostsJsonApi` which were difficult to read and created unnecessary configuration.
In order to remove this helper functions, we remove support for multiple hosts.  This allows us to pass through the single value directly into the env variable.
Considerations
Removing support for multiple hosts should have no effect, as we are not using them anywhere.
Except for on production, where multiple hosts were defined:

manage.content-hub.prisoner.service.justice.gov.uk
cms-prisoner-content-hub-production.apps.live-1.cloud-platform.service.justice.gov.uk

Only the first is being used, so the second one was removed.
Checklist

[ ] This PR contains only changes related to the above card
[ ] Tests have been added/updated to cover the change
[ ] Documentation has been updated where appropriate
[ ] Tested in Development

Closing this as it's now out of date with main.
Although it's still something we want to do.


text:
Need to support pass through of the the proxies keyword to the requests

Trying to use this behind the corporate firewall.   I need to be able to set the proxies paramaeter to the requests session

@dwmoran5104 Checkout my add-proxies-to-requests-session branch. This should solve it. Let me know if it works for you. Thanks.

@juanbrena Could you please create a merge request for this please?

authors: dwmoran5104 juanbrena minitriga


text:
Make multiple blank users rather than just one.

Presently if you make a bunch of sessions with a blank name, they are
all assigned to the same presenter.  This poses a problem if you later
want to change the presenters name.  It would change the name for all
unassigned sessions. This makes one presenter for each unassigned
session. This allows us to set different names at a later time.

Coverage increased (+0.08%) to 52.559% when pulling 15884f7f414b660060070ef186c0ae2ee7113fd1 on new_user into 3d985cb4c9b414f9245b3f7fb15385db5f83b940 on master.

authors: coveralls jcoyne

repo: minnestar/sessionizer


text:
目录数据存放在组件中，不安全，待解决。

目录数据存放在组件中，不安全也不够模块化，待解决。

已修复，将目录数据存放到lib中

已修复，将目录数据存放到lib中

repo: minorcell/perfedge


text:
🛑 Matrix is down

In 7c58d91, 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 c81c4a8 after 7 minutes.

repo: mint-lgbt/status


text:
🛑 Matrix is down

In 4a49e08, 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 c88b7eb after 7 minutes.


text:
🛑 Matrix is down

In f51c1fe, 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 55ecc9e after 5 hours, 57 minutes.


text:
[Bug]

Describe the bug
System group container for systemgroup.com.apple.configurationprofiles path is /private/var/containers/Shared/SystemGroup/systemgroup.com.apple.configurationprofiles

Doesn't sound likes a bug in this library.


text:
add more older versions

make them static
[x] 22.11
[ ] 22.05 > error: fcitx-engines is deprecated, please use fcitx5 instead.
[ ] 21.11
[ ] 21.05

22.11 will be the latest

repo: mipmip/home-manager-option-search


text:
New ShowRSS generates incorrect link

I have also been experiencing a complete failure to receive torrents recently. I am using the link generated by the New ShowRSS site.
I noticed that the generated link does not reference the New site. Once I added "new." to the link and prompted Catch to "Check Now", I received a flood of backlogged shows.
Hope this might help others resolve their problems.

@adrinavarro weird, I don't have "new" in my feed and it works fine... Any ideas?
@Seacidal could you post both feed urls so we can compare them?

No need to change the link, it is intentional and there is no bug. Otherwise you will have to change the link again when the old showRSS is shut down.
Any way to get a log of http responses? Maybe cloudflare is not playing very nice.

mipstian,
My old link was:  http://showrss.info/user/1494.rss?magnets=true&namespaces=true&name=null&quality=null&re=null
My new link is: http://new.showrss.info/user/1494.rss?magnets=true&namespaces=true&name=null&quality=null&re=null
I had already tried uninstalling/reinstalling Transmission and Catch, without success.
I read showrss' post and maybe it's something else entirely. I'm willing to try to produce a helpful log, but not sure where to find it. Thanks.

The only differences I can see between the two feeds are that the new one has the new. prefix in the xmlns:tv and <link> values, and only one of the two had a newline at the end.
I had no trouble downloading either.
I don't think those differences can cause this problem, but I guess the feed contents should be identical, right @adrinavarro ?

When posting the new ShowRSS feed link into Catch, I get the yellow ! icon next to the link field upon hitting OK.  Checking my feed produces no results and after clicking "Check Now" from the Manu Bar icon, the Last Update field doesn't update from "Never".  Uninstalled/reinstalled both like the @Seacidal.
From their blog post, though I'm sure that's only referring to their in house solution, which coincidentally only worked with a much earlier release of Transmission.

Transmission Push does not exist anymore in the new site.

feed://showrss.info/user/21729.rss?.magnets=true&namespaces=true&name=null&quality=null&re=null

@reedyfj this is a completely separate issue, but in short: you're not copying the feed properly from ShowRSS. Feeds addresses accepted by Catch begin with http. You also have a stray . in there. New ShowRSS will show you the feed in a green box, just copy-paste that.

@mipstian sorry, safari converted it to feed:// but http gives the same behavior.  copied straight from the green link generation:

Your custom feed address: http://showrss.info/user/21729.rss?magnets=true&namespaces=true&name=null&quality=null&re=null

Removing the filters (truncating from ? onward) produces the same results as well.

Please download and the install the latest version of Catch and then try
again...
Sent from my phone
On Fri, Feb 5, 2016 at 9:06 AM, reedyfj notifications@github.com wrote:
@mipstian [https://github.com/mipstian] sorry, safari converted it to feed:// but http gives the same behavior. copied
straight from the green link generation:
Your custom feed address:
http://showrss.info/user/21729.rss?magnets=true�namespaces=true&name=null&quality=null&re=null]
Removing the filters (truncating from ? onward) produces the same results as
well.
—
Reply to this email directly or view it on GitHub
[https://github.com/mipstian/catch/issues/105#issuecomment-180244088] .[https://github.com/notifications/beacon/AADKagOB6GHW1LdleD48zyPPWM3OGtpbks5phE-SgaJpZM4HTGsJ.gif]

@Seacidal are you still experiencing the issue when using the non-new version of the link? If so, could you open Console.app and see if Catch logs something when you "Check now"?

Sorry for my delay. I had one episode download in the last day or so.
In Console, I saw that Catch (when I did a "Check now") reported the following:
2/6/16 5:54:53.388 PM Catch[5526]: Forcing feed check
2/6/16 5:54:53.389 PM Catch[5526]: Scheduler status updated (polling = 1, checking = 1)
2/6/16 5:54:53.394 PM com.giorgiocalderolla.Catch.CatchFeedHelper[5556]: Checking feed
2/6/16 5:54:53.395 PM com.giorgiocalderolla.Catch.CatchFeedHelper[5556]: Downloading feed http://showrss.info/user/1494.rss?magnets=true&namespaces=true&name=clean&quality=sd&re=no
2/6/16 5:54:53.743 PM com.giorgiocalderolla.Catch.CatchFeedHelper[5556]: Feed downloaded
2/6/16 5:54:53.743 PM com.giorgiocalderolla.Catch.CatchFeedHelper[5556]: Parsing feed
2/6/16 5:54:53.744 PM com.giorgiocalderolla.Catch.CatchFeedHelper[5556]: Parsed 8 files
2/6/16 5:54:53.744 PM com.giorgiocalderolla.Catch.CatchFeedHelper[5556]: Downloading files (if needed)
2/6/16 5:54:53.744 PM com.giorgiocalderolla.Catch.CatchFeedHelper[5556]: All done
2/6/16 5:54:53.744 PM Catch[5526]: Scheduler status updated (polling = 1, checking = 0)
Looks like things might be working again. I noticed that the site http://new.showrss.info now generates links without the "new" at the beginning. Interesting? I also find that Catch still takes me to the old ShowRSS site.

That Catch sends you to old ShowRSS is by design.
Ok then.. Guess I'm closing :)

authors: Seacidal adrinavarro mipstian reedyfj showrss

repo: mipstian/catch


text:
bugfix: look at udp_payload for dhcp flags field, not overall packet

buf is the entire packet (with Ethernet/Ipv4/Udp headers), so while the existing code was a valid access, it's not what we're looking for.

Wow, nice catch !
This is an embarrassing bug, I think it deserves a release actually.

authors: haesbaert yomimono

repo: mirage/charrua-core

license: ISC


text:
use the splat operator when calling JSONSchemer

if the splat operator is not used when running with ruby 3.0.1 and rails 6.1.4 the following error is raised because the splat operator is required ArgumentError Exception: wrong number of arguments (given 2, expected 1)
might be required elsewhere but I only had the error raised on this line

Thank you!

activerecord_json_validator-2.0.1 has been released with your fix 😄 🎊

authors: jwoodrow remi


text:
contents(centos stream): add centos stream help

added basic usage for centos stream.
modify usage of epel for centos stream.
See also https://github.com/tuna/issues/issues/1603

It seems that I mistakenly clicked on re-request review.

repo: mirrorz-org/mirrorz-help


text:
Fix forbidden null value sending

I want to merge this change because... it fixes forbidden null sending as an attribute value.

PR intended to be tested with API branch:
Screenshots

Pull Request Checklist

[ ] This code contains UI changes
[ ] All visible strings are translated with proper context including data-formatting
[ ] Attributes [data-test-id] are added for new elements
[ ] Changes are mentioned in the changelog
[ ] The changes are tested in different browsers and in light/dark mode

Test environment config

API_URI=https://qa.staging.saleor.cloud/graphql/

Task linked: SALEOR-3700 Dashboard sends [null] as input where null is forbidden

authors: orzechdev patrys

repo: mirumee/saleor-dashboard


text:
Command + Qで実行するようにしていたが，macだと普通にウィンドウが閉じる

上記の通り
2キーのショートカットだと限界がありそうなので，押しやすい3キーの組み合わせを考える．

とりあえずOption+Qに


text:
Testing translation page

Once we've implemented this feature: https://github.com/mishmanners/mish-friendly-food/issues/108, we'll need to test to ensure translation is working correctly.
Can test for multiple scenarios.

We can write tests using GitHub Copilot, and then use SIRV to test it in real time.

Started PR here: https://github.com/mishmanners/mish-friendly-food/pull/112

repo: mishmanners/mish-friendly-food


text:
added filename

What does this project do?
Describe the features.
Why is that entertaining?
Explain the joke to me. Yes, please, kill the joke. Alternatively write another short joke here or a haiku.
How hard did you laugh? Was it just a silent giggle or was it full blown laughing and howling and screaming "OH MY GOD! EVERYONE! OH MY GOD! THIS IS THE FUNNIEST THING EVER! YOU'VE GOT TO SEE THIS!!!"
Text description, image or video are all valid answers here.

closing as this is, well... nothing?

authors: mislavcimpersak prankitsahu-io

repo: mislavcimpersak/awesome-dev-fun


text:
the file icon needs margin

this explorer icon showing overlapping files might need little spacing between the front one and the other below them, what do you think?
OR
just file icon without any files below might look good, given that all the icons give a "singular vibe" why not do that with explorer icon too? That's just my thought :)

Thanks for the suggestion! There are other icons that have the same "overlay" so that fits within that pattern:

nvm, I thought a margin between them would look cool. Anyway great icon taste. Keep polishing.

Thanks! I'll go ahead an close this one out, feel free to file another issue if you encounter any bugs.

authors: misolori sidbelbase

repo: misolori/vscode-fluent-icons


text:
🛑 Catalogue API is down

In cd9a035, Catalogue API (https://catalogue.apprentissage.beta.gouv.fr/api) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Catalogue API is back up in 75ad2d0 after 20 minutes.

repo: mission-apprentissage/upptime


text:
Completely blank tag cloud

I tried integrating this cloud into my solution and no matter what i do I cannot seem to get it to work... at first I thought maybe there is some script or something conflicting an causing problems with jQCloud, so what i did was create an extremely basic static html page just for testing this and I even linked to the versions of the .js and .css files on your site, but still all I get is a blank screen. Here's my markup:
<!DOCTYPE html>

<meta charset="utf-8" />
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<meta name="keywords" /><meta name="description" />
<title>Home Page - My Site</title>
<link href="https://netdna.bootstrapcdn.com/bootstrap/3.1.1/css/bootstrap.min.css" rel="stylesheet" type="text/css" />
<link href="http://mistic100.github.io/jQCloud/dist/jqcloud2/dist/jqcloud.min.css" rel="stylesheet" type="text/css" />

`<script type="text/javascript">`
    `var basic_words = [`
		`{text: "Lorem", weight: 13},`
		`{text: "Ipsum", weight: 10.5},`
		`{text: "Dolor", weight: 9.4},`
		`{text: "Sit", weight: 8},`
		`{text: "Amet", weight: 6.2},`
		`{text: "Consectetur", weight: 5},`
		`{text: "Adipiscing", weight: 5},`
		`{text: "Elit", weight: 5},`
		`{text: "Nam et", weight: 5},`
		`{text: "Leo", weight: 4},`
		`{text: "Sapien", weight: 4},`
		`{text: "Pellentesque", weight: 3},`
		`{text: "habitant", weight: 3},`
		`{text: "morbi", weight: 3},`
		`{text: "tristisque", weight: 3},`
		`{text: "senectus", weight: 3},`
		`{text: "et netus", weight: 3},`
		`{text: "et malesuada", weight: 3},`
		`{text: "fames", weight: 2},`
		`{text: "ac turpis", weight: 2},`
		`{text: "egestas", weight: 2},`
		`{text: "Aenean", weight: 2},`
		`{text: "vestibulum", weight: 2},`
		`{text: "elit", weight: 2},`
		`{text: "sit amet", weight: 2},`
		`{text: "metus", weight: 2},`
		`{text: "adipiscing", weight: 2},`
		`{text: "ut ultrices", weight: 2},`
		`{text: "justo", weight: 1},`
		`{text: "dictum", weight: 1},`
		`{text: "Ut et leo", weight: 1},`
		`{text: "metus", weight: 1},`
		`{text: "at molestie", weight: 1},`
		`{text: "purus", weight: 1},`
		`{text: "Curabitur", weight: 1},`
		`{text: "diam", weight: 1},`
		`{text: "dui", weight: 1},`
		`{text: "ullamcorper", weight: 1},`
		`{text: "id vuluptate ut", weight: 1},`
		`{text: "mattis", weight: 1},`
		`{text: "et nulla", weight: 1},`
		`{text: "Sed", weight: 1}`
	`];`
    `$('#demo').jQCloud(basic_words);`
`</script>`

I suppose I must be doing something wrong somewhere, but for the life of me I can't see it. Any help would be much appreciated. Note; when inspecting in FireBug, I can see that the jqcloud class is added to the div.. so something is happening, but for some reason nothing is visible.

Nevermind; I figured out that I needed to add some css as follows:
#demo {
margin: 0 auto;
width: 80%;
max-width: 600px;
height: 250px;
}
                    #demo span {
                        text-shadow: 0px 1px 1px #fff;
                    }

It really should be stated more clearly in the documentation that we should expect a blank without this.

It's explicitly said I the big red block "Container dimensions & position" I the doc


text:
Implement a component drawer

It seems as though we may increasingly need a drawer for general operations on components that do not necessarily fit well as events/methods/properties of the components themselves.
Some possible things:

getprops/setprops
type of?
copy/clone
name of component/component by name (for symmetry; I actually prefer a Name read-only property on the component and have a branch that implements this)

Are there other operations we might want to consider?

cc @fturbak

repo: mit-cml/appinventor-sources


text:
class name not match file path when DX of extensions

i got a folder for an extension and the structure is like this:
OpenGL
|--objects
|　|--Drawable.java (required files)
|　|-- ...
|--GLCanvas.java (this is the extensions file)
after packed as an extension, it will all become this structure in AndroidRuntime.jar: (all file in a same folder even in the different package)

and then when packaging by build server, the DX-fail log is like shown in the picture

and the only solution now, is to pack the required files into a jar file, and then declear require library, but it is such a unnessary step if the AndroidRuntime.jar is well packaged.

@ColinTree You may want to take a look at the copyRelatedExternalClasses method in ExternalComponentGenerator. Right now it only aggregates classes within the same package, not any descendant packages. You could create a patch that walks the package hierarchy to include other classes.

@ewpatton
I had fixed it locally.
But i m not sure which branch should i put it in?

Since this doesn't materially affect the companion, you can put it onto master.

On Oct 21, 2017, at 10:21, ColinTree notifications@github.com wrote:
@ewpatton
I had fixed it locally.
But i m not sure which branch should i put it in?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Sure, here it is: #963

you leave this open until the PR is accepted.

authors: ColinTree ewpatton marksherman


text:
Custom Dataset Implementation

Hi, thanks for the great work!
For training BEV segmentation with a custom dataset I noticed that unlike with the nuScenes dataset where the forward direction (vehicle X axis) of model predictions and the ground truth were to the right, for me they had to point to the bottom (90 deg CW rotation), so I was wondering why. My data collection setup is pretty similar to nuScenes,  one lidar, 6 cameras at the same angles and with the same resolution. The only difference is that unlike the nuScenes dataset where the lidar X direction points to the right, mine points forward (same as the vehicle X axis), so I wonder if that's been causing the issue.

Hi, I read in the link you shared about the work on how to generate simulated nuscenes datasets using carla and simbev, I'm very interested in this, may I ask why the previous link to the repository is gone?

Hi, I read in the link you shared about the work on how to generate simulated nuscenes datasets using carla and simbev, I'm very interested in this, may I ask why the previous link to the repository is gone? @GoodarzMehr

Hi,
SimBEV originally only supported CARLA's Town06 map, but I recently made it private to work on expanding it to all CARLA maps. I have a working prototype and hopefully I'll make it public in a month or so.

authors: GoodarzMehr baiguosummer

repo: mit-han-lab/bevfusion


text:
Phoas constant folding

Added constant folding in Phoas, as well as proof of its correctness.

If/once this is integrated into a compilation pipeline, will it also need an additional constant_folding_preserves_wf lemma? IIRC, @amanda4zx is developing some automation for this, would that work here?

If/once this is integrated into a compilation pipeline, will it also need an additional constant_folding_preserves_wf lemma? IIRC, @amanda4zx is developing some automation for this, would that work here?

Yes, the "wf" here refers to Phoas_wf (rather than the wellformedness of the flat expr), and it is needed in dephoasify_sound. I will push the code in Phoas_Interpret.v later today, but the automation may not work out of the box.

authors: amanda4zx leognon samuelgruetter


text:
[bug] Error when setting location in init hook

Call to undefined function
Invocation
My invocation was from the plugin, like this, without a hook:
PLUGINS\WPDocsify::adminRegister(array(
  'domain' => 'Documentation',
  'name' => 'Documentation',
  'slug' => 'documentation',
  'icon' => 'dashicons-editor-help',
  'location' => get_stylesheet_directory_uri() . '/documentation/',
));

or within the `init' hook:
add_action('init', function () {

  PLUGINS\WPDocsify::adminRegister(array(
    ...
  ));

});

Fatal Error
But when I call it like this, I get an error:
Uncaught error: Call to undefined function get_home_path()

It wasn't obvious to find and fix a problem, but I got the solution from this discussion here.
Using get_home_path() inside the admin_init hook is fine, but using it inside init or even earlier is not fine. So get_home_path() has to be called where wp-admin/includes/file.php is already included.
Possible fixes

The best fix is to introduce a new class method instead of the global get_home_path(). Just copy and paste from the original.
An alternative solution would be to use the ABSPATH constant instead of the get_home_path() function, but note that these two values will not be the same if WordPress is installed in a subfolder. The ABSPATH constant is initialised very early and has no restrictions on where it can be used.

Strange Behaviour
When I try to call everything from admin_init like this, there is no error, but name and slug are not set:
add_action('admin_init', function () {
  PLUGINS\WPDocsify::adminRegister(array(
    'domain' => 'Documentation',
    'name' => 'Documentation',
    'slug' => 'documentation',
    'icon' => 'dashicons-editor-help',
    'location' => get_stylesheet_directory_uri() . '/documentation/',
  ));
});

Code that Works, but it's Ugly
So the only correct, error-free solution at the moment is this, which seems ugly to me:
use MBC\inc as PLUGINS;

PLUGINS\WPDocsify::adminRegister(array(
  'domain' => 'Documentation',
  'name' => 'Documentation',
  'slug' => 'documentation',
  'icon' => 'dashicons-editor-help',
));

add_action('admin_init', function () {
  PLUGINS\WPDocsify::adminRegister(array(
    'location' => get_stylesheet_directory_uri() . '/documentation/',
  ));
});

Hi there, i think i understand the issue, and you are right its because the get_home_path is not available till later in the WordPress Lifecyle.
I think i can improve apon this by replacing that function as there is global variables set by WordPress that could replace it.
https://developer.wordpress.org/reference/functions/get_home_path/
there is also additional registration method you could use that does work with init callback.
however its only available in the v2 build
https://github.com/mitchell-b-chelin/WPDocsify/tree/v2
A working example of the registration is here.
https://github.com/mitchell-b-chelin/Bricks/blob/main/inc/documentation.php
Perhaps v2 would be better suited for you as it is designed to work with the Lifecyle of WordPress.
Hopefully this helps.

Sadly, I can't make it work on v2

I will review this issue in the new version of the plugin as i wish to merge v2 into main.
Would it be easier if added support for in the theme to have a
documentation.json
file which would offer the same as using the php overides?
And would it be useful to have a settings menu to allow changing the name rather then having to write any code?

authors: cvladan mitchell-b-chelin

repo: mitchell-b-chelin/WPDocsify


text:
[FALSE-POSITIVE] paycount.com

Describe the subjects (domains)
Dead phish parsing the PyFunceble testing to domain(0|1).lists
pyfunceble --dns 192.168.1.5 -t 0 -vsc --no-whois --filter 'paycount.com' -f hosts.ubuntu101.co.za.tmp

Status      Percentage   Numbers
----------- ------------ ------------
ACTIVE      0%           1
INACTIVE    99%          267
INVALID     0%           0

See also https://www.mypdns.org/T883 for in depth documentation

/move to Ultimate-Hosts-Blacklist/whitelist

authors: funilrys spirillen


text:
mitmproxy stuck on many connect

Steps to reproduce the problem:

install mitmproxy from master branch
run script with body modify and replay by beautifulsoup
send a 1000-2000 request to proxy
mitmdump -p 30000  -k  -w out.file -s bla1.py
bla1.py: just insert in body one tag with bs4 and send it to client

Any other comments? What have you tried so far?
Hello. mitmproxy not accepted new connection, and not working anymore, after 1000-2000 connect. And count of thread growing crazy:
`810
root@localhost:~# ps a -L|grep -i mitmproxy|wc -l
819
root@localhost:~# ps a -L|grep -i mitmproxy|wc -l
829
root@localhost:~# ps a -L|grep -i mitmproxy|wc -l
^[[A846
root@localhost:~# ps a -L|grep -i mitmproxy|wc -l
846
root@localhost:~# ps a -L|grep -i mitmproxy|wc -l
1010
root@localhost:~# ps a -L|grep -i mitmproxy|wc -l
1023
root@localhost:~# ps a -L|grep -i mitmproxy|wc -l
1034
root@localhost:~# ps a -L|grep -i mitmproxy|wc -l
1042
root@localhost:~# ps a -L|grep -i mitmproxy|wc -l
1042
root@localhost:~# ps a -L|grep -i mitmproxy|wc -l
1177
root@localhost:~# ps a -L|grep -i mitmproxy|wc -l
1204
root@localhost:~# ps a -L|grep -i mitmproxy|wc -l
1218
root@localhost:~# ps a -L|grep -i mitmproxy|wc -l
1230
root@localhost:~# ps a -L|grep -i mitmproxy|wc -l
1238
root@localhost:~# ps a -L|grep -i mitmproxy|wc -l
1996
root@localhost:~# ps a -L|grep -i mitmproxy|wc -l
`
But proxy not working, and connect failed after timeout :(
System information

(venv) koka@localhost:~$ mitmproxy --version
Mitmproxy: 3.0.0dev0976 (472d122)
Python:    3.5.3
OpenSSL:   OpenSSL 1.1.0g  2 Nov 2017
Platform:  Linux-3.16.0-4-amd64-x86_64-with-debian-8.0

https://github.com/mitmproxy/mitmproxy/issues/2713
https://github.com/mitmproxy/mitmproxy/issues/2575

I Have the same problem (beautifulsoup html in the response) ... not fixed in:

Mitmproxy: 4.0.4
Python:    3.7.1
OpenSSL:   OpenSSL 1.1.0i  14 Aug 2018
Platform:  Windows-10-10.0.17134-SP0

I also even close all connection with : ctx.kill_flow(flow) at the end of a response (Not a connection leak, maybe threading issue?).
Only after we stopped using beautifulsoup everything works again.

I Have the same problem (beautifulsoup html in the response) ... not fixed in:

Mitmproxy: 4.0.4
Python:    3.7.1
OpenSSL:   OpenSSL 1.1.0i  14 Aug 2018
Platform:  Windows-10-10.0.17134-SP0

Only after we stopped using beautifulsoup everything works again.
Update: any change to flow.response.content will cause hang...

Did you find some way to solve this problem?

At the end it was caused by my code bugs.... after fixing them all worked.False alarm on my part.

authors: Mradr gennadicho imom0 selfcac

repo: mitmproxy/mitmproxy


text:
Improve snapshots.mitmproxy.org

Linking this here because most people won't see the release repo: We'd love to have help on https://github.com/mitmproxy/release/issues/3 - this is a really nice task for new contributors and I'm happy to help with getting things going.
Let's continue further discussion in the linked repo! :smiley:

closed by d3a9a6ba9520c5c80de3433697a8f5f30fa33c67

authors: Kriechi mhils


text:
can't export course from xPRO CI Studio

Trying to export https://studio-ci.xpro.mit.edu/export/course-v1:XPRO+TESTING+TESTR0
Stuck on "Exporting" step

This issue appears to be resolved as I was able to export the linked course without any delays or error.

authors: blarghmatey pdpinch

repo: mitodl/ol-infrastructure


text:
ansible_mitogen: correct typo in MitogenViaSpec.mitogen_lxc_path()

self.host_vars -> self._host_vars

Thanks for creating a PR! Here's a quick checklist to pay attention to:

Please add an entry to docs/changelog.rst as appropriate.

Do I need to? It's a typo fix to unbreak something.

Do I need to? It's a typo fix to unbreak something.

Not if you don't want. Thanks for the fix, sorry for the wait.

authors: InsanePrawn moreati

repo: mitogen-hq/mitogen


text:
cargo insta test exits with exit code 0 even if there's a test failure

What happened?
I ran cargo insta test in a project with a test failure. cargo insta v1.29.0` returned an exit code 1, whereas the latest version returns an exit code of 0. This causes our CI to pass, even if there are test failures.
Reproduction steps

Create a project with a failing test, for example:
#[cfg(test)]
mod tests {
    #[test]
    fn it_works() {
        assert_eq!(3, 2);
    }
}

Run cargo insta test

Print the exit code: echo $status -> 0

This seems to have changed with insta 1.30

Install insta 1.29.0: cargo install cargo-insta@1.29.0
Run cargo insta test
Print the exit code: echo $status -> 1

Insta Version
1.30.0
rustc Version
1.68.0
What did you expect?
cargo insta to exit with code 1

This is now fixed on master. will be published soon.

Thanks!

I also ran into this bug today. Is there anything delaying the release of the fix? You may also want to consider yanking v1.30.0 since this is a pretty significant bug.

Sorry for the delay on this. It's out now.

authors: MichaReiser mitsuhiko not-my-profile

repo: mitsuhiko/insta


text:
Multiple critical issues

I looked at the code, and it's not really in a production-usable state.
There are mostly for the Windows version:

If fs::copy is interrupted in the middle (e.g. by power failure or killing the process), the new file will be corrupted and the old one will be deleted, resulting in an unrecoverable catastrophe. Instead, fs::copy must write a temporary file, and only afterwards (and after an fsync) be atomically renamed to the executable name, and also deletion of the current executable must happen only after the rename is successful
The code apparently only tries a single name for the temporary files, which might well already exists. Code that creates temporary files must of course retry until it finds a name that doesn't exist.
If there is a power failure, temporary files are left as-is and there is no recovery. Instead, on Windows, it should use PendingFileRenameOperations to recover, and on Unix probably a bunch of mechanics (e.g. creating a systemd unit if systemd is installed, with perhaps fallback to editing ~/.profile)
It should try to open the current executable with FILE_DELETE_ON_CLOSE and if that succeeds there is no need to do the respawn trick
There is a sleep for 100ms in the code. Such sleeps are always wrong since an sleeping for an arbitrary amount of time serves no purpose. Instead, either wait for what you need or retry in a loop.
Doing the self delete on init by checking the executable suffix is completely broken, since the user might have to decided to name the executable with a name having such a suffix. Instead, the code should be work by dropping a separately compiled executable that does only the deletion instead of copying the current executable, or if that's undesirable, by using an environment variable with a randomly-generated name.
As mentioned in the code, the Rust std functions must not be used before main (this is not a problem if a separate executable is used).
The code spawns a "ping" process at some point, which is completely absurd (among others, there may be no ping, no network stack, no 127.0.0.1 address, not to mention that libraries should not spawn random processes for no reason, etc.). That should be replaced with a proper solution that does whatever is needed. If you absolutely need to spawn a system executable (which I doubt and hope not) then at least spawn something that is reasonably guaranteed to exist, such as rundll32
The code uses file paths for all operations, which means that it's going to fail spectacularly if the user renames a file in the middle of the operation. Instead, file paths must be used once for opening and then fd/handle-based operations (the *at APIs on Unix) must be used afterwards. If that's not possible, there must at least be code to retry the whole thing if a file gets renamed by the user.
The code doesn't work at all if the executable has been deleted or renamed. It should detect that case and not do the deletion.
The code seems to assume that once the parent shuts down, then the executable is deletable, but this is obviously false, since something else might have opened the executable without FILE_SHARE_DELETE. It should handle this case by either waiting until deletion is possible, or scheduling it for the next reboot.
If you really need to get the parent process id, then you should use InheritedFromUniqueProcessId from NtQueryInformationProcess and not toolhelp32 (which is O(n) in the number of processes in the system).
The unix code sets file permissions after doing the copy, which is obviously a security disaster since the executable might be readable and even writable by everyone (or by the group) in the meantime if umask is set to maximally permissive

This is just the issues that I have noticed without really doing an exhaustive check. Given the apparent lack of any thought to most of these issues, I would expect more to be present and would not really recommend using this crate, at least until it's pretty much rewritten.

If fs::copy is interrupted in the middle (e.g. by power failure or killing the process), the new file will be corrupted and the old one will be deleted, resulting in an unrecoverable catastrophe

You are right that on the windows case the atomic write is missing, that has been changed. The fsync part you mentioned I don't think is particularly relevant. I have added the missing rename.

The code apparently only tries a single name for the temporary files

It uses 20 random characters for entropy, it's doubtful that this would ever clash.

If there is a power failure, temporary files are left as-is and there is no recovery

Yeah, that is a known limitation. I filed #5 for it, but I don't think the issue is particularly critical as the race for this is very tight. The files placed in temp locations particularly on windows are not affected by this anyways as the operating system will clean them up. On Windows you will in fact only lose files if the volume of the target exe and the temp folder is not shared.

It should try to open the current executable with FILE_DELETE_ON_CLOSE and if that succeeds there is no need to do the respawn trick

There is no known version of Windows where this works, so I would not even know how to test this.

Doing the self delete on init by checking the executable suffix is completely broken, since the user might have to decided to name the executable with a name having such a suffix.

I don't consider this broken, but it's undocumented behavior and maybe a more random suffix should be used to reduce the likelihood of this happening. I filed #6 to track this.

As mentioned in the code, the Rust std functions must not be used before main

Yeah, I will eventually get to this and replace them with native win32 code. That said, the section I picked for the function is run after the runtime initialized. Tracking in #7.

The code spawns a "ping" process at some point, which is completely absurd (among others, there may be no ping, no network stack, no 127.0.0.1 address, not to mention that libraries should not spawn random processes for no reason, etc.). That should be replaced with a proper solution that does whatever is needed. If you absolutely need to spawn a system executable (which I doubt and hope not) then at least spawn something that is reasonably guaranteed to exist, such as rundll32

The original code spawned rundll32. Unfortunately rundll32 can in theory shut down quite quickly. I am not aware of a version of Windows that does not have ping available so that concern does not seem particularly high. For the hypothetical case that there is no 127.0.0.1 the spawn of ping might be slow enough anyways to solve this issue. The spawn of an executable is necessary and the motivation is documented.

The code uses file paths for all operations, which means that it's going to fail spectacularly if the user renames a file in the middle of the operation.

Since the files in all cases need to appear in the file system there are no alternatives to this. This issue however applies to most uses of filesystem APIs and is not specific to this crate. Any "create file and move over atomically" operation has the same issue.

The code doesn't work at all if the executable has been deleted or renamed. It should detect that case and not do the deletion.

That is intentional.

The code seems to assume that once the parent shuts down, then the executable is deletable, but this is obviously false, since something else might have opened the executable without FILE_SHARE_DELETE. It should handle this case by either waiting until deletion is possible, or scheduling it for the next reboot.

This only applies to the copied executable which nobody can launch in the meantime unless they race intentionally for it to break it. There are however cases where the deletion of the main executable after the rename cannot work. This is basically a duplicate of #5

If you really need to get the parent process id, then you should use InheritedFromUniqueProcessId from NtQueryInformationProcess and not toolhelp32 (which is O(n) in the number of processes in the system).

I considered this, however the behavior of the current version was battle tested by rustup so I decided to keep it. The issue with InheritedFromUniqueProcessId is that in theory this is not the parent process and I did not want to investigate yet if this is an issue. If the performance is a problem I'm sure an issue will be opened. In general that operation is not particularly common anyways.

The unix code sets file permissions after doing the copy, which is obviously a security disaster since the executable might be readable and even writable by everyone (or by the group) in the meantime if umask is set to maximally permissive

I don't believe this to be an actual problem given that this operation is performed on a temporary. Maybe you could elaborate on the exact way in which this might be exploitable in a new issue.
I opened specific issues for the individual points that that I believe need to be addressed.

authors: lyphyser mitsuhiko

repo: mitsuhiko/self-replace


text:
asg.plot

Hi,
Thank you for the asg package!
I am making a plot with the asg.plot function. I am wondering is it possible to add the numeric correlation coefficients in the "network" graph?
BR,
Lotta

Thanks for the suggestion, this functionalitty is now integrated both in the package and the graphical version.
Detlef

authors: lottaimmeli mittelmark

repo: mittelmark/snha-gui


text:
OpenMVH7_PhotoLogging.py writes to wrong partition of SD card

https://github.com/mitwelten/mitwelten-iot-hardware-poc/blob/main/OpenMV/OpenMVH7_PhotoLogging.py

Re-formatting the SD Card (FAT32) to a single partition fixed the issue.

repo: mitwelten/mitwelten-iot-hardware-poc


text:
Minibasket Attribute Pricing Issues

Expected Behavior

We’ll need to update minibasket to include attribute pricing information

Current Behavior

The minibasket product information does not display or take into account attribute pricing.  So if I have $9 product, and $10 attribute, the minibasket shows that the item is $9.
In the default state, the minibasket also does not show the product’s correct sale price when the item has attribute pricing.

Attempted Fixes

FIX: Change the &mvt:item:formatted_subtotal; variable to instead use a custom calculated item subtotal to show the correct price, both taking into account attributes as well as discounts. The calculation should be the group:subtotal + looping through and adding in all option:subtotal values for that item’s attributes.
I’ve attached a screenshot of my example code for the fix as well.
For future reference: I was going to suggest we use the subtotal_comprehensive variables as a solution, but this led me to discover some issues with the subtotal_comprehensive variables available to us that Jon has now filed a bug for v10.

Steps to Reproduce (for bugs)

See Screenshot Attached

*Chrome / Windows10 / Desktop

This was addressed in Issue #39

authors: bdepauw influxweb

repo: mivaecommerce/readytheme-shadows


text:
Error encountered while adding controllnet after SD upgrade 1.93

`*** API error: POST: http://127.0.0.1:7860/sdapi/v1/txt2img {'error': 'AssertionError', 'detail': '', 'body': '', 'errors': "Conflict of field 'guidance' and 'guidance_end'"}
Traceback (most recent call last):
File "D:\stable-diffusion-webui\venv\lib\site-packages\anyio\streams\memory.py", line 98, in receive
return self.receive_nowait()
File "D:\stable-diffusion-webui\venv\lib\site-packages\anyio\streams\memory.py", line 93, in receive_nowait
raise WouldBlock
anyio.WouldBlock
During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "D:\stable-diffusion-webui\venv\lib\site-packages\starlette\middleware\base.py", line 78, in call_next
    message = await recv_stream.receive()
  File "D:\stable-diffusion-webui\venv\lib\site-packages\anyio\streams\memory.py", line 118, in receive
    raise EndOfStream
anyio.EndOfStream

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "D:\stable-diffusion-webui\modules\api\api.py", line 186, in exception_handling
    return await call_next(request)
  File "D:\stable-diffusion-webui\venv\lib\site-packages\starlette\middleware\base.py", line 84, in call_next
    raise app_exc
  File "D:\stable-diffusion-webui\venv\lib\site-packages\starlette\middleware\base.py", line 70, in coro
    await self.app(scope, receive_or_disconnect, send_no_error)
  File "D:\stable-diffusion-webui\venv\lib\site-packages\starlette\middleware\base.py", line 108, in __call__
    response = await self.dispatch_func(request, call_next)
  File "D:\stable-diffusion-webui\modules\api\api.py", line 150, in log_and_time
    res: Response = await call_next(req)
  File "D:\stable-diffusion-webui\venv\lib\site-packages\starlette\middleware\base.py", line 84, in call_next
    raise app_exc
  File "D:\stable-diffusion-webui\venv\lib\site-packages\starlette\middleware\base.py", line 70, in coro
    await self.app(scope, receive_or_disconnect, send_no_error)
  File "D:\stable-diffusion-webui\venv\lib\site-packages\starlette\middleware\cors.py", line 84, in __call__
    await self.app(scope, receive, send)
  File "D:\stable-diffusion-webui\venv\lib\site-packages\starlette\middleware\gzip.py", line 24, in __call__
    await responder(scope, receive, send)
  File "D:\stable-diffusion-webui\venv\lib\site-packages\starlette\middleware\gzip.py", line 44, in __call__
    await self.app(scope, receive, self.send_with_gzip)
  File "D:\stable-diffusion-webui\venv\lib\site-packages\starlette\middleware\exceptions.py", line 79, in __call__
    raise exc
  File "D:\stable-diffusion-webui\venv\lib\site-packages\starlette\middleware\exceptions.py", line 68, in __call__
    await self.app(scope, receive, sender)
  File "D:\stable-diffusion-webui\venv\lib\site-packages\fastapi\middleware\asyncexitstack.py", line 21, in __call__
    raise e
  File "D:\stable-diffusion-webui\venv\lib\site-packages\fastapi\middleware\asyncexitstack.py", line 18, in __call__
    await self.app(scope, receive, send)
  File "D:\stable-diffusion-webui\venv\lib\site-packages\starlette\routing.py", line 718, in __call__
    await route.handle(scope, receive, send)
  File "D:\stable-diffusion-webui\venv\lib\site-packages\starlette\routing.py", line 276, in handle
    await self.app(scope, receive, send)
  File "D:\stable-diffusion-webui\venv\lib\site-packages\starlette\routing.py", line 66, in app
    response = await func(request)
  File "D:\stable-diffusion-webui\venv\lib\site-packages\fastapi\routing.py", line 237, in app
    raw_response = await run_endpoint_function(
  File "D:\stable-diffusion-webui\venv\lib\site-packages\fastapi\routing.py", line 165, in run_endpoint_function
    return await run_in_threadpool(dependant.call, **values)
  File "D:\stable-diffusion-webui\venv\lib\site-packages\starlette\concurrency.py", line 41, in run_in_threadpool
    return await anyio.to_thread.run_sync(func, *args)
  File "D:\stable-diffusion-webui\venv\lib\site-packages\anyio\to_thread.py", line 33, in run_sync
    return await get_asynclib().run_sync_in_worker_thread(
  File "D:\stable-diffusion-webui\venv\lib\site-packages\anyio\_backends\_asyncio.py", line 877, in run_sync_in_worker_thread
    return await future
  File "D:\stable-diffusion-webui\venv\lib\site-packages\anyio\_backends\_asyncio.py", line 807, in run
    result = context.run(func, *args)
  File "D:\stable-diffusion-webui\modules\api\api.py", line 478, in text2imgapi
    processed = process_images(p)
  File "D:\stable-diffusion-webui\modules\processing.py", line 845, in process_images
    res = process_images_inner(p)
  File "D:\stable-diffusion-webui\extensions\sd-webui-controlnet\scripts\batch_hijack.py", line 56, in processing_process_images_hijack
    cn_is_batch, batches, output_dir, input_file_names = get_cn_batches(p)
  File "D:\stable-diffusion-webui\extensions\sd-webui-controlnet\scripts\batch_hijack.py", line 196, in get_cn_batches
    units = external_code.get_all_units_in_processing(p)
  File "D:\stable-diffusion-webui\extensions\sd-webui-controlnet\internal_controlnet\external_code.py", line 133, in get_all_units_in_processing
    return get_all_units(p.scripts, p.script_args)
  File "D:\stable-diffusion-webui\extensions\sd-webui-controlnet\internal_controlnet\external_code.py", line 146, in get_all_units
    return get_all_units_from(script_args[cn_script.args_from : cn_script.args_to])
  File "D:\stable-diffusion-webui\extensions\sd-webui-controlnet\internal_controlnet\external_code.py", line 174, in get_all_units_from
    all_units = [
  File "D:\stable-diffusion-webui\extensions\sd-webui-controlnet\internal_controlnet\external_code.py", line 175, in <listcomp>
    to_processing_unit(script_arg)
  File "D:\stable-diffusion-webui\extensions\sd-webui-controlnet\internal_controlnet\external_code.py", line 228, in to_processing_unit
    return ControlNetUnit.from_dict(unit)
  File "D:\stable-diffusion-webui\extensions\sd-webui-controlnet\internal_controlnet\args.py", line 414, in from_dict
    values = cls.legacy_field_alias(values)
  File "D:\stable-diffusion-webui\extensions\sd-webui-controlnet\internal_controlnet\args.py", line 318, in legacy_field_alias
    assert key not in values, f"Conflict of field '{alias}' and '{key}'"
AssertionError: Conflict of field 'guidance' and 'guidance_end'

`

Before this upgrade, it was still normal. I don't remember the original version number


text:
アクセス権のない API にアクセスしようとしていたら、ユーザに警告を出す

ref: #48

良さそう。とりあえずconsoleに出ていれば十分かなー。


text:
Responsive emails for different email-clients

Description
According to my understanding MJML provides a mechanism for designing and building emails. The emails are created with mjml's wysiwyg editor and then the html code is generated by mjml. The generated html code is responsive and is displayed evenly in different clients, such as Thunderbird, Outlook, IPhone, etc.
However, the template that I have generated and the HTML code resulting therefrom are not being displayed correctly in, for example Outlook 2019. Here the columns are displayed horizontally instead of vertically.

The email is displayed correctly in Thunderbird.

To Reproduce
Steps to reproduce the behavior:

Create an email template by using columns, e.g.

<mj-column background-color="#DCDCDC" width="100%" padding="0px 40px 0px 40px" padding-top="40px" padding-bottom="40px">   <mj-text font-family="Brandon Text" font-size="25px" text-transform="uppercase" padding="0px 0px 0px 0px" font-weight="bold" align="center"> Morgengold bringt's </mj-text>   <mj-text font-family="Brandon Text" font-size="25px" align="center" text-transform="uppercase" padding="0px 0px 0px 0px" font-weight="bold"> Freu dich drauf </mj-text> </mj-column> <mj-column width="80%">   <mj-text font-family="Arial" font-size="15px"> Falls Sie keine weiteren Produktinformationen mehr per E-Mail wünschen, teilen Sie dies bitte    Ihrem Morgengold Partner per E-Mail    an folgende Adresse mit: $MailFN$.   </mj-text> </mj-column> <mj-column width="100%" vertical-align="middle">   <mj-spacer height="32px" />   <mj-social font-size="15px" icon-size="40px" mode="horizontal">     <mj-social-element name="trustedshops" src="http://$Hostname$/assets/asset/nAZ1gr4IIds"       href="https://www.trustedshops.de/bewertung/info_XB23DF2C689CD1F51D7E00E279D369153.html">     </mj-social-element>     <mj-social-element name="facebook" src="http://$Hostname$/assets/asset/DW9DsuP3Js0" href="https://www.facebook.com/mein.morgengold/">     </mj-social-element>     <mj-social-element name="instagram" src="http://$Hostname$/assets/asset/_t7aanwcer0" href="https://www.instagram.com/mein.morgengold/">     </mj-social-element>   </mj-social>   <mj-text font-family="Arial" font-size="15px" font-weight="bold" align="center" padding="0px 0px 0px 0px">      mein.morgengold    </mj-text> </mj-column>

Render it to HTML.
Send the HTML to an email address on Outllook 2019 and Thunderbird clients.
See rendered email in Outlook client.

Expected behavior
The email should be displayed in the same way in both Outlook 2019 and Thunderbird.
**MJML environment **

OS: Windows 10
MJML Version 4.8.1
MJML tool used: MJML App v3.0.3

Email sending environment:

The rendered HTML is used to generate an email and send via Java code to email address.

Affected email clients (for rendering issues):

Email Client: Outlook 2018, Thunderbird current version
OS: Windows 10

Screenshots
See above.

You need to split your columns into multiple sections.
1 section = 100% of content. Here you have 280% of content inside a single section so outlook will display everything inline

authors: MarkyGitHub iRyusa

repo: mjmlio/mjml


text:
Build fails on Android Studio 3.3

* What went wrong:
Execution failed for task ':app:processDebugResources'.
> Android resource linking failed
  /Users/user/.gradle/caches/transforms-1/files-1.1/prettydialog-1.0.4.aar/bb595b75c4d148bb4ef3f90b1ef93ea0/res/values/values.xml:10:5-13:13: AAPT: error: expected reference but got (raw string) #000000.

Android Studio 3.3 Canary 6, Gradle 4.9, Build Tools 28.0.2.
Library version 1.0.4

RC version of new CoordinatorLayout was the cause, not this lib. Closing.


text:
add gin web fremework support for this project

i am using gin, so i made life easier

Thank you for pull request.
I have a few comments, if you don't mind:

Change import path in ginExample.go to github.com/mkevac/debugcharts. You forgot to change it.
Remove copyright and "add by" comments, please. Package already has a license and it is different than you specified. Your contribution will be seen here, on GitHub and we can add CONTRIBUTORS file if you want. Package is really very simple to fill it with comments like these.

        OK，no problem！发自网易邮箱大师
        在2015年12月30日 03:37，Marko Kevac 写道:Thank you for pull request.

I have a few comments, if you don't mind:
Change import path in ginExample.go to github.com/mkevac/debugcharts. You forgot to change it.
Remove copyright and "add by" comments, please. Package already has a license and it is different than you specified. Your contribution will be seen here, on GitHub and we can add CONTRIBUTORS file if you want. Package is really very simple to fill it with comments like these.
—Reply to this email directly or view it on GitHub.

Thank you!
I have added you to CONTRIBUTORS file.

authors: Mint-Zhao-Chiu mkevac

repo: mkevac/debugcharts


text:
🛑 FarmPi Mirror is down

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

HTTP code: 429
Response time: 639 ms

Resolved: FarmPi Mirror is back up in a060a85 after 5 minutes.


text:
consider truncation support

it may be useful for dodo to be able to truncate(2) a file
truncate(2) appears in POSIX

also closing, @phillid is on a roll.

repo: mkfifo/dodo


text:
Possible memory leak

In new version (9.26.1) is possible memory leak. When save files (deploy on save turned on) several times ... I have high CPU load... and extension don't deploy files anymore.

How often have you saved files?
How many files have you saved and how many files are part of your package(s)?
What operating system and VSCode version do you use?

Can you also post an example of your settings?
When I try to reproduce it (on Windows with VSCode 1.13) I can not see an increasement of memory in my task manager.

Strange.... Now I can't reproduce incasement... Maybe the stars perfectly aligned today but I have this issue four times. I will be watching
PS... I save files every 30 seconds. Every time I save only file by file. MacOS 10.12.3
Settings:
`{
"deploy": {
"packages": [
{
"name": "Anything",
"description": "Deploy All Files",
"exclude": [
"./",
".meteor/",
".history/",
".idea/",
".vscode/",
".gitignore"
],
"deployOnSave": true,
"syncWhenOpen": true,
"noNodeModules": true
}
],
    "targets": [
        {
            "type":"sftp",
            "name": "DB SPROOT IT",
            "dir": "**************",
            "host": "**************", "port": 22,
            "user": "**************", "password": "**************",

            "checkBeforeDeploy": true
        }
    ],
    "showPopupOnSuccess": false,
    "fastCheckOnSave": true
}

}`

So here it is... appeared randomly

authors: Zarogon mkloubert

repo: mkloubert/vs-deploy


text:
BIO memory leak with pkey_from_pem and x509__x509_from_string?

It looks like pkey_from_pem and x509__x509_from_string don't BIO_free their memory object when they succeed, only when they fail. Would this result in a memory leak? Or do the pkey and x509 cert objects retain a reference to the allocated memory and free it when they are freed?

@raybondo did you find any fix or workaround for this issue? Thanks.

Ah, thanks. I had added the BIO_free(mem); call to the two functions in our codebase. The fix #43 is identical to our internal change for x509__x509_from_string.
The pkey_from_pem leak still exists in the github version.

Thanks for your quick answer. What do you mean by "our codebase"? Do you have a fork where the issue is fixed? I'd be happy to use it if possible.

We have a private git repository that I pulled the luacrypto library into.
Our repo is not publicly hosted and contains other libraries as well, so
there's no actual fork that you could pull from, sorry.
I know, we don't use github properly.
On Mon, Oct 30, 2017 at 3:27 PM, Guido García notifications@github.com
wrote:

Thanks for your quick answer. What do you mean by "our codebase"? Do you
have a fork where the issue is fixed? I'd be happy to use it if possible.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mkottman/luacrypto/issues/45#issuecomment-340573753,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAN-g6CWULHkYSfX8VEQtFm4L74a2xXks5sxjFFgaJpZM4Jjcye
.

authors: palmerabollo raybondo

repo: mkottman/luacrypto


text:
Fix object property value in mlx_lm.server chat completions response not matching OpenAI spec

In mlx_lm.server's POST /chat/completions, the "object" property did not match OpenAI's spec.
The value produced was "chat.completions" or "chat.completions.chunk", but should be "chat.completion" or "chat.completion.chunk" for compatibility with clients expecting an OpenAI API.
Refer to:
The chat completion object

object string
The object type, which is always chat.completion.

The chat completion chunk object

object string
The object type, which is always chat.completion.chunk.

In particular, this solves a problem for me in which aider 0.64.1 reports hitting a token limit on any chat completion request to mlx_lm.server, no matter how small, despite apparently correct counts in the usage property.

I read the contributing guidelines a step late and will be happy to go learn how to add tests if requested!

No need to add a test for this but please run the formatting hooks so the linter doesn't fail.

authors: awni kconner

repo: ml-explore/mlx-examples


text:
Added ArcTan2 operation

Proposed changes
This PR adds the arctan2 operation, as discussed in #1065.
Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING document
[x] I have run pre-commit run --all-files to format my code / installed pre-commit prior to committing changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have updated the necessary documentation (if needed)

Thanks! Is that linux build supposed to pass? It seems like it's mainly having trouble finding the ArcTan2 symbol and I'm not sure what I missed.

Yes, linux tests should pass. I think you just need to add  to the list here: https://github.com/ml-explore/mlx/blob/main/mlx/backend/common/default_primitives.cpp#L36
That is just a way of saying that we should just use the default eval when accelerate is not available (e.g. on linux).

repo: ml-explore/mlx


text:
(WIP) 2nd beta inversion by post-search refinement

Partial solution to #40
It turns out the inner search required for 2nd beta inversion is far easier than the outer search, because the inner search is about finding a shared subtree instead of a shared scaffold. Since structural hashers already tell us when the same subtree is in multiple places, this shouldn't be too hard.
We can cast this as a post-search improvement problem, where we raise the upper bound of a partial invention by pretending any multiarg arguments are actually shoved into the body of the invention (overapproximating the fact that this post-improvement will move part of the arg into the body), and this higher upper bound means we can safely put off refining arguments into 2nd-beta-inverted-arguments till the end.
Initial tests of raising the upper bound don't show a huge decrease in runtime - around 2s -> 2.8s or 159 -> 516 partial inventions, and upper bound pruning and single use pruning both still fire a lot. My sense is it just means the search has to unfold partial inventions a bit more before being able to prune them. Certainly this could be a considerable slowdown (eg 50% here) but not an unreasonable one for adding a new very useful feature.
An alternative would be to do this argument refinement on the spot which would have the benefit of slightly tightening bounds and letting us find multiuses of these refined arguments, but I don't think it's worth it because it would massively branch the search. It's also not super hard to switch to this from having first implemented the post-refinement appraoch.
The two things dreamcoder does that this would not capture are

when it does 3 beta inversions within the same argument. I have trouble visualizing what that even means. My guess is it's not important. Regardless I'll have a better understanding after I finish this implementation, so it might be more possible to look into then. I would be quite surprised if any DC invention actually uses this.
when it uses the same higher order function in multiple places (as in, #1 is a function type and it's used multiple places within the body). I suspect this doesnt come up in dreamcoder. Also glancing over the origami domain in the DC paper the inventions dont look like they use this.

Negative IVars

When an arg with $0 as a free var is about to get downshifted during the Lambda::Lam branch of get_appzippers, we go through and replace variables that reference $0 with #-depth (a negative IVar) where depth is the number of lambdas along the zipper to #0 through the body (excluding the newly added lambda, so that $depth placed at #0 would directly reference this lambda (unless any lambdas were included in the refining-out, in which case that depth would have to be added to the variable too. But that's dealt with during refining out, here you simply do #-depth).
How will this interact with multiuse at the unrefined arg level?

It will be fine, ie if two unrefined args are identical (ie multiuse has been done at this level) then the refined args are guaranteed to also be refinable in an identical way so we aren't breaking any multiuse that we've already done. This is because from a given unrefined arg regardless of the context it's used in it's always possible to refine it in exactly the same way, since refinement has nothing to do with context beyond adjusting the index but that's already been captured by the negative ivar and can be calculated without looking at the context.

How will this interact with multiuse at the refined-out arg level?

It will be fine. At the refined-out level multiuse is based on structural hashing, and since #-i gets translated to $(i+refined_out_lambda_depth) then there should be a one to one correspondance between abstracting out multiple copies of the same thing and those all turning into the same result (ie refining out some subtree always transforms it in the same way regardless of what context it's put into)

Why negative IVars?

we can query free_ivars to see if they're present in a subtree easily
they wont get affected by shifting operations which we want in this case
they aren't being used for anything and it's hard to imagine them being used for much
they wont show up in regular trees only invention trees so we're less likely to affect other code than if we did something like negative Vars
theyll only show up in args and since there are never any ivars in args this should not clash with any existing code
we dont want to create a new Lambda:: enum variant for this because that would require dozens of changes to the code.

Subsuming Context Threading

This PR would actually subsume context threading and we can remove everything related to context threading (eg threaded_vars) and the cfg flags.
Instead context threading is achieved by just refining out negative variables

What can / can't be refined out?

You cant refine out a free variables in the arg. This points outside of the invention so it can't be moved into the invention body without creating a free variable in the finished invention (illegal)
You cant refine out a variable referencing a lambda that's within the argument unless youre also refining out that lambda itself, because you can't create a reference to the lambda within the argument from within the body.
In short, this means you can't refine out any term with free variables. It's okay for the term to have variables as long as they aren't free variables.
What you can refine out: arbitrary subtrees that don't have free variables and may or may not have negative variables
Useless refinement optimization: it's never worthwhile to refine out a size=1 term (even if it's multiused) unless it's a negative variable.
It's always better to maximally multiuse the refinement arg, ie if it appears multiple times you should refine out all of them not just one.

What should refinement output?

metadata on the ZTuple thats a list of list of refined out args. This is a good place because to_expr can use it and we no longer need the threaded vars on labelledzids etc.

Higher arity refinements

--refinement_arity can control the number of terms refined out
This isn't super clear yet but there are various thoughts:

For multiarg refinement I guess you want to only look for the cousin cases and watch out for usecollision esp if this happens only sometimes.
I might just suggest greediness with multiarg - complete the search with one, rewrite the unrefined args using that, do the search for another. Idk that's not optimal ofc.
you know a single arg wont collide with itself, and two args will only collide with each other if one is a subtree of the other - right we're working with subtrees where structural hashing makes everything easy!
I'm pretty sure the gains are additive between multiarg with subtrees. The only notable thing is some args preclude each other. Oh wait also different args result in different subsets and multiarg means intersecting the subsets.
you could potentially still do this though. First map over the arg possibilities to get utilities + subsets for each. Then loop over pairs of args (or triples for arity=3 etc) and abort if one arg is a descendent of another and then just keep track of the best utility youve seen (taking subsets into account). Dont exactly know how this would work since I haven't seen how utility calc happens / if its actually possible to decouple it from usages

Algorithm

I think we want to have preconstructed a mapping from a parent to all its descendents along with usage counts (+ maybe even multiplied by size so you get real cost).
Then we build up a hashmap by looping over all these and a) building the nodes subset for each of these options (since they all have diff nodes subsets) b) noting the global_hof_utility from this.
Then we do need to recalculate utility at this point since the nodes subset has changed which changes both num_uses and also the multiuse gains since those are location dependent.
I think we dont want to fully construct cloned ztuples for each of these things we just want to pick out the best one then create a new ztuple that has it as part of the LabelledZid I guess.

Todos

[x] undo threading
[ ] what exactly does the utility formula become?
[ ] how will we write rewrite_with_invention?
[ ] how will we write ZTuple::to_expr? This much should be super easy we can have start_ivar() handle it.
[ ] (later) If we really wanted to obey --inv-candidates we could return a bunch of candidates from this. Maybe we should read the inv-candidates thing for how many to return.
[ ] Use-conflicts within the refinement_arity args is easy, but what about use-conflicts that come from the act of refining since it damages the existing arg? Remember we'll have use-conflict checks before merging the donelist_buf into the donelist, but I guess the fear is what if we drop those alternatives during refinement because we found one that we thought was great but it turns out to not be good... Idk, should we be building up a list since we might want it for inv-candidates anyways? idk.
[ ] do this in initial_inventions too

closing this with #121


text:
[Question] does mlc ai for android  can run using nnapi?

Hello, i does android demo app possible using with nnapi support?

Hi @BlindDeveloper, we don't use nnapi. That may be something we can do in the long term roadmap, but for now we do not leverage that.

@MasterJH5574   I would really like this functionality to be added, but unfortunately it is deprecated   with the release android 15.
https://developer.android.com/ndk/guides/neuralnetworks?hl=en.

repo: mlc-ai/mlc-llm


text:
[CPP] Separate common utils out from llm_chat.cc

This PR separates out the tokenizer creation function, the random number generator out from llm_chat.cc as a preparation step for batching inference support, since these functions/modules are also used in the same way in batching inference.

Ah looks like I messed something up during rebase. Will resolve it later

Okay it's back to life now.


text:
MLDB-1705 function application path

Adds a test case for being able to apply functions with structured fields coming in.

+1

authors: jeremybarnes mailletf

repo: mldbai/mldb


text:
[Feature]: Create VPC endpoints in the AWS VPC

Describe the Feature
This feature would allow all the traffic to be routed inside of the VPC instead of going out and coming back in again. The code for the VPC endpoints can be found here in the VPC module.
Importance of the Feature
This is important to keep all the traffic inside of the AWS network instead of going out in the internet and coming back in again. It is important for security, and to reduce the egress costs.

hey  @aliabbasjaffri pls assign me this issue
thanks

Hello! Sure! All the best, feel free to ask any questions in our discord

repo: mlinfra-io/mlinfra


text:
Add primary_currency to user object

Hi @mmenanno! This client library is so wonderful to work with. Thanks for writing it!
When I was using it today, I couldn't get the User object via api.me. Looks like it's because of a change in the LunchMoney API. A primary_currency field was added to the User, and the gem didn't know how to handle it.
The error I got

Response from LunchMoney API

This PR adds primary_currency to the User object. Tested from my forked version of the repo. Let me know if this PR needs anything else!

Thanks for this! Code looks good initially, but let me figure out why the unit tests aren't running on your PR 👀

@SaalikLok I've bumped the gem version v1.3.0 should have your change

@mmenanno Thanks! 🙌

authors: SaalikLok mmenanno

repo: mmenanno/lunchmoney


text:
Cross-seed based on data files

After #52 we'll have an abstraction over searchees. This means that we'll be able to easily create searchees out of entries in your data dirs.
Plan:

add a --data-dir command line option that can be used multiple times
concatenate those with the torrents found in the session dir
be careful to deduplicate these

This one will be slick. Nice one

I'm keeping an eye on this 👀

Just an FYI, this is how https://github.com/BC44/Cross-Seed-AutoDL operates

You do have torrent files inside your qbit config, just gotta hunt them down.

I found that folder. Luckily the docker container maps it to the outside so I got access too. Thanks. Hope that wasn't too off topic.

@byter09 No worries! I am gonna mark as off topic though.

repo: mmgoodnow/cross-seed


text:
How do we install this?

Forgive me if it's been written somewhere but i can't figure out a way to install this extension. I've only been using OSX for a few months so i hope i'm not missing something obvious.

Hey coolnow, it's not written somewhere. Because it's not in the Safari Extension Gallery (yet), you have to download and install it manually. Visit the project's releases page (https://github.com/mmichaa/passafari.safariextension/releases) and download the latest .safariextz file. To install the safari extension, just double click it in finder. Then Safari will ask you to install it or not. I hope that this helped you. Regards, Michael.

Thank you for that link, i was trying to find the .safariextz on your
actual repo (new to this). Will try it soon!
On 15 April 2015 at 09:25, Michael Nowak notifications@github.com wrote:

Hey coolnow, it's not written somewhere. Because it's not in the Safari
Extension Gallery (yet), you have to download and install it manually.
Visit the project's releases page (
https://github.com/mmichaa/passafari.safariextension/releases) and
download the latest .safariextz file. To install the safari extension, just
double click it in finder. Then Safari will ask you to install it or not. I
hope that this helped you. Regards, Michael.
—
Reply to this email directly or view it on GitHub
https://github.com/mmichaa/passafari.safariextension/issues/1#issuecomment-93258596
.

Hey @coolnow , did you found a .safariextz yet?

authors: coolnow mmichaa


text:
Move CONTRIBUTING.md and README.md to .github folder

I think we will all agreed on this point: we have so mmmmaaannnny files in our root directory. It's possible to move "community health files" to a standard .github folder.

pro: here,
con: Gitlab don't look in github directory (not really a problem for me, this template is for small projects and big ones already knows how to deal with this).

👍 	for CONTRIBUTING but the README is maybe one of the most important part of the project, might be worth keeping it at the root.

@mnapoli opela

🎉

authors: mnapoli pocky

repo: mnapoli/project-template


text:
ENH: Bash entry for windows, normalize names

it seems like we should use _post2 rather than _2 in our names as it's more standard (and it's what we're calling our releases anyway)
This uses the _2 conda builds, so it should include a Bash entry on Windows now.

@hoechenberger if you're happy with this, could you merge and cut a release draft, then you or I could test on Windows? I'd encourage you to try VirtualBox again, the ~20 GB Windows test VM worked pretty well for me there in testing!

No, not happy. The only reason I append _number is because _YYYYMMDD got too long when I tried it early ob. But I think we have since changed the string before that and we can try and see if adding the date again would work.
I don't want to bother users with highly technical stuff like postwhatever, while still indicating which revision the installers have.

I want some sort of counter, or the date (which I'd prefer)

I think a counter might be better, for the same reasons as above: 1) users probably don't care about this technicality, and 2) it makes the name even longer. I'll revert to _2

Hooray, a conflict on Windows :(
The other unsolved issue here is why I had to -eq 3 here for .desktop and .app rather than -eq 5 on main. https://github.com/mne-tools/mne-installers/pull/114 should tell us if it's really from this PR or from (not) checking the exit codes of all commands.

https://github.com/mne-tools/mne-installers/pull/114 should be merged first, then I'll rebase this

I guess that's a problem where mne-feedstock does not build the mne-installer-menus target.

Bumping the build number should enforce a rebuild though … And in fact, the package was rebuilt: https://anaconda.org/conda-forge/mne-installer-menus/files
So I'm not sure where the conflict is coming from …
Oh, yeah, I might have an idea. The package is noarch so it's only built on Linux. We could make it non-noarch to avoid such breakage in the future, I think. But it really shouldn't happen in the first place :(

Let's try running now, maybe it was some temporary resolution error.
If it doesn't work, we should just revert https://github.com/conda-forge/mne-feedstock/pull/89 so that we can make the 1.0.1 installers.

authors: hoechenberger larsoner

repo: mne-tools/mne-installers


text:
Python 3.9 still listed as minimum supported version in environment.yml

Should environment.yml be updated to give 3.10 as the minimum version, like in pyproject.toml?

Yes PR welcome!

authors: larsoner tsbinns

repo: mne-tools/mne-python