help ?

can you explain to me where are the weights please ?  ,thanks
&& btw look at these numbers  :)

GTX850M

That's awesome! 5fps! :D
For a zip of the weight files used, you can check it out on ModelDepot, clicking on "Download Model" https://modeldepot.io/mikeshi/tiny-yolo-in-javascript
The location reference in this package is here in the source code. You can check out the repo hosting the weight files here (it's a bit of a mess, the config file is model2.json)
Ideally I can get it hosted somewhere better than GH but we don't have the data transfer capacity at ModelDepot to justify hosting the live model version there for now (hopefully in the future we can though!).
The weights were created through pulling the weights from the original darknet website, passing it through YAD2k (see this issue for loading tiny yolo weights), and then using tfjs's keras -> tfjs convertor to create the ultimate tf.js model weights.


Incorrect formatting of text from traceback

Applies To

[X] Notebooks (.ipynb files)
[ ] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
When a python cell throws an exception, links to source files are inserted but this messes up text that should not be linked, for example
AttributeError: 'Australia/Sydney' object has no attribute 'key'
is displayed as

And the output is converted to
AttributeError: 'Australia[/Sydney](https://file+.vscode-resource.vscode-cdn.net/Sydney)' object has no attribute 'key'

To reproduce
rasie AttributeError("'Australia/Sydney' object has no attribute 'key'")

This seem to occur when any text contains a forward slash. It makes it harder to share error messages with other library developers etc. since you cannot copy/paste the raw text of the traceback.
VS Code Version
Version: 1.78.2 Commit: b3e4e68a0bc097f0ae7907b217c1119af9e03435 Date: 2023-05-10T14:47:05.613Z Electron: 22.5.2 Chromium: 108.0.5359.215 Node.js: 16.17.1 V8: <IP_ADDRESS>-electron.0 OS: Linux x64 5.15.0-71-generic Sandboxed: No
Jupyter Extension Version
v2023.4.1011241018
Jupyter logs
No response
Coding Language and Runtime Version
Pytohn 3.10
Language Extension Version (if applicable)
No response
Anaconda Version (if applicable)
No response
Running Jupyter locally or remotely?
Local

this should be fixed in the latest version


documentation regarding training_loss() is confusing

I am using training_loss() inside tape.gradient but I am confused that whether we need to use closure inside tape.gradient or not.

Without knowing more about what you mean by "weird behaviour" it's pretty much impossible to help you. We've added the issue templates for a reason - please provide a minimal example that reproduces your issues.
Note that the SVGP model behaves slightly differently from the others - it doesn't store data inside, so its training_loss does take data. (This is documented in the training_loss docstring.)

Hi, Thank you for your reply. I solved my problem but I should mention one thing that in the beginning of the documentation closure() is used with training_loss() function . Then in tape.gradient section it should be clearly mention that here we use this function without closure. As I was thinking how to use closure () inside  tape.gradient.

It'd be great if you would contribute your suggested improvements directly as a pull request :) it's only people new to the code base who can point out where the documentation and tutorials are lacking clarity or depth!


Esper stats on global version

I just rebuilt the Esper Simulator on ffbedb using your data but I'm not sure if I got the right esper stats. One of my users just notified me that Siren's 3star max HP and Ifrit's 3star max HP don't match with his summon in the game nor the data on the english wiki. Are the base stats in the Entries array in Summons.json the proper stats?

Hey, the "problem" is that, technically, the 3* espers go up to max level 100, but in-game you can only level them to 60.
Apparently, they said there might be other chances to upgrade them further with events or something.

Alright, thanks for the explanation. I'll manually set the stats.


Adding Betterscan.io as a Scan Type

Description
Adding Betterscan.io as a new "Scan Type"
https://github.com/DefectDojo/django-DefectDojo/issues/7354
Used current SARIF implementation in new parser and also new file, since in the future there could be Betterscan changes
Test results
Use SARIF parsing, so I assume no need for test.
Documentation
Not needed

Hi @marcinguy please see the how to write a parse guide for other requirements we have for new parsers

Hi @marcinguy please see the how to write a parse guide for other requirements we have for new parsers

@Maffooch Thank you. Will check it.

@marcinguy we already have a SARIF parser. what is the value of this new SARIF parser?

@damiencarol Hi, thank you for your message.
I was asked for support for DefectDojo.
I can make a new format for Betterscan and it can be added here, but I also support SARIF.
The only value, I think, will be that it will be seen in DefectDojo and save me explaining that SARIF can be used for DefectDojo. I guess that was not clear to everybody.
Do you have other ideas how to add Scan Type of ”Betterscan" to DefectDojo"?
Original issue: https://github.com/DefectDojo/django-DefectDojo/issues/7354
Thanks,

@marcinguy why not simply add it to the Label of the SARIF importer, s.t. it will be shown in the UI? https://github.com/DefectDojo/django-DefectDojo/blob/master/dojo/tools/sarif/parser.py#L24. Furthermore you could also adjust the Documentation https://github.com/DefectDojo/django-DefectDojo/blob/master/docs/content/en/integrations/parsers.md#sarif

@italvi Great idea. So I will modify https://github.com/DefectDojo/django-DefectDojo/blob/master/dojo/tools/sarif/parser.py#L24 to return return ["Betterscan.io"]
Thanks,

@marcinguy I just tried it myself: You have to adjust the scan_label at https://github.com/DefectDojo/django-DefectDojo/blob/master/dojo/tools/sarif/parser.py#L21, s.t. return ["SARIF", "Betterscan.io"]. Then you can choose "Betterscan.io" in the import field:

Furthermore, if you scroll down in the import scan results site you will see:

You could also add something like:
def get_description_for_scan_types(self, scan_type):
        if scan_type == "Betterscan.io":
            return "Import Betterscan.io report. Currently only SARIF Format is supported."
        return "SARIF report file can be imported in SARIF format."

S.t. the text in the UI will not be "Betterscan.io SARIF report file can be imported in SARIF format."

@italvi Awesome. Thank you very much.
Create new PR for this change here, thought it would be the easiest way:
https://github.com/DefectDojo/django-DefectDojo/pull/7452

succeeded by #7452


copysign function inside @avx loop

Hi,
Fantastic package!
When I recently tried to use some sign functions inside the loop, for example
using LoopVectorization, BenchmarkTools

function mydotavx(a, b)
    s = 0.0
    @avx for i ∈ eachindex(a,b)
        s += a[i]*b[i]
        s += minmod(a[i],b[i])
    end
    s
end

minmod(a, b) = (sign(a) + sign(b)) / 2.0 * min(abs(a), abs(b))

a = rand(256); b = rand(256);

@btime mydotavx($a, $b)

there comes the error message:
ERROR: LoadError: MethodError: no method matching copysign(::NTuple{4,VecElement{Float64}}, ::NTuple{4,VecElement{Float64}})
Stacktrace:
 [1] macro expansion at /Users/hyzhou/.julia/packages/SIMDPirates/Mppjk/src/floating_point_arithmetic.jl:27 [inlined]
 [2] sign at /Users/hyzhou/.julia/packages/SIMDPirates/Mppjk/src/SIMDPirates.jl:80 [inlined]
 [3] minmod(::VectorizationBase.SVec{4,Float64}, ::VectorizationBase.SVec{4,Float64}) at /Users/hyzhou/Documents/Computer/Julia/Dante/test_loop.jl:12
 [4] macro expansion at /Users/hyzhou/.julia/packages/LoopVectorization/bI2Fm/src/reconstruct_loopset.jl:444 [inlined]
 [5] _avx_! at /Users/hyzhou/.julia/packages/LoopVectorization/bI2Fm/src/reconstruct_loopset.jl:444 [inlined]
 [6] mydotavx(::Array{Float64,1}, ::Array{Float64,1}) at /Users/hyzhou/Documents/Computer/Julia/Dante/test_loop.jl:5
...

I guess what is missing here is that we need to implement a new copysign function?
I am also wondering what we should do for more functions inside the loop like this.
Thanks!

Hi, thanks for the bug report.
Turns out my old sign function was broken.
Early on in SIMDPirate's history, shortly after it was forked from SIMD, it tried to do everything with NTuple{W,Core.VecElement{T}}s instead of wrapping them in structs. The primary motivation was that on some platforms, wrapping them in structs would often cause crashes when they crossed function boundaries. That bug has since been fixed, so now there's little reason not to wrap them in structs. @benchmark/@btime on anything returning a struct-wrapped NTuple{W,Core.VecElement{T}} was very likely to crash Julia (on those platforms).
But because ``NTuple{W,Core.VecElement{T}}is a type defined inBaseJulia, I can't overload methods defined in base Julia without commiting type piracy, so every function was given a name likevcopysign. But that meant I couldn't overload any base methods, like copysign, as that'd be type piracy, so vcopysignis defined onNTuple{W,Core.VecElement{T}}instead, whilecopysignis defined onSVec`s (which is a struct wrapping such tuples).
tldr: copysign was already defined, but there was a bug preventing it from being called.
I'll close this issue when I make it depend on the new SIMDPirates version with the fix.
julia> using LoopVectorization, BenchmarkTools
[ Info: Precompiling LoopVectorization [bdcacae8-1622-11e9-2a5c-532679323890]

julia> @inline minmod(a, b) = (sign(a) + sign(b)) / 2.0 * min(abs(a), abs(b)) # @inline seems necessary
minmod (generic function with 1 method)

julia> function mydot(a, b)
           s = 0.0
           @inbounds @simd for i ∈ eachindex(a,b)
               s += a[i]*b[i]
               s += minmod(a[i],b[i])
           end
           s
       end
mydot (generic function with 1 method)

julia> function mydotavx(a, b)
           s = 0.0
           @avx for i ∈ eachindex(a,b)
               s += a[i]*b[i]
               s += minmod(a[i],b[i])
           end
           s
       end
mydotavx (generic function with 1 method)

julia> a = rand(256); b = rand(256);

julia> @btime mydotavx($a, $b)
  54.581 ns (0 allocations: 0 bytes)
151.08749163626982

julia> @btime mydot($a, $b)
  92.835 ns (0 allocations: 0 bytes)
151.08749163626982

julia> a = rand(255); b = rand(255);

julia> @btime mydotavx($a, $b)
  54.419 ns (0 allocations: 0 bytes)
139.69162315609353

julia> @btime mydot($a, $b)
  159.942 ns (0 allocations: 0 bytes)
139.6916231560935

Bumped SIMDPirates version on master, and registering.


Ordering notifications

Hey,
Is there any way to order the notifcations by date? All get functions returns oldest first, and while the read function is order-able, it doesn't return the read notifications.

By default notifynder is ordering the notifications by date in a DESC order so newest first.
https://github.com/fenos/Notifynder/blob/master/src/Notifynder/Notifynder.php#L175
Which method are trying to use? Can you make an example please i'll glad to help you

Is it? Forgot to mention, I'm on laravel4, using version 2.5 of Notifynder
I'm using $notifications = Notifynder::getNotRead($currentUserId);

@Aryess Notifynder 2.0 (L4 branch) doesn't include ordering on getNotRead()
However, feel free to modify the following files from the following files, adding the order parameter like how it has been done on Notifynder 3, I'll maybe do it if I got time or @fenos will do it, but maintaining two different versions on two different Laravel version is a hard thing to do..
Notifynder Laravel 4, files to edit :
https://github.com/fenos/Notifynder/blob/L4/src/Fenos/Notifynder/Notifynder.php#L296
https://github.com/fenos/Notifynder/blob/L4/src/Fenos/Notifynder/NotifynderInterface.php#L160
https://github.com/fenos/Notifynder/blob/L4/src/Fenos/Notifynder/Notifications/Repositories/NotificationRepository.php#L185
https://github.com/fenos/Notifynder/blob/L4/src/Fenos/Notifynder/Notifications/NotifynderNotification.php#L160
*Notifynder Laravel 5 as reference:
https://github.com/fenos/Notifynder/blob/master/src/Notifynder/Notifynder.php#L175
https://github.com/fenos/Notifynder/blob/master/src/Notifynder/NotifynderManager.php#L351
https://github.com/fenos/Notifynder/blob/master/src/Notifynder/Notifications/NotificationRepository.php#L221
https://github.com/fenos/Notifynder/blob/master/src/Notifynder/Notifications/NotificationManager.php#L172
https://github.com/fenos/Notifynder/blob/master/src/Notifynder/Contracts/NotifynderNotification.php#L109
https://github.com/fenos/Notifynder/blob/master/src/Notifynder/Contracts/NotificationDB.php#L122

maintaining two different versions on two different Laravel version is a hard thing to do
No worries.
Thanks for the answer, I'll see what I can do.


NCS-689 Amend trading status model

NCS-689
Change trading status model to use boolean field instead of section status.

looks like there's a test error - https://ci.platform.aws.chdev.org/teams/common/pipelines/api-sdk-node/jobs/pull-request-analysis/builds/235#L610ebd94:37

looks like there's a test error - https://ci.platform.aws.chdev.org/teams/common/pipelines/api-sdk-node/jobs/pull-request-analysis/builds/235#L610ebd94:37

Sorted now


Azure VPN gateway| On a single tunnel a maximum of 1 Gbps throughput can be achieved ?

[Enter feedback here]
It is mentioned that "On a single tunnel a maximum of 1 Gbps throughput can be achieved."
Does it mean the maximum throughput of single tunnel is 1Gbps?  It cannot exceed?
We have noticed it could reach to 1.25 Gbps in practice.
https://github.com/jocortems/azurehybridnetworking/tree/main/Multiple-VPN-Tunnels-From-CiscoCSR-to-AzureVNG
Please help to confirm it and update this description accordingly.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 45d7ab31-fdf3-9b0f-1439-6a01fb257aab
Version Independent ID: af13a45f-3061-2f0e-7452-b52d700b4805
Content: About Azure VPN Gateway
Content Source: articles/vpn-gateway/vpn-gateway-about-vpngateways.md
Service: vpn-gateway
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@yinoa Thanks for the feedback! We are taking a look into this and will get back to you soon.

@yinoa Thanks for the feedback! We are taking a look into this and will get back to you soon.

@yinoa , I believe you are referring to Virtual WAN VPN gateway as described in below article:
https://docs.microsoft.com/en-gb/azure/virtual-wan/virtual-wan-about#what-is-the-total-vpn-throughput-of-a-vpn-tunnel-and-a-connection
And not normal Azure VPN gateway since normal Azure VPN gateway has different throughput based on different SKUs. However Virtual WAN VPN gateway operates on scale units multiplied by 2 instances of highly available VPN gateways, where each tunnel in a connection can support up to 1 Gbps.
Is that correct?
The doc in which you have opened this feedback is related to normal VPN gateway, hence wanted your confirmation on same.

@yinoa , I believe you are referring to Virtual WAN VPN gateway as described in below article:
https://docs.microsoft.com/en-gb/azure/virtual-wan/virtual-wan-about#what-is-the-total-vpn-throughput-of-a-vpn-tunnel-and-a-connection
And not normal Azure VPN gateway since normal Azure VPN gateway has different throughput based on different SKUs. However Virtual WAN VPN gateway operates on scale units multiplied by 2 instances of highly available VPN gateways, where each tunnel in a connection can support up to 1 Gbps.
Is that correct?
The doc in which you have opened this feedback is related to normal VPN gateway, hence wanted your confirmation on same.

@yinoa , I checked with the Virtual WAN Product group team and they mentioned - "while it can go to 1.25 Gbps, we document 1 Gbps as a conservative number for a single tunnel. There is no explicit throttle we put to cap it."
Hence, we are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.

@yinoa , I checked with the Virtual WAN Product group team and they mentioned - "while it can go to 1.25 Gbps, we document 1 Gbps as a conservative number for a single tunnel. There is no explicit throttle we put to cap it."
Hence, we are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Update Color.cs

Added typecast in "public static Color FromARGB(int argb)" to prevent an infinite loop.

Thanks @r-j-s, this is a great fix! 🚀


Content pages should use shouldComponentUpdate

They do now. Not sure why this hasn't closed by itself.


Wordcount - daily graph

Posting as a WIP because, though it works, I'm a little unsure if this fits in with the project style. Still WIP, can be cleaned up References (#21 pt. 3). Next steps would be including wordcount graph for, eg, last 14 days, which is perfectly doable in SVG, this branch or otherwise.

Very cool! I am totally slammed at work right now, but am excited to go through this next week.
Love the idea!

Ditto; will add as I can!

Thanks for taking a look at this when you have some time!  I'm using this project to teach myself Elm, and really appreciate it being open source.  This appears to work as an initial implementation.  In some instance, incomplete docs wind up in Elm-land (missing dailyWords, for instance).  This gracefully degrades by simply not showing a graph if not available, but this will be worth a follow-up.
QA

Add some words to the doc and refresh.  There should be a single bar.
Set a breakpoint at DreamSync.coffee line 236
Start typing; the breakpoint should be hit.
Add some additional days worth of words, including negative wordcounts.[0]
Remove the breakpoint and resume.
Refresh to see those days in there.

[0]
doc.dailyWords.push({words: 20, day: "2014-08-09"});
doc.dailyWords.push({words: -5, day: "2014-08-09"});

This totally rules. Thank you so much for the contribution!


Faceted browse against a JSON list of tags

If a table has a ["foo", "bar", "baz"] JSON column allow that to be faceted against.

[x] Support ?column__arraycontains=x filter queries
[ ] Support ?_facet_array=column faceting

Example table: https://latest-code.datasette.io/code/definitions
Here's a query that does facet counting against that column:
https://latest-code.datasette.io/code-a26fa3c?sql=select+count(*)+as+n%2C+j.value+from+definitions+join+json_each(params)+j+group+by+j.value+order+by+n+desc%3B

And here's the query for pulling back every record tagged with a specific tag:
https://latest-code.datasette.io/code-a26fa3c?sql=select+*+from+definitions+where+rowid+in+(
++select+definitions.rowid
++from+definitions+join+json_each(params)+j
++where+j.value+%3D+%3Atag
)&tag=filename
select * from definitions where rowid in (
  select definitions.rowid
  from definitions join json_each(params) j
  where j.value = :tag
)

Note that there doesn't seem to be a way to use indexes (even indexes on expressions) to speed these up, so this will only ever be effective on smaller data sets, probably in the 10,000-100,000 range. Datasette is often used with smaller data sets so this is still worth pursuing.

I'm going to go with ?_facet_array=definitions as the querystring argument for this.
And ?definitions_arraycontains=foo as the filter argument.

Challenge: facets can also be defined in metadata.json like this:
{
  "databases": {
    "sf-trees": {
      "tables": {
        "Street_Tree_List": {
          "facets": ["qLegalStatus"]
        }
      }
    }
  }
}

But... ?_facet_array=definitions doesn't fit in that data structure. Need to have an alternative mechanism for defining this kind of facet.

Demo: https://latest.datasette.io/fixtures/facetable?_facet_array=tags#facet-tags


Link from the overview page to the authentication page is broken

On the overview page link to the authentication page is broken. It links to https://docs.botframework.com/en-us/core-concepts/authentication.md, instead of https://docs.botframework.com/en-us/core-concepts/authentication

Thanks - we'll get this fixed.


Impossible amount of posts scanned

What problem has occurred? What issues has it caused?

As we can see on the picture, there is a spike in the posts scanned on May 27, 15:00, and one of the 10 minute scan report indicates that SD managed to scan 32896 posts in some 10-minute period. However, since the post scan rate indicated is 1.095 posts per second, we can calculate that the maximum posts that can be scanned in the 10-minute period is 1.095 posts/s * 10 min * 60 s/min = 657 posts.
There are problems with other reports too. For example, in the following report

We can see that 1002 posts were scanned. However, 1.04 posts/s * 10 min * 60 s/min = 624 posts < 1002 posts.
What would you like to happen/not happen?
Not to let 1002 be less than 624.

Okay I somehow managed to figure out the second is due to multithreading. However, the first still remains impossible, as 32896 / 657 = 50.


update express and related deps

This PR updates the express and related dependencies to the newest versions.

Coverage remained the same at 90.991% when pulling 9fd5521f07fc13ca6d95241c92055152864a6766 on Shegox:express-deps into 54131277b09b146514d095bb1041a13c908de8b5 on cla-assistant:master.


Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Branch name
ERROR: Dealer._add_filters() missing 1 required positional argument: 'req'
Commit ID
ERROR: Dealer._add_filters() missing 1 required positional argument: 'req'
Other environment information
No response
Actual behavior
ERROR: Dealer._add_filters() missing 1 required positional argument: 'req'
Expected behavior
No response
Steps to reproduce
ERROR: Dealer._add_filters() missing 1 required positional argument: 'req'

Additional information
No response

why close, same error

same error


Scroll to section on page load

What is the best way to trigger a scroll event after the page is loaded? I've tried adding a scrollTo method to componentDidMount but that doesn't seem to work. Thanks for all your great work!

If you are using react-router, you could use the browserHistory.listen callback to add a call to scrollTo when the location changes.
For example like this in your main app component:
let { animateScroll } = require('react-scroll');

browserHistory.listen(function (location) {
  if (location.hash) { // scroll to anchor
    const offset = document.getElementById(location.hash.slice(1)).getBoundingClientRect().top;
    animateScroll.scrollMore(offset);
  } else {
    animateScroll.scrollToTop();
  }
});

ReactDOM.render(
  <Router routes={routes} history={browserHistory} />,
  document.getElementById('react-root')
);

code
import { animateScroll } from 'react-scroll';
....
`componentWillMount() {
 browserHistory.listen(function (location) {
  if (location.hash) { // scroll to anchor
    const offset = document.getElementById(location.hash.slice(1)).getBoundingClientRect().top;
    animateScroll.scrollMore(offset);
  } else {
    animateScroll.scrollToTop();
  }
});`

error
@fargerio
"browserHistory is not defined"
stack:
"ReferenceError: browserHistory is not defined↵    at eval (eval at componentWillMount (webpack-internal:///534), :1:1)↵    at Root.componentWillMount (webpack-internal:///534:97:7)↵

Sorry forgot to mention, that you also need to import browserHistory from react-router:
let { Router, browserHistory } = require('react-router');


Add documentation to delete stale backend images/omap

Add documentation to delete the backend omap metadata and backend cephfs and rbd images.
if the PVC is created with storage class which is having the reclaimPolicy as Retain will not delete the PV object and backend omap metadata and backend image. Manual deletion of PV will result in stale omap keys, values and also stale cephfs and rbd images.
we need to document the steps to clean up the image metadata and backend images.

Should the metadata and the image be deleleted separately ？

Yes you need to delete metadata from rados seperately

ok I will take the job .

Awesome, @chenxu1990 assigning this to you


Day based bars is not supported yet

Lots of things get a bit weird if you try preload day based bars:
e.g.
    strategy = DayStrategy(
        instruments = [
#         ("ES", "FUT", "GLOBEX", "USD", ACTIVE_MONTH, 0.0, ""),
#          ("SPX", "IND", "CBOE", "USD", "", 0.0, ""),
          ("AAPL", "STK", "SMART", "USD", "", 0.0, "")
         ],
        resolution  = "1D",
        bar_window  = 100,
        preload = "4D"
    )

ES hangs with no history error message on Gateway.
INDes seem to just hang and not return anything.
For stock I got things going with two changes to the blotter.py
     # expand the resolutions
       if resolution[-1] in ("K", "V", "S"):
            self.backfill_resolution = "1 sec"
       elif resolution[-1]== "M":
            self.backfill_resolution = "1 min"
       elif resolution[-1]== "H":
            self.backfill_resolution = "1 hour"
       elif resolution[-1]== "D":
            self.backfill_resolution = "1 day"

```and

  # round to days if asking for days
    if resolution[-1]== "D":
        start_date = datetime(start_date.year, start_date.month, start_date.day)
        end_date   = datetime(end_date.year, end_date.month, end_date.day)
        first_date = datetime(first_date.year, first_date.month, first_date.day)
        last_date  = datetime(last_date.year, last_date.month, last_date.day)

I got the query for the stock back but....something is not parsing the dates well... it works for minute data but not for the day based bars...  (no idea where to debug from here)

1	1970-08-22 10:52:01	1	141.59	142.8	139.35	139.35	352675
2	1970-08-22 10:52:02	1	139.25	141.6	138.67	141.5	238385
3	1970-08-22 10:52:03	1	141.28	141.59	140.61	141	186233
4	1970-08-22 10:52:04	1	141.07	141.74	140.05	140.43	203203
5	2017-03-24 08:36:00	1	141.07	141.14	141.07	141.14	4
6	2017-03-24 08:37:00	1	141.14	141.14	141.14	141.14	0

I plan on adding a daily bar db real soon. In the meantime you should resample your daily bars to minute bars before preloading
min_df = daily_df.resample("1T").last().ffill()

I think I am going to try get a real data provider.  Using IB like for data seems flaky.
Thanks for the note.  I would love to chat sometime on your experience if you.
-Ric Pruss
On 18 April 2017 at 8:39:55 pm, Ran Aroussi<EMAIL_ADDRESS>wrote:
I plan on adding a daily bar db real soon. In the meantime you should resample your daily bars to minute bars before preloading
min_df = daily_df.resample("1T").last().ffill()
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I had the same hanging issue with INDes and  ES. Is this an issue with ezibpy.requestHistoricalData?


Automated cherry pick of #9646

Cherry pick of #9646 on release-6.3.
/cc  @koox00
NONE

Trying to auto merge this PR.


Propagate handlers in CallSettings extension methods

I've validated manually that these are the last calls which misuse
the shorter constructor. Thoughts about how to avoid similar
problems in the future would be welcome.

Odd. The test is passing locally - I've restarted the Travis build, and we'll see what happens. The fake scheduler is complex so maybe there was some oddity there...

And it passed... merging this PR as unrelated, but let's keep an eye to see if it happens again.


[MPoW] PubKey should be unique

After recover mode is enable in TEE side, if different accountId can bond same pubkey, it will steal the recovery_file, then different accounts can use the same plot size, it will destroy crust network.

Actually, solution can be:

adding accountId into WorkReport(different accountId cannot un-sig same pub-key work report);
checking pub key be unique(different accountId should have totally different tee identities);


No documentation for "error.unauthorized"

I'm working with a customer who is getting the error code "error.unauthorized" with the message "failed to authenticate provided token".  I don't see this error on the error documentation page.

This error is returned prior to UMAPI being invoked and is due to the authentication token being invalid: https://adobe-apiplatform.github.io/umapi-documentation/en/UM_Authentication.html


Feat/categories bulk actions

Details
This PR adds the ability to perform bulk actions on categories, such as disable, enable, or remove multiple categories at once.

Fixed Issues

$ https://github.com/Expensify/App/issues/35709
PROPOSAL: https://docs.google.com/document/d/1gk3xqOs7epMbUrSSiX8K7YcqfPLVgqEos0sf-D-GMDA/edit#bookmark=id.onqkmvshcfhp
Tests

[ ] Verify that no errors appear in the JS console

Log in as an admin of a paid policy group and navigate to the workspace categories settings.
Choose some categories that are currently disabled.
Click the options menu in the top right corner and select "Enable Categories".
Confirm that the selected categories are now enabled.
Choose some categories that are currently enabled.
Click the options menu in the top right corner again and select "Disable Categories".
Check that these categories are now disabled.
Choose any categories you wish to delete.
In the top right corner, click the options menu and choose "Delete Categories".
Verify that the selected categories have been deleted.

Network Failures Test

Log in as the admin of a paid policy group.
Go to profile settings, then navigate to "About" -> "Troubleshooting" and enable "Simulate Network Failures".
Go to the workspace categories settings.
Select some categories, whether they are currently disabled or enabled.
Use the options menu in the top right corner to either enable or disable the selected categories.
Wait for a moment, then look for an error message to appear under each category you tried to update, indicating a request failure.
Repeat steps 3 to 6 for the action of deleting categories.

Offline tests
Here are the simplified and clarified test steps:

Log in as an admin of a paid policy group and open the workspace categories settings.
Disable your internet connection.
Choose some categories that are currently disabled.
Click the options menu in the top right corner and select "Enable Categories".
Check that the selected categories now appear as pending (shown in gray).
Choose some categories that are currently enabled.
Click the options menu in the top right corner again and select "Disable Categories".
Ensure that these categories now show as pending (in gray).
Select any categories you want to delete.
In the top right corner, click the options menu and choose "Delete Categories".
Confirm that the chosen categories are marked as pending (grayed out) and have a strikethrough.
Reconnect to the internet and verify that:

Categories that were pending disable are now enabled.
Categories that were pending enable are now disabled.
Categories marked for deletion are removed.

QA Steps
The same steps as above.
PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message: https://expensify.slack.com/archives/C01GTK53T8Q/p1709744184746739

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/24796318/4c30060a-c81d-49f3-8db3-7137d5384d72

Android: mWeb Chrome
https://github.com/Expensify/App/assets/24796318/003b6364-6823-449a-b73e-a1fe00255c68

iOS: Native
https://github.com/Expensify/App/assets/24796318/1e898511-c091-4186-bf49-1dfe12b7a68a

iOS: mWeb Safari
https://github.com/Expensify/App/assets/24796318/bdc978e4-a757-4668-bb50-7e68ccd792d8

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/24796318/67bd259d-faae-48f5-a405-2cf8fb9e647a

MacOS: Desktop
https://github.com/Expensify/App/assets/24796318/ac32c9ae-3f38-476c-97ab-8cdffbafb865

@luacmartins There is an issue with the openWorkspace API call that seems to override pending actions for categories. Specifically, when a category is removed while offline, and then the page is reloaded or, on mobile, when navigating back and reopening the categories list, the action no longer appears as pending.

@ArekChr we have conflicts now

Conflicts resolved

@ArekChr new conflicts

@akinwale let's continue the review, that way we can address all issues once @ArekChr starts his day tomorrow

@ArekChr I am not getting any errors when I enable the Simulate Failing Network Requests option and try to enable / disable categories.
https://github.com/Expensify/App/assets/4160319/34328291-94f1-4760-a1e5-21cdaa202a99

@ArekChr I am not getting any errors when I enable the Simulate Failing Network Requests option and try to enable / disable categories.

Could you please wait a bit longer? It attempts several times before indicating a failure.

I think in another PR somewhere we updated the enabled/disabled text to use a smaller font size and a slightly different color. I'm not sure if that PR was merged yet, but maybe a quick pull of main couldn't hurt?

@luacmartins It appears that the collection policy free trial period has expired. Looks like I found bug, should we fix this by hiding the category menu item? What is the expected behavior here?

I think in another PR somewhere we updated the enabled/disabled text to use a smaller font size and a slightly different color. I'm not sure if that PR was merged yet, but maybe a quick pull of main couldn't hurt?

Pulled main

@luacmartins It appears that the collect policy free trial period has expired. Looks like I found bug, should we fix this by hiding the category menu item? What is the expected behavior here?

I started seeing that in dev too, maybe there's a regression somewhere. Gonna investigate.

@ArekChr we have conflicts again

@ArekChr A couple things for dealing with network request failures.

The enabled / disabled state does not revert back to its previous value. Is this expected behaviour?
If a category or multiple categories are deleted, and the request fails, there is nothing to indicate that the operation failed. I assume this isn't as critical, but just want to confirm that this is also expected.

https://github.com/Expensify/App/assets/4160319/6189698a-f8de-4875-9ed7-0641593af217

@ArekChr A couple things for dealing with network request failures.

The enabled / disabled state does not revert back to its previous value. Is this expected behaviour?
If a category or multiple categories are deleted, and the request fails, there is nothing to indicate that the operation failed. I assume this isn't as critical, but just want to confirm that this is also expected.

37734-web-network-failure.mp4

@akinwale I encountered the same issue again and after re-logging in, the updated state is still visible despite receiving an error message. It seems the state updates are going through even when we encounter simulation failure requests. Should we consider filing a bug report on this matter? @lucamartins, have you dealt with something similar in your experience?

@lucamartins, have you dealt with something similar in your experience?

TBH I've never gotten the Simulate failing network requests feature to work reliably. I always throw an error on our dev API to test these. I think we can move forward without blocking on that.

@ArekChr more conflicts and TS is failing again

🚀 Deployed to staging by https://github.com/luacmartins in version: 1.4.52-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

Tests passed when I merged it

Coming from https://github.com/Expensify/App/issues/38262
When a category is disabled while offline, the expected behavior was to make the entire category row grayed out. Only the status column was grayed out.

Coming from https://github.com/Expensify/App/issues/38262
When a category is disabled while offline, the expected behavior was to make the entire category row grayed out. Only the status column was grayed out.


'Error' texture on focused MenuButton

If I add a MenuButton to a scene and run it, whenever it has keyboard focus a texture with the word 'Error' is overlayed on the button. This occurs this both the default theme and custom themes.

Currently experiencing this, too, on v1.0 stable. It appears when you click on a MenuButton object and click somewhere else to dismiss the menu, the text "ERROR" appears on top of the object.

Is this issue still valid in the current master branch?

I'm not able to reproduce this in current master (bd736e5). I'll close it and if someone experiences the issue it can be reopened.

This can be reproduced with the "Tracks" button in the animation edtior:

For this demonstration, I set an exaggerated minimum size.
This is in current master (281d6fac1edd4a5b06692b850c00db6a5a9bbb46).


No UI release build Swift

When I debug on my iPhone the UI is there, all the images and colors. However when I get a release build and send it to my testers via Ad-hoc the UI is missing, the controls still work however the assets are missing. Any ideas?

Hi @jackintosh7! Are all the assets missing or just a few? This could be related to #64.

I pushed version 2.2.3 now. Could you try updating the pod and see if it fixed that issue?

I'm closing this due to inactivity and lack of more info. If you still have that issue please don't hesitate to open it again :)


xiaomi_gateway3.send_command on version 4.x

After upgrade to version 4.0.3, got some errors in home-assistant.log:
... Error executing script. Service not found for call_service at pos 1: Service xiaomi_gateway3.send_command not found

I use this automation to run some command when homeassistant or the gateway startup:
- trigger:
    - platform: homeassistant
      event: start
    - platform: state
      entity_id: binary_sensor.xxxxxxxxxxxx_gateway
      to: "on"
  action:
    service: xiaomi_gateway3.send_command
    data:
      command: miio
      host: <ip>
      data: >-
        {"method":"set_ip_info","params":{"ssid":"\"\"","pswd":"1; <command>"}}

Is there any equivalent service I can use on version 4.x?

Is there any equivalent service I can use on version 4.x?

No


⛰[EPIC]: CorrDiff integration

Placeholder for tracking the issues related to integrating the ResDiff model
### Integration Tasks
- [x] Use Modulus Docker image for ResDiff (https://github.com/NVIDIA/modulus/pull/266)
- [x] Compare the results generated with stochastic and deterministic samplers (https://github.com/NVIDIA/modulus/pull/260)
- [x] Use Modulus logging utilities (https://github.com/NVIDIA/modulus/pull/251)
- [x] Use Modulus distributed utilities (https://github.com/NVIDIA/modulus/pull/247)
- [x] Use Modulus checkpointing (https://github.com/NVIDIA/modulus/pull/282)
- [x] Use hydra configs (https://github.com/NVIDIA/modulus/pull/268)
- [x] Package and import the stochastic sampler (https://github.com/NVIDIA/modulus/pull/250)
- [x] Add profiling markers (https://github.com/NVIDIA/modulus/pull/306)
- [x] Black formatting (https://github.com/NVIDIA/modulus/pull/257)
- [x] Linting (https://github.com/NVIDIA/modulus/pull/262)
- [x] Code cleanup (https://github.com/NVIDIA/modulus/pull/262, https://github.com/NVIDIA/modulus/pull/274)
- [x] Remove dependency to `pickle` due to security issues
- [x] Remove the `persistance` decorator (https://github.com/NVIDIA/modulus/pull/274)
- [x] Update README (https://github.com/NVIDIA/modulus/pull/288)
- [x] Unit tests (https://github.com/NVIDIA/modulus/pull/281)
- [x] Docstrings (https://github.com/NVIDIA/modulus/pull/274)
- [x] Move files out of `experimental` (https://github.com/NVIDIA/modulus/pull/274)

### Issues
- [x] 'distributed.py' has to be removed but that breaks loading the existing checkpoints (https://github.com/NVIDIA/modulus/pull/274)

### Performance Optimization
- [x] Add support for CUDA Graphs (will not do)
- [x] Support for fp16 in inference (https://github.com/NVIDIA/modulus/pull/288)
- [x] Enable multi-GPU inference (https://github.com/NVIDIA/modulus/pull/281)
- [x] Enable batch_size > 1 for inference (https://github.com/NVIDIA/modulus/pull/281)

### Logging enhancement
- [x] Add W&B logging (https://github.com/NVIDIA/modulus/pull/316)

### Dataset
- [x] Get permission to publish the CWA dataset
- [x] Trim the Zarr dataset to include only the four target variables
- [x] Publish the dataset on NGC (https://github.com/NVIDIA/modulus/pull/330)

### Validation
- [x] Verify CorrDiff runs on multiple nodes

I'm gathering here the suggestions from our discussion:
Code structuring

[ ] Split generate.py, train.py and training_loop.py into functions/classes that handle more well-defined tasks
[ ] Reduce redundancy between the above files
[ ] Clean up handling of config files
[ ] Specify clearly in an interface what interface data loaders need to follow, to facilitate adding new loaders
[ ] Same as above, but for models (e.g. U-Nets)
[ ] Interoperability with diagnostic model code base (TBA)

Optimizations

[ ] AMP support
[ ] Support distributing generation over time steps as well as ensemble members

Moving these suggested items to a new epic for CorrDiff enhancements: https://github.com/NVIDIA/modulus/issues/353

Epic Completed!


add shardId metrics to read history branch

What changed?
Add ShardId metrics to read history branch to detect hotshards for this operation

Why?
Help detect hotshards

How did you test it?
Unit tests/Int tests pass

Potential risks
No risk

Pull Request Test Coverage Report for Build 01880804-9da9-40ef-802a-3dbeb89e4d1f

8 of 10 (80.0%)  changed or added relevant lines in 2 files are covered.
83 unchanged lines in 13 files lost coverage.
Overall coverage increased (+0.02%) to 57.238%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

common/persistence/persistenceMetricClients.go
7
9
77.78%

Files with Coverage Reduction
New Missed Lines
%

common/persistence/sql/common.go
2
50.65%

common/persistence/sql/sqlplugin/postgres/db.go
2
85.0%

common/task/weightedRoundRobinTaskScheduler.go
2
89.64%

service/history/task/transfer_standby_task_executor.go
2
86.1%

service/matching/matcher.go
2
91.46%

common/persistence/nosql/nosqlplugin/cassandra/workflow.go
3
59.1%

service/history/queue/timer_gate.go
3
95.83%

service/history/task/fetcher.go
3
91.75%

service/matching/taskListManager.go
3
81.15%

common/task/fifoTaskScheduler.go
7
82.47%

Totals

Change from base Build 018807fb-a304-4d9e-81b4-cefe3b944952:
0.02%

Covered Lines:
86440

Relevant Lines:
151019

💛  - Coveralls


Notify users when they accumulate drinks

As a user, I want to know when I accumulate drinks, so that I don't have to use !drankbank every time I duel, have a social, or participate in games.

I think it does this already?


text/template: does not document how values are formatted

go version devel +3c6b668 Thu Jun 2 00:22:03 2016 +0000 linux/amd64
I may well have missed it, but text/template package doesn't seem to specify
anywhere how values are actually displayed. For example, by reading the
documentation, it's not clear whether a value's String method will be called
to produce text to display in the template.

I agree; the closest thing in the documentation currently is

The default textual representation of the value of the pipeline is copied to the output.

The "default" it refers to is (AFAICT) the same "default format" that the fmt package documentation refers to. Perhaps the fix is simply to say that values are formatted as if using the %v verb.
/cc @robpike @adg


/stuck: Command that shows diagram for when people query about the best "I'm stuck" workflow

Complete the following REQUIRED checkboxes:

[x] I have thoroughly read and understand The Odin Project Contributing Guide

[x] The title of this issue follows the command name: brief description of request format, e.g. /help: add optional @user parameter

The following checkbox is OPTIONAL:

[x] I would like to be assigned this issue to work on it

1. Description of the Feature Request:
It's quite common for people to ask questions about or in some way express frustration that highlights or raises suspicion about less-than-ideal learning habits and expectations during exercises/projects. Quite commonly they'll be along the lines of not "fully understanding" everything after reading a lesson, or feeling like they did but as soon as they got to a project it's out the other ear already, not revisiting past lessons, or something that hints at them reluctant to ask for help/thinking asking for help should only ever be a last resort and they should struggle alone as long as is possible first.
@bycdiaz drew up a very neat little diagram the other day which I think (perhaps along with a brief text-version), would be really useful to put in an embed callable by /stuck or some equivalent/better name. It should help streamline and make giving advice for this a little more consistent.
Maybe also add the link to AI and learning at the very bottom of it just mentioning the dangers there.
2. Acceptance Criteria:
A list of checkbox items that explain the requirements needed to be met to resolve this request, e.g.:

[ ] Typing /stuck will bring up the slash command in Discord
[ ] Optional user mention option
[ ] Bot embed contains this image
[ ] (Optional) includes text version of the flow chart (e.g. 1. ... 2.... 2.1 ..go to step 3   etc.)
[ ] Mentions negative impact of using AI during learning with a link to https://www.theodinproject.com/lessons/foundations-motivation-and-mindset#a-note-on-ai-code-generation like with the /ai command

Sounds wonderful Mao! I'd love to hear the copyright holder's thoughts @bycdiaz on this as well 😄.
I am of the opinion it could be useful to add a note on the /question template command because many people will end up asking a question when stuck. Could you expand on your motivation for including note of the /ai command?

I wouldn't object to including the diagram at all :)

If this does go ahead, would the preferred way to handle the image be by uploading it elsewhere then linking the URL in the command embed? Or by using the attachment builder which will upload the image using a local file directly?

Gentle bump
Had a lot of cases over the last few months where I've linked the workflow message/image which has shown to be really helpful.

@bycdiaz @CouchofTomato Just wanted to clarify if the assignment to Carlos was intended/means the idea is approved and he's been asked to work on implementing this.
Based on his reaction on Discord, there may or may not have been a miscommunication?

@MaoShizhong, like the idea, you're assigned.


Add 1-argument ensure!($expr)

Amends a subltle difference from anyhow. This PR is basically duplicated from https://github.com/dtolnay/anyhow/pull/129.

Hi! This seems like a small and useful change, maintaining that compatibility with anyhow's ensure!.
It looks like this failed CI due to some problems that weren't really your fault and now it's gone stale. Would you consider rebasing and updating the review now that tests are more stable and there are more folks around to handle contributions?

No worries :)
Rebased - no conflicts, didn't check if the changes are still valid

Thanks so much for the contribution! :D


Incompatible Types: String cannot be converted to Int

Also added the Butterknife library. But when I try to do BindView, I am getting this error.
error: incompatible types: String cannot be converted to int
`
buildscript {
repositories {
jcenter()
}
dependencies {
classpath 'com.neenbedankt.gradle.plugins:android-apt:1.8'
classpath 'com.oguzbabaoglu:butterfork-plugin:2.0.0'
}
}
// It is important that the library apt plugin are applied before butterfork-plugin
apply plugin: 'com.android.library'
apply plugin: 'com.neenbedankt.android-apt'
apply plugin: 'com.oguzbabaoglu.butterfork-plugin'
`

I need to get butter fork working for my project which depends on internet library projects.
This issues is blocking.
Also noticed that the sample (butterfork/butterknife-sample/) is working fine (not sure what the difference is)
Please do suggest if there is something specific that needs to be done.

This won't work if Butterknife is also added as a dependency. They use the same package. I'll change the package name for the next update.


Passwords through sql.yaml need to be accessed and set

Using environment variables in compose.yaml instead.


Is this syntax valid?

I'm looking at the example and find the this syntax (ro.observe(ref.current), ro.disconnect) a bit hard to comprehend, could anyone explain what it does here? Thanks.
Full context:
import { useRef, useState, useEffect } from 'react'
import ResizeObserver from 'resize-observer-polyfill'

export default function useMeasure() {
  const ref = useRef()
  const [bounds, set] = useState({ left: 0, top: 0, width: 0, height: 0 })
  const [ro] = useState(() => new ResizeObserver(([entry]) => set(entry.contentRect)))
  // what does (ro.observe(ref.current), ro.disconnect) mean here? is this even legal syntax for that there's no pair data types in js?
  useEffect(() => (ro.observe(ref.current), ro.disconnect), [])
  return [{ ref }, bounds]
}

Ended up it's comma operator, all's good!


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-k8s-25-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2091

Questions ?
Please refer to the Backport tool documentation


Extra docs for Statamic static site generation

There were issues I ran into and had to resolve when using Statamic with their Static Site Generation plugin. This isn't an issue with Torchlight exactly but with how SSG is performed in the plugin.
When generating the pages via php please ssg:generate, views are generated with the raw Torchlight placeholders:

After creating a Statamic custom page generator and forcing the page to be rendered in full by manually calling the RenderTorchlight middleware, we get the full HTML code snippets:

Results can be seen here: https://stevebauman.ca/create-api-responses-with-eloquent-factories/
Here's the code for forcing code generation:
// app/Providers/AppServiceProvider.php

use App\CustomGenerator;
use Statamic\StaticSite\Tasks;
use Statamic\Facades\Markdown;
use Statamic\StaticSite\Generator;
use Torchlight\Commonmark\V2\TorchlightExtension;

/**
 * Bootstrap any application services.
 *
 * @return void
 */
public function boot()
{
    Markdown::addExtension(function () {
        return new TorchlightExtension;
    });

    $this->app->singleton(Generator::class, function ($app) {
        return new CustomGenerator($app, $app['files'], $app['router'], $app[Tasks::class]);
    });
}

// app/CustomGenerator.php

namespace App;

use Statamic\StaticSite\Generator;

class CustomGenerator extends Generator
{
    protected function createPage($content)
    {
        return new CustomPage($this->files, $this->config, $content);
    }
}

// app/CustomPage.php

namespace App;

use Illuminate\Http\Exceptions\HttpResponseException;
use Illuminate\Http\RedirectResponse;
use Statamic\StaticSite\GeneratedPage;
use Statamic\StaticSite\Page;
use Torchlight\Middleware\RenderTorchlight;

class CustomPage extends Page
{
    protected function write($request)
    {
        $response = (new RenderTorchlight())->handle($request, function ($request) {
            try {
                $response = $this->content->toResponse($request);
            } catch (HttpResponseException $e) {
                $response = $e->getResponse();
                throw_unless($response instanceof RedirectResponse, $e);
            }

            // This has to be set/overridden, as additional pages being generated
            // by Statamic SSG will have a `null` content type and Torchlight
            // relies on this to return the full response instead of Blade.
            $response->headers->set('content-type', 'html');

            return $response;
        });

        $html = $response->getContent();

        if (! $this->files->exists($this->directory())) {
            $this->files->makeDirectory($this->directory(), 0755, true);
        }

        $this->files->put($this->path(), $html);

        return new GeneratedPage($this, $response);
    }
}

I hope this helps someone who is looking to do the same thing! 😄

Steve this is amazing, thank you for taking the time to write this up. I'll add it to the docs, I really really appreciate it!

Thanks for the tip to look at this issue for a solution @joshuablum.
I have a pair of red underlines in my code that prevents this from working.
Markdown::addExtension(function () {
   return new TorchlightExtension;
});

Undefined type 'App\Providers\Markdown' And Undefined type 'Torchlight\Commonmark\V2\TorchlightExtension'`
Hope you can point me in the right way.

Looks like you might have forgotten to import Statamic's Markdown facade, @mbootsman?
Try to add use Statamic\Facades\Markdown, like mentioned here and here.

Thanks @joshuablum -- I missed that import in the example, I've added it in 👍

Don't worry, it's nothing more than a Facade, @stevebauman. Thanks for the great work, really appreciated!

I am shocked... SHOOK! ... to see that these classes are not final @stevebauman
😏

lmao I would rather DIE @simonhamp


Routing of followed sources fixed

Routing of followed sources fixed

Change the routing, is way more user friendly and better designed


add asciidoctorj-diagram to list of artifacts

Thanks for the suggestion. I regret to say, however, that this is opposite of the direction we are trying to head. This strengthens the coupling between AsciidoctorJ and AsciidoctorJ Diagram, which we're trying to break. Instead, the artifact listing should go in the repository from which it's produced, such as: https://github.com/asciidoctor/asciidoctorj-diagram
Having said that, I do see value in having a landing page that lists the versions of all the AsciidoctorJ-related jars. Though perhaps it's something we can compute at the time we build out the (new) docs. It's a good requirement.

Hi Dan, thanks for information.  As an newcomer to these projects it wasn't clear to me that this artifact wasn't generated here.  I just assumed it was because the other two were listed here.

I wouldn't expect you to know that. You've arrived at a transition point. Although the references are still lingering, they are on the way out the door (to their own repositories).


compress pwa icons

hmm. slightly changed the color

O0oh, how many bytes were saved? And how did you do it?

i used the imageOptim app


 Could you specify what error do you hit ?

Looks like you commented on https://github.com/docker/docker/issues/23115 as well; can you describe the exact issue on that issue? Is the "Download" hanging (i.e., it never finishes?)
I'll close this issue, to keep the discussion in a single place, please continue on https://github.com/docker/docker/issues/23115

@HackToday @thaJeztah   pull progress stop at  4.651 MB/6.459 MB

Is this from Docker Hub, or a private registry?

@thaJeztah from Docker Hub


fix pyjwt version

ref https://github.com/flavors/django-graphql-jwt/issues/242
ref https://stackoverflow.com/questions/65757394/module-jwt-has-no-attribute-expiredsignature

@mikedebock the wrong pyjwt version was the only issue I found breaking the build, see ref above (it's also unrelated to my changes)
related PR https://github.com/labd/django-cognito-jwt/pull/40

wait before merging this please, looking at other PRs I don't think this is the way to go officially

@mikedebock I think this is good, I formatted and took in consideration older PRs (which can be closed now)
https://github.com/labd/django-cognito-jwt/pull/38
https://github.com/labd/django-cognito-jwt/pull/37
tests pass in all environments too

Thank you for your contribution @MichelML !

my pleasure @mikedebock , if you can close the other two issues and publish a new version, it would be appreciated :pray:

the two PRs, sorry :D #38 and #37 can be closed

@mikedebock you can also close #34 as this PR adresses the issue

@mikedebock once last thing, it seems the release didn't work https://github.com/labd/django-cognito-jwt/runs/4487961949?check_suite_focus=true
maybe an expired token problem?

@MichelML I'm on it, checking at my colleagues for a new token.

@MichelML 0.0.4 is available: https://pypi.org/project/django-cognito-jwt/

Awesome!!!!

Thanks a lot for the support @mikedebock

You're welcome! Thank you for your contribution @MichelML !


Failed to Verify

I'm getting this error and I don't know why.

You either do not have all the files placed into the folder or your antivirus is blocking the injection. Add an exclusion for the folder you have xivalexander in.

That was it. For some reason, Windows Defender lost my settings when I had added the \game\ folder to the exclusions. Thanks Microsoft.


Just want to report increased CPU with latest update

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS Version: [e.g. Windows 10 1903 18362.175]
App version: [v0.11.0]

Additional context
Add any other context about the problem here. Does this problem occur again after restarting the app?
Log file
Please post the log file here so that we can understand your problem better. You can access it from Settings->About->Open log location.

@whdmike We usually expect users to fill out the issue template when reporting a bug or submitting a feature request, please reopen this issue when you have filled out the issue template.

I usually expect developers to be more user friendly. I am reporting
this to help you and to help improve the product. I am not developer or
some computer geek, I just love software and I like to help, sometimes,
by giving feedback. I think I will also "close" this contributing.
Thanks for your hard work and wish you luck with project.

@whdmike https://github.com/whdmike We usually expect users to fill
out the issue template when reporting a bug or submitting a feature
request, please reopen this issue when you have filled out the issue
template.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/files-community/Files/issues/2676#issuecomment-750322799,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKYBUADXFZNK5JAF44X5ZG3SWH4WZANCNFSM4VGSUCDA.

I usually expect developers to be more user friendly. I am reporting this to help you and to help improve the product. I am not developer or some computer geek, I just love software and I like to help, sometimes, by giving feedback. I think I will also "close" this contributing. Thanks for your hard work and wish you luck with project.
…
@whdmike https://github.com/whdmike We usually expect users to fill out the issue template when reporting a bug or submitting a feature request, please reopen this issue when you have filled out the issue template. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#2676 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKYBUADXFZNK5JAF44X5ZG3SWH4WZANCNFSM4VGSUCDA.

@whdmike User feedback is particularly important to us and we appreciate you sharing your concerns. In fact, Files is only the way it is because of feedback from users like you. In order for us to manage issues and track feedback, we usually expect issue reports to include information that can help us investigate the issue. The issue template is there to help guide the user to include information that will help us investigate the issue. In this particular case, a couple of things can help us.

The version number
When do you notice the increased CPU usage?
How large is the difference from previous versions?
Can you still reproduce the issue after restarting the app?

We hope that this gives a little bit of a better understanding into our process and that you will continue to report feedback in the future.

I am reopening this in anticipation for more information.

I understand, but sometimes as a user you just want to help and you
don't have time for a big report. I thougt if more users report about
higher CPU it will get your attention.
Files <IP_ADDRESS> I have 12 GB of ram, Intel I-3 processor and compared to
previous version, computer while using Files became significally louder
and slower. This is my observation and I stoped using Files when more
programs are open. This is what I can contribute at the moment. If no
one else is expiriencing the same, just close the case.

@whdmike Thank you for the extra information, I am keeping this open so we can investigate.

@whdmike We don't charge for our code, nor have we charged for our labor before. We don't owe the community any type of support for defects in it, and aim to fix these on a best effort basis. If you don't have time to fill out a ticket, then please don't bother opening one to begin with.
I'll close this now.

@duke7553, I am sorry, your communication skills are unfortunately on a
level of a computer geek who forgot how to talk to humans.


Unable to run test suite from a clean copy

Hi there!
I've just forked this project and want to run the test suite on my laptop... I'm wondering if there is some extra setup step that I'm missing.
If that's the case, I may revise the README.md and begin contributing to this project 😃
What I've done so far
✅ 1. fork and clone the repo
✅ 2. create a virtual env and install the dependencies from  requirements.txt and requirements_dev.txt with pip install -r
🔴 3. Run python setup.py test (I've tried also python -m pytest .)
Helpful things

this a freeze of my current env:

$» pip freeze
alabaster==0.7.12
Appium-Python-Client==0.50
Babel==2.8.0
behave==1.2.5
certifi==2019.11.28
chardet==3.0.4
coverage==4.3.1
coveralls==1.1
docopt==0.6.2
docutils==0.14
extras==1.0.0
fixtures==3.0.0
fuzzywuzzy==0.18.0
idna==2.9
imagesize==1.2.0
Jinja2==2.11.1
lettuce==0.2.23
linecache2==1.0.0
MarkupSafe==1.1.1
mock==2.0.0
needle==0.5.0
nose==1.3.7
parse==1.15.0
parse-type==0.5.2
pbr==5.4.4
Pillow==7.0.0
py==1.8.1
Pygments==2.2.0
pytest==2.9.2
python-mimeparse==1.6.0
python-subunit==1.4.0
pytz==2019.3
requests==2.23.0
requests-mock==1.2.0
screeninfo==0.3.1
selenium==3.141.0
six==1.14.0
snowballstemmer==2.0.0
Sphinx==1.5.1
sure==1.4.11
testtools==2.4.0
traceback2==1.4.0
typing==<IP_ADDRESS>
unittest2==1.1.0
urllib3==1.25.8

The failing tests are from toolium/test/test_visual_test.py

[...]
toolium/test/test_visual_test.py ....................FF.....FFF.FFF..FF....FF..
[...]
================================== 12 failed, 316 passed, 1 error in 39.14 seconds ===================================

Some failing example outputs

self = <needle.engines.imagemagick_engine.Engine object at 0x1059c3190>
output_file = '/Users/martero/workspace/bundle/toolium/toolium/test/output/visualtests/2020-03-24_151907_firefox/01_screenshot_ios_web__screenshot_suffix.png'
baseline_file = '/Users/martero/workspace/bundle/toolium/toolium/test/resources/ios_web_exclude.png', threshold = 0.1

    def assertSameFiles(self, output_file, baseline_file, threshold=0):
        diff_file = output_file.replace('.png', '.diff.png')

        compare_cmd = self.compare_command.format(
            compare=self.compare_path,
            baseline=baseline_file,
            new=output_file,
            diff=diff_file)
        process = subprocess.Popen(compare_cmd, shell=True,
                                   stdout=subprocess.PIPE,
                                   stderr=subprocess.PIPE)
        compare_stdout, compare_stderr = process.communicate()

>       difference = float(compare_stderr.split()[1][1:-1])
E       ValueError: could not convert string to float: b'ompare'

or
self = <needle.engines.imagemagick_engine.Engine object at 0x10591f510>
output_file = '/Users/martero/workspace/bundle/toolium/toolium/test/output/visualtests/2020-03-24_151907_firefox/01_screenshot_ios__screenshot_suffix.png'
baseline_file = '/Users/martero/workspace/bundle/toolium/toolium/test/resources/ios_excluded.png', threshold = 0.1

    def assertSameFiles(self, output_file, baseline_file, threshold=0):
        diff_file = output_file.replace('.png', '.diff.png')

        compare_cmd = self.compare_command.format(
            compare=self.compare_path,
            baseline=baseline_file,
            new=output_file,
            diff=diff_file)
        process = subprocess.Popen(compare_cmd, shell=True,
                                   stdout=subprocess.PIPE,
                                   stderr=subprocess.PIPE)
        compare_stdout, compare_stderr = process.communicate()

>       difference = float(compare_stderr.split()[1][1:-1])
E       ValueError: could not convert string to float: b'ompare'

=== EDIT ===
Python version
I suspect it's related to python3 since running 2to3 writes some changes,
Python 3.7.5
$» pip --version
pip 20.0.2 from /Users/martero/.virtualenvs/toolium/lib/python3.7/site-packages/pip (python 3.7)

thanks!

@rgonalo any advise or tip here?
thanks!

You are right, unit tests assume that ImageMagick is installed, but they should not.
I have updated unit tests to use PilEngine instead of ImageMagick: https://github.com/Telefonica/toolium/pull/168.
Let me know if it works for you.

Thanks! #168 did it :)
===================== 328 passed in 47.34 seconds =====================


Feature request: Clustering

Hi!
I wonder if it could be possible to cluster markers like Google Maps Android Marker Clustering Utility does.
Thanks!

I've done something similar in the past with arbitrary logic (zoom smaller than X, use clusters), but there's nothing formal on the horizon.  The next major version (3) is a long way out, but will use a plugin style architecture, and this feature might fit in nicely there.  I'm going to leave this issue open as a feature request until then.  FWIW, v3 is probably a good ways out.
Thanks for your feedback.

This library could be helpful in figuring this issue out.
P.S.
I, too, have done something related to clustering in the past. Feel free to check it out here, maybe you could get some more ideas...


Replace flake8 with ruff (which is equivalent but faster)

ruff is a very fast Python linter written in Rust that covers the flake8 rules in addition to a bunch of other popular linters. This sets up a config for it that covers the existing flake8 config, and adds additional linters while generating minimal errors with the existing code base. Since pylint is much slower than ruff (and I suspect slower than flake8 too), this config executes the ruff checks first in order to fail fast.

Hmm, looks like we're running the pre-commit hooks on all files in the lint action, and it doesn't seem to be picking up settings from pyproject.toml.


chore: Remove link locale

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

Thanks for the contribution, @nschonni
#sign-off

@iainfoulds: I'm sorry - only the author of this article, @wesmc7777, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

Verified removal of en-us link locale in all files.


Can't build installer - Unit 'cxGraphics' not found

Sorry, I can't build installer project due cx* and dx* units are missing:
[dcc32 Fatal Error] uMain.pas(7): F2613 Unit 'cxGraphics' not found.

These units are not necessary, you can just remove then from there. I'll remove it and commit in some minutes.

I sent the new source can you check ?

Yep, after pulling changes, adding DelphiAST and Log4pascal library paths, Installer.exe has been built.
Thank you.


Added independent branch 'noAntennaInfo'

An independent branch of QTNMSim is available, called noAntennaInfo. All antenna calculations were removed and a pure trajectory output implemented, including all the required information for antenna response calculations: the full 3-bectors of position (SI units), velocity (as relativistic beta vector) and acceleration (SI units), the step time vector and the kinetic energy (in keV) for each step of the trajectory.
Output files must be expected to increase significantly in size. An additional paragraph in the README describes further information on this branch version.

@yramachers Do you have an opinion on the long term status of this - essentially do we plan to maintain separate versions, or add this as a feature in main?

@TomGoffrey I would like this output to become an option, longer-term. Maintaining two separate branches makes no sense. For now it is the most convenient way to manage that feature but that can only stretch so far. I haven't got a good idea on how to merge this into the main branch, plenty of changes are required. That doesn't mean it is impossible but it needs more effort than I have available for a while. If this output format becomes an optional choice then there must be a strong hurdle separating the two output options since picking this option carries a large cost in output size when getting it wrong.


move dependencies to npm

bower is dead and all dependencies should be managed by npm

There are no dependencies. The only thing Kalendae relies upon is Moment, and it's bundled into the library. The bower dep is just there for people who want to use the unbundled version so they can have Moment globally. You'll notice that Moment is not included in the npm dependencies.

Then it's unclear to me why it is a dependency in bower.json but not package.json. Is it required by them eg for imports/requires?

I actually didn't write the Bower.json, it was added via a pull request, so I don't know why it's there. I've historically had a lot of problems with the bower registry because of third parties pushing my libraries up there and then letting them rot. I don't really interact with it at all, I just push tags for the new releases and supposedly that makes it work.

I guessthe line can be deleted as the dependencies are bundled here https://github.com/Twipped/Kalendae/tree/master/src


Client side validation

in this pull request, I developed functions to permit only information that matches a certain pattern in the contact form and throw errors whenever a given type of data is wrongfully entered.

Hi @Kidd254,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.

Invalid Code Review Request
❌You have submitted a project, that has not passed linter checks. Kindly fix the ESLint issues and submit your PR again.

❌ This project has to be completed with pair programming. Kindly check the work and submission mode here.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


cherry-pick (4.13): Fix double evaluation and filter sensitive settings (#5641)

Description
Fix the double evaluation and filter sensitive settings

Pull Request Test Coverage Report for Build 250858

0 of 0   changed or added relevant lines in 0 files are covered.
25 unchanged lines in 6 files lost coverage.
Overall coverage decreased (-0.03%) to 78.628%

Files with Coverage Reduction
New Missed Lines
%

/libraries/AdaptiveExpressions/BuiltinFunctions/GetNextViableTime.cs
1
93.94%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/SetProperties.cs
1
66.67%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/SetProperty.cs
2
60.0%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/EndDialog.cs
3
72.0%

/libraries/Microsoft.Bot.Builder.Dialogs/Memory/Scopes/SettingsMemoryScope.cs
3
96.0%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/HttpRequest.cs
15
75.86%

Totals

Change from base Build 245265:
-0.03%

Covered Lines:
18049

Relevant Lines:
22955

💛  - Coveralls


mac-7 & mac-9 lost external connection phone device.

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
mac-7 & mac-9 lost external connection phone device.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

Rebooting machines to restore both connections.


Feature request: Support disabling of extended type support

One mechanism for this would be to see if the user has an @Override in place on the x/setX method, as this tends to be dropped in extended type support. This has the benefit of not extending the FreeBuilder API (unlike, say, an annotation), but is non-discoverable, and could be very surprising for maintainers of code written using it.

Also, this might come for free if feature request #14 were to be implemented.


Update donation widget proportions

What:

Update donation widget proportions
Upgrade the header component version

Why:
Closes: https://app.asana.com/0/1168997577035609/1197758668914745/f
How:

Update the padding y from py-2 to py-3 to elements within the widget following the proportions being used in the charity donation widget

Media

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix Cagtegories feature

Removed the category: Category? property from MenuItem model

MenuItem had categoryId: String? and category: Category?, but category: Category? wasn't really used

Fix prepopulated items

There is aPrepopulate data button under the settings page, but it wasn't working correctly
writeStrategy was insertIfAbsent, but the document IDs were just the names of the category
means that they couldn't be added to other flights

Fixes #13

This branch did not build for me, with SwiftHEXColors linking failure. Neither did the Env.swift appear to be generated, however the build failure was in the linking phase.

I git-pulled this repository as a new project, and I saw the same issue, but it got fixed after #26.
I'll merge #26 to this branch as soon as it's merged into main.


Make requestBody required

Fix issue raised here.

There are several solutions:

Modify the schema to mention that the requestBody of paths is required. The OpenAPI Generator will likely use this information when generating the clients and inform the user that a request body object must be specified, even if it's an empty object.
Keep the schema as it is but modify the ROCC API service implementation so that it basically comply with optional request body, i.e. the API service must not return an error if the request body is missing.

My preference goes to Option 2. This solution increases the consistency of the API as both the user and the developer know that a request body object must always be specified. The developer can then validate this object against a model. The alternative is more complicated: first the developer has to look at the schema to check if the request body is required or not, and based on the answer implement an if condition in the controller of the API service.
Note that Option 2 is the solution adopted by Google Healthcare API. Moreover, we had a similar discussion a couple of months ago with @vpchung and @thomasyu888 about the returned object of DELETE operations. We decided to always return an empty object rather than nothing. The logic was the same and the alternative would have been that both the user and the developer would have to look closer at the schema to know if a given endpoint should return an object or not. Always returning an object makes our API more consistent.
Tagging @rrchai

Done


[WIP] Return custom buttons for service having nil service template

For bz https://bugzilla.redhat.com/show_bug.cgi?id=1595051.
Services with nonexistent service templates should return all buttons applicable to services, I spose.
@eclarizio can you :eyes: for me please?
@tinaafitz

@d-m-u if this can be backported can you add the gaprindashvili/yes label.

@miq-bot add_label gaprindashvili/yes

@miq-bot remove_label wip

Yes, that looks correct.

@gmcculloug @syncrou can you 👀 again for me please?

@d-m-u Please see failing tests.
We initially looked at using prepend but found that the included block in the CustomActionsMixin was not run which lead to other issues so we went with the current solution.
https://github.com/ManageIQ/manageiq/blob/master/app/models/mixins/custom_actions_mixin.rb#L4-L9
Open to suggestions on this one.

We initially looked at using prepend but found that the included block

Yeah, @d-m-u asked me about that, include and prepend are very different.  I'm still not sure what the goal is...

@bdunne Services can be backed by a ServiceTemplate or created without one.
Custom buttons can be attached directly to a ServiceTemplate and/or to the Service class (meaning all service instances get them).
The existing code assumes that there is a ServiceTemplate and if so we get the buttons from there.  If there is no ServiceTemplate then nothing is returned.  This change handles a stand-along Service (no parent ServiceTemplate) and supports the custom buttons that are defined on the Service class.

@eclarizio PTAL

Gaprindashvili backport details:
$ git log -1
commit 0378482d9f90600e3614cf65573b87f372bf04c2
Author: Greg McCullough<EMAIL_ADDRESS>Date:   Tue Jul 17 19:01:41 2018 -0400

    Merge pull request #17703 from d-m-u/bz1595051

    Return custom buttons for service having nil service template
    (cherry picked from commit 2d8f3e83bcc4e9ea519596957e99d9e69baf5041)

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1603031


1282 search filters

Implements #1282
Make sure to run yarn before trying this as the search fields won't show up on the older version of MUI DataGrid.
This includes an update to all of our DataGrids  - Operators, Operations and Users. Now when you search or sort that gets added to the URL searchParams. You can now refresh the page or save a bookmark and load your previous sort order 🎉

Since I've refactored all the DataGrids it would be good to make sure the Operators DataGrid for cas users is working still as well as the Users DataGrid for industry_user_admin.
As discussed at standup, the submission_date field doesn't have a search filter due to it being saved in a timestamp format, it wouldn't return any data if the user typed in a month ie jan. We may add this in the future if it's decided to be important.
No search for the Action column either as it's not a database value
Refactored all of the base page.tsx routes and children for our DataGrid components. page.tsx files receive searchParams props so we can pass those to the server components and fetch the correct page/sort/filter data on the serverside first render
I initially installed the new DataGrid v7 which supports column groups which I used for the search. However there are accessibility errors with it that I found in the accessibility PR so I upgraded to the latest v6 and used the experimental columnGrouping feature. I created a tech debt ticket to upgrade in the future . The upgrade is pretty straightforward aside from some style tweaks and testing selectors. Hopefully they fix the aria issues soon!

Go get some rest man 😜

First of all, apologies in advance for this suggestion because I know you've put a lot of time and effort into developing this feature.
I've just learned that the free version of MUI Datagrid has a search feature called Quick Filter. It doesn't show up above every column like the design, but the designers did the grid based on CIF because they thought that would be easiest for us, so the exact placement of the search may not be that important.
Quick search is interesting because it searches every column at once and actually worked on dates. I threw it into a test branch (I didn't want to suggest overhauling this PR without checking if it would actually work) and you can see how it looks here: https://www.loom.com/share/0398dfc3a99746cbb1db0ae462f0d0e9. (branch here. It's a lot less code and therefore more maintainable, so even though this PR is working, I think we should at least consider switching to quick search/column filters (column filters are also free and again not exactly like the design, but they're so much less code I think it's worth bringing up with the designers).

I saw the quick filter as well when I was researching how to implement the design. The reason it's seemingly working so well out of the box is that it's just doing a search for the single page data that we have fed into the page (including nice readable dates). It's not doing a database search so the dates wouldn't work once we did a proper implementation of this.
As for the column filters as discussed we would have implemented that in this ticket if I didn't find a way to implement the design. Most of the changes and refactoring in this ticket would stay the same aside from the columnGroups, the HeaderSearchCell and some minor DataGrid styling updates.

First of all, apologies in advance for this suggestion because I know you've put a lot of time and effort into developing this feature.
I've just learned that the free version of MUI Datagrid has a search feature called Quick Filter. It doesn't show up above every column like the design, but the designers did the grid based on CIF because they thought that would be easiest for us, so the exact placement of the search may not be that important.
Quick search is interesting because it searches every column at once and actually worked on dates. I threw it into a test branch (I didn't want to suggest overhauling this PR without checking if it would actually work) and you can see how it looks here: https://www.loom.com/share/0398dfc3a99746cbb1db0ae462f0d0e9. (branch here. It's a lot less code and therefore more maintainable, so even though this PR is working, I think we should at least consider switching to quick search/column filters (column filters are also free and again not exactly like the design, but they're so much less code I think it's worth bringing up with the designers).

I saw the quick filter as well when I was researching how to implement the design. The reason it's seemingly working so well out of the box is that it's just doing a search for the single page data that we have fed into the page (including nice readable dates). It's not doing a database search so the dates wouldn't work once we did a proper implementation of this.
As for the column filters as discussed we would have implemented that in this ticket if I didn't find a way to implement the design. Most of the changes and refactoring in this ticket would stay the same aside from the columnGroups, the HeaderSearchCell and some minor DataGrid styling updates.

OK, thanks! So all of the data grid complication is because of pagination, and we have to paginate so we don't have load problems.

OK, thanks! So all of the data grid complication is because of pagination, and we have to paginate so we don't have load problems.

Yeah the reason there is so many changes in this PR is because of refactoring all DataGrids due to changes with handling the state in the query params. This is a big quality of life upgrade and and is seen as best practice by many - Shon emailed me this article.
So even if we went to a single field search most of the changes would remain. As an aside, I definitely considered pushing to use the quick search and have a single search for all fields, though I don't feel strongly about which is better UX so I implemented the design as is.

I manually re-tested the focus and pagination and everything's working well. I found one styling bug:  that could be tech debt or part of a larger ticket for mobile display (I'm not on mobile but I use the accessibility settings on my computer to make everything bigger and it kind of has the same effect). I also had a mini row to the right of actions, but part of it being blue is new from this PR.

Thanks, I fixed this!


DuckDB loader example

This example shows just one of the many ways one would like to use DuckDB as a data loader. It illustrates:

downloading a raw csv file from Eurostat and keeping a local copy so we can iterate faster on the data loader
filtering data
exporting as parquet
making a chart

Eurostat is a huge beast, but I think this particular dataset is both simple enough to understand, and entertaining enough that you can just look at the chart and learn interesting stuff.

Issue to display "unknown" in the legend:
https://github.com/observablehq/plot/issues/651


Add finalizer removal hook during helm delete for common case for release CR

Describe your changes
Due to new finalizer behavior for releases, we may leave releases hanging around in the namespace after
chart has been deleted.
This change fixes that behavior for the common case where the operator is working in namespace mode
and operator namespace is same as the cr namespace.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
anijhawan@CYWWYVVT7L templates % helm ls -n operator-test
NAME            	NAMESPACE    	REVISION	UPDATED                             	STATUS  	CHART                      	APP VERSION
chart-1734653701	operator-test	1       	2024-12-19 16:15:02.657514 -0800 PST	deployed	yugabyte-k8s-operator-0.1.5	2.25
anijhawan@CYWWYVVT7L templates % helm delete -n operator-test chart-1734653701
release "chart-1734653701" uninstalled
anijhawan@CYWWYVVT7L templates % kubectl get release -n operator-test
No resources found in operator-test namespace.

can we also handle finalizers for ybuniverse in the hook?


Typo

The word 'joint' in 'create a new local Windows account and joint it to a local user group' has an extra 't'.

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

ID: 309bae29-c769-5051-6c3f-aebeb455cd34
Version Independent ID: bbdfad82-025e-8aee-b3fc-bcfb8b110674
Content: Accounts CSP
Content Source: windows/client-management/mdm/accounts-csp.md
Service: unspecified
GitHub Login: @nickbrower
Microsoft Alias: maricia

Fixed--thank you!


Adding book about go lang

What does this PR do?
Add resource(s)
For resources
Description
Interesting study material on go lang, helps to take the first steps in the language.
Why is this valuable (or not)?
I believe that go lang is a very interesting language to produce micro services, the more material about it the better.
How do we know it's really free?
It is an academic article.
For book lists, is it a book? For course lists, is it a course? etc.
This is not a book, it is a PDF document.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

added in #9203


Unable to submit empty rule attribute

This is for the use case that you want to nullify all rules for a given node_group.

@beergeek This is for you.

  Passing a false or undef assumes other properties in providers.  Is there one that is preferred?

I have no preference

Migrated to: https://github.com/WhatsARanjit/prosvcs-node_manager/issues/11


Image rows with invalid version number for PDM

Description
From the add_license_url DAG run, 43 images were found with an invalid license, "PDM 4.0" (version four specifically doesn't exist).
The following *invalid license(s)* were found and will be skipped:
╭────┬───────────┬───────────┬─────────╮
│    │ license   │   version │   count │
├────┼───────────┼───────────┼─────────┤
│  0 │ pdm       │       4.0 │      43 │
╰────┴───────────┴───────────┴─────────╯

Details of rows implied
These are the identifiers of the rows in a query to get more information.
Extracted with this query;
SELECT identifier FROM image WHERE license = 'pdm' AND license_url = '4.0';

SELECT * FROM image WHERE identifier IN (
'394789b1-59ed-477c-8cab-2c12c44f8881',
'c2cd1c63-c1c6-4ae3-86ba-a559878e5c29',
'a565409e-7b34-4729-9a03-ef7a2bbedf69',
'4a3477a5-88fe-4182-8075-1331fb025bdf',
'508e0900-3893-48af-a2a8-ca7f349a1bd0',
'a8581655-211d-4468-96e9-e91dda27ec16',
'02728167-46d9-44e4-9666-55c0f0b97bef',
'5a51a5e7-e511-4886-b410-42f659130aac',
'6965802b-a93a-453d-88e1-e4c414f8cd38',
'2b37a989-f5c1-4f9a-9c5f-07c64b7f455b',
'5c5d6856-6d7f-47df-9490-96c5900e016d',
'cf8d179d-0944-42e3-bf11-488765b22bd2',
'0a6f1a62-abd5-49e0-8f93-5c068c0aa6fa',
'162b0827-2825-43d2-ac64-c7e8575411a0',
'4414f2c9-0dcb-47f8-9998-b2a34e8b7863',
'9e95075a-2aaa-4683-aae2-d8d20f1343fb',
'32959d92-9141-4810-8e79-85fccf848027',
'f59aa983-1c54-4a61-8985-6a4e264b1c31',
'521123e9-1a5c-42d8-a7d8-464e263b84d1',
'8888f5c8-4ad1-488e-8cad-bbd1cc04334a',
'57a87e1d-61a6-4df5-accc-0cae8339ab1b',
'5492c666-cbd3-4bde-996b-47c9d4abeb62',
'ad00fa52-7a62-49b3-840a-cf09f1eebf61',
'b74a4327-f92f-4df0-bb56-660a39ac7fe4',
'f8f33710-2e7a-44c4-9cb6-1777b2086904',
'6221eebc-3e45-4c22-9ef8-c1f9ea766ade',
'4f506b2a-1d86-4760-a6ff-fcecb5f3491c',
'b2e6be7f-bf6f-4ccc-a1dd-725e7c42e9d1',
'9aba2139-d1f3-47d8-9a58-e3fe18006ff5',
'c6080bb3-b314-4e66-b413-9e5132837b28',
'fa1eee5c-8149-4646-89b2-c97dd9d04fe1',
'5c3180cd-4ab8-496e-98bc-55296e946deb',
'9a40cf3e-6f95-466b-8561-a977820197f2',
'd9fc460a-4200-4ccb-a663-852fc58b9538',
'be98bf10-7166-4f16-9e1b-98ed0d1e57ed',
'e344e6ee-41a4-4823-8679-aa8d9ce4ba36',
'dfe52b22-24c5-483b-88a5-6655af0bb1ec',
'b341be5d-021b-4724-b5a2-80de20badb5d',
'27b7e367-52e1-44df-9ffc-53103431d019',
'94a730f3-6b1e-467a-b0e4-c36e7c8616a0',
'b8b87ea5-4e09-487f-acd8-b73c6116f8c0',
'3262db1c-b3fa-4ee9-9c14-84c1ce5c23c6',
'60e346e8-e492-47e8-b5a9-353bc3bd289e'
);

On an analysis of these items, besides the confusion of potentially dealing with a CC0 license instead, given the "License information" provided in landing pages, see, for example, https://digitaltmuseum.org/021176009794/arko-1957, it was determined that these are, in fact, under PDM. From @AetherUnbound:

In the HTML source for this page, there's a metadata block (set to app.data), and all the references to license within it refer to PDM (in a few locations)

"licenses":[{"description":"Public Domain-m\u00e4rke (PDM)","label":"","link":null}]

Proposed solution
To fix this, we'll change the version to "1.0" like the other existing marks in the database. Note that in the query below, the extra two conditions of the last part are not strictly necessary but add a safety check.
UPDATE image SET license_version = '1.0', updated_on = NOW()
WHERE identifier IN (<list copied above>) AND license = 'pdm' AND license_version = '4.0';

Environment
This is in the catalog database.
Additional context
Related to #3885 and #4318.

The query makes sense! Do we also need to update the license_url field in meta_data as part of this query, or will we rerun add_license_url after it?

I'll rerun the DAG. Thanks for double-checking with me!


config: remove dmr version config

The previous version of dmr has reached the end of its lifecycle and will no longer accept new PRs. Therefore, delete the configurations for these versions.

/lgtm
/hold

/unhold


Module/Pubstream is not working

What it says... Have not investigated further yet...

Check admin->site->Import Public Streams and also admin->security->Block Public; both of which will turn this into a blank (but themed) page. We may need a message that the public stream functionality is not enabled on this site.

They are both set right... I suspect an issue with $load and $update...

Ha! Thanks...