Remove unwraps in execution_entry_point module

There was a todo message in the _execute method calling for the addition of error handling

Codecov Report

Merging #1137 (9680292) into main (677a657) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1137      +/-   ##
==========================================
- Coverage   89.53%   89.52%   -0.01%
==========================================
  Files          49       49
  Lines       13268    13262       -6
==========================================
- Hits        11879    11873       -6
  Misses       1389     1389

Files
Coverage Δ

src/execution/execution_entry_point.rs
94.50% <100.00%> (-0.08%)
:arrow_down:


Overlay barcode scanner in ionic view

I am having trouble on overlaying barcode scanner in ionic view.
Eg: i have a cordova-camera.html view with a header bar.
What i intend to achieve is to have the barcode scanner to stay in the view (with the header bar visible).
But my actual outcome is the barcode scanner take up the entire view.
How can it be achieved?

Hi @1nfinite , I would like to clarify, are you looking for something that is similar to camera preview but instead of camera, is it barcode?

Duplicate of https://github.com/phonegap/phonegap-plugin-barcodescanner/issues/223


Code quality fix - Null pointers should not be dereferenced.

This pull request is focused on resolving occurrences of Sonar rule
squid:S2259- Null pointers should not be dereferenced.
You can find more information about the issue here:
https://dev.eclipse.org/sonar/rules/show/squid:S2259
Please let me know if you have any questions.
Faisal Hameed

Coverage decreased (-0.1%) to 98.823% when pulling b3793963e9edf60c813a2bebb4db902de17bbcd8 on DevFactory:release/null-pointers-should-not-be-dereferenced-fix-1 into 1be03477b4749d21c7798c1b65d1ac9249abab4c on petitparser:master.

Thank you for the cleanup!


Fixed a copy-paste error in README.md.

The SimpleDB example had a reference to MongoDB variable. Changed to SimpleDB.

Merged, thanks


home: update top card with new slogan and additional button

closes #1483
closes #1503
Preview desktop:

Preview mobile:

i think "monero is money" might make more sense.

I like "Monero Means Money" more. It's more catchy and it's already around.

It was uppercase only on mobile. Fixed now.


Add missing RSpec aliases

Introducing xfeature, ffeature and fscenario

How does the test output look like:

Only Ruby 1.9.3 build (the second one? there are two builds on 1.9.3) fails. https://travis-ci.org/jnicklas/capybara/jobs/96986194
with message:
NoMethodError: undefined method `exclude_pattern=' for #<RSpec::Core::RakeTask:0x000000015f71b0`

Any assistance?

As a guess RSpec 2.2 doesn't support exclude_pattern=   -- don't break out these tests into a separate task and just test for the metadata in the test rather than that true == true.

@twalpole I have merged these tasks and refactored focus tag specs

rbx-2 uses rspec-core 3.4.1 and fails after removing rspec 2 aliases. need to investigate

Ok looks like the tests pass, please squash this down to one commit

Should I force push to bartuz/master or create new PR?

Just squash and force push

@twalpole thanks you assistance and patience! :)

Thanks a lot!


High CPU and RAM usage when tracking with CLI & GUI

