[CT-119] [Bug] run_results.json schema validation error with oneOf constraint on status

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Trying to validate the run_results.json artifact produced by the dbt test command, I used the jsonschema python package and the schema you expose here (run-results/v3.json).
The validation fails with the error below.
Expected Behavior
As the artifact and the json schema are produced by dbt, I expect the schema validation to be OK.
"oneOf" constraint seems to be unadapted as a status could appears in several enum.
Steps To Reproduce

Run dbt test
In a python3 console, run the following lines:

import jsonschema
import requests

dbt_runresults_schema = requests.get(url='https://schemas.getdbt.com/dbt/run-results/v3.json').json()
runresults_rawjson=json.load(open('path/to/target/run_results.json'), encoding="UTF-8"))

validate(instance=runresults_rawjson, schema=dbt_runresults_schema)

Relevant log output
>>> validate(instance=runresults_rawjson, schema=dbt_runresults_schema)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.9/site-packages/jsonschema/validators.py", line 934, in validate
    raise error
jsonschema.exceptions.ValidationError: 'warn' is valid under each of {'type': 'string', 'enum': ['pass', 'warn', 'error', 'runtime error']}, {'type': 'string', 'enum': ['pass', 'error', 'fail', 'warn', 'skipped']}

Failed validating 'oneOf' in schema['properties']['results']['items']['properties']['status']:
    {'oneOf': [{'enum': ['success', 'error', 'skipped'], 'type': 'string'},
               {'enum': ['pass', 'error', 'fail', 'warn', 'skipped'],
                'type': 'string'},
               {'enum': ['pass', 'warn', 'error', 'runtime error'],
                'type': 'string'}]}

On instance['results'][0]['status']:
    'warn'

### Environment

