tools: rewrite benchmark tool

rewrite the benchmark tool for v3 using cobra.

@gyuho Maybe you can also give this tool a try? set up a etcd3 cluster try this tool out?

@xiang90 Sure, I will give it a try.

Tried this locally, and confirmed that it works well. Thanks!
./benchmark --clients=1000 --endpoint '<IP_ADDRESS>:12378' range foo bar
10000 / 10000 Boooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo! 100.00 % 3s

Summary:
  Total:	3.6638 secs.
  Slowest:	0.4704 secs.
  Fastest:	0.1985 secs.
  Average:	0.3579 secs.
  Requests/sec:	2729.4435

Response time histogram:
  0.199 [1]	|
  0.226 [13]	|
  0.253 [441]	|∎∎∎∎∎∎∎
  0.280 [452]	|∎∎∎∎∎∎∎
  0.307 [1450]	|∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎
  0.334 [757]	|∎∎∎∎∎∎∎∎∎∎∎∎∎
  0.362 [2322]	|∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎
  0.389 [1165]	|∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎
  0.416 [1282]	|∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎
  0.443 [2002]	|∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎
  0.470 [115]	|∎

Latency distribution:
  10% in 0.2832 secs.
  25% in 0.3116 secs.
  50% in 0.3552 secs.
  75% in 0.4127 secs.
  90% in 0.4261 secs.
  95% in 0.4307 secs.
  99% in 0.4679 secs.

I am going to merge this in and add watcher bench. This is not perfect for sure (we might want to add more benchmark options for example), but I think this is a good start.


Fix continuous integration matrix

Python 3.6 is not available in ubuntu-latest (22.04).
This PR merely changes (ubuntu-latest, python-3.6) to (ubuntu-20.04, python-3.6) in the matrix strategy.

Proposal to add Python 3.9 to the matrix while we are at it.

Codecov Report
Base: 73.39% // Head: 73.39% // No change to project coverage :thumbsup:

Coverage data is based on head (ab22c9f) compared to base (0f25add).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #131   +/-   ##
=======================================
  Coverage   73.39%   73.39%
=======================================
  Files          11       11
  Lines        1075     1075
=======================================
  Hits          789      789
  Misses        286      286

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


SelectionRangeSlider Options List Not Updating

In some cases when changing the options of a SelectionRangeSlider, the options list is not updated. For this to occur, you must have both continuous_update set to True, and an index specified. I've seen similar behavior in react_ipywidgets 0.11 (ipywidgets 0.7) with the SelectMultiple widget. I believe it's some kind of race condition in the SelectionRangeSlider's JavaScript view, but I haven't been able to reproduce this issue with ipywidgets directly.
Test Case:
Click Update, and then try and adjust the slider range. The range remains 1-3, instead of updating to 1-10.
Set continuous_update to False, or remove the index, and the problem does not occur.
import reacton as react
import reacton.ipywidgets as w

@react.component()
def Test():
    options, setOptions = react.use_state(['1', '2', '3'])

    firstIndex = 0
    lastIndex = max(0, len(options)-1)

    def onClicked():
        setOptions([str(i) for i in range(1, 11)])
        print("what")

    print(options)

    with w.VBox() as vbox:
        w.Button(description="Update", on_click=onClicked)
        w.SelectionRangeSlider(
            description="Options",
            options=options,
            index=(firstIndex, lastIndex),
            continuous_update=True
        )

    return vbox

Test()

Software Versions:
python                          3.10.9
ipywidgets                    8.0.4
reacton                          1.2.2

I think this is an ipywidget bug.
import ipywidgets
options = ['1', '2', '3']
firstIndex, lastIndex = 1, 2

s = ipywidgets.SelectionRangeSlider(
            description="Options",
            options=options,
            index=(firstIndex, lastIndex),
            continuous_update=True
        )
def update(b):
    s.options = [str(k) for k in range(1,11)]
b = ipywidgets.Button(description="Update")
b.on_click(update)
display(b, s)

Since this behaves the same, do you agree?
Actually, Reacton can help you work around it, by using:
        w.SelectionRangeSlider(
            description="Options",
            options=options,
            index=(firstIndex, lastIndex),
            continuous_update=True
        ).key(repr(options))

Where the .key(..) function will make reacton not match this element to the widget with the previous options. It will create a new widget and destroy the old one. I guess there are better way that repr(options) though.

Yeah, seems likely to be a ipywidgets bug. That example does not reproduce the issue for me, but I suspect it's a race condition on the ipywidgets browser side. I was unable to find a case that reproduced it before now. I'll file a bug with ipywidgets and include your example and a reacton example. Hopefully that's enough for them to go on.
The .key() workaround works great! I use id(optoins) since I replace the options array each it changes. Thanks!

I think the issue is solved.


Fix physfsrwops_write

This bug report was migrated from our old Bugzilla tracker.
These attachments are available in the static archive:

Patch against tip (file_6398.txt, text/plain, 2015-07-30 20:34:32 -0400, 732 bytes)

Reported in version: all
Reported for operating system, platform: All, All
Comments on the original bug report:
On 2015-07-30 20:34:32 -0400, rettichschnidi wrote:

Created attachment 3544
Patch against tip
physfsrwops_write does not return what is specified in the documentation of SDL's read function in SDL_RWops.

On 2015-07-30 20:38:05 -0400, rettichschnidi wrote:

/read function/write function/

This got fixed at some point.


Referencing documents are not working if .md extension is used

🐛 Bug Report
When referencing other docs using .md extension, Docusaurus is failing to generate a proper link.
Have you read the Contributing Guidelines on issues?
Yes
To Reproduce
(Write your steps here:)

Follow the steps mentioned here: https://docusaurus.io/docs/en/navigation#referencing-site-documents
Create 2 documents (doc1.md, doc2.md)
Create a link in doc1.md ([link to document 2](doc2.md))

Expected behavior

The above steps should generates a proper link to doc2.html in both dev server and also when the site is built for publishing
Actual Behavior

The browser shows an error when clicking on the link in doc1.html
Cannot GET /test-site/docs/doc2.md

When building the site, docusaurus also throws following warnings:
[WARN] unresolved links in file 'doc1.md' > [ 'doc2.md' ]

Reproducible Demo
I will try to create a reproducible demo soon. But, the steps mentioned above are very simple.

This issue was also brought up before in #1137  but not sure why it was closed.

Ooops. Just found my mistake. I had a typo in the doc file name that was being linked to.
Closing it.

#1137 was fixed by #1138 and hence got closed. I believe this might be a regression.


Memory leak of GPU when start/stop Object Detection module many times

Hello, I can not re-open this issue. Then I create a new one. This problem still happens
#340
Thanks

Hi, the leak origin was identified and fixed.
It will be released in the upcoming ZED SDK 3.5.0, thanks for your patience.


Add CLI tests, return codes, version

Changes

[x] adds --version to top-level
[x] import and hoists __version__ to __all__
[x] returns non-zero error codes on failure
[x] prints errors to stderr
[x] adds pytest-console-scripts (and pytest-asyncio) configuration

[x] with some minimum versions, also pytest >=6.1

[x] don't try to use emoji on windows
[x] tests the cli
[x] fixes coverage uploading https://app.codecov.io/gh/jupyter/jupyter_events/pull/30

@blink1073 @Zsailer Welp, aside from the label thing, this looks ready to go. Is there a meeseeks command that would add one? Or PR templates?

This is fantastic. Thank you for this amazing work, @bollwyvl!


Always open the prometheus port on the pod if prometheus is enabled

Previously the port would not be opened if in container configuration was used.  Thus, if you used custom configuration then prometheus would not be able to scrape the metrics port.
Now we will always open the port if prometheus is enabled, and only mount volumes if custom configuration is not used.

@googlebot I signed it!


Catch Error when render SlotMachine not load image

I cannot rerender or catch error from SlotMachine ("@lucky-canvas/react": "^0.1.13") although img correct link but
// error: prizes[i].imgs[i] =>  'my-correct-link-image' 图片加载失败', i try onFinally or onError not working for me.!

That onFinally can only be used to capture errors in the component itself
It cannot be used to capture lucky-canvas errors

You need to describe what you specifically want to do

