ArangoDB start with systemd resulting in "No space left on device" error

My Environment

ArangoDB Version:        3.4.1 (Installed using rpm, Centos 7)
Storage Engine:             RocksDB
Deployment Mode:       Single Server
Deployment Strategy:   ArangoDB Starter
Configuration:               default conf with exception in stats=false and auth=false
Infrastructure:               own
Operating System:        centos 7
Total RAM in your machine:       16GB in VM.
Disks in use:        NFS
Used Package:            RedHat  Centos .rpm

Component, Query & Data : N/A
Affected feature: ArangoDB Start
AQL query (if applicable):
AQL explain (if applicable):

Dataset:

Size of your Dataset on disk:

Replication Factor & Number of Shards (Cluster only):

Steps to reproduce

Start any process that uses 8529 port. In my case I've started a python HTTPserver

python -m SimpleHTTPServer 8529

Now start the arangodb using the command:
[root@dbtest ~]# service arangodb3 start
Redirecting to /bin/systemctl start arangodb3.service
[root@dbtest ~]#
This command tries to start the arangodb but it doesn't say whether it is successfully started or not. But in /var/log/messages it says "Port Already InUse" error.

Problem:
I've installed systemd in my VM and because of it the arangodb3 process was keep on trying to start and it is keep on failing with "Port Already In Use" error. For each and every try to start arangodb, the memory space is getting increased in "/var/lib/arangodb3/engine-rocksdb/journals" and at some point of time it started throwing "No space left on device"
Aug 20 13:15:03 dbtest arangod: 2019-08-20T13:15:03Z [26657] ERROR cannot write to json file '/var/lib/arangodb3/VERSION-1.tmp': No space left on device
Aug 20 13:15:03 dbtest arangod: 2019-08-20T13:15:03Z [26657] ERROR {startup} writing VERSION file '/var/lib/arangodb3/VERSION-1' failed: No space left on device
Aug 20 13:15:03 dbtest arangod: 2019-08-20T13:15:03Z [26657] FATAL Database '_system' initialization failed (No space left on device). Please inspect the logs from the initialization procedure and try starting the server again.
Expected result:
If system is not able to start the arangodb3 why it is blocking the space in "'/var/lib/arangodb3" folder.

Hi @arunviswa,
Thanks for reporting. I was able to reproduce this issue by blocking the default port 8529 and starting the arangodb3 service as described (on Ubuntu 18.04 installing the 3.4.7 + 3.5 RC7 debian package).
3.4.7 + 3.5 RC7:
New log files of size 71 MB are frequently created in the /var/lib/arangodb3/engine-rocksdb directory when port 8529 is blocked. The directory size is increasing fast at a rate of 740 MB/minute.
3.3.24:
New log files in the /var/lib/arangodb3/rocksdb directory are created when port 8529 is blocked, but at a much lower rate of 1.6 MB/minute.


Support Python 3

What does this PR do?
This makes HA Proxy support Python 3.
Motivation
all aboard! 🐍 3⃣ 🚂
Review checklist

[x] PR has a meaningful title or PR has the no-changelog label attached
[x] Feature or bugfix has tests
[x] Git history is clean
[ ] If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo
[ ] If PR adds a configuration option, it has been added to the configuration file.

Additional Notes
Anything else we should know when reviewing?

Codecov Report

Merging #2849 into master will increase coverage by 9.33%.
The diff coverage is 85.29%.

@@            Coverage Diff             @@
##           master    #2849      +/-   ##
==========================================
+ Coverage    84.5%   93.84%   +9.33%
==========================================
  Files         631        7     -624
  Lines       36842      763   -36079
  Branches     4402      110    -4292
==========================================
- Hits        31134      716   -30418
+ Misses       4403       32    -4371
+ Partials     1305       15    -1290


[QUERY] How can I configure timeout while using ContainerRegistryManagementClient

Library name and version
Microsoft.Azure.Management.ContainerRegistry 6.0.0
Query/Question
We migrated our services from Net4.7.2 to dotnet core 6.0. We see the error "The request was cancelled due to the configured HttpClient.Timeout of 100 seconds elapsing."
The Kubernetes pod running this service has enough memory and CPU. The network does not seem to be an issue as the Kubernetes cluster in running on Azure and this client is trying to reach ARM.  Also, during the same time, the service can reach storage accounts. Also this is happening in multiple regions.
I want to add a retry logic in our code. For this I want to decrease the timeout from 100sec to 15sec, but I could not find timeout setting exposed. Do I have to create my own HttpClient?
My code is really simple
var containerRegistryManagementClient = ContainerRegistryManagementClient(resourceManagerUri, tokenCredentials) { SubscriptionId = subscriptionId };
var registry = await containerRegistryManagementClient.GetRegistryPropertiesAsync(
                subscriptionId: resource.SubscriptionId,
                resourceGroupName: resource.ResourceGroupName,
                registryName: resource.ResourceName,
                correlationId: correlationId);

Environment
No response

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Hey @jikuma the SDK you're using is being deprecated soon, please move to our new Azure Sdk for Container Register, and here is doc for how to config the retry policy by using the new SDK.


Proxy URL Rewriting

Is it possible to rewrite links on a page in a similar way to how Browsersync does it?
Example:
PHP server: site.localhost
webpack-dev-server: site.localhost:8080 (proxy site.localhost)
Right now when I load up site.localhost:8080, it displays absolute links on a page as site.localhost/link.html. Browsersync somehow automatically rewrites all those to site.localhost:8080/link.html so navigating pages keeps you running through webpack dev server.
Is something like that possible using this middleware? Tried a few things with onProxyRes, but I'm not sure if that can support what I'm looking for. Thanks!

Was able to find a solution at https://github.com/chimurai/http-proxy-middleware/issues/97#issuecomment-609891024 for anyone interested.


Global alert should be able to show errorMessage for custom alert

Fixed in https://github.com/near/near-wallet/pull/1746


Fix test

Error build.
import into a class of a non-existing assembly - "import static com.synq.testprimitive.TestUtil.sameInstant;"

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:testCompile (default-testCompile) on project testprimitive: Compilation failure
[ERROR] /var/lib/jenkins/workspace/generated-applications/Testprimitive/src/test/java/com/synq/testprimitive/web/rest/TestprimitiveResourceTest.java:[29,1] cannot find symbol
[ERROR]   symbol:   static sameInstant
[ERROR]   location: class com.synq.testprimitive.TestUtil

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Alexey Nesterenko 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.

I accepted license the CLA.


Update _index.md

Added my contributions. Because it's a long list, I moved my name to the end (to be fair to the other contributors).

I would propose the following as contributions:

https://github.com/civier
Funding: ARDC Australian Electrophysiology Data Analytics PlaTform (AEDAPT) (CI for $566k AUD) - contributing to initial conceptualisation, EOI writeup, scope of project, proposal writeup, teaming up with the Australian Imaging Service (AIS), recruiting collaborators, assisting collaborators with case studies
Design: piloting and evaluating existing solutions (e.g., CVL/ACCS)
Design: leading the Virtual Neuro Machine (VNM) hackathon project in the 2020 OHBM BrainHack, where the first version of Neurodesktop was developed
Development: allowing Neurodesk containers running in Neurodesktop to access sshfs mounts
Development: template for container recipe documentation
Development: software application containers - developer (bidscoin, MATLAB; in progress: MMVT)
Development: software application containers - facilitator (Fieldtrip, running arbitrary scripts using compiled MATLAB containers; in progress: SOVABIDS)
Documentation: for developers (contribution to "add tools", configuring Github)
Documentation: for users (copy and paste troubleshooting, accessing storage, installation on different platforms, using ARDC Virtual Desktop Service, screenshots)
Documentation: Neurodesk's original logo
User testing: HPC, NECTAR, Mac, Linux, ARDC Virtual Desktop Service
User testing: VNC and RDP interfaces, including multiple concurrent users
User testing: running containers as sandboxes on HPC
User testing: running Neurodesktop for long periods
Outreach: initiating a user survey and analysing its results (2020)
Outreach: taking part in a presentation for the Australian National Imaging Facility 2020
Outreach: leading the AEDAPT project in BrainHack Global 2021
Outreach: contributing to the ARDC TechTalk 2022 "Neurodesk and Neurodesktop: Software containers and CVMFS for low-maintenance, application-rich and FAIR desktop environment on PC, cloud or HPC"
Outreach: presentations in conferences (2nd author eResearch 2020; 2nd author OHBM 2021; co-author eResearch 2021; co-author Australasian Cognitive Neuroscience Society 2021)
Outreach: providing assistance to nodes of the Australian National Imaging Facility with installing/using Neurodesk
Administration: managing NECTAR resources allocated to the AEDAPT project
Administration: maintaining contacts with AEDAPT project collaborators
Administration: one of Neurodesk/AEDAPT representatives in ARCOS, AIS and NECTAR Interactive Analytics committees and working groups
Papers: co-author Neurodesk manuscript (in preparation; contribution to initial outline, input on first draft), co-author proceedings of the OHBM Brainhack 2021 (to be published in Aperture)

This is a different template for contributions compared to the existing one.
If this format is being proposed, I think we need to do this as a template change for the contributors page.

We can set guidelines to maintain the template and formatting of the contributors page.
I propose we impose the following. Each contributor entry consists of

Name
Github handle (or alternative org if unavailable)
Contribution description to Neurodesk as bullet point list limited to
a. 12 bullet points
b. 120 words

And we'll sort the contributors on the page how its currently sorted - based on the metrics on the github insights
@civier @stebo85 @TomEmotion
Does this sound like a suitable set up?
If this is okay, I can update the contributors page with these guidelines.

