[druid] Updating Druid refresh metadata tests

This PR updates the Druid tests for refreshing of the metadata in the following ways:

Removes the unnecessary enumerating of datasources as there's only one.
Adds a check that the column type also is updated during the refresh. Previously we only checked the updated metrics.

to: @GabeLoins

Codecov Report

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

@@           Coverage Diff           @@
##           master    #4887   +/-   ##
=======================================
  Coverage   77.13%   77.13%
=======================================
  Files          44       44
  Lines        8542     8542
=======================================
  Hits         6589     6589
  Misses       1953     1953

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 510ae84...8219ea4. Read the comment docs.

LGTM


LightCurveCollections and TargetPixelFileCollections Classes

Created a Collections class for LightCurves and TargetPixelFiles. This class collects together multiple lightcurves or tpfs and allows operations over all of them such as plotting, for-loops, etc.
Input to a collections class should be given as an array. Example usage would be:
lcc = LightCurveCollections([lc1, lc2, lc3])
for lc in lcc:
    lc.normalize().fold().bin().plot()

or to simply plot all of them:
lcc.plot()

@johnnyzhang295 @johnnyzhang295 Thanks, I think this is an improvement over earlier PRs and something we could merge.
I do wonder if we want to sub-class the Collection class from Python's built-in Sequence class.  In this case we could still support indexing by targetid by overriding __get__ to build the target=>index dictionary map when it is needed (it could be cached by making sure a _dirty_map boolean to True when a new item is appended).  The advantage is that we may have less code to maintain, and only need to build a mapping dictionary when needed.  Thoughts?

@barentsen I'm not too familiar with the Collections class in python. If we opt to use a cache for the dictionary map, I wonder if this would make the code too complex. Building the dictionary map can be done in linear time so I'm not too worried about its computational complexity.

Collections is ready to merge now! @barentsen.
Lots of improvements were made...

All collections-related code has now been moved to collection.py
LOTS of test coverage for LightCurveCollections and TargetPixelFileCollections :)
New Feature- Slicing/Indexing (Could only append before)
An issue with KeplerTargetPixelFile and targetid has been fixed

Thanks @johnnyzhang295!  Stay tuned for a final review!

Merging.  Thanks @johnnyzhang295!


TSC Agenda 2022/05/31

TSC Meeting Details
Date/time: 2022-05-31 @ 8am Pacific (Timezone Converter)
Location: Discord tac-tsc voice channel
This meeting occurrence will be run by @OBWANDO acting as interim co-chair until a co-chair is elected (Jeremy OoO)

Co-chair nominations: https://github.com/o3de/tsc/issues/33

Would like how we improve RFC visibility - ie common advertising process.

Do we have a clear permission continuity model so if key members leave we can maintain access? Discord, O3DE binaries, O3DE permissions for GitHub? Raised as a concern that we have a few 1 person processes.

Hi O3DE SIGs!
We’re building an O3DE project roadmap through early 2023, which will help inform our marketing roadmap & activities. We’d appreciate your help in surfacing your SIG roadmaps (e.g. key features/capabilities you’re working on & timing) so we can build them into this roadmap. Would love to hear from you by next Friday, June 3, if possible. Many thanks in advance for your collaboration!
If a roadmap template would help to collect this information, please let me know & I’ll create one.
(Nicole Huesman)

Minutes taken: In attendance:
Danilo
Finchy
Finite_State
Joe Bryant
JT
kberg
Laura Reznikov
Lumberyard-employee
Nick_L
philkeslin
RoddieKieley
Royal OBrien (Leading)
Seapip
stramer
Terry
Tobias Alexander Franke
Reminder from Royal that all Heads please join o3de-tsc mailing list
Reminder that during monthly SIG head meetings that we need a chair or co-chair present to discuss cross concerns.

lmbr-pip items:

Need to improve RFC visibility - How to move these up to more visibility - Maybe a webpage spot during the o3de reorg revamp that has a page or section with new items and RFCs along with blogs and feature RFCs
Also to have all RFCs go to SIG-ALL and a process to request marketing support
Add automation if possible, and to stay open to try whatever it takes to gain traction for visibility
Post to impactful changes - channel for RFCs and announcements - automation to thread GitHub comments into mailing lists

--
Need a clear permission continuity model of key members - groups - We do not have things for services that show where there is a need for multiple admins - Request to build a list of services (YouTube, discord, o3debinaries/etc.) and the respective members that have permissions or ownership to fill holes in - Finchy agreed to help create list and names
--
Release update - Comment made that timing allows marketing to spin up by 9/1 to begin building blogs and feature videos for release

Seapip - general issue - issue triage - when cross sig boundaries -
need better system when issue triaged with one sig or need help or move to another sig
when moves to another sig - because triaged - new sig doesn't see it or show up on radar -
talk about how to solve it - need mechanism for that - issues move across SIG and disappear -
with change to SIG-triage triage meeting where sigs are assigned - sig docs and sig-ui-ux through people
docs join a triage - look for automation of aging -
--
sig-release - Joe Bryant need SIGs to provide info for roadmap and ensure to keep the feature grid up to date -
sig-security - Seapip
sig-graphics-audio - Tobias - Future chat about streaming code - terrain modules (awareness)
sig-content - No Rep (Joe Bryan proxy) - Working on remote object RFC - deliver for next release - First iteration get remote projects running - any team to go fetch project from anywhere (github/perforce/etc) - 2nd piece templates so people can ge templates from anywhere - 3rd piece is gems - any object can be made remote - and UX - raise awareness around that - *** raised red flag on that (sig-platform colin) - dont think test is currently doing enough to test remote vs local  (engine centric) - remote objects not working - more cognizant of that - plan should include all tests needed
sig-core - Working on profiler gem - stabilized across platforms - new archive system adding to project to allow deprecation of legacy cloudpack -
sig-platform - Byrcolin - just XR and cross and issues with triage and joe element
sig-operations - Finchy - TSC have no chair or co-chair -need to add a possible sigs that don't have co-chair
sig-docs - stramer - actively engaged - RFCs for automated doc generation - auto gen component reference
working LUA and Script canvas and c++ API reference gen and cvar ref generation
sig-demos - Need to build chater - Joe Bryant, stramer and Royal O'Brien to contribute to it
sig-network - No rep
sig-ui-ux - No Rep
sig-build - No Rep
sig-simulation - No Rep
sig-testing - No Rep
does sig-operations (charter) - help with sig communications -
Elections are coming up - Notified Claire and LF team to begin process and look at staggering election dates.
Marketing / ui-ux / docs - SIGs determine cadence -


Upgrade to gcc 13.2 and binutils 2.41

new branches are ready:
https://github.com/ps2dev/binutils-gdb/tree/ee-v2.41.0
https://github.com/ps2dev/gcc/tree/ee-v13.2.0
Tested with neutrino.

Added upgrade to newlib 4.4 thanks to @fjtrujy


test

Docs Build status updates of commit 07515e0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

redirect-url/TOC.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Resolution of RGB and Depth Images (SUNRGBD)

Hi,
If I understand the code properly, is it that resolution of individual RGB and Depth images and thus the point cloud does not affect the training of F-PointNet? As the input to the network seems to be the point cloud in the field of view of the 2D box which can vary for different images and classes. And then all the data in that point cloud is re-sampled to have some constant dimensions in order to train the network. Am I correct in understanding the implementation of the network here? Please let me know. Thanks a lot in advance.
Kind Regards
Pranjal Biswas

Just to close this thread, resolution of RGB and Depth images don't exactly affect much cause the input of FPN is the frustum.


golem-network.com

see https://github.com/409H/EtherAddressLookup/commit/5012c911fef8f9aaad072c2af67c1ecb7d48cf66

In site footer


Improve Maps.me experience by opening AOI into app

Some feedback from users when using Maps.me after visiting the dashboard page:

What did you think you’ll see when you clicked the Maps.me button?
The AOI of the campaign open up in Maps.me
Instead it located the Maps.me app in the App Store (using iOS)
Opened Maps.me, but nothing happened

Idea proposed: open a bookmark or KML file directly into Maps.me to show area that should be mapped. This seems to be supported by Maps.me: https://support.maps.me/hc/en-us/articles/207895029-How-can-I-import-bookmarks-

@dimasciput will you follow up on the iOS side.
@meomancer will you follow up on the Android side.

@smit1678 @cchristelis
I have tested this on iOS, you can open the .kml file in Maps.me

