Parsing of replay ends with error: CDOTAPlayer does not have field: m_iPlayerID

New replays end with error
Tried for<PHONE_NUMBER> and<PHONE_NUMBER>
Old replays work fine.
Error log:
Error: entity CDOTAPlayer does not have field: m_iPlayerID
    at get@https://timkurvers.github.io/redota/redota-parser+viewer-f20ee975.js:1:57370
    at processPlayer@https://timkurvers.github.io/redota/redota-parser+viewer-f20ee975.js:551:5230
    at onEntities@https://timkurvers.github.io/redota/redota-parser+viewer-f20ee975.js:551:2438
    at 7187/i.prototype.emit@https://timkurvers.github.io/redota/redota-vendor-6c7c4893.js?82f7cdcab7c725abab2c:2:148540
    at onCSVCMsg_PacketEntities@https://timkurvers.github.io/redota/redota-parser+viewer-f20ee975.js:1:52802
    at 7187/i.prototype.emit@https://timkurvers.github.io/redota/redota-vendor-6c7c4893.js?82f7cdcab7c725abab2c:2:148540
    at onCDemoPacket@https://timkurvers.github.io/redota/redota-parser+viewer-f20ee975.js:1:48620
    at 7187/i.prototype.emit@https://timkurvers.github.io/redota/redota-vendor-6c7c4893.js?82f7cdcab7c725abab2c:2:148540
    at seek@https://timkurvers.github.io/redota/redota-parser+viewer-f20ee975.js:1:48130
    at step@https://timkurvers.github.io/redota/redota-parser+viewer-f20ee975.js:1:47646
    at start@https://timkurvers.github.io/redota/redota-parser+viewer-f20ee975.js:1:47624
    at ve@https://timkurvers.github.io/redota/redota-vendor-6c7c4893.js?82f7cdcab7c725abab2c:2:171821
    at o@https://timkurvers.github.io/redota/redota-vendor-6c7c4893.js?82f7cdcab7c725abab2c:2:171362
    at 2129/m/<@https://timkurvers.github.io/redota/redota-main-a667d387.js?82f7cdcab7c725abab2c:213:964
    at Ns@https://timkurvers.github.io/redota/redota-vendor-6c7c4893.js?82f7cdcab7c725abab2c:2:317045
    at 53/t.unstable_runWithPriority@https://timkurvers.github.io/redota/redota-vendor-6c7c4893.js?82f7cdcab7c725abab2c:2:360235
    at Vo@https://timkurvers.github.io/redota/redota-vendor-6c7c4893.js?82f7cdcab7c725abab2c:2:257583
    at Ps@https://timkurvers.github.io/redota/redota-vendor-6c7c4893.js?82f7cdcab7c725abab2c:2:316508
    at 4448/As/<@https://timkurvers.github.io/redota/redota-vendor-6c7c4893.js?82f7cdcab7c725abab2c:2:316419
    at B@https://timkurvers.github.io/redota/redota-vendor-6c7c4893.js?82f7cdcab7c725abab2c:2:359296
    at 53/w.port1.onmessage@https://timkurvers.github.io/redota/redota-vendor-6c7c4893.js?82f7cdcab7c725abab2c:2:358019
    at

Thanks for the report!
It seems Valve has introduced m_nPlayerID on CDOTAPlayer, but it does not follow the same 0-9 values that m_iPlayerID previously had, which throws off color indexing (among other things).
Hope to have a fix ready this week. May also tie into #46, which is related to player associations.

Addressed in 7a4ab8b663082de1d090cd9a2a62587d024411f5 and should work for both old replays as well as anything on patch 7.31.
New build is available on website.

Awesome!


Add type check for eventProxy function

Very that event proxy callback function is actually a function

Hi Akeem,
Any context for this PR? I had left out the type check in order to show an error when the handler was the wrong type. Though I guess the error would be a bit useless ("cannot call method call of undefined"). Seems like it would be worth showing some form of error though?  Interested to hear your thoughts.

Hi @developit, this really goes in conjunction with https://github.com/developit/preact-markup/pull/3
When I wrote a test for the negative case where the allow-events weren't enabled, I got the undefined error.
However, I'm fine with throwing an Error / Exception instead if that's what you'd prefer.

Ah, excellent, that's what I was looking for.  I'm going to check if there is a way to throw a more useful error here, will let you know.  Part of an ongoing effort to provide a better debugging experience, but retain the dev <-> prod parity, instead of requiring that people switch builds or apply globals to get suitably production-worthy builds.

Updated to throw an Error now

I think we might be able to combine this with #153. If it can cover all the cases for event handlers, need add more tests and look at compat, then I believe the TypeError would be similar to this one. Will keep you posted as I merge these on Thursday/Friday (sorry for vacation delays!).

Hey Akeem - I'm going to close this one out since the work for #153 ended up throwing a TypeError here for non-function handlers.  Thanks so much for your work!!


Horse Surfing

Horse Surfing from Red Hat Summit

I just closed this PR. I swear I had a proper qna.yaml but I did not include the correct file. I will keep that in mind should I contribute in the future.


IMAGE16 - Design edge case for what to display when there is no file

Background:
On Single run page, if there is no tomogram there is also no metatada to display. This is not the same as no value provided ("--")
Design Need:

make it clear to user that there is no file and no metadata to display
clarify how to display on the single run page and info panel
identify how this pattern applies for other missing files/metadata

we should probably try to address this as a part of multiple tomogram work #372

Confirmed may still happen after multiple tomograms work because:

we can't/wont create a portal standard tomogram in some cases
there maybe a time gap between raw data/TS being added to the portal and making the standard tomo


use nvidia gpu on https://unlimited.waifu2x.net/ on win 11

i am windows 11 os pc.
with nvidia GPU.
without build from source.
if i visit, https://unlimited.waifu2x.net/. can i use GPU instead of CPU.
for now, it will use 100% cpu for generating.
thank you

这是章泽鑫的邮箱，我收到你的邮件了。

unlimited.waifu2x still does not support GPU.
It can be supported when WebGPU works on google-chrome and onnxruntime without any problems.
But it still has various problems does not work.


Allow all protocols

Fixes a couple of documentation issues.
Allow specifying the protocol to connect to, handy if you want to connect via wss:// for instance.

@kelindar happy to have a discussion about this. I probably could have opened a issue first?

@sarge This was merged in https://github.com/kelindar/etop/pull/3 - no changes with this PR

Ah my mistake:)


test error when service worker file calls importScript

About the issue
I have a service worker configuration and calling workbox-sw via importScripts, but I'm receiving an error message:

More details
NodeJS version: v6.9.1
Yarn version: v0.27.5
Test framework: MochaJS
service-worker.js
...
importScripts('workbox-sw.prod.v1.0.1.js');

/**
 * Create an instance of WorkboxSW.
 * Also tells service worker to take control as soon as it's activated.
 */
const workboxSW = new WorkboxSW({ // eslint-disable-line no-undef
  clientsClaim: true,
  skipWaiting: true,
});
...

service-worker.spec.js
// service-worker.spec.js

import assert from 'assert';
import { connect, destroy } from 'sw-test-env';
import path from 'path';
import nock from 'nock';

const LOCAL_FAKE_URL = 'http://localhost:3000';
const SW_FILE = path.resolve(__dirname, '../build/service-worker.js');
const SW_WEBROOT_WORKER_FOLDER = path.resolve(__dirname, '../node_modules/workbox-sw/build/importScripts/');

let sw, fake;

describe('Service Worker', () => {

  before(() => {
    nock.disableNetConnect();
    nock.enableNetConnect(/(localhost|domainexample)/);
  });

  beforeEach(() => {
    fake = nock('https://domainexample', { encodedQueryParams: true });
    sw = connect();
  });

  afterEach(() => {
    nock.cleanAll();
    destroy();
  });

  after(() => {
    nock.enableNetConnect();
  });

  it('should add information in cache', () => {

    fake
      .get('/index.js')
      .reply(200, 'function(){}')
      .get('/index.css')
      .reply(200, '.a{ color: "red"; }');

    const sw = connect(
      LOCAL_FAKE_URL,
      SW_WEBROOT_WORKER_FOLDER
    );

    return sw.register(SW_FILE)
      .then((registration) => sw.ready)
      .then((registration) => {
        const urls = Array.from(sw.scope.caches._caches.get('static-assets')._items.keys()).map((req) => req.url);

        assert(sw._sw.state, 'activated');
        assert(urls, ['/index.js', '/index.css']);
        done();
      });
  });

});

Ping @popeindustries

Thanks for the report. This looks like a problem with the fake IndexedDB set up (at least that's related to the error I'm getting when I import workbox-sw). I'll take a closer look...

I've improved the handling of importScripts and added a test for workbox-sw to check that it (basically) works. I hope this fixes things.


Networking engine does not provide response HTTP status code

Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
3.2.0
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from master?
Yes
Are you using the demo app or your own custom app?
Irrelevant
If custom app, can you reproduce the issue using our demo app?
Irrelevant
What browser and OS are you using?
Irrelevant
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
Irrelevant
What are the manifest and license server URIs?
Irrelevant
What configuration are you using?  What is the output of player.getConfiguration()?
Irrelevant
What did you do?
I was trying to implement "License Wrapping", based on the official tutorial, as the result I had the following code:
player.getNetworkingEngine().registerResponseFilter(function (type, response) {
  if (!type == shaka.net.NetworkingEngine.RequestType.LICENSE) return;

  if (response.status === 200) ...One scenario

  if (response.status === 201) ...Another scenario
});

Unfortunately, my code did not work and it took some time in order to find out the reason. It seems that Networking Engine does not provide Response Status which is weird, because it provides almost all the information.
What did you expect to happen?
I was expecting that response will include response.status.
What actually happened?
As I mentioned before, my License Wrapping implementation did not work, because I was not able to compare response.status. I don't know the reason why it was excluded from response object.

Yeah, that sounds like a reasonable thing to add to me.


Maximization rather than Minimization with GPyOpt

I want to solve a maximization problem with GPyOpt, but I do not know what changes are needed for this goal? If we have an analytic function like f converting f to (-1)*f will work, however, when we have only some observations of f, how we should deal with this problem?

Just present the negative of the observation.
On 20 May 2016 13:55, "Ali Baheri"<EMAIL_ADDRESS>wrote:

I want to solve a maximization problem with GPyOpt, but I do not know what
changes are needed for this goal? If we have an analytic function like f
converting f to (-1)*f will work, however, when we have only some
observations of f, how we should deal with this problem?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/SheffieldML/GPyOpt/issues/17


CSRF Cookie Regex not specific enough

Summary
It's not working anymore. Every Account i test is recognized as invalid with the reason "errorCSRF".
That implies Spotify changed their CSRF check to prevent mass checking.
Steps to reproduce
Steps to reproduce the behavior:

Put a 100% working account into your Combo List
run checker.php
Output will be:

[!] Email ID<EMAIL_ADDRESS>[!] Password : insecurepw
[!] Invalid Account!
[!] errorCSRF

EDIT
Nevermind, the csrf check didn't change that much, but when the Regex to detect the set-cookie header ist to unspecific so it'll match the very first set-cookie header it finds, no matter if it's the csrf token or not.
Try /^Set-Cookie:\s*(csrf[^;]*)/mi, that worked for me

Thank You VERY much. I was just about to fix the regex


⚠ Bofrost.lu has degraded performance

In 0d31bac, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1841 ms

Resolved: Bofrost.lu performance has improved in f7674e8 after 21 minutes.


Different starting point in Clipper1 offset result vs Clipper2 offset result

Hi,
is it possible that offset results of the previous version start at different point of the same offset operation results performed with Clipper2?
Or should the new starting point be preserved, provided to use the same input?

is it possible that offset results of the previous version start at different point of the same offset operation results performed with Clipper2?

Yes.

Or should the new starting point be preserved

No.


Provide docs on using KEDA with HTTP scaling

Provide docs on using KEDA with HTTP scaling via Osiris.
Current docs we had on kedacore/keda:

HTTP scaling integration
KEDA enables scaling based on event sources where the event resides somewhere to be pulled. For events like HTTP where the event is pushed to the container, KEDA works side by side with HTTP scale-to-zero components like Osiris or Knative serving. By pairing KEDA with an HTTP scale-to-zero component you can provide rich event scaling for both HTTP and non-HTTP.

I haven't tried this before so:

Do we have a sample for this?
What docs should we add?
How do KEDA and Osiris work together or is this purely Osiris?

Do you know @jeffhollan?

Do we have a sample for this?

No. Not existing

What docs should we add?

I’m not sure we add much. If we did add something it would be in like the FAQ I think about “can KEDA scale HTTP requests”

How do KEDA and Osiris work together or is this purely Osiris?

Purely Osiris.  Osiris does 0-1 scaling for HTTP (as well as other alternatives like Knative or OpenFaaS).  If you wanted to do HTTP scaling 1->n based on request rate you could use KEDA, but I think what we’d recommend is something like the frontend or ingress or something is emitting metrics to prometheus.  KEDA then scales the deployment using the Prometheus scaler on some rps metric

Interesting, let me put it otherwise - If Azure Function customers move to Kubernetes and they use HTTP trigger; what would be your guidance then?
This feels like a gap in the guidance/docs imo 🤔

My guidance to them would be for now they'd have to rely on Kubernetes standard HPA scaling unless they are using an ingress controller that supports RPS scaling.
But would be on this doc https://docs.microsoft.com/en-us/azure/azure-functions/functions-kubernetes-keda
Which I plan to update today with latest KEDA info. I can include a blurb about this

It's documented in FAQ, so closing for now.


[docs] Fix typo in toggle-button.md

[x] I have followed (at least) the PR section of the contributing guide.

They are a two other arial-label= issues to fix in the codebase.

Added to batch


enhance(ws-api): support compact format to reduce api io

This PR

would not break old behaviour
support format='compact' control field for ws api to reduce response size

@634750802  PTAL

Can be merged?

@shczhen @Mini256  PTAL c34eb27
@Mini256 PTAL 3475178

Maybe we can use rowsAsArray ?
https://github.com/sidorares/node-mysql2#array-results

@Mini256
This argument cannot affect cached result, so we can only transform the result manually...


SoapServiceClient doesn't allow runtime binding/channel creation. New constructor perhaps?

Hi,
I'm using the library for my project where I have to scan the local area network for computers and discover the wcf services on them, and when there's one I create a channel factory for that service.
Unfortunately the SoapServiceClient class doesn't let you provide a runtime generated endpoint, so I have added it, and I'd like to ask you for your opinion about this change.

Hi,
Agree with you, sometimes it could be useful. I'll republish the package today. Thanks a lot.

Done https://www.nuget.org/packages/Nelibur/3.1.8 thanks again.


[#1813] fix(doc): Update the internal links for md files

What changes were proposed in this pull request?
Fix https://github.com/datastrato/gravitino/issues/1813
There are 5 files, and 13 links in total, the links are wrong.
Why are the changes needed?
The original link was not complete, so when we click the link, it jumped to an error page in Github.
Does this PR introduce any user-facing change?
N/A
How was this patch tested?
N/A

Thank for the fix @SophieTech88 .


First Translation Guide #75

Description
A new folder has been created for the translation guide and translations to be saved.
I hope to add Translation guide link to README.md after this branch is successfully merged.
Screenshots
There is a new folder called translations.

There is a translation guide inside the folder, and when someone translate documents into a new language, they create a new folder inside it.

Related Issue
https://github.com/priyankarpal/ProjectsHut/issues/75
Checklist
Please review and check off the following items before submitting your PR:

[x] I have tested my changes to ensure they work as expected
[x] I have included documentation for any new features or changes
[ ] I have formatted my code according to the project's style guidelines
[ ] I have written tests for any new features or changes
[ ] I have run the project's test suite and verified that all tests pass
[ ] I have updated the README and any other relevant documentation
[x] I have checked for any spelling or grammatical errors

Additional Information
The Korean translation will work with the new branch after this branch has merged for ease of project management.
It will be a sample for others.

I was going to make a new branch after this, but do you think it would be better to proceed here?

Looks good but could you add one language translation?

I think you can do it here.

Ok, I push again after I translated. :)

I have a question in the last sentence of this part. Please let me know if it is connected to the previous sentence or if there is a separate sentence with more words.

I have a question in the last sentence of this part. Please let me know if it is connected to the previous sentence or if there is a separate sentence with more words.

separate

The Korean translation has been completed. I created the folder as I wrote in the translation document. After merging, I would like you to add a link to the translation guide document on README.md in the main page.

The Korean translation has been completed. I created the folder as I wrote in the translation document. After merging, I would like you to add a link to the translation guide document on README.md in the main page.

but is there any problem?

Oh, I don't know why you have problem. But my devices I tested are fine.
And the results of 3 files seem I wanted. They are translated.
Desktop (OS: Window 10, Root: Chrome, MSEdge)

Laptop (OS: MacOS Ventura 13.2, Root: Chrome)

Mobile (OS: Android 13, Root: GitHub Application)

Oh, I don't know why you have problem. But my devices I tested are fine. And the results of 3 files seem I wanted. They are translated.
Desktop (OS: Window 10, Root: Chrome, MSEdge)
Laptop (OS: MacOS Ventura 13.2, Root: Chrome)
Mobile (OS: Android 13, Root: GitHub Application)

ok ok I think I'm using Brave so I'm facing this problem.

Thank you for chance to contribute ;)


GroupId and ArtifactId can be provided as parameters

https://github.com/vaadin/metathesis/pull/6 needs to be merged for this to work

Review status: 0 of 4 files reviewed, 2 unresolved discussions, some commit checks failed.

bin/metathesis.js, line 46 [r1] (raw file):
Running node bin/metathesis.js -h does not show the help for these commands

pom-template.xml, line 1 [r1] (raw file):
This file should be under the templates folder

Comments from the review on Reviewable.io

Review status: 0 of 5 files reviewed, 2 unresolved discussions, all commit checks successful.

bin/metathesis.js, line 27 [r1] (raw file):
Done.

pom-template.xml, line 0 [r1] (raw file):
Done.

Comments from the review on Reviewable.io

Review status: :shipit: all files reviewed, all discussions resolved, all commit checks successful.
Reviewed files:

bin/metathesis.js @ r2
gulpfile.js @ r2
pom-template.xml @ r2
template/GwtModule.template @ r1
template/pom.template @ r2

Comments from the review on Reviewable.io


Closes softlayer/sl-ember-components#987

this might be related to #980

Reviewed 14 of 14 files at r1.
Review status: all files reviewed at latest revision, 32 unresolved discussions, some commit checks failed.

tests/integration/components/sl-alert-test.js, line 32 [r1] (raw file):
This assertion should be removed per https://github.com/softlayer/sl-ember-components/issues/980 and https://github.com/softlayer/ember-style-guide/issues/67

tests/integration/components/sl-button-test.js, line 33 [r1] (raw file):
Move this test case to be the first one in the test suite.

tests/integration/components/sl-button-test.js, line 36 [r1] (raw file):
Invalid const name per style guide

tests/integration/components/sl-button-test.js, line 55 [r1] (raw file):
This assertion should be removed per https://github.com/softlayer/sl-ember-components/issues/980 and https://github.com/softlayer/ember-style-guide/issues/67

tests/integration/components/sl-button-test.js, line 60 [r1] (raw file):
This assertion should be removed per https://github.com/softlayer/sl-ember-components/issues/980 and https://github.com/softlayer/ember-style-guide/issues/67

tests/integration/components/sl-date-range-picker-test.js, line 21 [r1] (raw file):
This assertion should be removed per https://github.com/softlayer/sl-ember-components/issues/980 and https://github.com/softlayer/ember-style-guide/issues/67

tests/integration/components/sl-date-range-picker-test.js, line 33 [r1] (raw file):
This assertion should be removed per https://github.com/softlayer/sl-ember-components/issues/980 and https://github.com/softlayer/ember-style-guide/issues/67

tests/integration/components/sl-drop-option-test.js, line 18 [r1] (raw file):
This assertion should be removed per https://github.com/softlayer/ember-style-guide/blob/master/ember-testing.md#component-unit-tests

tests/integration/components/sl-drop-option-test.js, line 24 [r1] (raw file):
This test case should be rolled into the "Default rendered state" one

tests/integration/components/sl-modal-footer-test.js, line 14 [r1] (raw file):
Follow https://github.com/softlayer/ember-style-guide/issues/66

tests/integration/components/sl-modal-header-test.js, line 14 [r1] (raw file):
Follow https://github.com/softlayer/ember-style-guide/issues/66

tests/integration/components/sl-modal-header-test.js, line 35 [r1] (raw file):
Rename this test case

tests/integration/components/sl-modal-header-test.js, line 48 [r1] (raw file):
Remove this call.

tests/integration/components/sl-modal-header-test.js, line 52 [r1] (raw file):
Move this assertion to to the "Default rendered state" test suite

tests/integration/components/sl-modal-test.js, line 61 [r1] (raw file):
The template used for this test case should be the bare minimum of provided values.  Passing ariaLabelledBy, for example, should not be done.

tests/integration/components/sl-modal-test.js, line 235 [r1] (raw file):
This test case needs to be rolled into "Default rendered state one

tests/integration/components/sl-modal-test.js, line 248 [r1] (raw file):
Combine this test case with the one below and use this.set() calls for the animated property.