@civier - many of these are not quite true and vastly overstate your contribution - are you sure about this? If this is how you perceive your contributions I think we should discuss in person to clarify.
@aswinnarayanan, @TomEmotion what do you think?

Dear @stebo85,
This is indeed how I perceive my contributions, and I do not agree with your statement that many of the things I listed are not quite true and vastly overstated. I will be happy to continue discussing it, but I'm afraid it will have to wait until Tom has time to attend the matter. He has a lot on his plate when returning from leave next week, so that might take some time.

I am closing this pull request here and I added a contributions list that summarizes the important and relevant contributions from my perspective to the project trying to stay within reasonable length limits.


Issue/89

PR for https://github.com/SwiftWeekly/swiftweekly.github.io/issues/314
@SwiftWeekly/writers feel free to add your notes

Thanks @RomanVolkov ! 🎉 Awesome job 👍


Database Providers Capability

Ability to add other database providers like postgres, sqlite, mysql, oracle, etc.
I started down the path of a SQLite Management Studio that looks almost identical to SQL Operations Studio. I don't see a point in continuing that work when this exists. I should be able to just write a provider to connect to sqlite that works within this tool.

@JoshClose I'm going to close this as a dupe of #56 which is on our radar and should capture this feature request.  Feel free to reopen if you don't agree.

No problem! I did a search for a few key words that apparently weren't in that issue. ;) Thanks!


Central Style guide

Hi.
I've been working on the Department for Education School Perf. Tables (Alpha now moving to Beta) and wanted to ask if there is a central style guide for all the gov uk components etc.
So far I've found style guides like this http://rural-payments-styleguide.herokuapp.com/elements/snippets/#layout which gives us a lot structure but no skin  separation and no javascript examples plus its navigation is a bit painful for the less front end savy dev's.
I come from an OOCSS background where structure and skin is separated. We have three projects here that we want to skin but maintain a gov uk structure plus we have extra feature rich components like maps, mobile swipe interactions etc etc.
I'm now looking to build a style guide during our beta journey that gives the less front end savy developers a easy way to just grab html, js etc.. and get on. So what I'm trying to find is a one stop shop that gives us the following ability:

a style guide build tool process (like hologram) lets us integrate our skins with the gov uk sass components and generate a style guide which we host on each published DFE website
The style guide should contain overview\description of each (most) components, html and js examples etc..
simple navigation for less front end savy dev's to find what they need

Is there any such place the fulfils this?

hi Peter
We've been talking recently about the best way to support developers and designers, so it's helpful to know that you've been thinking about it too.
Two of the key people are away next week but I'll keep a note of this and will remember to raise it with them on their return.

Cool thanks

Hi @pflannery, are you happy to close this issue?

I'm happy to close.


Vimeo web component is not being defined

Hi there, really odd issue: I'm using the <Vimeo /> component, and while it is rendered in the DOM tree, I think the lite-vimeo web component is never defined.

When I run customElements.get("lite-vimeo") in the console, I get back undefined. This means that it's never rendered. However, I checked the Network tab and the thumbnail and everything is being downloaded. Any ideas?

Hi @aczw! Could you share a reproduction? The <Vimeo> component definitely registers the custom element:
https://github.com/delucis/astro-embed/blob/9d0c83cb875a263d218520af6633f5391e82de18/packages/astro-embed-vimeo/Vimeo.astro#L160-L163
And you can see it working in the docs too. So it might be something specific to your usage/project?

Hi again @aczw — given we don’t have a reproduction, I’m closing this issue. Feel free to reopen or make a new issue if you can share a reproduction.

Ah sorry about that, life has gotten in the way. I'll def let you know if I can reproduce it again in the future.


Target server import works correctly but returns an error

Hi,
When importing a target server, I see that the target server was correctly created in Apigee, but I get the following error:
Error: could not import targetserver, apigee responded with HTTP 200: 200 OK
After checking the source code, I noticed this condition that I think is causing the issue:
https://github.com/apigee/apigeecli/blob/553d525536b71636e79e181dd38571d6959ea7fa/client/targetservers/targetservers.go#L366
Thank you in advance for your time!

This is a case where the import of the target server failed because a) the input was invalid b) the target server already exists. Since you get a 200 OK, I think this is a case where the target server already exists. So it is safe to ignore.
@Helcaraxan FYI

This was a case where the target server didn't already exist. A 200 OK response will always trigger an error because the rest of dividing 200 by 100 will be 0 and not 2.

I see your point. I will create a fix for this. Thanks for identifying the issue.

I have submitted a potential fix. Once @Helcaraxan reviews it, I will merge and release a new version.

Apologies for this issue @ana-r-goncalves-alb.
All good on the PR @srinandan. Merge away.

Release v1.119 is published now. Please re-open if there is still an issue.


Don't validate removed inputs in the DOM

Fixes this kind of error where a required input is empty but also hidden because the element is destroyed (hidden) in the DOM.

Great, thanks!


Database integration

Epic
Integrate usage of MongoDB into react code.
User Stories

[x] As a user, I should be registered in the SpotiFry database when I login for the first time.
[x] As a user, each of my newly created playlists should be stored by Spotifry.
[x] As a user, I should be able to view my past custom playlists through SpotiFry.

All generated playlists are stored in the data base
Last five playlists now can be viewed at homepage


Preprocessing steps need reordering

Description of the bug
The current placement of strand inference before other processing creates difficulties, for example with short and/or adapter-heavy reads which don't align until they've been trimmed and filtered.
Command used and terminal output
No response
Relevant files
No response
System information
No response

Should be fixed in https://github.com/nf-core/rnaseq/pull/1154


FID's applicability for smaller datasets

Hi,
I have a few questions about FID score:

I have a dataset smaller than 2048 images, but I still want to compute FID score. I understand that 2048 images are required as to get a full rank covariance matrix. Computing the FID on my dataset still gives me sensible values, and no complex numbers or NaNs (or warnings). Can I still trust the FID score computed this way as a measure of visual quality?

In this paper, section 5.1, it is noted:

We observe that FID has rather high bias, but small variance, From this perspective, estimating the full covariance matrix might be unnecessary and counter-productive, and a constrained version might suffice.

What is your take on this? Does this hint to FID being usable with smaller datasets?

Do think a FID computed on lower level features maps of Inception is still meaningful? As the features at lower levels still have spatial extent, spatial pooling would have to be applied first. My thought here is to try to make FID work with smaller datasets.

I also ported your implementation to PyTorch, for people who do not want to have the Tensorflow dependency (see here). I hope that is okay with you.
Thanks!

@mseitzer this is an interesting question -- have you had any chance to think about this or apply it to any of your results with sample sizes < 2048?

For small sample sizes another option is to train an autoencoder and use the statistics (mean, covariance) of the coding/bottleneck layer for the FID.

@nickkimer I do not work on this anymore unfortunately.
One experiment someone could do is to measure the sensitivity of FID to the dataset size. So compute FID on differently sized subsets of the same dataset, and see how FID varies. Repeating this comparison with different datasets should at least give some level of insight to this question.

@mhex  Do you have some reference about your suggestions? I have a new and small dataset, therefore, I am planning to write a customized assessments methods. However, from my current understanding, it's not easy to directly rely on the autoencoder.


UI/UX of all calculators are modified

Finally, UI of all calculators is modified and are set to a particular theme. This pull request is related to the issue #28
Screenshots of newly modified calculators are attached herewith:

Thank you so much @singhkaran202 !!


Tcl6 64 bit userspace plus vmware

this will become the vmware follow-on after #854 is merged.
@frapposelli or you can build your own PR once I've removed the vmware stuff from #854

na, its all intertwined - i didn't realise OVT had been merged already.


Dropdowns not dropfrowns

fixing a bunch of small things in #249

adds full search functionality for all fields
removes full_name field and creates first_name and last_name fields in index search
with the above, we can now carry over information into the new_patient route, which is complete (methinks) and has been reflected in forms.py and views.py

:frowning:

:frowning: TURN THAT FROWN UPSIDE DOWN :smile:


⚠ ClamAV has degraded performance

In 0bc35af, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 254 ms

Resolved: ClamAV performance has improved in f5011e4 after 21 minutes.


feat(components): add useConditionalConfirm hook to shared components

overview
This PR moves the useConditionalConfirm hook used in PD to shared components, as well as adds test coverage around it. Closes #5589
useConditionalConfirm is intended for cases where we want to block and defer a particular user action until the user clicks "ok" in any kind of "are you sure?"
See useConditionalConfirm.js for an example of how this hook works.
changelog

Add unit test coverage for useConditionalConfirm
Move hook from PD to shared components

review requests
Code review — I've never written a test for a hook like this before so please take an extra careful look at the test
@IanLondon make sure that TerminalItem and StepItem work as they should (they use the hook for rendering ConfirmDeleteStepModal).
risk assessment
Low

Oop sorry @mcous forgot to add the comments in JSDoc but just did!


No references to getting help or support

vim-prettier 0.2.3
I have installed and using it now, thank you for the plugin.
doc\prettier.txt makes no mention on how to ask for help or support on the plugin.
No website given or email address supplied.
The only way I found this website was opening the changelog.md.
Most plugin authors put headers at the top of each of the files with something like this:
" vim-prettier.vim - A vim plugin wrapper for prettier
" ---------------------------------------------------------------
" Version:       0.2.3
" Author:    Whatever <you.want at gmail dot com>
" Last Modified: 2017 Aug 13
" Created:       2017-05-24
" Homepage:      http://www.vim.org/scripts/script.php?script_id=5572
" Contributors:  Blah...
"                Blah
"
" Help:         :h vim-prettier.txt

