BGDIINF_SB-1971: Added KML author in POST request

Now in the POST request you can specify the KML author. This will allow
to differentiates KMLs created by user (map viewer) to the one created
by E2E tests.

Good idea.
Just need to make sure, this will not cause troubles w.r.t. backwards-compatibility.
Especially when migrating the "old" kmls, that don't have the author field yet. Should be possible to simply batch-fill them with "" in that case, I guess.

This author field is only available in DB and not used anywhere. It will help in future to do some cleanup especially if we do E2E test on PROD we could safely delete kml files that came from E2E tests. For the migration we can then set the author to legacy-mapviewer for example.


Add additional early errors to disallow '(?i-i:)'

This makes it an early error to both set and unset the same flag, such as in (?i-i:.).
Fixes #8.

I've cleaned up the text to more clearly illustrate the semantics and reduce potential confusion for implementers. The result is that we independently check whether duplicate modifiers appear in the first set of flags, then independently check whether duplicate modifiers appear in the second set of flags, then check whether flags from the first set are present in the second set.


gaussianBlurAsync error when using kSize values other than (7, 7)

Describe the bug
Using values other than (7, 7) for the kSize in the gaussianBlurAsync method results in a crash.
To Reproduce
Steps to reproduce the behavior:

Use the code from https://github.com/rainyl/opencv_dart/blob/main/packages/opencv_core/example/lib/main.dart as your project.
Change blur = await cv.gaussianBlurAsync(im, (7, 7), 2, sigmaY: 2); to blur = await cv.gaussianBlurAsync(im, (8, 8), 2, sigmaY: 2);
When you select an image with the "Pick Image" button, it results in an error/crash.

If I set the kSize back to (7, 7) it works, but any other value doesn't.
Expected behavior
The blur method should not crash with values other than  (7, 7).
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop:

OS: Windows 10

Additional context
The error:
Exception has occurred.
CvException (CvException(code: -215, msg:OpenCV(4.10.0) D:\a\opencv.full\opencv.full\opencv\modules\imgproc\src\smooth.dispatch.cpp:294: error: (-215:Assertion failed) ksize.width > 0 && ksize.width % 2 == 1 && ksize.height > 0 && ksize.height % 2 == 1 in function 'cv::createGaussianKernels'
) in cv::createGaussianKernels of file D:\a\opencv.full\opencv.full\opencv\modules\imgproc\src\smooth.dispatch.cpp:294)

Maybe this is related to https://github.com/rainyl/opencv_dart/issues/180?

@Correct-Syntax Have you tried other odd values like (9, 9), (11, 9), (15, 15)? Please note that the kernel size of gaussianBlur must be ODD, refer to https://docs.opencv.org/4.x/d4/d86/group__imgproc__filter.html#gae8bdcd9154ed5ca3cbc1766d960f45c1

@rainyl Ah, I see. I'm used to the Python version of OpenCV where they must silently correct the values, not matter what is specified.


ARROW-6725: [CI] Disable 3rdparty fuzzit nightly builds

@yevgenypats I've created a follow-up issue where we can discuss how should we reliably execute the fuzzit job https://issues.apache.org/jira/browse/ARROW-6726

@kszucs great, I've commented there.


Suggestion: Make spells ignore i-frames

The spells are really fun to use without i-frames, I made a wand that was Homing + Increase Lifetime + Piercing -> Glimmer bolt with trigger -> Triple cast -> Chainsaw x3, this kind of wand is a boss shredder without i-frames and a boss shredder in noita as well. Make the spells fade after hitting once but with piercing, because they can hit multiple times the same enemy (like in noita) they should ignore i-frames and kill the boss really fast.
https://github.com/user-attachments/assets/5a60af76-7f04-4bff-ab0c-4c19807c6f8b

i've thought of doing that, but i fear some balance problems could come with it, specially knowing how chaotic the spells can be, im experimenting with the idea of making the spells just shred thru mobs, tho probably its best to make it a gamerule or a config


External assemblies not loaded properly

ModLib.FileDatabase.TypeData.Type attempts to load assemblies with Type.GetType. However, this does not work because they are not in the same file (ModLib.dll) or mscorlib.dll.

#6

Thanks for the catch, I've merged your pull request to fix it.


show tracks on scrollbar hover

show tracks on scrollbar hover to better UX, so user will know that area can be scrolled

You can extend default styles to achieve this.

[data-scrollbar="true"]:hover .scrollbar-track {
	display: block !important;
	opacity: 1 !important;
}


Stack overflow in LSP/lake build for snippet

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Check that your issue is not already filed.
Reduce the issue to a minimal, self-contained, reproducible test case. Avoid dependencies to mathlib4 or std4.

Description
The following MWE causes a stack overflow in lake build or causes the LSP to hang
inductive A
  | foo : A → A

inductive B : A → A → Type where
  | foo : B (.foo q) q

structure ImageR (R : α → α → Type) (a : α) where
  {b : α}
  (r : R a b)

instance : Ord (ImageR B p) where
  compare i j := match i.r with
    | .foo => match j.r with
      | .foo => sorry

Steps to Reproduce

Put this snippet in a lean project
lake build