tests/integration/components/sl-modal-test.js, line 313 [r1] (raw file):
Rename this test case to more accurately reflect what property presence causes aria-hidden to be true.

tests/integration/components/sl-panel-test.js, line 29 [r1] (raw file):
https://github.com/softlayer/ember-style-guide/issues/66

tests/integration/components/sl-panel-test.js, line 34 [r1] (raw file):
https://github.com/softlayer/ember-style-guide/issues/66

tests/integration/components/sl-panel-test.js, line 39 [r1] (raw file):
https://github.com/softlayer/ember-style-guide/issues/66

tests/integration/components/sl-panel-test.js, line 44 [r1] (raw file):
Merge this test case into "Default rendered state" one

tests/integration/components/sl-panel-test.js, line 71 [r1] (raw file):
This render can be removed

tests/integration/components/sl-panel-test.js, line 75 [r1] (raw file):
This assertion can be removed

tests/integration/components/sl-panel-test.js, line 86 [r1] (raw file):
Do we want to use hasClass() here?  If not, do we want to create a single reference to this.$( '>:first-child' ).find( '.panel-heading' ) to be used in both assertions?

tests/integration/components/sl-panel-test.js, line 99 [r1] (raw file):
Remove this call

tests/integration/components/sl-panel-test.js, line 103 [r1] (raw file):
Remove this assertion

tests/integration/components/sl-panel-test.js, line 127 [r1] (raw file):
Use hasClass()?

tests/integration/components/sl-tab-panel-test.js, line 84 [r1] (raw file):
Merge this test case with the "Default rendered state" one

tests/integration/components/sl-tab-panel-test.js, line 134 [r1] (raw file):
Merge this test case with the "Default rendered state" one

tests/integration/components/sl-textarea-test.js, line 262 [r1] (raw file):
Remove this test case

tests/integration/components/sl-textarea-test.js, line 403 [r1] (raw file):
Move this assertion to the "Default rendered state" test case

Comments from the review on Reviewable.io

Review status: all files reviewed at latest revision, 3 unresolved discussions, some commit checks failed.

tests/integration/components/sl-panel-test.js, line 44 [r1] (raw file):
no need; corrections to the default test make this one obsolete, as they test the same thing inherently

tests/integration/components/sl-panel-test.js, line 86 [r1] (raw file):
Can't since this is a child node.  We would have no reliable way of finding it.

tests/integration/components/sl-panel-test.js, line 127 [r1] (raw file):
can't

Comments from the review on Reviewable.io

Reviewed 10 of 10 files at r2.
Review status: all files reviewed at latest revision, 6 unresolved discussions, some commit checks failed.

tests/integration/components/sl-date-range-picker-test.js, line 33 [r1] (raw file):
A test case for this needs to be added to the test suite.

tests/integration/components/sl-date-range-picker-test.js, line 315 [r1] (raw file):
Remove this test and associated comment

tests/integration/components/sl-date-range-picker-test.js, line 10 [r2] (raw file):
There should be an assertion that there are to instances of sl-date-picker components with the correct classes on them

tests/integration/components/sl-date-range-picker-test.js, line 22 [r2] (raw file):
Remove this assertion.
A test case for this needs to be added to the test suite.

tests/integration/components/sl-panel-test.js, line 29 [r2] (raw file):
Replace with panelBody.length, after moving definition of panelBody before this assertion

tests/integration/components/sl-tab-panel-test.js, line 53 [r2] (raw file):
Replace with "Tabs display in expected order"

Comments from the review on Reviewable.io

Reviewed 6 of 6 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from the review on Reviewable.io

There are merge conflicts

Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from the review on Reviewable.io

Reviewed 1 of 1 files at r4.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


Can emqtt save the publish and subscribe information to the database as a record?

Environment

OS:  ubuntu 16.04 64bit
Erlang/OTP:
EMQ: V3.1.1

Description
A description of the issue
The publish and subscribe information received by the server is stored in the database, which can be set for a period of time, or when the data arrives at the server.Can the server do this by configuring or downloading plug-ins?If it is not possible, is there any way to implement it through additional programs?

Has been resolved！


Release v0.4.2

@rnestler here's your bugfix release.

:+1: Maybe add a note about the new meta-data in Cargo.toml
Also we should probably yank the 0.4.1 version.

Also we should probably yank the 0.4.1 version.

Nah, don't think it's that bad. Besides that, nobody besides us is using it :laughing:


Longview Client Overview

https://github.com/linode/manager/pull/2633 includes the top processes, a CPU and Network graph.  There are still a number of problems with these graphs in this PR.


Fixed "app_job" status "pending" after install 3rd-party app (#7863)

Solves the bug #7863

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Ledger hww login to raspiblitz not working after v1.8.0

Hi, before updating to v.1.8.0 I had my raspiblitz set up so that log in was via ssh with my key provided/confirmed through an ssh app on my hww (ledger nano). After the upgrade I repeated the steps I took when setting up this secure access process, earlier this year - but it doesn't seem to work. Specifically, after removing the password option, I'm given the error error<EMAIL_ADDRESS>Permission denied (publickey). Steps taken detailed below.
System: 8GB raspberry pi. SW v1.8.0.
HWW: Ledger Nano X (also same issue with S)
home computer: imac mini, ssh opened through a mac terminal
Steps used to set up ssh key login provided from ledger hww.

generate public key with ledger-agent
add this to ~/.ssh/authorized_keys, making sure permissions for both .ssh and authorized_keys were correct,
then remove ability to use password in /etc/ssh/ssh_config and sshd_config
restart both services.
attempt to ssh in to LN from a new terminal -> error<EMAIL_ADDRESS>Permission denied (publickey)
When connecting with ssh, it prompts the ledger for authorisation (good) but I don’t get any request for authorisation to log into the LN itself (i.e. before update I would need to approve twice on my ledger, now I am only asked for one approval and am then give then error<EMAIL_ADDRESS>Permission denied (publickey).).
I kept access to the LN thorugh a few ‘back up’ terminals / consoles so now I’ve rolled back to using pwdA log in. So I’m not in trouble but I would like to harden access to the LN. Have I missed a step, or has there has been a change in how I need to configure this (e.g. is pasting the public key into authorized_keys now not the right place to put it?). I'm still in the 'learning by doing' phase, so it is possible I've missed something obvious to someone more familiar with ssh config & management.

You got a ledger to work as an ssh key?  That's tremendous! If this issue gets fixed I would also like to also set that up.  I'm "learning by doing" with a trezor to ssh to my linux box.  but so far I'm afraid to turn off the password option.
Slight change of topic: How did you get your ledger to become a security key?  I thought I saw it in the ledger app list a while back, but now I can't find it.

Thanks - I leant heavily on a fusion of the following guides (although there was still, as far as I recall, some trial and error - I took unstructured notes as I attempted various modifications, & can attempt to tidy up if that's of interest)

https://thoughts.t37.net/a-step-by-step-guide-to-securing-your-ssh-keys-with-the-ledger-nano-s-92e58c64a005
https://www.digitalocean.com/community/tutorials/how-to-configure-ssh-key-based-authentication-on-a-linux-server
https://blog.ledger.com/ssh/
If you can afford it / have a trusted friend with another hww, I'd definitely recommend setting up an alternative hww with access before you turn off the pwd (and probably rehearse the full lightning node recovery option, if you've not already). Ledger's app is called 'SSH/PGP Agent' (at least, that's the display name on the device).

@chrisBonGit this is a cool use of a hardware wallet. Its a bit special so I will try to take a quick look into it for the v1.8.1 release. Basically not much changed from v1.72 to v1.8.0 on the SSH part.

I ran into a setup problem within my setup and missing a bit the time to go deeper on it. Pushing this issue a bit up the timeline - but happy if someone else can help out (test) in the meantime.

I'm not a SW dev, but I'm happy to learn, I work more on algo design and test of returned SW. If it could help, I'm happy to flash a SD card with the test-image & try it out on  my dev/learning-raspiblitz - I don't mind investing a bit of time to see if I can get hww back in operation. I am travelling for work for a few days, but will have a go this weekend - if you confirm that it could be helpful. Also, github noobie question, which branch should I take?

Any practical report on modifying your RaspiBlitz to being able to use the Ledger "SSH/PGP Agent" to SSH login would help.

Sorry for taking so long to respond. Below are the notes I made when I was able to successfully get this to work (I think this was sometime in early 2021, say Feb / March). I was not able to get this working after the upgrade to v1.8.0, I will try again if I can make some time.
Security Steps for RaspiBlitz
Lock down access to LNDs within Home Network
• Use switch to isolate LNDs from most of home, [Managed Switch ports]
Set up SSH key access (ledger agent)
• Using https://thoughts.t37.net/a-step-by-step-guide-to-securing-your-ssh-keys-with-the-ledger-nano-s-92e58c64a005 (also, but less clear help here https://blog.ledger.com/ssh/)
• Once ssh app on HWW & ledger agent is on computer, then the enabling tech is to have python running in terminal and use
○ ledger-agent username@hostname
○ Create a directory (as admin) on raspiblitz /~/.ssh
○ Copy public key into ~/.ssh/authorized_keys on remote server (LNDXX). NOTE must include ecdsa-sha2-nistp256 at start.
○ Use chmod to set permissions of .ssh folder to 700 (drwx) :q& authorised keys to 600
○ Restart ssh service:
§ service sshd restart
○ NOT REQUIRED. [Add public key locally (ie. On mac / linux machine) in ~/.ssh/ledger.conf] That was my misunderstanding. Doing this forces use of ssh key to start the ssh session, then another confirmation from node to log in…
• Repeat above for redundant hww
• Then, with one terminal already open and connected to node…
○ Remove password as an option
1. edit /etc/ssh/ssh_config, you need to do this as a super user (e.g. sudo vi …)
1. uncomment the line #PasswordAuthentication no (line 25)
2. Change no to yes above
2. Restart ssh server
1. service ssh restart
3. Test log in from a new terminal. KEEP safety line (i.e. terminal that is already connected to node) attached.
Remove Password access
1) On raspiblitz, edit /etc/ssh/ssh_config, you need to do this as a super user (e.g. sudo vi …)
a. uncomment the line 25, #PasswordAuthentication no
b. Change no to yes above
2) Restart ssh server
a. service ssh restart
3) Test log in from a new terminal
4) Note, some refs state that you also need to check sshd_config (I found lines commented out in my raspiblitz install)
a. PAM = privileged access management.
b. In sshd_config
i. Uncomment & Modified line 56; PassWordAuthentication from yes to no
ii. :Did not need to modify line 61 (ChallengeResponseAuthentication was already set to no)
iii. Modified line 84; UsePAM yes to UsePAM no
Steps in 2 successfully disables password (after a service ssh restart)


