[$1000]  Android - Console error when any report is opened - reported by @aneequeahmad

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

Action Performed:

Launch android app.
Click on any report.

Expected Result:
No console errors
Actual Result:
Console error Warning: Failed prop type: The prop anchorPosition.horizontal is marked as required in PopoverWithMeasuredContent, but its value is undefined.
Workaround:
unknown
Platform:

Where is this issue occurring?

Android

Version Number: 1.1.94-3
Reproducible in staging?: need repro
Reproducible in production?: need repro
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: @aneequeahmad
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1661901563422729
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01064e852643baf5f4
Upwork Job ID:<PHONE_NUMBER>500457984
Last Price Increase: 2023-01-04

@mvtglobally were you not able to reproduce this one when you tried on android?

@michaelhaxhiu when I use physical device, I can see the console. So I am not sure if it's repro or not

If we're not able to reproduce to confirm a bug then I'm going to close this for now.

@abekkala There is another bug reported https://expensify.slack.com/archives/C01GTK53T8Q/p1665657234159879
Do you want me reopen this bug?

@kavimuru this is still reproducible in v1.2.20-0. Please see the attached video:
Video:
https://user-images.githubusercontent.com/36359331/198207580-5803dab0-6442-47a5-a114-eac4aa7d909b.mov

Action Performed:

Launch android app.
Click on any report having attachment like image.
Long press on attachment(image) message.

Expected Result:

No console errors

Actual Result:

Console error Warning: Failed prop type: The prop anchorPosition.horizontal is marked as required in PopoverWithMeasuredContent, but its value is undefined.

PROPOSAL
Problem:
As anchorPosition is set to isRequired in component PopoverWithMeasuredContent which is not needed in every case.
https://github.com/Expensify/App/blob/66a05aa4015f96f5bab59929698d517a616620d1/src/components/PopoverWithMeasuredContent.js#L18-L22
Solution:
anchorPosition should not be set to isRequired cuz we don't need to pass prop anchorPosition to PopoverWithMeasuredContent like anchorPosition is not set to isRequired  in other components. for example

AddPaymentMethodMenu

https://github.com/Expensify/App/blob/d6ad76fbb83e3a45501dce6625520499f6195d63/src/components/AddPaymentMethodMenu.js#L15-L21
CODE CHANGE:
FROM
https://github.com/Expensify/App/blob/66a05aa4015f96f5bab59929698d517a616620d1/src/components/PopoverWithMeasuredContent.js#L18-L22
TO:
    anchorPosition: PropTypes.shape({
        horizontal: PropTypes.number,
        vertical: PropTypes.number,
    }),

This is still reproducible on dev main branch v1.2.25-0
VIDEO:
https://user-images.githubusercontent.com/36359331/200722675-feec8c4e-df9e-476d-92b8-9c6f4ae95d66.mov
cc: @abekkala

@abekkala , @aneequeahmad is still able to reproduce on  dev main v1.2.39-0. I am reopening this bug again.

It seems to be on dev only, correct? not reproducible on staging/production?

@abekkala Yes, it is dev only not reproducible on staging/production.

@deetergp based on this convo
Can I get your thoughts here?

After some internal discussion, here's where were at with this:

This only shows in dev because the logging of this kind is only shown in dev.
This warning doesn't happen in release builds because we turn off propType checking for better performance
If we did check types, it would warn in staging & production, but wouldn't be shown anywhere due to being sent to the error boundary
The warn is indicative of an issue and should probably be fixed.
(h/t to @roryabraham for filling in the gaps for me)

Now, with all that said, I'm not sure that removing the propType requirement for the anchorPosition is the correct solution. @aneequeahmad, can you explain why should this be optional here, or any of the other components you cross-checked?

If we did check types, it would warn in staging & production, but wouldn't be shown anywhere due to being sent to the error boundary

