[SPARK-32292][SPARK-32252][INFRA] Run the relevant tests only in GitHub Actions

What changes were proposed in this pull request?
This PR mainly proposes to run only relevant tests just like Jenkins PR builder does. Currently, GitHub Actions always run full tests which wastes the resources.
In addition, this PR also fixes 3 more issues  very closely related together while I am here.

The main idea here is: It reuses the existing logic embedded in dev/run-tests.py which Jenkins PR builder use in order to run only the related test cases.

While I am here, I fixed SPARK-32292 too to run the doc tests. It was because other references were not available when it is cloned via checkout@v2. With fetch-depth: 0, the history is available.

In addition, it fixes the dev/run-tests.py to match with python/run-tests.py in terms of its options. Environment variables such as TEST_ONLY_XXX were moved as proper options. For example,
dev/run-tests.py --modules sql,core

which is consistent with python/run-tests.py, for example,
python/run-tests.py --modules pyspark-core,pyspark-ml

Lastly, also fixed the formatting issue in module specification in the matrix:
-            network_common, network_shuffle, repl, launcher
+            network-common, network-shuffle, repl, launcher,

which incorrectly runs build/test the modules.

Why are the changes needed?
By running only related tests, we can hugely save the resources and avoid unrelated flaky tests, etc.
Also, now it runs the doctest of dev/run-tests.py properly, the usages are similar between dev/run-tests.py and python/run-tests.py, and run network-common, network-shuffle, launcher and examples modules too.
Does this PR introduce any user-facing change?
No, dev-only.
How was this patch tested?
Manually tested in my own forked Spark:
https://github.com/HyukjinKwon/spark/pull/7
https://github.com/HyukjinKwon/spark/pull/8
https://github.com/HyukjinKwon/spark/pull/9
https://github.com/HyukjinKwon/spark/pull/10
https://github.com/HyukjinKwon/spark/pull/11
https://github.com/HyukjinKwon/spark/pull/12

Test build #125769 has started for PR 29086 at commit 9d8555a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/30380/
Test PASSed.

Test build #125769 has finished for PR 29086 at commit 9d8555a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/125769/
Test PASSed.

Thank you @dongjoon-hyun!


Introduce cargo-watch.check-command option for Code extension

By this option you can replace check command in cargo watch by the something else like clippy.

CI is failing due to code-style issues in TypeScript. You can run npm run fix to fix them

bors r+
Thanks!


[DOCS] Problem with running the website after cloning

Description
There seems to be a problem when I run the website on my PC using the commands given in the readme.md. It throws an error stating that PWA support is disabled
Screenshots

Thanks man 😊

step 03: npm run dev
step 04: no more steps left🤝🏻

After step 3, it says node: --openssl-privacy-provider is not allowed in NODE_OPTIONS and does not run the server

step 03: npm run dev
step 04: no more steps left🤝🏻

After step 3, it says node: --openssl-privacy-provider is not allowed in NODE_OPTIONS and does not run the server

try again by doing
unset NODE_OPTIONS=""
this should fix

we should add this procedure on the Readme.md anoyne wanna fix this?

Can you assign me this  Francesco.

sure thnaks

done


Ability to Scan docker Image by providing tar archive directly

Hi,
When the image scan is executed the SecretScanner runs docker save and then extract the content of the saved tar then perform a scan on it.
It would be great to have a functionality to provide the tar image directly for scanning so that we don't have to use Docker daemon to run docker save first.

The idea is to run SecretScanner inside a Docker container where it has no access to Docker daemon from the host.
For now, I have implemented a workaround myself, by using a 3rd party tool to create image archive, and then supplying that archive to SecretScanner.
In order to achieve that, I had to add a functionality in SecretScanner to accept the tar archive as an input.

The idea is to run SecretScanner inside a Docker container where it has no access to Docker daemon from the host.
For now, I have implemented a workaround myself, by using a 3rd party tool to create image archive, and then supplying that archive to SecretScanner.
In order to achieve that, I had to add a functionality in SecretScanner to accept the tar archive as an input.

Thanks for the update. This is a useful feature for SecretScanner. Would you like to raise a pull request for this change?

For sure, I will do a pull request soon :)


Update rewrite rule

This change is

Signed CLA
/assign @bprashanth

This stills shows up as one of the top results in google and caused me many hours of wasted time, just want to keep someone else from going down the same road


Platform DRM

PLATFORM_DRM added to source and examples makefiles and to cmake build system
SUPPORT_MOUSE_CURSOR_RPI changed to SUPPORT_MOUSE_CURSOR_DRM, mouse cursor code added from @chriscamacho
public SetGraphicDeviceName(...) added to let the user select the graphics device to use (useful only for PLATFORM_DRM)
Config flag FLAG_INTERLACED_HINT added to let the DRM mode selection allow to choose also interlaced graphics mode, otherwise only a progressive mode is choosen

Please tell me if should any comments, changes etc.!
On Raspberry Pi 1-4 SetGraphicDeviceName(...) shouldn't be needed, it should select automatically the available one. On other Linux distributions the device name could differ, e. g. depending of the grapics card manufacturer.

The Travis CI needs additional build jobs for PLATFORM_DRM and preinstalled libgbm, libdrm and maybe libmesa or whatever provides EGL and OpenGL ES libraries. It should work on any Linux distribution that provides GBM and DRM which is any modern Linux distribution as far as I know.

@kernelkinetic Added two review comments.
About Travis CI, no worries, that script should be reviewed.

@raysan5 I would prefer the automatic selection if no DEFAULT_GRAPHIC_DEVICE_DRM is defined. So one can choose a specific device if necessary and it works out of the box otherwise. I'll also look if I can automatically select by checking all available interfaces if they can render, that's just a flag to check if I'm remembering it correctly. Anyways, I think I'll send a commit tomorrow ☺

Go bindings, not that it matters much, would prefer function, nice and clear, and can be accessed easily. Users can also pass CGO_CFLAGS='-DDEFAULT_GRAPHIC_DEVICE_DRM="/dev/dri/card1"' if and when they need,  but if it can be done with some detection/checking the best. I guess rare would be the cases where the user really wants to pick.

This commit uses the DEFAULT_GRAPHIC_DEVICE_DRM string if it's defined and otherwise tries for /dev/dri/card1 and if that fails for /dev/dri/card0 to support Raspberry Pi 0..4.
I've also implemented automatic selection of the render device but it seems that there are some quirks in Raspberry Pi driver implementation which makes it working on Linux but not on Raspberry Pi or vice versa. So at the moment this commit is what gives the most flexibility so far.

Merged! :D
Thanks to all the contributors for this amazing work!


regression-testSQL query conditions, if the data type is expanded with single quotes…

Problem summary
SQL query conditions, if the data type is expanded with single quotes, an error will occur case
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

LGTM


edit rate and edit images per iteration settings

Thank you for your work,
I have one question regarding your hyperparameters settings. In your settings, you edit and update one view after every 10 steps of NeRF training. How if we edit and update multiple views after 10 steps or larger steps, will it affect the results?

It doesn't affect the results much at all from our experiments


can't upload file of error_function.so

i can't upload file error_function.so on google colab what can I do now

Can you elaborate on what the problem / error is?
Did you try compiling the script.sh? If so, what was the error?

i can't upload file error_function.so on google colab what can I do now

Just place the following code snippet in a cell and replace the path/to/your/code/ with the path inside your drive..
!git clone https://github.com/val-iisc/lsc-cnn.git
!cp /contents/lsc-cnn/error_function.so path/to/your/code


Promotion

Hello jhlywa,
Could we have an example in your package (in "master" or in your jquery "widget") in which you can choose the piece you want to take when you are promoting a pawn.
It could be very good for people whom don't practise javascript every day.
Thanks for that tool, it is a good one :+1:
Thierry

//promote to queen. Other pieces are 'r' for rook, 'k' for knight and 'b' for bishop chess.move({ from: 'a7', to: 'a8', promotion:'q' });

Actually its n for the knight! I found the bug in enginegame.js on ai promote as well...