This helps people find support as well as when I prune files from my plugins directory (if the filename wasn't obvious) I know which plugin it was for.
And of course doc\vim-prettier.txt should have similar information or a section for getting help / support / kudos and so on.

Sorry, I see you have this section in plugin\prettier.vim:
"==========================================================================================================
" Name Of File: prettier.vim
"  Description: A vim plugin wrapper for prettier, pre-configured with custom default prettier settings.

Comment still applies (all files), but sorry for not noticing this first.

Hi @dfishburn,
Will include the above comment on all the other files. Also accepting a PR.
Thanks for reporting it :)

You might also want to reference the github link in the description of your plugin.  Many other people do this.


GN-4425: Smarter content wrapping

Overview

Be smarter about how we approach attempt to wrap structured nodes. More comments in the code.
Connected issues and PRs:
https://binnenland.atlassian.net/browse/GN-4425
Setup

Checkout
npm run start
Go to Regulatory Statement Sample

How to test/reproduce
Insert article, article, chapter, title, document title
https://github.com/lblod/ember-rdfa-editor-lblod-plugins/assets/769698/172d8c23-f316-4eae-bf22-b2ee94c574f1
Document title, article, article, title, chapter, chapter
https://github.com/lblod/ember-rdfa-editor-lblod-plugins/assets/769698/5516b3de-9c8c-4856-a9ed-18b465c9a9d4
Challenges/uncertainties

Checks PR readiness

[x] UI: works on smaller screen sizes
[x] UI: feedback for any loading/error states
[x] Check if dummy app is correctly updated
[x] Check cancel/go-back flows
[x] changelog
[x] npm lint
[x] no new deprecations

why draft? what still needs to happen?


remove all the join('')

It'd be nice if redraft worked with string returning functions rather than requiring a bunch of join and recursive join functions tacked on. Happy to investigate this if there is interest.

Main focus of redraft always was rendering to directly to React components. Handling strings in nested arrays may be bit awkward if you need a pure string output.
With this in mind I don't think there's real reason to change the default output. What's possible is to add a flag to merge the output to a single string, something like:
redraft(raw, renderers, { mergeOutput: true })

This would have to assume that all your rendering functions return strings.

Hey, I went with:
redraft(raw, renderers, { joinOutput: true })

It joins the output and the children of all the inline and entity renderers, you still have to join the block level children seeing as a single child there can be for example a separate <li> tag. Also mind you have to make sure all the renderers return strings it you plan to use this option.

Rendering to string should be much simpler now, while you need to join the block level output it wont receive any nested arrays so a single join('') should be enough. Currently I don't see much more could be done with this, so I'm closing this.


UNDERTOW-1227 Proposed fix

This fix was tested with a simple spring-boot application that defines a POST controller accepting multi-part requests. No additional unit tests were written.
I implemented a generic type for holding the getParts() and parseFormData() methods output, holding the raised exception when appeared, or the actual result.

Can one of the admins verify this patch?

This is ok to test

Thanks for the PR, however I have fixed this a different way (and also fixed the other cases that should throw IllegalStateException)

Hi @stuartwdouglas & @sermojohn I have arrived at this PR from this Stack Overflow issue. I see that the underlying issue UNDERTOW-1227 has been resolved in 1.4.22.Final with a slightly different change to the PR then what was presented here.
After upgrading to Spring Boot version 1.5.10.RELEASE and associated embedded undertow version I am still unable to catch the java.lang.IllegalStateException: io.undertow.server.RequestTooBigException: UT000020: Connection terminated as request was larger than 15728640 inside of a @ControllerAdvice is this to be expected?

Hi @stuartwdouglas & @sermojohn I have arrived at this PR from this Stack Overflow issue. I see that the underlying issue UNDERTOW-1227 has been resolved in 1.4.22.Final with a slightly different change to the PR then what was presented here.
After upgrading to Spring Boot version 1.5.10.RELEASE and associated embedded undertow version I am still unable to catch the java.lang.IllegalStateException: io.undertow.server.RequestTooBigException: UT000020: Connection terminated as request was larger than 15728640 inside of a @ControllerAdvice is this to be expected?
Basically what I am keen to do is to return a 400 Bad Request to the user when they attempt to upload a file which exceeds the maximum file size rather then simply closing the underlying connection which seem to be the case currently.

Facing the same issue with spring-boot


Delegate complexity to quarkus-groovy

A new extension has been added to the Quarkiverse that provides the support of Groovy in native mode, the code base could be simplified by delegating the complexity related to the native mode to the extension.

Depends on https://github.com/apache/camel-quarkus/issues/4960

Nice @essobedo !


Seperate Caching Redis from Webhooks Redis

Environment

Python version:  3.6
NetBox version:  2.6

Proposed Functionality
Having the ability to configure a separate Redis connection for the caching purpose and one for the webhooks delivery purpose.

Use Case
The Redis required for caching has different requirements regarding data persistency than the Redis for webhook.
While it doesn't matter for the caching Redis if the data is lost, I would try to avoid data loss for the webhooks Redis.
They also have two different access patterns: One is only ever accessed by one client, i.e. my Netbox instance, while the other is accessed by potentially many clients, i.e. more than one webhook worker.
And in a container-world this matters: The caching Redis would be an instance that is deployed as a sidecar to the Netbox instance, i.e. in the same pod. The webhooks Redis is a shared instance and would therefore be deployed on it's own, potentially with slaves for redundancy purposes, and definitely have some kind of persistent storage attached.
Therefore it would be nice if I could configure two Redis instances in Netbox, one for caching and one for delivering webhook invocations.

Database Changes
Not necessary

External Dependencies
None

I could see us mimicking the Django DATABASE dict in which several configurations can be defined.
Something like:
REDIS = {
    'webhooks': {
        ...
    },
    'caching': {
        ...
    }
}

The actual connection logic is already duplicated in that we open separate connections for cacheops and rq, so that won't require any additional overhead.


Added Parameters section and converted Usage to Examples

Closes #490

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/638cfd978bdca9429cc8909bcd93dd614777f943/

rename-table.md, line 34 at r1 (raw file):

### Rename a table

To rename a table, use the `ALTER TABLE` statement followed by the current table name in `database.table` format, the `RENAME TO` statement, and the new table name in `database.table` format:

As discussed, we probably don't need this introductory text.

Comments from Reviewable

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/5fefa81473d0c9ed15461d98ed181f7585432d75/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/0f9462163abe8ba18a39ff349661297cd76c784a/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/59e7f2ad5664881633c10c92db7bc2b2b7ab7066/

@Amruta-Ranade, I just pushed a fix to main.go and also regenerated the rename table diagram. The command you used didn't seem to regenerate it properly. We can look into this together more later.
Once all tests pass, if the rendered page looks good, I'll go ahead and merge. Thanks for this PR!

I just regenerated the diagram as well. The rendered page is updated with the new diagram now.

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/4e5cda8a5c8b2127f99738105b8d8baaac6292d8/


Unexpected input(s) 'recursive', 'recursive-path'

Describe the bug
Adding 'recursive' and 'recursive-path' to the workflow file results in the following error:
Warning: Unexpected input(s) 'recursive', 'recursive-path', valid inputs are ['entryPoint', 'args', 'config-file', 'working-dir', 'atlantis-file', 'find-dir', 'output-format', 'output-method', 'output-file', 'template', 'indention', 'git-push', 'git-push-user-name', 'git-push-user-email', 'git-commit-message', 'git-push-sign-off', 'fail-on-diff']
Seems like they might not be included in v0.11.0 even though the examples make it seem like they are. Is that intended?

How can we reproduce it?
- name: Render Terraform Docs (Recursive)
  uses<EMAIL_ADDRESS>  with:
    working-dir: .
    output-file: README.md
    output-method: inject
    recursive: true
    recursive-path: modules

Environment information
Github Actions

Sorry for the long delay, this feature has been added in v1.0.0 which just was released.

Works great now, thank you!


Bump Go version to 1.19

Description of your changes

Bumps Go version to 1.19 consuming the version update in upjet. Previously with Go 1.18, make lint has been failing in the CI. This is an attempt to observe whether these are related.
I have:

