Huobi PRO empty fetchBalance return

I'm a beginner. I want to be a bot. I want to ask you something! I don't show my account balance with console.log (await huobipro.fetchBalance ()),

but actually there is a balance,

excuse me, what did I make wrong? Thank you

Hi, @gavin369 ! Can you add the following line please to see what huobi actually returned to ccxt?
let huobipro = new ccxt.huobipro ({
    apiKey: 'YOUR_KEY',
    secret: 'YOUR_SECRET',
    verbose: true, // ←---------------------- this line added to see more verbose info
})

// ... then call the balance like you did in your code and check the onscreen output

Paste the output here, please. Use triple backticks for pasting code like for more readability:
&#96&#96&#96
code
&#96&#96&#96

&#96&#96&#96
'use strict';
var ccxt = require ('ccxt');
var async = require('async');
console.log ('添加交易所名称',ccxt.exchanges[56] )
console.log ('添加交易所个数',ccxt.exchanges.length)
var start = async function () {
// 在这里使用起来就像同步代码那样直观
let huobipro = new ccxt.huobipro ({
apiKey: 'key',
secret: 'secret',
})
while (true){
ticker = (await huobipro.fetchTicker('LTC/BTC')).info;
Balance = await huobipro.fetchBalance ();
console.log (Balance)
//console.log ('买1价',ticker.ask[0],'卖1价',ticker.bid[0],'买1深度',ticker.ask[1],'卖1深度',ticker.bid[1]);
//console.log (Balance)
await sleep(3000);
}
};
start();
&#96&#96&#96

You should not put &#96 there, use the backtick (`) on your keyboard 3 times in the beginning and 3 times in the end ) That's not a problem, just letting you know for future use ;)
The code is ok, but please paste the verbose output as in my previous post. We need you to add that verbose: true line and run your code again, then paste the output here.

Okay, I got it. First contact with GitHub. sorry！

@gavin369 no problem at all ) Welcome ) We will stand by to see your verbose output.

@gavin369 did you manage to get verbose output? We will be waiting to see your output in verbose mode, like written above.

@gavin369 alternatively, after you fetch the balance you can simply do:
console.log (Balance['info']);

... and paste the output here.

Now I'm not using console.log, but it's also printing information!
'use strict';
var ccxt = require ('ccxt');
var async = require('async');
verbose: true
console.log ('添加交易所名称',ccxt.exchanges[56] )
console.log ('添加交易所个数',ccxt.exchanges.length)
var start = async function () {
    // 在这里使用起来就像同步代码那样直观
    let huobipro = new ccxt.huobipro ({
        apiKey: 'apiKey',
        secret: 'secret',
        verbose : true,
    })

    while (true){
        //ticker = (await huobipro.fetchTicker('LTC/BTC')).info;
        Balance = await huobipro.fetchBalance();
        //console.log (Balance);
        //console.log (Balance)
        //console.log ('买1价',ticker.ask[0],'卖1价',ticker.bid[0],'买1深度',ticker.ask[1],'卖1深度',ticker.bid[1]);
        //console.log (Balance)
        await sleep(3000);
    }

};

@gavin369 ok, thank you for the info! I will now debug it a little and will get back to you shortly.

@gavin369 ok, this was a bug in balance parsing, but now it's fixed and Huobi should be working normally. Please don't forget to update the ccxt library to version 1.9.163+ (npm update ccxt in your package folder) and retry again. We will be happy if you report whether it works for you or not. Thx again for your participation!
P.S. Next time don't paste text in images and screenshots, paste code and verbose output in text enclosed with 3 backticks (`) for all text information. When we are programming, it is much better and much easier to read and edit, if both the code and the output are copied and pasted in text, not as an image. Anyway, thanks for your help! ;) Let us know if you have any other questions or difficulties with it. The library is under heavy development and we really appreciate your feedback! )

Okay，i know！ Thank you very much.


Costing builtin functions for showing things

I added some builtin show functions, and followed the steps in CostModelGeneration.md up to step 8. Please check if everything looks good before I start running the benchmarks.
In particular I'm not quite sure whether the memory models in CreateBuiltinCostModel are reasonable, especially the one for Data.
Also, step 8 in CostModelGeneration.md suggests running cabal bench plutus-core:cost-model-test, but I got cost-model-test: R Runtime Error: Error in filter.and.check.nonempty(., fname) : No data found for ShowBool, which I guess is expected since the csv file doesn't have the data for the new functions yet?
Once we get the cost model for all these functions, we can proceed to consider whether to combine them into a single function or not.

Also, step 8 in CostModelGeneration.md suggests running cabal bench plutus-core:cost-model-test, but I got cost-model-test: R Runtime Error: Error in filter.and.check.nonempty(., fname) : No data found for ShowBool, which I guess is expected since the csv file doesn't have the data for the new functions yet?

Yes.  Those error messages are maybe not very informative, but that means the frame that the R code reads from the CSV file doesn't contain any entries with ShowBool at the start.  If you really want to get past that you could stick some fake data in the CSV file (but then of course the models would be meaningless).

Is there any way that we could make it so that Show can only be used inside Trace to stop that kind of thing?

Great point! I think we should do that, and one method is adding checks in the plugin to only allow show... functions to be used within the first argument of trace. @michaelpj What do you think?

one method is adding checks in the plugin to only allow...

Well I forgot that not everybody uses the plugin :smile:
It would be difficult and/or annoying to check that in the CEK machine, I guess, so I don't know.

Oh wait.  Did you forget to add benchmarks for the new functions (Step 5)?  They should be in budgeting-bench somewhere, but I don't see anything.

Did you forget to add benchmarks for the new functions (Step 5)?

Right, good catch. Good thing I haven't started running it :smile:

Did you forget to add benchmarks for the new functions (Step 5)?

Right, good catch. Good thing I haven't started running it smile

Yes, that would have been embarrassing...

If a script says if show x == "xyz" then ... else ... we'll have to be sure that what it shows remains the same forever. Is there any way that we could make it so that Show can only be used inside Trace to stop that kind of thing?

Actually I think there's an easy solution... just not supporting showing Data at all. Is there any case where one would want to trace Data in a script? Why not just trace the original type, whether it's Datum or Redeemer or whatever?
Even if there is a legit reason for showing Data, it is already doable now, without using a show function: trace $ decodeUtf8 $ unsafeFromBuiltinData data. So there's little reason to provide a prettier show data function and risk behavior changes.
And finally, even if we really want to provide a prettier show data function, it doesn't have to be a builtin function... we can simply derive a PlutusTx.Show Data instance and use that.
So I'll remove the showData builtin.

If they're intended for debugging purposes then we could have a single very expensive costing function to encourage people to remove calls to the Show builtin before submitting scripts to the chain (or even disable it altogether if that's still an option).

If we want to do this we can just not include it in the set of builtins that's allowed on chain. But that's pretty inconvenient since we force people to have two compilation configurations, one for development and one for release. Plus, you can actually get the node to give you the evaluation logs, so it's sort of nice to be able to get this information out. If we don't actually ban it then I think it's very hard to prevent it being executed.

Is there any way that we could make it so that Show can only be used inside Trace to stop that kind of thing?

This doesn't help us: trace gets executed on the chain too, so its arguments will get forced. Hard to escape with a strict language.
That said, I expect that calling show on anything will be ruinously expensive, so people will probably try quite hard to do this anyway. Unclear.

Even if there is a legit reason for showing Data, it is already doable now, without using a show function: trace $ decodeUtf8 $ unsafeFromBuiltinData data.

I don't see how this function would work.

And finally, even if we really want to provide a prettier show data function, it doesn't have to be a builtin function... we can simply derive a PlutusTx.Show Data instance and use that

Yes, Data is a structured data type like lists and pairs, so we indeed don't need a primitive, and we can define it recursively in user-space.

one method is adding checks in the plugin to only allow...

Well I forgot that not everybody uses the plugin smile It would be difficult and/or annoying to check that in the CEK machine, I guess, so I don't know.

Oh, of course (slaps forehead). I was thinking in Haskell terms too, but I think it'd be really hard in Plutus Core: you'd probably need some special handling for that one builtin, which would need some extension to the language. OK, let's forget about that.

I don't see how this function would work.

Not trace $ decodeUtf8 $ unsafeFromBuiltinData data, but  trace $ decodeUtf8 $ serialiseData data.

Not trace $ decodeUtf8 $ unsafeFromBuiltinData data, but trace $ decodeUtf8 $ serialiseData data.

Thanks, I hate it :)

We only really need ShowInteger and ShowByteString as builtins, so I removed the rest.
I still wonder whether ShowByteString should just Prelude.show the ByteString, or should show a base64-encoded one.

@effectfully If we go with monomorphic "show" builtins, we only need two (for now), not five: showInteger and showByteString. All other builtin types can have their PlutusTx.Show instances defined without additional builtins.
I costed showInteger and showByteString; they are both nicely linear and have similar parameters, so (confirmed with @kwxm) from a costing perspective there's no problem combining them into a single builtin function.
This means there are three options:

Add a single show builtin

+: A single builtin for showing all builtin types
-: The PlutusTx.Show instances for lists and pairs do not use the builtin directly (because lists and pairs may contain non-builtin types). So in a script compiled from Plutus Tx, the show builtin will never be applied to lists or pairs. Its ability to show lists or pairs is thus largely redundant, and potentially confusing, especially if the behavior isn't exactly the same as using the PlutusTx.Show instances.
-: I have not costed such a show builtin on types other than Integer and ByteString, and indeed there are an infinite number of builtin types. That being said, since we are just converting things into strings, assuming a simple linear model for show (including for any future builtin types) is probably safe.

Add a single show builtin, but only allow it to be applied to Integer or ByteString. Applying it to other types results in EvaluationError.

+: A single builtin
-: Having a function called show which can only show Integer or ByteString sounds extremely dubious.

Add both showInteger and showByteString

+: Averts the disadvantages of the previous two options
-: Need two new builtins
-: Need to add additional builtin functions if we decide to add more primitive builtin-types in the future, such as Char, Double, or Natural.

My personal recommendation is (3), given that we only need two new builtins for now and there's no plan to add more primitive builtin types. What do you think?