Minor correction – JS errors would typically land the user in the error boundary and be logged in Firebase Crashlytics. JS warnings would go into the void, never to be seen again (unless you were looking at JS logs for the release app using Android Studio's LogCat or something similar)

@deetergp Yeah here is the list of components or PropTypes where anchoPosition is optional

AddPaymentMethodMenu.js

https://github.com/Expensify/App/blob/d6ad76fbb83e3a45501dce6625520499f6195d63/src/components/AddPaymentMethodMenu.js#L15-L21

HeaderWithCloseButton.js

https://github.com/Expensify/App/blob/5e297476d5ac1ed10f4151b3bbce0adb5f944d4c/src/components/HeaderWithCloseButton.js#L59-L64

modalPropTypes

https://github.com/Expensify/App/blob/e7c44da13bd4b8f270868073e8d3d0c7ddcb276c/src/components/Modal/modalPropTypes.js#L51-L56

popoverPropTypes

https://github.com/Expensify/App/blob/d4a1cb5029a7e713a7cd7983ae47c061d39ce308/src/components/Popover/popoverPropTypes.js#L10-L15

@aneequeahmad Yes, those other components have mixed requirements, but what does that mean for this component? It's being used by both EmojiPicker.js, where we are passing both the vertical and horizontal positions and in PopoverReportActionContextMenu where we pass it whatever happens to be going on in state. It seems to me with this component having the word Measured in its name, we'd want to be pretty explicit about its size. Rather than make it less specific, let's figure out why we aren't passing horizontal along with vertical in this situation.

I'm ooo now so this one will need to be re-assigned out.
@deetergp if we've determined that we do want this to be worked on we can either leave it internal and you can apply the Bug label to get a BZ team member co-assigned to manage the payments and checklist.
Or, if you want it to be External, the External and Bug label can be applied to auto-create an Upwork job and also get a BZ member assigned to manage this.

@deetergp You're right it makes sense what you said here. i'll investigate the component and let know here soon. Thank you for your comment.

UPDATE:
@deetergp I looked into the component PopoverReportActionContextMenu and found that

In this component propTypes we are omitting anchorPosition

https://github.com/Expensify/App/blob/66a05aa4015f96f5bab59929698d517a616620d1/src/components/PopoverWithMeasuredContent.js#L15-L16

In popoverPropTypes i found this comment Anchor position is optional only because it is not relevant on mobile and it was added in commit # 798ad5b as shown in diff image below

Also, popoverPropTypes has defaultProps where we are omitting popoverPropTypes from defaultModalProps
https://github.com/Expensify/App/blob/d4a1cb5029a7e713a7cd7983ae47c061d39ce308/src/components/Popover/popoverPropTypes.js#L24-L25
and the defaultModalProps also has optional popoverAnchorPosition here https://github.com/Expensify/App/blob/e7c44da13bd4b8f270868073e8d3d0c7ddcb276c/src/components/Modal/modalPropTypes.js#L50-L56
Conclusion:
From investigation explained above it seems like the prop under question is optional. What's your opinion ? @deetergp

@deetergp Any update here ?

No updates yet, was ooo. Will update later today.

@aneequeahmad Based on your investigation it does look like you're right and we can make it optional. Let's go with your proposal and thanks for the deep dive!

@deetergp Sound's good. Let do it. Thank-you for you comment.

I re-applied the External and Bug labels. @aneequeahmad has a workable solution so let's go with that.

Hi @deetergp - I'm assuming that I'm taking care of hiring, not @johncschuster ?

@aimane-chnaif Please see this comment here Anchor position is optional only because it is not relevant on mobile.
Also, popoverPropTypes has defaultProps where we are omitting popoverAnchorPosition  from defaultModalProps as this is optional and not required.

@aimane-chnaif nativeEvent has this object on android {"absoluteX":<PHONE_NUMBER>5214844, "absoluteY":<PHONE_NUMBER>101562, "duration": 521, "handlerTag": 5, "numberOfPointers": 1, "oldState": 2, "returnValue": false, "state": 4, "x":<PHONE_NUMBER>5214844, "y": 28.052824020385742} which doesn't have pageX or pageY in it as commented the horizontal and vertical position (relative to the screen) where the popover will display.
https://github.com/Expensify/App/blob/9cf07d2b81dfc0206932e1c5b38c190712564651/src/pages/home/report/ContextMenu/PopoverReportActionContextMenu.js#L46-L50
Does it make sense why it is not relevant on mobile ?

@aimane-chnaif Why are we complexing the issue which is The prop anchorPosition.horizontal is marked as required in PopoverWithMeasuredContent, but its value is undefined as we already know anchor position is optional.
What's main concern for this issue is the following

let's figure out why we aren't passing horizontal along with vertical in this situation.

Which i explained above in the comment

@aimane-chnaif pageX and pageY value exists for PointerEvent and doesn't exist for LongPress

@aimane-chnaif What are your thoughts on my comment here.
The issue that i reported here is the prop is not optional and it's required. I investigated and explained in detail why these values are not needed. I feel like what you said it makes event data consistent between native and web though actually not used is overEngineering.
Please read this issue from first comment so that you can get idea what i'm talking about.

Yeah, it sounds like @Ollyws's proposal, which would make it more consistent across the platforms, is the right way to go here.

Going to unassign @johncschuster since it looks like we both got added. I'll go ahead and hire @Ollyws and @aimane-chnaif

@Ollyws can you link me your Upworks profile?

👋 @Ollyws needs to be assigned to this issue right? Is there a PR to link as well, yet?

@deetergp @abekkala @johncschuster @jliexpensify Pretty please make sure that your issues correctly have the bug label on them if we're actively looking for proposals. Bug was just added to this issue, so before it was not on the WAQ initiative radar, but now it is. Then given that the bug is well over 30 days old, it's immediately something that we've got to prioritize as a result.

@aimane-chnaif, @deetergp I'm still not clear how can we add the wrong/irrelevant data just to make the data consistent  and this pageX and pageY property is not provided by the react-native-gesture-handler for LongPressGestureHandler.
Moreover, PageX and PageY and different from absoluteX and absoluteY. Value of x, y and absoluteX and absoluteY are alike and absoluteX and absoluteY value can be used instead of x and y as written in the doc.

Even in react-native doc pageX and pageY value only exist in nativeEvent of Synthetic Touch Events for View responder props. Also, I can't find any authentic source that says that absoluteX and absoluteY value can be given to pageX and pageY.
The react-native doc says: Method that provides pageX and pageY value is measure(callback) which determines the location on screen, width, and height of the given view and returns the values via an async callback. If successful, the callback will be called with the following arguments:

x
y
width
height
pageX
pageY

For new feature issues, Bug label should be added as well if we're actively looking for proposals?
i.e. https://github.com/Expensify/App/issues/12698 has lack of proposals maybe because of Bug label not applied.

Missed this last week, though we aren't added the bug label for NewFeature issues. Main reason is that we still favor bug fixing over features at the moment, though I'm sure there are process improvements we can still make.

Job - https://www.upwork.com/ab/applicants/1610433379500457984/job-details
Changed title to reflect payment date (huh, I thought it would update automatically?), hired @Ollyws and @aimane-chnaif and invited @aneequeahmad . Cheers!

@jliexpensify applied thanks

Everyone's been paid, thanks!

Hi @aimane-chnaif - apologies: I'll get a new Upworks link for both you and @Ollyws for the $500.

I ran into some errors on the Upworks site doing that (not sure what's happening), so I've just rehired you on the new job - that seems to have worked.

All sorted, thanks Oliver and Aimane!


[Event Request] codeunit 5817 "Undo Posting Management" - New event OnUpdatePurchLineOnBeforeInitOustanding

Hi,
we need an event OnUpdatePurchLineOnBeforeInitOustanding, similar to the one already present for Sales. It would be useful to add the source record (that means for example Purchase Receipt) in order to have all information related to any customization and let us to update PurchLine according to our customization.
Now I'm not able to retrieve information about the correction except UndoQty passed as parameter. Having Purchase Receipt Line record would help us to update other custom fields.
As suggestion:
UpdatePurchLine function call in "Undo Posting Management" codeunit: add SourcePostedDoc: variant (PurchRcptLine in my example) record as parameter
UndoPostingMgt.UpdatePurchLine call in "Undo Purchase Receipt Line" codeunit: pass PurchRcptLine record for SourcePostedDoc parameter
OnUpdatePurchLineOnBeforeInitOustanding event in "Undo Posting Management": pass new parameter SourcePostedDoc received
this issue is related to another opened by me for sales area: #14373
I hope that our need is clear.
Thank you

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[SMALLFIX] Use static imports for test utilities

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2910/
Test PASSed.

alluxio-bot, check this please


Simplify extend_rpc_modules arguments

Describe the feature
this is currently very verbose:
https://github.com/paradigmxyz/reth/blob/4dceabf06b9ef26422264ad2026f6e23fda38fbb/bin/reth/src/cli/ext.rs#L59-L66
these two arguments should be combined into a new helper container type generic over Reth: RethNodeComponents
like RethRpcComponents<'a Reth> {   pub registry: &mut ... }
Additional context
No response

I can take this


VART python code for yolo

Hi, Is there any VART python code example available which runs YOLO .xmodel files? I found DNNDK examples for the same but could not find VART examples. Can you please share the link if VART examples are available for the same?
Thanks!

We are very sorry that there are currently no examples of running YOLO .xmodel in our library


Add option to run "latest" migrations upon server initialization on a per-plugin basis

To be fully spec'd.  But this should leverage the knex migration API.

This actually could pose challenges, tgriesser/knex#607. An alternative is to give each plugin its own migrations table. Hrmm, we may have to get creative!

(Knex will refuse to run migrations if it looks like there are any migrations that have already been run but aren't in the current migrations directory.)

  Will have to think about the ramifications.

Here's a shot at it: https://github.com/BigRoomStudios/schwifty/compare/knex-instance...migrator


Add global stats

The tiles are currently out of alignment.

Stats page has been removed from the collection pages (will probably implement later for the global site)

User quote to show the need for this "since you remove the Statistics（ the original ： https://www.LibCrowds.com/statistics/）,I can't get some informations from that such as the daily/weekly degree of activity， if convenient , If  permited，may I  have a look？(/ω＼) For a research..."

When we eventually get to this, we can use the update feed from http://docs.pybossa.com/en/latest/api.html#account-index to get the latest actions on the server (users created, contributions, new tasks, etc.).

Following GDPR changes we no longer store real IP addresses for our users, so the map stuff (although never quite implemented) no longer makes sense. We could consider added some more charts somewhere but I don't think this is urgent.

Let's scrap this - IPs have been anonymised now, so no more location maps. Plus, we can use Jupyter notebooks for more analytical things.

We should remove the mapbox config var though


Enhanced table does not respect Kibana time range filtering

Greetings,
Maybe this is somewhat answered in a previous topic, but I am facing the following issue:
When trying to create an enhanced table, with count as metric, values seem static, regardless the time range I choose. Is this a Kibana issue? If yes, is there any suggested workaround?
Thanks in advance,
Dimitris

Hi,
There has been an issue about that, few months ago, for few Kibana versions.
But it has been fixed since then.
Can you tell me what is your Kibana version and enhanced table plugin version?

Hi,
There has been an issue about that, few months ago, for few Kibana versions.
But it has been fixed since then.
Can you tell me what is your Kibana version and enhanced table plugin version?

Thanks for the quick response!
I am using the latest table version (1.11.0) with Kibana 7.9.0. I will
upgrade to latest Kibana and see if the problem is fixed!
Στις Πέμ, 28 Ιαν 2021, 23:41 ο χρήστης Fabien Baligand <
<EMAIL_ADDRESS>έγραψε:

Hi,
There has been an issue about that, few months ago, for few Kibana
versions.
But it has been fixed since then.
Can you tell me what is your Kibana version and enhanced table plugin
version?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/fbaligand/kibana-enhanced-table/issues/171#issuecomment-769417774,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN4PIT6EWK62Q3RMZ4ZTCFLS4HKYVANCNFSM4WXFOX7A
.

Thanks for the quick response!
I am using the latest table version (1.11.0) with Kibana 7.9.0. I will
upgrade to latest Kibana and see if the problem is fixed!
Στις Πέμ, 28 Ιαν 2021, 23:41 ο χρήστης Fabien Baligand <
<EMAIL_ADDRESS>έγραψε:

Hi,
There has been an issue about that, few months ago, for few Kibana
versions.
But it has been fixed since then.
Can you tell me what is your Kibana version and enhanced table plugin
version?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/fbaligand/kibana-enhanced-table/issues/171#issuecomment-769417774,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN4PIT6EWK62Q3RMZ4ZTCFLS4HKYVANCNFSM4WXFOX7A
.

Hi,
I just made some tests with Kibana 7.9.3 and enhanced-table 1.11.0 ; and time range selection is processed as expected.
I made tests both on Visualize app and Dashboard app, with Enhanced Table vis and Document Table vis.
I don't reproduce your issue.

Hi,
I just made some tests with Kibana 7.9.3 and enhanced-table 1.11.0 ; and time range selection is processed as expected.
I made tests both on Visualize app and Dashboard app, with Enhanced Table vis and Document Table vis.
I don't reproduce your issue.

Hey, I upgraded to latest Kibana and everything works great. Thanks again for your time and effort! Much appreciated

Hey, I upgraded to latest Kibana and everything works great. Thanks again for your time and effort! Much appreciated

Nice to see that it works with last Kibana version!
so I close the issue.

Nice to see that it works with last Kibana version!
so I close the issue.


Possible fix to shell escaping env vars

So. I'm not positive we should do this. This is solves the escaping of env vars in the proper bash way (as blessed by shellcheck), which is to use an array sets of things. This is good.
This is going to require changing numerous formulae in both pantries which rely on the old version of settings a single sell variable. This seems really, really bad.
You can test it by building a sample package.yml (I called it foo) that looks like this:
distributable: ~

versions:
  - 0.0.1

build:
  script: |
    echo $ARGS
    for x in "${ARGS[@]}"; do
      echo "$x"
    done
  env:
    ARGS:
      - foo
      - bar
      - "baz bat"
      - --thing="foo bar"
      - "--thing=baz bat"

test: true

Since people shy away from bash arrays, I think it might be smarter just to treat them directly in the script rather than make this change wholesale. But I submit it for evaluation and discussion.

The fact that this passes all tests only makes me more nervous.

The flipside of my concern about arrays, of course, is that it's the right thing to do. strings of strings (potentially of string of strings of string) each with their own escaping concern is why shells have the reputation they do already. bash's solution is the array, so maybe we should be good boys and girls and just use it.

(since this is still taking up space in my brain)
hilariously, even things that look right don't seem to be based on testing. The following don't want to work:
export ARGS="thing 'foo bar' other_thing"
export ARGS="thing \"foo bar\" other_thing"
export ARGS="thing foo\ bar other_thing"

bash really wants you to use arrays for this use-case.

We’ll try this out after v0.19.0 has been out a little bit.

isn't the correct way to do this, this:
export ARGS="thing ""foo bar"" other_thing"

?
My concern is only due to my lack of familiarity with (ba)sh arrays and what might happen as a result of this change
OTOH we can revert it quickly enough.

I no longer remember if I tried double quotes. I'll test that.

So, this doesn't seem to work:
ARGS="$ARGS --with-pkgversion=""tea GCC {{version}}"""
../configure "$ARGS"

But, I agree that this is a massive change in style. I don't feel good about merging it; certainly without testing.
Unpopular opinion: switching our internal shell scripts to something like fish, where all variables are lists by default, solves this at the expense of exposing us to unknown fish idiosyncrasies and involves much, much, much more work and testing. Maybe the build scripts should .rb, .py, .ts, or some other real language? I don't think there's any one "best" solution, but bash is by far the least surprising.

Could you elaborate on what doesn’t work?
I'd be fine with fish, if it was widely known, but it isn’t. We have to make choices are contributors are comfortable with if we want contributions.

Possibly it didn’t work because we’ve already been doing the double-double-quote expansion for months:
function expand_env(env: PlainObject, pkg: Package, deps: Installation[]): string {
  return Object.entries(expand_env_obj(env, pkg, deps)).map(([key,value]) => {
    // weird POSIX string escaping/concat stuff
    // eg. export FOO="bar ""$baz"" bun"
    value = `"${value.trim().replace(/"/g, '""')}"`
    while (value.startsWith('""')) value = value.slice(1)  //FIXME lol better pls
    while (value.endsWith('""')) value = value.slice(0,-1) //FIXME lol better pls

    return `export ${key}=${value}`
  }).join("\n")
}

Agree that switching to fish is worse in most ways. I believe the thing that was failing was:
script: |
  ../configure $ARGS
env:
  ARGS:
    - --version-string="tea version {{version}}"

Including varieties of double-quoting.


Set up result output

Task List

[ ] Color Coded Heat Map
[ ] GUI

Branch: heatmap

Color Coded Heat Map
Considerations

What is the format of the output?

Useful Links

TermColor

System.out coloring library

GUI

Very cool! I think this is a very finalized look for what we need.
If you'd like, you can make a draft PR and use it as a pastebin for your branch progress, such as this 🤗


zsh-nix-shell: 0.6.0 -> 0.7.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/chisui/zsh-nix-shell/releases.
meta.description for zsh-nix-shell is: zsh plugin that lets you use zsh in nix-shell shell
meta.homepage for zsh-nix-shell is: https://github.com/chisui/zsh-nix-shell
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.7.0 in filename of file in /nix/store/ifqixbbgvq8ff5nzlfgy0j20n3cddc0p-zsh-nix-shell-0.7.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
zsh-nix-shell

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ifqixbbgvq8ff5nzlfgy0j20n3cddc0p-zsh-nix-shell-0.7.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A zsh-nix-shell https://github.com/r-ryantm/nixpkgs/archive/91dc753211caa5dd717878ac775e5e6bd3d0f03b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ifqixbbgvq8ff5nzlfgy0j20n3cddc0p-zsh-nix-shell-0.7.0
ls -la /nix/store/ifqixbbgvq8ff5nzlfgy0j20n3cddc0p-zsh-nix-shell-0.7.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

zsh-nix-shell

Maintainer pings
cc @herrwiese for testing.

Result of nixpkgs-review pr 234376 run on aarch64-darwin 1

1 package built:

zsh-nix-shell


Update upstream PostgreSQL Operator to v4.5.1

Updates to existing Operators

[x] Is your new CSV pointing to the previous version with the replaces property?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml ?
[x] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Submit operators to both upstream-community-operators and community-operators at once
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

@jkatz since i see some unresolved comments i label it as work in progress

@mvalarh All the feedback was addressed in the forced pushes. I didn't realize I had to resolve the comments as well. I have gone ahead and done so. Thanks :+1:

@jkatz Is it ready to be merged?

@mvalarh I incorporated all of the requested changes, so from my vantage point it is ready to be merged, but I defer to the esteemed committers of the community operators project on whether or not it is ready :wink:

@jkatz I am looking at problem why it is not able to create index from scratch. It is failing in our lemon test in Travis CI - Pull Request .

Hi -- is there anything we need to do on our end to fix this?

@jkatz let me retest

opened https://github.com/operator-framework/operator-registry/issues/539

Thank you!


Exception in Rust code bringing the entire dotnet server down

Hey, sorry to bother once again, but I was wondering if you have any idea for the issue I am facing.
For some reason, every now and then, I get an "exception" in Rust which basically completely breaks the application. The server goes down and has to reboot.
I am trying to pinpoint what is the root cause, but right now I am having a hard time reproducing the issue.
I was wondering if you have any insights on how to prevent the server from going down. If there is any way to capture the rust error, log it and move on.
Thanks!

Unfortunately not. It is one of 2 errors:

A panic in rust.
A null pointer.

We are working to solve the second problem with smart pointers, but it is still unsolved.

We're working on this on Rust side and should be able to show something soon.

I might try to fix the panic in rust issue then. Not sure if it is possible to fix that tho, but maybe just add some code to the way we call the rust binaries to catch panics and prevent the pod from going down.
How does ydotnet interact with Rust binaries? Is there any documentation I could read on the topic?

I think it is not possible, because a panic is basically a process.exit(ERROR) in Dotnet. But I would be happy if you could solve that.
There is no documentation, but this is how it works:

Y-crdt has a C binding layer: https://github.com/y-crdt/y-crdt/tree/main/yffi that is used for that.
In out native namespace we have all the code to talk with the channels: https://github.com/y-crdt/ydotnet/blob/main/YDotNet/Native/Document/DocChannel.cs#L15
Sometimes we have to follow pointers, then we always do that in the Native namespace, e.g. here: https://github.com/y-crdt/ydotnet/blob/main/YDotNet/Native/Document/Events/SubDocsEventNative.cs#L21
We expose CLR types that are converted from the native types. The conversion is done outside of the native namespace. The goal is to have a dependency to the native namespaces but not from this namespace to avoid a circular dependency hell. See: https://github.com/y-crdt/ydotnet/blob/main/YDotNet/Document/State/DeleteSet.cs#L10

I am closing this. It is only a meta issue.


Create does not handle well "all" as alias creating topic with all

Description
[wtrocki@graphapi app-services-cli (main)]$ rhoas kafka acl create --operation alter --permission deny --user<EMAIL_ADDRESS>--topic all
debugEnabled false
The following ACL rules are to be created:

  PRINCIPAL            PERMISSION   OPERATION   DESCRIPTION
 -------------------- ------------ ----------- ----------------
<EMAIL_ADDRESS> DENY         ALTER       TOPIC is "all"

Expected
[wtrocki@graphapi app-services-cli (main)]$ rhoas kafka acl create --operation alter --permission deny --user<EMAIL_ADDRESS>--topic all
debugEnabled false
The following ACL rules are to be created:
PRINCIPAL            PERMISSION   OPERATION   DESCRIPTION

<EMAIL_ADDRESS> DENY         ALTER       TOPIC is "*"

I think you have fixed this now @wtrocki


Support PEP 526 -- Syntax for Variable Annotations

Currently this code:
x: int = 3

gets compiled to this:
xint3;

PEP 526 is here: https://www.python.org/dev/peps/pep-0526/

Wow, that's bad.
Will repair, thanks!
BTW. The newest version of mypy will be bundled as soon as it's out, since it will contain an improved version of the API that will connect it to Transcrypt.
http://mypy.readthedocs.io/en/latest/command_line.html#integrating-mypy-into-another-python-application
The original API was found too intrusive by Guido, so it was replaced by this simple one.
https://github.com/python/mypy/pull/2744
As soon as this is released, Transcrypt will be adapted to use it, and with that the newest version of mypy with proper var annotation support.
Kind regards
Jacques

@JdeH Wow, thanks for the mega-quick response.
I am "new" to Python. That's why I went straight to the latest: 3.6.0.
My Transcrypt version is: 3.6.8 (installed via Pip).
As far as my little experiments go, I can confirm that mypy (0.470) also has problems with type-annotations in variables (it reports a parse error). The Python interpreter itself (the 3.6 version) of course has no issue. And the syntax also supported by the latest PyCharm.
Regarding your label: IS:bug? Well... I don't know. The 3.6 Python interpreter supports it, and you claim to be a 3.6 to JS compiler, so ... ;-) --- But if you are basing this on mypy the problem seems to be there.
mypy also does not support f-strings / PEP-498 (which Trascrypt does).

I also want to take the opportunity to thank you and all who make this project possible! :-)
As a web developer with a background in statically-typed languages and a fondness to whatever-to-JavaScript compilers the existence of this Python 3.6 (with optional type annotations) to JavaScript is enormously appreciated.

The  questionmark behind is:bug? isn't meant to express doubt that failing to generate code for x: int = 3 is a bug, it clearly is a bug. It only means that' we haven't yet reproduced it (since we didn't try, that is). So it's just a stage in our quality assessment pipeline.
About the statically typed languages: I have been programming in C++ from its birth, and before that in Pascal and Algol 60, so the added value of compile time typechecks is clear to me.
Of course dynamic typing has also big advantages (flexibility, brief code), but especially when working with a team, static typechecks on module boundaries can save lots of trouble.
I hope you enjoy Trancrypt, and your bug reports are very welcome.
Kind regards
Jacques

Committed an experimental fix for this to GitHub (but not yet on PyPi.)
Feel free to reopen if there are still problems.
Annotation-support for 3.6 is still incomplete.
Will get priority as soon as next version of mypy is released.

Thanks! For my purposes, it would be enough.
However, take into account that PEP-526 states that declaring the type of a variable without an assignment is also possible. What for? In the case of loops, for example.
This is also possible, and currently not supported:
i: int
for i in range(0,5):
    print(i)

Or:
sane_world: bool
if 2+2 == 4:
    sane_world = True
else:
    sane_world = False

As I said, I don't use this - and can gladly wait for mypy to be updated. Just wanted to point it out.

I've committed a fix for the "var annotation without assignment" case.


Not receiving an sms until I click the "request sms" link

My problem is pretty much what the title says. Everything works fine, but I have to click the "request sms" link manually every time, otherwise no sms is sent. Is this a bug or a feature?

Good point! I normally use this with the Authy App, so I don't notice that it doesn't send an SMS immediately.
We can add that as a setting, if that's something you would be interested in?
Otherwise, you can add it yourself by overriding the DeviseAuthyController like so:
# app/controllers/devise_authy_controller.rb
class DeviseAuthyController < Devise::DeviseAuthyController
  before_action :send_request, :only => :GET_verify_authy

  private

  def send_request
    Authy::API.request_sms(:id => @resource.authy_id)
  end
end

Then override the route too:
# config/routes.rb
Rails.application.routes.draw do
  devise_for :users, :controllers => { :devise_authy => :devise_authy }
  # other routes
end

Note that the request_sms route sends :force => true so that it sends an SMS even if the user normally uses the app. In the code above, it just makes a regular request which will send a push notification to the Authy app if the user uses that.
One thing I'm not sure about right now is what happens in the case where you use a third party authenticator. It may cause SMS messages to be sent, even if the user could just open their authenticator app. That would likely be worth testing.

Thank you, this works great! I think the setting would be a valuable addition, as many users know the process this way from websites like Paypal.

This library is no longer actively maintained. The Authy API has been replaced with the Twilio Verify API. Twilio will support the Authy API through November 1, 2022 for SMS/Voice. After this date, we’ll start to deprecate the service for SMS/Voice. Any requests sent to the API after May 1, 2023, will automatically receive an error.  Push and TOTP will continue to be supported through July 2023.
Learn more about migrating from Authy to Verify.
Please visit the Twilio Docs for:

Verify + Ruby (Rails) quickstart
Twilio Ruby helper library
Verify API reference
Coming soon: Look out for a new Devise plugin to use Twilio Verify with Devise

Please direct any questions to Twilio Support. Thank you!


Name conflict: atom

Details
I am currently the maintainer of the atom project ( https://github.com/nucleic/atom) (which predates the popular editor of the same name). As part of my maintenance effort I wrote docs that I would like to publish on read the docs.
However there is a name conflict with  https://readthedocs.org/projects/atom/ which does not have a single successful build and no recent activity (the github repo is a private one and cannot be reached).
I could not find any contact information for the maintainer and I would hence appreciate if a core team could contact the maintainer, to resolve the conflict.

Read the Docs username (if applicable): mdartiailh

@MatthieuDartiailh hey! The atom namespace should be free. I'm closing the issue. Let me know if anything.

I was able to add the project. Thanks @humitos.

You are welcome. Enjoy Read the Docs

And now my docs are live ! I really love readthedocs !

@MatthieuDartiailh it's mutual! Enjoy it!


lib/zocial/zocial.css is not included in the .jar file

Hi, I tried building the starter project on Ubuntu
Environment:
Node: v16.14.0
Ubuntu: 20.04.6 LTS

When I built with Keycloak 22.0.1 and Keycloakify 8.0.0-rc.2, I get this error:
node:fs:1392
  handleErrorFromBinding(ctx);
  ^

Error: ENOENT: no such file or directory, scandir 'keycloakify-starter/build_keycloak/src/main/resources/theme/keycloakify-starter/tmp_suLeKsxId_202f15f6/keycloak/account/resources'
    at Object.readdirSync (node:fs:1392:3)
    at crawlRec (keycloakify-starter/node_modules/keycloakify/bin/tools/crawl.js:43:35)
    at crawl (keycloakify-starter/node_modules/keycloakify/bin/tools/crawl.js:65:5)
    at transformCodebase (keycloakify-starter/node_modules/keycloakify/bin/tools/transformCodebase.js:52:50)
    at keycloakify-starter/node_modules/keycloakify/bin/keycloakify/generateTheme/downloadKeycloakStaticResources.js:86:63
    at step (keycloakify-starter/node_modules/keycloakify/bin/keycloakify/generateTheme/downloadKeycloakStaticResources.js:56:23)
    at Object.next (keycloakify-starter/node_modules/keycloakify/bin/keycloakify/generateTheme/downloadKeycloakStaticResources.js:37:53)
    at fulfilled (keycloakify-starter/node_modules/keycloakify/bin/keycloakify/generateTheme/downloadKeycloakStaticResources.js:28:58) {
  errno: -2,
  syscall: 'scandir',
  code: 'ENOENT',
  path: 'keycloakify-starter/build_keycloak/src/main/resources/theme/keycloakify-starter/tmp_suLeKsxId_202f15f6/keycloak/account/resources'
}
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

So I change Keycloak version to 21.1.2 and build successfully, but the .jar file misses this file build_keycloak/src/main/resources/theme/keycloakify-starter/login/resources/resources_common
lib/zocial/zocial.css, lead to the login form cannot display.
Finnally, I try Keycloak 22.0.1 and Keycloakify 9.0.0-rc.0. This time I see the file build_keycloak/src/main/resources/theme/keycloakify-starter/login/resources/resources_common
lib/zocial/zocial.css but the .jar file build fail because of this error:
node:child_process:828
    err = new Error(msg);
          ^

Error: Command failed: mvn package
    at checkExecSyncError (node:child_process:828:11)
    at Object.execSync (node:child_process:902:15)
    at keycloakify-starter/node_modules/keycloakify/bin/keycloakify/keycloakify.js:194:39
    at step (keycloakify-starter/node_modules/keycloakify/bin/keycloakify/keycloakify.js:56:23)
    at Object.next (keycloakify-starter/node_modules/keycloakify/bin/keycloakify/keycloakify.js:37:53)
    at fulfilled (keycloakify-starter/node_modules/keycloakify/bin/keycloakify/keycloakify.js:28:58) {
  status: 1,
  signal: null,
  output: [
    null,
    Buffer(715) [Uint8Array] [
       91,  27,  91,  49,  59,  51,  49, 109,  69,  82,  82,  79,
       82,  27,  91, 109,  93,  32,  69, 114, 114, 111, 114,  32,
      101, 120, 101,  99, 117, 116, 105, 110, 103,  32,  77,  97,
      118, 101, 110,  46,  10,  91,  27,  91,  49,  59,  51,  49,
      109,  69,  82,  82,  79,  82,  27,  91, 109,  93,  32, 106,
       97, 118,  97,  46, 108,  97, 110, 103,  46,  73, 108, 108,
      101, 103,  97, 108,  83, 116,  97, 116, 101,  69, 120,  99,
      101, 112, 116, 105, 111, 110,  58,  32,  85, 110,  97,  98,
      108, 101,  32, 116,
      ... 615 more items
    ],
    Buffer(0) [Uint8Array] []
  ],
  pid: 30770,
  stdout: Buffer(715) [Uint8Array] [
     91,  27,  91,  49,  59,  51,  49, 109,  69,  82,  82,  79,
     82,  27,  91, 109,  93,  32,  69, 114, 114, 111, 114,  32,
    101, 120, 101,  99, 117, 116, 105, 110, 103,  32,  77,  97,
    118, 101, 110,  46,  10,  91,  27,  91,  49,  59,  51,  49,
    109,  69,  82,  82,  79,  82,  27,  91, 109,  93,  32, 106,
     97, 118,  97,  46, 108,  97, 110, 103,  46,  73, 108, 108,
    101, 103,  97, 108,  83, 116,  97, 116, 101,  69, 120,  99,
    101, 112, 116, 105, 111, 110,  58,  32,  85, 110,  97,  98,
    108, 101,  32, 116,
    ... 615 more items
  ],
  stderr: Buffer(0) [Uint8Array] []
}
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Here is the package.json I use in 3 cases, only change version of Keycloak and Keycloakify:
{
  "name": "keycloakify-starter",
  "homepage": "https://starter.keycloakify.dev",
  "version": "4.7.0",
  "description": "A starter/demo project for keycloakify",
  "repository": {
    "type": "git",
    "url": "git://github.com/codegouvfr/keycloakify-starter.git"
  },
  "scripts": {
    "start": "react-scripts start",
    "build": "react-scripts build",
    "build-keycloak-theme": "yarn build && keycloakify",
    "storybook": "start-storybook -p 6006"
  },
  "keycloakify": {
    "extraThemeProperties": [
      "foo=bar"
    ],
    "extraThemeNames": [
      "keycloakify-starter-variant-1"
    ],
    "keycloakVersionDefaultAssets": "22.0.1"
  },
  "author": "u/garronej",
  "license": "MIT",
  "keywords": [],
  "dependencies": {
    "evt": "^2.4.15",
    "jwt-decode": "^3.1.2",
    "keycloak-js": "22.0.1",
    "keycloakify": "9.0.0-rc.0",
    "powerhooks": "^0.26.8",
    "react": "18.1.0",
    "react-dom": "18.1.0",
    "tsafe": "^1.6.0"
  },
  "devDependencies": {
    "@storybook/addon-actions": "^6.5.16",
    "@storybook/addon-essentials": "^6.5.16",
    "@storybook/addon-interactions": "^6.5.16",
    "@storybook/addon-links": "^6.5.16",
    "@storybook/builder-webpack5": "^6.5.16",
    "@storybook/manager-webpack5": "^6.5.16",
    "@storybook/node-logger": "^6.5.16",
    "@storybook/preset-create-react-app": "^4.1.2",
    "@storybook/react": "^6.5.16",
    "@storybook/testing-library": "^0.0.13",
    "@types/node": "^15.3.1",
    "@types/react": "18.0.9",
    "@types/react-dom": "18.0.4",
    "react-scripts": "5.0.1",
    "typescript": "~4.7.0"
  },
  "eslintConfig": {
    "extends": [
      "react-app",
      "react-app/jest"
    ],
    "rules": {
      "react-hooks/exhaustive-deps": "off",
      "@typescript-eslint/no-redeclare": "off",
      "no-labels": "off"
    },
    "overrides": [
      {
        "files": [
          "**/*.stories.*"
        ],
        "rules": {
          "import/no-anonymous-default-export": "off"
        }
      }
    ]
  },
  "browserslist": {
    "production": [
      ">0.2%",
      "not dead",
      "not op_mini all"
    ],
    "development": [
      "last 1 chrome version",
      "last 1 firefox version",
      "last 1 safari version"
    ]
  }
}

Please help me resolve this issue~ Many thanks <3

Hello @AnhTuan-AiT,
Thank you for reaching out.
Could you elaborate on your reasons for wanting to change keycloakVersionDefaultAssets? Contrary to what you might assume, this value doesn't need to match the version of Keycloak you're using. The default Keycloak assets simply provide a fallback styling for pages you haven't customized. When I initially translated the default Keycloak theme to React, the Keycloak version was 11.0.3; hence the default version remains the same. However, I am planning to upgrade it to version 22 soon.
If you specifically want to use the assets from Keycloak 22.0.1, for instance to use the latest PatternFly version, follow these steps:

[x] Update "keycloakVersionDefaultAssets": "22.0.1" in your package.json.
[ ] Update your src/keycloak-theme/login/Template.tsx file to actually use the default assets from Keycloak 22.0.1.

    const { isReady } = usePrepareTemplate({
        "doFetchDefaultThemeResources": doUseDefaultCss,
        "styles": [
<EMAIL_ADDRESS><EMAIL_ADDRESS>            `${url.resourcesCommonPath}/node_modules/patternfly/dist/css/patternfly.min.css`,
            `${url.resourcesCommonPath}/node_modules/patternfly/dist/css/patternfly-additions.min.css`,
            `${url.resourcesCommonPath}/node_modules/patternfly/dist/css/lib/pficon/pficon.css`,
            `${url.resourcesPath}/css/login.css`
        ],
        "htmlClassName": getClassName("kcHtmlClass"),
        "bodyClassName": undefined
    });

Further steps are theoretically optional but recommended for complete customization:

[ ] Modify useGetClassName to match the property names in Keycloak 22.0.1.
[ ] Eject all pages.
[ ] Resolve type errors.
[ ] Confirm that updating to the new asset version doesn't break your layout.

In summary, updating the assets is ideally something I should handle at the project level. I suggest focusing on building your theme using your own assets.
As for Keycloakify 9, it is not yet buildable and requires this PR to be merged into the Keycloak project.
Hope this helps.

That's right @garronej , I see that 11.0.3's login.css is a bit different so I want to try changing it to match the Keycloak version I'm using. I will wait for your update to version 22.
Thank you very much for your support <3


docs: changelog excludes now supports the use of regexes to improve filtering

Codecov Report

Merging #274 (cbc524a) into main (ac8dcc0) will decrease coverage by 0.23%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##             main     #274      +/-   ##
==========================================
- Coverage   84.32%   84.08%   -0.24%
==========================================
  Files          43       43
  Lines        2073     2099      +26
==========================================
+ Hits         1748     1765      +17
- Misses        257      263       +6
- Partials       68       71       +3

Impacted Files
Coverage Δ

internal/context/context.go
100.00% <ø> (ø)

internal/task/changelog/changelog.go
69.85% <71.87%> (+0.02%)
:arrow_up:

cmd/uplift/bump.go
82.89% <100.00%> (+0.22%)
:arrow_up:

cmd/uplift/changelog.go
86.08% <100.00%> (+0.12%)
:arrow_up:

cmd/uplift/release.go
84.80% <100.00%> (+0.12%)
:arrow_up:

cmd/uplift/root.go
80.48% <100.00%> (+0.48%)
:arrow_up:

internal/git/utils.go
79.65% <100.00%> (-0.42%)
:arrow_down:

internal/task/bump/bump.go
94.73% <100.00%> (-0.27%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Derive value type

PR Info

Closes https://github.com/SeaQL/sea-orm/issues/1349

Dependencies:

https://github.com/SeaQL/sea-orm/pull/1713 (need to check if syn v2 will break the new macro)

Dependents:

New Features

[ ]  added macro to derive for simple value type wrapper

Bug Fixes

[ ]

Breaking Changes

[ ]

Changes

[ ]

Looks really good so far!

So perhaps it's better to have a separate test case to test StringVec end to end.


App is flaky attempting to play a second game after ending with lost player

If a game ends and you respond to the lost player message, the app should return to a "virgin" state where it's possible to start a new game.   What I noticed instead was:

Although the remaining player was leader, he did not have a setup button until after he opened the find players dialog.
It seemed that he had lost the ability to save setups, possibly because some remnants of former private cards were left around?

The root of the problem is the number of things that have to be reset and the number of interdependencies between them, causing the prepareNewGame function to be quite fragile.  What you would really like is for the entire game-playing view controller to reload and re-initialize.   This might be possible but is tricky right now because ViewController is the root view controller of the app.   I think it would be much more possible to do it that way if the UI were reorganized down the lines of issue #37 so that some sort of nav tab view would be the root and the game playing view would then be capable of a complete reload.   In fact, I might not want to tackle that issue until after issue #54, which is likely to change quite a number of assumptions.

Because of the work under issue #60, this issue is now obsolete.   We will have to completely re-evaluate what works and what doesn't after the cutover.


Disabling fold coloring

Folding colors unrelated lines, above and below a fold. Unless it's already a feature, an option to turn off fold coloring or enable customizing it in Preferences > Appearance would help.

Hi @OverLordGoldDragon,
Currently we are working to enhance the syntax highlighting and we definitely will take this into account.
Thanks for reporting!


[RadioGroup] 'required' prop doesn't do anything

Bug report
Current Behavior
I'm adding the required prop but it doesn't trigger any native input validation.

As you can see in the screenshot, it skips over the radio group and stops on the validation of the text input.
Expected behavior
I expect a native validation message on the radio group.

 I figured out you must have a name specified on the RadioGroup.


InvalidArgumentError (see above for traceback): No OpKernel was registered to support Op 'BilateralSliceApply' with these attrs.  Registered devices: [CPU], Registered kernels:   device='GPU'

I have trouble training the network.
cudnn 5.1.0   tensorflow-gpu 1.1.0
how can i fix it?
Thanks
[1010] INFO train.py:113 | Adding 2 update ops
2017-09-11 23:24:52.279962: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.1 instructions, but these are available on your machine and could speed up CPU computations.
2017-09-11 23:24:52.279988: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.2 instructions, but these are available on your machine and could speed up CPU computations.
2017-09-11 23:24:52.280004: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use AVX instructions, but these are available on your machine and could speed up CPU computations.
2017-09-11 23:24:52.280007: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use AVX2 instructions, but these are available on your machine and could speed up CPU computations.
2017-09-11 23:24:52.280010: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use FMA instructions, but these are available on your machine and could speed up CPU computations.
2017-09-11 23:24:56.351927: E tensorflow/stream_executor/cuda/cuda_driver.cc:405] failed call to cuInit: CUDA_ERROR_NO_DEVICE
2017-09-11 23:24:56.352006: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:158] retrieving CUDA diagnostic information for host: im-gpuws01
2017-09-11 23:24:56.352018: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:165] hostname: im-gpuws01
2017-09-11 23:24:56.352081: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:189] libcuda reported version is: 375.66.0
2017-09-11 23:24:56.352141: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:369] driver version file contents: """NVRM version: NVIDIA UNIX x86_64 Kernel Module  375.66  Mon May  1 15:29:16 PDT 2017
GCC version:  gcc version 5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.4)
"""
2017-09-11 23:24:56.352180: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:193] kernel reported version is: 375.66.0
2017-09-11 23:24:56.352191: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:300] kernel version seems to match DSO: 375.66.0
Traceback (most recent call last):
File "/home/rox/hdrnet-master/hdrnet/bin/train.py", line 259, in
main(args, model_params, data_params)
File "/home/rox/hdrnet-master/hdrnet/bin/train.py", line 150, in main
with sv.managed_session(config=config) as sess:
File "/home/rox/anaconda2/envs/hdrnet/lib/python2.7/contextlib.py", line 17, in enter
return self.gen.next()
File "/home/rox/anaconda2/envs/hdrnet/lib/python2.7/site-packages/tensorflow/python/training/supervisor.py", line 960, in managed_session
self.stop(close_summary_writer=close_summary_writer)
File "/home/rox/anaconda2/envs/hdrnet/lib/python2.7/site-packages/tensorflow/python/training/supervisor.py", line 788, in stop
stop_grace_period_secs=self._stop_grace_secs)
File "/home/rox/anaconda2/envs/hdrnet/lib/python2.7/site-packages/tensorflow/python/training/coordinator.py", line 389, in join
six.reraise(*self._exc_info_to_raise)
File "/home/rox/anaconda2/envs/hdrnet/lib/python2.7/site-packages/tensorflow/python/training/supervisor.py", line 949, in managed_session
start_standard_services=start_standard_services)
File "/home/rox/anaconda2/envs/hdrnet/lib/python2.7/site-packages/tensorflow/python/training/supervisor.py", line 706, in prepare_or_wait_for_session
init_feed_dict=self._init_feed_dict, init_fn=self._init_fn)
File "/home/rox/anaconda2/envs/hdrnet/lib/python2.7/site-packages/tensorflow/python/training/session_manager.py", line 262, in prepare_session
sess.run(init_op, feed_dict=init_feed_dict)
File "/home/rox/anaconda2/envs/hdrnet/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 778, in run
run_metadata_ptr)
File "/home/rox/anaconda2/envs/hdrnet/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 982, in _run
feed_dict_string, options, run_metadata)
File "/home/rox/anaconda2/envs/hdrnet/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1032, in _do_run
target_list, options, run_metadata)
File "/home/rox/anaconda2/envs/hdrnet/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1052, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InvalidArgumentError: No OpKernel was registered to support Op 'BilateralSliceApply' with these attrs.  Registered devices: [CPU], Registered kernels:
device='GPU'
     [[Node: eval/inference/output/slice/BilateralSliceApply = BilateralSliceApply[has_offset=true, _device="/device:GPU:0"](eval/inference/output/slice/Reshape, eval/inference/guide/Squeeze, train_data/shuffle_batch)]]