Expected behavior: No stack overflow
Actual behavior: Stack overflow
Versions
[Output of #eval Lean.versionString or of lean --version in the folder that the issue occured in] 4.7.0-rc2
[OS version] NixOS 24.05pre-git (Uakari) x86_64

Does not reproduce anymore


Application does not open!

What happened: I have updated my desktop DeFi wallet to version 2.3.3 at first it struggled with downloading the snapshot and for a couple of times it gave me errors. then I decided to let it sync the blocks from scratch and it went up to 87% and again gave another error like node is disconnected.
Now the whole app is not coming up again and it just shows the logo
What you expected to happen: The wallet to start and run the syncing process.
How to reproduce it (as minimally and precisely as possible):

What are your environment parameters?

DeFi Wallet Version: 2.3.3
Operating System (OS): macOS Big Sur (11.2.3)

Anything else we need to know?:

Archive.zip

If you run the app from the command line, what does it say when you're just seeing the logo?

Here is a screenshot of command line and the application

So from looking at your message and the code it seems your wallet_map.json file may have some trouble in there. That file is in your defi folder.
If you know your way around computers, maybe you can look at it and figure what's wrong.
If not, I'd be happy to help, but to be honest I don't know if anything in there is supposed to stay private...

Alright, let me have a look and get back to you
thanks a lot

Sure thing!

I think I will be needing more help on this matter, lol
how can we proceed?

Come to the DFI telegram and I can help you from there.

Thanks for the help @John-Gee. @rezaghiasi as John pointed out, there might be some issue on your wallet_map.json. Need to check it for potential issues. You can post it here but you can delete some personal info like directory etc. Just need to view the structure

In my case the app doesn't open also, on my windows PC. Installed DeFi-Wallet-Setup-2.3.2 and DeFi-Wallet-Setup-2.3.3, but both the same. What do i have to do?

My App dosen't open also. I have a Mac (macOS Big Sur (11.2.3)). What should i do?

Hi @Mabarkubus , did you use the DeFi-Wallet-Setup-2.3.3.exe file extension type? Please backup your wallet.dat before performing a reinstallation.
Hi @gianlucast1 , for MacOS Big Sur, try removing the .defi file, here's a guide MacOS Big Sur: Unable to open app after installation.

I still had made a backup of my wallet and tried severel times a reinstallation. I don't know what to do now?

Hi @Mabarkubus , can you check if you've sufficient storage? On Windows, currently it requires >100GB of storage. The team is working on a light wallet and reducing the storage size required.

@izzycsy it's not working. I have the same problem, like on the screenshot in the massage from "rezaghiasi" 14 days ago.

Then this could be the reason, i'm only having 21GB of storage on Drive C...

@izzycsy it's not working. I have the same problem, like on the screenshot in the massage from "rezaghiasi" 14 days ago.

You may want to start from scratch, with a proper backup of your wallet then.

...but i had the "older" versions of the app DeFi wallet before version 2.3.1 in use, so why is this problem now. I deleted a lot of software and files on my system, but the app doesn't work again...

Hi @Mabarkubus , that's because the blockchain grows and you're running the blocks yourself in your desktop app. We understand that it's taking a lot of storage space and the team is working on reducing it somehow, including a Light Wallet. Thank you for your understanding.

@izzycsy it's not working. I have the same problem, like on the screenshot in the massage from "rezaghiasi" 14 days ago.

You may want to start from scratch, with a proper backup of your wallet then.

How can i do this?
Do you know how many storage i need on my mac Book to Start the Wallet? Now I have 32GB free. I don't know, but maybe I don't have enough space on my Mac.

Hi @gianlucast1 I doubt it's enough, currently it grew to this size. I would advice that you backup your wallet.dat and wait for a fix if you didn't manage to clear storage.

Thx for the massage. Is it possible to download the Wallet with the Snapshot to an external hard drive and run the wallet on the hard drive?

Yes you can do that, the snapshot is about 29Gb and the blockchain around 80Gb, you'll need enough space for both before deleting the snapshot.

Cool, thank you. So i will check it out.


Location of every event displayed accurately on map

Fixes issue #1054
Location of every event is accurately displayed on map, by passing session id through shared preferences.

Can you add after/before screenshots too?

@the-dagger
Before
https://cloud.githubusercontent.com/assets/10931958/22261710/03d48e58-e294-11e6-89c3-3daf190556ce.png
After
https://cloud.githubusercontent.com/assets/10931958/22261837/77d19c56-e294-11e6-9a87-e32879aef7ea.png
they are also mentioned in the issue #1054


LOG4J2-3080 Use SimpleMessage in Log4j 1 Category whenever possible

@garydgregory @carterkozak @rgoers Thanks for the review. Will merge this and port to master.


Forced Chunk Load Hack

Well, it works by forcing a cancellation of the chunk load when it's in the map.
Once teleported, map will be removed, and bukkit will take care of the chunk cleaning.

Closed by request. Check out PR #6


Feature Request: Manual Entry and Editing of Charging Log Entries for External Charging Sessions

I would like to request a feature that allows users to manually enter and modify entries in the charging log to document charging sessions conducted at public charging stations. The goal is to obtain a consolidated cost overview for my vehicle. While I predominantly charge at home using evcc, I occasionally charge externally. Including these costs in my statistics would provide a more comprehensive understanding of my total charging expenses.
Key Features:
Ability to manually add and edit charging sessions with the following fields:

Date and Time of the charging session
Charged kWh
Cost
Duration
Vehicle (selectable via dropdown)
Comments (optional)

Use Case:
This feature would be particularly useful for users who primarily charge at home but occasionally use external charging stations. By allowing manual entry of external charging sessions, users can maintain a complete and accurate charging log, facilitating better cost tracking and analysis.
Describe alternatives you've considered
Alternatively, providing API access to the charging sessions would also achieve this goal. This would allow users to:

Retrieve charging data to create a consolidated view in an external system.
Insert charging session entries via API.

Thank you for the suggestion. We don‘t plan to work on this, its beyond scope of evcc.
/cc @naltatis

Agree with @andig. The ability to manually create sessions would, beside the obvious data entry/management UI, open up a lot of special cases we'd have to address (optional fields for charging speed, duration, co2, ...). Sounds like a good opportunity for a third-party application. evcc charging data can be easily fetched from REST API or as CSV.


feat(insights): use separate deployment for Insights

Welcome to Cryostat! 👋
Before contributing, make sure you have:

[ ] Read the contributing guidelines
[ ] Linked a relevant issue which this PR resolves
[ ] Linked any other relevant issues, PR's, or documentation, if any
[ ] Resolved all conflicts, if any
[ ] Rebased your branch PR on top of the latest upstream main branch
[ ] Attached at least one of the following labels to the PR: [chore, ci, docs, feat, fix, test]
[ ] Signed all commits: git commit -S -m "YOUR_COMMIT_MESSAGE"

Fixes: #961
Depends on: https://github.com/cryostatio/cryostat-agent/issues/510
Description of the change:

Removes Go module dependency on the Runtimes Inventory Operator
Adds a Kustomize overlay to optionally build a bundle that includes a second deployment for the Runtimes Inventory Operator container image with its own RBAC
Bumps Kustomize version to support Replacements

Motivation for the change:

Removes a Go module dependency that is only useful in downstream Red Hat builds
When enabled. the Insights integration no longer counts towards Cryostat Operator's resource limits

How to manually test:

make bundle bundle-build ENABLE_INSIGHTS=true
Deploy an agent sample app with -Drht.insights.java.debug=true
Check for a line in the agent log like: Red Hat Insights - Payload was accepted for processing

/build_test


Data collection stuck at 91.3%

I am using version 1.6.0 of the addon in Thunderbird 78.10.0, with an account that has somewhere in the vicinity of 200,000 messages. I have yet to see the reports - every time I try to start it up, the data collection gets stuck at 91.3% complete, with the CPU still running high.
Below is the output from the debug console. Everything you see here is displayed within the first few seconds. ("Inbox > Non-3PLTA Today" is a saved search folder. That message comes up right away, not when the collection hangs. There are five other saved search folders, though, and it doesn't complain about any of those.) When it hits 91.3% after about 90-120 seconds, there are no additional messages indicating what is causing the hang.
Let me know what other information I can try to collect for you. Thanks!

Hi Keith,
thanks for reporting this. Unfortunately, special characters in search folder names make a lot of trouble in Thunderbird and as extension developer, I don't have any possibility to fix this (yet, I reported it some time ago https://bugzilla.mozilla.org/show_bug.cgi?id=1688612).
Can you check, if this issue is caused by the space character in your search folder name? E.g. does it work, if you rename it to Non-3PLTA-Today? This may be a possible workaround.

That was it. I only had to remove the spaces in the saved search folders; it didn't pose a problem in the normal folders. Just for good measure, I also found two folders with an & and a ' in them, and removed those.
This might be a good thing to add to the docs, since, as you said, there's a pretty easy workaround. Thanks for the quick response!

Glad it works, and thanks for reporting those additional characters. I will recheck this as soon as the next major release of Thunderbird landed this summer, as there will be a lot of core changes.

This might be a good thing to add to the docs

Very good idea, a "known issues" section or page somewhere in ThirdStats would be helpful for users. I try to figure out how this can be automated somehow.

Thanks for the quick response!

You're very welcome.

FYI the above mentioned bug report got fixed in TB 90 and will be part of TB 91 in summer.


doc: fix warnings and add guidance around the use of Py_FindMethod

Add a "Py_FindMethod and Generic Attributes" section to document the
use of Py_FindMethod.
Document how to upgrade old code that uses Py_FindMethod() from a custom
tp_getattr function.
Provide an example demonstrating how to replace general usage of
Py_FindMethod.
Closes #22
Signed-off-by: David Aguilar<EMAIL_ADDRESS>

I tried building the docs this morning and I hit a number of sphinx warnings. I've added the warning fixes to this PR, but if you'd prefer that they be split out to a separate PR please let me know. Thanks again for py3c!

Good idea, less is more. I just rebased and dropped that last section. Thanks for the review.

Thank you!


[Feature] Prevent component create with empty title [OSF-8162]

Purpose
Fix issue where a user submits empty (create component)
form through enter key press.
Changes

Prevent addProjectPlugin controller from sending request if project/component
title is not provided.

Side effects
N/A
Ticket
https://openscience.atlassian.net/browse/OSF-8162

Also be sure to add QA notes for the various conditions you find.


feat(wip): add initial structure of session replay

:bulb: Motivation and Context

#69

:green_heart: How did you test it?
:pencil: Checklist

[ ] I reviewed the submitted code.
[ ] I added tests to verify the changes.
[ ] I updated the docs if needed.
[ ] No breaking change or entry added to the changelog.

Amazing @thisames will take a look.

Everything is going well. In theory, the replay already works. However, I still need to thoroughly analyze the code and adjust other details. I also need to organize the folder structure and check if there is any code smell.
Additionally, here are the main tasks that are currently a priority:
[x] masks navigation context
[  ] Regular and improve the debounce (It currently works ok, but not enough)
[  ] Improve performance (It currently works ok, but not enough)
[  ] Add a solution to perfectly synchronize the mask with the current screenshot (maybe a manual schedule or queue?)
https://github.com/user-attachments/assets/4777577b-722c-4e94-8021-72783c046a2c

@thisames look at this PR since I need to get this merged in first and will conflict with a few things for you

@thisames look at this PR since I need to get this merged in first and will conflict with a few things for you

I think you can merge this https://github.com/PostHog/posthog-flutter/pull/117. Then I'll pull it and resolve the conflicts. But there's nothing that will impact my PR that much. I just need to adapt the initialization settings that I added to the style that you implemented.
https://github.com/PostHog/posthog-flutter/blob/197002162df1ddc9fbd5acf524a933a1ddbd1ae7/lib/src/posthog.dart#L25-L38
from what I understand, my init is equivalent to your setup.
Future<void> setup(PostHogConfig config) => _posthog.setup(config);

hello @thisames Hope you are doing well
I see you guys are working on the session recording is this feature will be available for Flutter web as well?

hello @thisames Hope you are doing well
I see you guys are working on the session recording is this feature will be available for Flutter web as well?

I think that @marandaneto may say better about that


Add more flexible assertions

I found current assertions to be too limiting and needed some more flexibility with partial matching and more flexible assertions in general. I Added the following functions to Apus.Test module to achieve that:
assert_delivered_message/0 - checks if any message was delivered
assert_delivered_message_matches/1 - allows custom function matching similar to what Swoosh provides with anonymous function test assertions
refute_delivered_message/0 - refutes that any message was delivered
Docs and readme were updated to reflect the changes above, plus added some additional assertion examples.

@swelham is there anything I can help with to make this easier to review/have it merged in? Cheers!

Really appreciated over here. Hope can be considered and reviewed soon :-)

Hey, thanks for the PR and sorry for the delay on this, having a crazy busy start to the year!
This looks good to me 👍
I will get it merged and published.

Thanks @swelham, much appreciated! I haven't tested these changes on older Elixir version, it seems that some of the test additions now break the build_legacy pipeline. If the pipeline is still relevant I'll come up with a fix for it


Is it safe to use the new react-router 2.0?

Hi. Im starting a new project and was wondering if I can use the new version of react-router without getting into any issues? Thanks.

I'll probably stay with react-router 1.x, as upgrading it causes many errors.

I'll probably stay with react-router 1.x, as upgrading it causes many errors.


[JAX] Added prepare phase for the FusedAttnForwardFFI

Description
In this PR, the FusedAttnForwardFFI is registered to XLA via 2 phases:

prepare where the te_cudnn_handle_init_ffi is called.
execute phase where the actual fused attention function is called.
In the past, the default was to have only the execute phase.
This helps to avoid memory allocation for the handle in the cudaGraph capturing area, thus resolving the issue we have in the distributed run with cudaGraph.

In general, all the custom calls that involve cuDNN kernels should be registered with these two phases in the future. Since the prepare function needs to have the same function argument list with the execute function ones, variadic args are used to make this te_cudnn_handle_init_ffi general so that it can be called in any custom calls.
In addition, I moved the common cudnn helpers out of fused attention so that they can be used in other kernels, i.e. the coming normalization.
Type of change

[ ] Documentation change (change only to the documentation, either a fix or a new content)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Infra/Build change
[ ] Code refractor

Checklist:

[x] I have read and followed the contributing guidelines
[x] The functionality is complete
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

@cyanguwa Please help to look at the changes in Common and let me know if there is an issue.

\te-ci jax L1

\te-ci jax L1

/te-ci jax L1


[RFE] At provide-certificate action, verify public keys in CSR and received certificate matches

During the processing of provide-certificate action, the tls-certificates-operator should use the relation-id provided as a parameter to recover the CSR set by the Requirer and compare main fields:
# Compare types as there are multiple and key values
if not csr.public_key === certificate.public_key:
    event.fail(message="Certificate and CSR public keys do not match")
    return
if csr.subject != certificate.subject:
    event.fail(message=f"Certificate has subject: {certificate.subject} whereas CSR has: {csr.subject}")
    return

This makes sense, I'll add this to our TODO.

https://github.com/canonical/tls-certificates-operator/pull/67 implements this and is merged.


support reproject to bounds edge

filter out infinity values

what if a bounding box has infinity values along every edge?
one solution could be to treat the bbox as a grid, drop points evenly spaced throughout the inside of the bbox, reproject every point, and then recalculate the bbox
that would be a fairly simple solution and without regression for the current point density approach.  the current point density approach drops points evenly along the edges.  this would just add points to the interior as well

not perfect, but if you reproject with nan_strategy = "skip" AND high density, you'll get close to what you want.


Add Johan Yim to Contributors list

adding myself to the contributors list

Hi  @johanyim, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added social share  to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


Separate instantiation from showing the view. Otherwise it can happen…

… that you end up in a situation where the UI is blocked.
In the original version where instantiation and showing is done in one step (dispatch main) it can be that the dispatch didn't occur when you call the hide method. Hide is checking as a first step if there is an instance, which is not, because the main thread didn't elaborate it yet, and exits.
Then at a point in time the main thread is showing the Indicator with blocking id, and never gets dismissed.
With this change the instance is there, even if it hasn't been shown and as both showing and hiding happen in the same thread (main) it will be shown and immediately hidden, and the UI is unblocked.

Good catch, thanks man :+1:

Actually it was a problem that I had :-)
Could u please update it on cocoapods too?
thanks
~hons
On Wed, Jan 20, 2016 at 12:19 AM, Goktug Yilmaz<EMAIL_ADDRESS>wrote:

Good catch, thanks man [image: :+1:]
—
Reply to this email directly or view it on GitHub
https://github.com/goktugyil/EZLoadingActivity/pull/14#issuecomment-173019932
.

I will update the pods tomorrow

Sorry for late update. I updated the repo now


test_matrix: fix warning

What was the warning?

Instead of deleting can you just print it:
cout << mat4 << endl;

Good point, can you do a pull request to warning branch?


Pre-commit hook for pylic

Hi,
thanks for the great package. Would it possible to offer a pre-commit hook for license checking?
I could be useful to run the license checking via pre-commit e.g. in CI, if someone added a new dependency, or a dependency changes its license.
Maybe like this?
.pre-commit-hooks.yaml:
- id: pylic
  name: pylic
  entry: pylic check
  language: python
  pass_filenames: false

Hi @afuetterer
Glad you find it useful :slightly_smiling_face:
This would definitely be a sensible addition!
Let me see if I can squeeze this in, else feel free to open a PR yourself! :slightly_smiling_face:

Turns out this was a very quick thing and I just could take exactly what you proposed @afuetterer :clap:
Please check if this works for you! I also added a small README section, I hope this is also fine.
This should work with the latest tag v.3.5.0.

I don't use pre-commit regularly so I'd be very interested in hearing from you if this feels right for you @afuetterer
:slightly_smiling_face:

Hi @ubersan, that was fast. Thank you. I will test this soon.

This works like a charm. Thank you. :slightly_smiling_face:

Thanks for the feedback, glad it works😊👍

Hi @ubersan and @afuetterer,
thanks for providing such a useful tool. I've just tried to use pylic as a pre-commit hook, however, it seems as if it only checks some globally installed packages and not the ones in my project's pyproject.toml file. As the environment created by pre-commit is usually empty, is there any way to also consider my listed dependencies from the pyproject.toml file in the checks?
To be more precise: It works perfectly when I manually call pylic check in my virtual environment with all the installed dependencies. But when it is called via pre-commit hooks, it always fails and says I have unnecessary safe licenses listed which are not used by any installed packages. The problem seems to be that not all my packages are installed.

Hi @TomTomRixRix
Glad it's useful for you.
Indeed, pylic only checks against the installed packages in your current environment. If I understand correctly, to take the pyproject.toml (or other dependency lists found in e.g. requirements.txt, ...) as basis I guess pylic  would need to do the dependency resolution to figure out which version will be relevant for every package. This would be necessary as the licenses can change between versions.
As this would be a major task and already tools like poetry, pip, ... take on this responsibility I think it would be easiest not to also do this.
Does this make sense?
I'm not sure if I have a solution for you to improve the situation outside of a venv :/

Thanks for your fast reply. Then I see just two possibilities that would be useful in my case:

Offer the possibility to manually run pylic check in the local environment without a pre-commit hook,
Write a GitHub action to run pylic in an already set up testing environment on a remote repo where the dependencies are resolved and installed.

Could you elaborate on what you mean in 1?  I guess I don't fully understand, but you can e.g. install it as dev dependency via poetry add pylic --group dev and then run locally via poetry run pylic check.
In regards to 2, yes this is surely a good solution as usually only on CI you have the exact environment/dependencies installed that you also later want to ship. It's thus a good idea to at least run the check there.

Exactly, option 2 should definitely work fine. The only drawback is that it only checks the license compatibility after commits have been done and pushed. So in the worst case scenario, when adding a dependency with an incompatible license, the developers only find this out once they've committed and pushed their code. Ideally, there would also be an option to find the incompatibility earlier, e.g. before even committing.
Thus option 1 might also be interesting. There developers could run pylic check or poetry run pylic check in their command line locally to see if all dependencies have compatible licenses. For this to work I would only have to add pylic as a dependency to my pyproject.toml and developers would have to install it into their virtual environment. However, as far as I understand these local checks cannot be done automatically with a pre-commit hook as the dependencies are not installed in the pre-commit environment.


Split message definitions from headers

All of the cFS messages are currently defined in C structures.  As a first step toward moving to "electronic data sheets" to describe the external data format, these need to be slightly modified to better separate the header portion of the structure from the payload portion of the structure.
Currently, message structures are typically defined by reserving a block of space for the header as a uint8 array of size CFE_SB_CMD_HDR_SIZE or CFE_SB_TLM_HDR_SIZE.  This approach has several issues:

Using a fixed-size block assumes only a single type of encapsulation (CCSDS) will ever occur.  This may not be the case, as other non-CCSDS encapsulations may be a requirement for some missions.
The fixed size block is not guaranteed to be properly aligned for a CCSDS header.  Since it is declared as a uint8 array, the compiler will not ensure any alignment this structure.  It is technically not valid to cast this as a CCSDS header since that contains uint16's.
This is unlikely to be compatible with electronic data sheets (EDS) no matter what specific implementation is used.  Since the definition of the message content (payload) and the message header (CCSDS or other format) will come from different data sheets, it becomes very problematic to have them mixed together like this.

To solve this problem requires a bit of restructuring: instead of declaring the format of the payload directly within the message structure, declare a separate "Payload" structure and define it in there.
This adds one extra layer to the structure tree but will improve flexibility going forward, and it will NOT change the external data format, so compatibility with ground systems is unaffected.  It only affects the syntax of code accessing members of the payload structure.

Imported from trac issue 4.  Created by jphickey on 2014-12-22T13:49:40, last modified: 2019-03-05T14:57:55

Trac comment by jphickey on 2014-12-26 10:12:52:
Pushed branch "trac-4-cfe_payload_separation" which addresses this issue.  Final commit is [changeset:4fdeade6] which includes the change for all core applications and unit tests.
In general:

For all command and telemetry messages a new "_Payload_t" structure is defined which reflects the content of the message //without// any headers.
The existing "_t" structure now just wraps together the Payload structure with a reserved block of space for a CCSDS header, exactly as was done before.
References are fixed in one of two ways: either the CFE_SB_MsgPtr_t (message header pointer) is changed to a a CFE_SB_MsgPayloadPtr_t, or the reference was fixed to have an extra ".Payload."

Merging this change will ease the forthcoming electronic data sheets (EDS) change since the Payload structures and the Header structures come from different data sheets, they are different structures.  This just updates the code to follow that model so the data sheet will drop-in easily.  There should be no difference in the binary format of the final message.

Trac comment by glimes on 2015-03-04 14:17:49:
Accepting this change into CFE will have to be done in lockstep
with accepting a corresponding change to several CFS applications.
Too bad "unnamed structure fields" are a C-11 feature, it would have
been a nifty way to do this in a source-compatible way.
Side note: this may change where the compiler inserts alignment padding
and could change the size of the packet. Consider the case where the
border between the header and the payload is in the middle of a word,
and the payload contains data elements that require word alignment: the
inner structure will have padding to assure the aligned field is at an aligned
offset within the inner structure, and separately, padding will be inserted
before the payload structure in the packet structure so that the payload
starts at an appropriately aligned offset.

Trac comment by jphickey on 2015-03-04 14:50:10:
Greg - it may be possible to do this for the apps separately.  In general, the apps define their own messages and most generally just talk to the ground station and not to each other (exceptions perhaps for the aggregator apps like HS/HK, these might be more complicated).
Also - the alignment issue is noted and I can confirm that this is not a problem in the currently defined messages since the combined CCSDS primary + secondary headers are either 8 or 12 bytes, it is always a multiple of 4.
There is actually a side benefit to this too: it ensures that all packet payloads can also be instantiated without encapsulation headers - this allows use of alternate encapsulations in the future, even the option for a transparent encapsulation added by "SB" as the packet goes out.
In general having message constructs with misaligned words (made possible by an odd-sized element before it) should be very much frowned upon as it may impede portability to other processors or other types of message encapsulation.

Trac comment by acudmore on 2015-04-06 14:26:03:
recommend accept.

Trac comment by jphickey on 2015-04-06 11:21:23:
This is ready for review/merge
Note - #68 may need to be merged at the same time which fixes some UT code that was only built when using UT_VERBOSE.

Trac comment by glimes on 2015-04-07 12:44:23:
Tested changeset [changeset:4fdeade] as part of the ic-2015-03-10 merge.

Trac comment by jwilmot on 2015-04-06 15:15:12:
Concur.

Trac comment by glimes on 2015-04-13 15:28:43:
Part of integration candidate 2015-03-10,
committed to cFS CFE Development branch on 2015-04-10
as part of merge [changeset:7d6f6d0].

Trac comment by glimes on 2016-02-25 10:17:32:
these will be fixed in CFE 6.5

Trac comment by jhageman on 2019-03-05 14:57:55:
Milestone renamed


JUKERFF

JUKERFF
Originally posted by @fnhckgrx in https://github.com/pallets/flask/pull/4189#pullrequestreview-883763998

Hallo zusammen

spam


[AutoPR monitor/resource-manager] Update monitor readme.nodejs.md to have latest generator flags

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2783

(message created by the CI based on PR content)

Codecov Report

Merging #2310 into master will decrease coverage by 0.04%.
The diff coverage is 36.1%.

@@            Coverage Diff             @@
##           master    #2310      +/-   ##
==========================================
- Coverage   52.77%   52.73%   -0.05%
==========================================
  Files        5630     5646      +16
  Lines      133962   134310     +348
==========================================
+ Hits        70697    70822     +125
- Misses      63265    63488     +223

Impacted Files
Coverage Δ

...azure/mgmt/monitor/models/action_group_resource.py
29.41% <0%> (-6.31%)
:arrow_down:

...-monitor/azure/mgmt/monitor/operations/__init__.py
100% <100%> (ø)
:arrow_up:

...or/azure/mgmt/monitor/monitor_management_client.py
100% <100%> (ø)
:arrow_up:

...gmt/monitor/operations/action_groups_operations.py
16.77% <100%> (ø)
:arrow_up:

...gmt/monitor/operations/metric_alerts_operations.py
18.46% <18.46%> (ø)

...itor/operations/metric_alerts_status_operations.py
30.61% <30.61%> (ø)