But you will need have Maps.me installed ( there's no way browser will know if you have it installed or not), and if you have it installed it will not automatically open the maps.me. So I think maybe we need change the button name.

@dimasciput, maybe we just note that in some helper text. Maybe this is interesting to note for @MariaSolovyeva while she is thinking about the call to actions on the UI.

Hi @meomancer, for android have a look at: https://stackoverflow.com/a/12901352/1158060

Added in https://github.com/hotosm/field-campaigner/pull/401


[nrf fromtree] Bluetooth: Disconnect L2CAP channel if peer sent too m…

…uch data
This was affecting L2CAP/LE/CFC/BV-26-C, L2CAP/LE/CFC/BV-27-C,
L2CAP/ECFC/BV-33-C and L2CAP/ECFC/BV-34-C qualification test cases.
Signed-off-by: Szymon Janc<EMAIL_ADDRESS>(cherry picked from commit c3bb5ad47289eb7bab1a0ee9106f0570dc45f96b)
Signed-off-by: Alexandru Carbuneanu<EMAIL_ADDRESS>

@CarbuneanuAlexandru can you check why CI is red? (cc @PerMac)
Also, could you send a PR to sdk-nrf with the corresponding update in sdk-zephyr revision?

Twister failures shouldn't be seen now. I restarted the CI.

@CarbuneanuAlexandru can you check why CI is red? (cc @PerMac) Also, could you send a PR to sdk-nrf with the corresponding update in sdk-zephyr revision?

here is the PR for sdk-zephyr: https://github.com/nrfconnect/sdk-nrf/pull/6078

@carlescufi the last build seems green, are we ok for the approval?


Create annotated LKG tag instead of lightweight

For #9842
When tagging builds as LKG, create annotated tags instead of lightweight ones

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

Codecov Report

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

@@            Coverage Diff             @@
##           master   #10177      +/-   ##
==========================================
- Coverage   61.35%   61.32%   -0.03%
==========================================
  Files         567      567
  Lines       30756    30756
  Branches     4408     4408
==========================================
- Hits        18869    18862       -7
- Misses      10903    10909       +6
- Partials      984      985       +1

Impacted Files
Coverage Δ

src/client/common/utils/platform.ts
64.7% <0%> (-11.77%)
:arrow_down:

src/client/linters/pylint.ts
94.82% <0%> (-3.45%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0%> (-0.73%)
: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 2008c14...9bf2584. Read the comment docs.

Not needed.


[hi] Fix README-hi.md

Problem:
The sentence below in the README-hi.md seems to be incomplete
"अपने fork में कोई भी परिवर्तन करना चाहते हैं, और जब आप उन परिवर्तनों को हमारे पास भेजने के लिए तैयार हों, तो अपने fork पर जाएं और हमें इसके बारे में बताने के लिए एक नया pull request बनाएं।"
the first part i.e "अपने fork में कोई भी परिवर्तन करना चाहते हैं" makes no proper connection with the other part

/assign

We would like to have better sentence structure. What solution do you recommend?

/language hi
/retitle [hi] Fix README-hi.md

I think starting the statement with " इस fork में आप अपने इच्छा अनुसार परिवर्तन कर सकते है  " would sound more correct as well as it would go with the flow with the entire statement

We need to mean the same as Make any changes you want in your fork, and when you are ready to send those changes to us, go to your fork and create a new pull request to let us know about it.

अपने fork में कोई भी परिवर्तन करना चाहते हैं (existing one) -> this means "want to make any changes to your fork" which is a kind of question to user while we want to convey a assertive sentence that developers can make changes to their fork if you get what I mean

Exactly, we need to mean want to make any changes to your fork. You can refer the en version I stated above.

"want to make any changes to your fork, and when you are ready to send those changes to us, go to your fork and create a new pull request to let us know about it. " this is what the (hi) translation sounds to me I guess that is abrupt

How about अपने fork के परिवर्तन करने के बाद जब आप उनको हमारे पास भेजने के लिए तैयार हों, तो अपने fork पर जाएं और हमें इसके बारे में बताने के लिए एक नया pull request बनाएं

perfect to me

Thanks

if you want you can go with your PR

Why did you unassign. You can make a PR, else this could be a good-first-issue with some tweaks to issue description.

/triage accepted

/assign

/reopen

You need to PR to https://github.com/kubernetes/website for things to change here.
/reopen


Off Canvas Back-Link Overlay Transition doesn't work in Safari

In 5.2.3, the off canvas back-link overlay should fade in within 300 ms:
.move-right .exit-off-canvas {
	...
	transition: background 300ms ease;
	...
}

This background transition doesn't work in Safari. The overlay appears suddenly and without any background transition. I tested this in Safari 7 (OS X) and Mobile Safari (iPhone 4/iOS 7).

We'll close this as we are beginning dev for F-6. We'll be happy to accept a PR as Caleb mentioned. Thanks!


Tab doesn't work, neither does Emmet

Hi, just launched atom now to find that I can't TAB. Emmet doesn't work either – I can't expand even by shift+cmd+e, it deletes the word (e.g. "#funky"). If I disable the emmet package, TAB works well.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/27266066-tab-doesn-t-work-neither-does-emmet?utm_campaign=plugin&utm_content=tracker%2F535900&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F535900&utm_medium=issues&utm_source=github).

/ Resolved by deleting the .atom hidden folder in my user folder on OS X. Not sure if I should keep this issue here or delete it, to my understanding, it's still an issue. I only have one other package – beautify, which I don't think can conflict with Emmet anyhow. Feel free to delete/close this issue though.

Not sure if related, but behavior sounds similar: when I hit tab to expand an emmet expression, the line just deletes. I can tab elsewhere just fine, but when I tab complete emmet expressions it just wipes out the line.

Interestingly, keybinding resolver is showing the emmet command was executed.

Opened a new issue for mine. #347


The team @go-maintainers does not exist in bazel-contrib

This team has to be copied from the old org (unless I just can't see it)

@alexeagle Are you able to make @bazel-contrib/go-maintainers truly public (rather than just org public)?


Document the rust-version field

I've tried to make the documentation here fairly comprehensive. I've also updated the first version for the 2021 edition, which should now be stable pending substantial unforeseen changes.
See #8072.

r? @alexcrichton
(rust-highfive has picked a reviewer for you, use r? to override)

r? @ehuss

Thanks!  Normally the documentation is updated along with the stabilization.  Are you intending to add more commits here to stabilize?
As for the 2021 stabilization, there still is some risk it could slip a release, since 6 weeks isn't much time.  Hopefully it won't, but I assume changing the value shouldn't cause too much of a problem since 2021 is still unstable?

Ah, wasn't sure about the exact wording you used in the tracking issue. I'll add a commit to stabilize.
It will be easy to update the first version for 2021 again if necessary. We can do that if/once 1.56 hits beta without 2021.

Add a commit to stabilize.

Stabilizing this seems reasonable to me.
@rfcbot merge

Stabilizing this seems reasonable to me.
@rfcbot merge

:umbrella: The latest upstream changes (presumably #9751) made this pull request unmergeable. Please resolve the merge conflicts.

Thanks!
@bors r+

:pushpin: Commit c43024f04641ba6ba011387332c9647c86214e35 has been approved by ehuss

:hourglass: Testing commit c43024f04641ba6ba011387332c9647c86214e35 with merge cc17afbb0067b1f57d8882640f63b2168d5b7624...

@ehuss thank you for all the reviews and mentoring, and your awesome work in Cargo in general!

:sunny: Test successful - checks-actions
Approved by: ehuss
Pushing cc17afbb0067b1f57d8882640f63b2168d5b7624 to master...

Is it just me, or was the FCP cut short for this issue? Shouldn't the FCP have happened in the tracking issue? Very confused about the process now.

We mostly use rfcbot just to get a positive approval from the entire team.  We don't broadcast these like the lang team does, and don't usually wait for the comment period.  Of course, anyone is welcome to leave comments at any time, and there is 6 to 12 weeks to change or revert any decision.


Documentation on filtering APIM named values by tags is unclear

The documentation for retrieving named values from APIM and filtering by tags is unclear. The documentions of the $filter field suggests that a filter can be applied to the tags field :

but if I try and filter using equals I get the following error :

{
"error": {
"code": "ValidationError",
"message": "The operand for a binary operator 'Equal' is not a single value. Binary operators require both operands to be single values.",
"details": null
}
}
Which seems reasonable as tags is an array. I also tried to figure out how to use the any function but couldn't get it to work.
Please can you provide an example of how to filter by tags in this case and also updating the documentation to give examples of such would be useful.

I figured out how to do this by googling around (I didn't know about odata) I finally got it work after reading this SO post :

This isn't intuitive at all, so I think the docs would definitely benefit from an example

Hold the phone, I'm back. What I actually wanted to do was filter out all named values with a specific tab so I changed my filter to tags/any(tag: tag ne 'no-deploy') and got back :
{
"error": {
"code": "InternalServerError",
"message": "Request processing failed due to internal error.",
"details": null
}
}
The docs says that the tag filter supports 'ne' so is there something I'm missing?

We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Terminal incorrectly wraps after 80 chars in remote ssh (bash)

I'm experiencing incorrect line wrapping in the integrated terminal, identical to what is described in https://github.com/microsoft/vscode/issues/106668
This happens in a remote SSH connection where the shell is bash.

What version of vscode are you using?

Hi, sorry for being that brief. I'm currently using:
Version: 1.55.2 (system setup)
Commit: 3c4e3df9e89829dce27b7b5c24508306b151f30d
Date: 2021-04-13T09:35:57.887Z
Electron: 11.3.0
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042

Duplicate of #120810


The extension doesn't work for short periods of time now and then. [I'm not sure if this is a bug]

Describe the issue/bug:
The extension doesn't work at times [keeps loading; the Brainly Tools icon is yellow] though internet connectivity is pretty normal/fast, and the extension works as expected a couple of minutes later. [A few other moderators faced the same issue too]
To Reproduce:
Not sure of how it can be reproduced.
[Tried re-installing the extension, using different browsers/apps, refreshing the page, logging in and out. These steps didn't work]
Screenshots [paste below, optional]:
Error shown in the console: [Mods who faced the issue had this same error in their console as wel]

[PS: I'm not 100% sure if this is a bug, around 6 moderators faced the issue including myself (PC and mobile), so I thought of reporting it]

https://brainly.in/app/profile/2709968

Here the extension works 100%, but at Console, this error appears:

This error is happening when the new Function tries to return an Object from the Brainly's script.
A suggestion for this code to work: Try to use the RegExp /(?<=fos\.Router\.setData\().+scheme:"http"}/gim instead

@HydroXBR , No it's probably an extension error (index come from extension, but I agree there is also a Brainly error but it doesn't matter with the extension error)
He may be just did a syntax error.

Seems like one case where the issue happens is when a user with extension access makes a new login, previously the extension took a good few minutes to load in, but it's taking a lot more time to load in (hasn't started working since I made a new login). [On brainly.in, not sure about other domains]
Tried logging in on a new browser (Brave), tried with different Brainly accounts, tried on incognito mode, the extension isn't loading in completely all cases. [Only the theme color loads in]
https://user-images.githubusercontent.com/68370357/118348916-fe260f00-b56a-11eb-8f59-49539eafa688.mp4
On Brave:

Moderator who reported the issue: https://brainly.in/app/profile/4451081
There are a few other moderators who haven't made any new logins, the extension keeps loading at times for them as well. [Edge, Yandex & Kiwi Users]

Hello everyone,
@seahindeniz is aware about the issue. Brainly did some changes on the website and the extension can't work correctly.
@seahindeniz will release a patch when he will be free to work on this.
Thank you for your understanding.
Stay patient :)

Any update?

No new releases for the moment @hashed17, we all hope to have a patch soon. Power to Sahin 🔥

Hey, not sure if this is a common fix, but following these steps solved the issue for some moderators who were unable to use the extension on PC:

Click on the three dots at the top right corner. [Menu option]
Click on 'Settings'.
Search for 'Cache'.
Click on 'Clear browsing data'.
Select 'Hosted app data', you may unselect the rest.
Click on 'Clear data'.


Update index.mdx

Summary
Damarcus Jones Ai chips
Screenshots (optional)

Documentation checklist

[x] The documentation style guide has been adhered to.
[x] If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
[x] Files which have changed name or location have been allocated redirects.

🧑🏾💻


Removed my packages

Sorry, I can no longer keep my packages up to date. My Stack installation has broken beyond repair by reinstallation, and I don't have time to fix that right now.

Moved to abandoned packages.
If you want to take care of them again in Stackage later, of course you're welcome to.

Ok, thank you.


Row [clickable] hover not working

There is no hover state when 'clickable' is TRUE. rowClick works though.
using: "@covalent/core": "^1.0.0-beta.5-1"

Hmm, does this happen in a particular browser?

Newest Google Chrome.

Hmm, cant seem to replicate this? Can you post a plnkr with the issue?

no traction on issue in over a year, closing


Clear separation for projects below <200K EOS worth of TVL

Update

[x] Add line to separate projects above >6M EOS worth of TVL  (max)

[x] Show maximum TVL requirement to reach above line

[x] Add line to separate projects below <200K EOS worth of TVL  (min)

[x] Show minimum TVL requirement to reach above line

[x] Add visual aid (color tag)

[x] Unpaid (Color=Red) below TVL
[x] Paid (Color=Green) within TVL
[x] Max (Color=Green) above TVL

Data configs

min_tvl_report
max_tvl_report
https://bloks.io/account/eosio.yield?loadContract=true&tab=Tables&account=eosio.yield&scope=eosio.yield&limit=100

Other Examples

Implemented


Allow additional ports to be passed to proxy

This might be needed in a debugging session later today.

Is it worth adding a debug option to check-connecion.ts that tries to connect to all provided ports?

Hmm, maybe, but I'm not sure what kind of request we'd make. For the app URL we know it should respond to a GET HTTP request because it's serving a webpage but it's not so obvious for the other ports.
I suggest we wait until we have a concrete case for this.


Add native support for modifying the log level programmatically at runtime

Summary
We need a new Java API to change the log level programmatically at runtime without restarting the application. At the moment, the only way of changing the log level is from the Config.toml file that needs to restart the application. We need a new API to change the level programmatically.

Related issue: https://github.com/ballerina-platform/ballerina-library/issues/4633

The proposed native API
  /**
   * Set the log level dynamically globally or for a specific module.
   * This is a native implementation of the setLogLevel function in the log-api module.
   *
   * @param logLevel   The log level to be set
   * @param packageName The package name of the module
   * @param moduleName  The module name
   */
  public static void setLogLevel(Environment env, String logLevel, String packageName, String moduleName);

This log level change is applied temporary for the running instance and it is not persisted. It resets after restarting the instance.

This support needs calling the ballerina util method from the Java side. This support is addressed in the issue[1]

https://github.com/ballerina-platform/ballerina-lang/issues/42128

This task is deprioritized because the initial control plan version will not support changing log levels in Ballerina nodes. So we are not going to ship this support in update 09. Move this issue onto hold till we get the requirement.


Boardeaser: Cleanup and build in repo

Sorry, I messed up here... To quick with where it was pushed!


Configure user.name in git_test.go run method.

In my work to add downstream testing to knative.dev/pkg this test is failing with:
        git_test.go:216: ["error" git]: Error running git [commit --allow-empty -m Hello Moto]: exit status 128
            Author identity unknown

            *** Please tell me who you are.

            Run

              git config --global user.email<EMAIL_ADDRESS>              git config --global user.name "Your Name"

            to set your account's default identity.
            Omit --global to set the identity only in this repository.

            fatal: empty ident name (for<EMAIL_ADDRESS>not allowed

Ref: https://github.com/knative/pkg/pull/2233/checks?check_run_id=3423565615#step:6:528
Tekton already configures user.email, but it seems to reject this on Actions without user.name as well.
/kind bug
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Docs included if any changes are user facing
[ ] Tests included if any functionality added or changed
[ ] Follows the commit message standard
[ ] Meets the Tekton contributor standards (including
functionality, content, code)
[ ] Release notes block below has been filled in or deleted (only if no user facing changes)

Release Notes
NONE

/retest


Enable extra-hardware when running Ironic introspection

Including the extra-hardware flag when running introspection
collects additional data for disks, bios, nics, cpus, kernel etc.

Build SUCCESS, see build http://<IP_ADDRESS>:8080/job/dev-tools/137/


Upgrade the default graphcool API endpoint playgrounds to graphql-playground

Because it's awesome :smiley:

Done 🙂

There's only a slight stylesheet issue between graphqlbin.com and the API endpoints on tabs, buttons and query text. Is that deliberate?

Ok it seems the font is not loaded properly, will have to look into that


Make the log name an required argument to all the examples

@brifordwylie - did you want me to add 'required=True' for the '-f' argparse argument to all the examples? It would make it a little bit tidier.
I can do that to all of them and then add as one pull request and approve it myself if you're ok with the action?
Cheers, Mike

@swedishmike that's a good idea, we might also just make it a positional argument... so that's it more like unix command line tools. So
python blah.py filename

I think I like that even better, if you want to give it a swing feel free.
Here's the docs on that if you need it: https://docs.python.org/2/library/argparse.html
Approving PRs yourself is fine for smaller stuff like this, obviously just make sure all the automated tests pass and do a few yourself.
Cheers and thanks so much for contributing to the project, it's great to have the help.

Sure thing - I'll give that a go and will test it locally thoroughly before even submitting the PR for automated tests.
I'll probably submit this tomorrow - gives me time to test and 'make it right'.
Absolutely no worries at all - it's all my pleasure. It's a great project and I'm happy that I can contribute in a small way.
Cheers, Mike

You probably know all this already but since your a dev on the project you can just check out this repo make a local branch and then push that local branch and make a PR
$ git checkout -b my_branch
<work on stuff>
$ git push -u origin my_branch

Yeah - just wanted to check that it was ok with you to do it that way and just didn't get around to it yet. ;)
Will do that from now on.

@brifordwylie All tested and merged now. Have a look and see what you think - if you're happy with it let me know and I'll close this issue.

Looks great :)


Verify Navier-Stokes mass and momentum

Verify against the lid-driven cavity benchmark.
Relevant FEniCS examples:

https://fenicsproject.org/olddocs/dolfin/1.6.0/python/demo/documented/navier-stokes/python/documentation.html
https://fenicsproject.org/olddocs/dolfin/1.6.0/python/demo/documented/stokes-taylor-hood/python/documentation.html

Complete as of commit  499a487d6049665e433588ced41ef295a11c3b3d


Setting Host headers causes: TypeError: hostHeader.startsWith is not a function

Since version 2.1.0, setting Host header causes fetch call to fail. This is a minimal example
const fetch = require('node-fetch');
(async () => {
    const resp = await fetch('https://google.com', { headers: { 'Host': 'example.com' } });
    console.log(resp);
})();

that throws following exception:
(node:5895) UnhandledPromiseRejectionWarning: TypeError: hostHeader.startsWith is not a function
    at calculateServerName (_http_agent.js:240:20)
    at Agent.addRequest (_http_agent.js:155:26)
    at new ClientRequest (_http_client.js:257:16)
    at request (https.js:245:10)
    at /home/user/tmp/node-fetch-host-header/node_modules/node-fetch/lib/index.js:1355:15
    at new Promise (<anonymous>)
    at fetch (/home/user/tmp/node-fetch-host-header/node_modules/node-fetch/lib/index.js:1342:9)
    at /home/user/tmp/node-fetch-host-header/index.js:4:24
    at Object.<anonymous> (/home/user/tmp/node-fetch-host-header/index.js:6:3)
    at Module._compile (module.js:660:30)
(node:5895) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 3)
(node:5895) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

In my production code I've also encountered TypeError: hostHeader.replace is not a function in almost identical code to the example above.

Here is a test case that reproduces the problem: https://runkit.com/gr2m/5aa826566fefc70013b04566

Weird, I thought this test should cover it?
https://github.com/bitinn/node-fetch/blob/master/test/test.js#L197-L209

@bitinn Lowercase header key, i.e. host, works. I don't know what the fetch spec[1] constraints the header keys, but it just felt natural to use what's on the wire in the end.
[1] https://fetch.spec.whatwg.org/

We needed a special handling on host header because nodejs http doesn’t allow it in headers (it’s set separately which we handled automatically)
I think we might have missed the capitalize version of Host. Should be easy to normalize.
But it’s always better to use the lowercase header name as nodejs http enforce lowercase header names anyway (not a fetch spec limit).
(From my phone)

On Mar 16, 2018, at 4:58, woky<EMAIL_ADDRESS>wrote:
@bitinn Lowercase header key, i.e. host, works. I don't know what the fetch spec[1] constraints the header keys, but it just felt natural to use what's on the wire in the end.
[1] https://fetch.spec.whatwg.org/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Seems like a duplicate of #416. Let's close this one.


Make improvements to FastHTML By Example

Small PR as I'm working my way carefully through this document.  I suspect I will make multiple PRs onto this document.
Changes made:

Use serve() instead of Uvicorn
Include links in places where there should be links
Make some of the language more succint (less is more)
Remove the bit about this doc serving as context because there is now llm-ctx.txt and friends
Update for new @rt decorator, and add a tabset panel so people can see the difference
Clarify some of the language assuming people are new to web development.
Move all Asides into Quarto callouts

Im only 25% of the way through just though I would open an early PR.  Don't want to make a big PR if these changes are not aligned with what you want to do!

[!Important]
I have generated a preview link for the page I changed to make reviewing easier on Netlify

cc: @johnowhitaker @jph00

This is great! BTW the text "This is needed to tell FastHTML to turn the main body into a full HTML page that includes the headers (including the pico link and our custom css) which we passed in" is wrong. A single FT element is sufficient for this to work (cc @johnowhitaker since IIRC you wrote that bit)

@pydanny you might be interested in this PR to see some nice nbdev/quarto features icymi

I'm a little confused by this

"This is needed to tell FastHTML to turn the main body into a full HTML page that includes the headers (including the pico link and our custom css) which we passed in" is wrong. A single FT element is sufficient for this to work (cc @johnowhitaker since IIRC you wrote that bit)

I can't seem to figure out how to change the code exactly to do this without a tuple?
from fasthtml.common import *
css = Style(':root {--pico-font-size:90%,--pico-font-family: Pacifico, cursive;}') # <1>
app = FastHTML(hdrs=(picolink, css)) # <2>

@app.route("/")
def get():
    return (Title("Hello World"),
               Main(H1('Hello, World'), cls="container"))

I've updated that sentence now to be more accurate.


Remove AGRC version of Utah Quaternary Faults from AGOL

Summary
Current AGRC version of Utah Quaternary Faults need to be removed from AGOL, they are being replaced by a version from UGS that is shared to the Geoscience group.

Triage

[x] Data Team Triage (@gregbunce)
[x] Dev Team Triage (@steveoh)
[x] Cadastre Team Triage (@rkelson)

Remove data from the following areas

[x] ArcGIS Online (@ZachBeck )
~[ ] Internal SGID (assigned to)~
~[ ] SGID10 (assigned to)~
~[ ] Open SGID (assigned to)~
[x] Google Drive (@ZachBeck)
~[ ] Open data (assigned to)~
~[ ] Mapserv (assigned to)~
~[ ] Retire application-specific prod and test SQL Databases (assigned to)~

Copy data to the following areas

[ ] Upload to UtahAGRC/AGRC_Shelved folder in AGOL (assigned to)
[x] Move existing AGOL item to AGRC_Shelved AGOL folder (@ZachBeck )
[ ] Upload to appropriate UtahAGRC/{SGID Category} folder in AGOL (for "static" datasets) (assigned to)

Update information in the following areas

[x] gis.utah.gov data pages (@ZachBeck)
~[ ] Stewardship doc and sgid-index (assigned to)~
[x] SGID.META.AGOLItems (@ZachBeck)
[x] AGRC_Shelved/AGOLItems_Shelved (in AGOL) (@ZachBeck)
[ ] SGID.META.ChangeDetection (assigned to)
~[ ] Backseat Driver (assigned to)~

Is there a website?

~[ ] Archive source code repository (assigned to)~
~[ ] Remove from the web server (assigned to)~
or
~[ ] Replace app with a static page with information (assigned to)~
or
~[ ] Redirect somewhere else (assigned to)~

Is there a map service?

~[ ] Stop (assigned to)~
~[ ] Delete (assigned to)~

Is there a forklift pallet?

[ ] Remove repo (assigned to)
[ ] Remove stale data from forklift hashing and receiving (assigned to)
[ ] Remove row from data/hashed/changedetection.gdb/TableHashes

Are there service dependencies?

[ ] Validate that the web api does not query it (assigned to)
[ ] Validate that AGRC or applications widgets do not reference it (assigned to)

Notification

[ ] Blog post (assigned to)
[ ] Twitter (@steveoh)

[ ] Intent
[ ] Completed

conductor results for tasks - 76

check
status

gregbunce has completed 1 out of 1 tasks
:+1:

@steveoh has completed 1 out of 2 tasks
:no_entry:

rkelson has completed 1 out of 1 tasks
:+1:

ZachBeck has completed 6 out of 6 tasks
:+1:

conductor results for tasks - 76

check
status

gregbunce has completed 1 out of 1 tasks
:+1:

steveoh has completed 2 out of 2 tasks
:+1:

rkelson has completed 1 out of 1 tasks
:+1:

ZachBeck has completed 6 out of 6 tasks
:+1:


first() turns an update query into a select

> knex('pool_enterprise.test').update({'state': 'awesome'}).first().toSQL()
{ method: 'first',
  options: {},
  timeout: false,
  cancelOnTimeout: false,
  bindings: [ 1 ],
  __knexQueryUid: '3698eb45-5165-46af-aa43-9376e048cb27',
  sql: 'select * from "pool_enterprise"."test" limit ?' }

Probably should just throw an error? I mean you can do an UPDATE RETURNING but I don't seen LIMIT clause allowed in the postgres docs
Tested on postgres.
How it happened is I had a limit supposed to be on a subquery but I got the parens wrong. Was a bit hard to debug.
Knex version:
<EMAIL_ADDRESS>  version "0.12.9"
  resolved "https://registry.yarnpkg.com/knex/-/knex-0.12.9.tgz#aa852138c09ed46181e890fd698270bbe7761124"

Thanks for the report. Throwing error sounds good.
Also probably knex should throw an error in every case where calling new builder method changes some update -> plain select.

Experiencing a similar issue when changing first() with del()

Hm, I kinda assumed update().first() would return a single updated row, instead of an array. Maybe a .single() method would be useful for such cases, if we want to avoid having a double meaning for .first()?

A note on the above, for posterity.
Since knex uses bluebird promises under the hood, it is possible to use the .spread method, which flattens returned array into function arguments.
knex('foo')
  .where('id', 1)        // match by PK, so we could only ever have one row updated
  .update({ s: 2 }, '*') // yet this normally would resolve to an array of 1 rows
  .spread((row) => {})   // .spread flattens the returned array into a single arg

Yes. It would make sense for update().first() to try to return first updated item if possible. But that throwing error in impossible cases would still be good to have.

With .returning not being supported by all dialects it might not be good to give .first() two different meanings between dialects.
I would opt for just throwing an error that update was changed to select.


sway: focus.followMouse supports yes/no/always

Description
Fixes #1197.
Checklist

[x] Code formatted with ./format.

[x] Code tested through nix-shell --pure tests -A run.all.

[x] Test cases updated/added. See example.

[x] Commit messages are formatted like
{component}: {description}

{long description}

See CONTRIBUTING for more information and recent commit messages for examples.

If this PR adds a new module

[ ] Added myself as module maintainer. See example.

[ ] Added myself and the module files to .github/CODEOWNERS.

build failure seems unrelated.

About the build failure. It is due to formatting not being as expected, see lines 299–300 in the log.

@rycee I re-implemented this per your suggestion and added explicit tests for:

sway with bool
sway with string
i3 with bool

Thanks for the guidance.

Thanks for reviewing @alexarice. I addressed your comments and amended/force-pushed.

@rycee Is there a way to rebase this onto master? Status checks seem to stop doing it manually and the option is greyed out in github

Only seem to have "squash and merge" available, though it would be nice to keep the commits separate if possible (possibly this is just my lack of github knowledge showing and there is a way to do this)

I think it's fair to squash these since the tests are directly related to the initial commit. I did so and also added dummy packages for sway-unwrapped, swaybg, and xwayland to avoid downloading these in the CI job. It's in master now since commit 5c639ff68abf0f16f2e63fb3f8aca016d54c7d10.
Many thanks for the contribution @colemickens and the nice reviewing @alexarice!


fix: added download excel and csv

Added export to excel and css
Can consider adding https://docs.sheetjs.com/docs/demos/grid/xs/ for viewing the responses

LGTM Merging


Add uris to the CommandInfo when a custom executor is set

@drexin: thank you for the contribution!


Content filter is out of control

The past few times I've tried to take a photo of a meal, I've been told that my image is too racy. Even with a dozen different angles, the same every time.
Steps to reproduce the behavior:

Take photo of meal -- the last two I've had an issue with are the red curry and the Leberkäse, but it happens for almost every meal.
See error, something about racy content, detected level 2, allowed level 1 (sorry, I didn't take a screenshot).

I would expect to be able to upload photos of meals, since that is part of the app's core functionality. I imagine that this bug it the reason why there are often no photos for meals.
I will provide screenshots the next time I encounter this issue.
I have an iPhone XR with iOS 17.5.1, the app version is 1.1.1.

I have looked through the logs and seen a couple (242 since April 2024) of declined images that got rejected because of adult, spoof, or racy level 2, where we currently only allow level one. I now bumped all allowed levels to 2.
For comparison, in the same time period, 137 images got rejected because of a higher detected level.
The levels are those from Google could vision safe search, where the levels meaning can be seen here.


xArchive: Not Compatible with forward slash

ISSUE DESCRIPTION:
When the target archive is the attached archive immediately below this, the object returned by System.IO.Compression.ZipFile is delimited by "/" rather than by "\"
MSBuild_12_WebApplicationTargets.zip
Details of the scenario you tried and the problem that is occurring
When I try to extract the attached archive, then "if ($archiveEntryFullName.EndsWith('\'))" resolves to false for the "Web/" folder and treats it as a file, and then it prints the ItemWithArchiveEntryNameIsNotFile message, and the test returns false, even though it should be true.
Verbose logs showing the problem
LCM:  [ Start  Test     ]  [[xArchive]MSBuild12WebApplicationTargets  "},
                           [[xArchive]MSBuild12WebApplicationTargets Retrieving the state of the archive with path MSBuild_12_WebApplicationTargets.zip and destination C:\\Program Files (x86)\\MSBuild\\Microsoft\\VisualStudio\\v12.0..."},
                           [[xArchive]MSBuild12WebApplicationTargets A directory already exists at the destination path C:\\Program Files (x86)\\MSBuild\\Microsoft\\VisualStudio\\v12.0."},
                           [[xArchive]MSBuild12WebApplicationTargets Testing if the archive at the destination path C:\\Program Files (x86)\\MSBuild\\Microsoft\\VisualStudio\\v12.0 exists..."},
                           [[xArchive]MSBuild12WebApplicationTargets Opening the archive at path MSBuild_12_WebApplicationTargets.zip..."},
                           [[xArchive]MSBuild12WebApplicationTargets An item with the same name as the archive entry exists at the destination path C:\\Program Files (x86)\\MSBuild\\Microsoft\\VisualStudio\\v12.0\\Web\\."},
                           [[xArchive]MSBuild12WebApplicationTargets The item at the destination path C:\\Program Files (x86)\\MSBuild\\Microsoft\\VisualStudio\\v12.0\\Web\\ has the same name as a file archive entry but is not a file."},
                           [[xArchive]MSBuild12WebApplicationTargets Closing the archive at path MSBuild_12_WebApplicationTargets.zip..."},
                           [[xArchive]MSBuild12WebApplicationTargets The archive at path MSBuild_12_WebApplicationTargets.zip does not exist at the destination C:\\Program Files (x86)\\MSBuild\\Microsoft\\VisualStudio\\v12.0."},
                           [[xArchive]MSBuild12WebApplicationTargets Testing whether or not the state of the archive with path MSBuild_12_WebApplicationTargets.zip and destination C:\\Program Files (x86)\\MSBuild\\Microsoft\\VisualStudio\\v12.0 matches the desired state..."},
LCM:  [ End    Test     ]  [[xArchive]MSBuild12WebApplicationTargets  in 0.0330 seconds."},
LCM:  [ Start  Set      ]  [[xArchive]MSBuild12WebApplicationTargets  "},
                           [[xArchive]MSBuild12WebApplicationTargets Setting the state of the archive with path MSBuild_12_WebApplicationTargets.zip and destination C:\\Program Files (x86)\\MSBuild\\Microsoft\\VisualStudio\\v12.0..."},
                           [[xArchive]MSBuild12WebApplicationTargets A directory already exists at the destination path C:\\Program Files (x86)\\MSBuild\\Microsoft\\VisualStudio\\v12.0."},
                           [[xArchive]MSBuild12WebApplicationTargets Expanding the archive at MSBuild_12_WebApplicationTargets.zip to the destination C:\\Program Files (x86)\\MSBuild\\Microsoft\\VisualStudio\\v12.0..."},
                           [[xArchive]MSBuild12WebApplicationTargets Opening the archive at path MSBuild_12_WebApplicationTargets.zip..."},
                           [[xArchive]MSBuild12WebApplicationTargets An item with the same name as the archive entry exists at the destination path C:\\Program Files (x86)\\MSBuild\\Microsoft\\VisualStudio\\v12.0\\Web\\."},
                           [[xArchive]MSBuild12WebApplicationTargets Closing the archive at path MSBuild_12_WebApplicationTargets.zip..."},
LCM:  [ End    Set      ]  [[xArchive]MSBuild12WebApplicationTargets  in 0.0460 seconds."},

Suggested solution to the issue
See https://github.com/PowerShell/xPSDesiredStateConfiguration/blob/9fd047b90f3d74c4a83e0ae0011da1af5b916bbd/DSCResources/MSFT_xArchive/MSFT_xArchive.psm1#L1153
Update the if statement so that it checks for both "/" and "\" ?
The DSC configuration that is used to reproduce the issue (as detailed as possible)
# insert configuration here

    @(12, 14) | ForEach-Object {
        xArchive "MSBuild$($_)WebApplicationTargets"
        {
            Destination = "C:\Program Files (x86)\MSBuild\Microsoft\VisualStudio\v$_.0"
            Path = Join-Path $InstallerPath "MSBuild_$($_)_WebApplicationTargets.zip"
        }
    }

The operating system the target node is running
OsName               : Microsoft Windows Server 2016 Datacenter
OsOperatingSystemSKU : DatacenterServerEdition
OsArchitecture       : 64-bit
WindowsBuildLabEx    : 14393.2608.amd64fre.rs1_release.181024-1742
OsLanguage           : en-US
OsMuiLanguages       : {en-US}
Version and build of PowerShell the target node is running
----                           -----
PSVersion                      5.1.14393.2608
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14393.2608
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Version of the DSC module that was used ('dev' if using current dev branch)
<IP_ADDRESS>

So.... this issue is blocking me from upgrading....


Add service discovery logs

Proposed Changes
The issues regarding communication with Consul are quite common. Currently the Ocelot logs don't show the basic information about this communication. I think it could be very useful for debugging purposes.

Thank you for the PR!
Could you Sync fork please? So, your develop branch is outdated!

The feature branch has been rebased onto ThreeMammals:develop!
Welcome to code review!

@TomPallister
This PR is ready to merge.


3035 update docs with pendo info

Closes #3035

Replaced the section labeled telemetry and the section labeled gitops CLI with the new documentation (as stated in the issue description).

Notes:

I've tried to match the original document formatting, so, when there were commands but just in bold font instead of backticks, I've just used bold markdown formatting. Also preserved all spaces (where there are two spaces in the doc I kept them).
I've only changed "Weave Gitops CLI analytics" to "Weave GitOps CLI analytics" because this is how we usually capitalize it and how it is capitalized throughout the doc.

Please see the notes.
@ozamosi sent you the link to the doc @josefaworks provided on Slack.


Fix allocating repeated measure entities

If we're trying to allocate a MEASURE_ token (a measure entity
copied from context), try reusing an existing value that was previously
allocated, rather than checking for a NUMBER_ entity and failing.
This was originally reported at https://community.almond.stanford.edu/t/generating-dataset-entity-error/269/8

Pull Request Test Coverage Report for Build 884

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.924%

Totals

Change from base Build 883:
0.0%

Covered Lines:
8391

Relevant Lines:
10102

💛  - Coveralls


Multi source in multi url

Hi,
Current Behavior
Mixed youtube, dailymotion, soundcloud urls. The player don't appear.
Expected Behavior
Would it be possible to mixe sources in the url array ?
Example
url={[
"https://www.youtube.com/watch?v=oUFJJNQGwhk",
"https://soundcloud.com/bgrz_music/aya-nakamura-feat-stormzy-plus-jamais-bgrz-kompa-remix-1",
"https://www.dailymotion.com/video/x7yt6x8",
"https://www.youtube.com/watch?v=jNgP6d9HraI",
"https://www.youtube.com/watch?v=ysz5S6PUM-U",
]}

Arrays of URLs only work for Youtube URLs, as it just proxies the built-in playlist function.
Also see https://github.com/cookpete/react-player/issues/1099#issuecomment-741646073

Arrays of URLs only work for Youtube URLs, as it just proxies the built-in playlist function.
Also see https://github.com/cookpete/react-player/issues/1099#issuecomment-741646073


RequestFactory.php added override

I have the use case where I just want to update one key in the form request and I don't want it to be like $factory->without('foo)->override('bar', 123). So I created a PR for it.

I wrote it in a simpler fashion myself. 🤷♂


feat: implement generic webapp processor

This PR adds a generic resource processor for web applications. Future proof. No hardcoded list of properties required.

The CIs are failing with a <protect> error except for I4H 2022.1, probably because of {$dbrole} is evaluated to :%DB_USER-VERIFY-DATA rather than :%DB_USER-VERIFY.
Only I4H 2022.1 evaluates this to :%DB_USER-VERIFY hence it passes.
I tried both
Do ##class(%SYS.Namespace).GetNSInfo($NAMESPACE, .nsInfo)
Return $Get(nsInfo("Resource"), "%DB_DEFAULT")

and
Set db = ##class(SYS.Database).%OpenId(dbDir)
Return $SELECT($IsObject(db): db.ResourceName, 1: "%DB_DEFAULT")

but neither seems to be giving the correct result, except on I4H 2022.1.
@isc-kiyer @isc-tleavitt Any clue why this is the case?

The CIs are failing with a <protect> error except for I4H 2022.1, probably because of {$dbrole} is evaluated to :%DB_USER-VERIFY-DATA rather than :%DB_USER-VERIFY.

The error is:
ERROR #5002: ObjectScript error: <PROTECT>^%CSP.REST.1 ^|^^\/usr\/irissys\/mgr\/USER-VERIFY-CODE\/|Test.Rest.Demo.1
That is, there is a lack of read privilege on the routine database. Our CI setup for verify creates separate routine and data DBs for the temporary namespace used.
This is actually a common/annoying problem - see e.g. https://github.com/intersystems/isc-perf-ui/blob/main/cls/pkg/isc/perf/ui/Installer.cls#L11 which is called from an invoke at https://github.com/intersystems/isc-perf-ui/blob/main/module.xml#L68. From a defense in depth perspective, we generally would only want to grant read privilege on the routine database, but such a role does not inherently exist (we need to create it as this automation does).
IMO the right solution is to bake the automation from this installer into IPM itself and make it easier to use/manage.

@isc-tleavitt Point of clarification: do we want to call this method on all web applications after they created, or only on those that allow unauthenticated access?

@isc-tleavitt Point of clarification: do we want to call this method on all web applications after they created, or only on those that allow unauthenticated access?

Great question - I think we should make it easy to opt in to it. Let's say that similar to ${dbRole} we have ${codeReadRole} and we automatically manage an appropriate role (derived from the package name and namespace name to ensure uniqueness system-wide) - that is, create if referenced, and on uninstallation delete.
Given that we need to do this with duplicated boilerplate code for every web application using isc.ipm.js or similar less-advanced constructs, I'm a big fan of automating it.
@isc-kiyer any concerns/objections here?

The CIs are failing with a <protect> error except for I4H 2022.1, probably because of {$dbrole} is evaluated to :%DB_USER-VERIFY-DATA rather than :%DB_USER-VERIFY.

The error is: ERROR #5002: ObjectScript error: <PROTECT>^%CSP.REST.1 ^|^^\/usr\/irissys\/mgr\/USER-VERIFY-CODE\/|Test.Rest.Demo.1
That is, there is a lack of read privilege on the routine database. Our CI setup for verify creates separate routine and data DBs for the temporary namespace used.
This is actually a common/annoying problem - see e.g. https://github.com/intersystems/isc-perf-ui/blob/main/cls/pkg/isc/perf/ui/Installer.cls#L11 which is called from an invoke at https://github.com/intersystems/isc-perf-ui/blob/main/module.xml#L68. From a defense in depth perspective, we generally would only want to grant read privilege on the routine database, but such a role does not inherently exist (we need to create it as this automation does).
IMO the right solution is to bake the automation from this installer into IPM itself and make it easier to use/manage.

@isc-tleavitt @isc-shuliu for this, why not look up the role for the routine database? It does mean we can no longer use the %SYS.Namespace call and would need to use Config.Databases but that's fine. I recommend two expressions: $routinesDbRole and $globalsDbRole to make available for evaluation. For web apps, would need routines at a minimum but may also want to grant access to globals (so the user role doesn't need access to the database. This is what we do in HS for example).

The CIs are failing with a <protect> error except for I4H 2022.1, probably because of {$dbrole} is evaluated to :%DB_USER-VERIFY-DATA rather than :%DB_USER-VERIFY.

The error is: ERROR #5002: ObjectScript error: <PROTECT>^%CSP.REST.1 ^|^^\/usr\/irissys\/mgr\/USER-VERIFY-CODE\/|Test.Rest.Demo.1
That is, there is a lack of read privilege on the routine database. Our CI setup for verify creates separate routine and data DBs for the temporary namespace used.
This is actually a common/annoying problem - see e.g. https://github.com/intersystems/isc-perf-ui/blob/main/cls/pkg/isc/perf/ui/Installer.cls#L11 which is called from an invoke at https://github.com/intersystems/isc-perf-ui/blob/main/module.xml#L68. From a defense in depth perspective, we generally would only want to grant read privilege on the routine database, but such a role does not inherently exist (we need to create it as this automation does).
IMO the right solution is to bake the automation from this installer into IPM itself and make it easier to use/manage.

@isc-tleavitt @isc-shuliu for this, why not look up the role for the routine database? It does mean we can no longer use the %SYS.Namespace call and would need to use Config.Databases but that's fine. I recommend two expressions: $routinesDbRole and $globalsDbRole to make available for evaluation. For web apps, would need routines at a minimum but may also want to grant access to globals (so the user role doesn't need access to the database. This is what we do in HS for example).

The role for the routine database would include write permission. Principle of least privilege means that we shouldn't grant the ability to modify code as an application role; we need a separate role that grants only read permission. That's what Shuheng has implemented based on the sample code I've copied to half a dozen projects that need to do this.

The CIs are failing with a <protect> error except for I4H 2022.1, probably because of {$dbrole} is evaluated to :%DB_USER-VERIFY-DATA rather than :%DB_USER-VERIFY.

The error is: ERROR #5002: ObjectScript error: <PROTECT>^%CSP.REST.1 ^|^^\/usr\/irissys\/mgr\/USER-VERIFY-CODE\/|Test.Rest.Demo.1
That is, there is a lack of read privilege on the routine database. Our CI setup for verify creates separate routine and data DBs for the temporary namespace used.
This is actually a common/annoying problem - see e.g. https://github.com/intersystems/isc-perf-ui/blob/main/cls/pkg/isc/perf/ui/Installer.cls#L11 which is called from an invoke at https://github.com/intersystems/isc-perf-ui/blob/main/module.xml#L68. From a defense in depth perspective, we generally would only want to grant read privilege on the routine database, but such a role does not inherently exist (we need to create it as this automation does).
IMO the right solution is to bake the automation from this installer into IPM itself and make it easier to use/manage.

@isc-tleavitt @isc-shuliu for this, why not look up the role for the routine database? It does mean we can no longer use the %SYS.Namespace call and would need to use Config.Databases but that's fine. I recommend two expressions: $routinesDbRole and $globalsDbRole to make available for evaluation. For web apps, would need routines at a minimum but may also want to grant access to globals (so the user role doesn't need access to the database. This is what we do in HS for example).

@isc-kiyer the role for the routine database would include write permission. Principle of least privilege means that we shouldn't grant the ability to modify code as an application role; we need a separate role that grants only read permission. That's what Shuheng has implemented based on the sample code I've copied to half a dozen projects that need to do this.

Ah ok that makes sense. For HS, our code databases are RO so although not ideal, granting write permission is fine.


Province of City is always wrong -- Only in belgium ..

Dont know if I'm doing something wrong but everything is correct except the Province of the city. Is this an issue observed by others aswel?

Nevermind, only seems to be the case in Belgium so it seems. No idea why though. Guess it's kinda like our government.

Update: in your file geoname2fips.py you write
VLAMMS BRABANT': 'FLANDERS'

But It's supposed to be VLAAMS BRABA?T.
However I cant seem to make it write a fips csv from it (absolutely clueless what parameter its accepting) so I can't test if that fixed worked or not...
Indeed, all the flemish cities are being marked as being in flemish brabant, fixing that to flanders would indeed fix it.
Can anyone help?

I found out the problem. Please take a look at following pull request:
https://github.com/sherpya/geolite2legacy/pull/26
I now have all the provinces working instead of just the general region (which was wrong and fixed on a single province for all cities)

this is fixed but I need to generate new csv, did you modified it by hand?
perhaps it needs to be converted in python module

I generated it with a bash script, I couldn't get the python module to work properly. I simply took a look of the geolite's extracted location file for the used BE provnames and at the fips 10-4 province id's and then generated the csv with all the correct ids for all BE cities, and replaced it with all the wrong be entries. I'll leave it to you to decide what to do in the python code...


Increment versions for machinelearning releases

Increment package versions for machinelearning releases

API change check
API changes are not detected in this pull request.


Background Event errors when trying to load a scene as a background

The problem
Describe the bug
When you set a Background event to use a scene as a background, it fails to do so.
First issue here:

Second:

If I comment out those two lines, then I get these:

and when I comment out all both
#new_node.get_meta('node')._update_background(argument, fade_time)
#new_node.get_meta('node')._custom_fade_in(fade_time)

it finally works and it runs the timeline with the custom background as expected.
System (please complete the following information):

Godot Version: 4.2.stable
Dialogic Version: (latest, main)

Solutions
Commenting out these lines work, so I guess something might have changed since the last time this was implemented.

I think that the reason why this problem pops up is because the scene you tried to use did not have a script extending the DialogicBackground class.
There is actually a check in place if the script on the root of the scene is (extending) DialogicBackground, but this check happens after the damage has been done.
There are really 2 ways for this problem to be solved:

put the check in the add_background_node function and enforce the developer to extend from DialogicBackground
make the code in the subsystem check for the existence of functions instead of relying on typing to dictate functionality.

The assignment of a viewport is not strictly required inside of the background subsystem, as it gets fetched from the newly fabricated node anyways, but it could be used inside of the background scene itself for advanced use. a bit of an edge case. If going with the second way of solving this problem, one way to go about the viewport property is to check if it exists before assigning it, if we want to keep it.


chore(deps): aws dependencies bump

Description of your changes

Fixes #
I have:

[ ] Read and followed Crossplane's contribution process.
[ ] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

example MR https://github.com/crossplane-contrib/provider-aws/pull/1686
and https://github.com/crossplane-contrib/provider-aws/pull/1667
todo

snyk validation
story https://github.com/crossplane-contrib/provider-aws/issues/1628


Web Chat Customization doc links result in 404s

Links to defaultStyleOptions.js file and createStyleSet.js file are 404.

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

ID: 2d32e774-cb21-7ebc-f5c4-71edf8cf05e9
Version Independent ID: 1403c378-79e7-a966-4d5b-a9f3f18e0790
Content: Web Chat customization - Bot Service
Content Source: articles/v4sdk/bot-builder-webchat-customization.md
Service: bot-service
GitHub Login: @ivorb
Microsoft Alias: kamrani

Solved


Plan to make it a scoped package

Curious to know if this package will eventually become a scope package like @polymer/pwa-helpers instead of pwa-helpers?

@motss We have talked about this but have not made the final decision yet. I am hoping we can have the decision in the next month or two.

Cool. Thanks for answering this. Glad to know that the team is making good progress. 👍

We decided to make it not a scoped package under @polymer. Similar to lit-element and lit-html.


[Performance] Timm_models beit_base, gmixer_2's performance dropped >20% compared to v2.1

bash inductor_xpu_test.sh timm_models amp_bf16 training performance xpu 0 static 1 0 beit_base_patch16_224

Quick check the E2E performance profile status.
The eager mode has almost the same kernel performance when the kineto profiler is enabled:
LLVM-TARGET:
Self CPU time total: 731.982ms
Self XPU time total: 149.012ms
SPIRV-TARGET:
Self CPU time total: 687.606ms
Self XPU time total: 148.457ms

But the CPU host time is unreasonable when the kineto profiler is enabled. We can skip the host time for now.
The graph mode has almost the same kernel performance too.
LLVM-TARGET:
Self CPU time total: 681.791ms
Self XPU time total: 126.993ms
gemm_kernel         0.00%       0.000us         0.00%       0.000us       0.000us      69.137ms        54.44%      69.137ms     315.694us           219
Triton kernel total time is: 58ms

SPIRV-TARGET:
Self CPU time total: 544.637ms
Self XPU time total: 124.709ms
gemm_kernel         0.00%       0.000us         0.00%       0.000us       0.000us      68.894ms        55.24%      68.894ms     314.584us           219
Triton kernel total time is: 55ms

Need to check the GPU bubble time further.

The performance gap is caused by the same issue as #707.
The performance of the LLVM target is good as SPIRV target with the torch/ipex.
Torch branch: https://github.com/intel-innersource/frameworks.ai.pytorch.private-gpu/tree/dev/triton-test-3.0
IPEX branch: https://github.com/intel-innersource/frameworks.ai.pytorch.ipex-gpu/tree/dev/triton-test-3.0
IGC driver version: ii  libigc1                              1.0.15136.24-775~22.04                  amd64        Intel graphics compiler for OpenCL -- core libs
The performance of the
bash inductor_xpu_test.sh timm_models amp_bf16 training performance xpu 0 static 1 0 beit_base_patch16_224
bash inductor_xpu_test.sh timm_models amp_bf16 training performance xpu 0 static 1 0 gmixer_24_224
bash inductor_xpu_test.sh timm_models amp_bf16 training performance xpu 0 static 1 0 mixer_b16_224
bash inductor_xpu_test.sh timm_models amp_bf16 training performance xpu 0 static 1 0 swin_base_patch4_window7_224
bash inductor_xpu_test.sh timm_models amp_bf16 training performance xpu 0 static 1 0 gmlp_s16_224
bash inductor_xpu_test.sh timm_models amp_bf16 training performance xpu 0 static 1 0 sebotnet33ts_256
bash inductor_xpu_test.sh timm_models amp_bf16 training performance xpu 0 static 1 0 tnt_s_patch16_224
bash inductor_xpu_test.sh timm_models amp_bf16 training performance xpu 0 static 1 0 ese_vovnet19b_dw

beit_base_patch16_224: 1.154x
gmixer_24_224: 1.906x
mixer_b16_224: 1.224x

There are two issues found:

One IPEX commit set the default log level to WARN accidently. Unexpected L0 barrier causes the GPU bubble time increased.
The IPEX bench utils uses the SYCL barrier to measure the Triton kernel time. But it has issue in successfully to measure the Triton kernel. That causes autotune may chose a sub-optimal Triton kernel to be used.

The first issue has already been fixed on IPEX dev/triton-test-3.0 branch.
The second issue is the oneAPI 2024.0 regression issue. It can be resolved by setting UR_L0_IN_ORDER_BARRIER_BY_SIGNAL=0.


Prerender error and workarounds

Preact-cli comes with prerender which render your / route using server-side rendering technique. So if you run npm start successfully but npm run build produces some errors, this mean that the prerender is failing because of some code requires DOM to work.
Workarounds:

Wrap your with this to make sure it won't execute when prerendering.

if (typeof window !== "undefined") {
  // Your code here
}

Turn off preredering by run npm run build -- --no-prerender

Had the same issue using HashRouter. npm run build -- --prerender=false didn't work for me but the window !== "undefined" check did.

@nephix I updated the command. So sorry. npm run build -- --no-prerender

@thangngoc89 what's expected outcome of this issue? Friendly error message with steps how to write code which could be used on node side?
I guess we could wrap prerender with try catch block that checks for TypeErrors (possibly with some whitelist of variables like window, document, location, localStorage and others) and prints user friendly message with steps on how to fix this.

IMO the example app that preact-cli creates should show how to handle the issue if DOM is required on /

@rkostrzewski Yup. A friendly error for user.

Hi !
I try to prerender the default template with no extra code but it fails.
i simple use  npm run build and i got this :

Unable to read file: /Users/jorel/Projects/proto/prerender/src/external "preact"
Error: No valid exports main found for '/Users/jorel/Projects/proto/prerender/node_modules/preact'
method: HteQ
at: /Users/jorel/Projects/proto/prerender/src/external "preact":1:17

I'm on macOS Catalina with node 13.6.0
Did i miss something ?

The issue was coming from node 13.6. I siwtched to 12.6.2 and the prerender is working fine. The only issue that i see now is that the home page seems to not be rendered. I cannot find the html file in the build folder.

You might want to update to the latest preact version, that should fix the first problem you had


Added custom overrides for kube-rbac-proxy-self.

Description
I have discovered that kube-state-metrics tends tends to experience CPU throttling in my kubernetes cluster. I want to get CPU throttling down to a decent level below 10%. Right now I don't have a way to change this.
This is a picture from my cluster. The sidecar called  kube-rbac-proxy-self experiences a CPU throttling of 19%. This gives bad performance

I use thanos along prometheus.
Type of change
What type of changes does your code introduce to the kube-prometheus? Put an x in the box that apply.

[ ] CHANGE (fix or feature that would cause existing functionality to not work as expected)
[ ] FEATURE (non-breaking change which adds functionality)
[ ] BUGFIX (non-breaking change which fixes an issue)
[x] ENHANCEMENT (non-breaking change which improves existing functionality)
[ ] NONE (if none of the other choices apply. Example, tooling, build system, CI, docs, etc.)

Changelog entry
By looking at existing code I am extending  kube-rbac-proxy-self with the same set of overrides as a similar sidecar container kube-rbac-proxy-main. This way I get a pitch point where i can add more CPU resources to the container.
Added configurable default values for sidecar container kube-rbac-proxy-self in deployment kube-statate-metrics.

Test and Verification
Edit the file called examples/kustomize.jsonnet. Add the following block at line 7.
      kubeStateMetrics+: {
        kubeRbacProxySelf+: {
          resources: {
            limits: { cpu: '120m', memory: '40Mi' },
            requests: { cpu: '60m', memory: '20Mi' },
          }
        }
      }

Run the install procedures from README.md followed by the generate procedure from github actions.
jb install github.com/ahysing/kube-prometheus/jsonnet/kube-prometheus@custom-requests-kube-rbacc-proxy-self
make --always-make generate validate && git diff --exit-code

Make sure that the file manifests/kubeStateMetrics-deployment.yaml is changed properly with the new limits you expected.

Please concider my change.
I need this.

thanks PhilipGough . this helps me a lot.


Support scanning Rust binaries built with cargo auditable

cargo auditable is a project by Rust's Secure Code WG. It embeds the list of dependencies into the binary itself, so that it can then be audited for known vulnerabilities.
Auditing such binaries is already supported by cargo audit and Trivy. It would be nice to get support for it in osv-scanner as well.
cargo auditable is used for all Rust builds by at least 5 Linux distributions, including Alpine. A number of organizations use cargo auditable, but to the best of my knowledge only Microsoft has spoken about it publicly.
There is already a Go library for extracting this data, which should make the integration quite easy: https://github.com/microsoft/go-rustaudit
I am the principal author of cargo auditable and I'm happy to answer any questions you might have.

Thanks for the FR. this indeed looks like a great fit for OSV-Scanner.

@G-Rath is this something we can put in your backlog?


flogogen template for activity.json misses required fields

Current behavior (how does the issue manifest): The template for activity.json in the flogogen command misses the required field ref
Expected behavior: flogogen should generate correct activities
Minimal steps to reproduce the problem (not required if feature enhancement):

flogogen activity HelloWorld
cat activity.json <-- output misses the ref field

Please tell us about your environment (Operating system, docker version, browser & web ui version, etc): 0.5.6
Flogo version (CLI & contrib/lib. If unknown, leave empty or state unknown): 0.5.6

The same applies to the trigger.json :(


Add -h flag to specify HTTP headers

Flag to specify HTTP headers, i.e:
gobuster -h "Header1: test1" -h "Header2: test2"

Upvote!

Hey @barrracud4, any chance you could rebase this so that we can get it moving? Keen to get through these PRs while I have the chance.

Hey @OJ
Sorry for not answering for a long time, check this:
https://github.com/OJ/gobuster/pull/133

Thanks @barrracud4 I'll close this one then :)


IE11: pfe-health-index colors

Description of the issue
IE11 bug in rendering the colors for the health-index.
Impacted component(s)

pfe-health-index

Branch: fix-issue-992-pfe-health-index-colors


fixed resnet50 config and sleep issue

This is to address the issue: https://github.com/argonne-lcf/dlio_benchmark/issues/95


feat(client): ts-migration for layout index file, Intro Component and header test file

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Related to the issue #42256

Hey @Ismailtlem ,
I appreciate that this is still a draft, but would like to clarify:

Migration should be incremental and painless. If you find that fixing the type and lint errors can't be done without migrating all the related files, feel free to use @ts-ignore or eslint-disable. However, this should only be a last resort when the problems cannot be addressed otherwise.

We are passed the point of trying to get as many of these files refactored as possible in a short time. So, I welcome you to take your time with typing the file/s without using unnecessary ignore comments.
At most, I expect some @ts-expect-error and // eslint-disable-next-line comments due to things like Graphql not being typed, and anything touching the gatsby-node file.


Feature/ForecastControllers

Controllers for forecast data

For getAll probably not required. but certainly useful for filtering.
Source entity, name, and designator are probably the most other useful.


ProvisionedConcurency not working as expected.

It seems I can`t get ProvisionedConcurency to work as its state in the docs:
This is what I am currently doing
 this.api = new sst.ApiGatewayV1Api(this,"api",{

 (...)

routes: {
  [route]: {
            function: "---handler",
            currentVersionOptions: {
              provisionedConcurrentExecutions: 5,
            },
          }
       }
    }
    (...)
   const fn = this.api.getFunction(route);
   const version = fn.currentVersion;

}
When checking on the AWS console, I see that new version is created correctly, but the concurrency is not provisioned.
I then tried creating a alias via:
 version.addAlias("AUTO_GENERATE_ALIAS", {
          provisionedConcurrentExecutions: 1,
        });

And my alias does get auto generate, with the correct values, but whenever I hit my api endpoint, it does not go through the provisioned alias, it cold starts a new request.
If I go to the console , manually change the endpoint on API_GATEWAY to point to the lambda and REDEPLOY the api from the console, then it does hit the new alias as expected.
In essence I want to remove the need for us to do any manual setup on AWS console, is this possible? Can I set my api gateway to point to the alias in the code?
Quick questions:
Do I even need a alias in the first place for the provisionedConcurrentExecutions?
Am I missing a better way of setting this up?
Any pointers would be appreciated.

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


Black output images

Any idea why I would get completely black images? I'm running this on a VM with Ubuntu 15.04 (so no GPU).
Here's the final image (out_900.png) and console output for a test I did using example images and a small image size.

$ th neural_style.lua -style_image examples/inputs/frida_kahlo.jpg -content_image examples/inputs/golden_gate.jpg -image_size 50 -gpu -1
[libprotobuf WARNING google/protobuf/io/coded_stream.cc:537] Reading dangerously large protocol message.  If the message turns out to be larger than<PHONE_NUMBER> bytes, parsing will be halted for security reasons.  To increase the limit (or to disable these warnings), see CodedInputStream::SetTotalBytesLimit() in google/protobuf/io/coded_stream.h.
[libprotobuf WARNING google/protobuf/io/coded_stream.cc:78] The total number of bytes read was 574671192
Successfully loaded models/VGG_ILSVRC_19_layers.caffemodel
conv1_1: 64 3 3 3
conv1_2: 64 64 3 3
conv2_1: 128 64 3 3
conv2_2: 128 128 3 3
conv3_1: 256 128 3 3
conv3_2: 256 256 3 3
conv3_3: 256 256 3 3
conv3_4: 256 256 3 3
conv4_1: 512 256 3 3
conv4_2: 512 512 3 3
conv4_3: 512 512 3 3
conv4_4: 512 512 3 3
conv5_1: 512 512 3 3
conv5_2: 512 512 3 3
conv5_3: 512 512 3 3
conv5_4: 512 512 3 3
fc6: 1 1 25088 4096
fc7: 1 1 4096 4096
fc8: 1 1 4096 1000
Setting up style layer  	2	:	relu1_1
Setting up style layer  	7	:	relu2_1
Setting up style layer  	12	:	relu3_1
Setting up style layer  	21	:	relu4_1
Setting up content layer	23	:	relu4_2
Setting up style layer  	30	:	relu5_1
Running optimization with L-BFGS
Iteration 50 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
Iteration 100 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
Iteration 150 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
Iteration 200 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
Iteration 250 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
Iteration 300 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
Iteration 350 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
Iteration 400 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
Iteration 450 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
Iteration 500 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
Iteration 550 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
Iteration 600 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
Iteration 650 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
Iteration 700 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
Iteration 750 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
Iteration 800 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
Iteration 850 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
Iteration 900 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
Iteration 950 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
Iteration 1000 / 1000
  Content 1 loss: nan
  Style 1 loss: nan
  Style 2 loss: nan
  Style 3 loss: nan
  Style 4 loss: nan
  Style 5 loss: nan
  Total loss: nan
<optim.lbfgs> 	reached max number of iterations

Unfortunately the content weight and style weight depend on the image size; looks like using the default style and content weights with such a small image caused the optimization to diverge (as evidenced by nan everwhere in the losses). Try setting -print_iter 1 to make it print losses every iteration, and try tweaking the -style_weight and -content_weight; if you see nans in the first few iterations then kill it and try different style and content weights.

I think I also had cases on which a simple re-run of the same neural-style command didn't result in NaNs.

nans are basically like the ice-nine of neural nets: everything they touch turns to nan.  Once you get a nan anywhere in the system, it's over :-P

hi, I meet the same problem, do you fixed it?


⚠ Bofrost.lu has degraded performance

In 66c0d76, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2318 ms

Resolved: Bofrost.lu performance has improved in 41e50c2 after 32 minutes.


Add Batch Job Definition

Context
With the addition of the Batch compute environment was in https://github.com/AlexsLemonade/scpca-portal/issues/844, and the Batch job_queue in https://github.com/AlexsLemonade/scpca-portal/issues/900, we are now ready to develop the configuration for Batch job definitions.
A note about EBS integration: As of January, EBS does integrate with Fargate, and was integrated into Terraform in May. It can be configured on the aws_ecs_service resource via a volume_configuration argument, which enables a new volume to be configured when the task is executed, and is decomissioned at the time of task completion. The challenge is that while AWS Batch does use ECS under the hood to orchestrate the container lifecycle of its jobs, there is no interface (i.e. argument) in the aws_batch_job_definition resource which allows for this functionality.
As such, I've been EFS in here as a place holder until we decide the best way forward.
A note about volumes: the necessary size for a block_device in api.tf is 1.4 TB, which is described over there as 3.5x the biggest project's size.
See here for more details: https://github.com/AlexsLemonade/scpca-portal/blob/9278ebd23a87206d77bafe096c85cd65039f92d8/infrastructure/api.tf#L86
Solution or next step
batch/job_definition.tf
resource "aws_batch_job_definition" "generate_computed_files_job"
    name = "generate-computed-files-job-${var.user}-${var.stage}"
    type = "container"

    platform_capabilities = [
        "FARGATE",
    ]
    container_properties = jsonencode({
        command = ["sportal", "generate-computed-files", "--scpca-id", # "<scpca-id>"]
        image = "ccdl/scpca_portal_api"

        # t2.medium has 2 vcpus and 4.0 GB of RAM
        vcpus = 2
        memory = 4000
        volumes = [
            {
                host = "efs-volume"
                efsVolumeConfiguration = {
                    fileSystemId = # needs to be configured, will read something like aws_efs_file_system.name.id
                }
            }
        ]
        mountPoints = [
            {
                sourceVolume = "efs-volume"
                containerPath = # path inside container where EFS volume is mounted
            }
        ]
        executionRoleArn = aws_iam_role.aws_ecs_task_execution_role.arn
    })

Closed with #901


Remove system attribute from namespace yaml files in ManageIQ domain

We have removed the system attribute from the
miq_ae_namespace table. Remove references to system
from all the namespace yaml file in ManageIQ domain
Links

https://github.com/ManageIQ/manageiq/pull/10213

@gmcculloug @tinaafitz
Please review

@mkanoor Looks good.

Euwe Backport details:
$ git log
commit e33dfdeb694fc33442964aafa1514caa386d8755
Author: Greg McCullough<EMAIL_ADDRESS>Date:   Tue Sep 27 14:45:42 2016 -0400

    Merge pull request #11547 from mkanoor/managiq_sytem_cleanup

    Remove system attribute from namespace yaml files in ManageIQ domain
    (cherry picked from commit 93195345dc5daa419b591f937252abf9788a24a2)


Running Boogie as Long-Lived Process

Hi,
So, I'm trying to figure out if it's possible to do this, or if Boogie would need to be updated.  Specifically, I want:

To run Boogie as a separate (long-lived) process capable of answering multiple queries
To interact with the process by piping Boogie code into it, and reading error messages out of it.

The reason for this is simply to reduce the cost of running Boogie during e.g. an incremental editing session.  At the moment, I have to fire it up every time things change and that is proving expensive.  The command-line options (as I can see them) appear to imply this should be possible:
/proverShutdownLimit n
     Set the time, in seconds, between closing the stream to the prover and killing  the
     prover process.  n defaults to 0.

However, I can't figure out how to do it.  Thoughts appreciated.

Hi @DavePearce,
Running Boogie as a long-lived query-answering process is not supported. It goes through a single pipeline of parsing, resolution, type checking, VC generation, and theorem proving. From what I recently heard from a colleague, even incremental parsing of (potentially incomplete) programs alone is an open research problem.
What is the granularity of interaction you wish for? Something coarse-grained like adding/retracting complete procedures, or really fine-grained edits to the code?
If your main problem is the overhead of running Boogie over and over again, the solution for your use case could be Boogie's verification result caching. The feature is described in Fine-Grained Caching of Verification Results by @RustanLeino and @wuestholz. Dafny exploits this feature when run as verification server, e.g., in the Emacs mode for Dafny. I found this to significantly improve reactivity in interactive coding sessions.
The -proverShutdownLimit option does not do anything in the code and should be removed. Anyway, the description sounds more like what -timeLimit does; restricting the time given to the theorem prover.
Hope that helps.

Hey Bernhard,
Thanks for the reply!  Sorry for my slow response, but was actually on holiday the last few weeks.
So, to answer your questions.  What am I looking for?

If your main problem is the overhead of running Boogie over and over again,

Yes, this is definitely the main driving factor at this stage.  You mention Dafny and I am very interested in how Dafny makes it work.  Do you know much about that?   And, how would I go about enabling result caching?
One problem is that I am running on a Linux machine under mono, and each query fires up a new process, etc.  Just being able to have a process running and then send it queries (even if they are completely fresh queries) would help.

Dafny is an open-source project:  https://github.com/dafny-lang/dafny.  Dafny uses Boogie as a library.  Dafny uses the verification result caching feature in Boogie that you are interested in.  You could study the paper mentioned above by @bkragl and then look at how Dafny invokes Boogie.


Preventing JavaScript URLs upon parsing markdown links

I was looking at using this project for a server-side MarkDown parser, but someone I was working with tried out the demo and pointed out that this markdown:
[some text](javascript:alert("xss"))

renders as an actual JavaScript link. Does markdown-wasm offer more fine-grained control to prevent this specific URL type from being parsed?
If not, we can attempt to remove them manually (using JavaScript), but it would be great if it could directly be dealt with in the Wasm parser itself.

Looks like that's considered a valid link under the CommonMark spec, which md4c (and thus markdown-wasm) strictly comply to: https://spec.commonmark.org/0.29/#link-destination
I'd recommend just passing output through DOMPurify.

In my particular case, javascript: links do nothing on my site, since our server sends Content-Security-Policy HTTP headers that strictly deny inline JavaScript and CSS, so I didn't even need to modify the output.
But yes, it is a valid URL, that I have no problem with, but users might appreciate a flag that prevents emitting them.

Ah, this would definitely be a nice option to add, to allow this as an opt-in (disallowed by default.)
For now, a post-processing step on the generated HTML is a good idea if you allow untrusted users to enter & render HTML.

4b48783c3e209d900a459a2bd76aa314cb62f7e1 (released with v1.2.0 on NPM) includes a fix for this. Starting with this version, by default any href starting with "javascript:" is now stripped. "javascript:" URIs can be explicitly enabled by setting the option allowJSURIs.
Example input:
[XSS test](javAscRipt:alert("xss"))

Output:

XSS test


Adding CONTRIBUTE.md

Hi, I really want to contribute to this repo, but don't really know how to. Perhaps a CONTRIBUTE.md would help?
Thank you :)

Sure, we're looking for contributions - what do you have in mind? There's no real guidelines for now, you can clone the repo and make PRs.

Sure thing @drcmda


chore(release): v1.1.1 🎉

:robot: I have created a release beep boop
1.1.1 (2023-07-25)
Bug Fixes

practs: bump best practs for faster cicd (b9a26fe)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/mhetrics/app-usage-events-react/releases/tag/v1.1.1 :sunflower:


Logo

I think this package could be even better with a logo in the README 😁  what do you think?

Codecov Report
Patch coverage: 75.00% and project coverage change: +0.97 :tada:

Comparison is base (ddd8c9a) 81.96% compared to head (19d7414) 82.93%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #34      +/-   ##
==========================================
+ Coverage   81.96%   82.93%   +0.97%
==========================================
  Files          11       11
  Lines         560      545      -15
==========================================
- Hits          459      452       -7
+ Misses        101       93       -8

Impacted Files
Coverage Δ

src/DDM.jl
82.68% <ø> (ø)

src/DiffusionRace.jl
85.29% <ø> (ø)

src/LBA.jl
95.77% <ø> (+7.53%)
:arrow_up:

src/LCA.jl
78.33% <ø> (ø)

src/LogNormalRace.jl
93.54% <ø> (+5.66%)
:arrow_up:

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

src/Wald.jl
60.00% <ø> (ø)

src/wald_mixture.jl
88.00% <ø> (ø)

src/utilities.jl
55.88% <75.00%> (-1.70%)
:arrow_down:

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

NICE! This looks awesome. Only commnet have: I would have liked to see the purple more clearly, either by making the dot purple or swapping the blue and purple distributions.

How 'bout that:

@DominiqueMakowski, thank you! I think this is really cool.
By the way, I cannot tell. Did you use the official julia Theme colors: https://github.com/JuliaLang/julia-logo-graphics
The RGB definitions can be found in the link.

Do you think the logo would also fit in the documentation?
https://documenter.juliadocs.org/stable/man/guide/#Adding-a-logo-or-icon

all good on my side, I think I fixed the documentation problems. Feel free to edit & merge :)


Release v0.2.1

The git commit history on the master branch is just nasty, is there an easy way to fix this? @Jspsun


US795042: Updates for use in classification worker

Ticket: https://internal.almoctane.com/ui/entity-navigation?p=131002/6001&entityType=work_item&id=795042

A developer build has not yet been created for this branch. Click here to go ahead and create the build...

CI Build Link:
http://sou-jenkins2.swinfra.net:8080/job/CAFapi/job/CAFapi~opensuse-opensearch2-image~US795042~CI

Still got 3 CVE's to address in Netty and commons-compress.

Still got 3 CVE's to address in Netty and commons-compress.

Addressed in https://github.com/CAFapi/opensuse-opensearch2-image/pull/19/commits/018741efac2b61cd5e4f02d24a50271768723489


New maintainer for Snacks

@Angel-125 has updated Snacks for KSP 1.1.2 and is the new maintainer for it.

Looks good to me. Merging now.