DJ mode / basic mixing

So the desire behind this issue is that I want to be able to talk over my stream sometimes, possibly bringing the volume down on the music while I do so, or to play silly things like this on top of the currently playing song.
How hard would it be to build this in a non-awful way? I'm interested in pitching in time or money to make it happen.

ho boy. I can see why you'd want that but it's pretty out of scope for the project. Adding a feature like this brings in a lot of design implications.

Yeah, makes sense. Maybe some day, yadda yadda.


Remove print statements in datasets

This is a second time I'm removing print statements in our datasets, so I've added a test to avoid these issues in the future.

The CI failures are unrelated to the changes.


Hide password by default from formatted printing

Applications should not render as clear text any data after the first colon (":") character found within a userinfo subcomponent unless the data after the colon is the empty string (indicating no password).
--- RFC 3986 section 3.2.1. User Information

According to this, println!("{}", uri); or such "usual" formatting should hide the password part of the userinfo component, unless explicitly specified.
To see full userinfo, uri.display_unsecure() or uri.display_secrets() would have to be provided.

This doesn't change API breakingly (i.e. downstream code compilable with the previous verisons are still compilable after this change), but this will break some kind of codes semantically.
Because of this, I treat this as a breaking change.

Is it enough if Display writes<EMAIL_ADDRESS>and Debug writes<EMAIL_ADDRESS>I think this may still have a risk that the user unintentionally prints sensitive information into logging file, error page, or something like that.

I decided not to hide the password by default.
Newly added  {BorrowedIri}::mask_password() method lets users hide or replace the password part.
The reason not to change the default is, some types requires core::fmt::Display to write the raw IRI including the password part, since strings generation by builder and proxy types are built on core::fmt::Display trait.
If some types hides the password and some others don't, it will confuse users and may become the cause of the vulnerability.
So I decided to let core::fmt::Display write the raw (uncensored) values anytime, and require users to explicitly mask IRIs.
Explicit masking provides useful features, like replacing password instead of removing.
(For example, it is possible to write<EMAIL_ADDRESS>as<EMAIL_ADDRESS>or<EMAIL_ADDRESS>


Build(deps-dev): Bump eslint-import-resolver-typescript from 3.2.1 to 3.2.4 (#3725)

Pull request opened by the merge tool on behalf of #3725

Merged in c04a7d45d9ce57bb13adf9bf74a9a01a92fd92c5


Fix JS_ReadString for wide strings on big endian targets

wide characters are stored in little endian order, so they must be swapped when read in a big endian target

Any chance of a test?

To test the serialization of strings:

use a wide character in example/hello.js and check for its presence at runtime when running the executable

More generally:

we could test the serialization on both little-endian and big-endian systems passing data to workers.

I am going to merge this patch.  Adding a test requires a better understanding of the CI tests and the cmake stuff, so I am not the best candidate for this.


Fix a couple of router related issues.

Add missing negation of compare when reconciling router gateway info and change the start of the provider network's subnet allocation pool to avoid conflicts between the router's fixed IP address and the ovnmetadata agent.

/lgtm

/test octavia-operator-build-deploy-kuttl

Superseded by https://github.com/openstack-k8s-operators/octavia-operator/pull/280


Outputs existing in local store but not in cache are not built

I want to use nix-build-uncached on persistent builders, so that builds are even faster, because dependencies are already cached.
I have this script to build new outputs and upload them
      nix-build-uncached ./ci.nix -build-flags '-o results/x'
      if [ -e results/x ]; then
        nix -v sign-paths -r --key-file $NIX_CACHE_KEY ./results/x*
        nix copy --to $BINARY_CACHE_URL ./results/x*
      fi

Now if I mess up the signing & uploading part, I end up in a situation, where the outputs are present on the builder but not in the cache and nix-build-uncached will not create any outputs.
Unfortunately nix-build --dry-run does not provide any information about those paths.

I have this problem too, but not yet a solution.


Modules: support relative path

Background
The import.file block imports custom components from a file or a directory and exposes them to the importer. Users can specify the path to watch via the argument filename.
Problem
The path is relative to the current working directory. This works fine when importing a module from the root config but it’s not ideal when the module itself has import.file blocks. Resolving the path relative to the module would be better in this case.
Request
The following use-cases should be supported:

import.file -> import.file
import.git -> import.file
import.string -> import.file

And if possible import.http -> import.file, but this one might be more tricky because it would only be possible if the nested import.file blocks would be turned into import.http blocks.

Should this become a proposal as it's a user-facing API?

There was already an accepted proposal for it: https://docs.google.com/document/d/16vRoiIxQVVlqqEGngm27Kcun6hF_BTpY2tPOSt05JWU/edit#heading=h.y9td4d74r21c


Fix searchPlaceholder option

Hello,
Thank you, your plugin is awesome.
Emoji are saved in unicode in my database, so i need to display only emoji which belong  to a specific unicode version in the window.
Maximum version of unicode available need to be 4.1
https://emojipedia.org/unicode-4.1/
Currently your plugin will display "mouse emoji" but browsers will display alt="🐁" if you inspect it.
https://emojipedia.org/mouse/
I saw the function "getSupportMode" in your code but i don't know how to use it.
Another issue :
I try to change 'searchPlaceholder' but it seems to have no effet. Indeed in emojionearea.js you have the word "SEARCH" used directly (L905) instead of the variable.
Thank you in advance!

I'm trying to be more clear. I don't want to display emoji which are display by a square by the browser.
some exemple here : https://emojipedia.org/unicode-6.0/
I can use the option "filters" but it's a huge work to do by hand.
Any idea ?

Why not use EmojiOne to add support for older browsers
It seems strange you want to use EmojiOneArea for editing, but then display them as native platform emoji

I want to use this library in my backoffice as an emoji selector helper.
On frontoffice, the library will not be used and thus the emoji should be natively displayed by the browser.

Currently your plugin will display "mouse emoji" but browsers will display alt=":mouse2:" if you inspect it.

I don't get your problem, It works as expected

Maximum version of unicode available need to be 4.1

It uses emojione, so probably it's their issue,

in emojionearea.js you have the word "SEARCH" used directly (L905) instead of the variable.

Thanks for your notice, this will be fixed

I'm trying to be more clear. I don't want to display emoji which are display by a square by the browser.

Please make a screenshot of your squares, i see icons, not  squares

You can have this problem when you are not use utf8 encoding of you page
<meta http-equiv="content-type" content="text/html; charset=UTF-8">

I can use the option "filters" but it's a huge work to do by hand.
Unfortunately in your case, I think it is the better way...
Or you could use emojione in your frontoffice too...

Thank you for your rapidity and answers.
Indeed your library works perfectly, it display mouse correctly. I figured out it's a browser problem.
Firefox will displayed mouse unicode but chrome will display a square for exemple.
Below screen of chrome vs firefox.
Chrome (v62):

Firefox (v52.1):

You can see in "alt" tag that firefox understand mouse unicode but not chrome.
With your library my users will be authorized to save "mouse" emojti and it's will be transfer and used in front. For the moment i did not use emoji in front so it will display unicode logo.
My customers using chrome (same version as us) will see a square and the others using firefox (same version as us) will see a mouse (same case for opera, safari, ...).
I know we can't really do nothing about it.
Do you think there is a simple way to display in emotjioneare only emotji which availble and supported by chrome and firefox last version.
So the "mouse" will not be available in emojionearea.
Maybe the only solution it's to filters available emotji using a hard coded list.

All emojis are valid utf8 symbols and all major browser supports this.
You just do it wrong.
Chrome Version 60.0.3112.90 (Official Build) (64-bit):

Make sure you have valid charset of your html page

We are not speaking about the same mouse emotji. Indeed "mouse-face" works correctly on chrome but not "mouse"
mouse-face : https://emojipedia.org/mouse-face/
mouse : https://emojipedia.org/mouse/
It's not depending on my website, if i take an official page : https://emojipedia.org/unicode-6.0/
In my Chrome Version 62.0.3202.94 (Build officiel) (64 bits) it display square instead of unicode correct logo.
I want to exclude all of that emotji. It is possible to give a range of available unicode used in of library ?
Thank you again.

do you see 3 lines or only 1 ?

3, maybe i need to install a specific font in my browser ?

And I see it perfect

It because default font what uses chrome have not full symbols on your system

Oh ok :) maybe it's because in work i'm in windows 8 with specific enterprise parameters.
I will try from another computer.
Sorry for the misunderstanding
I keep you in touch !

It works on my mobile (android/chrome). I will try to figure out the problem with my chrome entrerprise specific version.
Thank you mervick for your help and explication.
I will follow your commit, waiting for searchPlaceholder  correction.

It is not chrome, it's your windows

I let open the issue for the searchPlaceholder bug.


Code enhancements of login.go

Can you run test related to this file and post asciinema recording?

Sure ; I will do it . Thank you for the opportunity once again .
On Tue, 19 Sept 2023 at 21:46, Samy Fodil @.***> wrote:

Can you run test related to this file and post asciinema recording?
—
Reply to this email directly, view it on GitHub
https://github.com/Hameeda6/tau-cli/pull/1#issuecomment-1726960686, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AVRRCHPFUHY6Y7DLDF5RXLLX3JYI5ANCNFSM6AAAAAA47I2VMM
.
You are receiving this because you authored the thread.Message ID:
@.***>


demo_mug.py RuntimeError: CUDA out of memory. Tried to allocate 120.00 MiB V100 32G

Hello, I have a memory leak in the test, in the diffusion 1000 iterations when the memory rose from 14G to 32G so that can not be tested, please help to see what is the reason? python3.7.10 pytorch1.12.1+cu10.2 is installed in requirements.txt for all environments.

Hi, @Zenobia7, sorry for the late reply! But I do not have such an issue before and normally the memory of the program will not be more than 24G.


Document staging purpose

http://comments.gmane.org/gmane.linux.distributions.nixos/13447

[ ] document how merging is done

https://nixos.org/nixpkgs/manual/#idm140737319144352

That seems the proper place to me.

Reopen if you believe it needs improving.

http://comments.gmane.org/gmane.linux.distributions.nixos/13447

Sadly that link isn't available anymore. Probably due to the following:
http://gmane.org/about/:

At the end of July 2016, Gmane briefly closed and was acquired by Yomura in mid August.
Gmane is currenly being rebuilt in Python and PHP and uses ElasticSearch for its underlying document store. At the time of writing, we have just over 112M documents indexed.

Do we have a "backup" of that information?
And sorry for commenting on this old & closed issue...

It's in the nixpkgs manual.  I thought it would be understood from the discussion, but let me say it more explicitly.  There's no "named" (i.e. stable) link directly to that section, so that's why the exact old link doesn't work anymore.

I might be wrong here but I think you understood that the link to the section in the nixpkgs manual is broken: https://nixos.org/nixpkgs/manual/#idm140737319144352
But I actually meant the link we referred to inside the manual: http://comments.gmane.org/gmane.linux.distributions.nixos/13447
This is the relevant excerpt:

It's only for non-breaking mass-rebuild commits. That means it's not to be used for testing, and changes must have been well tested already. Read policy here.

@primeos: I see now, thanks!  That should get fixed.

The broken link hasn't been fixed yet. https://github.com/NixOS/nixpkgs/blob/77161de4546697f9bf2da6d081eeba4c399b3313/doc/submitting-changes.xml#L479


chore(e2e): Use new test namespace in Knative E2E tests

Enables better error analysis due to automatic dump of resources on test failure

Release Note
NONE

Related to #5400 to have a better error analysis when the E2E test fails again


"Reading time" articlelist-format identifier

I propose a "reading time" identifier for the articlelist-format.
This would indicate how long it takes to read the article.
We already have "article length", but that is not very informative for me.
Since we already have the article length it would be easy to calculate the reading time from that.
There are many feed readers that offer this functionality.

Sounds useful, thank you for the suggestion!
I expect we'll need to know the article's language to figure out how long it takes to read it? Atom and RSS 2.0 both have tags for that, but: 1) in RSS the tag is per-channel, i.e. all articles are assumed to use the same language; 2) in both specs, the tags are optional, so we can't rely on their existence.
Are there reliable ways to auto-detect the language?
I think we'll have to add a new sorting order and a new article attribute, too, so people can use this to sort their articlelist, and to filter it to their preference.