[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Closing in favor of https://github.com/upbound/provider-gcp/pull/29


Custom skill which is deployed - how to use in AVS SDK or Smart Screen SDK

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:
I have few custom skills with custom utterances which are already deployed . How do I use these skills in AVS Ubuntu or Smart Screen SDK.
What is the expected behavior?
Should return the expected result
What behavior are you observing?
Not responding to that question.
Provide the steps to reproduce the issue, if applicable:
When you test AVS by tapping t + enter, try calling the custom skill utterances or when you test smart screen tap A and speak by calling the custom skill utterances
Tell us about your environment:
Ubuntu or Raspberry Pi 4
What version of the AVS Device SDK are you using?
  <1.25.0>

Tell us what hardware you're using:

[ ] Desktop / Laptop
[ ] Raspberry Pi
[ ] Other - tell us more:

Raspberry Pi 4 , Ubuntu
Tell us about your OS (Type & version):

[ ] Linux
[ ] MacOS
[ ] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

Hello @kaygeeyem16
Were the instructions provided to test your custom skills useful and resolve your question?

Closing due to inactivity. Please re-open if necessary.


Add symfony/config compiler to adapters

@Ph3nol could you please add some description here?

I was talking about adding a symfony/config nodes configuration to adapters in order to check, prepare and validate parameters data.
Some examples: https://symfony.com/doc/current/components/config/definition.html
It's not a "must", but a way to do this.


Enhances new command template, Closes #4135

This PR enhances the new command template which allows the input of default properties.
Closes #4135

Thank you, we'll review it ASAP!


Fast scrollbar?

The side scrollbar in the recycler gridview does not do a fast scroll. Can we implement something like a fast scroll bar where a user can hold the scroll bar and scroll fast if they want to?

https://stackoverflow.com/questions/27798212/how-to-add-a-fast-scroller-to-the-recyclerview
This looks promising

No, it's a bit complicated to explain. The dashboard uses swipe refresh layout and app bar scroll, it will needs a nested scrolling. But with nested scrolling enabled fast scroller is a bit  weird, let's say it's not working properly.


mindspore

文件在mindspore文件夹下，readme中暂时提供百度网盘链接

Merge code about the support of  MindSpore, thanks Ruiqi @RuiqiWang00  for this contribution!


Add Gzip

Adds Gzip decoder to Ciphey and a test for it.

@SkeletalDemise @bee-san
Thank you for your greate job, I am kind of love decrypt now.


Move Docker rtsp-simple-server.yml config file from root path to a sub-path

Describe the feature
Currently rtsp-simple-server.yml resides at the root of the docker image which means if you want to override it with a local file you have to mount just the file.  This does not work if your files are on a volume like NFS since you can not "mount" a single file.  If this file was moved to a sub folder by itself then it would allow using NFS volumes for the config file.

You can mount anywhere and pass the yml config file path as argument to the executable.

Just use the arg parameter and point to the desired configuration file.

- apiVersion: v1
  kind: Deployment
  metadata:
    name: videoproxy
  spec:
    replicas: 1
    strategy:
      type: Recreate
    selector:
      matchLabels:
        app: videoproxy
    template:
      metadata:
        labels:
          app: videoproxy
      spec:
        containers:
        - name: main
          image: aler9/rtsp-simple-server
          args: ["/mynfsfolder/rtsp-simple-server.yml"]


feat: support uds

What this PR does:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

1.7.0


xmlnotepad: Add version <IP_ADDRESS>

A XML editor provided by Microsoft.
Homepage: https://microsoft.github.io/XmlNotepad/

[x] I have read the Contributing Guide.

Anything to persist? Settings/preferences.

Settings are stored in \AppData\Local\Microsoft\Xml Notepad


Manually block display from sleeping

From the TODO list:

Allow toggling/setting/resetting the "Manually block display from sleeping". It's an option available on KDE, I'm not sure if applications can access that.

You can't access the KDE applet's own "Manually block" toggle, that particular inhibition belongs to the applet itself. What you can do though is to add your own manual inhibition on top of that.  This will work in KDE, GNOME, and many other environments.

Thank you! Very much appreciated! Whenever I decide to implement it, it will be helpful!
Meanwhile, I've been using this project as-is for a few months. I use a few MIDI knobs to adjust the volume of each of the sound devices (HDMI output, analog headphones, bluetooth headphones), and another knob to adjust the display brightness+contrast. It's so useful that makes me wish computers had built-in analog knobs/sliders that we could program ourselves for different purposes.


add Issif as owner of the okta plugin for the PR approvals

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area plugins

/area registry

/area build

/area documentation

What this PR does / why we need it:
I'm the developer of this plugin, seems fair to me to be allowed to approve the PR from the community, especially because I'm still continue to develop/answer the issue/review the PR.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Issif
Once this PR has been reviewed and has the lgtm label, please assign mstemm for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

This is a majority vote as per our governance.
/hold until we get majority or a week has passed

and ofc big +1 from me :)

+1 @Issif! Thanks for the help.

LGTM label has been added.  Git tree hash: a574d253ea36fc7e5264f9942d5bc036188c2f19

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Issif, leogr, mstemm
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:

OWNERS [leogr,mstemm]

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


Focal Length in mm? Or Camera Sensor Size?

Hi, I'm doing 3D reconstruction tasks on Matterport3D. Can you provide me the focal length in mm or camera sensor size used in this dataset? I will be really helpful to me, thanks!

Upvoting this issue here. Would be really helpful if we have Focal Length in mm/FOV/Sensor Size!

Hi @YunfanZhang42. Thanks for upvoting, I just want to share some information I found that might be helpful to you. In the dataset organization section, it is mentioned that Matterport Pro camera consists of three Primesense Carmine. And some Primesense Carmine specs could be found at here. It looks like it is the camera spec used in dataset capturing, but I'm not sure.


Caching problem after initial routes scan in server-routes

Currently, in the server-routes, after the initial routes scan, any new routes created are not shown in server-routes
is it a good idea to add a clear-cache button so user can manually re-scan the routes or we could clear the cache whenever a new route is added(does nitro has a hook like that?)

Yes, we need to watcher to get the updates from the FS, which we haven't implemented, I think. PR is welcome if you want to work on it.
Meanwhile, @pi0 made the open API entry point in https://github.com/unjs/nitro/pull/1162, which we could leverage later.


counter example: write actual test on incrementIfOdd action

Now the test on incrementIfOdd doesn't test it actually. expectedActions can contain anything, but it will not be checked because store.dispatch is never called. So, it would be good to call some assertion to be sure that all actions had been matched.
Also, the verifyNoPendingActions helper can be useful in other tests to get more descriptive message rather than timeout of 2000ms exceeded

We should just use https://github.com/arnaudbenard/redux-mock-store.
Does it handle this case?

That's great, that it exists as a separate project, but it has almost the same code.
Also, they have an issue about it: https://github.com/arnaudbenard/redux-mock-store/issues/17
@gaearon what is your point about enhancing store API? Is it ok to have extra method verifyNoPendingActions for the mocked store? If it is, I can send a pull-request there, and then use it here instead of the current one.

I don't like that it's a manual operation. It would be best if we could use beforeEach() and afterEach() for that. This would involve changing the redux-mock-store API but IMO it would be worth it.

Another alternative is to always pass done to the mock store even for sync cases. Then the test that didn't dispatch will hang.

It would be best if we could use beforeEach() and afterEach() for that.

Of course. But it doesn't seem difficult if you have this method
var store;
it('is a test', () => {
})

afterEach(() => {
})

It would be best if we could use beforeEach() and afterEach() for that.

Of course. But it doesn't seem difficult if you have this method
var store;
it('is a test', () => {
})

afterEach(() => {
})

Then the test that didn't dispatch will hang.

That is the thing that I want to avoid. If you have a big project with many developers, you can encounter the error

It would be best if we could use beforeEach() and afterEach() for that.

Of course. But it doesn't seem difficult if you have this method
var store;
it('is a test', () => {
     store = mockStore()
     // ... testing ...
})

afterEach(() => {
    store.verifyNoPendingActions()
})

In the current master branch, these tests are gone. Closing this as not actual.


Import failure

Can’t import the module on z/os system, there is possibly something with the structure of the package that might need to be updated. This is the error message received.
>>> from pyzkiln_core import display_active_jobs
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/u/ibmuser/.local/lib/python3.9/site-packages/pyzkiln_core/__init__.py", line 11, in <module>
    from pyzkiln_core.zcmds import *
  File "/u/ibmuser/.local/lib/python3.9/site-packages/pyzkiln_core/zcmds/__init__.py", line 5, in <module>
    from pyzkiln_core.zcmds.display_active_jobs import *
  File "/u/ibmuser/.local/lib/python3.9/site-packages/pyzkiln_core/zcmds/display_active_jobs.py", line 2, in <module>
    from zoautil_py import opercmd
  File "/u/ibmuser/.local/lib/python3.9/site-packages/zoautil_py/opercmd.py", line 1, in <module>
    from zoautil_py.core import call_zoau_library
