Implement JsonSchemaManager based on Stargate V2 TableManager

Extracted tasks from #1720 related to the schema management:

Add JSON schema to an existing and valid Document API table
Get JSON schema from an existing and valid Document API table
Delete JSON schema from the existing Document API table (maybe not needed, seems V1 does not have it)
Check if JSON matches the schema

We could make this non-bound to the any specific TableManager impl and have methods like:
Uni<JsonNode> getJsonSchema(Uni<Schema.CqlTable> table);

Uni<Void> attachJsonSchema(Uni<Schema.CqlTable> table, JsonNode schema);

Uni<Boolean> validateJsonSchema(Uni<Schema.CqlTable> table, JsonNode document);

Then the components using the class could go for authorized or not authorized schema access.
Note that caching of JSON schema is not needed anymore, as the table schemas are cached.

Done in #1847.


parameterize and increase squid disk cache size

We're easily filling it and it's rolling over at 5G

Without an increased cache some jobs fail since pulling large Debian packages takes too long: http://build.ros.org/job/Idoc__zbar_ros__ubuntu_trusty_amd64/8/console

The cache might help mask this problem. but it shouldn't take over an hour to download 716 MB of data as it does in that job even without the cache at all.
It does look like the same packages are at issue here. Aka texlive-latex-extra-doc which is the same large deb that we've seen issues with before.
00:42:09.325 Get:128 http://archive.ubuntu.com/ubuntu/ trusty/main texlive-pictures all 2013.20140215-1 [2,414 kB]
00:42:33.383 Get:129 http://archive.ubuntu.com/ubuntu/ trusty/main texlive-latex-extra all 2013.20140215-2 [7,286 kB]
00:43:50.637 Get:130 http://archive.ubuntu.com/ubuntu/ trusty/main texlive-latex-extra-doc all 2013.20140215-2 [317 MB]
01:38:46.539 Get:131 http://archive.ubuntu.com/ubuntu/ trusty/main texlive-latex-recommended-doc all 2013.20140215-1 [35.0 MB]
01:45:07.552 Get:132 http://archive.ubuntu.com/ubuntu/ trusty/main texlive-luatex all 2013.20140215-1 [8,204 kB]

Are these configuration options documented anywhere?

I added commented values in the slave config: https://github.com/ros-infrastructure/buildfarm_deployment_config/blob/master/slave/common.yaml#L24-L25

The comment doesn't mention yet what those values do, what their unit is, etc.


Memory leak!

The plugin is great, but there is a small memory leak.
I noticed that my static memory graph was going up. I had some footage of the game before I installed the plugin and the graph didn't go up.
Then I disabled the plugin and the memory leak is gone.
Here is a short video:
https://github.com/pkdawson/imgui-godot/assets/96019707/8ccfc1a1-3677-4192-b363-4f13719fe902

Thanks for the report. I'm not able to look into it right now, but I'll do a thorough leak analysis with Visual Studio before the end of the month.
 But I try to minimize those anyway, so this is worth investigating.

 But we can avoid some allocations and do a little manual cleanup of other stuff, so there's no more constant increase of memory usage.


Documentation: ArgoCD apps in any namespace incompatibility

For those using Applications in any namespace, the docs should mention that Image Updater is not compatible with any Application resources installed in namespaces other than argocd.

That is unfortunate. This is exactly what I'm doing and I just installed and deployed it :D

This is a dupe of #601 and #612 cover this.

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

Project coverage is 56.16%. Comparing base (fc1999f) to head (b6d29cd).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #541   +/-   ##
=======================================
  Coverage   56.16%   56.16%
=======================================
  Files          31       31
  Lines        3105     3105
=======================================
  Hits         1744     1744
  Misses       1222     1222
  Partials      139      139

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

Looks like most recent effort to add support for this was closed: https://github.com/argoproj-labs/argocd-image-updater/pull/672
Could we merge this warning so folks are at least aware? My change also adds a brief warning about this to the documentation root.

Thanks @seanmorton @ElanHasson @JStacy86 for the fix and review!

Hasn't #687 added the support of Applications in any Namespace?

@mcanevet thanks for pointing that out. @seanmorton @ElanHasson @JStacy86 could you please verify if the current master branch works for you when using image-updater for apps in any namespace?

I can confirm the image-updater can work with apps in any namespace if you use the APPLICATIONS_API approach.
https://github.com/argoproj-labs/argocd-image-updater/issues/601#issuecomment-2249819819

Thanks folks!


ssh: golang/crypto/ssh prioritizes server's kex algo instead of parsing known_hosts first

Original issue was reported here: https://github.com/coreos/bugs/issues/1186
The problem is that golang ssh implementation compares client's ServerHostKeyAlgos and server's ServerHostKeyAlgos and chooses first match depending on the order in supportedHostKeyAlgos variable:
// supportedKexAlgos specifies the supported host-key algorithms (i.e. methods
// of authenticating servers) in preference order.
var supportedHostKeyAlgos = []string{
        CertAlgoRSAv01, CertAlgoDSAv01, CertAlgoECDSA256v01,
        CertAlgoECDSA384v01, CertAlgoECDSA521v01,

        KeyAlgoECDSA256, KeyAlgoECDSA384, KeyAlgoECDSA521,
        KeyAlgoRSA, KeyAlgoDSA,
}

If remote host had KeyAlgoRSA key before and it was written into client's .fleetctl/known_hosts, then SSH on the remote host was upgraded (i.e. ecdsa-sha2-nistp256 support was added), then crypto/ssh automatically chooses first match: KeyAlgoECDSA256 and doesn't take into consideration already existing fingerprints. This causes Failed building SSH client: ssh: handshake failed: host key mismatch error message on the fleetctl client's side.

OpenSSH logic:

load known_hosts keys
parse them, determine whether remote hostname/IP corresponds to the known_hosts
set hostkeyalgs order (order_hostkeyalgs)
send SSH2_MSG_KEXINIT
receive host's fingerprint and compare it to the entry inside known_hosts

golang/crypto/ssh logic:

send SSH2_MSG_KEXINIT
receive host's fingerprint and compare it to the entry inside known_hosts

ClientConfig struct contains HostKeyAlgorithms. We can ParseKnownHosts crypto/ssh function to parse known_hosts and set HostKeyAlgorithms inside sshClientConfig.
Here is an example on how to parse known_hosts files https://github.com/golang/crypto/blob/master/ssh/keys_test.go#L391


Add full encryption support for the wallet db

Currently only the private keys are obfuscated when a wallet is encrypted. Adding full encryption support for the whole wallet db (unlocking it on startup) would add privacy in regards to the transaction history and other parameters to be stored in the wallet.db when the new privacy kicks in.

[ ] Encryption of the full contents wallet DB
[ ] Unlock at launch using the GUI
[ ] Unlock at launch using the command line

Bounty: 2500NAV (1 Bounty extra high + 1 Bounty medium)

@aguycalled I'll take a shot at this :)

I think a good way to implement this would be to have 2 passwords?
1 for tx data and another for the keys
Issue I see with 1 password for both is that if you encrypt whole wallet.dat with 1 password, that means the wallet will be completely unlocked while running it.
Not just when sending TX.

yes i agree with doing it like that with 2 passwords. like a PIN for the whole wallet, and then the spending password

So PIN for the tx data? And password for the keys? Or allow using password for tx data as well?

People can always just use PIN in the password field anyway 🗡

i would allow the pin for decrypting the whole wallet to be alphanumeric and of arbitrary length

Yeah so user decides how simple/secure it will be (Just like the main password)

Should I allow a user to encrypt the tx data and leave the keys open? (Strange question, but I know some users might do it out of spite) Hahaha

I've decide, tx data encryption requires keys to already be encrypted 🍡


Fix occasional ES 503 test errors

Caused by accessing ES indices before shard allocation completed
http://stackoverflow.com/a/30311714
Going to test this a bit in Travis before committing

Also applies some lint + PY2/PY3 best practices

@benghaziboy @MichaelButkovic @collin

:+1: very nice


Apply standards only if out of spec

First pass at only applying standards if out of spec.
Doesn't yet do this for branch operations.
NOTE: My python skills are sus. As such, I may have broken things by splitting into multiple files. One way to find out I guess. ;)
Partial Fix #3

Thanks @bhamail - have pushed a couple of updates/improvements.


fix: allow to use %s placeholder in execute command

Description
Hey! It will be nice to use %s placeholder in --execute
Linked Issues
#10
Additional context

I am not very sure, as %s can be ambiguous, and in this case, ppl might also want more information like the old versions, etc. We might be better use an extensible templating syntax, like $version etc, and providing an object of context. Also, documentation would be needed to introduce this.

Closed due to inactive


README formatting messed up on PyPI

If I run python setup.py sdist upload locally then the README text on PyPI is correctly rendered reStructuredText. If, on the other hand, the package is submitted to PyPI from Travis then we get the current mess:

Works now


Bumping versions for version update (0.1.28)

Platforms Impacted

[X] iOS
[X] macOS

Description of changes
What's new:

iOS:
-- Avatar: Update to background and text color to be in sync with react native

Mac:
-- Link: No hover effects when disabled
-- Link: Add correct focus ring

Verification
Built and launched macOS and iOS demo apps.
Pull request checklist
This PR has considered:

[ ] Light and Dark appearances
[ ] VoiceOver and Keyboard Accessibility
[ ] Internationalization and Right to Left layouts
[ ] Different resolutions (1x, 2x, 3x)
[ ] Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)

Microsoft Reviewers: Open in CodeFlow

@chripear you should merge your change before?

@chripear you should merge your change before?

I will wait for his changes before merging PR

@chripear you should merge your change before?

I will wait for his changes before merging PR

seemed to be merged


Copy button added to code block in documentation