We could use a natural language detector for figuring out the language. Miniflux is doing this. The are using getlang, an offline natural language detector written in go.
But maybe we can just make it a user definable variable "reading speed in WPM" which defaults to 200.
That way a user can set it according to the language he reads in, the kind of text he normally reads (scientific, etc) and his personal reading speed.
The filter and sorting options for reading speed are a really good idea.

I just realized that "WPM" would not work for languages like chinese.

The are using getlang, an offline natural language detector written in go.

It appears that whatlang crate implements the same algorithm, though I didn't compare them closely. There's also lingua, which uses neural networks and claims to be more accurate.

But maybe we can just make it a user definable variable "reading speed in WPM" which defaults to 200.

I'm in two minds about this. I read in three languages, and I'm pretty sure my reading speed is different in each (although I didn't measure). I wouldn't want to add a couple thousand new settings, one for each language :)
OTOH, does the exact number even matter? I wouldn't expect Newsboat to tell me precisely how long it'll take, down to a second. I'll be mostly looking at the order of it, like "1 minute", "5 minutes", "30 minutes", "100500 minutes". What about you, @svemoe?
(Miniflux uses 500 runes per minute for CJK and 265 words per minute for everything else: https://github.com/miniflux/v2/blob/6e2e2d166574f35276dead58f345dedb234ac7ae/reader/processor/processor.go#L236-L249)

I'm basically of the same opinion as you on this. We don't need the exact number, the "ballpark" is good enough.
So we could implement it in the same way that miniflux does.

There's also lingua, which uses neural networks and claims to be more accurate.

I think accuracy is not very important, since we would only need it to differentiate between rune-based and word-based languages.
That should not be very difficult.

The answer is probably no, but is there any way we could avoid implementing this in Newsboat itself by e.g. using a filter? Not sure how we would forward this calculated information into Newsboat to make it usable. I'm just trying to evaluate alternatives, so we don't end up with feature creep and tons of new settings to tweak the algorithm.
I also want to avoid that Newsboat wastes computation time on this if users like me are not interested in a read time estimation.

we would only need it to differentiate between rune-based and word-based languages.

Oh, right! So we don't even need a new crate. We can simply count the runes, count the words (space-separated runs of non-space characters), and whichever number is bigger indicates the "type" of the language.

The answer is probably no, but is there any way we could avoid implementing this in Newsboat itself by e.g. using a filter?

Yes: a filter could perform the same calculation and put the estimated time e.g. into the title. If the time is at the beginning of the string, the sorting will work quite well (Newsboat 2.13 and up uses natural sorting, so "1, 2, 3, 10, 20", not "1, 10, 2, 20, 3"). To make this work with the filter language, the filter: could e.g. replace the author with the estimated reading time.

if users like me are not interested in a read time estimation

As a user, I'm not interested either. I read either in the web browser or in the Pocket mobile app, so I'm not looking at the article's size in Newsboat.
However, with my maintainer hat on (and thus a wider perspective), I can't help but notice that "estimated reading time" is way better than our current "article length":

the article length is just a number of characters, including HTML. HTML makes this a pretty noisy metric, because the same amount of text can be marked-up very differently, which affects the characters count but not the reading time;
the article length requires effort to be useful. If the article is 1.1 kilobytes in size, what does this tell the user? I had to go to lipsum.com and generate this many bytes to see that it's just a couple paragraphs — which my mind immediately translated into "a minute or so", "a quick read". Perhaps there are a few nerds who know their reading speed in bytes per second, so they can translate 1.1K into "45 seconds", but I'm not one of them :)

Reading time estimation doesn't suffer from these problems, and doesn't seem to bring any new ones. Am I missing something?
It's indeed feature creep, but I don't think it'll require any new settings, and the computation cost is on the order of running through the string maintaining a dozen bytes of state, and getting an integer as a result. That's non-zero, but is so low that I'd call it negligible. (But that's a guess based on what's being talked about.)