@zliu41 both 2 and 3 sound good to me. For 2 we could call the builtin showSupported or whatever to make it clear that it doesn't handle all the built-in types. I agree that 3 is the simplest and cleanest solution currently, but 2 is future-proof, which is why I like it. I don't think we should worry about costing (unless we're keen to merge this ASAP), because I'm making the machinery more expressive anyway (by accident, it was just an optimization originally) and it should allow us to easily merge multiple costing functions into a single one.
I assume the team's gonna pick 3 and I'm not against it.

I don't know how I feel about this.  Option 3 seems sensible, but I'm worried about what happens if we add more types, and also what'll happen if users come along and say "We really need show for data!!!" or other types.

I still wonder whether ShowByteString should just Prelude.show the ByteString, or should show a base64-encoded one.

I like 0x3f65129c4abb28, but that doesn't seem to be especially popular (although it does almost match our PLC concrete syntax).  The Prelude.show version is horrible!

I costed showInteger and showByteString; they are both nicely linear and have similar parameters, so (confirmed with @kwxm) from a costing perspective there's no problem combining them into a single builtin function.

A possible objection to this is that if we get things wrong here it's difficult to change them, so it might be bet to keep our options open for now.  @effectfully 's ongoing work (and other planned things) might mean that won't be such a problem in future though.

what'll happen if users come along and say "We really need show for data!!!" or other types.

I highly doubt a builtin for showing data is appropriate:

The justification to add a builtin function should be: either it cannot be done without the builtin, or is very inefficient. We can show Data without the builtin, and the cost should be similar.
Using a builtin makes it a lot harder to change the behavior of showing Data, as you pointed out.
Data contains lists and pairs, and we don't really want the builtins to handle showing lists or pairs, because we need to be able to show lists and pairs that contain non-builtin types.

I like 0x3f65129c4abb28

I'm fine with hex; it's less efficient than base64, but more readable.

@effectfully 's ongoing work (and other planned things) might mean that won't be such a problem in future though.

Agreed

I don't think we need showByteString. There is no one way to show a bytestring, and privileging on may be a mistake. We already have decodeUtf8, we could potentially add decodeHex, and pick on of them to use in the Show typeclass in plutus-tx. (I rather regret using decodeUtf8 for the IsString instance, I even wonder if we should remove it if we add decodeHex or something...)

We already have decodeUtf8, we could potentially add decodeHex

That assumes most or all of the ByteStrings on chain are utf8 or hex, is that the case? Even so, I'm not so sure show @ByteString should be a partial function.

showHex is total, right? All bytestrings can be shown as a sequence of bytes represented as hex.

showHex is total, right?

Yeah encodeHex would be total. I thought you meant decodeHex in a similar way as decodeUtf8.
So you also prefer hex to base64?

I don't know. I wish we didn't have to pick. It's annoying, especially since we need to include them as builtins :/ I guess conceivably we could implement a really horrible to/from hex conversion in Plutus Tx?


Allow containers in csv import

[x ] I agree to follow the Code of Conduct that this project adheres to.
[x ] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Is your feature request related to a problem? Please describe.
I have been trying to create an import that represents an SSIS package for a sql server process, but find draw io is too limited to do that
Describe the solution you'd like
In the current csv import spec i see no way to add tasks / objects inside a container to group them together. If that was there i could achieve my goal.
Describe alternatives you've considered
I will have to use another tool if this cant be achieved
Additional context
My ultimate goal is to be able to create diagrams from a csv file that represents the flow, loops, and containers typically found in SQL Server Integration Services packages

Hi David,
that worked thank you very much… but… that seems to make every shape a container.
  The CSV import example seems quite basic and I can find no documentation on line as to all the options and styles available for the CSV import.
What I am really after is something like this below, is there an example csv??
Where there are some shapes inside a container and some outside.  The other important aspect for me is having different Line labels, and colours based on the data in my CSV. I could only see how to hard code the same label on every arrow.
Really appreciate your help with this, if I can get it to work ill be using draw.io for quite a project.
Thanks,
Geoff
From: David Benson @.>
Sent: Tuesday, July 18, 2023 2:51 PM
To: jgraph/drawio @.>
Cc: tempest-1973 @.>; Author @.>
Subject: Re: [jgraph/drawio] Allow containers in csv import (Issue #3731)
Try "container=1" in the shape style.
—
Reply to this email directly, view it on GitHub https://github.com/jgraph/drawio/issues/3731#issuecomment-1640270563 , or unsubscribe https://github.com/notifications/unsubscribe-auth/BBKBLQZAPXFLN5GH4YK623TXQ2IEJANCNFSM6AAAAAA2MKMUXI .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/BBKBLQYCS767UMEPGZC4NQDXQ2IEJA5CNFSM6AAAAAA2MKMUXKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTBYSFOG.gif Message ID: @.*** @.***> >


Fixed import error for rapidfuzz 3.X.X

What does this PR do?
Fixes import error for rapidfuzz 3.X.X(tested 3.0.0, and 3.2.0)
Why was it initiated? Any relevant Issues?
rapidfuzz-test.py code:
from rapidfuzz import fuzz

# Should print around 96.55 based from the docs
print(fuzz.ratio("this is a test", "this is a test!"))

Using concrete python 3.11.4(x86_64):
nuitka --standalone --windows-force-stderr-spec=errors.txt rapidfuzz-test.py

Running the .exe file will trigger the following error:
Traceback (most recent call last):
  File "...\rapidfuzz-test.py", line 1, in <module>
  File "...\rapidfuzz\__init__.py", line 10, in <module rapidfuzz>
  File "...\rapidfuzz\distance\__init__.py", line 6, in <module rapidfuzz.distance>
  File "...\rapidfuzz\distance\OSA.py", line 8, in <module rapidfuzz.distance.OSA>
  File "...\rapidfuzz\_utils.py", line 105, in fallback_import
  File "importlib.py", line 126, in import_module
ModuleNotFoundError: No module named 'rapidfuzz.distance.metrics_py'

See: rapidfuzz hook for pyinstaller which has .metrics_py only since v3.0.0, which is not in the implicit configs.
The only test I did was to compile the code above as the change will only affect importing rapidfuzz.
PR Checklist

[X] Correct base branch selected? Should be develop branch.
[X] Enabled commit hook or executed ./bin/autoformat-nuitka-source.
[ ] All tests still pass. Check the Developer Manual about Running the Tests.
There are GitHub Actions tests that cover the most important
things however, and you are welcome to rely on those, but they might not
cover enough.
[ ] Ideally new features or fixed regressions ought to be covered via new tests.
[ ] Ideally new or changed features have documentation updates.

This is perfect. I am merging this, and adding it to Nuitka-Watch too.


Black screen on Android Emulator

Steps to Reproduce
Freshly installed Android Studio. I tried to run my app on Android emulator but my app fails to render and I get a black screen. When I connect my real device, everything works fine. iOS simulator also works fine. Also, the default "new project" flutter app works fine on Android emulator! Tested on Android O and M emulators.
When the app is opening, I get this black box on the bottom-left corner for half a second and after that, the whole screen goes black. If I keep clicking on the screen, it often keeps flashing again with this same black box.

Logs
There are no any errors in flutter run logs. But when the screen is flashing I get this log, not sure if helpful:  D/skia    ( 4813): OpList failed to instantiate.
Here are the verbose logs: https://hastebin.com/ehojeqitil.sql
Flutter Doctor
[✓] Flutter (on Mac OS X 10.12.6 16G1036, locale en-US, channel alpha)
• Flutter at /Users/jokr/Documents/flutter/flutter
• Framework revision e8aa40eddd (4 weeks ago), 2017-10-17 15:42:40 -0700
• Engine revision 7c4142808c
• Tools Dart version 1.25.0-dev.11.0
[✓] Android toolchain - develop for Android devices (Android SDK 27.0.1)
• Android SDK at /Users/jokr/Library/Android/sdk
• Platform android-27, build-tools 27.0.1
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)
[✓] iOS toolchain - develop for iOS devices (Xcode 9.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 9.1, Build version 9B55
• ios-deploy 1.9.2
• CocoaPods version 1.3.1
[✓] Android Studio (version 3.0)
• Android Studio at /Applications/Android Studio.app/Contents
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)
[✓] IntelliJ IDEA Community Edition (version 2017.2.5)
• Flutter plugin version 19.0
• Dart plugin version 172.4343.25
[✓] Connected devices
• Android SDK built for x86 64 • emulator-5554                        • android-x64 • Android 6.0 (API 23) (emulator)
• iPhone 7                     • A4A35184-4527-499F-B58A-66D0EF65A193 • ios         • iOS 11.1 (simulator)
• iPhone X                     • E39C9791-6662-43FE-B77F-4DFACEECBEE3 • ios         • iOS 11.1 (simulator)
EDIT: Here's the repo if anyone is interested: https://github.com/itsJoKr/iOS4Android

Fixed by adding --enable-software-rendering. Found here:  https://github.com/flutter/flutter/issues/12428

Using --enable-software-rendering should not be required.
We've had many reports like this.  e.g. https://github.com/flutter/flutter/issues/11947 https://github.com/flutter/flutter/issues/11484 https://github.com/flutter/flutter/issues/10617
@741g  from the android emulator team was able to help us with a previous time this occurred.  I thought we had fixed all these, but presumably not.
I'm not sure how we should be debugging these in the wild?

This could be a new, different breakage. It doesn't look like a black screen error, rather something else is going wrong. I'll look into it.

The "OpList fails to instantiate" error could just be a Skia regression?  See also https://github.com/flutter/flutter/issues/12980 and https://github.com/flutter/flutter/issues/13018 which mention that error.  FYI @chinmaygarde

Just to add. Usually, these three lines will be logged many times, especially when I try clicking and then it starts to flash.
[        ] D/EGL_emulation( 4325): eglMakeCurrent: 0xa1299b00: ver 2 0 (tinfo 0xad08b000)
[        ] D/skia    ( 4325): OpList failed to instantiate.
[  +15 ms] I/chatty  ( 4325): uid=10083(u0_a83) gpu_thread identical 2 lines

The eglMakeCurrent ver there gives us a clue as to which GLES version is used on the emulator; it's not indicative of an error in the emulator driver.
OpList failed to instantiate, however, could be.

Wondering if this was before/after https://github.com/flutter/engine/commit/e03cab39d764c2a15c341b4a8576352bb878263f was landed in the engine?

The flutter doctor output in the original report shows using an flutter/engine which is about a month old.  flutter/engine@7c4142808c

@brianosman @eseidel
I've switched the channel to master and upgraded flutter. I don't see black screen issue anymore on the same project.

Fantastic!


Implement new related nav design

What
Implement redesign of the related navigation sidebar on content pages. This includes an updated behaviour to the toggling of overflow content. The toggle button now no longer includes the number of hidden items, instead just reading "more x", includes the same chevron icon seen on the accordion and shows hidden items as individual list items instead of all hidden items bunched into a single list item.
Additionally removes the "Explore the topic" section at the bottom fo the page, the print buttons and the brexit CTA.
Why
So that we can test this new pattern with users.
The removals are because the "explore the topic" section is being retired and the print buttons are unnecessary for this specific experiment.
Example page
Visual changes

Context
Before
After

Full page desktop

Full page mobile

Related nav toggle closed

Related nav toggle button hover

Related nav toggle open

Preview here https://explore-prot-build-rela-4k27rn.herokuapp.com/guidance/trader-support-service

There's a gap between the new related content background and the footer on mobile (but doesn't match your screenshot)

Tested in Chrome and on my iPhone 12