```markdown
- OS:Debian GNU/Linux 10 (buster)
- Python:3.9.6
- dbt:0.21.1
- jsonschema:3.1.1

What database are you using dbt with?
redshift
Additional Context
As a workaround I changed in the schema the "oneOf" constraint of
"properties": {
        "status": {
          "oneOf": [
          ...

by a "anyOf" constraint which seems more relevant.

👋 @indyyyyy Sorry it's taken a min to get back to you on this.  I was able to replicate the issue-- it seems our jsonschema could use some love.  I'll categorize this as a bug and we'll get it prioritized ASAP.

For future reference here's a slight update to fix some typos in the reproduction code posted above:
import jsonschema
import requests
import json

breakpoint()
dbt_runresults_schema = requests.get(url='https://schemas.getdbt.com/dbt/run-results/v3.json').json()
runresults_rawjson=json.load(open('/path/to/run_results.json'), encoding="UTF-8")

jsonschema.validate(instance=runresults_rawjson, schema=dbt_runresults_schema)

We identified some limitations of the library (hologram) that we're using to generate JSONSchemas from internal dbt objects. As such, while we could make this fix as one-off, what we'd really need is a more rigorous conversion library and validation testing.
I hesitate to prioritize that larger effort over a continuing investment in structured logging, which we believe can offer a more powerful, reliable, and real-time metadata interface going forward.

Improvement to the accuracy of our jsonschema generation isn't something we're able to prioritize currently. We should document that the JSONSchemas at schemas.getdbt.com are better understood as descriptive of the information contained in the JSON artifacts, rather than usable for rigorous validation of every artifact. In the meantime, we'll continue discussion in #4617 about how to create some healthy distance between dbt-core's internal classes and the artifacts it produces for external consumption.


[types] Define a enum that represent both system transaction and user transaction

Motivation
This is the first step for #1173 to separate system transaction from SignedTransaction submittable by the user. In the PR we define a new enum to represent all variants of system transaction and user transaction that one full node should be able to process, persist and replay.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
We only introduce the definition here.

I got some clippy error for size difference between variants. Should we suppress the warning here just for sanity?

@zekun000  Since ideally most transactions will be user transactions, those being the largest variant should be Ok?

@zekun000 Since ideally most transactions will be user transactions, those being the largest variant should be Ok?

Yea that's what I thought as well. Getting rid of the Box may seems a bit cleaner from the code stand point but I might be wrong. Any preference? cc @wqfish

I'm fine to silence the clippy with a comment.

:v: @runtian-zhou can now approve this pull request

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

@bors-libra r+

:pushpin: Commit 6b8c3bf has been approved by runtian-zhou

:hourglass: Testing commit 6b8c3bf947c5c17d9b38990a4f8f175203e51de7 with merge 84550a27ebd2f98ec37cf70dc77e2e321cdb6a2f...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: runtian-zhou
Pushing 84550a27ebd2f98ec37cf70dc77e2e321cdb6a2f to master...


Enforce the locale to be set to English

Summary
As reported in #136, we could have a mix of translations when this default_locale is set to another language than English.
This PR resolves this by adding a around_action that set's the locale to English during the execution of the action.
I've also removed a docker volume and the version from the docker-compose.yml because the volume isn't being used, and the version is obsolete.
Fixes #136
Before

After

Thank you so much again! 🙏

This will cause error I18n::InvalidLocale in MissionControl::Jobs::QueuesController#index for websites that don't have the English locale available.


fix: Move files in /usr/etc to /etc

Adding files to /usr/etc is considered undefined behavior

Ok I've removed all mentions of the etc-merge service. Please doublecheck my work.

This good to go or are we still sorting out that containers.json thing? (Does that even affect Bluefin since we're not using the rechunker?)

This good to go or are we still sorting out that containers.json thing? (Does that even affect Bluefin since we're not using the rechunker?)

@castrojo The containers.json thing is only affecting main and ucore, since this is where we add the container policy to the image.  The issue there is we want to move the policy from /usr/etc to /etc, but Fedora's containers package already has taken "ownership" of that file, and doesn't allow us to overwrite it from our own RPM.
This has been reverted for now, but I don't think we have any issues here.


Not able to connect to ATEM Mini

Hi there,
have a problem with connection. ATEM Mini found but no access.
I have MacOS Catalina.
Thank you in advance for support.
Yuri

Atems only support control via ethernet.

Hi there,
have a problem with connection. ATEM Mini found but no access.
I have MacOS Catalina.
Thank you in advance for support.

George Maistoi


Italian language update

@mega5800
@activescott
:+1::tada: First off, thanks for taking the time to contribute! :tada::+1:
Please fill out the following checklist:

[ ] Added tests for any bug fixes that changed existing code
[ ] Added tests for any new behavior
[X ] All unit tests are passing (please make sure all tests are passing for your branch/PR at https://ci.appveyor.com/project/activescott/lessmsi/history )

If you need any help at all, feel free to submit the PR and @-mention activescott and I'll be happy to assist!

Hello @bovirus.
Let me help you with updating the tests as you changed the displayed Italian text.

@bovirus, here is the place you need to update: https://github.com/activescott/lessmsi/blob/18356d0b08ca1d026bd76ac0cdfe5f0b25da8e5b/src/Lessmsi.Tests/GUITests.cs#L50
I believe you only need to update the tests in lines 51-52, by providing  the updated Italian texts for these tests.

@mega5800
Sorry but I didin't understanmd what strings I should update.

@bovirus,
In the GUITests.cs file perform the next changes:

Line 51:
Change this line from: Assert.Equal("Selziona &Tutto", form.btnSelectAll.Text);
To: Assert.Equal("Selziona &tutto", form.btnSelectAll.Text);

Line 52:
Change this line from: Assert.Equal("&Deseleziona Tutto", form.btnUnselectAll.Text);
To: Assert.Equal("&Deseleziona tutto", form.btnUnselectAll.Text);

Hope this helps.

@mega5800
Selziona &Tutto -> Seleziona &tutto
But GUITests.cs is not master file tht should have strings in English?

@bovirus,
As the name suggests, GUITests.cs is a file for testing the GUI, nothing to do with master English strings.
Please, apply the changes I suggested in my previous comment, which I updated according to your Italian translation.
If you have any difficulties doing so, please open a ticket with the changes you've made so far, your desired behaviour and I will take care of that.

I'm closing this one since it appears to be stale. If ya'll want to work on it though feel free to re-open it or open a new one.

@activescott
Sorry for the delay.
For me all is OK.


GEOMESA-643 Exclude Yammer

yammer.metrics is excluded from kafka

12f8f0a57c000d577a8b295b8a5430ed03b84507

@mm64z see comments in https://geomesa.atlassian.net/browse/GEOMESA-643 I reopened the PR

Bad news: the kafka data store doesn't work without metrics. Can't read or write features.

I've requested that Kafka (and this dependency) be considered for 'works with' status.  If that happens, we will change these to provided dependencies and this PR will be OBE.

I'm closing this PR.  Either Kafka will be 'works with' or this ticket would break the Kafka DataStore.
Either way, we won't be merging this code.


create apko YAML from an SBOM

It would be neat to generate apko YAML files from an SBOM, something like:
syft packages alpine:latest --output cyclonedx-json | apko import -f cyclonedx > alpine-latest.yaml
But it seems like these SBOMs don't capture repository lists, or what is actually an /etc/apk/world dependency.  Maybe we can work with Anchore on this?

Let's do it!
We've talked before about the possibility of re-hydrating artifacts from SBOMs. (This came up in an sget conversation a couple times, I think). I think it's super cool.
If Syft started capturing repositories and world dependencies, would that be enough information to create a suitable apko YAML file?

Yeah, absolutely!


Added JWT-PreProcessor plugin details and wiki page.

Hi Team,
I developed a Jmeter PreProcessor plugin for JWT generation and I would like to publish it under the Jmeter-plugin manager. Please review my pull request.
Plugin Repo : https://github.com/sashikaR/jwt-preProcessor

Look at the CI failure. Can you compile your plugin using older JDK requirement? This will ensure more people can use it.

The recommended version is  Java 11

Hi, Please review the PR with the new build.

Hi, Updated the Manifest file in the plugin build. Please re-run the build.

Hi undera,
I need some guidance from you to resolve this issue. The class which I gave in markerClass is the implementation of PreProcessor interface and AbstractTestElement class in my plugin. So there is no need to define the main method in that class since it is trigger by the Jmeter application. Could you please guide me what is the class I should give in markerClass and installerClass?
Previously in my plugin's manifest included the Main-Class and I changed the build to remove it now. So not sure why the build is failing with main method is not found error.
Thank you.

If your plugin does not have custom installation procedures, you can just omit installerClass from descriptor

Hi,
Please run the build now. I removed the installerClass as suggested.
Thanks.

Looks good now.
Thanks for publishing your plugin!

Hi Andrey,
Thank you for your valuable feedback and support. It is my pleasure to
publish my plugin in jmeter plugin-manager.
Thank you.
On Tue, Aug 15, 2023 at 7:59 PM Andrey Pokhilko @.***>
wrote:

Merged #608 https://github.com/undera/jmeter-plugins/pull/608 into
master.
—
Reply to this email directly, view it on GitHub
https://github.com/undera/jmeter-plugins/pull/608#event-10096314802, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AK4QS7KU6BQ3Z7QOGYLOFH3XVNQBTANCNFSM6AAAAAA3P2B77Y
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Sashika Rangoda
BSc. (Hons) in ICT
University of Colombo


static code analysis

This issue lists suggestions for static code analysis with goal to ensure higher code quality.

Have single source for linting rules symmetrical in CI/CD and local environments
Bee code currently has three different rules and only one of those is enforced by CI. Problems with this way of linting: a) it causes confusion what are rules to be followed, b) rules which are not enforced by CI are going to be forgotten  c) rules executed in CI/CD should be symmetrical to local (so devs can check code locally)
What could be done: have single source for all linting rules in .golangci.yml and single make lint command to run those lints on entire code base.

Trailing white space check only works in CI
This check seems to be only working in CI on diff changes. It would be easier for devs if this check is symmetrical in CI/CD and local environments.
What could be done: implement command make whitespace-check which would check entire code base.

Increase golangci-lint strictness
In order to ensure higher code quality bee code base should consider including more linters. Below are suggested linters bee should include:

3.1) Compliant rules list
List includes rules which bee can start using right away because code is compliant with them.
    - asciicheck
    - bidichk
    - deadcode
    - depguard
    - errcheck
    - gofmt
    - goheader
    - gomoddirectives
    - goprintffuncname
    - gosimple
    - govet
    - importas
    - ineffassign
    - megacheck
    - misspell
    - misspell
    - nakedret
    - nilerr
    - promlinter
    - rowserrcheck
    - sqlclosecheck
    - staticcheck
    - structcheck
    - typecheck
    - unconvert
    - unconvert
    - unused
    - varcheck

3.2) Easy to fix list
List of rules which could be easily fixed (there are few problems in the code witch needs to be addressed)
  - errorlint ##important
  - errname
  - dogsled
  - bodyclose
  - exportloopref
  - forbidigo
  - goconst
  - gochecknoinits
  - gofumpt
  - nilnil
  - nlreturn
  - noctx
  - prealloc
  - predeclared
  - wastedassign
  - whitespace

3.3) Test related
List includes rules which affect how tests are written. These seem to be relatively easy to be fixed.
 - paralleltest ## important **
 - tparallel
 - testpackage
 - thelper

**  Using parallel test can reduce total test execution time. At the same time it enforces tests quality (for example using global variables in those tests can cause race condition).
3.4) High impact rules
These rules have high impact on code quality. Fixing some of those need more effort compared to linters above. Since fixing all issues could require bigger effort, some issues could be fixed and some temporally ignored in order to get these rules enabled for future code changes.
  - cyclop           # many issues; improves code readability; can start with some higher value then reduce it
  - dupl             # many issues; improves code quality
  - gochecknoglobals # many issues
  - gocognit         # many issues; improves code readability
  - gocyclo          # many issues; improves code readability
  - gocritic         # many issues; bugs, performance and style issues.
  - goerr113         # uniform error handling
  - nestif           # improves code readability
  - revive           # style issues; improves code readability (make code uniform)
  - stylecheck       # style issues; improves code readability (make code uniform)
  - unparam          # style issues; code quality
  - gomnd            # style issues; improves code readability
  - makezero         # performance
  - wrapcheck        # many issues; improves error handling
  - ifshort          # style issues; improves code readability

License header
There is no tool for checking code license in .go code. As it currently requires developers in code review process to ensure this it can lead to code being pushed without this header.
What could be done: implement command make header-check which would check if all files have necessary headers.

megacheck

Is deprecated.

rowserrcheck
sqlclosecheck

Only relevant to SQL.

bodyclose

Not needed for server-side since: https://github.com/golang/go/blob/master/src/net/http/request.go#L174-L177

gofumpt

We should stick with gofmt.

nilnil

Is too restrictive, consider a valid case where you return a nil (a.k.a empty) slice and no error: return nil, nil.

nlreturn

I'd drop this since It's opinionated.

gocyclo
cyclop

Those two seem to be doing the same.

revive
stylecheck

Not sure what they bring on top of the golint?

ifshort

Also, this seems too restrictive, sometimes you just need to do it on a separate line because in one line it's too long.

License header
There is no tool for checking code license in .go code. As it currently requires developers in code review process to ensure this it can lead to code being pushed without this header.
What could be done: implement command make header-check which would check if all files have necessary headers.

The goheader linter you've mentioned above does this: https://github.com/denis-tingaikin/go-header.
Overall, I would agree with the linters in sections 3.1, 3.2 and 3.3, except for those I specifically mentioned. However, some of the linters in section 3.4 are opinionated/restrictive, so I wouldn't recommend including them at this time and we can revisit them later.

Closing this Issues, after enabling all linters


Fix rotate buttons in animation example

The animation-example does not show icons for the rotate-buttons.
This PR suggests using two unicode characters for these buttons: ↻ and ↺.
I also added a title-attribute to the buttons.

Before:

After:

Thanks! Please merge.

Cheers for the review, Éric.

icons in http://openlayers.org/en/master/examples/brightness-contrast.html, http://openlayers.org/en/master/examples/export-map.html, http://openlayers.org/en/master/examples/hue-saturation.html are broken too.
font-awesome was updated to version 4.x but the css classes must be changed from icon-xxx to fa fa-xxx

You want to do it @fredj?

I hope we can work toward not relying on Bootstrap, Font Awesome, etc. in the "body" of the examples.  Instead, I think it would be nice if the copy/paste part of the examples was vanilla JavaScript + OpenLayers with elements that have an acceptable default style .  In our hosted examples, we can add custom styling, but people should be able to reproduce them without loading external libraries.

The changes in this PR are headed in that direction, or not?


update to docker v1.10.1

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:upmio/docker.git somewhere
$ cd somewhere
$ git rebase -i HEAD~3
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.


Istio CNI for Istio-1.1 Readiness Tracker

Using this issue as a landing spot to track readiness for Istio release 1.1.  The following work items are mandatory for 1.1.
Repo Issues

[X]  Owners permissions--not everyone in owners file have permissions to do "owners" stuff

[X] assign issues, assign reviewers
[X] approve PRs
[X] merge PRs

[X] CircleCI project hookup.  No cni sub-project under istio in CircleCI so no jobs are firing (Costin working) - Done jobs running now
[X] DCO check never completes (remove DCO check) - Removed the DCO check

Release Readiness Items

[X] hosting "release" candidate docker images

[X] nightly circleci job will build & push to docker.io -- #53
[X] keep decoupled--figure out how to generate & host release artifacts (container images, trigger helm chart publish) https://github.com/istio/istio/pull/9973

[X] hosting "release" helm chart

[X] istio-cni chart packaged by https://github.com/istio/istio.io/blob/master/scripts/grab_charts.sh  (https://github.com/istio/istio.io/pull/2827)
[X]  istio/istio github_release.sh package helm charts (https://github.com/istio/istio/pull/9496)
- Needs to be a specific tag need coordination on tag value

[X] Prow jobs

[X] istio/istio change triggered  -  Need PRs https://github.com/istio/test-infra/pull/1020, https://github.com/istio/istio/pull/9577 & issue: https://github.com/istio/test-infra/issues/1018
[ ] ~istio/cni change triggered~ -  This is no longer the plan

CircleCI istio/cni

[X] istio/cni e2e - Using DinD approach from the kubeadm-dind-cluster (KDC) proj... see PR #62
[X] Unit tests, lint
[X] Functional tests

[x] Documentation - https://github.com/istio/istio.io/pull/2902

[x] Add info to istio.io (how to doc for alpha features?) -- https://github.com/istio/istio.io/pull/2902

[x] installation options
[x] hosted k8s specific info
[ ] troubleshooting

[X] istio/cni README

[X] Clarify installation & current implementation behavior,  add new use cases

[x] Necessary changes in release 1.1 branch

Nice to have features or functionality for 1.1 release.

[ ] Finalize use-case scenarios

[ ] upgrade
[ ] coexistence

[ ] ~Prow job triggered by a change in the istio/cni repo -- https://github.com/istio/cni/issues/8~

For alpha features, just add the documentation as normal and the feature will be marked alpha in the feature status page here prior to release:
https://preliminary.istio.io/about/feature-stages/

All required milestones have been met,  although troubleshooting is maybe a still a bit weak.


Regarding stop button functionality

i have a button of play and stop the play pause functionality working fine but when i click on stop button it resets everything which works but when i play again nothing happens..please help

I have faced same problem in my current project but i have handle like this.
player?.event.playbackEnd.addListener(self, handleAudioPlayerStatePlayBackEnd)
func handleAudioPlayerStatePlayBackEnd(state: AudioPlayer.PlaybackEndEventData) {
            if state == .playedUntilEnd{
      // Reload your audio again and reset UI also
   }
 }
}

not working @sukh1993 sir,
my code is as follows, can you point me where i am wrong where to add what code
var audioURL:String?
private var isScrubbing: Bool = false
private let audioPlayer = AudioPlayer()
let audioSessionController = AudioSessionController.shared
override func viewDidLoad() {
super.viewDidLoad()
setupAudioPlayer()
configureTrackSlider()
createNowPlayingAnimation()
}
@IBAction func onClick_playAudioBtnTapped(_ sender: UIButton) {
if !audioSessionController.audioSessionIsActive {
try? audioSessionController.activateSession()
}
audioPlayer.togglePlaying()
}
@IBAction func onClick_stopBtnTapped(_ sender: UIButton) {
self.audioPlayer.stop()
self.nowPlayingImageView.stopAnimating()
}
private func setupAudioPlayer() {
audioPlayer.event.stateChange.addListener(self, handleAudioPlayerStateChange)
audioPlayer.event.secondElapse.addListener(self, handleAudioPlayerSecondElapsed)
audioPlayer.event.seek.addListener(self, handleAudioPlayerDidSeek)
audioPlayer.event.updateDuration.addListener(self, handleAudioPlayerUpdateDuration)
audioPlayer.event.didRecreateAVPlayer.addListener(self, handleAVPlayerRecreated)
audioPlayer.event.playbackEnd.addListener(self, handlePlayEnd)
audioPlayer.event.fail.addListener(self, handlePlayerFailure)
handleAudioPlayerStateChange(data: audioPlayer.playerState)
    if let url = self.audioURL {
        let item = DefaultAudioItem(audioUrl: url, sourceType: .stream)
        try? self.audioPlayer.load(item: item, playWhenReady: true)
        self.playPauseBtn.setImage(UIImage(named: "ic_audio_pause"), for: .normal)
        self.scaleInBookCover(isFirstLoad: false)
        self.nowPlayingImageView.startAnimating()
    }
}

// MARK: - AudioPlayer Event Handlers
extension AudioPlayerVC {
func updateTimeValues() {
    self.trackSlider.maximumValue = CGFloat(self.audioPlayer.duration)
    self.trackSlider.setValue(CGFloat(self.audioPlayer.currentTime), animated: true)
    self.currentTimeLbl.text = getTimeString(time: self.audioPlayer.currentTime)
    self.durationLbl.text = getTimeString(time: (self.audioPlayer.duration - self.audioPlayer.currentTime))
}

func setPlayButtonState(forAudioPlayerState state: AudioPlayerState) {
    if state == .playing {
        playPauseBtn.setImage(UIImage(named: "ic_audio_pause"), for: .normal)
        self.scaleInBookCover(isFirstLoad: false)
        self.nowPlayingImageView.startAnimating()
    } else {
        playPauseBtn.setImage(UIImage(named: "ic_audio_play"), for: .normal)
        self.scaleOutBookCover()
        self.nowPlayingImageView.stopAnimating()
    }
}

func handleAudioPlayerStateChange(data: AudioPlayer.StateChangeEventData) {
    print(data)
    DispatchQueue.main.async {
        self.setPlayButtonState(forAudioPlayerState: data)
        switch data {
        case .loading:
            self.startLoader()
            self.updateTimeValues()
        case .buffering:
            self.startLoader()
        case .ready:
            self.stopLoader()
            self.updateTimeValues()
        case .playing, .paused, .idle:
            self.stopLoader()
            self.updateTimeValues()
        }
    }
}

func handlePlayEnd(data: AudioPlayer.PlaybackEndEventData) {
    DispatchQueue.main.async {
        switch data {

        case .playedUntilEnd:
            if let url = self.audioURL {
                let item = DefaultAudioItem(audioUrl: url, sourceType: .stream)
                try? self.audioPlayer.load(item: item, playWhenReady: false)
            }
        case .playerStopped:
            self.audioPlayer.stop()
            self.stopLoader()
        case .skippedToNext:
            self.stopLoader()
        case .skippedToPrevious:
            self.stopLoader()
        case .jumpedToIndex:
            self.stopLoader()
        }
    }
}

func handleAudioPlayerSecondElapsed(data: AudioPlayer.SecondElapseEventData) {
    if !isScrubbing {
        DispatchQueue.main.async {
            self.updateTimeValues()
        }
    }
}

func handleAudioPlayerDidSeek(data: AudioPlayer.SeekEventData) {
    isScrubbing = false
}

func handleAudioPlayerUpdateDuration(data: AudioPlayer.UpdateDurationEventData) {
    DispatchQueue.main.async {
        self.updateTimeValues()
    }
}

func handleAVPlayerRecreated() {
    try? audioSessionController.set(category: .playback)
}

func handlePlayerFailure(data: AudioPlayer.FailEventData) {
    if let error = data as NSError? {
        if error.code == -1009 {
            DispatchQueue.main.async {
                print("Network disconnected. Please try again...")
            }
        }
    }
}

}

@nishant6042 try this its works for me
func handlePlayEnd(data: AudioPlayer.PlaybackEndEventData) {
    DispatchQueue.main.async {
        switch data {
        case .playedUntilEnd:
// pause audio player when  playedUntilEnd and reload next song or previous song
         self.audioPlayer.pause()
            if let url = self.audioURL {
                let item = DefaultAudioItem(audioUrl: url, sourceType: .stream)
                try? self.audioPlayer.load(item: item, playWhenReady: false)
            }
        case .playerStopped:
            self.audioPlayer.stop()
            self.stopLoader()
        case .skippedToNext:
            self.stopLoader()
        case .skippedToPrevious:
            self.stopLoader()
        case .jumpedToIndex:
            self.stopLoader()
        }
    }
}

not working @sukh1993 sir, i have done audiolayer.stop() in stop button click and in playedUntilEnd i have done as you said

And actually i my case theres no next song when current song ends it just have to start the same song again but when i click stop button the ui resets but when play again it doesnt do anything

pls help @sukh1993 sir

@nishant6042 hi
//If things is not working try this with some tricks
var isAudioStops : Bool = false

@IBAction func onClick_playAudioBtnTapped(_ sender: UIButton) {
 if isAudioStops == true{
   if let url = self.audioURL {
        let item = DefaultAudioItem(audioUrl: url, sourceType: .stream)
        try? self.audioPlayer.load(item: item, playWhenReady: true)
        self.playPauseBtn.setImage(UIImage(named: "ic_audio_pause"), for: .normal)
        self.scaleInBookCover(isFirstLoad: false)
        self.nowPlayingImageView.startAnimating()
    }
  isAudioStops = false
}else{
if !audioSessionController.audioSessionIsActive {
try? audioSessionController.activateSession()
}
audioPlayer.togglePlaying()
 }
}
@IBAction func onClick_stopBtnTapped(_ sender: UIButton) {
self.audioPlayer.stop()
self.nowPlayingImageView.stopAnimating()
self.isAudioStops = true // bool true
}

func handlePlayEnd(data: AudioPlayer.PlaybackEndEventData) {
    DispatchQueue.main.async {
        switch data {
        case .playedUntilEnd:
            self.isAudioStops = true // bool true
        case .playerStopped:
            self.audioPlayer.stop()
            self.stopLoader()
        case .skippedToNext:
            self.stopLoader()
        case .skippedToPrevious:
            self.stopLoader()
        case .jumpedToIndex:
            self.stopLoader()
        }
    }
}

not working still @sukh1993 sir

@nishant6042 check with break points, Is your player reload the url or not on play button. if It is not working change stop method to pause.

@sukh1993 sir working when doing pause instead of stop but doing that it pauses i want to restart from start when stop clicked

@nishant6042 I will working this issue, this is bug on swift Audio side.

So whats the alternate for now?

and also what should i do to play in background also when i leave the screen @sukh1993 sir

Hi @nishant6042
Are you manage cache?
means i load one audio and then play again, it will play from without load.
If you did this please let me know, how can be?

@nishant6042  also check this link :- https://github.com/jorgenhenrichsen/SwiftAudio/issues/53
about stop functionality


ignore KeyboardInterrupt in examples

Coverage remained the same at 87.889% when pulling d71f23910e2fabb47b266141d10ec77ebc3f7f2f on example-keyinterrupt-56 into 2bab06e540dc19d3e6dd7179d1d1a78589e7bf3f on master.


Update site returns 404

The update site specified in the readme (http://www.thymeleaf.org/eclipse-plugin-update-site/) is not working.

Guys looks like this is still an issue and what I find the best link so far is http://sourceforge.net/projects/thymeleaf/files/thymeleaf-extras-eclipse-plugin/2.1.1/update-site/   now  it should work hoping you have access to sourceforge downloads without any firewall issues or things like that. Thanks !

@rkand4 same problem for me, only direct url to springsource is working


fix(datastore): observequery local only collect

Issue #

https://github.com/aws-amplify/amplify-swift/issues/3100

Description

DataStore.observeQuery(...) has some internal logic to batch snapshots received based on total count or time elapsed while syncing. This logic is applied when DataStore thinks it is hasn't yet completed syncing.
When DataStore is in local only mode (i.e. when AWSAPIPlugin isn't being used), it is never "completed syncing", leading to the same throttling logic.
This change ensures that the throttling / collect logic is not applied when DataStore is in local only mode.
General Checklist

[ ] Added new tests to cover change, if needed
[x] Build succeeds with all target using Swift Package Manager
[x] All unit tests pass
[x] All integration tests pass
[x] Security oriented best practices and standards are followed (e.g. using input sanitization, principle of least privilege, etc)
[ ] ~Documentation update for the change if required~
[x] PR title conforms to conventional commit style
[x] New or updated tests include Given When Then inline code documentation and are named accordingly testThing_condition_expectation()
[ ] ~If breaking change, documentation/changelog update with migration instructions~

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #3214 (ef644ba) into main (659fdf9) will decrease coverage by 0.17%.
The diff coverage is 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main    #3214      +/-   ##
==========================================
- Coverage   64.74%   64.58%   -0.17%
==========================================
  Files        1090     1090
  Lines       36988    36998      +10
==========================================
- Hits        23948    23894      -54
- Misses      13040    13104      +64

Flag
Coverage Δ

API_plugin_unit_test
66.52% <ø> (ø)

AWSPluginsCore
69.55% <ø> (ø)

Amplify
47.84% <ø> (-0.02%)
:arrow_down:

Analytics_plugin_unit_test
86.39% <ø> (ø)

Auth_plugin_unit_test
75.19% <ø> (+0.03%)
:arrow_up:

CoreMLPredictions_plugin_unit_test
59.44% <ø> (ø)

DataStore_plugin_unit_test
80.15% <100.00%> (-1.01%)
:arrow_down:

Geo_plugin_unit_test
54.06% <ø> (ø)

Logging_plugin_unit_test
62.77% <ø> (ø)

Predictions_plugin_unit_test
35.17% <ø> (ø)

PushNotifications_plugin_unit_test
69.69% <ø> (ø)

Storage_plugin_unit_test
54.10% <ø> (ø)

unit_tests
64.58% <100.00%> (-0.17%)
:arrow_down:

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

Files Changed
Coverage Δ

...Plugin/Storage/StorageEngine+SyncRequirement.swift
80.18% <100.00%> (+0.38%)
:arrow_up:

...gin/Subscribe/DataStoreObserveQueryOperation.swift
83.17% <100.00%> (+0.65%)
:arrow_up:

... and 13 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Group Provider Label Did Not Match Expected Provider LabelGroup Name

time="2021-10-19T13:54:38Z" level=info msg="Group Provider Label Did Not Match Expected Provider LabelGroup Nameocp_adminExpected Labelldap-groupsync_ldapFound Label" source="groupsync_controller.go:140"
Seems there is no step to label a group in this operator README. According to the message, seems my group in ldap "ocp_admin" will not be synced because the label mismatch. It should have the label called ldap-groupsync_ldap but the fact is, it doesn't have any label.
This group is synced manually before but when trying to sync it using group-sync-operator seems it is skipped ?

Existing groups are not managed via the operator. If the group exists that you would like to sync, delete the group and the operator should resync

Closing due to staleness


Enable type checking for TS projects from eslint

Enable type checking for TS projects from Eslint in the Fiori project

Update minor version and consumption in project templates
Update eslint config in preview-middleware-client and address some eslint issues in the code.

Thanks all


Add the "Skills" section.

Description
The website needs a section presenting the website owner's skills. This section can be next to the "About" one.
Requirements
Requirements to close the issue:

[x] A list of skills is presented.
[x] #18

The skills are presented in the "about" page as bubbles with the technology logos inside.


AFD throws "CertificateNameCheckFailed" error in logs

AFD and app gateway internet-facing certificates are different from the ones I have internally on the backend behind the app-gateway.
I had to run:
az network front-door update --name myAFD --resource-group myRG
--enforce-certificate-name-check Disabled
to make it work.
Without it, afd throws "CertificateNameCheckFailed" error in logs. Log query:
AzureDiagnostics
| where ResourceProvider == "MICROSOFT.NETWORK" and Category == "FrontdoorAccessLog"
| where isReceivedFromClient_b == true
| where ErrorInfo_s <> "NoError"

@dariuszbz The Docs feedback is for the issues related to the document issues and making them better. Please share the document link of this is associated with a document.
As it looks more like an issue which needs troubleshooting, you can also raise this issue on our community forum on Microsoft Q&A

@dariuszbz Please convey the document that you are referring to for this feedback.

Sorry, my bad. No problems with the doc. The platform doesn't work as it
supposed to work in relation to the document. Again. Sorry to bother you.
Could you direct me to the proper forum, perhaps?
On Thu, 20 May 2021 at 16:29, NavtejSaini-MSFT @.***>
wrote:

@dariuszbz https://github.com/dariuszbz The Docs feedback is for the
issues related to the document issues and making them better. Please share
the document link of this is associated with a document.
As it looks more like an issue which needs troubleshooting, you can also
raise this issue on our community forum on Microsoft Q&A
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/75611#issuecomment-845222446,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEVINAX75CXNEBPK47JLPFDTOUTGLANCNFSM45GWCPYA
.

@dariuszbz You can raise your issue on Microsoft Q&A for these issues. For now we will be going ahead and closing this feedback.


Add let's encrypt support for pre-production domains

Summary
This adds the requested acme-challenges for the first four pre-production urls.
Metadata

Question
Answer

Did you use a meaningful pull request title?
yes

Did you apply meaningful labels to the pull request?
yes

Did you perform a self review first?
yes

Documentation reflects changes?
no

CHANGELOG reflects changes?
yes

Unit/Functional tests reflect changes?
n/a

Did you perform browser testing?
yes

Risk level
Low

Relevant links
n/a

@pfrilling Looks good. Just double checking those acme challenge strings don't need to be secured. Assuming that's the case this is /approved.

Actually, I didn't even need to merge this. I found a workaround to get the certificates issued.


List of URLs for retrieving data and documentation file with Methods

List access URLs and documenting methods of data gathering (when) and where
What is the data documentation (methods) [MetaData]

List all indexes (FEMA + Healthy Places Index CA + all listed on policy innovation)

@kelsranj


Docs: What configuration to set in root monorepo?

Hi,
I'd like to request better docs on what configuration should be used in the root package of a monorepo to enable lint-staged to run successfully over all packages. I simply have done this:
...
"lint-staged": {
    "*": "true"
  }
...

but it would be nice if I didn't have to.
I get an error about no configuration if I don't have this.
Thank you!

Hey,
I consider this a bug and will try to get it fixed in https://github.com/okonet/lint-staged/pull/1106

Thank you, @iiroj!

Version 12.3.5 is now released and hopefully fixed this issue. Can you confirm?

Works now! Thank you :)

@iiroj, I thought this was fixed but now I'm working in another repo and I have to add "lint-staged": { "*": "true" } to get it to work. Only difference I can see is that in the repo it's not working I have ./ui/.lintstagedrc.js, ./common/.lintstagedrc.js, and ./infra/.lintstagedrc.js whereas in the original repo where the update fixed the issue I have ./packages/ui/.lintstagedrc.js, ./packages/common/.lintstagedrc.js, and ./packages/infra/.lintstagedrc.js.
I can try and pull together more details if you need. Cannot share the repo. I'm using PNPM workspaces if that helps.

@bestickley Can you please elaborate on your setup? I am also using pnpm workspaces and husky with lint-staged.
Like what's in the base package.json and what's in the .lintstagedrc.js and if I need to consider something else.
Thanks a lot! :)

@Manubi, definitely:
package.json:
...
"lint-staged": {
    "*": "true"
  }
...

{ui,common,infra}/.lintstagedrc.js:
import { removeIgnoredFiles } from "../utils/lintStaged.js"

export default {
  "*.ts?(x)": async (files) => {
    const filesToLint = await removeIgnoredFiles(files);
    return [`eslint --fix --max-warnings=0 ${filesToLint}`, `jest --watchAll=false --findRelatedTests ${files}`, "tsc --noEmit"];
  }
}

I am experiencing the same issue and need to apply the same fix by creating a .lintstagedrc in the root with
{
    "*": "true"
  }

My version is:
    "lint-staged": "^13.2.2",


Make multiple_interfaces works on branch add-rrbot-corrections.

Correct URDF macro
Add multiple interface commands in the README.md
Fix controller name for forward_command_controller_position

Codecov Report

:exclamation: No coverage uploaded for pull request base (add-rrbot-corrections@a550d90). Click here to learn what that means.
The diff coverage is n/a.

@@                   Coverage Diff                   @@
##             add-rrbot-corrections    #101   +/-   ##
=======================================================
  Coverage                         ?   0.00%
=======================================================
  Files                            ?       3
  Lines                            ?     283
  Branches                         ?       0
=======================================================
  Hits                             ?       0
  Misses                           ?     283
  Partials                         ?       0

Flag
Coverage Δ

unittests
0.00% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out 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 a550d90...816a3d6. Read the comment docs.

@destogl @bmagyar IMHO this branch should be merged as soon as possible.
They are plenty of small corrections that people are proposing that are not useful because of the needed new organization proposed by this branch.
I fixed the CI, the format, tested the multiple interface, fixed some of the README.md
I did NOT try the diff example and they are probably a lot of small typos that I missed. But it looks like that the community is willing to help. So if you agree with that I would propose to merge as soon as it is possible this the branch to master and then fix the remaining problem in master. Or make it clear to contributors to work on top of this branch. Any feedback welcome.

Dear @destogl thanks a lot. You are probably super busy. But the next phase is to merge your PR #76. It cannot be done without the review of somebody else than you with writing permissions on the repository.
It would be nice if @Karsten1987 could have the writing rights on this repository. If this is fine with you, if you let me have the writing right I also can help on the ros2_control_demos repo for the merging and the review. This would ease the workload on you and @bmagyar.

FYI @Karsten1987 had merge right here for a week ore more now

@bmagyar Thanks for letting me know.


Remove file management and provide some sort of third-party application integration method

There is just to many ideas and needs about file uploads (file type, prefixing) and file upload widgets.
My current idea is to remove any file management features - just provide what the package name says - a CKEditor form field. As people want file management done as they need it there should be some way to integrate with the editor - I'll have to look what ckeditor has to say about this. Would be good to have this covered in the docs and to have at least one example for popular Django upload application/widget.
Suggestions, comments?

Looking at ckeditor 4.5.0 they added built in support for uploading drag and dropped images and media. Will have to check it, as it would solve problems with old gallerific widget being dead - and see if it must/should be supported by the package or exposed/binded to django file management app/apps.

Being able to uploading drag and dropp media sounds nice. But what about images or files already uploaded via other file managers or django image/file fields. I usually find myself in one of the two situations while using ckeditor editor in django:

I upload already uploaded images via ckeditor gallerific widget or
Inputing urls into ckeditor' image widget for image files which were already uploaded via other file manager.

I've used filebrowser and it works perfectly.
Had to load this JS:
'''
class Media:
js = ('filebrowser/js/FB_CKEditor.js',)
'''
and just added to the CKEDITOR_CONFIGS
'''
'filebrowserBrowseUrl': '/admin/filebrowser/browse/?pop=3',
'''

@ikresoft had django-ckeditor-filer app which provided integration between filer and ckeditor.As far as I know DjangoCMS also uses django-filer for file management and integration between ckeditor and django-filer is provided by django-ckeditor-filebrowser-filer app.

django-wysiwyg-redactor provides a simple way to customize the upload
https://github.com/douglasmiranda/django-wysiwyg-redactor#upload-handlers
https://github.com/douglasmiranda/django-wysiwyg-redactor/blob/master/redactor/handlers.py
https://github.com/douglasmiranda/django-wysiwyg-redactor/blob/master/redactor/views.py#L23-L73
it could at least serve as inspiration to create our own solution

:+1:

I am not using anymore this django-ckeditor-filebrowser-filer, so I figure out that is better to use some sort browser for all for example pictures, files, links for posts etc. This means that you can use browser for any kind of content just make plugin and register it. Unfortunately I dont have a time to finalize it its working for now but there some issue with design. So my point is use one browser for everything.
https://github.com/ikresoft/django-url-browser, old project is (django-nav-tree not using)

In current master I'm moving all file upload code to separate django application. This will allow using clean CKEdifor form field with other widgets and backends for upload.

This has happened a long time ago. ckeditor and ckeditor_uploader are two different apps, both in django-ckeditor.


Can the master node be launched as spot instance?

Currently the default behavior is to launch the master node as on-demand. This is the case even if I set "cluster_type=spot". Is there a way to launch the master node as spot instance instead?

You should be able to change the CloudFormation template to get what you want (MasterInstance is spot), but I suspect any spot revocation of the master node will result in the whole cluster failing with no recovery.

Can you point to where in the CloudFormation template I should make the change?

MasterServer would have to become a spot fleet. Then you would have to have to make sure the initialization procedure works. It's a bit more complicated than I thought, since using a spot fleet means you have to adjust the parameters in the MasterServer. Is there a particular reason you want to use spot for the master server? Using a small on-demand instance would be pretty cost effective. If you're worried about having a weak master node, you can find a way to SSH directly into the compute nodes.
Alternatively, you could default to launching instances manually via tools like Boto and then find a way to coordinate that with the rest of the system.

@mistycheney, the short answer is that there isn't a good way to use Spot for the master.    And then once it's working, you're not going to end up where you want, because the master failing is essentially suicide to the cluster; there's no recovery if the master node is lost.
If you're worried about price, a t2.small can act as a master node without an problem (and in many cases, a t2.micro is sufficient).  You might run into trouble building large software packages on the master node, but using compute nodes to build should work around that problem.


Improvements to per-port static/dynamic customization process?

I'm happy to work to implement improvements here, but first wanted to discuss and see what others thought.
I've got a solution using vcpkg linking to everything static with dynamic runtime (community triplets x64-windows-static-md and x86-windows-static-md). I had been pulling in ffmpeg manually (not with vcpkg) and dynamically (for licensing and size reasons), and wanted to move to pulling in using vcpkg.
I can totally do this with a custom triplet as shown in the triplets.md readme. But there are some pain points.

I have to add the custom triplet to vcpkg/triplets in my fork of vcpkg. (I pull in this fork as a subrepo in my repo to keep things stable, and pull the latest from microsoft/vcpkg as needed.) Now I've got a commit in my fork that will never be in microsoft/vcpkg, and feels like noise in my fork - it's definitely a triplet that would never be merged back into microsoft/vcpkg.
A way to add a triplet without having to pollute my fork would be nice. Maybe there already is and I just missed it?
Switching to a new triplet is a little painful. I have to update all of my projects (not too bad) and all of my dependencies will be rebuilt (a couple hours). Not the end of the world, but I'd rather not need to do that. ;-)

Port files are allowed to ignore the library linkage preference, as some projects only build static or dynamic, not both. What I really want to install is ffmpeg~dynamic - that is, an ffmpeg that builds dynamic regardless of the library linkage preference. What if vcpkg allowed specifying this via something like:
vcpkg install ffmpeg~dynamic?
For a given port, you would only be allowed to install one of the regular flavor (no special suffix), ~dynamic, or ~static.
FFmpeg is not the only library that can be built dynamic or static but where clients often have a preference for one option regardless of how their other dependencies are being linked. An example going the other way is protobuf, which builds both static and dynamic, but which recommends on its github page that only static linkage be used.

Attempting this via a custom triplet actually revealed that path isn't great for another reason. ffmpeg has many dependencies, and they need to be built dynamically too. Which means my triplet file had this added:
    set(VCPKG_LIBRARY_LINKAGE dynamic)
endif()```
actually needs that added for each of its 25 dependencies too.