We actually have an issue for the sorting order, albeit by length not reading time: https://github.com/newsboat/newsboat/issues/663


ci: update static check tools

This PR updates the static check tools to
the latest version.
Further needs to resolve all the errors after
updating the version.
Updates: #1586
Signed-off-by: Yati Padia<EMAIL_ADDRESS>

Need to add DNM tag here.

@yati1998 dnm is added

@yati1998 dnm is added

Thanks @humblec  :)

/retest all

/retest golangci-lint / golangci-lint

Getting closer!
@mergifyio rebase

@nixpanic @Madhu-1 please review this PR.  Also, need to remove the DNM tag.

/retest ci/centos/mini-e2e-helm/k8s-1.19

/retest ci/centos/mini-e2e/k8s-1.19

/retest ci/centos/mini-e2e/k8s-1.19

You can consider addressing the typos in the commit messages
ci: disbale forbidigo linter
ci: disabe gomoddirectives linter

oops, I don't want to do that, otherwise the approvals will be stale and it will be a problem for others to review again:(.  Is it important to do that? can't we skip this for this time?

/retest ci/centos/upgrade-tests-cephfs

/retest ci/centos/upgrade-tests-cephfs


GEODE-7864: Add missing curly braces to if statements

StripedExecutorService contained two instances of if statements
withoug curly braces

Authored-by: Donal Evans<EMAIL_ADDRESS>Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

The DUnit failure is a known flaky test tracked by GEODE-7118.


No std crypto

No std crypto
Description
Description of the pull request changes and motivation.
Type of change
Please delete options that are not relevant.

[x] Optimization

Checklist

[ ] Linked to Github Issue
[ ] Unit tests added
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

[ ] This change is an Optimization

[ ] Benchmarks added/run

Codecov Report
Attention: 61 lines in your changes are missing coverage. Please review.

Comparison is base (f96448c) 95.63% compared to head (f127b45) 95.42%.

Files
Patch %
Lines

math/src/fft/errors.rs
0.00%
18 Missing :warning:

math/src/polynomial.rs
0.00%
7 Missing :warning:

crypto/src/merkle_tree/merkle.rs
25.00%
6 Missing :warning:

math/src/msm/naive.rs
0.00%
6 Missing :warning:

math/src/field/extensions/cubic.rs
0.00%
5 Missing :warning:

...rt_weierstrass/curves/bls12_381/field_extension.rs
20.00%
4 Missing :warning:

math/src/field/fields/mersenne31/field.rs
0.00%
3 Missing :warning:

math/src/field/extensions/quadratic.rs
33.33%
2 Missing :warning:

...th/src/field/fields/p448_goldilocks_prime_field.rs
0.00%
2 Missing :warning:

math/src/field/fields/u64_goldilocks_field.rs
0.00%
2 Missing :warning:

... and 4 more

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #736      +/-   ##
==========================================
- Coverage   95.63%   95.42%   -0.21%
==========================================
  Files         136      136
  Lines       30882    30924      +42
==========================================
- Hits        29533    29510      -23
- Misses       1349     1414      +65

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Alloc isn't being compiled with clippy, and targets of the CI are not updated, the implementation could be broken and we wouldn't notice it.

I added them to the CI


codegen: Allow returning nothing

Currently, function is supposed to return something. If it returns nothing, it should return ().
This spec is awkward, let's fix.

done.


refactor(mempool): add docstring for queued transaction

This change is

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 83.37%. Comparing base (6e74e3f) to head (0d108d6).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #416   +/-   ##
=======================================
  Coverage   83.37%   83.37%
=======================================
  Files          37       37
  Lines        1720     1720
  Branches     1720     1720
=======================================
  Hits         1434     1434
  Misses        210      210
  Partials       76       76

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


fix: nvd3 line chart y axis bounds

🐛 Bug Fix

Previously the bound will not be applied if the extent of data is larger than the specified bound.
Address Airbnb's PRODUCT-70657

🏠 Internal

Add examples in storybook to demonstrate different settings for yAxisBounds
Set default time format for storybook.

@michellethomas @williaster @conglei @graceguo-supercat

That's a great idea. Haven't used knobs before. Will try that for the new components.


Fails to decode data after long time running

I have just updated both defmt and probe-run, and i am now starting to see a lot of
(HOST) ERROR failed to decode defmt data: [af, 6e, 39, 0, 0, ...]
Error: malformed data

They are not coming right away, but rather after running for a long time. I am not logging any significant amount of data, and it doesn't seem like it's a particular log frame that causes the error.
This is kind of a big issue, as above error also seems to halt the core, causing not only our logging, but also our application to stop running.
Would it be possible in cases of malformed data, to simply rotate a single byte and try decoding again, in a loop? It seems to work as a last resort re-synchronization mechanism, that is MUCH more desireable than a panic. That way only the malformed frame is lost.
The obvious alternative would be to finally add framing to the wire format, which i think would also solve a bunch of other issues/requests?

Are you using defmt-rtt? Does this happen with any executable or are specific messages etc. needed to reproduce this?

Yea, using defmt-rtt.
I can't seem to find the trigger.. It seems like it's random and can happen right after flashing, or 24 hours into the program, so it doesn't seem like its any particular place in the execution.
Is there anything i can enable to obtain more info?

Hi @MathiasKoch 👋🏾
I ran a small sample program (see below) for a couple of hours, but couldn't find the error you described.
Can you please elaborate a bit on your defmt usage and rough setup? What kind of data structures are you logging which produce this error?
Also can you please try to run probe-run with --verbose enabled. Maybe we can find something odd there.
Thank you!

sample code
#![no_main]
#![no_std]

use cortex_m::asm;
use defmt::Format; // <- derive attribute
use defmt_app as _; // global logger + panicking-behavior + memory layout

#[derive(Format)]
struct S1 {
    x: u8,
}

#[cortex_m_rt::entry]
fn main() -> ! {
    let s = S1 { x: 42 };
    let x = 43;

    loop {
        defmt::info!("s={:?}", s);
        defmt::info!("x={=u8}", x);
        asm::delay(25_000_000)
    }
}

Just skimming other issues I assume this is related to https://github.com/knurling-rs/defmt/issues/322

@MathiasKoch Same experience her, including the apparent "randomness". Ie sometimes right after flashing, sometimes after a few seconds or minds, sometimes after hours.

I think this has been solved by rzcobs encoding in v0.3.0?

That's good to hear @MathiasKoch!
Do you see data getting lost and defmt recovers the next frame, or is no data getting lost at all? I am trying to understand if the problem of "randomly malformed data after a long time" is solved or if it is just mitigated due to the recovery.
PS: Happy new year! 🎉

I am not sure yet, as we have not had time to upgrade our devices to v0.3.0 yet, so you might be right that this is still not fixed. Not sure?
Happy new year! :tada:

I'd be happy if you can report back in either case. If the root cause is fixed as well if it is not :)


Achieve 100% test coverage

Codecov Report

Merging #13 into master will increase coverage by 6.77%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master    #13      +/-   ##
========================================
+ Coverage   93.22%   100%   +6.77%
========================================
  Files           5      6       +1
  Lines         118    261     +143
========================================
+ Hits          110    261     +151
+ Misses          8      0       -8

Impacted Files
Coverage Δ

tests/utils/config.py
100% <100%> (ø)

tests/test_integration.py
100% <100%> (ø)
:arrow_up:

ddtrace_asgi/middleware.py
100% <100%> (+13.33%)
:arrow_up:

Continue to review full report at Codecov.

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


webview-about update

What does this PR do?
-About Page WebView for Mobile
Description of Task to be completed?

How should this be manually tested?
-Load the about-mobile  on the browser [ /about-mobile]
What are the relevant Trello Card stories?  Trello Card Item
-https://trello.com/c/xUnIui0h/104-about-page-webview-for-mobile
Pull Request Type (check one)

[x] Feature
[ ] Bug Fixes
[ ] Chore

Developer Checklist

[x] New code has been scanned for ESlint issues
[x] I have tested my code locally
[x] I made sure that there are no other pull requests that solve for this issue
[x] My code follows the same structure within the guidelines