In the existing documentation, we don't have any option to copy the example code. We have to manually select and copy it.
Material for MkDocs provides an easy way to integrate this feature. So, I have added that to our mkdocs.yml. This allows the users to copy the sample code easily and test it in their local system.
Also, in future, when we add more examples, the users will be able to take advantage of the simple copy button.
NOTE:
As I am facing an issue with testing the code (#97), this pull request is not tested as instructed in the development section. However, as it is a very minor change to the yml file of the documentation (which doesn't affect the code), I believe we can skip the testing for this one.

Last time I've tried to build the docs it wasn't able to build properly in the CI. The tool that's used to build the docs, novella, is no longer maintained. Seems to be a pattern with tooling I opted to use with the beaupy 🥲

Thanks for responding so promptly. May I ask, what was the initial reason to utilise novella? I have used mkdocs before and never felt the need to preprocess it in an isolated environment. mkdocs doesn't modify any files of the project (as far as I know). Can't we eliminate the need for novella and create a simple pipeline to publish it directly?

However, last time I've tried to build the docs it wasn't able to build properly in the CI.

How can we then merge this change into the project if the CI is building it properly?

Thanks for responding so promptly. May I ask, why we have novella integrated into this project? As far as I understand, it's basically a build system designed to process files in a temporary directory isolated from the project's source code. But, when we generate a static site using mkdocs, it doesn't modify any source code of the project. Can't we completely get rid of novella and modify the pipeline to build a static site using mkdocs build?

However, last time I've tried to build the docs it wasn't able to build properly in the CI. The tool that's used to build the docs, novella, is no longer maintained.

As the automatic merging is not working, what can I do to integrate the change into the project?

Apologies for brevity and lack of structure, I am only available from the phone atm.
I might be remembering this incorrectly but I believe novella was being used to build the markdown API doc from Python docstrings and to the best of my research, mkdocs didn't do that at the time I postulated the job to build the docs.
I am more than happy to look into alternatives soon.
As for the merging, automatic merging is not enabled for this repo, there is no issue merging these changes outside of the fact documentation doesn't currently build on release.

Thanks, I am going to merge these changes and I'll work on dealing with the documentation build issue.


typo in benchmark script

https://github.com/intel/xFasterTransformer/blob/main/benchmark/benchmark.py#L188

Appreciate your feedback. #261 is created to fix it.


update name of WP_STATELESS_MEDIA_ON_FLY constant

Change WP_STATELESS_MEDIA_ON_FLY  to WP_STATELESS_DYNAMIC_IMAGE_SUPPORT to be more accurate of the setting name and inline with our naming scheme.

@ewsopp we are having that constant for a very log time. Instead of replacing old constant we should keep them both and mark old constant as deprecated.

@alimuzzaman that's fine. Lets just add support for the new one and I'll remove the old from the documentation.


Feature-wipInitialize and submit code for rapid development of Docker

Proposed changes
Submitted shell script code for Dockerfile, Docker-Compose and related registers.
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

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

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

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

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

Hi, please check your own branch. This pr should not change so many files.

Hi, please check your own branch. This pr should not change so many files.

sorry~ :(


Deforum tab disappeared

Hello
today I wanted to set film aa deforum, but an error appeared (possibly after the Extension update). What should I do now? It detects deforum for me, but can't go to it.
from modules.ui_components import FormRow, FormColumn
ImportError: cannot import name 'FormColumn' from 'modules.ui_components' (D:\AI\stable-diffusion-webui\modules\ui_components.py)

ok i correct it
ui_components.py ater update missed Column Class, a add it and now work


vidcoder-beta: Add version 11.7-beta

Closes #1283

[x] I have read the Contributing Guide.

/verfity


Add methods for pausing/resuming performance metrics

Here is my take on some basic pause/resume functions for pausing timing and heap allocation measurements. I'll open an example soon over at rclcpp for demonstration.
Signed-off-by: Stephen Brawner<EMAIL_ADDRESS>

Tests for performance_test_fixture

Linux
Linux-aarch64
macOS
Windows

Tests for rclcpp benchmarks

Linux
Linux-aarch64
macOS
Windows

I quickly realize the logic was backwards. I think switching it to are_performance_measurements_active sounds better to me for some reason.

Rerunning tests of performance_test_fixture

Linux
Linux-aarch64
macOS
Windows

Err, forgot to run with benchmarks on

Linux
Linux-aarch64
macOS
Windows

Benchmark results:
---------------------------------------------------------------------------------------------------------------------------------------------------------------
Benchmark                                                                                Time             CPU   Iterations UserCounters...
---------------------------------------------------------------------------------------------------------------------------------------------------------------
PerformanceTestFixture/benchmark_pause_resume_timing/Enable Performance Tracking:1/manual_time             1155 ns          656 ns       605409 heap_allocations=0
PerformanceTestFixture/benchmark_pause_resume_timing/Enable Performance Tracking:0/manual_time             1155 ns          656 ns       606455
PerformanceTestFixture/benchmark_pause_resume_measurements/Enable Performance Tracking:1/manual_time       1160 ns          657 ns       602977 heap_allocations=0


fix: cplp-1042 - add data to seeding e.g. iam_identity_providers

https://jira.catena-x.net/browse/CPLP-1042

@Phil91 Could you please have a quick check after your vacation?

@evegufy looks fine to me, are the added files supposed to be empty?

Thank you! Yes, they are supposed to be empty, every file available under Seeder/Data should be in enabled for seeding as I don't find it intuitiv when the file is there but not enabled also for the most part they'll needed to be filled soon.
How did you decide whether to add the table to seeding via BaseEntityBatchSeeder class or via MappingEntityInsertOnlySeeder class?
To add the tables to MappingEntityInsertOnlySeeder was a bit of a gut feeling...

@evegufy looks fine to me, are the added files supposed to be empty?

Thank you! Yes, they are supposed to be empty, every file available under Seeder/Data should be in enabled for seeding as I don't find it intuitiv when the file is there but not enabled also for the most part they'll needed to be filled soon.
How did you decide whether to add the table to seeding via BaseEntityBatchSeeder class or via MappingEntityInsertOnlySeeder class? To add the tables to MappingEntityInsertOnlySeeder was a bit of a gut feeling...

@evegufy the decision on where to put it can be made pretty quickly. If the entity has an Id Property of type GUID and we want batch updating functionality, we would put it into BaseEntityBatchSeeder, if it has a combined key, e.g. CompanyServiceAccountAssignedRole and we don't want to have batch updating functionality, instead we always want to add the data in the JSON to the database we would use the MappingEntityInsertOnlySeeder. As soon as we have an entity with a combined key like CompanyServiceAccountAssignedRole and we want to do something special we would need a separate implementation of ICustomSeeder.
As an explanation of why IamServiceAccount and IamIdentityProvider are not in the BaseEntityBatchSeeder, both entities don't have a combined key, but per definition, we do a select on the Id column of an entity, both IamServiceAccount and IamIdentityProvider don't have an Id column, so we'd either need to rename the PrimaryKey Column of the entities, or use the MappingEntityInsertOnlySeeder 😄
For future improvement, I am planning to refactor the logic a bit, so that we only need one seeder which handles the selection of the database and inserts only the new values for us.


Trigger ArduCam Mini 5MP Plus

Hi,
I don't understand what is the difference, in terms of software, between the manual trigger and the internal one from the register. I mean, taking a photo in one way is the same as the other in terms of sending the capture command?
Thanks in advance,
giogrimo

the manual trigger is the almost the same as register command, the only difference is the manual trigger is issued by hardware signal, the register command is issued by software itself.

Ok, thank you.
giogrimo


fix the api request to change settings

It Fixes the API request to change the settings. Please closely test against the comments by Jeremy Audet.

state.insights should always be enabled whenever the user makes a change in the config
The 'View configuration manager playbooks' link should lead to https://github.com/RedHatInsights/config-manager/tree/master/playbooks
The three visible configurations should behave independently
When the user changes

the first configuration, should only affect the .state.remediation
the second configuration, should only affect .apply_state
the third configuration, should only affect .state.compliance_openscap

Codecov Report
Base: 47.65% // Head: 47.72% // Increases project coverage by +0.06% :tada:

Coverage data is based on head (ee32c1e) compared to base (a3bb7ab).
Patch coverage: 71.42% of modified lines in pull request are covered.

:exclamation: Current head ee32c1e differs from pull request most recent head 4513cdd. Consider uploading reports for the commit 4513cdd to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #71      +/-   ##
==========================================
+ Coverage   47.65%   47.72%   +0.06%
==========================================
  Files          56       56
  Lines         682      681       -1
  Branches      132      129       -3
==========================================
  Hits          325      325
+ Misses        341      340       -1
  Partials       16       16

Impacted Files
Coverage Δ

src/Components/Services/Services.js
0.00% <0.00%> (ø)

src/Components/Services/permissionsConfig.js
0.00% <ø> (ø)

src/Routes/Dashboard/index.js
0.00% <0.00%> (ø)

src/store/currStateReducer.js
100.00% <ø> (ø)

src/api/index.js
100.00% <100.00%> (+10.00%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@mkholjuraev Looking at Jeremy's comment, I'm wondering if the order of calling /states and /manage is important.

@bastilian thank you for having a look, I believe these 2 endpoints are independent of each other. /manage changes settings for apply_state, /states changes remaining.
Do you think that the order matters, if so can you tell me how?

When testing it seems that the "insights" setting in the states request always stays "enabled"

@bastilian it is on purpose. it is because of the comment (not sure if this is the optimal way though, I am open for suggestions) :
'.state.insights should be enabled every time the user changes settings. I have confirmed this with Christian Marineau.'


would you do to remove this "@" ?

npm install --save **@**angular2-material
would you do to remove this "@" ?
like this:
npm install --save angular2-material

https://docs.npmjs.com/misc/scope


Feature: Add Maniupulation gesture support to move holograms smoothly

Scenario:
The Holograms 211 course provides a basic implementation of the manipulation events.
This update will help add a more smooth and sophisticated approach to moving holograms where the user wants then to be.
Developers should be simply able to add this to the hologram in their app which they want to move.
Proposed change:

Update GestureManager.cs to support manipulation gestures
Add another script that consumes the manipulation data to help move holograms.

I'm working on this now.  Will have a PR out before the end of the week.

@micthom-msft thanks so much! Looking forward to it.

@NeerajW This was resolved with 3549fa022a51e87bb9be8f430461f9f53f1549bc.  Unfortunately I didn't use the magic words in the commit message but this should probably be closed.


Cannot trigger this event when switching from a dock to laptop mode

Currently I am testing if my app still can change the display brightness after switching from dock mode (IsSupported==false) to regular laptop mode (IsSupported==true). I notice that if I start my app with dock mode (connected to two monitors) and remove the laptop from the dock, IsSupported will remain false. The event is not triggered either, at least in visual studio the break point in the callback function is not hit. If I start my app without using the dock, then IsSupported will be true. Am I using the event wrong?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6eb1684d-bb79-3627-c972-5ab4614ca77d
Version Independent ID: 463b829c-7042-8d45-508c-1701fa0d28eb
Content: BrightnessOverride.IsSupportedChanged Event (Windows.Graphics.Display) - Windows UWP applications
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.graphics.display.brightnessoverride.yml&version=GBlive&_a=contents
Product: uwp
Technology: winrt-reference
Git Login: stevewhims
Microsoft Alias: stwhi

Hi, @laishiekai, thanks for reaching out with this issue. I experimented on my laptop (and on a desktop PC), but I wasn’t able to get a case like yours where the event was raised, so I couldn't determine whether or not the property value had changed. Unfortunately, then, from my experiments, I'm not seeing the unexpected API behavior that you're seeing.
In any case, For problems/errors/omissions with the actual documentation about how Windows APIs work, this feedback mechanism (Send feedback about > This page) takes you to the right place to raise those.
But this particular issue doesn't look like something that I can fix by making a change to the docs. It's really a question, it seems, as to whether the APIs are behaving as specified. I don't myself see any issue with the content itself, based on my experiments (although I understand that you're seeing unexpected behavior). So, if instead you click Send feedback about > This product, then you'll be taken to the Windows developer feedback site (more info about that below). Or, if you believe that this is a product bug, then see the fourth option below, about Microsoft Support.
Here are all of the developer support resources, for completeness.

The Windows developer feedback site is the right place to lobby for a change, a feature, or to make a suggestion.
For community assistance with development questions, there are our forums, and StackOverflow.
The Windows developer support site explains other support options.
The Visual Studio developer community is available, and responsive. The former User Voice feature is now there on the Suggest a feature tab, if you want to lobby for a change, a feature, or to make a suggestion.
For developer issues, or suspected product bugs, there's Microsoft Support; a support engineer will look at the repro and see whether the issue is a Microsoft product bug, or something else. If it's a bug with a Microsoft product then the ticket will be free of charge.

If you receive assistance via any support resource, and you believe that information would help us improve the docs, then please come back and let us know.
Thanks!
Steve

Thank you for your quick response! I feel like it might be the way I use the API. I have posted a question on SO:
https://stackoverflow.com/questions/58124485/brightnessoverride-issupportedchanged-is-not-triggered-when-i-switch-the-monitor

Great! I'll go ahead and close this issue. ~Steve


Update fin version 2.6.3 with .cork-build

Aggie Experts currently uses fin version 2.6.3.  Can we get a new version 2.6.4 that is built in the cork-build-repository.

2.6.3 is now registered with cork-build


Infinite installation in Ubuntu

Bug description
I'm currenly trying to install Superset on my virtual machine ussing docker
sudo docker-compose -f docker-compose-non-dev.yml up
But after
superset_init         | Init Step 4/4 [Complete] -- Loading examples
The proccess just went on cyclus
superset_app          | <IP_ADDRESS> - - [31/Oct/2024:14:19:08 +0000] "GET /health HTTP/1.1" 200 2 "-" "curl/7.88.1"
superset_app          | <IP_ADDRESS> - - [31/Oct/2024:14:19:38 +0000] "GET /health HTTP/1.1" 200 2 "-" "curl/7.88.1"
superset_worker_beat  | [2024-10-31 14:20:00,054: INFO/MainProcess] Scheduler: Sending due task reports.scheduler (reports.scheduler)
superset_worker       | [2024-10-31 14:20:00,057: INFO/MainProcess] Task reports.scheduler[776267e7-e913-420e-a4e1-5b094e23dd65] received
superset_worker       | [2024-10-31 14:20:00,063: INFO/ForkPoolWorker-1] Task reports.scheduler[776267e7-e913-420e-a4e1-5b094e23dd65] succeeded in 0.00419969399990805s: None
superset_app          | <IP_ADDRESS> - - [31/Oct/2024:14:20:08 +0000] "GET /health HTTP/1.1" 200 2 "-" "curl/7.88.1"
superset_app          | <IP_ADDRESS> - - [31/Oct/2024:14:20:38 +0000] "GET /health HTTP/1.1" 200 2 "-" "curl/7.88.1"
superset_worker_beat  | [2024-10-31 14:21:00,012: INFO/MainProcess] Scheduler: Sending due task reports.scheduler (reports.scheduler)
superset_worker       | [2024-10-31 14:21:00,015: INFO/MainProcess] Task reports.scheduler[690d92f0-45e5-4e08-8898-33d49370a158] received
superset_worker       | [2024-10-31 14:21:00,021: INFO/ForkPoolWorker-1] Task reports.scheduler[690d92f0-45e5-4e08-8898-33d49370a158] succeeded in 0.004534726000201772s: None
superset_app          | <IP_ADDRESS> - - [31/Oct/2024:14:21:08 +0000] "GET /health HTTP/1.1" 200 2 "-" "curl/7.88.1"
superset_app          | <IP_ADDRESS> - - [31/Oct/2024:14:21:38 +0000] "GET /health HTTP/1.1" 200 2 "-" "curl/7.88.1"

Screenshots/recordings

Superset version
3.1.3
Python version
3.10
Node version
I don't know
Browser
Firefox
Additional context
No response
Checklist

[x] I have searched Superset docs and Slack and didn't find a solution to my problem.
[x] I have searched the GitHub issue tracker and didn't find a similar bug report.
[ ] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.

I'm not sure if I understand the issue, but looking at the log seems the service is up and available as expected. You should be able to access Superset on port 8088.
@isaacschneuder seem you might be expecting an installation that reports being done or something like that, but docker-compose up simply starts the service and is expected to keep running until you CTRL-C to stop the service.

@mistercrunch Thank you for your answer.  I'm trying to install Superset on my virtual machine. I followed these instuctions:
git clone https://github.com/apache/superset.git
cd superset
git status
sudo docker-compose -f docker-compose-non-dev.yml up
I tried docker compose up, and after this i've got an error from screenshot:

Also, after docker compose start seems like the containers started, but when i try in my browser :8088, nothing is happening.
I thought, maybe I should intall celery and flask through pip, but there are the same errors.

Hello @isaacschneuder from your first screenshot it appears that Superset is running properly. Refer to the docs fore more info https://superset.apache.org/docs/installation/docker-compose/. I am closing this issue for now but feel free to comment if you are still experiencing some problems. Thank you!


Add a library call for S3 header ETAG field checksum testing

* add a hex_string=s3sum(filename_string[,blocksize_int]) call?
    - need to convert s3checksum C source to Go.
    - would be used to compare against an S3 header ETAG field to verify
        either single or multi-part downloads
    - may need to do something to auto-detect block size based on size of first
        block when multi-part formulated and block size not provided as param?
    - would assume a default blocksize of 8192*1024 if not provided, but needed.

s3sum(file[,blocksize]) added. now in test.

Tested against a variety of S3 headers. Added an enforced single part upload calculation for uploads from clients that have not followed multi-part upload recommendations. Testing concluded.


URL of the created page (v4 beta 2)

Describe your problem
When you create a page, the URL of it should be generated by the page’s title. At the moment there is a default nonsense text like no-content-1 or similar. Every time you have to click onto the blue line reading Aus dem Titel der Seite generieren (probably Generate from the title of the page in English). In older Bludit version the URL was generated automatically from the title.
Additionally, it would be fine if German umlauts would be changed automatically, like ü to ue etc., but that is of course not so important for the time being.
Steps to reproduce the problem
Create page, then click on URL ändern (I presume it is Change URL or so in English), and then you’ll see the URL is not generated from the title of the page.
Bludit version
4.0.0 beta 2
PHP version
7.4
PHP logs
—

Implemented in v4.0 rc2, the url is generated from title


Implement support for --test-suffix <suffixes> command line option

Add support for custom test suffixes, as used by the --test-suffix command line option. (https://phpunit.readthedocs.io/en/9.5/textui.html#command-line-options)

Can be closed - this is already implemented by the phpunit.testFileSuffix setting.


rpgp not building anymore

I'm currently getting this error on all ci/local deb building scripts:
Step 12/12 : RUN true     && cd /rpgp/pgp-ffi     && PREFIX=/opt/DeltaChat/rpgp make install
 ---> Running in 76480efd4de7
cd /rpgp/pgp-ffi/../ && RUSTFLAGS="-C codegen-units=1" cargo build --release --features nightly -p pgp_ffi
    Updating crates.io index
error: no matching package named `pgp` found
location searched: /rpgp
perhaps you meant: pgp
required by package `pgp_ffi v0.1.0 (/rpgp/pgp-ffi)`
make: *** [build-stamp] Error 101
Makefile:45: recipe for target 'build-stamp' failed
The command '/bin/sh -c true     && cd /rpgp/pgp-ffi     && PREFIX=/opt/DeltaChat/rpgp make install' returned a non-zero code: 2

What's the correct way to build and install rpgp? We should maybe also put that in the README.

Where are these lines from?


Vapor GUI becomes unreadable after enlarging the window.

User Nicholas Boselli reports on vapor mailing list that VAPOR GUI becomes unreadable after enlarging the window.
"I also encounter bugs when I try to enlarge the vapor window and it glitches out making everything unreadable."

Sam will reach out to user for more info.

The user did not have a screenshot nor a sequence to reproduce it. I'll close this bug report for now.


When the NAME of nodes is hostname, metrics-server cannot obtain node data.

k8s v1.12.0
metrics-server v0.3.1
addon-resizer v1.8.3

E1008 06:51:25.170819 1 manager.go:102] unable to fully collect metrics: [unable to fully scrape metrics from source kubelet_summary:kube-master: unable to fetch metrics from Kubelet kube-master (kube-master): Get http://kube-master:10255/stats/summary/: dial tcp: lookup kube-master on <IP_ADDRESS>:53: no such host, unable to fully scrape metrics from source kubelet_summary:kube-backup: unable to fetch metrics from Kubelet kube-backup (kube-backup): Get http://kube-backup:10255/stats/summary/: dial tcp: lookup kube-backup on <IP_ADDRESS>:53: no such host, unable to fully scrape metrics from source kubelet_summary:kube-worker: unable to fetch metrics from Kubelet kube-worker (kube-worker): Get http://kube-worker:10255/stats/summary/: dial tcp: lookup kube-worker on <IP_ADDRESS>:53: no such host]

When the name of the node is changed to IP, it is normal.

[root@kube-master metrics-server]# kubectl top nodes
NAME CPU(cores) CPU% MEMORY(bytes) MEMORY%
<IP_ADDRESS> 135m 13% 801Mi 42%
<IP_ADDRESS> 113m 11% 697Mi 37%
<IP_ADDRESS> 100m 10% 286Mi 15%
[root@kube-master metrics-server]#

k8s v1.12.0
metrics-server v0.3.1
addon-resizer v1.8.3

#143


--save-exact for npm install isn't converted correctly

Describe the bug
I found this fork through your contribution to Docusaurus. Seeing as the source repository appears to be dead I'm creating the issue here hoping you can fix this.
There is currently a bug with npm install pkg --save-exact getting converted to yarn add pkg-exact instead of yarn add pkg --exact
To Reproduce
Steps to reproduce the behavior:

Pass npm install pkg --save-exact to the npm-to-yarn with second argument set to yarn
Observe result

Expected behavior
Gets converted to yarn add pkg --exact
Package & Environment Details

Environment: Node v18.4.0
Version: 1.1.1

Additional context
N.A.

fixed in npm-to-yarn


[Modules] Updates to the Azure Monitor Private Link Service (AMPLS) Module

Description
Requested upgrade for AMPLS in issue #4035.
Upgraded the API version of the AMPLS resource and added the required properties. Also, added two UDTs to make parameters input for the user easier. This is a breaking change because it introduced new properties that are required in the newer version.
Type of Change

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Update to documentation

Checklist

[x] I'm sure there are no other open Pull Requests for the same update/change
[ ] My corresponding pipelines / checks run clean and green without any errors or warnings
[x] My code follows the style guidelines of this project
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation (readme)
[x] I did format my code

@AlexanderSehr
I have an issue with the "max" test. The latest API version introduces the accessModeSettings property which holds 3 properties: ingestionAccessMode, queryAccessMode and exclusions (optional).
The exclusions property is vague for me on how it works. In the Azure Portal you can choose the specific endpoint using a dropdown menu:

But, in the Bicep IaC it is a bit different because you can set any private endpoint connection name you want. It does not have to exist either and it holds any string value...
Also, what is weird to me is that the connection name is randomly generated:

Because of this randomly generated connection name I am not able to potentially set the exclusion for this private endpoint, because the exclusion is set before the creation of the connection. This might be an issue on the resource side. What do you think?

@AlexanderSehr
I have an issue with the "max" test. The latest API version introduces the accessModeSettings property which holds 3 properties: ingestionAccessMode, queryAccessMode and exclusions (optional).
The exclusions property is vague for me on how it works. In the Azure Portal you can choose the specific endpoint using a dropdown menu:

But, in the Bicep IaC it is a bit different because you can set any private endpoint connection name you want. It does not have to exist either and it holds any string value... See "thisisatest" for example.
Also, what is weird to me is that the connection name is randomly generated:
Because of this randomly generated connection name I am not able to potentially set the exclusion for this private endpoint, because the exclusion is set before the creation of the connection. This might be an issue on the resource side. What do you think?

Great question @johnlokerse, let me look into this a bit more. While doing so I'll also loop in the to-be module owner in AVM, @ahmadabdalla as he may have had experience with this property.

@AlexanderSehr
I have an issue with the "max" test. The latest API version introduces the accessModeSettings property which holds 3 properties: ingestionAccessMode, queryAccessMode and exclusions (optional).
The exclusions property is vague for me on how it works. In the Azure Portal you can choose the specific endpoint using a dropdown menu:

But, in the Bicep IaC it is a bit different because you can set any private endpoint connection name you want. It does not have to exist either and it holds any string value... See "thisisatest" for example.
Also, what is weird to me is that the connection name is randomly generated:
Because of this randomly generated connection name I am not able to potentially set the exclusion for this private endpoint, because the exclusion is set before the creation of the connection. This might be an issue on the resource side. What do you think?

Great question @johnlokerse, let me look into this a bit more. While doing so I'll also loop in the to-be module owner in AVM, @ahmadabdalla as he may have had experience with this property.

@ahmadabdalla Do you have an idea how this works? I have looked into this and I think it's not possible to get the private endpoint connection name (connection-) dynamically or set the name manually. I can get the tests running green with any value.

@ahmadabdalla Do you have an idea how this works? I have looked into this and I think it's not possible to get the private endpoint connection name (connection-) dynamically or set the name manually. I can get the tests running green with any value.

@AlexanderSehr @johnlokerse After further investigation from my end, it appears that this is a chicken and egg situation if we are to configure this from the start via IaC. The privateEndpointConnections child resource type under the insights/privateLinkScopes fails when being setup independently via IaC. This is the resource that gets a random name. And if we try to export an existing template with this child resource being in there, the template fails deployment for this resource type, but still goes in and creates another one with a random name too.
Given that we can still inject a name via the test file and that name shows up in the Portal

Is to me at least is an acceptable outcome on the things we can control, because it seems something we cannot easily deal with unless we start introducing complex logic.
We can however provide context in the exclude description field to become more informative for the end user. E.g. "Exclusions for the current created Private endpoints can only be applied post initial provisioning"

@ahmadabdalla Do you have an idea how this works? I have looked into this and I think it's not possible to get the private endpoint connection name (connection-) dynamically or set the name manually. I can get the tests running green with any value.

@AlexanderSehr @johnlokerse After further investigation from my end, it appears that this is a chicken and egg situation if we are to configure this from the start via IaC. The privateEndpointConnections child resource type under the insights/privateLinkScopes fails when being setup independently via IaC (at least in Bicep). This is the resource that gets a random name. And if we try to export an existing template with this child resource being in there, the template fails deployment for this resource type, but still goes in and creates another one with a random name too.
Given that we can still inject a name via the test file and that name shows up in the Portal

Is to me at least is an acceptable outcome on the things we can control, because it seems something we cannot easily deal with unless we start introducing complex logic.
We can however provide context in the exclude description field to become more informative for the end user. E.g. "Exclusions for the current created Private endpoints can only be applied post initial provisioning"

I agree. I have added a description to the exclusion property.
@AlexanderSehr Going to run the required actions now 😄

🚀

Off we go. Thanks @johnlokerse & @ahmadabdalla . What a beauty of a service 👀

One small recommendation (not relevant for the PR) for you @johnlokerse: Bicep is now at version 0.24.24. I recommend a choco upgrade bicep ;)

Done, thanks! 😄


Create Playlist defunct due to missing input text field

input text field in create new playlist has disappeared since upgrade from iOS 6 to iOS 7

closed by commit ac080f61234219716c4af4169b18924fc484bc04


Add gradient accumulation

Codecov Report

Merging #28 (0429d2f) into master (b97ec34) will decrease coverage by 0.04%.
The diff coverage is 82.75%.

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
- Coverage   91.86%   91.82%   -0.05%
==========================================
  Files          44       44
  Lines        6260     6287      +27
==========================================
+ Hits         5751     5773      +22
- Misses        509      514       +5

Impacted Files
Coverage Δ

eir/train.py
92.52% <82.14%> (-0.92%)
:arrow_down:

eir/setup/schemas.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


UI create states and scenarios

Create states via UI (perhaps with a default value and whether on or off).
Generate a file (or a file view) to copy paste into your app.

We have created an option to add ajax state that are recorded.
But we should also have the option to:

add ajax states directly
add non ajax states
add scenarios directly

Almost done, partially duplicated with https://github.com/outbrain/Leonardo/issues/33


drivers: can: stm32: Cleanup Kconfig enablement

Follow the pattern we have for other peripherals in that if the driver
class (CAN) is enabled than enable the driver for that class
(CAN_STM32).  Also have the STM32 CAN driver depend on being on a STM32
SoC.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

Here's some output from the build log:
[ 22%] Built target subsys__testsuite__ztest
/home/galak/git/zephyr/tests/drivers/can/api/src/main.c:46:8: error: variable 'test_std_msg' has initializer but incomplete type
 struct can_msg test_std_msg = {
        ^~~~~~~
/home/galak/git/zephyr/tests/drivers/can/api/src/main.c:47:3: error: 'struct can_msg' has no member named 'id_type'
  .id_type = CAN_STANDARD_IDENTIFIER,
   ^~~~~~~
/home/galak/git/zephyr/tests/drivers/can/api/src/main.c:47:13: error: excess elements in struct initializer [-Werror]
  .id_type = CAN_STANDARD_IDENTIFIER,
             ^~~~~~~~~~~~~~~~~~~~~~~
/home/galak/git/zephyr/tests/drivers/can/api/src/main.c:47:13: note: (near initialization for 'test_std_msg')
/home/galak/git/zephyr/tests/drivers/can/api/src/main.c:48:3: error: 'struct can_msg' has no member named 'rtr'
  .rtr     = CAN_DATAFRAME,
   ^~~
/home/galak/git/zephyr/tests/drivers/can/api/src/main.c:48:13: error: excess elements in struct initializer [-Werror]
  .rtr     = CAN_DATAFRAME,
             ^~~~~~~~~~~~~
/home/galak/git/zephyr/tests/drivers/can/api/src/main.c:48:13: note: (near initialization for 'test_std_msg')
/home/galak/git/zephyr/tests/drivers/can/api/src/main.c:49:3: error: 'struct can_msg' has no member named 'std_id'
  .std_id  = TEST_CAN_STD_ID,
   ^~~~~~
/home/galak/git/zephyr/tests/drivers/can/api/src/main.c:34:30: error: excess elements in struct initializer [-Werror]
 #define TEST_CAN_STD_ID      0x555
                              ^~~~~
/home/galak/git/zephyr/tests/drivers/can/api/src/main.c:49:13: note: in expansion of macro 'TEST_CAN_STD_ID'
  .std_id  = TEST_CAN_STD_ID,
             ^~~~~~~~~~~~~~~

Codecov Report

Merging #13715 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #13715      +/-   ##
==========================================
- Coverage   52.28%   52.24%   -0.04%
==========================================
  Files         307      307
  Lines       45413    45413
  Branches    10508    10508
==========================================
- Hits        23742    23726      -16
- Misses      16886    16898      +12
- Partials     4785     4789       +4

Impacted Files
Coverage Δ

subsys/testsuite/ztest/include/ztest_assert.h
35% <0%> (-35%)
:arrow_down:

subsys/testsuite/ztest/src/ztest.c
65.28% <0%> (-7.44%)
:arrow_down:

Continue to review full report at Codecov.

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

Change is fine with me but please complete with existing CONFIG_CAN_STM32 symbol clean up in samples/ and tests/

Huh?


[release/1.1] Add missing return statement on pull unpack

Cherry-pick of #2332

The vendor issue that we hit on another pr in master.  I think you have to update vendor locally and revendor for this branch to fix it.

Codecov Report

Merging #2334 into release/1.1 will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           release/1.1    #2334   +/-   ##
============================================
  Coverage        50.07%   50.07%
============================================
  Files               75       75
  Lines             7269     7269
============================================
  Hits              3640     3640
  Misses            2956     2956
  Partials           673      673

Flag
Coverage Δ

#linux
50.07% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM


Fixed bug for 'unsupported parameter for module: path' error

Re-submitting the PR after pull from upstream and fixing merge conflicts
This fixes the bug which gives this error: fatal: [<IP_ADDRESS>]: FAILED! => {"changed": false, "failed": true, "msg": "unsupported parameter for module: path"}

This change would only be required in Ansible 2.2 and earlier; this role requires 2.2 or later: http://docs.ansible.com/ansible/latest/modules/replace_module.html


Build snap in GitHub CI

This change builds and stores the snap on amd64 and offers the snap as a build
artifact for download and testing.
Signed-off-by: Nicolas Bock<EMAIL_ADDRESS>

We would have to cross compile which might not work for all architecture combinations we care for. I meant this change more as a way to run things in CI using the (amd64) hotsos snap. Sort of as a functional test.

The issue we ran into with the shebang not being properly rewritten might have come up here.

Done @hopem


C# Cant Load Custom resources.

Godot version:
3.2.4 beta 6 mono
OS/device including version:
Windows 10 x64
Issue description:
I've created a custom resource, but when loading, got a casting or invalid class name error.

Steps to reproduce:

Create a custom resource:

public class RStatistics : Resource
{

    [Export]
    public float timeAlive;

    [Export]
    public int kills,killStreak,oneHitKills,deaths,id;
}

On a scene, create a script, save and load it:

public class Test : Control
{
    public override void _Ready()
    {
        RStatistics r = new RStatistics();
        r.kills = 100;
        ResourceSaver.Save("test.tres",r);
        var loaded = ResourceLoader.Load("res://test.tres") as RStatistics; // also tried with ResourceLoader.Load<Rstatistics>() same error
        GD.Print(loaded.kills);
    }

}

Minimal reproduction project:
New Game Projects.zip

Hey 👋
This is not really a loading issue. It's the ResourceSave.Save miss handling the serialisation: #38191.


Add Type-Levels to the Includes

Currently, the new type levels are already usable in the LSP. However, they cannot be explicitly specified via includes as seen below.
  include
	TYPE-level
	TYPE-level.numeric-constraints
	TYPE-level.string-constraints

@SundermannC We're about to implement the TYPE-level and show which features need .numeric-constraints or .string-constraints to be included.
However, we don't know which features are contained in these includes. (len(..) function, comparison with strings, etc.)?
Please let us know which functions are reserved for TYPE-levels.
Comments in the following example (or additional constraints, etc) should do it:
namespace type_level

include
	TYPE-level
	TYPE-level.numeric-constraints
	TYPE-level.string-constraints

features
	some_feature
		mandatory
			String feature1 {type_level_value "tempval"}
			String feature
			Integer feature2 {type_level_value "5"}
				alternative
					feature3
					Real feature4
constraints
    feature1 == 'something'
    len(feature1) == len(feature) & feature2 > len(feature1)
    len(feature1) == 2
    len(feature1) == feature2
    feature2 > 6
    floor(feature2) == 5
    ceil(feature2) == 5

Furthermore, what is the type_level_value exactly? This does not seem to work in our current uvls implementation.

TYPE-level.numeric-constraints consists of:

floor(name)
ceil(name)

TYPE-level.string-constraints consists of any constraints that include string operations which should include:

len()
String comparsions e.g. feature1 == 'something'

The type_level_values are ordinary feature attributes that are used for conversion between levels in the uvl-parser2.0. We do not need explicit handling of them in the LSP for now.


I don't understand how to use this package.

Im developing a proffesional network and want the users to connect to each other. I found this package but how to i use it especcialy when you already have the user model which stores all the users. hoa to get sender id and receipt it.

its very simple if you go through the documentation thoroughly. Just give it a few reads. For any confusion, the sender_id and recipient_id will always be the User's ID, just changes depending who is the user sending the request and who is the user receiving the request.
For Ex: we have a table with 5 users with id's (1,2,3,4,5, respectively) -
So if the User with ID wants to send a friend to a User with ID 2, this is how you write it -
$user = Auth::user(); [logged-in user is the sender]
$recipient = User::where('id',2)->get(); [the user who you want to send friend request to]
$user->befriend($recipient); [here the system will automatically take the id of both the users and assign the $user->id as sender id and $recipient->id as recipient_id in the pivot table.

Hope this clears up.
-CR

I'm not good on writing english hope u'll understand me.
Thank you very much this is an eye open to me. I looked at the test file but i can't figure out how to pass the recipient id on where.
the test/FriendshipsTest.php file is like this if you want to send the friend requset
public function user_can_send_a_friend_request()
{
$sender    = createUser();
$recipient = createUser();
$sender->befriend($recipient);
$this->assertCount(1, $recipient->getFriendRequests());
}
So mine should i change be like this or there is another way of doing this
public function user_can_send_a_friend_request(Request $request)
{
$sender    = Auth::user();
$recipient = User::where('id',$request->recipeint_id)->get();;
$sender->befriend($recipient);
$this->assertCount(1, $recipient->getFriendRequests());
}
where the $request->recipient_id is submited from the form like this
{{ Form::open(['method' => 'POST','route' => ['friendships.sent']] )}}
@csrf

 Connect
{{ Form::close() }}

Hey, for future when writing/pasting code use the "insert code" which looks like this <>
As for your question, Tests are not designed for you, its only useful for those who knows the inner working and want to make sure if they make any changes to the code, they wont break anything. Looking at the tests without understanding the functions will not make any sense for you or anyone.
Just look at the instructions in the readme file as they are all what u will need when simply using this package.
As for your confusion, you can use a form, display the user a lists of members and who they want to be friends with, they can select one user and submit. Now you do not pass the recipient_id to the befriend function, you need to pass the recipient's user model,  i will give u a little snippet to explain better -
public function sendRequest(Request $request)
{
$sendTo = $request->recipient_id;
$recipient = User::find($sendTo);  // Loading the User Model with the id received from the form
$user = Auth::user(); // logged-in user, aka the sender
$user->befriend($recipient);
}

The above form snippet you posted wont work because the form is not sending anything, do something along the lines of this
{{ Form::open(['method' => 'POST','route' => ['friendships.sent']] )}}
@csrf
<input type"hidden" name="recipient_id" value="2"> [or whatever is the ID of the user you want to be friends with]
<button class="btn btn-success" type="submit">Connect</button>
{{ Form::close() }}

Hope this helps.

Thank you very much for the support. Let me work on it if i face any problem i'll ask for support.

i want to get the list of all the users except those we are connected as friends andi have sent the request. how can i do this??.

I don't think such a function is build into the system. If you want to display the user a list with all of your other users (minus friends) for them to add, then that is a good source for spamming. It is advised that you befriend a user if and only if you know something about them (like email, username, full name, etc).
Can you tell me your use case for this? or what you are trying to achieve?

spamming.

This is what i want to achieve in a proper way the current one is working but not a good approach
My Controller file
`class HomeController extends Controller
{
public function __construct()
{
$this->middleware(['auth', 'verified']);
}
//Show the application dashboard.
    public function index()
    {
        $users = User::select('name', 'id')->where([['id','!=',Auth::user()->id],['email_verified_at','!=',NULL]])->get();

        return view('home',compact('users'));
    }

}`
My view file.
` @foreach($users as $user)
@php
$auth_user = Auth::user();
$recipient = \App\User::findOrFail($user->id);
@endphp
@if(($auth_user->hasSentFriendRequestTo($recipient) ||
$auth_user->hasFriendRequestFrom($recipient) ||
$auth_user->isFriendWith($recipient) ||
$auth_user->hasBlocked($recipient) ||
$auth_user->isBlockedBy($recipient) ) == 1)
@else

@if(isset($user->user_detail->profile_picture))

@else

@endif

{{$user->name}}

@if(isset($user->user_detail->headline))
{{$user->user_detail->headline}}
@else
                                                @endif

                                            {{ Form::open(['method' => 'POST','route' => ['friendships.sent']] )}}
                                            @csrf
                                            <input type="hidden" value="{{$user->id}}" name="recipient_id"/>
                                            <button class="btn btn-outline-primary btn-sm "> Connect</button>
                                            {{ Form::close() }}

                                @endif
                            @endforeach`

I want the if statement in a View file to do it in the Controller

This is what i want to achieve in a proper way the current one is working but not a good approach.
My Controller file
class HomeController extends Controller { public function __construct() { $this->middleware(['auth', 'verified']);
My view file.
`@foreach($users as $user)
@php
$auth_user = Auth::user();
$recipient = \App\User::findOrFail($user->id);
@endphp
@if(($auth_user->hasSentFriendRequestTo($recipient) ||
$auth_user->hasFriendRequestFrom($recipient) ||
$auth_user->isFriendWith($recipient) ||
$auth_user->hasBlocked($recipient) ||
$auth_user->isBlockedBy($recipient) ) == 1)
@else

{{$user->name}}

                          {{ Form::open(['method' => 'POST','route' => ['friendships.sent']] )}}
                           @csrf

 Connect
                         {{ Form::close() }}

@endif
@endforeach`
I want the if statement in a View file to do it in the Controller

Okay, I get it, you want something like Facebook/Twitter has for suggesting users. Well I will have to get some free time to be able to create a PR that would do this out of the box with eager loading, but for now you are on the right path.
The STATUS column in the friendships table is used for describing the TYPE of Friendship. When you Deny, Block or Accept, all it does is change the Status integer (PENDING = 0, ACCEPTED = 1, DENIED = 2, BLOCKED = 3), so there is no need to run multiple checks, all you need to check is if that relation exists in the friendships table.
In your controller -
-----------------------------------
$users = User::all()->except(Auth::id()); // using except here to eliminate the logged in user from the collection

return view('yourview')->with(compact('users');
-----------------------------------
In your view -
-----------------------------------
@foreach($users as $user)
 @if(! Auth::user()->getFriendship($user))
  {{ $user->name }}
 @endif
@endforeach

the getFriendship() method returns the pivot relation between the Auth::user() and the $user from the Users table. So we call this method for each user in our users table and then display the user details only when the relation does not (!) exists. Check if this works for you.
-TT

Glad that it worked. Close this issue if you are done.


Server crash

Spark link
https://spark.lucko.me/syaMmWzAtt
Stack trace
https://paste.gg/p/anonymous/faa1e2ab5c574969a0082a17e4ae302a
Actions to reproduce (if known)
Szerver crash, but there is not crash report.  The server is hosting in apex.
Purpur version
Current: git-Purpur-1835 (MC: 1.19.2)*
Previous: git-Paper-256 (MC: 1.19.2)

You are 23 version(s) behind
Download the new version at: https://purpurmc.org/downloads

Agreements

[X] I am running the latest version of Purpur available from https://purpurmc.org/downloads.
[X] I have searched for and ensured there isn't already an open issue regarding this.
[X] My version of Minecraft is supported by Purpur.

Other
No response

Not gonna comment on the issue itself, but you did include quite a few ips and no stacktrace with that log.


docs: fix some broken links

Hello CF Team!
I stumbled upon #10339 which points out some of the links that are broken. I tried to find and fix ~15 links as suggested in the issue, the references are either self-explanatory or mention the commit that changed the link.
I am unsure about 9a2a17a1564d0244e999470faf0c23cea3e7402f though so would be nice if the reviewer could take a look. 🙂

Please also check 273bb95 more closely, I might have missed the correct destination for this link

Thanks for your help @WiGeeky


Provide CLI to setup sidebase with different packages pre-installed

Describe the feature
Provide a setup cli to setup sidebase e.g., asking questions:

what css component libraries do you want to use?
do you want to have a backend?
vite or webpack?
...

Example of where we once build a cli: https://github.com/sidestream-tech/unified-auctions-ui/pull/402
some libraries to use:

prompts

Additional information
No response

closed as duplicate of #13


Arithmetic operations with reference semantics

In the implementation of complex, the Add, Sub, etc. for references are forwarded via clone() for values. This is very bad for types which are expensive to copy. In my example, I use Complex<Ratio<BigInt>>.
Can the code be refactored in such a way, that no clone() is necessary? I guess the different arithmetic operations have to be coded out and not forwarded.
I could try that, if there is no objection against that ...

The main problem is that we added all of those implementations with just T: Clone + Num, and it's a breaking change to add additional constraints.  To implement ops directly on references, we would probably want to use NumRef, NumAssignRef, and RefNum.

In my example, I use Complex<Ratio<BigInt>>.

Note, Ratio also implements reference ops by cloning, which means that changing Complex alone wouldn't really help you.
I started looking at this for 0.3 (#70), but I'm skeptical. It requires a lot of code duplication on our part to directly implement each op with/without clones, and the additional T: NumRef + ... requirements end up propagating widely. The latter will affect all generic users too, sadly.


Twitter And Pin it buttons are not apearing

I m using this module to my ionic app here google+ and facebook ion are appearing but Twitter and Pin it button are not appearing.

Twitter is appearing in Chrome but not FireFox and Edge. What's funny is the DOM contains the HTML and yet the browser doesn't render.


Can I send email like this?

First of all, thank you sir. The code itself works well!
But I wanna ask you one question-
Could I put an html body, upload an image file with st.file_uploader(), then attach it to that body and send it?
Maybe it is more related to the function of st.file_uploader() but I have no idea.

I haven't experimented with it that way yet. I will try and let you know.


Are ENT functions school specific?

A few of our ENT functions (ac_grenoble, atrium_sud for example) have a specific school address in some of their requests. Is this ok? Do they work with other schools?

I don't really understand, why wouldn't they work well with others?

@bain3 no it will not work with other schools. I had already noticed the problem but forgot to report it.

I don't really understand, why wouldn't they work well with others?

Because sometimes they have a specific hard-coded "service" (like atrium sud: ?service=https:%2F%2F0060013G.index-education.net%2Fpronote%2F) which points to a specific school. I guess we need to pass the school url to the function and make this variable?

@bain3 Not sure but maybe it can work without this url param. Because I would be surprised if the site gives the cookies according to the param.

We'd need to test it. Honestly I wouldn't be surprised... All of this authentication stuff is super sketchy.

I use the atrium-sud ENT with another Pronote URL than the one hard-coded here and it works without any problem.


narrow: Save blue box position and restore on navigation.

This saves the blue box position as state on the location in brower history, with history.replaceState.
The position is restored when a narrow is activated.
Fixes #20066.

I'm planning to wait for code review to verify this method before writing tests.

This seems to work okay on testing. When I try racing through views really quickly, it seems to hang onto the history as expected.

I think save_narrow_state might better belong in the browser_history module; and if you do that, I expect that you won't be pressured to write tests for it.
(I think this is probably the type of code where node tests are quite low value compared to manual testing)

updates:

Moved the code to browser_history.js
Switched to throttling instead of debounce, as this ensures we call the function in a timely manner
Added another call to update_selection which is used when manually setting the narrow position, and used flush() to ensure this call happens without any delay

I think save_narrow_state might better belong in the browser_history module; and if you do that, I expect that you won't be pressured to write tests for it.

turns out browser_history also wants code coverage -- should I exclude this file or would you recommend something else?

Yeah that sounds reasonable; we've already achieved the goal of testing the existing logic, and this new logic is probably not going to get a ton out of node tests.

Okay that creates an import cycle (which I now recall dealing with earlier) -- so I think I'll just put it in its own file?

@timabbott okay got it passing CI! back to you :)

Posted one comment, otherwise lgtm. Thanks for all the work on this!

Pushed with that change!

Merged, thanks @evykassirer!


Is this active?

Hi Sarah,
I got here through the original project. It looks like it hasn't been updated in 4 years. Your project hasn't seen any updates in a while, but I don't know if it's been abandoned or it's just working fine.

Fell off the GitHub train while ramping up on a new job, sorry for the extremely delayed response.
I can confirm that hartjet/amazon-advertising-api-python is still active, it’s being maintained by a former coworker so it’s most likely to be similar to this version.


loss/losses are omitted from get_config/serialization

opening a new issue since https://github.com/keras-team/keras/issues/19820 is closed.
Based on this - https://keras.io/api/models/model_saving_apis/ I assume I should be able to save a partially trained model using model.save(), then load it using load_model(), and resume training. If my assumption is true, then there are more issues like https://github.com/keras-team/keras/issues/19820
For e.g., for a trained model with a (non-custom) Loss, model.get_config().get("losses") and model.get_config.get("loss") both return None. Whereas model.losses and model.loss work as expected. Consequently, those properties don't roundtrip via save() and load_model().

Is there any reason that you're not using ModelCheckpoint callback for saving and loading later to continue training?
I believe our Saving API is mainly designed for saving a finalized model and doesn't have to keep training information.
cc @nkovela1

I'm closing this issue for now but feel free to let us know if https://github.com/keras-team/keras/issues/19884#issuecomment-2197699458 is not clear or doesn't address your concern.


“Current IE status” should be “Current Microsoft Edge status”

…as for the most part new features are under consideration / in development for MS Edge, not IE. IE will mostly only get security and regression fixes.

Thanks, fixed!


Refactor sanitize_for_vue/sanitize_for_rails

I believe this was addressed in #781. Was there anything else you were thinking of @maebeale ?

Nope! :) Thanks, @exbinary !


Share players video feed directly to the layouts with webRTC

The layouts currently show the twitch avatar in the field.
But for for competition, one might want to see the players video feed. nestrischamps should be about easy setup. a game host should not have to cut player streams, or have to start a side video call on skype or hangout or zoom and then screen cap the video call player feed in OBS.
Proposal:

The producer window and the rendering layouts can be webRTC peers
When any window starts, it registers itself as a peer and gets a peer ID and send that to the nestrischamps server
When admin selects the player, the server instructs the layout it will be expecting a call from a given peer, and instructs the producer window to initiate a call to the rendering layout and submit its video feed

Complexities:

Handle call drops, reconnection, admin changing players, etc.
Producer page must allow player to select which webcam to pick
How to integrate the player feed in the layouts? small areas? Full blown? etc.

Optional Bonus:

The game frames themselves could be sent peer to peer (Nice for Say, Singapore players, whose game frame do a full roud trip to the US server even though both players and game host are in Singapore)

Video feed is now live with the layout ctm_video
However, it is an early POC:

There's no crop control, or privacy control
No webcam selector

Game frames will likely never be sent peer to peer, since we want the server to track the games.

Closing this as done. Improvements to the video feed have their own issues.


Flutter SDK Tools is marked as obsolete in android studio.

Steps to Reproduce

Install Android Studio
run flutter doctor

Logs
$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, v1.12.13+hotfix.8, on Linux, locale en_US.UTF-8)
[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    ✗ Android license status unknown.
      Try re-installing or updating your Android SDK Manager.
      See https://developer.android.com/studio/#downloads or visit https://flutter.dev/setup/#android-setup for detailed instructions.
[✓] Android Studio (version 3.6)
[!] Connected device
    ! No devices available

! Doctor found issues in 2 categories.

$ flutter doctor --android-licenses
Android sdkmanager tool not found (/home/pearman/Android/Sdk/tools/bin/sdkmanager).
Try re-installing or updating your Android SDK,
visit https://flutter.dev/setup/#android-setup for detailed instructions.

The reason for this is that Flutter SDK tools are not installed by default and they are not even shown in the SDK install manager in Android studio (It hides obsolete packages by default.)

I encountered the same issue while installing flutter. I'm still looking for a solution

Duplicate of #51712

Thanks a lot. I checked my Android Studio, and installed Android SDK Tools. The issues fixed.

I've done this and flutter still complains about sdkmanager being outdated, if I try to update it, I get this:
Exception in thread "main" java.lang.NoClassDefFoundError: javax/xml/bind/annotation/XmlSchema 	at com.android.repository.api.SchemaModule$SchemaModuleVersion.<init>(SchemaModule.java:156) 	at com.android.repository.api.SchemaModule.<init>(SchemaModule.java:75) 	at com.android.sdklib.repository.AndroidSdkHandler.<clinit>(AndroidSdkHandler.java:81) 	at com.android.sdklib.tool.sdkmanager.SdkManagerCli.main(SdkManagerCli.java:73) 	at com.android.sdklib.tool.sdkmanager.SdkManagerCli.main(SdkManagerCli.java:48) Caused by: java.lang.ClassNotFoundException: javax.xml.bind.annotation.XmlSchema 	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:602) 	at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178) 	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521) 	... 5 more
I can't seem to solve this (all I did was reinstalling flutter and the Android sdk as I can't find anyone else wit this problem).

Thanks a lot. I checked my Android Studio, and installed Android SDK Tools. The issues fixed.

but I can‘t find Android SDK Tools now,what's your Android studio Version？mine is 3.6.1

Thanks a lot. I checked my Android Studio, and installed Android SDK Tools. The issues fixed.

but I can‘t find Android SDK Tools now,what's your Android studio Version？mine is 3.6.1

I download a zip from https://androidsdkmanager.azurewebsites.net/SDKTools,and unzip it.now I resolve this Problem.

Thanks a lot. I checked my Android Studio, and installed Android SDK Tools. The issues fixed.

but I can‘t find Android SDK Tools now,what's your Android studio Version？mine is 3.6.1

Mine is 3.6.1,too. Try check Hide Obsolete Packages button.

https://robbinespu.gitlab.io/blog/2020/03/03/flutter-issue-fixed-android-license-status-unknown-on-windows/

It's not there for me.

and this is the flutter-doctor:
`[√] Flutter (Channel stable, v1.12.13+hotfix.9, on Microsoft Windows [Version 10.0.18363.778], locale en-US)
    • Flutter version 1.12.13+hotfix.9 at D:\FlutterSDK\flutter
    • Framework revision f139b11009 (4 weeks ago), 2020-03-30 13:57:30 -0700
    • Engine revision af51afceb8
    • Dart version 2.7.2

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at C:\Users\mojta\AppData\Local\Android\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • Java binary at: D:\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)
    X Android license status unknown.
      Try re-installing or updating your Android SDK Manager.
      See https://developer.android.com/studio/#downloads or visit https://flutter.dev/setup/#android-setup for detailed instructions.

[√] Android Studio (version 3.6)
    • Android Studio at D:\Android Studio
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)

[!] Connected device
    ! No devices available

! Doctor found issues in 2 categories.`

hey, try this:
First

Go to Android SDK Folder (in ur case: C:\Users\mojta\AppData\Local\Android\sdk)
Delete tools folder
Re-open Android Studio and navigate to SDK Tools tab
check Hide Obsolete Packages button
Install Android SDK Tools (Obsolete)
Install Android SDK Command-line Tools (latest)

Second

Go to Android SDK Folder (again)
Open the bin folder inside tools in shell
Type: sdkmanager.bat --licenses and accept all
Try now: flutter doctor or flutter doctor --android-licenses

Same with me android tools obselete not found

hey, try this:
First

Go to Android SDK Folder (in ur case: C:\Users\mojta\AppData\Local\Android\sdk)
Delete tools folder
Re-open Android Studio and navigate to SDK Tools tab
check Hide Obsolete Packages button
Install Android SDK Tools (Obsolete)
Install Android SDK Command-line Tools (latest)

Second

Go to Android SDK Folder (again)
Open the bin folder inside tools in shell
Type: sdkmanager.bat --licenses and accept all
Try now: flutter doctor or flutter doctor --android-licenses

this worked man tysm

hey, try this:
First

Go to Android SDK Folder (in ur case: C:\Users\mojta\AppData\Local\Android\sdk)
Delete tools folder
Re-open Android Studio and navigate to SDK Tools tab
check Hide Obsolete Packages button
Install Android SDK Tools (Obsolete)
Install Android SDK Command-line Tools (latest)

Second

Go to Android SDK Folder (again)
Open the bin folder inside tools in shell
Type: sdkmanager.bat --licenses and accept all
Try now: flutter doctor or flutter doctor --android-licenses

It worked Thanks

Finally got time to try this.
Thanks man it worked
On Fri, Jul 17, 2020 at 6:11 PM Sowndharya1143<EMAIL_ADDRESS>wrote:

hey, try this:
First

Go to Android SDK Folder (in ur case:
C:\Users\mojta\AppData\Local\Android\sdk)
Delete tools folder
Re-open Android Studio and navigate to SDK Tools tab
check Hide Obsolete Packages button
Install Android SDK Tools (Obsolete)
Install Android SDK Command-line Tools (latest)

Second

Go to Android SDK Folder (again)
Open the bin folder inside tools in shell
Type: sdkmanager.bat --licenses and accept all
Try now: flutter doctor or flutter doctor --android-licenses

It worked Thanks
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/51793#issuecomment-660084996,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMUCS54Y2ZXPPED2YLKKIMDR4BBG5ANCNFSM4K73UJKQ
.

problem fix here : https://youtu.be/LEvZUI95Zv4


submitFailed documentation format error

Location: http://erikras.github.io/redux-form/#/api/props?_k=9t2op3
API: submitFailed
Problem: Obviously a backtick is missed after the first false.

Thanks!


How to create an object in another location

Hello, I'm very confused about how to render using nerf_volume_renderer and have been struggling for days...
For example, I want to create the object centered around [-0.5,0.5,0] rather than [0,0,0] point. RandomCameraIterableDataset defined in data/uncond.py creates random cameras surrounding the [0,0,0] point. From what I understand, nerf_volume_renderer shoots rays from rays_o (camera position) in the direction of rays_d and accumulates them.
Therefore, I thought that in order to create an object in [-0.5,0.5,0] rather than [0,0,0], I would just use rays_d without changing it and add [-0.5,0.5,0] to rays_o. However, this only works properly when the camera is in the front view and does not work with random cameras. Only when rendering with front view, the object is rendered in the center of the image, which is what I want.
Below is an example of nerf optimization using DreamCraft3D. In the picture above, an object was created at the [0,0,0] position without changing ray_o, and the object is in the center of the image even when rendered in random view. In the picture below, to create the object at the [-0.5, 0.5, 0] location, [-0.5, 0.5, 0] was added to the components of rays_o and an object was created. In the front view, the object is rendered in the center of the image without a problem, but in the random view, the object is not rendered in the center of the image.

I suspected there was a problem with the direction of the ray, so I tested the code below, but the center point was [-0.5, 0.5, 0], so there seemed to be no problem.....
# move ray origin [-0.5, 0.5, 0]
rays_o = batch["rays_o"].clone()
rays_o[:,:,:,0] += dist_x # -0.5
rays_o[:,:,:,1] += dist_y # 0.5
rays_o[:,:,:,2] += dist_z # 0
batch["rays_o"] = rays_o

# view center point of world space
rays_d = batch["rays_d"].clone()
b, w, h, c = rays_d.shape #shape 1, 128, 128, 3
ww, hh = w//2, h//2
print(ww, hh, rays_d.shape) # center of image 64, 64
direction2origin = rays_d[:,ww, hh, :] # ray direction of center point
print("object_origin", rays_o[:, ww, hh, :] + direction2origin * batch["camera_distances"].item()) # center point of world space [-0.5, 0.5, 0]

My question is, do I need to modify something other than rays_o? I tried modifying things like c2w, camera_position, and mvp_matrix, but it didn't seem to affect rendering result. nerf_volume_rendering appears to only use rays_o and rays_d. Are there other factors to consider?
Also, since rays_d is understood as the direction of rays defined in world space, I thought it could be used as is when moving the position of an object. Isn't that correct?

The issue has been partially resolved. The camera parameter (rays_o) was accurate.
The core problem stemmed from blob_magic3d, which initializes density around the coordinates [0,0,0]. When using SDS Loss with Stable Diffusion and similar methods, a suitable gradient is generated even if the object is not centered within the camera's view. This means the process can function to an extent, despite the initial density not being positioned at the desired location. Ultimately, it was necessary to generate the initial density at the specific location I required, such as [-0.5,0.5,0].
To address this, I modified the get_activated_density function in implicit_volume.py as follows.
#density_bias = (self.cfg.density_blob_scale  * (1 - torch.sqrt(( (points)**2).sum(dim=-1)) / self.cfg.density_blob_std)[..., None]
density_bias = (self.cfg.density_blob_scale  * (1 - torch.sqrt(( (points - center_point)**2).sum(dim=-1)) / self.cfg.density_blob_std)[..., None]


Database Folder not Expanding in Azure data studio on mac

I have ADS on mac
I want to connect to server but I can't
Everything is ok
user pass and is created db on server
when i want to connect server after set user pass
I see all databases on server and can choose my db

.48.57 AM.png
but when  I connect to server I see green button
I see anther database name on server
but I can't see my database and its related files
why ?

Azure Data Studio Version:

Steps to Reproduce:

Duplicate of https://github.com/microsoft/azuredatastudio/issues/12992

Duplicate of https://github.com/microsoft/azuredatastudio/issues/12992


Cannot save file when starting character is // in first line

Issue Type: Bug

Ctrl+N to create a new file
Write "// some text" (without quotes)
Ctrl+S to save
It will say "Failed to save '// some text': [UriError]..."

This happens because vscode tries to take the default file name as the first line in the file (in this case slashes //). But I guess that is not valid file name.
VS Code version: Code 1.42.1 (c47d83b293181d9be64f27ff093689e8e7aed054, 2020-02-11T14:45:59.656Z)
OS version: Windows_NT x64 10.0.18362

Duplicate issue as #90685, #90433, #90790. Closing the issue.


Add xmldom dependency to FOAM

Ref

https://nanopay.atlassian.net/browse/NP-7192

Changes

Add xmldom dependency to FOAM

Should package-lock.json be included?

Yes, committing package-lock.json allows others to have the same dependency tree when npm install.


Do not remove binding alias in function declarations

Revert change that removes binding aliases from declaration files. Doing this correctly implies doing a lot more work than previously anticipated.
Fixes #56991
Fixes #56992

This also partially reverts #41044 as it removes all removal; I was actually expecting this to have caused d.ts errors because #41044 also introduced an error for unused renamings... but that didn't happen?

All of the examples that are broken feel like they should be fixable; I know some of this I've tackled in typeToTypeNode, but the same strategies probably don't work in declaration emit. I'm fine reverting this so it doesn't go out incorrectly and we can try and tackle it another day.

The problems are not unsolvable for sure. Even in declaration emit I think we ca solve it. But the problem is that it will introduce a lot of perf cost. Seems like it's not worth the complexity. (I belatedly arrived at the same conclusion as @andrewbranch 😅)

This also partially reverts #41044 as it removes all removal; I was actually expecting this to have caused d.ts errors because #41044 also introduced an error for unused renamings... but that didn't happen?

The error for unused renames is still there. I did not remove it. It only seems to pop up on some kinds of declarations for example function types let o: ({n : a}: any) => void;

After checking the above and noticing https://github.com/microsoft/TypeScript/pull/41044#issuecomment-1146271768, I'm now confident that this PR is working correctly.


Api items/wishlist

Separate in-stock items from wishlist items.
something like:
items/... for in stock items
items/wishlist/... for wishlist items

items routes will be divided into three calls:

/items/pending
/items/wishlist
/items/stock


[FLINK-33089] Drop support for Flink 1.13 and 1.14 and clean up related codepaths

What is the purpose of the change
Based on community agreement we drop support for 1.13/1.14 to allow the removal of specific (deprecated) codepaths and reduce CI time overall.
With this we introduce an explicit check and short-circuit exit on unsupported versions.
Brief change log

Remove 1.13/1.14 specific codepaths and tests
Short-circuit the controller loop on unsupported versions and trigger user event
New unit test

Verifying this change
New unit tests added, manual verification
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changes to the CustomResourceDescriptors: yes
Core observer or reconciler logic that is regularly executed: yes

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? [TODO]

cc @mbalassi @tweise @mxm
I went with a more aggressive approach in the pr to disable reconciliation of unsupported version instead of potentially causing silent problems / exceptions in other places.
If you feel that this is too much please let me know :)

I will wait until the 1.18.0 release is finalised for Flink before merging as I also included the respective updates from snapshot to 1.18 images for CI


Lesson not loading

Describe your problem and - if possible - how to reproduce it
Click to open task // OR // Progress from the previous task - opens page and nav bar appears with none of the lesson details showing. Profile card in corner loading constantly.
Console:
Bundle loading error true
loader.js:136 Loading the JSON for /responsive-web-design/applied-visual-design/create-a-more-complex-shape-using-css-and-html failed
Add a Link to the page with the problem
https://learn.freecodecamp.org/responsive-web-design/applied-visual-design/create-a-more-complex-shape-using-css-and-html/
Tell us about your browser and operating system

Browser Name: Google Chrome
Browser Version: Version 67.0.3396.99 (Official Build) (64-bit)
Operating System: Macbook Pro Running OS V.10.13.5

If possible, add a screenshot here

Hi there! 👋
Thanks for reporting, this issue. Can you please share a screenshot of the console tab on your browser's DevTools. On a macOS you can open it with  Cmd +  Option +  I and then click on the console Tab.

Hi, Issue is been resolved as I checked it this morning.
Thanks for the speedy responses all ends!!


BlockSynchronizer finality signature fetch optimization

When the BlockSynchronizer asks peers for finality signatures it should only ask for signatures it does not already have.

The BlockSynchronizer is already only asking for missing signatures.


[BUG] Wallet fails to reconnect after disconnection

Describe the bug
On the demo page (https://docs.family.co/connectkit/try-it-out), cannot reconnect to the page with a WalletConnect wallet after a successful disconnection.
To reproduce

Connect a WalletConnect wallet (i.e. Rainbow) to the demo page at https://docs.family.co/connectkit/try-it-out
Disconnect the wallet
Attempt to reconnect the wallet again, which fails. The QR code is read by the WalletConnect wallet, but is not able to complete the connection to the demo page. The only way to reconnect is to hard refresh the demo page.

Expected behavior
Successful reconnection to the demo page after disconnection
Environment details
I'm using the latest version of the Rainbow app on iOS on the demo page. In my own development environment Coinbase Wallet also fails the reconnection exercise. Metamask (injected wallet) reconnects fine.
Additional context
Using ConnectKit 1.7.2

nice catch @ddickson, there's a fix in #363 that will be released soon

This feature is now available in ConnectKit 1.7.3 🫡


Buttons Styling

Globally,
Regular links should be styled as links and not buttons
Bolded links should be the primary orange buttons
Italics links can be a secondary button (not seen any so far on the site)
Example pages

https://main--sling--aemsites.aem.page/whatson/entertainment/holiday/christmas-movies-shows
This should not be an orange button

compare to https://www.sling.com/whatson/entertainment/holiday/christmas-movies-shows

https://main--sling--aemsites.aem.page/drafts/amol/homepage has mix of text and orange buttons. compare with https://sling.com home page.

Don't break these
Tags, not buttons:
https://www.sling.com/whatson/entertainment/holiday

Examples of the secondary button (white instead of orange)


General grid

Add support for transform between general grid #8
Add demo script
Add readme

Codecov Report

Merging #10 (2d74a17) into main (16f2b78) will increase coverage by 18.58%.
The diff coverage is 19.56%.

@@             Coverage Diff             @@
##             main      #10       +/-   ##
===========================================
+ Coverage   25.15%   43.74%   +18.58%
===========================================
  Files          10       10
  Lines         648      679       +31
===========================================
+ Hits          163      297      +134
+ Misses        485      382      -103

Impacted Files
Coverage Δ

src/discrete/builder.jl
0.00% <0.00%> (ø)

src/discrete/kernel.jl
0.00% <0.00%> (ø)

src/functional/builder.jl
0.00% <ø> (ø)

src/operation.jl
100.00% <ø> (ø)

src/sample/sample.jl
96.61% <ø> (ø)

src/spectral.jl
80.67% <ø> (+0.47%)
:arrow_up:

src/utility/chebyshev.jl
0.00% <ø> (ø)

src/dlr.jl
63.38% <63.38%> (ø)

... and 1 more

Continue to review full report at Codecov.

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


Add version to relay payload

This adds version to /chains/:chainId/relay endpoint for specifying the version of the Safe relaying.
Test coverage for all existing Safe versions and relevant libraries/factories has also been included.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.01%) to 93.92%

Files with Coverage Reduction
New Missed Lines
%

src/routes/transactions/mappers/common/transaction-info.mapper.ts
1
78.17%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
6147

Relevant Lines:
6336

💛  - Coveralls


Create MERS fauxalysis dataset

Task Overview

[x] Download and align MERS structures from the PDB
[x] Create design units for every MERS structure (Structure_Source) and fragalysis compound (Compound_ID)
[ ] Run docking
[ ] Analyze docking results
[ ] Pick 1 best structure for each Compound_ID and provide a Fragalysis-like directory to hand off to medicinal chemists

We're currently in a design-make-test cycle for docking, analyzing, and picking the best structures but we have most of the code set up to do this!


python311Packages.msgspec: 0.18.3 -> 0.18.4

Diff: https://github.com/jcrist/msgspec/compare/refs/tags/0.18.3...0.18.4
Changelog: https://github.com/jcrist/msgspec/releases/tag/0.18.4
Description of changes

Things done

Built on platform(s)

[ ] 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
[ ] 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.

Result of nixpkgs-review pr 259710 run on x86_64-linux 1

6 packages built:

gallia
gallia.dist
python310Packages.msgspec
python310Packages.msgspec.dist
python311Packages.msgspec
python311Packages.msgspec.dist


Webpack inserts line break after semicolon in quoted string, breaks Google Font import with multiple font weights

Bug report

What is the current behavior?
Webpack inserts a line break after semi-colon in quoted string
If the current behavior is a bug, please provide the steps to reproduce.

Select multiple font weights for a Google font and copy the supplied @import snippet
Paste that code into a SCSS file, ex:

@import url('https://fonts.googleapis.com/css2?family=Montserrat:wght@400;600&display=swap');

Check Webpack's compiled CSS, ex:

@import url(https://fonts.googleapis.com/css2?family=Montserrat:wght@400;

600&display=swap);

What is the expected behavior?
Webpack should ignore semicolons in quoted strings

Other relevant information:
webpack version:  4.42.0
Node.js version: 10.16.0
Operating System: MacOS 10.15.4
Additional tools:

Thanks for your report. It would be great if you reduce your issue to a small reproducible example. Best put this example into a github repository together with instructions how to get to the problem.

Could you try to isolate the issue to figure out if this is related to sass-loader or css-loader or mini-css-extract-plugin?

@c-tanner You should be able to get around this by doing something like:
@import url('https://fonts.googleapis.com/css?family=Montserrat:400,600&display=swap');

No problems on css-loader/sass-loader side, maybe bug in a plugin for postcss-loader

Using latest laravel mix and plugin versions and stumbled upon same issue, would like to als oadd that changing syntax from:
@import url('https://fonts.googleapis.com/css2?family=Poppins:wght@400;500;600;700&family=Roboto:ital,wght@0,300;0,400;0,500;0,700;1,300;1,400;1,500;1,700&display=swap');
to:
@import url('https://fonts.googleapis.com/css?family=Poppins:300,400,500,700|Roboto:300,400,500,700,300i,400i,500i,700i&display=swap');
compiles css correctly and fonts are loaded correctly.

Answer here https://github.com/webpack/webpack/issues/10873#issuecomment-632024988, you you still have a problem, please open a new issue in css-loader with reproducible test repo, I will show where the error is


Fix function names in comments

Hi,
Fixed these two functions names in the comments. About New (previously named NewNotificationHandler), is there any reason not to name it NewHandler?. The places where it is used inside the repo are notification_test.go and cmd/prometheus/main.go, but it could be used outside of the repo too so maybe you want to leave it as it is. Hope it helps...
Cheers

NewHandler would make more sense to me. @fabxc?

It's quite common for idomatic go packages that just have "one thing" to simply stick with New.
https://blog.golang.org/package-names
Looking at the godoc, I don't see any confusion here and don't see anything added by adding a suffix.
If we were to change it, then to something more meaningful than Handler or Manager in general.
https://godoc.org/github.com/prometheus/prometheus/notification

"A function named New in package pkg returns a value of type pkg.Pkg."
But notification.New doesnt return notification.Notification. Rename HandlertoNotification`? Not sure about the best way.

notification.Notifier better? Or rename package to notifier, too.
Handler is really horrible.

I'd vote for notifier.Notifier for maximum consistency.

In case you finally decide for notifier.Notifier, do you want to rename this through the repository or I can update this PR? (It would be today but after several hours)

:+1:
It's fine, I'll do the renaming in a separate PR.


Update install.sh

Optimize: Mac Osx Detection
Optimize: Check Bash Version

This is just "if-else" contraction, it doesn't improve or enhance anything


Update bindable-icons and l-icon documentation

Upgrade to bindable-icons v0.3.0 and display all of the available icons in the l-icon documentation

Coverage remained the same at 73.489% when pulling a3bca4c51015116a11c8129515c727c82804f0fd on bindable-icons into c74d33e1cfb2e411d0723b7603bb9aeaf693d0d3 on master.