the got-10k dataset can not be downloaded

when I dowanload the got-10k dataset in http://got-10k.aitestunion.com/downloads.It shows the error that 500 Internal Server Error.So I can not download this dataset. can you help me solve this problem?thank you！

Hi, do you mean http://got-10k.aitestunion.com/downloads? This link works for me. Please let me know if there is still any problem when downloading.

http://got-10k.aitestunion.com/downloads is ok now.thank you for your reply.


Uncaught OSError in pathlib.py line 331

Aider version: 0.54.8
Python version: 3.10.13
Platform: macOS-13.1-arm64-arm-64bit
Python implementation: CPython
Virtual environment: No
OS: Darwin 22.2.0 (64bit)
Git version: git version 2.39.0
An uncaught exception occurred:
Traceback (most recent call last):
  File "aider", line 8, in <module>
    sys.exit(main())
  File "main.py", line 552, in main
    coder = Coder.create(
  File "base_coder.py", line 156, in create
    res = coder(main_model, io, **kwargs)
  File "base_coder.py", line 355, in __init__
    fname.touch()
  File "pathlib.py", line 1168, in touch
    self._accessor.touch(self, mode, exist_ok)
  File "pathlib.py", line 331, in touch
    fd = os.open(path, flags, mode)
OSError: [Errno 30] Read-only file system: '/chat-mode'

Thanks for trying aider and filing this issue.
Looks like you tried to run aider on a read only filesystem. That won't really work, but aider shouldn't crash. I've fixed that.
The fix is available in the main branch. You can get it by installing the latest version from github:
aider --install-main-branch

# or...

python -m pip install --upgrade git+https://github.com/paul-gauthier/aider.git

If you have a chance to try it, let me know if it works better for you.


Rifle Crush, Durandal and Lebel

About the PR
Rifles, like muskets, with a new cartridge inserted after each shot. It's still a work in progress.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

:cl: Nimfar11

add: Adds three rifles, the Crush, Durandal and Lebel. So far there is no difference between them.

different weapons should be recognizale inhand. This two have same sprites

huh

PR is broken


OpenCover stoped working

Hi guys,
i configurated my cake build script a while ago and open cover worked as expected. I did not touch the project for something like 2 months. When i tried to rerun the build script today open cover was not able to create results with the following msg:
Committing...
No results, this could be for a number of reasons. The most common reasons are:
1) missing PDBs for the assemblies that match the filter please review the
output file and refer to the Usage guide (Usage.rtf) about filters.
2) the profiler may not be registered correctly, please refer to the Usage
guide and the -register switch.
-> I checked the bin folder and the pdb file exists
My test task looks like this (I already removed the filters to make sure they are not the problem):
	var coverSettings = new OpenCoverSettings()
	{
		Register = "user",
		SkipAutoProps = true,
	};

	var reportGeneratorSettings = new ReportGeneratorSettings();

	var reportGeneratorSettingsCobertura = new ReportGeneratorSettings()
	{
		ArgumentCustomization = args => args.Append("-reporttypes:Cobertura" ),
	};

	foreach (var testProject in unitTests)
	{
		string projectName = testProject.GetFilename().ToString().Replace(".Tests.csproj", string.Empty);

		string testDir = "./TestResults";
		EnsureDirectoryExists(testDir);

		string openCoverDir = $"{testDir}/OpenCover/{projectName}";
		EnsureDirectoryExists(openCoverDir);

		string resultsFileName = $"{testDir}/{projectName}_TestResults.xml";
		var openCoverFilePath = new FilePath($"{openCoverDir}/_results.xml");

		var unitSettings = new NUnit3Settings
		{
			ShadowCopy = false,
			Results = new[] { new NUnit3Result { FileName = resultsFileName } },
		};

		OpenCover(tool =>
			{
				tool.NUnit3(testProject.ToString(), unitSettings);
			},
			openCoverFilePath,
			coverSettings
		);
		ReportGenerator(openCoverFilePath, openCoverDir, reportGeneratorSettingsCobertura);
		ReportGenerator(openCoverFilePath, $"{openCoverDir}/html", reportGeneratorSettings);
	}

Output File:

Is this a public project? Can you share a link? Are you pinning to a specific version of tools/addins/Cake?

No its not public. I tried the versions 4.6.519 and 4.7.922 of open cover.
None worked yesterday. Today though its working again.
The only thing, that changed was the company proxy, which was blocking nuget yesterday and the problem seems to be connected to that problem, which is strange, cause the binaries were already loaded.
Either way the problem is solved.


Remove extra token merged by accident.

Purpose
8ddf0ca40643401541d844a97df5542817fea1be introduced a spurious token
that breaks scrolling back to top in HEAD.
This change simply reverts that.

@abahgat thanks, well spotted :)


BufferIf:  Adjusted API to not collide

If only a pause observable was provided, the method signatures were ambiguous
What kind of change does this PR introduce?
API improvement to avoid ambiguous calls for BufferIf
What is the current behavior?
If BufferIf is called with only a pause observable provided, the compiler complains about ambiguous calls.
What is the new behavior?
Happy compiler
What might this PR break?
I don't think it will break much, as any users of those extra parameters will still see them as API options.  This should only improve users of BufferIf that only provide a pause observable.
Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Heya!  Not too familiar /w doing pull requests as I haven't done too many before.  Let me know if you need anything tweaked or addressed.  8)

Hey @noggog. Thanks for doing the pr. Yeah they have a inbuilt way for us to do a review and if any problems do a review and show the block of code we are talking about. You can add a new commit to your branch/fork and we will automatically see it. If the affected line of code we indicated feedback has changed GitHub will tell us.
We could also mark for approval which indicates we are happy with the work. I often do this for dynamic data for example if I want to indicate to @rolandpheasant that I'm happy with the work.
We can merge which adds the code to the main product.
Final case that can happen is we reject. So this happens if we are planning a similar feature or if it just didn't fit with the vision of the product. Often these types of issues can be avoided by submitting an issue first and waiting for the maintainer to respond.
Bit of an overview since you said you are new to the process.

Thanks, will let @RolandPheasant review as well. But makes sense to me. (@RolandPheasant  is the original author of DynamicData)


Cleanup

I organized the images into one folder under /doc/images and change the image references accordingly.

Thanks, this is very useful.


Fix single threaded builds

The definitions of mutexes in threading.h are not protected by
#if defined(MBEDTLS_THREADING_C)

this breaks single threaded builds.

Just for clarity: This fix is only relevant for amalgamated headers. In regular cases, threading.h is never included and thus the extern calls are not exposed

Merged offline with fix in final header guard


Implement kube-linter checks in the repository

The list of checks that need to be implemented is in issue #168, as well as the documentation on how to implement the linter into the repo.

Migrated to Jira: https://issues.redhat.com/browse/RHOAIENG-1639


mdformat 0.7.19 support

Apparently mdformat 0.7.19 (silently) introduced a breaking change (release notes), which causes mdformat-pyproject to crash on invocation.
To illustrate, in the CI of scipy-stubs the following command is run:
mdformat CODE_OF_CONDUCT.md CONTRIBUTING.md README.md SECURITY.md tests/README.md --check

But since upgrading to mdformat==0.7.19, it crashes:
Traceback (most recent call last):
  File "/home/runner/work/scipy-stubs/scipy-stubs/.venv/bin/mdformat", line 8, in <module>
    sys.exit(run())
             ~~~^^
  File "/home/runner/work/scipy-stubs/scipy-stubs/.venv/lib/python3.[13](https://github.com/jorenham/scipy-stubs/actions/runs/11915079805/job/33204492222?pr=183#step:7:14)/site-packages/mdformat/__main__.py", line 8, in run
    exit_code = mdformat._cli.run(sys.argv[1:])
  File "/home/runner/work/scipy-stubs/scipy-stubs/.venv/lib/python3.13/site-packages/mdformat/_cli.py", line 125, in run
    formatted_str = mdformat.text(
        original_str,
    ...<6 lines>...
        _filename=path_str,
    )
  File "/home/runner/work/scipy-stubs/scipy-stubs/.venv/lib/python3.13/site-packages/mdformat/_api.py", line 25, in text
    mdit = build_mdit(
        MDRenderer,
    ...<2 lines>...
        codeformatters=codeformatters,
    )
  File "/home/runner/work/scipy-stubs/scipy-stubs/.venv/lib/python3.13/site-packages/mdformat/_util.py", line 39, in build_mdit
    plugin.update_mdit(mdit)
    ~~~~~~~~~~~~~~~~~~^^^^^^
  File "/home/runner/work/scipy-stubs/scipy-stubs/.venv/lib/python3.13/site-packages/mdformat_pyproject/plugin.py", line 87, in update_mdit
    cli_opts = _reload_cli_opts()
  File "/home/runner/work/scipy-stubs/scipy-stubs/.venv/lib/python3.13/site-packages/mdformat_pyproject/plugin.py", line 71, in _reload_cli_opts
    arg_parser = mdformat._cli.make_arg_parser(enabled_parserplugins, enabled_codeformatters)
TypeError: make_arg_parser() missing 1 required positional argument: 'parser_extensions'

Thanks for reporting this @jorenham
To be completely fair, the change was not silently introduced. I was warned, but when I reviewed the code the change did not look menacing, and I did not manage to make the time to fully dive into it.
Seeing that it does indeed provoke problems, I'll prioritize this and try to release a fix ASAP

It turns out that the problem is actually related to something else, not the PR about the cli arguments groups, and this is why it went in undetected.
The origin is this change in the make_arg_parser signature.
I've already fixed it and I'm now testing whether all the plugin options are being properly parsed.

v0.0.2 has been released, which fixes this issue.

I have confirmed it fixes the issue in our projects

Thanks @csala :)


Custom SID Generation

I wanted a way to provide my own method of generating session identifiers instead of the default UUID4 method. I added a new optional parameter sid_generator to each of the session interface classes which stores the value as self.sid_generator. The default value is a new function in sanic_session.utils called default_sid_generator which simply returns uuid.uuid4().hex. Docstrings have been updated where available and unit tests have been written to test custom SID generation as well as UUID4 validation for the default generator.

Hello.
I'd like to suggest such approach:
sid_generator: Callable[[], str] = uuid4

instead of such:
sid_generator: Callable[[], str] = default_sid_generator

, as I think, it's more obvious and doesn't require reading utils.py to figure out which method is used by default.
Thanks.

Now PR conflicts with code, I close this issue. Please open new PR.


Create SEO/webmaster to get gp2040-ce.info into webcrawlers

We should have some kind of SEO/webmaster account for gp2040-ce.info to get into webcrawlers.
From FeralAI:
One other note, we may want to submit gp2040-ce.info to search engines manually for crawling/indexing. I haven't done this since the project should have control of that and I don't believe there are any SEO/webmaster accounts set up to handle this.

We are going to close this for now as project growth and discoverability has been good.  In the future we may look at promoting more via search results.