Make build script swift version default to 3.1

Make build script swift version default to 3.1. This was overriding the CMake default value.

@swift-ci please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - ae5dc78c12a06543ab85835727eb3cbeeae695b5
Test requested by - @zisko

@swift-ci please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - ae5dc78c12a06543ab85835727eb3cbeeae695b5
Test requested by - @zisko

@swift-ci Please smoke test linux

@shahmishal , is something going on with the linux bot, or is this a legit failure?

@swift-ci please test

@swift-ci Please smoke test linux

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - ae5dc78c12a06543ab85835727eb3cbeeae695b5
Test requested by - @zisko


Analyzer or DDC build script doesn't understand generic function types.

I have a CL: https://dart-review.googlesource.com/c/sdk/+/26620
When trying to build the DDC SDK, I get the following error:
$ tool/build_sdk.sh
*** Patching SDK
*** Compiling SDK to JavaScript
[error] Analysis of generic function typed parameters is not yet supported. (dart:core/set.dart, line 101, col 46)

That source line in set.dart contains the following declaration:
  static Set<T> castTo<S, T>(Set<S> source, {Set<R> newSet<R>()}) =>
      new CastSet<S, T>(source, newSet);

This should be a perfectly valid generic function type, so this seems like a problem with the analyzer (guessing it's the analyzer based on the error message talking about "Analysis").
Workaround: Change the parameter syntax to {Set<R> Function<R>() newSet} and it works without warning.

I'm assuming that this is stale. Please let us know if you're still seeing this problem.


[Feat]: Dark Mode

Feature Description
An icon or button in the navbar for the user to switch modes
Provide a detailed description of the feature
i wanna add an icon that will have some effects and when the user click on it It changes the mode
Put here any screenshots or videos (optional)
No response
By submitting this issue, you agree to

[X] I agree to follow this project's Code of Conduct
[X] I’ve read all the issues in the issue section, and this feature has not been proposed yet.

Other Information

[X] I'm a GSSoC '24 Extended contributor
[X] I'm a Hacktoberfest '24 contributor
[ ] I am willing to work on this feature (optional)

@Isabelle36 Home page is under progress so any changes you will made will clash with other contributor. You can wait for sometime.

ok

Hey @maniyaom I am assigned this project as a mentor under ggsoc-extd so can you add me as a collaborator on this project so I can help you and contributors to


Added a Survey form under forms section

Fixes Issue🛠

Closes #253
Description👨💻

Type of change📄

[] New feature (non-breaking change which adds functionality)
Added a survey form under form section

How this has been tested✅

Checklist✅

[] My code follows the style guidelines of this project
[] I have performed a self-review of my own code
[] I have commented my code, particularly in hard-to-understand areas
[] I have added demonstration in the form of GIF/video file
[] I am an Open Source Contributor

Screenshots/GIF📷

Merged 🎉 @Dhairya1Gupta


Delete discounts

Currently there is no way to delete discounts through the Admin Hub UI.
Is this intentional or an oversight?

Was this not handled by #840 ?

Yep it seems so, I missed that.


[Question]: how to add extension, spiral/jobs for example, to roadrunner?

Didn't find the info in the doc.
If I want to use spiral/jobs extension with roadrunner, do I need to edit cmd/rr/main.go, add spiral/jobs service, and rebuild roadrunner?
thanks.

@wppd Jobs module already contains RR, so, it is a self-sustained module. If you need only Jobs, you can safely use spiral/jobs. If you need to add other modules, you can add needed modules to the spiral/jobs.
You can see here a sample of all modules in one place: Repo
Note: we are working on the better module system for the RR 2.0

You can also spiral binaries which already include jobs: https://github.com/spiral/framework/releases

@wppd Feel free to close the ticket if you have no other questions.


Remove /moveto command

While originally well-intentioned, the /moveto #channel command seems to tend to stifle conversation instead of moving it to a better channel. IMO it should be removed, and people should just ask kindly for the conversation to move. As @mrasband put it, it's a bit cold.

Agreed on this, it tends to totally shut down the conversation rather than move it. If nothing else, we should consider alternatives. I think right now maybe /moveto could auto-generate a message from the current user and post as them instead (or just request people to nicely ask)? Something like:
Sorry to ask this, but it seems this conversation has gone off topic and is conflicting with ongoing on-topic conversations here. Can you please move to #<channel>


ActiveDirectoryAuthenticationProvider.AcquireTokenAsync method fails when ConnectTimeout set to 0 or >= int.MaxValue/1000

Well that was fun to debug!
https://github.com/dotnet/SqlClient/blob/3f0c4b1a706136e417efa3a49bc168342470513e/src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/ActiveDirectoryAuthenticationProvider.cs#L117
As the ConnectionTimeout is used to set the cancellation token's CancelAfter * 1000...

If ConnectionTimeout = 0 then 0 * 1000 = 0 - this causes the cancellation token to immediately expire, so all operations such as gather the tenant metadata fail with A task was canceled exception message (which is what led me down this rabbit hole).
If ConnectionTimeout = int.MaxValue (or any value greater than int.MaxValue / 1000) then int.MaxValue * 1000 throws as the result is larger than int.

Ideally this needs more defensive coding, eg:

When ConnectionTimeout == 0 we do not require a CancelAfter?
When ConnectionTimeout >= int.MaxValue / 1000 set the CancelAfter to int.MaxValue?

Thanks for pointing this out. We'll likely add this to our backlog


Added "CustomActionTarget"

Added a new target that uses an implementation of an interface to run a coded action.
By implementing IActionProvider interface and registering the instance in the CustomActionTarget you can perform any action when a LogEvent is logged.
Example
Logging to console using a custom action
The configuration file looks like this.
<?xml version="1.0" ?>
<nlog xmlns="http://www.nlog-project.org/schemas/NLog.xsd" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
  <targets>
    <target name="myTarget" xsi:type="CustomAction" layout="${date:format=HH\:MM\:ss} ${logger} ${message}" />
  </targets>

  <rules>
    <logger name="*" minlevel="Info" writeTo="myTarget" />
  </rules>
</nlog>

To write the code for the action we want to perform we must build a class that implements the interface IActionProvider (located in NLog.Targets namespace).
using System;
using NLog;
using NLog.Targets;

public class ConsoleActionProvider : IActionProvider
{
    public void Action(TargetWithLayout target, LogEventInfo logEvent)
    {
        Console.WriteLine(" Action Povider: {0}; Target {1}; Event: {2}",
             this.GetType(), target.Name, target.Layout.Render(logEvent));
    }
}

After that we must create an instance of this class and register it on the CustomActionTarget static action set.
class Example
{
    static void Main(string[] args)
    {
        ConsoleActionProvider provider = new ConsoleActionProvider();
        CustomActionTarget.Register(provider);

        Logger logger = LogManager.GetLogger("Example");
        logger.Info("log message");
        logger.Error("error message");
        Console.ReadKey();
    }
}

The result is:
 Action Povider: ConsoleActionProvider; Target myTarget; Event: 18:03:10 Example log messagege
 Action Povider: ConsoleActionProvider; Target myTarget; Event: 18:03:08 Example error message

It is possible to unregister an action provider by using CustomActionTarget.Unregister(provider)
P.S. I tried to add the new files to all the projects, but in doing so I may have added some unwanted changes to the solution files ( src/NLog.monodevelop.sln; All_Targets.sln and NLog.wp7.csproj) (because I only have visual studio 2012 and 2013). If I did please ignore those changes.
I also added unit tests and examples

Looks cool! I will check the code in detail this week
About the project files: Did you check https://github.com/NLog/NLog/blob/master/CONTRIBUTING.md#sync-projects?

Yes, I ran the msbuild script. The result was no errors, no warnings.

Well your unit tests are failing ;)
See AppVeyor details

From AppVeyor I see that the tests fail when checking the InternalLogger after an error.
But when I run these tests locally on my PC they don't fail. I also ran AppVeyor on the forked project on my account and these 3 tests were successful.
Do you have any idea on how to solve this?

Did you try the failing unit tests on all 3 .net solutions? (3.5, 4.0, 4.5)

The test run successfully on all 3 solutions.
I have another 3 tests that fail, but I don't think they have any connection with the new functionality.
The test that fail are

ImpersonatingWrapperTest, RevertToSelfTest - becuse I do not have the user NLogTestUser
Extension_should_be_auto_loaded_when_following_NLog_dll_format - because the test are run by xunit in a temporary folder where I do not have the extension dlls.

I don't like the idea that custom actions providers are registered statically. What if I need to have two CustomActionTargets configured and have first to invoke one provider, and second to invoke another one?

I agree. Multiple custom actions providers should be possible

This is very useful input. I agree with you, but I wanted the action providers to be developed as part of the consumer code. Given that the targets can be configured at run time I don't think there is any way to make the action provider register to a specific target.
Another approach wold be to let the target specify the provider action to run but this has several problems. You wold have to use reflection, you wold not be able to specify a specific instance of the provider and you wold not be able to register/unregister in the consumer code.
What do you think?

I propose:

extend the <target with an extra attribute, "action"
extend CustomActionTarget.Register with an extra parameter, "action"

Then we can register multiple actions. This can be done with a (case intensive) dictionary's and no usage of reflection is needed. ;)

Ok. I'll try and do that.

:+1:

Do you need help on this @sorin-bolos ?

Closing this because of the lack of activity and because we are closing stuff for the upcoming release. Feel free to reopen when it's active again.


Fine-tuning

Hello @eltociear @cnulab
Thanks for you great work. Can I fine-tune your model on new defect private dataset which is related with defects? If yes, How we can do it? Because there is no file and you can just give me the hint.
Regards,
Khawar

Thank you very much for your interest in our work.
I understand that you want to train (fine-tune) a new anomaly synthesis model on your private dataset. If yes, you first need to reimplement the SDASDataset class in datasets/sdas_dataset.py to ensure the correct loading of your dataset. Then, use your custom Dataloader in train_diffusion.py. I recommend that you fine-tune from the ImageNet diffusion checkpoint, as that might achieve better generalization.
If you have any further questions, please feel free to continue the conversation.


ssl error fixed on android

@HamzaNadeemIqbal Can you explain what you're doing here? Looks like you're just removing the error handling code?

This code was not letting my webview display url without ssl validation as i did not have access to the website itself, so i landed upon a suggestion to remove the handler which worked for me @jamonholmgren

Any update?


Gringotts & Towny

Hey guys, I'm having an issue setting up Gringotts to work with Towny. I am using Towny for 1.7.10
I assume Towny is not seeing Gringotts, when I try to make a town, it says I have insufficient funds. I have the appropriate amount of emeralds on my person, as well as some in a vault.
Any thoughts? Im pretty new to both plugins so any suggestions would be great!
-Cain