Caused by op u'eval/inference/output/slice/BilateralSliceApply', defined at:
File "/home/rox/hdrnet-master/hdrnet/bin/train.py", line 259, in
main(args, model_params, data_params)
File "/home/rox/hdrnet-master/hdrnet/bin/train.py", line 105, in main
model_params, is_training=False)
File "/home/rox/hdrnet-master/hdrnet/models.py", line 60, in inference
fullres_input, guide, bilateral_coeffs)
File "/home/rox/hdrnet-master/hdrnet/models.py", line 200, in _output
out = bilateral_slice_apply(coeffs, guide, im, has_offset=True, name='slice')
File "/home/rox/hdrnet-master/hdrnet/layers.py", line 147, in bilateral_slice_apply
sliced = hdrnet_ops.bilateral_slice_apply(grid, guide, input_image, has_offset=has_offset)
File "", line 82, in bilateral_slice_apply
File "/home/rox/anaconda2/envs/hdrnet/lib/python2.7/site-packages/tensorflow/python/framework/op_def_library.py", line 768, in apply_op
op_def=op_def)
File "/home/rox/anaconda2/envs/hdrnet/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 2336, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/home/rox/anaconda2/envs/hdrnet/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 1228, in init
self._traceback = _extract_stack()
InvalidArgumentError (see above for traceback): No OpKernel was registered to support Op 'BilateralSliceApply' with these attrs.  Registered devices: [CPU], Registered kernels:
device='GPU'
     [[Node: eval/inference/output/slice/BilateralSliceApply = BilateralSliceApply[has_offset=true, _device="/device:GPU:0"](eval/inference/output/slice/Reshape, eval/inference/guide/Squeeze, train_data/shuffle_batch)]]