Regtest wallet will brick after 1000 inscriptions

I have been running into an unusable regtest after a few batch inscriptions totaling in thousands of inscriptions. This might be related to 3130
Running on V 0.16 after 1000 inscriptions on the default ord wallet
ord -r wallet --server-url http://localhost:8080 outputs
error: error sending request for url (http://localhost:8080/output/e606c6518bdcfafbb3da7577ea3d20aaf76352a71945d15c3131984cae62f382:226): connection error: Connection reset by peer (os error 54)
because: connection error: Connection reset by peer (os error 54)
because: Connection reset by peer (os error 54)

I can see the output url on the regtest server. Each time I run the command its a new output its erroring on.
I get 1672outputs on the ord wallet using:
bitcoin-cli -regtest -rpcwallet=ord listunspent | jq length

This is probably because the wallet makes a request to the server for each output in the wallet, to see which inscriptions it contains. We can make these a single request, which should make it much faster.

I'm hitting this error too on mainnet 0.17.1.  Wallet has 1775 inscriptions and I get the connection reset error when doing any wallet command.
Also on 0.16.0 and later, the error originally manifested as "too many open files":
error: error sending request for url (http://<IP_ADDRESS>/output/<redacted>): error trying to connect: tcp open error: Too many open files (os error 24)
because: error trying to connect: tcp open error: Too many open files (os error 24)
because: tcp open error: Too many open files (os error 24)
because: Too many open files (os error 24)

I'm on a mac (monterey).  A stackoverflow post suggested increasing the open file limit with ulimit -n 10240.  That seemed to fix the "too many open files" error, but then I started getting the "connection reset" error.
For now I'm creating a new wallet to do new stuff, and to access my 1775 inscriptions it seems like downgrading to 0.15.0 is the only thing that works (annoying bc there are no runes in <0.17.0!)

Yeah, it's definitely the wallet. We'll have to add an RPC endpoint that takes a list of utxos and return information about them in one request. Instead of doing a request for every UTXO.

That makes sense, thanks for the reply.  Obviously this is really bad for power ord client users (on macs?), as their wallets are basically bricked in 0.16.0+ :( :( :(
As a temporary workaround I created a new wallet for new things, but would be great to be able to manage my existing assets from 0.16.0+

Agree this should be high priority as you cannot run any wallet commands.  I cannot transfer my inscribed collection after going over ~200 inscriptions on mainnet.  Even ord wallet balance breaks
wonder if executing ulimit -n 1024 will work as a temporary workaround
https://github.com/ordinals/ord/issues/3421
default max number of open files for me was:
$ ulimit -a |grep "open files"
open files                      (-n) 256

ok this worked for me
Before starting the ord server and before running the ord wallet commands, I ran:
ulimit -Sn 512

recommend including the S in the switch so it only sets the soft limit - allows you to increase it later

I am getting this same error trying to inscribe on mainnet.
It started after i did a batch inscribe of 1000... Which worked. Now I am unable to inscribe or do anything with that wallet.
ulimit -Sn doesnt fix it.

any update on this...? workaround?

I have found that doing ulimit -n 4096  before running ord server + before running ord commands does help. Any higher doesn't seem to make a difference.. Also make sure u type it in every terminal window before running ord.
But this only works up to a certain amount - seems to be when ord hits around 500+ Disk Threads in Activity Monitor.
Only seems to effect apple silicon devices..

0.18.3 brings some improvements to this. Also this pending PR #3655

As Raph mentioned on twitter, PR #3621 should have also helped to alleviate these issues.  @elocremarc @dozer-eth @bg-bm @10xdegen @freaktree - could you please test with 0.18.5 and see if this issue still exists? and if so, is it less severe?

Hm now I'm getting these errors:
error: JSON-RPC error: transport error: Couldn't connect to host: Resource temporarily unavailable (os error 35)

when running ord wallet balance and ord wallet inscriptions (i have ord server running in a separate tab).  One conflating factor is I have an external SSD and my original cord broke, so I'm using a (potentially) slower temporary replacement.
But this is concerning bc my previous workaround of downgrading to 0.15.0 will no longer work as I've been airdropped runes on that wallet, so risk accidentally spending those runes if using a runes-unaware version of ord.
Is anyone else also getting the error above with 0.18.5 or just me?  @elocremarc @bg-bm @10xdegen @freaktree

I added export RUST_LOG=DEBUG before running ord server and ord wallet balance.  At the time I ran this
843196 was the latest confirmed block height.  This is what I see:
-> ord server
[2024-05-12T21:22:52Z DEBUG hyper::proto::h1::io] parsed 2 headers
[2024-05-12T21:22:52Z DEBUG hyper::proto::h1::conn] incoming body is empty
[2024-05-12T21:22:52Z DEBUG hyper::proto::h1::io] flushed 361 bytes
[2024-05-12T21:22:53Z DEBUG bitcoincore_rpc] JSON-RPC request: getblockcount []
[2024-05-12T21:22:53Z INFO  ord::settings] Connecting to Bitcoin Core at <IP_ADDRESS>:8332/
[2024-05-12T21:22:53Z INFO  ord::settings] Using credentials from cookie file at `<redacted>`
[2024-05-12T21:22:53Z DEBUG bitcoincore_rpc] JSON-RPC request: getblockchaininfo []
[2024-05-12T21:22:53Z DEBUG bitcoincore_rpc] JSON-RPC request: getnetworkinfo []
[2024-05-12T21:22:53Z DEBUG bitcoincore_rpc] JSON-RPC request: getblockhash [843197]
[2024-05-12T21:22:53Z DEBUG bitcoincore_rpc] JSON-RPC error for getblockhash: RpcError { code: -8, message: "Block height out of range", data: None }
[2024-05-12T21:22:53Z DEBUG ord::index::updater] Outpoint channel closed
[2024-05-12T21:22:58Z DEBUG bitcoincore_rpc] JSON-RPC request: getblockcount []
[2024-05-12T21:22:58Z INFO  ord::settings] Connecting to Bitcoin Core at <IP_ADDRESS>:8332/
[2024-05-12T21:22:58Z INFO  ord::settings] Using credentials from cookie file at `<redacted>`
[2024-05-12T21:22:58Z DEBUG bitcoincore_rpc] JSON-RPC request: getblockchaininfo []
[2024-05-12T21:22:58Z DEBUG bitcoincore_rpc] JSON-RPC request: getnetworkinfo []
[2024-05-12T21:22:58Z DEBUG bitcoincore_rpc] JSON-RPC request: getblockhash [843197]
[2024-05-12T21:22:58Z DEBUG bitcoincore_rpc] JSON-RPC error for getblockhash: RpcError { code: -8, message: "Block height out of range", data: None }
[2024-05-12T21:22:58Z DEBUG ord::index::updater] Outpoint channel closed
[2024-05-12T21:23:03Z DEBUG bitcoincore_rpc] JSON-RPC request: getblockcount []
[2024-05-12T21:23:03Z INFO  ord::settings] Connecting to Bitcoin Core at <IP_ADDRESS>:8332/
[2024-05-12T21:23:03Z INFO  ord::settings] Using credentials from cookie file at `<redacted>`
[2024-05-12T21:23:03Z DEBUG bitcoincore_rpc] JSON-RPC request: getblockchaininfo []
[2024-05-12T21:23:03Z DEBUG bitcoincore_rpc] JSON-RPC request: getnetworkinfo []
[2024-05-12T21:23:03Z DEBUG bitcoincore_rpc] JSON-RPC request: getblockhash [843197]
[2024-05-12T21:23:03Z DEBUG bitcoincore_rpc] JSON-RPC error for getblockhash: RpcError { code: -8, message: "Block height out of range", data: None }
[2024-05-12T21:23:03Z DEBUG ord::index::updater] Outpoint channel closed
[2024-05-12T21:23:07Z DEBUG hyper::proto::h1::conn] read eof

-> ord wallet balance
[2024-05-12T21:22:52Z INFO  ord::settings] Connecting to Bitcoin Core at <IP_ADDRESS>:8332/
[2024-05-12T21:22:52Z INFO  ord::settings] Using credentials from cookie file at `<redacted>`
[2024-05-12T21:22:52Z DEBUG bitcoincore_rpc] JSON-RPC request: getblockchaininfo []
[2024-05-12T21:22:52Z DEBUG bitcoincore_rpc] JSON-RPC request: getnetworkinfo []
[2024-05-12T21:22:52Z DEBUG bitcoincore_rpc] JSON-RPC request: getnetworkinfo []
[2024-05-12T21:22:52Z DEBUG bitcoincore_rpc] JSON-RPC request: listwallets []
[2024-05-12T21:22:52Z DEBUG bitcoincore_rpc] JSON-RPC request: getwalletinfo []
[2024-05-12T21:22:52Z DEBUG bitcoincore_rpc] JSON-RPC request: listdescriptors []
[2024-05-12T21:22:52Z DEBUG bitcoincore_rpc] JSON-RPC request: getblockcount []
[2024-05-12T21:22:52Z DEBUG reqwest::connect] starting new connection: http://<IP_ADDRESS>/
[2024-05-12T21:22:52Z DEBUG hyper::client::connect::http] connecting to <IP_ADDRESS>:80
[2024-05-12T21:22:52Z DEBUG hyper::client::connect::http] connected to <IP_ADDRESS>:80
[2024-05-12T21:22:52Z DEBUG hyper::proto::h1::io] flushed 71 bytes
[2024-05-12T21:22:52Z DEBUG hyper::proto::h1::io] parsed 9 headers
[2024-05-12T21:22:52Z DEBUG hyper::proto::h1::conn] incoming body is content-length (6 bytes)
[2024-05-12T21:22:52Z DEBUG hyper::proto::h1::conn] incoming body completed
[2024-05-12T21:22:52Z DEBUG hyper::client::pool] pooling idle connection for ("http", <IP_ADDRESS>)
[2024-05-12T21:22:52Z DEBUG bitcoincore_rpc] JSON-RPC request: listunspent []
[2024-05-12T21:23:07Z DEBUG bitcoincore_rpc] JSON-RPC failed parsing reply of listunspent: JsonRpc(Transport(SocketError(Os { code: 35, kind: WouldBlock, message: "Resource temporarily unavailable" })))
error: JSON-RPC error: transport error: Couldn't connect to host: Resource temporarily unavailable (os error 35)

Let me know if there's any other output I can provide.  This is quite urgent as I need to access some assets in here soonish, but don't have a safe way to via ord client atm (I could prob use sparrow but I'm not sure what precautions I need to take with runes?)

Is there any way to find out the best way to get started with wallets and stacking sats I have a project that is all ready for deployment I'm just inexperienced so I am not sure how to push it out there

Thanks to the patience and diligence of Greg from the Ordicord, we were able to figure out that upgrading bitcoin core from 24.0.1 to 27.0 fixed it!
ord wallet balance still returns an error initially before the default "ord" wallet is loaded.  But after the wallet loaded I called it again and it worked!
Also prior to that, Greg helped manually craft txns with bitcoin-cli to move the inscriptions I needed to move (many thanks!!!).


[TASK] Time dependent Adaptive Monte Carlo

Issue Description
Is your feature request related to a problem? Please describe.
Adaptive Monte Carlo sampler should be able to handle time-dependent variables
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[ ] 1. Is it tagged with a type: defect or task?
[ ] 2. Is it tagged with a priority: critical, normal or minor?
[ ] 3. If it will impact requirements or requirements tests, is it tagged with requirements?
[ ] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
[ ] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[ ] 1. If the issue is a defect, is the defect fixed?
[ ] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[ ] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[ ] 4. If the issue is a defect, does it impact the latest release branch? If yes, is there any issue tagged with release (create if needed)?
[ ] 5. If the issue is being closed without a pull request, has an explanation of why it is being closed been provided?

archived


Verification of the response

Coverage decreased (-0.2%) to 88.31% when pulling 3647fdfc95e0317d0d89e6ec1763cb1ffba3b04d on dev_verification_of_the_response into aa8812edf4785fd4a6ac47904501b416f1114e8a on master.

Coverage increased (+1.4%) to 89.992% when pulling c1dee9be80acfe641799fc991ecdd5fc7da5cbc9 on dev_verification_of_the_response into aa8812edf4785fd4a6ac47904501b416f1114e8a on master.


Create mage-training-arena-telekinetic-puzzle

Create tiles for the Mage Training Arena for the Telekinetic puzzle room. The tiles are a refinement of the most efficient tile position to cast telegrab to save as many seconds on this horrible grind.
The tiles maximize the amount of tiles moved while the maze guardian is sliding, meaning that movement is prioritized only when unable to cast the next telegrab on the guardian. They also minimize the amount of time standing and waiting for the guardian to stop moving.
This is an improvement over the modest attempt for the tiles included in the MTA plugin when I spent time optimizing the puzzles to remove much of the inefficient standing around.

Thanks for the tiles! I'll check them out and merge if it all looks good :+1:

Hey Jamie, yeah, I just created these myself, no video or anything, just a lot of telekinetic puzzle grinding had me create these. I'll try to get this updated in the next few days

Any update on this?


Different GIDs needed by animation "members" and tile map.

Tiled stores GIDs for tiles starting with 1. The "members" property of tiles starts at 0. These should be alligned to prevent confusion.

Fixed in 5afaad84595e80b3004b9a0012bd73e825d1bd11.


Evaluate the need for pytestPath and noseTestPath in settings.json

Check if this is used today
If we don't know, then add telemetry to track its usage

As I mentioned on #5970, consider replacing python.testing.*{Path,Args} with python.testing.*Task.
These would be string settings selecting the name of one of tasks inside launch.json. The widget for selecting values for this setting should limit itself to tasks of "type": "python".

We're likely going to maintain pytestPath, but nose test has been deprecated already.  Closing this.


force_reply does not work in Topic aka Forum chats

Hi. This is easy to reproduce. Simply have your bot send a message with reply_markup {force_reply true} as per documentation, You'll find that a reply to message interface pops up as expected in (a) private chats, (b) group chats. But the moment you convert to Topics aka Forum, message will arrive but no force reply interface will show - chat will receive an ordinary message from your bot without forcing the user to reply. Playing with selective and replying to user directly or @user in text to ensure correct target is chosen makes no difference.
Kind of a big problem tbh. Thank you
Telegram  v <IP_ADDRESS>866 installed from Apple AppStore

It is a bug of the app that you use. You can create an issue for the app at https://bugs.telegram.org, or upvote one if it has been created already.

my bad, moving there and closing


docs(README): add wink-nlp text and links

Coverage remained the same at 100.0% when pulling cd739f4d9c97a318c2c420db1e9b302facadedeb on prtksxna:patch-6 into 8228d46b60a269ebd2bc990e0be77afa62aae3dd on winkjs:master.


Review: AI Gen Examples for ma.put

Link to your fork:
(https://github.com/numpy/numpy/compare/main...otieno-juma:numpy:ai-examples-ma-put)
Checks

spin build && python -m pip install . && spin docs && python tools/refguide_check.py --doctests

[x] All tests have passed
[x] A visual inspection of the generated docs is correct

Acceptance Criteria:

Close when the following are complete, in order:

[ ] Reviewed by @bmwoodruff
[ ] Any suggestions to fork have been merged.
[ ] Reviewed by Numpy Mainter:

When all tasks are complete:

[ ] Submitted as PR to NumPy.

@otieno-juma , It looks like AI decided to add a : to the end of examples.  You'll need to remove the colon, and then extra - below it.  The examples checkout fine.
I'm glad you submitted this one, as it is something we'll have to check as we process the examples.
Have you tried using https://github.com/possee-org/genai-numpy/blob/main/tools/review/example-checker.ipynb to generate correct output?
When you get the change made, ping me here.

I have made all necessary changes and combined my commits into one.

Looks good to me.  I'll put this on a list for Chuck to look at.

@charris, this is ready for your review.

@otieno-juma , you can send this to Numpy as a PR.


[D2] 3.3 Indicate Interfaces used

TODO: Indicate interfaces used from other groups in the component diagram
To be completed on:

Interface
Providing To

IRequest
Module 2 Team 3

IPerformanceEvaluation
Module 2 Team 3

IQuarterlyReport
Module 3 Team 2, Team 5 (Own)

ISearchSystem
Module 3 Team 5 (Own)

IFilterSystem
Module 3 Team 5 (Own)

ICriteria
Module 3 Team 5 (Own)

IReportFormat
Module 3 Team 5 (Own)

IAnalysisAlgorithm
Module 3 Team 5 (Own)

Checked By:

[ ] Yi Kiat
[ ] Ai Xin
[ ] Guan Lin

Review and closed on: 31/3/2023


Add clipBehavior to ListView, GridView, PageView

These widgets are missing from
https://github.com/flutter/flutter/pull/59364
With this change, developers can use clipBehavior for
https://github.com/flutter/flutter/issues/59424

I searched for the usage of pushClipRect globally, I am not sure are those widgets OverLay AnimatedSize AndroidView need too.

Looks like the analyzer is not happy about this.

I searched for the usage of pushClipRect globally, I am not sure are those widgets OverLay AnimatedSize AndroidView need too.

Yes, I should add it to them. Created https://github.com/flutter/flutter/issues/63246 to track the progress. I'll do it in another PR.


Fix #9612: Set node content for astra db

Description
Adds a check for the _node_content field for documents that may not have been originally indexed by Llama
Fixes #9612
Type of Change

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

[x] I stared at the code and made sure it makes sense

Suggested Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I ran make format; make lint to appease the lint gods

@nerdai small fix for Astra, basically to support setting the _node_content attribute if the document wasnt originally ingested by llama. Thank you!!

@hatianzhang @logan-markewich could you please review and merge?

Thanks very much @hatianzhang ! Let me know if there's anything you guys need before merging.


fix(deps): update gomod

This PR contains the following updates:

Package
Type
Update
Change

github.com/go-git/go-git/v5
require
minor
v5.4.2 -> v5.5.0

github.com/gruntwork-io/terratest
require
patch
v0.41.3 -> v0.41.4

Release Notes

go-git/go-git
v5.5.0
Compare Source
What's Changed

*: add collision resistent SHA1 implementation by @pjbgf in https://github.com/go-git/go-git/pull/618
*: replace go-homedir with os.UserHomeDir by @mvdan in https://github.com/go-git/go-git/pull/535
Remote: add RemoteURL to {Fetch,Pull,Push}Options by @noerw in https://github.com/go-git/go-git/pull/375
Remote: Push, add support to push commits per hashes by @tjamet in https://github.com/go-git/go-git/pull/325
Remote: Push, add ForceWithLease Push Option by @john-cai in https://github.com/go-git/go-git/pull/404
Remote: PushOptions add push-options by @S-Bohn in https://github.com/go-git/go-git/pull/399
Remote: Push, add atomic to push options by @john-cai in https://github.com/go-git/go-git/pull/406
Remote: add FollowTags option for pushes by @john-cai in https://github.com/go-git/go-git/pull/385
Worktree: use syscall.Timespec.Unix by @tklauser in https://github.com/go-git/go-git/pull/437
Worktree: Checkout, simplified sparse checkout by @john-cai in https://github.com/go-git/go-git/pull/410
Repository: don't crash accessing invalid pathinfo by @muesli in https://github.com/go-git/go-git/pull/443
storage: filesystem, switch from os.SEEK_* to io.Seek* by @abhinav in https://github.com/go-git/go-git/pull/421
config: add branch description support by @ninedraft in https://github.com/go-git/go-git/pull/409
revision: fix endless looping in revision parser by @michenriksen in https://github.com/go-git/go-git/pull/475
pumbling: optimise zlib reader and consolidate sync.Pools by @pjbgf in https://github.com/go-git/go-git/pull/608
pumbling: parse optimisations by @pjbgf in https://github.com/go-git/go-git/pull/602
plumbing: object, rename calculation uses too much memory by @jfontan in https://github.com/go-git/go-git/pull/503
plumbing: protocol/pakp and server, include the contents of GO_GIT_USER_AGENT_EXTRA. Fixes #529 by @stewing in https://github.com/go-git/go-git/pull/531
plumbing: protocol/pakp, avoid duplicate encoding when overriding a Capability value. by @tylerchr in https://github.com/go-git/go-git/pull/521
plumbing: protocol/pakp, update agent by @caarlos0 in https://github.com/go-git/go-git/pull/453
plumbing: protocol/pakp: Actions should have type Action by @abhinav in https://github.com/go-git/go-git/pull/420
plumbing: protocol/pakp: allow unsupported multi_ack capability by @pjbgf in https://github.com/go-git/go-git/pull/613
plumbing: transport/ssh, auto-populate HostKeyAlgorithms. Fixes #411 by @evanelias in https://github.com/go-git/go-git/pull/548
pumbling: format/packfile, resolve external reference delta by @ga-paul-t in https://github.com/go-git/go-git/pull/392
plumbing: format/packfile, prevent large objects from being read into memory completely by @zeripath in https://github.com/go-git/go-git/pull/330
plumbing: format/index, support v3 index by @john-cai in https://github.com/go-git/go-git/pull/407
plumbing: format/gitignore, Read .git/info/exclude file too. by @enisdenjo in https://github.com/go-git/go-git/pull/402
plumbing: format/gitattributes, Avoid index out of range  by @To1ne in https://github.com/go-git/go-git/pull/598
plumbing: format/config, Branch name with hash can be cloned. Fixes #309 by @dowy in https://github.com/go-git/go-git/pull/354
go.mod: update github.com/xanzy/ssh-agent to v0.3.1 by @tklauser in https://github.com/go-git/go-git/pull/403
go.mod: update dependencies to remove supply chain CVEs by @pjbgf in https://github.com/go-git/go-git/pull/620
examples: added "tag find if head is tagged" by @snebel29 in https://github.com/go-git/go-git/pull/374
examples: remote fix typo by @nep-0 in https://github.com/go-git/go-git/pull/408

Full Changelog: https://github.com/go-git/go-git/compare/v5.4.2...v5.5.0

gruntwork-io/terratest
v0.41.4
Compare Source

Modules affected

k8s

Description

Add support for Kubernetes network policies

Special thanks

Special thanks to the following users for their contribution!

@baquintanag

Related links

https://github.com/gruntwork-io/terratest/pull/1208

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

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


fixed requested changes

moved orbitcode to scene and deleted orbit.js
Removed the console logs i missed

I apologise, completely my fault for overlooking what I wrote. I meant move the orbit code FROM scene not TO scene. Sorry again


Move uglify invocation into npm scripts and remove grunt

Unfortunately the uglifyjs command line ends up being a little unwieldy, but that's a small price to pay for dropping grunt!
If the 'watch' functionality is useful to folks, I can add one of the options from https://www.keithcirkel.co.uk/how-to-use-npm-as-a-build-tool/#watch

Oh dear, windows' lack of globbing strikes again. I'll figure something out.

Some notes:

Windows doesn't support globbing (client/js/libs/*.js doesn't expand to the list of js files in windows shell)
Most nodejs libraries seem to solve this by adding a dependency on glob (which npm itself depends on), and parsing their cmd-line with it
UglifyJS is a unique snowflake and implements their own version of globbing, which will only glob-expand once per argument, so foo/*/*.js doesn't work.
Ultimately I think these libs/ should be either moved to dependencies installed via npm, included in the main bundle by import, or removed altogether.
Uglify can then be part of the webpack build process, and this npm script will go away.

So I think it's OK to explicitly list out the subdirectories for now.

First of, amazing, I am so happy someone tackled this!

Windows doesn't support globbing (client/js/libs/*.js doesn't expand to the list of js files in windows shell)

Guess what, I think we added AppVeyor right after the last mess up there as well :D

Ultimately I think these libs/ should be either moved to dependencies installed via npm, included in the main bundle by import, or removed altogether.
Uglify can then be part of the webpack build process, and this npm script will go away.

Agreed.

So I think it's OK to explicitly list out the subdirectories for now.

Added a comment to try with **.
EDIT: Their simple_glob function, which really should be called simple_glob_placebo, doesn't seem to support ** either, sigh...
So yeah, having the libs in the repo was always a non-permanent solution, and we've made progress on that, so I'm happy with the current solution unless we have a better idea.

** doesn't even work on OSX, because npm scripts are executed with sh, not
bash, and sh doesn't support ** :(
On Thu, Sep 22, 2016 at 22:43 Jérémie Astori<EMAIL_ADDRESS>wrote:

First of, amazing, I am so happy someone tackled this!

Windows doesn't support globbing (client/js/libs/*.js doesn't expand
to the list of js files in windows shell)

Guess what, I think we added AppVeyor right after the last mess up there
as well :D

Ultimately I think these libs/ should be either moved to
dependencies installed via npm, included in the main bundle by import,
or removed altogether.
Uglify can then be part of the webpack build process, and this npm
script will go away.

Agreed.
So I think it's OK to explicitly list out the subdirectories for now.
Added a comment to try with **.
EDIT: Their simple_glob function, which really should be called
simple_glob_placebo, doesn't seem to support ** either, sigh...
So yeah, having the libs in the repo was always a non-permanent solution,
and we've made progress on that, so I'm happy with the current solution
unless we have a better idea.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/thelounge/lounge/pull/628#issuecomment-249107771, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAKjAExccFWyDnnV7aR9YfYTSeVfUzHuks5qs2bmgaJpZM4KEExb
.

Perhaps include uglifyjs in your webpack PR? Then the glob issues will be gone, and simplify it.

This is cross-platform (appveyor passed, as did the OSX + Linux travis builds). Folding it into the webpack PR is cleaner, though, I agree :)

I looked into folding this into #609, but ran into some problems. The libraries we include via the uglified libs.min.js are not CommonJS modules; they simply inject themselves into the global namespace. Webpack works by following require() calls, so it won't work with the libs unless we convert them all to use require/import.
IMHO this is still a win for now because we get to drop Grunt. As we further refactor the client we can shift stuff in libs/ over to CommonJS style and phase out uglify.

Not cross-platform

What's your issue with this, @xPaw? It seems like AppVeyor is happy and @nornagon kept cross-platformism in mind.

That's why:

ERROR: can't read file: client/js/libs/*.js

I get that by running npm build in powershell.

Figured it out, had to run npm install...
👍

Awesome stuff, thanks @nornagon!


File type incorrectly identified as C++ rather C depending on filename

VSCode Version: Code 1.14.1 (2648980a697a4c8fb5777dcfb2ab110cec8a2f58, 2017-07-13T19:18:47.188Z)
OS Version: Windows_NT ia32 6.1.7601
Extensions:

Extension
Author (truncated)
Version

language-x86-64-assembly
13x
2.2.11

rainbow-brackets
2gu
0.0.6

read-only-indicator
ale
0.6.0

alignment
ann
0.3.0

Doxygen
bbe
1.0.0

python
don
0.6.7

EditorConfig
Edi
0.9.4

matlab
Gim
0.5.0

triggertaskonsave
Gru
0.1.2

vscode-dimmer
hoo
2.1.0

docthis
joe
0.4.7

select-part-of-word
mle
1.0.1

cpptools
ms-
0.12.1

indent-rainbow
ode
0.6.1

vscode-clearcase
Ope
1.10.0

vscode-icons
rob
7.10.1

indenticator
Sir
0.4.2

vscode-hexdump
sle
1.4.0

code-spell-checker
str
1.3.0

comment-snippets
Tob
0.2.0

selectline-statusbar
tom
0.0.2

change-case
wma
1.0.0

highlight-trailing-white-spaces
yba
0.0.2

(2 theme extensions excluded)

Steps to Reproduce:

Load file called dl_srp_ce.c (and many others with similar names).
Shows as C++ at both top of tab and in file browser window and status bar.
Rename the file and reload and it correctly shows as C

Reproduces without extensions: Yes, still fails without extensions (as seen in status bar)

I cannot reproduce:

Sorry I meant the filename was "dl_srp_ce.c".

@BillDenton also not reproducible, what icon theme are you using? I am using seti:

I'm using:
"workbench.iconTheme": "vscode-icons",
https://marketplace.visualstudio.com/items?itemName=robertohuertasm.vscode-icons

Without extensions (code --disable-extensions) and no changes in settings:

Without extenssions After changing settings:
"workbench.iconTheme": "vs-seti",

Tried again in a folder that doesn't have a .vscode directory and it is OK:

This is still without any extensions. The "settings.json" is in C:\Users<my username>\AppData\Roaming\Code\User. So looks like another settings problem. I'll diff this settings.json with my workspace one.

Found the problem. My workspace settings file has a lot of ""files.associations" including:
"dl_srp_ce.c": "cpp",
I want to use the same settings for all workspaces, so I'll delete my workspace settings.json. It only contains file.associations. I've obviously got confused at some point in the past. Apologies for wasting your time.

I'm not sure how this happened, but I think it may be due to the "Change Language Mode" keyboard shortcut, so I've disabled that. BTW: It would be useful to have a quick way of disabling keyboard mappings from the GUI. The "caused-by-extension" label should be removed, because it was caused by confusion between workspace and user settings.

I'm still having the same problem. My workspace settings (settings.json in .vscode) is slowly filling up ""files.associations". The list appears to start from the list in my user settings, but then includes an assortment of files that I have opened (just the filename, not path). Some of these have the correct language type, but others are wrong (e.g. cpp for a .c) file. Could be an extension. I haven't tried running for a significant time with extensions disabled.

This looks like https://github.com/Microsoft/vscode/issues/27404, as reported fixed in C/C++ for Visual Studio Code Version 0.12.2. After installing that update it looks like the problem report in my last comment has gone.

This looks like https://github.com/Microsoft/vscode/issues/27404, as reported fixed in C/C++ for Visual Studio Code Version 0.12.2. After installing that update it looks like the problem report in my last comment has gone. No it hasn't after exiting and opening VS Code, I have in my workspace settings the contents of both my user settings plus "filename.c": "c" (same for "h") for some of the files I have opened.


DateTimeOffset loses time zone offset

DateTimeOffset is stored as UTC ticks. The consequence is that the data read back lacks the time zone offset; however, it still accurately represents the correct moment in time.
Store this: "2014-06-11T20: 24: 01.8325787-07: 00"
You read back: "2014-06-12T03: 24: 01.8325787+00: 00"
They represent the same moment in time, but the first contained a timezone offset.
A work around is to store a DateTimeOffset's 'Offset' and 'Ticks' separately and rebuild an accurate DateTimeOffset on read. This does mean range queries are more difficult.

Anything on this?

Any other workaround or library that supports this?

Work around is to save both parts separatly in the database and build them back on load.

Yeah I just noticed this myself, sorry for the trouble.
My thoughts are to store the offset in an auxiliary column and keep the UTC date time in the main column. This should allow compares to keep working and will only fail if people write their own queries.

Just had this bite me.


Re-wrote tools sentence

Changed the sentence under Tools from "This space is intended to give new tools to beginners to start their Hacktoberfest." to "This space has new tools for beginners to start their Hacktoberfest Projects."

great :)


BundleDeployment Healthcheck

wip

closed in favour of https://github.com/operator-framework/rukpak/pull/711


Bug: Incorrect helm cd pipeline status in presence of GitOps cd pipeline.

📜 Description
After introduction of GitOps or Helm option for CD, when we create multiple cd pipelines with different deployment app types (Helm or GitOps)  ,either in parallel or in different workflows then deployment status of Helm pipeline is show as failed in "Build and Deploy" tab even if cd pipeline status is healthy in app details tab. This happens only when both GitOps and Helm pipeline are present.
👟 Reproduction steps

Create a cd pipeline with deployment app type as helm. (PipelineA)
Wait for PipelineA's deployment app status to become healthy.
Create another pipeline , either in parallel to above pipeline or create a separate workflow. (PipelineB)
After deployment app status of PipelineB become healthy, status of Pipeline A will be shown as Failed.

👍 Expected behavior
Status of PipelineA should not have been Marked as failed because in actual it is Healthy.
👎 Actual Behavior
Status of PipelineA (helm pipeline) is affected by presence of PipelineB (GitOps pipeline).
☸ Kubernetes version
1.22
Cloud provider

AWS

🌍 Browser
Chrome
🧱 Your Environment
No response
✅ Proposed Solution
No response
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find any similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

not reproducable ,already fixed


Add timeout parameter to get_joint_values states

For GetJointValuesState and GetJointValuesDynState a state can block indefinitely if parameter values are not published.
I propose adding an optional time out parameter  (default: None) that will return a 'timeout' outcome.
If no timeout value is given, then will block indefinitely.
I propose doing this in a kinetic_devel branch to avoid conflicts with existing  indigo code.

Was this ever addressed? I am looking into the possibility of having a timeout in a state, to avoid it getting stuck. Is there a way to do that?

Adding a timeout is quite straightforward and you can take a look at the flexbe_states/WaitState for an example (this state is essentially nothing else than a timeout).
Regarding this specific issue, I don't know whether somebody did it, at least there is no PR open. But feel free to open one in case you need to improve one of the state implementations here. I'm currently not working on this repo myself, but plan to revise and improve the existing state implementations once I finished my current clean-up iteration of the main repo.

I would also suggest making hardcoded join_states topic a state param so it can be configured. (To support reading joint_states from different paths on different states. e.g. in case of a multi robot coordination where different robots have joint states published on different topics.)
I would be happy to make a PR if it is a welcome enhancement.

This fell of my radar.  I'll try and submit the pull request this week


Adding require_valid_protocol option for isURL

The protocol option for the isURL validator acts as a whitelist of valid protocols, and if a protocol is present in the URL, it must match one of the whitelisted values regardless of other options.
In cases where validating the protocol is not needed or valid protocol names cannot be anticipated (for example, mobile apps can register ad hoc protocols), it would be helpful to disable protocol validation.
This pull request adds a require_valid_protocol option (set to true by default) for the isURL validator to skip checking the protocol against the protocols whitelist. Documentation and test updates are also included.

Sounds reasonable. Thanks!

Thanks for the quick review and release!


fix(exporter-metrics-otlp-grpc) Add explicit otlp-exporter-base depen…

https://github.com/open-telemetry/opentelemetry-js/issues/4676

@dyladan is this correct?


Add hot reloading

Hot reloading is a must-have for development purposes. Currently, this project lacks any form of hot reloading set up.

Added in #2. Closing this issue.


How to load custom glb model using phy engine?

I am making a web app using ThreeJS and React
It's very similar with the museum example of phy
I implemented almost of functionalities using Phy
However I encountered in issue to load custom museum model
phy.load(
'dices.glb',
onComplete
)
phy.load(
['dices.glb', 'yo.jpg'],
onComplete
)
This code are wrotten at the Phy engine guide
https://lo-th.github.io/phy/docs/index.html#manual/phy-load
But this code doesn't works in my project
I will share my main code:
const loadModels = () => {
	const models = ["./assets/models/musee.glb"];

	phy.load([...models], onCompleteLoadModels);
};

const onCompleteLoadModels = () => {}

This code occured errors at the crhome console like the following:

 index.tsx:86 SyntaxError: Unexpected token '<', "<!doctype "... is not valid JSON
at JSON.parse (<anonymous>)
at GLTFLoader.parse (Phy.module.js:17994:17)
at Object.onLoad (Phy.module.js:17889:11)
at three.module.js:44245:38

If I remove the above code, all works well
In one word, if I use simple model like box, the project works well
But if I gonna load new model, it occur error

It's a urgent project, so I would be happy anyone let me know how to overcome this issue

Best regards

Hi do you try with native three loader ?
is maybe a problem's on your model what exporter you use ? what compression ?
i only include the draco compression in phy.
you can use the loader of three if is work and use your model in phy.

Thank you for your kind support.
Regarding the model compression, I tested it using museum models in phy repository
But the error was same
When I use ThreeJS's default loader, there wasn't any problem
Could you write the code how to use loaded model that loaded through ThreeJS's default loader in phy?

you can use any model you want
in phy i use pool only to manage demo switch and don't reload model.
You have to take care of your model name and not use too complex mesh on simulation
phy.load(['./assets/models/chess.glb', './assets/models/chessclock.glb'], onComplete )
onComplete = () => {
    // get a object with list of model
    const clockMesh = phy.getMesh('chessclock');
    // add a box shape and remplace by the model. here is clockMesh.clock
    phy.add({ type:'box', size:[3,1.6,1.1], pos:[0, 0.8-0.02, -5.2], mass:1, mesh:clockMesh.clock, material:'Clock' })
}


How to make keys repeat

How would i make keys repeat when i hold down a key

Hey, I found this in the pimoroni discord: https://github.com/shane-powell/keybow-2040-gamer/blob/main/code.py

Thank you @waynelx for chiming in on this.
I'll close the issue now
Regards
Gee


docs: fedramp config exmaple

Pull Request Test Coverage Report for Build 509168629

0 of 0   changed or added relevant lines in 0 files are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 58.088%

Files with Coverage Reduction
New Missed Lines
%

pkg/metrics/sampler/sampler_routine.go
2
94.74%

pkg/integrations/v4/logs/cfg_watcher.go
4
67.5%

Totals

Change from base Build 503517880:
-0.02%

Covered Lines:
11427

Relevant Lines:
19672

💛  - Coveralls

Pull Request Test Coverage Report for Build 509168629

0 of 0   changed or added relevant lines in 0 files are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 58.088%

Files with Coverage Reduction
New Missed Lines
%

pkg/metrics/sampler/sampler_routine.go
2
94.74%

pkg/integrations/v4/logs/cfg_watcher.go
4
67.5%

Totals

Change from base Build 503517880:
-0.02%

Covered Lines:
11427

Relevant Lines:
19672

💛  - Coveralls


KAFKA-17703: Moved DelayedActionsQueue outside DelayedShareFetch

About
In reference to comments https://github.com/apache/kafka/pull/17177#issuecomment-2392017658 and https://github.com/apache/kafka/pull/17177#issuecomment-2392108397 , this PR addresses the following -

Move ActionQueue outside DelayedShareFetch class to SharePartitionManager where SharePartitionManager has the ability to add a delayed action to the ActionQueue.

Add TopicPartition as a key for delayed share fetch along with SharePartition (that is already present right now). This will be utilized later when we add the code to know if new data has been added to the topic partition.

Testing
The code has been tested with the help of unit tests.

@adixitconfluent : Currently, we check the produce/fetch purgatory when a replica becomes the follower or a replica is deleted from a broker through ReplicaManager. applyLocalFollowersDelta and ReplicaManager.applyDelta. Should we do the same for share purgatory?

@adixitconfluent : Currently, we check the produce/fetch purgatory when a replica becomes the follower or a replica is deleted from a broker through ReplicaManager. applyLocalFollowersDelta and ReplicaManager.applyDelta. Should we do the same for share purgatory?

@junrao , yes, I think we should do it. Moreover, with more use cases of delayed share fetch purgatory within ReplicaManager functionalities like the above, perform a checkAndComplete when HWM is updated and perform a shutdown of the delayed share fetch purgatory like the others in ReplicaManager, I feel its better we declare the delayed share fetch purgatory inside ReplicaManager along with the existing purgatories. We don't need a reference to this purgatory outside of ReplicaManager, we can just add accessor methods. What do you think?

Perform a shutdown of the delayed share fetch purgatory like the others in ReplicaManager
I feel its better we declare the delayed share fetch purgatory inside ReplicaManager along with the existing purgatories. We don't need a reference to this purgatory outside of ReplicaManager, we can just add accessor methods. What do you think?

We can do that too if it makes code simpler.

Also, there seems to be another existing issue. If a sharePartition is being initialized, any pending shareFetchRequest is blocked in the purgatory. So, after the initialization completes, we need to trigger a check in the share fetch purgatory to wake up pending delayed operations.

Hi @junrao, we are aware about the share partition initialization issues and we have a JIRA for it https://issues.apache.org/jira/browse/KAFKA-17510 , I've added what you mentioned in the description and it should be completed as part of that JIRA

We can do that too if it makes code simpler.
Thanks @junrao , I've created a JIRA https://issues.apache.org/jira/browse/KAFKA-17742 to address this in my future PRs.

Re-opening PR to trigger build.


remove nvm installation for SDK automation

fixes https://github.com/Azure/azure-sdk-tools/issues/7967

Rather than removing the installation steps, could we instead check and skip if nvm is already installed?

@raych1 for awareness

This init script currently is only used in the spec PR pipeline which has already installed the Node. We can ignore the NVM/Node installation in this script.

The automation probably don't need to use nvm to switch nodejs versions? Looking at the history nvm was used to install node v18, likely when 18 is not the default installed version.

The automation probably don't need to use nvm to switch nodejs versions? Looking at the history nvm was used to install node v18, likely when 18 is not the default installed version.

@jeremymeng
This PR is to revert the change added previously to install NVM to install node18 when the spec PR pipeline didn't support node18. The spec PR has already supported the node 18 and it will upgrade the node version per the lifetime. So, it doesn't need the JS language automation script to install node separately.

@jeremymeng  @raych1 I found the checkenforcer is always in pending status. but when I click button, the status is already success.

/check-enforcer override


Error in Project having UIKit and swiftUI

Getting Kingfisher.KingfisherError.ProcessorErrorReason.processingFailed(processor: Kingfisher.DefaultImageProcessor(identifier: ""), item: Kingfisher.ImageProcessItem.data(0 bytes)) on every image call

That means the downloaded data is 0 byte so it is not an image. I suggest maybe checking the actual data of your URL.

I checked that, I tried with URL in demo project but they through untrusted certificate error. they are images in that particular url.

So maybe the certificate error is the reason? Are you trying to access a domain under a self-signed cert or even within plain HTTP? Either one, maybe try to read and adjust your app with the guide here: https://developer.apple.com/news/?id=jxky8h89


feat: According to the document, supplement the missing default API

According to the document: https://casdoor.org/docs/permission/exposed-casbin-apis/
supplement the missing default API permissions.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Remove override models for next Garry's Mod update

Garry's Mod provides them now out of the box, might need to keep some.
https://commits.facepunch.com/481021
https://commits.facepunch.com/482428

The checked NPCs have their models combined by the game. NPCs written in bold are tricky to combine.

[ ] Alyx
[ ] Advisor
[ ] Antlion
[ ] Combine Dropship
[ ] Dog
[ ] Eli
[ ] Fast zombie
[ ] Barney
[ ] Blackout (EP1 Intro)
[ ] Kleiner
[ ] GMan
[ ] Lamarr
[ ] Vortigaunt
[ ] Mossman
[ ] Zombie

This can also be closed since Garry's Mod will provide all important models with their animations merged (upcoming October update).


Can you regenerate schema for a latest API?

Or just upload generator sources. Thank you!

https://core.telegram.org/bots/api#recent-changes


Ubuntu 20.04 repositories do not contain 'firmware-sof-signed' nor 'firmware-sof'

This prevents the sof-setup-audio script from finishing since it can't find and install it.
I managed to get around this issue by manually installing it from here: https://launchpad.net/ubuntu/jammy/+package/firmware-sof-signed
Then I just commented out this piece of the script;
sudo apt install -y firmware-sof-signed || sudo apt-get -o Dpkg::Options::="--force-overwrite" install firmware-sof-signed

The script completed after that.

Ubuntu 20.04 repos are to old to contain sof firmware, which is why I set the default version to 21.04. I can implement this feature, but I don't think anyone is affected by using Ubuntu 21.04 instead of Ubuntu 20.04. This could create a mini dependency hell.


Custom inherit model

I’ve an existing lua project which has a class model that one lua “class” can inherit form a C# class.
When porting this project to typescript, my plan is to write declaration class for that C# class, and have a ts class inherit from that one.
So my problem is, instead of __TS__ClassExtends , can I call a custom function in this case.

You can do it at runtime by reversing what __TS__ClassExtends does and adding you own logic: Playground

Good idea, thanks!

Well this does not work for me.
The base class has no prototype field, so things when wrong at this line in __TS__ClassExtends:
setmetatable(target.prototype, base.prototype)

I think the best thing I can do is implement my own __TS__ClassExtends method and replace the default one

In that case I'd recommend to create a plugin. Overriding lualib methods wouldn't be possible after #653

I did something like this in a lua file and require it before everything else:
require("lualib_bundle")
local vanilla__TS__ClassExtends = __TS__ClassExtends

local function __TS__ClassExtends(target, base)
	if base.prototype then
		vanilla__TS__ClassExtends(target, base)
	else
		// custom inheritance logic here
	end
end

_G.__TS__ClassExtends = __TS__ClassExtends

It looks like working as expected.

I'll see what I can do with plugins...

OK I've things up and running with a plugin now. It solves this problem and #921 for me.
I'll use the plugin in real projects, and open source it later once it's mature enough.
Thanks for all the help.


Question: Indentation control in {% include %}

Hi. I'm trying to use stencil for swift code generation (as many other existing applications of stencil) and found it very confusing when it comes to indentation control. It isn't very important for html code generation (as  jinja, liquid, django and many other implementations if this kind of templating usually do) because that html is then being interrupted by browsers and humans don't look in that geneted html. But for swift code generation it would be beneficial for generated code too look readable for humans too.
Simple example to describe the problem. Consider template
struct SomeType {
    {% include variablesSection%}
    {% include publicMethosSection %}
}

and then publicMethosSection too look like
func method1() {
    doSomething()
}

func method2() {
    doSomething()
}

Master template assumes some indentation as both includes are prefixed with 4 spaces, however when rendering includes inner templates all new lines generated by inner template break that indentation.
In terms of templating language design, what is the suitable way of ensuring that all new lines coming from include gets automatically indented same way as include clause in master template? I understand this feature  may not exist yet, but how you think it may look like?

I've been wanting this behaviour too. I guess the include block could prepend it's own indentation to every line of the included string.
As a related but different issue, it would be useful to pass in an optional context to the included file, instead of it getting the whole master context

This is fixed in #114


[DOCS] Typo mistake

Issue Description
Typo error in technologies : Shadcn
Screenshots or Examples (if applicable)

Proposed Solution (optional)
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)

@Nishitbaria Is it correct ?

Hey @Sid-80  , your contribution won’t be needed right now. I’m working on a new readme file and will be replacing the entire content.


RuntimeError with the new flux models

Congratulations and thank you on this amazing work.
Unfortunately when I try to use the flux models, I get the following error. It occurs both with t2v and i2v tasks with a different sized tensor.
  File "/mnt/efs/users/vsrinivasan/Projects/Pyramid-Flow/pyramid_dit/flux_modules/modeling_flux_block.py", line 717, in forward
    return self.processor(
  File "/mnt/efs/users/vsrinivasan/Projects/Pyramid-Flow/pyramid_dit/flux_modules/modeling_flux_block.py", line 861, in __call__
    hidden_states, encoder_hidden_states = self.varlen_attn(
  File "/mnt/efs/users/vsrinivasan/Projects/Pyramid-Flow/pyramid_dit/flux_modules/modeling_flux_block.py", line 300, in __call__
    concat_qkv_tokens[:,:,0], concat_qkv_tokens[:,:,1] = apply_rope(concat_qkv_tokens[:,:,0], concat_qkv_tokens[:,:,1], image_rotary_emb[i_p])
  File "/mnt/efs/users/vsrinivasan/Projects/Pyramid-Flow/pyramid_dit/flux_modules/modeling_flux_block.py", line 37, in apply_rope
    xq_out = freqs_cis[..., 0] * xq_[..., 0] + freqs_cis[..., 1] * xq_[..., 1]
RuntimeError: The size of tensor a (248) must match the size of tensor b (249) at non-singleton dimension 1

Any ideas on how I can fix this would be much appreciated.
Side note: I also get this warning despite having all the models downloaded from the huggingface hub.
An error occurred while trying to fetch ./models_flux/causal_video_vae: Error no file named diffusion_pytorch_model.safetensors found in directory ./models_flux/causal_video_vae.
Defaulting to unsafe serialization. Pass `allow_pickle=False` to raise an error instead.

I hope these two are not related.
Thank you very much in advance.

Hi, this warning does nit affect the normal running. I guess this bug is because you use the incorrect width or height setting? Can you share your setting?

Thank you for the prompt response.
I am using the default setting for the task i2v (and t2v).
Trying to run it on 3 GPUs in one node. No changes other than that.

What do you set the width and height? And do you use the multi-gpu inference?

The issue is fixed if I set:
GPUS=2
The issue can be reproduced by setting
GPUS=3
Setting GPUS=4 gives another new error
  File "/mnt/efs/users/vsrinivasan/Projects/Pyramid-Flow/pyramid_dit/flux_modules/modeling_flux_block.py", line 716, in forward
    return self.processor(
  File "/mnt/efs/users/vsrinivasan/Projects/Pyramid-Flow/pyramid_dit/flux_modules/modeling_flux_block.py", line 868, in __call__
    hidden_states = attn.to_out[0](hidden_states)
  File "/mnt/efs/users/vsrinivasan/Projects/Pyramid-Flow/pyramid-flow/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/mnt/efs/users/vsrinivasan/Projects/Pyramid-Flow/pyramid-flow/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/mnt/efs/users/vsrinivasan/Projects/Pyramid-Flow/pyramid-flow/lib/python3.10/site-packages/torch/nn/modules/linear.py", line 114, in forward
    return F.linear(input, self.weight, self.bias)
RuntimeError: mat1 and mat2 shapes cannot be multiplied (60x2048 and 1920x1920)

For now, the code is running for me. Its not necessary for me to use >2 GPUS (would be nice though:).
Thank you very much.


unnest is very slow .

unnest takes very long time compared to the pandas methods.
For example
import pandas as pd
from datar.all import *
d ={'name': [['a','b','c','d','e','f']]*200000}
test = pd.DataFrame(d)
test >> unnest(f.name)

# it takes 1m 37.6s

# pandas method
pd.DataFrame(np.concatenate(test.name.values))

#it takes only 2.2s

How about unchop?

@pwwang
unchop is very fast. Only 2s taken

You should use unchop in your case. See https://tidyr.tidyverse.org/reference/chop.html#ref-examples
In your case, you are unwrapping python lists not tibbles/dfs.
unnest has extra steps to parse tibbles.


Best way to cleanup flows in an environment

We have many flows in an environment and we are planning to cleanup those. Can someone suggest best approach to do that.

Hello!
Can you please ask this question in the Power Automate Community
https://powerusers.microsoft.com/t5/Microsoft-Power-Automate/ct-p/MPACommunity
In this GitHub, we only support solutions released by our team but aren't able to help with more general questions.
Thank you
Manuela


let unicorn read the right Gemfile on USR2

since unicorn knows Gemfile path in its real (not symbolic) one, it continues to use old Gemfile after receiving USR2.
my patch fixes this issue by specifying Gemfile in `current_path' (symbolic) via BUNDLE_GEMFILE environment variable.

Merged, thanks!


Floating layout engine

Hi,
I have implemented a Floating Layout Engine for those who would like to use such a layout. I always keep a workspace with a floating layout and use it for applications which do not behave as expected in other layout types.
I hope you also find it useful.
Note: the floating layout reads the Window properties (Location, Offset) in their original form. Therefore, if a Bar (or more) are defined, it needs to know the height of the Bar(s). By default, this is 30, and this is what the normal constructor does. If you have a different Bar height (e.g. 50) you want to call it in your configuration like so:
context.DefaultLayouts = () => new ILayoutEngine[] { new FullLayoutEngine(), new TallLayoutEngine(), new FloatingLayoutEngine(50) }

People in general seem kinda hesitant to review new layouts... I guess that means they don't feel the need for too many?
That said, this PR might help out people who are having issues with typical JetBrains products like Rider or CLion?
Maybe you can pull some of those users in here and see if it helps their case?
As for the code itself:

Note: the floating layout reads the Window properties (Location, Offset) in their original form. Therefore, if a Bar (or more) are defined, it needs to know the height of the Bar(s). By default, this is 30, and this is what the normal constructor does. If you have a different Bar height (e.g. 50) you want to call it in your configuration like so

Don't we have a BarPluginConfig-object type which could be used here instead, somehow?

Thanks for the PR!
A suggestion for this change: instead of returning a list of window locations that are just "the same location that each window is currently at", I wonder if it is a cleaner solution to instead provide a way for a layout engine to tell workspacer that a window should just have it's location ignored entirely, and only care about keeping it inside the current workspace. Thoughts?

This PR has been stale since @rickbutton's review/comment earlier on.
Does anyone of the current maintainers have an opinion on if this is OK now in it's current form, or if we should close this PR due to inactivity?

Closed due to inactivity


Eclipse plugin installation fails

Category: Bug
Severity: Low
Expected behaviour "./setup" should succeed without warnings about failed eclipse plugin installation
Actual/current behaviour: "./setup" outputs:

******** ATTENTION ********
WARNING: Installation of plugin AnyEditTools failed. Please try to install manually.
Installing eclipse plugin net.sf.eclipsecs from http://eclipse-cs.sf.net/update/
No Java runtime present, requesting install.

Steps to reproduce:  Remove all devon-ide introduced changes on system and run "./setup" inside devon-ide root directory again.
Comments: On macOS, Eclipse may need an additional "vm" option - either on command line or set via eclipse.ini. http://wiki.eclipse.org/Eclipse.ini#-vm_value:_Mac_OS_X_Example
I assume, the plugin install start of eclipse could just use the same additional option as the later executed eclipse start:
-vm "${JAVA_HOME}/bin/java"
macOS 10.14.4, devon-ide-scripts-3.0.0-beta2.tar.gz (from Maven)

Thanks for the feedback and the hint. I will add the vm option and also the vmargs options (as I have seen OOME once in a while) to the command for the plugin installation.

I could resolve the above error but still get this one:
Installing org.eclipse.eclemma.feature.feature.group <IP_ADDRESS>903112331.
Installation failed.
Cannot complete the install because of a conflicting dependency.
	Software being installed: EclEmma Java Code Coverage <IP_ADDRESS>903112331 (org.eclipse.eclemma.feature.feature.group <IP_ADDRESS>903112331)
	Software currently installed: EclEmma Java Code Coverage <IP_ADDRESS>809121651 (org.eclipse.eclemma.feature.feature.group <IP_ADDRESS>809121651)
	Only one of the following can be installed at once:
		Old EclEmma version (com.mountainminds.eclemma) cannot be installed together with new (org.eclipse.eclemma). Uninstall currently installed version and try the install again. <IP_ADDRESS>903112331 (com.mountainminds.eclemma.core <IP_ADDRESS>903112331)
		Old EclEmma version (com.mountainminds.eclemma) cannot be installed together with new (org.eclipse.eclemma). Uninstall currently installed version and try the install again. <IP_ADDRESS>809121651 (com.mountainminds.eclemma.core <IP_ADDRESS>809121651)
	Cannot satisfy dependency:
		From: EclEmma Java Code Coverage <IP_ADDRESS>809121651 (org.eclipse.eclemma.feature.feature.group <IP_ADDRESS>809121651)
		To: osgi.bundle; com.mountainminds.eclemma.core [<IP_ADDRESS>809121651,<IP_ADDRESS>809121651]
	Cannot satisfy dependency:
		From: EclEmma Java Code Coverage <IP_ADDRESS>903112331 (org.eclipse.eclemma.feature.feature.group <IP_ADDRESS>903112331)
		To: osgi.bundle; com.mountainminds.eclemma.core [<IP_ADDRESS>903112331,<IP_ADDRESS>903112331]
There were errors. See log file: /projects/ide-test/software/eclipse/Eclipse.app/Contents/Eclipse/configuration/1554130633195.log

Seems as if EclEmma is already part of the official Eclipse vanilla distribution and therefore we can ommit it in our plugin list. Anyhow Eclipse should perform an update as the plugin is outdated but lets forget about this detail for the moment to proceed...

Also I see this error:
Installing org.sonar.ide.eclipse.feature.feature.group <IP_ADDRESS>50804-1512-RELEASE.
Installing org.sonar.ide.eclipse.jdt.feature.feature.group <IP_ADDRESS>50804-1512-RELEASE.
Installation failed.
Cannot complete the install because one or more required items could not be found.
	Software being installed: SonarQube Java Configuration Helper <IP_ADDRESS>50804-1512-RELEASE (org.sonar.ide.eclipse.jdt.feature.feature.group <IP_ADDRESS>50804-1512-RELEASE)
	Missing requirement: Apache Apache HttpClient OSGi bundle 4.3.2 (org.apache.httpcomponents.httpclient 4.3.2) requires 'java.package; org.apache.commons.logging [1.1.0,1.2.0)' but it could not be found
	Cannot satisfy dependency:
		From: SonarQube Integration for Eclipse <IP_ADDRESS>50804-1512-RELEASE (org.sonar.ide.eclipse.feature.feature.group <IP_ADDRESS>50804-1512-RELEASE)
		To: org.eclipse.equinox.p2.iu; org.apache.httpcomponents.httpclient [4.3.2,4.3.2]
	Cannot satisfy dependency:
		From: SonarQube Java Configuration Helper <IP_ADDRESS>50804-1512-RELEASE (org.sonar.ide.eclipse.jdt.feature.feature.group <IP_ADDRESS>50804-1512-RELEASE)
		To: org.eclipse.equinox.p2.iu; org.sonar.ide.eclipse.feature.feature.group 0.0.0
There were errors. See log file: /projects/ide-test/software/eclipse/Eclipse.app/Contents/Eclipse/configuration/1554130748509.log

Seems that sonarlint has additinal dependencies to other plugins. IMHO sonarlint is too greedy anyhow and massively slows down the IDE so it should not come as default. Still it would be good to have a way for projects that want to have it to be able to install it automatically via their project specific git config (settings).

Fixed and PR #71 merged.


[MonorepoBuilder] Commands failing after migrating to PHP config

Recently we've migrated from monorepo-builder.yaml to monorepo-builder.php. We're using the validate command as part of our CI to make sure all packages are using the same dependencies version.
Since that change we're getting an error because this command is explicitly assuming that the config file is monorepo-builder.yaml however this file doesn't exist since we're using the PHP config.
https://github.com/symplify/symplify/blob/282a9c804a90c1ed3f6005c03029a70e34e8d17f/packages/monorepo-builder/src/VersionValidator.php#L25
The file above is causing the error when running validate, but after a quick search it's not the only place that refers to the config file statically.
So my question is what is the best way to deal with that? Should we set the config file name in the container so each service that need it can retrieve it?

How could we fix it here, so it's not a problem?

Not sure because this file name is returned within the conflicting versions array and the console reporter is trying to instantiate a SmartFileInfo for it. So if we don't want to have to modify everything using this array I think we need to find a way to resolve the right suffix.

@TomasVotruba I was thinking about setting the config filename as a container parameter so we can reuse it within services. The MonorepoBuilderKernel would be a good place for that as it knows about the config file and it has access to the container builder.
What do you think?

Why not, go for it!

Cc @samsonasik Could you look at this bug?

@TomasVotruba could you give me information for files that need to be checked/modified/added and step to reproduce? Thank you.

I think the goal is to explicitly check for yml + php here:
https://github.com/symplify/symplify/blob/282a9c804a90c1ed3f6005c03029a70e34e8d17f/packages/monorepo-builder/src/VersionValidator.php#L25
And in places where it's applied

Thank you, I will check.


New-HTMLTableRowGrouping for more then one field

Hey Master PrzemyslawKlys,
I was to do subgrouping for a table, anyway this can be done with  New-HTMLTableRowGrouping ? since it looks like it takes only one field.
for example, i have a global user list, i want to group them by geographic location, and then in that geo grouping, to group them again by manager.

@PrzemyslawKlys ,  any thoughts would be appreciated :)

I did quick implementation and I think I did it using a plugin
                                        rowGroup: {
                                            // Uses the 'row group' plugin
                                            dataSrc: 2,
                                            startRender: function (rows, group) {
                                                var collapsed = !!collapsedGroups[group];

                                                rows.nodes().each(function (r) {
                                                    r.style.display = collapsed ? 'none' : '';
                                                });

                                                var toggleClass = collapsed ? 'fa-plus-square' : 'fa-minus-square';

                                                // Add group name to

                                                return $('
')
                                                    .append('' + ' ' + group + ' (' + rows.count() + ')	')
                                                    .attr('data-name', group)
                                                    .toggleClass('collapsed', collapsed);
                                            },
                                        },

While what I can see:

https://datatables.net/extensions/rowgroup/examples/initialisation/simple.html
https://datatables.net/extensions/rowgroup/examples/initialisation/multipleGroups.html

        rowGroup: {
            dataSrc: [ 5, 1 ]
        },
        columnDefs: [ {
            targets: [ 1, 2 ],
            visible: false
        } ],

Is much easier but also with more options.


release candidate 0.2.0

This is an ongoing PR to finalize a candidate for our next release.

@danielhuppmann can you please take a look at 30e067d. This deprecates the "filters" argument and touched more code than I expected. Can you especially take a look at the tutorial notebook on "validation"?

Coverage decreased (-0.07%) to 91.844% when pulling 30e067d2e552c14f29a6ee42d5536b923ad7d04b on gidden:rc into bf7d87ed59a7388f744dc2d9379deeb6159af912 on IAMconsortium:master.

FYI @danielhuppmann I doubled checked the published docs versus a local build, and the output cells are the same

@danielhuppmann and @znicholls could you take a look at e00085c58941c8c and suggest any more "highlights" please!


imageglass-moon: add version <IP_ADDRESS>

Relates to https://github.com/ScoopInstaller/Extras/pull/8140 - while working on this I noticed there is a beta/development build for ImageGlass which is not in Versions, so I made this PR!

It is mostly based on the Extras imageglass manifest, with my fixes from the PR linked above.

[x] I have read the Contributing Guide.

/verify


Add SVG push support

SVG is a common image type used on the web and would also benefit from being pushed from the server.

Test works for me. I'm not sure if you want any additional tests. If so, let me know
$ ./vendor/bin/phpunit vendor/jacobbennett/laravel-http2serverpush/tests/AddHttp2ServerPushTest.php
PHPUnit 5.7.27 by Sebastian Bergmann and contributors.

..........                                                        10 / 10 (100%)

Time: 597 ms, Memory: 6.00MB

OK (10 tests, 21 assertions)


role update: search all assigned scopes for existing role definitions

[x] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[x] I adhere to the Command Guidelines.

@adewaleo, could you please review?


58 add documentation for resource collection

Q
A

Branch
6.2.x

Bug fix?
no

New feature?
no

BC breaks?
no

Deprecations?
no

Fixed issues
chameleon-system/chameleon-system#58

License
MIT

@UlrichKu I made some changes in the file - are you OK with that?

Looks good.

Thanks @UlrichKu


Add perms so I can cut alpha releases of random-job-builder

Description
Requesting permissions to cut releases of https://github.com/jenkinsci/random-job-builder-plugin so that I can cut alpha/beta releases of the new load generators work pre-Jenkins World (since @stephenc is on vacation and won't be able to review for full merge).
Requesting from @stephenc - thanks!
When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] All newly added users have logged in to Artifactory at least once

Thanks @stephenc!  (and yes, for alpha, I am not about to merge something in your repo without your review ;) )


Refactor/code quality

Addresses a number of code quality lints.
I haven't added any additional lints/checks to enforce these changes. Is that something that you want?

I like most of the change. thanks :)
However, I am not convinced by the match => if
In general, I prefer the match
Could you please remove this commit from the pr?
thanks

I like most of the change. thanks :) However, I am not convinced by the match => if In general, I prefer the match
Could you please remove this commit from the pr? thanks

sure thing. I've removed that commit.

Hey! Great job! This is getting pretty big, let's split it here and put further changes in another PR

Hey! Great job! This is getting pretty big, let's split it here and put further changes in another PR. Oh you we're faster than me :)

sure. happy to slice and dice this however you like! Will move further changes to a stacked PR

CI failures...
I don't think they're related to this PR. how stable is the CI normally?

It's pretty stable these days, but it fails every once in a while. It seems unrelated indeed. I'll rerun the CI

It's pretty stable these days, but it fails every once in a while. It seems unrelated indeed. I'll rerun the CI

loads of failures in GnuTests job. Still think it's unrelated to this PR? I can split the commits into separate PRs if it helps?

loads of failures in GnuTests job

That's to be expected. This tests against all GNU tests and we are not 100% compatible. The job will fail if there are any tests that pass on main but not on the PR.

I've created an issue for the test that failed: https://github.com/uutils/coreutils/issues/2969

@tertsdiepraam
finished! that's some epic CI

what tool did you use for that btw?

what tool did you use for that btw?

it was about 5% manual effort, and about 95% clippy

I guess it is with inactivated options of clippy as we have it in the CI?
Could you please update the CI to enable them?
thanks


Only pulls title of first episode

The script runs without any errors, but it only pulls the first episode title when multiple are watched on the same date. Is there any way you could edit it to include all of them?

Good catch! I've updated the script so it should work now for multiple movies/series/episodes on the same day.

Support for multiple episodes added in ff6323c6a90dc970bfad08bd6dcca0492d10d4ad.
Support for multiple movies/shows added in 7233c0439a7e451b8f77591d216d1849974dc83c.


[Bug] Cannot upload a zipped backglass

When uploading a zipped, cf link below, studio detects it is a backglass but goes through table import process then fails
https://vpuniverse.com/files/file/7335-class-of-1812-gottlieb-1991-2-3-screens-directb2s-b2s-db2s/?do=download&r=132238&confirm=1&t=1&csrfKey=2af0fa6aba6801d024c16f4c37293b07

Unfinished uploads v2


Update peermentors.html

Hey Mingina,
Is this what you meant?

You are only supposed to add your card to the peer mentorship page not edit the whole page.
Coz you have edited the page title and the header and that's not what you are supposed to do.
Pull the repo so you can see what other people have done.


Update sbt-lucuma to 0.7.0

Updates edu.gemini:sbt-lucuma from 0.4.4 to 0.7.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "edu.gemini", artifactId = "sbt-lucuma" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequest = { frequency = "@monthly" },
  dependency = { groupId = "edu.gemini", artifactId = "sbt-lucuma" }
}]

labels: sbt-plugin-update, early-semver-major, semver-spec-minor, commit-count:n:2

Superseded by #138.


If no search results are found; Search the web.

📝 Provide a description of the new feature
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
If you search in PowerToys launcher and no search results are found, then the top item should start a web search when you press enter, instead of saying that not all files are indexed.
If you'd like to see this feature implemented, add a 👍 reaction to this post.

@Tech-How
web search related requests are tracked in https://github.com/microsoft/PowerToys/issues/3245

@Tech-How
web search related requests are tracked in https://github.com/microsoft/PowerToys/issues/3245


some packages add the b similar when using rc

Issue #, if available:
Description of changes:
Adding another suffixes, we have a few packages that use the format x.y.x_bw like https://github.com/Azure/azure-sdk-for-python/tree/azure-mgmt-servicefabricmanagedclusters_2.0.0b5
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

lgtm

Is there any documentation somewhere? The PR added b to both preSuffixes and postSuffixes, which confuses me.

updates to only preSuffixes was my mistake.
which kind of documentation are you looking for?

I'm looking for a document about what b means as a pre-suffix. What about order compared to alpha, beta, etc? (e.g. Is 1.2.3pre1 older than 1.2.3b1?) Is b different from beta?

I'm looking for a document about what b means as a pre-suffix. What about order compared to alpha, beta, etc? (e.g. Is 1.2.3pre1 older than 1.2.3b1?) Is b different from beta?

that is something I cannot answer :( That specific project that I mentioned I did not find any docs for that.
if this makes no sense feel free to close
thanks!

Actually, I didn't find b even in apk-tools. I'm curious if 2.0.0b5 works with apk.
https://gitlab.alpinelinux.org/alpine/apk-tools/-/blob/acefa1acc1ce0e1871d17b4eafe4b1888f45d4d0/src/version.c#L73-74
Isn't it violating the apk version specification?

i think so, thanks! i will close this pr

If we confirm it should work with apk, we can reopen this PR. Thanks for your contribution.

thank you!


BINs are incorect or missing.

Hi Erdem,
Nice work.
BINs are incorect or missing. You should check.
and ..
TROY (Turkey Credit Cart). You better add it. :)

There are some BINs over on my Skeuocard project that you're welcome to grab! Take a look at this file, specifically.

Hi @basitiyidir ,
I just added type control for troy cards. you can see on this https://github.com/muhammederdem/credit-card-form/commit/e1d5acf76ae1b7123508ef2b5cc8d3cd5943b65c;


Release new version(s) to PyPi

We're overdue for a release. My goal is to release basically what is in master, and then prepare for a second release with current Django support. I'll update this issue with what those look like.

v1.4.7 released to PyPI, supporting Django 2.2: https://pypi.org/project/django-nose/1.4.7
A v2.0 should:

Drop support for Django 1.8, 1.9, 1.10, 1.11, 2.0, 2.1
Add support for Django 3.0, 3.1, and master
Drop support for Python 2.6, 2.7, and 3.4
Add support for Python 3.8

This might allow removal of some compatibility code.
This gives users an upgrade path:

Upgrade to Python 3.5 or later, django-nose 1.4.7 and Django 2.2.
Upgrade to django-nose 2.0
Upgrade to Django 3.x and later Python versions

Hello @jwhitlock, thank you for the development of the project.
As I understand, you are not currently maintaing this project as stated in #310, that being said I am happy to contrbuite to it so it can support Django 3.2.
I would you really appercaite if you would give any guildine on how to move forward, for example would  this #313, be a good start?
This library is being used by the OpenEdx platfomr, context here, so it's critcial for the team to make a decision in a timely manner on weather to keep  using this library or look for alternatives.
Thank you again for your time

Hi @ghassanmas!
I don't have experience with nose2, so I don't know if #313 is possible. I suspect that a new code effort would be best for a Django wrapper. It looks like someone tried with https://github.com/jpellerin/django-nose2, but that appears abandoned.
My advice to continue with django-nose would be to prepare a v2.0 release like I described above, with changes for today's releases:

Drop support for Django 1.8, 1.9, 1.10, 1.11, 2.0, 2.1
Add support for Django 3.0, 3.1, 3.2 and master
Drop support for Python 2.6, 2.7, and 3.4
Add support for Python 3.8 and 3.9

If you have an active project using django-nose, then you are in a better position than I am to make and test these changes. I could review PRs and give feedback, and help with the transition to a maintainer.

Hi @jwhitlock and @ghassanmas!
I'm working in a big active project and using django-nose.  We want to upgrade to Django 3.1+, but django-nose don't support it: typeerror: sql_flush() takes 3 positional arguments but 4 were given.
We want to help correct this issue, and upgrade django-nose to support new python and django versions.
It's just open issues, and send PRs to solve it? Can you guys help me a little?
Thanks.


feat: Add View Portfolio support via Sonarwatch

Added tool to fetch portfolio for an address(includes Fungible Tokens + NFTs)
#18

can you rebase the branch


Program that generates a file with list of upcoming contests and their info using web scraping in Python

Usage of BeautifulSoup required

The project was reviewed and merged. Thank you @VinayakGaikwad101
Feel free to star this repo and add more projects.


301 Redirect for old RSS flight levels

/rss/feed/high/ => /rss/feed/osg/ ?
/rss/feed/medium/ => /rss/feed/msit/ ?
/rss/feed/low/ => /rss/feed/wif/ ?

LGTM, do this.

They make more sense now since older ring levels are not considered anymore. Go for it.


why rescale weight of first layer if in barf mode

Hi, great work on NeRF. I am interested in these lines, can you tell me: why rescale weight of first layer if in barf mode
https://github.com/chenhsuanlin/bundle-adjusting-NeRF/blob/803291bd0ee91c7c13fb5cc42195383c5ade7d15/model/planar.py#L211
Thank you very much for such amazing observation on gradient !

Hi @leo-frank, this allows the initial optimization behavior to be equivalent to an MLP where the first layer consumes only a 2-dimensional input. This is very subtle though, and I don't really find it to make a big difference in practice.


Handle errors while scaling kubernetes cluster

Description
This PR fixes the issue #7920

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI

Bug Severity

[ ] BLOCKER
[ ] Critical
[x] Major
[ ] Minor
[ ] Trivial

How Has This Been Tested?

Issue #7920 happened because of the unhandled CloudRuntime Exception, I've replicated the same scenario even when Global setting enable.dynamic.scale.vm is set to false.
Tested the same before and after fix
Before fix, state stuck in "scaling"

After fix, state first changed to "Alert" and then moved to "Running"

@blueorangutan package

Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 7393

@blueorangutan test rocky8 vmware-67u3

@harikrishna-patnala a [SF] Trillian-Jenkins test job (rocky8 mgmt + vmware-67u3) has been kicked to run smoke tests

[SF] Trillian test result (tid-7993)
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server r8
Total time taken: 50614 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr8107-t7993-vmware-67u3.zip
Smoke tests completed. 107 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_01_deploy_vm_on_specific_host
Error
3603.69
test_vm_deployment_planner.py

test_02_deploy_vm_on_specific_cluster
Error
4.38
test_vm_deployment_planner.py

test_03_deploy_vm_on_specific_pod
Error
4.40
test_vm_deployment_planner.py

test_04_deploy_vm_on_host_override_pod_and_cluster
Error
4.46
test_vm_deployment_planner.py

test_05_deploy_vm_on_cluster_override_pod
Error
12.56
test_vm_deployment_planner.py

Code LGTM but needs testing. In my original issue, dynamic scaling config was set to true. It was the hypervisor plugin that was returning error during scaling. Using same vCPU and 2GB to 4GB scaling should reproduce the original issue on VMware. cc @harikrishna-patnala @kiranchavala

In both cases, CloudRuntimeException is thrown @shwstppr , so this covers that error as well.

KubernetesClusterScaleWorker.java:460

Thanks for testing @kiranchavala .
For sure this error or this case is not part of this PR but worth checking the case and fix it. I'll test it again with your reproduction steps.

@harikrishna-patnala will you be able to make changes for cases reported by @kiranchavala ?

checking it now, I'll see if that case can be covered here.

@harikrishna-patnala any progress/prognosis?

@harikrishna-patnala any update on this?

Sorry @DaanHoogland and @shwstppr could not finish this completely. Resuming it now.

I'm seeing some other cases where the cluster is stuck in "Scaling" state. Trying to fix them as well.

This is ready for review cc @kiranchavala @shwstppr @DaanHoogland

@blueorangutan package

Packaging result [SF]: ✔ el7 ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 7962

@blueorangutan test

@DaanHoogland a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[SF] Trillian test result (tid-8499)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 40740 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr8107-t8499-kvm-centos7.zip
Smoke tests completed. 109 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

@blueorangutan package

@blueorangutan package

@harikrishna-patnala a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el7 ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 7986

@blueorangutan test

@harikrishna-patnala a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[SF] Trillian test result (tid-8534)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 40913 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr8107-t8534-kvm-centos7.zip
Smoke tests completed. 108 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_08_migrate_vm
Error
45.83
test_vm_life_cycle.py

@kiranchavala would you like to continue testing this and also please verify the issue that you've raised


allow only nominal subtyping

Our compilation scheme doesn't handle structural subtyping correctly.  We rely on heritage clause to create vtables.   We need to consider restricting type compatibility and subtyping.

reviewed compilation scheme. structural subtyping seems ok.