Possibly related to #2896, when tracking another user from the CLI, the GUI opens a window just after the CLI prompts me with "Is this the  you wanted? [Y/n]". I clicked the button in the GUI to track, and thought I was done.
I came back to my computer this morning to find kernal_task running at 150% CPU and > 1GB RAM, and the CLI repeating these lines (13 times in all):
▶ INFO Checking for update, current version is 1.0.16-20160606110015+a548d0e
▶ INFO Using updater source: Remote (https://s3.amazonaws.com/prerelease.keybase.io)
▶ INFO Request "https://s3.amazonaws.com/prerelease.keybase.io/update-darwin-prod.json"
▶ INFO Checking update with version: 1.0.16-20160606110015+a548d0e
▶ INFO Update matches current version: 1.0.16-20160606110015+a548d0e = 1.0.16-20160606110015+a548d0e

I discovered an additional tracking window open behind another window confirming that the tracking was successful, and after clicking the confirmation button, the CLI gave me this:
▶ ERROR Failed to get track token%!(EXTRA libkb.IdentifyTimeoutError=Identification expired.)

Since then, another group of the updater lines have appeared in the terminal. Canceling the CLI ops dropped CPU usage but the memory is still high. I grabbed a spindump, but forgot to grab a sample of the Keybase process before I quit the GUI.
Keybase GUI Version: 1.0.16-20160606110015+a548d0e
OS X 10.11.5

can you run keybase log send if you see this in the future

Just ran it, but now sure how successful it was...
WARNING error opening log "/Users/morganestes/Library/Logs/keybase.updater.log": open /Users/morganestes/Library/Logs/keybase.updater.log: no such file or directory

Just after that it said it was complete, and gave me a log ID of 5ee8dbe7bfc965f6c984511c.

Thanks @morganestes, it worked!  I don't see anything weird in the logs though that would explain high CPU usage.  The kernel_task thing is pretty weird, hrm.
@patrickxb @maxtaco: can you think of anything weird that would happen if you click "Track" on the GUI popup, but leave the "[Y/n]" command line question unanswered?  Are there any locks held there, or constant polling back to the GUI?

There aren't any locks held. There is a token that captures the art aye
displayed to the user and on track, they token is cashed in.  But
definitely no busy waiting. Does the electron log show anything weird?
The track situation might also be unrelated.
On Friday, June 10, 2016, Jeremy Stribling<EMAIL_ADDRESS>wrote:

Thanks @morganestes https://github.com/morganestes, it worked! I don't
see anything weird in the logs though that would explain high CPU usage.
The kernel_task thing is pretty weird, hrm.
@patrickxb https://github.com/patrickxb @maxtaco
https://github.com/maxtaco: can you think of anything weird that would
happen if you click "Track" on the GUI popup, but leave the "[Y/n]" command
line question unanswered? Are there any locks held there, or constant
polling back to the GUI?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/3050#issuecomment-225261969,
or mute the thread
https://github.com/notifications/unsubscribe/AA05_wtqP0dAPdusVZole_yN7klgw5fOks5qKa7CgaJpZM4IzAMx
.

Can't see how that would result in CPU use inside kernelspace, though.

I haven't been able to reproduce this after restarting the GUI app. I've tried id-ing and tracking users, starting in the CLI and switching to GUI in the middle, but the only problem I had this time around was a single ▶ ERROR Tracking statement was stale when I clicked the track button in the GUI and finished answering the y/n questions in the CLI.

I was able to trigger it again, and have it running in a terminal now.

With CLI: keybase track <username> and observe the "Is this the  you wanted? [Y/n]" prompt.
When GUI opens, click the "Track" button; leave the window open without clicking the "Close" button.
Open a new terminal and run keybase update check.
Command completes successfully in that terminal session and returns to the prompt.
Command triggers the same command to run in the original terminal session that's still awaiting a response to the ays prompt.

Also in the other terminal window, a warning is thrown when manually checking to see if the updater is running:
$ keybase update check-in-use
▶ WARNING Continuing despite error in lsof: Error running /usr/sbin/lsof [-w -F pcuftn /keybase]: exit status 1 ()
{
  "in_use": false
}

Sent a new log 480e434541ee5489e910731c with more info.

The error about missing keybase.updater.log can be ignored (it's used by the new updater which isn't live yet).
The keybase update check-in-use is a command run by the updater to see if /keybase is in use (calls lsof)... erroring isn't necessarily bad here, and can just mean that /keybase is not in use.

Lsof returns a one (1) if any error was detected, including the failure to locate command names, file names, Internet addresses or files, login names, NFS files, PIDs, PGIDs, or UIDs it was asked to list.

So I don't think it is updater related.
I also tried reproing this (leaving close UI up), and wasn't able to (on a bleeding edge version).

Unless calling lsof on /keybase while Fuse or KBFS is in a particular state is causing the issue.

I tried to repro this using the new updater and wasn't able to. FWIW, my kernel_task is currently at 1.41GB and my other computer is 1.1GB so the mem usage doesn't seem that out of the ordinary.
The update check output being repeated does seem weird but not sure how that happening 13 times would cause 150% usage in kernel_task. The update check causes the v1 updater to trigger a check and there is log forwarding from the service, but it should exit after. Maybe it has something to do with log forwarding from the service to the client and something spinning in there?
I looked in the logs and I don't see anything strange (or something that seemed uncontrollably looping).

The kernel_task may be an unrelated red herring; it just jumped out as unexpectedly high (it's at 911MB right now). I haven't seen the updating loop issue again, but if I do I'll try to get a process dump before I close it all down. Hopefully it was just a one-off problem with my local environment and not Keybase itself.


Automated cherry pick of #6840: scheduler: fix memory over committed

Cherry pick of #6840 on release/3.0.
#6840: scheduler: fix memory over committed

Codecov Report

Merging #6843 into release/3.0 will decrease coverage by 0.37%.
The diff coverage is 9.90%.

@@              Coverage Diff               @@
##           release/3.0   #6843      +/-   ##
==============================================
- Coverage         6.92%   6.54%   -0.38%
==============================================
  Files              541     556      +15
  Lines            91753   98577    +6824
==============================================
+ Hits              6351    6455     +104
- Misses           84880   91598    +6718
- Partials           522     524       +2

Flag
Coverage Δ

#aFlag
6.54% <9.90%> (-0.38%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/cloudcommon/policy/policy.go
1.58% <ø> (+<0.01%)
:arrow_up:

pkg/cloudcommon/policy/resources.go
0.00% <ø> (ø)

pkg/cloudprovider/objectstore.go
2.51% <0.00%> (-0.01%)
:arrow_down:

pkg/cloudprovider/securitygroup.go
0.00% <0.00%> (ø)

pkg/compute/models/buckets.go
1.23% <0.00%> (ø)

pkg/compute/models/cloudproviders.go
0.97% <0.00%> (-0.01%)
:arrow_down:

pkg/compute/models/disks.go
0.54% <0.00%> (-0.01%)
:arrow_down:

pkg/compute/models/guestdisks.go
0.57% <ø> (+<0.01%)
:arrow_up:

pkg/compute/models/guestnetworks.go
0.21% <0.00%> (+<0.01%)
:arrow_up:

pkg/compute/models/regiondrivers.go
20.00% <ø> (ø)

... and 42 more

Continue to review full report at Codecov.

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


fix: Reduce wait time to two mins while loading spaces

Reducing wait time to 2 mins from 6 mins, as 6 mins is quite high and it feels like something is down

Why would it feels like something is down ? This function is run immediately on server start

Why would it feels like something is down ? This function is run immediately on server start

No not on server start, but try going to https://testnet.snapshot.org/#/ and follow a space 😄 it takes very long to see correct followers, i think these functions loadSpaces loadSpacesMetrics take long time on prod anyway, I feel waiting for 6 mins doesn't make anything better.

I see now. Theses counters will be faster with https://github.com/orgs/snapshot-labs/projects/12/views/1?pane=issue&itemId=55693630 later, and this run() should only updates space scores used for sorting


Implement an endpoint/function for making an initial commit.

Refer to this link for help: https://gist.github.com/lucis/864849a7f3c347be86862a3a43994fe0.

Not needed.


Update deprecated ag-grid props

UI/UX: Update deprecated ag-grid props
Summary: Code quality improvements.
Description
Some of the existing grid props and column definition props of the ag-grid AgGridReact component were deprecated in v19 and v20, as shown by these warning messages in the console.

Changelog

Replace deprecated ag-grid props with their current recommended equivalents

Verification instructions

Navigate to any page containing an ag-grid datagrid, e.g. the Grading page - it should load without any warning messages about deprecated props

Last updated 13 Jun 2020, 12:00AM

Pull Request Test Coverage Report for Build 5403

0 of 4 (0.0%)  changed or added relevant lines in 4 files are covered.
3 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.02%) to 34.021%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/commons/sourceRecorder/SourceRecorderTable.tsx
0
1
0.0%

src/pages/academy/dashboard/Dashboard.tsx
0
1
0.0%

src/pages/academy/grading/Grading.tsx
0
1
0.0%

src/pages/academy/groundControl/GroundControl.tsx
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/pages/academy/dashboard/Dashboard.tsx
1
0%

src/pages/academy/grading/Grading.tsx
1
0%

src/pages/academy/groundControl/GroundControl.tsx
1
0%

Totals

Change from base Build 5351:
-0.02%

Covered Lines:
1698

Relevant Lines:
4625

💛  - Coveralls

This is currently deployed on sourceacademy.nus.edu.sg, by the way.


Exit brakeman gracefully after parser errors

Jira link
HOTT-
What?
I have added/removed/altered:

[x] Fixed typo in brakeman config

Why?
I am doing this because:

ruby 6 syntax brakes the brakeman sexpression tokeniser and propagates errors since brakeman 4

@willfish FYI - this will have disabled running brakeman? - It will exit with and exit code of 0 and the checks will show as passing even if it finds an issue?

That's definitely not what I intended @jebw


Adaptive Card - Render PDF, Office Docs, Iframe (web page)

.NET HTML
JavaScript

Is your feature request related to a problem? Please describe.
Not finding a way to render office docs, pdf and webpage within a card
Describe the solution you'd like
Render a PDF, Office docs - excel, word, powerpoint and also a webpage as iFrame
Describe alternatives you've considered
nil
Additional context
nil

Hi, I am also Using teams Webview to render PDF URL using HTML code, embed tag but it is not displaying in teams Webview.
Is there any way we can render the PDF in Microsoft Teams?


Enable write color sets on animation publish automatically

Brief description
Add the switching on/off option for write color sets in Create Cache in Openpype settings

Description
Add the switching on/off option for write color sets in Create Cache in Openpype settings, which overrides the write color sets attributes in maya when creating point cache instance for the geometry.
It is implemented for being consistent regarding to the same implementation of switching on/off options in Create Animation for maya.

@moonyuet I agree with Murphy. all good, just please fix the label and the hound comments.

Get the name changed!


Multiple responses per collapser arg

Replaces #881.  Fixed memory consumption issue by replacing ReplaySubject with PublishSubject and adding an AtomicBoolean to CollapsedRequestObservableFunction to track if a value has been set on the response yet

NetflixOSS » Hystrix » Hystrix-pull-requests #175 SUCCESS
This pull request looks good

This did not degrade performance according to new JMH benchmarks (#882), and also worked as expected in Netflix canary.


Need help for record stream

Hello Friends. I'm beginner in c# and in my wpf app i try create button for record stream and stop record when click. May be some one have answer on my qustion? I try change option but it does not work. When i set play() with option second time start record but app load new stream and it wrong.
Thank you all for help.

Hello kapitaneo,
Try to use the following media option when calling the Play().
":sout=#duplicate{dst=std{access=file,mux=mp4,dst='[FileDestination]'},dst=display}"
Regards Tobias

An answer has been provided, no reply, closing.

I want to add a record start/stop toggle button event for both rtsp and vlcplayer  recording feature.how to achieve this....

I don't think this is possible with vlc to add/remove stream output at runtime.
With the latest beta version (with PR #313) you can set a DisplayVideoCallback (with SetVideoCallbacks) when a picture is ready to be displayed.
Maybe you could have some luck with that by saving pictures one by one by yourself. There are probably other solutions out there, but I can't help very much here.
However, don't expect anything that works out of the box without any work.


Adding support for all Rollbar's levels and raw messages

Allow to send debug/info/warning messages to Rollbar.
Rollbax.report(:warning, "Unexpected input", System.stacktrace())

I don't think this is a feature we should implement in this way. If we want to go ahead with this, I think we should have a separate Rollbax.report_message(level, message, stacktrace, custom \\ %{}, occurrence_data \\ %{}) function for this. Thoughts @ericmj?

I agree with @whatyouhide, Rollbax.report should be for errors/logs from logger. A more general function Rollbax.report_message should be used for generic reports to rollbar.

Closing in favour of #73.


Spring Webflux version

Does this project will support spring webflux version ?

It should, this is just a Spring Data project, so if Spring Data Redis works with WebFlux, so should Redis OM Spring, try it, if it doesn't please report back and we'll troubleshoot.

BTW, version 0.8.0 required Spring 3 or greater

@bsbodden Thanks for your answers


Transfering to Starter Kit if already on Locomotive Scroll

Hello! Thank you so much for this Plugin, I really hope this is gonna be fixing all my problems.
I have a Nuxt page with Locomotive Scroll running, but I am always bothered with flickering parts everywhere on scroll.
I just use the locomotive-scroll.min.js loaded in head.
Additionally I have this code (and some more) to the global page _.vue:
`
// Locomotive Scroll
var scroll = new LocomotiveScroll({
  el: document.querySelector("[data-scroll-container]"),
  smooth: true,
  smoothMobile: false,
  offset: ["0", "0"],
  lerp: 0.05,
  touchMultiplier: 0,
});

scroll.scrollTo(0);

if ($(".c-scrollbar").length > 1) {
  $(".c-scrollbar:first-of-type").remove();
}

var initDomHeight = $("#footer").offset().top;

var windowWidth = $(window).width();
var leftDist = 0;
var topDist = 0;

if ($("#startVideo").length) {
  var startOffsetVideoTop = $("#startVideo").offset().top;
  var gridWidth = $(".videowrap").outerWidth();

  $("#startTitle .h2").mouseover(function () {
    scroll.scrollTo(180);
  });
}

scroll.on("scroll", (func) => {
  // Change width of Video on scroll
  if ($("#startVideo").length) {
    var curOffsetVideoTop = $("#startVideo").offset().top;
    var percentPosition = (100 / startOffsetVideoTop) * curOffsetVideoTop;

    if (curOffsetVideoTop > 0) {
      var videoWidth =
        gridWidth +
        (windowWidth - gridWidth) *
          ((startOffsetVideoTop - curOffsetVideoTop) / startOffsetVideoTop);
    } else {
      var videoWidth = windowWidth;
      $("#startVideo video").attr("loop", "loop");
      if ($("#startVideo video")[0].paused) {
        $("#startVideo video")[0].play();
      }
    }

    $(".videowrap").css("width", videoWidth);
  }`

Do I have to change the whole code and remove everything from Locomotive scroll to be able to run the starter Kit, or can I install your package as well as having the lovomotice scroll definitions?
  var scroll = new LocomotiveScroll({       el: document.querySelector("[data-scroll-container]"),       smooth: true,       smoothMobile: false,       offset: ["0", "0"],       lerp: 0.05,       touchMultiplier: 0,     });

This is my current project:
https://gowest.go-stage.at/de/
user: customer
passwort: gowest

Hi!
I think you can easily include both the plugin and the component.
First of all install the dependencies and copy Locomotive Scroll folder.
After that use the plugin and configure nuxt.
(Wrap your content)[https://github.com/DidoMarchet/starter-kit-nuxt-locomotive-scroll/blob/main/pages/on-call.vue] with the component and pass the options.
Replace scroll.scrollTo(0);, scroll.on("scroll", (func) => {....} with this.$refs.scroller.locomotive.scrollTo(0); or this.$refs.scroller.locomotive.on("scroll", (func) => {....}.
I hope it could be useful and let me know,
Davide

Thank you very much, it works perfect so far!


Limit discord message length or it won't send a notification for long messages

I noticed that I was getting all test notifications, but actual notifications were not arriving.  After going through the logs I found:

(1) accepted ('<IP_ADDRESS>', 46074)
<IP_ADDRESS> - - [27/Dec/2021 01:38:40] "GET /login?next=%2F HTTP/1.1" 200 3037 0.002403
WARNING:apprise:Failed to send to Discord notification: Bad Request - Unsupported Parameters., error=400.
DEBUG:apprise:Response Details:
b'{"content": ["Must be 2000 or fewer in length."]}'

Easy enough "fix" on my part for now is to just remove the diff and so forth from the message.  But maybe add a calculated max length on that string?   Might have to take it in stages..  Not 100% sure as even the URL could possibly be that long..
Or really I guess, you could just chop it at 2k and call it good.
Thanks!

@BloodBlight ok nice, just added a link to the notification wiki page

yeah unsure, in this case people would also complain/make-an-issue that "it seems to be stuck at 2000 chars"
better to just link them to the wiki perhaps

Thanks, I do think that will be helpful.  But as all the tests had been working, and had even gotten notifications from it, I didn’t know to go looking for it (caught it by chance).  I had to go to the logs to even find out there was a problem.
Some sort of persistent feedback in the UI that there is a problem sending a notification would be nice. Even just a flag or something saying on a feed saying “Hey, go check your logs.”
Having a system wide alert/problem notification would be a nice addition.  But that isn’t a quick add and would require some additional state tracking, so...

Having a system wide alert/problem notification would be a nice addition.

yeah, I have thought about that, how to ask apprise for a status report.. i'm open to PRs :)

Just silently losing the message is a bit sad. I would propose reopening this and if its discord you chop it to 2k as suggested. If you are worried about people being confused you could chop another X number of characters off for some room to prepend or append "Truncated due to discord limit" or something special cased, could even link to the issue/wiki if you wanted too

Just silently losing the message is a bit sad. I would propose reopening this and if its discord you chop it to 2k as suggested. If you are worried about people being confused you could chop another X number of characters off for some room to prepend or append "Truncated due to discord limit" or something special cased, could even link to the issue/wiki if you wanted too

Just silently losing the message is a bit sad. I would propose reopening this and if its discord you chop it to 2k as suggested. If you are worried about people being confused you could chop another X number of characters off for some room to prepend or append "Truncated due to discord limit" or something special cased, could even link to the issue/wiki if you wanted too

I've reopened the issue, as I dont use notifications, I wont be writing the patch but I'm open to PR's (maybe you can write one, just check it starts with discord:// etc)

I have extended this feature to include URLs starting with https://discord.com/api/webhooks/ which apprise supports in addition to the base discord:// in #748

I have extended this feature to include URLs starting with https://discord.com/api/webhooks/ which apprise supports in addition to the base discord:// in #748

very nice!
will apprise cut the message down to 2k or does the problem persist?


Proposal: link recibe a function as "to" prop

Add support to send a function as "to" prop (like onClick but at the same prop)
return <Link to={pathOrFunction} />

This breaks the actual navigation. If you want to run something on click, use onClick.

I use this change, not break anything dude and reduce a lot the conditions when you got bunch of routes on different devices

See #5368. We're already working on this.


Section headers

Is there a way to use the coordinator view in a tableView with section headers?
The problem is that the section header stay stick and doesn't move with the animation.

I don't know if you solved this but here's what I did:
I made an extension of UITableView and in that file I put the following code
import UIKit

class HeaderInsetTableView: UITableView {

    var shouldManuallyLayoutHeaderViews =  Bool()
    var headerViewInsets = UIEdgeInsets()

    override func layoutSubviews (){
        super.layoutSubviews()

        if shouldManuallyLayoutHeaderViews {
            layoutHeaderViews()
        }
    }

    func setHeaderViewInsets (_headerViewInsets: UIEdgeInsets){
        headerViewInsets = _headerViewInsets

        shouldManuallyLayoutHeaderViews = !UIEdgeInsetsEqualToEdgeInsets(headerViewInsets, UIEdgeInsets.zero)

        setNeedsLayout()
    }

    func layoutHeaderViews () {

        let numberOfSections = self.numberOfSections
        let contentInset = self.contentInset
        let contentOffset = self.contentOffset
        let sectionViewMinimumOriginY = contentOffset.y + contentInset.top + headerViewInsets.top

        for section in 0  ..< numberOfSections  {
            let sectionView = (headerView(forSection: section))

            if sectionView == nil {
                continue
            }

            let sectionFrame: CGRect = rect(forSection: section)

            var sectionViewFrame: CGRect = sectionView!.frame

            sectionViewFrame.origin.y = ((sectionFrame.origin.y < sectionViewMinimumOriginY) ? sectionViewMinimumOriginY : sectionFrame.origin.y)
            if section < numberOfSections - 1 {
                let nextSectionFrame: CGRect = rect(forSection: section + 1)
                if sectionViewFrame.maxY > nextSectionFrame.minY {
                    sectionViewFrame.origin.y = nextSectionFrame.origin.y - sectionViewFrame.size.height
                }
            }

            sectionView?.frame = sectionViewFrame
        }

    }
}

Then on your view instead of using UITableView use the custom class and add this to your table:
tableView.contentInset = UIEdgeInsetsMake(100, 0, 0, 0)
tableView.setHeaderViewInsets(_headerViewInsets: UIEdgeInsetsMake(-235,0,0,0))

This is the source of the solution, I just converted the code.
http://www.b2cloud.com.au/tutorial/uitableview-section-header-positions/


Conference management

We currently don't have a way on the SMBExplorer exports to provide whether the conference is DH or not. May need a screen to manually toggle this from within the app

May also want to include season configuration on this (unless we have a season config screen) for # playoff teams. This would help us determine which teams made it to the playoffs on a season basis (although we can also infer this based on teams who have records in their home/season playoff schedule)


fix: CLIN-204 refactor get by ancestors endpoint

test fail (same for master and this PR)
master:

this PR:


Allow possibility to provide multiple additional args before message

When logging an error, it's possible to provide an initial object that will be logged alongside the message.
It would be good if it was possible to provide multiple ...objects instead, to make it easier to provide error payloads.
Example
err.payload = { foo: 1 };
logger.error(err, 'Something failed');

-->
logger.error(err, { foo: 1 }, 'Something failed');

Maybe there's a better signature for this, but the ability to pass along more payloads would be appreciated.

:+1:

@sommestad We should also make it a subclass of Bunyan. Something like this maybe...
export default class Logger extends Bunyan {

  _emit(rec, noemit) {
    // The Splunk timestamp thing we currently do...
    const overridden_rec = { timestamp: new Date() };
    return super._emit(overridden_rec, noemit);
  }

  error(...args) {
    const [payload, message] = this._formatPayload(args);
    super.error(payload, message);
  }

  _formatPayload(args) {
    // Prepare a payload object...
  }

  static create(options) {
    return new Logger();
  }

}


add localization tests into demo

localization and internationalization features should be demoed using demo tests

merged: https://github.com/qaprosoft/carina-demo/pull/123
during the testing Web-Localization-Demo-Test should be executed vs different locales

Verified.
Web-Localization-Demo-Tests are successfully executed in different locales:


python3Packages.apsw: <IP_ADDRESS> -> <IP_ADDRESS>

Description of changes
Release docs
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review pr 203509 run on x86_64-linux 1

3 packages marked as broken and skipped:

pgadmin
python310Packages.pytest-ansible
python39Packages.pytest-ansible

1 package failed to build and already failed to build on hydra master:

etesync-dav: log was empty

89 packages built:

ansible (python310Packages.ansible-core)
ansible-later (python310Packages.ansible-later)
ansible-lint (python310Packages.ansible-lint)
ansible_2_12
ansible_2_13
calibre
cozy
dyndnsc
etebase-server (python310Packages.etebase-server)
gkraken
gwe
home-assistant-component-tests.bmw_connected_drive
kargo
matrix-commander
mirage-im
opsdroid
pantalaimon
pantalaimon-headless
parsedmarc (python310Packages.parsedmarc)
pgadmin4
python310Packages.aiofile
python310Packages.aiomisc
python310Packages.ansible
python310Packages.ansible-compat
python310Packages.ansible-kernel
python310Packages.ansible-runner
python310Packages.apsw
python310Packages.bimmer-connected
python310Packages.caio
python310Packages.dependency-injector
python310Packages.etesync
python310Packages.fastapi
python310Packages.fastapi-mail
python310Packages.flask-admin
python310Packages.flask-security-too
python310Packages.geoip2
python310Packages.json-logging
python310Packages.matrix-nio
python310Packages.mocket
python310Packages.nplusone
python310Packages.ormar
python310Packages.peewee
python310Packages.pytelegrambotapi
python310Packages.slowapi
python310Packages.sopel
python310Packages.strawberry-graphql
ttp (python310Packages.ttp)
python310Packages.wtf-peewee
python39Packages.aiofile
python39Packages.aiomisc
python39Packages.ansible
python39Packages.ansible-compat
python39Packages.ansible-core
python39Packages.ansible-kernel
python39Packages.ansible-later
python39Packages.ansible-lint
python39Packages.ansible-runner
python39Packages.apsw
python39Packages.bimmer-connected
python39Packages.caio
python39Packages.dependency-injector
python39Packages.etebase-server
python39Packages.etesync
python39Packages.fastapi
python39Packages.fastapi-mail
python39Packages.flask-admin
python39Packages.flask-security-too
python39Packages.geoip2
python39Packages.json-logging
python39Packages.matrix-nio
python39Packages.mocket
python39Packages.nplusone
python39Packages.ormar
python39Packages.parsedmarc
python39Packages.peewee
python39Packages.pytelegrambotapi
python39Packages.slowapi
python39Packages.sopel
python39Packages.strawberry-graphql
python39Packages.ttp
python39Packages.wtf-peewee
s3ql
semgrep
soco-cli
sqlite-web
sublime-music
theharvester
vorta
weechatScripts.weechat-matrix

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:

ansible_2_12:

Package is missing a meta section.
If the package is using runCommand please make sure to inherit or create a meta section.
Using CC=cc, CXX=c++, AR=ar or LD=cc prevents cross compiling.
Please use CC=${stdenv.cc.targetPrefix}cc, CXX=${stdenv.cc.targetPrefix}c++, AR=${stdenv.cc.targetPrefix}ar or LD=${stdenv.cc.targetPrefix}cc.
got build log for '/nix/store/l2lqfcinp85k98fzg51mc663gq8nxa84-python3.10-ansible-core-2.12.10' from 'daemon'
When evaluating attribute ‘ansible_2_12’:
warning: unused-argument
Unused argument: lib.
Near pkgs/top-level/all-packages.nix:36900:18:
      |
36900 |   pkgsModule = { lib, options, ... }: {
      |                  ^

ansible_2_13:

got build log for '/nix/store/zasslnffxxdlk5ja195f43j1wjxdbv70-python3.10-ansible-core-2.13.6' from 'daemon'
When evaluating attribute ‘ansible_2_13’:
warning: unused-argument
Unused argument: callPackage.
Near pkgs/development/python-modules/ansible/core.nix:2:3:
  |
2 | , callPackage
  |   ^

ansible-later:

got build log for '/nix/store/1bnz0pf02hbz4lkngmsvry3pvaq4pdhx-python3.10-ansible-later-2.0.23' from 'daemon'
When evaluating attribute ‘ansible-later’:
warning: stale-substitute
Stale substituteInPlace detected.
substituteStream(): WARNING: pattern 'unidiff = "0.7.3"' doesn't match anything in file 'pyproject.toml'
Near pkgs/development/python-modules/ansible-later/default.nix:85:
   |
85 |     description = "Best practice scanner for Ansible roles and playbooks";
   | ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/stale-substitute.md
warning: stale-substitute
Stale substituteInPlace detected.
substituteStream(): WARNING: pattern 'toolz = "0.11.2' doesn't match anything in file 'pyproject.toml'
Near pkgs/development/python-modules/ansible-later/default.nix:85:
   |
85 |     description = "Best practice scanner for Ansible roles and playbooks";
   | ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/stale-substitute.md

ansible:

got build log for '/nix/store/3bm82kcfz7gkh51krcfiqd7ky0ahsklj-python3.10-ansible-core-2.14.0' from 'daemon'
When evaluating attribute ‘ansible’:
warning: unused-argument
Unused argument: callPackage.
Near pkgs/development/python-modules/ansible/core.nix:2:3:
  |
2 | , callPackage
  |   ^

gwe:

got build log for '/nix/store/hysmrl14v4rix0q20jsqjb2z74l215lc-gwe-0.15.5' from 'daemon'
When evaluating attribute ‘gwe’:
warning: meson-cmake
Meson uses CMake as a fallback dependency resolution method and it likely is not necessary here. The message about cmake not being found is purely informational.
Near pkgs/tools/misc/gwe/default.nix:53:3:
   |
53 |   nativeBuildInputs = [
   |   ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/meson-cmake.md
warning: unused-argument
Unused argument: makeWrapper.
Near pkgs/tools/misc/gwe/default.nix:5:3:
  |
5 | , makeWrapper
  |   ^

home-assistant-component-tests.bmw_connected_drive:

got build log for '/nix/store/72xr4nmiv1ckrcswkhwd96ibyqzbnnl2-homeassistant-test-bmw_connected_drive-2022.11.4' from 'daemon'
When evaluating attribute ‘home-assistant-component-tests.bmw_connected_drive’:
warning: unused-argument
Unused argument: fetchpatch.
Near pkgs/servers/home-assistant/default.nix:5:3:
  |
5 | , fetchpatch
  |   ^

opsdroid:

got build log for '/nix/store/6jgc01l56nangfh4sj1mxpasfqkcbq46-python3.10-opsdroid-0.25.0' from 'daemon'
When evaluating attribute ‘opsdroid’:
warning: python-include-tests
Consider adding a checkPhase for tests, or if not feasible, pythonImportsCheck.
Near pkgs/applications/networking/opsdroid/default.nix:30:
   |
30 |     description = "An open source chat-ops bot framework";
   | ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/python-include-tests.md

parsedmarc:

got build log for '/nix/store/6yxx7whdyvjnz38n68wi2xpl03bx6f4z-python3.10-parsedmarc-8.2.0' from 'daemon'
When evaluating attribute ‘parsedmarc’:
warning: unused-argument
Unused argument: python.
Near pkgs/development/python-modules/parsedmarc/default.nix:6:3:
  |
6 | , python
  |   ^

python310Packages.dependency-injector:

got build log for '/nix/store/r10qqcyrhv7x9s4s8qx4yq1f2lzrcz8i-python3.10-dependency-injector-4.35.3' from 'daemon'
When evaluating attribute ‘python310Packages.dependency-injector’:
warning: stale-substitute
Stale substituteInPlace detected.
substituteStream(): WARNING: pattern 'six>=1.7.0,<=1.15.0' doesn't match anything in file 'requirements.txt'
Near pkgs/development/python-modules/dependency-injector/default.nix:58:
   |
58 |     description = "Dependency injection microframework for Python";
   | ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/stale-substitute.md

python310Packages.flask-admin:

got build log for '/nix/store/lwmpsqb5791580kmlnsi0vvl9iaxy3fr-python3.10-flask-admin-1.6.0' from 'daemon'
When evaluating attribute ‘python310Packages.flask-admin’:
warning: unused-argument
Unused argument: enum34.
Near pkgs/development/python-modules/flask-admin/default.nix:8:3:
  |
8 | , enum34
  |   ^

python310Packages.matrix-nio:

got build log for '/nix/store/zz2wsyx1sqsn57g8crnbh6wjr01p3mrx-python3.10-matrix-nio-0.20.0' from 'daemon'
When evaluating attribute ‘python310Packages.matrix-nio’:
warning: stale-substitute
Stale substituteInPlace detected.
substituteStream(): WARNING: pattern 'jsonschema = "^3.2.0"' doesn't match anything in file 'pyproject.toml'
Near pkgs/development/python-modules/matrix-nio/default.nix:94:
   |
94 |     description = "A Python Matrix client library, designed according to sans I/O principles";
   | ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/stale-substitute.md

python310Packages.pytelegrambotapi:

got build log for '/nix/store/n86k49r9v3dfn0n7mxb6sj65ziy9igw3-python3.10-pyTelegramBotAPI-4.7.0' from 'daemon'
When evaluating attribute ‘python310Packages.pytelegrambotapi’:
warning: no-python-tests
Test runner could not discover any test cases: ‘Ran 0 tests in 0.000s’
Near pkgs/development/python-modules/pyTelegramBotAPI/default.nix:34:
   |
34 |     description = "A simple, but extensible Python implementation for the Telegram Bot API";
   | ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/no-python-tests.md

python310Packages.slowapi:

got build log for '/nix/store/ysqhbnpcrfwnx84f7pb0xkdb4qhj8c2p-python3.10-slowapi-0.1.6' from 'daemon'
When evaluating attribute ‘python310Packages.slowapi’:
warning: stale-substitute
Stale substituteInPlace detected.
substituteStream(): WARNING: pattern 'redis = "^3.4.1"' doesn't match anything in file 'pyproject.toml'
Near pkgs/development/python-modules/slowapi/default.nix:66:
   |
66 |     description = "Python library for API rate limiting";
   | ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/stale-substitute.md

python39Packages.ansible-core:

got build log for '/nix/store/n3d6ddr6rd884c8ldqa2xkypv2brkfsz-python3.9-ansible-core-2.14.0' from 'daemon'
When evaluating attribute ‘python39Packages.ansible-core’:
warning: unused-argument
Unused argument: callPackage.
Near pkgs/development/python-modules/ansible/core.nix:2:3:
  |
2 | , callPackage
  |   ^

python39Packages.ansible-later:

got build log for '/nix/store/aaiblav98pnbyx4xj23cz2cdls38a213-python3.9-ansible-later-2.0.23' from 'daemon'
When evaluating attribute ‘python39Packages.ansible-later’:
warning: stale-substitute
Stale substituteInPlace detected.
substituteStream(): WARNING: pattern 'unidiff = "0.7.3"' doesn't match anything in file 'pyproject.toml'
Near pkgs/development/python-modules/ansible-later/default.nix:85:
   |
85 |     description = "Best practice scanner for Ansible roles and playbooks";
   | ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/stale-substitute.md
warning: stale-substitute
Stale substituteInPlace detected.
substituteStream(): WARNING: pattern 'toolz = "0.11.2' doesn't match anything in file 'pyproject.toml'
Near pkgs/development/python-modules/ansible-later/default.nix:85:
   |
85 |     description = "Best practice scanner for Ansible roles and playbooks";
   | ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/stale-substitute.md

python39Packages.dependency-injector:

got build log for '/nix/store/635wjdxr1pnwfqrclvfg9qyv8dcf3rf9-python3.9-dependency-injector-4.35.3' from 'daemon'
When evaluating attribute ‘python39Packages.dependency-injector’:
warning: stale-substitute
Stale substituteInPlace detected.
substituteStream(): WARNING: pattern 'six>=1.7.0,<=1.15.0' doesn't match anything in file 'requirements.txt'
Near pkgs/development/python-modules/dependency-injector/default.nix:58:
   |
58 |     description = "Dependency injection microframework for Python";
   | ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/stale-substitute.md

python39Packages.flask-admin:

got build log for '/nix/store/czfypnv48h1gqcr8igifmwl3cvh9v7if-python3.9-flask-admin-1.6.0' from 'daemon'
When evaluating attribute ‘python39Packages.flask-admin’:
warning: unused-argument
Unused argument: enum34.
Near pkgs/development/python-modules/flask-admin/default.nix:8:3:
  |
8 | , enum34
  |   ^

python39Packages.matrix-nio:

got build log for '/nix/store/w2vsrm55i6jk5lq89fy8mnh45xddqa3i-python3.9-matrix-nio-0.20.0' from 'daemon'
When evaluating attribute ‘python39Packages.matrix-nio’:
warning: stale-substitute
Stale substituteInPlace detected.
substituteStream(): WARNING: pattern 'jsonschema = "^3.2.0"' doesn't match anything in file 'pyproject.toml'
Near pkgs/development/python-modules/matrix-nio/default.nix:94:
   |
94 |     description = "A Python Matrix client library, designed according to sans I/O principles";
   | ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/stale-substitute.md

python39Packages.parsedmarc:

got build log for '/nix/store/1qfk7kwhidvsdarj3666xdq68aqllgml-python3.9-parsedmarc-8.2.0' from 'daemon'
When evaluating attribute ‘python39Packages.parsedmarc’:
warning: unused-argument
Unused argument: python.
Near pkgs/development/python-modules/parsedmarc/default.nix:6:3:
  |
6 | , python
  |   ^

python39Packages.pytelegrambotapi:

got build log for '/nix/store/lzhmrh2pk4hfbzijhz06s7w8s6h15qz5-python3.9-pyTelegramBotAPI-4.7.0' from 'daemon'
When evaluating attribute ‘python39Packages.pytelegrambotapi’:
warning: no-python-tests
Test runner could not discover any test cases: ‘Ran 0 tests in 0.000s’
Near pkgs/development/python-modules/pyTelegramBotAPI/default.nix:34:
   |
34 |     description = "A simple, but extensible Python implementation for the Telegram Bot API";
   | ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/no-python-tests.md

python39Packages.slowapi:

got build log for '/nix/store/aibwdbyds77vmgsy805p264zabg6w27l-python3.9-slowapi-0.1.6' from 'daemon'
When evaluating attribute ‘python39Packages.slowapi’:
warning: stale-substitute
Stale substituteInPlace detected.
substituteStream(): WARNING: pattern 'redis = "^3.4.1"' doesn't match anything in file 'pyproject.toml'
Near pkgs/development/python-modules/slowapi/default.nix:66:
   |
66 |     description = "Python library for API rate limiting";
   | ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/stale-substitute.md

semgrep:

got build log for '/nix/store/rx96ahgc1f7j556h52n536b2l2f848cc-semgrep-0.112.1' from 'daemon'
When evaluating attribute ‘semgrep’:
warning: unused-argument
Unused argument: lib.
Near pkgs/tools/security/semgrep/default.nix:1:3:
  |
1 | { lib
  |   ^

warning: unused-argument
Unused argument: fetchFromGitHub.
Near pkgs/tools/security/semgrep/default.nix:2:3:
  |
2 | , fetchFromGitHub
  |   ^

vorta:

got build log for '/nix/store/dl4sg52bv6nsrmjy6d0khld09akjyqy7-vorta-0.8.9' from 'daemon'
When evaluating attribute ‘vorta’:
warning: stale-substitute
Stale substituteInPlace detected.
substituteStream(): WARNING: pattern 'setuptools_git' doesn't match anything in file 'setup.cfg'
Near pkgs/applications/backup/vorta/default.nix:90:
   |
90 |     description = "Desktop Backup Client for Borg";
   | ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/stale-substitute.md
warning: stale-substitute
Stale substituteInPlace detected.
substituteStream(): WARNING: pattern 'pytest-runner' doesn't match anything in file 'setup.cfg'
Near pkgs/applications/backup/vorta/default.nix:90:
   |
90 |     description = "Desktop Backup Client for Borg";
   | ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/stale-substitute.md


Extending test cases.

Additional test cases from checking the railroad diagrams.
Test cases which are not accepted yet by the parser start with the characters "% ? " since I could not decide if the change should be in the syntax definition or in the parser.

@walter-weinmann Can you please resolve this conflict? I can then merge and work on the new test cases, every-now-and-then.

Sorry - I can't see any conflicts - maybe the following old lines in select.tst are causing the trouble:
<<"select a < 0, f(a+2 as d), f2(a < 3 as f) from abc">>.

<<"select 'öüäéèà', 'شلاؤيثبلتهتنمةىخ','นี่คือการทดสอบ' from dual"/utf8>>.

<<"sElect
...

$ git checkout -b walter-weinmann-sqlparse_wwe_ext_test_cases master
...
$ git pull https://github.com/walter-weinmann/sqlparse.git sqlparse_wwe_ext_test_cases
From https://github.com/walter-weinmann/sqlparse
 * branch            sqlparse_wwe_ext_test_cases -> FETCH_HEAD
Auto-merging test/cursor.tst
CONFLICT (add/add): Merge conflict in test/cursor.tst
Automatic merge failed; fix conflicts and then commit the result.

Conflicted test/cursor.tst:
%%-*- mode: erlang -*-
%%-*- coding: utf-8 -*-

% Test control options
[{verbose, 0}, {tests, []}].

%%
<<<<<<< HEAD
%% TESTS
%%

"CLOSE name_cursor_1".
"DECLARE name_cursor_1 CURSOR FOR SELECT * FROM name_table_1".
"FETCH name_cursor_1 INTO name_column_1".
"OPEN name_cursor_1".
"DELETE FROM table_name WHERE CURRENT OF name_cursor".
"UPDATE name_table SET name_column_1 = :value_1 WHERE CURRENT OF name_cursor".
"UPDATE name_table SET name_column_1 = :value_1, name_column_2 = :value_2 WHERE CURRENT OF name_cursor".
=======
%% TESTS CLOSE
%%

% ? "CLOSE name_cursor_1".

%%
%% TESTS DECLARE
%%

% ? "DECLARE name_cursor_1 CURSOR FOR SELECT * FROM name_table_1".

%%
%% TESTS DELETE
%%

% ? "DELETE FROM table_name WHERE CURRENT OF name_cursor".

%%
%% TESTS FETCH
%%

% ? "FETCH name_cursor_1 INTO name_column_1".

%%
%% TESTS OPEN
%%

% ? "OPEN name_cursor_1".

%%
%% TESTS UPDATE
%%

% ? "UPDATE name_table SET name_column_1 = :value_1 WHERE CURRENT OF name_cursor".
% ? "UPDATE name_table SET name_column_1 = :value_1, name_column_2 = :value_2 WHERE CURRENT OF name_cursor".

>>>>>>> 059ed285bba469e2e8a16225710ce804add8bfbc

Hope it helps.

Thanks for the help - should be ok now.

Sorry - done.

Thanks @walter-weinmann


1.20.4 Nether doesn't change

Built a portal at bottom of world at y 0, went though, and all that is there is the portal.

ignore

Every dimension is void in this version of skyblock. In the nether there should be a small island of netherrack and nylium


New/enhanced component: Combobox

This could be made a part of the 'drop down' component.
Something like this: https://jedwatson.github.io/react-select/
@jstrachan

BTW ah it looks like @mkramb had already added react-select and react-virtualised to cdx-frontend - doh :)


Successful transaction in stripe does not populate cc payment fields

What I'm trying to achieve
Get card details (ex. last 4 digits) from Payment object when using stripe gateway.
Steps to reproduce the problem

Use stripe gateway
Pay with card
cc_last_digits, cc_brand, cc_exp_month, cc_exp_month are empty in Payment object.

What I expected to happen
Those data should be taken from gateway_response.

It is because only a token is passed from frontend to backend during payment in stripe. All the data including credit card brand, last digit, etc. are tokenized. There is no need to expose sensitive data such as credit card information to backend during payment. This makes the payment safer. The docs relate with stripe.js in https://stripe.com/docs/stripe-js might be useful to explaining this.

@jxltom thats not true. Gateway response has those data and its not considered sensitive.

@krzysztofwolski You're right. I thought you are referring the data posted but you are talking about gateway response. Yeah, it will be nice to update payment if gateway response has updated data. Currently, only transactions are created by gateway response but payment is not updated.


Refactoring of the Tabs Select Configuration Screen

[x] Refactor the tabs to track across all modules

[x] When you open a tab up it's open across all modules refactoring the management of the state and storage.

[x] You are able to switch between all modules without having to reopen the tabs

*Note: This will be a ticket worked with QA developers (coordinate with Quang and/or Adis)

Test Case: TC177
Test Passed


How  to get the scroll direction of the fscalendar

I want to get the scrolling direction of the calendar and get the targetPage ,Because I need to know the month after I swipe the calendar ,then take the month as http parameter

@DeliriousLee  Did you get the solution ?? Can you please share the solution ?


[FEAT] Graphql - configurable mutation InputType

Is your feature request related to a problem? Please describe.
In create and update mutation queries, operation name can be configured via metadata. variable.input.type can't be configured as it's "hard coded".
There is case (or at list mine ;)), where InputType.type can't be the same as the "hardcoded by refine" ones.
Describe alternatives you've considered
No response
Additional context
No response
Describe the thing to improve
For example, in the dataProvider.create, and in the same spirit that allow to configure operation :
const operation = metaData?.operation ?? camelCreateName;

The variables.input.type should be something like:
const variableInputType = metaData?.variableInputType ?? `${camelCreateName}Input`

What do you think about that ? Is it doable ?
Thanks for this great piece of lib !

Hey @florent-andre
GraphQL data provider needs various arrangements. This includes what you wrote. Can you create PR to support us? 🙏🏼

Yes, sure I can try (maybe this week or later on). Do you think metaData?.variableInputType is a good name ?
Regards

Yes, sure I can try (maybe this week or later on). Do you think metaData?.variableInputType is a good name ? Regards

I think metaData?.variableInputType can be found better, we are open to suggestions. We look forward to PR 🎉

Hello! I'll open a PR for this one!


Zoom level is too high after searching for a node

Search for a named building somewhere off the visible map area and iD zooms in to a level suitable to fill as much as possible of the map area with the building.  I just tried with a typical house and got level 22. This is good.  Maybe a little high, but I can see something sensible has been returned and change the zoom as necessary.
Now do the same for a named node.  Such as Cardigan (a town in Cardigan, Wales).  Zoom level is 24 and all that can be seen is a vague blur of background imagery zoomed far beyond what is sensible, with a few segments of ways that could be anything.  Only at zoom level 22 does it start to become intelligible.
I suggest limiting the zoom to 22 (maybe even 21) when the search result is a node.  Maybe even lower than that, because some nodes do not make much sense without more surrounding context.  A level 24 zoom just leaves people wondering what has gone wrong because everything is a blur.  It's even worse for St Dogmaels (a village in Pembrokeshire, Wales), which is in the middle of a field and probably ought to be moved closer to the centre of the village.

Good suggestion, thank you!   I adjusted the default max zoom for points to z19.


Converting division to PTX fails

self.compareCudaToCPU("fun((a,b)) { a/b }", "(2, 1)")
Fails with:
AssertionError: Failed with fun((a,b)) { a/b } on (2, 1): Exception(
    (
        "Code returns multiple types.",
        (BA00CC2F65A2..., ((f: <eval>.<unnamed function>.f(...*), i: (2, 1)),))),
    nothing)

Yeah this is because of the exception path. I think the right thing to do
is to modify it so that exceptions cause that particular frame to exit
early. We should keep a mask of which threads exited with real values. We
can then execute the early-exit threads on the CPU to capture whatever
actually happened. In the short-run, let's not worry about this...
On Mon, Mar 21, 2016 at 5:55 PM, Alexandros Tzannes <
<EMAIL_ADDRESS>wrote:

self.compareCudaToCPU("fun((a,b)) { a/b }", "(2, 1)")
Fails with:
AssertionError: Failed with fun((a,b)) { a/b } on (2, 1): Exception(
(
"Code returns multiple types.",
(BA00CC2F65A2..., ((f: ..f(...*), i: (2, 1)),))),
nothing)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/ufora/ufora/issues/242

--
Braxton McKee                             CEO / Ufora
www.twitter.com/braxtonmckee www.ufora.com


How to use Vue.extend to inject Vue instance into a component in Nuxt?

I am trying to create a ProseMirror editor with Nuxt.js on which my users can insert certain components into their inline text.
I should be able to render a new component node in NodeView using Vue.extend(). The problem is, my component needs to communicate with this.$store, and a newly-initialized Vue instance does not have it. Therefore, rendering the component with a new Vue instance would cause an error that suggests something like this.$store does not exist.
Is there anyway that I could access the Vue instance created by Nuxt.js and thereby call Vue.extend() to pass the Vuex instance to my component?
Editor.vue: https://github.com/v2land/v2land-frontend/blob/feature/comment-editor/components/Comment/Editor.vue
Event NodeView: https://github.com/v2land/v2land-frontend/blob/feature/comment-editor/components/Comment/View/Event.js

The current Event NodeView is a workaround, in which I created the component node in Editor.vue and pass it into the view. The ideal solution should be similar to vue-prosemirror-editor.

I inject the $store into a new Vue instance by calling Vue.prototype.$store = vue.$store. Problem solved.


Change min sat value and max. number of vouchers

Issue:
Min. sat value and max. number of vouchers.
My two sats:
Today  i made some changes to this extension on my own server. I asked myself for a long time "why is max amount of vouchers limited to 250 and why is min. amount 10 sats". It seems random to me. Since i got not a propper answer is played with my own lnbits and changed the values.
If there are no security thoughts behind that values i would argue that we now have the privilege to make small payments and withdrawls down to a piece of a cent. We should use that and change this extension.
Set max numbers of vouchers to 1000 makes it easier to fullfile some larger tables of withdrawls. 1000 even better for the monk inside me. lol
Summary:
Changed min. value/withdrawl from 10 sat to 1 sat
Changed max. amount of vouchers from 250 to 1000

Very good, I support that!
I have the technology to start with a Satoshi and should do so. We should leave the fiat era of restrictions behind us.
I can live with 1000 for the time being.
If there are fears that the lower withdrawal limit and the maximum number of withdrawals could be abused, I would rather increase the minimum time between two withdrawals. It is enough for people to request a withdrawal every 10 seconds. But I also believe that 1 second is enough. Every server should be able to cope with that.

Great idea! 👏 I support it too. As Axel said, it's time for no barriers. 🚧

To the developers @dni @arcbtc. Something speaks against this? At least I don't see any reason. I could really use the function.

hey! sorry for the long wait, i also have no idea why there are those limitations. if there is a reason for that there should been a comment imho. i have nothing against it.

so after talking to ben, the limit was set into place because trying to print out more than 250 QR codes hung peoples computers. can you try that @DoktorShift if printing 1k makes any issues?

so after talking to ben, the limit was set into place because trying to print out more than 250 QR codes hung peoples computers. can you try that @DoktorShift if printing 1k makes any issues?

Oh interessting. I actually never printed, just generated. I will try that tomorrow evening. Do not have a printer available rn.

so after talking to ben, the limit was set into place because trying to print out more than 250 QR codes hung peoples computers. can you try that @DoktorShift if printing 1k makes any issues?

Ugh... If you try to print out thousands of QR codes at the same time, you shouldn't be surprised if the system crashes. I will integrate the function on my website, where one QR code voucher will then be retrieved from the other. I would have to retrigger the entire 250 voucher, which would be quite annoying since you have to monitor it and constantly update it.
Maybe you can block the print button to a maximum of 250 vouchers? Or you can give an indication that printing is only possible up to a maximum of 250 vouchers.

Maybe you can block the print button to a maximum of 250 vouchers? Or you can give an indication that printing is only possible up to a maximum of 250 vouchers.

something like that. not quite sure what is best.
but there at least should be comments in the code why this 2250 limit is there.

Maybe you can block the print button to a maximum of 250 vouchers? Or you can give an indication that printing is only possible up to a maximum of 250 vouchers.

something like that. not quite sure what is best.
but there at least should be comments in the code why this 250 limit is there.

My solution:
Increase the amount of maximum QR codes and set a small pop up when press "Print" which give some short information that some system could fail printing high amounts of QR codes.

This was reduced to reduce strain on the server. Its not hard to  create multiple batches, either through the UI or programmatically


bug to http_fuzzer

my poc named "elasticsearch_unauthorized_access_poc" use model "http_fuzzer",
here is my poc results:

two ports match，but only show one raw !

OS: windows
OS Version: 10
Python Version: python 2.7.15

Hi
when you calling the __repeater try this to change default ports in an array.
__repeater(request, [extra_requirements["pma_scan_list"]], timeout_sec, thread_number, log_in_file, time_sleep,
                   language, verbose_level, socks_proxy, retries, scan_id, scan_cmd, condition, thread_tmp_filename,
                   sample_event, sample_message, target, ports, default_ports=[80, 443], counter_message)

if there is any more error please share more details. and if its fixed please let me know.
Regards.

Hey @aliluyala, does the issue persists even after the changes Ali suggested. If yes, please let me know, I would be happy to help, else we can close this issue.

@shaddygarg just to mention, I think this will be failed in file upload requests as well, in that case we need fundamental change...


Image building enhancement

What type of PR is this?
/kind enhancement
What this PR does / why we need it:
Accelerate Golang-based image building
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/lgtm


Track 90th, 99th percentile instead of missed_frame_build_budget_count

See https://github.com/flutter/flutter/pull/19121#issuecomment-419520765

@Hixie any more comment?


Fix #1258 - Serialize/deserialize security in "security" instead of "…

Fix #1258 - Serialize/deserialize security in "security" instead of "securityRequierement" to be spec compliant.

So, I appreciate the PR but this has been discussed before.  You must use our jackson serializers and when you do, the securityRequirement is correctly handled.  If you do not, well, you'll have other problems, this being the least of them.
We cannot accept this PR because (a) the serialization strategy works fine as-is and (b) this is a breaking change to a model signature, and this alone is not worthy of bumping to a 2.0 release.
During the 2.0 development we will fix this, so I'll keep your issue #1258

Unless I'm doing something wrong, I'm using your jackson serializer as you
can see in the test case I added in a comment of the original issue.
If I'm not wrong the current seralizer generates invalid swagger 2, doesn't
it ?
Le mer. 11 mai 2016 20:31, Tony Tam<EMAIL_ADDRESS>a écrit :

Closed #1783 https://github.com/swagger-api/swagger-core/pull/1783.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/swagger-api/swagger-core/pull/1783#event-657595992

I can guarantee you that our serializer is not producing invalid swagger, or all swaggers would be broken.  How are you enabling the serializer?

I'm using io.swagger.util.Json like this:
Swagger swagger = new Swagger();
swagger.setSecurity(Arrays.asList(new SecurityRequirement().scope("user").scope("movie")));
System.out.println(Json.mapper().writeValueAsString(swagger));

which produces:
{"swagger":"2.0","securityRequirement":[{"scopes":["user","movie"]}]}

(note that I updated the example in this comment because there was a mistake)

I want to bump up this PR, because the latest swagger-ui can't handle "securityRequirement" field generated by this generator on the authentication feature. (it expects a "security" field, like it should be !)
as per swagger-codegen, its security definitions only work for the "securityRequirement" field, not "security" as it is based on this code (removing the @JsonIgnore on the setters could fix this tho...)
This is a major issue that should be integrated before everyone forks it to fix it...

I'm a terrible person for not merging this.


while loging into calendar-site, instead of redirecting to current date user is getting redirected to the date on which they created a event

while loging into calendar-site, instead of redirecting to the current date, the user is getting redirected to the date on which they created a event

Notes:

need to look into caching.

Observed: if the date of an event is updated, it is still reflecting the old one on / for few minutes. To reproduce this, update an event's date by drag and drop and reload the page.


About Login function.

Hello Sir,
I try to implement the Login function into a button event. When i click that button, nothing happens. I tried to print the result of the response and it returns nil. In addition, I cannot get to the SoundCloud authorization page as well. Did I do anything wrong? Sorry I am just a newbie in iOS area.
Looking forward to your reply.
Bao

Could you please provide your code here?

Very thankyou for the fast reply. Here is my code. I just try to get to the authorization page and login.

import UIKit
import Soundcloud
class ViewController: UIViewController {
override func viewDidLoad() {
    super.viewDidLoad()
    // Do any additional setup after loading the view, typically from a nib.
    Soundcloud.clientIdentifier = "***********"
    Soundcloud.clientSecret  = "************"
    //Soundcloud.redirectURI = "YOUR_REDIRECT_URI"
}

override func didReceiveMemoryWarning() {
    super.didReceiveMemoryWarning()
    // Dispose of any resources that can be recreated.
}
@IBAction func onLogout(sender: UIButton) {
    let session = Soundcloud.session
    session?.destroy()
}

@IBAction func onLogin(sender: UIButton) {
    Session.login(self) { (result: SimpleAPIResponse<Session>) in
        print(result.response.result)
    }

}

}

Hi,
I have just fixed the problem by setting the redirectURI : "http://google.com". Now I can login. But I got the problem which is: "canOpenURL: failed for URL: "org-appextension-feature-password-management://" - error: "This app is not allowed to query for scheme org-appextension-feature-password-management" ". Never seen that before.
Bao


Minor fixes

This PR is probably not mergable. I use it as way to communicate things.

It will be good to have both of us in reviewers
Script that can copy common folders across SDK's (then we do not need to package anything into github actions etc.)

Assumes structure looks like this. I have already used it in the js-sdk. Now have changes that we might remove from autosync later.

I have added required trivial changes - split fetching and package generation.
This way we can split entire process into two phases:

Github action fetch openapi
Separate step that is language-specific

This solves couple of problems including simplicity to test and verify generation outside
@craicoverflow Those changes might sound radical (there is no pressure from me to get that merged etc.)
Looking for feedback to see if you agree with approach and we can reuse it in the JS SDK (I already replidated this flow to js-sdk)


ERROR: not all arguments converted during string formatting

Describe the issue:
When I create exp under annotation mode,  it failed with this info, And I have no idea what's wrong and how to locate this error.

How to reproduce it?:

We have refactored error handling for next release.
For now if you still have problem please attach the config file.


xamarin.forms android  Bluetooth.ScanForDevicesAsync    Wait forever

private async void MainPage_Appearing(object sender, EventArgs e)
        {
            try
            {
                var s = await Bluetooth.ScanForDevicesAsync();
            }
            catch (Exception ex)
            {

            }

        }

This should be resolved in 4.0.23.


Update Honduras Flag

fix #92. Here Color #00b5e2.

@jdecked  may i ask if this will be accepted?

@jdecked may i ask if this will be accepted?


Bug 1161171 - Stop stripping the cset URL parameter when showing error pages

I think it would be better to modify the go() function to just not throw away the cset and repo, which would avoid having to add boilerplate everywhere. It seems like that may even be as simple as modifying this line:
https://github.com/mozilla/bugherder/blob/e968f7d9e0f63af0541338e21d265e078653c676/js/bugherder.js#L533

Drive-by comments.
Think I agree with Ed here. I had a vague feeling that there was a specific reason I didn't persist these originally, but of course it's too much to ask that I might have documented why.
I suspect it might have been related to possible ambiguity in handling errors once Bugherder started handling multiple trees: if loading the pushlog failed, was it because the user specified right cset/wrong tree or wrong cset/right tree? However, it's probably not worth worrying about that scenario too much: most invocations these days will presumably be incoming traffic from Treeherder with the correct query string and no manual input. (We could probably file a new bug to add a <select> to the input form to specify the tree if necessary.)
The forEach over the persisted array below the linked line not only constructs strings for the fragment of query string to be persisted, but also normalizes them, turning things like debug=foo into debug=1. The logic would need to be changed here to not mutate the changeset/tree.
I think we'd also need to add logic for prioritizing parameters from the query string supplied in the go method's query parameter. (If there's a cset in the query parameter, it came from user input, so should take precedence over any saved state).

How about this?
As a side note, it seems like the items in 'persisted' only actually persist when their values are already '1'. (eg, "debug=foo" was removed, even without my patch, but "debug=1" is persisted)

I'd prefer to defer to Graham on this, I'm not sure I really follow the current PR (it seems a bit hacky?).
Feel free to land as-is if you'd prefer though - I don't really have much of a stake in bugherder any more :-)

As a side note, it seems like the items in 'persisted' only actually persist when their values are already '1'. (eg, "debug=foo" was removed, even without my patch, but "debug=1" is persisted)

Yeah this is what Graeme mentioned in https://github.com/mozilla/bugherder/pull/23#issuecomment-128082930


Add OpenImageIO's idiff tool to appleseed distribution

#2480
Do the same things like oiiotool.

Looks good to me, but I'd like @est77 to have a look too.

Fails on Travis because as @slongle noted, idiff is not available in prebuilt Linux dependencies.

Added idiff to linux deps.


upstream: Add SNI support to original_dst_cluster

Add SNI support to original_dst_cluster via a Host type with default transport socket options created from either from downstream connection's SNI or from the HTTP request's host header.
Risk Level: Low
Testing: Unit tests
Docs Changes: None yet
Release Notes: Original destination cluster now supports deriving a default value for SNI for the upstream TLS connection (if applicable) from the downstream connection (if TLS) or from HTTP request's Host header.

At high level this will be obsolete once #2690 (being worked in #8889, blocked by the refactoring #9026), we're trying to land this generically in router rather than in each type of cluster (dynamic_forward_proxy already does this). cc @Shikugawa @mattklein123

Good to know, I wasn't aware of these efforts. Will reopen as needed to keep some of the cleanups in this PR once #8889 lands.


[CLOSED] Remove IdGenerator class and object-constant-based calls of it

 Issue by KushalP
Friday 24 October 2014 at 15:24 GMT
Originally opened as https://github.com/alphagov/specialist-publisher/pull/290

Note: This is a speculative PR.
The IdGenerator object did very little. It can be summed up as the
following bit of code: SecureRandom.uuid. Keeping it around doesn't
add anything and it makes it hard to understand why the constant form
of the object is being called in the way it does.
Removing it means there's one less thing to understand in terms of
using the already existing *Builder classes.

KushalP included the following code: https://github.com/alphagov/specialist-publisher/pull/290/commits

 Comment by alext
Friday 24 October 2014 at 15:31 GMT

Looks good to me.  Would be good for someone from Product Gaps to give this a look as well.

 Comment by kalleth
Saturday 25 October 2014 at 09:32 GMT

"Ideally" we'd depend on abstractions, but I do agree that this makes the code simpler. I'm surprised at quite the amount of code this removes, honestly.
Happy to merge -- would do it now but i'm not sure whether "speculative" means don't merge it yet :)

 Comment by KushalP
Monday 27 October 2014 at 10:03 GMT

I'm happy for you to merge if you think it's good enough :+1:
Only reason I made it clear that it was speculative is that sometimes people have issues if PRs appear which don't have a story with them.

 Comment by james
Monday 27 October 2014 at 11:05 GMT

<3 @KushalP

 Comment by kalleth
Monday 27 October 2014 at 10:12 GMT

Thanks, @KushalP ! :+1:


feat: update demos with neccessary attribute for surge #415

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes: #415
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Global settings

Pull Request checklist
Note: these are not necessarily requirements

[ ] I reformatted the code with clang-format
[x] I checked to make sure my submission worked
[x] I am the author of submission or have permission from the original author
[ ] Requires update of the PS2SDK
[ ] Requires update of the gsKit
[ ] Others (please specify below)

Pull Request description

Global Settings
Add a global config for game settings (conf_game.cfg) this allows the user to choose between loading per game or global settings on a game by game/feature by feature basis.. global config is used by default and supports GSM, Cheats & PADEMU... per game settings take precedence over global if selected.

For a simple breakdown of how it works see here:
https://www.psx-place.com/threads/open-ps2-loader-v0-9-3.13415/page-45#post-211808

VMC
Added a quick fix for VMC GUI, commit message explains it... the issue was present in previous builds which is why it carried over to the new Game Menu.

Info Page
Also I gotta agree with @TnA-Plastic on this one, the UseInfoPage option is redundant now as it is always an option by choosing to press square or not before booting a game so having the toggle switch only confuses noobies.

Test Build
https://www.sendspace.com/file/z6cbop

I believe you missed one line change under the lng/lang_English.lng file.

LINE 50: "Remove Settings",
Plus, I thought, maybe it's best to remove the following lines instead.  My question is why leave lang text strings that no longer serve a purpose -- hence make translators keep translating lines that are no longer being used in the application.  -- Just a thought...
Here they are as far as I could spot them... (there may be more source files that may point to that line?...)
include/lang.h
LINE 149:     _STR_USE_INFO_SCREEN, // string no longer used
lng/lang_English.lng
LINE 166: Display Info Page
 src/lang.c
LINE 152:     "Unused", // string no longer used
Your thoughts?

Yea I forgot the template, but I think it’s better to leave the strings rather than remove if someone wants to add a new one they can just replace it that’s why I left the comments. Otherwise all the Lang files will need to be shifted up one line at said line.. easier to just leave it unused until someone wants a new Lang string.

@ElPatas1
This PR is now complete (pending your approval) when you've had a chance to test.
Best Regards.

Hello,
i teted already the test build with the previous PR, if nothing new is changed here
i merge it now, please confirm.
Best regards.

@ElPatas1
Only the English template has changed since the previous PR, so nothing further to test.
Thank you.

Hello,
merged, thank you.
Best regards.

The global functions are cool indeed as well as the removal of the OPL-setting, which is rather a theme-thing...

A little question here, how do we set GLOBAL to be off by default in OPL?

You dont, thats the point.. Global game settings only effect GSM, PADEMU & Cheats.. most people use the same settings for all of these throughout their game library.. set Globals so vast majority of your library use those settings, when you find a game that say your GSM settings don't work for switch GSM to per game and use settings that work for that game.

I see.
I think right now, since this feature is quite new, there will be some confusion for most end-users.
You see right now, when users had "let's say the CHEAT ENGINE enabled for a game" and if they browse that Cheat menu right now for that game -- they will find that it says it's not enabled and the GLOBAL settings is enabled.
So in order to fix this so the menu shows the true setting found from their game's CFG settings, they have to switch GLOBAL to Per Game so it finally shows the correct setting -- that they had enabled the CHEAT engine for that game.
Plus I also tested running the game without making the adjustment above, and found the cheats did not load for the game, so it seems that I will have to switch to the Per Game Cheat setting for each game that has Cheats in order for the Cheat Engine to be turned on.  NOTE: not all my games have game cheats -- so this is why the per game cheat feature was ideal for me.
So the other issue is that this new change will now make a lot of end-users have to adjust (again) all their settings for Cheats, GSM, and PADEMU for this to work...
So, can we change this?  Can we not make Global be by default when they first try this new release?  But give them that option to enable it, once they've read more about it and know if this settings fits them or not?
I think it would have been better (still not sure, since it's a new feature) that the GLOBAL setting would not be set to default, but instead per game for each of those settings, and then allow the end-user to choose to enable GLOBAL, once they learn more about it.
Another question, since I know it will come up in the help topics.  Let's say some people like using different GSM Video Settings for certain games.  Let's say some games they have it set to play at 480p and then they have a few games set to 720p.  How will that affect them if they set GLOBAL?   -- which of the two video modes will be the GLOBAL setting?
Or those that only want their PAL Game to be GSM forced to NTSC or vise-versa?

When the GUI reads GLOBAL you are setting and saving Global settings and vice versa, so whatever the Settings Mode switch is set to is what you are adjusting/saving/loading, per game, per feature.
I see what you are saying, but from my understanding (and the people that were involved in the discussion) most peoples Per Game Configs have the same settings for the vast majority of their games.. so really all you need to do is set Globals and then for certain games switch that Settings Mode switch and save, it will load your already saved Per Game settings (if you already have a cfg for that game).

I think that is where the issue is.
Currently it seems that the new feature is not reading the older Per Game Settings that were done in the past prior to this new feature.
So right now, all my games that I had Cheats or GSM (custom settings) are disabled.  I have to manually go into them, and toggle from Global to Per Game so the older CFG settings are read for that particular module.
For example:
I don't want to turn on GSM globally since only a very few games had GSM settings and they were a bit different (customized settings) in each case.
Plus the same goes with Cheats, only a few had them (about 25%), not all of them.
And the problem is right now, I have to go back and readjust all those so they can be enabled, since the new feature right now is ignoring all those past per game settings.

That's how it works, that was the best implementation moving forward.. unfortunately the idea of Globals is to be...well...Global and ON by default otherwise it defeats the purpose.. I can see your issue but unfortunately already made Per Game cfgs aren't going to have the CFG Key since they never existed before.
You could potentially use a batch file or something to add the keys if you wish to use all existing Per Game cfgs

I see, well, okay.  No problem.
You read my mind, the batch file or even OPLM Manager (after an update) might assist with that, at least to enable all the per game settings I had for my games and it might help other in the same boat as I find myself in right now.
So the task will be to:
Finds this line:
$EnableCheat=1
Then add this new line after it:
$CheatsSource=1
My guess that line represents the Per Game Setting?  Right?
And simply do the same for both GSM and also PADEMU.
What is the string for the Global setting?  So it makes sure to only follow what's found inside the conf_game.cfg file inside the OPL folder.
I'll inform the troops once we release the new version later on in a few hours.  I can already predict some incoming new help topics:  "Help!  My cheats are not working in the latest OPL version!"  "Help!  My GSM settings are not working in the latest OPL version!" and etc.
I think I will tell them that if they have lots of games that use the same settings, to enable the Global and set their settings accordingly to each one, then manually set those to Per Game for those with unique settings and etc.

Actually... We EVER HAD Globals (all 'off' or 'generic')! We just couldn't adjust them before! ;)
Some things to note: That's why I said, the Cheat-Functionality (specifically regarding support in the GUI) needs some rework as well! It's not the fault of the new implementation, but due to errors which were made in the past...
I once had a conversation with @sp193, where I stated something along the lines of that 'I will just let them do however they want to, because the future will prove me right!'... I wouldn't have thought that @KrahJohlito and @Tupakaveli would be those who would be the ones to prove some of these things tho', haha. ^^
Regarding Cheats: IMO that needs it's own menu with selection of cheats and some of the current cheat-settings could become redundant as well!

You know what's funny?   can be served ... :P )
(2) I failed to recognize the potential inefficiencies of Per-Game at the time I was slogging everything under that umbrella. I do remember a handful of people were MAD about Per-Game GSM (because it was a Global variable before) and lobbied for a choice between Global and Per-Game at the time, but Doc, SP193, and I were hearing none of it, and it mainly stems from the three of us having small hard drives and thus smaller gaming libraries. Going through all 15 to 20 games on our lowly 80GB drives?!? Didn't bother us three goony birds all that much - took about 8 to 10 minutes! But woe to the hell on Earth it foisted on those that had joined the 1TB or 2TB club! LMAO
We failed to recognize that those gamers (i.e those that could afford such large drives back then) would become the prevailing majority over time, especially once 1TB or 2TB drives hit the $50-$80 price range for us poor bastards ... ;)

So the task will be to:

Finds this line:
$EnableCheat=1
Then add this new line after it:
$CheatsSource=1
My guess that line represents the Per Game Setting? Right?
And simply do the same for both GSM and also PADEMU.
Yes, that is correct

What is the string for the Global setting? So it makes sure to only follow what's found inside the conf_game.cfg file inside the OPL folder.

There isn't one, the cheat engine for example first checks your per game cfg for $CheatsSource=1 and if it exists continues to load settings from there.. otherwise it uses conf_game.cfg.
@BatRastard

how it killed my previous work

I know you explained your thought process but I just wanted to single this bit out, I hope you don't still feel this way (it seems like you don't).. I think of it as expanding on your work rather than killing it, Per Game is still in there and still very useful.
If we only had Global and came across a game that didn't work with those settings it would be a PITA to change every time just for that game, that's why I think the current system works quite well together.. The initial 'getting used to it phase' will be annoying for some but I think overall its a good system.

Hm... What about a check, if global settings (file[s]) is[/are] present and if there is non, it defaults to the 'off-internals' or per-game-settings if that file is present?
Would that be a feasible solution or rather not?

@Jay-Jay-OPL
Dekkit was kind enough to throw together a batch file, get it here
https://www.psx-place.com/threads/open-ps2-loader-v0-9-3.13415/page-46
The soul purpose of this BAT is to bring per game cfgs that were made with the old system up to date with the new system without having to do it through the GUI for tonnes of games, so it is only needed to be used once and if you have a large collection.
As for OPLM it doesn’t need a batch converter but it would need to add the new keys when creating per game cfgs if the user wanted it to auto enable per game.

Thanks, we are also moments away from releasing a fix with OPLM, this is why we didn't go the BAT route.
We also noticed that during a previous pull that MODES 7 and 8 were finally removed from the GUI, so we are also contemplating whether we should also create an update to remove those modes from user's CFG settings or leave them there for older OPL compatibility builds and just add a notifier in the CFG Editor that they are obsolete (for now)?  Still unsure if we should...?
Yes, we plan to add the new keys to users that opt to use Per Game settings with OPLM.
Thanks again...

@KrahJohlito  -- No, I don't feel that way. Quite the opposite: not only does it expand on my work, it actually corrects a major blind spot and oversight: most hard drives less than 250GB are 8 to 10 years past their MTBF, and 500GB+ hard drives have been the new normal for quite some time. For the sake of efficiency and common sense, giving us a choice between Per-Game and Globalized for GSM, CHEATS, and PADEMU is necessary. Hell, I've got a 500GB laptop drive in my PS2 now with 70+ games and it's only half full. I see myself moving away from Per-Game in short order! :)


Slower on Radeon R9 M370X discrete GPU than Intel Iris Pro on 2015 MBP

The draw step is about 3x slower on my discrete GPU than my integrated one. Why?

This issue affected Pathfinder 1, which is now obsolete. Please feel free to reopen if this is still an issue with Pathfinder 2.


Update release issue template

Signed-off-by: Zelin Hao<EMAIL_ADDRESS>Description
Update release issue template based on the 2.4.0 release issue https://github.com/opensearch-project/opensearch-build/issues/2649.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2930 (1daa646) into main (565735b) will decrease coverage by 0.30%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2930      +/-   ##
==========================================
- Coverage   93.00%   92.69%   -0.31%
==========================================
  Files         165      165
  Lines        4518     4518
==========================================
- Hits         4202     4188      -14
- Misses        316      330      +14

Impacted Files
Coverage Δ

src/system/temporary_directory.py
64.44% <0.00%> (-28.89%)
:arrow_down:

src/system/os.py
93.75% <0.00%> (-6.25%)
:arrow_down:

src/paths/script_finder.py
100.00% <0.00%> (ø)

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


C# scripts reload in editor fixed

Co-authored-by: Michael Bickel<EMAIL_ADDRESS>This fixes the problem when trying to run the same script more than once in the editor (see #48590).
Originally started by in @dazKind in #48627. Changes were required to the merge the PR but unfortunatly he did not have the time to make them and the problem was never fixed. So I made the changes that were asked in his PR.

Done! Code style should be fixed now.

Looks good to me, but maybe you could amend the commit message to remove the mention to clang-format - passing style checks is the rule, not the exception, so is not worth mentioning in the commit message.

Any additional changes needed to merge this PR? It has been a few days since I commited the required changes. I am guessing this is several commits behind the official repo branch by now.

Thanks!
This should likely be changed in master too?

Yes. I did not use master branch because C# is quite unstable right now, but it would be nice to have the fix in godot 4 too.

Whoops, I didn't realize this was made against the 3.x branch. It's needed in master too, but no hurries. I'll add it to my list just in case to not forget.

Cherry-picked for 3.3.4.


Update MD file on new branch 2017-11-30_01-32-33

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

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


Add a tutorial

Add a quick tutorial for users who first launch the app, so they can see how to use it.

Some sort of OOBE makes sense so users can understand what it does.

Some sort of OOBE makes sense so users can understand what it does.


Added some documentation on how to set up on citycloud + flavor

Added openstack/citycloud flavor and added some snippets I used to set it up with.

Can we add something similar in compliantkubernetes.io?
Ideally, it should only require copying the MD in the right place, updating
mkdocs.yaml and checking with mkdocs serve.
On Mon, 1 Feb 2021, 13:27 Fredrik Liv<EMAIL_ADDRESS>wrote:

Added openstack/citycloud flavor and added some snippets I used to set it
up with.
You can view, comment on, or merge this pull request online at:
https://github.com/elastisys/compliantkubernetes-kubespray/pull/36
Commit Summary

Added some documentation on how to set up on citycloud + flavor

File Changes

M bin/init.bash
https://github.com/elastisys/compliantkubernetes-kubespray/pull/36/files#diff-2ad97c0cf99389f194621d4309036d5855241250db7714be8a1ac8eeae6a8d07
(10)
D config/common/group_vars/all/ck8s-all.yaml
https://github.com/elastisys/compliantkubernetes-kubespray/pull/36/files#diff-12cf9e4d9987e93079f4703b11575a5cb348afa44534381b7a4b2e0c3969e59c
(0)
A config/openstack/group_vars/all/ck8s-openstack.yaml
https://github.com/elastisys/compliantkubernetes-kubespray/pull/36/files#diff-3f3ba18bcff0da8725d9d059e68d3ec0b4371aef941fcc0053fe8b5753d634e5
(1)
A
config/openstack/group_vars/k8s-cluster/ck8s-k8s-cluster-openstack.yaml
https://github.com/elastisys/compliantkubernetes-kubespray/pull/36/files#diff-aa21c9de7d063dca7c2fec3f69cb5bdbf13779d1e6ec360a18f7d4b8a9f185a5
(2)
A docs/citycloud-snippets.md
https://github.com/elastisys/compliantkubernetes-kubespray/pull/36/files#diff-fc545d52b145b437ab3d2c617e600ab401757f821a6963cc406ba9b412ab58d9
(130)

Patch Links:

https://github.com/elastisys/compliantkubernetes-kubespray/pull/36.patch
https://github.com/elastisys/compliantkubernetes-kubespray/pull/36.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/elastisys/compliantkubernetes-kubespray/pull/36, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMVPILU4OMCQQM63GKDDQ3S42M4LANCNFSM4W4WN42A
.

Can we add something similar in compliantkubernetes.io? Ideally, it should only require copying the MD in the right place, updating mkdocs.yaml and checking with mkdocs serve.

Yes, this was my end goal. But I think we might want to actually take some more time to polish this before we add it there. So I thought I just add this here for now so people that need to set it up have something to copy-paste. And then we can create an issue and polish it for compliantkubernetes.io


GPU for Model Inference + Real time transcription

Hi Mohit,
Thanks for providing this pre-trained model.
I have a few questions on this:

Is there a way to run this model on GPU for inference?
Can we utilize these model weights to do real-time transcription using audio from the mic?

Thanks!

Is there a way to run this model on GPU for inference?
The model is trained using T2T (tensor2tensor) and exported using their in-built functionality. You would need to see checkout their docs if there are any specific flags for enabling inference on GPU.

Can we utilize these model weights to do real-time transcription using audio from the mic?
At the moment, no. We're working on a real-time model. Would still take a few weeks before we can release a good enough model for general consumption.

I think this issue should remain open until someone finds a way of doing this. GPU inference should be much faster than the current method.

Contributions are welcome! 😄

So I installed CUDA 10.0 (required for tensorflow-gpu==1.14.0), and I got it to run on GPU. It raises the following error:
Unimplemented: Generic conv implementation only supports NHWC tensor format for now

I'm unfamiliar with Tensorflow so I have no idea what caused it, and searching for it returned nothing...

Hi mohitsshah, is gpu based model available now.


ISO String

The ability to set the return format of the dates would be great.
Currently I am having to convert them after the return which is a little untidy:
return await sslChecker(domain)
    .then(res => {
        /** Make sure we return a valid date format for the frontend */
        res['valid_from'] = new Date(res.valid_from).toISOString();
        res['valid_to'] = new Date(res.valid_to).toISOString();
        return res;

Good One, Will do that this weekend and will add typings (typescript) as well :)

Great, thank you so much. Happy to help test.

Made a new release with your request, Small breaking changes worth to check 😉
Feel free to re-open the issue for any updates
Thank you for reporting again

Sorry for the late reply. I did implement this and sort out the breaking change and all works perfectly. Thank you so much


Custom generator, class provider etc should not support parameters.

Current design of annotations for CustomXXX(XXX.class) supports parameters. It could brings another problems which should be avoided by helper annotation for empty parameters.

Added helper annotation Empty as fix of this problem.


svml: implement {mm,mm256,mm512}{asin,atan}{ps,pd}

Codecov Report

Merging #334 into master will decrease coverage by 0.05%.
The diff coverage is 78.57%.

@@            Coverage Diff             @@
##           master     #334      +/-   ##
==========================================
- Coverage   89.67%   89.61%   -0.06%
==========================================
  Files          38       38
  Lines        5723     5751      +28
  Branches      910      922      +12
==========================================
+ Hits         5132     5154      +22
  Misses        317      317
- Partials      274      280       +6

Impacted Files
Coverage Δ

simde/x86/svml.h
87.05% <78.57%> (-0.95%)
: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 cafec4b...73f569d. Read the comment docs.


e2e: Update skip docs check to get just PR diff

What this PR does
The previous git check would lead to a large diff if the contributor's fork falls behind kubevirt main[1].
Using the diff based off a common ancestor of both commits[2], returns the
diff for the pull request.
[1] https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/11286/pull-kubevirt-e2e-k8s-1.29-sig-compute-migrations/1762051220236668928#1:build-log.txt%3A33
[2] https://git-scm.com/docs/git-diff#Documentation/git-diff.txt-emgitdiffemltoptionsgtltcommitgtltcommitgt--ltpathgt82308203-1

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer
/cc @dhiller @oshoval @fabiand

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

NONE

Thanks, nice
/lgtm

Btw, we can extend the logic, that if only unit tests are changed we also don't need to run all those e2e lanes

Seems it doesnt work on rehearse
https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_project-infra/3333/rehearsal-pull-kubevirt-e2e-windows-sysprep/1774339166738321408

Seems it doesnt work on rehearse https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_project-infra/3333/rehearsal-pull-kubevirt-e2e-windows-sysprep/1774339166738321408
maybe easiest solution is to detect the specific error and test if met fatal: Invalid symmetric difference expression x...y wdyt?

Its an issue with the rehearse plugin - it passes the git refs from the project-infra PR which don't exist in the kubevirt repo.

Ic, but we can just solve it easily until the real thing is fixed imho

Ic, but we can just solve it easily until the real thing is fixed imho

Yes sure - there are a few ways to fix it. I think Daniel was taking a look at it here - https://github.com/kubevirt/project-infra/pull/3275