Screenshots (if appropriate)

When loading the /about-mobile,  I have a 404 page at the top and the about page is below.
Any suggestions on how to fix this?

@Utieee so what I did was place a conditional in the initial routes file that looks at the URL. If it is about-mobile render that component as is, if not render the other router.
Let me know if you have any questions regarding the implementation.

@Utieee so what I did was place a conditional in the initial routes file that looks at the URL. If it is about-mobile render that component as is, if not render the other router.
Let me know if you have any questions regarding the implementation.

Good idea, Thanks!


[fix][test]Fix flaky test because the too short receive time

Reopen https://github.com/apache/pulsar/pull/20991
Fix https://github.com/apache/pulsar/actions/runs/5854803000/job/15895075150?pr=20984
Motivation
Because the performance of the machines in Git is poor, the message may not be received in 1 second or 300 milliseconds.
We need to adjust the time limit of receiving messages to avoid flaky test.
 Error:  org.apache.pulsar.client.impl.TransactionEndToEndTest.txnAckTestNoBatchAndCumulativeSub  Time elapsed: 3.92 s  <<< FAILURE!
  java.lang.AssertionError: expected object to not be null
  	at org.testng.Assert.fail(Assert.java:110)
  	at org.testng.Assert.assertNotNull(Assert.java:1483)
  	at org.testng.Assert.assertNotNull(Assert.java:1467)
  	at org.apache.pulsar.client.impl.TransactionEndToEndTest.txnCumulativeAckTest(TransactionEndToEndTest.java:748)
  	at org.apache.pulsar.client.impl.TransactionEndToEndTest.txnAckTestNoBatchAndCumulativeSub(TransactionEndToEndTest.java:712)
  	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
  	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
  	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
  	at java.base/java.lang.reflect.Method.invoke(Method.java:568)
  	at org.testng.internal.invokers.MethodInvocationHelper.invokeMethod(MethodInvocationHelper.java:139)
  	at org.testng.internal.invokers.InvokeMethodRunnable.runOne(InvokeMethodRunnable.java:47)
  	at org.testng.internal.invokers.InvokeMethodRunnable.call(InvokeMethodRunnable.java:76)
  	at org.testng.internal.invokers.InvokeMethodRunnable.call(InvokeMethodRunnable.java:11)
  	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
  	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
  	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
  	at java.base/java.lang.Thread.run(Thread.java:833)

Modifications
Unique the limited time for receiving messages. 5 seconds for messages that can be received and 1 second for messages that can not be received.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

Codecov Report

Merging #21273 (7ca0042) into master (643428b) will increase coverage by 36.21%.
Report is 1 commits behind head on master.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master   #21273       +/-   ##
=============================================
+ Coverage     36.96%   73.17%   +36.21%
- Complexity    12294    32392    +20098
=============================================
  Files          1698     1887      +189
  Lines        130510   140197     +9687
  Branches      14260    15436     +1176
=============================================
+ Hits          48240   102591    +54351
+ Misses        75982    29513    -46469
- Partials       6288     8093     +1805

Flag
Coverage Δ

inttests
24.17% <ø> (-0.15%)
:arrow_down:

systests
24.69% <ø> (-0.21%)
:arrow_down:

