Auto-update go-utils to latest version

:robot: Beep boop I am a bot
This is an automatically created PR to change keptn/go-utils to version 51257d664f819c10230a329ff458f82f0fe7075e.
Please consult https://github.com/keptn/go-utils/actions?query=workflow%3A"Auto+PR+to+keptn%2Fkeptn" for more information.

Codecov Report

Merging #2885 (7cc5a55) into master (047a7f7) will increase coverage by 2.71%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2885      +/-   ##
==========================================
+ Coverage   36.07%   38.79%   +2.71%
==========================================
  Files         104       33      -71
  Lines        6461     4173    -2288
  Branches      197        0     -197
==========================================
- Hits         2331     1619     -712
+ Misses       3822     2315    -1507
+ Partials      308      239      -69

Flag
Coverage Δ

moduleA
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

bridge/client/app/_services/clipboard.service.ts

bridge/client/app/_models/problem-states.ts

bridge/client/app/_models/trace.ts

...http-loading-bar/ktb-http-loading-bar.component.ts

bridge/client/app/_models/approval-states.ts

bridge/client/app/_services/http-state.service.ts

platform-support/openshift-route-service/main.go

bridge/client/test.ts

...ponents/ktb-event-item/ktb-event-item.component.ts

...nts/ktb-project-tile/ktb-project-tile.component.ts

... and 61 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #2885 (7cc5a55) into master (047a7f7) will increase coverage by 2.71%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2885      +/-   ##
==========================================
+ Coverage   36.07%   38.79%   +2.71%
==========================================
  Files         104       33      -71
  Lines        6461     4173    -2288
  Branches      197        0     -197
==========================================
- Hits         2331     1619     -712
+ Misses       3822     2315    -1507
+ Partials      308      239      -69

Flag
Coverage Δ

moduleA
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

bridge/client/app/_services/clipboard.service.ts

bridge/client/app/_models/problem-states.ts

bridge/client/app/_models/trace.ts

...http-loading-bar/ktb-http-loading-bar.component.ts

bridge/client/app/_models/approval-states.ts

bridge/client/app/_services/http-state.service.ts

platform-support/openshift-route-service/main.go

bridge/client/test.ts

...ponents/ktb-event-item/ktb-event-item.component.ts

...nts/ktb-project-tile/ktb-project-tile.component.ts

... and 61 more

Continue to review full report at Codecov.

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


Create a materialized view which contains several joins

Tpch q2 would be a good candidate.

@lmatz Could you please take a look?

We have executed tpch q2 for a period of time now:http://risingwave-perf-test-dashboard-metabase.us-west-2.elasticbeanstalk.com/collection/97-modified-tpch-q2-bs-medium-1cn
As the figure shows, the source throughput and sink throughput are both quite high, i.e. enough to saturate the throughput of an iceberg sink in the short term I suppose (without concurrent writes?).
But we need to integrate it with the sink. If the sink is ready, we can work in the kube-bench repo to construct the end-to-end pipeline.


Add openat-nocache

I'll do fcntl version separately

Ok so #232 was merged which (we hope) provides the same functionality as this but with a simpler API and implementation. We can keep this branch just in case the other approach doesn't work out, but closing for now.


Blog post about syntax sugar $

In EO we have $ object which is actually a syntax sugar and refer to current abstract object it's used in
So the next code:
[] > foo
  assert-that > @
    2.plus 2
    $.equal-to 4

actually means:
[] > foo
  assert-that > @
    2.plus 2
    foo.equal-to 4 #<EMAIL_ADDRESS>--> assert-that.equal-to

It would be nice to write a blog post about it

@Graur well done! I'm closing it


Add window{open/close/movetoworkspace} IPC events

I am making a custom status bar using the eww framework. And I need to know when windows open, close and move between workspaces.
Open/close is pretty straightforward. I tried to emit IPC events in Events::listener_mapWindow/Events::listener_unmapWindow and it worked fine.
Move, on the other hand, is tricky, because CWindow.m_iWorkspaceID is public and is being changed all over the place. There are at least 3 places where I need to emit the "move" IPC event:

Move window via keybind
Drag window and switch workspace
Pick window, switch workspace and drop it (without mouse movement)

I suggest hiding CWindow.m_iWorkspaceID behind getter/setter. Then it will be possible to emit the event just in the setter.
Would you be willing to merge that?

don't hide it - instead make a method (either in CWindow or CCompositor) like moveToWorkspace and do it there.
Yes, I would be willing to merge that, provided it's coded well, so please don't write ass code.

@vaxerski OK, here are the events
openwindow>>{address},{workspace_name},{class},{title}

closewindow>>{address}

movewindow>>{address},{workspace_name}

Any objections/suggestions?

lgtm.


Database Name not optional in Server info

Database Name is not optional in Server info page. If it is left blank, Save button cannot be clicked.

this is intentional. With the mongo db driver you cannot connect to a remote database without specifying which database you want to connect to. If you are creating a connection to either localhost or <IP_ADDRESS> then it will allow you to leave it blank.


ember-cli torii facebook oauth2 Popup was closed or authorization was denied

So I have an application created in ember-cli. Torii is being injected properly on my route, and the Facebook Oauth2 provider launches properly and allows me to log in and receive an authData response.
However it gets a bit odd at that point, if my application is listed under the apps on my Facebook account (not on the developers site but, my personal Facebook as a user would access), launching the authentication using "this.get('torii').open" still triggers the popup, however it closes immediately and reports "Popup was closed or authorization was denied" as opposed to any auth data / any indication that I am signed in.
The even stranger behaviour is, if I log into my Facebook and delete my app from the app settings and try to log in again, I will occasionally receive an authData response, otherwise it'll give me the usual "Popup was closed or authorization was denied" error. I am a bit new to ember-cli and Torii but I can't seem to pinpoint why I only sometimes receive the authData from Facebook properly and other times I don't. I feel like I might be doing something silly or overlooking something.

+1 I anecdotally see this with similar behavior.

@mixonic this also happened for me with LinkedIn. Signed in or signed out it happens.
Like I said in #133 with the timeout on the close of the popup it works.

It also happens with my own provider on Firefox / Mac OS X

I have the problem with facebook. google+ works fine. In Chrome it works most of the time. Safari and Firefox have problems. I used the setTimeout @rduque1 mentioned and it helps.

I see similar behavior. For me it's working fine in Chrome but failing in Firefox, every single time. Works fine with Safari.
I haven't tried with Facebook.

Just want to add my name to this list, not for Facebook but through Google.
Have been getting intermittent issues using Safari on Mac OS and reports of others having trouble too.
Adding a timeout suggested in #133 fixes it for me
redirect-handler.js
var winPopup = window;
setTimeout(function(){ winPopup.close(); }, 500);

Would you consider a PR for this? If so I'd be happy to create one.
Thanks

So perhaps what is happening is that the post message is being cancelled by the window closing. Perhaps these two lines are not running in order:
https://github.com/Vestorly/torii/blob/master/lib/torii/redirect-handler.js#L21-L22
If this is true, what I would like to see is the popup service send a message into the popup, and have that be the trigger for the close. This would be more deterministic than an arbitrary timeout.

Understood.
Looking at the https://github.com/Vestorly/torii/blob/master/lib/torii/services/popup.js file I must admit it looks a bit beyond my level of understanding — but i'd like to try to help out so will have a proper look this evening & see if I can work out how to do what you are asking.

@chrism you can find me on freenode as mixonic, in the #emberjs-dev room. I don't think it will be hard to implement the return post-message (or to test), happy to make sure you don't get blocked or waste any time!

@mixonic I've forked the repo and used that simple fix for now so i'm not blocked at all, i'd just like to help (if I can). i've taken a good look at the code & have run the tests, so far i've not found a way to make it work but i'm going to keep trying.
thanks for taking the time to offer your help — if i'm still no further forward in the next couple of days then please just let me know a good time for you & i'll jump on irc.

I'm also experiencing this in Firefox

This appears to be an issue in Mac OS X Chrome Canary as well. It does however work in the stable release of Chrome. Is there any new updates on this issue? Anyway I could help or check-in on the current progress.

The solution I proposed in this PR (#182) uses a timeout, but it's a timeout in the parent waiting for the message. In the normal flow (i.e., without any errors or failures), the popup will call postMessage() and then close itself. The parent will receive didClose set the timeout, but as soon as it receives the message, it will close the popup and cancel the timeout and so there's no apparent delay. Only in the case the message is never received will the timeout eventually trigger and reject the promise.
One question is whether the timeout should be configurable, either making it optional (i.e., wait forever if desired) or a different amount of time. It's currently hard-coded as 5 seconds. In retrospect, this seems very long, and perhaps should be made shorter.


GM route on kodadot

let's make it and we could have a dedicated route for a gallery that would filter GM from name/metadata or it could be actually trait GM and thus creators could get highlighted on this route

gm


Release funds for abandoned checkouts

I want to merge this change because it covers: https://github.com/saleor/saleor/issues/14768 and https://github.com/saleor/saleor/issues/14769

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations

Docs

[ ] Link to documentation:

Pull Request Checklist

[ ] Privileged queries and mutations are either absent or guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migrations are either absent or optimized for zero downtime
[ ] The changes are covered by test cases
[ ] All new fields/inputs/mutations have proper labels added (ADDED_IN_X, PREVIEW_FEATURE, etc.)
[ ] All migrations have proper dependencies
[ ] All indexes are added concurrently in migrations
[ ] All RunSql and RunPython migrations have revert option defined

All review comments have been applied on PR to main.


[FEATURE] switch accounts-- need to change to mainnet

Issue to be solved
when user switch accounts， even user is backup in testnet in old version，our app need to find mainnet address and switch to that account.
Suggest A Solution
No response

This issue already solved in Android prod version i2.1.0.
We add a pop up alert to lead user change to mainnet when they try to switch accounts.
The demo video is attached below.
https://github.com/Outblock/FRW-Android/assets/87978227/2334fa21-056a-4d97-b4d1-57119bdd789a


refactor: use zindex for checking if win is float

Relative may soon be delegated to only floats (returning nil for splits), if that's the route I/we decide to take with this PR in core. This PR preemptively addresses that change by checking zindex instead of relative to determine whether a window is floating. This doesn't change any behavior now, so even if that PR isn't merged there will be no side-affects from this, but this ensures we don't break any user configs if/when it is merged.

Can you please add a test? You can probably use screenshot testing for this. Example: https://github.com/echasnovski/mini.nvim/blob/main/tests/test_jump2d.lua#L433
If you set force, then a screenshot will be created, once you have one you can remove it, see: https://github.com/echasnovski/mini.test/blob/main/doc/mini-test.txt#L568

Will do - can't I just check whether the window is floating in the test though? I'm a bit confused on what screenshot would be used to check here

Well, it should not resize the float. So a screenshot will check the float is still intact.

Well, it should not resize the float. So a screenshot will check the float is still intact.

Gotcha, sounds good :)

