Missing test results in junit

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
xcpretty drops test results when generating junit report.
Command executed
run_tests scheme: 'Tests', device: 'iPhone 6', test_without_building: true, output_types: 'junit'

Please see the multiple open issues related to xcpretty incorrectly scanning for test results:

https://github.com/xcpretty/xcpretty/issues/322
https://github.com/xcpretty/xcpretty/issues/295
https://github.com/xcpretty/xcpretty/issues/344
https://github.com/xcpretty/xcpretty/issues/337
https://github.com/xcpretty/xcpretty/issues/329

If I run my tests in Xcode and count the total I get 542. If I run fastlane's run_tests action and check how many tests are in Xcode's report I get
$ grep TestSummaryGUID output/Tests.test_result/TestSummaries.plist | wc -l
542

And yet the fastlane summary (and junit report) say there are only 490. Looking in the junit report I can see that tests are being omitted even though they are present in the plists. The omitted tests seem to fluctuate randomly with each run.

For now I'm working around this by using trainer and running it like this so it's almost a drop-in replacement:
trainer path: "#{ENV['SCAN_OUTPUT_DIRECTORY']}/#{scheme_name}.test_result", extension: '.junit', output_directory: ENV['SCAN_OUTPUT_DIRECTORY']

It correctly detects all 542 tests.


Update ATT&CK™ to ATT&CK®

As a user of the ATT&CK website, I want to see accurate legal language relfecting the status of the ATT&CK trademark.
All instances of ATT&CK™ should be updated with ATT&CK®. See Butterick's Practical Typography for best practices on placement.

This also applies to page titles:
$ curl -s https://attack.mitre.org/groups/G0018/ | grep \<title\>
<title>admin@338 | MITRE ATT&CK™</title>


Update JS snippet from the ApplicationInsights-JS repo

This updates the JS snippet from the latest in the ApplicationInsights-JS repo. I've left in the minor differences of course like the {0} for the inst key and CDN path.
See https://github.com/Microsoft/ApplicationInsights-JS/pull/295 for additional context.
PS the snippet also seems to already have been a little diverged from what was in ApplicationInsights-JS, so this change pulls those in too as a side effect. Either way it should be consistent with that repo as of now.

Hi @dfederm, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (David Federman). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

👍

I was wondering why the diff was "so different" until we realized this was against master instead of develop. ;)  I'll close this PR, please open it against develop. :)

Hi @dfederm, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (David Federman). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


ibc  transfer

Summary of Bug
An attempt to complete the transmission of the ibc packet umee > axelar fails
invalid denom: transfer/channel-0/transfer/channel-208/transfer/channel-2/transfer/channel-288/transfer/channel-33/transfer/channel-3/transfer/channel-184/transfer/channel-33/uusdc

Version
Please provide the output of the following commands:

$ umeed version  6.3.0