unittests
72.48% <ø> (+40.35%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1452 files with indirect coverage changes


export my own solution?

Is your feature request related to a problem? Please describe.
The problem is data security. I am worried that if the codewars database service fails, all my solutions will be lost. I spent a lot of time on codewars and didn't want to waste my time.  So I want to export my own solution, It's kind of a backup.
Describe the solution you'd like
If there is an "export button" on the personal profile, it will be good.
Describe alternatives you've considered
I found some tools, they seem like web crawlers, request the page, parse the html, etc...

https://github.com/brunolm/codewars-exporter
https://github.com/lucasbflopes/codewars-kata-exporter

Additional context
No

(as long as you don't publish them everywhere public... Otherwise, this back up tool is just a bad idea. We already have enough troubbles like that with copycats without making the task even easier...)

@Blind4Basics yeah, it makes sense. The solutions are published everywhere public is another data security...


Drop unsupported non-string attribute values

Drop attributes that have non-string dimension values, as they are currently unsupported.

Ah, let's not forget to add the entry in the README about the exporter only supporting string dimension keys/values.


Fixes thank you page on cancel

Pass cancel_return_url to citrus
@girasquid @ivanfer

Do we want to preserve the fact that this transaction is cancelled, not necessarily failed? Or is treating a cancel as a fail fine?

@girasquid ideally we'd be sending a cancel URL in the initial request and they'd be forwarding to that. We can reach out to Citrus at some point and see if this is a change they can make. In the meantime, this is a much better experience for customers. Currently they just get taken to a "thank you" page after they click cancel.

Alternatively, we can leave it as "Cancelled" and just add that as a string to the offsite processing job.

This sounds like an improvement over what we have currently - :+1:. Having Citrus support cancel URLs down the road would be awesome.

:+1:

Citrus can't forward to cancel url .. reverted PR back to what it was before.

C'est bon?

:+1:


Tweak the RpcDeadline test to reduce flakes

This is to mitigate: https://github.com/grpc/grpc/issues/12917
See the comments in the code for details

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences


CTRE CANdles

Merge support for CTRE CANdles into master, and rename the talon_interface package to ctre_interfaces. All code built on Ben's laptop, and the CANdles worked on the robot-in-a-box.

I've reverted the deploy.sh script, but I also added a deploy_robot_in_a_box.sh script that will deploy to the robot in a box (it doesn't have a second jetson, so normal deploys don't work).

I've reverted the deploy.sh script, but I also added a deploy_robot_in_a_box.sh script that will deploy to the robot in a box (it doesn't have a second jetson, so normal deploys don't work).

Could you make it a command line option for deploy.sh? deploy gets updates more frequently than it should, would rather not have two copies floating around
You could make an alias to deploy with that flag in https://github.com/FRC900/2022RobotCode/blob/master/zebROS_ws/command_aliases.sh to create a deploy_robot_in_a_box command

I've reverted the deploy.sh script, but I also added a deploy_robot_in_a_box.sh script that will deploy to the robot in a box (it doesn't have a second jetson, so normal deploys don't work).

Could you make it a command line option for deploy.sh? deploy gets updates more frequently than it should, would rather not have two copies floating around You could make an alias to deploy with that flag in https://github.com/FRC900/2022RobotCode/blob/master/zebROS_ws/command_aliases.sh to create a deploy_robot_in_a_box command

I modified deploy.sh directly - adding the "box" argument will deploy to the robot in a box instead of the actual robot. Doing some testing now and will push afterwards.
# This can be an array of IP address if there are multiple Jetsons
#JETSON_ADDR=( )
# If we're deploying to the robot in a box, only deploy to 1 jetson (it doesn't have 2)
if [ $1 == "box" ]; then
    JETSON_ADDR=(<IP_ADDRESS>)
else
    # On the actual robot, deploy to both jetsons
    JETSON_ADDR=(<IP_ADDRESS> <IP_ADDRESS>)
fi

Moving over to 2023 repo, closing


FIX simulate_transaction add parameters from new RPC API

Incorporate new parameters from RPC API (https://docs.solana.com/api/http#simulatetransaction): replace_recent_blockhash, min_context_slot, and accounts.

@pyz4 would you mind taking a look at the broken integration test?

The check issues are from a TypeError in test_send_transaction_and_get_balance:
# transfer_tx needs to be a VersionedTransaction.
transfer_tx = Transaction().add(
     sp.transfer(sp.TransferParams(from_pubkey=stubbed_sender.pubkey(), to_pubkey=stubbed_receiver, lamports=1000))
)
sim_resp = test_http_client.simulate_transaction(transfer_tx)

the test_send_raw_transaction_and_get_balance test in integration/test_http_client.py. The returned balance is<PHONE_NUMBER>, but the assertion is against<PHONE_NUMBER>. May be that your change is using less gas?
Same thing in test_send_raw_transaction_and_get_balance_using_latest_blockheight. Balance is slightly higher than it should be. Would love to get @michaelhly 's opinion on if gas is likely the culprit.

The check issues are from a TypeError in test_send_transaction_and_get_balance:
# transfer_tx needs to be a VersionedTransaction.
transfer_tx = Transaction().add(
     sp.transfer(sp.TransferParams(from_pubkey=stubbed_sender.pubkey(), to_pubkey=stubbed_receiver, lamports=1000))
)
sim_resp = test_http_client.simulate_transaction(transfer_tx)

the test_send_raw_transaction_and_get_balance test in integration/test_http_client.py. The returned balance is<PHONE_NUMBER>, but the assertion is against<PHONE_NUMBER>. May be that your change is using less gas?
Same thing in test_send_raw_transaction_and_get_balance_using_latest_blockheight. Balance is slightly higher than it should be. Would love to get @michaelhly 's opinion on if gas is likely the culprit.

Yeah. We can change the hard coded balance amounts but I think we still need to fix the TypeError:
FAILED tests/integration/test_http_client.py::test_send_transaction_and_get_balance - TypeError: argument 'tx': 'Transaction' object cannot be converted to 'VersionedTransaction'


Use Object.is when testing for equality where the === operator was previously used

This affects the 'to be' and 'to equal' assertions and means that NaN is considered equal to itself, whereas -0 and 0 are considered different.
Consider this a spike at this point. I realize that the additional text required in the docs to explain the departure from === is a drawback in itself.
Closes #116.

Coverage decreased (-0.27%) to 93.71% when pulling 672efe97eac315664d9c2a86a13dbbb49f155420 on objectIs into ef9fea707b361d7802fd162a4c71a7b7f9b11bd5 on v6.

It looks good. I wunder what the performance impact will be. I also think we should use the native Object.is if it is available. I don't think it is a problem that we have to mention that we use Object.is sematic in the doc.

I wunder what the performance impact will be

From running the test suite multiple times before and after the change I couldn't measure any difference.

I also think we should use the native Object.is if it is available

That is how I implemented it, see https://github.com/sunesimonsen/unexpected/blob/objectIs/lib/utils.js#L11
The drop in coverage is due to the polyfill only getting exercised by make test-phantomjs btw.

Sorry I overlooked that fact that you have it chose the native version if it was available.
It is nice that the performance is the same. I think this is a good change.

LGTM :+1:


I can't restore backup from tachiyomi.

# Device information

Tachiyomi version: 0.8.72az
Android version: 5.1.1
Device: Samsung galaxy tab 4

Steps to reproduce

Go to settings 2. Click backup 3. Click restore backup 4. Only showing Dropbox and sound picker.                                         ## Issue/Request
I am supposed to be able to pick any file from file Manger

Other details
Additional details and attachments.

Please it is not completed

ah you edited it to actually add the issue, great

Thank you

Fixed

how

I downloaded a file manager that allowed root access.
So the file manager showed up when I wanted to restore.
It took me to the root directory first.
Then I navigate to storage/emulated/0/tachiyomi
Which is where I saves the backup
I was using my default phone file manager before
Thank you very much for creating this app
I couldn't update my tachiyomi anymore on my phone as the device is an old device.
And I only keep the android because of tachiyomi


Android hardware back button not working after Actions.main({type: 'replace'})

Version
Tell us which versions you are using:

react-native-router-flux v3.37.0
react-native v0.41.2

I use Actions.main({type: 'replace'}) to logout user, but after that hardware back button not working on any scene.

You didn't provide any test case or at least investigated what's causing this error. You should use StackOverflow for this type of question.


[Module Proposal]: Azure BuildIn policy assigment

Check for previous/existing GitHub issues/module proposals

[x] I have checked for previous/existing GitHub issues/module proposals.

Check this module doesn't already exist in the module indexes

[x] I have checked for that this module doesn't already exist in the module indexes.

Bicep or Terraform?
Bicep
Module Classification?
Resource Module
Module Name
avm/res/builtin-policy
Module Details
Module will allow assigne builtin polic and initiative to Managment Group / Subscriptions.
Do you want to be the owner of this module?
Yes
Module Owner's GitHub Username (handle)
No response
(Optional) Secondary Module Owner's GitHub Username (handle)
No response

Hi @mimachniak, thanks for proposing this module!
There's a chance that we already have the module you're looking for: https://github.com/Azure/bicep-registry-modules/tree/main/avm/ptn/authorization/policy-assignment
Can you please take a look and either confirm this is what you'd need or explain how your requirements are different? Thank you!


The requested content cannot be played by the player on iOS Simulator.

When I play video on iOS simulator it says "The requested content cannot be played by the player". But, in android everything is ok.

Please try wirh v7.0.0


clang -fopenmp depends on openmp

Issue:
clang has an option '-fopenmp' which claims that since version 3.8 clang has supported OpenMP. Which is true as far as it goes, but programs written for OpenMP will not compile using -fopenmp unless the openmp package is also installed. Maybe this is obvious to everyone in the world but me. But it is not the behavior of the old gcc package. So either a) openmp should be a dependency of clang, or at the very least b) the documentation for -fopenmp should state that the openmp package is required for that option to work (though, apparently, not on all platforms and versions of clang?)
Thank you for your consideration.

Environment (conda list):

Details about  conda and system ( conda info ):

Please add the details required in the issue template

Details added.
But here's an easy way to reproduce the error on MacOS 10.13:
# conda create --name clang c-compiler
# conda activate clang
# vi omp.c

Add the following into omp.c:
#include <omp.h>

int main() {}

Save and then do:
# clang omp.c -fopenmp

I get:
omp.c:1:10: fatal error: 'omp.h' file not found
#include <omp.h>
         ^~~~~~~
1 error generated.

Please open an issue in https://github.com/conda-forge/compilers-feedstock


The link returns a 404 page

Hacktoberfest notes:

due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
please read our contributing guidelines
be sure to check the output of Travis-CI for linter errors
if this is your first open source contribution, make sure it's not your last!

What does this PR do?
Add Resource(s) | Remove Resource(s) | Add info | Improve Repo
I had to remove a link to a symfony page that's returning a 404 error
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[X] Not a duplicate
[X] Included author(s) if appropriate
[X] Lists are in alphabetical order
[X] Needed indications added (PDF, access notes, under construction)

remove the category,, too

Sure!
Btw, I'm making a node script that checks all the links in the md files. It still makes some false positives, but it should be ready in a day or two :)


Add atom/rss feed for news posts

Danke @wm75

Found out where it lives by now: https://training.galaxyproject.org/training-material/feed
A nice feed icon on the page would be cool though.

Oh. I couldn't find that, cool. Yeah we can link/icon that and include it in the header metadata so it's easier to add to thinks/autodetect.


An attempt to make current date work with GitHub Actions

The workflow is not valid. .github/workflows/update-contributors.yml (Line: 22, Col: 27): Unexpected symbol: '$'. Located at position 1 within expression: $(date +'%Y-%m-%d') .github/workflows/update-contributors.yml (Line: 24, Col: 18): Unexpected symbol: '$'. Located at position 1 within expression: $(date +'%Y-%m-%d')
https://github.com/kaigionrails/conference-app/actions/runs/6782234624

hmm


kvclient(ticdc): region worker use local context instead of the global server ctx to control worker pool event handler

… global server ctx

What problem does this PR solve?

Issue Number: close #11638
What is changed and how it works?

use region worker local context to control the event handler

processEvent method is used by the worker pool event handler, which is controlled by the server level context which is passed in when the worker pool start.
Since region worker is a changefeed level instance, we should use local context to control it, so the event handler can return in time when the changefeed stopped.
Check List
Tests

Unit test
Integration test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
`None`

/retest

/retest

/retest


Update ui.css

This resets the box model on the root and body elements to not move around if they have pre-existing classes during initialization.

Thanks! Actually, the new Re-view 2 repo is here: https://github.com/livestyle/re-view
Sorry, my bad. Could you please create a new PR in this repo?

Looks as though you already have most of this in your latest stylesheet, when I can test the latest version I'll figure out if an other PR is necessary. Any reason you still have this repo?


URL 404s

Under the https://learn.microsoft.com/en-us/azure/azure-arc/servers/troubleshoot-extended-security-updates#error-trying-to-check-imds-again-hresult-12002 section, the URL https://microsoft.com/pkiops/certs 404s
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 5265708c-f38d-e961-cd0c-bfb00fb8dad7
Version Independent ID: a0f2fa8d-09aa-d083-d0f1-ea7a00193129
Content: How to troubleshoot delivery of Extended Security Updates for Windows Server 2012 through Azure Arc - Azure Arc
Content Source: articles/azure-arc/servers/troubleshoot-extended-security-updates.md
Service: azure-arc
Sub-service: azure-arc-servers
GitHub Login: @johnmarco
Microsoft Alias: johnmarc

@benwa Thanks for your comment! We will review and provide an update as appropriate.

#please-close

Not sure that's true, I'm still getting 404s

@benwa ok, let me check into it more.

@benwa You can't directly browse to the URL -- it doesn't have a directory listing feature. The leaf certificate used to sign the license document on-premises has an extension that points to the URI of the next intermediate certificate in its validation chain. Windows will read that URI and query that directly. In the example below, it will go to https://www.microsoft.com/pkiops/certs/Microsoft Azure TLS Issuing CA 06 - xsign.crt which you can browse to and download. (note: this example is NOT for ESUs, but the same process applies when the ESU license is downloaded)

Ah, understood. Thank you for the clarification!


Calling warning on Error class segfaults

Describe the bug
Calling the warning method on the Error class (rather than on an instance) segfaults.
To Reproduce
Run the following code:
Error.warning("Hello")
Expected behavior
Error should raise the warning with a generic tag.

This is now fixed in v0.6.1.


Pressable Button breaks when scaled in world space mode

Overview
Currently, the PressableButton functions by calculating distances in world space. If a consumer scales the button, then these world distances break. The button can still function if the ratio of push plane distances are maintained which can be done if set to Local space.
At the moment, this change just adjusts the behavior for newly instantiated buttons. Previously created buttons/prefabs/scenes etc will remain to be in the world space.
Of course, all prefabs should be in local space...but that may be disruptive to people who have already extended from these prefabs.

Default coordinate space mode is Local as this allows pressable buttons to scale and still function
Updated documentation that PressableButton default is Local and this is recommended setting
Added space mode enum popup in PressableButton inspector that on change will auto-update plane distance values
Removed space mode button conversion in PressableButton inspector
PressableButton UnityEvents are null, which is annoying for developers to have to instantiate themselves when trying to subscribe programmatically
Fixed summary tags from private fields to the public properties

Changes

Fixes: #4971
Possibly related: #5428

Verification

As a reviewer, it is possible to check out this change locally by using the following
commands (substituting {PR_ID} with the ID of this pull request):
git fetch origin pull/{PR_ID}/head:name_of_local_branch
git checkout name_of_local_branch

/azp run

/azp run

/azp run

/azp run


in operator ArgumentError

This is on Cerbos 0.20.0
Looking at a snippet of my query plan:
{
            "expression": {
              "operator": "in",
              "operands": [
                {
                  "value": "random_value"
                },
                {
                  "variable": "request.resource.attr.tags"
                }
              ]
            }

and then we look at the get_query function defined in cerbos_sqlalchemy:
        if operator == "in":
            return column.in_(value)

I get the following error:
sqlalchemy.exc.ArgumentError: IN expression list, SELECT construct, or bound parameter object expected, got 'random_value'.

Should the code in cerbos_sqlalchemy be the following instead?:
        if operator == "in":
            return column.in_([value])

The sqlalchemy docs say:

IN is available most typically by passing a list of values to the ColumnOperators.in_() method:

Making these changes locally fixes the issue for me.

https://github.com/cerbos/query-plan-adapters/blob/21de38586a17da0df8cd593d8867804e463d9c44/sqlalchemy/tests/test_query.py#L290
Seems like the tests only covers cases where value is a list

Oo, thanks for raising. Will draw up a fix


Please provide update.json

Could you please provide an update.json for easy installation and updates? Thanks in advance!

pling.com doesn't support update.json

I'm no module dev, so I might miss something here: what does pling.com have to do with providing an update.json here in your Github repo? To my knowledge, that file just holds:
{
    "version": "<your_module_version>",
    "versionCode": <your_module_versionCode>,
    "zipUrl": "<url_to_your_compiled_module.zip>",
    "changelog": "<optional_url_to_your_changelog.md>"
}

This could be placed anywhere in your repo here, usually it's directly in the repo root. Your release process then would include updating that file. What did I miss? Apologies if I'm totally off the track – as said, I'm no module dev myself.

pling.com doesn't have direct download file

Ah, I see – so that's what I missed. Would tagging releases and attaching the ZIP there be an option? Many other modules handle it that way. For some background to my question: I maintain a module repo for Magisk Modules which are FOSS – and I'd like to include your module there. Currently, using update.json is the only way it can automatically keep things up-to-date, though other means might be implemented in the future.

Oh, surprise! It seems the repo-util I'm using can create the module.zip directly from the git repo – I didn't expect that variant working that easy. Can you confirm the following ZIP of your module would be fine?
v1.13_13.zip
That then would not only mean I could add it straight away – but it would save me a lot of head-aches for several others queued up here, waiting to be added :smiley:

I will never support 'talking only' without sending proper logs. If you want a fix, then read #bug_report carefully, and send required files to the author.

Use discussion tab instead next time

Well, a clear "yes" or "no" from your end would have made a "discussion" unnecessary. But same as you said, I'm not about "talking only" – I just was asking for a straight answer I never got. I cannot use pling.com (doesn't work for me), you cannot tell me if just zipping up the cloned repo would produce a proper module (often it does, when developers keep the structure that way – but some modules need some different setup; as I cannot get a ready module to check against and you won't answer it I'll never know for yours) – so I have to give up on it then.  :cry:
So thanks for the invitation – but no need for a "next time" if I cannot get an answer anyway.

Support & Bug Report
https://t.me/androidryukimodsdiscussions/2618
If you don't do above, issues will be closed immediately


define with block statement inside if gets removed

Using a define that contains a block scope inside of an if statement removes the brackets. Given:
#define send_bits(s) \
{ int t = s;\
}

void main() {
  int s;
  if (0) {
    send_bits(s)
    send_bits(s)
    send_bits(s)
  }
}

The generated code:
package main

func main() {
	var s int
	if false {
		var t int = s
		_ = t
		var t int = s
		_ = t
		var t int = s
		_ = t
	}
}

I would expect there to be an inner bracket for each time the define is used. This might be an issue with the preprocessor but I didn't know if CXGO does some "optimization" that would remove these brackets unintentionally. If there isn't an if statement the brackets are generated.

Yes, it's an optimization that cxgo does. It should only do that if there's no declarations in that scope.

So in testing some other cases I have come to the conclusion that it has nothing to do with the define. Just putting the blocks directly inside the if statement gets them removed. It also doesn't matter if there are many or few lines in the block it always gets removed while inside an if statement. Do you know where this optimization code is so I can look for the bug?


Facing Crash Issue in IOS devices

Hi Team,
I have almost finished the application development. Most of the time what I am seeing is application got slow down if the users are trying to use the many features of the application. And some times it got stuck for very long time and I need to close and re-open the again to run it very smoothly.
I have checked many blogs regarding this and tried many error identification tools like Sentry and fixed all of the sentry issues. But still the application is bit sluggish.
Please let me know what are the details you need it to review and provide few suggestions.
Thanks.

Hi @Mohammedomer20610
The goal of this GitHub repo is not to assist in personal code or the whole package/plugin project but to improve Flutter framework reporting bugs or feature requests.
If you encounter behavior that you believe could be a Flutter framework bug, please file a new issue (select the appropriate issue template here https://github.com/flutter/flutter/issues/new/choose) and fill all information indicated in the template.
Instead, if you think that the issue might depend on your implementation, please see flutter's community for resources and ask questions like this. You may also get some help if you post it on Stack Overflow as well.
Closing this issue because it lacks the necessary information to proceed.


Button crashes game if you don't have BYG

Or so I assume from the logs.
crash-2023-01-02_15.51.44-client.txt

Thanks for the report, fixed with https://github.com/Globox1997/PatchouliButton/commit/84e2084653c097b4f5f9f76b998566ef98d224c7


Replace <malloc.h> with standard <stdlib.h>

Malloc.h is non-standard and won't compile on certain platforms that don't include one.
This will help build the library cross platform :D

malloc is required for the _aligned_malloc used for _WIN32 platforms. I had it guarded properly in scoped.h, but did not in the DirectXTexP.h header.
Note I already include  which is the C++ version of stdlib.h.

https://github.com/microsoft/DirectXTex/commit/5a7b9b51ed6cba2f2b9d6888e60625a4034a5e1e


When a new workitem is created in the detail UI, it should be assined a default area

Closing as out of date - no longer an issue.


Remove installkernel package since it is provided by systemd-udev

The 'installkernel' script is provided by systemd-udev; this gets rid of the old custom 'installkernel' package.

@christopherco @rlmenge @Camelron @BettyRain I think we'll need to evaluate what currently is using installkernel, as the one provided by systemd-udev probably isn't doing exactly everything our custom one is doing - for example, the specific filename of the vmlinuz and initrd, and our custom script is dropping the hardcoded /boot/linux-*.cfg file that our hardcoded grub.cfg uses.
However our kernel rpm also does some/all of that stuff, so I think the 'installkernel' script is only getting used by manual runs of 'make install' from a kernel build. Anyone know for sure if anything in our kernel package and/or image build is actually using 'installkernel' for anything?

However our kernel rpm also does some/all of that stuff, so I think the 'installkernel' script is only getting used by manual runs of 'make install' from a kernel build. Anyone know for sure if anything in our kernel package and/or image build is actually using 'installkernel' for anything?

Correct. Our installkernel package & script is only used in manual runs of make install. It is not used by our kernel package or image builds.

@christopherco @rlmenge @Camelron @BettyRain I think we'll need to evaluate what currently is using installkernel, as the one provided by systemd-udev probably isn't doing exactly everything our custom one is doing - for example, the specific filename of the vmlinuz and initrd, and our custom script is adding the hardcoded /boot/linux-*.cfg file that our hardcoded grub.cfg uses.
However our kernel rpm also does some/all of that stuff, so I think the 'installkernel' script is only getting used by manual runs of 'make install' from a kernel build. Anyone know for sure if anything in our kernel package and/or image build is actually using 'installkernel' for anything?

The Mariner-specific /boot/linux-*.cfg has been removed in https://github.com/microsoft/CBL-Mariner/pull/7618


How can I use graphview to show my data in json like listbuilder?

I have no idea how to initialize the nodules from a json or Map file knowing that the elements in the tree are increasing.

yes its a bit challanging to write it using json, You would have to parse the json data yourself and then add the nodes in the graph

I'll try to do.
thanks for the feedback.
Nabil Mosharraf<EMAIL_ADDRESS>escreveu no dia domingo,
1/11/2020 à(s) 10:36:

yes its a bit challanging to write it using json, You would have to parse
the json data yourself and then add the nodes in the graph
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nabil6391/graphview/issues/13#issuecomment-720060553,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIRGBGBFQOJ6P3EZJ2VR3DTSNUT3BANCNFSM4TFNIPWQ
.

I tried to do the following:
graph.addEdges ([
Edge (
Node (GetNodeText (
txt1: "1",
)),
Node (GetNodeText (
txt1: "2",
))),
Edge (Node (GetNodeText (
txt1: "1",
)),
Node (GetNodeText (
txt1: "3",
))),
]);
And I'm getting:
The following NoSuchMethodError was thrown during performLayout ():
The getter 'depth' was called on null.
Receiver: null
Tried calling: depth

@Vivaldo-Roque create the nodes at the top, and then just add the edges
Something like this:
final Node node1 = Node(getNodeText());
final Node node2 = Node(getNodeText());
final Node node3 = Node(getNodeText());
final Node node4 = Node(getNodeText());
final Node node5 = Node(getNodeText());
final Node node6 = Node(getNodeText());
final Node node8 = Node(getNodeText());
final Node node7 = Node(getNodeText());
final Node node9 = Node(getNodeText());
final Node node10 = Node(getNodeText());
final Node node11 = Node(getNodeText());
final Node node12 = Node(getNodeText());
graph.addEdge(node1, node2);
graph.addEdge(node1, node3);
graph.addEdge(node1, node4);
graph.addEdge(node2, node5);
graph.addEdge(node2, node6);
graph.addEdge(node6, node7);
graph.addEdge(node6, node8);
graph.addEdge(node4, node9);
graph.addEdge(node4, node10);
graph.addEdge(node4, node11);
graph.addEdge(node11, node12);

😅 @nabil6391 it looks like i will have a long job.

Perfect package! About JSON. Could you use same JSON structure as https://github.com/r3code/vue-vis-network
      nodes: [
        {id: 1,  label: 'circle',  shape: 'circle' },
        {id: 2,  label: 'ellipse', shape: 'ellipse'},
        {id: 3,  label: 'database',shape: 'database'},
        {id: 4,  label: 'box',     shape: 'box'    },
        {id: 5,  label: 'diamond', shape: 'diamond'},
        {id: 6,  label: 'dot',     shape: 'dot'},
        {id: 7,  label: 'square',  shape: 'square'},
        {id: 8,  label: 'triangle',shape: 'triangle'},
      ],
      edges: [
        {from: 1, to: 2},
        {from: 2, to: 3},
        {from: 2, to: 4},
        {from: 2, to: 5},
        {from: 5, to: 6},
        {from: 5, to: 7},
        {from: 6, to: 8}
      ]

@bubnenkoff, me?

Plugin developer
Вт, 3 нояб. 2020 г. в 18:56, Vivaldo Roque (Wython) <
<EMAIL_ADDRESS>
@bubnenkoff https://github.com/bubnenkoff, me?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nabil6391/graphview/issues/13#issuecomment-721216454,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABRWNFX7BHNEQAWT2MZNWYDSOAR27ANCNFSM4TFNIPWQ
.

@bubnenkoff Yes its possible, but in that case we are making the UI generic with the labels and shape. I wanted it to be very dyanmic.
Still I will have a look if we can come with an easy solution as well

😅 @nabil6391 it looks like i will have a long job.
Update:
I was able to initialize the tree with the data stored in the sqlite database.
It wasn't that easy.

Hey, can you please help me out with the same issue. I need to display the graph with data from database, but am unable to do that currently. Any help would be appreciated. Thanks.

😅 @nabil6391 it looks like i will have a long job.
Update:
I was able to initialize the tree with the data stored in the sqlite database.
It wasn't that easy.

Hey, can you please help me out with the same issue. I need to display the graph with data from database, but am unable to do that currently. Any help would be appreciated. Thanks.

Here is the project take a look »»» https://github.com/Vivaldo-Roque/family_tree
may have some bugs.


Add links for reporting and FAQ to README.md

This covers the corresponding reporting mechanism for #28. A link to our growing FAQ is also added.

Excellent, thank you!


File to message in c++

Hi,
I just came across this repo, great stuff !
Is there any way this can be used to write a function fileToMsg, that would basically do in c++ what
rostopic pub /topic type_of/msg -f  /to/file.yml does ?

If I understand you correctly, no.
This library can be used to handle messages from the subscriber side, not the publisher once

It MIGHT be expended to generate messages also, but this is not its purpose and I don't have time to do it. Pull Requests welcome

I hope @facontidavide doesn't mind my shameless self-promotion but this is exactly one of the intended use cases of ros_babel_fish.
For example, it is used by qml_ros_plugin which also contains code for filling messages from a variant map similar to your described use case.


Run it in docker

we can add more args as well improve the code from here
still need to edit run.sh to update with our vars
and add more vars to be called on dockerfile
its working, opens chrome, logs in and deletes posts
your code is running on docker +_+
thanks for this code, I wanted to delete my stuff long ago
this is how I can give it back, cheers
Marco

Thanks for working on this, I'll test it out as soon as I can

I was able to build the image successfully
I'm going to merge and then probably see if I can do something with the new Github image registry.

@mmatoscom Is run.sh necessary? Looks like it was just there for testing

Hi, I made this run.sh to get people unfamiliar with Docker deleting posts right away.
I also put this run.sh in the Dockerfile header. Docker does not need it, can be deleted.
Still, need to:

fix run vars (2FA delete by year, unlike pages, archival)
I've made a Makefile, maybe helps somebody, make build, make run, prompts for facebook email, user, pass, (and 2FA, year, etc), exports them to container entrypoint. You can take a look at my fork to check. Its not fully functional, I am working on it.
If you want I could help with some travis pipeline, testing etc

I forgot to mention was thinking about to cherry pick only the final version with the Dockerfile and create a new PR

@mmatoscom I plan on merging this PR soon. Anything I can do to help?

Hi @weskerfoot , I am so sorry, I had an hectic week
will try to fix it this weekend
we cant go live without the correct parameters from your script exported to container ARGS

No problem! I've been really busy too
On Thu, Jul 18, 2019, 8:17 AM Marco Matos<EMAIL_ADDRESS>wrote:

Hi @weskerfoot https://github.com/weskerfoot , I am so sorry, I had an
hectic week
will try to fix it this weekend
we cant go live without the correct parameters from your script exported
to container ARGS
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/weskerfoot/DeleteFB/pull/73?email_source=notifications&email_token=AAC4L33AZUKRI5AFWINRDNDQABNNNA5CNFSM4H7GVUW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2IJEUI#issuecomment-512791121,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAC4L32BEWZDNUASTKCFLV3QABNNNANCNFSM4H7GVUWQ
.

Once it's ready I am also going to start publishing images to Docker Hub

@mmatoscom I plan on merging this PR soon. Anything I can do to help?

My friend I am trying to export all options to the container, will try to fix this tomorrow, sorry for delay, have been really busy these days but interested in finish and merge. thanks

@mmatoscom FYI my other branch https://github.com/weskerfoot/DeleteFB/pull/72 introduces a new flag --conversations that archives and deletes messenger convos.

@weskerfoot  working on it today.

@weskerfoot  is there a channel or messenger we could use to speak?

@mmatoscom Do you use IRC? I am on Freenode most of the time. I can just create the channel #deletefb

just joined there

I have added the working files
kindly have a look when you have time
its just git clone, then make
we could do more from here, lets merge this working version before starting developing further
thanks =)

Awesome! I'll try to take a look soon, thanks!
On Thu, Aug 8, 2019, 6:54 AM Marco Matos<EMAIL_ADDRESS>wrote:

I have added the working files
kindly have a look when you have time
its just git clone, then make
we could do more from here, lets merge this working version before
starting developing further
thanks =)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/weskerfoot/DeleteFB/pull/73?email_source=notifications&email_token=AAC4L3YUKDQX3CNRANLYP3TQDP3PTA5CNFSM4H7GVUW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD33HZ5Q#issuecomment-519470326,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAC4L3YRLTYUBNZNUTII6YTQDP3PTANCNFSM4H7GVUWQ
.