...mgmt/monitor/models/metric_alert_resource_patch.py
31.25% <31.25%> (ø)

...azure/mgmt/monitor/models/metric_alert_resource.py
33.33% <33.33%> (ø)

...nitor/azure/mgmt/monitor/models/metric_criteria.py
38.46% <38.46%> (ø)

...r/azure/mgmt/monitor/models/metric_alert_status.py
44.44% <44.44%> (ø)

... and 27 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 b40e0e3...37a8a25. Read the comment docs.

Node PR


Needs an 1.19.2 update

Describe the Suggestion
The mod works well on its own but needs an all around update to be compatible with other modpacks. I use Create with the NewCraft modpack and its no longer compatible with the game version.
Screenshots and Videos
No response
Additional Context
No response

updated: https://github.com/Fabricators-of-Create/Create/commit/5c6c284cb9630a3e0bc0aacc0a76a055ebdd4443
Seriously though, if you want bugs fixed you need to be specific. I have no idea what NewCraft is or what's wrong with adding Create to it.


Add JWT support to Xpanse's authentification

Regarding the API authentication for Xpanse, they currently call Zitadel's API to validate the user's token each time. This approach might potentially strain Zitadel's capacity in the future. To mitigate this, we should contemplate integrating JWT for Xpanse to independently verify tokens.

[ ] Integrate JWT support at the Xpanse API level.
[ ] Incorporate configurations into Xpanse for switching between JWT and traditional ways.

I will take it.


[npcollector] Add namespace to Network Path Collector

What does this PR do?
[npcollector] Add namespace to Network Path Collector

Motivation
Namespace is used for NDM resolution in backend.

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Did some QA on my side. Tested in two ways, first looked at the agent debug logs to see that a passed namespace was properly set and that a missing namespace resulted in default.
I also added an API key and checked the raw events being stored to ensure the namespace was being propagated back to storage. I did not test resolution as I don't have a great way to simulate devices on my path.
Everything looked good with regard to the changes made here, but one potential issue came to mind. This code only works if namespace is set in datadog.yaml, so it only works OOTB for a customer if their agent is running the SNMP integration with autodiscovery:
datadog.yaml
network_devices:
  namespace: my-namespace

but an NDM customer may have it configured in the SNMP integration at the init_config or even instance level.
conf.d/snmp.d/conf.yaml
init_config:
  namespace: my-namespace

instances:
- ip_address: <IP_ADDRESS>
  community_string: public
  namespace: my-instance-namespace

I don't know if we want to get into the business of complex logic around which namespace to send, but maybe we need a note in the public docs for customers not running SNMP from the same agent or if they're using not using autodiscovery.

I don't know if we want to get into the business of complex logic around which namespace to send, but maybe we need a note in the public docs for customers not running SNMP from the same agent or if they're using not using autodiscovery.

Agree with you analysis, we don't cover init_config and instance level namespace.
I think we need to document what's required for NDM Resolution (matching namespace).
@ken-schneider I think we can possibly address that with NETPATH-212
network_path:
  configs:
    - selector:
        ip: <IP_ADDRESS>/24
        # we can add other selectors like:
        #   - reverse dns hostname
        #   - port
        #   - protocol
      # Actions to apply:
      namespace: my-ns
      max_ttl: 42
      timeout: 10
      allowed: false # to prevent creating paths for selected NPM connections
                     # default to true
    - ... # more configs

/merge


Update Populate

Along the week 7, some minor changes have been made to the db. It is expected that the populate will now be either incomplete or non working. The populate.json file shoud be updated so that it can continue tu be useful.

Story Points: 2
See the discussion in Backend S2 W8
Powered by Parabol


Referencing a secure param in a regular (non-secure) param is not flagged

The following scenarios should be blocked:
Scenario 1:

main.bicep:@secure()
param secureParam string

param insecureParam string = secureParam

Scenario 2:

main.bicepparam:using 'main.bicep'

param secureParam = 'MYSECRET'
param insecureParam = secureParam

main.bicep:@secure()
param secureParam string

param insecureParam string

Maybe a separate issue, but we should in general be flagging whenever a sensitive value is assigned to a non-sensitive slot or vice versa. For example, the following template raises no diagnostics but should raise two:
@secure()
param secret string

param nonSensitive string

resource e 'Microsoft.CostManagement/exports@2024-08-01' = {
  name: secret    // <-- Should flag assignment of sensitive data to non-sensitive slot
  properties: {
    definition: {
      timeframe: 'BillingMonthToDate'
      type: 'Usage'
    }
    deliveryInfo: {
      destination: {
        container: 'containerName'
        sasToken: nonSensitive   // <-- Should flag assignment of non-sensitive data to sensitive slot
      }
    }
  }
}

We should be able to do this based on existing type validation flags. I'm not sure if we accurately track "taint," though (i.e., is a variable whose value is based on a secure parameter itself be flagged as secure?).


SourceControl: avoid the builtin FS monitor