win_get_width and win_get_height should be enough for this I think, not sure screenshot is needed.

Looks fine for me. Thanks!


Python 3 support, add the ability to run the server programatically.

Python3 support
Logging uses logging formatting
Socks5Server() class can be run programatically (my use case is a transient in-process proxy server).
Flag to force running in the foreground.

This fixes https://github.com/fengyouchao/pysocks/issues/2
It might break python 2 (I haven't tested), but who cares. If you're still on python 2, you should feel bad.

Clients sending unknown address types no longer cause an exception in the server, and I fiddled with some of the struct unpacking.
You seem to be using byte_to_int() in a bunch of places where you decode a byte as signed, and then truncate the upper bits to get an unsigned value. Why not just tell struct to unpack the values as unsigned? Or unpack them as their native sizes?
I switched some of the port number unpacking to just extract the values directly, though I'm uncomfortable changing EVERYTHING without some sort of testing facility to be certain I didn't overlook some subtle behaviour.


Initial Whitelist Support

This PR creates a new configuration variable, '$wgSlackIncludeNotificationsFrom' as an array, which serves as a converse (whitelist) of $wgSlackExcludeNotificationsFrom.  The two should be able to work together, with the Include creating a more global whitelist that the exclusion lists works on.
Simply not defining the variable should preserve previous behavior.

Sorry for the late reply! Thanks for the PR, I'll try to check this as soon as possible :)

Seems to be working well, thank you for the contribution! I'll add you to the list of contributors alongside the request and update the version.


Dom manipulation to sort.icon cards

according to #86 , CSS ordering is to be substituted with JS sorting for colors to allow tabbing to work.
This PR includes DOM sorting for all criteria, related refactorings and unit and e2e tests for this new approach.
DOM rearranging is slower than the previous solution, but that was expected and is the only solution for the tabbing problem (without updating tabIndex everywhere).

Would you mind to solve the conflicts @jorgeamadosoria?

Would you mind to solve the conflicts @jorgeamadosoria?

conflicts are solved.

New conflict in public/scripts/index.js 🔧

New conflict in public/scripts/index.js 🔧

this is done.

I can't answer on the previous thread.
I thought setTimeout was making more than one call. If it is like you said, then nevermind, this is fine.
I have a different issue:
This is also not going to be enough for the compact layout. Maybe instead of 15 the slice should be 30 or even 50 (compact can have up to 9 cards per line, so 3 lines would be 27 instead of 15).
Maybe that also takes care of most of the cases if people have done a little bit of scrolling, not too much.
More importantly: visually, there is not that big of a delay between ordering the whole list and ordering it by slices, so I don't know if the gain is enough to justify the complexity. This is just my opinion, however: if it is up to me, simpler is better than faster, unless the code is too slow for confort. I don't think the sorting is too slow for confort just yet.
In short, this is clever, but I don't think there is any need for it. If you still think we need it, I would recommend a larger slice to account for compact (30 cards should be fine), a comment along the lines of "Above the Fold item sorting" or something like that on the first call, and "rendering the rest of the sorted items" in the setTimeout. Something to mitigate the complexity.
Other than that, the code works just fine.

Seems that the scroll before sorting should be setted to 0 in axis Y, so the page would be seen at the start. Currently, the scroll behaves erratically when sorting because travels to some random point of the page.

I think that the async implementation is necessary because the blocking would be a great performance regression.
After reflecting on it a bit, seems that the scroll before sorting should be setted to 0 in axis Y, so the page would be seen at the start. Currently, the scroll behaves erratically when sorting because travels to some random point of the page, so it is an improvement anyways.
    $body.classList.add(selected);
    window.scrollTo(0, 0);
    domUtils.sortChildren(document.querySelector('ul.grid'), selected);

With this line and the asynchronous implementation seems to me that it should be fine, unless I'm missing something happening when sorting by relevance on search.

Scrolling to the beginning is definitely good, I agree.
I would make a domUtils function for setting the scrolling view value. Or you can leave it like that and I will abstract that later on.
If you are going to keep this, then merge it at will.

Some improvements in the tests and this will be ready 👍🏼


Zoom draw.io diagrams

Describe the feature you'd like
A way to zoom in diagrams or have them fill the whole screen when clicked, simialr to what other wiki software does (such as cloudformation).
Describe the benefits this would bring to existing BookStack users
At the moment its difficult to zoom in diagrams - we can open them in a separate tab and zoom on the browser but this zooms all open tabs of the browser. Having a way to have larger views of the image of the diagram when clicked would allow us to more easily read the diagram and thus make diagrams even more useful.
Can the goal of this request already be achieved via other means?
Not sure how it can be done unless zooming for the whole browser set of tabs which makes the rest of the fonts very large to view a larger diagram
Have you searched for an existing open/closed issue?

[X] I have searched for existing issues and none cover my fundamental request

How long have you been using BookStack?
1 to 5 years
Additional context
No response

Well drawio images are just images when rendered. So this could maybe be folded into a more general enhancement to viewing images.


Feature/fix issue 432

Fixes issue #432

Coverage decreased (-0.04%) to 99.958% when pulling 583c86ea26d8fe3cb0002aa4cead1bbd21bc3ccf on feature/fix-issue-432 into 988b97859e38ff8cd6658c332877a2af31cd94c3 on develop.


Required label is not displayed in expense details conversation while the receipt is scanned

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.30-0
Reproducible in staging?: Yes
Reproducible in production?: yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @Puneet-here
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1705704475627709
Action Performed:

Navigate to a collect workspace chat
Submit a receipt for smartscan
Navigate to the receipt details

Expected Result:
The required fields should display the "required" label even when the expense is smartscanning.
Actual Result:
Required fields don't show the "required" label.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01c21a61e56bc3b0d3
Upwork Job ID:<PHONE_NUMBER>449107968
Last Price Increase: 2024-01-24

Thanks for creating Isa! @cead22 do we need the C+ here if you're handling internally?

bump @cead22 please?

bump bump @cead22 ?

Expected Result:
The required fields should display the "required" label even when the expense is smartscanning.

One clarification is that when money requests are created, we no longer show the Required labels. In this case, if a category is required in the workspace, we should be showing the Missing category violation

The issue here is two fold, and I think we should probably discuss in slack how we want to handle them. @zanyrenney do you want to lead those discussions? cc @JmillsExpensify

We're not showing violations for money request created with a receipt while the receipt is still scanning
We're not showing violations for "partial transactions", ie, transactions without a merchant or amount set

ooh, I can bring the conversation to slack if that would be helpful but I have not been close to the design decisions on this wave 6 stuff so not super confident on the context behind why it was designed this way initially. cc @cead22

following the slack convo here though - https://expensify.slack.com/archives/C02MW39LT9N/p1706718905047439?thread_ts=1706717450.983399&cid=C02MW39LT9N

I posted a couple of threads on the wave 6 channel https://expensify.slack.com/archives/C02MW39LT9N/p1706728695835139

Thanks @cead22 I replied there, i proposed that if we want to do nothing for the violations for money requests created with a receipt while the receipt is still scanning, we can consider that done on this issue and just tackle the second issue of violations for partial transactions. Are you cool with that @cead22 ?

wow sorry for the double tag 😅


add a trait to register custom type and it's methods, getters, setters, etc.

What this PR do
Add a trait RhaiCustomType with a method to build the type i.e. register the type itself, it's methods, getters, setters, etc...
Like mlua's UserData trait.
Why
Make the code clearer :

Where the engine is created : only one line for each custom type engine.build_type::<MyType>()
The code to build the type is next the type itself

Notes

The concept is named "build" a type to avoid confusion with the concept of "register" a type.
The trait (RhaiCustomType) and it's method (build) could have a better name.

Example
use rhai::{Engine, RhaiCustomType};

#[derive(Debug, Clone, Eq, PartialEq)]
struct TestStruct {
    field: i64
}

impl TestStruct {
    fn new() -> Self {
        Self { field: 1 }
    }
    fn update(&mut self, offset: i64) {
        self.field += offset;
    }
}

impl RhaiCustomType for TestStruct {
    fn build(engine: &mut Engine) {
        engine
            .register_type::<Self>()
            .register_fn("new_ts", Self::new)
            .register_fn("update", Self::update);
    }
}

fn main() -> Result<(), Box<rhai::EvalAltResult>> {
    let mut engine = Engine::new();

    // Register API for the custom type.
    engine.build_type::<TestStruct>();

    assert_eq!(
        engine.eval::<TestStruct>("let x = new_ts(); x.update(41); x")?,
        TestStruct { field: 42 }
    );

    Ok(())
}

Also care to add a test?

I think the style in https://github.com/tguichaoua/rhai/blob/4432d5063a44c082d1f20c4c4889865b927ed7d9/tests/build_type.rs is best for the user, and it gives expansion possibilities later on without changing the API.


Remove Fresh Rebuilds section

What are you changing in this pull request and why?
The Fresh Rebuilds section doesn't seem to add any additional information as explained in https://github.com/dbt-labs/docs.getdbt.com/issues/5747
Checklist

[x] Review the Content style guide so my content adheres to these guidelines.
[x] For docs versioning, review how to version a whole page and version a block of content.
[x] Add a checklist item for anything that needs to happen before this PR is merged, such as "needs technical review" or "change base branch."

Hi @brunocostalopes Thank you for bringing this to our attention. I understand where you're coming from, but I think this section is specifically defining the freshness+ argument, which isn't mentioned in the section below (thought it is in the sample). Let's merge these sections, rather than delete one entirely.
I appreciate you catching this oversight.

Hi @matthewshaver, no worries, glad I could help even if only in a small way :)


