MultiAnd/MultiOr optimizations

This is a step toward https://github.com/runtimeverification/haskell-backend/issues/3099, however, a version that uses HashSet was actually slower due to the high overhead of hashing of TermLikes (also discussed here https://github.com/runtimeverification/haskell-backend/issues/3182)

Ok this might be a fail :/ Will re-run, but seems like running against the current master, this branch (dirty-ghc8107) has worse performance
 Test                                            | dirty-ghc8107 time | ac0e3c0c403bbca4d812d921f8783f4dfcdbcb29-ghc8107 time | (dirty-ghc8107/ac0e3c0c403bbca4d812d921f8783f4dfcdbcb29-ghc8107) time
-------------------------------------------------+--------------------+-------------------------------------------------------+-----------------------------------------------------------------------
 erc20/ds/totalSupply-spec.k                     | 113.87             | 109.96                                                | 1.0355583848672245
 erc20/hkg/totalSupply-spec.k                    | 51.7               | 49.92                                                 | 1.0356570512820513
 specs/examples/sum-to-n-spec.k                  | 37.22              | 35.92                                                 | 1.0361915367483296
 specs/benchmarks/storagevar02-overflow-spec.k   | 115.35             | 111.28                                                | 1.0365744069015097
 specs/mcd/flipper-ttl-pass-spec.k               | 157.7              | 152.09                                                | 1.036886054309948
 specs/mcd/vat-deny-diff-fail-rough-spec.k       | 404.97             | 390.17                                                | 1.0379321834072328
 specs/mcd/dstoken-burn-self-fail-rough-spec.k   | 1678.85            | 1617.49                                               | 1.037935319538297
 specs/benchmarks/storagevar00-spec.k            | 99.86              | 96.15                                                 | 1.0385855434217368
 erc20/ds/allowance-spec.k                       | 259.04             | 249.24                                                | 1.0393195313753811
 erc20/hkg/approve-spec.k                        | 158.39             | 152.25                                                | 1.0403284072249588
 specs/mcd/dai-symbol-pass-spec.k                | 212.26             | 203.95                                                | 1.0407452807060553
 specs/examples/solidity-code-spec.md            | 161.05             | 154.63                                                | 1.0415184634288301
 specs/functional/storageRoot-spec.k             | 44.84              | 43.02                                                 | 1.0423059042305904
 specs/mcd/vow-fess-fail-rough-spec.k            | 377.88             | 362.46                                                | 1.042542625393147
 erc20/ds/balanceOf-spec.k                       | 177.92             | 170.48                                                | 1.0436414828718912
 specs/mcd/vat-move-diff-rough-spec.k            | 717.43             | 686.93                                                | 1.0444004483717408
 specs/mcd/flipper-tau-pass-spec.k               | 158.56             | 151.73                                                | 1.0450141699070719
 specs/mcd/cat-file-addr-pass-rough-spec.k       | 226.72             | 216.91                                                | 1.0452261306532664
 erc20/ds/approve-failure-spec.k                 | 114.76             | 109.7                                                 | 1.0461257976298997
 specs/mcd/end-subuu-pass-spec.k                 | 145.8              | 139.36                                                | 1.046211251435132
 specs/benchmarks/functional-spec.k              | 48.14              | 45.94                                                 | 1.0478885502829778
 specs/benchmarks/bytes00-spec.k                 | 204.84             | 195.45                                                | 1.0480429777436686
 specs/mcd/vat-subui-pass-rough-spec.k           | 352.15             | 335.38                                                | 1.0500029816924086
 erc20/hkg/balanceOf-spec.k                      | 98.11              | 93.27                                                 | 1.0518923555269648
 specs/mcd/vat-subui-fail-rough-spec.k           | 258.53             | 245.67                                                | 1.0523466438718605
 specs/mcd/vat-subui-pass-spec.k                 | 360.2              | 342.21                                                | 1.0525700593203005
 specs/mcd/vat-addui-fail-rough-spec.k           | 257.21             | 244.17                                                | 1.0534054142605562
 specs/mcd/dsvalue-read-pass-summarize-spec.k    | 133.97             | 127.11                                                | 1.0539690032255526
 specs/benchmarks/storagevar01-spec.k            | 108.51             | 102.94                                                | 1.0541091898193122
 specs/mcd/flopper-cage-pass-spec.k              | 233.99             | 221.72                                                | 1.0553400685549341
 specs/benchmarks/address00-spec.k               | 128.61             | 121.83                                                | 1.0556513174095052
 erc20/ds/transfer-failure-2-a-spec.k            | 488.77             | 462.98                                                | 1.055704350079917
 erc20/ds/approve-success-spec.k                 | 470.79             | 445.73                                                | 1.0562223767751777
 specs/mcd/vat-mului-pass-spec.k                 | 270.02             | 255.59                                                | 1.0564576078876324
 erc20/hkg/allowance-spec.k                      | 127.83             | 120.88                                                | 1.0574950363997353
 specs/mcd/flipper-addu48u48-fail-rough-spec.k   | 104.46             | 98.77                                                 | 1.0576085856029158
 erc20/hkg/transferFrom-success-2-spec.k         | 501.86             | 474.4                                                 | 1.0578836424957843
 erc20/ds/transfer-failure-1-c-spec.k            | 367.59             | 346.94                                                | 1.059520378163371
 erc20/hkg/transfer-failure-2-spec.k             | 169.11             | 159.56                                                | 1.0598520932564552
 erc20/ds/transferFrom-failure-1-d-spec.k        | 444.29             | 419.1                                                 | 1.0601049868766403
 specs/mcd/dai-adduu-fail-rough-spec.k           | 87.16              | 82.17                                                 | 1.06072775952294
 specs/functional/lemmas-spec.k                  | 36.05              | 33.97                                                 | 1.0612304974977922
 erc20/hkg/transferFrom-failure-2-spec.k         | 382.67             | 360.57                                                | 1.0612918434700613
 erc20/ds/transfer-failure-2-b-spec.k            | 252.03             | 237.31                                                | 1.0620285702246006
 specs/mcd/dsvalue-peek-pass-rough-spec.k        | 262.58             | 246.92                                                | 1.063421351044873
 specs/mcd/cat-exhaustiveness-spec.k             | 150.74             | 141.68                                                | 1.0639469226425748
 specs/benchmarks/storagevar03-spec.k            | 89.17              | 83.71                                                 | 1.0652251821765621
 specs/benchmarks/overflow00-nooverflow-spec.k   | 133.53             | 125.3                                                 | 1.065682362330407
 erc20/ds/transferFrom-failure-2-c-spec.k        | 288.69             | 270.69                                                | 1.0664967305774133
 specs/benchmarks/encodepacked-keccak01-spec.k   | 177.37             | 165.98                                                | 1.068622725629594
 specs/benchmarks/requires01-a0gt0-spec.k        | 133.0              | 124.38                                                | 1.069303746583052
 specs/benchmarks/storagevar02-nooverflow-spec.k | 118.69             | 110.99                                                | 1.0693756194251736
 erc20/hkg/transfer-success-2-spec.k             | 213.67             | 199.42                                                | 1.0714572259552704
 specs/benchmarks/keccak00-spec.k                | 212.09             | 197.4                                                 | 1.074417426545086
 specs/bihu/forwardToHotWallet-failure-2-spec.k  | 276.88             | 257.44                                                | 1.0755127408328153
 specs/benchmarks/ecrecover00-siginvalid-spec.k  | 280.01             | 258.98                                                | 1.0812031817128736
 specs/benchmarks/overflow00-overflow-spec.k     | 131.69             | 121.74                                                | 1.0817315590602925
 specs/benchmarks/staticloop00-a0lt10-spec.k     | 226.24             | 209.04                                                | 1.0822809031764256
 specs/mcd/dsvalue-read-pass-spec.k              | 278.29             | 257.13                                                | 1.0822930035390659
 specs/benchmarks/requires01-a0le0-spec.k        | 102.89             | 94.85                                                 | 1.0847654190827623
 erc20/hkg/transfer-success-1-spec.k             | 357.16             | 328.57                                                | 1.0870134217974863
 specs/benchmarks/ecrecover00-sigvalid-spec.k    | 349.96             | 321.45                                                | 1.0886918649867787
 specs/benchmarks/staticarray00-spec.k           | 179.41             | 164.6                                                 | 1.0899756986634266
 specs/bihu/forwardToHotWallet-failure-4-spec.k  | 504.28             | 462.28                                                | 1.0908540278619019
 erc20/hkg/transfer-failure-1-spec.k             | 292.71             | 266.37                                                | 1.09888500957315
 specs/bihu/forwardToHotWallet-failure-1-spec.k  | 207.62             | 188.1                                                 | 1.1037745879851144
 specs/bihu/forwardToHotWallet-failure-3-spec.k  | 310.26             | 281.01                                                | 1.1040888224618342
 specs/benchmarks/encode-keccak00-spec.k         | 339.87             | 306.52                                                | 1.1088020357562314
 TOTAL                                           | 16917.860000000004 | 15961.849999999999                                    | 1.059893433405276

We should see why it's making performance worse on KEVM.

This is now on par with master performance wise (on the KEVM profiling run). I have ran the profiling twice and there seems to be no significant improvement or regression either way, however there was a recorded improvement in performance of the C semantics @dwightguth is working on.


New file

issue #3

In anadi.sh are used some commands like $(pwd), those commands are supported in Windows ?
Did you try it on windows machine ?

Hi @Brijeshthummar02 any news?
Do you need some help?

@Cereal84 to be honest i'm not sure if  those commands are supported in Windows ?

@Cereal84 to be honest i'm not sure if  those commands are supported in Windows ?

Those things are part of the "Windows Support", otherwise the risk is to have not the support for Windows.
I don't know if those commands are supported (or not) by Windows, unfortunately ilI have no Windows machine.

@Brijeshthummar02 the support must be added in both install and run script, otherwise even if the installation goes well the user cannot use it anyway.
I haven't Windows installed so I cannot test it. I expect that who is adding the support in some O.S. does the test too in order to see if everything works.

Close this Pull Request because the user seems not more interested in resolving it.


[Suggestion] - delete project.lock.json

I have hit this issue in the pastw ith NuGet restore not working, and excluding the project.lock.json file from source control fixed it for me, and allows NuGet restore to work in typical fashion. This is something you may want to try. My understanding is that this file should only be committed to source control, when the dependencies / packages are also being committed.

Thanks for the suggestion! I removed project.lock.json from the project, but it still didn't automatically restore for me. That is, I still had to do the dnu restore.