@chao-xian Could you test again please? I'm not seeing the gap...

Using this https://explore-prot-build-rela-typc1y.herokuapp.com/guidance/trader-support-service

@owenatgov huh... I definitely was able to reproduce this on my personal iPhone to double check.

May've been an old implementation before I sorted out that spacing properly 🤷


jsonfeed 1.1 "authors" not supported?

I tried copying the content of a jsonfeed example into a feed:
{
    "version": "https://jsonfeed.org/version/1.1",
    "user_comment": "This is a microblog feed. You can add this to your feed reader using the following URL: https://example.org/feed.json",
    "title": "Brent Simmons’s Microblog",
    "home_page_url": "https://example.org/",
    "feed_url": "https://example.org/feed.json",
    "authors": [
        {
            "name": "Brent Simmons",
            "url": "http://example.org/",
            "avatar": "https://example.org/avatar.png"
        }
    ],
    "items": [
        {
            "id": "2347259",
            "url": "https://example.org/2347259",
            "content_text": "Cats are neat. \n\nhttps://example.org/cats",
            "date_published": "2016-02-09T14:22:00-07:00"
        }
    ]
}

NetNewsWire 5.1.3 doesn't pick on any author here. If I change "authors" to "author" and just put one object instead of a list inside, it is handled as expected:
{
    "version": "https://jsonfeed.org/version/1.1",
    "user_comment": "This is a microblog feed. You can add this to your feed reader using the following URL: https://example.org/feed.json",
    "title": "Brent Simmons’s Microblog",
    "home_page_url": "https://example.org/",
    "feed_url": "https://example.org/feed.json",
    "author": {
        "name": "Brent Simmons",
        "url": "http://example.org/",
        "avatar": "https://example.org/avatar.png"
    },
    "items": [
        {
            "id": "2347259",
            "url": "https://example.org/2347259",
            "content_text": "Cats are neat. \n\nhttps://example.org/cats",
            "date_published": "2016-02-09T14:22:00-07:00"
        }
    ]
}

The jsonfeed 1.1 spec says about this:

JSON Feed version 1 specified a singular author field instead of the authors array used in version 1.1. New feeds should use authors, even if only 1 author is needed. Existing feeds can include both author and authors for compatibility with existing feed readers. Feed readers should always prefer authors if present.

I found this today as well, I assume it is the same issue so I did not open a new.
If I view a feed which NNW pulls directly ("On My Mac" or iCloud), the author field populates. If I pull the same feed through an aggregator, the author field is blank.
I tested this on both Inoreader and my own FreshRSS instance, and across all of [Big Sur, Monterrey, iOS 14, iOS 15] with [6.0.2, 6.0.3b1, git], the behaviors were the same. I am happy to provide more testing as requested, the author field is important for feeds which are an aggregation of many authors.


test_get_all fails

For some reason, find/find_all works well works, but calling hashes() on a corpus results in completely wrong hashes being returned.
Running OSX 10.10.3 with Python 2.7.6, just cloned from the repo and followed the instructions.
Could it have something to do with 64 vs 32bit when compiling, which I am getting errors about?
During compile I get a lot of these errors:
warning: implicit conversion loses integer precision: 'uint64_t' (aka 'unsigned long long') to 'Simhash::hash_t' (aka 'unsigned long') [-Wshorten-64-to-32]

simhash-py has only been tested in 64-bit environments. If you're compiling it for 32-bit, you're on your own for help.

I'm not compiling in a 32-bit environment, I investigated the issue further, and setup.py simply compiles a universal binary by default. However, forcing it to only compile for x86_64 doesn't help (obviously, as it was using the 64-bit implementation in the universal binary by default)

The issue actually lies in simhash-cpp, namely the const_iterator is wrongly implemented and does not properly loop through the Judy array. I'll submit a fix.

Could you please confirm that this has been fixed by pulling in recent updates from simhash-cpp in #19?

I'm going to assume this was resolved with the prior PR since there's been no update on this ticket in a long while.


Wrap name in box to prevent unwanted hyphenation

https://github.com/maucejo/elsearticle/blob/d48036c6f25dc28f5a23507fb83377c4bded946c/src/elsearticle.typ#L197-L203

Thanks @jamesrswift - It is how I wanted to solve it and you confirm my intuition :)


Retrain.py try download remote archive

I want manually set input directory, but retrain.py downloading remote archive. Why so?
python retrain.py --image_dir=D:\123\flower_photos\

I apologize but I am having a hard time understanding what the problem is, where the problem is, and what version it affects. Please resubmit and pay attention to the issue template (https://github.com/tensorflow/tensorflow/issues/new) . Please provide all the information it asks. Thank you.


feat: new poster html

What does it do?

Updates poster with new layout. -  https://trello.com/c/ODDwDtIa/35-atualizar-cartaz-html-com-novo-layout

This is how it looks when sending to print

after last changes, looks like this


How can I set the expiration time of Notification

>  By default, push notifications expire three days from the time they are received by the device. If needed, you can override this default with an explicit expiration time (toasts have a max of three days).
Where can I do that? Is there an example? I don't know how I can set the expiration time with NotificationHub. I only find a way to set the expiration time of a registration.

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

ID: bce1599b-38fc-79e6-d29c-c8d81cdd30f9
Version Independent ID: df2b1c9e-2d08-0654-1464-f88f9aafd9a6
Content: Choose a notification delivery method - Windows UWP applications
Content Source: windows-apps-src/design/shell/tiles-and-notifications/choosing-a-notification-delivery-method.md
Product: uwp
Technology: design
GitHub Login: @mijacobs
Microsoft Alias: mijacobs

Hey @eluchsinger, for push notifications, you can set the expiration time by using the request headers.
To do that, you use the X-WNS-TTL header. The value is in seconds. If you specify it, once the Windows PC receives the notification, it'll expire it in X seconds.
So, if you wanted your notification to expire 1 hour after it's received, you would send the following HTTP request
POST https://db3.notify.windows.com/?token=AgUAAADCQmTg7OMlCg%2fK0K8rBPcBqHuy%2b1rTSNPMuIzF6BtvpRdT7DM4j%2fs%2bNNm8z5l1QKZMtyjByKW5uXqb9V7hIAeA3i8FoKR%2f49ZnGgyUkAhzix%2fuSuasL3jalk7562F4Bpw%3d HTTP/1.1
Authorization: Bearer EgAaAQMAAAAEgAAACoAAPzCGedIbQb9vRfPF2Lxy3K//QZB79mLTgK
X-WNS-RequestForStatus: true
X-WNS-Type: wns/toast
X-WNS-TTL: 3600
Content-Type: text/xml
Host: db3.notify.windows.com
Content-Length: 196

<toast launch="">
  ...
</toast>

As for doing that via Notification Hub, I'm not an expert there. I assume there's some way to set custom headers from Notification Hub to achieve this.


Made names on cards clickable issue#10

Description
Please include a summary of the changes and the related issue.
Add issue number at the end
Issue: https://github.com/justsanchez/FESonline.github.io/issues/
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] This change requires a documentation update

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation

Also fill out the Pull Request info for every PR

Sorry about that, I'm used to pressing CTRL + Enter to jump to the next line. When I tried doing that when creating the pull request it would just submit it instead of jumping to the next line.
Won't happen again, I'll fill out the pull request info next time.
Get Outlook for iOShttps://aka.ms/o0ukef

From: justsanchez @.>
Sent: Tuesday, July 2, 2024 8:55:48 PM
To: justsanchez/FESonline.github.io @.>
Cc: Rivera-hernandez, Jovani @.>; State change @.>
Subject: Re: [justsanchez/FESonline.github.io] Made names on cards clickable issue#10 (PR #16)
[This email originated from outside of UNCW]
Also fill out the Pull Request info for every PR
—
Reply to this email directly, view it on GitHubhttps://github.com/justsanchez/FESonline.github.io/pull/16#issuecomment-2204803199, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BHXRQ4DEOOGPHDQXFYPY2FTZKNDZJAVCNFSM6AAAAABKIQ7EZ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBUHAYDGMJZHE.
You are receiving this because you modified the open/close state.Message ID: @.***>

Have fixed errors.


MultiMap is overriding my entries

The fact that the index is calculated like this (in CaseInsensitiveHeaders), it overrides my entries.
private static int index(int hash) {
        return hash % 17;
}

It is intended to not allow you to have more than 17 entries and override them?
If so, could you please let me know what can I use instead?

@TatianaBuica did you find a bug ? do you have a reproducer ?

When I am trying to make the following insert into a MultiMap, keys doorsUser and encryptionAlgorithm are always fighting for index 4.
MultiMap form = MultiMap.caseInsensitiveMultiMap();
form.add("doorsUser", "aa");
form.add("encryptionAlgorithm", "");

The fact that the index is calculated as I mentioned above, it is a bit confusing and I am not sure if this is a bug.

is there a bug ? i.e is data overwritten when you use the MultiMap ?
it's expected to have collision inside the implementation, that's how hash tables work https://en.wikipedia.org/wiki/Hash_table#Collision_resolution

I ran the code you provided:
  public static void main(String[] args) {
    MultiMap form = MultiMap.caseInsensitiveMultiMap();
    form.add("doorsUser", "aa");
    form.add("encryptionAlgorithm", "");
    for (Object o : form) {
      System.out.println("o = " + o);
    }
    System.out.println("doorsUser=" + form.get("doorsUser"));
    System.out.println("encryptionAlgorithm=" + form.get("encryptionAlgorithm"));
  }

and I got
o = doorsUser=aa
o = encryptionAlgorithm=
doorsUser=aa
encryptionAlgorithm=


Invalid metrics for some tall chars when zooming in the browser. And now different height for bold and regular font

there is another way to fix it:
in pixi.js/packages/text/src/TextMetrics.js
al line 544:
canvas.height = height + 1;

instead of
canvas.height = height;

and then, at 584 line:
properties.ascent = baseline - i + 1;

instead of
properties.ascent = baseline - i;

it just adds 1px to the top of the canvas. this method has no side-effects
have a look please
@bigtimebuddy @themoonrat
Originally posted by @iv-menshenin in https://github.com/pixijs/pixi.js/issues/5594#issuecomment-485430676

This issue relates to https://github.com/pixijs/pixi.js/issues/5594#
After these changes on our codebase, we have a side-effect: we have offset of baseline for regular font versus bold font.

And this commit should be reverted:
https://github.com/pixijs/pixi.js/commit/3c019c77ca5482e2213b2acc1d9860d260cdabb6

A "+1" sounds like a hack without some evidence as to why it's the best solution. Surely actually calculating the height is better than fudging it?

not. I do not claim to be the ultimate truth. I just wanted to say that the solution with BASELINE gives side effects while we only need to increase the canvas by 1 px
In any case, you know better than me how to fix it.

And I am tried magic with these settings:

METRICS_STRING
BASELINE_SYMBOL
BASELINE_MULTIPLIER