I should be able to test it out and then suggest any potential changes soon.

Sorry I've been really busy, I'll get to this today

dont worry, I have been busy as well, take your time
cheers

I'm building it at the moment. I think I'm going to have to figure out a workflow for building / releasing everything and uploading it to a docker registry.

I could help with Travis CI

@mmatoscom Travis CI would be fine to have, can you open a new issue for it?


Allow discardable results on NIOAtomic for mutating functions

Motivation:
Make the NIOAtomic API look a little cleaner
Currently when using add or sub, we have code the tlooks like this,
_ = self.activeConns.add(1)
channel.closeFuture.whenComplete { _ in
    _ = self.activeConns.sub(1)
}

Modifications:
Adding @disrcardableResult to the to following function signatures.

public func add(_ rhs: T) -> T {
public func sub(_ rhs: T) -> T {

Result:
If the API consumer wishes to discard results, the above looks much cleaner.
self.activeConns.add(1)
channel.closeFuture.whenComplete { _ in
    self.activeConns.sub(1)
}

@swift-nio-bot test this please


TypeScript version dependency for tslint@next

Picking up on https://github.com/palantir/tslint/pull/726#issuecomment-151720524
Specifically:

I think we ought to fix TS to a specific version for the next branch. So instead of the dependency being typescript: next, we ought to fix it to something like typescript: 1.8.0-dev.20151027 and then update that each week.

At the moment with tslint@next depending on typescript@next this forces our project to also depend on typescript@next. Otherwise, we see errors like these.
But being forced to depend on typescript@next means our CI build is not stable from one day to the next (because typescript@next points to a nightly build)
One option would be to fix the dependency as suggested by @JKillian and bump on a weekly basis. This has the downside of constraining our project to also depend on that same version and only moving when tslint@next updates its dependency.
Would it instead be possible to have tslint@next depend on TypeScript as a peer dependency? Would that work?
I'll admit now I don't understand all the factors at play here.. so please dismiss this suggestion if it's a load of rubbish!
cc @weswigham given his previous thoughts/comments on this topic

Thanks for bringing this up again and the writeup! I'm in favor of making TS a peer dependency. That makes it so TS is installed in a deterministic location for tslint, custom rules, and apps to consume.

@myitcv Do you think this should also be done for the master branch?

I think we should do the same on the master branch, yes. We'll make that change with 3.0, for same reason I mentioned above (deterministic location for tsc).

Sounds good @adidahiya @JKillian

published 3.0.0-dev.3 with the peer dependency change.

3.0 release PR: #804


feat: support for M1 macs (arm64)

Is your feature request related to a problem? Please describe.
All Docker images are currently built for x86, so it is difficult to run Cactus on ARM-based machines (M1 Macs, e.g.)
Describe the solution you'd like
Additional builds for ARM, or a more streamlined way to create them manually.
Describe alternatives you've considered
For M1 Macs with Rosetta2 installed, the Cactus packages themselves can be built successfully. However, it does seem to require a workaround which involves installing an intel-based version of git and npm (which Rosetta2 then translates) before going through the normal steps in BUILD.MD.
Unfortunately, due to the precompiled binaries and images that are pulled, the scripts that utilize test ledgers cannot currently run on ARM-based machines. Exploring potential fixes for this- one solution could be just manually creating ARM versions of the images for the sake of running Cactus tests and example applications.

More info: https://lists.hyperledger.org/g/maintainers/message/141

@petermetz any idea if reproducing what Sean and @ryjones did on the solang image would work for our images as well?

The faio2, qaio, baio images are not supported by M1 Macs

@RafaelAPB would it make sense to build the ones you can on ARM?

@RafaelAPB Sorry for the slow response. The answer is yes, that is exactly what we want for all of our images so that it's entirely transparent to both Mac ARM users and everyone else.

@RafaelAPB would it make sense to build the ones you can on ARM?

@ryjones @RafaelAPB Yes, it would, but ultimately I'd like all of our images to be supported on all the platforms anyway. It's just a lot of work that needs to be done, but we'll get to it for sure. :-)

@petermetz @ryjones sorry for the late answer. Yes, ultimately it would be better if we supported ARM as well. Some of the test ledgers run fine on both supports though (Substrate/Polkadot test ledger for instance)

Note to self: rent something in the cloud and update the build process docs & readme to make sure we properly support M1

@petermetz we already have an M1 mac at MacStadium


ARROW-11239: [Rust] Some unit test failures

This PR is used to demonstrate some test failures, but not intended to fix the problems.

I am aware of this, but I was unable to work on it. I can only work on this over this weekend :(

I am aware of this, but I was unable to work on it. I can only work on this over this weekend :(

Both ARROW-11239 and code of this PR were updated.
https://github.com/apache/arrow/pull/9193/commits/b81176852536b66838ab33b706a0e404dc8765a6 added two tests to show the failures

Both ARROW-11239 and code of this PR were updated.
https://github.com/apache/arrow/pull/9193/commits/b81176852536b66838ab33b706a0e404dc8765a6 added two tests to show the failures

e20ee33 added new tests that passed locally.

e20ee33 added new tests that passed locally.