@mgharbi    could you help me ?Thanks

I have gpu environment and surely have read  https://github.com/mgharbi/hdrnet_legacy/issues/5#issuecomment-323860996.
but i still can not fix my problem.


[AIRFLOW-2167] cleanup nonexistent import errors on the first iteration in addition to periodically

Make sure you have checked all steps below.
JIRA

[x] My PR addresses the following AIRFLOW-2167 issues and references them in the PR title. For example, "[AIRFLOW-2167] cleanup nonexistent dags on the first iteration in addition to periodically"

https://issues.apache.org/jira/browse/AIRFLOW-2167

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

This PR tweaks airflow/jobs.py to cleanup nonexistent import errors on the first iteration of the scheuduler and periodically as defined by dag_dir_list_interval.  This way, a scheduler setup with a small number of runs will still have old errors cleaned up.
Previously, a scheduler would only run this cleanup after the number of seconds in dag_dir_list_interval had elapsed.  For schedulers defined with a small number of runs, this meant the cleanup never occurred.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

extremely good reason:  This is a very minor code tweak.  No new functionality is added, existing functionality is only triggered one additional time.

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

[x] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Codecov Report

Merging #3088 into master will decrease coverage by 0.09%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master   #3088     +/-   ##
========================================
- Coverage   73.29%   73.2%   -0.1%
========================================
  Files         179     179
  Lines       12500   12499      -1