everything looked bad

@iv-menshenin  That is a separate issue unrelated to the previous PR. Here is a playground using text from the last 4.8.7 version of PixiJS: https://www.pixiplayground.com/#/edit/Sgm5c~TGHlUfmN6bTBEss
Bold text and unbold text take up different amounts of space, so when you put them side by side they may not all have the same baseline.
I'd have no idea how to begin on solving that. You've have to measure every style variant I guess, and then pick something that they can all work on. It might be more in the scope of a non official pixi plugin: https://github.com/tleunen/pixi-multistyle-text

How did this happen? how can I remove the previous comment?


New builder without a suffix failing with an exception for StrictlyPositiveDouble

Description
Like discussed in #104, we would like to introduce an experimental builder without a suffix for the StrictlyPositiveDouble on all platforms that should throw an exception in case of a failure.
/**
 * Returns this number as a [StrictlyPositiveDouble], which may involve rounding
 * or truncation, or throws [IllegalArgumentException] if this number is
 * negative.
 *
 * Here's some usage examples:
 * ```kotlin
 * 1.toStrictlyPositiveDouble() // 1.0
 * 0.toStrictlyPositiveDouble() // IllegalArgumentException
 * (-1).toStrictlyPositiveDouble() // IllegalArgumentException
 * ```
 *
 * Instead of throwing an [IllegalArgumentException] when this number is
 * negative, you can use the [toStrictlyPositiveDoubleOrElse] function for
 * returning the result of the specified callback, or the
 * [toStrictlyPositiveDoubleOrNull] for returning `null`.
 */
@ExperimentalNumberApi
fun Number.toStrictlyPositiveDouble(): StrictlyPositiveDouble

