Can't seem to get read-only mode to work on MoonRiver.

Hi there!
I'm not able to get useDapp's read-only mode working on MoonRiver. The same code works fine on other networks, and the RPC I'm using works fine as well. Would appreciate any input.
ReactDOM.render(
  <React.StrictMode>
    <DAppProvider config={{readOnlyChainId: ChainId.Moonriver, readOnlyUrls: {[ChainId.Moonriver]: 'https://moonriver.api.onfinality.io/public/'}}}>
      <ApolloProvider client={client}>
        <App />
      </ApolloProvider>
    </DAppProvider>
  </React.StrictMode>,
  document.getElementById('root')
);

Tested on Polygon and had no issues, and the dapp works fine with a wallet connected. When connected to MoonRiver, I get the active state coming back as true, but the useContractCalls and useTokenBalance hooks just return undefined.
EDIT: Error Details. Tried both the purestake RPC (shown) and the OnFinality RPC.

Hey! I tested this out with Moonriver to see if I could reproduce the issue, but I wasn't able to reproduce it. It seems to work as it should.
Your config matches what I used, with the exception of using Apollo. I console logged the results from calling both the useTokenBalance and useContractCalls hooks and I do see that undefined (or [undefined] for useContractCalls) is returned while the data is fetching, and then once the data has been retrieved it returns the results I was expecting.
For useContractCalls, I grabbed the WMOVR contract on Moonriver and used the following code snippet to get the result:
 const abi = new ethers.utils.Interface(wmovrAbi)
 const contractCalls = useContractCalls([{ abi, address: "0x98878B06940aE243284CA214f92Bb71a2b032B8A", method: "symbol", args: []}])
 console.log(contractCalls)

The response returned was ['WMOVR'] as expected.
If you want some help debugging the issue you can feel free to respond here, or reach out to me on the Moonbeam discord if it's easier - my username there is also @eshaben