actually needs that added for each of its 25 dependencies too.

not so difficult use.
if(PORT MATCHES"(ffmpeg|otherdep1|otherdep2)")
    set(VCPKG_LIBRARY_LINKAGE dynamic)
endif()

I have to add the custom triplet to vcpkg/triplets in my fork of vcpkg.

ever heard of overlay triplets?

Thanks for the help @Neumann-A . When the installation plan is made for ffmpeg and it lists the dependencies, is that list complete recursively, if that makes sense? Or might one of those dependencies itself have a dependency that I also need to add to the list.
I had not seen overlay triplets. That's perfect! :-)

use vcpkg depend-info <port> to get the complete list of dependencies of a port. The installation plan is only complete if you start with zero deps being installed.

Awesome, thank you! Quite excited to have ffmpeg building in my project via vcpkg. Tried it on a dummy project and it was great having source and debugging automatically.

I'm successfully building exactly what I want now with overlay-triplets. vckpg depend-info is also great to know - in this case I ended up only building ffmpeg with dynamic linking and not it's dependencies, as that mirrors how I used to be consuming it. Thank you again.


Add API doc links to array/bag/dataframe sections

Fixes #9629

I don't know of an easy way.  I probably won't spend time looking.  I also think that, while this is weird, it's not necessarily bad.  If I'm looking at API docs it might not be bad to show me "here are where the other API docs are in case you're interested".