Pull-Requests KPIs

Problem
At AsyncAPI, we prioritize the prompt acknowledgment of new contributors who submit pull requests, offering timely code reviews, and ensuring efficient code merging. We view this quick turnaround time for reviews as a vital indicator on how successful we are at engaging our community of contributors.
Solution
To solve the problem we first need to get visibility around our key engineering metrics that are split into 3 main categories:

Pull-Requests KPIs
Release KPIs
Issues Interaction KPIs

In this Bet we will focus on Pull-requests KPIs
Pull request flow

Metrics we want to track
We would like to track theses metrics in the repos we monitor
Feature lead time
merge (date) - first commit (date)

PR lead time
merged (date) – opened at (date)

Time to first review
first review created at (date)  - last commit created at (date)

Review time
last review created at (date)  – first review created at (date)

Last review to merge time
merged (date) – last review created at (date)

List of repositories we should monitor

asyncapi/asyncapi-react
asyncapi/bindings
asyncapi/cli
asyncapi/glee
asyncapi/modelina
asyncapi/parser-api
asyncapi/parser-js
asyncapi/spec
asyncapi/spec-json-schemas
asyncapi/studio

Solution design

Store the metrics in a database // or analytic service (preference for solutions that supports open source)
Users must see the metrics in a dashboard to track them over time
The system must offer alerting capability (slack integration)

Rabbit holes
Not known for now
Scope

[ ] Define the technical solution // POC
[ ] Dashboard
[ ] Alert system

Out of bounds
The alert system should be hard coded, it must offer the team the possibility to define the KPIs as part of the working agreement (PRs should have first time review < 24h)
Success criteria
We are able to get visibility on the metrics and reduce the review time.

Are we going to collect the metrics in each stage or after a PR is merged? I guess it makes sense to collect it on PR merge since it takes less juice out of our precious runtime of GitHub actions.
But collecting it in different stages would give us fresh data. 🤔

if this is Postman only initiative and related to multiple repos, why do we have it here in spec repo?
My 2 cents:

I suggest you first focus on one repo, to decrease complexity. You will also see how complex issue you want to tackle, and then you can multiply it by N repos you want to track
don't try to reinvent but use ready solutions like https://ossinsight.io/analyze/asyncapi/asyncapi-react#issues

reduce the review time

metrics will not help reducing review time. Might be better to just have a dedicated channel in Slack for you folks, where you will be getting notifications (something like we have now with channel for new issues and PRs) about your repos, and only info about "new issues or PRs that do not fulfil KPI" - your team would have to actively react, and improvement-of-KPI for you would be basically less alerts to the channel. This is better solution as you do not need a database, you do not need to handle github events storage complexity and any dashboards. You would just have a GH workflow that runs in a repo, checks time and drops a message to slack.

@derberg you're right. I'll move this issue to asyncapi/studio as a first repo we start with.
Already explored ossinsight it not enough.

Moved to https://github.com/asyncapi/studio/issues/802


[DOCFIX] Sync docs in 1.8 branch with 1.9-SNAPSHOT

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/148/
Test FAILed.

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/Alluxio-Pull-Request-Builder/149/
Test PASSed.

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/Alluxio-Pull-Request-Builder/150/
Test PASSed.


ctrl-c is broken

hangs at "[2024-04-28 20:17:11][INFO] Stopping API Status Poller..."
after 2 more "ctrl-c":
^C[2024-04-28 20:17:38][ERROR] Unhandled Exception
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/lib/python3.11/multiprocessing/spawn.py", line 122, in spawn_main
    exitcode = _main(fd, parent_sentinel)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/multiprocessing/spawn.py", line 135, in _main
    return self._bootstrap(parent_sentinel)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/multiprocessing/process.py", line 332, in _bootstrap
    threading._shutdown()
  File "/usr/lib/python3.11/threading.py", line 1590, in _shutdown
    lock.acquire()
KeyboardInterrupt
Unhandled Exception
[2024-04-28 20:17:38][ERROR] Unhandled Exception
Traceback (most recent call last):
  File "/home/tower/.local/bin/counterparty-server", line 8, in <module>
    sys.exit(cli.main())
             ^^^^^^^^^^
  File "/home/tower/counterparty/counterparty-core/counterparty-core/counterpartycore/cli.py", line 445, in main
    server.start_all(args)
  File "/home/tower/counterparty/counterparty-core/counterparty-core/counterpartycore/server.py", line 731, in start_all
    api_status_poller.stop()
  File "/home/tower/counterparty/counterparty-core/counterparty-core/counterpartycore/lib/api/api_v1.py", line 502, in stop
    time.sleep(0.1)
KeyboardInterrupt
Unhandled Exception
Traceback (most recent call last):
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/tower/.local/bin/counterparty-server", line 8, in <module>
    sys.exit(cli.main())
             ^^^^^^^^^^
  File "/home/tower/counterparty/counterparty-core/counterparty-core/counterpartycore/cli.py", line 445, in main
    server.start_all(args)
  File "/home/tower/counterparty/counterparty-core/counterparty-core/counterpartycore/server.py", line 731, in start_all
    api_status_poller.stop()
  File "/home/tower/counterparty/counterparty-core/counterparty-core/counterpartycore/lib/api/api_v1.py", line 502, in stop
    time.sleep(0.1)
KeyboardInterrupt
  File "/usr/lib/python3.11/multiprocessing/spawn.py", line 122, in spawn_main
    exitcode = _main(fd, parent_sentinel)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/multiprocessing/spawn.py", line 135, in _main
    return self._bootstrap(parent_sentinel)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/multiprocessing/process.py", line 332, in _bootstrap
    threading._shutdown()
  File "/usr/lib/python3.11/threading.py", line 1590, in _shutdown
    lock.acquire()
KeyboardInterrupt
^CException ignored in: <module 'threading' from '/usr/lib/python3.11/threading.py'>
Traceback (most recent call last):
  File "/usr/lib/python3.11/threading.py", line 1590, in _shutdown
    lock.acquire()
KeyboardInterrupt:

fixed here https://github.com/CounterpartyXCP/counterparty-core/pull/1748


Fix theme.json string extraction

In #320 we limited extraction of theme.json files to ones in the (theme) root and in a /styles folder.
We forgot the fact that there's wp-includes/theme.json in core and lib/theme.json in Gutenberg which are currently forgotten because of that, thus those strings aren't extracted nor translatable in these projects. But they should be!
We probably need to drop that limitation and just scan any theme.json file we encounter...
Meta changes (in sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-customizations/inc/cli/class-make-core-pot.php) are probably required too once that lands. To avoid any regression there.
/cc @ocean90 @oandregal

Sharing this thread for reference.

WordPress 5.8 was released in July 2021. The theme.json strings for WordPress, Gutenberg, and themes were available in GlotPress in December 3rd, 2021. The code to extract them was part of i18n-command 2.2.9. The change in 320 was part of of i18n-command 2.4.0. Are we able to detect which version of WordPress first used i18n-command? That'd give us a hint about when this started happening.
The reason I mention this is that I suppose we have to trigger the extracting string process for every version of WordPress starting in 5.8. Otherwise, the theme.json strings won't be there.
I don't know how the string extraction process works, so I can't speak more about the traceability. This is what I know: the strings for WordPress 5.8 were there when I checked and they are not now in 5.8. Something must have changed them post-release — after December 3rd, 2021 to be exact. I checked and the next events I see are:

a minor 5.8.3 release in January 6th, 2022
a major 5.9 release in January 25th, 2022

Do we re-trigger the extraction process for every minor? Perhaps we used i18n-command 2.4.0 in 5.8.3? Just oversharing in case it's helpful.

@swissspidy looking at the current code, does this codepath receives all files of a given project? If so, can we just add two other checks so that lib/theme.json (gutenberg) or wp-includes/theme.json (wordpress-develop) are also processed?
You also mentioned the following:

That change was made with the assumption that theme.json is supposed to live only in themes and in the root folder of theme. This helped avoid an issue where, when extracting strings in core, theme.json strings from all the default themes were erroneously included (because they live in wp-content.

Don't we have the same issue with strings coming from PHP? Themes can have strings defined in their functions.php file, so how do we deal with those? I suppose there are various ways to make it work: running the string extraction process with an empty wp-content folder, telling the extractors (PHP, theme.json, etc.) to ignore anything coming from wp-content, adding an allowlist of files (current approach). I don't know enough the specifics to suggest any approach, but wanted to raise it to make sure we consider a holistic solution.

I don't want do add hardcoded file paths to the string extractor.
I think we just need to scan every theme.json file that exists, and make sure that dotorg is able to exclude the ones it doesn't need.

I suppose there are various ways to make it work: running the string extraction process for core with an empty wp-content folder (or an ignore flag), telling the extractors (PHP, theme.json, etc.) to ignore anything coming from wp-content, adding an allowlist of files in the extractors (current approach for themejsonextractor). I don't know enough the specifics to suggest any approach, but wanted to raise it to make sure we consider a holistic solution.

Thanks, I appreciate you thinking of all these possibilities!
As you can see at https://meta.trac.wordpress.org/browser/sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-customizations/inc/cli/class-make-core-pot.php, Meta has multiple include/exclude configs for the three projects in use (dev, admin, continents & cities). Most likely a change to those configs would be needed.

The reason I mention this is that I suppose we have to trigger the extracting string process for every version of WordPress starting in 5.8. Otherwise, the theme.json strings won't be there.

I think that will be automated anyway whenever there is a new release. We don't need to know specific version numbers or dates.

As you can see at https://meta.trac.wordpress.org/browser/sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-customizations/inc/cli/class-make-core-pot.php

I've been trying to access that link for the whole morning but I'm getting a 504 🤔

Here's the same file on GitHub: https://github.com/WordPress/wordpress.org/blob/83888a04df5f3d7d218503755543a73a6235c3ef/wordpress.org/public_html/wp-content/plugins/wporg-gp-customizations/inc/cli/class-make-core-pot.php


Feature Request: Corral Now

I'd love to have a feature where you can click a specific tab to put into the corral immediately.
Frequently I'll have a bunch of tabs open while doing research, and I'd love to be able to pick one to 'save for later' while I'm drilling to the bottom of ~100 open tabs.

It's unfortunately impossible to add items to context menus on tabs. Extensions are limited to adding context menus on elements within a rendered HTML page and the context menu of the extension's icon in the URL bar row.
You can already right click inside the page you are on, choose the Tab Wrangler menu, and select "Close tab and save URL Immediately" like in this screenshot:

Watch #69 too; I'm going to add a configurable keyboard shortcut for wrangling the current active tab.


added c# in cheatsheetlinks.json

Hey Gaurav I am so sorry but this repo has been excluded from Hacktoberfest, yet, I'll take your Pull Request for the sake of Contribution
Thank you so much!


LGPL licensing of FFmpeg - is packaging as one exe even allowed?

I am just bringing this up in case this hasn't been thought of yet:
If this project spits out a single exe containing nwjs-win with the whole webkit stuff (the description makes it sound like it does), has anyone considered the problems caused by including the LGPL-licensed FFmpeg / ffmpegsumo.dll into the exe?
The LGPL would either require the ffmpegsumo.dll to be very easily exchangable by the end user with a desired alternative compiled .dll (which is hard to fulfill unless the dll is shipped openly separated from the exe instead of contained in it, or unless you provide a detailed guide along with each such exe blob on how to do replace the contained ffmpegsumo.dll) or that the entire resulting thing is released under GPL (which many users of Web2Exe might not want to do)..
Edit: obviously, this is more a problem of the user of Web2Exe rather than the project itself. Still, it would be good if this could be properly addressed if this filed issue applies in any way

@JonasT, The ffmpegsumo.dll is not included in the exe. It comes included in the same directory as the exported application and can be included/excluded by choice.
But your point is very correct IF you decide to use the patented technologies of ffmpegsumo (mp3, etc). As far as I know, Roger has compiled only the open source technologies into the standard version of NW.js, which is what I use to pack the exes. See here: https://github.com/nwjs/nw.js/wiki/Using-MP3-&-MP4-(H.264)-using-the--video--&--audio--tags.
This means that unless you explicitly decide to include the LGPL licensed functionality, your exported exe should be under this license: https://github.com/nwjs/nw.js/blob/nw13/LICENSE

Ah, sorry for the unnecessary remark then!
About your FAQ suggestion: why would the app need to be open-source if LGPL-licensed functionality of FFmpeg is used? (Are you referring to the special case when the optional GPL parts of FFmpeg are enabled?)
Also, patents are an entirely different matter and not directly relevant on whether one makes the app open-source or not. It's more like whether you will be a poor person because Thompson demands royalties, lol

No worries :)
The app wouldn't need to be open source, that's a bad choice of wording. It would just need to be released under the same license (you couldn't charge money for it without paying royalties)
And yep, you're right about the patent thing :)

The MIT or BSD licenses and also the GPL have wildly different implications, none of them is that you cannot sell the program. Therefore you really shouldn't word it that way, because sale of the program is always possible (even if you enabled the GPL FFmpeg extensions - you would just need to provide the source to all customers).
On the other hand, patents aren't even free to use when you don't charge for your program. In practice, nobody might care - but there are no guarantees.

Ah, that's very interesting information. I wasn't aware that you could still sell the program under the LGPL and GPL licenses.
Thank you very much for the clarification of these details! :) It seems like you know much more about the licensing concerns than I do. I honestly haven't put much concern (other than keeping the exported exe as open as possible) into the licensing since NW.js's license is what the exe's are released under. Looks like I have some reading to do in terms of what the different licenses actually require the users to do. I assumed I knew, but your comments have convinced me otherwise.


New Pull Request Prompted for Perennial Branch's Parent

Master is a known perennial branch. However when I create a new pull request, I see the following message.

If I enter nothing, the solution is incorrect, if i enter master an error will appear
@charlierudolph

Thanks for reporting this issue. Hopefully our rewrite in Golang will fix this issue. We will look at this bug again after that is done.

Unable to reproduce with the current version


iterating on Stacks' persistent store

If it's not obvious by the code quality, Stacks is my first substantive Rust project. In addition to learning on the go I've been sketching out a bunch of ideas.
The rough edges really show in the persistence layer, which is all in store.rs. This issue is an attempt to describe where I'd like to get to, and an invitation for input from anyone who is into this sort of thing.
Persistence in Stacks is roughly, a stream of events (think event sourcing/modeling) stored in sled and a CAS, kept on the file system via cacache's api. The actually app works against a materialized view of the event stream (view.rs).
Something I've been tripping over myself to tease out is, there's interesting meta information that is associated with the content of a clip, and other meta information associated with the particular instance of clipping some given content.
For example, the time and the app you clipped some content from is interesting. If you later interact with the same content, that's interesting too. Sometimes you "clean up" a clip, by quickly editing, in order to paste it for a given context. This is now new content with a new content hash, but it's still interesting to see when and how it relates to the original content. It seems to make most sense to track this information against the unique event sled ids (which is a scru128) in the event stream.
And then there's interesting meta information fundamentally related to the content, regardless of when and how that content is used. Primarily there's mime type and content type. Mime type is currently roughly the http mime type that makes sense to associate with the content. And the content type is additional specialization of the content, which is mostly relevant for text content. Is it HTML, JSON, Markdown, a URL, email, etc. Some other meta information that would be interesting are things like, image dimensions, Open graph data for urls, word count, tiktoken counts etc.
(still fleshing this out)

And then there's interesting meta information fundamentally related to the content, regardless of when and how that content is used. Primarily there's mime type and content type. Mime type is currently roughly the http mime type that makes sense to associate with the content. And the content type is additional specialization of the content, which is mostly relevant for text content. Is it HTML, JSON, Markdown, a URL, email, etc. Some other meta information that would be interesting are things like, image dimensions, Open graph data for urls, word count, tiktoken counts etc.

ah! the metadata should be kept on the CAS.. keeping it on the event stream is awkward

This issue is really: https://github.com/cablehead/xs


Android: Bitmap sampled

When using SetImage, the bitmap is in SampleSize (value 4) when using byte[] or path to image, but not on Stream.
Could you consider making this sample size an option (even if default value is 4) and give the opportunity to get the sample size (when we retrieve coords, we want to know how to transform them to get back coords to our original image)

Fixed.


make RestBuilder public

Because RestBuilder is package protected, activeRestGateway can not be configured.
Sorry for the bug...
Regards,
Khanh

Coverage remained the same at 88.173% when pulling ac861379f70e5e3c71d71832367bd4886bad2c45 on jetoile:makeRestBuilderPublic into 1615412e25f2a6683a1d8165b0a1ef3d1186ef26 on sakserv:master.

Thanks @jetoile - sorry I missed this. I'll merge this now. Do you need me to cut a new release with this fix?

If you have time, it would be great ;)

Merged. I'll work to cut a new release today. Thanks for the contribution!!

0.1.11 has been released.


Add TemperatureResponse class

The current implementation I'm using is below. This is useful for being able to sample a temperature response function, and in theory could be an anlytic function, or just interpolating between points.
class TemperatureResponse:
    """
    A discretely sampled temperature response function.

    Parameters
    ----------
    temps : astropy.units.Quantity
        Temperatures.
    responses : astropy.units.Quantity
        Response function.
    """
    # TODO: use astropy quantity input here
    def __init__(self, temps, responses):
        if temps.shape != responses.shape:
            raise ValueError('Shape of temps must be the same as '
                             'shape of responses')
        self.temps = temps
        self.responses = responses

    def __repr__(self):
        return f'TemperatureResponse, T={self.temps}, response={self.responses}'

    # TODO: use astropy quantity input here
    def sample(self, temps):
        """
        Sample the temperature repsonse function at temperatures *temps*.

        Uses linear interpolation.
        """
        resps = np.interp(temps, self.temps, self.responses,
                          left=np.nan, right=np.nan)
        return resps

you might be able to use the Tabular1D model in astropy modelling to do this?

This is a nice approach. I wonder if this could be the expected type for the temperature response input arguments to Model


Official support for Debian 12

ISSUE TYPE

Improvement Request
Enhancement Request
Feature Idea

COMPONENT NAME

All

CLOUDSTACK VERSION

4.19

CONFIGURATION

OS / ENVIRONMENT

Debian 12.7 Stable
SUMMARY

Can we have official support on Debian 12 Stable? The Ubuntu package has a dependency package python3-mysql.connection that is not included in Debian stable. Thank you!

@gobiursus
The issue has been fixed by #9738
You can find the workaround in the description
Actually we have run lots of smoke tests on debian12 recently, the test results look fine. You can search "kvm-debian12" in the pull requests.

@gobiursus The issue has been fixed by #9738 You can find the workaround in the description
Actually we have run lots of smoke tests on debian12 recently, the test results look fine. You can search "kvm-debian12" in the pull requests.

Thank you!

@gobiursus The issue has been fixed by #9738 You can find the workaround in the description
Actually we have run lots of smoke tests on debian12 recently, the test results look fine. You can search "kvm-debian12" in the pull requests.

Thank you. I will just be waiting for Cloudstack to release Cloudstack 4.20.


Our build pipeline takes way too long. Let's move some bits to a daily build instead.

Some elements of our PR build pipeline take a long time to run and very rarely fail. This slows down the PR process substantially. We should prune those out and make a daily build which would run on main, and if it took an age that would not matter. Any errors picked up by the daily build would then become high priority issues for us to fix.
AB#4257

It's important to ensure all training completes as expected before allowing merges. IMO anything < 30 mins for a PR build is acceptable. Better to merge good code slowly than have to hotfix bad code quickly.


How to treat QR codes in Valora

Expected Behavior

Users can tell when a scanned QR code is out of date
Users can tell the different between a QR recipient and regular phone contact recipient

Current Behavior