PARAM SIZE MISMATCH

RuntimeError: Error(s) in loading state_dict for SiameseConvNet:
Unexpected key(s) in state_dict: "conv4.weight", "conv4.bias", "fc1.bias", "fc2.bias".
size mismatch for conv1.weight: copying a param with shape torch.Size([48, 1, 11, 11]) from checkpoint, the shape in current model is torch.Size([5, 1, 11, 11]).
size mismatch for conv1.bias: copying a param with shape torch.Size([48]) from checkpoint, the shape in current model is torch.Size([5]).
size mismatch for conv2.weight: copying a param with shape torch.Size([128, 48, 5, 5]) from checkpoint, the shape in current model is torch.Size([12, 5, 5, 5]).
size mismatch for conv2.bias: copying a param with shape torch.Size([128]) from checkpoint, the shape in current model is torch.Size([12]).
size mismatch for conv3.weight: copying a param with shape torch.Size([256, 128, 3, 3]) from checkpoint, the shape in current model is torch.Size([16, 12, 3, 3]).
size mismatch for conv3.bias: copying a param with shape torch.Size([256]) from checkpoint, the shape in current model is torch.Size([16]).
size mismatch for fc1.weight: copying a param with shape torch.Size([1024, 40800]) from checkpoint, the shape in current model is torch.Size([1024, 28560]).
Im getting this error in the torch.load call

Seems you have cloned the repo before the last commit and the script has downloaded the previous version of the model. Try deleting model file and running DownloadData.py again, that should download the current model. This doesn't seem to be a issue with the code, so I'm closing the issue. Please do re-open it if in case you still encounter the error.
Thanks.

You are right.. Thanks for the response
On Fri, 24 May 2019, 20:06 Aftaab Zia<EMAIL_ADDRESS>wrote:

Closed #3
https://github.com/Aftaab99/OfflineSignatureVerification/issues/3.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Aftaab99/OfflineSignatureVerification/issues/3?email_source=notifications&email_token=AEDMCN5HK2AFKSS5DDOMCJTPXAOAXA5CNFSM4HPBNIZ2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGORUELNMA#event-2366158512,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEDMCN4LCX45YCX3M3L3Z5DPXAOAXANCNFSM4HPBNIZQ
.


Per-status-page robots.txt

📑 I have found these related issues/pull requests
I couldn't find any related issues.
🏷 Feature Request Type
Status-page
🔖 Feature description
"Search engine visibility" should be toggleable for different status pages.
✔ Solution
I have some status pages that should be public, and I have some that I don't want to be public. Being able to set only some status pages as hidden (or set them exempt from the instance-wide policy) would be really useful for that.
❓ Alternatives
No response
📝 Additional Context
No response

Non public status pages is not a feature that we currently offer.
If we would, those would not be included in that report.
I see no reason to make this into another setting.
Lets track this together with private status pages in the following issues:

(parent) https://github.com/louislam/uptime-kuma/issues/1050
(related, not duplicate) https://github.com/louislam/uptime-kuma/issues/938

=> closing as a duplicate


How get_document_analysis_status is used?

The status used on the Step Funtions flow is set by start_process_scanned_invoice_workflow.
The get_document_analysis_status is not referenced in the template.yaml, I think that not is used.

get_document_analysis_status is no longer being used. Thanks for pointing this out.


.NET Core 1.0 Support

I did like to know if either or both of these libraries have been ported to the new .NET Core 1.0 Framework.

We support netstandard 1.3 and thus .NET core.

how about .net core 2 ? anyone know doc we can use elastic in .netcore 2

1.3 should be forward compatible with 2.0 as 2.0 is a superset of 2.0.


postmessage events causing high cpu usage

I have an app that communicates with an iframe via postmessage events. My event listener is getting flooded with "AUGURY_INSPECTED_APPLICATION" messages. CPU at 100% due to this.

@bmayen we will look into this. Would you be able to provide us with an isolated repo/site where this is reproducible?

@bmayen Is this issue still occurring?

Sorry, was on a tight schedule and had to simply disable Augury for that project. Am no longer active on that. If the issue comes up again, I will update.

Okay, for now I will close to issue as it has not been reported any other time. If it comes up again we'll investigate further.

I'm getting this whenever I open developer tools on any page, even if it is not using Angular.

steps to reproduce:

create an angular app
add this HostListener to app.component.ts

@HostListener('window:message', ['$event'])
onFrameMessage(event: MessageEvent) {
  console.log({event});
}

open augury from dev tools
enjoy 250% cpu

stackblitz example: https://stackblitz.com/edit/angular-fjstb9
augury version used: 1.23.0

I can still experience the same issue in 2019.
The steps to reproduce are exactly the ones described by tal-abziz

@hmicn Which version of Augury you are on? We just release new version yesterday. Can you check on that. Thanks

Hi @sumitarora thanks for the reply. I currently use Augury 1.24.0. I'll check on the new version and I'll get back to you

Still I experience the same issue. Using postMessage. I'm listening to the message and send a network request to the API. It worked perfectly on localhost. Then I run the prod build on GCP compute instance, it start to run infinite amount of time.
when the callback of the listener execute for the first time, it receive the message that I sent from postMessage, after that it just receive an object with a property,
{messageSource : "AUGURY_INSPECTED_APPLICATION, ...'}. This triggers the callback and sending infinite amount of network request with that object.
When I disable the Augury, it works without any problem.


Make the library Multiplatform and upgrade config

I would like to use this library in multiple Kotlin Multiplatform projects of mine, but only JVM is supported. I found a crypto library, which implements SecureRandom, as well as multiple stable random libraries across other platforms to be used in common code.
The modification also comes with configuration change to using Kotlin Gradle DSL.
I already forked this repository and made my changes (not finalized yet).

Just so we don't duplicate work, @nik-sta should I attempt to finish up my changes and make a PR, or is it your task to find a proper way to make the library multiplatform? Either way, I am happy to assist and make changes! :)

Hi, @HLCaptain; thanks for taking the time to discuss your changes with us. Generally speaking, we like your proposal and code changes. The one detail we need to work on (or make up our minds) is if there is an alternative for the crypto library. The proposed one is unacceptable and is insufficient to accept using it.
You are more than welcome to contribute to this repo; therefore, please make your pull request, and we can work together on it :)

We would highly welcome the change to make this library multiplatform capable. Let's find a way together to make this possible.

Has any progress been made on this? I see a draft PR but no progress in months


Cherry-pick PR #398 to release-v1.3

Cherry-Pick Details

Original PR Title: Switch Nutanix Client to use Session Auth instead of Basic Auth
Original PR URL: https://github.com/nutanix-cloud-native/cluster-api-provider-nutanix/pull/398
Original Commit SHA: 03c61d34afe159fe91940724e44f821dfeed1d1c
Cherry-Pick Reason: Cherry-picking changes related to session auth to release-v1.3 so when a user upgrades from 1.2.5 to 1.3.4, they don't fall back to basic auth

/test e2e-k8s-conformance

/test e2e-k8s-upgrade


Move dependabot.yml into .github directory

@aligator created the dependabot.yml file in the wrong directory. For me, this is the proof that he actually manages to get even the simplest tasks wrong! 😄 🥳

This PR is included in release v0.6.0.


perf: Remove not needed complexity of local cache hash

1. Why is this change necessary?
Performance.
2. What does this change do, exactly?
Remove unneeded cache hash creation, which consists of two function calls, json_encode and md5.
3. Describe each step to reproduce the issue or behaviour.
-
4. Please link to the relevant issues (if any).
-
5. Checklist

[x] I have rebased my changes to remove merge conflicts
[ ] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

🤖 Generated by Copilot at 2f586dd
This pull request improves the performance and readability of the CurrencyFormatter class by simplifying its cache logic and removing unnecessary arguments. It also adds a changelog entry for the change in changelog/_unreleased/2023-07-02-remove-not-needed-cache-hash-creation-for-cache-in-the-currencyformatter.md.

🤖 Generated by Copilot at 2f586dd

Simplify currency formatting by using locale and default format (link, link)
Remove unnecessary hash creation and caching logic for formatter (link)

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-28971
Please use this issue to track the state of your pull request.

Thank you, @aragon999, for your contribution! 💙

Thank you @leichteckig for writing the tests <3


PostmarkException: Received valid, but incompatible JSON input

I'm getting an exception while sending email using Postmark PHP API.
Here are the logs for the same :
Postmark\Models\PostmarkException: Received valid, but incompatible JSON input. in /app/vendor/wildbit/postmark-php/src/Postmark/PostmarkClientBase.php:165
Stack trace:
#0 /app/vendor/wildbit/postmark-Postmark\PostmarkClientBase->processRestRequest('POST', '/email', Array)

Hi.
Normally, this means that there's some sort of issue with the data provided
to our API (we defer practically all of the validation to the API to avoid
problems when rolling out validation updates).
If you can contact support (http://support.postmarkapp.com/), with a
snippet of the code you're using to send, we can try to help identify what
the specific issue is.
//Andrew
On Thu, Feb 16, 2017 at 2:46 PM, Nitesh Sisodia<EMAIL_ADDRESS>wrote:

I'm getting an exception while sending email using Postmark PHP API.
Here are the logs for the same :
Postmark\Models\PostmarkException: Received valid, but incompatible JSON
input. in /app/vendor/wildbit/postmark-php/src/Postmark/
PostmarkClientBase.php:165
Stack trace:
#0 /app/vendor/wildbit/postmark-Postmark\PostmarkClientBase->processRestRequest('POST',
'/email', Array)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/wildbit/postmark-php/issues/33, or mute the thread
https://github.com/notifications/unsubscribe-auth/AACKASljWVz72VsHC-pcnBd1YZ-EFv5Dks5rdKeygaJpZM4MDdg8
.

Hi,
I am sure this has been solved for @niteshsisodia but if someone lands here from search results, thought to mention that I got this when not using the right order of the parameters, basically I skipped some of them by mistake and resulted in the same error.
Regards,
Cata

My error was I used an array for multiple bcc addresses, but Postmark needs a comma separated list for that. Bingo :)
Thanks