ImportError: CEE3512S An HFS load of module /u/ibmuser/.local/lib/python3.9/site-packages/zoautil_py/core.abi3.so failed. The system return code was<PHONE_NUMBER>; the reason code was 0BDF0C27. ```

@tim-odom It seems like the problem you are having is related to your ZOAU installation. Please check if ZOAU installation is correct, including all your environment variables. Also make sure you have the latest ZOAU version.

@luisamarti I already did that before I made the issue but it didn't change anything. Joe suggested I open the issue for it.

@tim-odom can you go into the directory mentioned in the error message , /u/ibmuser/.local/lib/python3.9/site-packages/zoautil_py , issue this command   ls -alT  , and send us the output

@tim-odom It seems like the file core.abi3.so  is tagged incorrectly. Can you change the tag to binary with this command chtag -b core.abi3.so and try again.

@luisamarti that worked but I seem to be getting a different error now that doesn't have any return codes
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/u/ibmuser/.local/lib/python3.9/site-packages/pyzkiln_core/__init__.py", line 11, in <module>
    from pyzkiln_core.zcmds import *
  File "/u/ibmuser/.local/lib/python3.9/site-packages/pyzkiln_core/zcmds/__init__.py", line 5, in <module>
    from pyzkiln_core.zcmds.display_active_jobs import *
  File "/u/ibmuser/.local/lib/python3.9/site-packages/pyzkiln_core/zcmds/display_active_jobs.py", line 2, in <module>
    from zoautil_py import opercmd
  File "/u/ibmuser/.local/lib/python3.9/site-packages/zoautil_py/opercmd.py", line 2, in <module>
    from zoautil_py.datasets import hlq as get_hlq
  File "/u/ibmuser/.local/lib/python3.9/site-packages/zoautil_py/datasets.py", line 8, in <module>
    import logging
  File "/usr/lpp/IBM/cyp/v3r9/pyz/lib/python3.9/logging/__init__.py", line 32, in <module>
    import sys, os, time, io, re, traceback, warnings, weakref, collections.abc
  File "/usr/lpp/IBM/cyp/v3r9/pyz/lib/python3.9/re.py", line 124, in <module>
    import enum
  File "/usr/lpp/IBM/cyp/v3r9/pyz/lib/python3.9/enum.py", line 2, in <module>
    from types import MappingProxyType, DynamicClassAttribute
ImportError: cannot import name 'MappingProxyType' from 'types' (/u/ibmuser/.local/lib/python3.9/site-packages/zoautil_py/types.py)


correct capitalization of clusterIP in an example

This change is

Thanks @robstarling  - if you could sign the CLA, this should be a quick PR to merge.

I've duplicated this in #6067.


Regenerate aws-sdk SQS interface (v1.28.0)

New SQS client version released from aws-sdk-go-v2 - v1.28.0
See full changelog.

Codecov Report

Merging #18 (96170ec) into main (8407361) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff            @@
##              main       #18   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines          408       408
=========================================
  Hits           408       408


Karma throws exception

Karma throws this exception launching tests with ts-mockito:
Error: XHR error (404 Not Found) loading http://localhost:9876/ts-mockito

Can you share some more details? How you import ts-mockito into your project?

I'm installing with with npm npm install ts-mockito --save-dev
then
`
import {it, describe, expect} from "angular2/testing";
import {HardService} from "./HardService";
import {EasyService} from "./EasyService";
import {mock} from "ts-mockito";
describe('Hard Service Tests', () => {
let mockEasyService: EasyService = mock(EasyService);
let hardService: HardService = new HardService(mockEasyService);
it('getRandomLengthDogName should return the dog length number', () => {
    // Arrange.
    var dogName = 'foo bar';

    // Act.
    var length = hardService.getRandomLengthDogName();

    // Assert.
    expect(length).toEqual(dogName.length);
});

});
`

Hmmm I'm using webpack. Can you give me a link to your project with sample app using systemjs?

@NagRock is not public so I cannot share any url. BTW I'm using a classic angular2 installation

Ok so I will try it but it will be faster if you will put a new project on github.

how are u loading ts-mockito?

I've forked angular quickstart and added ts-mockito to it:
quickstart-fork
You can clone it and run:
npm run install
npm run test-once
And remember to get instance of mocked object before using it:
let mockEasyService: EasyService = mock(EasyService); let hardService: HardService = new HardService(instance(mockEasyService)); // instance(mockEasyService)
And look at my commit to see what need to be added in quickstart to run ts-mockito with systemjs
commits

@NagRock Thanks


Solver backport

backport of #355

Coverage increased (+0.004%) to 87.305% when pulling ef1a1ae40949291ecb2b00c67a33b7ddc1267746 on gertjanvanzwieten:solver-backport into<PHONE_NUMBER>d7ae754bae6a529afac4119cc7651a on nutils:v4.x.

Coverage increased (+0.004%) to 87.305% when pulling ef1a1ae40949291ecb2b00c67a33b7ddc1267746 on gertjanvanzwieten:solver-backport into<PHONE_NUMBER>d7ae754bae6a529afac4119cc7651a on nutils:v4.x.

Coverage increased (+0.004%) to 87.305% when pulling ef1a1ae40949291ecb2b00c67a33b7ddc1267746 on gertjanvanzwieten:solver-backport into<PHONE_NUMBER>d7ae754bae6a529afac4119cc7651a on nutils:v4.x.

Coverage increased (+0.004%) to 87.305% when pulling ef1a1ae40949291ecb2b00c67a33b7ddc1267746 on gertjanvanzwieten:solver-backport into<PHONE_NUMBER>d7ae754bae6a529afac4119cc7651a on nutils:v4.x.


Calculate combined Python & Typescript code coverage and add CI checks

Description
Right now our various automated pytest runs in CI (unit, integration, etc.) calculate code coverage, but it's really hard to use this coverage reporting for anything useful. For example, if you look at a recent "Pytest-Unit-Ops" job and hunt hard enough, you'll see the coverage report for that test suite (https://github.com/ethyca/fides/actions/runs/3208546383/jobs/5244559179):

As you can see, it's 65% from that suite of unit tests. We also generate similar coverage reports for the other suites, but since we don't combine coverage across reports, we don't have a clear report that tells us what % of our Python code is covered by automated tests! Additionally, on the Typescript side, we don't have any automated coverage reporting.
Therefore, the goal of this issue is to report on the combined test coverage for both Python and Typescript code across all suites (unit, integration, e2e) in every pull request, and then throw check failures if coverage is reduced by a PR.
Acceptance Criteria

MUST combine test coverage across all automated checks in a pull request (unit, integration, E2E)
MUST calculate coverage for all Python source code
MUST calculate coverage for all Typescript source code
MUST report coverage in each pull request
MUST throw a check error in coverage is reduced by a pull request

@NevilleS , @seanpreston  is this going to complete for Q4? Trying to update Aha! dashboard


fix: 抽离disable delete,避免删除账号失败

这个 PR 实现什么功能/修复什么问题:

抽离disable delete
避免因删除保护导致删除账号失败

是否需要 backport 到之前的 release 分支:

release/2.12

/area region
/cc @swordqiu @yousong @tb365

Codecov Report

Merging #3933 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #3933      +/-   ##
=========================================
- Coverage    6.96%   6.96%   -0.01%
=========================================
  Files         503     504       +1
  Lines       84801   84824      +23
=========================================
- Hits         5905    5904       -1
- Misses      78395   78420      +25
+ Partials      501     500       -1

Flag
Coverage Δ

#aFlag
6.96% <0%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/compute/models/elasticcache_instances.go
1.26% <ø> (ø)
:arrow_up:

pkg/compute/models/dbinstances.go
1.02% <ø> (ø)
:arrow_up:

pkg/compute/models/guests.go
0.59% <ø> (ø)
:arrow_up:

pkg/compute/models/purge.go
0% <0%> (ø)
:arrow_up:

pkg/compute/models/lockableresource.go
0% <0%> (ø)

pkg/cloudcommon/db/modelbase.go
1.6% <0%> (-0.03%)
:arrow_down:

pkg/appsrv/response.go
66.15% <0%> (-3.08%)
:arrow_down:

pkg/cloudcommon/db/caller.go
36.27% <0%> (+1.98%)
:arrow_up:

Continue to review full report at Codecov.

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

/lgtm

/lgtm

/lgtm

/approve


Cannot run the app locally - conference_secrets.dart missing

If I want to run the app to the app, the file conference_secrets.dart is missing. It is not checked in. Hence, one cannot run the app on the local device. Please provide information, how one can start to fork your app.
conference_data.dart
import 'package:conference_data/src/conference_secrets.dart';

I stumbled upon the same problem, but eventually fixed it. Here is what I had to do:

via git, check out the branch not-for-rod/mock-api-response, as per the README
go into packages/conference_data/lib/src/conference_data.dart

it's a different file from packages/conference_data/lib/conference_data.dart, make sure you got the right one

remove the failing import:

import 'package:conference_data/src/conference_secrets.dart';

set a url placeholder to prevent compile errors (mock data will automatically be used instead). Make sure it's a valid url scheme (has http and such,) as it will still be parsed:

// This url is just here so the program compiles and it will be parsed, but no calls will be made with it.
final apiClient = ApiClient(baseUrl: 'http://wontbeused.not');

Now the app shold be compilable and runnable and should show some (hard coded) data.
I had to use the command flutter run --flavor Production lib/main.dart to run it on an iOS Simulator.

Thanks @schasi for chipping in :)
I also had this mentioned at the bottom of the README file here. I'm afraid that the conference lead did not specifically permit me to share the base URL publicly, that's why I have this file gitignored.


-T subkey to debug time, elapsed on hash calculation

Hello!
Working on reproducing #18, an idea comes to mind: to add -T subkey to debug time, elapsed on hash calculation. This can be helpful on large files to calculate.
Default: OFF
Output example:
# gost3411-2012 -T /path/to/some/file

GOST R 34.11-2012 (/path/to/some/file) = bebebebebebebe
Time =  1.234567 seconds

Similary to #14 this will violate DOTADIW principle I follow in Streebog.
You should use time to measure the time usage:
# truncate -s 4G /tmp/hugefile
# time gost3411-2012 /tmp/hugefile
GOST R 34.11-2012 (/tmp/hugefile) = 14ede2aafef0915cfb441e3....

real    0m37.113s
user    0m36.052s
sys     0m1.061s

which is way more meaningful in understanding of how the time resource was used.

Ok, can be closed.


[KYUUBI #2207]Support newly added spark data types: TimestampNTZType

Why are the changes needed?

support newly added data types: TimestampNTZType
since  Spark3.3.0
How was this patch tested?

[X] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[X] Run test locally before make a pull request

Codecov Report

Merging #2224 (54ef78d) into master (86cd685) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #2224      +/-   ##
============================================
- Coverage     61.79%   61.77%   -0.03%
  Complexity       69       69
============================================
  Files           331      331
  Lines         16016    16022       +6
  Branches       2031     2036       +5
============================================
  Hits           9897     9897
- Misses         5297     5298       +1
- Partials        822      827       +5

Impacted Files
Coverage Δ

...org/apache/kyuubi/engine/spark/schema/RowSet.scala
91.13% <0.00%> (-1.17%)
:arrow_down:

...ache/kyuubi/engine/spark/schema/SchemaHelper.scala
90.10% <0.00%> (-4.15%)
:arrow_down:

...org/apache/kyuubi/ha/client/ServiceDiscovery.scala
89.28% <0.00%> (-3.58%)
:arrow_down:

...rg/apache/kyuubi/engine/trino/TrinoStatement.scala
69.87% <0.00%> (+2.40%)
:arrow_up:

Continue to review full report at Codecov.

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

thanks, merging to master


Modify file_picker plug-in and want to test it.  but get  cmd\options.go:5:2: imported and not used: "xxx/", What is correct way to put plugin in options.go?

I have modified file_picker plug-in and want to test it.
I get this error "imported and not used"
C:\go-flutter-master\example\file_picker_example>hover run
Using engine from cache
go: finding github.com/go-gl/glfw latest
go: finding github.com/chunhunghan/file_picker latest
go: downloading github.com/chunhunghan/file_picker v0.0.0-20190701011006-457d50e58024
go: extracting github.com/chunhunghan/file_picker v0.0.0-20190701011006-457d50e58024
go: finding github.com/gen2brain/dlgs latest
# C:\go-flutter-master\example\file_picker_example/desktop/cmd
cmd\options.go:5:2: imported and not used: "github.com/chunhunghan/file_picker"
Go build failed: exit status 2

options.go
package main

import (
	"github.com/go-flutter-desktop/go-flutter"
	"github.com/chunhunghan/file_picker"
)

var options = []flutter.Option{
	flutter.WindowInitialDimensions(800, 1280),
}

go get success
C:\Go>go get -v github.com/chunhunghan/file_picker
github.com/go-flutter-desktop/go-flutter/embedder
github.com/go-flutter-desktop/go-flutter
github.com/chunhunghan/file_picker

What is correct way to put plugin in options.go?
Could you provide an example?
Thanks

Problem Fixed. I will provide example for File Picker. Thanks.


Utilize new destroy-app.js when present.

1.13.9 of ember-cli will have a destroy-app.js similar to start-app.js. See this PR.
This patch uses that file if it's present.

Rebased this - should be good to go to master.

thanks @blimmer!

I updated package.json and tagged, but then realized I can't npm publish.
@dgeb - Can you add me to npm owners for ember-cli-mocha (npm owner add rwjblue from your local ember-cli-mocha working directory should do the trick)?

@rwjblue sure thing - just added you

Thank you, sir!
Published as v0.9.5.

Nice - glad not to be a blocker any more!


Use modulo on Character/Outfit/Movestyle IDs

This should allow people using CharacterAPI mods to join again without crashing other players, they'll just show as base game characters instead.

Clever! I was going to do this, but I thought that maybe it would try to play an animation that wouldn't match the modulo'd character and break stuff. I'll need to do some testing before merging this for that reason.

To just throw a comment, just doing modulo is not a good idea on Characters enum. Not only we have MAX, which is not a character, we also have a few characters that should be excluded. This is a method that populated what characters you can pick on a dance pad
private void PopulateListOfSelectableCharacters(Player player)
{
  selectableCharacters.Clear();
  for (int i = 0; i < 26; i++)
  {
	  Characters characters = (Characters)i;
	  if (characters != Characters.headManNoJetpack && characters != Characters.headMan && characters != Characters.legendMetalHead && player.character != characters && IsCharacterUnlocked(characters))
	  {
		  selectableCharacters.Add(characters);
	  }
  }
  Shuffle(selectableCharacters);
}

I don't know what will happen if you make them available, but I assume the game would just outright crash.

I don't have the Characters enum serverside as the server doesn't build against the assembly. Perhaps I should just set invalid values to Red for now?

Actually, what am I thinking? We can flag invalid characters, and when it's received by the client, pick a random selectable character in place. This'll require an update in 1.4.0, so for now I'll set invalid players to Red.


fix locale category rewrite

The next.js is having an error while trying to automatically rewrites each source and destination prefixed to handle the configured locales because of the search category route. I added a locale false to this route to fix this problem
This is the error that was fixed when we tried to change the category:

This pr is related with this issue https://github.com/vercel/commerce/issues/119


SUDDEN FAILURES

[16.02.2017 21:59:48.587] [DEBUG] [THR 4     ] [    MAIL LOG] Email logger starting
[16.02.2017 21:59:49.259] [ INFO] [THR 4     ] [        MAIN] Loggers initialized: File, SystemEvent, Email, Request
[16.02.2017 21:59:53.029] [ INFO] [THR 4     ] [CONFIGURATION] Using 4 MS Word processes
[16.02.2017 21:59:53.780] [DEBUG] [THR 4     ] [     DB LOAD] Account already exists
[16.02.2017 21:59:53.795] [DEBUG] [THR 4     ] [     DB LOAD] All questions already exist
[16.02.2017 21:59:53.857] [DEBUG] [THR 4     ] [     DB LOAD] All questions already exist
[16.02.2017 21:59:53.857] [ INFO] [THR 4     ] [          DB] Initial data loaded
[16.02.2017 21:59:53.857] [ INFO] [THR 4     ] [        MAIN] Email controller configured
[16.02.2017 21:59:53.951] [ INFO] [THR 4     ] [        MAIN] Resend controller configured
[16.02.2017 21:59:53.967] [ INFO] [THR 4     ] [HTTP CONTROL] Using encrypted connections
[16.02.2017 21:59:55.127] [ CRIT] [THR 4     ] [ HTTP SERVER] EXCEPTION (InvalidOperationException)
Instance 'WmiApSrv' does not exist in the specified Category.
   at System.Diagnostics.CounterDefinitionSample.GetInstanceValue(String instanceName)
   at System.Diagnostics.PerformanceCounter.NextSample()
   at System.Diagnostics.PerformanceCounter.get_RawValue()
   at ITCC.HTTP.Server.Service.ServerStatistics`1.GetProcessInstanceName(Int32 pid) in C:\EsdDemo\Repository\lib\ITCC.Library\ITCC.HTTP.Server\Service\ServerStatistics.cs:line 543
   at ITCC.HTTP.Server.Service.ServerStatistics`1.GetCurrentProcessInstanceName() in C:\EsdDemo\Repository\lib\ITCC.Library\ITCC.HTTP.Server\Service\ServerStatistics.cs:line 530
   at ITCC.HTTP.Server.Service.ServerStatistics`1..ctor(Int32 criticalMemoryValue, MemoryAlarmStrategy memoryAlarmStrategy) in C:\EsdDemo\Repository\lib\ITCC.Library\ITCC.HTTP.Server\Service\ServerStatistics.cs:line 55
   at ITCC.HTTP.Server.Core.StaticServer`1.PrepareStatistics(HttpServerConfiguration`1 configuration) in C:\EsdDemo\Repository\lib\ITCC.Library\ITCC.HTTP.Server\Core\StaticServer.cs:line 118
   at ITCC.HTTP.Server.Core.StaticServer`1.Start(HttpServerConfiguration`1 configuration) in C:\EsdDemo\Repository\lib\ITCC.Library\ITCC.HTTP.Server\Core\StaticServer.cs:line 61