========================================
- Hits         9162    9150     -12
- Misses       3338    3349     +11

Impacted Files
Coverage Δ

airflow/jobs.py
79.62% <100%> (-0.47%)
:arrow_down:

airflow/task/task_runner/bash_task_runner.py
93.33% <0%> (-6.67%)
:arrow_down:

airflow/utils/helpers.py
51.14% <0%> (-2.88%)
:arrow_down:

Continue to review full report at Codecov.

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

@caseybrown89  thanks for your contribution. the change lgtm with one nit-picking comment.
Also I believe you don't need to restart the scheduler every n runs (I do recall this used to be a bug, but I believe it is no longer the case). Curious any particular reason for this?

Also I believe you don't need to restart the scheduler every n runs (I do recall this used to be a work-around for a bug, but I believe it is no longer the case). Curious any particular reason for this?

@jgao54 I think we're still doing this out of habit.  Should this be maintained as a scheduler option if it's not intended to be used?

@caseybrown89 there may be corner cases that people are using it so it doesn't hurt to maintain it, but definitely not required to put a lifetime on the scheduler afaik.

@caseybrown89  are you still working on this PR?


sqla how to set schema of table

How to define the schema of {schema}.{table} in luigi.contrib.sqla.CopyToTable? Do I need to update the class to allow custom metadata class or __table_args__?

I've made changes to add self.metadata = sa.MetaData(), but this will need some testing, any other thoughts on this would be appreciated.

Are you asking about the table and column definitions?
Here's an example of a table definition using sqla.CopyToTable from my own usage.
class LoadInspectionKPI(CopyToTable):
    columns = [
        (["Inspection_Date", Date()], {}),
        (["Inspection_Person", String(120)], {}),
        (["Vendor", String(250)], {}),
        (["Carton_ID", Integer()], {}),
        (["Serial_Number_Count", Integer()], {})
    ]
    table = 'inspection_kpi'

If you actually want work right with the MetaData you can bind it like this...
(snip from on of my classes that override run() in CopyToTable
output = self.output()
engine = output.engine
meta = MetaData()
meta.reflect(bind=engine)

When I refer to schema I'm referring to a database schema {database}.{schema}.{table}, not the table definition.
Like when using MetaData(schema='schema_name'), which is why I opted to change the class to allow for custom MetaData class versus redefining the run() for every CopyToTable Task.

Much needed indeed!

closing this issue.

[x] It has been inactive for +4 months.
[x] It's not about luigi core, so not as many users are affected about this.
[ ] The change seems quite big, it's unlikely to be sporadically picked up.
[x] The owner haven't responded or disappeared.
[ ] I don't understand what this issue is about.
[ ] There exists a reasonable workaround for this.
[ ] We need to check if this hasn't been fixed by now (for old issues).
[ ] This is kind of by design and not a bug.
[ ] Resolving this would probably add a lot of complexity.
[ ] This is a question not an issue. Use SO or the mailing list.

Every open issue adds some clutter, and we try to make the issues fewer and make it easier for new collaborators to find. Currently we try to close any issue that meets the first checkbox + one other.
Feel free to reopen this issue at any point if you have the intent to continue to work this. :)