There should be a function is_promotion(from, to) in chess.js and to return true if the move is valid and is promotion move. After that this issue can be solve like this:
var piece_code = 'q'; if ( is_promotion(from, to) ) {       //a modal window lets the user to choose a promotion piece       piece_code = choose_piece(); } chess.move({ from: from, to: to, promotion: piece_code });

You are absolutely right. Actually I'm doing this in a hackish way, where I check if the move exists in chess.moves({ verbose: true });. The dialog you mentioned should be handled with a Promise/await pattern. Unfortunately this is ES6, but webpack could compile the module down to a compatible state machine.
You are welcome to implement this feature in a separate branch, which could be merged into the master branch later on. This would definitely improve the handling for everyone.

//promote to queen. Other pieces are 'r' for rook, 'k' for knight and 'b' for bishop chess.move({ from: 'a7', to: 'a8', promotion:'q' });

Actually its n for the knight! I found the bug in enginegame.js on ai promote as well...

That's not a bug... 'N' is standard for knight in chess, 'K' is king.

Maybe, this will be helpful.
function isPromoting(fen, move) {
  const chess = new Chess(fen);

  const piece = chess.get(move.from);

  if (piece?.type !== "p") {
    return false;
  }

  if (piece.color !== chess.turn()) {
    return false;
  }

  if (!["1", "8"].some((it) => move.to.endsWith(it))) {
    return false;
  }

  return chess
    .moves({ square: move.from, verbose: true })
    .map((it) => it.to)
    .includes(move.to);
}


Added pipenv support

Experimenting with using pipenv this semster instead of using sudo with pip to see if we get less breakage.

Never worked correctly in all environments (Windows, Intel/Mac, M1/Mac)


[BitcoinMove]Introduce Bitcoin Move framework

Summary
Follow #1236
Introduce Bitcoin Move framework

Migrate Bitcoin relative Move source to frameworks/bitcoin-move.
Use 0x4 for BitcoinMove.

Closes #1191

Discussion: Should we migrate all move frameworks to frameworks dir.
TODO: update docs

I think migrate all move frameworks to frameworks dir is reasonable


Suggestion from Alex Jones#2846

"add a bankrob system where you can have up to 5 people join and the more people the higher chance of success but you share the reward/punishment if you win/lose"
like a group crime pretty much

deferred to next major bot update


[FEATURE] Add button link to Open-Source project page

Detailed description
Add a button link to the Community section. Clicking on this button takes you to the Open-Source projects page.
Context
The Community section summarizes how important it is to make Open-Source contributions. So, I believe that providing a way for users to get to the available Open-Source projects in that section would be great.
Possible implementation
New Recording - 10_13_2022, 2_32_23 PM.webm
Additional information
Nil
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hey @abdullmustyy , I have merged that pr, now you can work on that.
Thanks.


interpreted when input Chinese when seraching history

When I used Chinese Language Input, it would be interpreted, for example, I need to input "王"wang, when I input 'n', the input field of Maccy would be put 'wan' and input tools would be disappered.

Can you please provide step-by-step instructions on how to reproduce the problem and the expected and actual result?

When we search for anything in many after 1 second it just disappeared.

Closing since there was no response. Feel free to re-open with more details.

That bug is already fixed. Maybe it was an OS Issue. Anyway, your app is working fine.


Fix #23: Application- and InstanceName should be optional.

Fix #23: Application- and InstanceName should be optional.

@tom-englert  i agree on that statement. was to force being consistent and give "guidance". What you could add to this PR then is having to modify Extensions methods giving more entry point then without "Application" and "Instance" names
what do you think ?

Not sure which extension methods you are talking about, can you point me to the methods?

https://github.com/pada57/serilog-sinks-influxdb/blob/main/Serilog.Sinks.InfluxDB/LoggerConfigurationInfluxDBExtensions.cs

I don't think these can be changed so easily without loosing backward compatiblity.
But you can always pass null for the application name, if you want to override it with some custom logic.

What could be done is to use the latest c# with nullable enabled and annotate the parameters correctly.

Any chance to get a release with all PRs until next week?
Anything I can help with?

Will work this weekend on it

What could be done is to use the latest c# with nullable enabled and annotate the parameters correctly.

We use DotNet standard which does not allow nullable if not mistaken

It needs C# 8 or later, however the value for this small interface is not too much.

Will work this weekend on it. Which one more urgent?

Only all together will work


Maybe it got reverted at some point or straight up broke again.

yes i forgot to add the component for killrandomhead and nobody is saying anything....


optionsFromVsCode - indent_scripts wrong default

indent_scripts is set default as 'normal' - not 'keep'
https://github.com/HookyQR/VSCodeBeautify/blob/master/Settings.md
If you have 'keep' setting, the formatter behaves differently than for normal HTML file

I go with the default from the library on this one.
https://github.com/beautify-web/js-beautify/blob/master/tools/template/beautify-html.wrapper.js#L52


chore: set python3 version for ort

Codecov Report

Merging #77 (094a023) into main (2bad1fc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #77   +/-   ##
=======================================
  Coverage   31.57%   31.57%
=======================================
  Files          15       15
  Lines         627      627
=======================================
  Hits          198      198
  Misses        429      429

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 2bad1fc...094a023. Read the comment docs.

:tada: This PR is included in version 1.4.11 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


python3Packages.pydeconz: 77 -> 78

Motivation for this change
Update to latest upstream release 78
Change log: https://github.com/Kane610/deconz/releases/tag/v78
Things done

[x] 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"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] 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 118179 run on x86_64-linux 1

3 packages built:

home-assistant
python38Packages.pydeconz
python39Packages.pydeconz


Minimum loading for each truck?

Hi, I am working on setting up a CVRPTW code. I want each vehicle to carry at least 50% of its capacity. I have different costs associated with each vehicle, so obviously the optimiser will prefer using vehicles with lesser costs. But I want to use all vehicles atleast to 50% of their max capacity (the total demand for customers is more than sum of 50% capacities of all vehicles). Please help how to achieve this.
PS: I am a newbie to ortools.
Using python 3.7

Please use the mailing list.

You can put soft lower bound on cumulvar at vehicle start location.
https://developers.google.com/optimization/reference/python/constraint_solver/pywrapcp#setcumulvarsoftlowerbound


MINOR:Optimized the path compression in quick union

Changes are following:

Optimized the path compression logic in quick union
Previously, the logic compressed the path one step up for a node at a time.
Now, the new logic compresses the path to the root node at a time.

@bbejeck @vvcephei could you take a look?

retest this please

retest this please

@bbejeck @vvcephei , if by chance you have time, could you review this?

@bbejeck, Thank you so much for reviewing my PR! I appreciate it.
Those two questions are very good questions.
I was doing some experiments in the last three days and trying the proof that the path compression should be faster.
However, based on the result of my experiments, I would say the performance of the trunk version of the code for most of the test cases I made is better than what I made, the path compression version.
I was confused about the result I have for days. The path compression should be faster than others.
After doing researches on the internet, I found some reasons they are following:

The path compression needs more nodes in the topology chain; if just a few nodes, say less than hundreds, then the trunk version works better. If we try tens or hundreds of thousands of nodes, then the path compression wins.
The path compression needs to introduce another collection to maintenance the nodes which are required to be updated. I tried different collections, like HashSet, LinkedList, ArrayList, Deque and ArrayDeque and some customized data structures, none of them wins. They are all more expensive compared with the trunk version, which does not use another collection.
The trunk version is actually NOT no complete path compression, it is called path splitting.
The amortized cost of path splitting is similar to the path compression version. That is why I changed the comment.
Also, thanks @guozhangwang for taking a look at this PR!!

@xinzhg thanks for the detailed explanation and for taking the time to investigate the performance implications thoroughly.  As for our UnionFind not being a complete path compression, I'm not sure I agree as the implementation looks like what I would expect, but I could be wrong.
If others are ok with the comment change, it's fine with me.
\cc @guozhangwang @mjsax @vvcephei

QuickUnion is only used when the topology is build. Thus, it's not on the hot code path and there is no reason to do any performance micro-optimizations IMHO. I am wondering what "experiments" you did?
We can update the comment of course. I leave it to @guozhangwang to merge or not.

@mjsax Thanks for introducing the scenario Kafka using QuickUnion for. I agreed that it is not on the hot path.
One of the experiments I did is to build a linked list with 30 nodes, so the topology chain looks like node 1 -> node 2 -> node 3 -> ... -> node 30 . And I always call the leaf node (which is node 1) for 15 times. For the path compression version, in the first iteration, it compresses the path for every node. But it also means the first iteration takes longer time. and the next iterations will be faster.
For the trunk branch version, path splitting version, it will move each node to its grandparent at a time. Each iteration does the same thing. The amortized cost of both is similar overall.

Thanks @xin-au! Will merge this to update the comment.

@mjsax Thank you so much!


[fix] black linting

Applied black formatting with --skip-string-normalization for the current version of the codebase
Answers this issue and then was discussed in slack.

@vchulski can you please add CI check for new PRs? Otherwise, we need to repeat it quite often.

@triklozoid that's what I suggested in Slack, but I was suggested just to re-lint the current code.
Anyway, the workflow is added to the CI and it seems to be working correctly.


map ux tweaks 2

Investigate and fix fluctuating bug after map loading process, appears sometimes -  when map is declared as loaded by the system but after refreshing the page it disappears. (Looks like process of the map loading not finishes ans rolls back).
Sort map list alphabetically.
After the synchronization, the Map Translator page needs to be reloaded to show all maps. Fix it to show all maps without refresh.
try to fix SVG image position and scale after upload.

Knowledge gained of many hours of debugging and  figuring out why map uploading fails randomly. IMO it is not trivial, so maybe, i'll save a couple of hours for somebody. So.
getDataSource method at data-source.ts

const getDataSource = (opts: SqljsConnectionOptions) => async () => {
  await initialize();
  const ds = new DataSource(opts);
  setInterval(() => {
    (ds.driver as SqljsDriver).save();
  }, 10000);

  return ds;
};

had really speed up  our DB operations but as a price we have all changes to the db persisted exactly after 10 seconds timeout  after data source init.
So if the user closes tab/navigates to other page before 10secs,  we may have weird situation when sql query is fulfilled, even we get its results as generated entities/ids/etc. BUT  only in memory,  no data really persisted to the  localStorage yet, so the next call to query this 'like-saved-data' fails (even not fails, just returns empty set).
Such an errors really hard to debug, so be aware.
To avoid, i think it is sufficient to place  call await singletons.driver.save() when you want data to be really saved.
Ah, also i noticed that such a deferred persistance causes strange behaviour of the button 'clear site data'  at the Google Chrome's devtools... - after cleaning, the data will apper again if you don't refresh page during 10s....


Need to pass options through to Psych

I've been using yaml_extend and like it. Thanks!
I now need to pass the symbolize_names option to Psych. Specifically I need to pass symbolize_names: true.
I don't see a way to do this. Is there a way to pass options thru to Psych?  Am I overlooking something?

Nevermind. I've figured out a workaround.

Thank you for your report. I think others could need this feature as well. I will reopen this issue to add the symbolize_names as additional pass through option.


Docs: add instructions about chartmuseum

Signed-off-by: Charlie Chiang<EMAIL_ADDRESS>
Updated Build your Own Registry with ChartMuseum and addon push command
Use addon init command in Build Your Own Addon

Corresponding feature PR: https://github.com/kubevela/kubevela/pull/4261

does this doc for 1.5 only?

Yes, 1.5 or later


Found nil error occurs after a pause

When we record and stop, code works fine. But when we pause and then continue and stop, then on clicking "keep" the app is crashing. I am getting error "unexpectedly found nil while unwrapping an Optional value". Any workaround for this? I want to use continue and pause so have to get it working somehow.

Simple nil check in updateMeter


Track memory usage

Investigate memory usage in Page server.

How much is spent on which layer? Are there leaks?
Add counters for tracking the memory usage and expose them as prometheus metrics.
Perform checkpointing based on memory usage.
Pause receiving WAL if too much memory is already used.

Ok to close @hlinnaka

Much of this has been done, or obsoleted by other changes. Closing.


Refactor Exporter and API's FileManager to allow files to be on a remote location (S3 for example)

Currently, the storage used for processed files is only local which can cause problems when using Docker.
The idea is to be able to use different storage such Amazon S3, Azure Storage, local files...

Very closely related to #149


Feature: Model serving merge into main

Description

Add model serving #626
How Has This Been Tested?

Merge criteria:

[X] The commits are squashed in a cohesive manner and have meaningful messages.
[X] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[X] The developer has manually tested the changes and verified that the changes work

/hold

/lgtm


UD vs DREAM status

Using Status: UD and the correct working group gets you the correct boilerplate for that group. In the case of CSS for instance, you get the addition to the Introduction that says what CSS is, and CSS specific additions to the conformance section.
However, using Group: csswg, even with Status: UD also seems to hardcode reporting issue to the csswg's github (cannot replace it using Repository or Issue Tracking), which isn't right for an Unofficial Draft, especially when you host  it elsewhere. It also set the copyright line to

Copyright © 2017 W3C® (MIT, ERCIM, Keio, Beihang). W3C liability, trademark and document use rules apply.

which doesn't seem right of an Unofficial draft either.
Using Group: csswg and Status: DREAM together has the same problems, plus broken styling.
If you use Status: DREAM without a group, these problems go away, but you don't get the relevant boilerplate.
Fixing a few of the following things would help:

[ ] fixing the combination of Group: *** and Status: DREAM to use the same stylesheet as no group and Status: DREAM, rather than a non existing https://www.w3.org/StyleSheets/TR/2016/W3C-DREAM
[ ] Changing the copyright line for Group: **** + Status: DREAM (and maybe also for Status: UD) to what you get when you're not in a group
[ ] Don't point to the groups's default Issue tracker when Status is DREAM or UD, or alternatively, allow it to be overridden by an explicit Repository.

The copyright is easy to do right now - just submit a csswg/copyright-DREAM.include file that's identical to the default one.  Feel free to PR me for that.
I agree with the other two as well, and will work on them.

Okay, so I'm looking into this more.  Specifically, the third one is caused by the CSSWG's own defaults:
{
	"Repository": "w3c/csswg-drafts",
	"!Issue Tracking": "GitHub Issues",
	"Boilerplate": "repository-issue-tracking no",
	"Infer CSS Dfns": "yes"
}

So you can override the Repository key, it just doesn't matter, because the group turns off the generation of the issue tracking from the repository metadata. You can turn that back on if you like - just repeat that Boilerplate line with a yes (or other boolish true).
But you can't remove the special Issue Tracking line - custom metadata is purely additive.
Similarly, the first issue isn't caused by Bikeshed, it's a result of the CSSWG's own boilerplate, which links to the W3C stylesheet.  Bikeshed does autogen this URL, and will happily do so for statuses that don't exist in the W3C, but ultimately it's a matter of the boilerplate using a particular <link> rather than including an inline stylesheet.

Really, you just shouldn't fight against this. Use no Group at all if you're not working in the CSSWG repo; the CSSWG boilerplate and defaults are tuned for the WG repo.  (The boilerplate also links to default.css, for example, tho the effects of that are pretty minimal.)


Expandable item - update header

I think it would make sense to have a setting for enabling notifying the header item of expandable items. I have following use case:

a custom RecyclerView.ItemDecoration that takes care to add margins around items and that also adds a margin below the last item

If the last item is a collapsed header, it gets a margin from my ItemDecoration. If I now expand the last item, this margin stays because the adapter is not notifying the header so that it invalidates it's view and recalculates it's state.
My solution is to manually notify the RecyclerView to update the header in a custom click listener, but that's cumbersome. A simple flag for the builder would be nicer.

Not sure if this is in the scope of the library, since it's not an common use case.

Actually, it's my second use case for something like this after showing a counter of sub items in the header...


2.10版本异步问题

调用agent_chat的时候报异步处理的错误，请问如何解决
INFO:     <IP_ADDRESS>:59574 - "POST /chat/agent_chat HTTP/1.0" 200 OK
2024-07-05 08:58:55,195 - runners.py[line:118] - ERROR: Task exception was never retrieved
future: <Task finished name='Task-63257' coro=<AsyncClient.aclose() done, defined at /data/anaconda3/envs/langchainv3.0/lib/python3.11/site-packages/httpx/_client.py:1996> exception=RuntimeError('unable to perform operation on <TCPTransport closed=True reading=False 0x7f1ba45b0e90>; the handler is closed')>
Traceback (most recent call last):
File "/data/anaconda3/envs/langchainv3.0/lib/python3.11/site-packages/httpx/_client.py", line 2003, in aclose
await self._transport.aclose()
File "/data/anaconda3/envs/langchainv3.0/lib/python3.11/site-packages/httpx/_transports/default.py", line 383, in aclose
await self._pool.aclose()
File "/data/anaconda3/envs/langchainv3.0/lib/python3.11/site-packages/httpcore/_async/connection_pool.py", line 313, in aclose
await self._close_connections(closing_connections)
File "/data/anaconda3/envs/langchainv3.0/lib/python3.11/site-packages/httpcore/_async/connection_pool.py", line 305, in _close_connections
await connection.aclose()
File "/data/anaconda3/envs/langchainv3.0/lib/python3.11/site-packages/httpcore/_async/connection.py", line 171, in aclose
await self._connection.aclose()
File "/data/anaconda3/envs/langchainv3.0/lib/python3.11/site-packages/httpcore/_async/http11.py", line 265, in aclose
await self._network_stream.aclose()
File "/data/anaconda3/envs/langchainv3.0/lib/python3.11/site-packages/httpcore/_backends/anyio.py", line 55, in aclose
await self._stream.aclose()
File "/data/anaconda3/envs/langchainv3.0/lib/python3.11/site-packages/anyio/_backends/_asyncio.py", line 1198, in aclose
self._transport.write_eof()
File "uvloop/handles/stream.pyx", line 699, in uvloop.loop.UVStream.write_eof
File "uvloop/handles/handle.pyx", line 159, in uvloop.loop.UVHandle._ensure_alive
RuntimeError: unable to perform operation on <TCPTransport closed=True reading=False 0x7f1ba45b0e90>; the handler is closed

0.2.x 版本已经停止维护。
0.3.1 版已经发布，优化了配置方式，修改配置项无需重启服务器，可以更新尝试。0.2.x 版本已经停止维护。


Implement highlight on hover using a Color Picking technique

For now this is only a visual effect, the state is not synced between clients. We can discuss in an issue whether this could be useful or not.
https://user-images.githubusercontent.com/21197331/194020360-5515801c-fee8-4d9d-99ef-f8d40dcdc568.mp4

Thanks @martinRenou, It looks great!
For sharing state with other users, I don't think it is useful but is it possible to highlight an object when we select it from the control panel?

So actually, maybe instead of an "highlight on hover" behavior, it would make sense to implement a click selection, and when you click on the piece it selects it in the control panel?

Freecad has both hover and selection events, but I think the selection is more useful.

That way we can both select from the control panel and select from the 3D view.

That's great!

Let's do just like FreeCad! The color-picking being super cheap, I think we can afford it.

I implemented both an hover logic and a click logic for selection:
https://user-images.githubusercontent.com/21197331/194053313-84059174-b6af-4217-af0d-5cc29fb7ba17.mp4
We can bind the control panel UI to the actual 3D selection in a separate PR

Thanks!


Update @kadira/storybook

React Storybook supports npm2 from v2.2
Related issue: https://github.com/airbnb/react-dates/issues/2

Thanks! Closed #2.


Fix algorithm error handling

The name of this branch does not fully capture what it is about.  It started out that way but evolved as I found a bunch of issues with algorithms.  There are several major changes here:

The primary purpose was to try to adapt all the functions in algorithms to be usable in the parallel framework.  That meant meshing them with mspass_func_wrapper and making sure they all follow the suggested mspass model - write errors to elog and kill if necessary but throw an exception only for fatal errors.
I got caught up in the need to add a feature previously not implemented for the scale algorithm. This revision allows an optional window parameter that defines a time window that should be used to compute an amplitude for scaling.   That required some reworking of the C++ code, and in the process I fixed some oddities in the old pybind11 code.   A key point is the scale python function now acts like a normal function for a map operator returning a scaled version of the data and it allows an optional windowing.
I modified WindowData to use the mspass_func_wrapper.  I fixed some definite oddities in the pybind11 code for the old version.
I updated the test code for scale and WindowData.   It passed for me, and I'm assuming this will show it passed.  I MUST poitn out, however, that the test for scale should be extended.  I just patched the existing test script and didn't add tests to verify the new windowing option works in every situation.  I don't think there will be an issue, but that should somehow be put on some list of things to patch someday that are not yet critical enough to spend a lot of time on right now.

Codecov Report

Merging #225 (ba089ab) into master (897de8f) will decrease coverage by 0.15%.
The diff coverage is 30.23%.

@@            Coverage Diff             @@
##           master     #225      +/-   ##
==========================================
- Coverage   43.70%   43.55%   -0.16%
==========================================
  Files         117      117
  Lines       15041    15103      +62
==========================================
+ Hits         6574     6578       +4
- Misses       8467     8525      +58

Impacted Files
Coverage Δ

cxx/python/algorithms/basic_py.cc
77.77% <0.00%> (+1.70%)
:arrow_up:

cxx/src/lib/algorithms/deconvolution/CNR3CDecon.cc
0.00% <0.00%> (ø)

...c/lib/algorithms/deconvolution/GeneralIterDecon.cc
0.00% <0.00%> (ø)

python/mspasspy/algorithms/RFdeconProcessor.py
0.00% <0.00%> (ø)

cxx/src/lib/algorithms/slice_and_dice.cc
25.25% <5.00%> (-30.31%)
:arrow_down:

cxx/include/mspass/algorithms/amplitudes.h
59.37% <36.00%> (-7.74%)
:arrow_down:

python/mspasspy/util/decorators.py
65.35% <42.85%> (-1.31%)
:arrow_down:

python/mspasspy/algorithms/window.py
50.71% <45.56%> (+0.35%)
:arrow_up:

cxx/python/algorithms/amplitudes_py.cc
67.44% <50.00%> (ø)

python/mspasspy/algorithms/signals.py
63.09% <100.00%> (ø)

... and 4 more

Continue to review full report at Codecov.

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

@haruming Could you please make sure the formatting and the test coverage for the python code are OK? I probably won't have time to get to this, but we do want to merge this branch asap.

Sure, I will pull it down and see if there is anything missing.


Dismiss stale pull request approvals when new commits are pushed

Dunno if this is voxpupuli's policy but it seems that this setting is not enabled in this repository:

Would it reasonable to enable it? In my opinion it makes sense to reset approvals if new code is pushed to a change request.
Relevant docs: https://docs.github.com/en/github/administering-a-repository/managing-a-branch-protection-rule

Yeah, although sometimes I approve with a minor type in question. Which means for a simple spell check correction I'll have to re-approve, but probably less harmful than additional code that gets pushed and not reviewed.

Yeah, although sometimes I approve with a minor type in question. Which means for a simple spell check correction I'll have to re-approve, but probably less harmful than additional code that gets pushed and not reviewed.

@bastelfreak Any input from the voxpupuli's side?

@bastelfreak Any input from the voxpupuli's side?

hi,
I enabled it for this repo

hi,
I enabled it for this repo

I'm not 100% sure, but maybe our release process breks if reviews are enforced and that's why we didn't enable it some time ago. In case it fails we can disable the approval settings again.

I'm not 100% sure, but maybe our release process breks if reviews are enforced and that's why we didn't enable it some time ago. In case it fails we can disable the approval settings again.

Yes release process broken.
 bundle exec rake travis_release
Cleaning for module build
Bumping version from 1.1.1 to 1.1.2-rc0
rake aborted!
Blacksmith::Error: Command ["git", "--git-dir", "/tmp/puppet-nftables/.git", "--work-tree", "/tmp/puppet-nftables", "push"] failed with exit status pid 67553 exit 1
remote: error: GH006: Protected branch update failed for refs/heads/master.
remote: error: At least 1 approving review is required by reviewers with write access.
To github.com:voxpupuli/puppet-nftables.git

Think the tag is pushed with the rc commit as well ? So the release never happens. Pushing the 1.1.1 semi-manually.

Yes release process broken.
 bundle exec rake travis_release
Cleaning for module build
Bumping version from 1.1.1 to 1.1.2-rc0
rake aborted!
Blacksmith::Error: Command ["git", "--git-dir", "/tmp/puppet-nftables/.git", "--work-tree", "/tmp/puppet-nftables", "push"] failed with exit status pid 67553 exit 1
remote: error: GH006: Protected branch update failed for refs/heads/master.
remote: error: At least 1 approving review is required by reviewers with write access.
To github.com:voxpupuli/puppet-nftables.git

Think the tag is pushed with the rc commit as well ? So the release never happens. Pushing the 1.1.1 semi-manually.


Fix Althea docker build.

Changes:
1. Updates binary copy from the build stage.
2. Updates uid:gid within the container to althea.
3. Fixes https://github.com/althea-net/issue-tracker/issues/3

Thanks for updating this dockerfile!


[CTD-271]-made several changes to the build script.

a) Removed the step for lintian checking when building a tar file - as I was getting failures on Android daemons due to expected linking problems.  Ultimate solution will be to use lintian overrides but that is rather complicated.  So for the time being I am not running lintian on tar builds.  Lintian still is executed on deb package builds and will still catch a lot of problems.
b) Fixed routine to clean up daemon binaries prior to doing build
c) Moved android target code closer to the beginning for quicker
debugging.
d) Fixed a few things associated with new output folder "./build"
e) Lintian errors print inline rather than in "less" - still need to
catch lintian errors and make them cause CI to fail.
f) Added lib folder to tar files (systemd startup service files)