There is no way to know if a QR code is out of date. Because QR codes have addresses encoded, users have no way of knowing if they are about to send to an outdated code
When a transaction is done via QR code, the encoded QR data (e.g., address, user's name) is used to populated a "recent recipient" in Valora. This potentially creates a duplicate recipient that is indiscernible from a recipient populated from the user's phone contacts

What does outdated mean here?

It means a user has verified a new address to their phone number


CiaB: noVNC address incorrect

When trying to access the noVNC console of started VMs, the OpenStack dashboard (horizon) fails to work.
http://<IP_ADDRESS>:6080/vnc_auto.html?path=%253Ftoken%253D....
gives me "Connection reset".
The same non-working URL is also reported by openstack console url show.
Checking the qemu command line, I tried the same URL with <IP_ADDRESS> instead of .254 to make it work.
This fortunately succeeded.

Do you have the latest version? The current novnc address is set to https://api.in-a-box.cloud:6080/vnc_lite.html.

This was an installation from the end of Dec. I can retry with a current installation.

Try this. I installed a CiaB yesterday and both the console and the log worked there.

It works on a fresh install.
THANKS!


POST methods doesn't work on Windows Phone 8.1 App

I'm trying to create some Refit methods for the Gitter API.
Implementing the GET methods worked just fine, but when I tried a POST method I got an 404 exception.
I translated this Rest API method :
POST /v1/rooms/:roomId/chatMessages
Parameters
text: Required body of the message.

Into this :
[Post("/rooms/{id}/chatMessages")]
Task<Message> SendMessage([AliasAs("id")] string roomId, [Body] string text, [Header("Authorization")] string accessToken);

"Response status code does not indicate success: 400 (Bad Request)."
A first chance exception of type 'System.IO.FileNotFoundException' occurred in mscorlib.ni.dll
A first chance exception of type 'System.Net.WebException' occurred in System.Net.ni.DLL
A first chance exception of type 'Refit.ApiException' occurred in Refit.DLL
A first chance exception of type 'Refit.ApiException' occurred in mscorlib.ni.dll
A first chance exception of type 'Refit.ApiException' occurred in mscorlib.ni.dll
The thread 0x898 has exited with code 0 (0x0).
You can find the full source code in my Github Repository

This how a made it to work with Refit Link

Is the PUT method working in your link?

Yep :)

I have the same issue. I'm getting this weird message when dealing with Windows Phone Silverlight 8.1. Any ideas on how to solve it? What's the cause of this?


roulette timing should be displayed in seconds

→ Not displayed here : I think roulette timing should be displayed in seconds, so that would be 0.5s, 1s, 2s etc instead of 50, 100, 200 ? It took me a long while to understand what it was x)

Which command is triggering this? I'm having a hard time narrowing this down.

I have absolutely no idea - @chasehult marked this as a good first issue, I don't actually know where or how to do this

Actually this is a pipeline ticket: https://github.com/TsubakiBotPad/pad-data-pipeline/blob/6348342a003ad3dfeebc4fa395e7d1b6b6dc8df4/etl/pad/raw/skills/en/enemy_skill_text.py#L336


Retry issue

Comment 1

Closing issue since it's done

Closing issue since it's done

Comment 3

Comment 3


Add tests for health

Codecov Report

Merging #61 into master will increase coverage by 2.64%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master      #61      +/-   ##
==========================================
+ Coverage   40.19%   42.83%   +2.64%
==========================================
  Files          10       10
  Lines        1779     1779
==========================================
+ Hits          715      762      +47
+ Misses        988      922      -66
- Partials       76       95      +19

Impacted Files
Coverage Δ

api/handlers.go
27.44% <0%> (ø)
:arrow_up:

api/health_linux.go
60.46% <100%> (+60.46%)
:arrow_up:

api/helpers_linux.go
17.35% <0%> (+17.35%)
:arrow_up:

Continue to review full report at Codecov.

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


Publish 07/25/2022 3:30 PM PT

Docs Build status updates of commit afc751b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/connect-to-scc-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-online-powershell-v2.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/v1-module-mfa-connect-to-scc-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-MailDetailATPReport.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-AntiPhishPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-Mailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-SafeAttachmentPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-AntiPhishPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-AtpPolicyForO365.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-SafeAttachmentPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Redirect to https

Pull Request Test Coverage Report for Build 1396

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

Totals

Change from base Build 1395:
0.0%

Covered Lines:
44

Relevant Lines:
44

💛  - Coveralls


Weird error when creating asyncComponent with router

Warning: Failed child context type: Cannot read property 'asyncComponentsAncestor' of undefined
    in AsyncComponent (created by Route)
    in Route
    in div
    in Router (created by ConnectedRouter)
    in ConnectedRouter
    in Provider

looks like there is no validation - that context could not exist - so it throws that error - but after that works as expected (from my point of view)

looks like related to the https://github.com/facebook/react/issues/3392

any workaround?

I have a fix for this in progress.

Fix is queued in master, should be released shortly.


Emit destructors correctly for block nodes