How to set the selected item in the dropdown programatically?

I'm trying to set the selected item in the dropdown programmatically when the page loads via a sessionStorage variable that I save when the user originally selects an item, but i'm not having success. Any ideas on what i'm doing wrong?

      <ui-select ng-model="users.selectedUser" theme="select2" on-select="impersonate($item, $model)">
        <ui-select-match placeholder="Impersonate">{{$select.selected.email}}</ui-select-match>
        <ui-select-choices repeat="person in allUsers | filter: $select.search track by person.email">

        </ui-select-choices>
      </ui-select>

'use strict';

angular.module('myapp').controller('NavbarCtrl', function ($scope, Messages, Users, $location, $rootScope,) {

  $scope.impersonate = function(item, model) {

    // Set the selectedUser so that when the page is refreshed we can still display the user in the dropdown
    sessionStorage.user = model;
        $location.path('/');  // Redirect to dashboard
    );
  };

  /**
   * Gets all the users
   */
  var getAllUsers = function() {
  	Users.query(function(users) {
  		$scope.allUsers = users;
  	});
  };

  $scope.users = {};  // Set this so we can get the selected user from the view

  // If a user has been selected previously, we want to show that user in the dropdown when the page is loaded
  if(sessionStorage.user) {
    $scope.users.selectedUser = sessionStorage.user;
  }

  console.log('$scope.users.selectedUser = ');
  console.log($scope.users.selectedUser);

  getAllUsers(); // Call on page load

});

I finally figured it out. The problem was in code that I hadn't actually posted. I was trying to save a JSON object to sessionStorage and apparently you have to stringify it first and then parse it when you retrieve it from sessionStorage..


Use Alpha-2 code as 'country'

To follow the homogeneous use in the rest of the Countries, I propose to modify the value of 'country' by the Alpha-2 code of the ISO-3166 standard.
https://www.iso.org/glossary-for-iso-3166.html

Wrong branch.

Wrong branch.


Bring changes from main to rc-branch

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


ReliableTopicTest.testMessageFieldSetCorrectly occasionally crashes on github actions. [API-1809]

C++ compiler version: msvc-14.2
Hazelcast Cpp client version: 5.0.0
Hazelcast server version: 5.1.3
Number of the clients: -
Cluster size, i.e. the number of Hazelcast cluster members: -
OS version (Windows/Linux/OSX): Windows
Please attach relevant logs and files for client and server side.
Expected behaviour
Pass test
Actual behaviour
Crashes on test
Steps to reproduce the behaviour
It might be a different test but this is what I suspected.
Here is the link.
ReliableTopicTest_testMessageFieldSetCorrectly.txt

This failed again on Test CPP client v5.1.0 with enterprise server in the client compat. tests of 5.3.3 HZ release. https://github.com/hazelcast/client-compatibility-suites/actions/runs/6535724054/job/17771144765#logs
[ RUN      ] ReliableTopicTest.testMessageFieldSetCorrectly
08:42:58,116  INFO [Main] - Ping ... Pong ...
Oct 17, 2023 8:42:58 AM com.hazelcast.internal.server.tcp.TcpServerConnection
WARNING: [localhost]:5701 [dev] [5.3.3-SNAPSHOT] Connection[id=1, /<IP_ADDRESS>:5701->/<IP_ADDRESS>:35360, qualifier=null, endpoint=[<IP_ADDRESS>]:35360, remoteUuid=943d04dc-d7d5-4720-a84f-882823ece7cf, alive=false, connectionType=CPP, planeIndex=-1] closed. Reason: Exception in Connection[id=1, /<IP_ADDRESS>:5701->/<IP_ADDRESS>:35360, qualifier=null, endpoint=[<IP_ADDRESS>]:35360, remoteUuid=943d04dc-d7d5-4720-a84f-882823ece7cf, alive=true, connectionType=CPP, planeIndex=-1], thread=hz.nifty_faraday.IO.thread-in-0
java.io.IOException: Connection reset by peer
	at sun.nio.ch.FileDispatcherImpl.read0(Native Method)
	at sun.nio.ch.SocketDispatcher.read(SocketDispatcher.java:39)
	at sun.nio.ch.IOUtil.readIntoNativeBuffer(IOUtil.java:223)
	at sun.nio.ch.IOUtil.read(IOUtil.java:197)
	at sun.nio.ch.SocketChannelImpl.read(SocketChannelImpl.java:379)
	at com.hazelcast.internal.networking.nio.NioInboundPipeline.process(NioInboundPipeline.java:115)
08:42:58,156  INFO [Main] - Cleaning the R.C. Cluster Manager
08:42:58,156  INFO [Main] - Shutting down the cluster.
	at com.hazelcast.internal.networking.nio.NioThread.processSelectionKey(NioThread.java:383)
	at com.hazelcast.internal.networking.nio.NioThread.processSelectionKeys(NioThread.java:368)
	at com.hazelcast.internal.networking.nio.NioThread.selectLoop(NioThread.java:294)
	at com.hazelcast.internal.networking.nio.NioThread.executeRun(NioThread.java:249)
	at com.hazelcast.internal.util.executor.HazelcastManagedThread.run(HazelcastManagedThread.java:111)

Oct 17, 2023 8:42:58 AM com.hazelcast.client.impl.ClientEndpointManager
INFO: [localhost]:5701 [dev] [5.3.3-SNAPSHOT] Destroying ClientEndpoint{connection=Connection[id=1, /<IP_ADDRESS>:5701->/<IP_ADDRESS>:35360, qualifier=null, endpoint=[<IP_ADDRESS>]:35360, remoteUuid=943d04dc-d7d5-4720-a84f-882823ece7cf, alive=false, connectionType=CPP, planeIndex=-1], clientUuid=943d04dc-d7d5-4720-a84f-882823ece7cf, clientName=hz.client_114, authenticated=true, clientVersion=5.1.0, creationTime=1697532176068, latest clientAttributes=null, labels=[]}
Oct 17, 2023 8:42:58 AM com.hazelcast.core.LifecycleService
INFO: [localhost]:5701 [dev] [5.3.3-SNAPSHOT] [localhost]:5701 is SHUTTING_DOWN
Oct 17, 2023 8:42:58 AM com.hazelcast.internal.partition.impl.MigrationManager
INFO: [localhost]:5701 [dev] [5.3.3-SNAPSHOT] Shutdown request of Member [localhost]:5701 - bfb83d8a-de3c-4421-9bfe-2026e6a52c3d this is handled
Oct 17, 2023 8:42:58 AM com.hazelcast.instance.impl.Node
INFO: [localhost]:5701 [dev] [5.3.3-SNAPSHOT] Shutting down connection manager...
Oct 17, 2023 8:42:58 AM com.hazelcast.instance.impl.Node
INFO: [localhost]:5701 [dev] [5.3.3-SNAPSHOT] Shutting down node engine...
Oct 17, 2023 8:42:58 AM com.hazelcast.instance.impl.NodeExtension
INFO: [localhost]:5701 [dev] [5.3.3-SNAPSHOT] Destroying node NodeExtension.
Oct 17, 2023 8:42:58 AM com.hazelcast.instance.impl.Node
INFO: [localhost]:5701 [dev] [5.3.3-SNAPSHOT] Hazelcast Shutdown is completed in 14 ms.
Oct 17, 2023 8:42:58 AM com.hazelcast.core.LifecycleService
08:42:58,173  INFO [Main] - TServerEventHandler.deleteContext client
INFO: [localhost]:5701 [dev] [5.3.3-SNAPSHOT] [localhost]:5701 is SHUTDOWN
./scripts/test-unix.sh: line 60: 38562 Segmentation fault      (core dumped) ${TEST_EXECUTABLE} --gtest_output="xml:CPP_Client_Test_Report.xml"
Test FAILED. Result:139


Pointer double free issue with ImagePrefetcher

Check List
Thanks for considering to open an issue. Before you submit your issue, please confirm these boxes are checked.

[x] I have read the wiki page and cheat sheet, but there is no information I need.
[x] I have searched in existing issues, but did not find a same one.
[x] I want to report a problem instead of asking a question. It'd better to use kingfisher tag in Stack Overflow to ask a question.

Issue Description
What
Crashed when prefetching images in a row.
Reproduce
You can test it on the latest Kingfisher codebase in ImagePrefetcherTests.swift.
func testPrefetchMultiTimes() {
        testURLs.forEach { stub($0, data: testImageData) }

        for _ in 0..<10000 {
            let prefetcher = ImagePrefetcher(resources: testURLs)
            prefetcher.start()
        }
    }

You may need to run it serveral times before the crashed happened.
Other comment

Each prefetcher instance runs on an independent serial queue, while they are using the same image downloader. When downloading images, the downloader concurrently overwrites the same task's onTaskDone delegate's block, which cause a double free issue.

Thanks for reporting this. I will check it as soon as I can.

Should be fixed by making all prefetcher use the same queue.

Will the new version release in coming days?

5.1.1 was released.


Where can i download models by hand?

i want to download models by hand because  Download failed: A connection error occurred
is there any url i can downlaod models?
Whisper model file: /Users/UserName/.cache/whisper
Whisper.cpp file: /Users/UserName/Library/Caches/Buzz
Faster Whisper model file: /Users/UserName/.cache/huggingface/hub

Having same issue here. Please help.

You can get the modesl on HuggingFace:

Whisper.cpp
faster-whisper stores models in different repos, you can get large-v2 model here