Add better error checking to FixedPoint from_string conversion

As of now we can pass some odd strings that contain . and get difficult-to-interpret errors because we are presently just looking for .. We could use a regex that verifies the number format (+/-) (digits w/ possible _). We should also wrap the int cast in a try/except that throws a more informative error.

Resolved by #455


Revert "Added documentation about escaping values for $GITHUB_ENV file (#21582)"

Reverts nrz77/docs#1

c


Tags as field not facet?

Hi,
I have too many tags in my library so these don't work as facets.
Thats OK as I can exclude with KERKOAPP_EXCLUDE_DEFAULT_FACETS and create new, simpler, facets based off sub-collections & configure with KERKOAPP_COLLECTION_FACETS.
Great!
But now my tags don't show up on the item page.
Is there any way round this?
Could I tweak the code to create a new field, not facet, in composer.py?
And if so how do I tweak the template to display this (ideally as simple text field eg "Keywords: aaa, bbb, ccc") ?
eg I think this works as I can now search on my tags
if 'z_tags' not in exclude:
self.add_field(
FieldSpec(
key='z_tags',
field_type=self.secondary_id_field_type,
scopes=['all'],
extractor=extractors.TagsFacetExtractor(
include_re=self.default_tag_include_re,
exclude_re=self.default_tag_exclude_re
)
)
)