The issue I had was the the TemplateId a number, defined at the top right corner of the templates - not the name the template.

I just encountered this error by passing the wrong datatypes to PostmarkClient::sendEmail() (empty arrays instead of NULL, for example). My code was "working" on my end, but I was passing in wrong values.


P4HeaderStack messages and the size of the P4 program's corresponding header stack variable

It seems reasonable that everything is fine if a P4HeaderStack message has the same number of headers in it as the maximum number of headers declared in the P4 program -- the first header in the message should correspond to element 0, the next element 1, etc.
If the message has more than the P4 program's maximum size for the header stack, is that an error?
If the message has less than the P4 program's maximum size for the header stack, is that an error, or should the message's header stack value be appended with enough invalid headers at the end of the header stack (i.e. the largest index values)?
The P4Info file seems to have no explicit representation of header stacks, and no way for the control plane to know what the maximum allowed size of a particular header stack is.
Same issues/questions exist for stacks with elements that are header_union.

I personally consider this issue fairly low priority, as the only way it is relevant in P4Runtime v1.0 is if someone has a P4 program where a Register extern has a header stack as one of its elements stored in each register array element, which seems like an unusual use case to me.

Actually the P4Info does have the max size information I think (https://github.com/p4lang/p4runtime/blob/master/proto/p4/config/v1/p4types.proto#L147). But it may be a good idea to specify what is expected from the P4Runtime client. I am tempting to say that the client should provide a "value" for each element of the stack, but we can also support providing fewer values, because headers & header unions have the notion of validity.

Huh.  Not sure how I missed the max size information, but yes it is there.
If we want to preserve read-write symmetry here, one way would be to require that all N headers are specified in the message explicitly.  The other would be that all of the "last" invalid headers should never be present in the message (in the read-write symmetry form, at least).
Another detail with read-write symmetry and P4HeaderUnion is probably that it should be required that invalid header_union values must have valid_header_name equal to the empty string and valid_header unset.


esp_wifi_80211_tx sends RTS, not arbitrary frames (IDFGH-9284)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
v4.4.2
Operating System used.
Windows
How did you build your project?
Other (please specify in More Information)
If you are using Windows, please specify command line type.
None
Development Kit.
ESP32-CAM
Power Supply used.
USB
What is the expected behavior?
Broadcast message 08 02 00 00 DE AD BE EF 42 72 DE AD BE EF 42 72 DE AD BE EF 42 72 00 00 08 06 07 05 03 00 09  + CRC32
(Data packet from AP [From AP mode tests] with sequence 0x00 0x00 [Appears to be overwritten; acceptable] and three DE:AD:BE:EF:42:72 dummy MACs, followed by test payload [8675309] - CRC32 can be created in program or hardware)
What is the actual behavior?
Broadcasts a burst (~24) of frames with payload:  B4 00 56 04 (3x dummy MAC) + CRC32
(Request to send)
Steps to reproduce.

Set up WiFi
Send packet with esp_wifi_80211_tx
Packet sniffed with promiscuous mode adapter using Wireshark.

Debug Logs.
(Typical)
Building Frame
8 2 0 0 DE AD BE EF 42 72 DE AD BE EF 42 72 DE AD BE EF 42 72 0 0 8 6 7 5 3 0 9
63 44 E0 7E
8 2 0 0 DE AD BE EF 42 72 DE AD BE EF 42 72 DE AD BE EF 42 72 0 0 8 6 7 5 3 0 9 63 44 E0 7E

(Lowest two lines confirm arrays contain information as desired)

More Information.
Day 4 of attempting to broadcast packets for unidirectional data flow. Issues persisted in previous versions, updated to v4.4.2 via Arduino board manager (via 2.0.5, as 2.0.6 has a 404 and cannot install through the board manager). System functions as intended except esp_wifi_80211_tx does not transmit packet. Function broadcasts multiple RTS frames instead of the frame desired. As it is not connected to an access point (or is an access point) these fail and the desired frame is never sent.
Please advise if backtrace or linked debugger is required, as well as how to perform these actions with the Arduino IDE.
Code follows: (note: may contain remnants of camera control code)
#include "esp_camera.h"

#include "freertos/FreeRTOS.h"

#include "esp_event.h"
#include "esp_system.h"
#include "esp_event.h"
#include "esp_wifi.h"

#include "freertos/task.h"
#include "freertos/event_groups.h"
#include "esp_log.h"
#include "nvs_flash.h"

#define CAMERA_MODEL_AI_THINKER

#include "camera_pins.h"
#include "esp_system.h"

esp_err_t esp_wifi_80211_tx(wifi_interface_t ifx, const void *buffer, int len, bool en_sys_seq);

uint8_t BadMAC[] = {
  0xde, 0xad, 0xbe, 0xef, 0x42
};

uint8_t DataFrameHeader[] ={
  0x08, 0x02, 0x00, 0x00,
  0xde, 0xad, 0xbe, 0xef, 0x42, 0x72,
  0xde, 0xad, 0xbe, 0xef, 0x42, 0x72,
  0xde, 0xad, 0xbe, 0xef, 0x42, 0x72
};

uint8_t Sequence[] = {
  0x00, 0x00
};

uint8_t Payload[1024];

uint8_t FCS[4];

uint8_t TestInfo[] = {
  0xc4, 0x00, 0xe8, 0x07, 0x9c,
  0x8e, 0xcd, 0x25, 0xa3, 0x28
};

size_t TestLength = sizeof(TestInfo);

uint32_t crc32_formula_reflect( size_t len, const void *data, const uint32_t POLY = 0xEDB88320 )
    {
        const unsigned char *buffer = (const unsigned char*) data;
        uint32_t crc = -1;

        while( len-- )
        {
            crc = crc ^ *buffer++;
            for( int bit = 0; bit < 8; bit++ )
            {
                if( crc & 1 ) crc = (crc >> 1) ^ POLY;
                else          crc = (crc >> 1);
            }
        }
        return ~crc;
    }

void BuildFrame(){
  Serial.println("Building Frame");
  uint8_t LoadPay[] = {0x08, 0x06, 0x07, 0x05, 0x03, 0x00, 0x09};
  int TempLen = sizeof(DataFrameHeader) + sizeof(Sequence) + sizeof(LoadPay);
  uint8_t Temp[TempLen];
  memcpy( &Temp[0], &DataFrameHeader[0], 22 * sizeof( uint8_t ) );
  memcpy( &Temp[22], &Sequence[0], 2 * sizeof( uint8_t ) );
  memcpy( &Temp[24], &LoadPay[0], sizeof(LoadPay) );
  for(int i = 0; i<(sizeof(Temp)/sizeof(Temp[0]));i++){
    Serial.print(Temp[i],HEX);
    Serial.print(" ");
  }
  Serial.println("");
  uint32_t crcW = crc32_formula_reflect(TempLen,Temp);
  byte* crcB = (byte*)&crcW;
    for(int i = 0; i<4;i++){
    Serial.print(crcB[i],HEX);
    Serial.print(" ");
  }
    Serial.println("");

  uint8_t Temp2[TempLen+4];
  memcpy( &Temp2[0], &Temp[0], TempLen);
  memcpy( &Temp2[TempLen],&crcB[0],4);

  for(int i = 0; i<TempLen+4;i++){
    Serial.print(Temp2[i],HEX);
    Serial.print(" ");
  }
    Serial.println("");
    if (ESP_OK != esp_wifi_80211_tx(WIFI_IF_STA, Temp, TempLen, false)){
    ESP_LOGE("raw tx", "tx failed");;
  }

}

void setup() {
  Serial.begin(115200);
  Serial.setDebugOutput(true);
  Serial.println();

  wifi_init_config_t cfg = WIFI_INIT_CONFIG_DEFAULT();
  ESP_ERROR_CHECK(esp_wifi_init(&cfg));
  ESP_ERROR_CHECK(esp_wifi_set_storage(WIFI_STORAGE_RAM));
  ESP_ERROR_CHECK(esp_netif_init());
  ESP_ERROR_CHECK(esp_event_loop_create_default());
  esp_netif_create_default_wifi_sta();
  ESP_ERROR_CHECK(esp_wifi_set_mode(WIFI_MODE_STA));

  wifi_config_t wifi_config = { };
  wifi_config.sta.channel = 1;

  ESP_ERROR_CHECK(esp_wifi_set_config(WIFI_IF_STA, &wifi_config));
  ESP_ERROR_CHECK(esp_wifi_start());
  ESP_ERROR_CHECK(esp_wifi_set_ps(WIFI_PS_NONE));

  Serial.print("IDF Version: ");
  Serial.println(esp_get_idf_version());
  BuildFrame();

}

void loop() {
  delay(10000);
  BuildFrame();
}

I would like to add that I have observed this issue, with a few extra points.

I am sending a null data frame (0x48) in order to keep the message as short as possible.
Sometimes the arbitrary frame goes through exactly as desired, other times, it tries to use RTS/CTS first.
My device never receives CTS, even from devices which are correctly sending CTS to others (such as my AP)
The RTS seems to spam out pretty extreme.  A single call to esp_wifi_80211_tx() can result in a dozen RTS calls.

As a user, this is the behavior I would want-

if esp_wifi_80211_tx() fails to actually send due to no CTS, I would like a callback function to inform me of this failure once the device gives up.
I would like to be able to tell it not to use RTS/CTS, with a specified window of time to look for an opening to send without it.  Something like "listen for 50 milliseconds and attempt to send", again, I would like a callback or even potentially a blocking function.
I would expect a single call to esp_wifi_80211_tx() to result in a configurable number of RTS retries (down to 1).

In general, I would like a little more control over either what the esp_wifi_80211_tx() does regarding RTS/CTS, or callback functions that can deliver the results of the attempted transmission.  I will submit the callback as a feature request.

For context: I did some research: this is a deliberate. Espressif removed this capability with a version change, I forget which one. You can no longer send arbitrary frames without rolling back to an older version of IDF.

For context: I did some research: this is a deliberate. Espressif removed this capability with a version change, I forget which one. You can no longer send arbitrary frames without rolling back to an older version of IDF.

What version was this changed in?  I'll downgrade no problem.

@Syphontwo @faragher we will consider the  feature request.

This feature request and https://github.com/espressif/esp-idf/issues/13319 are repeated. We will close this ticket first and please move to https://github.com/espressif/esp-idf/issues/13319 for more updates, thanks.


chore: Add comments on calls to kubeClient Update()

Fixes #N/A
Description
Add some additional comments on locations that we call kubeClient.Update()
How was this change tested?
make presubmit
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

6 of 6 (100.0%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.01%) to 78.99%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/node/termination/terminator/eviction.go
2
88.89%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
8271

Relevant Lines:
10471

💛  - Coveralls


Improvements to responder chain management

This PR aims to improve the responder chain management of PARViewController so that we (hopefully) avoid hitting the remaining hard-to-diagnose class of crashes due to the garbage data being present in the responder chain during deallocation of views, as described here: https://github.com/mekentosj/papers-mac/issues/3994
Myself and @defragged worked on this, and are confident that it is working as intended and the responder chain appears intact during adding and removing objects from it, but of course this should have a decent level of scrutiny pointed at it before merging.

@cparnot could you take a look at this?

While the code is straightforward, I don't understand what kind of problems observing view.window helps avoiding? Are you catching cases where the responder chain needs to be patched that are not detected by KVO on nextResponder?

@mz2 I just remembered about this PR - what shall we do RE: mac beta?

@danielrbrowne I would indeed like it to go in. @defragged could you respond to @cparnot's feedback please?

We went for view.window as that's what you suggested in your pseudocode here @mz2: https://github.com/mekentosj/papers-mac/issues/3994
If removing a view from its parent view (and by extension, window) triggers nextResponder to change, then it probably makes more sense to observe that instead. Although I'm not hugely familiar with responder chain stuff, so I'd have to experiment to make sure.

If removing a view from its parent view (and by extension, window) triggers nextResponder to change, then it probably makes more sense to observe that instead. Although I'm not hugely familiar with responder chain stuff, so I'd have to experiment to make sure.

It definitely triggers the nextResponder change, but that's why there is an observer for nextResponder in the first place :-)
It probably does not hurt to add an observer on view.window, it just adds much more code. Also the observing is done differently, in that for the nextResponder observing I use the helper class PARObjectObserver, which takes care of properly removing the observer. I don't remember if I checked that KVO on nextResponder actually is triggered when the view is removed from its superview.
What I see as problematic in the current (public) PARViewController implementation  is that the patchResponderChain method does not remove the view controller from the previous responder chain when the nextResponder of the view is set to nil.
In the end, what I am still trying to understand is a specific scenario that leads to a crash, even without the above fix or yours. As soon as the view controller is deallocated, even if that happens before the view is gone, the KVO is disabled by setting the observer object to nil, so any subsequent changes in the view's nextResponder won't trigger any call into the deallocated view controller.

I don't believe the issues we're seeing are down to observers hanging around. Rather view controllers that are being patched into the responder chain getting deallocated, not removed from the chain, then are later messaged.
That's the hypothesis anyway.

I don't believe the issues we're seeing are down to observers hanging around. Rather view controllers that are being patched into the responder chain getting deallocated, not removed from the chain, then are later messaged.

Ah, yes, that makes sense. In this case, what we need to do is remove the view controller from the responder chain upon deallocation. The tricky bit in dealloc is wether or not the view has been deallocated already. Calling self.view can have side effects and create the view, but you already have code that keeps track of that. Why not add in dealloc the following (pseudo-code)
 if (self.par_isViewLoaded))
{
    self.view.nextResponder = nil;
}