After testing this out further, I was able to reproduce the issue whenever my wallet is not connected. I see the same errors:
"Error: could not detect network (event="noNetwork", code=NETWORK_ERROR, version=providers/5.5.0)
    at Logger.makeError (http://localhost:3000/static/js/bundle.js:8274:19)
    at Logger.throwError (http://localhost:3000/static/js/bundle.js:8284:16)
    at Web3Provider.<anonymous> (http://localhost:3000/static/js/bundle.js:18971:21)
    at Generator.throw (<anonymous>)
    at rejected (http://localhost:3000/static/js/bundle.js:18534:32)"

I traced the error back to an Ethers issue and opened the following PR: https://github.com/ethers-io/ethers.js/pull/2464

I've managed to build a version of @ethersproject/networks with the change from @eshaben. As I was experiencing the same issue with another chain.
However, it didn't seem to resolve the issue with the read-only mode.

This now seems to be working for me as of 0.10.0 :)


The ticker and Current price should refresh only during market hours.

On holidays and off hours, the stock prices should show the last closing price. Perhaps we use moment js to see the day & time and call the functions only if needed.

Also, ticker should only scroll if there are items in your list


Added Vectr, Folio, Plant, Fathom, Heap

Signed-off-by: Paramdeo Singh<EMAIL_ADDRESS>

@oedmarap, many thanks for your contribution!
I knew about Plant but forget to add it. Other tools are new for me 😊

@oedmarap, many thanks for your contribution!
I knew about Plant but forget to add it. Other tools are new for me blush

No probs! Half of the stuff I come across I bookmark and forget, so I find this project to be super neat :smile:


How to run on Linux

Disclaimer: I'm a noob.
Since the prebuilt .exe doesn't do anything except open a blank commandline (which does appear to block some Midi ports or something, but no gui), I've tried running the script on Linux.
First of all, there were three python modules I installed with pip (construct, mido, rtmidi), but now it just returns more obscure errors.
Do I need more modules? Am I running the .exe wrong?
$ python3 zoomzt2-gui.py
Traceback (most recent call last):
  File "/mnt/data/Downloads/B1Xon_v1.30_Win_E/zoom-zt2-prebuilt-0.4.0/zoomzt2-gui.py", line 289, in OnInit
    self.frame = _479389042__675845753_MyFrame(None, wx.ID_ANY, "")
  File "/mnt/data/Downloads/B1Xon_v1.30_Win_E/zoom-zt2-prebuilt-0.4.0/zoomzt2-gui.py", line 57, in __init__
    if self.ConnectPedal():
  File "/mnt/data/Downloads/B1Xon_v1.30_Win_E/zoom-zt2-prebuilt-0.4.0/zoomzt2-gui.py", line 246, in ConnectPedal
    if self.pedal.connect():
  File "/mnt/data/Downloads/B1Xon_v1.30_Win_E/zoom-zt2-prebuilt-0.4.0/zoomzt2.py", line 115, in connect
    for port in mido.get_input_names():
  File "/home/bernie/.local/lib/python3.9/site-packages/mido/backends/backend.py", line 169, in get_input_names
    devices = self._get_devices(**self._add_api(kwargs))
  File "/home/bernie/.local/lib/python3.9/site-packages/mido/backends/backend.py", line 163, in _get_devices
    return self.module.get_devices(**self._add_api(kwargs))
  File "/home/bernie/.local/lib/python3.9/site-packages/mido/backends/rtmidi.py", line 50, in get_devices
    rtapi = _get_api_id(api)
  File "/home/bernie/.local/lib/python3.9/site-packages/mido/backends/rtmidi.py", line 34, in _get_api_id
    return rtmidi.API_UNSPECIFIED
AttributeError: module 'rtmidi' has no attribute 'API_UNSPECIFIED'
OnInit returned false, exiting...

okay progress:
rtmidi is the wrong package, I needed python3-rtmidi, which I installed via apt, instead of pip.
This does not produce errors when starting zoomzt2-gui.py, but instead just does nothing.
No gui is shown.

More progress:
you need the wx-common apt package.
The program just fails silently and does nothing when the package is absent, instead of producing any meaningful error.
Had to find that out by painfully scraping through code.
Now it shows a gui and appears to connect to the effectpedal.


rT.variables in mg/dL

First part of fix for #29
Check in Trio needed to find where suggestion.isf is used and apply a glucoseformatter to it. I did not find deviation and BGI being transferred to Trio from oref.

Checking on rT.target_bg, there seems to be an error in the calculation of insulinForManualBolushttps://github.com/nightscout/trio-oref/blob/d1dfb7099d59e4f7779e5fc5c75f8ca299d91ef2/lib/determine-basal/determine-basal.js#L1623 as it uses a non converted glucose value eventualBGand a converted glucose value target_bg, which will screw up that value for mmol users. Changing rT.target_bg also into non converted value - it duplicates current_bg though.

successful test documented in Trio PR https://github.com/nightscout/Trio/pull/327


Robustness of move_std

The algorithm used by move_std is prone to numerical errors. A while back I rewrote the implementation of the equivalent function in Pandas to use what some call Welford's method, a one pass stable algorithm, see here for the relevant PR. I took some timings on the Pandas code, and it resulted in a 25% slowdown.
Would you like to see something like that implemented for bottleneck?

Sounds like a good idea to me. But I'd like to hear what others think. Anyone?
bn.nanstd uses a two-pass algorithm. If the bottleneck is memory access, which I believe it is, then perhaps the one-pass algorithm you are proposing is faster. If so, assuming comparable stability, then maybe the place to start is with bn.nanstd?

If you are not frontally opposed to it I'll put a PR together, and we can discuss its merits after it is up and running.

That's great. Is this for move_std or nanstd?

I think it is only clearly advantageous in move_std, the two pass algorithm is typically faster, and probably even more accurate, for a non-moving window.

Indeed, this seems like a nice improvement :+1:.

What should go into the PR? Just the move,pyx file in templates, or the generated C files as well?


Refractor monarch variable names to boost readbility

Previously block dimensions such as q, p are very non-intuitive and make the code hard to understand and build on. In developing my Monarch PEFT project, I renamed them to fix the block-diagonal matmul taxonomy with terms like blk_in, blk_out to clearly indicate block dims.
This could significantly encourage follow-up work using this code base, as folks have complained about hard-to-use code :)

Precision tests passed in my repo https://github.com/SprocketLab/sparse_matrix_fine_tuning/blob/main/tests/ops/test_blockdiag_butterfly_multiply.py

Cool! Has this been tested in this repo in any models?

If you mean convergence tests, not yet--only passed precision tests. Can probably try that in a few days. Thanks!

Thank you! Just some basic curves should be fine — in case there’s some
end-to-end disruption.
On Sun, Dec 15, 2024 at 9:49 AM Wenxuan Tan @.***>
wrote:

If you mean convergence tests, not yet--only passed precision tests. Can
probably try that in a few days. Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/HazyResearch/m2/pull/40#issuecomment-2543974311, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDDIIWBWZV34ZL2JMF4GM32FW6LTAVCNFSM6AAAAABTRAQ73CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBTHE3TIMZRGE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Source dependency resolution should check for changes in a Git repository no more than once per Gradle invocation

For consistency with binary dependency resolution, and to avoid expensive remote operations, source dependency resolution should perform each remote query no more than once per build. This means:

List the versions available in each remote repository once per build and cache the results. Currently this is done (at least) once per selector.
Don't perform a Git update for any working directory. We should check out each revision into its own working directory that is never updated, only garbage collected.

These should also apply when doing multiple resolutions in parallel.

Split the second point to new issue

Fixed in https://github.com/gradle/gradle/pull/6116


Adding ability to specify an alternate temp directory

Adds the ability to specify an alternate temp directory via the ':scm_copy_remote_tmp_dir' in deploy.rb

?

:+1:

Any plans to merge this into master?

Please merge this!

I think this repo is officially dead :(


Add additional material discussing DateVectors and C++ templates?

Hi, thanks so much for writing this book- I've recently been working on my first project with Rcpp and it's been extremely helpful.
There are two areas where I have run into problems in my work that aren't mentioned here: one is the slightly unusual behaviour of DateVectors and the other is the fact that template programming doesn't work as I expected.
Specifically:

DateVectors are not vectors of Dates, but doubles (see related SO question). This caught me out and caused some confusion. I think it would be helpful to clarify this in Chapter 16 where it currently says "Date is a scalar type corresponding to an element of DateVector."
Template programming requires some extra boilerplate in Rcpp, as per this Rcpp Gallery example.

I have some notes put together on the first point, and a very brief paragraph on the second point. If you think these could be good additions I'm happy to put together a PR and you can take a look. If so, let me know where you think this extra info would fit in the book.
Thanks!

Hi @lewinfox
Thank you for your suggestion!
I think these 2 points are valuable to add to the book and I really welcome your kind PR.
As you pointed out DateVector and also DatetimeVector are not Rcpp::Vector of Date or Datetime.
I implicitly express the situation by the word "corresponding to", but I knew nobody would notice my intention. Act\tually, these two classes (NewDateVector, NewDatetimeVector) were not yet implemented at the timing I wrote the chapter.
I think it is good idea to add a section or paragraph at the top of Chapter 16 and 17, where explaining DateVector and DatetimeVector is internally NumericVector and you can use getDate() or getDatetime() if you want a std::vector of Date or Datetime.
Regarding templates, I think it is useful for readers to add a new chapter explaining the macro RCPP_RETURN_VECTOR and RCPP_RETURN_MATRIX. I think the best place for the chapter is next to the Chapter 18 RObject, or it may be good to add the explanation as a new section of Chapter 18.
I'm wondering what is the best name for the chapter/section.
My candidate for now is ...

Creating a function that receives various data types.

I appreciate your comments on my idea and looking forward to seeing your PR.
Thank you again.

Great, OK! I'll put a PR together and we can go from there.

@teuder I've added material on DateVectors here. I also noticed an issue where duplicated section names had caused links in the sidebar to point to the wrong pages, so I fixed that too.
I need to a do a bit more research on the macros as I'm new to C++ and my knowledge is pretty limited. Shall I raise a PR for the changes made already, or would you rather wait until I've written both sections?


Add Qodana configuration file

Add Qodana config at project root dir.
The QodanaGitHub action should hopefully use it.

I am going to merge this to see if it works after the qodana.yml file exists at the project root. My guess is no, in which case I will roll the changes back.


getPayload() should gracefully handle a token with 3 parts that isn't a valid JWT

See #592.

Fixed in d5993f2a5252b7b39fec8909940e9a632a792205.


Cobertura coverage: No coverage data found

Hello,
I'm generating a code coverage Code Coverage Files_100.zip
file under linux using gcovr. However, when during the Publish Code Coverage Results step, the processing of the file fails, sadly without any further information why.
The file is visible on the server, but the results are not displayed in the interface.
I also attached the log content below.
2016-11-04T14:10:28.8768020Z ##[section]Starting: Publish Code Coverage Results $(Build.SourcesDirectory)/coverage-summary.xml
2016-11-04T14:10:28.9258350Z ==============================================================================
2016-11-04T14:10:28.9273410Z Task         : Publish Code Coverage Results
2016-11-04T14:10:28.9287450Z Description  : Publish code coverage results to VSTS/TFS
2016-11-04T14:10:28.9301680Z Version      : 1.0.16
2016-11-04T14:10:28.9315780Z Author       : Microsoft Corporation
2016-11-04T14:10:28.9330020Z Help         : [More Information](https://go.microsoft.com/fwlink/?LinkID=626485)
2016-11-04T14:10:28.9344570Z ==============================================================================
2016-11-04T14:10:29.1653850Z ##[debug]agent.workFolder=/home/build/tfs_agent/_work
2016-11-04T14:10:29.1670380Z ##[debug]loading inputs and endpoints
2016-11-04T14:10:29.1686650Z ##[debug]loading INPUT_CODECOVERAGETOOL
2016-11-04T14:10:29.1702830Z ##[debug]loading INPUT_SUMMARYFILELOCATION
2016-11-04T14:10:29.1717820Z ##[debug]loading INPUT_ADDITIONALCODECOVERAGEFILES
2016-11-04T14:10:29.1733220Z ##[debug]loading ENDPOINT_AUTH_...
2016-11-04T14:10:29.1748220Z ##[debug]loading ENDPOINT_AUTH_SCHEME_...
2016-11-04T14:10:29.1763850Z ##[debug]loading ENDPOINT_AUTH_PARAMETER_..._ACCESSTOKEN
2016-11-04T14:10:29.1778890Z ##[debug]loading ENDPOINT_AUTH_SYSTEMVSSCONNECTION
2016-11-04T14:10:29.1794390Z ##[debug]loading ENDPOINT_AUTH_SCHEME_SYSTEMVSSCONNECTION
2016-11-04T14:10:29.1809330Z ##[debug]loading ENDPOINT_AUTH_PARAMETER_SYSTEMVSSCONNECTION_ACCESSTOKEN
2016-11-04T14:10:29.1824300Z ##[debug]loaded 9
2016-11-04T14:10:29.1839230Z ##[debug]codeCoverageTool=Cobertura
2016-11-04T14:10:29.1855460Z ##[debug]summaryFileLocation=/home/build/tfs_agent/_work/1/s/coverage-summary.xml
2016-11-04T14:10:29.1871410Z ##[debug]reportDirectory=null
2016-11-04T14:10:29.1886600Z ##[debug]additionalCodeCoverageFiles=/home/build/tfs_agent/_work/1/s/code_coverage.xml
2016-11-04T14:10:29.1901860Z ##[debug]codeCoverageTool: Cobertura
2016-11-04T14:10:29.1917070Z ##[debug]summaryFileLocation: /home/build/tfs_agent/_work/1/s/coverage-summary.xml
2016-11-04T14:10:29.1932240Z ##[debug]reportDirectory: null
2016-11-04T14:10:29.1947150Z ##[debug]additionalCodeCoverageFiles: /home/build/tfs_agent/_work/1/s/code_coverage.xml
2016-11-04T14:10:29.2117560Z Reading code coverage summary from '/home/build/tfs_agent/_work/1/s/coverage-summary.xml'
2016-11-04T14:10:29.2230010Z ##[warning]No coverage data found. Check the build errors/warnings for more details.
2016-11-04T14:10:33.9414720Z ##[debug]Processed: ##vso[codecoverage.publish codecoveragetool=Cobertura;summaryfile=/home/build/tfs_agent/_work/1/s/coverage-summary.xml;additionalcodecoveragefiles=/home/build/tfs_agent/_work/1/s/code_coverage.xml;]
2016-11-04T14:10:33.9452250Z ##[section]Async Command Start: Publish code coverage
2016-11-04T14:10:33.9507470Z Modifying Cobertura Index file
2016-11-04T14:10:33.9523450Z Publishing code coverage files to TFS server.
2016-11-04T14:10:33.9538870Z Uploading 1 files
2016-11-04T14:10:33.9553750Z ##[debug]File: '/tmp/Code Coverage Report_100/summary100/coverage-summary.xml' took 859 milliseconds to finish upload
2016-11-04T14:10:33.9568450Z ##[debug]Detail upload trace for file: Code Coverage Report_100/summary100/coverage-summary.xml
2016-11-04T14:10:33.9582920Z ##[debug]File 'Code Coverage Report_100/summary100/coverage-summary.xml' will be uploaded in one chunk.
2016-11-04T14:10:33.9598310Z ##[debug]Attempt '1' for uploading chunk '1' of file 'Code Coverage Report_100/summary100/coverage-summary.xml'.
2016-11-04T14:10:33.9612950Z ##[debug]Generate new HttpRequest for uploading file 'Code Coverage Report_100/summary100/coverage-summary.xml', chunk '1' of '1'.
2016-11-04T14:10:33.9628670Z ##[debug]Start uploading file 'Code Coverage Report_100/summary100/coverage-summary.xml' to server, chunk '1'.
2016-11-04T14:10:33.9643570Z ##[debug]Finished upload chunk '1' of file 'Code Coverage Report_100/summary100/coverage-summary.xml', elapsed 800 (ms), response code 'Created'.
2016-11-04T14:10:33.9659950Z File upload succeed.
2016-11-04T14:10:33.9675630Z Uploading 1 files
2016-11-04T14:10:34.9725610Z Published '/tmp/Code Coverage Report_100' as artifact 'Code Coverage Report_100'
2016-11-04T14:10:34.9740610Z ##[debug]File: '/tmp/Code Coverage Files_100/code_coverage.xml' took 935 milliseconds to finish upload
2016-11-04T14:10:34.9755480Z ##[debug]Detail upload trace for file: Code Coverage Files_100/code_coverage.xml
2016-11-04T14:10:34.9769930Z ##[debug]File 'Code Coverage Files_100/code_coverage.xml' will be uploaded in one chunk.
2016-11-04T14:10:34.9784360Z ##[debug]Attempt '1' for uploading chunk '1' of file 'Code Coverage Files_100/code_coverage.xml'.
2016-11-04T14:10:34.9799050Z ##[debug]Generate new HttpRequest for uploading file 'Code Coverage Files_100/code_coverage.xml', chunk '1' of '1'.
2016-11-04T14:10:34.9813480Z ##[debug]Start uploading file 'Code Coverage Files_100/code_coverage.xml' to server, chunk '1'.
2016-11-04T14:10:34.9828290Z ##[debug]Finished upload chunk '1' of file 'Code Coverage Files_100/code_coverage.xml', elapsed 934 (ms), response code 'Created'.
2016-11-04T14:10:34.9843080Z File upload succeed.
2016-11-04T14:10:35.4658930Z Published '/tmp/Code Coverage Files_100' as artifact 'Code Coverage Files_100'
2016-11-04T14:10:35.4673330Z ##[section]Async Command End: Publish code coverage
2016-11-04T14:10:35.4717530Z ##[section]Finishing: Publish Code Coverage Results $(Build.SourcesDirectory)/coverage-summary.xml

Have checked the file and noticed the issue.
We are looking Lines while the generated XML has lines and for the same reason it's failing.
Will be fixed in next agent release and simple way to fix is to change report to Lines

@dragongame sorry for confusion. I overlooked a bit.
We support only http://cobertura.sourceforge.net/xml/coverage-04.dtd schema
Please upgrade the coverage reporting format to latest (04) one and it will work.

@nigurr thank you for your further looking into this.
Looks like for now we won't be able to have code coverage for c++ files from linux, as the tool we are using seems not maintained (gcovr).
I'll see if I can find a newer one.
Thank you again for you time.

Hi @dragongame we run in the exact same issue, do you find a new tool in order to replace gcovr ?
The latest version is still bound to dtd 0.3 :(

My comments on this;

Changing lines to Lines did not fix the problem for me; same error.
The DTD schema specifies <!ELEMENT method (lines)> i.e. lowercase, so the file I have should be supported.

Can we please reopen this issue?

ok, this thing really does publish this small coverage result, but the MAIN coverage tab is EMPTY. is this expected?

@libPhipp

No, you can have the gcovr generated html report in the coverage tab. For that, you'd have to tell gcovr to produce an html report and you have to tell vsts in the "Publish Code Coverage Results" task where this html report is.

@libPhipp any guidance? we are using:
:gcov:
    :html_report_type: cobertura

and no html files are being generated? when we were using :html_report_type: detailed it was not picking those up. And I cant really put both values in there, cant I?


doc: An alternative to writing toPythonApplication s

The advent of pkgs/by-name hierarchy created an interesting alternative: instead of cluttering all-packages.nix with more lines and more merge conflicts, the creation of toPythonApplication expressions can be made at pkgs/by-name.
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Marking

@infinisil as mastermind of by-name hierarchy
@ShamrockLee as beta-tester at https://github.com/NixOS/nixpkgs/pull/291176

calling it an alternative doesn't seem right, isn't by-name the preferred way of doing things now?

This format provides a more robust way to override dependent python packages, and also make the dependent toPythonPackage overridable.
However, we do hope to maintain a consistent overriding interface. Considering the following scenario:
Let's say that we have pkgs.youtube-dl packaged in this format:
{ python3Packages }:
with python3Packages; toPythonApplication youtube-dl

where python3Packages is python311Packages. If, for some reason, python3Packages.youtube-dl breaks, and the maintainers decide to switch to python310Packages.youtube-dl, should we add a line in all-packages.nix and specify youtube-dl = callPackage pkgs/by-name/yo/youtube-dl/package.nix { python3Packages = python310Packages; }, or should we just change the argument from python3Packages to python310Packages?

i don't actually know python, so i'm not much help here, but if such a fundamental issue as "how do we package for different python versions" isn't solved yet, then it sounds like this packaging method isn't ready.

BTW, why do we prefer python3*Packages over python3*.pkgs while we have to override it by overriding python3* and then access python3*.pkgs? After the adoption of this pattern, we'll need to do such kind of overriding often, and it's necessary to have a sensible way of overriding.

i don't actually know python, so i'm not much help here, but if such a fundamental issue as "how do we package for different python versions" isn't solved yet, then it sounds like this packaging method isn't ready.

I personally prefer the second approach, i.e. pin the version in all-packages.nix and remove the line to un-pin it. My point is that when it comes to the choice between keeping overriding interface consistent and avoiding all-packages.nix, the overriding interface consistency should win.

I think using toPythonApplication like this is great! The only thing to watch out for is whether this causes a change in the .override interface when you switch from a toPythonApplication in all-packages.nix to one in package.nix. I believe it does, so to prevent breakages you'd have to only use this for new packages or so.

where python3Packages is python311Packages. If, for some reason, python3Packages.youtube-dl breaks, and the maintainers decide to switch to python310Packages.youtube-dl, should we add a line in all-packages.nix and specify youtube-dl = callPackage pkgs/by-name/yo/youtube-dl/package.nix { python3Packages = python310Packages; }, or should we just change the argument from python3Packages to python310Packages?

This is indeed a concern, and I agree with your conclusion:

I personally prefer the second approach, i.e. pin the version in all-packages.nix and remove the line to un-pin it. My point is that when it comes to the choice between keeping overriding interface consistent and avoiding all-packages.nix, the overriding interface consistency wins.

This is also what is recommended in pkgs/by-name/README.md.

calling it an alternative doesn't seem right, isn't by-name the preferred way of doing things now?

toPythonApplication cals are usually inlined at all-packages.nix.

Ops, my failure!

Ops, my failure!

(Discussed in the docs team meeting today)

calling it an alternative doesn't seem right, isn't by-name the preferred way of doing things now?

toPythonApplication cals are currently inlined at all-packages.nix.

I think it's fine for something to be recommended even if older instances don't use the pattern (yet).

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/2024-03-14-documentation-team-meeting-notes-113/41462/1

@DanielSidhion please review again.

Some minor suggestions here:

We should probably avoid prone-to-irrelevance naming like "legacy method" and "new method",

My intention is to remove the legacy reference as soon as possible (right after converting the code), however it requires some testing.

especially for document anchors, which are hard to change without breaking hyperlinks.

Good point.
What do you think about removing the session marks and rewording?

My laptop is awaiting repair, and I might be slow to response until setting up the machine I borrowed. Thank you for your patience.

Some minor suggestions here
We should probably avoid prone-to-irrelevance naming like "legacy method" and "new method",

My intention is to remove the legacy reference as soon as possible (right after converting the code), however it requires some testing.

Does that mean there will be a bulk migration from the legacy reference approach to the new one?

Does that mean there will be a bulk migration from the legacy reference approach to the new one?

Yes:
https://github.com/NixOS/nixpkgs/pull/295499

The new "note about migration" looks great!
Do we need to remind people to follow the "changing the implicit default value" guide of name-based package directory when using a specific Python version?

Closing to clean up.
I will take the reverse approach: clean the code first.


Lag Spikes [1.17.1]

After downloading this mod, I experienced huge lag spikes during gameplay. It happens more or less often based on the amount of mods downloaded, and the amount of world discovered. No cause is seen in the logs. It only occurs in the 0.11.0-pre version.

Code of MC itself in 1.17 is different from 1.16, most code of BE is same except some adaptations, so this is mostly issue of MC itself.
What you can try to do:

Limit entities spawn to lower values (there are mods that allows this)
Install optimisation mods (like sodium, phosphor, lithium, hydrogen, ferrite or others)
Increase you memory limits (as you lags related to amount of mods or worlds this means that you probably just don't have enough memory for all calculations)
Try different JVM tags

We are trying to optimise BetterEnd as much as we can - parallel calculations and loading, smarter variable usages, code refactoring, but we can work only in MC limits, they are not too wide and we already did most things. So, try to follow steps above, probably they will help you.


Handle pattern parentheses in FormatPattern

Summary
This PR fixes the duplicate-parenthesis problem that's visible in the tests from https://github.com/astral-sh/ruff/pull/6799. The issue is that we might have parentheses around the entire match-case pattern, like in (1) here:
match foo:
    case (1):
        y = 0

In this case, the inner expression (1) will think it's parenthesized, but we'll also detect the parentheses at the case level -- so they get rendered by the case, then again by the expression. Instead, if we detect parentheses at the case level, we can force-off the parentheses for the pattern using a design similar to the way we handle parentheses on expressions.
Closes https://github.com/astral-sh/ruff/issues/6753.
Test Plan
cargo test

Does this address https://github.com/astral-sh/ruff/issues/6753 ? If not, what's missing?

Yes I believe it should (sorry, I missed that issue, I stumbled upon this independently). Added a test + linked the issue.

I still need to figure out the parenthesize_if_expands, working on that now.

Okay @MichaReiser I think this is ready for review based on your feedback.


Warn of file replacement while exporting project

Describe the solution you'd like
When you are exporting a project file, and the name matches an existing file in the selected folder, warn the user that the previous file will be lost.
Describe alternatives you've considered

A pop-up window with the warning message
Warning message below the output file name

Please provide more details of how to reproduce your issue. Any screenshots or video would be very useful.

Export DBeaver project.

Then set a name:

@MontejoJorge Thank you for the idea.

I would like to try to get with this, how do you think I should warn the user? pop up or warning message?

We already have pop-ups warns about overriding in some other wizards (for example, in data transfer). Let's make a pop-up here too to be consistent.


[Kogito-8223]Introduce a docker registry for the Container builder tests with Docker and Podman

See:https://issues.redhat.com/browse/KOGITO-8223
Signed-off-by: desmax74<EMAIL_ADDRESS>
Description of the change:
Motivation for the change:
Checklist

[ ] Add or Modify a unit test for your change
[ ] Have you verified that tall the tests are passing?

May I ask what is the use case here ? I cannot see any description in issue or PR ...

Thank you !


Adding strings or

I mentioned this in #31 but this deserves an issue of its own
You can make our version of alidade say anything like this.
http://<IP_ADDRESS>/project/slide/I can make this say anything gives this

But it also renders CSS. Like this.
http://<IP_ADDRESS>/project/slide/<style>* {background-color: blue;}</style>
This could just look bad. But could also load 3rd party data by loading background images as url
Inserting

There may be quite a lot of scope for XSS, since the templating layer does not enforce escaping based on origin of content.  I think the 404 sprinkling may help, but this could be a longer piece of work.  We can make sure that the user-supplied content in the forms is safe as part of one of the other issues.

Most of the functions that are responsible for rendering user data have been switched over to inline or file-based twig templates.  The main slide template is still raw PHP, but this is probably going to be converted to allow better sidebox handling.

Form controls are now rendered by template blocks, which implements proper output escaping.


Mark default address

Set default: true for new identity addresses. i.e. in the curly brackets here:
https://github.com/praekelt/seed-jsbox-utils/blob/develop/lib/identity_store.js#L73

@gsvr please review

👍


feat: debugger package implementation

Proposed Changes

Splitting and reorganizing node utils from algokit-utils-ts

@neilcampbell - ready for review.
Will remove the tar archive before final merge - installing from github would have required a lot more effort.
See the latest on utils-ts https://github.com/algorandfoundation/algokit-utils-ts/pull/315 for related changes


[SPARK-12666][CORE] SparkSubmit packages fix for when 'default' conf doesn't exist in dependent module

What changes were proposed in this pull request?
Adding a "(runtime)" to the dependency configuration will set a fallback configuration to be used if the requested one is not found.  E.g. with the setting "default(runtime)", Ivy will look for the conf "default" in the module ivy file and if not found will look for the conf "runtime".  This can help with the case when using "sbt publishLocal" which does not write a "default" conf in the published ivy.xml file.
How was this patch tested?
used spark-submit with --packages option for a package published locally with no default conf, and a package resolved from Maven central.

Test build #59701 has started for PR 13428 at commit 27d0c21.

Test build #59701 has finished for PR 13428 at commit 27d0c21.

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

Merged build finished. Test PASSed.

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

Test build #59779 has started for PR 13428 at commit ece86f1.

Using the wildcard for the fallback conf "default(*)" has the side effect of resolving transitive dependencies from all scopes - so it will grab test jars, etc
Would it be more appropriate to use "default(runtime)" instead?  It has the tradeoff of being less flexible but I tested using spark-csv publishLocal and it only used the required jars.
cc @JoshRosen

@BryanCutler Am I dreaming or is this for real. I've been trying to solve this issue for a very long time. I honestly want to give you a high five now :)
default(runtime) makes sense to me. Some jars don't have sources, javadocs, etc. and I wouldn't want things to fail in that case. Pulling in all test dependencies is also useless.

Haha, no problem, I'll take a virtual high-five :P
I agree default(runtime) does make more sense,  I'll change that up.

Test build #59786 has started for PR 13428 at commit b87e0c9.

Test build #59779 has finished for PR 13428 at commit ece86f1.

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

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

Merged build finished. Test PASSed.

Test build #59786 has finished for PR 13428 at commit b87e0c9.

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

Merged build finished. Test PASSed.

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

Hi @brkyvz , did you ever get a chance to try this out and see if it works for you?  I imagine you probably have been pretty slammed with the 2.0 release..

LGTM. @JoshRosen Can we merge this to 2.0.0? This is a very nice fix that I couldn't figure out for a long time. It's amazing because it's so simple

Test build #62673 has started for PR 13428 at commit b87e0c9.

Test build #62673 has finished for PR 13428 at commit b87e0c9.

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

Merged build finished. Test PASSed.

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

ping @JoshRosen , would you mind taking a look at this?  Thanks!

Two suggestions, otherwise I'm not familiar enough with Ivy - I'll trust you guys to know what you're doing. :-)

Modulo the minor review comments already left on this PR, this looks good to me as well. Is there any risk in putting this into 2.0.1 / 2.1.0? Will this change the resolution behavior for any cases which didn't fail before this patch?

Test build #63812 has started for PR 13428 at commit df1ac92.

Test build #63812 has finished for PR 13428 at commit df1ac92.

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

Merged build finished. Test PASSed.

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

Thanks for the review @JoshRosen, I made the requested changes and tested it out once more.  I think it is low risk because it is pretty well isolated to this particular issue and only improves on how it was before.

ping @JoshRosen, I think this should be good to go now

LGTM; I'll merge when I get home tonight.

Merged to master and branch-2.0.

Thanks @JoshRosen, @brkyvz and @vanzin!


Update libStorage dependency to 0.5.2

This patch updates the libStorage dependency to 0.5.2.

Codecov Report

Merging #789 into release/0.8.2 will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           release/0.8.2    #789   +/-   ##
=============================================
  Coverage           7.41%   7.41%
=============================================
  Files                 21      21
  Lines               2590    2590
=============================================
  Hits                 192     192
  Misses              2381    2381
  Partials              17      17

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 738db67...cce9118. Read the comment docs.


cannot able to set break points in jadx [gui]

Issue details
unable to set break points in jadx gui , so i pulled the recent code and gradlew compiled it still it didnt work, if i double click/left click/right click, only line numbers are vanishing/not vanishing , cannot able to set break points
https://github.com/skylot/jadx/assets/135871751/d5c94fe6-461d-4abc-898e-3c452c340c8d
Jadx version
latest version
Java version
18.0.2
OS

[ ] Windows
[x] Linux
[ ] macOS

it worked , i forgot to select show dalvik code option


Error: received handshake message out of context

Hi! I'm getting "Error: received handshake message out of context" in /var/log/openvas/gsad.log after startup.
logs.txt
Do you have any ideas about it?

Same problem here...

Hmm how much memory do you have available? I've seen things like this when you're limiting docker's memory

I see the same error. I've configured Docker for Mac with 12 GB of memory.

@lancelakey can you provide your docker command to start the container and any logs?

I solved the problem, i've stopped the gsad service and ran it manually like so:
/usr/sbin/gsad --listen=<IP_ADDRESS> --port=4000 --mlisten=<IP_ADDRESS> --mport=9390
@mikesplain is there any chance you'll change the Dockerfile to support that?

@udiromano Can you try the new build once https://github.com/mikesplain/openvas-docker/pull/109 finishes building? That should fix it!

Closing due to inactivity. Please reopen if you're still seeing issues!

Same on the latest. Details in logs.
Logs and creation: https://gist.github.com/lexfrei/4d8f4b994eadf886720fb3d479908d5d
Container inspect https://gist.github.com/lexfrei/483a26da33cf2c46c8fdc3dd8d75031b

@mikesplain I can't reopen this issue. Can you reopen it for me plz?

Are you still able to connect to gsad or is the connection refused? Which browser do you use when accessing gsad? The problem should be on the client side of gsad aka your browser see https://lists.gnu.org/archive/html/libmicrohttpd/2011-09/msg00003.html

curl: (52) Empty reply from server
✗: 52 @ Wed Mar  7 18:51:16 MSK 2018```
Same in latest Safari and Chrome

My bet is gnutls handles some tls stuff more strictly then others.

Worked as expected: there is no http handling on https port. It's not friendly, but correct.

Hi,
I'm getting this error but when running this image behind a reverse proxy like traefik.
Maybe it has to do with the fact that the proxy is terminating ssl ? I'm using letsencrypt certs in the traefik proxy, so there's no need to run openvas with ssl enabled. How can I disable https on this image to see if that's the issue?
thanks/

@juanluisbaptiste you need to pass --http-only as a startup parameter to GSAD
I am not sure where to set it in this image.
https://www.mankier.com/8/gsad


Overflow on absolute comparison of signed integers

fn main() {
    approx::abs_diff_ne!(std::i32::MAX, std::i32::MIN);
}

Expected behavior: Runs fine.
Actual behavior: thread 'main' panicked at 'attempt to subtract with overflow'.

More corner case:
approx::abs_diff_ne!(-1, std::i32::MAX)

won't overflow on subtraction but will be overflow on abs

Oh huh, that's interesting. We actually implement these comparisons for integers?

Right, seems like it's here: https://github.com/brendanzab/approx/blob/0527afee5faf712e877634ccf4b9790cac5a1d63/src/abs_diff_eq.rs#L35-L87


center of vehicle frame

the data here is 001/segment-10750135302241325253_180_000_200_000_with_camera_labels.tfrecord
According to the docs, the box center is in vehicle frame. E.g., in this photo, the center of the box is
(12.425728132752083, -4.773993024558877, 2.878885135010904)
so this box is about 12m forward, 4.7m right and 2.9m upward.
The y and z might be correct. But this box doesn't look like 12m forward... where is the center of the vehicle frame?
I have also calculated the box center location in the camera frame, the result is
[[ 4.70110318] [-0.70939735] [10.91130435]]
Looks like the camera center is about 1.5m from the vehicle frame's center, is this correct?

the center is in the middle of the SDC's rear axle.  It is better to check the validity of the label in BEV or 3D view?

hi @peisun1115
thanks for your answer. I have checked using a 3D point cloud viewer and I found you are right. It's easier to see from the top.
I have another question, I found that in each data frame, the camera calibration data is the same. There are many photos in one data frame, from my understanding, the extrinsic matrix of the camera may be different in different photos. However, I use this camera calibration data to drawing the bounding box and found the position is correct. How could this happen?

They are different for different cameras. Check the camera name in this CameraCalibration proto

But for one same camera, e.g., the front center camera, does the extrinsic matrix should be different for the different photo?

extrinsics is fixed for each camera in a scene. it represents the location of the camera in vehicle frame. it does not change

@peisun1115 Thanks for the explanation!


Prevent execution of tasks on an already shutdown EL

Per #1383 , crash if a task is attempted on a closed Event Loop
One note: I created some test cases here, but wasn't able to figure out how to integrate them into IntegrationTests but perhaps you can point me in the right direction and I'll add them.
Modifications:
Using externalState to determine if a SelectableEventLoop is open, and added a computed property to the EventLoop protocol so that the default method implementations can implement this behavior (and it may be useful to query this information from user code).
Result:
see test cases

@Lukasa @weissi Alrighty, made some updates to this one, lmk what you think.

Ok cool, I think I like this. I'd like to see what @weissi thinks.

Awesome! Happy to make updates if needed.

@swift-nio-bot test this please

CC @Lukasa

@swift-nio-bot test this please


unterminated string literal

in write_fantasy_novel
chapter_titles = ast.literal_eval(storyline)
Certainly, here's a more polished and expanded version of the storyline for "$$$," presented in the same list of dictionaries format as requested:
^
SyntaxError: unterminated string literal (detected at line 1)

Also getting this error:
`
Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/IPython/core/interactiveshell.py", line 3553, in run_code
exec(code_obj, self.user_global_ns, self.user_ns)
File "", line 5, in <cell line: 5>
novel, title, chapters, chapter_titles = write_fantasy_novel(prompt, num_chapters, writing_style, claude_true)
File "", line 266, in write_fantasy_novel
chapter_titles = ast.literal_eval(storyline)
File "/usr/lib/python3.10/ast.py", line 64, in literal_eval
node_or_string = parse(node_or_string.lstrip(" \t"), mode='eval')
File "/usr/lib/python3.10/ast.py", line 50, in parse
return compile(source, filename, mode, flags,
File "", line 21
This redesigned odyssey transports the reader through a gauntlet of growing suspense, emotional crescendos, heart-stopping showdowns, and the characters' relentless battle against the forces of darkness.
^
SyntaxError: unterminated string literal (detected at line 21)`


Document HAS of shaped CArray

The https://docs.raku.org/language/nativecall#Embedding_CStructs_and_CUnions_with_HAS should be expanded to include inline arrays within Struct/Union.  The example in the Rakudo tests is:
class InlinedArrayInStruct is repr('CStruct') {
    has int32 $.a is rw;
    HAS int32 @.b[3] is CArray;
    has int32 $.c is rw;
}

It's slightly non-obvious and it always takes looking at the test suite to remind myself how to do it.

Thanks for the report. Related #2965


Remove some dependencies

Fixes #140

Alright, thanks for the information, and thanks for taking over as well!


[WIP][RELAY][QUANTIZATION] automatic data-driven calibration and per-channel scales

Support for features described in #2651. Currently this PR focuses mainly on vision models where there is a well known dataset (ImageNet).

I hit a max_pool2D realize issue with this IR:
fn (%data: Tensor[(1, 3, 224, 224), float32]) -> Tensor[(1, 1000), float32] {
  %0 = relay.op.annotation.simulated_quantize(%data, meta[relay.Constant][0], -127f, 127f, kind=1, passthrough=1, granularity="channel", layout="NCHW")
  %1 = relay.op.annotation.simulated_quantize(meta[relay.Constant][1] /* ty=Tensor[(64, 3, 7, 7), float32] */ /* ty=Tensor[(64, 3, 7, 7), float32] */, meta[relay.Constant][2], -127f, 127f, kind=2, granularity="channel", layout="OIHW")
  %2 = nn.conv2d(%0, %1, strides=[2, 2], padding=[3, 3], channels=64, kernel_size=[7, 7])
  %3 = relay.op.annotation.simulated_quantize(meta[relay.Constant][3] /* ty=Tensor[(64, 1, 1), float32] */ /* ty=Tensor[(64, 1, 1), float32] */, meta[relay.Constant][4], -127f, 127f, kind=2, granularity="channel", layout="NCHW", op_hint="broadcastable_add")
  %4 = add(%2, %3)
  %5 = nn.relu(%4)
  %6 = relay.op.annotation.simulated_quantize(%5, meta[relay.Constant][5], -127f, 127f, kind=1, passthrough=1, granularity="channel", layout="NCHW")
  %7 = nn.max_pool2d(%6, pool_size=[3, 3], strides=[2, 2], padding=[1, 1])
  %8 = relay.op.annotation.simulated_quantize(%7, meta[relay.Constant][6], -127f, 127f, kind=1, passthrough=1, granularity="channel", layout="NCHW", op_hint="nn.max_pool2d")
  %9 = relay.op.annotation.simulated_quantize(meta[relay.Constant][7] /* ty=Tensor[(64, 1, 1), float32] */ /* ty=Tensor[(64, 1, 1), float32] */, meta[relay.Constant][8], -127f, 127f, kind=2, granularity="channel", layout="NCHW", op_hint="broadcastable_mul")
  %10 = multiply(%8, %9)

When realize %10, %8 is int8 while %9 is int32, so realize will fail.

I hit a max_pool2D realize issue with this IR:
fn (%data: Tensor[(1, 3, 224, 224), float32]) -> Tensor[(1, 1000), float32] {
  %0 = relay.op.annotation.simulated_quantize(%data, meta[relay.Constant][0], -127f, 127f, kind=1, passthrough=1, granularity="channel", layout="NCHW")
  %1 = relay.op.annotation.simulated_quantize(meta[relay.Constant][1] /* ty=Tensor[(64, 3, 7, 7), float32] */ /* ty=Tensor[(64, 3, 7, 7), float32] */, meta[relay.Constant][2], -127f, 127f, kind=2, granularity="channel", layout="OIHW")
  %2 = nn.conv2d(%0, %1, strides=[2, 2], padding=[3, 3], channels=64, kernel_size=[7, 7])
  %3 = relay.op.annotation.simulated_quantize(meta[relay.Constant][3] /* ty=Tensor[(64, 1, 1), float32] */ /* ty=Tensor[(64, 1, 1), float32] */, meta[relay.Constant][4], -127f, 127f, kind=2, granularity="channel", layout="NCHW", op_hint="broadcastable_add")
  %4 = add(%2, %3)
  %5 = nn.relu(%4)
  %6 = relay.op.annotation.simulated_quantize(%5, meta[relay.Constant][5], -127f, 127f, kind=1, passthrough=1, granularity="channel", layout="NCHW")
  %7 = nn.max_pool2d(%6, pool_size=[3, 3], strides=[2, 2], padding=[1, 1])
  %8 = relay.op.annotation.simulated_quantize(%7, meta[relay.Constant][6], -127f, 127f, kind=1, passthrough=1, granularity="channel", layout="NCHW", op_hint="nn.max_pool2d")
  %9 = relay.op.annotation.simulated_quantize(meta[relay.Constant][7] /* ty=Tensor[(64, 1, 1), float32] */ /* ty=Tensor[(64, 1, 1), float32] */, meta[relay.Constant][8], -127f, 127f, kind=2, granularity="channel", layout="NCHW", op_hint="broadcastable_mul")
  %10 = multiply(%8, %9)

When realize %10, %8 is int8 while %9 is int32, so realize will fail.

Do you know why %9 is int32 in this case? It looks like its (KIND) is weight, which we usually set to int8 quantization.

I hit a max_pool2D realize issue with this IR:
fn (%data: Tensor[(1, 3, 224, 224), float32]) -> Tensor[(1, 1000), float32] {
  %0 = relay.op.annotation.simulated_quantize(%data, meta[relay.Constant][0], -127f, 127f, kind=1, passthrough=1, granularity="channel", layout="NCHW")
  %1 = relay.op.annotation.simulated_quantize(meta[relay.Constant][1] /* ty=Tensor[(64, 3, 7, 7), float32] */ /* ty=Tensor[(64, 3, 7, 7), float32] */, meta[relay.Constant][2], -127f, 127f, kind=2, granularity="channel", layout="OIHW")
  %2 = nn.conv2d(%0, %1, strides=[2, 2], padding=[3, 3], channels=64, kernel_size=[7, 7])
  %3 = relay.op.annotation.simulated_quantize(meta[relay.Constant][3] /* ty=Tensor[(64, 1, 1), float32] */ /* ty=Tensor[(64, 1, 1), float32] */, meta[relay.Constant][4], -127f, 127f, kind=2, granularity="channel", layout="NCHW", op_hint="broadcastable_add")
  %4 = add(%2, %3)
  %5 = nn.relu(%4)
  %6 = relay.op.annotation.simulated_quantize(%5, meta[relay.Constant][5], -127f, 127f, kind=1, passthrough=1, granularity="channel", layout="NCHW")
  %7 = nn.max_pool2d(%6, pool_size=[3, 3], strides=[2, 2], padding=[1, 1])
  %8 = relay.op.annotation.simulated_quantize(%7, meta[relay.Constant][6], -127f, 127f, kind=1, passthrough=1, granularity="channel", layout="NCHW", op_hint="nn.max_pool2d")
  %9 = relay.op.annotation.simulated_quantize(meta[relay.Constant][7] /* ty=Tensor[(64, 1, 1), float32] */ /* ty=Tensor[(64, 1, 1), float32] */, meta[relay.Constant][8], -127f, 127f, kind=2, granularity="channel", layout="NCHW", op_hint="broadcastable_mul")
  %10 = multiply(%8, %9)

When realize %10, %8 is int8 while %9 is int32, so realize will fail.

Do you know why %9 is int32 in this case? It looks like its (KIND) is weight, which we usually set to int8 quantization.

I am also trying to debug it and no clue yet.
It will be great that we can solve it together.
Reproduce steps:

Download pretrained model from: https://dl.fbaipublicfiles.com/octconv/ablation/a02_resnet-26_alpha-0.500.tar
untar
loading code:

def load(prefix, epoch=0):
  sym, arg_params, aux_params = \
    mx.model.load_checkpoint(prefix, 0)
  net, params = relay.frontend.from_mxnet(symbol=sym,
                          shape={"data": (1, 3, 224, 224)},
                          arg_params=arg_params,
                          aux_params=aux_params)
  return net, params

net, params = load("./model/a02_resnet-26_alpha-0.500/checkpoint-0", 0)

I hit a max_pool2D realize issue with this IR:
fn (%data: Tensor[(1, 3, 224, 224), float32]) -> Tensor[(1, 1000), float32] {
  %0 = relay.op.annotation.simulated_quantize(%data, meta[relay.Constant][0], -127f, 127f, kind=1, passthrough=1, granularity="channel", layout="NCHW")
  %1 = relay.op.annotation.simulated_quantize(meta[relay.Constant][1] /* ty=Tensor[(64, 3, 7, 7), float32] */ /* ty=Tensor[(64, 3, 7, 7), float32] */, meta[relay.Constant][2], -127f, 127f, kind=2, granularity="channel", layout="OIHW")
  %2 = nn.conv2d(%0, %1, strides=[2, 2], padding=[3, 3], channels=64, kernel_size=[7, 7])
  %3 = relay.op.annotation.simulated_quantize(meta[relay.Constant][3] /* ty=Tensor[(64, 1, 1), float32] */ /* ty=Tensor[(64, 1, 1), float32] */, meta[relay.Constant][4], -127f, 127f, kind=2, granularity="channel", layout="NCHW", op_hint="broadcastable_add")
  %4 = add(%2, %3)
  %5 = nn.relu(%4)
  %6 = relay.op.annotation.simulated_quantize(%5, meta[relay.Constant][5], -127f, 127f, kind=1, passthrough=1, granularity="channel", layout="NCHW")
  %7 = nn.max_pool2d(%6, pool_size=[3, 3], strides=[2, 2], padding=[1, 1])
  %8 = relay.op.annotation.simulated_quantize(%7, meta[relay.Constant][6], -127f, 127f, kind=1, passthrough=1, granularity="channel", layout="NCHW", op_hint="nn.max_pool2d")
  %9 = relay.op.annotation.simulated_quantize(meta[relay.Constant][7] /* ty=Tensor[(64, 1, 1), float32] */ /* ty=Tensor[(64, 1, 1), float32] */, meta[relay.Constant][8], -127f, 127f, kind=2, granularity="channel", layout="NCHW", op_hint="broadcastable_mul")
  %10 = multiply(%8, %9)

When realize %10, %8 is int8 while %9 is int32, so realize will fail.

Do you know why %9 is int32 in this case? It looks like its (KIND) is weight, which we usually set to int8 quantization.

I am also trying to debug it and no clue yet. I double checked %9 is float32, but %8 is int8, so it failed in the check at quantize.cc:715 (dtype is int32)
    if (lhs->dtype == Float(32)) {
      ldata = Cast(ldata, dtype);
    } else {
      CHECK_EQ(lhs->dtype, dtype);
    }

It will be great that we can solve it together.
Reproduce steps:

Download pretrained model from: https://dl.fbaipublicfiles.com/octconv/ablation/a02_resnet-26_alpha-0.500.tar
untar
loading code:

def load(prefix, epoch=0):
  sym, arg_params, aux_params = \
    mx.model.load_checkpoint(prefix, 0)
  net, params = relay.frontend.from_mxnet(symbol=sym,
                          shape={"data": (1, 3, 224, 224)},
                          arg_params=arg_params,
                          aux_params=aux_params)
  return net, params

net, params = load("./model/a02_resnet-26_alpha-0.500/checkpoint-0", 0)

Thanks to @ajtulloch finding a solution to this bug.
Problem:  when a bn follows with max_pool, MulRealize will fail to handle int8 output from max_pool
diff --git a/src/relay/pass/quantize.cc b/src/relay/pass/quantize.cc
index 3a2e54c8..4059dc3a 100644
--- a/src/relay/pass/quantize.cc
+++ b/src/relay/pass/quantize.cc
@@ -340,18 +340,9 @@ Expr MulRealize(const Call& ref_call,
     const auto* rhs = new_args[1].as<QRealizeIntExprNode>();
     Expr ldata = lhs->data;
     Expr rdata = rhs->data;
-
     DataType dtype = cfg->dtype_activation;
-    if (lhs->dtype == Float(32)) {
-      ldata = Cast(ldata, dtype);
-    } else {
-      CHECK_EQ(lhs->dtype, dtype);
-    }
-    if (rhs->dtype == Float(32)) {
-      rdata = Cast(rdata, dtype);
-    } else {
-      CHECK_EQ(rhs->dtype, dtype);
-    }
+    ldata = Cast(ldata, dtype);
+    rdata = Cast(rdata, dtype);

     Expr ret = ForwardOp(ref_call, {ldata, rdata});
     Expr dom_scale = FoldConstant(Multiply(lhs->dom_scale, rhs->dom_scale));

In order to make quantized model running with AVX512, we have to set data_type to uint8. Currently the type_solver is not able to handle this: int32 = conv(uint8, int8)
A dirty way to make the pipeline running is:
diff --git a/src/relay/pass/type_solver.cc b/src/relay/pass/type_solver.cc
index 84f72e0d..c083f1e1 100644
--- a/src/relay/pass/type_solver.cc
+++ b/src/relay/pass/type_solver.cc
@@ -6,9 +6,9 @@
  * to you under the Apache License, Version 2.0 (the
  * "License"); you may not use this file except in compliance
  * with the License.  You may obtain a copy of the License at
- *
+ *
  *   http://www.apache.org/licenses/LICENSE-2.0
- *
+ *
  * Unless required by applicable law or agreed to in writing,
  * software distributed under the License is distributed on an
  * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
@@ -116,6 +116,9 @@ class TypeSolver::Unifier : public TypeFunctor<Type(const Type&, const Type&)> {
       return lhs->resolved_type;
     } else {
       Type resolved = this->VisitType(lhs->resolved_type, rhs->resolved_type);
+      if (!resolved.defined()) {
+        return rhs->resolved_type;
+      }
       CHECK(resolved.defined())
         << "Unable to unify parent types: "
         << lhs->resolved_type << " and " << rhs->resolved_type;

We have to make type_solver working in clean way to handle quantized type solver.

Given the importance of quantization, it would be great if we can work to merge one version of the PR before we iterate.
@eqy can you work to get one version of the PR ready so we can merge it soon. Then @vinx13 @antinucleon @ZihengJiang can help iterate.
Thanks

Hi @tqchen @eqy @ZihengJiang  @vinx13 ,  I tried this PR for channel quantization. It works in resnet(according to the code of @antinucleon as before). But it failed on my own detection model, which works on master or my old version. the error infos as below. Do you have any idea about this error?  Under what cases will cause this error? help you can provide some clues. Thanks.
[11:58:41] /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/src/relay/pass/pass_manager.cc:377: Executing module pass : InferType with opt level: 0

[11:58:41] /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/src/relay/pass/pass_manager.cc:377: Executing module pass : InferType with opt level: 0

Traceback (most recent call last):
  File "core/test.py", line 76, in <module>
    quantize_method=pFramework.quantize_method)
  File "/mnt/tscpfs/xiaotao.chen/Repositories/tvm_usage_summary/core/TVMPipeline.py", line 68, in __init__
    granularity='layer', gpuid=self.device_id)
  File "/mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/python/tvm/relay/quantize/quantize.py", line 909, in autoquantize
    graph, lib, params = relay.build(graph, target)
  File "/mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/python/tvm/relay/build_module.py", line 196, in build
    params)
  File "/mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/python/tvm/relay/build_module.py", line 107, in build
    self._build(func, target, target_host)
  File "/mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/python/tvm/_ffi/_ctypes/function.py", line 209, in __call__
    raise get_last_ffi_error()
tvm._ffi.base.TVMError: Traceback (most recent call last):
  [bt] (8) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbcfe6b) [0x7f6589e4ae6b]
  [bt] (7) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbcebab) [0x7f6589e49bab]
  [bt] (6) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbb5ebd) [0x7f6589e30ebd]
  [bt] (5) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbafa3e) [0x7f6589e2aa3e]
  [bt] (4) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbb4fcf) [0x7f6589e2ffcf]
  [bt] (3) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbb1908) [0x7f6589e2c908]
  [bt] (2) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xc43e73) [0x7f6589ebee73]
  [bt] (1) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbb84d0) [0x7f6589e334d0]
  [bt] (0) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbb3db9) [0x7f6589e2edb9]
  File "/mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/src/relay/backend/graph_plan_memory.cc", line 87
TVMError: Check failed: tok.size() == 1U (3 vs. 1) :

Hi @tqchen @eqy @ZihengJiang @vinx13 , I tried this PR for channel quantization. It works in resnet(according to the code of @antinucleon as before). But it failed on my own detection model, which works on master or my old version. the error infos as below. Do you have any idea about this error? Under what cases will cause this error? wish you can provide some clues. Thx.
[11:58:41] /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/src/relay/pass/pass_manager.cc:377: Executing module pass : InferType with opt level: 0

[11:58:41] /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/src/relay/pass/pass_manager.cc:377: Executing module pass : InferType with opt level: 0

Traceback (most recent call last):
  File "core/test.py", line 76, in <module>
    quantize_method=pFramework.quantize_method)
  File "/mnt/tscpfs/xiaotao.chen/Repositories/tvm_usage_summary/core/TVMPipeline.py", line 68, in __init__
    granularity='layer', gpuid=self.device_id)
  File "/mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/python/tvm/relay/quantize/quantize.py", line 909, in autoquantize
    graph, lib, params = relay.build(graph, target)
  File "/mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/python/tvm/relay/build_module.py", line 196, in build
    params)
  File "/mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/python/tvm/relay/build_module.py", line 107, in build
    self._build(func, target, target_host)
  File "/mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/python/tvm/_ffi/_ctypes/function.py", line 209, in __call__
    raise get_last_ffi_error()
tvm._ffi.base.TVMError: Traceback (most recent call last):
  [bt] (8) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbcfe6b) [0x7f6589e4ae6b]
  [bt] (7) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbcebab) [0x7f6589e49bab]
  [bt] (6) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbb5ebd) [0x7f6589e30ebd]
  [bt] (5) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbafa3e) [0x7f6589e2aa3e]
  [bt] (4) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbb4fcf) [0x7f6589e2ffcf]
  [bt] (3) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbb1908) [0x7f6589e2c908]
  [bt] (2) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xc43e73) [0x7f6589ebee73]
  [bt] (1) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbb84d0) [0x7f6589e334d0]
  [bt] (0) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbb3db9) [0x7f6589e2edb9]
  File "/mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/src/relay/backend/graph_plan_memory.cc", line 87
TVMError: Check failed: tok.size() == 1U (3 vs. 1) :

looks like you have nested tuple that is not yet supported by graph memory planer

Hi @tqchen @eqy @ZihengJiang @vinx13 , I tried this PR for channel quantization. It works in resnet(according to the code of @antinucleon as before). But it failed on my own detection model, which works on master or my old version. the error infos as below. Do you have any idea about this error? Under what cases will cause this error? wish you can provide some clues. Thx.
[11:58:41] /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/src/relay/pass/pass_manager.cc:377: Executing module pass : InferType with opt level: 0

[11:58:41] /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/src/relay/pass/pass_manager.cc:377: Executing module pass : InferType with opt level: 0

Traceback (most recent call last):
  File "core/test.py", line 76, in <module>
    quantize_method=pFramework.quantize_method)
  File "/mnt/tscpfs/xiaotao.chen/Repositories/tvm_usage_summary/core/TVMPipeline.py", line 68, in __init__
    granularity='layer', gpuid=self.device_id)
  File "/mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/python/tvm/relay/quantize/quantize.py", line 909, in autoquantize
    graph, lib, params = relay.build(graph, target)
  File "/mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/python/tvm/relay/build_module.py", line 196, in build
    params)
  File "/mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/python/tvm/relay/build_module.py", line 107, in build
    self._build(func, target, target_host)
  File "/mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/python/tvm/_ffi/_ctypes/function.py", line 209, in __call__
    raise get_last_ffi_error()
tvm._ffi.base.TVMError: Traceback (most recent call last):
  [bt] (8) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbcfe6b) [0x7f6589e4ae6b]
  [bt] (7) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbcebab) [0x7f6589e49bab]
  [bt] (6) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbb5ebd) [0x7f6589e30ebd]
  [bt] (5) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbafa3e) [0x7f6589e2aa3e]
  [bt] (4) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbb4fcf) [0x7f6589e2ffcf]
  [bt] (3) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbb1908) [0x7f6589e2c908]
  [bt] (2) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xc43e73) [0x7f6589ebee73]
  [bt] (1) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbb84d0) [0x7f6589e334d0]
  [bt] (0) /mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/build/libtvm.so(+0xbb3db9) [0x7f6589e2edb9]
  File "/mnt/tscpfs/xiaotao.chen/Repositories/tvm_channel/src/relay/backend/graph_plan_memory.cc", line 87
TVMError: Check failed: tok.size() == 1U (3 vs. 1) :

looks like you have nested tuple that is not yet supported by graph memory planer

Thanks for your reply. the detection model has 3 outputs. That grouped by mxnet.sym.group() . Maybe the error caused by the below code in this PR, which tuple the graph.body. The model can work normally with graph memory planner on master or old version. Do you have any suggestion to fix this in my case? Or how to avoid this by modify my model ? :
        graph = relay.Function(graph.params,
                                relay.expr.Tuple([graph.body]+additional_outputs))
        target = 'llvm -mcpu=core-avx2'
        #target = 'cuda'
        with relay.build_config(opt_level=0):
            graph, lib, params = relay.build(graph, target)
            ctx = tvm.nd.context(target)

@XiaotaoChen You can try fattening the nested tuple here:
graph = relay.Function(graph.params, relay.expr.Tuple([graph.body]+additional_outputs))

@vinx13 why use fmulti_ref in QuantizeAnnotate? in "resnet18_v1" from gluon, this will add a simulated_quantize node after nn.maxpool_2d node which output a int8 expr and lead to an error in realize stage:
  %17 = clip(%16, a_min=-127, a_max=127);
  %18 = cast(%17, dtype="int8");
  %19 = nn.max_pool2d(%18, pool_size=[3, 3], strides=[2, 2], padding=[1, 1]);
  %20 = right_shift(%19, 1) an internal invariant was violated while typechecking your program [23:40:49] /home/yy/deeplearning/tvm/src/relay/op/type_relations.cc:120: Check failed: t0->dtype == t1->dtype (int8 vs. int32) :
; an internal invariant was violated while typechecking your program [23:40:49] /home/yy/deeplearning/tvm/src/relay/op/type_relations.cc:120: Check failed: t0->dtype == t1->dtype (int8 vs. int32) :

With  fmulti_ref:

Without fmulti_ref:

I am using newest master code.

@traveller59 let's move discussion to https://discuss.tvm.ai

I started taking a look at this PR and it looks like some of the pass infra has changed it was opened, so I'm going to incrementally see what breaks as I try to backport the new changes. The first seems to annotate being removed in the new infra.
@jroesch @ZihengJiang @vinx13 @zhiics

Hi @eqy @vinx13 , I try this PR on my detection model with weight/act granularity=layer. and remove the graph.body on calibration stage. in the realize pass. there is a bug in UnifyDTypeScale called by ConcatenateRealize.  in this line: Expr dom_scale = ChooseDomScale(nptrs, min); the scalar of dom_scale is 0, that cause the later error. Then i check the inputs->dom_scale, which used by Concatenate op. to find the lhs_expr->dom_scale=0, and the rhs_expr->dom_scale is normal. So i go back to check all dom_scale of quantize_op in calibration stage. But all act scale >0, such as [4], [8]. i don't know why the dom scale of quantize op can be changed. hope you give some clues. Thanks.

@vinx13 what is the relation of this PR and your #3538? @tqchen said the issue #2651 was closed by #3538.

@masahi #3538 computes scales directly to minimize kl divergence, this PR select scales from predefine candidates by mean square error

@vinx13 thanks, good to know we have multiple methods available.

close for now due to inactive status, would be great if we can follow up @vinx13


Deprecating gopkg.in

Hello Aaron,
Is the code currently in master branch considered v1 or v2?
If I want the latest v1, should I still be using gopkg.in/v1 or the v1 tags on volatiletech/authboss?
Attempting to import v1.1.0 from volatiletech failed to compile, because the code in v1.1.0 itself imports from gopg.in/v1, specifically the internal packages causing the errors.
It failed with this:
vendor/github.com/volatiletech/authboss/auth/auth.go:11:2: use of internal package not allowed

v2 is meant to be used as semver.
v1 is still meant to be used with gopkg.in


Centering with fixed width

Hello!

Swiper Version: 4.5.0
Platform/Target and Browser Versions: Ubuntu Chrome / Firefox
Live Link or JSFiddle/Codepen or website with isssue: https://jsfiddle.net/wvps1h2t/1/

What you did
Configure swiper with fixer width and centered.
width: 50,
centeredSlides: true,

Expected Behavior
Centering will be applied.
Actual Behavior
Slides starts from left side and centering is ignored.
Thanks, Vlad.

Set slidesPerView: 'auto' and .swiper-slide { width: 50px } in CSS https://jsfiddle.net/sj75aqhp/


[HUDI-6450] Fix null strings handling in convertRowToJsonString

Change Logs
Fix null strings handling in HoodieCDCRDD.convertRowToJsonString.
Impact
Avoid NPEs when reading CDC info for entries with null strings.
Risk level
Medium
Documentation Update
NA
Contributor's checklist

[x] Read through contributor's guide
[x] Change Logs and Impact were stated clearly
[x] Adequate tests were added if applicable
[x] CI passed

@zaza You need to link it to JIRA and update the jira description. Refer other PR's.
I created JIRA for you - https://issues.apache.org/jira/browse/HUDI-6450, So you can prepend [HUDI-6450] to the subject.

@zaza please add unit test for this.

@zaza There is a test failure: https://github.com/apache/hudi/actions/runs/5422435507/jobs/9859095006?pr=9064, can you resolve that?

Test has passed: https://dev.azure.com/apache-hudi-ci-org/apache-hudi-ci/_build/results?buildId=18237&view=results


add verified field to groups

This will not work as expected for CI currently, as verifying groups requires an auth token from a human Chainguardian. Before this merges, updates to the api are necessary.


Config fix

Resolves #111

@denar90 can you create tests at the cmd line tool level (https://github.com/paulirish/pwmetrics/blob/master/test/cli.js) to ensure this case does not break in the future? I agree with @paulirish there should be a cleaner/ less convoluted way to do this? I've haven't had the time to think about this but I'll try during the weekend. If I can't find an alternative and we get more tests for this then LGTM

I agree with you guys that it looks like overhead. The reason I did it to simplify reading the code. If package.json => go one way, read just function, if file => read another function. Don't need to follow some conditions and stuff. I'm also open for improving it :)

@paulirish @denar90 I modified the PR code with my proposed changes, the relevant part is in the bin/cli.js file. What do you guys think?

LGTM.  yeah this is great.
over to @denar90 for confirmation this solves #111

Nice solution 👍


release-21.1: sql: ensure user has correct privileges when adding/removing regions

PR: https://github.com/cockroachdb/cockroach/pull/62450
From release notes:

ALTER DATABASE .. SET PRIMARY REGION now requires both CREATE and ZONECONFIG privilege on all objects inside the database when adding the first region to the database. The same behavior applies for dropping the last region using ALTER DATABASE ... DROP REGION. [#62450][#62450] {% comment %}doc{% endcomment %}

Permissions are documented for both SET PRIMARY REGION and DROP REGION in v21.1 docs


dotnet aws delete-deployment fails if resource has been added to a security group

Describe the bug
I deployed a Console Application that reads from a Database to a Fargate Service.
After it was deployed I updated my RDS's Security Group to allow the new Application to read the database.
I then ran dotnet aws delete-deployment and got this error message:
03/15/2021 15:46:18 | DELETE_FAILED        | AWS::CloudFormation::Stack               | ContosoFargateService2
Unhandled exception: AWS.Deploy.CLI.FailedToDeleteException: The stack ContosoFargateService2 is in a failed state. You may need to delete it from the AWS Console.
   at AWS.Deploy.CLI.Commands.DeleteDeploymentCommand.WaitForStackDelete(String stackName) in C:\codebuild\tmp\output\src000293299\src\src\AWS.Deploy.CLI\Commands\DeleteDeploymentCommand.cs:line 122
   at AWS.Deploy.CLI.Commands.DeleteDeploymentCommand.ExecuteAsync(String stackName) in C:\codebuild\tmp\output\src000293299\src\src\AWS.Deploy.CLI\Commands\DeleteDeploymentCommand.cs:line 73
   at AWS.Deploy.CLI.Program.<>c.<<Main>b__4_2>d.MoveNext() in C:\codebuild\tmp\output\src000293299\src\src\AWS.Deploy.CLI\Program.cs:line 217
--- End of stack trace from previous location where exception was thrown ---
   at System.CommandLine.Invocation.CommandHandler.GetResultCodeAsync(Object value, InvocationContext context)
   at System.CommandLine.Invocation.ModelBindingCommandHandler.InvokeAsync(InvocationContext context)
   at System.CommandLine.Invocation.InvocationPipeline.<>c__DisplayClass4_0.<<BuildInvocationChain>b__0>d.MoveNext()

Cloud Formation in AWS Console shows:

My Fargate Task Definition has been deleted which is good.  But the CloudFormation stack still exists and is in a delete failed state.
Expected behavior
My application and related AWS Resource to be successfully deleted and no error message should be displayed.
Environment (please complete the following information):

CLI Version: [eg. 1.0.0] 0.1.1-beta.20210315205437.06db70a
Include the output of dotnet --info 5.0.201
OS Info: Windows 10

This is a :bug: bug report

This should no longer be an issue as you can add additional security groups to your deployment instead of the other way around.


Adding a UIView

So I see how you guys added a UIIMage to the Scene. However, I wanted to add a UIView. So using your code I created the code below. Everything work except one thing. I could not get the background to be transparent. I ended up converting my view to a UIIMage and have a current workaround. However, I think it would be nice if your SDK supported adding UIViews.
One other odd thing I noticed: When I added the views using:
self.sceneLocationView.addLocationNodeWithConfirmedLocation(.
They were smaller the farther away. However when I add Images using the same method. They stay the same size no matter the distance. How do i controle that feature?
open class LocationViewNode: LocationNode {
    ///An image to use for the annotation
    ///When viewed from a distance, the annotation will be seen at the size provided
    ///e.g. if the size is 100x100px, the annotation will take up approx 100x100 points on screen.
    public let view: UIView

    ///Subnodes and adjustments should be applied to this subnode
    ///Required to allow scaling at the same time as having a 2D 'billboard' appearance
    public let annotationNode: SCNNode

    ///Whether the node should be scaled relative to its distance from the camera
    ///Default value (false) scales it to visually appear at the same size no matter the distance
    ///Setting to true causes annotation nodes to scale like a regular node
    ///Scaling relative to distance may be useful with local navigation-based uses
    ///For landmarks in the distance, the default is correct
    public var scaleRelativeToDistance = false

    public init(location: CLLocation?, view: UIView) {
        self.view = view

        let plane = SCNPlane(width: view.bounds.size.width / 100, height: view.bounds.size.height / 100)
        plane.firstMaterial!.diffuse.contents = view
        plane.firstMaterial!.lightingModel = .constant

        annotationNode = SCNNode()
        annotationNode.geometry = plane

        super.init(location: location)

        let billboardConstraint = SCNBillboardConstraint()
        billboardConstraint.freeAxes = SCNBillboardAxis.Y
        constraints = [billboardConstraint]

        addChildNode(annotationNode)
    }

    required public init?(coder aDecoder: NSCoder) {
        fatalError("init(coder:) has not been implemented")
    }
}

I have ended up with the same solution but however I sometimes get white empty SCNNode without my UIView which is loaded from a Nib. It is not all the nodes but only few and it seems to be random. Do you have the same problem @jwilson85 ?

@jwilson85 to improve your code you should use:
plane.firstMaterial!.diffuse.contents = view.layer

Because as Apple documentation says:

SceneKit cannot use a layer that is already being displayed elsewhere (for example, the backing layer of a UIView object).

This is why I had the white SCNNode, using a CALayer is more reliable.

Thanks I’ll try that

I seem to be having the same problem with the missing UIViews. Adding view.layer instead of the view itself seems to have helped a bit, but instead of white empty views I now sometimes get completely invisible views. Did anyone experience this as well?

@adrgrondin
plane.firstMaterial!.diffuse.contents = view.layer
I have used the above your code but not getting events for button so my question is how can i get buttons events?


purge onedrive folders on gh action

Description
runs the golang purge script for onedrive folders
when running the pwsh purge scripts.  This will
move test-generated folders in onedrive into
a recycle bin.  However, that recycle bin cannot
be purged via api, and must be manually emptied
periodically for each user.
Type of change

[x] :computer: CI/Deployment

Issue(s)

#1525

Test Plan

[x] :green_heart: E2E

Knowing very well you know what you are doing

famous last words.


[QUESTION]: Where i can find documentation and guide how to use tidal-dl?

Which tool
tidal-dl
Describe your question
It looks like link to documentation for CLI version is missing.
Add screenshots
No response

For now you can see the archived version:
https://web.archive.org/web/20230330100857/https://yaronzz.com/post/tidal_dl_installation/#🎉Tidal-dl
Maybe I'll try to update the readme from the archived version later.

Actually I see yarnozz change the url on his website:
https://doc.yaronzz.com/post/tidal_dl_installation/
I'll update the readme.


Update node-exporter role for running with new data model

Wait for changes in prometheus and kubernetes role.


Compare performance of softmax implementations

CuDNN is supposed to have very efficient softmax implementations for both, the forward and the backward step. It would be interesting to set up a benchmark and compare to our own implementation on different (small and very large) matrix sizes.
Not sure, but CuDNN seems also to have a working log-softmax?

Quite surprising results. Which also confirm my previous suspicions, that library creators only test for square matrices.


wasi-crypto: unable to build on Windows CI

@jedisct1, I thought it would be interesting to build in wasi-crypto to the benchmarking engine used in Sightglass over in #4207. I changed the wasmtime-bench-api default features to include wasi-crypto, but when I did, I observed build errors on Windows related to pqcrypto-internals:
https://github.com/bytecodealliance/wasmtime/runs/6700498078?check_suite_focus=true#step:12:399
Understanding that neither the wasmtime-bench-api crate or the wasi-crypto feature are completely "done," this issue is not a high priority one; I did want to get your eyes on this to see if there is anything simple that can be done to fix things. In the meantime, I will leave wasi-crypto disabled by default in wasmtime-bench-api.

Thanks for reporting this!
I think it was fixed here, about a week after the crate published :/
So, the next release won't have that issue. As a workaround, the wasi-crypto hostcalls can be compiled without the default features, effectively removing the pqcrypto dependency.

We've since removed wasi-crypto from in-tree, so Im' going to close this.


Bug Report: 0.7.0 - The program start by default with UI in English

Is there an existing issue for this?

[X] I have searched the existing issues

What happened or didn't happen
@Timthreetwelve
My OS is Windows 10 23H2 Pro 64bit italian.
After installed 0.7.0 it started with UI in English.
My expectation is that program start by default with same language of the OS (Italian).
Thanks.
Steps To Reproduce
No response
Version
0.7.0
Windows Version
Windows 10 23 H2 Pro Italian
Relevant log output
Install the program
Start the program.
Anything else?
No response

@bovirus

I could change the default for the Use Windows display language option (in green box) to true (checked) to do this. There is already logic in place that falls back to en-US if the language translation doesn't exist.
Does that sound like it would address your issue?

@Timthreetwelve
Of course it's possible to change the language via Settings,.
My expactation is that by defaulkt the option "Use Windows display language" was set,.
Then please make this change.

Yes, it will still be possible to change the language via Settings.
I will make this change to each of my multilingual apps. I'm currently working on updating HWiNFO VSB Viewer. It will be the first release to have this change.
Thanks for the suggestion.
Tim

Thanks.


[6.x] Split hasValidSignature method

This splits up the functionality of the hasValidSignature method into two separate checks - whether the signature matches the given path/URL, and whether the expires timestamp has since passed or not.
The benefit of providing access to these separately is it allows apps to handle these cases differently. It would allow you to catch InvalidSignatureException and then provide a better user experience when a signature was actually correct, but has just since expired.
My use case is that some emails we send include signed URLs that will automatically log our users in and then redirect them to the page the user intended to go to. In the case that the URL has expired I would still like redirect them to the intended page, but just skip the automatic login.

Maybe we should introduce an ExpiredSinatureException which extends the InvalidSignatureException?

@netpok I was actually ready to put a PR together that was exactly like that, but felt that implementation may have been a little too heavy. This PR was a simpler compromise.
https://github.com/laravel/framework/compare/6.x...dwightwatson:signature-expiry?expand=1

Could you do this PR?


Transfer admin rights to another device

Is this PR still needed or will it be merged? It's been here for quite some time without anything happening.
@bitionaire @orangehelicopter


Example page fails

Thanks for opening an issue!
Please select the type of issue you're reporting. For questions.

[x] Bug
[ ] Feature Request
[ ] Question

Problem Report
When I go to the example page, I see the following:

System Info
Browser: Version 71.0.3578.98 (Official Build) (64-bit)
OS: OSX
Reproduction
If this is a bug report, please provide a reproduction of the issue by going to
http://www.webpackbin.com/41YFBvekG.
Paste a link here to your working reproduction.

I can't reproduce this, unless you're talking about the issue template link, which I've replaced to codesandbox in 4ce15ee.


How to end a specific netconf session connection through session ID ?

Hello，I wanna know how to cut off a specific neconf session through session ID, whatever use the code or spcript.
I have tried the "sr_session_stop(subscription)" / “sr_session_stop(session)” / "sr_disconnect(connection)" interface function, they are all not worked right.

You can affect NETCONF sessions only using another NETCONF session. So connect to netopeer2 and use kill-session to kill the other session.


Handle kamelets dependencies

Please describe the feature that you want to propose
Support the dependencies array.
The current rules are:
a. The flow contains a Processor like to or choice: In this case, camel:core needs to be added
b. The flow contains a few Camel Components: In this case, I need to add camel:component-name for each one.
c. The user adds a 3rd party dependency: In this case, this is allowed in the source code editor for the time being.
This issue spun off from: https://github.com/KaotoIO/kaoto-next/issues/110

It also needs to consider Languages and Dataformats for dependencies.


Revert "Optimize DefUseManager allocations (#4709)"

This reverts commit d18d0d92e55f44da6af0dc87fb0e3c8034e9a3ac.
This is reverted because it causes a 7X slowdown when legalizing
SPIR-V with NonSemantic.Shader.DebugInfo.100 instructions.
This is due to the creation of very large UseLists for several
heavily used operands for this extension combined with the fact
that this commit changed the performance of Uselists to O(n).

Yeah, I am seeing an assert in InterfaceVariableScalarReplacementTest.ReplaceInterfaceVarsWithScalars with Debug build.

Regression fixed with #4850

@s-perron PTAL

I'm going on vacation for a week.  @kuhar Can you take a look?

@greg-lunarg would you be able to share an input that exhibits the 7x slowdown (either here or email me in private)? I'd like to step through the code to see what's going on.

@kuhar You will need to get the shader directly from @danginsburg. Dan will contact you to hand you the shader.
The problem originally manifested using dxc which calls spirv-opt to do legalization. Here is the dxc command line to recreate the problem:
/cygdrive/c/Users/greg/dev/dxc/bin/Release/bin/dxc.exe  -EMainPs -Wno-effects-syntax -T ps_6_0 -fspv-target-env=vulkan1.3 -fspv-debug=vulkan-with-source -spirv -Fo csgo.frag.vdb.fast.spv csgo.frag

Essentially, the problem is that every NonSemantic.Shader.DebugInfo.100 instruction uses the following operands:
%2 = OpExtInstImport "NonSemantic.Shader.DebugInfo.100"
%void = OpTypeVoid

For large shaders, there are 1000s of these instructions. When these instructions are added to DefUse, this creates a UseLists of 1000s of entries.
The change that is being reverted changed the representation of UseList in DefUse to use find and insert algorithms of O(n), which caused serious performance degradation for shaders with DebugInfo instructions.
While this is true for OpenCL.DebugInfo.100, it is twice the problem for NonSematic.Shader.DebugInfo.100 since the latter also has its own DebugLine instruction which can double the number of Debug instructions.


libevdev: 1.11.0 -> 1.12.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/libevdev/versions.
meta.description for libevdev is: "Wrapper library for evdev devices"
meta.homepage for libevdev is: "http://www.freedesktop.org/software/libevdev/doc/latest/index.html"
meta.changelog for libevdev is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/dpvqycbaja3g7sf1syl81yrcqsgzbm8z-libevdev-1.12.0/bin/mouse-dpi-tool had a zero exit code or showed the expected version
/nix/store/dpvqycbaja3g7sf1syl81yrcqsgzbm8z-libevdev-1.12.0/bin/libevdev-tweak-device passed the binary check.
Warning: no invocation of /nix/store/dpvqycbaja3g7sf1syl81yrcqsgzbm8z-libevdev-1.12.0/bin/touchpad-edge-detector had a zero exit code or showed the expected version
1 of 3 passed binary check by having a zero exit code.
1 of 3 passed binary check by having the new version present in output.
found 1.12.0 with grep in /nix/store/dpvqycbaja3g7sf1syl81yrcqsgzbm8z-libevdev-1.12.0
found 1.12.0 in filename of file in /nix/store/dpvqycbaja3g7sf1syl81yrcqsgzbm8z-libevdev-1.12.0

Rebuild report (if merged into master) (click to expand)

3825 total rebuild path(s)

3792 package rebuild(s)

3792 x86_64-linux rebuild(s)
33 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
AusweisApp2
ChowKick
CuboCore.coreaction
CuboCore.corearchiver
CuboCore.corefm
CuboCore.coregarage
CuboCore.corehunt
CuboCore.coreimage
CuboCore.coreinfo
CuboCore.corekeyboard
CuboCore.corepad
CuboCore.corepaint
CuboCore.corepdf
CuboCore.corepins
CuboCore.corerenamer
CuboCore.coreshot
CuboCore.corestats
CuboCore.corestuff
CuboCore.coreterminal
CuboCore.coretime
CuboCore.coretoppings
CuboCore.coreuniverse
CuboCore.libcprime
CuboCore.libcsys
OSCAR
adapta-gtk-theme
adwaita-qt
aj-snapshot
ajour
albert
alfaview
alfis
almanah
amarok
amarok-kf5
ams
android-file-transfer
anki
ankisyncd
antimicrox
antimony
ao
apitrace
apmplanner2
apostrophe
appcsxcad
aqemu
arx-libertatis
astroid
asymptote

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/dpvqycbaja3g7sf1syl81yrcqsgzbm8z-libevdev-1.12.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A libevdev https://github.com/r-ryantm/nixpkgs/archive/c4cbf1331164786b2eacc387572564990eacc23f.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/dpvqycbaja3g7sf1syl81yrcqsgzbm8z-libevdev-1.12.0
ls -la /nix/store/dpvqycbaja3g7sf1syl81yrcqsgzbm8z-libevdev-1.12.0/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @AndrewMorsillo for testing.

Changelog: https://lists.freedesktop.org/archives/input-tools/2021-November/001573.html


add navbar with search bar

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: Thilini-Weerasinghe sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


bbs extension

Latest progress on the bitbucket_server extension. This might still need a bit of tidying up.

So just to comment briefly on this. Most views are now working properly. The biggest issue was with the pull-requests/diff view because there was no easy way to pull out the commit ids being compared for each diff without interacting with the Bitbucket Server UI. The commit list for each pull request within a diff view is stored in a drop down list and these ids are not fetched unless the user specifically clicks this dropdown. The first implementation included a click through jquery so the list could be fetched. However this is unacceptable for the user since he would see a dropdown opening and closing when looking at a diff. If I was using an extension that required this I would personally stop using it instantly.
So to bypass this I discovered that I am able to call BBS API directly. The call to the API returns a list of all commits within a pull requests. Each commit object has a link to its parents. This way we can safely retrieve the list of commits within a pull request without having to fiddle with the UI.
Obstacles
The hardest part is knowing when to update a view. All of the views use javascript heavily to update the views and multiple elements are being re-rendered all the time (e.g.: single file view is re-rendered on window resize). To handle this we pick specific DOM elements that we know will be updated with the next UI updates and bind them to a DOMSubtreeModified event. When the UI is updated the event is triggered and runs the update_view function after a fixed timeout (currently 1500 ms)
Testing
This has not been tested thoroughly and currently all builds are failing. I suggest that we wait with merging this pr and bumping the version until we have a soldid testing method for this part of the code. One thing we could do is try to create a docker environment that runs both bbs and codecov enterprise in an environment with data already setup with repos and code coverage reports. My point here being that even if we can mock the current setup for bbs these tests would be counterproductive. The bbs pages cannot really be tested with static html pages because so much of the data comes through javascript. Another idea would be to just manually test this whenever a new version of bbs is out. Automation could be hard since a setup requires some interaction by the user unless we find a solid way of running both bbs and codecov enterprise with a fixed license and data.
This plugin has currently been tested manually on Chrome Version 56.0.2924.87 (64-bit).

#43

@woot2531 #42

@woot2531 any questions on this. It has not been merged into master so it should not be the cause of issue #43. We are still testing this with bbs so this pr is only for viewing the latest updates.


typos / grammar

Those options control what happens when you the default "Clear all" action is executed.
Change to...
These options control what happens when the default "Clear All" action is executed.

Thanks for the report @jlahijani, I have corrected that sentence on the master branch. Except for the title-case spelling of "Clear All". This was indeed inconsistent with the field label, so I have changed it to "Change all" everywhere instead.
Let me know if you find any other errors!


callback to allow for transformation of matched items whilst keeping original list

It would be nice to be able to return all items in the array with a transform function to allow for adding properties to the matched items but keeping the original list.
For example: If I wanted to return a list of the matched items with a visible key attached so allow for animation on the nodes that are not matched eg collapsing them from view.

How about this:
matchSorter(items.map(/* whatever */), ...etc)


cleanup dacpac extension telemetry

Cleaning up the telemetry in the dacpac extension that was added last year in #13859. Summary of the changes:

send ErrorEvents when a DacFx operation fails instead of previously it was an ActionEvent with ExportStatus/ImportStatus/etc as a property
removed most of the different telemetry view enums. This is all in the Data tier application wizard and the action names are enough to differentiate
removed properties that were always true or never got sent
removed unnecessary try catches. If there was an exception thrown in STS, it would be reflected in the result.success being false. I checked in kusto to verify that the exceptionOccurred property was never true for any event sent from the dacpac extension, so this confirms that the code wasn't getting called.
removed unused providername parameter for getService()

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

61 of 65 (93.85%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.001%) to 48.038%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/dacpac/src/wizard/dataTierApplicationWizard.ts
54
58
93.1%

Totals

Change from base Build<PHONE_NUMBER>:
-0.001%

Covered Lines:
14877

Relevant Lines:
29396

💛  - Coveralls

How did you test these changes, out of curiosity?

How did you test these changes, out of curiosity?

I only checked that the right paths were getting hit so ActionEvents were getting sent when result.success is true and ErrorEvents were getting sent when result.success was false or result was undefined. The properties and measurements that were already there are in kusto, so I didn't check in fiddler that they're getting sent.


[feature] Add support for Python 3.13

Is your feature request related to a problem? Please describe.
Python 3.13 is currently in the release candidate stage and is scheduled for official release on October 1, 2024 (in approximately 2 weeks). To ensure compatibility with the latest version, we should begin updating our workflows to support Python 3.13.
Describe the solution you'd like
Update the workflow configuration to include Python 3.13 in addition to all previously supported versions.

Completed in https://github.com/EndstoneMC/endstone/commit/84c80eaadb7d92cce2f3b8b6d7e94aecba1ed863.

👀


Question about ETT dataset

In data_loader.py, I found that only 20 months' data has been used. And I wanna to ask why not using all data?
Thank you!

In data_loader.py, I found that only 20 months' data has been used. And I wanna to ask why not using all data? Thank you!

Hi, here we are referencing the settings of the Time Series Library (TSlib). The dataset-related content in the experiment is also consistent with it. If you want to use more data, you can adjust the settings in the data_loader accordingly.

In data_loader.py, I found that only 20 months' data has been used. And I wanna to ask why not using all data? Thank you!

Hi, here we are referencing the settings of the Time Series Library (TSlib). The dataset-related content in the experiment is also consistent with it. If you want to use more data, you can adjust the settings in the data_loader accordingly.

Thanks!


Launching on Cuda

Hi! After launching the program via run_1OpenCLDevice, the program generates 96k keys/second. The video card is not involved in the processes. Nvidia RTX 3050 Laptop graphics card, Cuda 12 is installed. Intel has disabled the second graphics card. Maybe additional settings are required?

Hi, can you have a look in the Windows TaskManager into GPU and open the CUDA graph?  Maybe your CPU is very slow. I`m already working on a lot of improvements to use the GPU only. Greetings, Bernard

gpu cuda is not displayed for some reason

On the left side, you need to open the performance overview in the menu bar with the charts inside the task manager.

You are using too less RAM. Please also use a drop down of the GPU window (e. G. 3D or copy and select CUDA)

Sorry to waste your time. But I didn't understand anything, what should I do?

Yes, CUDA is operational. However, your RAM is insufficient to keep the LMDB fully in memory. As a result, your SSD is performing swapping operations, leading to the high load on the SSD.

How much RAM do you recommend?

It depends on the database. For the light database, I recommend 16GB. For the full database, you need a minimum of 64GB.

OK. Thanks. Have a nice day!