Hi, it looks like you're very close to have it done!
For the field_type parameter, I suggest that you use field_type=STORED instead of self.secondary_id_field_type. Tags are already searchable by default (through the text_tags field), so all you need is a stored field that you will be able to retrieve for display.
Once that is done, I think you'll be able to display your new field by adding something like the following in the item.html.jinja2 template, perhaps right above the field(_("Notes") line:
    {{- field(_("Keywords"), item.z_tags|join(', ')|escape) if item.z_tags }}

More tweaking might be needed to sort the tags or to improve the formatting. And I might be forgetting stuff since the above is untested code 😉, but I hope it helps.

Also, remove the scopes parameter. It's unneeded because the text_tags field will be used for search, not your new field.

That works perfectly.
Thanks a lot!

One last suggestion: add your new field in KerkoApp's config.py instead of altering Kerko's composer.py. It will make things easier should you want to upgrade Kerko in the future.


Added opensearch output plugin to converter

Signed-off-by: Asif Sohail Mohammed<EMAIL_ADDRESS>Description

Added OpenSearch output plugin mapping file
Added an expression in mapping file to support negation of boolean Logstash attribute values

Issues Resolved
Resolves #746
Resolves #710
Check List

[x] New functionality includes testing.
[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed with a real name per the DCO

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 #754 (7b0af33) into main (cb00d69) will increase coverage by 0.16%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #754      +/-   ##
============================================
+ Coverage     91.54%   91.70%   +0.16%
  Complexity      627      627
============================================
  Files            75       75
  Lines          1844     1844
  Branches        158      158
============================================
+ Hits           1688     1691       +3
+ Misses          120      117       -3
  Partials         36       36

Impacted Files
Coverage Δ

...java/com/amazon/dataprepper/pipeline/Pipeline.java
92.85% <0.00%> (+4.28%)
:arrow_up:

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 cb00d69...7b0af33. Read the comment docs.

I'll create separate PR's for both the issues addressed in this PR.


Make it clear that CLI uses osm-system as default namespace for various commands

Please describe the Improvement and/or Feature Request

The CLI should clearly indicate to users when it is using osm-system as the default namespace
Scope (please mark with X where applicable)

New Functionality          [ ]
Install                    [ ]
SMI Traffic Access Policy  [ ]
SMI Traffic Specs Policy   [ ]
SMI Traffic Split Policy   [ ]
Permissive Traffic Policy  [ ]
Ingress                    [ ]
Egress                     [ ]
Envoy Control Plane        [ ]
CLI Tool                   [X]
Metrics                    [ ]
Certificate Management     [ ]
Sidecar Injection          [ ]
Logging                    [ ]
Debugging                  [ ]
Tests                      [ ]
CI System                  [ ]
Project Release            [ ]

Possible use cases

Users may not be aware that commands such as uninstall and policy use osm-system as the default namespace.

Adding an example:
osm policy check-pods bookbuyer/bookbuyer-8465f7dfc-9nctv bookstore/bookstore-v1-6cfc68bdd4-jxms8

Error: Error checking if permissive mode is enabled

The returned error is misleading.
For anyone who might hit this issue.
The fix was to provide the namespace the mesh is installed in via --osm-namespace

Reference PR: https://github.com/openservicemesh/osm/pull/2838


Adding ENEM Challenge Scenario & Maritaca AI model (Sabiá 7B)

Description
This pull request is intended to:

Add support for a PT-BR model developed by Maritaca AI called Sabiá.
Add a new PT-BR scenario called ENEM Challenge. This is a multiple choice question answering scenario that has all questions from the Exame Nacional do Ensino Médio (ENEM), which is an advanced High-School level exam widely applied every year by the Brazilian government to students that wish to undertake a University degree.

Thanks again! Note that I pushed a commit to your branch in order to fix the lint errors.


Move to new player api

Don't merge this one until player PR (Playapiupdates #62) to update the api has been delivered and etcd key /player/url has been updated to have value https://game-on.org/players/v1/accounts

This change is

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


"Smalltalk" with players

greet players that get in a certain range and have not been greeted for a time(?)
accuse players of hurting them if that is the case

This looks promising https://en.wikipedia.org/wiki/SHRDLU


--show-url flag seems to have no effect in dvc get

I executed dvc get with --show-url flag, but it still tries to download the file.  Also, the flag is shown in the description of the dvc get page, but not in the syntax at the top.

MIgrating this to the DVC core repo to investigate if needed.


Add task toolbox to DruidInputSource

Add task toolbox to DruidInputSource

The TaskToolbox can be useful in the InputSource for various reasons.
This PR introduces a new interface TaskInputSource in the indexing-service module to allow the addition of a provided toolbox to the input source. This is not possible within the InputSource interface as the TaskToolbox class isn't accessible in its module.
It also alters the DruidInputSource to fetch used segments using the RetrieveUsedSegments actions via the overlord when a toolbox is available. (It previously relied on the Coordinator's snapshot which could be stale at times)

This PR has:

[x] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] a release note entry in the PR description.
[x] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

Thanks for the changes, @AmatyaAvadhanula !

Thanks for the reviews @imply-cheddar, @kfaraz


Can not get tern to install globally on Windows 10

Hi,
I am running Windows 10 and emacs 24.5.1 (through Cygwin).
I have read all of #256 and I have what I think is a (much) smaller issue.  I seems like I just can't get tern installed globally correctly.
When I check C:\Users\myusername\AppData\Romainig\npm I see all of the other globally installed npm packages but nothing related to tern.
I do see tern inside the C:\Users\myusername\AppData\Romainig\npm\node_modules directory, but when I run tern from the command line I get:
'tern' is not recognized as an internal or external command.  (even if I am in that folder).
If, through Cygwin, I cd into that directory, I can run tern  (it just says listening on port...).  However, when I try to run a .js file, in emacs I get:
File mode specification error: (file-error "Searching for program" "no such file or directory" "tern")
I even went so far as to add  C:\Users\myusername\AppData\Romainig\npm\node_modules\tern to my path but this made no difference.  I would really appreciate any help.
Thanks

You could set the tern-command variable in Emacs to "C:\Users\myusername\AppData\Romainig\npm\node_modules\tern\bin\tern".

When I run that I now get:
Error running timer tern-reparse-on-idle': (wrong-type-argument stringp 99)`

Sorry, yes, it has to be a list, so '("C:\Users\myusername\AppData\Romainig\npm\node_modules\tern\bin\tern")

Sorry to be such a bother. But that gave me this error:
Request failed: Could not start Tern server
module.js:339^M
    throw err;^M
    ^^M
^M
Error: Cannot find module 'C:\cygdrive\c\Users\willy\AppData\Romaning\npm\node_modules\tern\bin\tern'^M
    at Function.Module._resolveFilename (module.js:337:15)^M
    at Function.Module._load (module.js:287:25)^M
    at Function.Module.runMain (module.js:467:10)^M
    at startup (node.js:134:18)^M
    at node.js:961:3^M

My sincerest apologies.  This was due to an error in my PATH on Windows.  Correcting that and reinstalling tern makes it work.

@computersarecool I seem to be facing the exact same issue. What kind of error in yout PATH were you facing?


Enhance TTS Model with Advanced Speech Parameters and Improved Voice Cloning

This Pull Request introduces several key enhancements to the MetaVoice-1B text-to-speech (TTS) model, focusing on improving AI capabilities and user interaction:
Advanced Speech Parameters:
Added functionality for dynamic adjustment of speech stability and speaker similarity. Users can now fine-tune the top_p (speech stability) and guidance (speaker similarity) parameters through sliders, allowing for more personalized and controlled speech output.
Enhanced Voice Cloning:
Improved handling of uploaded voice samples for cloning. The script now includes validation for file size and duration, ensuring that uploaded samples are suitable for high-quality voice synthesis. Samples must be between 30-90 seconds and less than 50MB to ensure optimal performance.
User Interface Improvements:
Updated the user interface to provide a more intuitive experience. Users can choose between preset voices and uploaded target voices, with automatic layout adjustments based on the selected option. The interface now features clear labels and better organization for ease of use.
Robust Error Handling:
Enhanced error handling to manage edge cases and provide informative feedback. The script includes comprehensive checks and error messages for input validation, such as handling text length limits and ensuring uploaded files meet the required criteria.
These updates aim to enhance the functionality, usability, and robustness of the MetaVoice-1B TTS model, delivering a more versatile and user-friendly text-to-speech solution.

This pull request introduces several significant enhancements to the MetaVoice-1B text-to-speech (TTS) model. The updates focus on improving the model's AI capabilities and user interface, ensuring a more robust and user-friendly experience.
Key Enhancements
Dynamic Speech Parameters:
Added sliders for top_p (speech stability) and guidance (speaker similarity) to allow users to fine-tune speech output. These adjustments provide greater control over the generated speech, enabling more personalized results.
Enhanced Voice Cloning:
Improved handling of uploaded voice samples, including validation for file size and duration. Samples must now be between 30-90 seconds and less than 50MB, ensuring high-quality and consistent voice synthesis.
User Interface Improvements:
Updated the interface to offer a more intuitive experience. Users can easily switch between preset voices and uploaded samples, with automatic layout adjustments based on their selection. The UI enhancements aim to streamline user interaction and improve overall usability.
Robust Error Handling:
Enhanced error handling to manage edge cases and provide clear feedback. The script now includes comprehensive checks for text length limits and file validity, with informative error messages to guide users.
Benefits
Enhanced Flexibility: Users can now customize speech parameters to achieve the desired voice characteristics.
Improved Quality: Voice cloning is more accurate and reliable with the added validation checks.
Better User Experience: The updated interface and error handling make the tool more intuitive and user-friendly.


Patrol ban CSGO

I have got patrol ban in CSGO.
I have some questions about it. Could I have info about some topics:

demo of the match, for which I have been banned
the results of patrol decisions and what ranks/stats/hours have those patrol players.
In the profile I have been adding Steam profile links of cheaters I played with, could it be a trigger for ban?
During last games I always played with cheaters, but earlier it was not often, could it be a trigger for ban?

I ask these, because I am sure I am banned for a mistake.
My Steam profile: https://steamcommunity.com/id/iamnotdotaplayeranymorerofl/
My inventory contains expensive skins. More than 700$ charged on account.
Faceit account was also active, as I were playing not only matchmaking, but Faceit also.
Yes, I were often toxic, because of cheaters.  I were ruining game when I play with cheaters in team, but do I deserve ban for that? Yes, I would agree for: ban chat/temporal ban/etc, but not ban forever...
Steam Supports ignore all of my questions, they only answer with useless template messages.
All I've used during CSGO runnig - Faceit Anticheat, Razer Central, HyperX NGENUITY, Lightshot app.
I really hope to get clarfication about this ban.
Thank you in advance.

Hello @a-korzhov, sorry, but account issues including bans are not handled on Github.


What's the "native support" for Themed Title Bar or Frameless Window on MacOS?

Hey!
First of all, I really appreciate you maintaining this plugin.
I noticed a changelog entry under 1.6.0 release and I was wondering if you could provide guidance for it.

Migrating MacOS users on 2022.2 builds to not use the provided themed title bar, as it has native support now.

How can I get a frameless titlebar and/or themes titlebar in Rider from 2022.2+?
The primary reason I'm asking this is to eliminate OS titlebar entirely and leave the whole space for tabs only.
Thank you!

How can I get a frameless titlebar and/or themes titlebar in Rider from 2022.2+?

Excellent question. If you are wanting to completely remove the titlebar, so that the tabs just show up at the top (I'm assuming the toolbar is off), I don't think there is any thing in the JetBrains platform that supports that feature.
The space up at the top can be removed, however I don't think the open/minimize/close controls can be removed. Which would probably be annoying because they would overlay on the left most tab.
I'll take a look when I get some free time in the future, thanks!


udp_msgs: 0.0.5-1 in 'jazzy/distribution.yaml' [bloom]

Increasing version of package(s) in repository udp_msgs to 0.0.5-1:

upstream repository: https://github.com/flynneva/udp_msgs.git
release repository: git@github.com:ros2-gbp/udp_msgs-release.git
distro file: jazzy/distribution.yaml
bloom version: 0.12.0
previous version for package: 0.0.3-7

udp_msgs
* Clean up CI, remove release helpers
* Contributors: Evan Flynn

Holding for Jazzy sync


Resolve undefined errors in oci.py

Resolves some undefined errors I found while exploring the codebase.

@swarajpande5 writing a test for pulling and oci image model would be useful to make sure the oci image path doesn't break long-term


Add contribution guide doc

Describe the solution you'd like
Add a contribution guide doc, describes how to add a new protocol, and kyanos internal implementation.

I think the error message should be more general.
it just needs to add in the FAQ that only local containers are supported.


Questions about HOW they will do this

Someone needs to explain whether or not there will still be the same per-user profile install or if it will behave like all the other office apps. Ever twitter thread I've seen that points to the roadmap item or this documentation has been asking this question.

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

ID: 7936ca5e-c658-4956-b174-99eaf1eb67d4
Version Independent ID: 9ecc0b61-a941-f5e1-96c3-d32fae59eacd
Content: Deploy Microsoft Teams with Office 365 ProPlus
Content Source: DeployOffice/teams-install.md
Service: o365-proplus-itpro
GitHub Login: @DHB-MSFT
Microsoft Alias: danbrown

@npherson -- thanks for the question. Will reply back when we have an answer and will update the article as needed.

@npherson  -- Teams will be installed with Office 365 ProPlus in the same manner Teams is currently installed if you had used the existing Teams MSI installer. For each new user that signs into the device, the Teams installer will launch and the Teams application will be installed in the user's appdata folder.
Hope that answers your question. I'll make an update to the article to include that info.  Thanks again for your question, which helps make the documentation better.

Thank you for the answer! While it is incredibly disappointing that this is how it will work, at least enterprises know and can start planning how to mitigate the pain. Can we update the document to make it clear that the existing Teams installer behavior won’t change with this like everyone is hoping?
From: Daniel H. Brown<EMAIL_ADDRESS>Sent: Thursday, February 14, 2019 10:53 AM
To: MicrosoftDocs/OfficeDocs-DeployOffice<EMAIL_ADDRESS>Cc: Nash Pherson<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: EX: Re: [MicrosoftDocs/OfficeDocs-DeployOffice] Questions about HOW they will do this (#291)
@nphersonhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnpherson&data=02|01|Nash.Pherson%40microsoft.com|3be23ea027994be3205108d6929ce0f5|72f988bf86f141af91ab2d7cd011db47|1|0|636857599825176589&sdata=ByIqKVXjliWX864hFw%2FcLlZFKpTJpl4kc82BoHqD%2FE0%3D&reserved=0 -- Teams will be installed with Office 365 ProPlus in the same manner Teams is currently installed if you had used the existing Teams MSI installer. For each new user that signs into the device, the Teams installer will launch and the Teams application will be installed in the user's appdata folder.
Hope that answers your question. I'll make an update to the article to include that info. Thanks again for your question, which helps make the documentation better.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-DeployOffice%2Fissues%2F291%23issuecomment-463703000&data=02|01|Nash.Pherson%40microsoft.com|3be23ea027994be3205108d6929ce0f5|72f988bf86f141af91ab2d7cd011db47|1|0|636857599825176589&sdata=QMgoaKoQtItoFPb5IjkIkOXqrSXvY%2Byg1jiemuL6i0E%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAVvb9Yu7qcjDqKjwFmAexhJC7ngBX05Eks5vNZRsgaJpZM4a6G9z&data=02|01|Nash.Pherson%40microsoft.com|3be23ea027994be3205108d6929ce0f5|72f988bf86f141af91ab2d7cd011db47|1|0|636857599825176589&sdata=p2iskB8v%2Fi5X0oqxSW984ZqY7U8uh3WlgjDFGfDq7w4%3D&reserved=0.

@npherson  -- yes, I plan to update the article today/tomorrow to make it clear that the Teams install behavior isn't changing.

FYI, article updated. See 2nd bullet point in the "Additional info" section at the end of the article. Thanks again for your question/feedback. It helps make the documentation better.


Custom domains settings drop after each deployment

This issue is a: (mark with an x)

[x ] bug report -> please search issues before submitting
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
We're trying to work with .NET Aspire. We built a system on it and now try to deploy it from github via GitHub Actions to Azure. We did everything as in the instructions and it worked. But One thing started to annoy critically. Every time we deploy the new version - custom domain settings are just gone. The same is about Ingress settings. My changes drop after each release.
Steps to reproduce

Deploy .NET Aspire app via GitHub Actions
Set up custom domains
Deploy again

Expected behavior
New version is running but previous network and domains settings are the same
Actual behavior
Ingress settings of the app and custom domain settings are dropped.

@relounge thank you for raising this with us. Could you share the repo or Bicep you're using. The  issue may need updating of the boilerplate Bicep to include the custom domain.

Please see here for guidance on how you can add the custom domain to your Bicep file (found in infra/resources). https://learn.microsoft.com/en-us/azure/templates/microsoft.app/containerapps?pivots=deployment-language-bicep#customdomain

@simonjj sorry, I cannot share the repo as it's our production repo. As for the Bicep file - I did not have it until you mentioned it. I deployed our aspire app using the instructions I found in the MS documentation and used this file as a workflow file for Github actions.
Then I started digging and found "next-steps.md" file. As I understand I need to run "azd infra synth" to generate 3 infra files including the one you mentioned and set it up, right? I must confess I never worked with Bicep and when I looked into resources.bicep file I didn't understand how to set up custom domains. I also noticed other created files after "synth" command in the AppHost project inside "infra". All of the files are yaml. Still not sure how it works, but we'll try to set it up, deploy and see what happens next

I don't get it. All of the examples here are in Bicep/JSON or Terraform syntax, but the files generated for containerized services by "synth" command were yaml files. This adds a bit of complexity. Will try to guess how it works

So I:

ran "azd infra synth" command and got bicep and yaml files in return
modified the services with domian bindings I needed (I hope it was correct)
added those new files to the repo and ran the deployement again
...and nothing has changed.

Here is how I changed the service yaml file:
ingress:
external: true
targetPort: {{ targetPortOrDefault 8080 }}
transport: http
allowInsecure: true
customDomains:
- bindingType: SniEnabled
certificateId: [ here I added the certificate name (not id) I found in azure]
name: [ here I added my domain]
I think I need some help. We choose Aspire to get rid of docker files hell. Now I see we got bicep and yaml hell instead.

Found a solution for custom domains from here https://github.com/Azure/azure-dev/issues/1765
This command helped: azd config set alpha.aca.persistDomains on
Still working on ingress settings. It hides the endpoint after each deployment.

thanks for updating us @relounge. i will share your feedback with the appropriate teams.


Why is "children" a required property of ContextMenuTrigger?

https://github.com/vkbansal/react-contextmenu/blob/master/src/ContextMenuTrigger.js#L12
It doesn't seem like it should be.  I want to wrap a childless React component in the ContextMenuTrigger.

I guess that makes sense. Will you be willing to open a PR with the change?

@mgummelt : is your custom component accept PropTypes?


procodile not found on server despite being installed

I have installed procodile on my server (gem install procodile) and can verify it's installation by running procodile on the command line.
However, when I deploy via Capistrano 3 (using the most recent version of procodile-capistrano, 1.0.2), I get the following error:
cap aborted!
SSHKit::Runner::ExecuteError: Exception while executing on host MYIP: Exception while executing on host MYIP: procodile restart -r /home/deployer/apps/myapp/current exit status: 127
procodile restart -r /home/deployer/apps/myapp/current stdout: bash: procodile: command not found
procodile restart -r /home/deployer/apps/myapp/current stderr: Nothing written

I get this error from a basic deploy (bundle exec cap production deploy) when I invoke|execute Procodile from an after hook (after :publishing, 'procodile:restart').
Since submitting the ticket, I have managed to get Procodile to fire up the correct processes with a bundle exec cap production procofile:start (or stop or restart).
Long and short, trying to understand why I cannot successfully invoke the 'procodile:restart' from a Capistrano deploy hook.

Not sure if this will help or not but I was able to get this rolling with the Cap tasks copied from procodile-capistrano made with a small callback change. Rather than after 'deploy:finished', "procodile:restart", the following works:
before 'deploy:starting', 'procodile:stop'
after 'deploy:finished', 'procodile:start'

... if you think it's a worthwhile change, I can fork that repo and submit a PR.

Sorry for the delay replying to this. I don't think stopping & starting is the solution as it would negate any zero-downtime deployment strategies one might have within processes. I'm not really sure why your application can't restart though.
The most likely cause is some sort of PATH issue on your server but that doesn't really explain why start/stop would work.

No worries, Adam! We've got it working as we need it so going to close this ticket and thanks for your work on Procodile!

Good good 👍


Fix Memviewslice docstrings

Fixes #2634

I think a test would help to understand a) what this does and b) if it does the right thing. It would also help in keeping this working in the future. See the embedsignatures.pyx test.


Azure queue: add missing param decodeBase64

What content needs to be created or modified?

Documentation doesn't show param decodeBase64 for the binding azure queue
Describe the solution you'd like

Where should the new material be placed?

The associated pull request from dapr/dapr, dapr/components-contrib, or other Dapr code repos

Additional context

/assign

FIXED


Report multiple missing unfoldings at the same time

Describe the feature you'd like
Currently Plutus plugin reports only a single missing unfolding.
Big contract will have multiple, so it would be nice to report them as a list instead.
Describe alternatives you've considered
Alternative is to recompile entire code each time unfolding is captured, which is much longer process and error prone, since Plutus plugin gives no indication of progress, nor location: no line number, nothing.

nor location: no line number, nothing

That seems wrong. The plugin should give you errors with locations.
I never figured out a way to make the plugin give errors in the style that the compiler does, but maybe I should give it another try.

Big contract will have multiple, so it would be nice to report them as a list instead.

Accumulating errors would be nice, but we'd have to rewrite it to be tolerant of failure instead of just bailing out as soon as it sees a problem.

@mgajda if you're experiencing this problem still, please specify how to reproduce it (I know it's been a long time). It seems this shouldn't be happening, given Michael's response:

That seems wrong. The plugin should give you errors with locations.

@michaelpj

Accumulating errors would be nice, but we'd have to rewrite it to be tolerant of failure instead of just bailing out as soon as it sees a problem.

Sounds like a very useful feature to have, but seems to be considered low priority right now, so I'm adding the label.


JWT plugin does not create credentials for consumer identified with UUID

Summary
SUMMARY_GOES_HERE
JWT plugin does not create credentials when passed a uuid that identifies a user (either username or custom_id).
Steps To Reproduce

Register a new user:
curl -X POST kong:8001/consumers --data "username=ad005c48-508a-41d0-b303-8921e97609c3"
Create credentials
curl -X POST kong:8001/consumers/ad005c48-508a-41d0-b303-8921e97609c3/jwt
Expected Result: 200, Actual Result: 404

Additional Details & Logs

Kong version: 0.9.6

Kong debug-level startup logs ($ kong start --vv)
2016/12/09 14:08:37 [verbose] Kong: 0.9.6
2016/12/09 14:08:37 [debug] ngx_lua: 10006
2016/12/09 14:08:37 [debug] nginx: 1011002
2016/12/09 14:08:37 [debug] Lua: LuaJIT 2.1.0-beta2
2016/12/09 14:08:37 [debug] PRNG seed:<PHONE_NUMBER>29
2016/12/09 14:08:37 [verbose] no config file found at /etc/kong.conf
2016/12/09 14:08:37 [verbose] no config file found at /etc/kong/kong.conf
2016/12/09 14:08:37 [verbose] no config file, skipping loading
2016/12/09 14:08:37 [debug] admin_listen = "<IP_ADDRESS>:8001"
2016/12/09 14:08:37 [debug] anonymous_reports = true
2016/12/09 14:08:37 [debug] cassandra_consistency = "ONE"
2016/12/09 14:08:37 [debug] cassandra_contact_points = {"<IP_ADDRESS>"}
2016/12/09 14:08:37 [debug] cassandra_data_centers = {"dc1:2","dc2:3"}
2016/12/09 14:08:37 [debug] cassandra_keyspace = "kong"
2016/12/09 14:08:37 [debug] cassandra_port = 9042
2016/12/09 14:08:37 [debug] cassandra_repl_factor = 1
2016/12/09 14:08:37 [debug] cassandra_repl_strategy = "SimpleStrategy"
2016/12/09 14:08:37 [debug] cassandra_ssl = false
2016/12/09 14:08:37 [debug] cassandra_ssl_verify = false
2016/12/09 14:08:37 [debug] cassandra_timeout = 5000
2016/12/09 14:08:37 [debug] cassandra_username = "kong"
2016/12/09 14:08:37 [debug] cluster_listen = "<IP_ADDRESS>:7946"
2016/12/09 14:08:37 [debug] cluster_listen_rpc = "<IP_ADDRESS>:7373"
2016/12/09 14:08:37 [debug] cluster_profile = "wan"
2016/12/09 14:08:37 [debug] cluster_ttl_on_failure = 3600
2016/12/09 14:08:37 [debug] custom_plugins = {}
2016/12/09 14:08:37 [debug] database = "postgres"
2016/12/09 14:08:37 [debug] dnsmasq = true
2016/12/09 14:08:37 [debug] dnsmasq_port = 8053
2016/12/09 14:08:37 [debug] log_level = "notice"
2016/12/09 14:08:37 [debug] lua_code_cache = "on"
2016/12/09 14:08:37 [debug] lua_package_cpath = ""
2016/12/09 14:08:37 [debug] lua_package_path = "?/init.lua;./kong/?.lua"
2016/12/09 14:08:37 [debug] lua_ssl_verify_depth = 1
2016/12/09 14:08:37 [debug] mem_cache_size = "128m"
2016/12/09 14:08:37 [debug] nginx_daemon = "on"
2016/12/09 14:08:37 [debug] nginx_optimizations = true
2016/12/09 14:08:37 [debug] nginx_worker_processes = "auto"
2016/12/09 14:08:37 [debug] pg_database = "kong"
2016/12/09 14:08:37 [debug] pg_host = "<IP_ADDRESS>"
2016/12/09 14:08:37 [debug] pg_port = 5432
2016/12/09 14:08:37 [debug] pg_ssl = false
2016/12/09 14:08:37 [debug] pg_ssl_verify = false
2016/12/09 14:08:37 [debug] pg_user = "kong"
2016/12/09 14:08:37 [debug] prefix = "/usr/local/kong/"
2016/12/09 14:08:37 [debug] proxy_listen = "<IP_ADDRESS>:8000"
2016/12/09 14:08:37 [debug] proxy_listen_ssl = "<IP_ADDRESS>:8443"
2016/12/09 14:08:37 [debug] serf_path = "serf"
2016/12/09 14:08:37 [debug] ssl = true
2016/12/09 14:08:37 [verbose] prefix in use: /usr/local/kong
2016/12/09 14:08:37 [verbose] preparing nginx prefix directory at /usr/local/kong
2016/12/09 14:08:37 [verbose] saving serf identifier to /usr/local/kong/serf/serf.id
2016/12/09 14:08:37 [debug] searching for OpenResty 'resty' executable
2016/12/09 14:08:37 [debug] /usr/local/openresty/bin/resty -V: 'sh: /usr/local/openresty/bin/resty: No such file or directory'
2016/12/09 14:08:37 [debug] OpenResty 'resty' executable not found at /usr/local/openresty/bin/resty
2016/12/09 14:08:37 [debug] resty -V: 'nginx version: openresty/<IP_ADDRESS>'
2016/12/09 14:08:37 [debug] found OpenResty 'resty' executable at resty
2016/12/09 14:08:37 [verbose] saving serf shell script handler to /usr/local/kong/serf/serf_event.sh
2016/12/09 14:08:37 [verbose] SSL enabled, no custom certificate set: using default certificate
2016/12/09 14:08:37 [verbose] default SSL certificate found at /usr/local/kong/ssl/kong-default.crt
2016/12/09 14:08:37 [verbose] running datastore migrations
2016/12/09 14:08:37 [verbose] migrations up to date
2016/12/09 14:08:37 [verbose] dnsmasq not running, deleting /usr/local/kong/pids/dnsmasq.pid
2016/12/09 14:08:37 [debug] searching for 'dnsmasq' executable
2016/12/09 14:08:37 [debug] /usr/local/sbin/dnsmasq -v: 'Dnsmasq version 2.76  Copyright (c) 2000-2016 Simon Kelley'
2016/12/09 14:08:37 [debug] found 'dnsmasq' executable at /usr/local/sbin/dnsmasq
2016/12/09 14:08:37 [debug] starting dnsmasq: /usr/local/sbin/dnsmasq -p 8053 --pid-file=/usr/local/kong/pids/dnsmasq.pid -N -o --listen-address=<IP_ADDRESS>
2016/12/09 14:08:37 [verbose] serf agent not running, deleting /usr/local/kong/pids/serf.pid
2016/12/09 14:08:37 [debug] checking 'serf' executable from 'serf_path' config setting
2016/12/09 14:08:37 [debug] serf version: 'Serf v0.8.0'
2016/12/09 14:08:37 [debug] starting serf agent: nohup serf agent -profile 'wan' -rpc-addr '<IP_ADDRESS>:7373' -event-handler 'member-join,member-leave,member-failed,member-update,member-reap,user:kong=/usr/local/kong/serf/serf_event.sh' -bind '<IP_ADDRESS>:7946' -node 'Joeys-MacBook-Pro.local_<IP_ADDRESS>:7946_7899b7ed127b434cb9a95a32c6e08f37' -log-level 'err' > /usr/local/kong/logs/serf.log 2>&1 & echo $! > /usr/local/kong/pids/serf.pid
2016/12/09 14:08:37 [verbose] waiting for serf agent to be running
2016/12/09 14:08:38 [debug] sending signal to pid at: /usr/local/kong/pids/serf.pid
2016/12/09 14:08:38 [debug] kill -0 cat /usr/local/kong/pids/serf.pid >/dev/null 2>&1
2016/12/09 14:08:38 [verbose] serf agent started
2016/12/09 14:08:38 [verbose] auto-joining serf cluster
2016/12/09 14:08:38 [verbose] no other nodes found in the cluster
2016/12/09 14:08:38 [verbose] registering serf node in datastore
2016/12/09 14:08:38 [verbose] cluster joined and node registered in datastore
2016/12/09 14:08:38 [debug] searching for OpenResty 'nginx' executable
2016/12/09 14:08:38 [debug] /usr/local/openresty/nginx/sbin/nginx -v: 'sh: /usr/local/openresty/nginx/sbin/nginx: No such file or directory'
2016/12/09 14:08:38 [debug] OpenResty 'nginx' executable not found at /usr/local/openresty/nginx/sbin/nginx
2016/12/09 14:08:38 [debug] nginx -v: 'nginx version: openresty/<IP_ADDRESS>'
2016/12/09 14:08:38 [debug] found OpenResty 'nginx' executable at nginx
2016/12/09 14:08:38 [debug] starting nginx: nginx -p /usr/local/kong -c nginx.conf
2016/12/09 14:08:38 [debug] nginx started
2016/12/09 14:08:38 [info] Kong started