Steps to Reproduce
Steps to reproduce the behavior:
hermes v1.8.0 ( https://hermes.informal.systems/quick-start/installation.html )
nano config.toml

[global]
log_level = 'debug'

[mode]
[mode.clients]

enabled = true
refresh = false
misbehaviour = true

[mode.connections]
enabled = true

[mode.channels]
enabled = true

[mode.packets]
enabled = true
clear_interval = 100
clear_on_start = true
tx_confirmation = false
auto_register_counterparty_payee = false
ics20_max_memo_size = { enabled = true, size = "32KiB" }
ics20_max_receiver_size = { enabled = true, size = "2KiB" }

[rest]
enabled = false
host = '<IP_ADDRESS>'
port = 3000

[telemetry]

enabled = false
host = '<IP_ADDRESS>'
port = 3001

[telemetry.buckets]

[tracing_server]
enabled = false
port = 5555

[[chains]]
id = 'umee-1'
rpc_addr = 'https://umee-rpc.polkachu.com'
grpc_addr = 'http://umee-grpc.polkachu.com:13690'
event_source = { mode = 'push', url = 'ws://umee-rpc.polkachu.com/websocket', batch_delay = '500ms' }
rpc_timeout = '20s'
trusted_node = false
account_prefix = 'umee'
key_name = 'key'
memo_prefix = 'Relayer by Craving_for_Knowledge'
store_prefix = 'ibc'
default_gas = 300000
max_gas = 3500000
gas_price = { price = 0.01, denom = 'uumee' }
gas_multiplier = 1.1
max_msg_num = 30
query_packets_chunk_size = 10
max_tx_size = 2097152
clock_drift = '5s'
max_block_time = '30s'
trusting_period = '14days'
trust_threshold = '2/3'
address_type = { derivation = 'cosmos' }

 [chains.packet_filter]
 policy = 'allow'
 list = [
   ['transfer', 'channel-33'], # axelar
 ]

 [[chains]]
id = 'axelar-dojo-1'
rpc_addr = 'https://axelar-rpc.polkachu.com'
grpc_addr = 'http://axelar-grpc.polkachu.com:15190'
event_source = { mode = 'push', url = 'ws://axelar-rpc.polkachu.com/websocket', batch_delay = '500ms' }
rpc_timeout = '20s'
trusted_node = false
account_prefix = 'axelar'
key_name = 'key'
store_prefix = 'ibc'
default_gas = 350000
memo_prefix = 'Relayer by Craving_for_Knowledge'
max_gas = 5500000
gas_price = { price = 0.07, denom = 'uaxl' }
gas_multiplier = 1.1
max_msg_num = 30
query_packets_chunk_size = 10
max_tx_size = 180000
clock_drift = '5s'
max_block_time = '30s'
trusting_period = '7days'
trust_threshold = '2/3'
trust_threshold = { numerator = '1', denominator = '3' }
address_type = { derivation = 'cosmos' }

 [chains.packet_filter]
 policy = 'allow'
 list = [
   ['transfer', 'channel-36'],
 ]

hermes --config $HOME/config.toml clear packets --chain umee-1 --port transfer --channel channel-33  --packet-sequences 14116

logs https://gist.github.com/studentmtk/9f52280e67ed654103904a826ecb408a

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

If I understand correctly, is this behavior related to exceeding the length of the denom? ( https://github.com/cosmos/ibc-go/blob/main/docs/architecture/adr-001-coin-source-tracing.md )


Refactor Routing to use ProjectReference and move into src/Http folder

This implements https://github.com/aspnet/AspNetCore/issues/4246 for projects that were under src/Routing.
Changes:

Stop producing Microsoft.AspNetCore.Routing.DecisionTree.Sources. I could not find any usages of  outside of the Microsoft.AspNetCore.Routing assembly, so I've moved this to be a subfolder of code in the Microsoft.AspNetCore.Routing project.
Stop building src/Routing as a submodule. Use ProjectReference instead
Move projects from src/Routing into src/Http/Routing/ and src/Http/Routing.Abstractions/
Update package baselines and dependencies to support building routing projects and samples

Two test failures in HttpSys: FYI @jkotalik @muratg

Microsoft.AspNetCore.Server.HttpSys.FunctionalTests.Microsoft.AspNetCore.Server.HttpSys.Listener.ResponseCachingTests.Caching_SendFileWithFullContentLength_Cached
Microsoft.AspNetCore.Server.HttpSys.FunctionalTests.Microsoft.AspNetCore.Server.HttpSys.ServerTests.Server_MultipleStopAsyncCallsWaitForRequestsToDrain_Success

These appear to be unrelated to Routing.

I'm torn between putting this in Http vs Middleware. @JamesNK @rynowak?

While routing has middleware, it does feel more fundamental than other middleware. Plus routing has link generation, and that isn't middleware specific. I prefer Http.

If there are no more objections, I'd like to merge this later today.


Bug when remove the last replace item into search bar

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:  1.78.2
OS Version: arm64

Steps to Reproduce:

Find some text
Put a replacement into replace the input
Click to close the last into from some folder
This crashes the entire vscode

Seems that is some mismatch into my local development, sorry


Fix typo in README.md

Hello, I've fixed a small typo in your README.md file. It was duplication of to build sentence :)

Thank you for spotting (and fixing) that! Every help is welcome. 👍


Pneumatic conveying example renewal

Description
As part of Lethe's version launching, this example was revisited and the text has been verified.
Checklist (will be removed when merged)
See this page for more information about the pull request process.
Code related list:

[x] All in-code documentation related to this PR is up to date (Doxygen format)
[x] Copyright headers are present and up to date
[x] Lethe documentation is up to date
[x] The branch is rebased onto master
[x] Code is indented with indent-all and .prm files (examples and tests) with prm-indent

Pull request related list:

[x] Labels are applied
[ ] There are at least 2 reviewers (or 1 if small feature) excluding the responsible for the merge
[ ] If this PR closes an issue or is related to a project, it is linked in the "Projects" or "Development" section
[ ] If any future works is planned, an issue is opened
[ ] The PR description is cleaned and ready for merge

Hey @voferreira you Ahmed body changes are also in this PR, do you think you could cherry pick just the pneumatic conveying?

@blaisb Sorry, I didn't see your commit before fixing it and I fixed it recommitting. Do you still want me to try to cherry-pick?

I am not finished with this example yet, though. I am still testing the CFD-DEM and there will be the postprocessing missing.


RP - remove xplat/include versions

Internal review URL

@tdykstra 	looks like you can't test redirection

@Rick-Anderson yes you can -- redirection works in staging.  If you're seeing it not work, there's likely something wrong with the redirection json.


Added Vestfold Fylkeskommune (vfk.no)

Domain used by multiple schools that specialize in different things.
Subdomains:
skole.vfk.no - used by students

@MatsAnd could you please create a file lib/domains/no/vfk/skole.txt, if the students use skole.vfk.no. If vfk.no is the address for the whole commune, it's not what we can add to the repository.

@philipto here you go: https://github.com/JetBrains/swot/pull/1942


SUP-2585: Add Support for changing Build Drivers

Per Issue: https://github.com/buildkite-plugins/docker-compose-buildkite-plugin/issues/457
To be able to fully support using the cache-to parameter, the Build Driver needs to be changed to use a different driver other than the default docker driver.
This PR will add the basic ability to change the driver used in a per Job basis and clean up the Build Instance afterwards to avoid issues of lingering Build Instances.

Bringing out of Draft for initial reviews whilst adding tests

thanks @toote will do the clean up and of the docs and a bit of refactoring/changing where it makes sense, went down the DRY route with some ideas but definitely make sense to make it more clear in some situations with the pre-exit hook


Feature Request: cupyx implementation of scipy.sparse.linalg.spsolve

Dear CuPy developers,
We would like to request that you consider implementing scipy.sparse.linalg.spsolve.
We are exploring porting some part of the DESI spectroscopic pipeline to GPUs (https://github.com/desihub) and we would like to use scipy.sparse.linalg.solve in CuPy.
At the moment we are using cp.linalg.solve instead, but we are finding that it is the current bottleneck in our workload:
overview_gpu_only.txt
If you would like more information please let us know.
Thank you very much,
Laurie Stephey

Thank you for raising a feature request!  We welcome pull-request for this feature if anyone is interested in writing it.
@lastephey Please be aware that line_profiler results may not be correct for CuPy code as CuPy (CUDA) runs asynchronously. You can use nvprof/nvvp to diagnose which CUDA kernel is actually taking time.

I see, thank you for the profiling advice @kmaehashi . In the future we will use nvprof/nvvp.

Regarding this issue, does CuPy have UMFPACK sparse direct solver?


Make constructor of ThrowableAssertAlternative public

Summary
The constructor of org.assertj.core.api.ThrowableAssertAlternative is currently package protected. I'd like to create a class named MyExceptionAssertAlternative with some more methods dealing with the fields of MyException. It would be used like this:
assertThatMyException().isThrownBy(() -> { throw new MyException("boom!", 123); })
                          .withMessage("%s!", "boom")
                          .withMessageContaining("boom")
                          .withNoCause()
                          .withStatus(123);

If there is an alternative, I'd like to hear it.
I could make a PR but as this is a really simple one, is it necessary? Thanks.

That's a little more complicated than expected:
The constructor of org.assertj.core.api.ThrowableTypeAssert must be public too and its fields protected.

sure, no problem

Fixed in https://github.com/joel-costigliola/assertj-core/commit/bd44eabb7380aba5916a36cc6a66de6a857f0a31, let me know if you that is sufficient.


Fix syntax error in HLG docs example (#6945)

[x] Tests added / passed
[x] Passes black dask / flake8 dask

Thanks!


git pull for branch not always master fixes #73

git clone will automatically set the remote origin, so this should function as normal

/cc @codeaholics @geophree - see any issues with this?

Is the upstream set correctly when the branch is checked out for the first time?

Relies on git clone, and it defaults to origin/master.
https://github.com/tes/bosco/blob/master/commands/clone.js#L164

:+1: Although we should be using execFile


[WIP] Web3 provider

Summary
A description of the changes proposed in the pull request.
Fixes #number
Checklist

[ ] Change is small and easy to review (please split big changes into multiple PRs)
[ ] Change is consistent with architecture
[ ] Change is consistent with test architecture
[ ] Change is consistent with naming conventions
[ ] New code is covered with tests
[ ] Tests related to old code are updated
[ ] Documentation is up to date with changes

Ok, looks like a good direction, I would propose to split that into boilerplate PR (project directory, typescript config, etc) and second - base for web3 provider.
Not sure if we should have example project thou. Maybe we fork popular project like UniSwap and try to implement with it?

Closing for now.


User Model Refactor

Description
User services have been refactored to follow the signature (sequelize : Sequelize) => async (service_args) => {} to allow for unit testing with an injected custom sequelize object (no real databases involved).
User request handlers have been refactored so that every API request has a handler and controller method. The handler only deals with validation and converting data to express responses. The controller handles all main logic, service calls, and errors.
IMPORTANT: We use sequelize.models to handle database queries so *.model.ts files no longer need to export the model. Only the definer and model interface are needed.
New User Schema

Email ( required, unique )
Password ( required )

8-30 chars
must include lower case
must include upper case
must include a symbol/number

Display name( required )

1-30 chars

[REMOVED] Username

The only thing I think you should change is changeDisplayName, when you put in the same display name as your current one, it works successfully, which is technically fine. However, doing the same thing to the password throws an error; this should be consistent. Below are other small things that I noticed, but don't need to be changed to be approved.

That's actually intentional. I think it's fine to allow someone to change display name to the same one. I only prevent that for passwords to promote better password security.
Fixed the 404 error for authentication


Quick Accent popup appears after returning from lock screen and remains frozen

Microsoft PowerToys version
0.70.1
Installation method
Microsoft Store
Running as admin
No
Area(s) with issue?
Quick Accent
Steps to reproduce
Return to the Windows desktop for the current user session.
It might be needed to press the spacebar to get reproduce it properly.
✔ Expected Behavior
This popup should never appear unwantedly.
❌ Actual Behavior
After returning to the desktop from the Windows lock screen, this popup appears and remains frozen, until I press any key followed by the spacebar:

This problem has been away for a while now. But since a few days, it seems to be reintroduced.
Apparently this bug has never been fixed properly.
Also see this issue: #22006
I might have used the spacebar to get out of the login screen (I don't remember).
Other Software
Running on Windows 10.

Not able to repro this, going to close. Let us know if you see this again!


Enhance the query API to support only including datasets based on the month of acquisition

For example, so that you can specify that you want datasets from 2005 but only only if they were acquired in Jan/Feb/Mar.

Implemented in 0.1.0 snapshot.
Can now specify months parameter to list_cells and list_tiles functions.
For example...
    cells = list_cells_as_list(x=[120], y=[-25],
                               acq_min=parse_date_min("2005"),
                               acq_max=parse_date_max("2005"),
                               satellites=[Satellite.LS5, Satellite.LS7, Satellite.LS8],
                               dataset_types=[DatasetType.ARG25],
                               months=[Months.January, Months.February, Months.March],
                               config=config)

There are also default values for Australian seasons so you can do, for example,  months=MONTHS_BY_SEASON[Season.SUMMER].


Firebase failed upload

I got an error about uploading to Firebase:
 💥   | firebase_app_distribution                        | 40          |
+------+--------------------------------------------------+-------------+

ERROR [2019-12-23 15:53:46.98]: fastlane finished with errors

Looking for related GitHub issues on fastlane/fastlane...
Search query: Shell command exited with exit status 1 instead of 0.
]0;Firebase CLIi  getting app details...
i  uploading distribution...

Error: failed to upload distribution. it took longer than expected to process your binary, please try again

URL: https://api.github.com/search/issues?q=Shell%20command%20exited%20with%20exit%20status%201%20instead%20of%200.%0A%1B%5D0%3BFirebase%20CLI%07%1B%5B1m%1B%5B36mi%20%1B%5B39m%1B%5B22m%20getting%20app%20details...%0A%1B%5B1m%1B%5B36mi%20%1B%5B39m%1B%5B22m%20uploading%20distribution...%0A%0A%1B%5B1m%1B%5B31mError%3A%1B%5B39m%1B%5B22m%20failed%20to%20upload%20distribution.%20it%20took%20longer%20than%20expected%20to%20process%20your%20binary%2C%20please%20try%20again%0A+repo:fastlane/fastlane
ERROR [2019-12-23 15:53:47.02]: Error finding relevant GitHub issues: undefined method `map' for nil:NilClass
bundler: failed to load command: fastlane (/home/circleci/repo/vendor/bundle/ruby/2.6.0/bin/fastlane)
FastlaneCore::Interface::FastlaneShellError: [!] Shell command exited with exit status 1 instead of 0.
]0;Firebase CLIi  getting app details...
i  uploading distribution...

Error: failed to upload distribution. it took longer than expected to process your binary, please try again

This issue is in CircleCI when I do parallel distribution for bulk of envs. Is it possible to change timeout or catch this error with try again e.g. for 2 or 3 times?
Or maybe it's an interaction limit in Firebase?

Hi tenhishadow, how large is the distribution you're trying to upload? There should be a 900 MB upload limit that might be affecting you here.

Oh, no.
It's just 57M
 tenhi@nlb08  ~  du -h app-arm64-v8a-staging.apk                                                                                 ✔  2  21:08:19
57M     app-arm64-v8a-staging.apk

But I do parallel execution for 2 ios and 2 android.

And as you can see, another build has been distributed correctly.
Here it is just something like a slow processing or smth else.

@kevinkokomani any ideas?

Hey, sorry for the delay.
I haven't heard specifically of an issue with parallel distribution, but that could be an avenue to look into. Does uploading builds work consistently if you're not distributing in parallel? That might be worth doing some testing with.
You're also using CircleCI - is this a locally hosted Circle instance or hosted on the cloud?
We also have seen transient issues where uploading distributions can fail occasionally when uploading from a place far away from our North American servers. Do the upload failures happen consistently overall, or have you noticed it working as intended at least some of the time?

Closing this issue as we have since improved binary upload speeds, so timeouts should be less likely to happen now.


Bug - List of departments for Demo Account does not have "Other Department" option

Description
The list of departments on the demo account "First Visit" does not provide an option for government workers that are not in one of the listed departments.
We should add a "Other Government Department" or "Government Department not listed below" to the list.

After discussing with @gobyrne, we can simplify this fix by simply changing the text of the "Not in Government" option to something like "Other department or Not government"/"Autre ministère ou non gouvernement".

Added via admin panel on staging server.
User is prompted for a government email when selecting 'Other Department'.
@gobyrne can you review?


Rename publicodes-react into @publicodes/react ?

Currently we publish the packages @publicodes/api and  publicodes-react. For consistency we might want to use @publicodes/react instead of the latter.
I'm in favor of keeping publicodes (ie not @publicodes/core) and use the @publicodes namespace for all official packages including react and api (and other to come) instead of the publicodes- prefix that anyone can use.
In any case we need to make a decision before 1.0.

Maybe @publicodes/react-ui ?


iframe  exception in console

Hello all,
I installed this app on my server. The "npm run dev" command works fine.
I created also an account on okta and updated auth.js file as explained in here
The issue is the following exception launched in the developer console with the result that i cannot autenticate myself.

Refused to display 'https://{MYOKTADOMAIN}.oktapreview.com/error/404?fromURI=%2Foauth2%2Fv1%2Fauthorize%3Fclient_id%3D{CLIENTID}%26redirect_uri%3Dhttp%253A%252F%252Flocalhost%253A8080%26response_type%3Did_token%2520token%26response_mode%3Dokta_post_message%26state%3DuifCspEhBZh1tIonaBeWejnjz8KX9devstqt7Cn8SlAaf17elNSsovuG8Kk022VW%26nonce%3DkM9nAzR5BWF6A1vLbHc94MUwdFnjAnT4rSVlWGwNu69dlaPqe9Qv9r6JMu5AkMyI%26prompt%3Dnone%26sessionToken%3D20111IvBufhvEdENA1OSF3_-ax_Qyze7LgWa7TJGYYeYyVtoQHMR-Dp%26scope%3Dopenid%2520email%2520profile' in a frame because it set 'X-Frame-Options' to 'sameorigin'.

How to solve?
Really thanks for your time and help.

Did you setup Okta to have the redirect_uri from your server? Is your server still localhost, or something else?

I don't know exactly what you intend for redirect_uri .
I didn't insert the word localhost , but my IP address.
So I have http://{my-ip-server-address}/8080
Following the okta configuration for trusted origins:

Following the okta configuration for the application that I created:

Make sure you didn't accidentally copy the *-admin from the URL for your org...
I've seen a lot of people do this and it will cause the iframe error because the admin domain is protected from click-jacking.  Sometimes the widget / authjs library will use an iframe if it is doing postMessage.


The apply button is added to the editor

Description:
You can save an email in the editor.
A little bit of information:

If the required field "Contact segment" is empty, the apply button won't be active.
If the required fields "Subject" and "Internal Name" are empty, the default value will be generated (e.g. Email 2022-01-01 12:01:00).
If the email form is not valid, the email will not be saved, and the modal will be displayed.

How can I add a translation?

@volha-pivavarchyk the same way you added the other text? If I remember correctly then you have to do it in the plugin.

I removed some unused code


Color Columns SVG fix

Fixes a typo in color columns block.
Fixes a querySelector scope issue that messes up the svg loading
Resolves: MWPW-143794
Test URLs:

Before: https://main--express--adobecom.hlx.live/express/colors/light-blue?martech=off
After: https://color-svg-fix--express--adobecom.hlx.live/express/colors/light-blue?martech=off

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

Project coverage is 35.52%. Comparing base (a0f6ea2) to head (4dd782f).

Additional details and impacted files
@@            Coverage Diff             @@
##            stage     #713      +/-   ##
==========================================
+ Coverage   34.84%   35.52%   +0.67%
==========================================
  Files         131      131
  Lines       25513    25513
==========================================
+ Hits         8891     9063     +172
+ Misses      16622    16450     -172

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


improve UX. message list auto scroll to bottom when chatting

Cool. I will merge now and give it a try. 🎊


How is orchestrator working with more than one MySQL Clusters ?

Hi ,
if there're more than one MySQL clusters , the orchestrator checking these MySQL clusters in parallel or in sequential?
And if there're more than one MySQL Master failed , the orchestrator do the recovery in parallel or in sequential ?
Thanks.

everything runs concurrently, including failovers.

My advice is to ask questions on https://groups.google.com/forum/#!topic/orchestrator-mysql/


Qiskit no longer maintained

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:  Version: 1.57.1
Commit: 507ce72a4466fbb27b715c3722558bb15afa9f48
Date: 2021-06-17T13:28:32.912Z
Electron: 12.0.7
Chrome: 89.0.4389.128
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 20.5.0

OS Version: Darwin  20.5.0 Darwin Kernel Version 20.5.0: Sat May  8 05:10:33 PDT 2021; root:xnu-7195.121.3~9/RELEASE_X86_64 x86_64

Steps to Reproduce:

Install https://marketplace.visualstudio.com/items?itemName=qiskit.qiskit-vscode&ssr=false#overview
Multiple error messages appears

Reason is that the https://github.com/qiskit-community/qiskit-vscode is no longer maintained and not compatible with VSC. Should be removed from repo. Same with https://marketplace.visualstudio.com/items?itemName=AkashGutha.qiksit-snippets

Dup of https://github.com/microsoft/vsmarketplace/issues/23


Download pretrain model error

The link of pretrained models is not available in the Readme

I confirmed that this link is available  and I can download the pre-trained model through this link .


Resize neighbouring windows/adjust split dynamically

📝 Provide a description of the new feature
Be able to dynamically change the grid by dragging the split between windows.
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
This is something that Windows can do (although it is limited and temperamental).
Say you have set up a grid (let's say a 1x2 grid).
You decide you want to give one of the windows more screen space, so you go to resize window A.
But now window B doesn't border with window A. You then need to manually adjust B to border A.
I would like to be able to resize both windows simultaneously (just like when you create a custom template in FancyZones where you drag the splits).
Maybe when I Shift-click (to match FancyZones existing shortcut) the window's border it could then resize both the active window and it's neighbour.
This would save the job of having to set up a template and switching to it, greatly improving the programs flexibility. Particularly
It would also mean you can get the program displaying correctly without having to go back and tweaking the template each time.

If you'd like to see this feature implemented, add a 👍 reaction to this post.

Already tracked in #6611 by #254


FancyZones Editor Doesn't open on Remote Desktop

Microsoft PowerToys version
0.63.0
Installation method
Windows Store
Running as admin

[ ] Yes

Area(s) with issue?
FancyZones Editor
Steps to reproduce
Running a fullscreen remote desktop connection.
On the remote machine, PowerToys is installed.
In the PowerToys Settings window > FancyZones > Launch layout editor - clicking this doesn't do anything.
Using the Activation shortcut does work and the editor shows.
✔ Expected Behavior
Layout editor opens
❌ Actual Behavior
Nothing happens
Other Software
No response

Fix / Workaround : It works after the remote machine has been rebooted.

Thank for the report. Could you please provide the logs from the remote machine?
/bugreport

PowerToysReport_2022-10-24-15-24-54.zip
Note the reboot occurred around
2022-10-24 12:19
After which the issue was fixed.


records: reuse parent record for versions

Reuse the parent record when fetching all record versions. Usually we
get all record versions when we want to reindex all versions (e.g.
parent was updated). Thus, we inject the parent record into all
versions, so we avoid having to refetch the parent from the database
for every record version.

Merged via other PR


npm start not working

I tried to start my first application on react. But it is not working

Sorry to hear that! But we can't help you unless you fill out the issue template.


viewport-addon: configure => configureViewport

Issue:
configureViewport is more convenient by following a naming convention for addons and to avoid aliasing. (and to make @igor-dv happy 😉 )
What I did
Added configureViewport function (should we drop the other one as it's an alpha version anyways?)
How to test
yarn test
Is this testable with Jest or Chromatic screenshots?
Yes
Does this need a new example in the kitchen sink apps?
No
Does this need an update to the documentation?
Yes
If your answer is yes to any of these, please make sure to include it in your PR.
Please tag your pull request with at least one of the following:
["cleanup", "BREAKING CHANGE", "feature request", "bug", "documentation", "maintenance", "dependencies", "other"]

should we drop the other one as it's an alpha version anyways?

Yes, I think so

Awesome!
Thanks @igor-dv


Subtitle or Caption text tracks not showing with Youtube & Vimeo Providers

Web components / default theme
Can be seen on the examples, after changing the src URL to either Youtube or Vimeo.
The subtitles never show, same for captions text tracks. We see the subtitle options appear inside the settings menu as expected, but the words don't actually show in the player.
https://stackblitz.com/edit/vidstack-examples-mllagx?file=video-player.html

Thanks for letting me know! Fixed in 1.9.6@next :)


Correction API Management AAD steps

Correction on step 19, changed Secret ID by Secret value.

@mauricio-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

Thanks for the correction @mauricio-msft! LGTM
#sign-off


Setting compound=false fails

I want the edges of my graph to connect together.
After looking in the GraphViz documentation I think I have to set the graph attributes concentrate=true and compound = false. Somehow setting compound=false does not work.
After generating the code for the
graph, the compound attribute is still set to true.
My code:
Compound.txt

I see where the issue is: https://github.com/KevinMarquette/PSGraph/blob/6d673abb35fbadaf5d7a6a81866eb6473189ecd4/PSGraph/Public/Graph.ps1#L94
I'll need to track down why I am setting that value to figure out the best way to resolve this. I think it has to do with me adding subgraph to subgraph edges.

On your original comment about getting the edges to connect together, can you provide an example of what you are looking for?
You can do this to see if that attribute is going to work for you:
$Graph -replace 'compound="true"','compound="false"' | Export-PSGraph -ShowGraph

compound is documented as allowing edges between clusters and that is what I am using it for. But I should add a check to see if it is already specified and not change it if it is.

Thanks for your help.
What I want to do is connecting two or more edges going from two or more nodes to one specific node.
Something like this:

Instead of:

I thought this would automatically work with concentrate set to "true". I tried your suggestion with replacing the string. Unfortunately the "concentrate"-attribute it doesn´t work like this. It only combines multiple edges from one node to another so there aren´t any duplicate edges.
Do you have any other idea how I can make this work?

A last resort option would be to create a new node that is a point and invisible. A,B,C would edge to the point and it would point to D. You would then have to remove the arrows from the first edges. I have not tried manipulating the edges that hard to have an idea. You may have to find some graphviz forums or groups to answer that question.
I'm not sure how much work that would be to add as a custom feature of my edge command.


Usage with SpringBoot - ApplicationRunner

As of now the library uses ApplicationRunner, thus the migrations will run after the app has started.
I would like to have the application start fail if my migrations fail.
I did not find a way to configure this.
Would it be a welcome addition to the codebase to add this feature if it's not yet configurable?

Hi @axelhodler ,
that's true if you use SpringBootMongockBuilder. The reason why is because you want to inject your dependencies. So you need to wait at that phase for the migration to run.
However, I believe SpringMogockBuilder(Notice it's not SpringBoot) fits your requirements.
Please, let me know if I am right or I haven't understood your issue.

@dieppa exactly what i wanted. Thanks a bunch and keep up the good work 👍


update to aspire 8.1

#2  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @gabrewer and the rest of your teammates on  Graphite


feat: added support for idle connection timeout in load balancer

Description
added support for idle_connection_timeout. Git issue: https://github.com/terraform-ibm-modules/terraform-ibm-landing-zone/issues/609
Release required?

[ ] No release
[x] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

:tada: This PR is included in version 5.1.0 :tada:
The release is available on:

GitHub release
v5.1.0

Your semantic-release bot :package::rocket:


Remove extra spaces from exception message

Without this change, it prints with extra spaces -- e.g.:
E           elasticsearch.exceptions.ImproperlyConfigured: No defined connections, you need to                     specify at least one host.

Thanks for the patch, could I ask you to please sign the CLA (0) so I can merge this?
0 - http://www.elasticsearch.org/contributor-agreement/

No problem. CLA signed.

Merged as 0246c5b
Thanks!


Some questions about forward_with_cfg

Thank you for the fantastic work and for making the code open-source. I have a small question regarding the forward_with_cfg function.
def forward_with_cfg(self, x, t, y, cfg_scale):
      """
      Forward pass of SiT, but also batches the unconSiTional forward pass for classifier-free guidance.
      """
      # https://github.com/openai/glide-text2im/blob/main/notebooks/text2im.ipynb
      half = x[: len(x) // 2]
      combined = torch.cat([half, half], dim=0)
      model_out = self.forward(combined, t, y)
      # For exact reproducibility reasons, we apply classifier-free guidance on only
      # three channels by default. The standard approach to cfg applies it to all channels.
      # This can be done by uncommenting the following line and commenting-out the line following that.
      # eps, rest = model_out[:, :self.in_channels], model_out[:, self.in_channels:]
      eps, rest = model_out[:, :3], model_out[:, 3:]
      cond_eps, uncond_eps = torch.split(eps, len(eps) // 2, dim=0)
      half_eps = uncond_eps + cfg_scale * (cond_eps - uncond_eps)
      eps = torch.cat([half_eps, half_eps], dim=0)
      return torch.cat([eps, rest], dim=1)

Why do we need these two lines:
half = x[: len(x) // 2]
combined = torch.cat([half, half], dim=0)

instead of simply setting combined = x:
def forward_with_cfg(self, x, t, y, cfg_scale):
      """
      Forward pass of SiT, but also batches the unconSiTional forward pass for classifier-free guidance.
      """
      # https://github.com/openai/glide-text2im/blob/main/notebooks/text2im.ipynb
      combined = x
      model_out = self.forward(combined, t, y)
      # For exact reproducibility reasons, we apply classifier-free guidance on only
      # three channels by default. The standard approach to cfg applies it to all channels.
      # This can be done by uncommenting the following line and commenting-out the line following that.
      # eps, rest = model_out[:, :self.in_channels], model_out[:, self.in_channels:]
      eps, rest = model_out[:, :3], model_out[:, 3:]
      cond_eps, uncond_eps = torch.split(eps, len(eps) // 2, dim=0)
      half_eps = uncond_eps + cfg_scale * (cond_eps - uncond_eps)
      eps = torch.cat([half_eps, half_eps], dim=0)
      return torch.cat([eps, rest], dim=1)

When I made this change, the model was no longer able to generate images correctly. As I understand it, the two approaches:
half = x[: len(x) // 2]
combined = torch.cat([half, half], dim=0)

and combined = x should be equivalent. Could you please help me understand where the issue arises when changing it to combined = x?
Thank you very much!

In my case, it's totally the same results.


remove sphinx option not available on leap versions

leap alabaster theme version is a bit old (0.7.10) and it does not
support the show_relbars config as that appears on 0.7.11
Instead of silently ignore unkown parameters, alabaster will instead
die, which is pretty bad.
So we need to drop that option to be able to package docs for leap
15.0/15.1

oh no - I really liked the prev/next buttons...

Me too! And you can see yourself progressing in the deploy and all of that!
I was so proud of this docs!


How to integrate the customized flutter engine into the flutter sdk after compilation

Use case
There are three developer in our flutter app development team. How can the other two use the flutter engine I compiled.
Maybe we can  integrate it into the flutter sdk, but how to do this?
Proposal
Is there any documentation?

Hi @SpringSea28

Maybe we can integrate it into the flutter sdk, but how to do this?

The tool doesn't support this currently, please see https://github.com/flutter/flutter/issues/134426#issuecomment-1724136103.
Closing this as a duplicate of https://github.com/flutter/flutter/issues/134426.

Thanks for your reply.  But #134426  seems to be no solution. Could you give me some more help? thanks！

The solution for that is: "You will have to setup a custom CI workflow to distribute locally built engines.". Please see flutter's community for resources and you may also get some help if you post it on flutterhelp.

@huycozy Thank you for your patience.
"You will have to setup a custom CI workflow to distribute locally built engines."，can you please tell me how to setup a custom CI workflow?
Please help me again, thank you very much！

Sorry, this seems to take times to set up such a CI. I may try it later when having spare time. In the meantime, you can try this simpler solution:
From your side:

Build your custom engine
Push it to your forked repository
Share your forked repository to your team

From your team side:

Pull your forked repository with your changes on engine
Link to the custom engine path with --local-engine, --local-engine-src-path and --local-engine-host flags when running app with flutter tool, see https://github.com/flutter/flutter/tree/master/docs/tool#using-a-locally-built-engine-with-the-flutter-tool

Please note that the above steps are from my idea only, I haven't tried it yet. Hope it helps!

@huycozy ,thank you very much for your answer.
Because the compiled engine is 40-50G, your suggestion may not be very suitable for us.
Thank you very much, you are a very patient and enthusiastic person.


builtins: add generator builtin for span payloads

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

Updated the crdb_internal.payloads_for_span builtin to return a table instead of a JSONB array. Each row of the table represents one payload for the given span. It has columns for payload_type and payload_jsonb. [#60784][#60784] {% comment %}doc{% endcomment %}

Fixed by https://github.com/cockroachdb/docs/pull/9907.


⚠ T:Riza Api has degraded performance

In 3842546, T:Riza Api (https://infra.triza.dev/api) experienced degraded performance:

HTTP code: 200
Response time: 1464 ms

Resolved: T:Riza Api performance has improved in 2ca21d0.


Trying to setup teleconsole in alpine

To reproduce:
$ docker run -it alpine sh
 # now from inside alpine container
apk add wget sudo
wget https://github.com/gravitational/teleconsole/releases/download/0.4.0/teleconsole-v0.4.0-linux-amd64.tar.gz
tar xf teleconsole-v0.4.0-linux-amd64.tar.gz
./teleconsole

results in the following error: sh: ./teleconsole: not found.
Can also check the recorded shell here: https://asciinema.org/a/236995

sorry, but it's outside the scope of Teleconsole.

Any alternative?


Implement additional buffer opening functions.

Hey @marshall-lee, thanks for contributing! I don't personally use these commands very often but the code sure looks fine to me. @teoljungberg, what are your thoughts?

I use neither of the commands, but I can see the value in having them.
I left some minor comments / questions on the implementation.

Hey @marshall-lee, are you still interesting in working on this?

Hi @marshall-lee, I just merged #12 which adds the funciton PickBufferCommand where you can add your own buffer commands.
Since that resolves this issue, I'm closing this PR. Thanks for your contribution!


Full Emulation, Language, and Link Cable Support

Hi, I saw a GitHub issue about emulation support, but I have a threefold followup question. When will multi-language support (not just other non-English games, but a multi-language version of Poké Transporter GB itself) be available? Also, would it be possible to enable emulation with just a mod for a GBA emulator? If I wanted to do this, my app is unable to be updated, as the host site is basically dead. Also, when will a GBA Link Cable be supported for the transporter?

Hi there!! To answer your questions-

The exact timeframe for the multi-language support is unfortunately still up in the air. It's currently in development, but I haven't had as much time to work on the project as I would have wished. But it is coming!!
Emulation is tricky, because while the connection is able to be emulated correctly, I haven't been able to find an emulator that supports GBC link cables specifically. Using a modified emulator is theoretically possible, but I've currently been focusing on fixing bugs instead of modding an emulator.
Unfortunately, a regular GBA cable will never be supported due to how the two cables are wired. There is theoretically a way to do it with two GBA cables, but I haven't had the chance to fully look into that

If you need a beta tester for the french version Pokemon games I'm up for it


Add examples for making your own dataset

Actually its more straightforward than I thought... I will repoen if I have any specific questions.
Im reading the Guide in split screen with the examples.

Im a bit confused on what edge_attr is. Some definitions or requirements would be really helpful @rusty1s .

edge_attr contain the edge features with shape [E, D] (see https://rusty1s.github.io/pytorch_geometric/build/html/notes/introduction.html).

edge_attr contain the edge features with shape [E, D] (see https://rusty1s.github.io/pytorch_geometric/build/html/notes/introduction.html).

Im not sure how I missed that... Thanks. Ill give it a shot and let you know if I get stuck.

@rusty1s , do all nodes need to be for training or testing? What would be the process to add nodes that are supporting?

You can mask out nodes that are not relevant to the loss function (see https://github.com/rusty1s/pytorch_geometric/blob/master/examples/cora.py)

Thanks for sharing the pytorch_geometirc.
I have a question for making my own dataset.
Take the three nodes figure (link )as an example.
The corresponding code is as follows.
edge_index = torch.tensor([[0, 1, 1, 2],                            [1, 0, 2, 1]], dtype=torch.long)
I searched COO format in google, and got nothing.
Could you pls help to figure out what is the COO format, and why the figure is defined as a (2*4) matrix?
Thanks!

For a graph with 4 nodes (not the open triangle in the above figure), for instance link.
The edge_index is defined as:
edge_index = torch.tensor([[1, 2, 2, 3,3,4,4,1], [2, 1, 3, 2,4,3,1,4]], dtype=torch.long)
Is it right?
If it is right, what is the motivation to design edge_index in a (2*4) shape?
Thanks!

Hi,
we represent adjacency matrices by edge_index and edge_attr similar to the sparse tensor format of PyTorch (https://pytorch.org/docs/stable/sparse.html), except that we do not wrap it in a torch.sparse_coo_tensor (we can differentiate w.r.t. edge_attr, something which is currently not supported for sparse tensors in PyTorch). You can read more about the coo format here or here.
The reason to design edge_index as a [2, E] tensor instead of [E, 2] is the contiguous memory representation of the row and col data.
Your example looks correct, however, indices should start at zero:
edge_index = edge_index - 1

Thanks for the quick response! Problem solved. Thanks!


Bugfix: getting registered DimensionType by type id.

See https://github.com/SpongePowered/SpongeForge/issues/1429
This helps compatibility with Forge mods using vanilla DimensionType and Forge's DimensionManager.
I've done a search and the only place WorldManager.getRegisteredDimensionIds() is referenced in Sponge code is from Forge's DimensionManager.

Many thanks for looking into this! Big kudos to you.

  Sponge is impressive, and has my full support.


<EMAIL_ADDRESS>rebuild with retagged version

Build<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

:beers: @BrewTestBot has triggered a merge.


DeserializeObject fails on property with type Dictionary<TKey, TValue> where TKey is an Enum

We have a .NET WebAPI controller that is serializing a Person class which contains a property of type Dictionary<AddressType, Address> where AddressType is an enum.  We are attempting to share the same .cs source files between our WebAPI server code and Bridge.NET client code for Person, AddressType, and Address.  When calling DeserializeObject on the client, we get the following error:
Invalid Enumeration Value
    at ctor (http://localhost:63375/Scripts/bridge/bridge.js:5226:31)
    at new ctor (http://localhost:63375/Scripts/bridge/bridge.js:5409:35)
    at Function.parse (http://localhost:63375/Scripts/bridge/bridge.js:4441:23)
    at Function.DeserializeObject (http://localhost:63375/Scripts/bridge/newtonsoft.json.js:1196:61)
    at Function.DeserializeObject (http://localhost:63375/Scripts/bridge/newtonsoft.json.js:1255:84)
    at Function.DeserializeObject (http://localhost:63375/Scripts/bridge/newtonsoft.json.js:1402:78)
    at Function.DeserializeObject (http://localhost:63375/Scripts/bridge/newtonsoft.json.js:1402:78)

I tried to demonstrate the issue on Deck, but I think it is failing in a different way because metadata is not being produced:
https://deck.net/5ca0a241e9c94e02a82ace4ddb6f44ea

See also Bridge.NET forums' thread: DeserializeObject fails on property with type Dictionary where TKey is an Enum

Deck reproducing the issue: https://deck.net/a17ac5321a9baa1e4075d2a5c33ad2b2
The string used here can be generated from native .NET if you run the same code, uncommenting the lines where it calls SerializeObject(). Corresponding dotnet fiddle (lines still commented out, like deck above): https://dotnetfiddle.net/xTQlOF


DM-22219: Add editor page for VSCode.

Possibly make a note about setup environments that people might have trigger from .bash_profile at login.

I've already forgotten exactly what you meant by this, though I imagine we discussed it in person at one point.  I'm going to merge as-is, and let you update this later if you like to avoid a big context switch for either of us.


Navbar Right-Dropdown Alignment

When align a dropdown to the right in a navbar, the dropdown-content goes out of the page.
v4 beta-example: https://jsfiddle.net/kh0sfpm8/

That's right because you want to align your dropdown to the right, so you just have to add dropdown-menu-right and everything is fine see : https://codepen.io/Johann-S/pen/gxNqrm


Added Predicates (not only for Match).

Towards #1157

Current coverage is 94.71%

Merging #1172 into master will increase coverage by +0.02% as of 3f76b42

@@            master   #1172   diff @@
======================================
  Files           78      78
  Stmts         9179    9185     +6
  Branches      1746    1746
  Methods          0       0
======================================
+ Hit           8692    8700     +8
- Partial         88      90     +2
+ Missed         399     395     -4

Review entire Coverage Diff as of 3f76b42

Powered by Codecov. Updated on successful CI builds.


Test hangs when referencing request form two features

Follow-up on https://stackoverflow.com/questions/57502369/is-there-a-better-way-to-reference-sub-features-so-that-this-test-finishes
Test hangs when running setup mentioned in stack overflow. I've attached a skelton project in the zip.
hangtest.zip
When I run at the command line mvn clean test, then use "Run Test" code sense in the UserRunner.java I get this error
java.lang.IllegalAccessError: tried to access field com.intuit.karate.http.HttpRequestBuilder.paths from class com.intuit.karate.http.HttpRequestBuilderAccAccess
	at com.intuit.karate.http.HttpRequestBuilderAccAccess.get(Unknown Source)
	at net.minidev.json.reader.BeansWriterASM.writeJSONString(BeansWriterASM.java:21)
	at net.minidev.json.JSONValue.writeJSONString(JSONValue.java:596)
	at net.minidev.json.reader.JsonWriter.writeJSONKV(JsonWriter.java:392)
	at net.minidev.json.reader.JsonWriter$7.writeJSONString(JsonWriter.java:145)
	at net.minidev.json.reader.JsonWriter$7.writeJSONString(JsonWriter.java:128)
	at net.minidev.json.JSONValue.writeJSONString(JSONValue.java:596)
	at net.minidev.json.JSONValue.toJSONString(JSONValue.java:632)
	at net.minidev.json.JSONValue.toJSONString(JSONValue.java:610)
	at com.intuit.karate.JsonUtils.toJson(JsonUtils.java:130)
	at com.intuit.karate.JsonUtils.toJsonDoc(JsonUtils.java:172)
	at com.intuit.karate.core.FeatureResult.getCallArgPretty(FeatureResult.java:159)
	at com.intuit.karate.core.Engine.callHtml(Engine.java:296)
	at com.intuit.karate.core.Engine.stepHtml(Engine.java:373)
	at com.intuit.karate.core.Engine.saveResultHtml(Engine.java:403)
	at com.intuit.karate.junit5.FeatureNode.lambda$next$0(FeatureNode.java:88)

if I un-comment the last line in special-request.feature the error no longer occurs, so I believe this is the root error behind the StackOverflow post.
For some reason, running mvn clean test doesn't actually run the karate tests.(Not sure why, maybe junit version difference) but running with VS code's code hints in UserRunner.java does.

just make this one line change:
* def currentRequest = karate.prevRequest

This is explained in the documentation: https://github.com/intuit/karate#karate-prevrequest
If that is not your intent, raise a feature request. the HttpRequestBuilder is not a public API.
The maven behavior is a known issue which apparently goes away if you upgrade your maven install, but we anyways fixed it: https://github.com/intuit/karate/issues/823


Aerosol HISTORY for M21C

Updates the R21C History follow to incorporate changes to the aerosol…output collections

@acollow, thank you.
1- The description of aer_inst_1hr_glo_L needs to change to '1-Hourly' .
2- I see that you left the reftime = 1 for the adg collections after changing from 2-hourly to 1-hourly. I presume you tested that the output comes out at the appropriate time, right?

Ok, I updated the "1-hourly".
In my test, I got hourly files on the half hour (see /discover/nobackup/acollow/r21c/c24_aerohist/scratch)

Ok, I updated the "1-hourly". In my test, I got hourly files on the half hour (see /discover/nobackup/acollow/r21c/c24_aerohist/scratch)

Ok great.

@elakkraoui @sdrabenh is out until Thursday, I think. If you are good with this, I can merge it into R21C for you.

@elakkraoui @sdrabenh is out until Thursday, I think. If you are good with this, I can merge it into R21C for you.

Yes please, you can merge it. Thanks.

@elakkraoui @sdrabenh is out until Thursday, I think. If you are good with this, I can merge it into R21C for you.

Yes please, you can merge it. Thanks.

@elakkraoui Merged!


update: Web Application with examor

Hi, This is a fun learning app I made, hope it will be added to the list

Thanks


Be explicit about how OIDs are used in sharded filepath

In your documentation, you state:

These filters ensure that large files aren't written into the repository proper, instead being stored locally at .git/lfs/objects/{OID-PATH} (where {OID-PATH} is a sharded filepath of the form OID[0:2]/OID[2:4]/OID), synchronized with the Git LFS server as necessary.

So with the example OID of:

oid sha256:4d7a214614ab2935c943f9e0ff69d22eadbb8f32b1258daaa5e2ca24d17e2393

would the sharded filepath become:

.git/lfs/objects/4d7/7a2/4d7a214614ab2935c943f9e0ff69d22eadbb8f32b1258daaa5e2ca24d17e2393

or

.git/lfs/objects/4d/7a/4d7a214614ab2935c943f9e0ff69d22eadbb8f32b1258daaa5e2ca24d17e2393

Your syntax is confusing, as I don't know if it is inclusive or exclusive range syntax, python indexing syntax, etc. that you are using.  I'm thinking it's the latter. . .
Please update docs with a sharded filepath example based on the example OID you already provide.  Alternatively, please answer the above, and I can submit a PR

The latter:
.git/lfs/objects/4d/7a/4d7a214614ab2935c943f9e0ff69d22eadbb8f32b1258daaa5e2ca24d17e2393

I'll get a quick spec tweak out soon.  Thanks for pointing the ambiguity out.


Clodfront Subdomain Takeover No Longer Possible

Hi,
Bad news for cloudfront subdomain takeovers.Now aws want a ssl certificate for the domain you want to add to your CF distribution.Read [more here](https://docs.aws.amazon.com/AmazonCloudFront/latest/DeveloperGuide CNAMEs.html#alternate-domain-names-requirements) .When i try to do a subdomain takeover i get this error
https://i.postimg.cc/QCZwky0Y/aws-subdomain-takeover-not-possible.png

Correct link for the announcement
https://docs.aws.amazon.com/AmazonCloudFront/latest/DeveloperGuideCNAMEs.html#alternate-domain-names-requirements


Land/Water conditional sounds not playing (Wolf, Dog and Husky)

The conditional step sounds for Wolf/ Dog / Husky) are not playing in TEN but are being triggered in Wadtool.
https://user-images.githubusercontent.com/84292688/147072311-217fe5ad-f83e-46cb-b8d5-b63b3d858b56.mp4
https://user-images.githubusercontent.com/84292688/147072886-b243af55-102f-4b23-b173-8ab82078a527.mp4

Yeah, I can confirm Lwmte's message. The wolf is not triggering the water sounds because although it's in the 1 click water, the code detects it's in the upper room. I could see the On dry and Always sounds yes work, only issue is with the On water ones.
I'm afraid the solution will have to wait for:

Pathfinding fix Right now (Nov 2023) pathfinding is incomplete, enemies don't know very well how to avoid water rooms. So Lwmte applied a hack to force enemies death if these enter in a water room (clearly by any mistake in their behaviour). So if we fix this issue of wolves (and low creatures) can enter in properly water rooms, they may die for that hack.

Water color fix Another issue we've got with the item object shader, is that it can only paint the ambient color from 1 room, so if we put enemies in the water room, they will get the water color in all the body. (In case of low enemies, we are saying they will get the water color in all its meshes). So fixing the shader should also be a preferential step before go on with this issue.

At the end I made a room number recalculation in the sounds function to fix this issue.
May have been more ideal to have an item.RoomNumber that really works to get the room where the enemy is standing, however change that, could provoke many more issues in unexpected places (including the interaction of creatures with floordata and bridges).
So  I did here what I learnt from the water color fix, and I used a local variable to store the real room where the item is, in this way wolves that are in water now detects the water room correctly so they play the water sounds.
Also I removed the camera conditions (so now is not required the camera were in water room to can listen the water sounds, nor it to be out of water to can listen the land sounds. If this feature is not a but and is really demanded, notify to implement that in addition to the current condition (not instead of the current conditions as originally was).
The fix has been pushed to the branch:
https://github.com/MontyTRC89/TombEngine/pull/1256
Is in testing and review process, once it get merged if everything goes well I'll close this issue.


Deprecate ghcide tool and delete the VSCode extension

Closes #935

I am an happy user of the ghcide tool and I'd like to ask to not deprecate it.
I know that I am supposed to use the Haskell-language-server. Unfortunately, that's not an option for me. Just like hie in the past, it comes with too many features and dependencies that cause too much overhead to build and make them work in my development environment -- each time I tried it took me a long time to install it and get it to work and it would usually break soon due to dev-environment changes (like a new cabal or ghc version or other often non-obvious reasons).
What I like about ghcide is that it provides a non-opinionated set of basic IDE features. Those features cover everything that I need most (type checking, completion, jump to definition, show type) in a single easy to build, efficient, and robust tool. Haskell-language-server is adding features on top of it that I consider less important. Moreover many of the features the additional features of haskell-language-server seem opinionated to me. And they come at the price of a much more difficult to install and to use product.

To reiterate, in my experience, ghcide is the first and only Haskell IDE tool (at least for vim) that just builds (even relatively quickly with a simple cabal install) and just works out of the box.

@larskuhtz thanks for the feedback. It seems looking at your list of issues, the main two are:

HLS is less GHC/Cabal/whatever changes compatible. Is this still true? It certainly was with HIE, but I was hoping HLS was less the case. Are there any specific dependencies that have caused you problems? Most of the GHC/Cabal compatibilities should be coming from hie-bios (which is shared) or GHC bumps (which are tested to the same versions with HLS and Ghcide).
HLS takes longer to build. Do we have numbers on just how long? Would be useful information to have.
Are there any other reasons you don't want to use HLS, but prefer Ghcide?

It may be that with some flags you can trim down HLS to very little over and above Ghcide (maybe you already can and we just haven't documented), or it may be that the work we've done means its no longer as problematic as it used to be.

@ndmitchell I haven't used HLS yet. I tried to compile it using cabal v2-run ./install.hs --project-file install/shake.project hls. It failed with an TH related error for which it was not obvious to me how to fix it. ghcide builds with a simple cabal build on a plain ghc/cabal setup, which is a big plus for me. (To be fair, HLS actually failed while compiling ghcide, however when I build ghcide directly, I can just repeat the build and it passes on the second try. In the context of an HLS build that doesn't work).
HLS has dependencies that have been causing build issues in the past or would only build with patches, like for instance brittany. Overall the project structure of HLS is more complex and my experience tells me that projects of that kind tent to be fragile -- e.g. can take a longer time to be ported to new ghc versions. Ideally, an IDE tool should be available broadly across many different dev-environments, because I can't afford to switch or reconfigure my development tools for each project that I work on.
I understand that more advanced IDE tools provide a lot of value to teams working in relatively homogeneous and stable environments where it pays off to spent the extra time to setup and maintain those tools. But I also think that its desirable to have tools that are focused on common basic needs and just work everywhere with little dependencies. To me it seems that HLS belongs to the former while Ghcide belongs to the latter category of tools.
So, I don't want to argue against HLS -- I will continue to try it and gladly use it if it works for me. I just like to continue to use ghcide, at least as a fallback solution that is available when HLS isn't.

@larskuhtz i agree with almost all arguments.
I think @pepeiborra plan is to slim down hls to get a version with lot less dependencies, see https://github.com/haskell/haskell-language-server/pull/647#issuecomment-739138203
That said, i think we will keep ghcide executable at mid (long?) term to use it in ghcide tests.
It would be great to open an issue in hls about your failing install but i want to note there are precompiled executables for all major os's and supported ghc versions that make you not need to even compile it: https://github.com/haskell/haskell-language-server/releases/tag/0.6.0

Exactly. As a user, you want to have a small ide with few dependencies. I want the same thing!
But as discussed in #936 we want to break down some of the monolithic completions code into multiple providers using the nice plugin model in HLS. Thus it seems natural to move this out of ghcide and into an HLS plugin.
These two desires should not be in opposition.

We need to make improvements to the HLS project definition and build system so that it's easy to select and build only a subset of the plugins.
And we need to get the current ghcide users onto HLS so that we can move the completions and other code without impacting them.

This PR is to get 2 in motion, and before merging I will add a link to this discussion for anyone interested in why we are doing this.

I think @pepeiborra plan is to slim down hls to get a version with lot less dependencies

That would be great, too, if HLS would be modular in a way that a build would just ignore dependencies that one doesn't care about or that might be less stable. What I really like about ghcide is that it "just builds" and "just works" with a vanilla ghc dev environment.
In any case my comments are meant as a compliment for ghcide and not criticism of hls.


[DOC] Updated broken links in the README and contributing docs files

PR Description
Please describe the changes proposed in the pull request:

Updated the Broken Links in the README and Contributing doc files.

**This PR resolves #817 **

PR Checklist

Please ensure that you have done the following:

[x] PR in from a fork off your branch. Do not PR from <your_username>:dev, but rather from <your_username>:<feature-branch_name>.

[x] If you're not on the contributors list, add yourself to AUTHORS.rst.

[x] Add a line to CHANGELOG.md under the latest version header (i.e. the one that is "on deck") describing the contribution.

Do use some discretion here; if there are multiple PRs that are related, keep them in a single line.

Automatic checks
There will be automatic checks run on the PR. These include:

Building a preview of the docs on Netlify
Automatically linting the code
Making sure the code is documented
Making sure that all tests are passed
Making sure that code coverage doesn't go down.

Relevant Reviewers

Please tag maintainers to review.

@ericmjl
@loganthomas
@samukweku

@nvamsikrishna05 kindly fix the failing checks.

@nvamsikrishna05 have a look at the notebook and kindly resolve it, using the same idea as your other PR

@samukweku It's the same issue as other PR, so I am waiting for that PR to merge and will update this after that.

@nvamsikrishna05 we're good to merge in the latest dev branch! Can you do that? I'll be able to do a reviw shortly. :smile:

@ericmjl Updated the PR. you can review it now. Thanks.

With @samukweku and @loganthomas both finishing the review, I'm good to go too.

Thanks @nvamsikrishna05!


Feature/minsulee2/dev env

Updates:

Setup dev mode
Update readme

I had a mistake at #4 (it's directly merged to main). I missing develop branch 😢 . After this pr merge, I will resolve this problem.

Thanks for the reviews! Merging now :)


Shortcut to symbol

Is there a way to map a shortcut to a symbol not on the keyboard?

Currently not. Sorry! Your request is currently tracken in #1207

...except for ALT codes...


Panic when linting tests/cases/compiler/sourceMap-LineBreaks.ts

Error
thread '<unnamed>' panicked at crates/oxc_linter/src/rules/eslint/no_useless_escape.rs:70:28:
byte index 670 is not a char boundary; it is inside '\u{85}' (bytes 669..671) of `// @target: ES3
// @sourcemap: true

Repro:
RUST_BACKTRACE=1 cargo run --bin=oxc_cli lint ./tasks/coverage/typescript/tests/baselines/reference/sourceMap-LineBreaks.js threads=1

Note to self: run linter on all conformance suite to make sure nothing crashes.


8k context limit for 16k models with .txt attachments

I'm not able to give more than 8k tokens with a 16k or 128k gpt4-preview model.

Looks like turbo-16k only has an 8k token limit for output so you'll need to use a gpt-4 model like 32k or 1106-preview

Did you try it on the bot on the server?

the issue is the input token limit. I also tried it on the server bot with gpt-32k and same issue.

The file is just too big, I don't understand your issue here @ashra-academy

yes, but this file is not over the token limit. it is about 12k. And the gpt4 (32k limit) in open ChatGPT can handle the same input.
So this is clearly a bug in the bot.

@ashra-academy It is certainly not a bug in the bot, the bot summarization threshold is set to 5k. Your file is bigger than that so it won't work. I set the threshold to 100k, you can try again now with the bot on the server.

oh I didn't know about such a threshold parameter. It probably better be above 128k since the latest gpt4-preview is that much. Also how can I set this parameter on my own bot?

Above 128k doesn't make sense as it would hit the token limit before even beginning to summarize and also leaves no space for the summary afterwards, so something like 110k is more reasonable, you can set it with /system settings summarize_threshold


util: Simplify the listImagesFromTargetState helper

Change-type: patch

Resolves: #
HQ:
See:
Depends-on:
Change-type: major|minor|patch

Contributor checklist

[ ] Includes tests
[ ] Includes typings
[ ] Includes updated documentation
[ ] Includes updated build output

I self-certify!


Empty results when searching and NULL relation

I'm facing the same problem as this unsolved problem in stackoverflow:
http://stackoverflow.com/questions/29331403/searching-with-flask-admin-returns-empty-results-when-a-searchable-column-is-nul
When a relation is NULL and is part of the searchable elements, it returns empty every time although the rest of parameters are correct.

Any idea about this issue?

Seems like Flask-Admin is using INNER JOIN somewhere, when building query. I will try to troubleshoot.

@mrjoes yes, that makes sense. Is there any plan for fixing it?

@mrjoes Could it be replace with a LEFT JOIN?

These issues might be related?: https://github.com/flask-admin/flask-admin/issues/547
It would really help if there was a Gist that reproduces the issue.

@pawl  #547 talks about problems with inputs different than strings.

@msempere I tried reproducing this issue with this gist, but couldn't reproduce it: https://gist.github.com/pawl/96afa9e1edbbe6b7e021


Interaction with plugins which take actions on first loaded database

I haven't done a truly sanitized test of this yet (just the impacted plugins), but I noticed that after switching my package from running "datasette data.sqlite" to "datasette /folder" format, the app crashes when you try to use a plugin that takes an action on the first-loaded database, such as Upload CSVs. This is what happens when merely selecting Upload CSVs after adding the Upload DBs plugin:
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/dist-packages/datasette/app.py", line 1254, in route_path
    response = await view(request, send)
  File "/usr/local/lib/python3.9/dist-packages/datasette/app.py", line 1442, in async_view_fn
    response = await async_call_with_supported_arguments(
  File "/usr/local/lib/python3.9/dist-packages/datasette/utils/__init__.py", line 977, in async_call_with_supported_arguments
    return await fn(*call_with)
  File "/usr/local/lib/python3.9/dist-packages/datasette_upload_csvs/__init__.py", line 46, in upload_csvs
    db = [
IndexError: list index out of range
INFO:     <IP_ADDRESS>:46308 - "GET /-/upload-csvs HTTP/1.1" 500 Internal Server Error

The screen when this happens looks like this:

I think opening this issue against this plugin as opposed to the CSV one is the right call, because I expect the issue to occur with multiple plugins. (I have not tested it extensively, but I believe datasette-saved-queries just silently fails to create its table, for example.)
I really like the paradigm this plugin creates, which does not require an initial database file, but I suspect it introduced some issues that may need to be addressed across the plugin ecosystem.

Should be fixed in this release: https://github.com/simonw/datasette-upload-csvs/releases/tag/0.8.1

Opened a new Datasette core issue to track this and potentially come up with a more general solution:

https://github.com/simonw/datasette/issues/1807


Authors not found (404) even though they exist

A lot of authors return 404s via the API, even though both their resource overview and profile page exist and are publicly accessible.
Possibly related to #17
A few examples:
https://api.spigotmc.org/simple/0.1/index.php?action=getAuthor&id=45870 => https://www.spigotmc.org/resources/authors/45870/

https://api.spigotmc.org/simple/0.1/index.php?action=getAuthor&id=797245 => https://www.spigotmc.org/resources/authors/797245/

https://api.spigotmc.org/simple/0.1/index.php?action=getAuthor&id=800551 => https://www.spigotmc.org/resources/authors/800551/

https://api.spigotmc.org/simple/0.1/index.php?action=getAuthor&id=798575 => https://www.spigotmc.org/resources/authors/798575/

they all don't have identities, maybe thats it?

Yeah, was thinking the same. Looks like it's doing an inner join on 3 tables, including the one seemingly containing the identity info, so it might just skip everyone without identity info.
https://github.com/SpigotMC/XenforoResourceManagerAPI/blob/master/support/Database.php#L55-L63

Though, I've also stumbled upon a couple users that are returned by the API which have an empty identities array (instead of an object)

if matters, while that api action returns 404, if you use action=getResourcesByAuthor&id=45870  it shows their resources.

I'll do some investigation on this ASAP.

Though, I've also stumbled upon a couple users that are returned by the API which have an empty identities array (instead of an object)
e.g.
https://api.spigotmc.org/simple/0.1/index.php?action=getAuthor&id=68433
https://api.spigotmc.org/simple/0.1/index.php?action=getAuthor&id=68413

Good catch on the object vs array. That was unintentional. I'll make that more consistent.

So, on my local install of XenForo every new user has the xf_user_field_value's automatically generated although they are blank.
I am guessing Spigot's installation is not doing this, for whatever reason.
I am not sure if a different type of join would work better or if I should just do two separate queries for the necessary data.
Comments?

@simpleauthority yeah I just picked a random user (45870) and xf_user_field_value is indeed empty for them

Doing a left instead of an inner join should, in theory, help with this. Instead of not returning the users without identities at all, they should return NULL identities when left-joining.

Well, it's worth a try for sure. I'm not sure I understand why for some users it works but for others it doesn't when xf_user_field_value is always present but empty. The join should work in any case. I can't reproduce locally, so the way we'd have to test would be push to prod and see if it helped which is less than ideal.

Could just wipe em from your test db.
I'm not sure why they're not filled in on Spigot; could be xenforo versions or way users are created

Could just wipe em from your test db.
I'm not sure why they're not filled in on Spigot; could be xenforo versions or way users are created

That's the thing though. On my test db, I have a user with blank xf_user_field_value's like you described (none are null) and it works fine. Since they're not null, the inner join should work just fine.
I'm open to trying the left join, but I can't reproduce locally so I would have no way of knowing if the change of join works or not.

I'm open to trying the left join, but I can't reproduce locally so I would have no way of knowing if the change of join works or not.

DELETE FROM xf_user_field_value WHERE user_id=99
Its a test DB, you should just be able to do that and see if it still works

I gotcha. I'll try it soon.

I think the pattern from what i see is when a user doesn't have any resources.

This is weird, any updates on the solution?

I just regained access to an XF installation, so I'll be debugging this soon.

I just regained access to an XF installation, so I'll be debugging this soon.

Alright sounds good

Works fine locally with missing rows in the xf_user_field_value table -- in my local install, they're never missing, but they are in Spigot. This may be a patch fix from XF that I have which Spigot doesn't...unknown. Regardless, #29 fixes it.


Switch to camelCase option names

Is your feature request related to a problem? Please describe.
Standard, Vue and React all lint against snake casing. Snakes also cause issues with fonts and cut out at certain sizes, creating errors. https://github.com/microsoft/vscode/issues/35901
Describe the solution you'd like
For version 5 ish, deprecate and log a message. Remove in version 6.
Describe alternatives you've considered
I've considered learning French, it's not for me.
Additional context
https://eslint.org/docs/rules/camelcase

There is no reason to change. CamelCase is your preference and it is supported. It is not mine and everyone is satisfied.


PropertyBuilder specific methods can be used by different property types

In EF6 we have type specific types and its' methods (e.g. StringPropertyConfiguration and IsUnicode()) to configure mapping between C# Entity and SQL Table. Now, in EF Core we can apply IsUnicode() method of PropertyBuilder FluentAPI even on datetime type and so on. Would not it be better to have only type specific PropertyBuilders as it was previously implemented in EF 6 ? Thanks.
EF 6
    public StringPropertyConfiguration IsUnicode(bool? unicode)
    {
      this.Configuration.IsUnicode = unicode;
      return this;
    }

EF Core
    // TProperty has no constraint
    public virtual PropertyBuilder<TProperty> IsUnicode(bool unicode = true)
    {
      return (PropertyBuilder<TProperty>) base.IsUnicode(unicode);
    }

Further technical details
EF Core version: 2.0
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Windows 10
IDE: Visual Studio 2017

@rooonikz There were several reasons for doing things differently in EF Core, but the probably the most important is that EF Core can be used with any CLR type that a database provider can map, while EF6 only worked with a limited set of pre-defined CLR types.


Add vit staging tool navigation

Builds off of this PR.
Implements this Pivotal Card.

The location of the VIT is going to change per discussions.


missing return value

# omni complete handler
def lsp#omniFunc(findstart: number, base: string): any     # <-----
  if !bufnrToServer->has_key(bufnr())
    return                                                                             # <------
  endif

Thanks for reporting the issue. I have pushed 44cadde9f80f7fa89d03638afeb955e58707232b to fix this issue.

thx.
--
shane.xb.qian

From: Yegappan Lakshmanan @.>
Sent: Friday, January 21, 2022 11:31:47 PM
To: yegappan/lsp @.>
Cc: Shane-XB-Qian @.>; Author @.>
Subject: Re: [yegappan/lsp] missing return value (Issue #33)
Thanks for reporting the issue. I have pushed 44caddehttps://github.com/yegappan/lsp/commit/44cadde9f80f7fa89d03638afeb955e58707232b to fix this issue.
—
Reply to this email directly, view it on GitHubhttps://github.com/yegappan/lsp/issues/33#issuecomment-1018614538, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADHMHQPZLF6LBITQSQVVAADUXF36HANCNFSM5MPD6D3A.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID: @.***>


Add Soundcloud Support

It would be nice to have support for embedding soundcloud players.
It's a popular audio streaming platform used by musicians, labels and podcasts.

Anyone able to make a PR for it?

I can take a look at this.
I have vacation days for the next two weeks.
It looks like soundcloud supports oEmbed.
I think it makes sense to implement support for generic oEmbed resources instead of implementing a soundcloud specific solution.
We could use an oembed php library and try to use oEmbed for all unknown services.
@dereuromark what do you think?

I am fine either way.
We could for now add basic support for soundcloud the current way and start a new branch with improved feature support as generic solution on top?

I took a look into your current implementation in my lunchbreak.
So you just extract the Id from the input url by regex, then insert the id into another url and build an iframe?
Then this is very easy to implement.

Created a pull request
https://github.com/dereuromark/media-embed/pull/44/commits

Awesome, thank you!
As for the generic rewrite, any maybe using value objects and more, we should have a separate discussion, and create a new develop branch for a new major release then.

https://github.com/dereuromark/media-embed/issues/45


vault-worker fails to start in nightly docker-compose

🐞 Bug Report
Affected Services
Vault-worker component in nightly docker-compose.
Description
Originally reported by @jdharms:

docker-compose up vault-worker crashes because insecureskipverify has been renamed to insecureSkipVerify.

Closed via PR #153


Migration Errors Attempting Dev Branch v0.8.2

Recently installed a fresh v0.7.2 docker compose instance of Archivebox, only transferring over my /data folder from my previous VM (was having some strange issues with my previous install and decided to just try and start fresh).  Got things up and running and then just for kicks I decided I would try out v0.8.2, so I did the following:
cd ~/archivebox
[edit docker-compose.yml to archivebox:dev]
docker compose pull
docker compose up

It then gets through a fair number of 'ArchiveResult' migrations that I don't fully understand, but eventually the archivebox container crashes and dies with the following error:
archivebox-1            |    Updating 2169 SnapshotTag.snapshot_id values in place... (may take an hour or longer for large collections...)
archivebox-1            | Migrated 0/2169 SnapshotTag objects...
archivebox-1            | Migrated 100/2169 SnapshotTag objects...
archivebox-1            | Migrated 200/2169 SnapshotTag objects...
archivebox-1            | Migrated 300/2169 SnapshotTag objects...
archivebox-1            | Migrated 400/2169 SnapshotTag objects...
archivebox-1            | Migrated 500/2169 SnapshotTag objects...
archivebox-1            | Migrated 600/2169 SnapshotTag objects...
archivebox-1            | Migrated 700/2169 SnapshotTag objects...
archivebox-1            | Migrated 800/2169 SnapshotTag objects...
archivebox-1            | Migrated 900/2169 SnapshotTag objects...
archivebox-1            | Migrated 1000/2169 SnapshotTag objects...
archivebox-1            | Migrated 1100/2169 SnapshotTag objects...
archivebox-1            | Migrated 1200/2169 SnapshotTag objects...
archivebox-1            | Migrated 1300/2169 SnapshotTag objects...
archivebox-1            | Migrated 1400/2169 SnapshotTag objects...
archivebox-1            | Migrated 1500/2169 SnapshotTag objects...
archivebox-1            | Migrated 1600/2169 SnapshotTag objects...
archivebox-1            | Migrated 1700/2169 SnapshotTag objects...
archivebox-1            | Migrated 1800/2169 SnapshotTag objects...
archivebox-1            | Migrated 1900/2169 SnapshotTag objects...
archivebox-1            | Migrated 2000/2169 SnapshotTag objects...
archivebox-1            | Migrated 2100/2169 SnapshotTag objects...
archivebox-1            |    Updating 1 Tag.id, ArchiveResult.uuid values in place...
archivebox-1            | Traceback (most recent call last):
archivebox-1            |   File "/usr/local/bin/archivebox", line 8, in <module>
archivebox-1            |     sys.exit(main())
archivebox-1            |              ^^^^^^
archivebox-1            |   File "/app/archivebox/cli/__init__.py", line 181, in main
archivebox-1            |     run_subcommand(
archivebox-1            |   File "/app/archivebox/cli/__init__.py", line 118, in run_subcommand
archivebox-1            |     module.main(args=subcommand_args, stdin=stdin, pwd=pwd)    # type: ignore
archivebox-1            |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
archivebox-1            |   File "/app/archivebox/cli/archivebox_server.py", line 64, in main
archivebox-1            |     server(
archivebox-1            |   File "/app/archivebox/util.py", line 160, in typechecked_function
archivebox-1            |     return func(*args, **kwargs)
archivebox-1            |            ^^^^^^^^^^^^^^^^^^^^^
archivebox-1            |   File "/app/archivebox/main.py", line 1311, in server
archivebox-1            |     run_subcommand('init', subcommand_args=['--quick'], stdin=None, pwd=out_dir)
archivebox-1            |   File "/app/archivebox/cli/__init__.py", line 118, in run_subcommand
archivebox-1            |     module.main(args=subcommand_args, stdin=stdin, pwd=pwd)    # type: ignore
archivebox-1            |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
archivebox-1            |   File "/app/archivebox/cli/archivebox_init.py", line 43, in main
archivebox-1            |     init(
archivebox-1            |   File "/app/archivebox/util.py", line 160, in typechecked_function
archivebox-1            |     return func(*args, **kwargs)
archivebox-1            |            ^^^^^^^^^^^^^^^^^^^^^
archivebox-1            |   File "/app/archivebox/main.py", line 356, in init
archivebox-1            |     for migration_line in apply_migrations(out_dir):
archivebox-1            |                           ^^^^^^^^^^^^^^^^^^^^^^^^^
archivebox-1            |   File "/app/archivebox/util.py", line 160, in typechecked_function
archivebox-1            |     return func(*args, **kwargs)
archivebox-1            |            ^^^^^^^^^^^^^^^^^^^^^
archivebox-1            |   File "/app/archivebox/index/sql.py", line 156, in apply_migrations
archivebox-1            |     call_command("migrate", interactive=False, stdout=out)
archivebox-1            |   File "/usr/local/lib/python3.11/site-packages/django/core/management/__init__.py", line 194, in call_command
archivebox-1            |     return command.execute(*args, **defaults)
archivebox-1            |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
archivebox-1            |   File "/usr/local/lib/python3.11/site-packages/django/core/management/base.py", line 459, in execute
archivebox-1            |     output = self.handle(*args, **options)
archivebox-1            |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
archivebox-1            |   File "/usr/local/lib/python3.11/site-packages/django/core/management/base.py", line 107, in wrapper
archivebox-1            |     res = handle_func(*args, **kwargs)
archivebox-1            |           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
archivebox-1            |   File "/usr/local/lib/python3.11/site-packages/django/core/management/commands/migrate.py", line 357, in handle
archivebox-1            |     post_migrate_state = executor.migrate(
archivebox-1            |                          ^^^^^^^^^^^^^^^^^
archivebox-1            |   File "/usr/local/lib/python3.11/site-packages/django/db/migrations/executor.py", line 135, in migrate
archivebox-1            |     state = self._migrate_all_forwards(
archivebox-1            |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^
archivebox-1            |   File "/usr/local/lib/python3.11/site-packages/django/db/migrations/executor.py", line 167, in _migrate_all_forwards
archivebox-1            |     state = self.apply_migration(
archivebox-1            |             ^^^^^^^^^^^^^^^^^^^^^
archivebox-1            |   File "/usr/local/lib/python3.11/site-packages/django/db/migrations/executor.py", line 255, in apply_migration
archivebox-1            |     state = migration.apply(state, schema_editor)
archivebox-1            |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
archivebox-1            |   File "/usr/local/lib/python3.11/site-packages/django/db/migrations/migration.py", line 132, in apply
archivebox-1            |     operation.database_forwards(
archivebox-1            |   File "/usr/local/lib/python3.11/site-packages/django/db/migrations/operations/special.py", line 196, in database_forwards
archivebox-1            |     self.code(from_state.apps, schema_editor)
archivebox-1            |   File "/app/archivebox/core/migrations/0059_tag_id.py", line 53, in update_archiveresult_ids
archivebox-1            |     assert tag.name
archivebox-1            |            ^^^^^^^^
archivebox-1            | AssertionError
archivebox-1 exited with code 1

Any thoughts on what might be going on here?  Thanks!!

It's complaining that you have a Tag with an empty name=``. I recommend deleting that Tag in the Admin UI as it will only cause problems, but in the meantime I also just pushed a code fix for this edge case 1d31b88f.
Thanks for helping test the beta release!
You can pull the latest :dev build and re-run to get the fix. It should pick up the migrations were it left off without issue:
docker compose pull archivebox
docker compose down
docker compose up

Comment back if you encounter any other issues and I'll reopen the ticket.


CASMINST-6103:  Make sure the HOSTNAME env is set in start-goss-servers.sh

Summary and Scope
Found that goss-servers doesn't have access to the HOSTNAME env at startup.   This makes sure it is set to run any tests that need to know the current hostname.
Issues and Related PRs

Resolves CASMINST-6103

Testing
Tested on:

drax

Test description:
Tested without the change and verified that goss-servers service fails.   Tested with this change and verified that the goss-servers services is running and I was able to run the ncn-healthcheck-master endpoint.
Pull Request Checklist

[x] Version number(s) incremented, if applicable
[x] Copyrights updated
[x] License file intact
[x] Target branch correct
[ ] CHANGELOG.md updated
[x] Testing is appropriate and complete, if applicable
[ ] HPC Product Announcement prepared, if applicable

/backport release/1.4


Cannot install due to requirement of "sklearn"

🐛 Bug Report
When I try to pip install dora I encountered this error:
Collecting sklearn (from dora->-r requirements.txt (line 4))
  Downloading https://pypi.tuna.tsinghua.edu.cn/packages/a4/0b/d1c703256cf293be77b7db44dbef62251fe02a97d0bef981f7120b0b0c0f/sklearn-0.0.post11.tar.gz (3.6 kB)
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [18 lines of output]
      The 'sklearn' PyPI package is deprecated, use 'scikit-learn'
      rather than 'sklearn' for pip commands.

      Here is how to fix this error in the main use cases:
      - use 'pip install scikit-learn' rather than 'pip install sklearn'
      - replace 'sklearn' by 'scikit-learn' in your pip requirements files
        (requirements.txt, setup.py, setup.cfg, Pipfile, etc ...)
      - if the 'sklearn' package is used by one of your dependencies,
        it would be great if you take some time to track which package uses
        'sklearn' instead of 'scikit-learn' and report it to their issue tracker
      - as a last resort, set the environment variable
        SKLEARN_ALLOW_DEPRECATED_SKLEARN_PACKAGE_INSTALL=True to avoid this error

      More information is available at
      https://github.com/scikit-learn/sklearn-pypi-package

      If the previous advice does not cover your use case, feel free to report it at
      https://github.com/scikit-learn/sklearn-pypi-package/issues/new
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

Environment

Python version: Python 3.10.12
Operating system: Linux version 5.4.0-91-generic (buildd@lcy01-amd64-017) (gcc version 9.3.0 (Ubuntu 9.3.0-17ubuntu1~20.04)) #102-Ubuntu SMP Fri Nov 5 16:31:28 UTC 2021

May the requirement be modified? I found a temporary workout:
SKLEARN_ALLOW_DEPRECATED_SKLEARN_PACKAGE_INSTALL=True pip install dora

Dora doesn't explicitely require sklearn, the issue is coming from some other package we depend on mostl ikely.

as mentioned in the readme: pip install -U dora-search


Sync integration/go.mod with root go.mod

the root go.mod no longer pins these two imports to specific versions as of:
https://github.com/containerd/containerd/pull/5613
Signed-off-by: Davanum Srinivas<EMAIL_ADDRESS>

@dims Can you rebase this branch against main to re-run the tests?

@kzys Done


add to_textgrid and to_aud so it's useful with vak.predict

closing this because a more general approach is described in #105


Slashlinks can contain emoji

This is a bug. We should be allowing only unicode word characters. Or else, if we allow emoji, we should update the spec and think through the phishing implications emoji might have.

Wait, nvm. What I'm seeing is the note content, not the slug. The slug is truncated at the-. This is WAI.


New: Display OGC API collection (OGC1)

This is the first of a series of 3 OGC samples. It shows an OGC feature layer
Please refer to common-samples/issues/2059 for details.

Since Viv is out today, I'll directly go to the next phase of review... 😆


[DOXFIX] delete System

change "system configuration" to "configuration"
https://alluxio.atlassian.net/browse/ALLUXIO-2009?filter=-4

Can one of the admins verify this patch?

thanks @silvaco for the contribution.
Can you please explain a bit more about this PR?

LGTM. Thanks for the patch!


feat(v2): create useDocusaurusContext

Motivation
Create a wrapper context function that allows access to context easily. It's quite troublesome now to have to import useContext and @docusaurus/context. Bundle them together as useDocusaurusContext.
Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan
Site still builds and can navigate each page.
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

in the future we might want to create index.js on client/exports so that we can do import ｛ useDocusaurusContext } from ‘@Docusaurus’

Yes I thought of that, but I think it should be in another PR, not this 😄


Cursor pagination

Continue #754
Things to do:
I released the clear_order_by in sea-query 0.25
So the dependency is to upgrade sea-orm to sea-query 0.25.1

Is there a PR for upgrading sea-query to 0.25?

Is there a PR for upgrading sea-query to 0.25?

Hey @tyt2y3, check https://github.com/SeaQL/sea-orm/pull/834

Is there any plan to have cursor in other directions? Like if we want to have created_at DESC to list most recent first?

Hey @penso, thanks for asking!!
This is exactly what .last(N) method is for.
use sea_orm::{entity::*, query::*, tests_cfg::cake};

// Create a cursor that order by `cake`.`id`
let mut cursor = cake::Entity::find().cursor_by(cake::Column::Id);

// Filter paginated result by `cake`.`id` > 1 AND `cake`.`id` < 100
cursor.after(1).before(100);

// Get last 10 rows (order by `cake`.`id` DESC but rows are returned in ascending order)
for cake in cursor.last(10).all(db).await? {
    // Do something on cake: cake::Model
}

Docs: https://www.sea-ql.org/SeaORM/docs/basic-crud/select/#cursor-pagination

Wouldn't it be easier to have:
let mut cursor = cake::Entity::find().cursor_by(cake::Column::Id, .desc)
for cake in cursor.first(10).all(db).await? {
}

so we don't need to convert/reverse the resulted array if we actually want to deliver the last ID first and in descending order?

Hey @penso, I think we have add two additional method - first_desc and last_desc - to the cursor:
// Get first 10 rows (order by `cake`.`id` ASC but rows are returned in descending order)
for cake in cursor.first_desc(10).all(db).await? {
    // Do something on cake: cake::Model
}

// Get last 10 rows (order by `cake`.`id` ASC but rows are returned in descending order)
for cake in cursor.last_desc(10).all(db).await? {
    // Do something on cake: cake::Model
}

Is there any drawback to just rev the Vec?

Is there any drawback to just rev the Vec?

Good question! @penso?
Note that first_desc and last_desc are performing Vec:::rev under-the-hood loll

No drawback than I can think of, except the mental thinking. I've done a "manual" cursor this week-end and will try to move to cursor_by and see if it's easier or not.

Please! And let us know your experience after using it :)


updated fishtape url

Updated the URL for fishtape to the new one to fix the build error on 301.

This has already been addressed, but thank you.


cp2k: enable SIRIUS planewave code and optimisations

Description of changes
This PR enables a set of new features for CP2K, namely the Planewave-Code SIRIUS with k-point sampling. Consequently, SIRIUS and its dependencies were also added in this PR. Furthermore, I've tried to enable GPU acceleration for CP2K. The cudatoolkit seems to be broken on current master (or not working on machines with pure AMD hardware?), but the AMD version compiles fine.
Otherwise, the GPU versions, especially CUDA, are mainly untested as I'm lacking this hardware.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Just one more thing: spla fails on Darwin. Limit spla and spfft to Linux?

Just one more thing: spla fails on Darwin. Limit spla and spfft to Linux?

Unfortunate, but only due to missing OpenMP libraries :thinking: Probably easy to fix if one has any knowledge about darwin. I haven't, so yes, have limited to linux :smile:

I think it is as simple buildInputs = ... lib.optional stdenv.isDarwin llvmPackages.openmp?
However, were these packages ever intended to run on Darwin?

costa fails on aarch64-darwin.

OK, here's another round with OpenMP libraries for Darwin in COSTA and SIRIUS.

Still does not build on aarch64-darwin. Keep in mind that CP2K is limited to x86_64-linux here also.
Maybe it's best to limit all new packages to linux? If someone wants to use them on darwin, they can turn it back on and test it properly.

I've limited sirius and costa to linux, now.


Support OLLAMA_URL and OLLAMA_HOST

Restores the original behavior of OLLAMA_URL,
which requires that the full path be specified,
including the /api prefix. Also supports the
OLLAMA_HOST behavior requested in
resolved ggozad/oterm#37 by adding the /api
prefix when constructing the OLLAMA_URL.

I will note that if this is rejected, the oterm README should be fixed — if the user sets OLLAMA_URL as currently described, the resulting URLs look like http://ollama:11434/api/api/generate

You are absolutely correct, this is better.