This disables the builtin FS monitor on the checkouts of the
dependencies.  This ensures that the global configuration of
the setting does not influence the repository local behavior
as that can cause problems on Windows. Due to the filesystem
behaviour on Windows, we cannot remove a directory with open
descriptors - which the monitor implies. This should resolve
the problem where a series of swift package commands would
result in failures.
[One line description of your change]
Motivation:
[Explain here the context, and why you're making that change. What is the problem you're trying to solve.]
Modifications:
[Describe the modifications you've done.]
Result:
[After your change, what will change.]

@swift-ci please test

@swift-ci please smoke test

Based on some web searches it seems that this is a Windows-specific feature... is that right?  If not, does this change have any effect on other platforms, and should the disabling be conditional to Windows?

@swift-ci please test package compatibility

@compnerd do you want to cherry-pick this into 5.6 branch?

Ah, yeah, that we should do.  I'll put that up in a minute.


History listed in back button is unhelpful

See attached.

For the most part this is just based on the <title></title> tags. Improvements have been made but a few scenarios where it's still unhelpful. Will push into next milestone and finish from there.


Change wording/messaging for SLC statuses.

Some of the feedback given is that our messaging about a courses status is confusing, especially the "Incomplete" status. Change "Incomplete" to "In Progress" and think about other possible changes that could be made.

Being addressed in PR #822

 Change wording/messaging for SLC statuses. #832


Parcelable support?

PocketKnife only supports Bundle, and I'm actually using IcePick to save/restore view state in a friendly manner to save boilerplate. IMO, parcelable support would make PocketKnife a fully featured swiss knife ;)

This is a reason I added BundleSerializer and IntentSerializer. So users can persist other types of data.


Usage of generalized convolution

Hello, I'm trying to use the generalized convolution of sparse tensors and want to ask you guys how to use it, cause the issues that I've looked seems to support generalized convolution as in MinkowskiEngine.
In MinkowskiEngine, the generalized convolution is supported by taking coordinates as inputs during the forward() on the convolution operation as in here, which allows us to compute the features on the coordinates that is not directly in the input coordinates.
Looking at the torch-sparse issues in github, generlized convolution seems to be implemented, but I can't find the right documentation for it :(
Could you help me with  this issue? I want to compute the features that are within the neighborhood of the input coordinates.
Thanks a milliions for the wonderful repo!

Hi @96lives,
To enable generative transposed sparseconv, you may simply pass in the generative=True argument to the convolution module:
def make_up_block(in_channels, out_channels, generative=False):
    conv = partial(spnn.Conv3d, transposed=True, generative=generative)
    return nn.Sequential(
        conv(in_channels, out_channels, kernel_size=3, stride=2),
        spnn.BatchNorm(out_channels),
        spnn.ReLU(inplace=True),
    )

This is a sample code snippet in which you can get a generative transposed sparseconv block by passing in generative=True.
I will release another example codebase for indoor object detection that utilizes this operator soon.
Best,
Haotian

@kentang-mit  Thanks for the quick reply!
However, the spnn.Conv3d does not take in the generative argument currently.
Is there a quick way to fix this?

Hi @96lives,
The source code of torchsparse 2.1.0 is still under internal quality check, so what you see is the 2.0.0 source code which does not have that argument. But for the library you installed from the wheels, the generative argument is there. Feel free to directly add it.
Best,
Haotian

Hi, @kentang-mit
Sorry, but I'm not quite sure, if there is an argument.
For the simple code of just initializing the conv3d:
import torch
from torchsparse import nn as spnn

@torch.no_grad()
def main() -> None:
    conv3d = spnn.Conv3d(
        in_channels=1,
        out_channels=1,
        generative=True
    )

if __name__ == '__main__':
    main()

I get an assertion error as the following:
 python torch_sparse_example.py
Traceback (most recent call last):
  File "/home/ds/gca-outdoor-deploy/torch_sparse_example.py", line 15, in <module>
    main()
  File "/home/ds/anaconda3/envs/gca-lightning/lib/python3.10/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
    return func(*args, **kwargs)
  File "/home/ds/gca-outdoor-deploy/torch_sparse_example.py", line 8, in main
    conv3d = spnn.Conv3d(
  File "torchsparse/nn/modules/conv.pyx", line 53, in torchsparse.nn.modules.conv.Conv3d.__init__
AssertionError

@kentang-mit
Thanks for the reply!!  Is works.
Still I'm quite not sure as how to obtain the features of the generative convolution.
For example, I wish to obtain the neighborhood features from the following convolution afther the UNet (code snippet is from the examples directory of the repo):
import numpy as np
import torch
from torch import nn

from torchsparse import SparseTensor
from torchsparse.backbones import SparseResNet21D, SparseResUNet42
from torchsparse.utils.quantize import sparse_quantize
from torchsparse import nn as spnn
import torchsparse.nn.functional as F
F.set_kmap_mode("hashmap")

@torch.no_grad()
def main() -> None:
    device = 'cuda:0' if torch.cuda.is_available() else 'cpu'

    for backbone in [SparseResUNet42]:
        print(f'{backbone.__name__}:')
        model: nn.Module = backbone(in_channels=4, width_multiplier=1.0)
        model = model.to(device).eval()

        conv3d = spnn.Conv3d(
            in_channels=model.decoder_channels[-1],
            out_channels=1,
            transposed=True,
            generative=True
        )
        conv3d = conv3d.to(device)

        # generate data
        input_size, voxel_size = 100, 0.2
        inputs = np.random.uniform(-100, 100, size=(input_size, 4))
        pcs, feats = inputs[:, :3], inputs
        pcs -= np.min(pcs, axis=0, keepdims=True)
        pcs, indices = sparse_quantize(pcs, voxel_size, return_index=True)
        coords = np.zeros((pcs.shape[0], 4))
        coords[:, -3:] = pcs[:, :3]
        coords[:, 0] = 0
        coords = torch.as_tensor(coords, dtype=torch.int)
        feats = torch.as_tensor(feats[indices], dtype=torch.float)
        input = SparseTensor(coords=coords, feats=feats).to(device)

        # forward
        outputs = model(input)
        out_conv = conv3d(outputs[-1])
        #  @kentang-mit: expect out_conv to have coordinates more than 100
        # To be exact, the neighborhood of coords, with kernel size 3, but currently, the out_conv.C.shape[0] == 100

        # print feature shapes
        for k, output in enumerate(outputs):
            print(f'output[{k}].F.shape = {output.feats.shape}')

if __name__ == '__main__':
    main()

I expect out_conv to have coordinates more than 100.
To be exact, the neighborhood of coords, with kernel size 3, but currently, the out_conv.C.shape[0] == 100
Could you help me with the generative argument?
I'm not quite sure how to use it without the documentation.
I really appreciate your help :)


UART/serial bitbang driver

Is your feature request related to a problem? Please describe.
Hardware UART does not always allow custom baudrates. A bitbanging UART driver would provide a solution in such a case.
Describe the solution you'd like
There are already I2C and SPI bitbanging drivers implemented. Maybe a similar driver could also be implemented for UART?
Describe alternatives you've considered
-
Additional context
-

And W1 bitbang drivers,haha!

I2C and SPI have dedicated clock line so timing is not that critical on the bit-bang driver. However, as UART (in general) has no dedicated clock line, timing is very critical on the software side to to manipulating the signal, where even a timer interrupt coming it will disrupt I/O (let alone NMIs). So it is highly unlikely there can be a usable implementation. So closing it.


[VS Code] Agent terminated with exit code:<PHONE_NUMBER> and signal null: Failed to load the dll from [C:\Users\SREE.vscode\extensions\ms-vsliveshare.vsliveshare-0.3.246\dotnet_modules\hostfxr.dll], HRESULT: 0x960000The library hostfxr.dll was found, but loading it from C:\Users\SREE.vscode\extensions\ms-vsliveshare.vsliveshare-0.3.246\dotnet_modules\hostfxr.dll failed  - Installing .NET Core prerequisites might help resolve this problem.     http://go.microsoft.com/fwlink/?LinkID=798306

<!--
For Visual Studio problems/feedback, please use the Report

@jasongin I've tried but it doesn't resolve my same problem.

Well,I think I've figured it out.Probably caused by  one of .net framework 4.6.1's patches.After more test I'll give more information.
@jash-backbenchers @jasongin

Hello everyone.After several testes,I can offer a solution.

install KB4049016 patch.Once successfully installed,you can enjoy the live-share
if you windows update can't see the patch above,you can install all patches that haven't been installed shown in windows update.When done,you can also use live-share.
@jash-backbenchers @jasongin


Kicking players

This branch adds a "Kick player" button which then calls an op-fw endpoint and kicks the player from the server.

Implements #111


iroha_wasm_builder: remove cargo invocation

Right now there is a problem -- we have to invoke cargo from our iroha_wasm_builder to be able to build Rust Smartcontract in WASM. That can cause problems with cargo recursion problem (cargo doesn't like to be invoked multiple times from it self).
There is a wasm-builder crate which can help us with that. But I'm not 100% sure it can do all what we need. We need to do a research and try to use it

Maybe we can use cargo as library.

That can cause problems with cargo recursion problem (cargo doesn't like to be invoked multiple times from it self).

@Arjentix, What are some of the problems that can be encountered? I only seem to find https://github.com/rust-lang/cargo/issues/6412, and this one:

is already worked around by setting CARGO_TARGET_DIR to OUT_DIR in wasm_builder
wouldn't be solved by using cargo as a library, as the build dir lock is still there

@DCNick3 , the problem was found when we were using our cargo lints. Because of that in iroha_wasm_builder we unset CARGO_RECURSION env var. Probably now there's no problem, need to test it.

I think it's actually called RUST_RECURSION_COUNT :wink:

I don't think that using cargo as a library is a good idea.
Mainly because its API is perma-unstable and is not gonna be stabilized soon.
Other problem I can see is ignoring the rustup config, but ig with us using a pinned nightly version for building smart contracts that's wouldn't be a problem.
And, well, it doesn't seem necessary IMO: the RUST_RECURSION_COUNT workaround is not necessary w/o cargo lints (I wasn't getting any problems when disabling it).
There's also CARGO_ENCODED_RUSTFLAGS workaround: (https://github.com/hyperledger/iroha/blob/f9f5ede6791a99ec788a05755867e1fcb497a9e2/wasm_builder/src/lib.rs#L402C1-L406C47). This one still seems to still be required, but I think it's better then chasing a perma-unstable API.


Not able to navigate through the emojis using keyboard

https://www.loom.com/share/2a92f16533004eb69fa19f4f2b1ae202
bug _t

@neerajdotname https://vimeo.com/793585243/b475b77819

@AbhayVAshokan https://www.loom.com/share/ff4c303c29134d7d9f6e714cba574dfb

@gaagul _a let's take this up.

Revert the option+arrow key shortcut in emoji suggestion.
Show the emoji suggestion popup only when one letter is typed. Else return null.
Don't show a popup at all when there are no results.


Get link to current theme in Marketplace

When getting the current theme via !theme current, provide a link to the link's site with the VSCode Marketplace.

Can I work on this if its still available?

Go for it @0xKC!

:tada: This issue has been resolved in version 3.1.0 :tada:
The release is available on:

Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


Unexpectedly found nil while unwrapping an Optional value: file DivRise/Request.swift, line 72

Were you able to fix it I am struggling with it too. Thanks!


Angular 2.0 Integration

Could you please provide a way to integrate with Angular 2.0. Basically our requirement is to include the dynamic form only for few pages and static for few. Could you please let us know how we can leverage this using the json-forms

I am not very familiar with Angular to give you a generic response. Maybe if you specify what your problem is I could help you better.

Hi,
Thanks for replying!
I am using the base project to start with Angular 2: https://github.com/mgechev/angular2-seed
My requirement is to have some pages dynamic and some pages static. So I want to use this brutusin/json-forms library inside my base project project for the dynamic functionality.
I am pretty new to Angular 2 too and not able to find out how the integration can be done. Could you please help in integrating this or guide me through.
Thank you!


Be able to kill and relaunch the app during a test

Use case:
I have feature flags within my app that are linked with Firebase remote config, and can be set/modified via a debug menu.
Sometimes these feature flags don't get picked up or reflected in the app until killing the app and relaunching.
It would be nice to be able to kill the app and then relaunch by calling the launchApp command again, or a differing one such as relaunchApp.
I think this scenario could also be useful for testing changes and state are reflected as expected after the app is removed from memory/the stack.

I believe stopApp command is what you are looking for: https://maestro.mobile.dev/reference/app-lifecycle#stopapp

Perfect, thanks Dima!


Fix typo

It is spelled "example", not "exampe". I think.
Motivation:
Fix spelling
Modifications:
Added the letter "l"
Result:
Text passes the spellchecker.

@swift-ci please smoke test


feat/(VIST-CPC-1202): Implement Get All Visit For Customer Side

JIRA: link to jira ticket
Context:
Implementing a get all visits for the customer side.
Does this PR change the .vscode folder in petclinic-frontend?:
It doesn't change it.
Changes
What are the various changes and what other modules do those changes affect.

Created a "visits" section in the navbar from the customer side.
Created rhe visits page for customer
Created a table that fetches the data for the customer (their visits)

Before and After UI (Required for UI-impacting PRs)
Before:

After:

I dont understand why you used BFFApiGatewayController in your test, but other than that, I don't see any problem

I used it in order to access the right method to get the visit for the customer. In this case, I had to get the visit for the pet and then getting the visit for the customer. Basically, I used last year's method that was used in the old version.


Download issue

Hello,
Thanks for your awesome work. Now I am trying to download the dataset using the provided download.py.
However, as shown in the above figure, the size of all downloaded subsets could not exceed 5.9 GB, so I want to if there is any constraint on the server or any other things I should check.
Thanks

Hi.
This is strange, I tried again and it's okay to download full-sized data. Did your server have any restrictions for downloading big-sized files?
By the way, we have shared baiduyun link and you can also download from this way.

It seems that there may be some constraint on the server. I have downloaded other larger files, but I haven't met such problem.
It also stops at the some place.

Is there any constraint on the connection of your server? it seems always stop after downloading for around 1 hour

Thank you for the notice. We are contacting the admin for this problem. You can download it from Baiduyun during this time period.

It seems that there is indeed a time-out constraint on the NTU server, thanks for offering the Baiduyun version. But I want to know if there will be a google drive version?
It's okay if you don't plan to upload it to google drive, I am asking this just because google drive is more convenient for me.

By the way, it would also be great if you could split the dataset into more small splits on your server. Just a suggestion.

Thank you for the suggestions.
We will consider uploading it to google drive in the near future.
Yes, the NTU server does have restrictions for downloading time, we will split it into much smaller files.
Thanks again.


request

Boss, can you write a hook vm_readv?

Based on the example of hook mkdir, taking the x86 architecture as an example, you can find the __NR_process_vm_readv and its corresponding syscall code in unistd_64.h, and replace the original __NR_mkdir.


Use computer's mouse to control Android device without mirroring and without losing control of your PC (like second display)

[x] I have checked that a similar feature request does not already exist.

I found a few issues related to using scrcpy as a second display (i.e. inverting scrcpy), but that's not quite what I need.
My tablet sits next to my monitor. And sometimes I need to use it.
I want to move the mouse cursor to it as if it were a second monitor. But not translate the windows screen to it, but just let me control the tablet itself.
Similarly with the keyboard.
Scrcpy actually has everything for this, but you need to tell the system that a second monitor has been connected.

Scrcpy actually has everything for this

No, it has not:

SDL needs a focused window to capture input events
with relative mouse (HID), once the mouse is on your device, we can't know where the mouse actually is on the device (so we can't know when it reaches the border of the screen), and we can only send relative moves, not absolute positions.

SDL needs a focused window to capture input events
You can't activate focus when you move the mouse to the second display?

to tell the system that we have a second screen, we most likely need a separate video driver (judging by other programs for the second display). Theoretically, it can be used to detect cursor transfer to the second screen.

we most likely need a separate video driver

that will need a lot of works (windows indirect display driver, ...). I have a similar setup with you and i use autohotkey switch mouse foucus between pc and scrcpy, some thing like:
; Press Alt+S to switch between pc and scrcpy
!s::
{
	DetectHiddenWindows true
	; if window already exist
	if WinExist("scr_v510_otg"){
		; and it is active, hide it
		if WinActive("scr_v510_otg"){
			WinMinimize
			WinHide
		}
		; else bring it to front
		else{
			WinShow
			WinActivate
			ControlClick "x1 y1"
		}
		return
	}
	; window have not started yet
	else{
		; start adb server, if adb server already started it will skip this step
		Run("adb start-server", , "Hide")
		NewPID := ProcessWait("adb.exe", 15)
		; if adb server take more than 15s, likely it failed to start
		if not NewPID
		{
			; display error message and exit.
			MsgBox "Cannot start adb server.", "Scrcpy error"
			return
		}
		; if there is only one device, connect to it.
		Run "path\to\vbs script\V510_otg.vbs"
		if WinWait("scr_v510_otg", , 10){
			try {
				WinActivate
				; move mouse focus to scrcpy
				ControlClick "x1 y1"
			} catch Error as e {
				MsgBox "Scrcpy start failed!", "Scrcpy error"
			}
			return
		}
		MsgBox "Scrcpy start timeout", "Scrcpy error"
		return
	}
}

and then in vbs script modify some flag like this:
scrcpy --no-video --mouse-bind=bhsn:+s++ --window-width=400 --window-height=250 --keyboard=sdk --mouse=uhid --window-borderless --window-title=scr_v510_otg

@tantien11, I have it even easier now - bt mouse with trigger switching between 2.4 and bt modes. On bt tablet, on 2.4 laptop xD
But it'll be cool if in a couple of years this fr does get implemented

Integrating with input-leap (formerly Barrier) might be a simpler method to achieve this, like QEMU does (source).
Basically, it connects to an input-leap server and announce a display. The user can configure the display's virtual location in input-leap control panel. When mouse is moved to the configured edge, input-leap server will send input events (mouse, keyboard, clipboard, file drag and drop) to us.
Problem is input-leap has no documentation.
Absolute mouse input is possible by emulating a touch screen or stylus (the coordinates are percentage based, so still need to know screen resolution (and maybe orientation)). I guess pointer capture (in Android apps) won't work in this mode.

I quickly explored this idea, in TypeScript: https://github.com/yume-chan/leap-scrcpy
It works, except Android doesn't draw cursor for stylus, except except, it does when --show-touches is specified (so it can also be a workaround for #44).
One thing I haven't done is to retrieve the screen orientation and transform the coordinates. Stylus coordinates are always based on the natural (vertical for my phone) orientation.
https://github.com/user-attachments/assets/2c67e85a-d54a-44e3-ba4d-a5286fce0667
Here I ran another instance of Scrcpy so you can see my phone's screen. It's a little bit laggy because it was connected wirelessly.


Regression on 19.0.0 vs rc16 in java interop

I'm observing some regression on 19.0.0 with interop:
https://travis-ci.com/reactiverse/es4x/builds/111547426
So with rc16 or even on OpenJDK 11 or OpenJ9 the code:
ctx.runOnContext(fn);

Where ctx is a Java object added to the engine bindings and fn is a JavaScript function. With 19.0.0 this code now fails with:
JavaObject[io.vertx.core.impl.VertxImpl@128cff9b (io.vertx.core.impl.VertxImpl)] failed due to: UnsupportedTypeException

However the same code works when not running on graalvm and downgraded to rc16. Also note that rc16 worked as expected too.

@dougxc I observed that this error only happens on java objects added to the context using bindings. Objects that are "added" using:
Java.type(...)

Are not affected.
So basically we expose VertxImpl and Verticle and that fails on the tests.

To add even more I get the bindings as:
context.getBindings("js");

And just put the objects.

@wirthi I've isolated the issue to:
final Context.Builder builder = Context.newBuilder("js")
 .allowHostAccess(hostAccess);

If I disable this and set allow all access to true the previous behaviour works. It looks that:
    hostAccess = HostAccess.newBuilder()
      .allowPublicAccess(true)
      .allowArrayAccess(true)
      .allowListAccess(true)
      .targetTypeMapping( // ... irrelevant for the bug but needed in my application

Isn't enough anymore.

Just a blind guess (considering that your code worked on rc16 and that there were some changes after rc16): does it help to add allowAllImplementations(true) into your HostAccess configuration?

allowAllImplementations(true) is not available, at least i'm pulling the SDK from maven so I can't compile the code right now.
Totally hacking my maven repo and replacing the current SDK jar with the one in: jre/lib/boot/graal-sdk.jar allows me to compile and it solves the issue.
So this means that I am now blocked from using the stable release until the SDK lands on maven. As the project builds both on graalvm and openjdk so I can't assume that the jar is present and use it as a "system" dependency.

Fixed with allowAllImplementations(true) and the new SDK just released to maven central.
https://travis-ci.com/reactiverse/es4x/builds/111714671

@pmlopes please use HostAccess.newBuilder(HostAccess.ALL).targetTypeMapping(...) in the future. We don't plan to do breaking changes, but if we need to you will be safe with this.


Fix schema extensions in umbrella applications

I found a bug in the function that finds the schemas for extension.
I was trying to extend a schema in a umbrella application and it was returning only: Schema extends xxxx, but the schema for xxxx was not found., although the schema was clearly present.
Digging through the code, I found that the function Mix.Dep.children/0 was removed since Mix v1.3. It seems that it was replaced by the Mix.Dep.cached/0, but because conform is compatible with Elixir v1.0 and up, I chose to use the Mix.Dep.loaded/1 function that maintains the compatibility.
It now finds the correct schemas and I think it might solve some issues people are having with configuration in umbrella applications.

Good find! Thanks!


Research bitcoin.conf configuration within GUI

Many bitcoin core users configure bitcoin.conf file to customise their node to their needs. However,

there’s no official documentation describing the file’s settings,
there’s no way to access nor configure the file within the GUI,
users use third party tools to configure that (but they also seem lacking for some and are not fully descriptive regarding the features)

Goal of the research would be to find out if it's worth implementing some part of the configuration within the GUI, and If yes, what's the useful way to do it
I came up with a research draft HERE
Related issues: https://github.com/BitcoinDesign/Bitcoin-Core-App/issues/52

Part of this research might also lead to improving the documentation and/or Lopp configurator?

@GBKS that was my reasoning too

Just poked around a bit and the Loop configurator breaks things down in these categories:

Bitcoin Core
Debugging & testing
Chain selection
Mining
Networking
Transaction relay
RPC API
Wallet
ZeroMQ

If I ask ChatGPT about why people might edit this file, it comes up with these categories (some of this might be hallucination):

RPC configuration
Network configuration
Security settings
Database and storage settings
Logging configuration
Mining configuration
Node connectivity
Memory usage
Fee and transaction settings
Tor integration

Just does not seem like a casual user should have to touch 99% of these and this is more into hobbyist, pro, dev land. The casuals probably need some of the wallet and fee settings, maybe RPC so they can connect other applications, and select settings related to performance and bandwidth that we can bundle and recommend (e.g. a simple low, medium, high bandwidth usage selector. I would be curious if there are some that 50% of people who edit this touch, and some that 0.001% touch, to get a feel for priority.


feat(flutterfire_ui): add autofillhints

Description
This PR adds autofill hints to the email / password and OTP fields.
Related Issues
Replace this paragraph with a list of issues related to this PR from the issue database. Indicate, which of these issues are resolved or fixed by this PR. Note that you'll have to prefix the issue numbers with flutter/flutter#.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[X] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[X] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[X] All existing and new tests are passing.
[X] I updated/added relevant documentation (doc comments with ///).
[X] The analyzer (melos run analyze) does not report any problems on my PR.
[X] I read and followed the Flutter Style Guide.
[X] I signed the CLA.
[X] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[X] No, this is not a breaking change.

Hey, is there anything I can do to help merge this PR? :)


[Monaco] Use syntax highlighting for .srt

Description of the new feature / enhancement
Use Monaco Monarch for syntax highlighting in SubRip files.
Scenario when this would be used?
Previewing srt files
Supporting information
No response

@PesBandi
Go for it.


Is it possible to calculate the Dice loss ?

Hi, I would like to calculate the Dice loss, instead of mAP ? How can I do that ? Please help me, thank you very much.

Hi, we do not support it, but you could make a PR to add it; This is how we compute the mAP, https://github.com/fizyr/keras-maskrcnn/blob/master/keras_maskrcnn/utils/eval.py, to start you could try to modify this to get the Dice loss


Uncaught Error: only one instance of babel-polyfill is allowed

If including https://cdn.jsdelivr.net/particle-api-js/5/particle.min.js on a site that already has executed "require('babel-polyfill');", the script won't load and the API is not available.
The issue has been raised over at babel a few times (e.g. https://github.com/babel/babel/issues/4019), but the issue seems to still exist.
A quick solution would likely be to do
if (! window._babelPolyfill) {
require("babel-polyfill");
}
in Particle.js to avoid requiring babel twice.

This is already fixed in a later version. See https://github.com/particle-iot/particle-api-js/blob/master/README.md#jsdelivr-cdn for instructions on how to include the later version through cdnjs


[Bug]: Unable to send test email through Email settings [Admin Settings] if toggle is off .. An error is thrown

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When I click on test email with the Enable TLS Protected Connection toggle off, it throws an error.
Steps To Reproduce
Login as instance admin to release [CE or EE]
Navigate to Admin Settings, and click on the Email section
Here, enter all the details for email setup.
Ensure that the Enable TLS Protected Connection toggle is turned off.
Click on Test Email.
Note that an error is thrown.

If the Toggle is set to on - no error is thrown. Email gets sent successfully.
This is misleading, as no where does it indicate that the toggle should be mandatorily set on.
Note: Till the previous release, the user name and password field would only be enabled if the toggle was on. However, there was a change introduced in this PR to anyway have the fields visible and editable, whatever the state of the toggle.
Public Sample App
No response
Environment
Release
Issue video log
No response
Version
Cloud

Fixed.


Create Inteface Demo

Create a demo app that shows how interfaces can be used to handle line functions, such as filling initial line information and updating information when the user makes changes.
The easiest way is to mimic a concept for sales lines, but greatly simplified, which can fill a line based on line type via an interface.

The app has been completed and pushed to GitHub


Upgrade to Hibernate ORM 5.6.8.Final

Changelog:

https://hibernate.atlassian.net/issues/?jql=project %3D 10031 AND fixVersion %3D 32056 ORDER BY priority DESC%2C key ASC

In particular see https://hibernate.atlassian.net/browse/HHH-15147 : metamodel gen should work fine on the examples now even in "Jakarta mode"

merged it :)
@DavideD when you next rebase the Jakarta branch, you can remove the commit that disables the examples.


Add CreateReactionsAsync method

Added the CreateReactionsAsync  method which takes IEnumerable<DiscordEmoji> to put as reactions onto the message.

Unless this is a thing in API, nope.


Remove 'K' mapping

This is now built into Neovim: https://neovim.io/doc/user/various.html#K-lsp-default

FYI, this is included in https://github.com/nvim-lua/kickstart.nvim/pull/936

That's been merged. Can this be closed?

Looks like this is no longer relevant. CLosing. Please feel free to reopen if I'm in error.


Create user manual

Acceptance criteria:

[x]  The user manual has step-by-step explanation of how to use the CarEship Android app.
[x] The language in the user manual is passive, clear and concise, with accordance to the conventions of technical writing.

User manual is now completed, and can be found here


Perhaps one benchmark fixed at 1000 generated structures and another fixed at 10000 generated structures

MPTSMetrics10, MPTSMetrics100, MPTSMetrics1000, and MPTSMetrics10000


Koji add-on for Indy

retest this please

This was successful: http://ci.commonjava.org:8080/job/indy-PRs/97
However, because of connectivity issues the Jenkins server couldn't post a comment here.


Corrigir tradução de "TesteFile" para "TestFile"

Summary
Fix translation to English of console message when creating a test file.
What is happening
SUCCESS: lib/app/app/modules/loan/loan_module.dart modified
File test test/app/app/modules/loan/loan_controller_test.dart created
TesteFile: 'lib/app/app/modules/loan/loan_module.dart'
What should happen
SUCCESS: lib/app/app/modules/loan/loan_module.dart modified
File test test/app/app/modules/loan/loan_controller_test.dart created
TestFile: 'lib/app/app/modules/loan/loan_module.dart'

PR já submetido com correção, aguardando análise/aprovação: https://github.com/Flutterando/slidy/pull/173

Thanks!!!


Milestone for prototype

@RhodesPeter @skibinska
I've created a prototype milestone for this project. At some point today could you add all your closed and open issues that you have been or will be working on for thursday's deadline? Just go into an issue and click on the milestone tab to the right to add it to it like so:

Having the issues grouped under the milestone will help us review what we accomplished during sprint-review. 👍

I am closing this issue as we have completed the prototype and pitched the MPV. We will make two new milestones for the sprints.


wip: Adding Context aware APIs to the client

This follows from the from idea from @colinsullivan1 at https://github.com/nats-io/go-nats/issues/221#issuecomment-288129861 and needs another pass but opening wip PR already for early comments.
Currently it supports the following 2 APIs (plus an extra one):
func (nc *Conn) RequestWithContext(ctx context.Context, subj string, data []byte) (*Msg, error)
func (s *Subscription) NextMsgWithContext(ctx context.Context) (*Msg, error)
+ func (s *Subscription) SetContext(ctx context.Context)

Since "context" package is supported only after Go1.7, I added a +build go1.7 tag just in case so that it does not break if using older versions of the runtime plus a small cntxt interface as part of the Subscription state which also implements Context interface, this way in those clients it just does not get used unless using more recent versions of Go.

Coverage decreased (-0.7%) to 94.435% when pulling 02986f8b4b9977325eacf1de87aa326f3d017f43 on wallyqs:context-support into 6949c8e06a246e4177961aab22940b5c411e48f0 on nats-io:master.

Closing in favor of https://github.com/nats-io/go-nats/pull/275


Can't add news item

I'm getting the following errors:
Exception Type: AttributeError at /admin/website/news/add/
Exception Value: 'datetime.date' object has no attribute 'utcoffset'

Template Error:
Template error:
In template /usr/local/lib/python3.7/site-packages/django/contrib/admin/templates/admin/change_form.html, error at line 33
   'datetime.date' object has no attribute 'utcoffset'   23 : {% endblock %}
   24 : {% endif %}
   25 :
   26 : {% block content %}

   27 : {% block object-tools %}
   28 : {% if change %}{% if not is_popup %}
   29 :

   30 :     {% block object-tools-items %}
   31 :

   32 :         {% url opts|admin_urlname:'history' original.pk|admin_urlquote as history_url %}
   33 :         {% trans "History" %}
   34 :

   35 :     {% if has_absolute_url %}
{% trans "View on site" %}
{% endif %}
   36 :     {% endblock %}
   37 :

   38 : {% endif %}{% endif %}
   39 : {% endblock %}
   40 : <form {% if has_file_field %}enctype="multipart/form-data" {% endif %}action="{{ form_url }}" method="post" id="{{ opts.model_name }}_form" novalidate>{% csrf_token %}{% block form_top %}{% endblock %}
   41 :

   42 : {% if is_popup %}<input type="hidden" name="{{ is_popup_var }}" value="1" />{% endif %}
   43 : {% if to_field %}<input type="hidden" name="{{ to_field_var }}" value="{{ to_field }}" />{% endif %}

I did a poor job of testing this locally. :( I wanted to get it onto ml-test so I could use real data to test.
@johnsonkuang, can you fix this?

Yes, definitely something is wrong. Recall that this PR updated models.py and thus requires makemigrations and migrate to work... not sure if that is being done correctly on UW servers. Remember how we've often had to manually edit PostgreSQL tables/fields :(

It may be that the change from the datefield to the datetimefield broke the objects in the database. Im not sure about that theory though. Will look at it over the weekend

@johnsonkuang Some todos from when we talked about earlier today:

Write a command to convert existing datetime --> datetimefield (this can be done in website > management > commands). You can use this command by running docker run -ti -v ${pwd}/db:/code/db -v ${pwd}/media:/code/media --entrypoint=python [tag] manage.py [command name].

I excluded the migrations folder earlier so I don't think you'll be able to manually edit those.

Test by checking out an old commit (or branch!), then adding some news items. Afterwards, you can go back to your current branch, run the commands, and see if the site works

But shouldn't makemigrations do this automatically for us (convert datetime
-> datetimefield?)? Or does it not have that kind of power?
On Fri, Jul 13, 2018 at 4:20 PM, Aileen Zeng<EMAIL_ADDRESS>wrote:

@johnsonkuang https://github.com/johnsonkuang Some todos from when we
talked about earlier today:

Write a command to convert existing datetime --> datetimefield (this
can be done in website > management > commands). You can use this
command by running docker run -ti -v ${pwd}/db:/code/db -v
${pwd}/media:/code/media --entrypoint=python [tag] manage.py [command name].

I excluded the migrations folder earlier so I don't think you'll
be able to manually edit those.

Test by checking out an old commit (or branch!), then adding some
news items. Afterwards, you can go back to your current branch, run the
commands, and see if the site works

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jonfroehlich/makeabilitylabwebsite/issues/442#issuecomment-404977267,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABi-9Vao4kY0c5GyBoRI1_DHG4S4Jwaaks5uGSsggaJpZM4VPkQx
.

--
Jon Froehlich
Associate Professor
Paul G. Allen School of Computer Science & Engineering
University of Washington
http://www.cs.umd.edu/~jonf/
http://makeabilitylab.io
@jonfroehlich https://twitter.com/jonfroehlich - Twitter

It got cleared to None when we tested it earlier

After fiddling around with the models and the commands, I think it would be best if we just reverted back to using the Date field, as it currently stands that there is no way of making a data migration as the old date simply becomes overwritten and converted to None before the command can get to it. As such, I think it would be wiser to just revert back and roll back this feature. We would just lose a visual display setting, instead of "X hours ago" if something was posted within 24 hours, it will simply display the date.

OK, I think that's a reasonable compromise. @johnsonkuang, I'd also like you to open up a new ticket to convert the Date field to a DateTime field for the future as ultimately this is the functionality we want. However, as now, no reason to delay the website because of this.

Fixed by reverting back to old field type.


Get item properties from API

Item detail pages show properties of certain kinds of items (armor, weapons, etc.). However, these properties are currently taken from a static .json file (/data/item-props.json). Now that this information is available via the API, the site should be updated to request this information from the API.
The first step would be adding the properties to do-fetch-items.js, something like so:
properties {
    __typename
    ...on ItemPropertiesWeapon {
        caliber
        ergonomics
        recoilVertical
        recoilHorizontal
        repairCost
        fireRate
        effectiveDistance
        sightingRange
        maxDurability
        fireModes
    }
    ...on ItemPropertiesArmor {
        class
        durability
        repairCost
        speedPenalty
        turnPenalty
        ergoPenalty
        zones
        material {
            name
        }
    }
    ...on ItemPropertiesChestRig {
        class
        durability
        repairCost
        speedPenalty
        turnPenalty
        ergoPenalty
        zones
        material {
            name
        }
        capacity
        pouches {
            width
            height
        }
    }
}

Then the page render code would be modified to use these properties instead of the json properties.

This same method of getting item properties could be used in #61 to get all the ammo attributes as well. Adding it all to the main items query would obviate the need to do a separate ammo query.


Optimize memory usage by reducing the size of ReceiveTableEntry

Problem
ReceiveTableEntry is 96 bytes (on both 32-bit and 64-bit). That's where most of the memory usage here is coming from. This breaks down like so, afaict:
48 bytes PacketHeader
20 bytes PayloadHeader
4 byte hole due to SecureSessionHandle alignment
16 bytes SecureSessionHandle
8 bytes PacketBufferHandle (64-bit) or 4 bytes 32-bit
4 bytes hole for alignment, 32-bit only, presumably ARM-only.
But... why are we storing all that stuff in ReceiveTableEntry? We should be able to recover the PacketHeader and PayloadHeader from the packet buffer; that's where we got them from to start with, no? So all we really need to store here is the 24 bytes of SecureSessionHandle + PacketBufferHandle (+ alignment bits on 32-bit ARM).
The following is the total save with this change on ESP32:
Total sizes:
DRAM .data size:   0 bytes
DRAM .bss  size:   -736 bytes
Used static DRAM:   -736 bytes (  62596 available, 49.8% used)
Used static IRAM:   0 bytes (  14405 available, 89.0% used)
Flash code:   -175 bytes
Flash rodata:   -48 bytes
Please note, you can not see the full change from CI report until we enable exchange layer with #5627

Summary of Changes
Clean up unnecessary items from ReceiveTableEntry to save memory

Fixes #5847

@saurabhst @msandstedt ?


state store and other fixes

Add a single -store-url option to specify the statestore for netplugin/netmaster
migrate to new etc client
add full support consul in objdb
simplify state store API and eliminate netplugin config file
Move away from relying on etcd's client URL for local ip address
separate out VTEP interface from control interface
cleanup unused godep files
reconstruct endpoint db when netmaster restarts.
pull in fix from PR #289 & #322.
misc fixes in system test

I'm going to merge this since sanity passed


Create Jumpbox VM for the CDC Subscription (Dev Resource Group)

I want to create a Jumpbox VM, so that developers can access private Azure resources in FHIR solution

IT Service Desk Ticket RITM0110617 opened requesting Windows VM in CDC FHIR Dev resource group


cargo-fmt in CI and format code

This PR adds cargo-fmt in the actions workflows and formats the code, so the lint succeeds.
cargo-fmt configuration is set to default.

bors merge

Ah yes, the setting was not yet disabled by @philippkeller . Waiting for them to resolve this.

bors retry


Call function locally using call

This updates the calling convention so that f(...) now calls a function locally, i.e. basically like a Python function call.
Up until 2022-12-05, it would call the function remotely but that was then deprecated in favor of f.call and the warning was increased to an error on 2023-01-23 in https://github.com/modal-labs/modal-client/pull/230
Also

Removed old deferred function calling methods .enqueue and .submit.
Saw an opportunity to not do from datetime import ... at all

This is a similar interface to Ray's, where f(...) is a local call and f.remote(...) returns a future.
This is clear to me. UX-wise, it is nice to do an additional operation (i.e. call f.call(...)) if I want things executed remotely. With that said, the verb call isn't clear. Calling it f.remote(...) would make it a copy of Ray's terminology ... but it does make sense?! Thoughts?

Yeah I tend to agree that maybe .remote would make more sense? There is a lot of discussion in https://github.com/modal-labs/modal-client/pull/97 about terminology. These things are possible to change (although require 2-3 months of deprecation). I'm fine opening up the discussion again. However let's keep that separate from this PR


Add Georgian university

Add IAKOB GOGEBASHVILI TELAVI STATE UNIVERSITY

URL of the University: http://tesau.edu.ge

@beqa1991 Pull request merged. Thank you!


Include bdgenomics.adam-0.23.0_SNAPSHOT-py2.7.egg in distribution tarball

I don't believe pyadam will work out of the box after downloading and decompressing the distribution tarball.

Going off of this @heuermh , what is the best way to use ADAM python APIs in an external project? https://github.com/bigdatagenomics/mango/pull/307 depends on the ADAM python apis, but it is not accessible through pip. Is there a plan to add these to pip, or is there an alternative way in which we can access adam python in an alternative project?

I don't know.  Who are our go-to python experts?  ;) @pbashyal-nmdp may also be interested.

I can definitely set up a PyPl account and push it so it is accessible by pip (see here: http://peterdowns.com/posts/first-time-with-pypi.html) The process seems quite painless. The only issue with putting this library on pip is that is it relatively useless without the jars. Should we put it up there anyways @heuermh and @fnothaft ?

How do you have your packaging set up? I assume you're using a virtualenv? We will push to pip at the release. In the intermediate, once you've activated your virtualenv, you can cd into adam-python and run make develop, which will install the bindings into the virtualenv.

WRT pip, PySpark is pushed to pip and has some fancy logic that moves the Spark jars into a python package. We need to play around with that a bit, but it is doable.

Yes, I have been cding into adam-python and running pip install and that is working for testing. I am just trying to figure the best way to handle things when we merge into the Mango master branch. We don't need the jars because those are built into the mango jar, we just need the python APIs.

Ah, gotcha. In the PiPy config, we should be able to make it so that installing the JARs is optional, if that would help.

When I run make sdist  it produces dist/bdgenomics.adam-0.23.0-SNAPSHOT.tar.gz When I try to pip install the distribution, it fails not finding the version module.
(venv) > pip install dist/bdgenomics.adam-0.23.0-SNAPSHOT.tar.gz
Processing ./dist/bdgenomics.adam-0.23.0-SNAPSHOT.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/private/var/folders/7l/5w7957112mjb8yjf6lf13c2cycqhx4/T/pip-qQtkk4-build/setup.py", line 20, in <module>
        from version import version as adam_version
    ImportError: No module named version

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/7l/5w7957112mjb8yjf6lf13c2cycqhx4/T/pip-qQtkk4-build/

I'd like the dist/bdgenomics.adam-0.23.0-SNAPSHOT.tar.gz to be part of the release. I can then download the release, use pip install to install the python package as well as install the packages for R etc..


how can i download the file with errors laravel?

Can you please tell me I have a file with error logs that occurred before I connected the project, can I download it somehow?

I am sorry, I am not sure what you are asking?
If you added Sentry at some point all error logs before that point but also after that are still where they were and are still being written too. Sentry does not change anything about what is written to the error log so that is still working with Sentry enabled too.
Laravel usually keeps error logs in storage/logs so what you are looking for might be in there?

I am sorry, I am not sure what you are asking?
If you added Sentry at some point all error logs before that point but also after that are still where they were and are still being written too. Sentry does not change anything about what is written to the error log so that is still working with Sentry enabled too.
Laravel usually keeps error logs in storage/logs so what you are looking for might be in there?

@stayallive the files have not disappeared anywhere, they are, I mean, is it possible to load those errors from them that were previously in order to analyze.

@stayallive the files have not disappeared anywhere, they are, I mean, is it possible to load those errors from them that were previously in order to analyze.

Ahh, I see!
No that won't be possible, or useful since most extra information Sentry collects would not be in those logs. But regardless it's not possible to "import" from existing error logs.

Ahh, I see!
No that won't be possible, or useful since most extra information Sentry collects would not be in those logs. But regardless it's not possible to "import" from existing error logs.


Synapse Accesscontrol Readme issues

Section link:

Suggestion:
1 Add [jdk]: https://docs.microsoft.com/en-us/azure/developer/java/fundamentals/?view=azure-java-stable
2 Add [azure_sub] : https://azure.microsoft.com/en-us/free/
@scottaddie , @joshfree , @jameszliao-msft , @lmazuel  for notification.

Close this issue, due to PR https://github.com/Azure/azure-sdk-for-java/pull/20054 merged.


Update with @GetMapping

I have read in updates, @GetMapping and @PostMapping added. I think it is good if you can update the source and guide with this and any related updates too.

Superceded by #54


Changed after_destroy to before_destroy for cleaning derivatives

Fixes #762
@projecthydra-labs/hyrax-code-reviewers

I'm assuming that the spec fails if you switch the callback to after_destroy? Is that correct?

@jeremyf Yes, the spec fails for after_destroy.

Works for me, @tap87. Any further comment on changing the timing of when derivs are destroyed, @projecthydra-labs/hyrax-code-reviewers ?

Thanks @jeremyf @mjgiarlo I updated the spec to use hyrax my_works_path instead of the hardcoded value.


[constraint-layout] Bump package to 2.0.0-alpha2 and add motionlayout fixup

Support Libraries Version (eg: 23.3.0)
same as master, only constraint-layout is updated
Does this change any of the generated binding API's?
Yes
Describe your contribution
Generate a constraint-layout nuget referencing the current alpha of ConstraintLayout which also brings a preview of MotionLayout.

@Redth fyi we were thinking of using this to demo basic ConstraintLayout support in the designer (via MotionLayout specifically) during the dev16 P2 timeframe. Still plenty of time to get around to it but I figured I would mention it :-) .

@garuma yup we've actually moved these bindings here: https://github.com/xamarin/XamarinComponents/tree/master/Android/ConstraintLayout but @newky2k will be working on a PR there today or tomorrow with your changes.  Thanks :)  Should have this out shortly!

Closing as we are tracking in https://github.com/xamarin/XamarinComponents/issues/484


Fix typo in Content.jsx

One small change to fix simple typo in types.Content.jsx

Hey thanks for the PR, but looks like the code changed.   hope it didn't break anything of yours.   Content is still a bit, err in flux.     The subschema-g10n module is forcing a few hopefully non-breaking changes...

All good here thanks for asking, it just jumped out at me as I was trying to understand how everything works. This is a really cool library, if I spot anything more useful I'll raise an issue / pull request.
I'm very interested to test how the g10n works, I went for doing it on the server for ease.

The G10n plan - doing some thinking here - bear with me.
The idea write your schema in the default locale, but include it with the subschema-g10n-loader.   That will do some transforms to the schema (adding keys to g10n) and extract the content to localization files.   Then when the locale changes it will read the localization files and update the schema.
It parses ICU so you get pluralization, number formatting, date formatting, currency and so on.  In a format professional translators understand.
My big worry right now is the ICU is parsed on the client (and the server in isomorphic mode).  But I would rather store the intermediate representation.  So I may make the webpack loader do that too.
I am thinking it will have a batch mode that will just convert all schema, and allow for concat to work... So you don't have to recompile all of your app for each locale, just concat the localized schema to the app and should be good to go.   It would create 1 app file per locale.    My work project has ~250 countries to support, so I need the localization to go fast.  The way it is now, takes an hour to generate all the files, and with every new file its taking minutes longer.  I am hoping to get it to < 30s.
Meeting with the localization team tomorrow to make sure they are kosher with this, if so I hope to have g10n out a bit afterwards.


Fix error when process function returns false

Check if the destination file exists before calling timestamp sync

When does this fail, exactly?

I'm afraid I cannot recall the exact situation right now (meaning I cannot send test code). It was something related to having a custom process function in grunt-contrib-copy, where the copy operation failed when there was no corresponding folder to the one being copied.

Seems weird to me. But if it can fail, maybe we should have that in ` a try/catch instead.
/CC @shama.

@ErjanGavalji Do you have any code or example that reproduces the issue?


Skeleton Effect Guide

Is your feature request related to a problem? Please describe.
I saw almost every clone of this repo it seems like very less people applied the skeleton effect to their project.
Describe the solution you'd like.
using skeleton we can attract more users because before our data is loaded it will show a temporary template.
hence I created a skeleton effect template using react js and I fetched data from random data API (await API).
Describe alternatives you've considered.
No response
Add any other context or screenshots about the feature request here.

assign this to me @gurjeetsinghvirdee @Rayman-Sodhi

Assigned u @Dharmesh177
Please deploy your project also


Doesn't cause problems during a regional.

As stated in the title, the robot made it through a whole regional without crashing, setpoints slipping, or any other similar programming problems.
This is a difficult issue to observe; it doesn't happen very often. I hope it can be reproduced and debugged in the upcoming Idaho regional.
(Nice job, Nik. ;))

This issue is occurring only in an internal development release and will be updated to "Feature" status upon release.

See https://github.com/DenfeldRobotics4009/2016_Freckles/releases/tag/1.0.1