Remember to rename the model. Check logs (on Windows, it's %LOCALAPPDATA%\Buzz\Buzz\Logs) for the desired path (on Windows, mine is C:\Users\USERNAME\.cache\whisper, but it used to be C:\Users/USERNAME\AppData\Local\Buzz\Buzz\Cache) & filename. You can see the direct download URL by the way.
I think it's also stored in the code:
https://github.com/chidiwilliams/buzz/blob/a3b54891e8c403d28b7bf2d86443ad310d2d1b51/buzz/model_loader.py#L108-L118


Submit to WinGet.

Request
File a new package request for Gittyup at https://github.com/microsoft/winget-pkgs/issues/new?assignees=&labels=Help-Wanted%2CPackage-Request&projects=&template=package_request.yml&title=[Package+Request]%3A+ for https://github.com/Murmele/Gittyup/releases/download/gittyup_v1.4.0/Gittyup-win64-1.4.0.exe. You can cite https://www.virustotal.com/gui/url/4be9ee273d6b0d66791d48e3c5411edd079ca549011fcc1e391d51062e8a8440.
I'll do so if nobody else does.
Rationale
It's Windows 10 and 11's default and official package manager.
(Lack of) Duplicates
https://github.com/Murmele/Gittyup/issues?q=sort%3Aupdated-desc+is%3Aissue+is%3Aopen+winget and https://github.com/microsoft/winget-pkgs/issues?q=sort%3Aupdated-desc+is%3Aissue+is%3Aopen+gittyup return 0 results.

I don't have any problems with it. Thank you!

https://github.com/Murmele/Gittyup/issues/801#issuecomment-2371108352

@Murmele, submitted at https://github.com/microsoft/winget-pkgs/issues/174936#issue-2545260707. Thanks.


1.0.0-rc.0 release

Added pod pause and resume support
Added bind mounts support
Added CRI-O support
Added QEMU Q35 machine type support
Added readonly rootfs support
Added annotations passing to the OCI config file
Fixed hyperstart users and groups management
Fixed the installation and build scripts
Fixed networking MAC address configuration
Shortlog:
0b08b32 api: Implement PausePod() and ResumePod().
0e47e70 pod: Make state handling functions set in-memory state.
d2e6aaa mounts: Add function to perform unmounts
3719480 mounts: Store mounts after bind-mounts are handled
63cbcca state: Add functionality to store/fetch mounts to/from filesystem.
a249fde test: Add test for checking if a path is system mount
0ef264e mounts: Handle bind mounts
90ef63f container: Add comments for setContainerState().
6b8f0b5 tests: Add test to verify mounts are parsed and stored
a98a913 mounts: Parse and store mounts from oci spec file.
53b7cad vendor: Add CRI-O vendoring to our list of dependencies
c53cfa8 oci: Implement CRI-O support
bdefe47 hyperstart: Limit the hostname length to prevent from hyperstart errors
f663cb2 hyperstart: Prevent container rootfs from being mounted readonly
c3a4570 hyperstart: User and groups are not handled properly
b25540e container: Get pod handler from any container
c611050 network: Determine if netns was provided or created
ab994f0 api: Execute poststart and poststop hooks outside of netns
2e3765e shim: Stop the shim if container has only been created
2ccf287 api: Remove the session ending call
181d371 logging: Use the logrus.FieldLogger interface in SetLogger
a46174b logging: Rename SetLog to SetLogger
64b2808 utils: Fix the script installing CNI plugins
bf18ba1 test: Add test to verify the readonly config for rootfs is assigned
2c6a037 oci: Pass the read-only config from the oci spec file
6b7c753 tests: Use defer with call to unmount
725de4d api: Add 'all' parameter to KillContainer().
b42fa98 tests: Add test to verify read-only bind mounts
b51fd53 rootfs: Support mounting the rootfs as readonly
8d49dfc oci: copy container's annotations to pod
8155768 pod: Adds RW lock to access pod's annotations
c1cb7eb pod: add function to get pod's annotations
8509c34 pod: add function to set or add annotations
148ba85 container: Add function to get container's annotations
449497a vendor: Update runtime-spec to v1.0.0-rc5 to be CRI-O compliant
9e7005c vendor: Update all vendored repositories
a43b84a tty: Set the console as the controlling terminal for shim
122b44e build: Make mock binary paths dynamically generated by Makefile
4704c5f travis: Enhance travis.yml
bb2d085 ci: Introduce go static and go test scripts
a587134 build: Enhance Makefile to build and install all binaries
22da7fd ci: Add semaphore scripts to simplify semaphore settings
2d8cecc oci: Make OCI config file and bundle directory paths public.
36faae0 container: Create function to create a Process.
7f7e7b6 tests: Fix-up for Container start time.
07ff99a api: Remove redundant call to fetchPod().
bf695ef api: Add HypervisorConfig to PodStatus.
70c7917 pod: rename GetContainers to GetAllContainers
3ad4c50 api: Make StatusContainer() include container start time.
c0e2098 pod: Add function to get a specific container
ff66ff4 container: Remove duplication for shim and process handling.
c2b3139 api: Make StatusPod() call StatusContainer().
e3e6f68 oci: Return correct Bundle path in StatusToOCIState().
60138bd oci: Add annotations to OCI State object.
38f24d4 oci: Add bundlepath to container Annotations.
b566e59 oci: Move OCI config path from pod to container annotations.
e2e5f8d annotations: Add annotations to ContainerStatus.
5a2b64d annotations: Add annotations to PodStatus.
24fd25f OWNERS: Add OWNERS files
bd0e479 oci: Rename Group to PrimaryGroup
00ef6ed tests: Change test config to test for additional gids
aa52eb4 oci: Add support for additional groups while running a container
df324ac network: Factorize common code between network implementations
9726597 network: Fix wrong MAC address configuration
fcceae7 README: Change shim path in virtc README
7dd546e networking: Turn off multicast snooping on integration bridge
782e907 CONTRIBUTING: Fix the md links
267bb85 vendor: Update vendoring with latest dep tool version
5d33348 network: Fix benchmarks sporadic issues with netns
702b16f README: Fix proxy compilation instructions in README
d37c723 README: Fix shim compilation instructions in README
fbc84a7 readme: Add Semaphore CI build badge
6081e64 virtc: Default QEMU machine type is pc-lite
be0f200 hypervisor: Missing a configured path is not an error
a9cf7a7 qemu: Build the QEMU binary path
4c185df virtc: Add "--machine-type=" option.
5817687 qemu: Add support for 'q35' machine type.
e614a37 hypervisor: Add machine type to HypervisorConfig.
b267d75 qemu: Use virtio-net-pci for our network driver
c1338cc hyperstart: Copy pause binary instead of creating a symbolic link
4d69498 qemu: Wait for the end of the VM
2b81883 qemu: Set script and downscript to "no" by default
bd08145 pod: Fix typo.
008eeb9 oci: Fix annotation namespacing.
72cb6df hyperstart: Do not use hard links
e934235 api: Speed up container status query.
Signed-off-by: Samuel Ortiz<EMAIL_ADDRESS>

lgtm

Coverage remained the same at 66.008% when pulling d1f3686b13f038d92b20e345c588c8621f4a6a03 on topic/releases into 7ecd41d58a948f4302fb9f1ef3aea1235e98053e on master.

Coverage remained the same at 66.008% when pulling d1f3686b13f038d92b20e345c588c8621f4a6a03 on topic/releases into 7ecd41d58a948f4302fb9f1ef3aea1235e98053e on master.


Add Holidays

Implement a Holiday field which identifies which days are holidays. This will allow insertion of "Holiday" dates within the DTR instead of leaving them blank or manually changing them. Frontend view must result to this:

Please note that the "TIANGGE DAY" should be replaced with "HOLIDAY" and should have black font color and yellow background color, resulting to this:

ALSO, ITALICIZE THE WORD. LMAO

Commit 71d4a4e, which is also the v1.2.0-beta release addressed and implemented this feature by modifying the Dragcheck utility script from DTR-Generator unto a full blown library. Similar on how to select regular work dates, you can just right-click the dates to select which of them are holidays. Some changes in the DTRG base code are also done to implement the "Holiday" row.
As the feature is already tested and deemed working, this issue will now be closed.


build: switch gyb to Python3

Change the build system to invoke gyb with python3 instead of python2.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @stephentyrone @drodriguez @tachoknight @edymtt @shahmishal @JDevlieghere @gwynne

@swift-ci please clean test

CC: @nathawes - you may also be interested in this since SwiftSyntax depends on gyb'ed output

Build failed
Swift Test Linux Platform
Git Sha - 05330c474d5a4cc79c1ce92783cb58e1f6401e76

Build failed
Swift Test OS X Platform
Git Sha - 05330c474d5a4cc79c1ce92783cb58e1f6401e76

@swift-ci please test macOS platform

@swift-ci please test Linux platform

Build failed
Swift Test Linux Platform
Git Sha - 05330c474d5a4cc79c1ce92783cb58e1f6401e76

Build failed
Swift Test OS X Platform
Git Sha - 05330c474d5a4cc79c1ce92783cb58e1f6401e76

@swift-ci please test Linux platform

Build failed
Swift Test Linux Platform
Git Sha - 05330c474d5a4cc79c1ce92783cb58e1f6401e76

@swift-ci please test Linux platform

@nathawes I can't seem to figure out why the tests on macOS are failing with this change.  Happen to have any hints on why the tests are hitting this error:
The loaded '_InternalSwiftSyntaxParser' library is from a toolchain that is not compatible with this version of SwiftSyntax
Run lit-test-helper -help for more help.

swift-syntax repo also invokes gyb via its build-script to generate its swift code. def calculate_node_hash() is used to ensure swift-side parser and swift-syntax library are compatible.
If swift-side uses python3 and that function generates different hash than python2 from swift-syntax, there will be a mismatch.

@akyrtzi ah, thanks for that explanation.  I'll try to get swift-syntax synced up then

@swift-ci please test macOS platform

Build failed
Swift Test OS X Platform
Git Sha - 05330c474d5a4cc79c1ce92783cb58e1f6401e76

@swift-ci please test macOS platform

Build failed
Swift Test OS X Platform
Git Sha - 05330c474d5a4cc79c1ce92783cb58e1f6401e76

@swift-ci please clean test

Build failed
Swift Test Linux Platform
Git Sha - 05330c474d5a4cc79c1ce92783cb58e1f6401e76

Build failed
Swift Test OS X Platform
Git Sha - 05330c474d5a4cc79c1ce92783cb58e1f6401e76

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 102621acb9160017be31d16c2b05c6c0e54c5a62

@swift-ci Please Python lint

@swift-ci Please Python lint

@swift-ci please test

@swift-ci please test

@swift-ci please test Windows platform

/cc @shahmishal

Thanks @compnerd!


Fix display issue when field has lost focus

You can repro this issue on any external ajax request with a slow connection (go to network settings and throttle traffic to simulate).  If the user gets impatient and tabs off the field, when the AJAX request returns it will still show the display list even when the field no longer has focus.  It seems like a reasonable limitation and check to ensure the current field has focus immediately prior to show the list container.
See How show easyautocomplete dialog only if cursor is in input for more info

@KyleMit Thanks for the contribution to this repository 👍


removed reason param from cluster::guild_member_kick

This now has conflicts, as cluster_sync_calls was split into a .h and .cpp. can you resolve it please? :) i'll then merge

Yep. Ill fix it later


Added more colors and fixed color overlap

There was new (or unused) options for received and sent reply colors and some other accent color.
The color overlap i saw was the button for going back down (to the latest message) in a chat.
Also you can edit hex code directly in the app if you enable dev options

i remembered this is not how its done so ill be back


fix: refactored pg-core

added setLoggerAndExecutor for set execute fn
updated qb compareSql
added createMany fn

:tada: This PR is included in version 3.32.0-next.2 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Corrected the tag links in list and show views