OK I removed the commented code.


As a feature extractor for point cloud

Hi,
Is your pre-trained model suitable as a feature extractor for a point cloud shape, similar to PointNet? If it is suitable, which variable in the code should I output?
I have observed that the shape of GPT_extractor is [batchsize, 64, 1024], which seems to be the features of the points group. Are there any features based on individual points?
Thanks!

I haven't yet come up with a good way to directly map the token features extracted by PointGPT back to point features. However, perhaps they could be used as global features, and then another network could be used to extract per-point features. Adding the two together would leverage PointGPT's pre-trained network parameters, which should improve the ability to extract features for each point.


Add Curacao holidays

Hi,
First pull request. Hopefully all requirements are met to add Curaçao holidays to the list.
Regards,
Raychel

Pull Request Test Coverage Report for Build 992

104 of 104 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 986:
0.0%

Covered Lines:
9603

Relevant Lines:
9603

💛  - Coveralls

Thank you! 👍

Thank you! +1

Thank you for this project!!!
Regards...


use test.sh in circle.yml (re. #120)

@rhuss something like this, for #120 ? This is UNTESTED and my blind attempt / guess...

#223 would help with half of what's required here - fish-pepper would be a non-issue.
s2i tools in container would be much more interesting.. wouldn't that require "docker in docker" ? I don't think that is feasible, or something that I would have cycles for sinking significant time into further investigating...

Whoa! I'm cleaning up many of my very old open PRs, from MANY years ago.. closing!


JSQMessagesCollectionViewFlowLayoutInvalidationContext missing?

hey, just thought to report that i just downloaded the latest zip and dropped it into my app (i'm old-school and am not using pods) and noted that JSQMessagesCollectionViewFlowLayoutInvalidationContext is missing from the project. A simple drag'n'drop into the Layout folder resolves it..

Strangely, I am seeing the same issue. I did a git clone of the develop branch, dropped the JSQMessagesViewController dir into my project and added #import "JSQMessages.h" to my bridging header.
.../JSQMessagesViewController/Views/JSQMessagesCollectionView.h:22:9: 'JSQMessagesCollectionViewDelegateFlowLayout.h' file not found

Please ignore my last comment. My problem was that I was upgrading from an older version of JSQMessagesViewController and forgot to update the references in my Xcode project. I fixed it by deleting the files from the project window and re-adding them.


Portal UI error message for "Failure threshold" doesn't match tooltip or allowed values

Please provide us with the following information:

This issue is a: (mark with an x)

[ ] bug report -> please search issues before submitting
[X] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
Portal > Container Apps > Revision Management > New Revision > Add > Edit Container > Health Probes
"Failure threshold" only accepts values between 1 and 10 inclusive. But for values outside that range the error message is "The value must be greater than or equal to 10."
Steps to reproduce

Portal > Container Apps > Revision Management > New Revision > Add > Edit Container > Health Probes
Set "Failure threshold" to 0 or any number larger than 10.

Expected behavior [What you expected to happen.]
This is where it gets more confusing. The Health Probes doc suggests setting the FailureThreshold to 48, but the Portal UI won't accept that value.
Expectations:

The Portal UI provides an accurate error message and tool tip
The Health Probes doc uses a valid value for FailureThreshold

Actual behavior [What actually happened.]
Confusion ensues - and my app still fails for some reason
Screenshots

Additional context
Portal

Thanks @charris-msft, I have logged a bug against the Portal team to get this fixed.


--show-summary=false does not work

Search first

[X] I searched and no similar issues were found

Description
The --show-summary flag is documented on various commands e.g. here:
https://docs.liquibase.com/commands/update/update.html
However when using it like so:
new LiquibaseCommandLine().execute({
  "update",
  "--changelog-file=(...path to changelog...)",
  "--show-banner=false",
  "--show-summary=false"
});

I get the the following error:
2023-06-21 17:27:38.181 ERROR l.i.c.LiquibaseCommandLine - Unknown option: '--show-summary=false'
picocli.CommandLine$UnmatchedArgumentException: Unknown option: '--show-summary=false'
	at picocli.CommandLine$Interpreter.validateConstraints(CommandLine.java:13604)
	at picocli.CommandLine$Interpreter.parse(CommandLine.java:13556)
	at picocli.CommandLine$Interpreter.parse(CommandLine.java:13501)
	at picocli.CommandLine$Interpreter.parse(CommandLine.java:13396)
	at picocli.CommandLine.parseArgs(CommandLine.java:1552)
	at liquibase.integration.commandline.LiquibaseCommandLine.registerValueProviders(LiquibaseCommandLine.java:567)
	at liquibase.integration.commandline.LiquibaseCommandLine.execute(LiquibaseCommandLine.java:350)

We would like to be able to switch the summary off since it interferes with e.g. build servers which expect the output to be logged in the usual way (and formatted as such) rather than dumped to System.out.
Steps To Reproduce
As above
Expected/Desired Behavior
It should be possible to switch the changelog summary off with using the (documented) --show-summary=false setting.
Or, if this setting is not and never will be supported, the documentation should remove it.
Liquibase Version
4.21.1
Database Vendor & Version
No response
Liquibase Integration
No response
Liquibase Extensions
No response
OS and/or Infrastructure Type/Provider
No response
Additional Context
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR (Thank you!)

After a bit of debugging / searching around the code I've also tried the following combinations, none of which seem to work either
--show-summary=OFF
--show-summary OFF
--showSummary=OFF
--showSummary OFF

The allowable arguments for --show-summary are OFF, SUMMARY, and VERBOSE.  I added a note to our documentation system to get that updated.
I was able to execute update by doing this:
       new LiquibaseCommandLine().execute(new String[] {                 "--show-banner=false",                 "update",                 "--changelog-file=test.xml",                 "--show-summary=OFF"
--show-banner is a global argument, so it needs to appear in the list before the command.

Thanks for the suggestion, but it might be more confusing to the user to try to describe the positional behavior of the exceptions.  They are allowed on either side to support legacy behavior.
Basically, the arguments which are specific to the command should go to the right, and others to the left.


sanitycheck: parse ecma2020 constructs (??)

To help with PRs such as #3642

@gfwilliams this PR assumes we can parse all syntaxes from the 2020 edition
https://github.com/espruino/BangleApps/blob/a8f1aa97e51bf9fa3d2b3777c63248b243cc0199/bin/sanitycheck.js#L29
... but I don't think that's the case (e.g. destructuring isn't supported), so I have two questions:

Is there a list anywhere of what's supported and what isn't?
Does this matter or do we just accept that we might pass some things through sanitycheck which then fail to parse on-device?

Is there a list anywhere of what's supported and what isn't?

https://www.espruino.com/Features
I guess ideally it'd be really nice to throw errors for unsupported stuff. Is there a 'visitor' for acorn maybe, so we can iterate over everything in the syntax tree and check for unsupported stuff that gets parsed (eg spread I guess)?

I've gone with upgrading to 8.0 which lets acorn parse ?? without an extra dependency, at the price of having to specify an ecma version. A visitor would be ideal but I'm afraid I don't have capacity atm (happy to close this for now to keep the PRs clear)

Thanks! No I'm fine if you want to merge this - I guess the Lint config (or something in espruinotools) should ideally be the one to catch it anyway.

Cool, sounds good - I suppose there's existing constructs that are parsed but not warned about too, so the lint config does sound like the right approach (e.g. destructuring)

Thanks! I think this should probably be in https://github.com/espruino/EspruinoTools (which the IDE as well as BangleApps uses). We could then use the tools themselves (there is some commented out code in https://github.com/espruino/BangleApps/blob/master/bin/sanitycheck.js#L565-L573) to parse which could check for supported constructs as well as potentially some other things (like libraries that don't exist)

Ah, so the tools can accurately tell if the JS will be parsed by espruino? That's cool

I think that'd be best - we already do attempt to parse with Acorn so adding some code to run over that wouldn't be too hard I hope


chore: Drop setFlags from options parsing

Fixes #N/A
Description
This drops the unneeded setFlags parameter from the options parsing now
How was this change tested?
make presubmit
make apply
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 79.505%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
7960

Relevant Lines:
10012

💛  - Coveralls


Issue with Chrome extension: SwitchyOmega

Describe the bug
When SwitchyOmega is enabled in Chrome/Chromium, GreenTunnel doesn't work. Neither [direct] nor [System Proxy] works. Disabling the extension solves the problem immediately.
Desktop (please complete the following information):

OS: Linux (Ubuntu)
Browser Chrome or Chromium
Version 80

Hi Vahid Jaan,
I test it with chrome version of this extension and it works perfect, I think your issue is related to SwitchOmega.
can you check GT logs to see if any unusual log is there? (use -v '*' option)


Salbutamol Quantity Measure

Had a quick idea for a new measure when we have time /resource for it.
Quantity of salbutamol inhalers per patients on asthma/copd list
There is rising concern with the overuse of salbutamol inhlaers and the National Review of Asthma Deaths cited it specifically as a key thing to watch out for as a persons asthma was out of control and recommended systematic population monitoring. Ideally we'd do at a individual level but practice level monitoring may help support practices identify if there repeat med systems are leading to overprescribing of salbutamols.
Separately salbutamol MDIs are quite aharmful to the environment so we want to ensure no overprescribing or waste. This measure could be helful on our Greener NHS dashboard.

Discussed with @richiecroker  and we think this would be worthwhile exploring when we have time. @AnnaB45 Not for now but this might be a good one to help understand measure development, python, website etec.

Hi, my name is Mattia I am a Data Analyst working within the NHSX Analytics Unit. I would like to work on building the python script to ingest the data and process the quantity of salbutamol inhalers per patient on the asthma/COPD list at a CCG level. Is this the full BNF code list HERE for salbutamol inhalers? I'm struggling to find the data for the number of patients on the asthma/COPD list - would someone be able to direct me to it? I assume it is open access data? Thank you in advance for your time and support. @brianmackenna

we have created a prototype of this measure, and have decided that it wasn't of high enough quality to continue.


New libWebRTC build please

While testing the H264 support I ran into some shortcoming in the current libWebRTC build bundled with the plugin. Basically it doesn't specify the profile-level-id attribute, but doesn't really use the default (LoL). It also doesn't specify the packetization-mode.
This has since been fixed without the need for ugly workarounds on Chrome > 51, so I guess building the latest libWebRTC should do.
Reference: https://bugs.chromium.org/p/chromium/issues/detail?id=591971
PS: This would also be a good opportunity to tag the 3.0.0 release :wink:

/cc @ibc

Let me do it during this week (ping me if necessary)

Thanks! (you'll get the cake once it lands :wink: )

So, howis my build progressing? I have a nice and sweet cake here...

Ping me tomorrow morning please.

Good morning!

libwebrtc updated (being pushed right now) but there are some ObjC changes I must handed in Swift land.

I see this:
a=rtpmap:107 H264/90000
a=rtcp-fb:107 ccm fir
a=rtcp-fb:107 nack
a=rtcp-fb:107 nack pli
a=rtcp-fb:107 goog-remb
a=rtcp-fb:107 transport-cc
a=fmtp:107 level-asymmetry-allowed=1;packetization-mode=1;profile-level-id=42e01f

good?

YES, that's the Kosher SDP we are after!
:cake: :cake: :cake: :cake:


Enhancement: Tortoise-ORM Plugin

This is a WIP PR, adding a plugin for tortoise-orm.
It resolves #166
Missing:

test for the DTOFactory async method
doc update

@Goldziher anything you want looked at in particular? Or just a once-over?

@Goldziher anything you want looked at in particular? Or just a once-over?

Code review


Simplified file selection interface

Currently, the user needs to put in the exact path of the file that needs to be created/edited; it would be nice to have a kind of "shortcut" or a simpler way, for example:

the user wants to create a new product but isn't familiar with the folder structure yet:

offer an easy way to "create product"
choose to correct location for the user
create the correct folder for the user
generate a correct id for the user
etc.

I am wondering if we could have some kind of "shortcuts" on the initial view, as this is the first thing the user would see.

Design: Buttons for shortcuts added to Figma.
Technical implementation: Followed up by "Automation" issue: #49


⚠ Samsung has degraded performance

In 3581ba9, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 3002 ms

Resolved: Samsung performance has improved in d9f7c6f.


Default (i.e. non-legacy) versioning causing error

@stovmascript
First, thanks for this.  It's a great project.
For me, the legacy option correctly versions the Xcode project, but the default does not finish running.
I do not have an android folder in my project, and I'm running react-native-version --target ios
Currently, it just logs the following:
[RNV] Versioning iOS...
[RNV] undefined

...followed by the usage output.
I'm running through the source to see if I can give anything more helpful.  From briefly scanning the source, I tried running this:
RNV_ENV='ava' react-native-version --target ios

and I get TypeError: errGrp.map is not a function, which seems to refer to index.js:356

Update: 🙈 the original problem came from the project name in the package.json not exactly matching the *.xcodeproj filename.  The package.json had a dash in the name while the Xcode project file didn't.
Would searching for any *.xcodeproj files in the specified ios folder be an accepted solution (I will be happy to submit a PR when I get time)?  Or should there be another option to specify not just the ios folder but the ios project folder?
Also, the error logging for that area could be fixed.  It seems like it expects an array of arrays but was given an array of Errors?  Again, if I get time, I can look into that more, too.
Thanks for your time and the awesome project!

@blakek Thanks, glad you find it useful.
Yeah, logging will need to be revised at some point. I'd like to use some basic CLI logging lib, if it won't be too much hassle to implement.
Thanks for submitting the PR, lets continue there on the topic of the Xcode filename...

@blakek New version (2.4.1) is out now on npm.


docs(suspensive.org): beautify main page by star animation

TO-BE

AS-IS

PR Checklist

[x] I did below actions if need

I read the Contributing Guide
I added documents and tests.

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

Project coverage is 73.93%. Comparing base (ce9d5ae) to head (9c59a94).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1398   +/-   ##
=======================================
  Coverage   73.93%   73.93%
=======================================
  Files          70       70
  Lines         587      587
  Branches      129      129
=======================================
  Hits          434      434
  Misses        141      141
  Partials       12       12

Components
Coverage Δ

@suspensive/react
100.00% <ø> (ø)

@suspensive/react-dom
95.55% <ø> (ø)

@suspensive/react-native
100.00% <ø> (ø)

@suspensive/react-query
83.47% <ø> (ø)

@suspensive/react-query-4
0.00% <ø> (ø)

@suspensive/react-query-5
0.00% <ø> (ø)

@suspensive/jotai
0.00% <ø> (ø)

@suspensive/codemods
41.97% <ø> (ø)


Use the linter package for visualizations

Your package looks great, but it's re-inventing the wheel. I suggest you use linter for message visualization.

I dont get how this is "re-inventing the wheel", now that this package was one of the first linter packages for Atom and linter did't exist back then this was created.
That aside, why should I transfrom to use linter then there already is a csslint package for linter?
Lastly I think its fine to have both packages, some will like the way linter shows errors, other the way atom-message-panel does. (Im one of them)

The choice is yours, but we have quite a few cool features like lintOnFly and scopes like line, file and project.


Is someone maintaining this library ?

If someone is maintaining this library then kindly Review the PRs otherwise i am thinking i should adopt this library.
Kindly let me know if you agree

its a great idea

Apologies for the lack of response to all issues/PRs. I built this library for a single project (and it works flawlessly in that environment) and have since been carried away with it. I am happy to add you as a contributor to help clear the issues and I can hopefully get some time to have a go at it also soon.

Hi @oli107   kindly send me an email, i have some confusion related to how you generated the gradle library path for Material Range bar. Lets talk about that. You can find my email under my profile

@oli107 Hey can you please send me an email. There is something related to collaborators that i want to talk about. Its kind of urgent


Support chrome extensions

It would be useful to enhance the browsing experience in Franz with standard Chrome browser extensions.
Feature Description

Franz uses Google Chrome instances for services.
Google Chrome supports a wide range of extensions.
Users could customize their experience using existing Chrome plugins.

I tried to copy extensions over from a regular Chrome installation into ...../Franz/Partitions/service-xxxxx, and added Secure Preferences, but there are probably more steps involved in activating them. A shortcut to navigate to chrome://extensions/ would be useful (such a link within a Franz service does not work, at least not when trying to message it to myself in slack, then clicking on it).
Motivation and Context

Mouse gestures (as an example) would allow actions like back without cluttering the UI with browser elements.
Since all services need passwords, a password manager integration would be of use to many.
Both could easily be done by allowing the user to add existing extensions.

One idea that came after submitting the issue above would be to allow references to https://chrome.google.com/webstore/detail/... to open within the own window instead of opening the external browser for them. That way, we might be able to cheat by sending us extension links through each service somehow, then click and install extensions.

Electron does not support the full chrome extensions API. There is an compatibility layer but I haven't had the time to test this yet.

+1 to this, if it's possible. It'd be fantastic to have LastPass working in Franz.

I would love this too! I would love to use Gmelius with Google Inbox inside Franz!

1 For create a Chromebook application(ChromeOs).

Whoever makes chrome extensions work first will be my goto solution. Subscribed to issue to make sure I get updated once this works.

+1 for app.grammarly.com support. Thanks!

+1 for password manager extensions and the awesome markdown-here  for fancy emails and code snippets.

+1

Anything new? :small_airplane:

I'm using Franz in my company for messaging, like Slack, e-mail, etc. I have some extensions like Grammarly and Google Translate that gives me more productivity on a day-to-day basis. I would be absolutely great if I could manage which extension is enabled for each service.
Also, it seems to have some blocking issues to achieve that, take a look: https://gist.github.com/alexstrat/0bc561e769664fc9ab520b73c0a2dd2c

I also have some crucial extensions in chrome. I would love if franz comes as a chrome extension.
Any updates on this issue..?

+1

+1.  being able to use Gmelius would be great.

+1 even providing any type of workaround here would be nice. perhaps there's some way to install extensions on chromium and then have that waterfall towards franz?

Just another shameless +1. Is there any current workarounds available? I would love to use some tools that I use every day in Gmail among other services, but I always have to switch back to a chrome instance to use them due to this limitation.

+1 that would be awesome

+1 Being able to use Chrome extensions such as https://simpl.fyi/gmail/ would be amazing!

+1 on chrome extensions
Trello and GMail have amazing extensions that really boost the workflow

+1 Grammarly is the reason why I use Vivaldi with many chat applications tabs pinned and set to restore once opening the browser, exclusively for that, and Chrome for normal navigation.

Any update if this is possible and if so, what is needed for this to be done

+1 for having chrome extensions available in franz.
We want to use hubspot inside of our g suite mailboxes.

Interesting: https://github.com/getstation/electron-chrome-extension

Any update on this?

+1 for having chrome extensions available in franz.
We want to use hubspot inside of our g suite mailboxes.

Exactly what I want to do. Any luck with Franz?

I want this feature to be able to use DarkReader extension (brings dark mode to all websites) so I can have a full dark mode Franz

+1

+1
Consider that :

it's good for productivity
one less app / no more switching

it would enable so much more functionalities...

Looks like this can be done in electron using the BrowserWindow api this is the code for electron I will like to help to add this feature https://github.com/electron/electron/blob/5dcc354432bd9b6ca09c5cb887eac3a216820b8a/lib/browser/chrome-extension.js#L360
(this message was copied from Rambox where they didn't implement yet this feature but looks like it's possible)

Looks like this is an encouraging direction too:
https://www.electronjs.org/docs/api/extensions
Although not in an electron release, certain recipes could bundle extensions which would be handy e.g. for corporate apps that require specific extensions to allow access to work (my org has blocked my Calendar access to anything not running a specific security extension which exists for Chrome and Firefox, but not Electron today).


Copyright Issue

I coded this bot and you posted to GitHub without my permission. Please take down this repo or I will report this repo and user. And stop using my project in Team IC's server cause I left and you need to take down anything about me.

ohh
but you remember that you have this project for team ic :D.


Release migrations_macros update

hey, any chance we can get a new release of migrations_macros? 52d6c9e107d34512a858608f3b2891a600a7b39f is on the current master but has never been released.
Thanks!

Master is currently tracking 2.0, which is not ready for release.


Not working with image created by Nix

Hi, I can't seem to get this working when I try to convert a container image built by Nix to WASM. Here's what my Nix expression looks like:
# default.nix
let
  nixpkgs-src = builtins.fetchTarball {
    url = "https://github.com/NixOS/nixpkgs/tarball/7731670498f0a22c361c9d68f73d382bce05d7dc";
    sha256 = "sha256-M59UCWsadEFO+DMdZFMtTlxpFyT3RQIpBLuEX8oziVc=";
  };
  nixpkgs = import nixpkgs-src {};
  haskellEnv = nixpkgs.haskellPackages.ghcWithPackages (p: [ ]);
  image = nixpkgs.dockerTools.buildLayeredImage {
    name = "small-haskell";
    tag = "latest";
    contents =  [
      haskellEnv
      nixpkgs.bashInteractive
    ];
    config = {
      Cmd = ["${haskellEnv}/bin/ghci"];
    };
  };
in image

And here are the commands I'm running:
$ nix-build default.nix
$ docker load < result
$ c2w --version
c2w version v0.6.2
$ c2w small-haskell:latest out.wasm
<...>
#120 [bochs-dev-packed 1/2] RUN /tools/wasi-vfs/wasi-vfs pack /Bochs/bochs/bochs --mapdir /pack::/minpack -o packed && mkdir /out
#120 128.3 panicked at 'capacity overflow', library/alloc/src/raw_vec.rs:517:5
#120 128.4 thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: the `wasi_vfs_pack_fs` function trapped
#120 128.4
#120 128.4 Caused by:
#120 128.4     wasm trap: wasm `unreachable` instruction executed
#120 128.4     wasm backtrace:
#120 128.4         0: 0x57515 - <unknown>!<wasm function 540>
#120 128.4         1: 0x5aa96 - <unknown>!<wasm function 567>
#120 128.4         2: 0x5880b - <unknown>!<wasm function 549>
#120 128.4         3: 0x2e180 - <unknown>!<wasm function 229>
#120 128.4         4: 0x3688d - <unknown>!<wasm function 293>
#120 128.4         5: 0x3ce1a - <unknown>!<wasm function 330>
#120 128.4         6: 0x513aa - <unknown>!<wasm function 497>
#120 128.4     ', crates/wasi-vfs-cli/src/bin/wasi-vfs.rs:5:32
#120 128.4 note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
#120 ERROR: process "/bin/sh -c /tools/wasi-vfs/wasi-vfs pack /Bochs/bochs/bochs --mapdir /pack::/minpack -o packed && mkdir /out" did not complete successfully: exit code: 101
------
 > [bochs-dev-packed 1/2] RUN /tools/wasi-vfs/wasi-vfs pack /Bochs/bochs/bochs --mapdir /pack::/minpack -o packed && mkdir /out:
128.4     wasm backtrace:
128.4         0: 0x57515 - <unknown>!<wasm function 540>
128.4         1: 0x5aa96 - <unknown>!<wasm function 567>
128.4         2: 0x5880b - <unknown>!<wasm function 549>
128.4         3: 0x2e180 - <unknown>!<wasm function 229>
128.4         4: 0x3688d - <unknown>!<wasm function 293>
128.4         5: 0x3ce1a - <unknown>!<wasm function 330>
128.4         6: 0x513aa - <unknown>!<wasm function 497>
128.4     ', crates/wasi-vfs-cli/src/bin/wasi-vfs.rs:5:32
128.4 note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
------
container2wasm1665798681:488
--------------------
 486 |
 487 |     FROM bochs-dev-${OPTIMIZATION_MODE} AS bochs-dev-packed
 488 | >>> RUN /tools/wasi-vfs/wasi-vfs pack /Bochs/bochs/bochs --mapdir /pack::/minpack -o packed && mkdir /out
 489 |     ARG OUTPUT_NAME
 490 |     RUN mv packed /out/$OUTPUT_NAME
--------------------
ERROR: failed to solve: process "/bin/sh -c /tools/wasi-vfs/wasi-vfs pack /Bochs/bochs/bochs --mapdir /pack::/minpack -o packed && mkdir /out" did not complete successfully: exit code: 101
exit status 1

Am I using the tool incorrectly?

@vaibhavsagar Thanks for reporting this.

#120 128.3 panicked at 'capacity overflow', library/alloc/src/raw_vec.rs:517:5

Seems to related to memory? Could you check if it causes out of memory?

Thanks for your response! Where do you do think I'm running out of memory, is it the Docker daemon, and how would I check this? The computer I'm running this on has 32GB of RAM installed which I think should be enough?

I ran it again while watching htop in a separate window and I got the same error even though total system RAM usage did not go above 3GB. I tried having docker stats open as well but weirdly did not see anything.

It seems that wasi-vfs fails to package large files that are over several GB. I keep on investigating the root cause but, as a workaround, you can avoid the failure by using a smaller image. (On my machine, small-haskell is 2.79GB. Could you create smaller image if possible?)

Thanks for investigating! I also tried with haskell:slim-buster and got the same error, even though it's a little smaller (2.28GB). Do you know what the size limit is?

I managed to get this working with a <1.5GB container, I'm guessing the limit is 2GB.


DNS resolution issues // How to disable Netbird DNS for self-hosted?

Describe the problem
I'm self-hosting netbird using docker-compose and experience DNS issues once in a while.
To debug, I want to disable netbirds DNS server temporarily but I don't see any option for that in the UI.
Where can I disable netbird DNS service for my peers using the self-hosted variant?

Hello @maaft since our release https://github.com/netbirdio/netbird/releases/tag/v0.12.0 we support DNS settings and there you can disable the DNS configurations to your peers. You can update your dashboard and management services containers to the latest versions.


Info

{prefix}info - Replied with an embedded message containing:

brief description of the bot
dev credits
affiliation credits (if any)
some bot info like latency and version
a footer telling the user to use the help command for command information

I believe we probably need to have multiple "-info" commands. As for example "userinfo", "guildinfo", "botinfo", etc..

I believe we probably need to have multiple "-info" commands. As for example "userinfo", "guildinfo", "botinfo", etc..

When you have the time, create a new issue for each of them, with project "Commands" and the label "Command"
Or make an issue that explains all of them and make it with project "Commands" and the label "Feature"

{prefix}info - Replied with an embedded message containing:

brief description of the bot
dev credits
affiliation credits (if any)
some bot info like latency and version
a footer telling the user to use the help command for command information

I am not sure if latency should be included. We can have a separate "ping" command for that...?


fix: add more permissions to the cronjob

The cronjobs needs at least permissions to list pipelineruns.
I have added get and watch as well.

/lgtm


Refactor custom return types

An approach for #350.
Right now in our custom return types we have some duplicate boilerplate code and the appropriate logic is inside __richcmp__, __str__ and __hash__  functions. This PR moves the logic into separate traits and implements them for native rust types and PyObject. This allows us to introduce convenient macro rules to expand the definition of our custom iterator based on the input types and name. Hopefully this approach makes it easy and fast to introduce new return types (?).
That being said, I am a little bit concerned about the performance. For example, we need to acquire the python Gil even for methods that we don't actually use it (is there any chance that compiler will optimize this?). In any case, even if we decide that this approach makes sense, we should benchmark it (but how?).
There is also PathMapping that does not fit nicely into the macro rule since although it holds a HashMap<usize, Vec<usize>> the PathMappingValues are Vec<NodeIndices>.

Pull Request Test Coverage Report for Build 918329395

583 of 605 (96.36%)  changed or added relevant lines in 1 file are covered.
12 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.7%) to 96.583%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/iterators.rs
583
605
96.36%

Files with Coverage Reduction
New Missed Lines
%

src/iterators.rs
3
95.68%

src/astar.rs
9
91.74%

Totals

Change from base Build 895763804:
0.7%

Covered Lines:
5596

Relevant Lines:
5794

💛  - Coveralls

That being said, I am a little bit concerned about the performance. For example, we need to acquire the python Gil even for methods that we don't actually use it (is there any chance that compiler will optimize this?). In any case, even if we decide that this approach makes sense, we should benchmark it (but how?).

I wouldn't be too concerned about this, for the most part acquiring the gil from a function called by python is low overhead since we already have the handle from the calling python. I think it would only be a potential issue if we were calling these functions from a separate thread where we might have to wait for the gil, but that shouldn't matter in the python interface code. Also the functions which are using the gil now when there weren't before are things like __str__ and __hash__ which aren't super performance critical. It would be a bigger deal for things like __getitem__ etc which get called a lot. The compiler is also probably smart enough to optimize away away code if it's never used in the generated function (but we'd have to look at the output from the compilation to confirm) there is a compiler flag to control this optimization: https://doc.rust-lang.org/rustc/codegen-options/index.html?highlight=dead-code#link-dead-code
As for benchmarking we can run some terra benchmarks that are good examples for how some of the custom return types are used in practice. For things like hash() and str() we'll probably need to come up with some synthetic benchmarks with large data sets in the custom container formats.

@mtreinish done in c660181

This is super nice! I just rebased two pending commits with custom return types, it removes so much boiler plate: https://github.com/Qiskit/retworkx/pull/312/commits/4374d4ac3ff2c113aa0d0e7df1e43031504af7c0 and https://github.com/Qiskit/retworkx/pull/355/commits/a5473e1d6bf37c2c34aa81a491600c1f630d7ec3


how to inline dynamically generated Javascript in index.html

When my application boots on the client, it needs to know if the user is already authenticated so that the proper user interface can be displayed. To avoid the latency associated with an additional request to the API, I was thinking of embedding the current authentication state in a <script> block in index.html that would set a global variable that can then be used as initial state for the redux reducer that handles authentication.
My questions are:
a) is this best practice for React.js/redux SPAs that create-react-app targets?
b) how do I do this?  I would like to use the same mechanism for development as for production. Is this possible with create-react-app or do I need to eject?

This approach sounds reasonable. Does this help?

I saw that note about placeholders, but can't put it together.
In particular, it says: "Then, on the server, regardless of the backend you use..." - during development, I'm using the server that CRA provides.  How do I tell this server to "read index.html" in memory.
I assume it'll require some webpack devserver plugin, but the webpack config is hidden inside CRA - do I need to eject?

PS: it seems that CRA fetches environment variables that start with "REACT_APP_" in env.js which are then used in the index.html processing in webpack.config.dev.js here.
I absolutely do not see how I can interpose a script that contacts the API and embeds its return into index.html.  Maybe I could change, as a work-around, index.html to include
<script src="%PUBLIC_URL/api/getinitialstate.js"></script>

but that would obviously defeat the entire idea of saving an additional client request.
Also, as a matter of philosophy, if I need to make changes to a file such as node_modules/react-scripts/config/webpack.config.dev.js I should basically be ejecting, right?

In particular, it says: "Then, on the server, regardless of the backend you use..." - during development, I'm using the server that CRA provides. How do I tell this server to "read index.html" in memory.

You can't. The assumption is that you check process.env.NODE_ENV. If it is 'development' then you use some fake data you prepared for development. Otherwise you use window.DATA_YOU_INJECTED_IN_HTML.

Closing as it doesn't appear actionable for us.


node-red itself is included in the package.json - installation fails under node.js v8

  "dependencies": {
    "chance": "^1.0.2",
    "client-sessions": "^0.7.0",
    "cookie-parser": "^1.4.1",
    "express-session": "^1.13.0",
    "jsonwebtoken": "^6.1.0",
    "node-red": "^0.13.3",
    "passport": "^0.3.2"
  },

Seriously!? Node-red itself listed as a dependency! And an outdated version at that. Please remove the dependency.

No need to be rude - will get to it soon, or submit a PR and I'll approve.

Updated in bdc7067


Duty Roulette: Expert Korean

Some unused strings in other languages are removed, too.

Thanks for the pull request.  This conflicts with a number of the changes from #1009.  Can you resolve them?


TouchableOpacity (and other basic-onPress components) doesn't work when Collapsible starts off as collapsed

Having "collapsed={true}" and rendering click/press-ables in the accordion disables the ability to press items.
renderDates in the example below contains <TouchableOpacity>'s with simple <Text>'s. Having "collapsed={false}" on render will allow me to press stuff, but not wise versa.
<Collapsible             ref={component => this.topMenu = component}             collapsed={true}           >             { renderDates } </Collapsible>

Hi,
I can confirm this issue. Here's a way to reproduce the problem:

class PendingShootings extends React.PureComponent {
  state = {
    collapsed: true,
  };
  toggle = () => {
    const collapsed = !this.state.collapsed;
    console.debug('collapsed',collapsed);
    this.setState({collapsed});
  };

  render() {
    const initialHeight = 100;
    return (
      <Collapsible collapsed={this.state.collapsed} collapsedHeight={initialHeight}>
          <TouchableWithoutFeedback onPress={this.toggle}>
            <View height={initialHeight} backgroundColor={Colors.tertiary} padding={10}>
              <Texts.Normal white>Vous avez un shooting a confirmer</Texts.Normal>
            </View>
          </TouchableWithoutFeedback>
        <View>
          <Texts.Normal white>Accepter ou refuser?</Texts.Normal>
        </View>
      </Collapsible>
    );
  }
}

Here the toggle function is never called.
What I wonder is if it's expected that the Collapsible children contains any press listener, because it's actually not implemented that way in examples. Maybe this could be better documented if not supported?

Hey @slorber can you make this example on https://snack.expo.io/, so we can see it?

remove the code:  pointerEvents={collapsed ? 'none' : 'auto'}

Nop. There is not working if we put TouchableOpacity or TouchableHighlight inside the _renderContent and _renderHeader. I also remove code pointerEvents={collapsed ? 'none' : 'auto'}.

TouchableHighlight  is ok, do you restart the RN server ?

Ok. I will say to replace TouchableOpacity with TouchableHighlight and Restart the RN. I will inform. it's work or not.

No, Its not working. i Will Restart also RN. but TouchableOpacity is also not working inSide the _renderContent.

Can you give me demo in which its working so that I will implements. Please help me. I will use only this collapsible class for expand - collapse.

Do you bind the function approvalAction ？ Use the arrow function, try again?

Hey guys, closing this as inactive as there's been no activity for over 30 days. If the problem re-occurs on the latest version of react-native-collapsible then ping me to reopen or create a new issue.

For us it worked with removingpointerEvents={collapsed ? 'none' : 'auto'} from Collapsible.js line 184. Since our is part of a menu that is not displayed when collapsed.

@iRoachie I'm also facing this issue. As mentioned before it works if I remove pointerEvents={collapsed ? 'none' : 'auto'}  from Collapsible.js. Maybe you could add support to configure this so we don't need to fork/change in the react-native-collapsible module. Unless there is some other workaround that I'm unaware of?
Snack:
https://snack.expo.io/@bennyb/collapsible-with-collapsedheight
Thanks

Great, thanks!


bazel: change bazelisk for M1 support

This uses a fat binary version of bazelisk to better support arm bazel
Signed-off-by: Keith Smiley<EMAIL_ADDRESS>

This should work when rebased.


Two differently styled Text components inside a TextInput display the same style sometimes

Description
The general idea is to create a <TextInput> with a bolded text for a user's name, like you might see on Slack or Instagram etc: **@someone** hello world.
This is done in my React Native app using a <TextInput> with two <Text> components inside it. The first <Text> component has a bold style, while the second <Text> has no special style.
If I change the text string inside the second text, the behavior differs. Sometimes, I tap on the end of the TextInput, and the style of the second text changes to match the style of the first one. That's wrong. The style should not change. With other strings, the style does not change, and that is the expected behavior.
So, the behavior is inconsistent and unexpected, which indicates a bug in <TextInput>.
Sample code:
    <View style={{paddingHorizontal: 24, borderColor: 'black', borderWidth: 2, margin: 2, borderRadius: 2}}>
    <TextInput multiline>
        <Text style={{"fontWeight":"bold","color":"blue","backgroundColor":"white"}}>David Tabaka</Text>
        <Text> The quick brown fox jumped over the lazy dog and the</Text>
    </TextInput>
    </View>

The bug is causing problems in another public project as well as in my own project - https://github.com/dabakovich/react-native-controlled-mentions
React Native Version
0.70.9
Output of npx react-native info
info Fetching system and libraries information...
System:
OS: Linux 5.19 Ubuntu 22.04.2 LTS 22.04.2 LTS (Jammy Jellyfish)
CPU: (12) x64 Intel(R) Core(TM) i7-10710U CPU @ 1.10GHz
Memory: 5.27 GB / 46.77 GB
Shell: 5.1.16 - /bin/bash
Binaries:
Node: 14.20.0 - ~/.nvm/versions/node/v14.20.0/bin/node
Yarn: 1.22.19 - /usr/bin/yarn
npm: 6.14.17 - ~/.nvm/versions/node/v14.20.0/bin/npm
Watchman: Not Found
SDKs:
Android SDK:
API Levels: 23, 26, 27, 28, 29, 30, 31, 32, 33
Build Tools: 28.0.3, 29.0.2, 29.0.3, 30.0.2, 30.0.3, 31.0.0, 33.0.0
System Images: android-23 | Intel x86 Atom, android-23 | Intel x86 Atom_64, android-23 | Google APIs Intel x86 Atom, android-23 | Google APIs Intel x86 Atom_64, android-29 | Android TV Intel x86 Atom, android-29 | ARM 64 v8a, android-29 | Intel x86 Atom, android-29 | Intel x86 Atom_64, android-29 | Google APIs ARM 64 v8a, android-29 | Google APIs Intel x86 Atom, android-29 | Google APIs Intel x86 Atom_64, android-29 | Google Play ARM 64 v8a, android-29 | Google Play Intel x86 Atom, android-29 | Google Play Intel x86 Atom_64, android-30 | Google APIs Intel x86 Atom, android-30 | Google Play Intel x86 Atom_64, android-32 | Google APIs Intel x86 Atom_64
Android NDK: Not Found
IDEs:
Android Studio: Not Found
Languages:
Java: 11.0.18 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.1.0 => 18.1.0
react-native: 0.70.9 => 0.70.9
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
Create the example default project for React Native.
Add the following code snippet. Make sure not to add it inside a <Section> component, which will confuse the issue.
          <View style={{paddingHorizontal: 24, borderColor: 'black', borderWidth: 2, margin: 2, borderRadius: 2}}>
            <TextInput multiline>
              <Text style={{"fontWeight":"bold","color":"blue","backgroundColor":"white"}}>David Tabaka</Text>
              <Text> The quick brown fox jumped over the lazy dog</Text>
            </TextInput>
          </View>
          <View style={{paddingHorizontal: 24, borderColor: 'black', borderWidth: 2, margin: 2, borderRadius: 2}}>
            <TextInput multiline>
              <Text style={{"fontWeight":"bold","color":"blue","backgroundColor":"white"}}>David Tabaka</Text>
              <Text> The quick brown fox jumped over the lazy dog and the</Text>
            </TextInput>
          </View>

Now run the app on an Android device (I'm using a Samsung A71).
Tap the first TextInput at the end. You will see nothing strange happen - the styles do not change, which is correct.
Next, tap the second TextInput at the end. You will see the style of the first Text get applied to the second Text (see video recording attached). That's the bug.
Snack, code example, screenshot, or link to a repository
Here's a repo that demos the issue.
https://github.com/fullStackOasis/react-native-styled-text-in-textinput
That repo can be used to demo the issue. It also contains a screenrecording in the README. The screenshot below shows how the second Text has had its style changed after tapping at the end of the TextInput as described.

My only test devices are Samsung mobile phones. The bug happens on a Samsung Galaxy A71 5G and a Samsung Galaxy A32 5G.
I tried this with the emulator -avd Pixel_5_API_32 and the bug doesn't happen there.

Anyone?

Is there something I can do to help get this triaged?

Could you try this on the latest stable version of React Native?

@cortinico Thank you for taking a look!
The issue also occurs on React Native 0.71.
I tried to upgrade my sample project, but it wasn't a clean upgrade. Instead, I built a new project and applied the few small changes need to demo the issue. So I did this:
nvm use 14.20.0
npx react-native init StyledTextInTextInput71 --version 0.71

Then I applied the few minor changes to App.tsx.
Project uses React Native 0.71.10. There's video in my new demo project that shows the exact same bug occurring. Here's a new link to my demo project:
https://github.com/fullStackOasis/react-native-styled-text-in-textinput-71
The new output of npx react-native info is here:
npx react-native info
info Fetching system and libraries information...
System:
    OS: Linux 5.19 Ubuntu 22.04.2 LTS 22.04.2 LTS (Jammy Jellyfish)
    CPU: (12) x64 Intel(R) Core(TM) i7-10710U CPU @ 1.10GHz
    Memory: 1.81 GB / 46.77 GB
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 14.20.0 - ~/.nvm/versions/node/v14.20.0/bin/node
    Yarn: 1.22.19 - /usr/bin/yarn
    npm: 6.14.17 - ~/.nvm/versions/node/v14.20.0/bin/npm
    Watchman: Not Found
  SDKs:
    Android SDK:
      API Levels: 23, 26, 27, 28, 29, 30, 31, 32, 33
      Build Tools: 28.0.3, 29.0.2, 29.0.3, 30.0.2, 30.0.3, 31.0.0, 33.0.0
      System Images: android-23 | Intel x86 Atom, android-23 | Intel x86 Atom_64, android-23 | Google APIs Intel x86 Atom, android-23 | Google APIs Intel x86 Atom_64, android-29 | Android TV Intel x86 Atom, android-29 | ARM 64 v8a, android-29 | Intel x86 Atom, android-29 | Intel x86 Atom_64, android-29 | Google APIs ARM 64 v8a, android-29 | Google APIs Intel x86 Atom, android-29 | Google APIs Intel x86 Atom_64, android-29 | Google Play ARM 64 v8a, android-29 | Google Play Intel x86 Atom, android-29 | Google Play Intel x86 Atom_64, android-30 | Google APIs Intel x86 Atom, android-30 | Google Play Intel x86 Atom_64, android-32 | Google APIs Intel x86 Atom_64
      Android NDK: Not Found
  IDEs:
    Android Studio: Not Found
  Languages:
    Java: 11.0.19 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 18.2.0 => 18.2.0
    react-native: 0.71.10 => 0.71.10
  npmGlobalPackages:
    *react-native*: Not Found

Anything I can do to help?

Anything I can do to help?

I'd say the best at this point would be to try to fix this issue and send us a PR as that would highly increase the chances of getting this fixed

That's very strange, I don't get that bug on my S21.


Release  v0.0.38

Description
This release fixes the tag versioning issue in the last release.
Fixes # (issue)
Feature # (details)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration. Is it a  breaking change which will impact consuming tool(s).

[ ] Test A
[ ] Test B

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Cool, took a quick look. So the release 0.0.37 was broken and no artefacts were polished which has broken install test:
For comparison: https://github.com/Azure/draft/releases

@Tatsinnit Hey , I attempted to fix a bug in the last release by updating the tag to include the bug fix commit. However, I found out this isn't permitted. The reason is the Go checksum database captures a permanent checksum the first time it sees a new module version. This checksum ensures the module remains unchanged from previous downloads, which explains the checksum mismatch error we encountered.


[5.4] - Blade @parent directive clarification

Laravel Version: 5.4.4
PHP Version: 7.1.0
Database Driver & Version: MySQL 5.7.17

Description:
What is the expected behavior when you call the @parent directive in a child view?????
With the new @extend parent child execution order, since the child view is executed first, which of the sections in both views in the hierarchy is considered parent section that will replace the ---parent--placeholder---?

Don't think this is a new blade directive.

What is the expected behavior when you

Please ask questions on the forums.


Fix fixed-cidr daemon option

Logic is broken when gateway ip falls in ip-range and ip-range is not the first block in the network

Fixes #17797

this is a vendor-in PR and it was reviewed in libnetwork.
LGTM.

LGTM


Support ignoring dependencies per language

Why?
#22 breaks down the report by programming language, but currently it is tricky to break down the ignore section by programming language. Deadpendency could look at what dep file the ignored deps come from, then use that information. However, really ignored deps should already be specific to a language. For example you may want to ignore a dep request in javascript, but then use a dep request in Haskell. Currently you either ignore for all languages, or allow for all.
Acceptance Criteria

Support the following:

ignore-failures:
  javascript:
    - this-is-js-dep
    - other-languages-can-use-this-dep-name

global ignores will no longer be supported. There is no reason to want to ignore the same dep name in many languages.

This issue pokes at some minor flaws in the current modelling. I'd like to resolve these all together (or before doing this issue).
Deduplicating dependencies is per language
If we have a JavaScript dependency of blah and a Haskell dependency of blah, they are not considered duplicates and both will be fetched. This is good.
It get's more complex when you have duplicates of JavaScript and TypeScript, or Java/Kotlin or C# / VB.NET where the registry is the same. These will resolve to the same dependency report, so should be considered duplicates.
Ignore per registry
Similarly, ignoring would make sense at the registry level, not the language level.

As from <IP_ADDRESS> deduplicating is now per registry not programming language.

<IP_ADDRESS> updates to support ignoring deps per language. Global ignores are working for now (undocumented feature), but will be removed in the future once a good deprecation plan is in place.


Syntax highlighting for code-snippets in get-started

https://workboxjs.org/get-started/webpack
https://workboxjs.org/get-started/gulp
https://workboxjs.org/get-started/npm-script

❤

Fixed by https://github.com/GoogleChrome/workbox-microsite/pull/70


Question: Support Dx9 to Dx10/11

I'm a macos user, and love playing old fighting games which are dx9 via wine. Would you be open to support translation from dx9 to dx10 or 11? The reason being is Dx9 support on wine is very slow but dx10 and dx11 is fast and have translations to the underlying metal API.

I don't plan to do this.  I am currently only working on DirectX from 9 and older.  However, there are tools that convert from Dx9, Dx10 and Dx11 to Vulcan, which should run good under Wine.  The project is called dxvk.


Disable navigation on the navigator

I need to temporarily disable navigation on the navigator. From the documentation I have read that I can invoke 'setMouseNavEnabled(enabled)' on the navigator class. This method works as expected for the viewer but not it's navigator. Inspecting the navigator does show that 'mouseNavEnabled' is false however I am still able to navigate via the navigator.

Thanks for reporting. Will get that fixed right away.

After looking at the navigator object I found that currently the navigator can be disabled by:
viewer.navigator.element.innerTracker.setTracking(false);
and likewise it can be re-enabled by:
viewer.navigator.element.innerTracker.setTracking(true);
Cheers!

Fixed by #575

@rleonard04
This should work with the new patch:
viewer.navigator.setMouseNavEnabled(enabled)


mixin removes securityDefinitions

Problem statement
when mixing 2 specs with OAuth2 securityDefinitions, the resulting merged spec keeps the security specifications at the operation level but omits the top level securityDefinition, resulting in an invalid spec.
Environment
swagger version: current master
go version: 1.9.1
OS:   ubuntu

Workaround: manually inserted the same securityDefinitions in both specs and it works


[BACKLOG-15104] Fixes unknown version in splash screen

@smaring

Build Skipped
:white_check_mark: There was nothing to build. The changes are probably excluded by the build rules. If this is an error, please contact your system administrator.


CI: Add cargo-audit step

It checks for known vulnerabilities in the dependencies.
I would also recommend setting up a Travis cron job to run the CI every week.

Good idea, thank you! :)


simplify the docker run command and base container

Would you be up for a PR for a chainguard based image that should simplify the docker run option? They are well maintained secured images and very tiny.
docker run \
    --mount type=bind,source=$PWD,target=/src \
    -w /src \
    -it \
    --user $(id -u ${USER}):$(id -g ${USER}) \
    ghcr.io/miniscruff/changie \
    new

docker run -it --rm $(PWD):/app ghcr.io/miniscruff/changie new

It's still really small, but I've moved away from scratch to the chainguard nonroot static binary image and it's a great default to use, and has default nonroot user and such set. Should trim down the size of run instructions.

I could maybe add an extra docker image in goreleaser but I don't think modifying the original will be a good idea. Not sure of ghcr.io/miniscruff/changie:chainguard is a good idea either though. Tbh the docker image was more of an after thought.

Kinda forgot about this but I think realistically its better to leave it as is considering docker is probably the least common method of using changie as it is.


Need to run syncdb twice?

Hi,
I recently added this to my app and it seems like robots_* tables aren't created the first time syncdb is done, but it is the second time.
Is this correct?  I don't have a problem with this, I'm just wondering if this is normal or not.
Thanks,
Shige

Sorry, it's my mistake.  I have several settings files and I use a build settings that doesn't include robots.


Typo reat => react

Codecov Report

:exclamation: No coverage uploaded for pull request base (v4@b98720c).

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 b98720c...c1de082. Read the comment docs.

Tanks!


XML roundtrip tests

Over the past two days I fixed issues with XML rountripping. In both cases I had a test case.  Would be nice if there's a directory and I can just drop a file there (and list it in a .py file) and it will be tested for roundtripping and other stuff ala tables_test.py; should probably be in ttLib though, and no need to have the compiled version necessarily.  Anyway, attaching the two tests here.

tests.zip

Thanks Behdad!
I'll add that when I get back to work on Monday.

Did this get in? I'm seeing roundtrip issues with 3.13.1

@jarrodlombardo could you be please provide more info about the specific issue your are seeing?

Thanks. https://github.com/fonttools/fonttools/issues/999

Anything else we need to do here?

#1089 fixed the problem I was seeing in #999 .

We have a lot of XML roundtripping in our tests now. Not what I was proposing, but good enough...


Update README

Please describe the changes in your PR. If it is addressing an issue, please reference that as well.

Updating the README to reflect the recent work done. Also, removing parts that are now more appropriate in docs.
Moving CONTRIBUTING to the top level.

Now we are talking! Love it!

Oh, it might be that this pip install pipecat-ai[canonical] needs to be this pip install "pipecat-ai[canonical]" depending on the shell.


Delete irrelevant runtime/README.md file

There also some files in https://github.com/NativeScript/android-runtime/tree/1ec030fe357f98b64ed1e7ccddbbed1cb5b70df0/test-app/app/src/main/assets/app/modules. I'm not sure what those are.

@jasssonpet the libCalc.so files are a native library that we use for testing.