Kong error logs (<KONG_PREFIX>/logs/error.log)

Kong configuration (registered APIs/Plugins & configuration file)
Registered apis:  mockbin (see 5 min start guide).

Operating System
MacOS Sierra

Known issue. Kong checks the format of the :id parameter of the /consumers/:id end point, and if it is a GUID/UUID it assumes you refer to a Kong internal ID, which is not the case here. Hence the 404, as the UUID is not a Kong ID, but one you passed in from the outside.
Workaround is to either prepend your ID with some fixed prefix, like user-<uuid>, or to have an actual username in the username, and use the custom_id for your own UUID.

considering this answered. Thx @DonMartin76 :+1:


Update case-study-looking-through-holes-in-your-reality.md

cleanup markup

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/case-study-looking-through-holes-in-your-reality.md
:white_check_mark:Succeeded
View

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.


MeanField with TangentSpace

A proposal about mixing TS with MeanField, for MI application mainly.
@qbarthelemy @toncho11

Oki


License

Hi,
Great work on LWM! I noticed the weights are licensed under the Apache license but derived from Llama 2, do both the Llama 2 license and the Apache license apply to the weights?
Thanks!
(Related to #10)

Hi, I'm also curious about this. It seems the licenses are a bit conflicting, or rather this GH says Apache 2.0 but HF says LLaMA.

Hi, this codebase is licensed under Apache 2.0. The models, since are derivatives of Llama 2, are under the license of Llama 2.
Will make this more clear in documentation.


Double refresh of beans

Hello!
I have Spring Boot Application with spring cloud starter auto configuration for zookeeper.
I met some problems with refreshing context when I do some changes in zookeper nodes.
This is my pom.xml:
    <properties>
        <java.version>1.8</java.version>
        <spring-cloud.version>Greenwich.RELEASE</spring-cloud.version>
        <camel.version>2.23.1</camel.version>
        <spring-boot.version>2.1.2.RELEASE</spring-boot.version>
    </properties>

    <dependencies>
        <dependency>
            <groupId>org.springframework.boot</groupId>
            <artifactId>spring-boot-starter-web</artifactId>
        </dependency>
        <dependency>
            <groupId>org.springframework.cloud</groupId>
            <artifactId>spring-cloud-zookeeper-config</artifactId>
        </dependency>
        <dependency>
            <groupId>org.springframework.cloud</groupId>
            <artifactId>spring-cloud-starter-zookeeper-config</artifactId>
        </dependency>

And I have two questions:

When I configure bootstrap.yml, for example:

  cloud:
    zookeeper:
      connect-string: localhost:2181
      config:
        enable: true
        root: config
        defaultContext: apps
        profileSeparator: '::'
  application:
    name: my_application

then created two propertySourceContext: "config/apps" and "config/my_application" in ZookeeperPropertySourceLocator
It was not obvious to me, that application.nameis used to build path to zookeper node.
Why do i need defaultContext and application.name both to build paths? I really want understand that)
When i use  defaultContext and application.name with same name,
then it leads to creation of two two propertySourceContext and  all RefreshScope beans refreshing two times, is it normal?)