[16.02.2017 21:59:55.127] [DEBUG] [THR 4     ] [    MAIL LOG] Sending email with 1 entries
[16.02.2017 21:59:55.158] [ CRIT] [THR 4     ] [        MAIN] EXCEPTION (NullReferenceException)
Object reference not set to an instance of an object.
   at ITCC.HTTP.Server.Core.StaticServer`1.Start(HttpServerConfiguration`1 configuration) in C:\EsdDemo\Repository\lib\ITCC.Library\ITCC.HTTP.Server\Core\StaticServer.cs:line 104
   at EsdDemo.Server.Core.Controllers.HttpServerController.Start() in C:\EsdDemo\Repository\src\EsdDemo.Server.Core\Controllers\HttpServerController.cs:line 45
   at EsdDemo.Server.Core.Common.ServerApplication.StartControllers() in C:\EsdDemo\Repository\src\EsdDemo.Server.Core\Common\ServerApplication.cs:line 81

[16.02.2017 21:59:55.158] [ INFO] [THR 4     ] [        MAIN] EsdDemo service has started successfully

Second exception is fixed, will be included in next version

Fixed in v2.8.2


Questions about finding RA/DC in Tetra.c

Comparing Tetra.c and tetra.py, I had a few questions about the differences.

Why does tetra.py have a fine map and course map, but tetra.c does not?
It looks like tetra.c is meant to compare 1000 images to their expected output, rather than to actually output the RA and Dec of the image. I'm trying to alter it to input an image and output the image's center RA/Dec. To find the RA/Dec of the image, is it just a matter of taking the image_stars vectors and finding a rotation matrix to the star's RA and Dec and then applying that to the vector (1,0,0)?
Thanks so much.

The python version is an older implementation (a year older), which isn't as powerful as the C version.  Whereas the C version's wrapper code is for performance measurements, the python version implements every step of star tracking (i.e. centroiding, star ID, attitude determination).  That's what the fine and course sky maps are used for in the Python code, most notably rapidly finding unidentified stars in the image using an initial attitude estimate.  The sky maps are also used for speeding up iteration over all star patterns, but this is unnecessary in the C implementation due to optimizations at compile time.
The C implementation only performs the second step of star tracking, star ID, so it only outputs the catalog IDs of a subset of the stars in the image.  The rotation matrix can then be found by using the identified stars' catalog and image vectors.  I believe the rotation matrix is usually defined as the matrix which left multiplies (1,0,0) to give the star camera's pointing in the catalog reference frame (i.e. the reverse of what you said).  The function that does this is only present in the python code.

Awesome thanks so much!


feat(longhorn-UI): Allow specyfing loadBalancer IP/SourceRanges

This change allows users to specify loadBalancerIP/SourceRanges while using loadBalancer service type in Longhorn-UI service.

Thank you @docent-net for the contribution, sorry for the late reply, we missed this PR before.
Could you please file PR to this here? Because this repo is mirrored from the https://github.com/longhorn/longhorn/tree/master/chart, thank you.

Sure @jenting, https://github.com/longhorn/longhorn/pull/3485


Creating and Saving Mono Nodes Fails

Godot version:
3.1
Issue description:
Given a C# class "NodeScript" which inherits from Node, I expected the following to work:
var node = new NodeScript();

var packedScene = new PackedScene();
packedScene.Pack(node);

var restoredNode = (NodeScript) packedScene.Instance();

But the restoredNode loses the association with NodeScript upon being loaded. When I save the PackedScene to a file (my actual intent), I also see that the Node is saved with a CSharpScript association, but no actual association to the subclass.
I think the saved scene exhibits #17689, but the underlying issue is that the Ctor of NodeScript doesn't actually associate the right script with the node.
Minimal project:
ScriptChange.zip