Coverage remained the same at 77.752% when pulling 32759a4591462048791fbd48f6f9adfa6caf7da1 on divyanshumehta:exp into 4af5da04e9db9e18c78c7551b57fa51810f5d354 on NIT-dgp:github_master.

@divyanshumehta Can you use the selectize classes in place of simple bootstrap ones?

I dont think selectize offers class to display tags. So we will have to use labels only went through their documentation, though it offers some customizations in the tag input fields, but none for displaying the tags. So I think for now we will need to use label class only

Sorry for taking too long to merge this PR. I have made some tweaks and merged it manually in the master.


Allow branch deletion through the GUI in explorer

A Feature Request.
Right now the context menu on right click on a branch name in repository view only shows

Refresh for local branches
Refresh and Open Branch in Remote for remote branches

Can we add another option(s)

Delete Local Branch for local branches
Delete Local Branch and Delete Remote Branch for remote branches

I don't know how useful it will be to others but I personally hate to type out the commands every time in terminal(not a big terminal user I am).
...Also thanks everyone for the great tool.

GitLens 6 is out now with new commands that are run in the terminal -- let me know what you think.

That's Awesome


forever some/script/with/slashes.js does not work

nt. May be a regression in flatiron.

This is fixed in<EMAIL_ADDRESS>


fix readme bug

PR types
Bug fixes
PR changes
Models
Describe
update install shell in README.md

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.root seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Could you provide a Docker image file? Howdown cannot be installed successfully.

I appreciate your willingness to help, and I apologize for any confusion.
Since I'm not currently in a Docker environment, I can attempt to assist you further if you provide a more detailed description of the issue you're facing.
Please share any error messages or specific challenges you've encountered, and I'll do my best to help you troubleshoot the problem.

I appreciate your willingness to help, and I apologize for any confusion.
Since I'm not currently in a Docker environment, I can attempt to assist you further if you provide a more detailed description of the issue you're facing.
Please share any error messages or specific challenges you've encountered, and I'll do my best to help you troubleshoot the problem.

After replacing bark_ssg with pip install bark, I was able to run the program.Then I found that the name of the encoder in Hugging Face is different from that in the yaml file:image_encoder_path: "pretrained/CLIP-ViT-H-14-laion2B-s32B-b79K". After modifying it, the following error was reported

Firstly, it appears that you have not installed the accelerate package. Installing it can save GPU memory and accelerate inference.
Secondly, based on the provided error information, potential issues can be classified into two scenarios:

If you did not use the default configuration file, configs/vlog_read_script_sample.yaml, for sample_scripts/vlog_read_script_sample.py and instead made modifications to the script file, you should compare it with the scripts in the repository I provided. Check for any differences in formatting, as content generated by GPT may occasionally deviate from the interface I designed.

If you used the default configuration file, configs/vlog_read_script_sample.yaml, it's possible that you accidentally modified some scripts within it. In this case, you can directly download and replace the original files with the script files I provided.

Firstly, it appears that you have not installed the accelerate package. Installing it can save GPU memory and accelerate inference.
Secondly, based on the provided error information, potential issues can be classified into two scenarios:

If you did not use the default configuration file, configs/vlog_read_script_sample.yaml, for sample_scripts/vlog_read_script_sample.py and instead made modifications to the script file, you should compare it with the scripts in the repository I provided. Check for any differences in formatting, as content generated by GPT may occasionally deviate from the interface I designed.
If you used the default configuration file, configs/vlog_read_script_sample.yaml, it's possible that you accidentally modified some scripts within it. In this case, you can directly download and replace the original files with the script files I provided.

I did not modify the video_prompts.txt。Could you tell me the version number of your ast?

My Python interpreter version is 3.10.11, and the ast version corresponds to Python.

Firstly, it appears that you have not installed the accelerate package. Installing it can save GPU memory and accelerate inference.
Secondly, based on the provided error information, potential issues can be classified into two scenarios:

If you did not use the default configuration file, configs/vlog_read_script_sample.yaml, for sample_scripts/vlog_read_script_sample.py and instead made modifications to the script file, you should compare it with the scripts in the repository I provided. Check for any differences in formatting, as content generated by GPT may occasionally deviate from the interface I designed.
If you used the default configuration file, configs/vlog_read_script_sample.yaml, it's possible that you accidentally modified some scripts within it. In this case, you can directly download and replace the original files with the script files I provided.

I did not modify the video_prompts.txt。Could you tell me the version number of your ast?

My Python interpreter version is 3.10.11, and the ast version corresponds to Python.


update broken links for screencasts

and add notes about content being slightly different (since they were for the 2023 workshop)

note @gavin this data was stored in the repo for the wiki which is different than the workshop repo.  If you look to the right of the wiki there will be a link to clone the wiki locally

I used this method to store the screenshots and screencasts rather than embedding them in issues so the files would be centralized with the content.

all links fixed


[action] new create_xcframework action

Note ❗ This PR is mutually exclusive with https://github.com/fastlane/fastlane/pull/17837
Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context

Resolves #17836
Conflict with #17837: only one should be merged into master
Description

I have created a new create_xcframework action, separated from the xcodebuild action, although this action also generates a xcodebuild command. I think it made more sense to do it this way as the set of parameters of -create-xcframework is quite small and completely independent from the rest of parameters of xcodebuild (I think only -output it's used in other commands)
I'm rather new to Ruby, so please feel free to comment anything about the code (without hurting my delicate developer heart ❤😝 ). Specifically, the -create-xcframework action supports a list of libraries which may have an associated headers directory. I have modeled this input parameter as a hash with library paths as keys and headers directory paths as values, using an empty string in the case that a library doesn't have associated headers. If you can think of a better way of expressing this in Ruby just tell me!
Testing Steps

I have added a test suite providing tests for the most common scenarios.

bundle exec fastlane test

Hey @jgongo ! 👋
Thanks for opening this alternate PR! I haven't reviewed your code yet, but I'm glad you were able to stretch your Ruby skills 😎
I personally prefer this option so far, since it's more complete than the other one provided (it is a full solution, as opposed to partial), and keeps the code more modular. However, keep in mind fastlane's vision, specially regarding actions and plugins. For this reason, I'm not sure if this action will be approved to be merged into fastlane core. However, there's always space for community-maintained actions and plugins 🤗
I'll leave it up to @joshdholtz  to decide which PR to accept, and whether it should belong to this repo 😃
Once again, thank you for your contributions!! ❤

Ok, just let me know if this can be accepted into fastlane core or not. If not I'll publish it as a plugin. Thanks!!

Hi @joshdholtz sorry to insist, but I'm waiting for this to continue working in a project. Could you please tell me if this will be accepted into fastlane core? If not I can publish this as an independent action and declare a dependency to that... thanks, and sorry again.

Hi @joshdholtz sorry to insist, but I'm waiting for this to continue working in a project. Could you please tell me if this will be accepted into fastlane core? If not I can publish this as an independent action and declare a dependency to that... thanks, and sorry again.

@jgongo Hey! Sorry about the wait. Me and my wife just had a baby so I'm getting to things a little bit slower than usual. I do appreciate the friendly ping though! ❤
I'm hoping to get to reviewing more PRs this afternoon and another release cut soon there after.

@jgongo Hey! Sorry about the wait. Me and my wife just had a baby so I'm getting to things a little bit slower than usual. I do appreciate the friendly ping though! ❤
I'm hoping to get to reviewing more PRs this afternoon and another release cut soon there after.

Congratulations, @joshdholtz !! I wouldn't have bothered you if I knew!! 😄 I've been there, so take it easy and rest while the baby is sleeping 😉

Congratulations, @joshdholtz !! I wouldn't have bothered you if I knew!! 😄 I've been there, so take it easy and rest while the baby is sleeping 😉

@jgongo No worries at all! I put it as my GitHub status but I don't actual know how that gets used or seen 😝 And thank you!! ❤

@jgongo No worries at all! I put it as my GitHub status but I don't actual know how that gets used or seen 😝 And thank you!! ❤

You're welcome!! Thanks for accepting it!!

You're welcome!! Thanks for accepting it!!


The flow rate does not change as expected in keyviz

Please answer these questions before submitting your issue. Thanks!

What did you do?
If possible, provide a recipe for reproducing the error.

workload: oltp_insert, 200 threads
topology:  1pd + 1tikv * 1*tidb

What did you expect to see?

Flow is discontinuous and does not meet expectations

What did you see instead?
Flow is continuous.

What version of PD are you using (pd-server -V)?
4.0.0

/pick


Use Devlin's algorithm to identify even more cycles

Started doing it here, but was having a hard time using the code.

Devlin ran his tests on my KBase model (think it still had the acetate loop issue), and sent me the results: https://github.com/C-CoMP-STC/GEM-mit1002/blob/4d620ff91b5d07d17b00bd21ec6a3e0a4bfe9da8/macaw-results/helen-alteromonas_test-results.csv

I took the csv file that Devlin sent me, and I plotted the proportion of reactions had any entry other than "ok" for each test.

Not really sure what to do next, so I read Devlin's paper:

It is important to note that the tests in MACAW are designed to flag reactions that are worth examining for potential errors as well as reactions that may simply be near erroneous reactions, so the percentages listed here should not be interpreted as the percentage of reactions in each GSMM that contain one or more errors.

Don't want to go any further on the analysis on this until I run it on the correct version (ModelSeedPy) of the Amac model.

Ran on the (nongapfilled) ModelSEEDpy model.
What do the different columns mean?

dead_end_test

one or more metabolite IDs separated by semicolons indicating which dead-end metabolites participate in that reaction.
"only when going forwards" or "only when going backwards" if it was a reversible reaction that was not a dead-end but had a reactant or product that could only be consumed by all other reactions it participates in or only be produced by all other reactions it participates in.

dilution_test

"always blocked" if the reaction was incapable of non-zero fluxes regardless of whether or not any dilution constraints were imposed on the model.
Devlin's documentation says that the other options are "blocked by dilution" or "unblocked by dilution", but my results all have lists of metabolites

It looks like the metabolites listed are all in the reaction (as either reactants or products)
So I'm assuming that the reaction was blocked when that metabolite has a dilution reaction
Not sure how to tell if any of the metabolite lists came from "unblocked by dilution", but Devlin's documentation says that is rare

pathway

Combines all of the flagged reactions into a network, usually with several distinctly connected components that are not connected to one another.
The entry is a discrete integer indicating which connected component each reaction is in.
Reactions that were not flagged by any tests or not connected to any other reactions that were flagged by any tests (this only happens with reactions flagged by the dead-end or diphosphate tests, and is generally uncommon) are always assigned a "pathway" of 0.


Colors scheme problem in Matlab 2020a

After analyzing the POD modes for some PIV vector files, when trying to represent the data with contour flood (u, ,v, vorticity, etc.), the colormap scheme is altered from the default 'jet' scheme.
I used Matlab version 2019b on Mac OS 10.15.7.
This problem seems to not occur when I use Matlab 2016b.

Hi @bengida1989 - could you open me please a teamviewer to a machine with Matlab - will try to fix it.