I use @EventListener(RefreshScopeRefreshedEvent.class)(for refreshing method) for catching refresh my ZooKeeper, but any change in my ZooKeeper call this method:(
Can i subscribe on some Node in zookeeper and refresh only certain beans?
I saw the method named public boolean refresh(String name)  in  RefreshScope.class,  how can I invoke it?

Thanks!
P.S. Sorry for my best english!

the defaultContext is a place to put properties that are common to apps. It shouldn't be the same as spring.application.name.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

I realized that i should not specify the same values.
What about the second question?
Can I subscribe to a specific node, or will any change in zookeeper always refresh context?

Thank you very much for your attention!
I figured this out a bit and just want to clarify. When changing the node in the zookeeper RefreshScopeRefreshedEvent event throws or is there a more specific event?

I think I found the answer to my question)
I can catch the refresh event and check on instance of TreeCacheEvent
then determine the updated node and do what you want)
Example
 @EventListener(RefreshEvent.class)
    public void onRefresh(RefreshEvent event)  {
...
}

If there is a better way please correct me!
Thank you very much for your attention!


documentaion and distribution todos

[x] add docs for AWS, GC and DO deployment
[x] remove mockjsons
[x] link to open-event FOSSASIA sample
[x] improve and correct text in Readme, add choice of technologies

thank you so much @mariobehling for doing points 1 and 4

@championswimmer I think this is a misunderstanding. Those points are not done entirely. I just added some documentation and empty mds to draw the path.

installation docs added


Clarify intent of --mapRoot

Add two sentences to make it clear that --mapRoot does not create the map files in the specified path.

Fixes #1207

This feels reasonable to me, somehow I can't get to the original issue  to verify but I'm open to this being the right behavior.


[Cleanup test<PHONE_NUMBER>56][test_000056_s.mp4] ctts box in stbl box  is not helpful.

currently, IAMF specification assume that PTS and DTS are same like other ordinary audio codec.
Instead of ctts which has the preskip value, according to https://aomediacodec.github.io/iamf/#isobmff-singletrack-basicencapsulationscheme, the preskip value is stored in elst.media_time.
Though ctts is useful for ordninary video codec which is PTS and DTS are generally different,
in IAMF bitstream, ctts box in stbl box  is not helpful.
I think that ctts box makes our IAMF decoder misunderstand,
it should be omitted.

There are same issues in test_000021_s, _000022_s, 000024_s, 000025_s, 000026_s, 000027_s,  000028_s
000032_s, 000033_s, 000034_s, 000035_s, 000037_s,
000045_s, 000046_s, 000048_s, 000049_s,
000050_s, 000051_s, 000052_s, 000053_s, 000054_s and 0000_55_s


How to fix the mistake?

hello everyone,i think i need some help.
KafkaJSNonRetriableError: Consumer group was not initialized, consumer#run must be called first
    at Object.commitOffsets (/opt/nodework/node_modules/kafkajs/src/consumer/index.js:362:13)
    at Runner.eachMessage (/opt/nodework/pincong-img-download-2.js:105:40)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at async Runner.processEachMessage (/opt/nodework/node_modules/kafkajs/src/consumer/runner.js:151:9)
    at async onBatch (/opt/nodework/node_modules/kafkajs/src/consumer/runner.js:287:9)
    at async /opt/nodework/node_modules/kafkajs/src/consumer/runner.js:339:15 {
  retriable: false,
  helpUrl: undefined
}

"kafkajs": "1.15.0"
“node”: "14.15.0"
“server”: "Linux version 4.15.0-124-generic (buildd@lgw01-amd64-029) (gcc version 7.5.0 (Ubuntu 7.5.0-3ubuntu1~18.04)) #127-Ubuntu SMP Fri Nov 6 10:54:43 UTC 2020"
async function main(){
  const groupId = "message";
  const topic = "data";
  let index = 0;
  const consumer = kafka.consumer({ groupId });
  await consumer.subscribe({ topic, fromBeginning: true });
  try{
    await consumer.run({
      eachMessage: async ({topic,partition,message}) =>{
        const data = JSON.parse(message.value.toString("utf-8"));
        try{
          await consumer.commitOffsets([{
            topic: topic,
            partition: partition,
            offset: Number(message.offset) + 1
          }])
          await insert(data);
        }catch(e){
          console.error(e);
        }
        return;
      }
    })
  }catch (e){
    console.log(e)
    return
  }
}

so,what should i do?

There's no need to call consumer.commitOffsets from within eachMessage. A soon as eachMessage resolves the offset will be committed automatically:
await consumer.run({
      eachMessage: async ({topic,partition,message}) =>{
        const data = JSON.parse(message.value.toString("utf-8"));
        await insert(data);
  }
})

I'm not entirely sure how to trigger this behavior though. The only thing I can think of is stopping the consumer, but even then I would expect that to have exited out without calling eachMessage.

thanks,and i closed the kafka autocommit,so it doesn't need to commit?


Added Timepicker UI component

Title

Added a time picker ui component.

Check List (Check all the applicable boxes)

[x] Bug(s) fixed
[x] Enhancement done
[x] New Feature(s) added
[x] Documentation changed
[x] All new and existing tests passed.
[x] My code follows the code style of this project.

Could you add the image here, i mean the output image?

Yeah sure here it is @thekavikumar

Rename the file timepickerstories.tsx to timepicker.stories.tsx

YEAH updated @thekavikumar

Just close all issues which are merged to avoid confusion @thekavikumar


"Invalid value for field 'resource.name'" if username contains "." character

I had the following error when trying to use this project to create a DCOS cluster on GCP :
terraform apply -var-file desired_cluster_profile.tfvars
[...]
 google_compute_network.default: 1 error(s) occurred:

* google_compute_network.default: Error creating network: googleapi: Error 400: Invalid value for field 'resource.name': 'guillaume.dupin-tfa0be-dcos-network'. Must be a match of regex '(?:[a-z](?:[-a-z0-9]{0,61}[a-z0-9])?)', invalid
[...]

It is because GCP resources are named with the username (returned by the script "whoami.sh") but if this name contains a dot ("."), it is not a valid character in a GCP resource.
Maybe a fix would be to remove dots (and any forbidden character) from the whoami result to avoid such error.

Thanks @yogeek for filing this. I will go ahead and add this check in GCP and related projects to ensure it provides the valid character.


Added a few Programs and Algorithms

I've added a few programs and Algorithms which will definitely help the community in building their DSA concepts stronger.

In depth explanation of Count Sort technique with the code in the DSA folder.
Full Code for Cycle Sort with explanation with Complexity Analysis.
Added a problem, First Missing Positive Integer with the Problem Statement along with the constraints and Test Cases with a optimized solution.

#hacktoberfest2021

@vijaygupta18 Please review. #123