With this new builder, we can deprecate with a warning level the experimental builder Number.toStrictlyPositiveDoubleOrThrow (introduced by the issue #132) for using the new builder introduced by this issue instead.
@Deprecated(
    "Use the toStrictlyPositiveDouble function instead.",
    ReplaceWith("this.toStrictlyPositiveDouble()")
)
fun Number.toStrictlyPositiveDoubleOrThrow(): StrictlyPositiveDouble

Dependencies
This issue is blocked by the following ones:

[ ] #133

Checklist

[ ] Implement and test the Number.toStrictlyPositiveDouble builder.
[ ] Deprecate the Number.toStrictlyPositiveDoubleOrThrow builder.
[ ] Refactor.
[ ] Check the generated documentation.
[ ] Check and update the public API binaries.
[ ] Update the Unreleased section in changelog.

Like discussed in #104, we will not add type converters that could fail with an exception.


Update concept-conditional-access-conditions.md for rebranding

Rebrand, “Windows Account extension" to “Microsoft Single Sign On”

@ejahja Can you also include some blurb in this docs page that...

This is required for Mac devices that enable Platform SSO and for devices that have opted-in to Secure Enclave backed device identity (https://learn.microsoft.com/en-us/entra/identity-platform/apple-sso-plugin#registeredenrolled-device-conditional-access-policy-support-in-chrome)
That admins can deploy it to their devices automatically with MDM [1, 2]

With some wordsmithing for clarity as needed

@MicrosoftGuyJFlo
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@ejahja - Thank you for your contribution. Would you take a moment to sign the Contributor License Agreement (CLA)? After the CLA is signed, someone can review your pull request. Thanks!

@microsoft-github-policy-service agree company="Microsoft"

thanks @ejahja
over to you for review @MicrosoftGuyJFlo

This should not be in the public repo.
@iambmelt @ejahja @EmLauber please reach out internally.
Not approved.


Chef solo version detection fails on a Debian 9

Each time I'm trying to cook on my Debian node, I have the following error :
ERROR: RuntimeError: Couldn't find Chef >=0.10.4 on XXX.XX.XXX.XX. Please run `knife solo prepare<EMAIL_ADDRESS>to ensure Chef is installed and up to date.

This is due to the fact that :

Debian doesn't have sudo by default, but this is workaround-able
The chef-solo --version is not outputting what the Chef::Knife::SoloCook.chef_version method expects.

In the awk command, knife-solo is expecting to have an output similar to Chef: 15.0.300, while on my node the chef-solo --version command outputs Chef Infra Client: 15.0.300.
In the case I didn't nothing wrong and the command output is fine, I then suggest the to change the cmd to the following :
sudo chef-solo --version 2>/dev/null | awk '/Chef:/{print $2} /Chef Infra Client:/ {print $4}'

So that the Chef: and Chef Infra Client: are both handled and knife-solo works in both cases.
If this is fine for you, I can open a PR.

I also need this pr!

As mentioned on https://github.com/matschaffer/knife-solo#deprecated I don't really intend on making any further releases of knife-solo.
You're welcome to do whatever you'd like with the code, but it seems the chef-solo flow has diverged quite a bit with the recent chef relicensing.
I haven't used chef actively in well over 5 years now so don't think I can reliably continue the project.


PHP Fatal error:  Interface 'Psr\Log\LoggerInterface'

One of our developers is using firelogger and developed issues with this on our linux server
I downloaded your current package and placed it on the webserver and got the following message in the apache logs when i ran basic.php (also had to copy FireLogger.php to firelogger.php):
PHP Fatal error:  Interface 'Psr\Log\LoggerInterface' not found in /var/www/html/firelogger.php-master/firelogger.php on line 36
I am using Centos 7 and php version 5.4.16. Is there something I need to add to my php implementation to make this work?

I'm sorry. I cannot help. Haven't touched this project for more than 4 years.


/server-components page doesn't exists

https://react.dev/server-components
Google Search Result:

Not found:

https://react.dev/blog/2020/12/21/data-fetching-with-react-server-components

I think this is out of the google cache now, so i'm going to close this.


Get-AzDataLakeStoreAccount returning Gen1 lakes only

This issue is related to datalakestore module. When I run the following powershell command it is only returning ADLS Gen1 storage accounts, not Gen2 lakes.
Get-AzDataLakeStoreAccount

And it is no where mentioned in the documentation that it only works with Gen1 lake.
https://docs.microsoft.com/en-us/powershell/module/az.datalakestore/get-azdatalakestoreaccount?view=azps-2.7.0

Sorry, but this repo is for documentation only. This sounds like a product bug, so please file a report with engineering.

@Murthy582 For Azure Data Lake Gen2, the Get-AzStorageAccount cmdlet gets a specified Storage account or all of the Storage accounts in a resource group or the subscription.
Hope this helps

@Murthy582 Get-AzStorageAccount cmdlet returns blob storage accounts and adls gen2 accounts.
Get-AzDataLakeStoreAccount cmdlet returns adls gen1 accounts.
Hope this helps.

Got it, Thank you. I can see ADLS Gen2 accounts with Get-AzStorageAccount command.
I'm trying to setup ACLs using PowerShell cmdlet. Is there any powershell command to get/set ACLs of a file/folder in ADLS Gen2 storage account?


Well , if you are interested in adding more advanced features check this out.

https://github.com/soyalk/probalk/issues/2

May add these in the future

oh great which ones did you like ?

Thank you!

you are welcome :)


duplicate declaration of method 'disableFeedbackForm'

https://github.com/testfairy/react-native-testfairy/blob/0b1981ead2b629664beeaf896f9f874919aa004d/ios/RCTTestFairyBridge.m#L182
I think you guys writes something's wrong..
I guess it should be
RCT_EXPORT_METHOD(disableAutoUpdate) {
instead of
RCT_EXPORT_METHOD(disableFeedbackForm) {

Thanks for letting us know @MisterFail. I've fixed the issue and pushed a new release, it should be out (v2.19.2) within the hour.
https://github.com/testfairy/react-native-testfairy/commit/5831653ebf85887b52ae9a2260a899d2dc2b2c37
Cheers


Can't initialize a project with Ruby 2.7.0 and middleman 4.3.9

Expected behavior and actual behavior
Expected
❯ middleman init wastelandwarfare
# Project is created...

Actual
❯ middleman init wastelandwarfare
Traceback (most recent call last):
	4: from /Users/brian/.rbenv/versions/2.7.0/bin/middleman:23:in `<main>'
	3: from /Users/brian/.rbenv/versions/2.7.0/bin/middleman:23:in `load'
	2: from /Users/brian/.rbenv/versions/2.7.0/lib/ruby/gems/2.7.0/gems/middleman-cli-4.3.9/bin/middleman:3:in `<top (required)>'
	1: from /Users/brian/.rbenv/versions/2.7.0/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
/Users/brian/.rbenv/versions/2.7.0/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require': cannot load such file -- middleman-core/profiling (LoadError)

Steps to reproduce the problem (from a clean middleman installation)

install rbenv
install ruby 2.7.0 via rbenv
set rbenv shell to 2.7.0

Additional information

Ruby version:  2.7.0
Middleman version: 4.3.9
OS version: macOS 10.15.5

I have this problem as well.  cannot load middleman-core/profiling

@spilth got any workaround at the moment you care to share?

No work around except giving a different static site tool a try.

ah.  I already use this one.  I went ahead and uninstalled and went back to middleman 4.3.8 which does work.  @spilth

MIddleman seems to have been having alot of bugs lately

I'm sure they'll address this soon.  4.3.8 was working for me just fine and still is.  I'll just wait until this is closed before updating.

Probably going to have to revoke this version. First time deploying from a windows box, I think it may have somehow missed some files. I'll dig in tomorrow. In the meantime, point the ole Gemfile at 4.3.8

Can confirm that 4.3.9 is causing this issue. Pinning the version at 4.3.8 resolves it

Okay, fixed. 4.3.10 is good to go. It was an issue with the git setup on the windows machine that was publishing.


panic: qtls.ConnectionState not compatible with tls.ConnectionState

Hi, I got an error when running brainfuck-psiphon-pro-go. here is the screenshot

I am still updating golang. any Idea if updating is not fixing this?

Hi, I got an error when running brainfuck-psiphon-pro-go. here is the screenshot

I am still updating golang. any Idea if updating is not fixing this?

same issue but solved by try another distro (old one) and you are done.

Hi, is the issue solved? If not, try psiphon-tunnel-core in this file (copy paste psiphon-tunnel-core)
https://mega.nz/file/XfwGzYID#VPu6YRlB0OVkZMeUbisLqnTEoJvJ24i0daCIgekCU9U


Retrieve external css and js from CDN

Preferably checking CDN performance.

Just a small speed tweak. With the side effect being that there is no need to keep these scripts in our repository (unless someone reports a problem with CDNs failing).


feat: Initial implementation of importAccountsByEncryptedUrl on MNW and Meteor Wallet

Description
This PR contains initial implementation for adding importAccountsByEncryptedUrl interface for My Near Wallet and Meteor Wallet. WEP-211
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

Type of change.

[ ] FIX - a PR of this type patches a bug.
[x] FEATURE - a PR of this type introduces a new feature.
[ ] BUILD - a PR of this type introduces build changes.
[ ] CI - a PR of this type introduces CI changes.
[ ] DOCS - a PR of this type introduces DOCS improvement.
[ ] STYLE - a PR of this type introduces style changes.
[ ] REFACTOR - a PR of this type introduces refactoring.
[ ] PERFORMANCE - a PR of this type introduces performance changes.
[ ] TEST - a PR of this type adds more tests.
[ ] CHORE - a PR introduces other changes than the specified above.

@andy-haynes I have commented Meteor wallet's buildImportAccountsUrl until we know for sure the wallet type of the Meteor Wallet is BrowserWallet type.


Stream content timeout not working for given site

Summary.
I would expect this to timeout but it doesn't.
r = requests.get('http://mywatchedmovies.com', stream=True, timeout=1)
But the following does
r = requests.head('http://mywatchedmovies.com', stream=True, timeout=1)
Not sure why?
Stackoverflow question: https://stackoverflow.com/questions/46158883/requests-stream-timeout-not-working-for-given-website
Reproduction Steps
import requests

def fetch_url(url, timeout, max_content, headers):

    r = requests.get(url, stream=True, timeout=timeout, headers=headers)
    r.raise_for_status()

    if int(r.headers.get('Content-Length',0)) > max_content:
        raise ValueError('response too large')

    content = r.raw.read(max_content+1, decode_content=True)
    if len(content) > max_content:
        print(url)
        raise ValueError('Too large a response')
    return {'url':url,'content':content,'status_code':r.status_code}

timeout = 10
max_content = 5000000
headers = {'User-Agent': 'Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/41.0.2228.0 Safari/537.36'}

fetch_url('http://mywatchedmovies.com', timeout, max_content, headers)

System Information
$ python -m requests.help

{
  "chardet": {
    "version": "3.0.4"
  },
  "cryptography": {
    "version": ""
  },
  "idna": {
    "version": "2.6"
  },
  "implementation": {
    "name": "CPython",
    "version": "3.5.2"
  },
  "platform": {
    "release": "4.4.0-1032-aws",
    "system": "Linux"
  },
  "pyOpenSSL": {
    "openssl_version": "",
    "version": null
  },
  "requests": {
    "version": "2.18.4"
  },
  "system_ssl": {
    "version": "1000207f"
  },
  "urllib3": {
    "version": "1.22"
  },
  "using_pyopenssl": false
}

Uh...this is a weird one.
Requests timeouts operate on a per-socket-event basis: that is, a timeout of 1 will fire if it takes more than one second for a server to send us some data: any at all. In this case, the server is sending us plenty of data.
The server is returning a 302 redirect to login.php. We would like to re-use that TCP connection for the next request, so we attempt to drain the response body. However, the response body is chunk-transfer-encoded, and seems to be completely endless: after an initial 746-byte chunk that contains some actual content the server just repeatedly sends the exact same 307-byte chunk endlessly forever. Because the server is active the whole time we never time out: we just get stuck in a loop trying to drain this socket, and fail.

Lukas: Thanks for getting back to me, I don't really need the data coming back I basically just don't want it to break my script. If I use requests.head() I get the timeout, which is good. Basically I am trying to weed out junk sites. I could add in requests.head() but then am I not hitting the server twice? I have to do many millions of domains as efficiently as possible. Do you have any suggestions?

In this case it's hard to do: due to a patch added a while back you can't prevent Requests trying to follow a redirect. ☹
In this case, you can try doing head(). You will hit the server twice, but only one TCP connection will be used.

This is probably a noob question but is it a correct reading of http://docs.python-requests.org/en/master/user/advanced/?highlight=tcp#session-objects
That each method request.get() and request.head() is initializing a new session and a new TCP connections? So if I wanted to use the same connection I should use something like this:
s = requests.Session()
s.head(url)
s.get(url)

Rather than:
requests.head(url)
requests.get(url)

To me this is saying that.
requests.head(url)
requests.get(url)

Is sufficient? As:

The requests module is not stateless; it just lets you ignore the state and effectively use a global singleton state if you choose to do so.*
And it (or, rather, one of the underlying libraries, urllib3) maintains a connection pool keyed by (hostname, port) pair, so it will usually just magically reuse a connection if it can.

https://stackoverflow.com/questions/24873927/python-requests-module-and-connection-reuse

If you want to connection pool you must use a Session.

Thank you sir. If you want to close this issue feel free. I have pulled down hundreds of millions of domains with requests and there has only been a few edge cases. Not sure if this one is worth your time. Appreciate what you do.


Add 209 new reactions after fixing the growth problem

Main improvements in this PR:
Try to be as clear as possible: Is it fixing/adding something in the model? Is it an additional test/function/dataset? PLEASE DELETE THIS LINE.

First, construct two draft models using RAVEN Toolbox. Model file are in Saccharomyces_cerevisiae_draftmodel_kegg and Saccharomyces_cerevisiae_draftmodel_metacyc
Then compare the draft models with yeast8 to find the new reactions.
In terms of the new reations from step 2, check if their are reasonable using metacyc, yeastcyc, uniprot, SGD and KEGG.
Get the final new reactions.

I hereby confirm that I have:

[x] Tested my code with all requirements for running the model
[x] Selected develop as a target branch (top left drop-down menu)
[x] If needed, asked first in the Gitter chat room about this PR

@edkerk that's my fault, i wiil detail more information.

I have reorganized the data, to fit #302.

[ ] Make sure that there are no more duplicated metabolites (3f1646d)
[ ] Make sure that there are no unused metabolites (26b55d8)
[ ] There should be much more metadata provided in DBnewRxnsMets.tsv, DBnewRxnsRxns.tsv and now also DBnewRxnsGenes.tsv
[ ] Many of the subsystems are very unique, and this does not make sense in accordance to #11 and #307.

But overall, I'm not convinced whether all these reactions should be included. What criteria were used to include them? What experimental evidence is there to support them? [To facilitate this, I changed the layout of the yeast-GEM.txt file (cb966bc, using exportForGit), which makes for easier diff-ing in 25b724b.]
Some examples:

rxnID
reaction equation
grRule

r_4855
oxygen[c] + Melatonin[c] => Formyl-N-acetyl-5-methoxykynurenamine[c]
YJR078W

r_4810
oxygen[c] + Serotonin[c] => Formyl-5-hydroxykynurenamine[c]
YJR078W

These are probably not correct. The breakdown of melatonin and serotonin, which are not yeast metabolites, has the same EC number as the reaction from tryptophan to N-formyl-kynurenine, which is a reaction in NAD biosynthesis. Actually, there are four reactions in this map with the same EC number, but only one of these is part of a functional pathway.
There are more examples like this, also based on MetaCyc. So how were these reactions selected?
Then there are also other problematic reactions. The following two reactions are modifying proteins, which is outside the scope of a metabolic network. Moreover, they are actually half-reactions of pyruvate dehydrogenase and alpha-ketoglutarate dehydrogenase (both already in the model, and associated with the same genes). So no need to include these:

rxnID
reaction equation
grRule

r_4833
coenzyme A[m] + pyruvate-dehydrogenase-acetylDHlipoyl[m] => acetyl-CoA[m] + pyruvate-dehydrogenase-dihydrolipoate[m]
YNL071W

r_4834
succinyl-CoA[m] + N6-dihydrolipoyl-L-lysine[m] <=> coenzyme A[m] + N6-S-succinyldihydrolipoyl-L-lysine[m]
YDR148C

There are other reactions that act on non-specific substrates:

rxnID
reaction equation
grRule

r_4755
2 H+[c] + H2O[c] + L-Selenocystathionine[c] => ammonium[c] + pyruvate[c] + Selenohomocysteine[c]
YGL184C or YHR112C or YFR055W

r_4835
H2O[c] + S-Substituted-L-Cysteines[c] => ammonium[c] + pyruvate[c] + Thiols[c]
YGL184C or YFR055W

There has been some discussion about including non-specific substrates (#219), but these genes are already associated to existing reactions (r_0308), so there is no value of including it as non-specific reactions.
There are also examples of fluorinated and chlorinated compounds that would not occur in S. cerevisiae.
Overall: The list of new reactions should be carefully cureated, to make sure that the models that are added make sense. More reactions is not perse better, even if it would not directly affect some of the model metrics (predicted growth rate, gene essentiality etc.).

@edkerk Is there any issue about the new reactions that I need to fix?

I have refactored the script and location of datafiles to match the generic curation format introduced in #313. See code/modelCuration/v8_6_1.m for how the model curation is performed.

I reiterate the last sentence of the previous comment: The list of new reactions should be carefully curated, to make sure that the models that are added make sense. More reactions is not perse better, even if it would not directly affect some of the model metrics (predicted growth rate, gene essentiality etc.).
So you should go through the list of reactions 1-by-1 and manually check whether they make sense. You uploaded draft models from KEGG and MetaCyc, but there is no explanation given which reactions are then included and why. I quickly looked through the new reactions, and found some more issues:
Double check that it is not duplicate of an existing reaction.

rxnID
reaction equation
grRule

r_0916
ATP[c] + ribose-5-phosphate[c] => AMP[c] + H+[c] + PRPP[c]
(YKL181W and YER099C) or (YKL181W and YHL011C) or (YKL181W and YBL068W) or (YER099C and YOL061W) or (YBL068W and YOL061W)

r_4723
ATP[c] + D-ribose 5-phosphate[c] <=> AMP[c] + H+[c] + 5-Phospho-alpha-D-ribose 1-diphosphate[c]
YBL068W or YHL011C or YER099C or YOL061W or YKL181W

The first reaction was already present, while the second reaction has different metabolite names, it represents the same reaction. This also highlights that there are duplicate metabolites, which otherwise would have made it easier to spot.
Double check that there are no duplicate metabolites.
See above, even if the reaction would not have been duplicate, then ribose-5-phosphate and ´D-ribose 5-phosphate` are highly likely the same metabolite, so make sure there is only one of them present.
Double check whether the reaction is likely to be present in S. cerevisiae

rxnID
reaction equation
grRule

r_0481
glutathione disulfide[c] + H+[c] + NADPH[c] => 2 glutathione[c] + NADP(+)[c]
(YCL035C and YPL091W) or (YDR098C and YPL091W) or (YDR513W and YPL091W) or (YER174C and YPL091W)

r_4711
2 glutathione[c] + NAD[c] <=> glutathione disulfide[c] + H+[c] + NADH[c]
YPL091W

The first reaction is how glutathione oxidoreductase is widely accepted to function. The new reaction is reversible, uses NADH and has a much simplified gene association. What strong evidence is there to include the second one?
Double check the gene associations
See both examples above, the new reactions have much simplified gene associations, while the old reactions indicate complexes with subunits. What strong evidence is there to have the simplified gene association?
See previous comment
But it's worthwhile to have another look at the previous comment as well, as these issues are not fully resolved. How is the localization determined? Be very careful with reactions predicted by MetaCyc, it can quickly draw in non-native substrates.

@edkerk Hi, Ed. I encounter a problem. When I fail to run deletion = cobra.flux_analysis.deletion.double_gene_deletion(model, gene_list1=pair1, gene_list2=pair2)  in python using yeast-GEM from both main branch and develop branch. Even if I change the version of cobrapy, I can not solve it. So, I am wondering if  saveYeastModel.m has changed.
The error is below:

Traceback (most recent call last):
File "D:\Anaconda\envs\python38\lib\site-packages\IPython\core\interactiveshell.py", line 3444, in run_code
exec(code_obj, self.user_global_ns, self.user_ns)
File "", line 47, in
deletion = double_gene_deletion(model,
File "D:\Anaconda\envs\python38\lib\site-packages\cobra\flux_analysis\deletion.py", line 393, in double_gene_deletion
return _multi_deletion(
File "D:\Anaconda\envs\python38\lib\site-packages\cobra\flux_analysis\deletion.py", line 144, in _multi_deletion
with ProcessPool(
File "D:\Anaconda\envs\python38\lib\site-packages\cobra\util\process_pool.py", line 56, in init
pickle.dump((initializer,) + initargs, handle)
TypeError: cannot pickle 'SwigPyObject' object

@edkerk @hongzhonglu Is there any methods to solve the above problem?

Hmm, even if saveYeastModel is changed, it would still produce a valid SBML file that cobrapy should be able to import without issues. Just to confirm that it is really a problem with the model itself, have you tried running it on another model (non yeast-GEM, maybe E. coli?).

@edkerk @hongzhonglu double_gene_deletion and single_gene_deletion could be perfectly performed in iML1515 and yeast-GEM 8.5. But in the latest yeast-GEM, somthing goes wrong.

However, matlab can run double_gene_deletion with solvable problem. And I am working on it.

I went through all suggested reactions, checked them one-by-one. With the quality of the current yeast-GEM, one should be careful to include new reactions, there should be more evidence than it appearing in KEGG. I checked with the following strategy:

Check if the reaction is not a partial reaction, which is already represented in the model as the complete reacton.
Compare the new reaction with existing reactions annotated to the same gene: if there is a difference (in e.g. substrate or co-factor), find evidence in literature if the new reaction is supported and/or likely to be present. Not only guided by KEGG or UniProt, but search for more solid evidence.
If the above are true, then see if the reactants and/or products connect to existing metabolites. If so, then include the reaction in that compartment, but do not add it to other compartments. This should rather be addressed by a thorough curation of all reaction compartmentalizations. If the reaction does not connect to the existing metabolic network, then just add it to whatever compartment is suggested.

I went through all suggested reactions, checked them one-by-one. With the quality of the current yeast-GEM, one should be careful to include new reactions, there should be more evidence than it appearing in KEGG. I checked with the following strategy:

Check if the reaction is not a partial reaction, which is already represented in the model as the complete reacton.
Compare the new reaction with existing reactions annotated to the same gene: if there is a difference (in e.g. substrate or co-factor), find evidence in literature if the new reaction is supported and/or likely to be present. Not only guided by KEGG or UniProt, but search for more solid evidence.
If the above are true, then see if the reactants and/or products connect to existing metabolites. If so, then include the reaction in that compartment, but do not add it to other compartments. This should rather be addressed by a thorough curation of all reaction compartmentalizations. If the reaction does not connect to the existing metabolic network, then just add it to whatever compartment is suggested.

I agree with the detailed strategy. With a standard workflow, we can add new reactions more efficiently and credibly.


Changing widths of integers across round-trips breaks relocations

I pointed out in https://github.com/paritytech/parity-wasm/pull/196#issuecomment-371278025 that LLVM/lld sometimes compiles integers to larger representations than they need. That particular case was an i32.const 0 compiled as 0x41 0x80 0x80 0x80 0x80 0x00 instead of just 0x41 0x00. parity-wasm can read it just fine, but it serializes it back out as 0x41 0x00 instead of the original representation.
This is a problem for relocations since relocations are measured in byte offsets from the start of the corresponding section. So a round-trip breaks all relocations after such a value.
It seems to me from testing that lld does this for every relocatable value, probably intentionally so that loaders / linkers doing have space to write the relocated value without needing to move bytes around.
How do you think this should be fixed?

Each RelocationEntry could have some reference to the corresponding Var*Int* (or to the section containing the Var*Int*. But this crosses section boundaries and be brittle.

Or, each Var*Int could store the width it originally had, and the serializer could artificially expand it to that width. But this breaks the From and Into impls for these types.

I believe we can just use separate type with custom serialization for such cases

Is it really about original length?
As far as I remember, wabt has a switch like “non canonical lebs” for such cases, and if it set then all lebs will be encoded with it’s maximal length.

Is it really about original length?

Yes. Only relocatable addresses are being emitted as 5-byte values. Non-address constants are being emitted as the smallest width.

I suggest to just add VarInt32NonCan or similar, and let it serialize to full 5 bytes always or remember it's deserialized length (not sure the latter is even required)

It does mean that the const opcodes like I32Const and the load/store opcodes like I32Load will have to change to hold VarInt32NonCan instead of i32 / u32

Yeah, this is no-go, i thought it was happening in relocation sections only

Another solution will require a lot of work
code/global section can be generic over VarInt32/VarInt32NonCan(or "VarInt32FixedLen"), with regular VarInt32 being default
Not sure if it worth troubles

How would that help? Like I said earlier, some opcodes contain smallest width and some opcodes contain largest width. It's not always one or always the other.
I'm thinking of storing a map of original offset -> opcode index in the code section and having the relocation section serialize differently based on how the code section was serialized. Haven't yet written the code to see how feasible it is because it means I need to have a custom Writer to be able to get the new opcode offsets out from CodeSection::serialize and then somehow propagate them to RelocSection::serialize...

something like this:
https://gist.github.com/anonymous/da3588dedc6427ec1c49e804a26ec12f#file-playground-rs
but surely it is not worth it

Parameterizing the whole Opcode means that opcodes like I32Load(u32, u32) can't have different widths for their two values.

Opcodes needs to hold both type of Opcode like I said above so you cannot parameterize it.

You just add two width fields (if both operands actually do require so), and you don't need to hold both type of Opcode, you just use width-preserving variant whenever you need it, and use it for all opcodes (since its abilities to serialize match those of regular variant if the representation is the most compact originally).

I made a prototype of how it would look for individual opcodes to store the original width with the value: https://github.com/Arnavion/parity-wasm/commits/min-width I only did it for I32Const and Call, the two opcodes that relocatable.wasm uses. I don't think it's particularly invasive. Tell me what you think.
That said, I want parity-wasm to be able to relocate modules in the future, which means it needs to be able to update these relocated values arbitrarily. I don't know if I want to depend on lld to always leave enough space like it does now. So I'm still going to try to implement my idea of storing a map in the CodeSection for original offset -> opcode, and changing the offsets in the RelocSection according to how the CodeSection is serialized.


calls: 0.1.5 -> 0.2.0 with init of callaudiod and feedbackd

Motivation for this change
To update calls and add its new dependencies: callaudiod and feedbackd.
The feedbackd package is from @masipcat's #94820, I just bumped the version. I can remove it if that PR gets merged first, but its necessary for this version of calls to build right.
Also callaudiod and feedbackd should probably be setup as modules at some point, possibly as a part of #88767. I might try my hand at it sometime.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

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

3 packages built:

callaudiod
calls
feedbackd

Updated calls to 0.2.0 and fixed name and version for feedbackd to match the naming rules. Callaudiod cannot be updated, because calls depends on 0.4.0, this should change in the next calls version.

Updated calls to 0.2.0 and fixed name and version for feedbackd to match the naming rules. Callaudiod cannot be updated, because calls depends on 0.4.0, this should change in the next calls version.

Result of nixpkgs-review pr 107681 1

3 packages built:

callaudiod
calls
feedbackd

Result of nixpkgs-review pr 107681 1

3 packages built:

callaudiod
calls
feedbackd

Package builds and runs. Cant test the actual functionality.

One thing i noticed is that when i close it, the process still runs until i kill it with CTRL+C.
[nix-shell:~/.cache/nixpkgs-review/pr-107681]$ calls

(calls:27003): dbind-WARNING **: 22:18:50.186: Couldn't connect to accessibility bus: Failed to connect to socket /tmp/dbus-GXwHDL6PHu: Connection refused

(sm.puri.Calls:27003): Gtk-CRITICAL **: 22:18:50.276: gtk_application_uninhibit: assertion 'GTK_IS_APPLICATION (application)' failed

^C

Package builds and runs. Cant test the actual functionality.

One thing i noticed is that when i close it, the process still runs until i kill it with CTRL+C.
[nix-shell:~/.cache/nixpkgs-review/pr-107681]$ calls

(calls:27003): dbind-WARNING **: 22:18:50.186: Couldn't connect to accessibility bus: Failed to connect to socket /tmp/dbus-GXwHDL6PHu: Connection refused

(sm.puri.Calls:27003): Gtk-CRITICAL **: 22:18:50.276: gtk_application_uninhibit: assertion 'GTK_IS_APPLICATION (application)' failed

^C

Yeah the description thing makes sense to me, just curious does nixpkgs have any specific guidelines for descriptions? I usually just copy directly from nixpkgs.

One thing i noticed is that when i close it, the process still runs until i kill it with CTRL+C.

I think thats intended, it is supposed to be also be a daemon, so it doesn't kill on application close. Its so it can receive phone calls.

Yeah the description thing makes sense to me, just curious does nixpkgs have any specific guidelines for descriptions? I usually just copy directly from nixpkgs.

One thing i noticed is that when i close it, the process still runs until i kill it with CTRL+C.

I think thats intended, it is supposed to be also be a daemon, so it doesn't kill on application close. Its so it can receive phone calls.

I usually just copy directly from upstream.

That makes sense most of the time. We have a few simple rules.
https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#submitting-changes

Its so it can receive phone calls.

i see. good

I usually just copy directly from upstream.

That makes sense most of the time. We have a few simple rules.
https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#submitting-changes

Its so it can receive phone calls.

i see. good

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 107681 run on x86_64-linux 1

3 packages built:

callaudiod
calls
feedbackd

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 107681 run on x86_64-linux 1

3 packages built:

callaudiod
calls
feedbackd

Updated feedbackd to 2021-01-25

Updated feedbackd to 2021-01-25

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 107681 run on x86_64-linux 1

3 packages built:

callaudiod
calls
feedbackd

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 107681 run on x86_64-linux 1

3 packages built:

callaudiod
calls
feedbackd

Thanks for merging!

Thanks for merging!


New GTM Twitter Event Pixel Gives Errors

Describe the bug
When using the event pixel an error in GTM preview mode occurs stating that 'Tag Twitter - Purchase failed access_globals permission check. Prohibited execute on global variable: twq.exe.apply.'
To Reproduce
Steps to reproduce the behavior:

Add event and base pixel template to GTM container
Add base pixel + event tag
Enter preview mode in GTM
Note that when the event tag fires an error occurs

Expected behavior
Tag should fire and read succeeded in preview mode

I get the same error.

I get the same error
@ahimelman,  I've had to add the permission 'twq.exe.apply' for "execute", to the Twitter Base Pixel template for it to work. If you're not using the Base Pixel, you can add the permission to the Event template
But this means that I'll no longer get updates from the official Template, as I've diverted from it...

Thank you for raising this, and apologies on the delayed response here. Pushing a fix now to both this and the base tag template


Port for STM32 with HAL library

Hi . I used ESP-Flexy stepper for my position control projects  ( Medical Pumps and & CNC axis and etc )which my MCU was ESP32 and I usually designed GUI with LVGL and of course every thing works fine and perfect .
Now I have a project with STM32 and I'm using STM32Cube IDE with Hal library + TouchGFX + FreeRTOS . so I must modify ESP-FlexyStepper library to work with HAL . firstly I edited all bellow functions :
digitalWrite() -> HAL_GPIO_WRITE_PIN();
digitalRead() -> HAL_GPIO_READ_PIN();
micros() -> __HAL_TIM_GET_COUNTER(&htim2); ---->  timer 2 configured at 1Mhz clock
delayMicrosecond() -> delay_us(uint16_t us); which is defined as bellow (Not reseting tim2 counter):
void delay_us(uint16_t us) {
	uint32_t startUs = __HAL_TIM_GET_COUNTER(&htim2);
	while (__HAL_TIM_GET_COUNTER(&htim2) - startUs < us){
		;  // wait for the counter to reach the us input in the parameter
   }
}
secondly in my main program , created a task to call processMovement() :

In terms of my driver (Lead shine CS-D1008) , minimum pulse width is 2.5 us , I added a delay_us(3) after digitalWrite(stepPin, HIGH) in processMovement() function .

In my last test , position control works fine due to accuracy but motor is running with vibration ( not such motion quality and silence motion with ESP32) and I think it's a problem with  timing calculations in this functions :
// figure out how long before the next step
DeterminePeriodOfNextStep();

@jafarniknam I am sorry, but I cannot help you with you Port, since I do not own and have no experience with STM MCUs.

@jafarniknam I am sorry, but I cannot help you with you Port, since I do not own and have no experience with STM MCUs.

You helped me before with your excellent library 🥇


Update usage of deprecated ember-mocha describeComponent()

This project uses semver, please check the scope of this pr:

[ ] #patch# - backwards-compatible bug fix
[X] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

Closes: https://github.com/ciena-frost/ember-frost-modal/issues/79
CHANGELOG

Updated integration tests to remove the deprecated use of describeComponent()
Updated service unit test to use setupTest() from ember-mocha
Updated ember-test-utils version

Visual Acceptance tests:
No changes

👍


Allow a stream to be delayed for a specified number of seconds

Describe the feature
I can envisage situations where it would be useful to be able to delay an RTSP stream:

Perhaps a live stream needs to be delayed for safety/security reasons.
Perhaps recording needs to be triggered when something happens, but the previous X seconds of footage leading up to the trigger is required for the recording.

I'd therefore like to specify a delay for a given path, if possible.
I accept that this likely isn't simply a case of feeding the RTSP stream into a fixed size FIFO buffer it is processed by the server, so I'll understand if it's too large a feature to implement any time soon.  (That said, if it's possible to estimate the required FIFO buffer size for the stream and the requested delay, I would be happy with the actual delay being only approximately what was requested!)

Hello, you can already perform this with ffmpeg:
ffmpeg -i rtsp://original-source -vf tpad=start_duration=5 -f rtsp rtsp://new-delayed-source

Thanks for the tip! :-)
I've had some issues with this approach, unfortunately.  I tried this yesterday and it seemed to work perfectly, but then when I tried it again later on the same machine, it didn't work!?
Now my ffmpeg's output is:
[rtsp @ 0x561ecbabc9c0] method SETUP failed: 461 Unsupported Transport
[AVFilterGraph @ 0x561ecbc30d60] No such filter: 'tpad'
Error reinitializing filters!
Failed to inject frame into filter network: Invalid argument
Error while processing the decoded data for stream #0:0

It definitely worked on the same machine earlier, so I don't know what's going on there.  I've tried it on my laptop (sending the stream to the the server on the other machine) and that works, although the 5s delay is actually only 2 or 3 seconds.
Also, I didn't realise that using the delay would mean I couldn't use -vcodec copy so the stream has to be re-encoded, inevitably losing quality and increasing processor usage.
So whilst I can probably use this as a workaround (if I can ever get my server's ffmpeg to remember that it knows what tpad is!), it might not be sustainable in the long-term.

You can merge this and this, adding a time.Sleep(5 * time.Second) in between and you can do the job without re-encoding.

Thanks, I'll give that a go (if you'll excuse the pun).

This feature can be achieved by using GStreamer or FFmpeg inside runOnReady, as mentioned in the README section about remuxing. For instance, in order to add a 5 seconds delay to a H264 stream:
paths:
  my_camera:
    source: rtsp://my-original-source
    runOnReady: >
      gst-launch-1.0 rtspsrc location=rtsp://<IP_ADDRESS>:8554/$MTX_PATH latency=0
      ! rtph264depay
      ! queue max-size-buffers=0 max-size-time=0 max-size-bytes=0 min-threshold-time=5000000000
      ! rtspclientsink location=rtsp://<IP_ADDRESS>:8554/$MTX_PATH_delayed

The delayed stream will be available in path /my_camera_delayed.
Native support won't be developed.


bazarr: fix unpackPhase error

Without overwriting the installation in MacOS fails with the following error
Description of changes
Fix a broken declaration on macOS by overwriting the output at the unpackPhase.
       > Running phase: unpackPhase
       > unpacking source archive /nix/store/mc6llwhmkwkxqqcajzanacl4wg559hpv-bazarr.zip
       > replace libs/apprise/plugins/NotifyMatterMost.py? [y]es, [n]o, [A]ll, [N]one, [r]ename:  NULL
       > (EOF or read error, treating as "[N]one" ...)
       > do not know how to unpack source archive /nix/store/mc6llwhmkwkxqqcajzanacl4wg559hpv-bazarr.zip

-->
Things done
Modified the unpack phase to use the unzip -o flag.

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[x] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Please change your PR to merge into NixOS:master instead of NixOS:nixos-24.05 and reword your commit and PR title to: bazarr: fix unpackPhase error

Mass ping, please resubmit.


application always get ' unfortunately app has stopped '

after android app injection and install the final app i always get ' unfortunately app has stopped '
i use parrot sec os linux 3.4
i download apk from googleplay , apkpure .... any site
and it still the problem !!

@e403 Can you try Pandora from the Google Play Store? That app is a good baseline and has proven to be very stable after injection.

i get this bro
[+] Handle the payload via resource script: msfconsole -r backdoor-apk.rc
[] Generating RAT APK file...done.
[] Decompiling RAT APK file...done.
[] Decompiling original APK file...done.
[] Merging permissions of original and payload projects...done.
[] Running proguard on RAT APK file...done.
[] Decompiling obfuscated RAT APK file...done.
[] Creating new directories in original project for RAT smali files...done.
[] Copying RAT smali files to new directories in original project...done.
[] Fixing RAT smali files...done.
[] Obfuscating const-string values in RAT smali files...done.
[] Locating smali file to hook in original project...done.
[] Adding hook in original smali file...done.
[] Adding persistence hook in original project...done.
[] Recompiling original project with backdoor...done.
[] Generating RSA key for signing...done.
[] Signing recompiled APK...done.
[] Verifying signed artifacts...done.
[] Aligning recompiled APK...done.
[!] Failed to align recompiled APK

@e403 Can you do the following for me on your system?
root@kali:~/Code/github/backdoor-apk/backdoor-apk/third-party/android-sdk-linux/build-tools/23.0.3# pwd
/root/Code/github/backdoor-apk/backdoor-apk/third-party/android-sdk-linux/build-tools/23.0.3
root@kali:~/Code/github/backdoor-apk/backdoor-apk/third-party/android-sdk-linux/build-tools/23.0.3# ./zipalign
Zip alignment utility
Copyright (C) 2009 The Android Open Source Project

Usage: zipalign [-f] [-p] [-v] [-z] <align> infile.zip outfile.zip
       zipalign -c [-v] <align> infile.zip

  <align>: alignment in bytes, e.g. '4' provides 32-bit alignment
  -c: check alignment only (does not modify file)
  -f: overwrite existing outfile.zip
  -p: page align stored shared object files
  -v: verbose output
  -z: recompress using Zopfli
root@kali:~/Code/github/backdoor-apk/backdoor-apk/third-party/android-sdk-linux/build-tools/23.0.3#

please wait ..

I can't understand the following ?
i go to the path
"backdoor-apk-master/backdoor-apk/third-party/android-sdk-linux/build-tools/23.0.3"
and these files only exist :
dx  lib  NOTICE.txt  zipalign

@e403 Once in that directory try to execute the zipalign binary. For example:
# ./zipalign

@e403 Can you share the output of running zipalign via the CLI?

Closing this issue for now. Create a new issue if needed.


Add ability to detect Facet as a vendor

Is your feature request related to a problem? Please describe.
Add ability to detect Facet as a script.
Additional context
Facet was just released, and it would be cool if Wappalyzer would have an entry for it. For instance, facet.run is using the CDN script (<script src="https://api.facet.run/js?id={ID}"></script>.

@mkotsollaris  - We have a new category called 'Feature Management'. I am assuming that's right fit for this.. This category has solutions like 'LaunchDarkly' / 'Split' etc. Can you please confirm if this is most appropriate category.
Also, do you have another example site apart Facet.run?


use maxTimestamp for logAppendTime timestamps

When consuming records that were produced to kafka with the new Record format and when the log.message.timestamp.type configuration for the broker/topic is LogAppendTime, we need to fetch the record timestamp from maxTimestamp field and not from FirstTimestamp + delta. Because Kafka send -1 as the FirstTimestamp in this case.
This Pr fix this.
Similar implementation can be found for other clients:
https://github.com/apache/kafka/blob/trunk/clients/src/main/java/org/apache/kafka/common/record/DefaultRecord.java#L325-L327
https://github.com/Shopify/sarama/blob/a6c1f7eaf9e4ed696183f5be645361ac97ab38bf/consumer.go#L526-L528

After some tests I see that there is many cases where even if firstTimestamp is not -1, the calculation of the message timestamp is wrong. I got this from my tests:
RecordBatch with two records:
max_timestamp: 2019-02-10 00:30:59 +0000

Record 1: record.create_time =  first_timestamp + record.timestamp_delta = 2019-02-09 23:27:01 +0000 + 0 = 2019-02-09 23:27:01 +0000
Record2: record.create_time =  first_timestamp + record.timestamp_delta = 2019-02-09 23:27:01 +0000 + 3837565 = 2019-03-26 09:26:26 +0000

When reading the same messages with ruby-kafka 0.5.5 or with java, the timestamp of these messages is 2019-02-10 00:30:59

@nguyenquangminh0711 Can you please take a look

Thanks ! When a new version will be released ?

I'll let @nguyenquangminh0711 decide that :)


Feature/dbt init

Adds an init task
# Usage:
dbt init pardot
# Results in:
pardot/
  dbt_project.yml
  .gitignore
  models/
  analysis/
  test/

👍

@drewbanin just need to you to:

add a section to the readme file indicating how to create a new project using dbt init as the recommended approach.
increment the version in the dockerfile.
push this release to pypi.


⚠ Bofrost.at has degraded performance

In 9f8950b, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 877 ms

Resolved: Bofrost.at performance has improved in 2527c45 after 16 minutes.


Slice interpolation

This issue is intended as a record of some of our offline & email discussions.
We've discussed three different types of approaches:

Shape based slice interpolation
Convex hulls
Contour slice interpolation

1. Shape based slice interpolation
Resources

Paper: Raya and Udupa "Shape-Based Interpolation of Multidimensional Objects" IEEE TRANSACTIONS ON MEDICAL IMAGING, VOL. 9, NO. I , MARCH 1990
This StackOverflow question (useful to look at, although it's not n-dimensional & I think there's a subtle bug in there too)

Work done

Gen put together a demo (in particular, see this notebook)
James put together https://github.com/jni/zarpaint/pull/32 based on the notebook, and is considering how the user interaction is best handled (tricky to do well)
Draga talked to Caleb Hulbert about how Paintera's shape interpolation mode handles this

Problems

You get artifacts if there is not significant overlap of pixels between the labelled slices. So that's a problem, especially if you have to label long thin things like filaments of neurons, that may not be well aligned with the image axes.

Plan
Continue figuring out the user interface. This will be necessary regardless of the interpolation algorithm used.
2. Convex hulls
Resources

scikit-image convex_hull_image function
The quickhull algorithm. Although, it turns out the scikit-image is using quickhull already (because the scipy ConvexHull uses it)

Work done

Gen tried out convex hulls in this notebook

Problems

It's super, super slow. It's not useable because of this, and I don't see a way to make it as fast as we'd need.
It's also not solving the actual problem. You'd get most of your object labelled, but you'd still need to edit every single slice at the edges, which defeats the point.

Plan
Discontinue the convex hull approach.
3. Contour slice interpolation
Resources:

ND morphological contour interpolation: https://www.insight-journal.org/browse/publication/977
Paper: https://pubmed.ncbi.nlm.nih.gov/18632365/
The napari-nd-annotator plugin (in particular, this file). Uses opencv contours, BSD-3 licence (same licence as zarpaint). Note that this plugin currently does suffer from performance problems when there are ~30 labelled objects. It's more likely that's caused by other parts of the plugin though, and not the contour slice interpolation algorithm

Plan
Try contour slice interpolation in place of the shape based euclidean distance transform method above. See if the results are nicer on a broader range of datasets (at first glance, it certainly seems like it)

@jni and @jamesyan-git - the content about the napari-nd-annotator is new, we haven't talked about that yet. I think it would be useful to look at, even if there are some problems with the performance (see here) and documentation (although I did write up some notes here)

@GenevieveBuckley The performance drop happens regardless of the interpolation part, it is rather related to UI problems. For a single object, interpolating ~20-30 slices is done in ~0.01s (with 300 contour points), so theoretically up to 100 similar-sized objects it would run under 1s (although I assume this could be further optimized).

The contour stuff in nd-annotator is so great! Makes me feel rather validated, thanks @bauerdavid! 😂

Link to https://github.com/jni/zarpaint/pull/32


Add artic 5.3.2 primers

Artic primers 5.3.2 are out and available. It's probably time to add them.

Should be resolved in https://github.com/UPHL-BioNGS/Cecret/pull/157


This Library Conflicts With fengyuanchen CropperJs Library

Hi!
i am working on the project and used your library. But after using Your library, my existing cropperjs Stopped working. Kindly fix this issue.

Thanks for opening this issue, @muhammadfaisal970! Uppload uses Cropper.js as a dependency (package.json#L84) and uses it for the Crop service (crop/index.ts) where it simply imports Cropper, creates a new object of the class, and uses the methods getCroppedCanvas() and setAspectRatio().
We don't do anything to the window scope and only use methods on our initialized object, not on the Cropper class. If there is a conflict, this might be because Cropper.js has trouble supporting multiple instances, so perhaps it's a good idea to discuss this issue there?
If, after looking at the code, you think this is a bug with Uppload, can you reproduce a demo on CodeSandbox so we could have a look at what's going wrong?

Flow:
Used uppload library to upload image-> when image uploaded i will assign that image to cropperjs for further modifications!!!. At that point my cropper functions stops working


[BitSail][Docs] correct the jump link in documents.

Signed-off-by:
Pre-Checklist
Note: Please complete ALL items in the following checklist.

[x] I have read through the CONTRIBUTING.md documentation.
[x] My code has the necessary comments and documentation (if needed).
[x] I have added relevant tests.

Purpose

Fix the jump link between documents.
Approaches

Just correct the relative path.
Related Issues

N/A
New Behavior (screenshots if needed)

N/A

@lichang-bd @BlockLiu Please merge this pull request when you have time.

Great work!!


[ZEPPELIN-493] Test fails VFSNotebookRepoTest.setUp:58 » IO No such file or directory

This PR fixes https://issues.apache.org/jira/browse/ZEPPELIN-493.
VFSNotebookRepoTest passes on CI without problem, however, it does fail when i run the test on my pc.

Looks good to me

I'm merging it into the master


Add FFLAGS to FCFLAGS

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Fixes #1.

@conda-forge-admin, please rerender


X11 bug around fullscreen when app startup/init

Bevy version
0.7.0
Relevant system information

cargo 1.62.1
rustc 1.62.1 (stable)
operating system:
Ubuntu 22.04 LTS (X11)

winit::platform_impl::platform::x11::window: Guessed window scale factor: 1

`AdapterInfo { name: "NVIDIA GeForce RTX 3080", vendor: 4318, device: 8714, device_type: DiscreteGpu, backend: Vulkan }`

What you did
works:
use bevy::prelude::*;
use bevy::window::WindowMode;

fn main() {
    App::new()
        .insert_resource(ClearColor(Color::rgb(0.4, 0.4, 0.4)))
        .insert_resource(WindowDescriptor {
            mode: WindowMode::Windowed,
            width: 0.0,
            height: 0.0,
            ..Default::default()
        })
        .add_plugins(DefaultPlugins)
        .add_system(fullscreeen)
        .run();
}

fn fullscreeen(
    input: Res<Input<KeyCode>>,
    mut windows: ResMut<Windows>,
) {
    let window = windows
        .get_primary_mut()
        .unwrap();

    if input.pressed(KeyCode::LAlt) && input.just_pressed(KeyCode::Return) {
        if window.mode() == WindowMode::Windowed {
            println!("Changing to fullscreen");
            window.set_mode(WindowMode::Fullscreen);
        } else {
            println!("Changing to windowed");
            window.set_mode(WindowMode::Windowed);
        }
    }
}

not working:
use bevy::prelude::*;
use bevy::window::WindowMode;

fn main() {
    App::new()
        .insert_resource(ClearColor(Color::rgb(0.4, 0.4, 0.4)))
        .insert_resource(WindowDescriptor {
            mode: WindowMode::Fullscreen,
            width: 0.0,
            height: 0.0,
            ..Default::default()
        })
        .add_plugins(DefaultPlugins)
        .add_system(fullscreeen)
        .run();
}

fn fullscreeen(
    input: Res<Input<KeyCode>>,
    mut windows: ResMut<Windows>,
) {
    let window = windows
        .get_primary_mut()
        .unwrap();

    if input.pressed(KeyCode::LAlt) && input.just_pressed(KeyCode::Return) {
        if window.mode() == WindowMode::Windowed {
            println!("Changing to fullscreen");
            window.set_mode(WindowMode::Fullscreen);
        } else {
            println!("Changing to windowed");
            window.set_mode(WindowMode::Windowed);
        }
    }
}

What went wrong
only changes are what mode app start in. Fullscreen or Windowed
if i run desktop in x11 and try startup app with Fullscreen (from start) it crash.
thread 'main' panicked at 'Error in Surface::configure: parent device is lost', /home/*****/.cargo/registry/src/github.com-1ecc6299db9ec823/wgpu-0.12.0/src/backend/direct.rs:214:9

if i run desktop in Wayland - it work fine.
if i run desktop in x11  and startup app with "Windowed" mode. it work fine and can easy change to Fullscreen after...
sorry if my english is bad.

I've tested this on 0.8.1 and it works as intended here.
@LLNet  Could you try against latest stable release?


Add support for nullable types in collections

NICE - did you test is locally?

It's a bit sad, but to test it locally with Invite, I need to merge it first :cry:

NICE - did you test is locally?

It's a bit sad, but to test it locally with Invite, I need to merge it first :cry:


Lack of microseconds and nanoseconds

Which version of Serilog.Sinks.Grafana.Loki are you using?
v8.1.0
Which version of .NET are you using?
net7.0
Describe the bug
.net 7.0 and later provides a .Microsecond and .Nanosecond fields for the DateTimeOffset class. Existing code ignoring such precision and send only milliseconds.
This will cause the log line strings to be in disorder.

To Reproduce
Send a several log records during a single millisecond
Expected behavior
Respect microsecond and nanosecond part of datetime object.
Log/SelfLog output or exception with stacktrace
No response
Application or code sample, which could be used to reproduce a bug
No response
Additional context
No response
I have read the documentation

[X] I have read Common Problems section in project wiki

Published as a part of v8.2.0-beta.2

@mishamyte in the beta release https://github.com/serilog-contrib/serilog-sinks-grafana-loki/releases/tag/v8.2.0-beta.2
you mistakenly linked issue 204 as "Lack of microseconds and nanoseconds in .NET 7 and later",
but its this issue #208 or PR #209

@Mario-Hofstaetter thank you for finding that out. Human mistake, fixed))


Add responsive hamburger menu button to navbar with state hook

I noticed for smaller screen sizes the menu items stacked, and thought collapsing to a hamburger style icon would be more functional as more pages are added to a project.  I was able to do this using Bulma's "is-active" class and React's State Hook to toggle it.
I'm new to GitHub and web development so hopefully I've done this correctly and this is helpful.

Not accepting any PRs until there's a new maintainer.