refactor: back rfl tactic only via apply_rfl

building upon #3714, this implements the second half of #3302.

(Likely needs a stage0 update after 846300038f0e692fe08ea83d410f96c233624489)

I am a bit confused now. After a stage0 update, so that
@[builtin_tactic Lean.Parser.Tactic.applyRfl] def evalApplyRfl

kicks in I would have expected that with
syntax (name := applyRfl) "rfl" : tactic

the new code is used reliably.
Nevertheless
theorem add_le_iff_le_sub (a b c : Int) : a + b ≤ c ↔ a ≤ c - b := by
  conv =>
    lhs
    rw [← Int.add_zero c, ← Int.sub_self (-b), Int.sub_eq_add_neg, ← Int.add_assoc, Int.neg_neg,
      Int.add_le_add_iff_right]
  -- rfl here works, so why does the rfl within `conv` no work?

fails with
unsolved goals
abc: Int
⊢ a ≤ c + -b ↔ a ≤ c - b

It seems to me that somehow the
   evalTactic (← `(tactic| try rfl))

in
@[builtin_tactic Lean.Parser.Tactic.Conv.conv] def evalConv : Tactic := fun stx => do

is broken.
Maybe I can work around that, do another stage0 update, and then it works…

I still see mathlib failures
 Error: ./././Mathlib/Analysis/NormedSpace/Star/ContinuousFunctionalCalculus/Instances.lean:223:35: error: type mismatch
  HEq.rfl
has type
  HEq ?m.240013 ?m.240013 : Prop
but is expected to have type
  IsSelfAdjoint 2 : Prop

which I did not expect. It seems that the macro_rules are duplicated in std4, proposing a fix at https://github.com/leanprover/std4/pull/702.

Now that’s a perf benchmark with lots of changes for a change:
http://speed.lean-fro.org/mathlib4/compare/74d6d12b-3209-44e1-b7d2-7265a9de7635/to/ef55fa22-f97d-4d12-8940-a2430e8a147d
Many modules benefit, a few regress, one badly (+213.3B instructions). Overall good (Build instructions -1.823 %). I have a hunch where we are doing extra work now and an idea of how to improve.


error: java.lang.Double cannot be cast to java.lang.String

node: 12.16.1
react native: 0.63.5
react-native-screenshot-prevent: 1.12.0
when we use the method "RNScreenshotPrevent.addListener", It called the error(' java.lang.Double cannot be cast to java.lang.String')
How can I fixed it?

Can you show what you are doing inside "RNScreenshotPrevent.addListener"?


Release updates for 3.0.3

Make the necessary updates to build 3.0.3

Codecov Report

Merging #469 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #469   +/-   ##
=======================================
  Coverage   60.12%   60.12%
=======================================
  Files          49       49
  Lines        3155     3155
=======================================
  Hits         1897     1897
  Misses       1258     1258

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 4fde829...0858f2c. Read the comment docs.


Migrate away from PermissionUtils, towards SessionContext

Problem statement
Now that we have a SessionContext to parse auth and provide permissions to the application, we need to refactor a few key areas that are still using permissionCheck. The goal is to migrate off of the old permission parsing utils and rely on the okta token being parsed by the SessionContext as our means of making sense of permissions.
What you need to know
In the app are comments labeled PERMISSIONS REFACTOR; search that and you'll see notes left for this work. The three main notes are:

App.tsx is handling redirects through permissionCheck and they should be determined by the active memberType of the user logging in. By default, the login action sets this to the first org in a user's array, so if I'm in 3 orgs, my first org in the claims list dictates that redirect. We do not currently support non-admins changing active membership/user-type.
AuthorizedRoute is handling permission parsing with permissionCheck and should instead handle it using the memberships provided by SessionContext
ReportStreamHeder is showing the admin org selector based off of permissionsCheck. This should NOT be coupled to our active memberType, but since permissionCheck is likely going away, a simple check to ensure someone's token has PrimeAdmins should do for "hard checking" admin access.

Acceptance criteria

[ ] We can refactor out the PermissionUtils file without sacrificing functionality.

To do

[ ]

@kevinhaube - think we can close this. Can you confirm?

Yep! We fully rely on session membership state now 👍🏼


Update Colloid potential to work with HOOMD4.

@mphoward @astatt this code is ready for y'all to take a look at! The documentation will need cleaned up, but the actual potential is working and recreates the old tests!

Looks good! I have some minor comments, then it is good to go!

@mphoward, I got all the changes implemented. This is ready for you again!


Formatting does not apply to text in check box answers

Is your feature request related to a problem? Please describe.
Some check box answers can be rather large. Formatting can help break up the text and highlight key information.
Describe the solution you'd like.
I would like to be able to use text formatting options in the text of checkbox or radio button answers. Is this possible?
Example: Admin ID: Decom_Options
What priority is this?
A clear description of whether this is high, medium, or low priority for you and why (eg, it is majorly blocking a user group, it has been requested several times, etc).
Associated civic entity
Seattle
User Group
If this request or piece of feedback comes from a resident, add the label 'Resident feedback'.
If this request or piece of feedback comes from a CBO or TI, add the label 'TI'.
If this request or piece of feedback comes from an admin, add the label 'Program Feedback'.
Additional Context
Add any other context or screenshots about the feature request here.
Done when:
What conditions need to be met before this issue can be closed?

@Miguel-314 can you include some screenshots of what you would like to format?

@rockycodes here is an example of a current question on our Weatherization Application. These program requires applicants to select one of the following options to convert their oil heat to electric heat. The options need to clearly explain to the applicant the legal ramifications of each option. It would be great if we could use, page breaks, bold or italics to help applicants identify key aspects of each option.
Here is an example of a legal statement in the same form. We are using some text formatting here but I think this is another area page breaks would be beneficial.

Thank you! That makes sense. Providing page breaks should be a relatively simple change and I'll look into allowing formatting for question options as well.

[like]  Jimenez, Miguel reacted to your message:

From: rocky @.>
Sent: Tuesday, February 20, 2024 10:48:19 PM
To: civiform/civiform @.>
Cc: Jimenez, Miguel @.>; Mention @.>
Subject: Re: [civiform/civiform] Formatting does not apply to text in check box answers (Issue #6648)
CAUTION: External Email
Thank you! That makes sense. Providing page breaks should be a relatively simple change and I'll look into allowing formatting for question options as well.
—
Reply to this email directly, view it on GitHubhttps://protect2.fireeye.com/v1/url?k=31323334-50bba2bf-31321b84-4544474f5631-a7905c7b74cbcd66&q=1&e=e8db46b3-a312-4eac-955d-b29cb002c49d&u=https%3A%2F%2Fgithub.com%2Fciviform%2Fciviform%2Fissues%2F6648%23issuecomment-1955247299, or unsubscribehttps://protect2.fireeye.com/v1/url?k=31323334-50bba2bf-31321b84-4544474f5631-7fc61c89b42948a4&q=1&e=e8db46b3-a312-4eac-955d-b29cb002c49d&u=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA6DPYONU46FRZK5CGKOJBF3YUUR3HAVCNFSM6AAAAABDJHNN72VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJVGI2DOMRZHE.
You are receiving this because you were mentioned.Message ID: @.***>

@rockycodes Hi Rocky, are you working this currently? Can i move it to IN PROGRESS?

Yep!

@Miguel-314 line breaks in question text should work now. I'm still working on formatting question options.

Great! Thank you!
Are there any special instructions on how to use page breaks or is it as simple as using the enter key to create lines where page breaks are needed?
https://docs.civiform.us/user-manual/civiform-admin-guide/using-markdown#urls

Nope! It's just using the enter key 😄

Perfect. Thank you!

@Miguel-314 I had to put this down for a bit, but I'm picking it back up again. Question for you -- would you like to have line breaks in the question options as well? Or just the ability to format text/add links.

I think line breaks makes sense as well. I have a new program coming online that will benefit from links in the answers too. I have a current work around but this would make it look cleaner. Thank you!

Ok great! I'm going to open a separate issue for line breaks, since it will require a bit more work. But links will work in the fix I'm working on for this issue.

@rockycodes I think I might have found a bug. I am working to format a question with a bullet list. Here is the result I am getting.

@Miguel-314 that definitely seems like a bug. I'll look into it.

This issue will close with the PR I'm about to merge, so I made a new bug issue for lists: https://github.com/civiform/civiform/issues/7240


Console friendly symbols

Could you replace the ✅, ❌, and ⚠ symbol with simple characters (e.g. O for ✅, X for ❌, S for ⚠) when the command is run from ASF directly? Current version will show ⍰ character instead.

Putty:

Windows (Conemu):

Windows (Windows termianl)

Thanks for the suggestion.


Add address to bottom popup

Hardcoded Santa Barbara address to the bottom popup on the main page

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 46.817%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
326

Relevant Lines:
575

💛  - Coveralls

Just realized this is not possible due to the way we have our locations database set up.


Import Ethernet-over-USB app

https://github.com/fobnail/fobnail/pull/2 has newer firmware version, closing.


I'm unable to run dashboard

npm install
npm start
There might be a problem with the project dependency tree.
It is likely not a bug in Create React App, but something you need to fix locally.
The react-scripts package provided by Create React App requires a dependency:
"eslint": "^6.6.0"
Don't try to install it manually: your package manager does it automatically.
However, a different version of eslint was detected higher up in the tree:
/Videos/react-material-dashboard-master/node_modules/eslint (version: 5.16.0)
Manually installing incompatible versions is known to cause hard-to-debug issues.
If you would prefer to ignore this check, add SKIP_PREFLIGHT_CHECK=true to an .env file in your project.
That will permanently disable this message but you might encounter other issues.

Me too, same issue

update the eslint to
"eslint": "^6.6.0"
And now it will run on browser correclty

Just Remove The "eslint": "5.16.0",  from package.json in devDependencies then it will fix.

Hi there, we just released an updated version. This should now be fixed.


update to description

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azuresmps-4.0.0/Azure/Set-AzureVMAccessExtension.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Refactor IteratorAdaptor::fold: Replace use of unwrap with InplaceUpdatable

Also add new struct utility::InplaceUpdatable:
Implements associated methods for safely replacing a &mut value without
any unsafe fn and doesn't pose any UB under panic unwind.

Nice! Looks very good! Thanks!


YouTube video's aren't displayed correctly on mobile

As reported on Twitter, YouTube video's aren't displayed correctly on mobile.
Example url: https://murze.be/inside-the-browsers-event-loop

iframe {
max-width: 100%;
}
should do the trick :-)

Thanks for the tip!


Trades for binance

Implement subscribing for trades in xchange-stream repository

Created a pull request #172 in xchange-stream repository


cannot use Auth0 social connectors with ionic 2

I there,
I downloaded your ionic sample. It works great on desktop browser. But I found after trying to run on either real device (my android), or emulator GenyMotion, or Ionic View it fails.
Im running an Android Samsung Note 2
Bascially, Auth0 for Ionic 2 for android fails.
What happens when you try to use social connector (live.com ) for instance on a phone, is you get redirected via phones browser to login page, you can login successfully, and page says 'Ok',
BUT..... it does not close and redirect back to the Ionic app. The App just sits there in a frozen state doing nothing because I suppose it is expecting a code to be delivered back to it, so it can say "happy days", you are now authenticated. But this does not happen
Can you please test your ionic 2 sample on an actual Android, or use an emulator, or Ionic View to see what I mean, and confirm that your sample works, and its not just me. I have tested against emulator running Samsung Galaxy S5 as well.
Cheers

And I can also confirm the same behavior using IOS phone

Same problem here. Any clues ?

Probable related: https://github.com/auth0-samples/auth0-ionic-samples/issues/49

Thanks for pointing the other thread!  However, I tried the solutions mentionned there (reinstall inappbrowser and setting sso to false). However, still not working :-(

bump, same problem

@alphakennyn updating to last version of auth0 lock didn't help?

@Amialc nope, currently on 10.11.0

Im having this problem as well. Really frustrating.

Two issues,

Using 'Ionic View'  "Social Login" is not working.
Blank space at the bottom.

Hi ! I have the same problem here. Do you know if there is a fix ?

Are you using WKWebView ? If so, try removing the plugin and trying with the older UIWebView. I just filed another issue for this exact problem.
https://github.com/auth0/auth0-cordova/issues/42

Same problem here. Can anybody sign in with an Auth0 account, i.e. not a social connection?


Teaking the way exceptions are handled

Coverage decreased (-0.1%) to 68.186% when pulling 2b5d74270a1f8bbf0ec50408dbd2a12c3dd323e0 on kevgliss:flask-upgrade into b3aa057d587715f713062e3e3dd6b84cde4e62fe on Netflix:master.


PoS NetworkManager ConnectIps Fixes

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1121
#1120  👈
#1119
feature/pos-networking-and-syncing

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

Join @AeonSw4n and the rest of your teammates on  Graphite


Shell Application

Feature Request - A functionality that allows developers to use the same commands in a shell like application.
$./cli
> foo
foo you
> bar
invalid params
> bar beer
here's your beer!

Hi @manan19, thank you for the suggestion. I think it's easy enough to compose CLI with readline (see @codegangsta doing it here https://github.com/codegangsta/hip/blob/master/hip.go#L64-L78) so I feel like we should leave this out of the core. This was also discussed as issue #44.

Let me know if you have any questions about composing this library with reading input lines though!

Thanks @jszwedko, that works fine for me.


Create a GitHub project board for M3

Create a GitHub project board for M3 and add a link to the board on our github.io site.
Milestone 2 Checklist:
https://courses.ics.hawaii.edu/ics314s23/morea/final-project/checklist-final-project-m2.html

Created M3 project board, made the project public, and assigned each team member an admin role. Link to the M3 project board is added on our github.io site: https://rio-connect.github.io/#milestone-3


CollectionView#reveal() behavior inconsistentent among platforms

Problem description
On Android, CollectionView#reveal() scrolls the minimal amount of distance possible to show the item in the view. On iOS, the item to be shown will be shown in the center of the view instead.
Expected behavior
Related to https://github.com/eclipsesource/tabris-js/issues/2112. In case #2112 cannot be implemented, the scroll behavior of CollectionView#reveal() on iOS and Android should be aligned.
Environment

Tabris.js version: nightly and 2.9.0
OS: Android and iOS

Code snippet
Run tabris serve -im dist/collectionview-scroll.js in snippets/ and execute tabris.contentView.find(tabris.CollectionView).first().reveal(40) in the interactive console.

I would argue that it follows the native patterns established on each platform. When we implement https://github.com/eclipsesource/tabris-js/issues/2112 you should have more control of the exact behavior.

Can not be fixed.


[#noissue] Added Logs to record route request

This PR introduces a new logging statement to the handleRouteRequest() method in the stream handling logic of our application. The new log statement will record the details of each request being handled, as well as the remote address from which the request originated.
By logging this information, we aim to improve the traceability and debuggability of our request handling procedures. This will be particularly beneficial in instances of unexpected behavior or failures, as it will allow us to pinpoint the exact request and source of any issues.

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

Thank you for your contribution.


Fix crater quality-of-life fix

Unfortunately my fix for reproduction commands in #1103 – introducing bash quotes – interfered with the outer JavaScript quotes, breaking the 'copy reproduction command' entirely...
This PR remedies this by alternating the quotes style. For this and all future changes, I have now got a local fontc_crater setup, and have verified the fix. The documentation for doing so was excellent, and I wish I'd checked for it the first time; sorry for the trouble!
Note: this is a personal contribution independent of my employer, and so I've submitted from a fork under my personal profile and email to make this distinction

No problem, thanks for the prompt fix


Check if rust-analyzer can parse project

What
Add ci job that checks if rust-analyzer can parse project.
Why
A couple times now we have done things that are technically allowed in Rust but are not supported by rust-analyzer. Everytime we've walked the change back because rust-analyzer compatibility is important for developer productivity. Let's prevent these things from happening by checking if rust-analyzer can parse the project.
Same as https://github.com/stellar/rs-soroban-env/pull/371.
Known Limitations
This relies on parsing the output of rust-analyzer analysis-status which is not stable, so it may change or break over time (https://github.com/rust-lang/rust-analyzer/issues/13115). We can remove this check if it doesn't live up to its hoped impact.

I just went through all the examples and remove the cyclic self-references. The changes are in this PR.
We had removed these previously in #59, but looks like they crept back into some new examples we wrote.
For anyone adding examples here (@sisuresh @jonjove?), can you take a look at the diff to see what we're doing now? Instead of adding a self-referencing dev dependency with testutils feature enabled, we add a cfg(test) everywhere as an alternative way to trigger any testutils feature gated code, that way testutils functionality is just always available in tests.


The Child failed send MAC Data Frames to its Parent when its Parent send announce message to another channel.

The Child failed send MAC Data Frames to its Parent when its Parent leaves the current channel  to send announce message to another channel.
The rloc16 of this Child is 0x2805， it consecutive unicast MAC Data Frames sent to 0x2800， but there is no Mac layer Ack.

Checked the log, the Parent is sending announce on other channels at this time. What is the maximum time when Parent leaves the current channel to send announce message to another channel? Is there a definition in the standard?
Is this kind of packet loss as expected?

Add log and wireshark packets, Network Key: 5d7838199c00a8dd596dd687ecb1b84e
5d7838199c00a8dd596dd687ecb1b84e.zip
send_failed_when_announce.log

The MLE Announce message should take on the order of tens of milliseconds - basically the time it takes for the radio to switch to a different channel, transmit a broadcast message, and return back to the main channel.
It appears that your device is only attempting to transmit a unicast message a total of 4 times (unless the ack message is getting dropped from the sniffer capture). Thread Specification requires 15 retransmissions when no ack is received and OpenThread implements that requirement.

The sleep end device in our system base on the Thread Specification 1.1.1, the macMaxFrameRetries is 3.

In OpenThread, the number of retries is controled via the build config OPENTHREAD_CONFIG_MAC_DEFAULT_MAX_FRAME_RETRIES_DIRECT. I would encourage you to increase the number of retries to 15. Even better if you can upgrade your device to Thread 1.3, which includes many other enhancements such as this.

Closing stale issue.


update for new home az contract from nma

Reason for Change:

NMA introduces a new field apiVersion in home az endpoint for the purpose of fixing the ipv6 issue in swift azr, we need to make changes on cns to correspond to this contract change.
Issue Fixed:

Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[x] adds unit tests
[ ] relevant PR labels added

Notes:

/azp run Azure Container Networking PR

Hey @timraymond , can you approve again? Thanks

/azp run Azure Container Networking PR


Cannot install dep ensure on Windows

i can't install dep on windows
go version: go version go1.10 windows/amd64
windows version:  win10 1709
download dep from https://github.com/golang/dep/releases/download/v0.4.1/dep-windows-amd64.exe
double click. no react

@silloy That's because it's not an installer but the actual dep application. Just rename the file to dep.exe and move it to your Go bin directory (typically C:\Go\bin.)

@tomc974 thanks

@tomc974 i did that but in command prompt, its telling me that dep isnt recognized as a internal or external command but i need dep for a project

@damilolakasumu You have to add the directory in which you copied dep to your PATH environment variable. Instruction to do so can be found online.

If you're using chocolatey there is a dep package: https://chocolatey.org/packages/dep
choco install dep should do the trick.

@silloy That's because it's not an installer but the actual dep application. Just rename the file to dep.exe and move it to your Go bin directory (typically C:\Go\bin.)

@silloy That's because it's not an installer but the actual dep application. Just rename the file to dep.exe and move it to your Go bin directory (typically C:\Go\bin.)

thx


Issue with exampe

I cloned this repo and the first time I upload an image, this happens (but only the first time after Meteor starts):

W20150305-14:05:17.421(-5)? (STDERR)       next();

W20150305-14:05:17.421(-5)? (STDERR)       ^

W20150305-14:05:17.422(-5)? (STDERR) TypeError: undefined is not a function

Could not reproduce. :/ did you solve it?

Not officially but I can no longer reproduce it either.


Update the OWNERS file

Rajat has left, so removing him from the file.
Signed-off-by: Phil Cameron<EMAIL_ADDRESS>

@shettyg @girishmg @dcbw @squeed @JacobTanenbaum
PTAL

@pecameron
Rajat still has committer privileges, if he decides to come back. Lets leave it as-is for a few months and then revisit.

@shettyg When Rajat left Red Hat he told me that he was not expecting to be involved with ovn. I am OK with leaving the OWNERS file as is for a while.


clean_build_artifacts doesn't remove all *.mobileprovision's

I'm using two sigh actions in my fastfile to download two provisioning profiles: one for the main app, and one for the app's share extension.
I'm on Fastlane 1.29.0.
Here's two excerpts from my fastlane log:
[16:46:16]: ------------------
[16:46:16]: --- Step: sigh ---
[16:46:16]: ------------------
[16:46:17]: Starting login with user 'email@redacted'
[16:46:19]: Successfully logged in
[16:46:19]: Fetching profiles...
[16:46:25]: Found 1 matching profile(s)
[16:46:25]: Downloading provisioning profile...
[16:46:26]: Successfully downloaded provisioning profile...
[16:46:26]: Installing provisioning profile...
./AdHoc_com.myapp.mobileprovision
[16:46:26]: ------------------
[16:46:26]: --- Step: sigh ---
[16:46:26]: ------------------
[16:46:26]: Starting login with user 'email@redacted'
[16:46:28]: Successfully logged in
[16:46:28]: Fetching profiles...
[16:46:33]: Found 1 matching profile(s)
[16:46:33]: Downloading provisioning profile...
[16:46:34]: Successfully downloaded provisioning profile...
[16:46:34]: Installing provisioning profile...
./AdHoc_com.myapp.share-extension.mobileprovision

[16:47:43]: -----------------------------------
[16:47:43]: --- Step: clean_build_artifacts ---
[16:47:43]: -----------------------------------
[16:47:43]: Cleaning up '/Users/karol/Desktop/myapp/myapp.ipa'
[16:47:43]: Cleaning up '/Users/karol/Desktop/myapp/AdHoc_com.myapp.share-extension.mobileprovision'
[16:47:43]: Cleaning up '/Users/karol/Desktop/myapp/myapp.app.dSYM.zip'
[16:47:43]: Cleaned up build artifacts 🐙

cc @lmirosevic

That's correct, right now the sigh path variable can only hold one path.

It would make sense to store multiple paths, I'll update this right now.

Done, this will be in the next release :+1:

:sparkling_heart:


extract common tags from ALB names

Right now the ALB name from cloudwatch metrics is something like app/www-stack-1234567890. Ideally we could extract the ALB name portion to make it easier for users to understand.

Fixed in #895.


[FR] Relax requirements for "Installed" items

Problem
Both Component and Assembly parts have to be Trackable to be installed in one another.
My usecase: tracking of computer parts inside computer assemblies. Every computer has: case, motherboard, hdd, etc.
I would like to create "Computer" part as assembly and all of its sub-parts as components and install them into "Computer" to know precisely where they are stored. Making them all trackable is over-complicate everything.
Suggested solution
Remove requirement for assembly to be trackable to enable install feature.
Remove requirement for components to be trackable to enable install feature.
Change Location description of installed part from "Installed in Stock Item 639" to "Installed in "Part name"".
Describe alternatives you've considered
Relax only trackable requirement.

Sounds like a simple general-purpose asset-management solution would be better suited for your use case.
I personally would not treat this with high importance but if it does not break anything else why not. @sintech would you be willing to implement this? Should be fairly easy.
What do you think @SchrodingersGat @eeintech ? yes 🚀, alternative solution 👍 or  no👎 ?

Here is how I interpret the idea: installing a number of "thing" into a number of "another thing" without specifically having a way to point which "thing" item goes to which "another thing" item.
I think most of the logic is already present in the build allocation process, but this would have to be ported over to stock items. And maybe a global "stock item allocation" makes more sense, this global allocation could let the user execute allocation either from the "Stock" (to either stock items or build orders) view or "Build" view (from stock items).
For the stock side, those would be the new relationships:

install serialized StockItem into serialized StockItem (already exists)
install generic StockItem into serialized StockItem (example: 10 screws into a single, trackable case)
install generic StockItem into generic StockItem (example: 150 screws into 15 cases)

In the second case, the item being installed should have the option to split into:

items being installed into serialized item
leftover items from installation

Now adding to this, in the third case, the item receiving the items being installed also needs to have the option to split into:

items in which first item is present/installed
leftover items still missing first item to be installed

@SchrodingersGat probably has a better idea of the allocation logic and if can be re-used in its current state.

@sintech I think what you are proposing would quickly become very complicated. The current "install into" logic is implemented in a particular way so that the BOM / assembly relationship defines what sort of parts can be logically installed into other parts.
If any part could be installed into any other part, how do you actually select the parts in the software, when you have thousands of different parts available in your database? Just scrolling through and selecting the one you want could become very tedious.
The existing logic allows you to narrow that down, so that the user does not have to search through your entire database of parts.
This is why the BOM relationship exists.  Some of these parts are "trackable" - i.e. you want these to be tracked when they are installed inside an assembly. Some are not trackable - e.g. screws? - and you don't want to track these for the lifecycle of the assembly.
Specifying the BOM and which parts you want tracked does take some time at the start of the process. However I feel strongly that the "garbage in, garbage out" adage applies here. To make the InvenTree system work well for your application, you need well curated data.
I think that relaxing the requirements here would turn into a bit of a mess. In our system we have ~1000 different parts, and a lot of these parts are extremely similar (but, importantly different). The BOM (and build order system) ensures that we know exactly which parts should go into which other parts, and the user cannot install other random parts without updating the BOM first.
Does this make sense? Have I interpreted your request accurately?

Thank you guys for comments.
@SchrodingersGat I agree about requirement for component to be in assembly BOM to be installed.
But why both assembly and component have to be trackable? How this helps with mess avoidance?
You also mentioned tedious part selection process (if any part could be installed), how is this differ from adding a line to a BOM?

@sintech I guess I missed this part of your solution:

Remove requirement for component to be a part of assembly BOM to be installed.

That I completely disagree with, and completely agree with @SchrodingersGat
For the two first points:

Remove requirement for assembly to be trackable to enable install feature.
Remove requirement for components to be trackable to enable install feature.

I think there are fair request because this is basically how allocation currently works: there is no need to have the parts be "trackable" to allocate them into a build order. And the build order can create stock items that are not trackable too. So I don't see much difference in bypassing the build order and do the same at the stock level.

@eeintech Yes, I admit that component must be a part of assembly BOM to be installed. I was a bit rash about it.
If all agree that "trackable" requirement could be removed I would like to try to implement it by myself including Location display fix.

@sintech I think that this requirement has been met now? Can I close this out? Please let me know if you think this has not actually been addressed.

@SchrodingersGat, no, part still has to be trackable to appear in "Install Stock Item" "Part*" field. And stock item should have assigned serial number to appear in "Stock Item*" field. And container part has to be trackable too.
Looks like nothing has changed in 0.6.x related to this FR.
Please reopen.

Just my 2-cents: If you want to track what parts are installed into what stock you want to track details about it -> so the trackable makes sense. Also, it ensures that there is only one part assigned to the stock item.

I agree with @matmair here - I'm not sure what a suitable alternative would look like.

I described my usecase in the first post: tracking of extension boards inside computer case or any other modules inside mother enclosure.
As far as I have only one item of each kind of sub-module I don't see any reason to make them trackable. In other case when I will have for ex. two video cards of the same model it makes sense to make them trackable. But again if they are identical does it matter where is what.
So why limit this very useful feature to only trackable parts. Users will decide what approach is better for them.

@sintech there are things happening in the business logic regarding if a part is trackable or not. So it matters that a part is marked trackable. If we rove that flag we will just have to make a new flag for that or hide a lot of parts in the selection -> the amount of work for the user is the same.
This feature only works if certain requirements are met. The limits are very intentional and I think no one will appreciate it if the system gets unreliable but the trackable flag is gone.
I am not sure if your use case makes sense for the system. Tracking single-item per model individual parts is better suited for an asset tracking system as there is a certain level of setup needed per part.

I am in agreement with @matmair here - I don't think the proposed change makes sense for InvenTree

@matmair , I'm sorry but I don't get your point. Why system will become unreliable?
It was already discussed that BOM requirement for container part is closing all issues with inner part selection. Assembly requirement doesn't allow any other parts to become container.
What happened in last 6 months that you guys have changed your opinion?
I'm not asking you to develop this if you don't like it, just don't reject it. I will do a PR eventually.

Hello @sintech
I'm sorry for your frustration, @matmair and @SchrodingersGat have been pushing forward with a lot of new features recently and it's an incredible effort given they are doing this on their free time.
The problem is, many more users means many more requests for new features... which could grow a, right now stable, software into a monster if one keeps on adding more and more things without framing the project.
I think I understood your original idea, which sounded reasonable at first, but I understand also the reasons not to implement it.
You best bet is to show them that this is feasible without adding monstrous complexity and making them uncomfortable with that new feature :smiley:

I'm not asking you to develop this if you don't like it, just don't reject it. I will do a PR eventually.

@sintech if you have a PR that works I see no reason to not look at it.

@matmair , I'm sorry but I don't get your point. Why system will become unreliable? It was already discussed that BOM requirement for container part is closing all issues with inner part selection. Assembly requirement doesn't allow any other parts to become container. What happened in last 6 months that you guys have changed your opinion?

There are architectural challenges with your requests - you will see yourself, especially when starting to add unit tests. Maybe your fresh view will find ways I could not think of - I would be glad to be proven wrong.
There are a lot of use cases for InvenTree and everything that is there should generally keep working in exactly the current way.
Things like tracking need to be reliable. Without a clear flag that enforces specific details about a part on each save() operation there could be edge cases that result in a duplication of tracking codes. If that ever makes it to master/release we might stirrup a lot of installations irreparable.
The more I discover the userbase I am less willing to take risks on security and business logic / API endpoints. We could cause quite a lot of damage.

The problem is, many more users means many more requests for new features...

And also that. Feature creep is real and causes performance problems left and right. There are installs with over 10.000 parts and a over a million stock tracking entries.
In closing: please feel free to speak out and add your view. In this case, I do not see the added value fitting the cost but that is neither final nor a set truth.

@sintech we've recently made some changes to how "installed items" work which may be sufficient for your requirements here? I'm closing out this issue for now, LMK if there are any further issues.


Update community files (#2963)

Signed-off-by: Knative Automation<EMAIL_ADDRESS>

/retest

/retest

OCF Webhook is merging this PR


[Edit secretary] requirements review

https://github.com/ita-social-projects/what-front/wiki/SRS-Edit-Secretary

https://github.com/ita-social-projects/what-front/wiki/SRS-Edit-Secretary


Fix(docs): "Edit this page" button broken link #6370

One-word change so that "Edit this page" goes to the right place, tested locally on a few pages.

Oooh, I wasn't familiar with the semantic-pull-request format. I'll change the title to lowercase fix and see if that helps.


[Feature]: add setHeaderTextColor to miniApp

Is your feature request related to a problem? Please describe.
  useEffect(() => {
    if (!miniApp || !theme) return;

    miniApp.setBgColor(theme === 'dark' ? '#18181b' : '#f4f4f5');
    miniApp.setHeaderColor('#18181b');
    /**
     * @TODO set header text color
     */
    // miniApp.setHeaderTextColor('#f4f4f5'); // section_header_text_color
  }, [
    miniApp,
    theme,
  ]);

I couldn't miniApp.setHeaderTextColor( above there.
Plz support section_header_text_color of Bot API 7.0 @December 29, 2023
Describe the solution you'd like
both for @telegram-apps/sdk-react and @telegram-apps/sdk
and I think there are more features about the MiniApp theme are supposed to be supported such as section_header_text_color, section_separator_color, subtitle_text_color and destructive_text_color etc...
Describe alternatives you've considered
N/A
Additional context
super thanks

I found useThemeParams but could just get sectionHeaderTextColor there

There is no way to implement such a functionality because the platform does not provide a corresponding method

tks for ur reply
something I found in their docs:
for section_header_text_color in ThemeParams

Optional. Bot API 7.0+ Header text color for the section in the #RRGGBB format.
Also available as the CSS variable var(--tg-theme-section-header-text-color).

so it is only possible to get but cannot set/change it?

and I cannot even find the method showAlert in sdk-react, is that possible?

so it is only possible to get but cannot set/change it?

Yes, you can't change it.
showAlert is the method from the Telegram SDK. Internally, it just uses the Telegram SDK openPopup method, so you can implement the same using @telegram-apps/sdk


Telemetry testing and fixes

Attempts to make telemetry functional in preparation for the rocket implementation, including by:
-Adding tests for current telemetry features
-Fixing a few bugs
-Adding serializability to endpoints??

What would be the reasoning behind removing packet discriminators completely?

LGTM
(sorry for late I was busy)


[BUG]: Predefined port

Describe the bug.
I faced this problem when I started the website and it failed to load since my port: 3000 was busy, next.js has ability to move start website to the next port (3001 or 3002) if default port 3000 is busy.
But in the website code, this feature is overridden by defining port to 3000 in package.json file causing errors if default port 3000 is busy.
Expected behavior
There should be no port to be defined which will enable usage of next.js port shifting capability that
Screenshots

How to Reproduce

port 3000 must be busy
run npm run dev
errors will be shown in console

🥦 Browser
Mozilla Firefox
👀 Have you checked for similar open issues?

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

🏢 Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

Are you willing to work on this issue ?
Yes I am willing to submit a PR!

@akshatnema shall I open a PR for this issue?


Propagate client secret type throughout pipeline

e.g. for adding a "client_amr" claim (e.g. shared secret vs private key JWT)

PR https://github.com/IdentityServer/IdentityServer4/pull/1367


android: update connlib dependency and integration

Replaced connlib dependency to use rust/connlib/clients/android/lib project
Added rust-android-gradle to android project
Set the cargo build target directory to rust/connlib/clients/android/lib/build/cargo-target
Moved logger, session, and vpn classes to their independent packages.
Added SessionCallback contract for the session callbacks.

@pratikvelani Looks like some CI build errors due to the restructuring.

@pratikvelani There are a couple CI changes still needed -- I'll fix these in another PR. Deferring to @francesca64 for approval.


Apply Program Year Maximums before sending eCert

Description of Task:
Student eCerts must be trimmed for each funding type such that the sum of each funding type (across all applications within a Program Year) does not exceed Program Year maximums established by policy.
Context:
Camunda calculates award amounts independantly for each application, based on the inputs provided on the application form and collected from supporting users.
However, the amount awarded on any given application may have impacts on the award amounts the student will receive on their next application within the same program year. This is a policy rule.
E.g. of Policy Rule:

BCAG has a limit of $10,000 per program year (example only, not real numbers)
Student submits app #1 for PY 23/24 and is assessed for $4,000 for BCAG. Student receives $4000
Student submits app #2 for PY 23/24 and is assessed for $4,000 for BCAG. Student receives $4000
Student submits app #3 for PY 23/24 and is assessed for $4,000 for BCAG. However, they should not receive all $4000 - they should instead be subjected to the annual PY maximums and receive only $2000.

@Signori proposed the best way to approach these calculations is at the time of eCert generation, along with all the other last-minute adjustments which are made.
E.g. of Proposed Approach:

Student submits app #3 for PY 23/24 and is assessed for $4,000 for BCAG. (student has already received $8000 BCAG that PY, and the annual max is $10,000.
Student NOA for App #3 displays a BCAG eligible award = $4000. IER field also reflects $4000.
When eCert is being finalized, a calculation is performed which sums all previous BCAG amount received during that PY, and ensures the latest 'eligible award' is reduced to comply with the annual maximum. In this case, the amount is reduced to $2000.
The student 'Assessment Summary' screen displays a 'final award' amount of $2000 for BCAG, along with some type of waring message explaining the reason for the discrepancy between 'eligible award' and 'final/actual award'.

Acceptance Criteria:

[ ] TBD based on full business identification of the funding type maximums
[ ]

Suggestion to make the changes at ECERT would mean that the assessment becomes worthless for all secondary PT applications as the entire application is built around program year limits.

Going to be addressed on #2571


Fix most of the tests

Overview
This gets us from most tests failing to most tests passing; there are about 25 left which still fail or error out.
Checklist

[X] PR has a name that won't get you publicly shamed for vagueness

Testing Instructions

./scripts/server
./scripts/test
Most of them should pass! 🎉

I decided to keep working on this since we're running low on cards, but I'm going to merge this and make another PR for the next set of fixes since some of them are a bit more involved.


API calls failing for Fedora 40, prevents use of rig add

Hi, completely new to rig so may be missing something obvious, but after some trouble with packages not working well with R 4.4 I figured it was a good time to check out rig as I've heard it's a good way to manage multiple R installations.
Unfortunately, I can't get off the ground on Fedora 40 - using rig available or rig add just return the following error (e.g. for rig add 4.3.3):
[ERROR] Failed to resolve R version: Cannot download JSON: HTTP status client error (404 Not Found) for url (https://api.r-hub.io/rversions/resolve/4.3.3/linux-fedora-40/x86_64)

I'm guessing this is because the info for Fedora 40 hasn't been added yet on the server side? If so, is there a timeline on this? I could get around the error for rig available by trying rig available --platform linux-fedora-39  but can't see an equivalent to the platform argument for rig add that would allow me to work around it. Thanks.

We don't support Fedora 40 yet.

I started adding support for Fedora 40 here: https://github.com/rstudio/r-builds/pull/217

Fedora 40 is now supported, both on x86_64 and aarch64.


Added a blue and red viz function for segmented images

This PR fixes issue #102.
I added a function that takes as input a segmented image and returns it in a red(myelin) and blue (axon) display.

@oumayb since this new feature is unrelated to the branch dev_issue73/pull request 96, it would have been cleaner to have created your branch off of master instead of dev_issue73 like you did here; branching off of  dev_issue73, we'd need to wait until that branch gets merged before being able to apply your new feature to master.
I took the liberty to create a new branch dev_issue102_cp, and cherry-picked (copied) your new feature's commit (525018c) into that branch. Can you create a pull request for that branch to be merged into master, and close this one? If you want to work on that branch a little bit more, you only need to do a git pulland then git checkout dev_issue102_cp on your local computer.

@mathieuboudreau I got your point.
Thanks for creating dev_issue102_cp, I'll create my branches off of master from now on.

@oumayb no worries – thanks!
You don't always have to make your branch off of master; if the bug fix/feature you want to make is related to another branch, branch it off of that one.
A+


Pull

A new inspection was created.

The inspection completed: 20 new issues, 44 updated code elements


No error is reported if worker fails to start up, connection stays spinning

Description
If you create a connection but it fails for some reason on the server, then it stays spinning.
Steps to reproduce

Start a connection to a DHE server where the worker fails (I set heap size to something too big, e.g. 4096gb)

Expected results

Connection should fail, error should be reported

Actual results

Connection stays in spinning state, even though on the server can see the worker failed with an error:

Versions
v0.1.14-pre release

Probably a duplicate of https://github.com/deephaven/vscode-deephaven/issues/158

Closing as duplicate


Add federation sync configuration SQL table and RPC endpoints

This PR adds functionality to configure the universe federation sync routine. It facilitates both a general configuration and a universe/asset specific configuration.
This PR is part of the work necessary to complete https://github.com/lightninglabs/taproot-assets/issues/530 .

All set/query db transactions are now atomic.
I've added an enum to signify which proofs should be synced. I think it's the cleanest solution because each sync process will either exclude proofs, sync issuance, or sync all proofs. We can also expand the enum (which comes close to Oli's key-value idea.)
Let me know what you guys think @guggero @Roasbeef

I assume the code to actually sync according to the setting will be done in a follow-up PR?

Yep plan is to do this as we work through https://github.com/lightninglabs/taproot-assets/issues/544.

Needs a rebase!


LS9 L1 preparation

[x] Add test case with a cut-down dataset.
[x] Get test case to pass
[ ] ~Update to match the newer S2 L1 preparation tools~ do in future PR

Example metadata for @dunkgray's test datasets are here: /g/data/v10/agdc/jez/c3/ls9


Assasins Creed 1 black screen issue - DX10 title

After you pass the intro cutscene , you can't see the profile creation menu. It is just black. FPS counter etc are all there but nothing rendered to screen. You can hear the main menu sound effects also.
Software information
Game: Assasins Creed 1 Director's Cut - GOG version
d3d_compiler 43 via winetricks
System information

GPU: GTX 1050 4 GB ( Nvidia Prime - Nvidia card is always active )
Driver: 396.51
Wine version: Wine 3.11 Staging ( updated to 3.13 but no difference )
DXVK version: 0.70

Apitrace file(s)

Put a link here

Log files
Game has three executables named :
1-) AssassinsCreed_Game.exe
2-) AssassinsCreed_Dx9.exe
3-) AssassinsCreed_Dx10.exe
Logs:
-d3d11.log from DX10 exec:
AssassinsCreed_Dx10_d3d11.log
-d3d10.log from DX10 exec:
Empty file so it cannot be uploaded

dxgi.log from DX10 exec:
AssassinsCreed_Dx10_dxgi.log

-d3d11.log from Game.exe:
AssassinsCreed_Game_d3d11.log
-d3d10.log from Game.exe:
AssassinsCreed_Game_d3d10.log
-dxgi.log from Game.exe:
AssassinsCreed_Game_dxgi.log

Hmm, the exact same game works for me with

fresh 64bit wine prefix
copying 32bit dxvk dlls to windows/syswow64
setting dxvk dlls to native in winecfg
winetricks d3dcompiler_43

Comparing my logs to yours, the main difference (apart of my gpu being a gtx680) is the driver version. I'm on 396.51.02, yours appear to be 396.51

I also have 64 bit fresh wineprefix
-I just used new DXVK script method
-They automically set to native in winecfg
-Installed winetricks d3dcompiler_43 , then actived it on Winecfg and set to native , builtin. Deactivating has the same result

Just updated Wine to 3.13 but nothing changed.
I'm using graphics ppa and latest version on there is 396.51.

That's odd. The game shouldn't even start with deactivated d3dcompiler_43.

It starts and works without d3dcomplier_43.
But i assume it works in DX9 mode this way.

Did you have to manually add d3dcompiler_43 to dll overrides? If yes, installing it might have gone wrong, as winetricks should add the overrides itself.

Thanks , that worked. However , before closing that issue i have noticed 2 more issues.
First:
Exit menu renders like that:

So you can't see yes or no prompts.
Second:
MSAA seems deactived on options menu.

Yes, it's the same for me. But everything else seems to work fine.

So you also have the exit issue and MSAA issue.
Maybe that is an Nvidia driver related issue.
Thanks for your help , i will close that issue.


[22.05] python3Packages.tensorflow: 2.8.1 -> 2.8.4, python3Packages.tensorflow-bin: 2.8.1 -> 2.8.4

Description of changes
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41883
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41911
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41909
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41908
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41901
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41907
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41900
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41899
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41898
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41896
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41897
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41894
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41895
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41891
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41893
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41889
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41890
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41887
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41888
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41886
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41880
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41884
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2022-41885
See #201997 for unstable.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg eval


more sense in pt-BR translations

Why: Some words were conjugated incorrectly and with wrong meanings
Closes [issue link]

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x ] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x ] For content changes, I have completed the self-review checklist.

gh pr checkout 21392

@LucasHT22 - Thanks for reaching out! We don't accept changes to our translated files as we use an internal automated solution for translations. We will flag your request for our team. :yellow_heart:
See https://github.com/github/docs/blob/main/contributing/types-of-contributions.md#earth_asia-translations.
I'm going to go ahead and close this PR. Please take a look at our help wanted section to find open issues you can work on.
Thanks for your interest in improving the docs! ✨


Unit test user model

https://github.com/sahat/hackathon-starter/issues/882

[x] Test for comparePassword
[x] Test for gravatar

Thank you !


fix: the protential SQL injection issue when users toggled the AllowRawSQLQuery feature gate on

What type of PR is this?
/kind bug
What this PR does / why we need it
As
https://github.com/clusterpedia-io/clusterpedia/blob/630cf19415ce0148bb1645c7a6f81e686d366051/pkg/storage/internalstorage/util.go#L58-L65
has stated. There are potential SQL injection threats of the current implementation. This Pull Request uses the prepared statement feature of Where(...) which GORM natively support and will pass the required parameters of whereSQL from whereSQLParameters field.
For more about the SQL injection of GORM, please read it more here: https://gorm.io/docs/security.html
I have added several commonly used SQL injection test cases to test out the prepared statement:

Alway true where clause, as known as OR 1=1
Comments based, as known as # and --
UNION based, as known as UNION NULL, NULL, NULL, NULL --

Which issue(s) this PR fixes
None. This is a known issue.
Special notes for your reviewer
Should we keep AllowRawSQLQuery to be turned off? Users may not be able to transit all the implementations while upgrade is planned, and users may not be able to process the user input correctly even when prepared statements is used.
Does this PR introduce a user-facing change?

1. `AllowRawSQLQuery` feature is now switched to `true` as default value since the current implementation enables user to supply parameters of `whereSQL` where raw SQL or fuzzy search is required and needed.
2. Added new `whereSQLParameters` for callers of Clusterpedia API to pass the parameters of raw SQL. Users can send raw queries more safely by using `?` as the placeholder of parameters (which is compatible for GORM) and include all the corresponding parameters in newly added `whereSQLParameters` field.

**Please update all the existing SQLs of raw queries to pass the parameters of raw SQL in `whereSQLParameters` field instead of string concatenation since it introduces the potential SQL injection when attackers obtained the access to Clusterpedia API.**

Disclaimer: the potentials of SQL injection will **NEVER** be resolved if implementations of caller still use string concatenation to concat the raw SQL query with parameters directly without `whereSQLParameters` when using `whereSQL`, end users are responsible for defensing and protecting the resources from malicious access at all the time.

/auto-cc


[Documentation] Adding random_split example to tutorial

Stack from ghstack:

-> #843

random_split is one of the most commonly used functionality. It will useful to include that in the tutorial such that users don't try to use torch.utils.data.random_split with IterDataPipe.

@nivekt has imported this pull request.  If you are a Meta employee, you can view this diff on Phabricator.


Type Saftey for Dragon Backend

This PR will:

Address small type errors in dragon prototype.
Promote type safety my moving type information out of instance variables.
Move serialization/de-serialization of request/response schema out of backend code and into dedicated class to prevent unexpected type coercions using pydantic.BaseModel.parse_obj.
Remove branches based on schema "instance var typenames" from backend and replaces with functools.singledispatchmethod

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Fixes #7564 - Malformed URLs in API git/commits response

Fixes #7564: Correction to two different URLs in the /git/commits response, the tree url and the HTML url. Probably should be backported to 1.9

Codecov Report

Merging #7565 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #7565      +/-   ##
==========================================
- Coverage   41.25%   41.24%   -0.01%
==========================================
  Files         469      469
  Lines       63618    63618
==========================================
- Hits        26247    26242       -5
- Misses      33948    33952       +4
- Partials     3423     3424       +1

Impacted Files
Coverage Δ

routers/api/v1/repo/commits.go
72.91% <100%> (ø)
:arrow_up:

modules/process/manager.go
76.81% <0%> (-4.35%)
:arrow_down:

modules/log/event.go
64.61% <0%> (-1.03%)
:arrow_down:

Continue to review full report at Codecov.

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


idmapd not starting on SLES 11 SP3

ERROR: service[idmap] (nfs::_idmap line 31) had an error: Mixlib::ShellOut::ShellCommandFailed: Expected process to exit with [0], but received '6'
---- Begin output of /sbin/service nfs start ----
STDOUT: Not starting NFS client services - no NFS found in /etc/fstab:
STDERR: ..unused
---- End output of /sbin/service nfs start ----
Ran /sbin/service nfs start returned 6
NFS service is not finding any NFS mount points in /etc/fstab, it consider NFS being useless, and exit with error.
However, if adding 'NFS_START_SERVICES=yes' to '/etc/sysconfig/nfs' file, service starts successfully.
This is the way how init script is written for SLES 11 SP 3.
Can you add the line, please?
To /etc/sysconfig/nfs, 'suse' section
NFS_START_SERVICES=yes
Thanks,
Vladimir

Could you add the line to the template and pull request it, please?  There isn't a public SUSE basebox I can use to quickly test this with test-kitchen.

Released in 2.1.0


Esri Requests

[x]  Thomas Fuchs at Esri noticed that some of the textures are 8 bit depth textures that don't work with SLPKs.

[x] Cheryl Lau at Esri identified a need to change some paths to update the Complete Street Example for textures.

Both of these requests will likely warrant a new version number.

Textures for asphalt were made 24 bit with https://github.com/d-wasserman/Complete_Street_Rule/commit/ed287bde7aba5e3ffa7c41f5f04ec9b1b92b5454

Asset info changes addressed with #47


fix pep8 errors + run flake8 in Travis

Codecov Report

Merging #117 into master will decrease coverage by 0.00%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##           master     #117      +/-   ##
==========================================
- Coverage   79.77%   79.76%   -0.01%
==========================================
  Files          40       40
  Lines        2131     2130       -1
==========================================
- Hits         1700     1699       -1
  Misses        431      431

Impacted Files
Coverage Δ

demo/example/foo/validators.py
100.00% <ø> (ø)

demo/example/foo/views.py
63.63% <ø> (ø)

demo/example/settings.py
100.00% <ø> (ø)

demo/manage.py
53.84% <0.00%> (ø)

plans/context_processors.py
100.00% <ø> (ø)

plans/contrib.py
85.41% <ø> (ø)

plans/enumeration.py
82.35% <ø> (ø)

plans/mixins.py
88.88% <ø> (ø)

plans/signals.py
100.00% <ø> (ø)

plans/taxation/__init__.py
75.00% <ø> (ø)

... and 15 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 d4f8fea...41b2968. Read the comment docs.

Excellent!
I think for making it consistent for everyone to contribute, black formatter can be used


Clarifying Errors in Config file

After an embarrassing amount of troubleshooting, I've learned that if you have a syntax error in your config file (for example leaving out a comma in an array), Magic Mirror behaves as if you don't have a config file at all.  It has the same message as when you do a fresh install: add a config file.
I recommend adding some descriptive text if there is a syntax error in the config file.

I actually went through this as well and while I can fairly easily figure things out, the first time this happened I too was a bit baffled. But I figured it was working just before I made a change, so it had to be that. Went back to look and sure enough ...

I Agree. I'll add some info about this to both the diadem as well as the "no config"-screen.

hi :)
i had the same "problem" yesterday when i was building my first mirror... Thanks to you all for this awesome project!
Do you have any tips for editing the config files? i'm using the standard text-editor directly on the raspberry via vnc, but there is no highlighting at all :(

@zombi27: if you have a mac, check out: https://github.com/MichMich/MagicMirror/wiki/Configuring-the-Raspberry-Pi#enable-file-sharing-with-os-x
This will allow you to edit the files on your mac using file sharing. (Using Sublime Text or Atom)

Btw, there is a same solution for Window, but that's not yet in the Wiki.

thanks! Sublime Text and Atom are looking great :)


Cleanup docusaurus leftover

Remove the @theme aliasing and update regen docs to render correct import for starlight

No changes needing a change description found.

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs


not able to plot polytope

Dear Developer
I try to plot a polytope via intvalpy. i face the following error message
IndexError                                Traceback (most recent call last)
Input In [50], in <cell line: 1>()
----> 1 v=plot_poly3dnew(s2n[0], ax, color ='C1')
Input In [42], in plot_poly3dnew(poly, ax, alpha, color)
35 dim=poly.dim
37 if dim == 3:
---> 38     v = ip.lineqs3D(-poly.A, -poly.b, size=(3,3), show=False)
39     for i in v:
40         x, y, z = i[:,0], i[:,1], i[:,2]
File ~\AppData\Local\Programs\Python\Python39\lib\site-packages\intvalpy\visualization.py:607, in lineqs3D(A, b, show, color, alpha, s, size, bounds)
605 #print("St = BoundaryIntervals(At, bt) ", St)
606 if len(St) > 0:
--> 607     Pt = Intervals2Path(St)
608     P = []
609     for l in range(len(Pt)):
File ~\AppData\Local\Programs\Python\Python39\lib\site-packages\intvalpy\visualization.py:160, in Intervals2Path(S)
158         index = k
159         break
--> 160 es = S[index, 2:4]
162 if np.max(np.abs(bs-es)) > 1e-8:
163     P.append(es)
IndexError: index 3 is out of bounds for axis 0 with size 3
I checked the dimension of polytope and it is 3D polyhedron. I do not know why the BoundaryIntervals failes to creae the facet.
your suggestion will be more helpful to figure out the causes for this problem
any comments is highly appreciated
Thank you
Best regards
Muthu
Researcher
TU freiberg, Germany

Dear Herr. Muthu,
thank you very much for your feedback!
Please, can you give an example of such a system when a similar error occurs?
Best regards
Artem

@mvnayagam
Dear Herr. Muthu,
fixes for this error were made in release 1.6.1.
You can install the latest version using the command : pip install intvalpy --upgrade
Thanks again for your example!
Best regards
Artem

Dear Herr Artem Androsov,
firstly, let me thank you for your valuable comments on my
questionnaire and the following updates in the code.
I am writing an article and want to cite your package. I searched in GitHub
for the corresponding article.
However, it is in the Russische language.  For general readers, do you have
it in English version?
Please let me know other corresponding articles to be cited!
Thank you
Best regards
Muthu
On Sun, Feb 18, 2024 at 6:50 AM Artem Androsov @.***>
wrote:

@mvnayagam https://github.com/mvnayagam
Dear Herr. Muthu,
fixes for this error were made in release 1.6.1.
You can install the latest version using the command : pip install
intvalpy --upgrade
Thanks again for your example!
Best regards
Artem
—
Reply to this email directly, view it on GitHub
https://github.com/AndrosovAS/intvalpy/issues/23#issuecomment-1950970753,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYL2SDYKR3NX7NGSGP5PDMTYUGJBRAVCNFSM6AAAAAA6MD2XJCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJQHE3TANZVGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Mit freundlichen Grüßen
Dr. rer. nat. Muthu Vallinayagam
Technische Universität Dresden
Helmholtz Zentrum Dresden Rossendorf
Ph: (+49) 151 7154 7767

@mvnayagam
Dear Herr. Muthu,
I am very glad that this package was useful to you.
Unfortunately, there is an article in Russian only now... We plan to translate
it into English in the future, but it's hard for me to say exactly when we will do it.
Best regards
Artem


Add missing php & .htaccess files to docker image.

If using the directions in the README / internet-pi project, these changes shouldn't be required... the -v "$PWD":/var/www/html should mount all the proper files into the html dir.


Docker deploy from bundle file not setting alias as service name

Description
Aliases in the created service manifest are set to the network name not the service name.
https://github.com/docker/docker/blob/914a72541fecfc2e4312015befb7200dd05f4462/cli/command/stack/deploy_bundlefile.go Line 49
Steps to reproduce the issue:

Deploy bundle file to docker swarm
Run docker service ls, and inspect one of the services
The alias is not set to the service name

Describe the results you received:
The alias set to the network name.
Describe the results you expected:
The alias set the the service name.
Additional information you deem important (e.g. issue happens only occasionally):
Output of docker version:
]
~ $ docker version
Client:
Version:      1.13.0-rc4
API version:  1.25
Go version:   go1.7.3
Git commit:   88862e7
Built:        Sat Dec 17 01:34:17 2016
OS/Arch:      linux/amd64
Server:
Version:      1.13.0-rc4
API version:  1.25 (minimum version 1.12)
Go version:   go1.7.3
Git commit:   88862e7
Built:        Sat Dec 17 01:34:17 2016
OS/Arch:      linux/amd64
Experimental: true
Output of docker info:
~ $ docker info
Containers: 45
Running: 30
Paused: 0
Stopped: 15
Images: 34
Server Version: 1.12.1
Storage Driver: aufs
Root Dir: /var/lib/docker/aufs
Backing Filesystem: extfs
Dirs: 208
Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
Volume: local
Network: null host bridge overlay
Swarm: active
NodeID: c59i1zljr8t4p6c50257u6tgi
Is Manager: true
ClusterID: 6jv3x0jrttn73dbfr30mnjf79
Managers: 1
Nodes: 1
Orchestration:
Task History Retention Limit: 5
Raft:
Snapshot Interval: 10000
Heartbeat Tick: 1
Election Tick: 3
Dispatcher:
Heartbeat Period: 5 seconds
CA Configuration:
Expiry Duration: 3 months
Node Address: <IP_ADDRESS>
Runtimes: runc
Default Runtime: runc
Security Options: seccomp
Kernel Version: 4.4.17-moby
Operating System: Alpine Linux v3.4
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 7.787 GiB
Name: ip-192-168-33-91.eu-west-1.compute.internal
ID: Y7GW:P7Y4:EOOK:ORE5:ABMG:WL7E:IS32:D74L:GSW6:XQM3:AJON:OIXG
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Username: cf1buildserver
Registry: https://index.docker.io/v1/
Experimental: true
Insecure Registries:
127.0.0
Additional environment details (AWS, VirtualBox, physical, etc.):
AWS docker preview Beta14

In this file the alias is set to the service name
https://github.com/docker/docker/blob/feb8b1c5818c178a9d51f539c8afca7f0c759ac4/cli/compose/convert/service.go
line 163


Remove version constants from PsrHttpAdapterInterface

I feel that they should go somewhere else...

Probably a Version class/interface?

Drop it :)


[VS Code] Could not join the collaboration session as it has already reached the maximum of 5 participants.

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

@lowelltr Hey! As the message indicates, we currently limit the number of concurrent guests in a Live Share session to 5. If you need more than this, the host can set the liveshare.features setting to experimental, which increases the limit to 30 (mileage may very thought!).
Out of curiosity: what is your team using Live Share for that requires more than 5 guests? Thanks!


access denied

Hi
I am trying to make render images in jade, and I can open the urls in separate browser windows, but when using insertAdjacentHTML, I can see the browser icons adding up, but inspecting the DOM in the console I can see 403 forbidden error. Here is the response:

AccessDenied
Access Denied
BB6E832B7178DC18

4cm9AKXA3d/DT7MLGFC6EkHeQuAjiwptzL0Pv82uzdK4ggIPT2y+rFV9cQAHeJOVss0OHFv83cU=

I am using the default key, but manually setting the public key, the issue remains the same.

Could you please provide a minimal project/example that reproduces this issue?

I was using bad javascript. This is fixed. Thank you

:+1:


Refactor Drawables

[x] Adhere to width/height/x/y
[x] Background / foreground / border color
[ ] 3D Border?

Better color management: Check.
3D border: No check - not done right now.


Schoolcountfix

Pull request overview
Fixing school count figure on dashboard homepage
Pull request checklist
Please check if your PR fulfils the following:

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[x] Tests have been run locally and are passing (run_tests_locally())
[x] Code is styled according to tidyverse styling (checked locally with tidy_code())

What is the current behaviour?
School count figure currently is number of schools reporting on the thursday, not the friday as the date suggests
What is the new behaviour?
Amended so count figure is being pulled from Friday

@rmbielby this is the fix I mentioned yesterday which we want to try and slide in as close to the data update as possible


DOC: Indicate members of the confusion matrix in code comments

Suggested by https://github.com/Joeycho in https://github.com/InsightSoftwareConsortium/ITK/issues/3490#issuecomment-1193159763

Closing in favor of #3496.


Add a knob to make per_connection_buffer_limit_bytes of cluster configurable

Description:

EG can configure per_connection_buffer_limit_bytes on xDS Listener currently via #2600, not available to configure per_connection_buffer_limit_bytes on xDS Cluster, default to 32KB. A new knob will be better for entire buffer sizing control of  Envoy provisioned by EG.

[optional Relevant Links:]

https://gateway.envoyproxy.io/latest/api/extension_types/#connection
https://www.envoyproxy.io/docs/envoy/latest/api-v3/config/cluster/v3/cluster.proto#config-cluster-v3-cluster

hey, @aoledk
Do you have any ideas or are you currently working on this PR?

hey, @aoledk Do you have any ideas or are you currently working on this PR?

@ShyunnY Perhaps this feature should be implemented via BackendTrafficPolicy, similar way like https://gateway.envoyproxy.io/latest/api/extension_types/#connection works. And I'm not working on this Issue.

/assign

@guydc When EG is used as L4 proxy and serves massive long lived TCP connection, the buffer size of <Downstream, Envoy> and <Envoy, Upstream> connections should be controlled, to reduce memory consumption of Envoy.


Add iter_tabs and iter_tabs_mut

Is your feature request related to a problem? Please describe.
I need to iterate over all tabs in my app, and I don't want my logic to handle internal docking structure with leaves.
Describe the solution you'd like
iter_tabs and iter_tabs_mut methods to directly iterate over tabs data
Describe alternatives you've considered
state
    .iter_nodes()
    .filter_map(|e| {
        if let Node::Leaf { tabs, .. } = e {
            Some(tabs)
        } else {
            None
        }
    })
    .flatten()

This requires my app to take internal docking logic in consideration, as well as makes _mut version impossible

Closed by #196


⚠ Processors has degraded performance

In 6530a8c, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 43 ms

Resolved: Processors performance has improved in aeb4159.


Fix/readme

This PR fixes the changes for the updated version of aws-ecs-fargate workflow:

Update licensing to current dates
Re-worked and updated readmes with descriptions and current information on building, running, and publishing your workflow; with redirects
Add comments to each section in the ops.yml file

LGTM. The README is more clean now.


how to del expire key in hash？

when i use redisson ,just like redissionCache.put(key, value, this.time, this.timeUnit), value is obj,but the key just use once, now we do some stress-testing ,there are lots of expire keys ，i don't know how to clean these expire keys.

Expired keys in RMapCache are cleaned automatically by separate thread


Use more fields when filtering in contributors query

At present search is done in fullName and orcid. We should include lastName, too. A use case is searching a contributor without their middle name doesn't return the expected contributor, e.g. filtering with "Martin Eve" doesn't return "Martin Paul Eve"s record.

I attempted splitting the filter string on spaces in https://github.com/thoth-pub/thoth/pull/278/commits/d28fef68872b6734c1bcdece7344280164656005, but the results returned were poor, as the number of hits potentially increased every time a new word/name was added to the filter string - making it harder to see actual matches.


Proof of concept for a PersistenceProxy Backend for Ripple

WIP

[x] Make it work with both Riak and the PersistenceProxy
[x] Store objects
[x] Fetch objects
[x] Reload objects
[x] Delete objects
[x] Integration specs
[ ] Improve error handling
[ ] Add more coverage tests
[ ] Use inside a Rails app for better integration testing

We decided not to move forward


Update indexing routes and osgi config

Indexing routes were not filtering messages property, and were assuming
the presence of the wrong headers in reindexing messages; Verified manually
Resolves #93

Just by way of FYI, fcrepo-camel/4.5.0 will not have the JmsHeaders class. Instead, camel users will be encouraged to follow this pattern:
from("broker:queue:foo")
  .process(new EventProcessor())
  .filter(header(FCREPO_URI).contains("..."))
    .to("direct:route");

The headers CamelFcrepoIdentifier (FCREPO_IDENTIFIER) and CamelFcrepoBaseUrl (FCREPO_BASE_URL) will continue to be supported, but, instead routes should start making use of the CamelFcrepoUri (FCREPO_URI) header. That header will also work just like HTTP_URI with the http4 component:
setHeader(FCREPO_URI).constant("http://example.org/fcrepo/rest")
.to("fcrepo:localhost");

this route will send the fedora request to http://example.org/fcrepo/rest, not http://localhost.

And if you're wondering why I would remove the JMS header class -- those headers are not and will not be part of the future Fedora API specification. Instead, the spec will concern itself only with the message body, which is what code should start depending on, since a non-JMS-based messaging system will certainly not contain org.fcrepo.jms.* headers.
Eventually, the message format will conform to the W3C Activity Streams 2.0 spec. At that point, existing code that follows the pattern above will need little to no changes. See here for some sample message serializations: http://fcrepo.github.io/fcrepo-specification/#message-examples

Ah, I see.  It looks like we'll need to be on fcrepo-camel 4.5 before we can use the new recommended approach.  How about we move towards the recommended FCREPO_URI pattern as part of #94 (that way we can test it in an IT), and leave the obsolete JmsHeaders in place until that happens?

@birkland yes, you'll need to be on fcrepo-camel/4.5 before implementing this -- for now, keeping the jms headers in place makes sense.

Merged 9bd1ebcf


Invoke-PnPBatch: Issue submitting sequential batches

We need to update large libraries with up to 1m documents per library.
Using Set-PnPListItem in a loop with batching, trying to submit 1000 document per batch:
foreach ($item in $Items){
Set-PnPListItem  ... -Batch $batch
if(  $Count % 1000 -eq 0)
Invoke-PnPBatch -Batch $batch
}
}
On submitting a second batch we are getting 20 lines the same error (batch size here is 10):
The 'Type' attribute is invalid - The value 'Object[]' is invalid according to its datatype 'http://schemas.microsoft.com/sharepoint/clientquery/2009:MethodParameterTypeType' - The Enumeration constraint failed.          200 https://...
Tried to use -Force, the same
Is it a bug? is there a workaround?

just to be sure, do you call $batch = New-PnPBatch after every invokation of Invoke-PnPBatch -Batch $batch ? as I think you need a brand new batch object after you have submitted it with Invoke-PnPBatch

Yes, I've tried. It says in doc, you cannot submit twice (not sure what that means)
Also tried adding $batch = $null before re-declaring, all with no difference.
The feature can really save us lot's time as we are migrating 2m records, any other ideas?
Thanks!

@soaway - Sorry for the delay. We've closed a couple of issues related to batch, if the problem continues, please open a new issue with all the details. Thank you.

Thanks for following up, Veronica.


Migrate Structopt to clap

fixes #89

:clap: (sorry, couldn't help myself). LGTM :D

I didn't quite get what was the sorry for 🫠 but thanks for quick review 😃

:clap:, pun intended

😂 okay got it now 👏🏻

Concept ack
There is a format error, and an examples to be fixed to make the CI happy

fixed the example and formatting too

@RCasatta can you run the workflow now 😅

sure no worries just glab I could contribute will work on the crate separation part now :)


Implement FastAdaBelief?

Hello! There is a newish paper improving upon AdaBelief, FastAdaBelief, which is more or less a combination of AdaBelief and SAdam. So far it does seem to match the performance it promises in its paper in my experiments, and it has time-dependent epsilon and time-dependent beta2 so there is only one hyperparameter -- beta1. I have implemented it here. If you'd like to add this to optax I'd be happy to do a pull request.

@evanatyourservice I'm also interested in this. Would you still like to submit a PR?

Hey Carlos! Iirc my implementation I posted is a little off from their paper, but when I implemented it correctly it didn’t really stand up to the test of time so to speak, adabelief alone is a much better optimizer :)

Interesting. The paper's experiments show FastAdaBelief outperforming AdaBelief. Your experiments yielded the opposite result?

Yeah I remember finding something wrong with my implementation but then after I fixed it it still wasn’t holding up to some tests. I totally forget what I tested it on, probably vision and time series back then, but definitely worth checking out if you want! I’ll make sure my implementation above is updated to the paper.
Interestingly the adabelief author made a new optimizer called acprop, could be worth checking out too

Ah I just realized I didn’t put my fastadabelief implementation public, I’ll post it on GitHub hopefully tonight. Maybe we should test it a little before suggesting optax add it. I’ll also add an acprop repo. I liked the theory behind acprop I think it would def be worth a PR

That sounds great!

@carlosgmartin sorry carlos i actually probably won't have time to get to this!

No worries! I can do it: https://github.com/google-deepmind/optax/issues/965.


Load clients into the race with custom gamemode/player type

Can someone review the ASM and make sure the patch macros are correct?

@stblr implemented it separately into the "rollback" branch.


"Illegal reflective access" warning should be addressed

Describe the bug
Version 2.11 of the Maven plugin (available on Maven Central) generates the following working during formatting:
09:24:58.285 [INFO] --- fmt-maven-plugin:2.11:format (default) @ xxxxx ---
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by com.google.googlejavaformat.java.JavacTokens$CommentSavingTokenizer (file:/Users/xxxx/.m2/repository/com/google/googlejavaformat/google-java-format/1.10.0/google-java-format-1.10.0.jar) to field com.sun.tools.javac.parser.JavaTokenizer.reader
WARNING: Please consider reporting this to the maintainers of com.google.googlejavaformat.java.JavacTokens$CommentSavingTokenizer
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

To Reproduce
Execute the format goal
Expected behavior
The warning should not occur
Additional context
I have reverted to the 2.10 plugin release, and the issue no longer occurs. This is a valid workaround for my team, so we're not blocked. You're still using this version on master, there's still a potential issue in subsequent releases.
I've also created an issue in the google-java-format project, as this is most likely in their code.

So, those arguments certainly work. However, it would be nice if we eventually don't not have to add them.

See also #99

These warnings are still issued for JDK 9  to 15, despite https://github.com/spotify/fmt-maven-plugin/pull/140

Closed by https://github.com/spotify/fmt-maven-plugin/pull/150


Add git submodule pointing to example app repository

Feature to add the existing sample application as a git submodule to directly track the Master branch and being able to update the two repositories independently.
Using the structure recommended by pub.dev to create an example application

Hello @sonikro  @Perdiga , can you please add my colleague @stevenbarash as a contributor with WRITE permissions so we can continue to collaborate on this project together?
Additionally, it would be great if you can make me a "super" admin so I don't have to rely on you for things like this.
Respectfully,
Mark


⚠ Fortnite has degraded performance

In cea1ce4, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1589 ms

Resolved: Fortnite performance has improved in 00c283b.


Graph is empty after first reload of the page (on ~daily basis)

We have CCF installed on AWS and are experiencing a rather strange behavior in the Cloud Usage chart: The very first time one of our devs accessed the site, we had a "normal" chart.

But any other dev got an empty graph. We've tried this on different Linux distributions and with different browsers. The very first dev also lost the graph, once reload was hit, or the exact page was opened in a second tab; so we tried the incognito mode as well, to make sure there isn't some client side issue with stored data. A day later we where once again able to see the curve for the very first time of opening the page.

We tried debugging this ourselves and had following findings:

The date-range changed; on the first screenshot (with graph), the x-Axis shows the entire date range, on the second one it cuts of the range after March 28th.
The data points are still there; we could make them visible via moving the mouse over the graph, see the tool tips, and even locate the data points on the graph.
Using the browsers devtools we found that there is an element .apexcharts-canvas .apexcharts-element-hidden with opacity = "0"; switching this to 1 will show at least the data points.
When manually making the points visible by overriding the opacity settings, and comparing the graphs we found that all data points where duplicated.

This points to the issue #1069 , from March last year; however the solution suggested there was to use Node 18. We are currently using release: 'release-2024-02-11'; and we rechecked that Node 18 is used by client and server, which is the case.
So, though the symptoms are identical to issue #1069 , the root cause must be somewhere else.

There are 2 instances from the same domain, which have slightly different behavior:

One is as described above. The doubling of points correlates to half the time range, e.g. choosing 1M gives 15 days.
The other doesn't have double points and half time range. But otherwise the issue is the same.

Also, to validate against issue https://github.com/cloud-carbon-footprint/cloud-carbon-footprint/issues/1069, we tried CCF on an entirely separate AWS account, setting it up by Terraform. (The one above was set up using Helm). Regardless of node 16, 18 or 20, there is no issue, i.e. the graphs are always visible.

"I'm facing the same issue.
I'm using Helm for deployment on EKS with the release version release-2024-02-11."
first run

second run

logs:
cloud-carbon-footprint-api-5bcd879796-h9xcr_cloud-carbon-footprint.log
"It seems like a caching issue."

https://github.com/cloud-carbon-footprint/cloud-carbon-footprint/issues/1069
It seems like my issue is identical to that one, but I'm using the latest image from the Docker Hub repository and still encountering the problem.

I solved it using the cache in Mongo Atlas.

It turned out that the cache was disabled on our own account, where the problem was not visible. On activating the cache the problem appears, unrelated to node version. I guess if we follow https://github.com/ddias00 's advice and utilize MongoDB, it will let the problem vanish again.
So - The problem is only with local cache, which will be deprecated, but is still the default!

@pygospa Thanks for opening this issue and for making it so detailed! Big thanks to everyone on the collaborative troubleshooting.
From first impression, this definitely sounds like a cache-specific issue mixed in with some weird behavior from Apex charts. @jochen-schuettler I completely agree with your point that even though the local cache method is no longer supported, this is still a problem since it's the default option. I believe it has come up enough that warrants an investigation into the root cause and fix.


Error

Errno: 3
Test: MACH_PORT_VALID(task_port)
Filename: ViewController.m
Line:215
Function: inject_libary
I get this error when i click jailbreak and it gets around 37/48

@viiejo please provide us with more information. What iOS are you on and does this happen every time you jaibreak?

Closing this due to no response after 2 days.


Home and Feed screens are empty

When running the sample on the simulator, the login via Google account is working but then the Home and Feed screens fail to load any images. The logs:
2017-08-09 15:23:27.575825+0800 FriendlyPix[83333:19626080] [MC] Reading from private effective user settings.
2017-08-09 15:23:49.112 FriendlyPix[83333:19625887] KZPropertyMapper: Ignoring value at index posts as it's not mapped
2017-08-09 15:23:49.113 FriendlyPix[83333:19625887] KZPropertyMapper: Ignoring value at index _search_index as it's not mapped
2017-08-09 15:23:49.113 FriendlyPix[83333:19625887] KZPropertyMapper: Ignoring value at index displayName as it's not mapped
2017-08-09 15:23:49.113 FriendlyPix[83333:19625887] KZPropertyMapper: Ignoring value at index photoUrl as it's not mapped
Looks like the same issue as https://github.com/firebase/friendlypix/issues/64

I fixed the original bug over there. Are you uploading a new image but not seeing it?


Update modules with a metadata file

Description
Source: https://github.com/Azure/bicep/issues/5138
Could use the metadata file to automatically generate the readme description text.

On the deownside - we'll bloat the modules further. I'd suggest to wait with this one until at least the PG has a final design


Release sentinel 2023 04 01 preview

ARM API Information (Control Plane)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[X] new service onboarding
[X] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[X] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[X] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[X] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

/azp run

Please get breaking change reviewed and come back for ARM review

@zizw123

All the new
changes on this branch has been reviewed and signed off by ARM review team.
We have 3
breaking changes flagged and we have gone through the ARM breaking change
review process, the office hour on 4/10. This is the ADO item to track the
work: Scenario
17908651: Cross version validation for release-Sentinel-2023-04-01-preview -
Boards (visualstudio.com)
1. One is
approved by Mike.
2. Two changes on watchlist are still in review, and
following up with Mike. This breaking change was introduced on 2022-01-01
Preview by the following PR.
Dev sentinel
2022 01 01 preview by anat-gilenson · Pull Request #18025 ·
Azure/azure-rest-api-specs (github.com)

❌ 1025   - RequiredStatusChange
The   'required' status changed from the old version('True') to the new   version('False').   New: Microsoft.SecurityInsights/preview/2023-04-01-preview/Watchlists.json#L481:7   Old: Microsoft.SecurityInsights/stable/2023-02-01/Watchlists.json#L469:7
4/10 office   hour   Follow up:   provide evidence of approval from past. Hassan to follow up

❌ 1049   - RemovedXmsEnum
The new   version is missing a 'x-ms-enum' found in the old version.   New: Microsoft.SecurityInsights/preview/2023-04-01-preview/Watchlists.json#L494:9   Old: Microsoft.SecurityInsights/stable/2023-02-01/Watchlists.json#L482:9
4/10 office   hour   Follow up:   provide evidence of approval from past. Hassan to follow up

Breaking changes previously approved in #22983

Signing off, most commits in this PR have been signed off before, the diffs are typo fixing.

@MaryGao @msyyc could you help to review the SDK breaking changes?

@tadelesh @zizw123 , it seems we got all the approvals. Can we get a final approval and merge please?

Sync with js sdk owner. Approved the sdk breaking.


Update DBConnection

closes #235
I've try to update to connection from 1.x to 2.0.1, but seems some test didn't pass

in test auth connection leak sometimes will have one tcp_count when test just started, thinking about the way to clear ports before this test start
in Elixir v 1.6 will got a warning that some db_connection behaviour didn't use

lib/mongo/protocol.ex:1: Undefined callback function handle_prepare/3 (behaviour ''Elixir.DBConnection'')
lib/mongo/protocol.ex:1: Undefined callback function handle_rollback/2 (behaviour ''Elixir.DBConnection'')
lib/mongo/protocol.ex:1: Undefined callback function handle_fetch/4 (behaviour ''Elixir.DBConnection'')
lib/mongo/protocol.ex:1: Undefined callback function handle_declare/4 (behaviour ''Elixir.DBConnection'')
lib/mongo/protocol.ex:1: Undefined callback function handle_deallocate/4 (behaviour ''Elixir.DBConnection'')
lib/mongo/protocol.ex:1: Undefined callback function handle_close/3 (behaviour ''Elixir.DBConnection'')
lib/mongo/protocol.ex:1: Undefined callback function handle_begin/2 (behaviour ''Elixir.DBConnection'')
lib/mongo/protocol.ex:1: Undefined callback function handle_commit/2 (behaviour ''Elixir.DBConnection'')

but I didn't see any behaviours before, just wonder how to bypass those warning

Thanks for this. I will take a look through it shortly.

Any news on this one?

I ended up with some workaround / didn't have time to check this 🤷♀

Unfortunately I have not been able to test these changes yet either. I am hoping to find a bit of time soon.

I merged master locally and the entire test suite has passed. Will be testing this out in our staging environment shortly.

If I run from this branch and just pop into an interactive session, sitting there briefly, the following happens:
** (stop) exited in: GenServer.call(#PID<0.373.0>, {:server_description, %{address: "localhost:27017", arbiters: [], election_id: nil, error: nil, hosts: [], last_update_time: -576460660915, last_write_date: nil, max_wire_version: 5, me: nil, min_wire_version: 0, op_time: nil, passives: [], primary: nil, round_trip_time: 4, set_name: nil, set_version: nil, tag_set: %{}, type: :standalone}}, 30000)
    ** (EXIT) time out

If I try to make an explicit find/4 call to the local Mongo server:
** (exit) exited in: GenServer.call(:mongo, :topology, 5000)
    ** (EXIT) time out
    (elixir) lib/gen_server.ex:924: GenServer.call/3
    (mongodb) lib/mongo.ex:819: Mongo.select_servers/4
    (mongodb) lib/mongo.ex:796: Mongo.select_server/3
    (mongodb) lib/mongo.ex:442: Mongo.find/4

Going to play with some config.

Update: so far no luck. Updating this driver to use dbconnection 2.0 is beyond me. :(

@kiote I think the easiest thing to do would be to just test this branch to make sure things are working correctly. Most important would be testing various failure scenarios (closed connections, that sort of thing).

@ankhers is right about this - one of the big problems I wasn't able to figure out (and I suspect it's due the callbacks not being implemented) is that connections just swell up and never get closed when using pooling.
@ankhers : Any chance you'll be able to take a swing at this PR?

I will try to find the time to dig through this soon.

I am closing this in favour of #279. Thank you for the contribution!


http cache

Please tell me whether it supports http cache or not? How can I set if it is true.Thanks a lot!

See the #103
Closing this as duplicate


Add scene::Transform

This addresses issue #93

We really need #21 before this is published.

Latest commit is unrelated but important.


ribbon

:rocket: This description was created by Ellipsis for commit 535f6efaa787dfdf5f35443472d60c23a911d2da

Summary:
Introduced a GithubRibbon component to display a GitHub ribbon on the application, enhancing visibility and access to the source repository.
Key points:

Added GithubRibbon component in /components/GithubRibbon.js
Imported and used GithubRibbon in App class in /app.js
GithubRibbon displays a GitHub ribbon linking to a specified repo

Generated with :heart: by ellipsis.dev

@elipsis-dev what if I want the ribbon to be green?