Towny isn't directly aware of Gringotts. Instead, Gringotts integrates with Towny.
Have you checked https://github.com/MinecraftWars/Gringotts/wiki/Troubleshooting ?

Does it work for you?

Assumed obsolete, closing.


regression in casting between 35 and 36.

Describe the bug
Previously (df 35) if you did an invalid cast, it would fail during logical planning. However, in datafusion 36, it does not fail during planning as expected.
A way to visualize this is via explain. It'll show an empty result set if the planning errored out.
To Reproduce
A simple example of an invalid cast, casting string to int;
Datafusion 35
DataFusion CLI v35.0.0
❯ explain select CAST('test' as int);
0 rows in set. Query took 0.016 seconds.

Datafusion 36
DataFusion CLI v36.0.0
❯ explain select CAST('test' as int);
+---------------+------------------------------------------------------------+
| plan_type     | plan                                                       |
+---------------+------------------------------------------------------------+
| logical_plan  | Projection: CAST(Utf8("test") AS Int32)                    |
|               |   EmptyRelation                                            |
| physical_plan | ProjectionExec: expr=[CAST(test AS Int32) as Utf8("test")] |
|               |   PlaceholderRowExec                                       |
|               |                                                            |
+---------------+------------------------------------------------------------+
2 rows in set. Query took 0.008 seconds.

### Expected behavior

in this example, i'd expect the cast to fail during planning as it did in 35

### Additional context

datafusion 35 will fail at 'simplify_expressions`

```sql
❯ select CAST('test' as int);
Optimizer rule 'simplify_expressions' failed
caused by
Arrow error: Cast error: Cannot cast string 'test' to value of Int32 type

while 36 does not seem to specify anything about optimization, and seems to fail at execution instead of planning
❯ select CAST('test' as int);
Arrow error: Cast error: Cannot cast string 'test' to value of Int32 type

nevermind, this appears to have been planned https://github.com/apache/arrow-datafusion/pull/8957

Thanks @universalmind303  -- if you need a different behavior (like fail early rather than fail when the cast is tried) maybe we can make a way to control the behavior via a setting or something


Shouldn't Inline Sequentialization check for moverness?

In the code below (updated NoChoice.bpl), I removed the mover type of async actions which are being inlined and the verification still went through. Is this expected? Shouldn't we verify the mover type of the actions before inlining?
// RUN: %parallel-boogie "%s" > "%t"
// RUN: %diff "%s.expect" "%t"

var {:layer 0,2} x:int;

////////////////////////////////////////////////////////////////////////////////

atomic action {:layer 1} MAIN ()
refines MAIN';
creates INC, DEC;
{
  async call INC();
  async call DEC();
}

atomic action {:layer 2} MAIN' ()
{
}

////////////////////////////////////////////////////////////////////////////////

async action {:layer 1} INC ()
modifies x;
{
  x := x + 1;
}

async action {:layer 1} DEC ()
modifies x;
{
  x := x - 1;
}

I think I see why it worked now, we assume by default ISL is being used for action refinement if ISR is not, and when we use ISL we assume that all eliminated actions should be left movers so we check for left moverness of all eliminated actions even though they are not explicitly specified as left in their definition


Cant bind array of objects

If i have in my view:

And in the controller:
$scope.arrayOfObjects = [
  {src: ''souce_url'},
  {src: ''souce_url'},
  {src: ''souce_url'},
];

Why could happen this error in console?

*Notes:
-Controller is working well.
-Array is visible in view.
-Array is not empty.

Hi Jovany
What version of angular you are using ??

close issue, yeah, it was the crush of versions. thanks a lot.


fix: connect through apigw

Fixing the connect through API gw

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.33%. Comparing base (8c614c9) to head (fea774d).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #9      +/-   ##
==========================================
- Coverage   97.38%   97.33%   -0.06%
==========================================
  Files          17       17
  Lines         804      787      -17
==========================================
- Hits          783      766      -17
  Misses         21       21

Flag
Coverage Δ

unittests
97.33% <100.00%> (-0.06%)
:arrow_down:

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

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


How to read env in plugin

I use docker run -e "env_name=xxx"   kong:2.4.0-ubuntu  to export env into container. Then I hope to get it by using os.getenv("env_name") in lua plugin. But it doesn't work and responds with nil.

Hi @Asilent,
This is expected behavior; in order to be able to read an environment variable from Lua code, you need to specify it in the Nginx env directive. In Kong, that can be done with directive injection.
In your case, you can achieve that by setting the following Kong configuration as an environment variable: KONG_NGINX_MAIN_ENV=env_name.
Closing this, but feel free to reopen if needed.
Happy Konging! 🦍

Alternatively you can read the variable when the plugin code loads, since that is executed in the master process where variables are still available

I use directive injection like -e KONG_NGINX_HTTP_CUSTOM_ENV=env_value to add custom env when start the docker.

Afrer this action, kong container can't restart with error.
The Error is as follows:

What should I do?

Hi @Asilent,
This is expected behavior; in order to be able to read an environment variable from Lua code, you need to specify it in the Nginx env directive. In Kong, that can be done with directive injection.
In your case, you can achieve that by setting the following Kong configuration as an environment variable: KONG_NGINX_MAIN_ENV=env_name.
Closing this, but feel free to reopen if needed.
Happy Konging! 🦍

Can I reopen this issue?

@Asilent you did not specify them as inidcated by @gszr .
This is the format:
export KONG_NGINX_MAIN_ENV=env_name

The name consists of 2 parts; KONG_NGINX_MAIN and ENV, and the value is env_name. The first part tells Kong where to inject the directive; in the Nginx main config file. The second one is the name of the directive ENV which causes Kong to inject it as name (lower-case) + value;
env env_name;

So you need to specify 2 environment variables for each variable you need

to tell Kong it should keep that variable around
the actual variable with its value

So you need to specify:
export KONG_NGINX_MAIN_ENV=HTTP_SSO_ENDPOINT
export HTTP_SSO_ENDPOINT=http://sso.my
export KONG_NGINX_MAIN_ENV=PROXY_SSO_ENDPOINT
export PROXY_SSO_ENDPOINT=http://sso.my

Now there is a problem, because the second KONG_NGINX_MAIN_ENV will overwrite the first... To get around that use a little hack, specify it like so (must be quoted because of the ;):
export "KONG_NGINX_MAIN_ENV=HTTP_SSO_ENDPOINT;env PROXY_SSO_ENDPOINT"
export HTTP_SSO_ENDPOINT=http://sso.my
export PROXY_SSO_ENDPOINT=http://sso.my

If you check the generated config file, you'll see that Kong injects this line:
env HTTP_SSO_ENDPOINT;env PROXY_SSO_ENDPOINT;

which is a valid nginx config, but now contains both variables. You should now be able to access those variables from your plugin code.

@Asilent you did not specify them as inidcated by @gszr .
This is the format:
export KONG_NGINX_MAIN_ENV=env_name

The name consists of 2 parts; KONG_NGINX_MAIN and ENV, and the value is env_name. The first part tells Kong where to inject the directive; in the Nginx main config file. The second one is the name of the directive ENV which causes Kong to inject it as name (lower-case) + value;
env env_name;

So you need to specify 2 environment variables for each variable you need

to tell Kong it should keep that variable around
the actual variable with its value

So you need to specify:
export KONG_NGINX_MAIN_ENV=HTTP_SSO_ENDPOINT
export HTTP_SSO_ENDPOINT=http://sso.my
export KONG_NGINX_MAIN_ENV=PROXY_SSO_ENDPOINT
export PROXY_SSO_ENDPOINT=http://sso.my

Now there is a problem, because the second KONG_NGINX_MAIN_ENV will overwrite the first... To get around that use a little hack, specify it like so (must be quoted because of the ;):
export "KONG_NGINX_MAIN_ENV=HTTP_SSO_ENDPOINT;env PROXY_SSO_ENDPOINT"
export HTTP_SSO_ENDPOINT=http://sso.my
export PROXY_SSO_ENDPOINT=http://sso.my

If you check the generated config file, you'll see that Kong injects this line:
env HTTP_SSO_ENDPOINT;env PROXY_SSO_ENDPOINT;

which is a valid nginx config, but now contains both variables. You should now be able to access those variables from your plugin code.

if I run on k8s, how to write this env KONG_NGINX_MAIN_ENV?


Fails to deploy

Cloning this repo down (changing nothing at all) and following the guide doesn’t deploy for me. Have tested in both Sydney and Virginia (the default) regions.
Here’s the final lines of the deploy
Running: `/app/bin/hello_elixir eval HelloElixir.Release.migrate` as nobody
         2021/07/08 00:38:42 listening on [fdaa:0:2f24:a7b:ad6:f8da:1834:2]:22 (DNS: [fdaa::3]:53)
: not foundpp/bin/hello_elixir: /app/releases/0.1.0/env.sh: line 2:
         Main child exited normally with code: 127
         Starting clean up.
Error Release command failed, deployment aborted

Any thoughts on how to get this deployed, or whether I need to make changes to the repo before it’ll work?

I just updated this demo repo for Phoenix 1.6, esbuild, and the latest fly configs.


[DO NOT MERGE] Auto PR to update dependencies of pilot

This PR will be merged automatically once checks are successful.
none

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/pilot-presubmit.sh
5b9f03070552b1c0bcd01acab5e5121ca036c355
link
/test pilot-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


In CamerZoom, Run FOV and Zoom FOV don't run well simultaneously

If you run and zoom at the same time, or one within the other, the FOV changes don't play nice together and cause a jarring tween. A solution that is working well for me is to Stop each other's Coroutines, not just their own. So in CamerZoom.cs the methods would look like this (new lines outlined in red):

I fixed it but it's a temporary solution because it's not the nices but it works. Basically i made that you can zoom during running and zoom will stop once you start running


Tracking button not working as expected on freeCodeCamp forum

Description of the bug
The tracking button doesn't change to other options available.
To Reproduce
Steps to reproduce the behavior:

Go to the community forum : https://forum.freecodecamp.org/
Click on any topic from the home page.
Select the tracking button which is available at the right bottom corner.
Try changing to other options.
See an error, the button still says "tracking".

Expected behavior
The button is expected to change to the option which is selected. Eg: if we select "Watching", the button should show watching as text on it.
Screenshots

Desktop (system details):

OS: [Windows 10]
Browser: [Firefox]
Version: [79.0 (64-bit)]

Thank you, for taking the time to report this.
I can confirm that this is also happening with me. I will also add that the button does change the value, but does not display this change. It appears to be an issue with the state change not being saved, when the component is reloaded.
However, (and someone else can confirm), I think this is a bug that needs to be reported to Discourse (the forum platform provider), not freeCodeCamp.

However, (and someone else can confirm), I think this is a bug that needs to be reported to Discourse (the forum platform provider), not freeCodeCamp.

@Sky020 FCC should report it or at least check on their GitHub issues to see if it is an outstanding issue already.  Would you be up for the task?

I can confirm that this is also happening with me. I will also add that the button does change the value, but does not display this change. It appears to be an issue with the state change not being saved, when the component is reloaded.

Thank you @Sky020  for putting down it appropriately.
I'm happy to be of some help to the FCC.

Would you be up for the task?
@RandellDawson , Sure, I will look into it.

I am almost certain this is the broken theme. You can confirm by switching to the default Discourse theme available for debuging.
/cc @ahmadabdolsaheb

Yep, confirmed myself.

@RandellDawson , A Discourse member says that this is a modification, and not part of the default Discourse: https://meta.discourse.org/t/freecodecamp-topic-timeline-tracker-button-bug/159804/2
What do we do in this case?

@Sky020 I did not think about the theme being the issue.  @raisedadead How were you able to guess this was the problem all along?

I tested it with the default Discourse theme.

I use this for my needs because the freeCodeCamp themes are broken for most admin areas, etc.

a lot of the icons were replaced with words one of which was the bell icon.
The word "tracking" should be replaced with "notification" so it would not be confused with "tracking" which is also a notification option.  let me if this would resolve this issue so I move forward with a pr.

Reopening.

Issue 1: It's not mapped correctly at all places.
Custom theme:

Default:

Issue 2: Its lowercased, which is not consistent with other Labels.

Issue 3: It does not solve the OP's issue, that is the button's label does not change to the selected option of the user and they have to open the drop down to see their choice.

Thanks for noticing that. The above-mentioned pr that completely removes the words in favor of icons should do the trick.


Query auto completion

Having in mind recent feedback from redit and feedback I'm getting from others, this should have it's own issue.
It would be a major enhancement for the app if query box would have auto completion (existing tables, columns, etc.). I haven't checked if ace editor comes with API which could be used for this feature (if it does not there are alternatives like CodeMirror).

Yeah. That would be an awesome feature. I don't have much time to implement it right now. But if you are  interested in implementing it. Please do. Let me know if you need help in anything.
By the way. Looks like ace has support for auto complete - https://github.com/ajaxorg/ace/wiki/How-to-enable-Autocomplete-in-the-Ace-editor.

Autocomplete would be amazing!

Although issue was resolved with this commit, I'll leave this issue open for further discussion on autocomplete enhancement.
As mentioned in  #128 , both basic and live autocompletion should be optional (hence user should be able to enable and disable it). Furthermore, there's way to include tables and columns in autocompletion by writing custom completer, explained here.

The metadata is now loaded into the auto completion list. I guess is just missing apply the custom configurations to consider this task done. I can work on that tomorrow.

Completed with https://github.com/sqlectron/sqlectron-gui/commit/a50497c79ba6bfed1e856f4aea1fe73c132e48d2


Show diff in cell widget

Update cell widget to show diff in the suggestion panel

Looking good, nice work!


Profiles management at logon/logoff (low priority)

Feature request
I'm submitting a proposal to ...

[ ] new feature
[X] change an existing feature

Describe your request
I'd like to submit 2 enhancements to the profile management.
At logout, profiles, profiles config, and all related data with the profiles are not deleted fr om the database => We could ask if the user wants to delete them and perform the cleanup.
At login, if I read the code well, every profiles not related to the current user are deleted, with the related data. We shouldn't if the user wants to keep them.
Describe an alternative
A warning in the readme about loosing data when connecting with an another profile, i.e. for some test. And encourage people to save their data before performing such operation.
Development tips
A bit tricky, as we may need to associate every profile with an account. And maybe it's not a good idea... Discussion opened.

I realized too late that the profile account association was missing
it would be interesting to add a multiple account management
but it would start to become difficult to manage everything, for something that might be used by one or two people...
i think that the simplest thing (as you said) is to ask to user what to do before logout (only first time)
and save the preference, so make it configurable in expert settings if an user want to change behaviour after the choice

Forgot that this cause a big problem with profiles
the profiles of other account will showed also in other accounts...

you could just do that for those who want only to logout, so then re-login with same account (eg. only because a person doesn't want others to enter... and not to use netflix addon with multiple different account)

Am I correct if I say "the profile list is built at every addon start (for any route)"?
If yes, then it means the only data we may want to keep are the ones in the shared database for stream continuity.
Is there any other data for a profile we may keep?

not all routes, and not at every addon py execution,
updating profile data occurs first start and when the login has expired
at least that should work...
currently stream_continuity is in table relation with profiles for delete/update actions
yes, it is the only user data that is automatically deleted

at the end all profile data is restored at login
no other data is deleted,
We do not save anything in the profile tables, it is only for convenience of retrieving data

i close this issue, the loss of stream continuity data at logout it's not a big loss
i can however add a warning message


Update Composer dependencies (2021-09-26-00-09)

Visual regression test passed!
View the visual regression test report


Model dependency causing failed CF updates

Imagine Mode2 references Model1 using the $ref keyword, and both models are defined in the models section of the serverless.yml file:
Model2:
{
    "$schema": "http://json-schema.org/draft-04/schema#",
    "title": "Model2",
    "type": "array",
    "items": {
        "$ref": "https://apigateway.amazonaws.com/restapis/API_ID/models/Model1"
    }
}

serverless.yml
custom:
  documentation:
    models:
      - name: "Model1"
        contentType: "application/json"
        schema: ${file(Model1.json)}
      - name: "Model2"
        contentType: "application/json"
        schema: ${file(Model2.json)}

This will cause a cloudformation update failure when Model2 is being applied earlier than Model1. This is due to the CloudFormation update template not including any dependency between the models. With Model1 not yet existing, the canonical $ref URL will be invalid and AWS will reject Model2 and fail the stack update completely.
Essentially, there is a need to specify how models depend on each other, optionally similar to other resources using the "dependsOn" keyword. Or even better, detect the dependency automatically and inject the "dependsOn" in the CF template correctly.
Can someone please confirm if this is a valid issue? I'd be happy to make a PR myself with some guidance.

I'm experiencing the same issue here. Any response from anyone related to the project?


System.ArgumentNullException: Value cannot be null.

Hello, we are using FFImageLoading in a PCL Project and sometimes the FFImageLoading throw this exception: "System.ArgumentNullException: Value cannot be null"
this is the only stack that the problem gives to us....
<forms:CachedImage HorizontalOptions="Start"
                                   VerticalOptions="Center"
                                   WidthRequest="50"
                                   HeightRequest="50"
                                   DownsampleWidth="50"
                                   DownsampleHeight="50"
                                   DownsampleToViewSize = "true"
                                   Source="{Binding MyImage">

 <forms:CachedImage.Transformations>
     <transformations:CircleTransformation />
  </forms:CachedImage.Transformations>
</forms:CachedImage>

what we saw here is that when we are not using CircleTransformation the error is seems to not appear again...
thanks

Somehow, memory cache entries with null cache key are being added. Could you run it with verbose logging to debug when exactly it occurs (for which urls/images)? Something like that:
		protected override void OnCreate(Bundle bundle)
		{
			base.OnCreate(bundle);
			FFImageLoading.Forms.Droid.CachedImageRenderer.Init();

			var config = new FFImageLoading.Config.Configuration()
			{
				VerboseLogging = true,
				VerbosePerformanceLogging = false,
				VerboseMemoryCacheLogging = false,
				VerboseLoadingCancelledLogging = false,
				Logger = new CustomLogger(),
			};
			ImageService.Instance.Initialize(config);

			global::Xamarin.Forms.Forms.Init(this, bundle);
			LoadApplication(new App());
		}

		public class CustomLogger : FFImageLoading.Helpers.IMiniLogger
		{
			public void Debug(string message)
			{
				Console.WriteLine(message);
			}

			public void Error(string errorMessage)
			{
				Console.WriteLine(errorMessage);
			}

			public void Error(string errorMessage, Exception ex)
			{
				Error(errorMessage + System.Environment.NewLine + ex.ToString());
			}
		}

We have attached the log error.
When this error occurs, the image shows, but it isn't a circle.
The transformation has not been applied.
log.txt

Thanks! It should be fixed with this: https://github.com/luberda-molinet/FFImageLoading/commit/073f4772ab3c4c71d2b2473691bfaff9a7149dcb

Thanks a lot.
Any ideias about when will be a release or a prerelease with the fix?

FFImageLoading 2.1.8-pre-146 nuget which contains discussed fixes released.

The exception is no longer happening.
But a single image on the ListView isn't a circle.
The image with the error changes each time I start the app.

Is it always the same image?
What is the source? Is it a local file / url?
The important thing is to set Transformations property before Source property.

No, it isn`t the same image.
The source is a local ImageSource
This is how I use the component on a ViewCell

                  <forms:CachedImage HorizontalOptions="Start"
                                     VerticalOptions="Center"
                                     WidthRequest="50"
                                     HeightRequest="50"
                                     DownsampleWidth="50"
                                     DownsampleHeight="50"
                                     DownsampleToViewSize = "true"
                                     Source="{Binding Imagem}, Mode=OneWay}">

                    <forms:CachedImage.Transformations>
                      <transformations:CircleTransformation />
                    </forms:CachedImage.Transformations>

                  </forms:CachedImage>