Ethos is especially difficult when dealing with labelled yields (e.g., #.my_label << 17) which may jump over an arbitrary number of block exits and need to destroy everything in-between.

Is this the same issue as gh-61?

That one is only about scope state. This is about objects declared in a block.

With the changes made for #81 this should be fixed now.


Include hMetis for GCG

hMetis is optionally used by GCG for detecting arrowhead structures, but it is unfortunately difficult to track down because the links from SCIP and GCG's sites are currently (and were in the past) dead. However, GCG's GitLab CI runs tests with hMetis and it downloads a binary from https://or.rwth-aachen.de/hmetis/hmetis-2.0pre1.tar.gz.
The included license is as follows:

Copyright and License Notice
The hMETIS package is copyrighted by the Regents of the University
of Minnesota. hMETIS may not be sold or redistributed without prior
approval.
It can be freely used for educational and research purposes by non-profit
institutions and US government agencies only. All other organizations are
allowed to use hMETIS for evaluation purposes only for a period of up to
30 days from the time of download. Any further uses will require obtaining
a license from the University of Minnesota's Patent and Technology Marketing
Office.
As unestablished research software, this code is provided on an ``as is''
basis without warranty of any kind, either expressed or implied. The
downloading, or executing any part of this software constitutes an implicit
agreement to these terms. These terms and conditions are subject to change
at any time without prior notice.

GCG also says it requires zsh to enforce memory and runtime limitations (which is hilariously jank, but what can we do?)

Played around a bit locally and came up with this:
diff --git a/packages/gcg.nix b/packages/gcg.nix
index 38911e2..1ba3b8c 100644
--- a/packages/gcg.nix
+++ b/packages/gcg.nix
@@ -25,6 +25,8 @@ in
     in
       builtins.concatStringsSep "." matches;

+    cmakeFlags = [ "-DHMETIS=ON" ];
+
     nativeBuildInputs = with pkgs; [
       cmake
       git # to obtain git commit hash
@@ -54,6 +56,21 @@ in
       for file in ${builtins.concatStringsSep " " debugFiles}; do
         sed -i '1s/^/#define SCIP_DEBUG\n/' $file
       done
+
+      # Fix `zsh` path
+      substituteInPlace \
+        src/gcg/dec_hcgpartition.cpp src/gcg/dec_hrgpartition.cpp src/gcg/dec_hrcgpartition.cpp \
+        --replace-fail "zsh -c" "${pkgs.zsh}/bin/zsh -c"
+
+      # Fix `hmetis` path (it will be installed there later)
+      substituteInPlace \
+        src/hmetis.h.in \
+        --replace-fail "@HMETIS_EXECUTABLE@" "$out/bin/hmetis"
+    '';
+
+    postInstall = ''
+      # Install `hmetis` binary where it was expected
+      install -Dm755 $src/hmetis -t $out/bin
     '';

     enableParallelBuilding = true;

It would be better to have hMetis's code on GitHub and build it from source, but this works for now. I found some old email communication with Dr. George Karypis on Ubuntu's packaging/licensing of parMetis in 2003, but hMetis is MIA for now: https://launchpad.net/ubuntu/focal/+source/parmetis/+copyright

However, GCG's GitLab CI runs tests with hMetis and it downloads a binary from or.rwth-aachen.de/hmetis/hmetis-2.0pre1.tar.gz.

Given the strict licensing I do not know if this should be public. On the other hand, the x86_64 binary is also in the GitHub and the official site is down...
Due to hMetis being unfree I am somewhat unsure if hMetis should be used by default.
I've created #2 for now.


Output redirect from mkdir to cd shows unexpected internal prefix

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
I am experiencing an unexpected behaviour when I created a directory and tried to redirect the output (folder-object) to cd to navigate into the created folder directly.
Expected behavior
PS E:\> mkdir Test | cd
PS E:\Test>

Actual behavior
PS E:\> mkdir Test | cd
PS Microsoft.PowerShell.Core\FileSystem::E:\Test>

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.2.6
PSEdition                      Core
GitCommitId                    7.2.6
OS                             Microsoft Windows 10.0.17763
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals

This quirk is expected and correct (but annoying).
TLDR Use cd (mkdir test)   or mkdir test | % tostring | cd instead.
First part of the explanation is that many PowerShell commands including Set-Location (alias CD)  have parameters like this
    [Parameter(ParameterSetName='Path', Position=0, ValueFromPipeline=$true, ValueFromPipelineByPropertyName=$true)]
    [string]$Path,

    [Parameter(ParameterSetName='LiteralPath', Mandatory=$true, ValueFromPipelineByPropertyName=$true)]
    [Alias('PSPath','LP')]
    [string]$LiteralPath

This sets the actions for piped objects as follows

If they have a property named Path,  use that as the -Path parameter.
If not, but if they have a property named LiteralPath , PSPath or LP use that as the -LiteralPath parameter (there cannot be a literalpath if path already exists)
If they have no such property, convert the object to a string and use that as the -Path parameter

(LiteralPath doesn't process wildcards which affects [ xx ]  with the file system provider,  other providers might consider * and/or ? to be legal in a path)
The second part is that if you create a new directory or get one with Get-Item / Get-ChildItem (or one of their aliases) the System.IO.DirectoryInfo object returned doesn't have a  property named path  the name used is  FullName , but PowerShell adds a PSPath property to files and directories,
Putting the two parts together means Get-Item PowerShell | cd   Is equivalent to
 $p = Get-Item PowerShell
 Set-Location -LiteralPath $p.PSPath

Which is seldom what we want (but cd / Set-Location Does allow the current location to be something which does not have a PSDrive, -  for example  cd Microsoft.PowerShell.Core\Registry::HKEY_CLASSES_ROOT. )
If we convert to a string before piping into cd, the string becomes the -Path parameter, and
 $p = Get-Item PowerShell
 Set-Location $p

will assume $P is the -path parameter, and convert it to a string. Doing that returns the fullname property, and works as expected, i.e. cd $p  is equivalent to doing Set-Location -path $p.ToString()
We can skip storing the variable, so  cd (mkdir test)   creates the directory and passes it as the -path parameter which is then converted to a string holding the full name.
$p | command and command $p should, ideally, do the same thing and they don't here,  cd $p does what is expected and $p | cd works differently. This annoyance could be fixed by either or both:

Making Fullname an alias for -Path in  commands where it is a problem. This could be done with proxy function (but most people won't go down that route).
Adding an alias property .Path to file and directory objects (it is possible to define and load a types.PS1xml file to do this, but again most people won't)

I don't know if the effort : reward ratio means it has never been worth doing it, or whether either of those would have side-effects we don't want.  I can't invent a case where there are side effects, but that doesn't mean there aren't any.

@jhoneill That's quite an investigation and write-up! I propose two extra solutions which may be simpler:

At the end of Set-Location, if the path can be simplified, set location to the simplified one
Since this is mostly a visual bug, simplify location if possible only in the prompt

I vote for option 2, as it is less likely to be a breaking change if some edge case relies on the current location being in this weird long form.
@ThomasLey You can test if function Prompt { (Resolve-Path Get-Location) -replace $HOME, '~' } solves your issue, if not then try replacing Resolve-Path with some other function.

@dkaszews :-)
You can't say with 100% certainty what the path should be simplified to
Should  get-item $env:temp | cd   resolve to TEMP: or C:\users\me\AppData\Local\Temp
And it is possible (if unlikely) that someone wants to do
> get-item hklm: | cd

And later be able to do
> cd '..\HKEY_Classes_root'

This  isn't possible with cd HKLM:  - but with that cd \ works and with  get-item hklm: | cd it fails.
If a script has cd Microsoft.PowerShell.Core\Registry::HKEY_LOCAL_MACHINE\ I wouldn't want Set-location to assume that HKLM: was what was wanted.
But I'm reasonably  happy that assuming get-itemPowerShell | cd meant change to the full name not to the PSPath instead of assuming we want the PSPath..
I'm not sure masking having a different directory is a good idea it would warn you why (e.g.) start $pwd  doesn't like the directory and so on.

It is fundamentally follow PowerShell provider conceptions.
Right fix should rather be in Formatting System or prompt function.
Dup #10522

I  think we should close this one as duplicate and take discussion to 10522

Good analysis, @jhoneill, but just as an aside, regarding one of the workarounds: That Get-Item . | Set-Location and Set-Location -LiteralPath (Get-Item .) bind the -LiteralPath parameter differently - solely based on the incidental aspect of whether the value came from the pipeline or was passed as an argument - is problematic in its own right:

See #6057

@mklement0  yes, interesting point .
ValueFromPipelineByPropertyName  doesn't act as  Value From Argument By Property name. If there were such a parameter option I have multiple places where it would have simplified function bodies, (if an object is piped use its ID property, but if something passed as an argument, check if it has an ID property, and if so use that in place of the whole object)
But I'd want them kept separate.
I think, also, one piped object can go to many parameters with ByPropertyName, but an argument must bind to exactly 1 parameter.


Detector Diagnostic Counts and Aggregations

This pull request includes files that count and aggregate the 30 second raw data files for detector diagnostic tests included in the PeMS Guide.
The files also provide the basis to address issue #84 by utilizing the total sample counts to perform the analysis to see if a sufficient number of samples have been received.
Fixes #79, #82, #80.

@kengodleskidot Ian figured out the group by issue, which was that your CTEs needed two! I added them to the correct places and committed the changes to this branch. See the latest commit

Ah, right, good point about the time
Ian Rose (he/him)
Principal Data Engineer, Data Services | CA Office of Data and Innovation
(916) 931-8386 | @.***
Follow us: web https://innovation.ca.gov/ | blog
https://innovation.ca.gov/blog/ | twitter
https://twitter.com/californiaodi
On Tue, Mar 12, 2024 at 4:47 PM Ken Godleski @.***>
wrote:

@.**** commented on this pull request.
In
transform/models/intermediate/int_pems__diagnostic_samples_per_station.SQL
https://github.com/cagov/caldata-mdsa-caltrans-pems/pull/96#discussion_r1522270276
:

   DATE(sample_date) = DATEADD(year, -1, CURRENT_DATE())

   and TO_TIME(sample_timestamp) >= '05:00:00'

   and TO_TIME(sample_timestamp) <= '21:59:59'

I was able to remove the DATE cast from sample_date but since
sample_timestamp includes the date and time I used the TO_TIME cast to
extract only the time to do the comparison in the where clause. I tried
building the file without to_date on the sample_timestamp and ran into an
error.
—
Reply to this email directly, view it on GitHub
https://github.com/cagov/caldata-mdsa-caltrans-pems/pull/96#discussion_r1522270276,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABLWQN4V5A64HS33XHIP6K3YX6H2XAVCNFSM6AAAAABEAQ2QO2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSMZSHE2DCNRTG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Before I ran into the ymllint issue I was thinking of combining stg_pems__set_assignment and stg_pems__det_diag_threshold_station as CTE's in a single int_ file. Does that seem like a worthwhile effort based on your review?

Sure, I tend to like having fewer models rather than more, especially if you don't envision them being used separately

commented out incrementality for now until I resolve #120

I didn't believe green check marks existed anymore and then I saw two 🥲 @ian-r-rose @kengodleskidot

Alrighty, this is ready, who wants to do the honors? @kengodleskidot @ian-r-rose


Querybase for Android

Hi David,
Do you, by chance, have plans to create an Android version of Querybase?
Regards,
Tsadok

@tsadoklf @LOG-TAG Unfortunately not at the time. But I do think it's a great idea :)


Azure File CSI Driver Operator is GA in OCP 4.11

Epic: https://issues.redhat.com/browse/STOR-736
CC @openshift/storage

/lgtm

/approve

/test verify


Projects: Project counts should be updated and associate to search bar

When we use the search bar to filter out projects, the project counts should be updated and associated to the searched projects

The counter is just the JavaScript code within that projects.html file that counts the number of elements with the specific class (or id). That should theoretically take care of itself once the search bar is implemented.

I can take this on. I deleted a previous comment as I was wrong on how the search bar was implemented

Found  results for
this line of code already returns the number of search results

opensource-project\static\js\projects.js
this is the file where we could alter the JS code to make it so that we can always display the count as indicated with the code provided


[AMBARI-14714] Fix some unit tests

What changes were proposed in this pull request?
  Most of the remaining ones are due to missing cluster settings.
How was this patch tested?
Performed blueprint deployment.
Ran unit tests.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2594/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2597/
Test FAILed.
Test FAILured.


Syntax Highlighting Part II

Use span[data-code-token="comment"] * {all: inherit;} to disable string styling within comments (A byproduct of parsing strings first).
Use span[data-code-token="line-number"] {user-select:none;} to prevent numbers from being selected with the rest of the code.


Creating repo with repo name containing '-' using ftxui_template breaks code

In ftxui_template, repository name is used for changing project name in CMakeLists.txt, configured_files/config.hpp.in, src/main.cpp files. template_janitor.yml is literally replaces namespace name to repo name. If repo name contains hypens or any other characters which are not allowed as part of identifiers. it breaks the code.
Please add a note regarding this issue, this will save time for people, Thank you.

Fixed in https://github.com/cpp-best-practices/cpp_boilerplate_project/pull/32 will be upstreamed here shortly

Fixed with #12

Thank you @lefticus, sorry for the late reply.


Add requests retry mechanism

Sometimes a segment download fails, which crashes the whole download.
I added a retry mechanism with requests.Session(), based on this article

Thanks again, really nice and quick solution.


Fixing browser fallback

The typescript definitions mention that the package exports AbortController as object https://github.com/mysticatea/abort-controller/blob/a935d38e09eb95d6b633a8c42fcceec9969e7b05/dist/abort-controller.d.ts#L43
This PR makes sure that this is also the case for commonjs implementations.

Note: this is a duplicate of #22. I proposed this PR to offer a different implementation, but it differs only by aesthetics.


[Known Issue] Error: Failed to resolve: com.twitter.sdk.android:twitter:2.0.0 when syncing with Gradle

If you are upgrading a project to version 1.0.0 you may encounter the error Failed to resolve: com.twitter.sdk.android:twitter:2.0.0 when syncing your project with Gradle. Version 1.0.0 has added a new required configuration step. To resolve this issue you must add the Fabric repository to your repositories:
repositories {
    // ...
    maven { url 'https://maven.fabric.io/public' }
}

Adding this to readme will be better.

@kirtan403 we have it in the auth README.  You don't need this repository addition if you're only using database or storage.

@samtstern  Ohh.. I didn't checked the auth one. I checked the main one. Thanks for pointing that out. 😄 Is it really needed even if I do not use twitter authentication?

@kirtan403 it's needed just to get gradle to be happy since the Twitter Android SDK is listed as a transitive dependency in the pom.xml.  If you use ProGuard on your application this should not have a large size impact.
We are looking for ways in later versions to remove this friction.

I have added fabric repository to build.gradle(App) as -
repositories {
    mavenLocal()
    flatDir {
        dirs 'libs'
    }
    maven { url 'https://maven.fabric.io/public' }
}

Getting Error -

Error:Could not download gson.jar (com.google.code.gson:gson:2.6.1): No cached version available for offline mode

I added gson dependency but didn't work.
@amandle  @samtstern Please Suggest.
@kirtan403 did your issue resolved by adding fabric repo only ?

@jay-thakur see this stack overflow post on how to turn off offline mode: http://stackoverflow.com/q/28712025/4548500
And of course make sure you are online.

@SUPERCILEX thanks. Worked.

Even adding the reference the problem remains guys (same as reported in the issue title). I can compile only in the 0.6.2 version.
My gradle:
(...)     repositories {         jcenter()         maven { url 'https://maven.fabric.io/public' }     } (...)
I'm syncing with the online configuration.
Thanks!

I had the same problem in 1.0.0 and solved it.
I have an android app module with an android library module (authentication).
The authentication module has dependency on firebaseUI. I received the error above and solved it by adding the repositories to both build.gradle files. Not sure why this works, but I hope it helps:
app/build.gradle:
android {

  repositories {
    maven { url 'https://maven.fabric.io/public' }
  }
...

authentication/build.gradle:
android {

  repositories {
    maven { url 'https://maven.fabric.io/public' }
  }
...

Yeah @GaborPeto,  i have two modules and the problem is solved now when i just put the repository in the app\build.gradle
Before i was trying to put the declaration on build.gradle general file common to all sub-projects/modules
There is no need for me to repeat the repository in the two build.gradle files for both of modules.
Anyway thanks very mutch GaborPeto for the advice!

I'm pretty sure I tested that the I got tge error if I didn't put the repositories into both build.grdale files. But if you're saying it is working I might just double check it as I really dont like the fact that I have to duplicate that piece of config.

So just an update on my previous comment:
Removing the config
repositories {
    maven { url 'https://maven.fabric.io/public' }
  }

from either of app/build.gradle or authentication/build.gradle results in Failed to resolve: com.twitter.sdk.android:twitter:2.0.0 exception.

@GaborPeto Add repository to allprojects as well.
// Top-level build file where you can add configuration options common to all sub-projects/modules.
buildscript {
repositories {
jcenter()
maven { url 'https://maven.fabric.io/public' }
}
dependencies {
classpath 'com.android.tools.build:gradle:2.2.2'
classpath 'com.google.gms:google-services:3.0.0'
    // NOTE: Do not place your application dependencies here; they belong
    // in the individual module build.gradle files
}

}
allprojects {
repositories {
jcenter()
maven { url 'https://maven.fabric.io/public' }
}
}
task clean(type: Delete) {
delete rootProject.buildDir
}

Thank @isurum, that works too

@yawarli Check the first post.
Add this line:
maven { url 'https://maven.fabric.io/public' }
inside repositories in the screenshot you posted.

Same Issue :)

@yawarali, put it in your allprojects repositories. 😄

@yawarali  You already have repositories section (just above the error). Just add that single line after jcenter().

@kirtan403 I did as you said. Please see second screenshot.
@SUPERCILEX  what is "allprojects"? Where is it?

@yawarali:

@SUPERCILEX I was just creating it in paint! Thanks 😄

@kirtan403 I have a secret trick for being so fast: the "Snipping Tool". It's the best thing ever! You can quickly select an area to take a screenshot of and then Ctrl+C and Ctrl+V it directly into a post. It's way faster than PrtScr->paste into paint->crop->select all->copy 'n paste. Anyway, just me being nerdy. 😄

@SUPERCILEX haha! I didn't knew it can be pasted directly here! I though I can just drag the jpg/png here! Now windows ink tool with screen sketch is also good! Thanks for the tip 😃

Hahaha guys new error
Error:Execution failed for task ':app:processDebugManifest'.

Manifest merger failed : uses-sdk:minSdkVersion 15 cannot be smaller than version 16 declared in library [com.firebaseui:firebase-ui-auth:1.0.1] C:\Users\Qaisar Aslam Mayo\AndroidStudioProjects\FirebaseDemo\app\build\intermediates\exploded-aar\com.firebaseui\firebase-ui-auth\1.0.1\AndroidManifest.xml
Suggestion: use tools:overrideLibrary="com.firebase.ui.auth" to force usage

FirebaseUI only support minSdk 16 so you have to go to your app build.gradle and set minSdkVersion 16.

@kirtan403 @SUPERCILEX  Thanks guys all issue are resolved. This is my first experience, real time communication.

@yawarali Your welcome 👍  Enjoy firebase 😄

Glad to hear that! Have fun! 😄

In my case com.twitter.sdk.android:twitter:2.0.0 was solved adding the code below into app/gradle and upgrading minSdkVersion to 16:   android {     repositories {             maven { url 'https://maven.fabric.io/public' }     } }
Thanks! :) @GaborPeto @SUPERCILEX
P.S. Sorry for the format, I have never posted an issue before :(

just an update in case this is still confusing some:
Adding the dependency to the allprojects section did not work, I had to add it to the android section in the same module that had the firebase-auth-ui dependency.

@samtstern Since 1.1.0 has been released, do you think this issue can be closed now? It was mostly intended as an upgrade guide for 1.0.0 which is now outdated.

I think given that it's in the Auth README we can close this issue.  It's not "fixed" but the workaround is documented.

V1.0.0 is working without issues but dependencies not resolved for all versions after it.

I found the solution:
1- in project build.gradle add fabric repository in one place as below:
allprojects {
    repositories {
        jcenter()
        maven { url 'https://maven.fabric.io/public' }
    }
}

2- in project manifest file make sure to change rtl support to false as it conflict with firebase-ui:
    <application
        android:icon="@mipmap/ic_launcher"
        android:label="@string/app_name"
        **android:supportsRtl="false"**
        android:theme="@style/AppTheme">
        <activity android:name=".MainActivity">
            <intent-filter>
                <action android:name="android.intent.action.MAIN" />
                <category android:name="android.intent.category.LAUNCHER" />
            </intent-filter>
        </activity>
    </application>

I followed the guide in the README for version 1.2.0 adding the needed dependency and fabric repository
and ran into thease errors here after Gradle sync..
Error:(3) Error retrieving parent for item: No resource found that matches the given name 'android:TextAppearance.Material.Widget.Button.Borderless.Colored'.
Error:(4) Error retrieving parent for item: No resource found that matches the given name 'android:TextAppearance.Material.Widget.Button.Colored'.
Error:(3) Error retrieving parent for item: No resource found that matches the given name 'android:TextAppearance.Material.Widget.Button.Borderless.Colored'.
Error:(4) Error retrieving parent for item: No resource found that matches the given name 'android:TextAppearance.Material.Widget.Button.Colored'.
Is there a documented workaround for errors in this area?

@BrownEli that sounds like something different, please file a new issue

@BrownEli  I am getting the same error, how did you solve it?

I just used a newer version on the library.
On May 22, 2017 23:32, "Mohammed Faisal"<EMAIL_ADDRESS>wrote:

@BrownEli https://github.com/browneli I am getting the same error, how
did you solve it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/firebase/FirebaseUI-Android/issues/392#issuecomment-303212161,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANFFcS802kvX5xBCxqBM2LmbKtQ521dDks5r8fDvgaJpZM4KqEBo
.

@BrownEli Thanks and here by 'library' you mean firebase auth ui. Right???

I am having this issue now !! How to fix this problem

My guess is because, firebaseui is also also adding the facebook app id,
you need to replace what the library is doing.
This should work! Let me know if it helped.

On Aug 31, 2017 12:40, "nabeelnazir163"<EMAIL_ADDRESS>wrote:

[image: help]
https://user-images.githubusercontent.com/30865060/29917075-33d39ff0-8e5a-11e7-97fb-142e878239e5.png
I am having this issue now !! How to fix this problem
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/firebase/FirebaseUI-Android/issues/392#issuecomment-326244749,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANFFcZAKMlv1nSU7sm1CnASS7jx-ZRnhks5sdn-CgaJpZM4KqEBo
.


Failed to execute 'pipeTo' on 'ReadableStream'

~/labs via  v18.1.0 via 🐍 v2.7.16 via 💎 v3.1.0
❯ deno run -A --no-check --allow-net --unsafely-ignore-certificate-errors https://raw.githubusercontent.com/lucacasonato/fresh/main/init.ts my-app
DANGER: TLS certificate validation is disabled for all hostnames
The target directory is not empty (files could get overwritten). Do you want to continue anyway? [y/N] y
error: Uncaught (in promise) TypeError: Failed to execute 'pipeTo' on 'ReadableStream': Argument 1 is not of type WritableStream.
  await raw.pipeTo(proc.stdin.writable);
            ^
    at makeException (deno:ext/webidl/00_webidl.js:79:12)
    at Array.WritableStream (deno:ext/webidl/00_webidl.js:866:15)
    at ReadableStream.pipeTo (deno:ext/web/06_streams.js:4285:41)
    at generate (https://raw.githubusercontent.com/lucacasonato/fresh/main/src/dev/mod.ts:127:13)
    at https://raw.githubusercontent.com/lucacasonato/fresh/main/init.ts:218:7

Mh, what is your Deno version? You need to be on the latest release. You can upgrade with deno upgrade.

Ah, righto.
Also, I am in a guarded env, so:
❯ deno task start --unsafely-ignore-certificate-errors
Warning deno task is unstable and may drastically change in the future
Task start deno run -A --watch=static/,routes/,islands/ --no-check dev.ts "--unsafely-ignore-certificate-errors"
Watcher Process started.
Download https://raw.githubusercontent.com/lucacasonato/fresh/main/dev.ts
Sending fatal alert BadCertificate
error: error sending request for url (https://raw.githubusercontent.com/lucacasonato/fresh/main/dev.ts): error trying to connect: invalid peer certificate contents: invalid peer certificate: UnknownIssuer
    at file:///Users/hhegadehallimadh/labs/my-app/dev.ts:3:17
Watcher Process finished. Restarting on file change...

Is there an ENV VAR that I can set to skip cert checks?

DENO_TLS_CA_STORE=system will cause Deno to use your system CA certs. Alternatively put the --unsafely-ignore-certificate-errors before the dev.ts in the "start" task in deno task.

Closing because of non response.


Unable to pass translated string as param to translate pipe?

In the readme example you will find this:

TranslatePipe
You can call the TranslatePipe with some optional parameters that will be transpolated into the translation for the given key.
Example:
Say {{ 'HELLO' | translate:{value: "world"} }}
With the given translation: "HELLO": "hello {{value}}".

If we assume that the key Hello will be translated, and we also assume we have English and Spanish languages, this will result in the text:
Hello World
Hola World
The second of which is incorrect, the ideal end result would be Hola Mundo. A real world example of this is for example:
"{field} Is a required field" where I need {field} to be reference for English or referencia for Spanish.
How would you achieve this? I have tried piping another translate in as the param value but this obviously passes the resulting string to the original translate which then doesn't get updated on further language changes. e.g.
{{ 'Forms.Validation.Required' | translate : { field: 'Users.Reference' | translate } }}
Ideally id like to pass a key in as the param value rather than a string?

+1 for this

This is the same feature requested here: https://github.com/ocombe/ng2-translate/issues/147
I'll close this one as a duplicate, you can follow in the other issue :)

Not sure if I'm understanding correctly but I don't think that solves the issue? I need to pass in a variable translation to a key. E.g. "{Reference} is a required field" where the word reference is variable and requires translation, and the rest of the string also requires translation.
From the look of the linked documentation that only allows you to give the same key different namespaces.


How to install your package ?

Hello, I try to install your package but I don't success.
Context

Vue3 (3.2.47)
Vite (3.1.2)
Node (18.16.3)

Steps

yarn add sandpack-vue3
Add the dependancy into main.ts (import SanpackPlugin from "sandpack-vue3"; + app.use(SanpackPlugin());)
Add into types of tsconfig.json : "sandpack-vue3/global"

Error
Internal server error: Failed to resolve import "sandpack-vue3" from "src/main.ts". Does the file exist?

Thank you

can you provide a github repo？

My bad, it was an issue with my monorepo.


[SPARK-44249][SQL][PYTHON] Refactor PythonUDTFRunner to send its return type separately

What changes were proposed in this pull request?
Refactors PythonUDTFRunner to send its return type separately.
Why are the changes needed?
The return type of Python UDTF doesn't need to be included in the Python "command" because PythonUDTF knows the return type. It can send the return type separately.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Updated the related tests and existing tests.

cc @allisonwang-db @zhengruifeng @HyukjinKwon

Merged to master.


Add the possibility of showing data decoded on signTypedData

Hey all <3
I'm using signTypedData method, and I"m receiving this metamask prompt:

That's ok and it's expected. But for the user singing the TX, it is not easy to know what they are signing (because the data property is encoded in bytes). Would be amazing if we can show that data property decoded. Something like:
{
   To: 0x00000000000
   Value: 0
   Data: encodedData
   DecodedData: ContractFunction(param1, param2, param3)
}

This is a feature request for Metamask, not safe sdk


Setting up automatically my L515 without using Intel Realsense Viewer

Required Info

Camera Model
{ L515 }

Firmware Version
(<IP_ADDRESS>)

Operating System & Version
{Win (10)}

Platform
PC

SDK Version
{  2 }

Language
{C++ }

Segment
{others }

Issue Description
Hi everyone,
I'm using my L515 camera in a C++ code, right now in order to set it up I open Intel Realsense Viewer and set it there on "short range", but I would like to automatically have it set this way through my code, I looked up at the librealsenseSDK library but I really don't understand how to use JSON file with it. I assumed that saving the config into a JSON file (only once on Intel Realsense Viewer) and then using a certain method would work, yet after reading and trying for a couple of hours I don't know how to set up a rs2::device according to my camera and how to import the informations from my JSON file to my code.
Could anyone provide any help please,
best regards

Hi @CheCamCheCam  The Visual Preset can be defined in C++ code instead of a preset.  I hope that https://github.com/IntelRealSense/librealsense/issues/9071#issuecomment-878552273 will be a helpful reference for implementing the L515 Short Range preset.

Hi, thank you for the link, I think it works as I managed to implement the lines in a way that doesn't cause errors and run the code but I wonder if there is a way to see if the modifications were made ? In any case thanks a lot for the help !

Thank you very much !

Hi @CheCamCheCam  Do you require further assistance with this case, please?  Thanks!


how to use mdb file to trian?

--train_nips path_to_dataset
--train_cvpr path_to_dataset
--valroot path_to_dataset
when i entry the baidupan:i found all are lock.mdb data.mdb,sorry i don not know how to use them,plz tell some?

Yes

Hi~ Please use the folder root (containing data.mdb and lock.mdb) of the dataset as path_to_dataset.

Hi~ Please use the folder root (containing data.mdb and lock.mdb) of the dataset as path_to_dataset.

thanks. what is for validation dataset? you mentioned here the datasets svt_p_645, svt_647, IIIT5K_3000,ic15_2077 and some other datasets for validation. Do we have to combine these datasets and produce one lmdb file for all of them or choose one of these datasets for validation?
thanks

Either way, as you like~


Ability to set an arbitrary ownCloud URL

As a web app developer, I want my users to be able to save data in their own ownCloud instances, so that they remain in control of their data and I do not need to provide storage for these users.

As a user of a given web app, I want to save my data/files into my ownCloud, so that they remain in my control.
(similar to "save to Dropbox")

Currently the web app developer must specify a static ownCloud URL in the config file, so it implies that there is an existing instance that is designed to be used with that web app.
These user stories imply more than just being able to select a target folder. It would require the webapp backend to also have a way to keep connected to the ownCloud instance and access the data there.
@LukasHirt @pmaier1

I am thinking in this case how the authorisation would be handled. Would it then be something like with iOS and android - there would be predefined endpoints which could be used?

OIDC js client will be removed and we'll be relying solely on getting bearer token via prop. This feature is not going to be possible with this approach.

OIDC js client will be removed and we'll be relying solely on getting bearer token via prop.

Does is make sense to keep both approaches?
a) File picker can do its own authentication
b) File picker can receive tokens from 3rd party
I think there are use cases for both (not necessarily the one requested in this ticket).

@pmaier1 It would be possible. I just understood from our last call that wouldn't be the goal. We can simply pass a check - if the bearer token is not provided, fall back to authorisation.

I like that idea. While it's currently not necessary to specify arbitrary oC URLs, it might still be needed to have a dedicated authentication for the file picker.


Failed assertion: boolean expression must not be null

Steps to Reproduce

Run flutter create bug.
Update the files as follows: ...
...

Expected results:
Actual results:

Logs

So how do I solve this? Pls
On Thu, May 21, 2020, 11:47 PM Jonah Williams<EMAIL_ADDRESS>wrote:

Closed #57762 https://github.com/flutter/flutter/issues/57762.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/57762#event-3362604144, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AMYBSOMZ2CVG3IBWWXOYXA3RSWVRZANCNFSM4NHHYJ5A
.

The Github issue tracker is for bugs and features requests for the Flutter SDK. Help and support questions like these are better suited for Q&A sites like StackOverflow or on one of the flutter community sites at https://flutter.dev/community .
Also -you posted an issue with literally zero content in the body. How could anyone possibly help you?

I have solved it
On Thu, May 21, 2020, 11:58 PM Jonah Williams<EMAIL_ADDRESS>wrote:

The Github issue tracker is for bugs and features requests for the Flutter
SDK. Help and support questions like these are better suited for Q&A sites
like StackOverflow https://stackoverflow.com/questions/tagged/flutter
or on one of the flutter community sites at https://flutter.dev/community
.
Also -you posted an issue with literally zero content in the body. How
could anyone possibly help you?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/57762#issuecomment-632387394,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMYBSOL3DPDQSA56YISVRDDRSWWXTANCNFSM4NHHYJ5A
.


Rename _validate_model_analysis_input_parameters() to _validate_rai_insights_input_parameters()

Description
Seems like this function was not renamed when RAIInsights class was created out of ModelAnalysis class
Checklist

[ ] I have added screenshots above for all UI changes.
[ ] Documentation was updated if it was needed.
[ ] New tests were added or changes were manually verified.

Codecov Report

Merging #1373 (b78f5ba) into main (05ab54f) will increase coverage by 13.69%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             main     #1373       +/-   ##
============================================
+ Coverage   86.30%   100.00%   +13.69%
============================================
  Files         100         8       -92
  Lines        4702        64     -4638
============================================
- Hits         4058        64     -3994
+ Misses        644         0      -644

Flag
Coverage Δ

unittests
100.00% <ø> (+13.69%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ai/responsibleai/rai_insights/rai_base_insights.py

...sibleai/responsibleai/rai_insights/rai_insights.py

.../responsibleai/_managers/error_analysis_manager.py

raiwidgets/raiwidgets/error_analysis_dashboard.py

...sibleai/responsibleai/_tools/shared/base_result.py

erroranalysis/erroranalysis/__init__.py

...nsibleai/responsibleai/_managers/causal_manager.py

...onsibleai/responsibleai/serialization_utilities.py

...ponsibleai/responsibleai/modelanalysis/__init__.py

erroranalysis/erroranalysis/_internal/metrics.py

... and 82 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 05ab54f...b78f5ba. Read the comment docs.


CDC table missing features

TODO
Please note down the follow-up
Originally posted by @fuyufjh in https://github.com/risingwavelabs/risingwave/issues/16513#issuecomment-2099641444

after https://github.com/risingwavelabs/risingwave/issues/15267


Is there a memory leak when running the webserver module ?

I am running  phantomjs as a web server. Here is my server-side code :
var system = require('system');
var server = require('webserver').create();
var port = system.args[1];
var listening = server.listen(port, function(request, response) {
var url = request['post'].url;
fire(url, request, response);
})
var fire = function(url, request, response) {
var page = require('webpage').create();
page.onLoadFinished = function(status) {
page.close();
response.close();
}
page.open(url);

}
In case concurrent requests, I create a new page for each request, and invoke page.close() to release the memory.
For test, I send different urls one by one (not concurrently) to the server,
and monitor the memory usage.
I found that the memory usage keeps increasing, and never drops even after I stopped sending requests.
does it indicate a memory leak, or just because I wrote the code improperly ?
thanks !

@quanghoc release() has been marked deprecated. Does this issue still exist?


Return map of all supported language codes and translations

Create a function T9n.getLanguages() that returns a map of language codes and translations:
[
  {name: "English", code: "en"},
  {name: "Italian", code: "it"},
  {name: "ro", code: "ro"},
]

T9n.getLanguages() return a language list, but I found some problems:

packages.js defined: DEFAULT_LANGUAGES, it's no 'hu' or else.
T9n.getLanguages() list that the zh-cn should be zh_cn, but I must use T9n.setLanguage('zh-cn') to change Chinese Language. For example: T9n.setLanguage('hu') /T9n.setLanguage('zh_cn') is no effect on client, but still can use T9n.setLanguage('en')/('de')/('vi')/('zh-cn')/....
I m confused for that if I want to make Login Interface with International supported.

It sounds like your working with an older version of t9n. Can you please check that your version of package.js is 1.1.0.

Yes，it's softwarerero:accounts-t9n 1.1.0. And I confused zh_tw is ok, zh_cn can't , but zh-cn is correct in there. And I can't find bug about section of "@registerMap" or "@replaceParams".

I see what happened: 1.1.0 was released on 3rd of April and Luca fixed this on the 15th of April. I did a new release 1.1.1 now, please give it at try.

With version 1.2.0 comes T9n.getLanguageInfo() which returns a structure like above.
Off course I could only provide translations for some languages, so all contributors are invited to a string for t9Name.


Allow generation of all functions in modules that appear in more than one metadata input

Fixes #1201

/azp run

/azp run


The package isn't available via PyPI service

Hi
Did you add your package to PyPI? It can't be installed from there

TorIpChanger is not on PyPi. You can install it with pip directly from GitHub, see https://stackoverflow.com/q/20101834/4183498 for more details.

https://pypi.org/project/toripchanger/


Customize time limit other than 1 sec.

Hello, I like this crate and its concise output very much!
Why don't you add customizable time limit version of bench() and bench_env()?
My idea is to add the following functions and to use the last argument to specify the limit.

bench_limit()
bench_env_limit()

I'm experimenting to bench my crate https://crates.io/crates/wavelet-matrix using easybench like below:
WaveletMatrix::new(), N = 1000000, values <<PHONE_NUMBER>:  Only generated 0 sample(s) - we can't fit a regression line to that! Try making your benchmark faster.
WaveletMatrix::new(), N = 100000, values <<PHONE_NUMBER>:    180559443 ns   (R²=0.997, 6 iterations in 5 samples)
WaveletMatrix::new(), N = 10000, values <<PHONE_NUMBER>:      13169803 ns   (R²=0.977, 78 iterations in 22 samples)
WaveletMatrix::new(), N = 1000, values <<PHONE_NUMBER>:        1345688 ns   (R²=0.982, 789 iterations in 45 samples)

This sounds like a reasonable addition.
Perhaps the best solution is to just run the benchmark until the R2 is high enough, up to some very large time limit. What do you think?

I'm kind of opposed to the approach where you auto-increase the time limit searching for better R2.
The reason includes:

I like the handiness and quickness of 1 sec very much.
On my note PC, the clock speed seems to be reduced when CPU gets too hot in the longer iterations. When you increase the execution time, you will get more worse R2 values.

I've folked easy-bench and implement bench_limit() and got the results like below:
    WaveletMatrix::new(), N = 10000000, 16-bit values:<PHONE_NUMBER> ns   (R²=1.977, 8 iterations in 6 samples)
(snip)
    WaveletMatrix::new(), N = 10000000, 32-bit values:<PHONE_NUMBER>0 ns   (R²=14.193, 8 iterations in 6 samples)
(snip)
    WaveletMatrix::new(), N = 10000000, 64-bit values:<PHONE_NUMBER>4 ns   (R²=68.467, 8 iterations in 6 samples)

from: https://github.com/sekineh/wavelet-matrix-rs/blob/77212c01c0eb445bc531dcd9dd8f8718cd1c73a0/BENCH.md
At last I gave up using the longer time limit.
Currently I'm just using one-time iteration for longer duration measurement.  The hand-written routine produced more predictable results in my use case for benchmark that takes > 1 sec.
    use std::time::*;
    let time_start = Instant::now();
    let wm = WaveletMatrix::new(&vec);
    let dur = time_start.elapsed();
    let dur_ns = dur.as_secs() as f64 * 1e9 + dur.subsec_nanos() as f64;
    println!(
        "{:>24}, N = {}, {}: {:>10} ns   (only 1 iteration)",
        "WaveletMatrix::new()",
        num,
        desc,
        dur_ns,
    );

    WaveletMatrix::new(), N = 10000000, 16-bit values:<PHONE_NUMBER> ns   (only 1 iteration)
(snip)
    WaveletMatrix::new(), N = 10000000, 32-bit values:<PHONE_NUMBER> ns   (only 1 iteration)
(snip)
    WaveletMatrix::new(), N = 10000000, 64-bit values:<PHONE_NUMBER>8 ns   (only 1 iteration)

from: https://github.com/sekineh/wavelet-matrix-rs/blob/ea7ac8112647c7195066e74f35b5981999f94364/BENCH.md
My PC is Panasonic Let's note CF-MX3 with Core i5 4310U which looks like ultra-low voltage.
The Xeon or other desktop-class CPUs may work in a different way, though.

I notice you're getting R² values greater than 1. This looks like a bug in easybench - I'll try to reproduce it.

Actually I used my folked version of easybench.
https://github.com/sekineh/easybench-rs
I made two trivial commits on it.
So I might have introduced the bug (R2 > 1) in my repository only.

What about decreasing the minimum amount of time and increasing the maximum based on R2? That way it would usually run tests even more quickly, but could take longer on slower tests.

I think that’s what I had in mind in https://github.com/asayers/easybench-rs/issues/1#issuecomment-350602671, but I’m a bit unsure about it now. One complication is that when there are only 2 samples the R² is always 1, so we’d have to take at least 3 before using it.


Masked Errors

Errors from api responses seem to get masked; though I've not deeply investigated it, it seems the yardarm client returns only the api documented error response.
For example, if the api spec shows the following example documented error code:
                "example": {
                  "errorCode": "400.",
                  "messages": [
                    "Bad Request"
                  ]
                }

No matter what the actual 400 response is (bad id/missing field/etc.), the yardarm client will only return the documented example string, which in the above example is Bad Request

I believe this was covered in my response to #10


Fix/issue 47

@fblundun - can you take a look at this one please?

Sure!

@chuwy everything looks good to me!

Hey @chuwy - sorry I got confused, I thought this was a release PR not a ticket PR. Can you put together a PR for the next release? It's fine to push back tickets that are not going to be done soon...

@alexanderdean what about #38? Do I need to implement it? Currently in 0.4.0 we have only two issues implemented (one of them in no way critical for r79).

It would be good to do #38 and its sibling #44 - thanks @chuwy !


Update domains.json

stellar.org.za and accountviewer.stellar.org.za are a phishing sites attempting to collect stellar secret keys

Thank you for the PR!
accountviewer.stellar.org.za
Fake Stellar site phishing for secrets
https://urlscan.io/result/80cd8a60-1b76-483d-9f01-23fb222388c8/
https://urlscan.io/result/73c228f8-4e07-4952-8364-0654ae228d7b/


[Bug]: Game crashes when starting to make 1.16.5 new world

General Info

[X] I am running a modpack
[ ] I can reproduce this issue consistently in single-player
[ ] I can reproduce this issue consistently in multi-player
[X] I have searched for this issue previously and it was either (1) not previously reported, or (2) previously fixed and I am having the same problem.
[X] I am crashing and can provide my crash report(s)
[X] I am using the latest version of the mod

Forge version
forge-36.2.18
Mod version
CamoCreepers-1.16.5-1.1.0
Java version
1.8.0_51, Oracle Corporation
Issue Description
When I went to start a single player world from the main menu, instead of pulling up the world creation screen the game lagged briefly (around 1-2 minutes) before crashing.
Additional Information
crash-2021-12-05_12.23.49-client.txt

I fixed this issue in 1.18 and have backported the fix to both 1.17.1 and 1.16.5 :)

Yay thank you :)


⚠ Arch Linux Package Mirror has degraded performance

In 4ac937d, Arch Linux Package Mirror (https://pkg.fef.moe/archlinux/) experienced degraded performance:

HTTP code: 200
Response time: 1135 ms

Resolved: Arch Linux Package Mirror performance has improved in 69c90f6 after 22 minutes.


Dynamic loading of locales

Nice work,  however when dealing with unknown locales, say based on the browser settings of users in a Web application,  it would make sense to dynamically load locale data from a Json files right?

Hello @nkev
I think I understand what you asking about dynamically loading locales, but don't really see any benefit loading from JSON, it's just an extra step.
eg. without JSON
local.New() --> use locale
vs with JSON
Define struct(with exposed fields!) --> Unmarshalled json into struct --> use locale
I used to have a map auto generated with a mapping of locale string to the New() function which would probably be what you're looking for without the JSON, I could add it back in if you'd like, but there are some down sides:
eg. using the map, because it references all the locales, causes all locales to be compiled into your application whether you will ever use them or not and increases compile time by a bit.
locales is really just a helper package though and is intended to be wrapped by either the user or another package as it provides all of the information for building an i18n package, but doesn't actually provide functionality for storing translations. I created an i18n package universal-translator that you regsiter all your applications supported locales, avoiding unnecessary locales from being compiled in, and even a fallback locale for unsupported ones and it has functions for selecting the locale based on ones you've extracted from the browser's headers(I may add another helper function for extracting the headers for the user soon).
Sorry for being so long winded, please let me know if this addresses you question about dynamically loading locales.


fix: mapping nnul crud operator to notNull

Mapping of nnul crud operator to notNull in strapi-v4 is missing

Thank you @npanti ⚡


Implement UTF8 byteLength for browsers

With this we do not have to allocated 4x too much memory when
creating strings.
Fixes #1859

Closing this. I'll add a new one with encode and decode.


PSM-393 Upgrade com.coveo:fmt-maven-plugin 2.9 -> 2.10

This PR contains the following updates:

Package
Update
Change

com.coveo:fmt-maven-plugin
minor
2.9 -> 2.10

Release Notes

coveo/fmt-maven-plugin
v2.10
Compare Source
v2.9.1
Compare Source

Renovate configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

Suggested commit message:
PSM-393 Upgrade fmt-maven-plugin 2.9 -> 2.10

See:
- https://github.com/coveooss/fmt-maven-plugin/releases/tag/2.9.1
- https://github.com/coveo/fmt-maven-plugin/compare/2.9.0...2.9.1

Suggested commit message:
PSM-393 Upgrade fmt-maven-plugin 2.9 -> 2.10

See https://github.com/coveooss/fmt-maven-plugin/compare/2.9.0...2.10.0

Github releases and version in Maven central do not match. 😞

This version packages a version of GJF which requires Java 11, while for OSS code we still retain Java 8 compatibility. Looks like we should go with 2.9.1 for now. (Or require Java 11 for building and Java 8 only for build artifact compatibility. Will look into that.)

Downgraded to 2.9.1. Suggested commit message:
Upgrade fmt-maven-plugin 2.9 -> 2.9.1 (#9)

See https://github.com/coveooss/fmt-maven-plugin/compare/2.9.0...2.9.1


Enrich events with cloud/company information

Problem
We use jistu for sending telemetry events for our cli/framework. It would be nice to know, which cloud providers our users use the most, along with information about a company, which can be known from IP address, like geo fields.
Solution
There is a library which does it https://github.com/bcoe/which-cloud. And maybe maxmind also have those databases, but I didn't check.

Now Jitsu can pull company data from MaxMind DB. Here's the: https://jitsu.com/docs/other-features/geo-data-resolution


feat(source): goldenmangas

Closes #174
Checklist:

[ ] Updated source's version for individual source changes
[ ] Updated all sources' versions for template changes
[x] Set appropriate nsfw value
[ ] Did not change id even if a source's name or language were changed
[x] Tested the modifications by running it on the simulator or a test device

delete Cargo.lock and recompile to fix building

source doesn't appear to work after fixing building. cleaning up old prs, but feel free to fix and reopen or re-pr.


(PUP-9914) Fixed ParsedFile behaviour for records

Also:

remove whitespaces in: lib/puppet/transaction/report.rb
do an extra check in: lib/puppet/type/exec.rb:576

CLA signed by all contributors.


Bug: The line counting in the code is breaking

If unformatted code is inserted into the editor, the line counting breaks. This happens because the line length exceeds the container's width.
Lexical version: 0.17.0
Steps To Reproduce

Paste some unformatted code in lexical editor
Resize screen

Link to code example:

As a quick solution, you can use CSS to address this issue. Here's a possible approach:

But then we face the issue of losing the menu in the top right corner.
The solution will have to be:
1 Recalculating lines when the screen is resized.
2 Allow wrapping code in a container.

I will try to fix this issue