After debugging this a little further, CSharpScript::create_for_managed_type does not set the path of the created Script instance. Indeed at runtime, it really can't.
A proper fix probably requires #15661.

Can anyone still reproduce this issue in Godot 3.5 or any later release?
I've been unable to reproduce with the provided MRP in v3.5.stable.mono.official [991bb6ac7].

I would be surprised if this was fixed in 3.x as the presumed cause of the bug should still be there: scripts created from C# (new Class()) don't have a path assigned.
This should be fixed in master already, though, as we re-use the same script that's loaded from disk: 67db89988d3c9283d4d58b328b959e1531b16dae.

Oh, I misunderstood the reproduction steps. It's still happening in 3.x indeed.


feat: $ prefix named file as island component

With this PR, the component file whose name has $ as a prefix, like /app/routes/dir/$counter.tsx, will be treated as an island component.
Before this PR, the filenames which are island components are below:

Under /islands - /islands/conter.tsx
An underscore and .island.tsx suffix - /app/routes/dir/_counter.island.tsx

From my experience using HonoX, it's sometimes easy to handle if the island component is in the same directory with the route file. So the 2 is better. But I'd make it a simpler and shorter name.

$ prefix - /app/routes/dir/$counter.tsx

Why $ - this is just my taste, but I think it's easy to understand.

I'll work on #159 after merging this PR.

Hey @usualoma !
What do you think about this? If it's okay, please review it!

Hey @usualoma
Updated: https://github.com/honojs/honox/pull/171/commits/7ff4899a4ce3afcbfdb8a39e5200e0c97d623a49
Could you review it again?

Merging!


add DataArray.pipe to api docs

[ ] closes #xxxx
[ ] tests added / passed
[ ] passes git diff upstream/master | flake8 --diff
[ ] whatsnew entry

Yes, thanks!


nvALT launches with window hidden from url scheme

2.2b (111) fixed the "No Window on Launch Bug" for me (Issue #162), with one exception, launching from the url scheme nvalt://find/*.
2.2b (111) running on OSX 10.8.4

This is still very broken. If you keep NVAlt open (either in the dock or menu bar mode) but with the window closed when not in use, the using applescript to activate or even launch does not show the window as it should, and therefore the URI fails to work. This breaks all workflows that involve e.g. Quicksilver or Alfred unless you never close the NVAlt window...


fixes #1032 Dark mode issues

Nice work @bakic.
I have just pushed some improvements to discuss:

Stay with a toggle for the Dark Mode switch
Mirror changes to NLP admin


fix: #1599 - Server passwords are printed during startup

Tested on a clean config:
Fixes https://github.com/znc/znc/issues/1599
password
[ ?? ] Launch ZNC now? (yes/no) [yes]: yes
[ .. ] Opening config [/tmp/znc/configs/znc.conf]...
[ .. ] Loading global module [webadmin]...
[ .. ] Binding to port [+6969]...
[ ** ] Loading user [zarthus]
[ ** ] Loading network [freenode]
[ .. ] Loading network module [simple_away]...
[ >> ] [/home/zarthus/.local/znc/lib/znc/simple_away.so]
[ .. ] Adding server [chat.freenode.net +6697 <censored>]...
[ .. ] Loading user module [chansaver]...
[ .. ] Loading user module [controlpanel]...
[ .. ] Forking into the background...
[ >> ] [pid: 4248]
[ ** ] ZNC 1.8.x-git-1788-1eac5ce9 - https://znc.in

no password
[ ?? ] Launch ZNC now? (yes/no) [yes]: yes
[ .. ] Opening config [/tmp/znc/configs/znc.conf]...
[ .. ] Loading global module [webadmin]...
[ .. ] Binding to port [+6969]...
[ ** ] Loading user [zarthus]
[ ** ] Loading network [freenode]
[ .. ] Loading network module [simple_away]...
[ >> ] [/home/zarthus/.local/znc/lib/znc/simple_away.so]
[ .. ] Adding server [chat.freenode.net +6697]...
[ .. ] Loading user module [chansaver]...
[ .. ] Loading user module [controlpanel]...
[ .. ] Forking into the background...
[ >> ] [pid: 4305]
[ ** ] ZNC 1.8.x-git-1788-1eac5ce9 - https://znc.in

No servers
[ ** ]
[ ?? ] Launch ZNC now? (yes/no) [yes]: yes
[ .. ] Opening config [/tmp/znc/configs/znc.conf]...
[ .. ] Loading global module [webadmin]...
[ .. ] Binding to port [+6969]...
[ ** ] Loading user [zarthus]
[ .. ] Loading user module [chansaver]...
[ .. ] Loading user module [controlpanel]...
[ .. ] Forking into the background...
[ >> ] [pid: 4334]
[ ** ] ZNC 1.8.x-git-1788-1eac5ce9 - https://znc.in

You're parsing the server twice... Can it be parsed only once?

I don't understand/know the technical complications of modifying pConfig->server directly, if that's what you mean.
Modifying content supplied in the config seems like a bad idea to me.

I as an admin kind of like the verbosity of the existing output.. but I will commit that solution on top of this and you can choose which one you'd like to merge/cherry pick


Adjust build to be kinder to multi-architecture.

Add code to build.gradle (stolen^H^H^H^H^H^Hborrowed from the CLI build)
to determine the building processor architecture and target
'linux-FOO' for that architecture.
Set the gogradle flag to prefix gogradle stages with 'go' (eg
goBuild), hence avoiding confusion with gradle build-in steps and
unwilling execution of 'test', etc.
Add a dependency on ':goBuild' to the 'distDocker' builds to ensure
that the proxy is properly build before building the Docker images.
Bump the Go version used by gogradle for the proxy build to 1.11.4
to match the version used in the docker images.  We could make this
a build argument to avoid having it in multiple places.
Documentation updates, including linting.

What's the status on this PR? Going to to label it WIP for now

this  is pretty much obsolete as now it does not build anymore with gogradle

Abandoned


[DNR] Set TarLongFileMode in the Assembly Plugin

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

Default TAR files aren't capable of handling long names, and this may be
causing issues with the assembly plugin in jenkins.
THIS IS AN EXPLORATORY CHANGE AND NOT SUITABLE FOR MERGE OR REVIEW AT THE CURRENT TIME.

Resolved by #417


test to debug travis failure in master

this is how i found the failure in Circle, but travis doesn't provide an ssh access.
I want to land it and see what exactly makes our tests disable colors

Codecov Report

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

@@           Coverage Diff           @@
##           master    #4145   +/-   ##
=======================================
  Coverage   60.55%   60.55%
=======================================
  Files         196      196
  Lines        6777     6777
  Branches        6        6
=======================================
  Hits         4104     4104
  Misses       2670     2670
  Partials        3        3

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 c06e2a9...630c148. Read the comment docs.


Add link to video that explains the scenario

Add link to video that explains the scenario

Pull Request Test Coverage Report for Build 1186

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 91.006%

Totals

Change from base Build 1184:
0.2%

Covered Lines:
2044

Relevant Lines:
2246

💛  - Coveralls


Add "execute next line", not just "step over next line"

When debugging & stopping at a breakpoint, I often want to see things evolve as lines are executed one by one. So I'd like to have an "execute next line" or "step through next line" button, not just a "step over [next line]" button

I don't understand what you mean.
VS Code surfaces exactly the stepping actions available from the underlying debugger.

Ah, it seems I simply misunderstood the meaning of "step over", which in my ears sounds like "skip next line and move to the one after that". But it actually does execute the next line. Wish it was called "execute next line" or "step through next line", but that's not a big deal for me anymore :). Closing the issue.


Update portalfx-pde-keyvault.md

Update doc for adding key filtering

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

The URL here needs to be updated: https://github.com/jasonxdhu/portaldocs/blame/b68a0f5a5d846f72752f3be506c636664e04a38e/portal-sdk/templates/portalfx-pde-keyvault.md#L25
The latest correct URL would be: https://msazure.visualstudio.com/One/_packaging?feed=Official&package=Microsoft.Portal.Extensions.KeyVault&version=1.0.50&protocolType=NuGet&_a=package
However, the commit for key filtering isn't on the master branch yet for version 1.0.50. Before merging this PR, we'd need to merge develop to master, do a buildreq, then change the URL to be something like 1.0.51.


sokol_gfx Shader compilation does not add SLOT_tex define if sampler is not used.

So, I might be doing this wrong, but if you change the fragment shader of arraytex-sapp.glsl to:
void main() {
    vec4 c0 = texture(tex, uv0);
    vec4 c1 = texture(tex, uv1);
    vec4 c2 = texture(tex, uv2);
    //frag_color = vec4(1.0); <- commented out
}

The autogen'd shader .h file does not contain a define for SLOT_tex.
I'm compiling this on macOS Catalina, xcode 11.

That's a side effect of the dead-code elimination in the SPIRV-Tools optimizer passes which are run in sokol-shdc. Since the code which uses c0, c1 and c2 has been removed, the optimizer also removes all the code assigning those values (the texture() function calls), which means that all references to the "tex" uniform are removed as well, and in the end, the entire uniform is removed because there's no code using it.
And with the uniform removed from the generated output shader code, the code which looks at the shader reflection information doesn't find any texture uniforms, and doesn't create SLOT_tex.
I agree that this behaviour is a bit annoying sometimes, since experimenting with shader code also requires changing the C/C++ side, but so far it hasn't been annoying enough to add a separate "debug mode" to shader cross-compilation. I'm also not sure whether it's enough to skip the SPIRVTools optimizer passes, because SPIRVCross (which runs after that) might also do similar things.

Got it. I agree that the behaviour makes sense as is. Perhaps it is worth calling this behaviour out in the readme somewhere? It took a bit of time to figure out what was happening :)