Source="{Binding Imagem}, Mode=OneWay} should be Source="{Binding Imagem, Mode=OneWay}

Yes, is coded that way.
Sorry, I was testing something else.

                  <forms:CachedImage HorizontalOptions="Start"
                                     VerticalOptions="Center"
                                     WidthRequest="50"
                                     HeightRequest="50"
                                     DownsampleWidth="50"
                                     DownsampleHeight="50"
                                     DownsampleToViewSize = "true"
                                     Source="{Binding Imagem, Mode=OneWay}">

                    <forms:CachedImage.Transformations>
                      <transformations:CircleTransformation />
                    </forms:CachedImage.Transformations>

                  </forms:CachedImage>

Could you try this:
<forms:CachedImage HorizontalOptions="Start"
				 VerticalOptions="Center"
				 WidthRequest="50"
				 HeightRequest="50"
				 DownsampleWidth="50"
				 DownsampleHeight="50"
				 DownsampleToViewSize = "true">

	<forms:CachedImage.Transformations>
	  <transformations:CircleTransformation />
	</forms:CachedImage.Transformations>

	<forms:CachedImage.Source>
		<Binding Path="Imagem"/>
	</forms:CachedImage.Source>

</forms:CachedImage>

That worked, I can no longer simulate behavior.
Thank you.

I fixed this with: https://github.com/luberda-molinet/FFImageLoading/commit/272c51f994803598673cbf72f7fb8abba4726bd4. It will prevent such behavior in future.

2.1.8-pre-150 released. It will allow your previous code to work correctly without any modifications.