i want catch error when "error: prizes[i].imgs[i] => 'my-correct-link-image' 图片加载失败' ", onError not working for me :(

If you see this
error: prizes[i].imgs[i] => 'my-correct-link-image' 图片加载失败'
That means there is a problem with your image path, or it is an unresolved image. This is an internal error thrown by me because you must solve it to run the lucky-canvas component correctly

i sure this link correct, but i get error above (when i click link from error, it still exaclty) ;(

Send me the link and I'll take a look

this error happens sometimes, i cannot send link because this link private and had to account ;(

That could be because this link cannot be accessed by my component
The internal rendering of my component is like this
const img new window.Image(src)

Thank you, i try reproduce and report it later 👍

hi @buuing, i has been reproduce it with specific device:

device info:

iPhone 14 Pro Max
iOS 16.6.1

https://codesandbox.io/s/mutable-currying-6n6ht5

iPhone 14 Pro Max not render

on web and iPhone 11, android working


How to fix error Cannot query field "path" on type "GithubContributors"

❔ Question
After running npm run dev, I have always the error
error There was an error in your GraphQL query:
Cannot query field "path" on type "GithubContributors".

Can you please suggest what is wrong, and what am I missing ?
🔦 Context
I'm trying to build a website for my company, using the gatsby theme based on Adobe Spectrum.
🌍 Your Environment
Windows x64, NodeJS v14.15.4, npm 6.14.10
.env file content
REPO_GITHUB_TOKEN=xxxxxxxxxxx
REPO_OWNER=AdobeDocs
REPO_NAME=dev-site-documentation-template
REPO_BRANCH=main

🧢 Your Company/Team
EMEXAL

It could be because your repository and some md files in that repository are not pushed on GitHub or REPO_OWNER and REPO_NAME don't match your GitHub organization and repository name.

It could be because your repository and some md files in that repository are not pushed on GitHub or REPO_OWNER and REPO_NAME don't match your GitHub organization and repository name.

Actually, nothing was pushed on GitHub. I've pushed everything now, but I can still see that the github token has never been used.

Actually, nothing was pushed on GitHub. I've pushed everything now, but I can still see that the github token has never been used.

Actually, nothing was pushed on GitHub. I've pushed everything now, but I can still see that the github token has never been used.

Actually, nothing was pushed on GitHub. I've pushed everything now, but I can still see that the github token has never been used.

I'm wondering if the .env file is in the right place. I have it in the same directory as package.json. @icaraps do you think this is OK ?

I'm wondering if the .env file is in the right place. I have it in the same directory as package.json. @icaraps do you think this is OK ?

@emexal-company Yes that's fine. I don't know the details of your use case but you only need env for local development and if you're building a doc site like https://github.com/AdobeDocs/dev-site-documentation-template.

@emexal-company Yes that's fine. I don't know the details of your use case but you only need env for local development and if you're building a doc site like https://github.com/AdobeDocs/dev-site-documentation-template.

My original requirement is to create a website using Gatsby + Spectrum. I have followed the steps in the README, but nothing shows up (apart the error mentioned here), when I wrote npm run dev. Remark: I'm a beginnger in Gatsby, and it will be my first website using this framework.
I thought it was easy to startup with the README, but I'm really struggling from a few days...
I'm performing some debugging. githubFetch from gql.js is never called... So I still don't understand how the github API will be called. I can see also that the writen values in my .env file are correctly visible in the variables process.env.REPO_, when placing a breakpoint inside gatsby-config.js

My original requirement is to create a website using Gatsby + Spectrum. I have followed the steps in the README, but nothing shows up (apart the error mentioned here), when I wrote npm run dev. Remark: I'm a beginnger in Gatsby, and it will be my first website using this framework.
I thought it was easy to startup with the README, but I'm really struggling from a few days...
I'm performing some debugging. githubFetch from gql.js is never called... So I still don't understand how the github API will be called. I can see also that the writen values in my .env file are correctly visible in the variables process.env.REPO_, when placing a breakpoint inside gatsby-config.js

@emexal-company to get started quickly I recommend to use one the templates eg https://github.com/adobedocs/dev-site-product-template/. On that page there's a green button "Use this template". You can click on it and it will create the repository for you. Then run the commands npm i && npm run dev.

@emexal-company to get started quickly I recommend to use one the templates eg https://github.com/adobedocs/dev-site-product-template/. On that page there's a green button "Use this template". You can click on it and it will create the repository for you. Then run the commands npm i && npm run dev.

I tried like you said, but I got exactely the same error.

I tried like you said, but I got exactely the same error.

OK strange could you please share a repository url to reproduce the issue ? @shazron do you think it could be related to Windows x64 ?

OK strange could you please share a repository url to reproduce the issue ? @shazron do you think it could be related to Windows x64 ?

The repository I'm using is the following https://github.com/emexal-company/spectrum-gatsby

The repository I'm using is the following https://github.com/emexal-company/spectrum-gatsby

Steps:
git clone https://github.com/adobedocs/dev-site-product-template/
cd dev-site-product-template
npm i
npm run dev

I can repro on Windows 10 x64 (in a VM). I cannot repro on macOS 11.0.1 Big Sur.

Steps:
git clone https://github.com/adobedocs/dev-site-product-template/
cd dev-site-product-template
npm i
npm run dev

I can repro on Windows 10 x64 (in a VM). I cannot repro on macOS 11.0.1 Big Sur.

Thanks @shazron for confirming there's an issue when using Windows. I've submitted a fix can you please review and merge if all fine https://github.com/adobe/gatsby-source-github-file-contributors/pull/8 ?

Thanks @shazron for confirming there's an issue when using Windows. I've submitted a fix can you please review and merge if all fine https://github.com/adobe/gatsby-source-github-file-contributors/pull/8 ?

Reviewed and merged. Published https://github.com/adobe/gatsby-source-github-file-contributors/releases/tag/0.3.1

Reviewed and merged. Published https://github.com/adobe/gatsby-source-github-file-contributors/releases/tag/0.3.1

From my side, I tested it, and it fixed the problem. Isn't it better also to update the dependency inside package.json ?
https://github.com/adobe/gatsby-theme-aio/blob/main/packages/gatsby-theme-aio/package.json

From my side, I tested it, and it fixed the problem. Isn't it better also to update the dependency inside package.json ?
https://github.com/adobe/gatsby-theme-aio/blob/main/packages/gatsby-theme-aio/package.json

Thanks @shazron
@emexal-company yes<EMAIL_ADDRESS>was released with the updated dependency.

Thanks @shazron
@emexal-company yes<EMAIL_ADDRESS>was released with the updated dependency.


feat(ingest): hide cartesian product warnings in GE profiler

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

@hsheth2 there is a linter error in the pr


Fix issue 1465: fix jit in example

fixes #1465

An alternative approach is to update the check in jit function to:
if static_argnums and max(static_argnums) >= len(args) + len(fun.__defaults__):

The downside is fun.__defaults__ is type level and does not consider whether the default argument has been supplied or not. This may let the actually wrong static_argnums pass the check, eg. consider this function:
@jit(static_nums=(0,1,2) # wrong
def f(a,b=2):
  pass

An invocation of f(1,2) will pass this check since 2 < 2 + 1, although one could argue nothing would break (the dyn_argnums compute won't be impacted)

@mattjj @shoyer Can you take a look please? Am curious your thoughts on we could handle static_argnums check for default arguments.

I changed this PR to use a wrapper function to avoid applying static_argnums to a keyword argument. This should make everything work until we add keyword static argument support.


fix(cli): fix missing executable permissions for index.js

Fixes sh: 1: code-pushup: Permission denied errors when executing Code PushUp binary on Linux :crossed_fingers:

The compare command fails because it's being run in main - not very practical, but I'll replace it all with the GitHub Action soon.


update the sample user-data for cloud

Signed-off-by: David Cassany<EMAIL_ADDRESS>

Updates docs along with rancher-sandbox/cOS-toolkit#424
I noticed this sample user-data.yaml file is copied three times in the same page of docs. I am wondering if there is a way to embed a remote yaml file into the page. So it all gets synched with images/sampler-user-data.yaml from the main source repository. Any idea?

yup, with https://gohugo.io/templates/partials/ we should be able to include the same piece across different pages

oh, a remote one, didnt read that :D

oh, a remote one, didnt read that :D

well, having it duplicated only once would be already an enhancement ;)

AFAIK there is only getJson or getCSV, not getYAML -> https://gohugo.io/templates/data-templates/#get-remote-data

Updates docs along with rancher-sandbox/cOS-toolkit#424
I noticed this sample user-data.yaml file is copied three times in the same page of docs. I am wondering if there is a way to embed a remote yaml file into the page. So it all gets synched with images/sampler-user-data.yaml from the main source repository. Any idea?

Oh yes there is, I remember I crafted a special shortcode on the KubeCF docs for that purpose, I'll backport it here

Opened https://github.com/rancher-sandbox/cos-toolkit-docs/pull/9 to add the githubembed shortcode :+1:


Enhance vault UI: add role_id and secret_id

Now the Infrabox only support get secrets with token. But token always have a short-life. The better way is to use role_id and secret_id, which is no longer expired. This PR is for adding an entrance to configure role_id and secert_id in UI.

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

closed this PR as it has been merged to #500


Suggestion: Inform People About Roles In Discord

Add a #roles channel in the discord server and tell people about roles and how to get the roles. Because newcomers and people not knowing the feature might not have put on the roles. Maybe add a section to #readme about it

Rather than adding a roles channel, I think we can use the existing #bot-chat channel for the commands. I've added a post in #readme about how to get roles.

🟩 for someone:
This markdown can be added to the bottom of the discord-readme.md file:
**Roles**

Our server has a number of roles you can assign to yourself, such as roles for your specific tech stack or roles that we ping for certain events.

To see the roles that are available, head over to #bot-chat and use the command `becca!role`. Becca will send a list of the roles you can assign to yourself.

To assign a role, use `becca!role <role name>` and Becca will give you that role. For example, if I wanted the `angular` role, I would send the command `becca!role angular`.

Can I do this?

Go for it!

Submitted a PR @nhcarrigan #1659


Require pyedflib v0.1.25 for tests

resolves #40
The underlying issue was fixed in v0.1.24 but that version required NumPy>=1.22 (see holgern/pyedflib#164), so we should require v0.1.25 instead.

Thanks @hofaflo!


Please pre-compiled before publish

"get-value" is great. I'm using<EMAIL_ADDRESS>but it not pre-compiled to es5 that cause break in older browser and uglify-webpack-plugin, can you fix it, pre-compiled es6 to es5 before publish. thx a lot.
https://github.com/facebook/create-react-app/blob/master/packages/react-scripts/template/README.md#npm-run-build-fails-to-minify

You should set up your bundler to precompile this package if you need to target older browsers. Also, the only es6 type of feature in this package is const or let. You could also try finding out if the webpack plugin supports using a newer version of uglify (there's one that will work with es6).

@doowb Thanks. I get it.


Toggle nested ListItems also when clicked (not just when tapped on the right icon toggle)

I think nested list items shouldn't be only collapsed/expanded when tapping on the right icon toggle. Should also happen when I click on the list item itself. That would result into a better user experience I think.

+1

Is there any update on a nice way to achieve this functionality ?


Ubiq Tokens Support?

Ubiq (UBQ) token should already be visible in coins table:
Example:

Price example:

You can also try increasing number of max pages if it's a low market cap coin:
$ cointop  --max-pages 50

I was refering to https://shinobi-info.ubiq.ninja/tokens

Ah got it. Do you know if CoinGecko supports them? If not, I think in the future we can add support for custom graphnode queries so you can plug in any Ethereum token list, but it's not an easy task to do right now

CoinGecko doesn't seem to support it but I ask the coin maintainers to submit it


Change the arguments of syncDiff to the same as Sequelize constructor

Using url affects compatibility as older versions of postgresql does not support.
Instead, passing config object should be more compatibility to other nodejs db clients, thus maybe helpful when dbdiff plan to support other database later on.
P.S. Sometimes is troublesome when database user password contain special characters, e.g. '@', '/'...

Yeah, good idea. But there are two things I would like to discuss:
With the tools that I'm used to use I use a lot URLs so I would like to keep compatibility also with URLs. Can you make Sequelize.prototype.syncDiff accept both (url, options) and (database, username, password, options)?
Also you removed the self.sync() to the master database. Any particular reason to do that? I think that the default sync() without force: true is safe.
Thanks for your PR!

I checked Sequelize constructor source code. I found that it can handle the conversion between URLs and connection config object.
Therefore, just get config object from sequelize instance instead of passing-in argument should do the job.
About removing self.sync(), my intention is to allow the library to production all table creation sql query instead of queries responsible for unsynchronized part only. We can manually sync other master  database first if we want to get the latter result.
P.S. I am currently working on a project where we need a auto migration system (generating sequelize migration compatible js file). We want to avoid the use of sequelize.sync() because It magically done "ONLY PARTS" of the things. It is hard for my team to keep track on the changes.

The test doesn't pass.
When I said I wanted to keep support for URLs I meant this:
// old method
Sequelize.prototype.syncDiff = function(url, options) {
// new method
Sequelize.prototype.syncDiff = function(database, username, password, options) {

Both ways should be supported. Probably checking the number of arguments passed to the method.
if (arguments.length === 2) // use the url
else // use database, username, password and options

sequelize is currently doing this:
`var Sequelize = function(database, username, password, options) {
var urlParts;
options = options || {};
if (arguments.length === 1 || (arguments.length === 2 && typeof username === 'object')) {
options = username || {};
urlParts = url.parse(arguments[0]);
// reset username and password to null so we don't pass the options as the username
username = null;
password = null;
// SQLite don't have DB in connection url
if (urlParts.pathname) {
  database = urlParts.pathname.replace(/^\//, '');
}

options.dialect = urlParts.protocol.replace(/:$/, '');
options.host = urlParts.hostname;

if (urlParts.port) {
  options.port = urlParts.port;
}

if (urlParts.auth) {
  username = urlParts.auth.split(':')[0];
  password = urlParts.auth.split(':')[1];
}

}`
So I think we can pass either url or config object to sequelize constructor first and get config object for db client from sequelize.config. I haven't tested it with URL though.

Yeah, in order to accept your changes syncDiff(url, options) should work. There is a test for that and right now is not passing.


AppSourceCop throwing exceptions in app.json

Describe the bug
AppSourceCop are throwing a lot of NullReferenceExceptions in app.json (41 exceptions for my current project)
They all seems to come from two rules:

'Microsoft.Dynamics.Nav.AppSourceCop.Upgrade.RuleDestructiveTablePropertyChangesNotAllowed'
'Microsoft.Dynamics.Nav.AppSourceCop.Upgrade.RuleDestructiveTableFieldPropertyChangesNotAllowed'

Exceptions:
{
	"resource": "/e:/xxx/App/app.json",
	"owner": "_generated_diagnostic_collection_name_#0",
	"code": "AD0001",
	"severity": 4,
	"message": "Analyzer 'Microsoft.Dynamics.Nav.AppSourceCop.Upgrade.RuleDestructiveTableFieldPropertyChangesNotAllowed' threw an exception of type 'System.NullReferenceException' with message 'System.NullReferenceException: Object reference not set to an instance of an object.\r\n   at Microsoft.Dynamics.Nav.AppSourceCop.Helper.UpgradeVerificationHelper.GetTableTypeKindFromTableApplicationTypeSymbol(ISymbol tableApplicationObjectTypeSymbol) in F:\\a\\586\\s\\source\\Prod\\Microsoft.Dynamics.Nav.AppSourceCop\\Helper\\UpgradeVerificationHelper.cs:line 64\r\n   at Microsoft.Dynamics.Nav.AppSourceCop.Upgrade.RuleDestructiveTableFieldPropertyChangesNotAllowed.AnalyzeTableExtension(SymbolAnalysisContext ctx) in F:\\a\\586\\s\\source\\Prod\\Microsoft.Dynamics.Nav.AppSourceCop\\Upgrade\\RuleDestructiveTableFieldPropertyChangesNotAllowed.cs:line 107\r\n   at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.<>c__DisplayClass49_2.<ExecuteSymbolActionsCore>b__0() in F:\\a\\586\\s\\source\\Prod\\Microsoft.Dynamics.Nav.CodeAnalysis\\DiagnosticAnalyzer\\AnalyzerExecutor.cs:line 655\r\n   at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.ExecuteAndCatchIfThrows_NoLock(DiagnosticAnalyzer analyzer, Action analyze, Nullable`1 info) in F:\\a\\586\\s\\source\\Prod\\Microsoft.Dynamics.Nav.CodeAnalysis\\DiagnosticAnalyzer\\AnalyzerExecutor.cs:line 1077'",
	"source": "AL",
	"startLineNumber": 1,
	"startColumn": 1,
	"endLineNumber": 1,
	"endColumn": 1
}

{
	"resource": "/e:/xxx/App/app.json",
	"owner": "_generated_diagnostic_collection_name_#0",
	"code": "AD0001",
	"severity": 4,
	"message": "Analyzer 'Microsoft.Dynamics.Nav.AppSourceCop.Upgrade.RuleDestructiveTablePropertyChangesNotAllowed' threw an exception of type 'System.NullReferenceException' with message 'System.NullReferenceException: Object reference not set to an instance of an object.\r\n   at Microsoft.Dynamics.Nav.AppSourceCop.Helper.UpgradeVerificationHelper.GetTableTypeKindFromTableApplicationTypeSymbol(ISymbol tableApplicationObjectTypeSymbol) in F:\\a\\586\\s\\source\\Prod\\Microsoft.Dynamics.Nav.AppSourceCop\\Helper\\UpgradeVerificationHelper.cs:line 64\r\n   at Microsoft.Dynamics.Nav.AppSourceCop.Upgrade.RuleDestructiveTablePropertyChangesNotAllowed.CompareTables(SymbolAnalysisContext ctx, ITypeSymbol oldTable, ITypeSymbol newTable) in F:\\a\\586\\s\\source\\Prod\\Microsoft.Dynamics.Nav.AppSourceCop\\Upgrade\\RuleDestructiveTablePropertyChangesNotAllowed.cs:line 133\r\n   at Microsoft.Dynamics.Nav.AppSourceCop.Upgrade.RuleDestructiveTablePropertyChangesNotAllowed.AnalyzeTableExtension(SymbolAnalysisContext ctx) in F:\\a\\586\\s\\source\\Prod\\Microsoft.Dynamics.Nav.AppSourceCop\\Upgrade\\RuleDestructiveTablePropertyChangesNotAllowed.cs:line 114\r\n   at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.<>c__DisplayClass49_2.<ExecuteSymbolActionsCore>b__0() in F:\\a\\586\\s\\source\\Prod\\Microsoft.Dynamics.Nav.CodeAnalysis\\DiagnosticAnalyzer\\AnalyzerExecutor.cs:line 655\r\n   at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.ExecuteAndCatchIfThrows_NoLock(DiagnosticAnalyzer analyzer, Action analyze, Nullable`1 info) in F:\\a\\586\\s\\source\\Prod\\Microsoft.Dynamics.Nav.CodeAnalysis\\DiagnosticAnalyzer\\AnalyzerExecutor.cs:line 1077'",
	"source": "AL",
	"startLineNumber": 1,
	"startColumn": 1,
	"endLineNumber": 1,
	"endColumn": 1
}

I just realized that this might be the same issue as #5233, please mark this as duplicate if so.
To Reproduce
I don't have a clean repo, but I have it in my BC15 repo, with AppSourceCop configured against our BC14 app.
Just let me know if I can fetch some debugging data for you, or if you want to share screen or something.
Expected behavior
I don't expect those Exceptions. I want the rules to be applied.
Screenshots

5. Versions:

AL Language:  4.0.182565
--

Business Central: N/A

Hi @jwikman,
The easiest for us is if we can get our hands on an extension that reproduces the issue otherwise we can end up spending a lot of time searching for the issue.
So please provide a zip file with all there is needed to reproduce the issue.
We do have some changes in the AppSourceCop coming in the next release, so hopefully it is fixed there.

@thpeder, can I send you a zip as a private message or mail somewhere?
I don't have a small repo, it's our full AppSource app...

This issue is fixed in the next update.


Setting nextClip.start = thisClip.end

Hi,hope you are well.
can you please see this one line code of premiere pro scripting?
I followed your tut. And trying this.
nextClip.start = thisClip.end.ticks;

I just want next clip will be start at the end of first clip. But having
this error at extendscript ( Cannot set property start.)
here is the full code.
var project = app.project;
var sequence = project.activeSequence;

var videoTracks = sequence.videoTracks;
var thisTrack = videoTracks[0];
var clips = thisTrack.clips;
var thisClip = clips[0];
var nextClip = clips[1];

nextClip.start = thisClip.end.ticks;

Thanks

From the TypeScript definitions in Adobe's official starter code (https://github.com/Adobe-CEP/Samples/blob/master/PProPanel/jsx/PremierePro.14.0.d.ts) it appears that the start Time object is read-only,

that is why you can't set it.
However, I tried setting the end Time object of a clip and it also didn't work - so either the TypeScript definition is outdated, or there is another way to set the Time Object of a Track Item. @bbb999 any ideas? I've tried
newTime = new Time();
newTime.seconds = 20;
nextClip.end = newTime;

and
nextClip.end.seconds = 20;

but neither work - is it fair to assume that both start and end Time objects of a TrackItem is read-only?
Regarding the initials question about how to insert a clip next to the previous clip, here is a solution that adds an additional clip either with the insert or overwrite method. The unfortunate consequence is that you have to clean up after yourself and delete the original clip, here is some sample code:
var videoTrackNr = 0;
var audioTrackNr = 0;
var overwrite = true;

var project = app.project;
var sequence = project.activeSequence;

var videoTracks = sequence.videoTracks;
var track = videoTracks[videoTrackNr];
var clips = track.clips;
var thisClip = clips[0];
var nextClip = clips[1];
var nextClipAudio = sequence.audioTracks[audioTrackNr].clips[1];

var thisClipEnd = thisClip.end;

var nextClipProjectItem = nextClip.projectItem;

if (overwrite) {
  sequence.overwriteClip(
    nextClipProjectItem,
    thisClipEnd,
    videoTrackNr,
    audioTrackNr
  );
} else {
  sequence.insertClip(
    nextClipProjectItem,
    thisClipEnd,
    videoTrackNr,
    audioTrackNr
  );
}

nextClip.remove(false, true); // inRipple, inAlignToVideo
nextClipAudio.remove(false, true); // inRipple, inAlignToVideo

Result of running the above code snippet.

Wow it works.
I have no words to thank you Louwrens.
I must quit using PR if i could not do this.
I usually do it using after effects but Now premiere using GPU for rendering and it's much more faster. Thats why I'm trying shift my works to pr.
Thank you so much sir.

I wish you to make a video about this in your channel as well.

@shakilmax I've added a ticket on our video backlog: #21
We've got a few other video's lined up next, but once we are done with them we'll give #21 a go... 👍

Hi guys,
I believe its not read only. For example, app.project.sequences[0].videoTracks[2].clips[0].start=3, puts the clip start at 3 seconds for me. Also, in the extend script pdf premiere pro scripting guide suggests that it is read/write.
Best, Ales

Thanks @Fougl for the update.... ☺

When I use  Xclip[0].end = 3,  it works well , but also throws an error
so i do it with whese code.
// Move Next Clip To Previous End
var sequence = app.project.activeSequence;  // active sequence
var clip1 = sequence.videoTracks[1].clips[0]  // first clip
var clip2 = sequence.videoTracks[1].clips[1]  // second clip

var new_end_time = clip2.end.seconds - clip2.start.seconds + clip1.end.seconds  // second clip new end time , this way, dont change the clip duration

// use "clip.start" or "clip.end" will throw an error. I dont know why.

try {
    clip2.start = clip1.end.seconds
} catch (error) {

    try {
        clip2.end = new_end_time
    } catch (error) {
    }
}```


1st PR to get the ball rolling

Current content starts with a lot of assumptions about existing methodology and has a lot of advocacy of what should happen in agile. I think audiences are looking for more practical information on options and their consequences, rather than prescriptive content. People want good information, not to be told what to do. Base content was a bit of a manifesto for continuous localization. So, I think this is just a starting point.
I think we also need more graphics, to illustrate timelines and the coordination of responsibilities.
Looking forward to reading your thoughts on these first tidbits!

Docs Build status updates of commit 6a163d5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

globalization/methodology/agile-development.md
:bulb:Suggestion
View
Details

globalization/methodology/continuous-delivery.md
:bulb:Suggestion
View
Details

globalization/methodology/index.md
:white_check_mark:Succeeded
View

globalization/reference/glossary.md
:white_check_mark:Succeeded
View

globalization/methodology/agile-development.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

globalization/methodology/continuous-delivery.md

Line 3, Column 14: [Suggestion: duplicate-descriptions - See documentation] Attribute 'description' with value 'A faster cadence of development and release requires continuous delivery of localized content.' is duplicated in 'methodology/continuous-delivery.md(3,14)', 'methodology/software-localization.md(3,14)'.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit d869819:
:warning: Validation status: warnings

File
Status
Preview URL
Details

globalization/methodology/agile-development.md
:warning:Warning
View
Details

globalization/methodology/index.md
:warning:Warning
View
Details

globalization/reference/glossary.md
:warning:Warning
View
Details

globalization/methodology/continuous-delivery.md
:bulb:Suggestion
View
Details

globalization/methodology/agile-development.md

Line 19, Column 41: [Warning: bookmark-not-found] Cannot find bookmark '#starting-early' in 'methodology/index.md', did you mean '#start-early'?

globalization/methodology/index.md

Line 104, Column 4: [Warning: file-not-found - See documentation] Invalid file link: 'pseudolocalization.md'.

globalization/reference/glossary.md

Line 68, Column 34: [Warning: file-not-found - See documentation] Invalid file link: 'mbcs'.
Line 68, Column 48: [Warning: bookmark-not-found] Cannot find bookmark '#sbcs' in 'reference/glossary.md', did you mean '#ansi'?
Line 115, Column 28: [Warning: file-not-found - See documentation] Invalid file link: 'dbcs'.
Line 115, Column 42: [Warning: bookmark-not-found] Cannot find bookmark '#sbcs' in 'reference/glossary.md', did you mean '#ansi'?
Line 139, Column 34: [Warning: file-not-found - See documentation] Invalid file link: 'dbcs'.
Line 139, Column 48: [Warning: bookmark-not-found] Cannot find bookmark '#mbcs' in 'reference/glossary.md', did you mean '#ansi'?

globalization/methodology/continuous-delivery.md

Line 3, Column 14: [Suggestion: duplicate-descriptions - See documentation] Attribute 'description' with value 'A faster cadence of development and release requires continuous delivery of localized content.' is duplicated in 'methodology/continuous-delivery.md(3,14)', 'methodology/software-localization.md(3,14)'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 270e582:
:warning: Validation status: warnings

File
Status
Preview URL
Details

globalization/methodology/index.md
:warning:Warning
View
Details

globalization/methodology/continuous-delivery.md
:bulb:Suggestion
View
Details

globalization/methodology/continuous-localization.md
:bulb:Suggestion
View
Details

globalization/methodology/agile-development.md
:white_check_mark:Succeeded
View

globalization/reference/glossary.md
:white_check_mark:Succeeded
View

globalization/toc.yml
:white_check_mark:Succeeded
View

globalization/methodology/index.md

Line 103, Column 4: [Warning: file-not-found - See documentation] Invalid file link: 'pseudolocalization.md'.

globalization/methodology/continuous-delivery.md

Line 3, Column 14: [Suggestion: duplicate-descriptions - See documentation] Attribute 'description' with value 'A faster cadence of development and release requires continuous delivery of localized content.' is duplicated in 'methodology/continuous-delivery.md(3,14)', 'methodology/continuous-localization.md(3,14)', 'methodology/software-localization.md(3,14)'.

globalization/methodology/continuous-localization.md

Line 3, Column 14: [Suggestion: duplicate-descriptions - See documentation] Attribute 'description' with value 'A faster cadence of development and release requires continuous delivery of localized content.' is duplicated in 'methodology/continuous-delivery.md(3,14)', 'methodology/continuous-localization.md(3,14)', 'methodology/software-localization.md(3,14)'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Docker deamon not running error

i installed fm right after installing python3.11 and Docker and i run the the code below to create ERPNext ;
# create example.localhost site with frappe, erpnext and hrms, version -> version-15
fm create example --apps erpnext:version-15 --apps hrms:version-15

but i get this error;
❌ Docker daemon not running. Please start docker service.

and i checked if Docker is active by sudo systemctl is-active docker , i see it is active
i could not find the problem, fm installed on a freshly installed ubuntu 22.04 all updates, upgrades and restarts are done before.
System information:

fm --version : 0.9.0
lsb_release -a :

No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 22.04.3 LTS
Release:        22.04
Codename:       jammy

docker version: Docker version 24.0.7, build afdd53b
docker-compose version:

docker-compose version 1.29.2, build unknown
docker-py version: 5.0.3
CPython version: 3.10.12
OpenSSL version: OpenSSL 3.0.2 15 Mar 2022

Log
~/frappe/logs/fm.log : fm.log

As I can see in the logs
[2023-12-21 15:34:04,820] INFO: RUNNING COMMAND: create erpfast --apps erpnext:version-15 --apps hrms:version-15
[2023-12-21 15:34:04,820] INFO: --------------------
[2023-12-21 15:34:09,940] DEBUG: - -- -- -- -- -- -- -- -- -- -
[2023-12-21 15:34:09,940] DEBUG: DOCKER COMMAND: docker version --format json
[2023-12-21 15:34:09,967] DEBUG: {"Client":{"Platform":{"Name":"Docker Engine - Community"},"Version":"24.0.7","ApiVersion":"1.43","DefaultAPIVersion":"1.43","GitCommit":"afdd53b","GoVersion":"go1.20.10","Os":"linux","Arch":"amd64","BuildTime":"Thu Oct 26 09:07:41 2023","Context":"default"},"Server":null}
[2023-12-21 15:34:09,968] DEBUG: permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Get "http://%2Fvar%2Frun%2Fdocker.sock/v1.24/version": dial unix /var/run/docker.sock: connect: permission denied
[2023-12-21 15:34:09,972] DEBUG: RETURN CODE: 1
[2023-12-21 15:34:09,972] DEBUG: - -- -- -- -- -- -- -- -- -- -

Seems like your user doesn't have permission to run docker commands. To fix this issue you can add your current user to docker group using usermod -aG docker <youruser>

As I can see in the logs
[2023-12-21 15:34:04,820] INFO: RUNNING COMMAND: create erpfast --apps erpnext:version-15 --apps hrms:version-15
[2023-12-21 15:34:04,820] INFO: --------------------
[2023-12-21 15:34:09,940] DEBUG: - -- -- -- -- -- -- -- -- -- -
[2023-12-21 15:34:09,940] DEBUG: DOCKER COMMAND: docker version --format json
[2023-12-21 15:34:09,967] DEBUG: {"Client":{"Platform":{"Name":"Docker Engine - Community"},"Version":"24.0.7","ApiVersion":"1.43","DefaultAPIVersion":"1.43","GitCommit":"afdd53b","GoVersion":"go1.20.10","Os":"linux","Arch":"amd64","BuildTime":"Thu Oct 26 09:07:41 2023","Context":"default"},"Server":null}
[2023-12-21 15:34:09,968] DEBUG: permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Get "http://%2Fvar%2Frun%2Fdocker.sock/v1.24/version": dial unix /var/run/docker.sock: connect: permission denied
[2023-12-21 15:34:09,972] DEBUG: RETURN CODE: 1
[2023-12-21 15:34:09,972] DEBUG: - -- -- -- -- -- -- -- -- -- -

Seems like your user doesn't have permission to run docker commands. To fix this issue you can add your current user to docker group using usermod -aG docker <youruser>

thank you for reply, i added the user to docker group, it did not solve the problem, still gives same error.

Are you able to run docker commands like docker version --format json normally using the same user ?


PSU control

would it be possible to send gcodes before a print regardless of the printer state? i have my PSU controlled by my raspberry so the printer cannot come operational before the power is on.

No, you can't send gcode, but you should be able to use system commands. https://github.com/jneilliii/OctoPrint-PrintScheduler/wiki/Helpful-System-Commands


Optimize task creation/deletion times

Resources' read/create/delete operations should be deduplicated and carried in parallel when possible in order to speed up the user-facing operations. The only sane way of doing this is by representing the resources as a directed acyclic graph and walking it as Terraform does.
#423 doesn't log create/read/delete steps with the INFO level because they are being performed several times. Adding a <nil> check to avoid duplicate reads would allow us to move those logs back to the INFO level.

I sometimes still suffer the super long creation times for just two tasks. Very hard to reproduce

Takes me ~1min to apply and ~5min to destroy a minimal script single task with no workdir files to sync.

While this sounds reasonable, I think we need a long-term strategy fortpi in all its incarnations. Reimplementing terraform is not a path we want to go down.


SAM to FASTQ functionality

Say you are only given a longranger processed BAM file that you would like to use for other purposes. Many tools cannot use BAM format directly. It could be useful to support a fast SAM/BAM conversion to fastq that preserves BX tags, RX tags and even MI tags in the header.
This is pretty easy to script, but I think this could be useful if written efficiently. It would also present the opportunity to encourage some sort of standardized fastq header format for chromium data.

Is there a preferred format for this header? This would be easy to incorporate, just give me the format.

Ideally, it would be similar to whatever longranger basic provides, but I haven't figured out where the RX tags are stored in the FASTQ file generated.
Maybe something like this:
@ReadID BX:Z:AAACACCAGAAACCCG-1 RX:Z:AAACACCAGAAACCCG MI:i:22359430

Make sure you make it look at the flag to reverse it back it the alignment has reversed the read.

You can use samtools fastq -TBX for this purpose.


[Typo] Remove trailing forward slash from BASH path

The install from source instructions under Linux/MacOS reads in part:
echo "export PATH=$(pwd)/bin/:...

This should perhaps have the trailing forward slash removed from the first path:
echo "export PATH=$(pwd)/bin:...

Otherwise you get something confusing (if harmless?) like:
$ which raku
~/rakudo/bin//raku

I agree with this suggestion, but this ticket should be in the rakudo.org website repo at https://github.com/rakudo/rakudo.org.
(It's an easy enough change so perhaps someone can fix the issue and close it here)

(Sorry, I see you were also working on something in a fork!)

No problem. I am unfamiliar with this source.
You seem to have found two relevant files, so I'll close my pull request -- thank you @coke


Adding IANA_SVC_NAME definition to docs/design/identifiers.md

To finalize https://github.com/GoogleCloudPlatform/kubernetes/issues/9734 as pointed out in https://github.com/GoogleCloudPlatform/kubernetes/pull/9736

@bgrant0607 @thockin
PS:  I'll be off next week and I won't be able to follow up if any modification is needed.

LGTM

Thanks


Way to print out world properties

While working on oregen for various dimensions, I couldn't find a way to get the name of the current generator. Is there any way to print out things like dimension.generator and if not could such a command be added?

Seems like this should be the role of another mod.

I don't know about "dimension.generator", but if you choose to use "dimension.generator.class", you can find the name of the mod's dimension generator class.  Usually, the class used in a mod is of the style ChunkProvider(something), where (something) is usually specific to that mod (for example, ChunkProviderEnd or ChunkProviderTwilightForest).


Favicons are assumed to be in root

I have checked all the prerequisites below and I'm yet experiencing a problem

[x] Read the README.md
[x] Have the extended version of Hugo installed
[x] Used the exampleSite's config.toml as a reference

Describe the bug
If you set a baseURL in a subdirectory of a website/domain (e.g. http://www.example.com/somedirectory), favicons are not loaded properly. This is due to the leading / in this line of code:
https://github.com/onweru/compose/blob/dec3f94f9dea42b28921b13ca12d3827c90cbf77/layouts/partials/head.html#L4
It should instead be: {{- $iconsPath := "favicons/" }}
Desktop (please complete the following information):

OS: Windows 10
Browser: Firefox, Chrome

@PetrosPapapa, thank you.


Indicate Target Platform in RPM File Names

As an Open GEE RPM user, I want to be able to tell what platform a given RPM would work on from its file name.
For example, shared libraries used by Open GEE have different versions on RHEL 6 and RHEL 7.  Therefore, an installation of an RPM containing Open GEE compiled for RHEL 6 would fail to run on RHEL 7, because the linked would not find the referenced shared libraries.  As a user, I want to be able to tell whether I can install a given RPM on my platform from its file name.
Notes:
Conventionally, Cent OS 6 and RHEL 6 user the el6 ID in RPM file names, and Cent OS 7 and RHEL 7 use el7.  Use the conventional names that users are already familiar with.
AC:

The RPM build tasks should produce RPM file names that identify the platform the RPMs can run on.

Can you provide examples in the AC of what filenames on some platforms ought to look like?


iotagent-mosca publishes QoS0 messages out of order

when messages are sent to the iotagent-mosca in rapid succession they are often received by the subscribers (kafka, ...) out of order.
According to the MQTT specfication, QoS0 messages of the same topic must be processed in order when they are not lost.

Duplicated #595


Support abort controller

AWS SDK v3 supports abort controllers as an option that can be passed to any command. It would be helpful if that was supported, without having to manually pass params to the AWS SDK
https://github.com/aws/aws-sdk-js-v3?tab=readme-ov-file#abortcontroller-example

Yes good idea! Wouldn't be too hard to implement 👍

@rjmackay abortSignoal is now available since v1.12.2. Docs: https://www.dynamodbtoolbox.com/docs/getting-started/usage#aborting-an-action
Cheers!


Updating with current season mode

I hope someone can point out an error I am making but at the moment when I try to refresh the current season using current-season-mode, I get the error message that I am trying to use it on an uninitialized database.
"Error: option '--current-season-mode' passed on an uninitialized database.
First load a season using the '--default-mode' flag!"
I am able to access the database and run queries perfectly fine so I don't understand why it is telling me my database is uninitialized. When I keep all the same info and switch to default-mode, I get an error that there are duplicate keys (because the database already exists). This is the first SQL database that I have created on my own computer so I'm not sure what issue I am having.

Did you load all of the tables or just a subset when you ran default season mode?
-------- Original Message --------
On 4 Apr 2024, 3:29 am, data-rob wrote:

I hope someone can point out an error I am making but at the moment when I try to refresh the current season using current-season-mode, I get the error message that I am trying to use it on an uninitialized database.
"Error: option '--current-season-mode' passed on an uninitialized database.
First load a season using the '--default-mode' flag!"
I am able to access the database and run queries perfectly fine so I don't understand why it is telling me my database is uninitialized. When I keep all the same info and switch to default-mode, I get an error that there are duplicate keys (because the database already exists). This is the first SQL database that I have created on my own computer so I'm not sure what issue I am having.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Did you load all of the tables or just a subset when you ran default season mode?

When I first built the database I didn't skip any tables and I just selected the 2023-2024 season. When I tried to refresh using current-season-mode I tried including every table as well as skipping every table except for player_game_log but both times it told me I was passing it on an uninitialized database. I also tried every table and just the player_game_log with the default-mode but that's when I got the duplicate error message.

I am able to successfully refresh my database (SQLite) using:
> DB_NAME="nba" DB_HOST="localhost" DB_USER=nba_sql DB_PASSWORD=nba_sql python stats/nba_sql.py --default-mode --seasons 2023-24 --skip-tables pgtt shot_chart_detail play_by_play player_game_log

> DB_NAME="nba" DB_HOST="localhost" DB_USER=nba_sql DB_PASSWORD=nba_sql python stats/nba_sql.py --current-season-mode

The way we check that a database is already initialized is by querying the season table and getting the latest season. It is possible that this isn't working for you for some reason. Can you run a SELECT * FROM season; and add what it returns?
Are you using the GUI or the commandline?

I am able to successfully refresh my database (SQLite)

For added context I am using postgres.

The way we check that a database is already initialized is by querying the season table and getting the latest season. It is possible that this isn't working for you for some reason. Can you run a SELECT * FROM season; and add what it returns?

It returns just the season_id as 2023.

2. Are you using the GUI or the commandline?

I am using the GUI from the nba_sql_v0.0.12_windows.exe file.

@data-rob re-open this with repro steps if you can.


Add method for injecting defaults

Following instance:
{"foo": 1}

Is valid against following schema:
{
  "type": "object",
  "properties": {
    "foo": {"type": "integer", "default": 0},
    "bar": {"type": "string", "default": "baz"}
  }
}

Though .validate method won't inject default value for bar. And I believe this is correct behavior. Instead we should have method like .setDefaults that when invoked mutates original JSON by setting missing bar (but not foo of course).
/cc @knservis

.setDefaults is interesting. It makes schema backwards compatibility easier. What happens in JSON Schema if you have a default, make the field "required", but it's not present in the instance - presumably validation fails?

Yes, validation fails at least at https://www.jsonschemavalidator.net/

Would you not run setDefaults before validate?

This would be a good idea to call it before, but this is up to consumer.
Question here: should cases like required AND default be invalidated by one of optional igluctl linters? For me it looks like user doesn't quite understand what he wants.

At least a warning is warranted. Something like "WARN required AND default found for field x! Required is redundant!". An error is also an option if that definition will cause errors. The warning will alert the user to a possible omission, but the execution is possibly unaffected by removing required in the case that default is present. Either way is fine as the user needs to know that this is pointless.

Yep. This is exactly how all linters work. If it sees type: string and maximum: X (integer's validation) - it yields. Schema is valid, validator can proceed, but this is very likely user missed something, usually just because he/she refactored schema and forgot something.

I'm thinking this can be a separate enrichment.


Error on router rule with an anonymous function

// Custom rule
$route['test'] = function (){
return 'welcome/index';
};
Visit the url: http://loacalhost/test
Get Error:
A PHP Error was encountered
Severity: Warning
Message: explode() expects parameter 2 to be string, object given
Filename: core/Router.php
Line Number: 390

Docs :
http://www.codeigniter.com/user_guide/general/routing.html#callbacks


Example with webpack integration

Hi
I tried to include this library in an asp.net core Angular JavascriptServices application and have the hardest time to get it up and running. Do you have an example how it does need to be configured?  I installed the package and change my root module but I am not sure what else needs to be done.
Thank you in advance!
Torsten

Needed to add "@types/applicationinsights-js": to the devDependencies in the package.json.

Ahh would it help if we add it to the dependencies here? Or should we make note of it in the Readme?

I would add this as a dependency to have it "working out of the box" but Readme would do it as well.  THANK YOU!

Yeah let me just add it as a dependency, thanks for pointing it out!


Update pdf.py

fix semantra UnicodeEncodeError when load pdf @win11

Many thanks for the PR and for contributing!


Relationship table accessibility

When relationship tables are published to HTML, the links are formatted as a set of div elements. While this was acceptable under Web Content Accessibility Guidelines(WCAG) 1.0, WCAG 2.0 calls for lists of links to be formatted using list markup. HTML processing should be updated to conform to the recommendations of WCAG 2.0.

I've gotten the same guidance from accessibility experts I work with, and we've had to override the default output to meet WCAG. I'm working on getting this one into the base code for 3.0 for HTML5 and XHTML.


First cut at 1.0 announcement. Much work remains.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: geeknoid
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS [geeknoid]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Put comments in the Google doc please

@geeknoid where is the google doc? :)


myria: delete client package

Code is unused and untested

Coverage increased (+1.27%) to 59.25% when pulling 21e667d75a86d93d386746f80ba554c60f15d566 on delete-client-pkg into 34170b1638141eaa2fcd535e9e2934d453fa4bd9 on master.


Symfony 4 compatibility

Could you please add compatibility for Symfony 4?
Problem 1
    - Installation request for fousky/idoklad ^0.2.0@dev -> satisfiable by fousky/idoklad[0.2.x-dev].
    - Conclusion: remove symfony/filesystem 4.1.x-dev
    - Conclusion: don't install symfony/filesystem 4.1.x-dev
    - fousky/idoklad 0.2.x-dev requires symfony/filesystem ~2|~3 -> satisfiable by symfony/filesystem[2.1.x-dev, 2.2.x-dev, 2.3.x-dev, 2.4.x-dev, 2.5.x-dev, 2.6.x-dev, 2.7.x-dev, 2.8.x-dev, 3.0.x-dev, 3.1.x-dev, 3.2.x-dev, 3.3.x-dev, 3.4.x-dev].

@honzagula ok, I'll try

@honzagula released https://github.com/Fousky/iDoklad/releases/tag/0.2.7-dev Can you try it?

Yeah, it works. Thank you!


mjpage doesn't seem to do anything.

Following installation instructions:
$ npm install mathjax-node-page
$ node_modules/mathjax-node-page/bin/mjpage

doesn't do anything, the command just hangs. Same if I try to pass some simple HTML:
$ node_modules/mathjax-node-page/bin/mjpage '<title>test</title>test $\alpha \le \beta$' > /tmp/test.html

Changing the --fontURL option to somewhere I know fonts are installed & available has no effect. There's no error message nor a --debug option so I have no idea where to go from there.
I'm using npm version 5.3.0.

Thanks for the report. I'll look into why it's not printing the usage when missing input.
From the file source; Usage: mjpage [options] < input.html > output.html
But note that single-dollars are not configured by default (just like MathJax). If you need to pass a string, try mjpage --dollars <<< "<title>test</title>test $\alpha \le \beta$"

Thanks for the quick answer. The problem isn't simply there when the input is empty or because I used the wrong notation. Your example also does nothing and hangs, as well as if I simply run mjpage with some HTML that doesn't contain any TeX code at all. Essentially, mjpage is not working no matter what.

Hm... The code I pasted works fine for me on a clean setup using node 8.2.1 / npm 5.3. Not sure how to help you, I'm afraid.

Closing this as abandoned. If there is more information on how to reproduce the problem, I'm happy to revisit this.


Fix tox.ini formatting when envlist ends with a comma

I am here to learn - so if you would do things differently, or when you have any comments, don't be shy.
This fixes #41.
modified:   CHANGELOG.md
modified:   src/tox_ini_fmt/formatter/tox_section.py
modified:   tests/test_main.py

Codecov Report

Merging #43 into main will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #43   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            9         9
  Lines          292       294    +2
  Branches        66        67    +1
=========================================
+ Hits           292       294    +2

Flag
Coverage Δ

#tests
100.00% <100.00%> (ø)

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

Impacted Files
Coverage Δ

src/tox_ini_fmt/formatter/tox_section.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


get all feature state

Hi, i'm wondering is there or will there be support for getting all features and their state?
Something like:
client.getAllFeatures()

For more context, i want to get all the features that are toggled on for a specified user or user type
like
context.attr("userId",id).build.get()

then
client.getAllFeatures().stream().filter(FeatureState::isEnabled).map(FeatureState::getKey).collect(Collectors.toList());

I don't see why not - its just a Map in the ClientFeatureRepository so adding that API would be easy.

Oh nice, was just wondering if there will be any design issues, if not i think i can help make a PR for it


Sync up gitignore to githubs recommendations fixes #215

Yeah, as it says we don't. These ones we have must have been added months ago before the .gitignore was setup. This commit is removing these files.
https://github.com/tl-its-umich-edu/student-dashboard-django/tree/master/dashboard/__pycache__

I see. No more questions then!


Why Longhorn volume state is detached and Health unknown , but log show attached to a node?

Question

(https://github.com/longhorn/longhorn/assets/8872412/894a0bb6-92f5-4a80-9363-4d94e7b434f0)
Environment

Longhorn version:1.4.1
Kubernetes version:1.23.13
Node config

OS type and version:CentOS8
CPU per node:
Memory per node:
Disk type: lvm
Network bandwidth and latency between the nodes:

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal):

Additional context
Add any other context about the problem here.

In my case, this fixed and issue:
The volume was 7.82/8Gi. I added +2Gi to it. After resizing, I recreated the pod and it started well.

In my case, this fixed and issue: The volume was 7.82/8Gi. I added +2Gi to it. After resizing, I recreated the pod and it started well.

You mean this error is because the volume is full ,expanding the capacity of the volume can solve the problem
right?

Welp, the last volume was in 7.98/8Gi. +2Gi did nothing, the recreation of the pod not fixed anything. +1Gi fixed the issue.

Hi @shaolila,
Is this issue still there?
Need more information for investigating.
How many nodes do you deploy? How many control nodes do you set? How many worker nodes do you set?
Before this issue happened, what did you do?
Hi @lexfrei,
Thanks for the testings.
Could you send us the support bundle if/after this happened again?


FastAPI Integration

Link to Main PR:

https://github.com/code-kern-ai/cognition-gateway/pull/66

expected a requirements change for the removed packages for gql & correct version from fastapi & starlette
Note that currently the starlette & fastapi are commented out in the common-requirements.txt & old versions (to support gql via starlette) were added in the requirements.in. Reach out if we should do a quick intro to this :)

[ ] resolved


rename polynomial.RotateCoeffs

This function should be given a more descriptive name, e.g. combine_rotation_and_polynomial.

or prepend_rotation_to_polynomial


Shell module is not accepting " or ' in the command.

  - name: "Get Version of {{ item }}"
    shell: cmd="my-service {{ item }} version | cut -d ' ' -f2"
    debug: true
    with_items:
    - "cassandra"
    - "zookeeper"

ERROR
SHELL [ cmd => my-service cassandra version | cut -d   -f2 ]
failure => exec'ed command

Output
-----
stderr: cut: the delimiter must be a single character
Try 'cut --help' for more information.
echo: write error: Broken pipe

stdout:

@sudharsh can you get this one?

This one is fixed with 3940eef44dafa8de7ea09d51ccaedc09cf3e7d78


Migration to Flutter 2 and Null safety

@huitingyeong thank you for the plugin!
Is this plugin going to migrate to Flutter 2 and Null safety?

I'd like to know this as well.

@MrCsabaToth has kindly done a PR for null safety


"unhandled exception escaping from a worker thread"

librealsense
2.50.0 RELEASE

OS
Windows

Intel RealSense Viewer / Depth Quality Tool has crashed with the following error message:
Unhandled exception escaping from a worker thread!
Error type: Unknown error (-529697949)!

Hi @HeloGiThUb  Please follow the advice in the case at https://github.com/IntelRealSense/librealsense/issues/10681 and let me know whether it resolves your problem.  Thanks!

when using depth camera D455, i always see it and i can't  solve it. Can you help me a hand, please! Thanks you very much!!

Does the Motion Module start working normally if you enable it after this message has appeared, please?

The Motion Module will work with USB 2.1 too.  The image below shows my own D455 displaying the Gyro and Accel streams in USB 2.1 mode.

Are you using the offical short 1 meter long cable supplied with the camera, please?  If you are using your own choice of cable then it is important to choose a high quality one that is suited to transferring data and is not just a device charging cable.  Cable quality is especially important when using cable length longer than 2 meters, otherwise the camera connection can degrade or disconnect.

Actually, i changed among cables. Long or short cables just can't connect the Depth camera. So, do you have other options? thanks you

Next, please try completely uninstalling and reinstalling the two RealSense camera drivers (Depth and RGB) in the Windows Device Manager interface using the instructions at the link below.
https://support.intelrealsense.com/hc/en-us/community/posts/4419989666323/comments/4431239847443

Hi @HeloGiThUb  Do you require further assistance with this case, please?  Thanks!

No, i don't. thanks you so much

Thanks very much for the update, @HeloGiThUb  - as you do not require further assistance, I will close this case.  Thanks again!


speech separation evaluator returns negative sdr sar

Hello, thanks for your work!
I'm learning to evaluate my separated signals. My reference signal is original voice of single target speaker, estimated signal is the voice of the target speaker after the two speakers are separated. Both of them are .wav files. But mir_eval library returns negative SDR SAR. Is there something wrong about my input or anything else?
Looking forward for your reply

SDR and SAR are expressed in decibels (logarithmic scale) and therefore can be negative. Please see this paper for more details.


Build fail on windows

I'm currently using zig version 0.12.0-dev.2063+804cee3b9, when I try to build and run the project it throws this errors

I've already tested using version 0.13.0 and the nominated version, it actually gives me different errors involving deadlocks, cleaned zig global cache yet nothing

I've already tested using version 0.13.0 and the nominated version, it actually gives me different errors involving deadlocks, cleaned zig global cache yet nothing

Yeah but what error specifically appear when u change the version?, and in the error you already got how is the Lazy path function set?

I've already tested using version 0.13.0 and the nominated version, it actually gives me different errors involving deadlocks, cleaned zig global cache yet nothing

Yeah but what error specifically appear when u change the version?, and in the error you already got how is the Lazy path function set?

In version 0.13.0-dev.351+64ef45eb0 this warnings appear first when I build

then it's followed by a lot of these kind of errors

And about the lazypath, I've run it again with 0.12 nominated version and the error seems to be different for some reason

They appear to come from the cache since they go to my global cache directory when the error is being thrown, so maybe the problem is a version when I run it on 0.12 since the build files from the cache says that I should run it on 0.13.0 or the 0.13 nominated version, but when I run it on one of those versions is when the deadlock error appears

The targeted Zig version is 0.13.0 / 2024.5.0-mach , it won't work with other Zig versions.

The targeted Zig version is 0.13.0 / 2024.5.0-mach , it won't work with other Zig versions.

I'm on 0.13.0 (Windows x86_64), and I'm also getting the same mutex Deadlock detected error as @NexWan:

This is with just cloning the template repo, not modifying anything at all.

Oh just fantastic, it compiles with ReleaseSmall/Fast, but not Debug... 😕

Oh just fantastic, it compiles with ReleaseSmall/Fast, but not Debug... 😕

I've just changed it to ReleaseSmall and indeed it worked


Federated SignIn for React Native

Issue #, if available: https://github.com/aws/aws-amplify/issues/45
Description of changes: Adding Federated Signin for Facebook and Google on React Native.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #340 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #340   +/-   ##
=======================================
  Coverage   90.29%   90.29%
=======================================
  Files          53       53
  Lines        2556     2556
  Branches      521      521
=======================================
  Hits         2308     2308
  Misses        237      237
  Partials       11       11

Continue to review full report at Codecov.

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

Since this implementation is taking a hard dependency on expo, I'm closing this CR and will have another one for Federated signin.


Cannot find module './modules/UnimplementedView'

Canary changed strategy to import modules. With 7.0.2, React Native Web works. More info here:
This is cross issue because I don't know where the bug is.
https://github.com/necolas/react-native-web/issues/1220

I'm not sure what's happening here. You didn't add the serverless target right?

No. I also checked Next.js RNW example with the same result.


Service Discovery for Laravel Portals

Implementation of Service Discovery so that each instance of the portal, which spawns up, registers itself to the loadbalancer.
Options -
consul-template
keepalived

Setup base consul server for service discovery
Consul dashboard accessible from - http://ec2-52-14-33-15.us-east-2.compute.amazonaws.com:8500/ui/
TO-DO:
Integrate Consul-template , Registrator & HAProxy.
I came across Registrator while reading about dynamic service discovery. It basically keeps looking for docker containers on the machine it is present, and once it finds a new container spawned, it registers this in the consul registry.


Generated command is not runnable

The command generated by the ToCurlAsync method actually cannot be run anywhere, as far as I know.

First, the command is generated as multi-line, without escape characters. There should be option to use escapes either for cmd.exe (^), PowerShell (`) or Bash (\).
Second, it uses 'single quotes' to enclose values which is illegal in cmd.exe, which expects "double quotes" in its default settings.
Third, for PowerShell, which is now default command line on Windows, does recognize curl as alias for Invoke-WebRequest, with completely different argument set. To invoke real curl, curl.exe has to be used, which would not work in Bash.

Proposed solution
Include (preferably global) configuration option as to which code should be generated. As it has to be compatible either with cmd.exe (the legacy Windows command line), PowerShell or Bash.

I just pushed a version that includes the missin \.
I just use it to import into Postman and there are not needed (see here my post https://imperugo.medium.com/log-curl-from-your-http-request-797562a14ad9).
Now it works also from terminal.
Take a look and let me know
.u

Going to cose it.
Please feel free to open it again if you still have the problem with the new version

It only fixes part of the problem. Now the command is runnable, but only in Bash, nowhere else. I think the best solution would be to create injected configuration object, which would hold information about the platform (Bash, CMD or PowerShell) and then it will create command with the right command (curl or curl.exe), right quotes and right escapes for the given platform.

Okay, added a PR #8

Thanks @ridercz
I've merged your PR and added the support for the http2 protocol.


feat: inject environment variables from .env files into edge functions locally

Summary
This allows any environment variables that are coming from dot-env files (.env) to be accessed in edge functions.
Similar to what is described here: https://docs.netlify.com/configure-builds/environment-variables/#build-locally

For us to review and ship your PR efficiently, please perform the following steps:

[ ] Open a bug/issue before writing your code 🧑💻. This ensures we can discuss the changes and get feedback from everyone that should be involved.
[ ] Read the contribution guidelines 📖. This ensures your code follows our style guide and
passes our tests.
[ ] Update or add tests (if any source code was changed or added) 🧪
[ ] Update or add documentation (if features were changed or added) 📝
[ ] Make sure the status checks below are successful ✅

A picture of a cute animal (not mandatory, but encouraged)

@danez - I'm not sure what part of this PR you want me to review. I took a look through and couldn't identify any changes that will impact what's shown in the cli docs at https://deploy-preview-5620--cli.netlify.app/
Could you point me to the part of this PR that needs my review?
I do see that docs.netlify.com documents some limitations around this behavior that will need to be updated/removed (on the Environment variables and functions page and the Edge Functions limits page)- I can open a PR to get the ball rolling on those changes.

@danez - I'm not sure what part of this PR you want me to review. I took a look through and couldn't identify any changes that will impact what's shown in the cli docs at https://deploy-preview-5620--cli.netlify.app/
Could you point me to the part of this PR that needs my review?
I do see that docs.netlify.com documents some limitations around this behavior that will need to be updated/removed (on the Environment variables and functions page and the Edge Functions limits page)- I can open a PR to get the ball rolling on those changes.

Sorry, i didn't mean to ask for a review. I just wanted to let you know that this is something that is changing which we might want to document :)


allow to pass Capabilities into BrowserWebDriverContainer#withDesiredCapabilities

I would like to use ChromeOptions to configure chrome instance
  public static ChromeOptions buildChromeOptions() {
        ChromeOptions chromeOptions = new ChromeOptions();
        chromeOptions.addArguments("--lang=pl", "--window-size=1600,900", "--disable-extensions");
        Map<String, Object> prefs = new HashMap<>();
        prefs.put("intl.accept_languages", "pl");
        chromeOptions.setExperimentalOption("prefs", prefs);
        return chromeOptions;
    }

+1
Using more recent versions of Selenium (3.X), the following log message is printed when creating a BrowserWebDriverContainer with DesiredCapabilities:
INFO: Using 'new ChromeOptions()' is preferred to 'DesiredCapabilities.chrome()'
unfortunately BrowserWebDriverContainer only exposes:
public SELF withDesiredCapabilities(DesiredCapabilities desiredCapabilities) instead of the parent interface Capabilities.

I'm using selenium-jupiter for junit 5 and it works great

@kamkie @peternitro Would someone like to contribute this feature?


[improve][test] add a test to ensure the exclusive consumer will reconnect even if received ConsumerBusy error

Motivation
Background

When the customer disconnects, the consumer automatically tries to reconnect.
When a connection is disconnected, the broker will remove the consumer related to this connection.
The client does not reconnect when it receives a ConsumerBusyException.

Issue

time
client disconnect
broker disconnect

1
disconnect

2
reconnect

3
Send cmd-sub to the broker
disconnect

4
Since the old consumer has not been removed in the broker memory, get a consumer busy error

5

Remove the old consumer in memory

6
Stop reconnect

At this time, there is no consumer under this subscription.

A mechanism to fix the issue above
Once a consumer has subscribed successfully, the consumer will try to retry regardless of any exceptions received. see: https://github.com/apache/pulsar/blob/master/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java#L875
Modifications
Add a test to ensure that the above issue has been fixed
Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: x

/pulsarbot run-failure-checks

Add a test to ensure that the above issue has been fixed

@poorbarcode - do you have a link to the PR where this issue was fixed?

@michaeljmarshall

do you have a link to the PR where this issue was fixed?

The PR https://github.com/apache/pulsar/pull/20737 fixes a issue one consumer will reconnect more than one times.
Issue: if we use an exclusive consumer, and hit the issue which https://github.com/apache/pulsar/pull/20737 tries to fix, the consumer will get a ConsumerBusyException, which makes the consumer close. This test to confirm this scenario will not occur

Thanks for the explanation @poorbarcode


PostgreSQL support

From @adriancole on March 11, 2016 1:19
@Koizumi85 asked on gitter about PostgreSQL support. spanstore-jdbc takes advantage of some SQL syntax which might not be supported in PostgreSQL. Before we start down this path, let's figure out some basics. For example, which version of PostgreSQL is needed, so that we can test the configuration desired.
Copied from original issue: openzipkin/zipkin-java#93

@Koizumi85 nag..

From @eirslett on March 21, 2016 10:21
Could it be supported as an external/optional module? I'm not sure it's a good idea to reuse the JDBC code, even if that's tempting.

From @Koizumi85 on March 21, 2016 10:28
Actually i think with postgresql 9.5 the actual code could possibly work if there were a possibility to switch the database type in Jooq..
Background: psql9.5 supports "upsert" semantics which are heavily used here.
But postgresql 9.5 is fairly new and is not hosted in the repositories of Ubuntu LTS and other linux distributions and there are a lot of sysadmins who don't want to take the postgresql repositories for LTS reasons.

From @eirslett on March 21, 2016 12:11
9.1 or 9.3 are the safest choices, I think.

From @Koizumi85 on March 21, 2016 13:10
9.1 and 9.3 should be almost identically unless the json column type of 9.3. but i don't think we would need this here, or?
So I think a 9.1 compatibility would be nice.

From @eirslett on March 21, 2016 13:17
Sounds good to me. The JSON query capabilites would be nice to have (for things like annotations), but it wasn't added in MySQL until 5.7.8, I think many linux distros are at MySQL 5.5 or so. And the existing SQL span store doesn't use JSON.

Any word on this? We dev against PG and would love to not have another database dep just for zipkin (especially on dev machines)

your are lucky number 3 person to be interested in this! https://blog.codinghorror.com/rule-of-three/
if we started this, would you help maintain and/or test it? Which version of postgresql can you use?

Yes! Definitely. Our company is about to launch and one of our goals is to contribute to active open source projects, so we have buy-off from an organizational level to contribute to FOSS.
We're on 9.5+, but we could easily support earlier versions. I agree to your previous statements that 9.1 would be a good goal to shoot for.

ooo this sounds suspiciously like an offer to help :) @dwelch2344 would you be up to designing this?
ps the version discussion is dated. ex 9.4 is the current supported version https://www.postgresql.org/download/linux/ubuntu/
@Koizumi85 @eirslett @klette @mansu @jcarres-mdsol any thoughts on this one?

Is there any update on this front?  I would like to see PostgreSQL support for Zipkin as well.

no one is actively working on it quite yet, hoping someone can pick it up. Regardless, it is probably best started as a separate repo I think.

best as a separate repo as we are pretty out of resources on this one. adding another DB might hurt our memory and bump build execution past 20m. that said, maybe circleci might make that better.. wdyt @abesto

First, we need to pick a minimum version Ex 9.5 supports upsert and if we use the existing schema from mysql, it would be the easiest to port. On that note, we should decide if we are just porting, or looking to design a high performance schema.
IOTW is the motivation just using postgresql? or are we trying to make a high-performance alternative that uses postgresql?

I think most companies don't really need Cassandra on the backend.  I suspect a high-performance alternative for medium-size data will something that will appeal to this audience.  What do you think?

I would happily volunteer, but I don't have enough knowledge of Zipkin nor PostgreSQL to take on the task...

I agree with @hwasungmars. Implementing PostgreSQL support would be a good opportunity to provide a high-performance implementation backed by a relational database. I suppose there are a lot of people who aren't used to NoSQL databases yet and would appreciate a relational backend which provides a good performance

I think most companies don't really need Cassandra on the backend. I
suspect a high-performance alternative for medium-size data will something
that will appeal to this audience. What do you think?

no insight into deployments of cassandra vs elasticsearch vs postgresql to
be honest
On Wed, Oct 5, 2016 at 10:25 PM, Hwasung Lee<EMAIL_ADDRESS>wrote:

I think most companies don't really need Cassandra on the backend. I
suspect a high-performance alternative for medium-size data will something
that will appeal to this audience. What do you think?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/zipkin/issues/1232#issuecomment-251690230,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAD61y6FX0wM0026fk-Ge_huEIJ17lvHks5qw7NzgaJpZM4Jgz3S
.

I think most companies don't really need Cassandra on the backend. I suspect a high-performance alternative for medium-size data will something that will appeal to this audience. What do you think?

I believe Cassandra is the most widely used Zipkin backend for production workloads.

no one is actively working on it quite yet, hoping someone can pick it up. Regardless, it is probably best started as a separate repo I think.

This is a good idea. Historically, we've already abandoned one DB backend (HBase, the support for that was eventually discontinued), and having implementations in separate repositories means it's up to module maintainers to make sure the backends are up to date. If support for ObscureDB/FadDB is implemented, and then the original authors lose interest and/or nobody else wants to maintain it, then the burden will not be in the core team, it will be on the developers who want to run Zipkin on XxxDB. (I'm not saying PostgreSQL is obscure, it's widely used in the tech world)
There is a balance to strike, between supporting modules in the core repo or separate repos. MySQL was chosen as the relational/simple-to-setup SQL alternative, and Cassandra for the high-scale deployment solution. Elasticsearch was recently added to leverage some of the query functions supported in Lucene (I think).

I sadly have not much to give but would love to see a postgres backend:P

I would be happy to try to review schema ideas/implementations. Do not hesitate to ping me if that happens.

I too would like see a PostgreSQL backend

Is there any work going on for postgres persistence support ? Any way I can help ?

work here needs champions to run it through. If you like, you can look at
the work-on-progress voltdb experiment as an example, though in the schema
likely completing normalization of the data is preferred (unclear on this
really)
https://github.com/adriancole/zipkin-voltdb
Anyway, you can create a repo based on this to start toying with it. Right
now, no one has worked on postgres

Came across this :
https://github.com/IAMTJW/zipkin/tree/master/zipkin-storage/postgresql
Any chance of this getting merged(accepted) ?

Came across this :
https://github.com/IAMTJW/zipkin/tree/master/zipkin-storage/postgresql
Any chance of this getting merged(accepted) ?

first, the author would need to volunteer to contribute the code.
technically though, this looks like a port of the mysql code, which isn't a
good schema to emulate. so, in its form I dont expect that specific code to
be merged.


Make the App's title optional on web

Title (in web) results in updating the title element which is  a global property. This is problematic in embedded and multiview modes as title should be managed by host apps. This PR makes the title optional, hence if not provided it won't result in the website title being updated.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] I followed the breaking change policy and added Data Driven Fixes where supported.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@yjbanov @Piinks do you happen to know why not instantiating the Title widgets leads to Navigator not being found in https://github.com/flutter/flutter/blob/master/packages/flutter/test/widgets/app_overrides_test.dart#L79?

do you happen to know why not instantiating the Title widget leads to Navigator not being found

I do not, maybe @chunhtai has some context?


in categories please get rid of the white cube

from the QA google sheet line 26,

link broken. Correct link (hope) very long link

might be similar to https://github.com/kiwicom/smart-faq/issues/402 but the other way around probs. Lemme take a look


Library not supporting TLS 1.2 on old Android Devices

I am using PRDownloader library for downloading images and other files in Android devices having OS 4.2.2. Our server is configured with TLS1.2 support. PRDownloader is not working with our Server.
I googled and found older Android does not support TLS 1.2. But there is an alternative to fix this.  links are

https://blog.dev-area.net/2015/08/13/android-4-1-enable-tls-1-1-and-tls-1-2/
https://stackoverflow.com/questions/28943660/how-to-enable-tls-1-2-support-in-an-android-application-running-on-android-4-1

for example using HttpsURLConnection we can set custom sslfactory as shown below.
HttpsURLConnection.setDefaultSSLSocketFactory(new MySSLSocketFactory());
But there is no option to set such sslfactory in PRDownloader libarary.
Please help me with how to support TLS1.2 in PRDownloader Library.

any updates?


ref(*): remove etcd now that all state is being handled by db/k8s

This switches confd to source in env var for gunicorn and a few other minor things, for now
Following items need to be figured out and #9 touches on some of that

[x] etcd_set_default protocol "${DEIS_PROTOCOL:-http}"
[x] etcd_set_default secretKey "${DEIS_SECRET_KEY:-$(openssl rand -base64 64 | tr -d '\n')}"
[x] etcd_set_default builderKey "${DEIS_BUILDER_KEY:-$(openssl rand -base64 64 | tr -d '\n')}"
[x] Builder reading app names, users and keys from an alternative location

Depends on #242 as it ditches one of the etcd requirement that is embedded in the models

SECRET_KEY needs to be set (https://docs.djangoproject.com/en/1.9/ref/settings/#secret-key) but we shouldn't be publishing it anywhere really

Will read the secrets from https://github.com/deis/charts/pull/67

Was told the protocol is hard-coded to http right now so no need to try to solve for that until we tackle it platform wide

Will this PR allow us to remove etcd entirely?

Yeah

that's pretty :bomb:.com

+1 on everything except for the reliance on confd to template out envvars. We should just remove confd entirely and rely on os.getenv in settings.py much like how we do with the database credentials/config.

That's not as friendly when it comes to gunicorn - You'd have to cycle the full RC to set new worker count oppose to just dropping on the pod and setting a new env var. While that isn't exactly the cleanest you can edit the RC and do that without any real downtime, in theory, if confd works

@helgi got it. sounds like it'd be pretty hard to take out confd then. could take it out after we rewrite the controller in Go...
Aaron runs away

In theory, yes. In practice, is there actually a way we can inject environment variables into the running processes through kubectl? If not then I'd still be in favour of removing confd since we wouldn't be able to do live reloads, simplifies the codebase and reduces the image size by a few megs.

I'll leave that for another PR - Right now confd also writes out a specific file with "prod" configuration, which doesn't happen on testing so more work would be required than to simply ripping out confd and making things use env vars via pythong

Code LGTM.
Tested with example-go and node app.


Add console commands to reroll a specific recipe

Sometimes you get a seed that feels great to play, until you hit that one recipe which is just really annoying. Alternatively, the odd softlock may have slipped through the cracks and prevent a mid-late game from concluding. In these cases, being able to reroll the offending item's recipe only would really help.
The biggest problem with implementing this is the fact that a post factum reroll no longer has access to all the logic that went into the seed. While the command could make an educated guess about accessible items based on the depth difficulty of the item, it still cannot do much more than an almost full random reroll.

Considering the way randomisation builds on itself in the randomiser, a satisfactory implementation of this command does not seem feasible.


Update progress bar and notifications

it is necessary to assemble all the components according to the notifications at the bottom of the plugin and update them to ant.design

#584 main task

progress bar prototype click to start
figma link
component link
ant link

figma link

Убираем successfully везде!

notifications are reworked in 3.2.272
task was decomposed and progress bar will be updated in #726


docker-compose: change overview from /compose/overview to /compose/

the /compose/ (index page) didn't contain any useful information, and wasn't included in the navigation, so we might as well use "overview" as the landing page, which makes the URL also a bit more logical.

@chris-crone

Deploy preview for docsdocker ready!
Built with commit 9e0520055ac5f65dd2c3a708e5722f442b728973
https://deploy-preview-8939--docsdocker.netlify.com

SGTM, thanks @thaJeztah

@usha-mandya ptal

That makes sense. Thanks for looking into this @thaJeztah.


Merging upgraded Sdk Common Projects

includes upgraded Sdk Common projects
build changes
Sdk common tests passing locally

Force pushing this PR because we do not want to merge with existing assets.
This push will be a baseline for future PRs


Mouseover on Nodes in scope taking time to respond

browser: Chrome Version 56.0.2924.87 (64-bit)
I also installed the new chrome garafana plugin, and not sure if this has anything to do with it.
( ID: aihaocdgpjomchhocbnlhoaildnoollo
Inspect views: background.html (Inactive)))
Running one of the katacodas with kubeadm and displaying it cloud.  The browser crashed two times (but I am not able to reproduce this).   The nodes take some time to respond when mousing over, particularly when the details panel is displayed, although this is not always the case.
Here's a video:
http://recordit.co/ZBpez2YZfz

moved to service ui


array3d test

Not producing same output as Java and C++.   Related to issue on dimensions and subclassing.

I have fixed the array indexing issues.  When referring to an array element, you alwasy use the slowest varying index first, and so on .
A[i][j][k] -- i  is the slice, j is row, k is column
I think this issue is fixed in both the tests and the Array.py in t he client and is consistent with C++ and Java


icon.getImageSource() uses old cache even when icons are updated

[x] Review the documentation: https://github.com/oblador/react-native-vector-icons
[x] Search for existing issues (including closed issues): https://github.com/oblador/react-native-vector-icons/issues

Environment: Android
Happens on both the emulator and real devices
Description
Custom font using icomoon. Icons rendered using the source:
const CustomIcons = createIconSetFromIcoMoon(selection);
const source = CustomIcons.getImageSource('icon-name', 30);

source has a uri like<EMAIL_ADDRESS>source can be used to render an icon like: <Image source={source} />
All this works fine on a clean install but there are issues when the icons are updated...
REPRO STEPS:

Clean install app on device
Open app view icons, everything looks good
Update/ change/ delete some icons from a custom font
Run the app again with react-native run-android or installing an APK update
View the icons and some of the icons are incorrect

Changing the size in image get source for the broken icons after the update fixes them AKA:
icon.getImageSource('icon-name', 29);
This is because it changes the file name to<EMAIL_ADDRESS>thus it doesn't already exist in the cache.
Rendering the icons directly from the icon set like: <CustomIcons name="icon-name" size={30} /> works fine regardless of size being changed or not.
But for our use case it is necessary to get the icon image sources but this seems to be broken.
If you uninstall or clear the app cache the icons behave normally/ are rendered correctly.
If there is a way I can clear out the icon cache or delete specific files from the cache I could create a work around for this problem myself but I am also unsure of how to do this in react native.

I was having the same issue.  I solved it by using react-native-fs to delete the cached images:
import RNFS from 'react-native-fs';

const removeAllCachedPng = async () => {
  const cacheFiles = await RNFS.readDir(RNFS.CachesDirectoryPath);

  // remove all pngs from cache, this should purge all existing icons, because
  // this breaks if the icons update otherwise.
  return Promise.all(
    cacheFiles.filter(f => f.name.endsWith('.png')).map(
      f => RNFS.unlink(f.path)
    )
  );
};

I'm still experiencing this issue. I've created a repro here. Ref: https://github.com/oblador/react-native-vector-icons/issues/1205


fix: added logging configuration to autoscaling lambda to ensure it l…

…ogs to created log group

According to https://github.com/aws-cloudformation/cfn-lint/issues/2958, the E3002 for LoggingConfig seems to be a known issue that can be fixed but updating the cfn lint. The mitigation suggests to use either cfn-lint version v0.83.4 or use cfn-lint --update-specs to update it and try. @smone77 could you please help with updating the cfn-lint version.

@anichari fixed lint checks