Alternatively, we could set the nextResponder back to what it was before patching (self.nextResponder) but that begs the question: is this next responder object sill allocated?! (the view is strongly referenced, I supposed, but next responder, I don't know? Probably unsafe_unretained behinf the scene; weak if we are lucky; though if we assume it crashes, then it's not a safe weak).
In the end, breaking the responder chain when the view controller is deallocated is probably OK, since it probably means the views are being dismantled. And by probably, I mean "very likely" ;-)

Ah, yes, that makes sense. In this case, what we need to do is remove the view controller from the responder chain upon deallocation.

Presumably in addition to when the view is removed from the responder chain? Is there a scenario where the view controller would be deallocated before the view it's managing is removed from the responder chain?

Oh, I think I remember why we might have done view.window. The unpatching requires knowledge of the nextResponder, and (unless I'm missing something in PARObjectObserver) if we watch for nextResponder using PARObjectObserver, we don't have any way of getting at the previous value of nextResponder (as it's already nil).

Presumably in addition to when the view is removed from the responder chain?

Yes.

Is there a scenario where the view controller would be deallocated before the view it's managing is removed from the responder chain?

I don't know of a concrete example, but if that happens, it leaves open the possibility that the view controller is messaged because of its next responder status, as you suggested. It's possible that nextResponder is using a zeroing weak reference in NSResponder, in which case that would not be an issue but I strongly suspect that's not the case.

Oh, I think I remember why we might have done view.window. The unpatching requires knowledge of the nextResponder, and (unless I'm missing something in PARObjectObserver) if we watch for nextResponder using PARObjectObserver, we don't have any way of getting at the previous value of nextResponder (as it's already nil).

Well, actually, the previous value of nextResponder of the view is stored in self.nextResponder :D
I think what would be very useful, actually, is to define a weak  NSResponder *parview_nextResponder property that we can use to store the "old" nextResponder, which will be safer to use than self.nextResponder. Then, we can easily "unpatch" in dealloc or other places where it's needed.

I think what would be very useful, actually, is to define a weak NSResponder *parview_nextResponder property that we can use to store the "old" nextResponder, which will be safer to use than self.nextResponder. Then, we can easily "unpatch" in dealloc or other places where it's needed.

I was thinking something along these lines. Sadly, being as we're in the world of 10.7 and 10.8 on Papers (where NSViewControllers don't play well with weak properties), I'm not sure if I'd be comfortable doing this (as it's another thing we have to remember to nil out when we're done with it).
We could use bog standard KVO to observe the nextResponder property, and register to be notified for the previous and new values of nextResponder (in the same way as, or instead of, how we watch the view.window property), but I worry that I'd then be losing some property of PARObjectObserver that I'm overlooking.

We could use bog standard KVO to observe the nextResponder property, and register to be notified for the previous and new values of nextResponder (in the same way as, or instead of, how we watch the view.window property), but I worry that I'd then be losing some property of PARObjectObserver that I'm overlooking.

That would be one way, but  if we can't get a zeroing weak reference to nextResponder, we might as well simply use self.nextResponder because we maintain the responder chain as follows whenever the property is changed:
view --> old_responder
view --> view controller --> old_responder

And thus we always have access to the old nextResponder.
The issue is no matter how we do it, we get an unsafe unretained reference, with no way to know if it's still valid. In practice, this is probably safe: the superview or else the window is typically the next responder, and it's an owner of the view.
Now that I think of it, NSWindow cannot be used with zeroing weak reference, so even there, we'd have to be careful what we store in parview_nextResponder.
OK, here is another option. In unpatch, we try to restore the nextResponder of the view using self.nextResponder but only if self.nextResponder is the view superview or window. The should pretty much cover 99.99% of "real-world" responder chains in AppKit.

OK, here is another option. In unpatch, we try to restore the nextResponder of the view using self.nextResponder but only if self.nextResponder is the view superview or window. The should pretty much cover 99.99% of "real-world" responder chains in AppKit.

And that's safe because we know the superview or window returned by the class will always be a valid object. Right?

@defragged bump


OAuth, OAuth2 multiple definitions in Android projects

If you add RestSharp.Portable.OAuth2 as NuGet package to your Xamarin Android app, you get
"The type 'RestSharp.Portable.Authenticators.Resource' exists in both..." error messages because ApplicationName and Hello string resources defined in both RestSharp.Portable.OAuth and RestSharp.Portable.OAuth2 assembly.

Please try again with 2.4.1. The problem should be fixed now.

It's working fine now.


Strange IP address appearing in flynn cluster add

I'm trying to run (I've tried to remove IDs and my real domain name in the below in case they are sensitive):
flynn cluster add -p {SOME_LONG_ID}+{SOME_SHORT_ID}= default flynn.example.com {SOME_LONG_ID}

The output I get is:
Error writing CA certificate: Get https://controller.flynn.example.com/ca-cert: dial tcp: lookup controller.flynn.example.com on <IP_ADDRESS>:53: no such host

The funny thing is I don't know where it's getting <IP_ADDRESS> from? The servers have internal IPs of <IP_ADDRESS>, <IP_ADDRESS> and <IP_ADDRESS> and public floating IPs that are NATed to them by OpenStack.
My local machine is on a 192.168.x.x network, so it may be that, but if I dig from my local machine:
dig -tany controller.flynn.example.com @<IP_ADDRESS>
;; QUESTION SECTION:
;controller.flynn.example.com.	IN	ANY

;; ANSWER SECTION:
controller.flynn.example.com. 600	IN	CNAME	flynn.example.com.

Can anyone shed some light on why this is going wrong and how to fix it?

<IP_ADDRESS> is the DNS server that the Flynn CLI is trying to do a DNS lookup against. What is the output of dig -tany flynn.example.com @<IP_ADDRESS>?

$ dig -tany flynn.example.com.com @<IP_ADDRESS>

;; QUESTION SECTION:
;flynn.example.com.			IN	ANY

;; ANSWER SECTION:
flynn.example.com.		510	IN	A	<IP_ADDRESS>
flynn.example.com.		510	IN	A	<IP_ADDRESS>
flynn.example.com.		510	IN	A	<IP_ADDRESS>

(obviously the 1.2.3 is replaced with the real public IP address part, but again, I don't know what needs to be kept secret here).

When I try the flynn cluster add again, I now get:
Error writing CA certificate: Get https://controller.flynn.civo.com/ca-cert: dial tcp <IP_ADDRESS>:443: i/o timeout

Sounds like you have a firewall or routing issue, can you curl that URL or connect to the IP on port 443?


TypeError: Cannot read property 'initialize' of undefined

Issue summary
Hi shopify-node-api team,
Thanks for making this node API.
I am trying to build an embedded app with Vue/Koa.js, I just start by following the tutorial of @shopify/koa-shopify-auth. However, when I tried to run the example app via command node server.js,
it return an error said:
Shopify.Context.initialize({
                ^
TypeError: Cannot read property 'initialize' of undefined

It confused me, so I came here following the getting start tutorial of @shopify/shopify-node-api, and carefully checked every input arguments(API_KEY, API_SECRET_KEY, SCOPES, HOST, all these value are correct).
Unfortunately, I got just the same error.
Expected behavior
I expected the example app can successfully executed, or at least the function Shopify.Context.initialize could work.
Actual behavior
when I tried to run the example app via command node server.js,
it return an error said:
Shopify.Context.initialize({
                ^
TypeError: Cannot read property 'initialize' of undefined

Even I checked the tutorial at here, it still end with same the error.
Steps to reproduce the problem

Setup the files
type node server.js

Reduced test case
Following is my code(sever.js and package.json):
server.js
import 'isomorphic-fetch';
import dotenv from 'dotenv';
import Koa from 'koa';
import session from 'koa-session';
import shopifyAuth, {default as verifyRequest} from '@shopify/koa-shopify-auth';
import Shopify, {ApiVersion} from '@shopify/shopify-api';

dotenv.config();
const port = parseInt(process.env.port, 10) || 3000;
const {SHOPIFY_API_SECRET, SHOPIFY_API_KEY} = process.env;

//initialize the library
Shopify.Context.initialize({
    API_KEY: SHOPIFY_API_KEY,
    API_SECRET_KEY: SHOPIFY_API_SECRET,
    SCOPES: [process.env.SHOPIFY_APP_SCOPES],
    HOST_NAME: process.env.SHOPIFY_APP_URL.replace(/^https:\/\//, ''),
    IS_EMBEDDED_APP: true,
    API_VERSION: ApiVersion.October20,
    SESSION_STORAGE: new Shopify.Session.MemorySessionStorage()
  });

const server = new Koa();
server.keys = [SHOPIFY_API_SECRET];
server.use(session(server))

.use(
    shopifyAuth({
        afterAuth(ctx){
            const {shop, accessToken} = ctx.session;
            console.log('First Trial Success!', accessToken);
            ctx.redirect('/');
        },
    }),
)

.use(verifyRequest())

.use(ctx => {
    ctx.body = '😎';
});

server.listen(port, ()=>{
    console.log('> Ready on http://localhost:${port}');
});

package.json
{
  "name": "koala",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "type": "module",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "author": "Dan Hsu",
  "license": "ISC",
  "dependencies": {
    "@shopify/app-bridge-utils": "^1.29.0",
    "@shopify/koa-shopify-auth": "^4.0.2",
    "@shopify/shopify-api": "^1.1.0",
    "@types/koa": "^2.13.1",
    "dotenv": "^8.2.0",
    "isomorphic-fetch": "^3.0.0",
    "koa": "^2.13.1",
    "koa-router": "^8.0.8",
    "koa-session": "^6.1.0"
  }
}

Checklist

[x] I have described this issue in a way that is actionable (if possible)

Hey, I had the same issue, fixed it by installing babel. My exact steps were:

Installing @babel/core, @babel/preset-env and @babel/register
Adding an index.js file like so:

require("@babel/register");

// Import the rest of our application.
module.exports = require("./app.js");

Adding the following babel config file:

module.exports = {
  presets: [
    [
      "@babel/preset-env",
      {
        targets: {
          esmodules: true,
        },
      },
    ],
  ],
  ignore: ["node_modules"],
};

So when you run the index.js file, the issue should be resolved. Hope that helps!

Hi @NoahHowadt, Thank you for your idea. It works!
Just have a little question, why the index.js file need to write in es5?
require("@babel/register");

// Import the rest of our application.
module.exports = require("./app.js");

Hey @danhsucowboy, I'm glad it worked for you!
As to why the index.js file is in es5, I'm sorry, I don't know. I just took the code from the shopify-app-node github repo. Tried to convert it to es6, but it didn't seem to work so I just left if as it is.

Got it, thank you very much!
I'll close this issue.

That worked thanks. Their doc on Github are wrong then
https://github.com/Shopify/shopify-node-api/blob/main/docs/getting_started.md#set-up-environment


feat: add cookie authentication

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (fc7796f) 81.19% compared to head (3e9bb3b) 81.53%.
Report is 2 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #5      +/-   ##
==========================================
+ Coverage   81.19%   81.53%   +0.34%
==========================================
  Files           7        7
  Lines         117      130      +13
==========================================
+ Hits           95      106      +11
- Misses         22       24       +2

Files
Coverage Δ

drf_anonymous_login/authentication.py
85.18% <100.00%> (+0.56%)
:arrow_up:

drf_anonymous_login/views.py
77.77% <0.00%> (-6.23%)
:arrow_down:

... and 1 file with indirect coverage changes

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

The branch needs to be rebased. Also make sure to add this feature to the CHANGELOG.md


Tiny issues w.r.t data sampling in few-shot iMAML example

I was looking through the iMAML few-shot example , and noticed some tiny mistakes:

the phase is not set correctly in the sinusoid generation (atm it's more like a frequency): y_train = jnp.sin(phase * x_train) * amplitude instead of y_train = jnp.sin(x_train - phase) * amplitude
only a fixed number of pre-generated tasks is used to train the meta-learner, rather than tasks being sampled on-the-fly like in the original implementation (see here and here).

It turns out that I was also working on reimplementing MAML (not implicit) but with jaxopt for the unrolled gradient and my results on the sinusoid dataset match those of the paper. I'd be happy to implement the tiny changes, but also add the MAML version. I saw you implemented the iMAML example, wdyt @fabianp ?

thanks @zaccharieramzi ! Your fixes would be most welcomed. I was actually concerned on why the example was so slow and not getting great results. This settles it then!
Can you send a pull request for the changes? Thanks again!

Yes I think on the slowness the problem is the for loop in the loss computation.
I also had implemented it via a for loop, but a vmap is much more efficient, it just means that the parameters should be duplicated at the leaf level (if you consider them a pytree) and not at the tree level.
I have some pressing things on my plate atm, in like 2 weeks I can definitely send a PR.

Sounds great, thanks!
On Thu, Mar 30, 2023, 15:55 Zaccharie Ramzi @.***>
wrote:

Yes I think on the slowness the problem is the for loop in the loss
computation.
I also had implemented it via a for loop, but a vmap is much more
efficient, it just means that the parameters should be duplicated at the
leaf level (if you consider them a pytree) and not at the tree level.
I have some pressing things on my plate atm, in like 2 weeks I can
definitely send a PR.
—
Reply to this email directly, view it on GitHub
https://github.com/google/jaxopt/issues/417#issuecomment-1490348912, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACDZB26ZIP3F3MXOU37CXLW6WGELANCNFSM6AAAAAAWK4JB4E
.
You are receiving this because you were mentioned.Message ID:
@.***>


Linux Mint builld problem

By default cloned sources builds with some errors:
$ make cc -I/usr/include/libusb-1.0   -Wall    cynpro.c  -lusb-1.0   -o cynpro cynpro.c: In function ‘led_color_rgb’: cynpro.c:57:3: error: ‘for’ loop initial declarations are only allowed in C99 mode    for (size_t i = 2; i < 88; i++)    ^ cynpro.c:57:3: note: use option -std=c99 or -std=gnu99 to compile your code cynpro.c: In function ‘led_intensity’: cynpro.c:100:3: error: ‘for’ loop initial declarations are only allowed in C99 mode    for (size_t i = 2; i < 88; i++)    ^ cynpro.c: In function ‘led_switch’: cynpro.c:126:3: error: ‘for’ loop initial declarations are only allowed in C99 mode    for (size_t i = 2; i < 88; i++)    ^ cynpro.c: In function ‘led_breathing_color_pattern’: cynpro.c:146:3: error: ‘for’ loop initial declarations are only allowed in C99 mode    for (size_t i = 2; i < 88; i++)    ^ cynpro.c: In function ‘led_breathing’: cynpro.c:180:3: error: ‘for’ loop initial declarations are only allowed in C99 mode    for (size_t i = 2; i < 88; i++)    ^ cynpro.c: In function ‘walk_devices’: cynpro.c:238:3: error: ‘for’ loop initial declarations are only allowed in C99 mode    for (ssize_t i = 0; i < len; i++)    ^ make: *** [cynpro] Error 1
Trying to add -std=c99 option to build cause following errors
$ make cc -I/usr/include/libusb-1.0   -Wall -std=c99    cynpro.c  -lusb-1.0   -o cynpro cynpro.c: In function ‘parse_options’: cynpro.c:268:3: warning: implicit declaration of function ‘getopt’ [-Wimplicit-function-declaration]    while ((c = getopt(argc, argv, "hl:c:b:i:")) != -1)    ^ cynpro.c:276:18: error: ‘optarg’ undeclared (first use in this function)        if (strcmp(optarg, "green") == 0)                   ^ cynpro.c:276:18: note: each undeclared identifier is reported only once for each function it appears in cynpro.c:321:11: error: ‘optopt’ undeclared (first use in this function)        if (optopt == 'c' || optopt == 'l' || optopt == 'b'            ^ make: *** [cynpro] Error 1

Issue fixed with adding getopt headers. I make pullrequest little bit later.


code improvements (the image is calculated to base 64 but from local …

cleanup of image in utils.py

I've merged it but had to revert the feature due to the tool breaking on python 3.11 and earlier


New-InMemoryModule does not get current AppDomain reference in a cross-platform manner

I changed the way I got an AppDomain reference recently in order to support PowerShell Core. The GetValue overload I use doesn't exist in .NET 2 however. To fix this, I just need to call the overload that's present in all versions.

Fixed


Broken link in documentation

https://kubernetes.io/docs/admin/multiple-schedulers/ states

Kubernetes ships with a default scheduler that is described here.

"here" points to https://kubernetes.io/docs/admin/kube-scheduler/ which gives a 404.

@JensRantil  Where should this link point to ? Any idea ? I can correct it.

@vbmade2000 Maybe https://github.com/kubernetes/community/blob/master/contributors/devel/scheduler.md?

@JensRantil @vbmade2000 , this issue duplicates the issue #3063

@surajnarwade Nice find. Closing.


Pessimistic test framework versions for bundle gem command

Here's a first draft of the change discussed in #3806.
Use a pessimistic version requirement for RSpec and Minitest in
gemspecs generated by the bundle gem command. It is best practice to
at least depend on major versions of development dependency gems.

Hard code major versions of the two testing frameworks.
Add a spec that tests that only specific version requirements are
used in development dependencies.

Some questions:

The test forbundle gem contains some duplication for cases concerning different gem names (with dash, with underscore etc). The shared example group that I added could have been used in more of the nested contexts, but I did not feel that those were testing different paths in my case.
I mostly struggled with the naming of the shared example group. Probably someone could come up with something a little less stilted...

Looking forward to your feedback.

Not sure why the specs are failing for Rubygems < 2.0. remove_push_guard seems to be called in the before block. Shouldn't that prevent the error?

A couple of reasons... the guard removal code wasn't actually being called where you inserted the test, and the specific? method only exists on RubyGems 1.8.1 and higher. I pushed a commit with fixed tests to #3885.

Thanks! :+1:

Merged in 26c4665788cd2601211cc54935411b8227c2f132. Thanks for the feature!


Redux in components

In this step I used Redux store to display books. I also added  some UI elements to  components and make them dispatch actions.

Hi @Masouma864,
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 closed pull request. Your pull request is expected to be open until it's approved. Also, as the previous reviewer mentioned, the linter checks aren't working. Kindly fix it.
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.

I fixed linter error and did pull request


Fix LCD pins to work correctly

Summary
Fix LCD pins to work correctly
Impact
None
Testing
With LVGL demo

@acassis the check failure is fixed here: https://github.com/apache/incubator-nuttx/pull/1389

@acassis the check failure is fixed here: #1389

@xiaoxiang781216
I merged the #1389 and reran the CI but it still fails...

@acassis the check failure is fixed here: #1389

@xiaoxiang781216
I merged the #1389 and reran the CI but it still fails...

This PR need rebase to the latest mainline.

This PR need rebase to the latest mainline.

@xiaoxiang781216
Thanks.
@acassis
Please rebase this PR to the latest master.

Thank you @xiaoxiang781216 and @masayuki2009


Spec for the new Query Role API

The rest specs for the new query role API in 8.15 https://github.com/elastic/elasticsearch/pull/108733 .
The docs for it are at https://github.com/elastic/elasticsearch/pull/110473 .

is there no json spec for this in the server code? we cannot validate this nor generate the output without that at the moment

Thanks @l-trotta . I've addressed your review. Please take another look.

following what the validator is saying, description should also be added as property to RoleDescriptorRead

Thank you for your help on this @l-trotta ! Much appreciated.


[Issue] No baseline for 15.1.1531.7

There is no baseline file for 15.1.1531.7 available, which results in numerous "NoHashMatch" entries in the result CSV when executing CompareExchangeHashes.ps1

This baseline is now available could you re-try?


External Web Dependencies

I would like to use this software on my internal ESX environment which doesnt have internet access. In inspecting the page source there are references to fonts.googleapis.com, jquery themes, and twitter api widgets that never load causing very bad lag when loading webcommander.
Was webcommander ever intended to work in a non-internet connected environment? If not then this issue can be closed.

You could download those external files and reference to them locally.

I will take care of that on Monday and get back to you.

Went ahead and downloaded the multiple versions of jquery and there dependencies. I commented out the links to the google fonts as well. UI worked as advertised.


OwlBase.sol

Put all functions that are being used by all contracts in one base contract to be inherited from.

/cib


Need to set up documentation site framework

Install JSDoc for frontend documentation of components: https://www.inkoop.io/blog/a-guide-to-js-docs-for-react-js/

Don't need for our repo

We could actually use this because our code base is starting to get quite a bit bigger (and messier). I suggest we use Docsify to generate and write documentation. I want to document:

Frontend: Components that are used
Backend: API endpoints that can be called
MongoDB: Organization of databases and collections, as well as the standard format for documents.

We could actually use this because our code base is starting to get quite a bit bigger (and messier). I suggest we use Docsify to generate and write documentation. I want to document:

Frontend: Components that are used
Backend: API endpoints that can be called
MongoDB: Organization of databases and collections, as well as the standard format for documents.


Fix compatibility with 2024.1 eap's backwards incompatible API of OAuthRequestHandlerBase

2024.1 is breaking the API we're using for "Sign in to AppMap" in an incompatible way. I've reported this at https://youtrack.jetbrains.com/issue/IDEA-346395, but so far JetBrains doesn't seem likely to fix this.
This PR is fixing the problem on our side to allow users of 2024.1 to sign in to AppMap.
This can be tested by using the "Sign in" action (at Tools > AppMap) with the current 2024.1 eap of a JetBrains IDE. It should also be tested with 2022.1, our earliest supported version.
I've successfully tested this with both versions.

:tada: This PR is included in version 0.59.0 :tada:
The release is available on:

v0.59.0
GitHub release

Your semantic-release bot :package::rocket:


Remove java-repo-tools directory.

It is not needed anymore, now that we package the parent pom in Maven
central.
Rather than using submodules, this time I clone the java-repo-tools repo
in the test script. (I probably should have done that for the other
repos, too, but it felt especially important for this quickstart repo).
@lesv PTAL

Oops. I didn't actually delete it. Wait to merge.

Okay. Actually removed now. Since the ./travis.sh script clones the repo, the folder reappeared when I tested manually. I added java-repo-tools to the gitignore to prevent that.

LGTM - but I no longer see a reference to the repo tools repo.  What magic installs the files?

There's still a git clone command in travis.sh.


Add file downloader

Introduces an http package and the option to download files
It will later be used in Kubernetes components to download the necessary artefacts for airgapped installs

I'd want to explore a little better Github's API before merging this.

I'd want to explore a little better Github's API before merging this.

I decided against using the Github API for now. It is fairly straightforward to list the release assets using GetReleaseByTag and downloading those using DownloadReleaseAsset.
However, by relying on those, we still need to filter only the necessary assets for a particular build and using the API can very easily run into rate limiting errors. In order to resolve these, then we'd also have to introduce GitHub authentication and given the time constraints we have, this is not something which will provide us with a huge benefit.
We'll instead rely on using the download functionality introduced in this PR against the so called web download urls (e.g. https://github.com/rancher/rke2/releases/download/<version>/<asset>) to download the relevant artefacts.


feat: add build step patching, and time and duration messaging

Checklist

[ ] Affected Issues have been mentioned in the Closing issues section
[ ] Documentation has been written/updated
[ ] PR title is ready for changelog and subsystem label(s) applied

Closing issues
closes #1082
partially closes #2908

smol change to the message printing - now in a one liner with the timezone too
STEP Post-Rollout Tasks: Completed at 2021-11-09 23:39:44 (UTC) Duration 00:00:01 Elapsed 00:00:23

also added a teensy 0.5s sleep after apply to allow the logs to pause whilst pushed to try to pin message to the bottom.

Sweet. I think for now the tiny message grab at the bottom is unavoidable though. Even with a forced sleep it might still capture them if the controller is a bit slow on a busy cluster.
With more staged build script, or staged build pods, we could probably handle this better in lagoon-kbd in the future though.


[WIP] Add Android compatibility

Addresses https://github.com/doefler/react-native-social-share/issues/30 & https://github.com/doefler/react-native-social-share/issues/26
This pull request adds Android compatibility for sharing via Twitter. Note that this only provides the tweet method at present, and Facebook is not supported yet.
Sharing via Twitter will check for the presence of the Twitter application, and will use its provided intent if available. Otherwise, it will open a web browser to compose the tweet.
TODO:

[ ] Get https://github.com/doefler/react-native-social-share/pull/34 merged in
[x] Support Facebook sharing
[x] Support the various options that iOS supports

Due to known issues with shelling out to the Facebook application-provided Intents, Android can't handle all of the arguments that iOS can. I'll update the README to reflect accordingly.

This PR is ready to merge once #34 is merged in (in order to develop on it, I had to cherry-pick the commits from that PR onto this branch).

I merged both PR @Jberlinsky
Great work 😄
I made a small correction in the code sample in the readme.md

@Jberlinsky let me know if you need other changes for it to work.
As I understand it was difficult since #34 wasn't merged.
After that I will do a version update to NPM

@doefler That should just about do it, thanks!

@Jberlinsky cool, I will do a NPM version update then 👍


Don't run nightly without changes

Motivation
npm is getting flooded with nightly builds.  Most days, especially weekends, don't have contributions so we can skip those releases.
Changes

Abort the nightly build if there were no changes in the last 25 hours.
Simplify the pack logic.
Run the nightly job when pushing to the nightly branch; this helps testing changes to the nightly job.

Tests

Outdated


Bump sctp-proto to 0.2.2

Another bump moving some logging from debug to trace in sctp crate

Thanks


Add support for dnf config-manager

Is your feature request related to a problem? Please describe.
It is annoying to add packages like Charm Glow and Mods to my fedora system. I have seen other projects like Tailscale handle this much more elegantly.
Describe the solution you'd like
Add a working url on: https://repo.charm.sh/yum/charm.repo
That has the following raw text:
[charm]
name=Charm
baseurl=https://repo.charm.sh/yum/
enabled=1
gpgcheck=1
gpgkey=https://repo.charm.sh/yum/gpg.key

Describe alternatives you've considered
Currently I am using the following url instead:
https://raw.githubusercontent.com/ublue-os/bluefin/main/etc/yum.repos.d/charm.repo
In the past I had also relied on third-party COPRs like https://copr.fedorainfracloud.org/coprs/tokariew/glow/ but nobody has done this for mods
Additional context
Example with (Tailscale)[https://tailscale.com/download/linux/fedora]:
With tailscale we are given the following install instructions:
Add the Tailscale repository and install Tailscale:
sudo dnf config-manager --add-repo https://pkgs.tailscale.com/stable/fedora/tailscale.repo
sudo dnf install tailscale

I would like to be able to do the same with the packages from charm.sh

Hey!
I don't use fedora myself, so pardon if I'm asking a dumb question, but doesn't the instructions in the readme already work for fedora?
According to the docs, creating the repo file in /etc/yum.repos.d/ works on both cases, and then you can yum update/dnf update.
I agree that having the repo file in a URL looks cleaner, though.

Yes it was


How to select city, state or zip code?  By noaa query of locations search?  Then map to alert?

How to select city, state or zip code?  By noaa query of locations search?  Then map to alert?  Asking here, since I did not see anyway to ask you a question directly?

This module is no longer supported and pretty buggy. I believe there are other libraries to wrap around the same service much better than this one.

Yes, I found some alternate methods.  But thank for the reply!  Most
appreciated.
Schor-
On 05/22/2017 07:27, Josh wrote:

This module is no longer supported and pretty buggy. I believe there
are other libraries to wrap around the same service much better than
this one.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Dolphman/siren/issues/1#issuecomment-303116219,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABatzaMrq9ZuhWydncsWoS9qRCxaHvojks5r8ZtUgaJpZM4Nh21B.


Replace ProviderMethodModel and ConsumerMethodModel with MethodDescriptor and add some unit test

What is the purpose of the change
Replace ProviderMethodModel and ConsumerMethodModel with MethodDescriptor and add some unit test
I saw this comment on ProviderMethodModel and ConsumerMethodModel, so I modified it by the way

Codecov Report

Merging #8018 (65e8852) into master (9f6b536) will decrease coverage by 0.73%.
The diff coverage is 52.38%.

@@             Coverage Diff              @@
##             master    #8018      +/-   ##
============================================
- Coverage     60.85%   60.12%   -0.74%
+ Complexity      491      489       -2
============================================
  Files          1089     1090       +1
  Lines         43876    43947      +71
  Branches       6413     6404       -9
============================================
- Hits          26699    26421     -278
- Misses        14208    14557     +349
  Partials       2969     2969

Impacted Files
Coverage Δ

...va/org/apache/dubbo/rpc/model/ServiceMetadata.java
67.85% <ø> (+19.58%)
:arrow_up:

.../dubbo/metadata/report/MetadataReportInstance.java
12.50% <0.00%> (-40.63%)
:arrow_down:

...java/org/apache/dubbo/rpc/model/ConsumerModel.java
52.83% <40.00%> (+1.88%)
:arrow_up:

...java/org/apache/dubbo/rpc/model/ProviderModel.java
59.70% <40.00%> (+11.94%)
:arrow_up:

...a/org/apache/dubbo/rpc/model/ApplicationModel.java
96.55% <100.00%> (+21.55%)
:arrow_up:

...main/java/org/apache/dubbo/rpc/AsyncRpcResult.java
50.47% <100.00%> (ø)

...ookeeper/ZookeeperDynamicConfigurationFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...gcenter/wrapper/CompositeDynamicConfiguration.java
0.00% <0.00%> (-73.34%)
:arrow_down:

...rg/apache/dubbo/rpc/model/ProviderMethodModel.java
0.00% <0.00%> (-72.73%)
:arrow_down:

...pport/zookeeper/ZookeeperDynamicConfiguration.java
0.00% <0.00%> (-57.15%)
:arrow_down:

... and 74 more

Continue to review full report at Codecov.

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

Thanks, Burning, that's a great job.
But I am afraid we cannot accept this patch for the time being. For some historical reasons, some user's inner frameworks rely heavily on these models, so we need to wait for them to eliminate dependencies first before we can refactor them.


Admin SDK example

The example code only shows how to use the Http protocol connection to FCM.
There should also be an example using the Admin SDK.

@kroikie do you have an Admin SDK example for FCM?

Currently the example directly calls the v1 API, I will update it to include or even replace the current implementation with the Admin SDK usage.


fix: Links in emails are rendered as plaintext

This PR fixes the email templates to render the links as actual hyperlinks rather than plaintext.
Also updates the year in the emails to 2020.

we'll also need to update the images in the emails as well

I think we can just add them on the release/guh-2020 branch, no need to have the hackathon's logo on master


MaskErrors does not work since version 0.240.0

Since version 0.240.0, the MaskErrors extension fails to mask errors.

Describe the Bug
See example script here:
import typing
import strawberry

from strawberry.extensions import MaskErrors

@strawberry.type
class Book:
    title: str
    author: str

def get_books():
    raise Exception("Something went wrong")

@strawberry.type
class Query:
    books: typing.List[Book] = strawberry.field(resolver=get_books)

schema = strawberry.Schema(query=Query, extensions=[MaskErrors(error_message="Internal server error")])

In the previous version it would show Internal server error as the message. Since 0.240.0 it shows this:
{
  "data": null,
  "errors": [
    {
      "message": "Something went wrong",
      "locations": [
        {
          "line": 2,
          "column": 3
        }
      ],
      "path": [
        "books"
      ]
    }
  ]
}

System Information

Operating system: macos
Strawberry version (if applicable): 0.240.0 and up

Additional Context

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

I'll check this today! thanks for the report 😊

Thanks!


How Can I get MediaProjection's interface from NDK?

Using java to write the screencapture is easily,but In my project, I want to get MediaProjection's interface from NDK,I can't find any useful docs, In other word, My Question is How to capture screen from NDK,please help me ~

There is no interface from NDK: all supported functionalities are inside header files under NDK, if there is header file, there is no support.  You app will have to do it from Java side API.

It will broken design for different language, why not plant to support it ?

Only the really necessary features are exposed to native side; for example there is no native api for UI, just use JNI to callback Java side to perform the same functionality.

so,It's the only way that use aosp  to build private library!

@ggfan
My works is like this： using NDK to write a CPP  class which will capture the screen ,and then pass the yuv data to next class  to encoder or render, If Android not supported the way which is that takes a screenshot from ndk, the code will be broken by different language，and then broken struture design.Yes I can call JNI to callback java side ,but,do you consider the efficiency?  by the way, Why  can
can audio get highly support in NDK?


Initial Pull Request

This is everything I've done so far. Please review.

It might be worth running http://www.jshint.com/ on the JS code; there are a fair amount of style things it could help you with, and it's a lot more forgiving than JSLint.

Also, there seems to be almost every style of variable declarations in the code (foo, fooBar, FooBar, $foo, and foo_bar are all there) -- variable naming could be a bit more consistent (I'm used to seeing methods written fooBar() style, but vars less so.
If there's a method to your madness, please explain and I'll lay off.


[PARANOIA 25th Anniversary Edition] Added missing "fallback_for" translation entry

Changes / Comments
I just noticed that due to a bug, the i18nOutput console command didn't extract the "fallback_for" translation entry, messing up my previous release of the PARANOIA 25th Anniversary Edition character sheet. This has been fixed and documented. Note that I'm reporting the i18nOutput bug separately through normal support channels.
On an unrelated note, my Roll20 profile has yet to be credited as a sheet author despite identifying my user ID in sheet.json. Please verify.
Roll20 Requests

Does the pull request title have the sheet name(s)? Yes.
Is this a bug fix? Yes.
Does this add functional enhancements (new features or extending existing features) ? No.
Does this add or change functional aesthetics (such as layout or color scheme) ? No.
If changing or removing attributes, what steps have you taken, if any, to preserve player data ? N/A
If this is a new sheet, did you follow Building Character Sheets standards ? N/A

Character Sheet Info Roll20 Internal Use only.


OL9: Request update irqbalance to 1.9.4 to fix regression

Hello,
As it turns out, irqbalance 1.9.2 introduced a performance-impacting regression that was patched by upstream & included in 1.9.4 a few days later:
https://github.com/Irqbalance/irqbalance/issues/301
Downgrading to irqbalance-1.9.0-3.el9 fixed it for our system. Please either backport their patch or offer 1.9.4 as soon as reasonable. Thanks!

Thanks for raising this issue.  We will look into this.


Ironpython has problems about some python-native libs and pyd problems.

Recently, we've been working on making our old python code compatible with IronPython which uses some native python libs, such as plist-lib, sqlite3 and others. We noticed that IronPython has less efficiency than native-python code called by C/C++. Especially , the sqlite3, which would take 4 time as long to insert 1M records, compaired with code called by C++.
We are also working on handle some iOS files on PC and Plist-lib is necessary, but it will not working with IronPython as XML-lib always cause a crash. I'm wodering if there will be a way to handle those problems? and may be we can fixed it by our own under you guidance.
thanks

The implementation of sqlite is written in C# on IronPython, it has not seen a lot of optimization, so I am not surprised by it being slower than the C implementation. This is something we would have to look at.
Can you give a test case for plistlib so we can see what the issue is?

@slide plistlib uses expat so https://github.com/IronLanguages/ironpython2/pull/460 may fix the issue.

Thanks for your answer！
Here is a simple plist file content, you can copy it into a text-file.
==========begin of the file=============

==========end of the file==============
When I use built-in plistlib , error happens:
>>> res = plistlib.readPlist('LoCoLastUnUploadDic.plist')                                                               Traceback (most recent call last):                                                                                File "", line 1, in                                                                                File "C:\Program Files\IronPython 2.7\Lib\plistlib.py", line 78, in readPlist                                           File "C:\Program Files\IronPython 2.7\Lib\plistlib.py", line 401, in parse                                            ImportError: cannot import expat from xml.parsers
It seems like xml-lib caused the error.

Ok, this should be fixed with the PR that I currently have open that will incorporate an implementation of pyexpat by @slozier. I'll test out with the file you provided

I just ran a quick test and it looks like plistlib works with #460


renderFlipX and renderFlipY is not working according to the specifications.

Hi)
The documentation says to change the direction of your components rendering, you can also use renderFlipX and renderFlipY - https://docs.flame-engine.org/1.0.0/components.html#render-positioncomponent
but the code editor swears that -  The setter 'renderFlipX' isn't defined for the type 'SpriteAnimationComponent'.

From the changelog I can see that renderFlipX and renderFlipY were replaced with flipHorizontally() and flipVertically respectively in rc14. Docs must have gotten out of sync then.


Typo in installation instructions

Hey guys,
I  think you have a typo in the readme:
pip install streaming is probably the wrong command to install this library :smile:
Keep up the great work!
Jo

whoops, thanks for reporting. I was unifying the README's of projects I was publishing on PyPI and then it must have slipped in.

Fixed with 9bdd3c9a92777fd1736af806c267a10066c039e4