[docs] Alias classnames to clsx

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

Upgraded<EMAIL_ADDRESS>to<EMAIL_ADDRESS>which uses clsx
Alias classnames to clsx
Fixed a typo

If you want I can release virtualized with clsx.

I agree with @eps1lon. I don't think that we should bother. I would rather ignore the issue.

@merceyz Thank you for taking the time to work on it. However, I think that it would be better to see initiatives like https://github.com/JedWatson/react-select/pull/3630.
+6 months later, no progress: https://github.com/JedWatson/classnames/issues/187.


offer more configuration for textDocument/hover

Because the hover result is either a string or a markdown string, it's hard to offer users more configuration over their hover results.
A good example of this is this issue raised against gopls: https://github.com/golang/go/issues/32561. Some users want to see full documentation on hover, while others prefer to see one sentence of the documentation, while others may prefer to see none at all.
Personally, my preferred solution would be to offer users a "see more" option, something like holding down Command and hovering over an identifier. Right now, that just previews the declaration of the identifier, but perhaps it could be something like a detailed hover?
Is there any chance that the textDocument/hover could have a detailed configuration, or maybe there could be an additional textDocument/detailedHover? I would suspect that most languages return some combination of documentation and signatures/type declarations, so maybe hover could even be more structured than just a string?
Otherwise, I will likely add additional configurations to gopls to control the level of hover. I figured it was worth raising this issue because I suspect other languages would encounter these questions as well.

To stay in line with other request we should improve this in the following way:

have a HoverItem result type
have a content property that is MarkupContent
have an optional details property that is MarkupContent
have a hover/resolve request to resolvethe details value is needed.

I am open to such a improvement. However it needs to be speced and we need to have a reference implementation for it.

This feature would also help for emacs-lsp, where the hover request is used both to provide the documentation for the current focal point, but also to get summary information for variables in the local context.  So have either a two-level request, or a two-level reply would help.
See https://github.com/emacs-lsp/lsp-ui/issues/292

To add a bit more to this issue - https://github.com/golang/go/issues/33352 was just raised in gopls. To me, it seems like the best solution for this issue would be to have a more structured return type from textDocument/hover. I would expect that most languages return some combination of documentation and a signature--would it be possible to separate the two in the hover result?

@dbaeumer: Just to clarify, would I be able to contribute the solution for this issue, or would it have to be someone who works on the LSP?

@stamblerre sorry for the late reply. Yes, I would accept a PR for this. However as state we need a reference implementation for it in one of the editors / tools. It can't only be a spec update.

My use case for a hover extension is that often for a particular position I have different infos corresponding to different ranges under the position.
I understand this would cover my case of returning multiple pairs of hover info + range.


Implement read and write functionality for JSON col

Summary:
This commit contains changes necessary for reading from and writing to a JSON column.
Tests will be added for read and write when we finalize our approach for testing this against custom build of SqlServer that has JSON support.

At some point, perhaps not in this PR, you will need to update the conversion matrix used by smi and make sure that this warning in the code is satisfied:
https://github.com/dotnet/SqlClient/blob/0c4519871c9b69f7e05fe90f59d224d5f9ab3e3e/src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/Server/ValueUtilsSmi.cs#L2698-L2704
The CI runs on Release builds so the CI will not save you from violating assertions in this project.
You will probably need the sql server team to define the extensions to the conversion operations tables that start at https://github.com/dotnet/SqlClient/blob/0c4519871c9b69f7e05fe90f59d224d5f9ab3e3e/src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/Server/ValueUtilsSmi.cs#L2920
I suspect that json to binary might be the only conversion but someone needs to make that decision.


Enhance mesheryctl to Update Existing Components and Models from Local CSV Directory

Current Behavior
At present, mesheryctl supports updating component metadata and models by referring to a Google Spreadsheet. This spreadsheet holds the necessary information for updating the system's components and models, and mesheryctl pulls the data, processes it, and updates the relevant registry entries.
The command works as follows:
// Updating models from Meshery Integration Spreadsheet
mesheryctl registry update --spreadsheet-id [id] --spreadsheet-cred [base64 encoded spreadsheet credential] -i [path to the directory containing models].

This functionality limits users to Google Sheets as the only source for updating models and components.
Desired Behavior
We would like to enhance the existing functionality by allowing mesheryctl to update models and components directly from a local directory containing CSV files. This will enable users to bypass Google Sheets and use local CSV files for updating components and models.

Users should be able to specify a path to a local CSV directory, and the system should process the files to update the existing models and components.
The structure and format of the local CSV files should match the format expected from the Google Sheets data (i.e., columns and data structure should be consistent).
If both a Google Sheets URL and a local CSV directory are provided, the local CSV directory should take precedence, and the Google Sheets data should be ignored.
Validation should be added to ensure that the CSV files are correctly formatted and contain the necessary data for updating components and models.
Appropriate error handling should be implemented to manage cases where CSV files are missing, unreadable, or improperly formatted.

Implementation
The following changes should be made:

Extend the current mesheryctl registry update command to accept a new flag: --csv-dir [path-to-local-directory].
Add validation to ensure that the CSV files in the specified directory contain the necessary fields for updating the models and components.
Implement functionality to parse and process the CSV files, updating the registry entries for existing models and components.
Add error handling for missing or improperly formatted CSV files, with informative error messages.
Ensure that local CSV files override any Google Sheets input if both are provided.
Update the Meshery documentation to reflect these changes.

Example Command:
// Updating models from a local CSV directory
mesheryctl registry update --csv-dir /path/to/csv-directory

// Example to update with both Google Sheets and CSV, CSV takes precedence
mesheryctl registry update --spreadsheet-id [id] --spreadsheet-cred [base64 encoded spreadsheet credential] --csv-dir /path/to/csv-directory

Acceptance Tests

Verify that the system successfully updates models and components when provided with a local CSV directory.
Verify that the system processes the CSV files correctly and ensures the data structure matches the expected format.
Verify that the system handles errors appropriately for missing or improperly formatted CSV files.
Ensure that local CSV files take precedence over Google Sheets input when both are provided.

Mockups
No visual mockups are required for this enhancement, as it is a CLI-based functionality.

Contributor Resources

⌨ mesheryctl Contributing Guide
Beginner's guide to contributing to Meshery and mesheryctl
mesheryctl Command Tracker
Meshery CLI Commands and Documentation
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

i want to work on this

@chetak123 i am working on this one.

@chetak123, are you still up for this issue? Think you can wrangle it?

Hi @leecalcote, I noticed that this issue hasn't seen any progress for a long time, and I am very interested in it. Could you assign this issue to me?

I'll work on this, please assign me

Hi @leecalcote , I'd love to work on this issue if it's still open to grab. I've finished the setup steps and could run meshery locally.

Hi @leecalcote @Aijeyomah , can you please assign the issue to me? i would love to get into it

Here's a helpful reference to study while working through this issue.

@nganphan123 and @abhisheksatpathy, I missed your comments previously. My apologies. Please jump in here and learn with @weilirs.


append to Gemfile

in my case Gemfile had
group :test do
  #...
end

And what i've got after running generator is:
group :test do
  #...
endgem 'botstrap-saas'

@andriytyurnikov Thanks for posting this issue. Can you tell me which generator options you used?

@aaronvb those were defaults for 2.0.1

@andriytyurnikov thanks! I'll look into this

@aaronvb Any update on this one? I'd like to get a fix into v3 if possible.

@justin808 working on this now


Can't access WebUI for PufferPanel V2

Hello, I set up everything using the instructions on the website but when I try to connect using https://<IP_ADDRESS>:8080(<IP_ADDRESS> is my server ip) it doesn't connect it says "This site can’t be reached <IP_ADDRESS> refused to connect." I would love some help, thanks.
I am using ubuntu server 20.04.3
I allowed access of the ports through firewall
I am new to this so if there is any other info you need just ask, thanks.

HTTPS doesn't work with ips as you need a certificate for the host and those certificates are only given out for domains (not considering self-signed certs as browsers don't trust those), try using HTTP instead
If that doesn't work, please consider coming to our Discord and ask for help there as it's easier to help you and more people tend to be active there than there are here

bro thank you so much that worked that seems like something I would have tried but I guess not, thanks a lot!


docs: update time bar demo

Checklist

[ ] npm test passes
[ ] tests and/or benchmarks are included
[ ] commit message follows commit guidelines

Description of change

修复 https://github.com/antvis/G2/issues/3085 中反馈的 DEMO 问题

Pull Request Test Coverage Report for Build 407850646

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.006%) to 93.38%

Totals

Change from base Build 399768748:
0.006%

Covered Lines:
10673

Relevant Lines:
11084

💛  - Coveralls


Fix 'Object reference not set to an instance of an object' in FSI when .ToString() returns null

Fixes #706.

Looks good to me.  Thanks for taking care of this.


Bump containerd and cri-dockerd

Proposed Changes
Bump containerd and cri-dockerd
Bump containerd to v1.7.15
Bump cri-dockerd to v0.3.12
Types of Changes
version bump
Verification
check versions
Testing
Linked Issues

https://github.com/k3s-io/k3s/issues/9884
https://github.com/k3s-io/k3s/issues/9885

User-Facing Change

The embedded containerd has been bumped to v1.7.15
The embedded cri-dockerd has been bumped to v0.3.12

Further Comments

Should we add these to the list of what we have updatecli take care of?

I'd say yes, except we're carrying patches on top of both of these upstreams, so it requires some manuall massaging to rebase our patches on top of the new tags from upstream, and then push the tags to our fork. If/when we ever get to zero patches I'd say yes.