[Improvement] Add the out direcotory in the project to the git ignore file

What would you like to be improved?
After we build Gravitino, there will be some out directory under the project folder, please see:

How should we improve?
Add them to the .gitignore file or avoid generating such files.

hello, I want to solve this issue. Please assign it to me, thanks!

@four1er do you still want to work on this?

@justinmclean If possible, can I try it?

It seems that we can't reproduce the issue in the main branch.

Closing as we can't reproduce, please ask to reopen if you come across the issue again.


Installer

Make a setup installed for a one-click install experience, perhaps with Rust?

Implemented as of 0.6.0!


Force MathJax content to be LTR

When in RTL mode and writing mathjax, it's practically impossible to correctly edit the mathjax parts because all the special symbols are treated as RTL while the english characters are LTR.
A quick path for this is to force all .math objects to be aligned to the left. Just add this css:
.math { direction:ltr; unicode-bidi:bidi-override; }

It's not perfect, but as soon as you finish typing the closing $ signs, the entire text between them will be correctly aligned. This can be used also to partly solve https://github.com/benweet/stackedit/issues/773 with code blocks.
I currently add this myself using a client script, but I hope to see this in main :)

How exactly do you add what you said to the css? Is it in the Default Template section in the advanced settings?

I used a browser extension (like Tampermonkey in Chrome) to run custom javascript that adds that css. Here's the script:
function addGlobalStyle(css) {
    var head, style;
    head = document.getElementsByTagName('head')[0];
    if (!head) { return; }
    style = document.createElement('style');
    style.type = 'text/css';
    style.id = "tapermonkey_css";
    style.innerHTML = css;
    head.appendChild(style);
}

addGlobalStyle('.math { direction:ltr; unicode-bidi:bidi-override; }');

I set it to run on stackedit's domain

Quite impressive, I'll be happy to here your comment on my comment at #773.


prometheus label shortcomings using default config

(This is from a Slack discussion on Thursday 5/25)

The mixer stores the HTTP Path in a Prometheus label called "method".  The HTTP Method should appear in "method".  The path should appear in a different label (such as "path").
The mixer configuration causes FQDNs to be stored in target but hostnames to be stored in source.  It would be best if the default configuration stored source and target similarly.
The "source" field for traffic that reaches a microservice via the ingress should have the ingress name as the source.
Traffic from the outside into the ingress should also appear in Prometheus.
The default configuration records the target pod's version label but not the source pods.  Both the source and target should be recorded by the default configuration.

We've had several back-and-forth about the content of each label in the metrics and about what we can properly extract and put in various labels and what should be exposed in general for default metrics.
I'm going to rename this issue to "Review default metrics configuration" to highlight the larger conversation and drive consensus on approach. I will separate out the ingress issues, as I believe that is distinct from content of metrics (mixer is not currently integrated with ingress).
A couple of starting points:

For input, we get the request headers and source and target UIDs (example: kubernetes://pod.namespace).
Right now, we use the kubernetes adapter to generate the following fields:

pod ip
pod labels
pod name
pod namespace
fqdn for service (constructed, in part, from a configurable label on the pod)
service account name for the pod

We currently construct metrics labels as follows:
source: source.labels["app"] | "unknown"
target: target.service | "unknown"
service: target.labels["app"] | "unknown"
method: request.path | "unknown"
version: target.labels["version"] | "unknown"
response_code: response.code | 200

At the very least, it seems we could replace method and version with something like:
request_path: request.path | "unknown"
target_version: target.labels["version"] | "unknown"
source_version: source.labels["version"] | "unknown"

It is unclear if it is useful to have a fqdn for target (and/or for source).

Moved conversation re: ingress to #649

Is this issue still relevant? Any new info? Should this be moved to 0.3?

It is still relevant, but can be done in separately from everything else. Like a p2.

I'm going to close this. Issues 2-5 have been addressed. Issue 1 has been partially addressed (removing method). API Management support will complete Issue 1.


Suggestion for a AggregateDocument function

Hi Zach,
Do you have plans to create a AggregateDocument function?
It would be really cool if I could do things like:
AggregateDocument[ myColletion
        ,{"$project" ->  {"author" -> 1}}
        ,{"$group" -> {"_id" -> "$author", "count" -> {"$sum" -> 1}}}
        ,{"$sort" -> {"count" -> -1}}
        ,{"$limit" -> 5}
]

tks
Murta

Hadn't gotten that far, but yes, thanks for the suggestion. I'll play with some designs soon.
(If you need that functionality immediately, you can access the the underlying java object for the collection with collection["collection"] and use jlink to call the aggregation commands that you need.)
Semi-related, I keep forgetting to add Sort options to all the query functions (just made #7 for that).

Hello, I am trying to test your code but it's appear this problem every time I use functions like "DatabaseNames[]", "FindDocuments[collection]", etc.:
Java::excptn: A Java exception occurred: java.lang.NoSuchMethodError: org.slf4j.Logger.isTraceEnabled()Z
at com.mongodb.diagnostics.logging.SLF4JLogger.isTraceEnabled(SLF4JLogger.java:36)
at com.mongodb.connection.DefaultConnectionPool.getPooledConnection(DefaultConnectionPool.java:258)
at com.mongodb.connection.DefaultConnectionPool.get(DefaultConnectionPool.java:93)
at com.mongodb.connection.DefaultConnectionPool.get(DefaultConnectionPool.java:82)
at com.mongodb.connection.DefaultServer.getConnection(DefaultServer.java:72)
at com.mongodb.binding.ClusterBinding$ClusterBindingConnectionSource.getConnection(ClusterBinding.java:86)
at com.mongodb.operation.OperationHelper.withConnectionSource(OperationHelper.java:237)
at com.mongodb.operation.OperationHelper.withConnection(OperationHelper.java:212)
at com.mongodb.operation.ListDatabasesOperation.execute(ListDatabasesOperation.java:101)
at com.mongodb.operation.ListDatabasesOperation.execute(ListDatabasesOperation.java:53)
at com.mongodb.Mongo.execute(Mongo.java:772)
at com.mongodb.Mongo$2.execute(Mongo.java:759)
at com.mongodb.OperationIterable.iterator(OperationIterable.java:47)
at com.mongodb.OperationIterable.forEach(OperationIterable.java:70)
at com.mongodb.MappingIterable.forEach(MappingIterable.java:50)
at com.mongodb.MappingIterable.into(MappingIterable.java:60)
at com.mongodb.Mongo.getDatabaseNames(Mongo.java:436).
Do you know which is the problem? How can I solve the problem?
Thank you!!

@julio-pope can you move that to a new issue please? I don't think it's related to this topic.

(I will move this answer when you move the question, but for now--)
I think that might be because you have an old SLF4J (<1.2.12) on your java classpath. Can you post your JavaClassPath[] please? MongoDB will only use SLF4J (logging) if the class is present, so the solution would be to either upgrade to a newer version or remove it from your classpath so MongoDB doesn't try to load it in the first place.

Feedback from Mike H on syntax:
Change the OP example to a list of stages:
AggregateDocuments[ myColletion,{
        ,{"$project" ->  {"author" -> 1}}
        ,{"$group" -> {"_id" -> "$author", "count" -> {"$sum" -> 1}}}
        ,{"$sort" -> {"count" -> -1}}
        ,{"$limit" -> 5}
        }
]

or more generally AggregateDocuments[ myColletion, myPipeline], where myPipeline is a list containing one or more pipeline steps which are lists. Each pipeline stage takes the form {$operator -> operand} where the operand is normally a list.


Test Time 14

Test Time 14

@amihaiemil this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2020-12-23T11:21:31.613979.
Estimation here is 128 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.

@MihaiDejeu please keep in mind that manual assignment of tickets is a bad practice and it is discouraged. Next time, please let me elect the assignee.

@amihaiemil thank you for resolving this ticket. I've just added it to your active invoice. You can always check all your invoices and more on the Contributor Dashboard.


Corrected typos and added DigitalOcean link

Please verify my changes and I hope they are useful.

Merge conflicts, please check what has already been merged


docs: Move integration docs to /api.

The main points of interest here are the image that needed to be moved, and the fact that images need to be embedded with html tags rather than markdown syntax (at least I couldn't find a way to do so).

Tests seem to be failing because chat.zulip.org does not run with this commit yet.

Yep, that's normal for this sort of big transition.  I'm going to try to integrate #7326 first, so this will likely need a rebase.

That was a nasty rebase. Added a commit for the outdated bots guide link.

I think we want the docs to link to zulipchat.com/api, not chat.zulip.org/api, since chat.zulip.org isn't a production server (i.e. it has significantly lower uptime than zulipchat.com).

Updated, and r/chat.zulip.org/zulipchat.com

Merged, thanks @derAnfaenger!  I'll deploy all the way out to zulipchat.com so that we confirm the links are all the links working again.
@rosatamsen FYI, since this is a big developer docs move.

@timabbott
Luckily there weren't any other files that needed to be updated other than tutorials/index.rst


How to Update the route on Users location change

Hi,
Is there any method or anyway by which we can update the route depending on the location of the user?
I am using the below code to set a route from the user's current location to a specific cordinates, but when the user changes location the route doesn't update.
Is there any way I can update the route when the user changes location.
The Code
await controller.enableTracking();
GeoPoint geoPoint = await controller.myLocation();

print (geoPoint.latitude);
print(geoPoint.longitude);

RoadInfo roadInfo = await controller.drawRoad(
GeoPoint(latitude: geoPoint.latitude, longitude: geoPoint.longitude),
GeoPoint(latitude: 24.913608364530518, longitude: 67.03123161285491),
interestPoints : [GeoPoint(latitude: 24.913608364530518, longitude: 67.03123161285491)],
   roadOption: RoadOption(
   roadWidth: 20.0,
   roadColor: Colors.red,
showMarkerOfPOI: true
 ),
);
   print("${roadInfo.distance}km");
   print("${roadInfo.duration}sec");

I've also tried using
await controller.enableTracking().then((value) => {
    GeoPoint geoPoint = await controller.myLocation();
      ...
    });

But it doesn't work.
Can someone help me out on this issue?

turn by turn navigation not implement yet it need  specific implementation (infrastruction + code )
drawRoad in the map is just static and we using open source routing server
you can change it manually every 1 min for example and adjust camera

Just out of curiosity, Do you plan to implement turn-by-turn navigation in the future, or it's not the scope of this project?

I already think about a lot but for next 3 months i will focus on ios and web support
but when i will find greate solution for it of course i will added but it will be as external library

Just out of curiosity, Do you plan to implement turn-by-turn navigation in the future, or it's not the scope of this project?

Even I am looking out for navigation support using OSM


Remove unused icon SVG files and React components

From the Slack discussion earlier:

jimmymow 6:09 PM
sup friends
i think the feather stuff was from a design contributor like two years ago, i don't think those are used anymore

mrfelton 6:11 PM
Probably a good maintenance chore to do a review of the icons that we have in the codebase and remove any unused ones. I suspect about 30% of them can go

There are a bunch SVG icons and corresponding React components under renderer/components/Icon/ many of which are unused. Probably a good idea to do some clean-up and remove the unused ones.

Closed via #2797


Add Bangladesh holidays

Hey there! Noticed most of holiday names are in their mother tongue – how about a sprinkle of Bengali magic to celebrate diversity?
https://github.com/spatie/holidays/blob/main/src/Countries/Belgium.php
https://github.com/spatie/holidays/blob/main/src/Countries/Brazil.php
https://github.com/spatie/holidays/blob/main/src/Countries/Hungary.php
'শহীদ দিবস' => '02-21', 'মুজিব জয়ন' => '03-17', 'Genocide Remembrance Day' => '03-25', 'স্বাধীনতা দিবস' => '03-26', 'ঈদুল ফিতর' => '04-10', 'পহেলা বৈশাখ' => '04-14', 'মে দিবস' => '05-01', 'বুদ্ধ পূর্ণিমা' => '05-19', 'ঈদুল আযহা' => '06-16', 'আশুরা' => '07-16', 'জাতীয় শোক দিবস' => '08-15', 'জন্মাষ্টমী' => '08-24', 'ঈদে মিলাদুন্নবী' => '09-15', 'দুর্গা পূজা' => '10-08', 'Armed Forces Day' => '11-21', 'Martyred Intellectuals Day' => '12-14', 'বিজয় দিবস' => '12-16', 'বড়দিন' => '12-25',

Dont think bangla would be a good choice here.
Also naming should match official or govt names. Ex 21 Feb.
Only fixed dates are added here.
So we cant add Eid holidays.

Dont think bangla would be a good choice here. Also naming should match official or govt names. Ex 21 Feb.
Only fixed dates are added here. So we cant add Eid holidays.

Got it!

Thanks!


nzTree: unable to disable multi-select

Version
1.6.0
Environment
Chrome: 69.0.3497.100
Reproduction link
https://stackblitz.com/edit/ng-zorro-antd-start-ybwqkh
Steps to reproduce
using nz-tree component, and I can not found any way to disable multiple selection, and nzMultiple  is not work for me.
What is expected?
choose only one tree node anyway
What is actually happening?
unable to disable multiple selection

@vthinkxie maybe I should not use checkbox for single selection.


Documentation for NA term

This term is used in valueCounts and statistical operations: std, mean, varianceAndMean
https://kotlin.github.io/dataframe/valuecounts.html
In documentation, there is no explanation of what it means. What's the difference from NaN?
In the following function, skipNA used when the value is NaN.
public fun Sequence<Float>.mean(skipNA: Boolean = skipNA_default): Double {
    var count = 0
    var sum: Double = 0.toDouble()
    for (element in this) {
        if (element.isNaN()) {
            if (skipNA) continue
            else return Double.NaN
        }
        sum += element
        count++
    }
    return if (count > 0) sum / count else Double.NaN
}

I mentioned it in the KDocs in https://github.com/Kotlin/dataframe/pull/214, but indeed, we need it on the documentation website too.


Request

Hi Nik,
I've been trying to get the below working with PageMenu, but I just can't seem to get both the navigation bar AND the subview below it to scroll - any ideas?
https://github.com/JoeFryer/JDFPeekaboo
Thanks,
Owen

Is the navigation bar translucent? If so the frame for page menu needs to be set appropriately

I haven't set it to be translucent - should I do so? Would I set the frame for pagemenu on the view controller that I will scrolling on, or the one that presents it?
Any examples would really help! :)


bring 2.0.0 dev up to current

creating this pull request as I'm not sure if 2.0.0 dev is protected

@andrew-welker 👍


Accuracy of AggregatingMergeTree Materialized View outside of primary key

Hello,
we have observed that the aggregate merge results (from the combination of -State and -Merge functions) in a materialized view are accurate only when grouping or filtering by fields that are contained within the primary key of the view. When filtering (either via -If or via where combination) the calculated numbers are vastly different (orders of magnitude) compared to the ones calculated on the original table.
Is this expected behavior? Unfortunately I do not have time at the moment to try and setup a toy example, but if someone is interested I would be able to show this on our data.

It sounds like a problem that should be investigated.
But, It isn't easy to understand the problem without an example.

The AggregatingMergeTree allows you to create MV with more fields than are in the primary key.
Like:
CREATE MATERIALIZED VIEW ... AggregatingMergeTree((date), (date, page))
AS SELECT date, page, src, countState() AS queryCount
FROM source
GROUP BY date, page, src

It will surprisingly work fine until the first merge when it will somehow miss most of the countState,
without throwing any errors. Maybe you made the same mistake I did and primary key is inconsistent with the grouping?

Hi Vavriza,
yes, this is the case for me as well. I am wondering whether this is a bug of the engine or a limitation that should be documented (and perhaps, if this is actually the criterion behind, a query on a MV that is known to return wrong results should throw an error beforehand?).

Looks like I have the same problem using SummingMergeTree.
@ludv1x, do you have any plans to change this behaviour or state it in the documentaion?

I think the docs should be updated to be explicit about requiring the all the key fields in the GROUP BY clause.  I ran into this when I was first using it too.

Following up here, in addition to the data being inconsistent in the result, it's also completely nonsensical and should throw an error.  Take below for example.  We're filtering on hour, but incorrectly were grouping by more than what's in our sorting key.  The first query correctly returns no results, however the subsequent query returns pretty much random data from the table/materialized view.
clickhouse-staging-0f6a61f82b85b6edf :) ...

SELECT
    sourceID,
    type,
    name,
    hour,
    sumMerge(countState) AS seenCountByEvent,
    sumMerge(enforcedEventsCountState) AS enforcedEventsCountByEvent
FROM default.hourly_events
WHERE (hour < '2019-04-29 00:00:00') AND (sourceID = '0xdeadbeef')
GROUP BY (sourceID, hour, type, name)

Ok.

0 rows in set. Elapsed: 0.002 sec.

clickhouse-staging-0f6a61f82b85b6edf :) ...

SELECT
    sourceID,
    type,
    name,
    hour,
    sumMerge(countState) AS seenCountByEvent,
    sumMerge(enforcedEventsCountState) AS enforcedEventsCountByEvent
FROM default.hourly_events
WHERE sourceID = '0xdeadbeef'
GROUP BY (sourceID, hour, type, name)

┌─sourceID───┬─type──┬─name────────────────────────┬────────────────hour─┬─seenCountByEvent─┬─enforcedEventsCountByEvent──┐
│ 0xdeadbeef │ page  │ null                        │ 2019-05-01 02:00:00 │             5099 │                        5099 │
│ 0xdeadbeef │ page  │ Lorem Ipsum                 │ 2019-05-01 01:00:00 │             7160 │                        7160 │
│ 0xdeadbeef │ track │ Dolor Sit                   │ 2019-05-01 03:00:00 │                1 │                           1 │
│ 0xdeadbeef │ track │ Consectetur Adipiscing      │ 2019-05-01 03:00:00 │             1247 │                        1247 │
│ 0xdeadbeef │ track │ Eiusmod Tempor              │ 2019-05-01 00:00:00 │             1995 │                        1995 │
└────────────┴───────┴─────────────────────────────┴─────────────────────┴──────────────────┴─────────────────────────────┘

5 rows in set. Elapsed: 0.004 sec. Processed 4.33 thousand rows, 393.95 KB (1.10 million rows/s., 100.27 MB/s.)

The code here suggests that it shouldn't be possible to GROUP BY in a select query where the fields in the group by aren't included in the original primary key for the materialized view:
https://github.com/yandex/ClickHouse/blob/83a35d7644cad823533c3a6459c8c777f9d10481/dbms/src/DataStreams/AggregatingSortedBlockInputStream.cpp#L31-L43

@ludv1x Are you going to do something about it? It's actually a problem. At least, we have to notice about this in the documentation about AggregatingMergeTree.

Same issue here. Resolved when GROUP BY clause contains same columns as view sorting key.


Update run-a-terraform-plan-with-the-terraform-apply-step.md

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://656f31e6a1a936008711ba96--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://656f3732cf151d0089def671--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://656feb6a4c4ba91a3b77bc80--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://656fed12777e271f145f9584--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://656fee167cdec4211df7e089--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://657053d1087e7161270c3a28--harness-developer.netlify.app


fix(gux-listbox): fix keyboard accessibility issue for list boxes wit…

Fixes an accessibility issue for listboxes that contain dividers. When identifying the next or previous option in the list to select on arrow down or up, gux-listbox was selecting the next element sibling. If the list contained a divider or heading, it selected that element, preventing further arrow navigation to the rest of the list items.
See gux-time-zone-picker for a working example.
This fix is somewhat similar to the solution in place for gux-list, in that it identifies a set of valid selectable tag names.
ENGAGEUI-7811

@daragh-king-genesys Thanks for approving and merging!  Should I also fix this in v4? This will be an issue there too.


Issue running Gekko after install.  Is this a node issue?

Note: this is the technical bug tracker, please use other platforms for getting support and starting a (non technical) discussion. See the getting help page for details.
I'm submitting a ...
[*] bug report
[ ] question about the decisions made in the repository
Action taken (what you did)
Hi all,
I am trying to run gekko on a windows 10 machine using the basic download and install options.  When I try to run the program with the command "node gekko --ui", I get the following.

I downloaded the dependencies correctly, so I think I might have downloaded the wrong version of node, or maybe I need to change something in the command prompt to run gekko correctly.  Please help!

is there another directory inside Downloads/gekko-develop called gekko-develop? If so run:
cd gekko-develop
node gekko --ui

Note: I just released a new version with a lot of upgrades, you might want to download that one :)


Link update: Github icon in footer

The footer on the map site has a Github icon that currently isn't linking to anything. The icon should link to:
https://github.com/opensupplyhub/pyoshub

CAN until Mariel confirms it's exhaustive.


Improve documentation

Improve documentation : Fix typo + Adding default values + Adding descriptions

Next version is under the beta branch, several command have been removed and there are changes in parameters. Can you refactor to that branch?

Ok. I have merge branch 'v3.0.0-Beta1' into my branch.
Everything seems good. No additional changes.

Thank you for the PR.


XCode 10 GM and Swift 4.2 support fails to compile

AnimatedCollectionViewLayout.swift:78:33: 'UICollectionViewScrollDirection' has been renamed to 'UICollectionView.ScrollDirection'

Xcode 10 with Swift 4.2 support is now released. Thanks for submitting the issue!

Works great, thank you!