"Ignored error in Collectfast:"

I'm frequently seeing unspecified, hard-to-identify (let alone debug) errors when running collectstatic using Collectfast.  Is there any way to get the errors to print so I can find out what's going on?
These occur within my build process on CircleCI, which adds another layer of complexity.

It seems there isn't (https://github.com/antonagestam/collectfast/blob/master/collectfast/management/commands/collectstatic.py#L133).
This exception handling could probably be further developed so that setting a flag (or using settings.DEBUG?) makes the exception propagate. I think the reasoning I had when writing this was basically that errors in Collectfast should never prevent collectstatic from running in production environments.
Do you perhaps have the time to look into this further?

@jdotjdot Would you mind helping me out testing the push I just made to fix this? You can install the development version with pip install git+https://github.com/antonagestam/collectfast.git@dd21a4d5bbb966495b006a5e660108f2bfa2aee5#egg=collectfast.

Ugh I realized I never responded to this... so sorry about that. For what it's worth, I am seeing error output in collectfast now.

@jdotjdot No worries, thanks for the feedback! 🍰


Feature/percentage mode

Added percentage mode functionality, and updated the sample project with a mini-example

Hey, I am not quite familiar with Github, or gradle or travis :D So give me a shout on anything that needs updating.
Cheers

Travis failed, but this is cause it's trying to compile the sample project against the old version of the library (1.02) that doesn't have the percentage.
How do we handle this ?

Just change the gradle dependency of the sample project to compile project(:library)
(remove compile 'com.github.jorgecastilloprz:fillableloaders:1.02@aar' so it does not try to get the dependency from maven central)

Shall I change the version under this commit as well ? from 1.02 to 1.03 ?

Yes, change it to 1.03, please.

Hi. Can you update your branch and pull the new develop changes? I have merged a master commit which had not been merged time ago. Then i will review and test your code.
Thanks in advance!

I think that they are already included, this branch is already up-to-date with develop and master

@JorgeCastilloPrz Could you please accept this and push the new version to maven?

If you want this PR to be merged, i would need:

A new sample page with the new percentage feature added by XML custom attribute.
A real sample of a loading which a percentage being changed along the way of a "mock" download.
Cleaning the code for the setPercentage() method, as it is pretty dirty.

Hey, thanks for your quick review. I'll try to get this done during the coming weekend.

Hello. Did you work on the last changes we talked about?. Thanks in advance.

hey, sorry for the delay but unfortunately it's gonna get delayed more. I am about to undergo surgery so I doubt this is gonna get looked at for at least a few weeks.
Cheers

This seems like it is not moving.  Can we move this a bit just to get it merged in?

Hey, I made a pull request a long time ago but unfortunately I don't have any time to spend on this. Please feel free to continue the pull request.

I have sent PR to athkalia and I hope he can merge it in so this PR gets updated with all requested features except making setPercentage pretty (which I have no idea how).

Thanks mate, I'll have a look during the weekend.

I would prefer it sooner, since Jorge will also have to take a look at it.

Can you have a look at the checkstyle issues please?

@athkalia care to add spaces in FillableLoaderPage.java between the brackets on lines 133 and 135 as that is why CI is failing?

It's passing locally for me, same for you @Letme I take it ?

Yeah it is passing locally, I checked also the library:lint (which is failing on Travis) and it is executed and passes (finds just 3 errors instead of 4). None of the Lint warnings are produced by changes in this patch (at least on my local build).
@JorgeCastilloPrz can you advise what you want us to do?

I do not have any ideas what might be wrong. Can someone help please?

One of the differences I spotted is build number of Java (95 on my computer and 76 on Travis). @athkalia  can you check yours?
$ javac -J-Xmx32m -version
javac 1.7.0_95
$ java -Xmx32m -version
java version "1.7.0_95"

I could not find any reference on Travis how to fix this.

Thanks for fixing CI :). I will retake this PR this weekend and work on it to get merged. New version will be deployed to maven central pretty soon!.
Thanks for your work guys :+1:

I hope you do not have to do too much. Still I have set-up the Coverity for my fork of the project and it found 2 defects which I will probably get fixed in near future. I would suggest you set up Coverity as that is much stronger static analysis tool than lint and for opensource it is free (github enables usage as service).

Please, if you could, it would be nice if you can fix those issues before I
review all the changes and merge your code this weekend.
Thanks in advance!
El 12 may. 2016 3:45 p. m., "Crt Mori"<EMAIL_ADDRESS>escribió:

I hope you do not have to do too much. Still I have set-up the Coverity
for my fork of the project and it found 2 defects which I will probably get
fixed in near future. I would suggest you set up Coverity as that is much
stronger static analysis tool than lint and for opensource it is free
(github enables usage as service).
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/JorgeCastilloPrz/AndroidFillableLoaders/pull/5#issuecomment-218761513

I would rather put that on separate PR as this defects are not in scope of
this patch.

Reviewed 2 of 6 files at r2, 1 of 1 files at r4, 11 of 12 files at r5, 1 of 1 files at r6, 3 of 3 files at r7.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Add documentation addition - adding libraries

Thanks for the library, it's really useful. In the docs, there is the autocomplete component, however to use this, you must add the places library when registering with vue. This is missing from the documentation, would be worth adding this - I'd have done it, but couldn't find the repo for the docs:
To use the AutoComplete component, you must add the places API in the google console and add the library to the libraries array when registering with vue:
const app = createApp(App);
app.use(VueGoogleMaps, {
    load: {
        key: 'YOUR_API_KEY_COMES_HERE',
        libraries: ['places']
    },
}).mount('#app')

@Kyle123
Thanks for the feedback.
The docs are hosted in a separate Repository
https://github.com/fawmi/vue-google-maps-docs
To use some component, they should either be defined as you explained or imported all at once.
I will document this behavior, but you can open a PR in the Docs repository as well.

How can I get place from coordinates of my marker?

@vBoykoGit
Can you please open a separate Issue for that?  This issue is about adding individual libraries.


regress-sv_br_ml20181012d_iv

Surelog issue created: https://github.com/chipsalliance/Surelog/issues/2339

Issue fixed.


frontend changes for making a bot an incoming webhook

Add 'Type of bot' option for bots by adding dropdown in
settings->"Your bots". This dropdown has 2 options: 'Custom Bot'
and 'Incoming Webhook Bot'.
This will enable users to create and add a bot as an incoming webhook
(in addition to create/add full-featured bots).

Automated message from Dropbox CLA bot
@abhijeetkaur, it looks like you've already signed the Dropbox CLA. Thanks!

The new UI can be seen here.

actually, a good idea would be to change the realm-admin edit-bot UI to be the same as the normal user edit-bot UI

Has the above already been implemented? As to me, edit-bot UI is same for both the admin and normal user.

Has the above already been implemented? As to me, edit-bot UI is same for both the admin and normal user.

there are 2 bot editing UIs; one in the /#organization area and the other in /#settings/your-bots.

@timabbott since changing the edit-bot UI is completely separate from this (also I am getting stuck at it). I guess I should keep this PR clean.

I created #5272 and #5273 to track follow-ups to this.


Lower Min SDK

My project uses 23 as its min SDK, which is higher then Accompanists Webview min SDK which was 21. Was there a reason for the bump to 24 or can it be revered to 21 or 23?

@jobobby04 Thanks for your suggestions! I will downgrade it to 21.

Thanks!


动态拼接bug /Dynamic stitching bug

Describe the bug
A clear and concise description of what the bug is.
Extension Version
i18n Ally
Framework/i18n package you are using
vue-i18n
To Reproduce
Steps to reproduce the behavior:
ex:
vue page
{{ $t('common.'+type)}}
Can't be detected, the whole page will have problems, not displayed, and the layout is disordered
Desktop (please complete the following information):
版本: 1.41.0 (user setup)
提交: 9579eda04fdb3a9bba2750f15193e5fafe16b959
日期: 2019-12-11T17:59:52.065Z
Electron: 6.1.5
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Windows_NT ia32 10.0.17763

Thanks for the feedback!
For your question:

Can't be detected

The extension can NOT detect dynamic keys since it's impossible to tell what the value of type is through static analysis.

the whole page will have problems, not displayed, and the layout is disordered

What do you mean disordered? Can you provide some screenshots?

@inwikipedia I have added a fix to ignore keys end up with "." in v1.4.7. You can update to it can see if the problem still exists.

Closing this for now. If you have any questions, feel free to open it


different button ui for simple and booking product  in compare section  in velocity

Bagisto Version

[ ] master branch

Steps To Reproduce

go to compare section

Actual Result

@ghermans
This is not an issue. the button size will be according to the text


core/Files.cc:119 enforced condition gs.files[_id] has failed

Input
# typed: true

A = def f; end

Observed output
enforce
Expected behavior
no enforce

related:
# typed: true

U0 = Integer ||= 5

another one:
#typed:true
X=redo

and another:
# typed: true
U = ()


Allow to override/extend printer implementations

I would like to use my own open() for a ESC/POS printer that I work with.
However, all implementations live in a closure and there is no way to extend or provide an override.
I would expect at least something simple as:
const receipt_data = receiptline.transform(doc, {
	encoding: "cp866",
	command: "escpos",
	commands: {
		open: function (printer) { return "..." }
	},
})

Thank you for your feedback.
The command object interface was closed and experimental, but now it is stable.
We will consider extending the command object by users.

I think this would be fantastic! I have an Xprinter that supports ESCPOS, but does not have the "Buzzer" like an Epson printer might, or at least does not understand the command FS ( A pL pH fn m. The print works, it just prints AOC at the start of every receipt. Using sii removes that issue, but barcodes no longer work.
I can work around this with string manipulation, but it would be nice to have more control for some of the generic brand printers that are ESCPOS compatible.

Hello!
We have just released receiptline V1.7.0.
Printer implementations are now public. receiptline.commands.*
You can now override/extend printer implementations.
Please see receiptline/example/command/custom.js.
Thank you.


Add the use of reverse method in tests

Switched the code so that tests will use reverse instead of hard-wired URLs, so if we change the URLs in the future, it will automatically change in the tests.
fix #159

@avivz450 I call reverse billion times in the tests and that's a lot of function calls.
How can I optimize it?

@avivz450 I call reverse billion times in the tests and that's a lot of function calls.
How can I optimize it?
It looks like you're calling reverse the same amount of times you called the URLs before this PR, why do you think this is a problem?

@avivz450 I call reverse billion times in the tests and that's a lot of function calls.
How can I optimize it?

It looks like you're calling reverse the same amount of times you called the URLs before this PR, why do you think this is a problem?

Because previously it was just client.get("some-url") and now it's client.get(reverse("url-name"))
I mean before it was a string and now it is a function call. and it's a problem because there are tests where I call 5+ times to the same function that returns the same result which I can save the return value and use instead.
What I think could be the best option is that when I run the Pytest then I would call once to all the reverse of each URL, save it in global variables like landing_page_url., and use it in all the needed tests. And I tried to do it in the conftest.py file but it doesn't work... So if you have an idea to how to make it global to all tests, and if not maybe I should do this in each test, I mean in the beginning of the test to save all the URL paths to local vars by calling reverse and use the vars instead calling multiple time to the function.

@avivz450 I call reverse billion times in the tests and that's a lot of function calls.
How can I optimize it?

It looks like you're calling reverse the same amount of times you called the URLs before this PR, why do you think this is a problem?

Because previously it was just client.get("some-url") and now it's client.get(reverse("url-name"))
I mean before it was a string and now it is a function call. and it's a problem because there are tests where I call 5+ times to the same function that returns the same result which I can save the return value and use instead.
What I think could be the best option is that when I run the Pytest then I would call once to all the reverse of each URL, save it in global variables like landing_page_url., and use it in all the needed tests. And I tried to do it in the conftest.py file but it doesn't work... So if you have an idea to how to make it global to all tests, and if not maybe I should do this in each test, I mean in the beginning of the test to save all the URL paths to local vars by calling reverse and use the vars instead calling multiple time to the function.

I don't know another way to make these variables global.
I thought maybe putting in the same class tests where the same reverse call is made, and save the output of the same reverse call in a variable ,but it's not sure there is a logical connection between them.
I think what you suggested is good enough.


Does drupal-vm actively support multiple domains/virtual-hosts in one vm?

For small projects or tests I need different domains (different codebase and different database) but I do not want to set up a vm for each domain, and neither I want to use drupal multi-sites for that.
Currently I am still using the code from drupalpro (https://bitbucket.org/mediadoneright/drupalpro, originally from the quickstart vm) in my vm to create virtual hosts like drush quickstart-create --domain=example.dev. It works well, one thing I do not like is that I need to manually sett the hosts /etc/hosts entry.
What I'd like is to have a vagrant command to just set up a new (virtual) host with database and all that stuff I need to run a new site. Is this possible with drupal-vm? If so, it would be nice to have some instruction in the wiki.

@osopolar - Yes, you can run as many separate docroots, or combinations of single site docroots and multisite docroots, etc., as you'd like.
The configuration for this is basically split two ways (all inside config.yml):

You can sync as many folders as you want via vagrant_synced_folders. And these folders can be synced into any location inside the VM (I generally prefer /var/www/[site-name]).
You can set up as many virtualhosts as you want (with whatever Apache vhost options you want) in the apache_vhosts variable; point multiple virtualhosts at one docroot for multisite, or multiple virtualhosts at multiple docroots for a site-per-codebase approach.

Whenever you make changes to vagrant_synced_folders, you need to run vagrant reload to reboot the VM and pick up the new synced folders.
Whenever you make changes to apache_vhosts, you need to run vagrant provision to update the Apache configuration (this will automatically restart Apache as well).

Thanks, great.

ohh, I needed to add a new vhost and did the provision, I assume it re-installs all the packages and takes quite a bit of time .......

@ragavendra - Once it does the initial provision, it just confirms all the config is correct; it doesn't need to reinstall anything. If first provision takes maybe 10 min, future provisions should only take 1-2 tops. You can really speed it up if you use a local Vagrantfile to also disable the galaxy role installation (make sure to re-enable it whenever updating Drupal VM though!): http://docs.drupalvm.com/en/latest/other/overriding-configurations/#extending-the-vagrantfile-with-vagrantfilelocal

ohh I ran into '* The following settings shouldn't exist: galaxy_role_file' , never mind I added the new host within the box .......

So this seems like it should work, @geerlingguy. But my question is how do we address this in the documentation?

Set drupal_core_path to the same value as the destination of the synced folder you configured earlier:

If you can have multiple sites, wouldn't the core path change from site to site?

I think one of our resident geniuses has answered this. Seems that value is ignored unless you are using one of the site install routines or a build workflow (i.e. composer or make)

Hm, I think the docs lacks some of the details in geerlingguy answer in this thread.  And the "drupal_core_path" info is still confusing ... If this thread still is correct, the docs should be updated. (I found several issues related to this.)

@hansfn - So, there's a bit of a tradeoff either way. The vast majority of users have only one Drupal site in a given Drupal VM instance, so there are some conveniences and documentation bits that assume one site from the beginner's perspective. Also, most of the drupal_* variables will only apply to one site, not to many.
However, it is easy enough to add two, four, or hundreds of sites to a VM instance using the apache virtualhosts var (or nginx if using NGINX), the mysql databases var, etc. If doing so, the drupal_* vars will still only apply to the first/default site of the bunch, so if you wanted to do something like have 5 docroots with 50 virtualhosts pointing at them (using multisite or something like Domain Access), then you'd either have to manage the databases/installation manually after the VM is running, or add a post-provision script that does all of that setup post-setup.

The docs do describe multisite usage http://docs.drupalvm.com/en/latest/deployment/multisite/

@geerlingguy Thx for the quick reply. So maybe the best solution is to just have one Drupal site in each instance and stop thinking more about this. I have a habit of being (too) concerned about performance - which is the reason I have waited so long before I started playing with VMs.
@joestewart I'm not interested in using multisite. (I'm very familiar with it - used it a lot.) I did read the multisite docs before posting here. Actually, the rather well documented multisite setup made me expect some more info about using multiple separate Drupal installations ;-)

Sure, the reason I pointed there is the vhost and db portion is similar.  Just need to ensure a mount point for the code.


New translations en_us.json (Chinese Simplified)

Hello. Is this the finished translation for Simplified Chinese?
If you don't mind, I can add it to the new update.


[WIP][5.3][SourceKit][InterfaceGen] Don't print clang decls marked with the swift_private attribute.

Cherry-pick of https://github.com/apple/swift/pull/32046 for 5.3.

Explanation:
ObjC decls marked with the swift_private attribute (via the NS_REFINED_FOR_SWIFT macro) are intended to be hidden from framework clients, but still exposed for use in the Swift code within the framework or its Swift overlay. We were still printing them in the generated interface of the framework though, counteracting the intention of these being hidden from clients.

Scope of issue: Affects interface generation only.

Origination: Since NS_REFINED_FOR_SWIFT was introduced (not a regression).

Risk: Low. The fix is simple and only impacts interface generation. The compiler and other sourcekitd requests are unaffected.

Testing: Added a regression test for the new behavior and all existing tests pass.

Reviewers:

Resolves rdar://problem/62464954

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 75dd1bfdbce1fbcdf3d9c283d336d5fdbbbad79b

Build failed
Swift Test OS X Platform
Git Sha - 75dd1bfdbce1fbcdf3d9c283d336d5fdbbbad79b

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 75dd1bfdbce1fbcdf3d9c283d336d5fdbbbad79b

Build failed
Swift Test OS X Platform
Git Sha - 75dd1bfdbce1fbcdf3d9c283d336d5fdbbbad79b

@swift-ci please nominate


RTL: ProductTour checkpoint arrow style error

As the title suggests, the ProductTour RTL support is incomplete.
Descrption
As I was testing frontend-app-learning both in local dev (master) and in our [demo site] (fennx.fennectech.net) (nutmeg.1)
I've noticed that user tours in Arabic have 2 style issues with the checkpoint dialog arrow: (with screenshots)
Issue 1
when popper tooltip placement is top or bottom. the arrow appears shifted to the right by half the size of the dialog.

Placement: top (expected blue arrow in the center top of the dialog, appearing far to the left)

Placement: top (expected gray arrow in the center bottom of the dialog, appearing far to the left)

Issue 2
when popper tooltip placement is left or right, the arrow direction is reversed.

**Placement: right ** (gray arrow expected to the left, appearing on the right)

I currently lack a working paragon dev environment to provide a fix myself.
I also lack information about why the ProductTour arrow was styled this way in the first place.
Possible reason for issue 1

Thepopper library does a good job listening to changes on the #pgn__checkpoint element and positions the arrow accordingly.
The style overrides provided in src/ProductTour/Checkpoint.scss define the shape of the arrow and adjust its position slightly, but they enter in conflict with element.style applied by popper to the #pgn__checkpoint-arrow element. The overrides work well for LTR, but things need to be rewritten for RTL support.
I tried to disable almost every positioning rule in Checkpoint.scss applied to #pgn__checkpoint-arrow and I could have the arrow nearly centered.

I'd be glad if the person who wrote this part of the style could assist us in providing a fix for the two issues above.
Thanks in advance

@ghassanmas that PR looks strange. I think I was misunderstood.
The issues are with the tooltip arrow that points to the user tour's checkpoint anchor (outline, user tour link, tabs, etc.)

As I have noticed. The docs website for paragonµ does not show
The learning MFE uses v19.18.3 of paragon. Maybe it has been solved?

the same is happening with this frontend-app-authn password tooltip


merge master new doc changes to dev

Summary
Details
References
Example
Checks

[ ] Tested Changes
[ ] Stakeholder Approval

hard to merge directly


Scram Kitty DX [PCSE00642]

Game summary

Game name: Scram Kitty DX
Game serial: PCSE00642

Vita3K summary

Version: v0.1
Build number: 647
Commit hash: https://github.com/vita3k/vita3k/commit/42cde29

Test environment summary

Tested by: IllusionMan1212
OS: Windows 10 Pro
CPU: AMD Ryzen 5 1600
GPU: NVIDIA GTX 1050Ti
RAM: 8 GB

Issues
Crashes on [23:20:22.289] |W| [unimplemented_impl]:  Unimplemented sceNgsVoiceKill import called. [23:20:22.290] |W| [unimplemented_impl]:  Unimplemented sceNgsAT9GetSectionDetails import called. [23:20:22.301] |W| [unimplemented_impl]:  Unimplemented sceNgsVoiceSetParamsBlock import called.
Screenshots
Not at the moment
Log
vita3k.log

Vita3K v0.1.5 2603-8daf6c6f
goes ingame now
ingame music is stuffed
vita3k.log


error: value SqlDBQuery is not a member of org.apache.spark.sql.hive.HiveContext

When running sqlContext.SqlDBQuery(config) I get
error: value SqlDBQuery is not a member of org.apache.spark.sql.hive.HiveContext
All other examples are running OK. Is documentation updated?

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

ID: 3a43d656-dbdf-ae04-f88d-488aa1dc78c5
Version Independent ID: 155c9eb2-92df-1619-4284-17c46a419f1f
Content: Spark Connector with Azure SQL Database and SQL Server
Content Source: articles/sql-database/sql-database-spark-connector.md
Service: sql-database
GitHub Login: @allenwux
Microsoft Alias: xiwu

PS. Git documentation is describing different command: sqlContext.azurePushdownQuery(config) but it's still not working.

@maskodnv Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Issue is with the Run DML or DDL query in Azure SQL Database or SQL Server Scala example:

SqlDBQuery is a function defined in org.apache.spark.sql.SQLContext instead of org.apache.spark.sql.hive.HiveContext. This connector is designed for SQL Server and Azure SQL Database instead of Hive.

#please-close

HiveContext is a subclass of SqlContext so that shouldn't be the issue.
But the method appears to be called sqlDBQuery (lowercase s).

Please fix the documentation.....

The issue is still exists even when i use org.apache.spark.sql.SQLContext.SqlDBQuery(config) this gives another error.
error: value SqlDBQuery is not a member of object org.apache.spark.SQLContext


Updated new signature for scrollTo

As of react-native 0.20 previous scrollTo has been deprecated.

@nick before either accepting or rejecting my PR, consider backward compatiblility. I think we need to either use peerDependecies or use if (React.version >= 20) { new syntax } else { old syntax }. this approach was mentioned by @skv-headless in here https://github.com/brentvatne/react-native-scrollable-tab-view/pull/158

But this code will stop the animation of view :(

Think this is now fixed


"Get this page" and "Get all" doesn't grab the correct creation dates on rule 34 paheal

What is the expected behavior? What do you get instead?
I want grabber to give me the correct created date but instead it always gives me Wednesday, December 31, 1969, 5:59:59 PM for the images I bulk download off of paheal rule 34. When I download the images individually this doesn't happen.
What version of the program are you using? On what operating system?
Windows 8, Grabber 5.2.2

This issue looks a lot like issue #797, which was fixed in 5.2.1.
I just tried and could not reproduce in 5.2.2, using the %date:format=yyyy-MM-dd% %md5%.%ext% filename. Both normal and batch downloads contained the date. Did you do any other manipulation?

I have to say that it also happens to me. I find that simply raising the number on the Images Limit fixes it a bit, but that still makes the loading a bit of a pain.


Long runs may be longer than the default sudo lock timeout

If a chef run takes too long on OS X, it may lock the user out of sudo due to timeout.

Closing as we've deprecated this cookbook


upgrade to credhub 1.9.1+ to fix growing audit table issue

Raised at https://github.com/cloudfoundry-incubator/credhub/issues/33
This looks to be resolved in https://github.com/pivotal-cf/credhub-release/releases/tag/1.9.1 "Replaced the internal (DB-only) audit tables with external log files"

@ramonskie @rkoster this has been done in bosh-deployment master branch

our ci will pick the latest bosh-deployment by default..
our current issue is with the latest concourse release thats failing due to windows jobs that has been added


Bottom overflowed issue is coming while I am trying to inserting the Url from the toolbar option.

here is the issue

Fixed via 3d73d30, thanks for reporting!


Fixed #34479: add watchfiles

Hello!
From what I understood from ticket description, we need to import the watchfiles module and extend its class
Thanks in advance for review

Thanks for the patch, however, there is no more than the initial draft from Tom, #16747.


Onboarding Guide Request

Please describe the resource requested
A onboarding guide that explains the organizational structure, how to join, the meaning of different membership levels, and an overview of how you can be involved would be exceptionally helpful.
Why would you like it added
So that folks joining the CDSC project know what is going on and how they can be involved. What is the legal structure and why was it chosen? Is it even called the CDSC or is it now LFESS?
How soon do you need this added
As new people come on board and want to contribute.
Additional context
There's been so much change in the organizational structure over different iterations that I've lost track of the current organizational structure.

Thanks @Henry-WattTime - can you review the doc added in #4 and let me know if that addresses the asks?

This looks great, I added a few minor comments.


Unable to connect to cloudSQL instance using IAM Authentication

I am unable to connect to my cloudSQL instance using IAM Authentication. Using the db password works fine. The error I am receiving:
ERROR: Error opening connection SASL: SCRAM-SERVER-FIRST-MESSAGE: client password must be a string

Application Default Credentials is configured.

Hi @fredsco - would you be able to share a possibly redacted version of the JSON representation of your config to confirm it? The one I have that uses IAM authentication successfully looks like this
"sqltools.connections": [
			{
				"previewLimit": 50,
				"driver": "CloudSQL-PostgreSQL",
				"name": "mydb",
				"group": "dev",
				"instanceConnectionName": "myproject-dev:us-central1:mydb",
				"database": "mydb",
				"username"<EMAIL_ADDRESS>				"impersonatedServiceAccount"<EMAIL_ADDRESS>				"iamAuthentication": true
			}
		],


The next and prev button clicks of the disabled state of the [Bug Report] el-pagination component trigger the current-change event

Element UI version
2.3.3
OS/Browsers version
window10
Vue version
2.5.17-beta.0
Reproduction Link
https://jsfiddle.net/oscar_xi/nh7ho83e/
Steps to reproduce
点击next按钮或者prev按钮
What is Expected?
不触发current-change事件
What is actually happening?
会触发current-change事件

#10628


HDFS-16494.Removed reuse of AvailableSpaceVolumeChoosingPolicy#initLocks().

Description of PR
When using the default constructor to build the AvailableSpaceVolumeChoosingPolicy, initLocks() is used twice, which is actually unnecessary, the purpose of this pr is to avoid this from happening.
Details: HDFS-16494
How was this patch tested?
Not too stressful for testing. Because this class is relatively mature.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
17m 55s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
34m 48s

trunk passed

+1 :green_heart:
compile
1m 29s

trunk passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
compile
1m 21s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
checkstyle
0m 58s

trunk passed

+1 :green_heart:
mvnsite
1m 29s

trunk passed

+1 :green_heart:
javadoc
1m  3s

trunk passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
1m 27s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 25s

trunk passed

+1 :green_heart:
shadedclient
25m 38s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 17s

the patch passed

+1 :green_heart:
compile
1m 25s

the patch passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javac
1m 25s

the patch passed

+1 :green_heart:
compile
1m 13s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
javac
1m 13s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 53s

the patch passed

+1 :green_heart:
mvnsite
1m 19s

the patch passed

+1 :green_heart:
javadoc
0m 53s

the patch passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
1m 25s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 28s

the patch passed

+1 :green_heart:
shadedclient
25m 29s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
325m 43s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 37s

The patch does not generate ASF License warnings.

450m 30s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.mover.TestMover

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4048/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4048

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux e83832001c7d 4.15.0-163-generic #171-Ubuntu SMP Fri Nov 5 11:55:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 57207da5948665cf4f7653bf24a812abd2e9073b

Default Java
Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4048/2/testReport/

Max. process+thread count
2343 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4048/2/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Here are some unit tests that are failing. E.g:
org.apache.hadoop.hdfs.server.namenode.TestFileTruncate
org.apache.hadoop.hdfs.server.mover.TestMover
It looks like these exceptions have little to do with the update I submitted.
Can you help review this pr, @aajisaka  @virajjasani   @tomscut .
Thank you very much.

Thanks @tomscut .

Can you help review this pr, @Hexiaoqiao  @ferhui .
Thank you very much.

Thanks @Hexiaoqiao .

Committed to trunk. Thanks @jianghuazhu for your contributions! Thanks @tomscut reviews!


Make identity tests run 230x faster

This PR improves the runtime of Identity unit tests by avoiding waiting real-world clock time when testing code that uses delays, such as polling for token updates.
On my Surface Laptop this took the runtime from 26s to 113ms
This is accomplished by replacing the use of core-http's simple delay function with one that is able to be mocked for testing.

This looks ready to merge!


Create Firebase setup article

Create Firebase setup article
:gear: Release Notes

Adds articles to explain how to set up and use the Firebase Local Emulator suite, as well as creating a cloud-based backend on Firebase.

:pencil: Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

[X] I agree to follow the Code of Conduct and Contributing Guidelines.

This was addressed by #59.


Support for GitHub Enterprise Server (GHES)

Hello @ysk8hori
Thank you for providing such an outstanding Workflow.
As a suggestion, I would like to propose adding support for GHES (GitHub Enterprise Server), as it would enable more users, including us, to benefit from these Actions in a broader range of environments.
To access GHES, the GITHUB_API_URL environment variable is required. Fortunately, thanks to GitHub Actions, this variable is automatically provided as a default environment variable, meaning no code changes are necessary to utilize it.
For more details, please refer to the official documentation:
GitHub Actions - Default Environment Variables
Please let us know if there are any changes or adjustments needed to align this contribution with your vision.

Hi @nacal !
Thank you for your excellent suggestion! I completely agree with your proposal.
One thing I’d like to confirm is the default URL behavior of Octokit.
https://github.com/octokit/action.js/blob/24a544a612c98c85b51ee38d29ca34e638ef09f6/src/index.ts#L35
From what I’ve seen, it seems to use GITHUB_API_URL as its default. Did it not work as expected in your environment? If you could verify this, it would be greatly appreciated.
If the default URL doesn’t function correctly, I’ll be happy to merge your Pull Request right away!
Looking forward to your reply. Thanks again for your contribution!

One more thing: could you please run npm run update-workflowfiles and commit the updated workflow files? 🙇🏻
This script will handle the commit for you.
The files test-no-params.yml and test-with-params.yml are designed to validate the functionality of delta-typescript-graph-action directly in the CI environment. Since these workflows reference a specific commit hash of delta-typescript-graph-action, it’s necessary to update this hash and trigger the CI to ensure everything works as expected.
Thanks again for your cooperation!

@ysk8hori
Thank you for checking.
You're absolutely right; it defaults to using GITHUB_API_URL. That was my mistake.
However, it doesn't seem to work correctly in my environment.
It might be caused by a different issue, so I’ll continue investigating.
Feel free to close this PR.

Thank you for your effort.
If you run into any issues or need help, feel free to open an issue. I’m happy to assist.
For debugging, you can use this configuration to get detailed logs:
steps:
  - uses<EMAIL_ADDRESS>    with:
      ...
      debug: true
      ...


reusable property edit widget for window function selection

In the same style of pothos-gui edit wigets:
https://github.com/pothosware/pothos-gui/tree/master/EditWidgets
Any arbitrary widget can be registered into the property tree at "/gui/EntryWidgets/*"
This window function edit widget can be used in Periodigram, Spectrogram, and pothos-blocks taps designer block.

moved to plotters toolkit: https://github.com/pothosware/pothos-plotters/issues/3


document.activeElement doesn't stay on checkboxes

https://twitter.com/ryanflorence/status/933244473492037632?s=21

If an element has focus and you click another:

onBlur of the original element is called
inside onBlur, document.body is document.activeElement, not the new element
next tick the new element is document.activeElement
It's crazy.

I guess the checkbox works as intended... we were just not waiting for the next tick, and in our instance it was going to a parent instead of body
https://html.spec.whatwg.org/multipage/interaction.html#dom-documentorshadowroot-activeelement-dev

The activeElement attribute's getter must run these steps:

Let candidate be the DOM anchor of the focused area of this DocumentOrShadowRoot's node document.
Set candidate to the result of retargeting candidate against this DocumentOrShadowRoot.
If candidate's root is not this DocumentOrShadowRoot, then return null.
If candidate is not a Document object, then return candidate.
If candidate has a body element, then return that body element.
If candidate's document element is non-null, then return that document element.
Return null.

So, I guess it's working as intended... we could get around with this by using the relatedTarget from the blur events in the drop down blur handlers... right now they try to look at document.activeElement ... after moving it to the contentContainer for checking if it's still a child we're good... but we could move it back and wait for the next tick or use the relatedTarget instead...
closing for now :D


add2app .gitignore has .android and .ios in it.

This seems like a bug if the entire wrapper project is omitted from source control

That is by design, as it's generated code. Not everything goes into source control.
(Of course, if you have cause to modify the generated code then you're at liberty to check it into source control).

This doesn't seem right because it won't regenerate when another dev pulls it down
And won't build because the Gradle script is missing.
Keeping generated code out of source control makes sense if it regenerates automatically...

It regenerates when another dev pulls it down, yes. That's kind of what generated code does.

(In this case, it's generated as part of the initial and any subsequent flutter packages get runs)

@neiljaywarner does flutter packages get recreate the platform-specific generated code for you?

@zoechi  it sure does, I'm so sorry. I did run the outer app and it gave an error which is why I thought it did not autogenerate, and this is new/experimental features.
I also tried asking several people about it several different ways including the flutter-dev email group and the gitter channel (twice i think) before posting it as a possible bug, but again, I'm sorry. With flutter being so IDE agnostic, i should've looked harder for a CLI way like that.
in addition, i doesn't seem like a package I'm getting, plus my goal is/was to have little/no setup required on the devs for the outer project.
so i wonder

is there any way to get it to "just work" by default, at least with help from the Intellij/Visual studio plugins or something?
Can we treat it as an area for improvement in the documentation please
perhaps change local.properties to have a comment header, similar to Android's comment header.

again, I'm sorry about that.

Adding Flutter to existing apps is still experimental and lots of work in progress.
Flutter usually runs flutter packages get before run/build, but perhaps not in that scenario (haven't used it myself yet).

@zoechi  is there an ETA for add2app being "not experimental"?  Should we be concerned about it being in production, or is it just a ymmv thing?

Add-to-app is no longer experimental.
There was some attempt to document this with https://flutter.dev/docs/development/add-to-app/ios/project-setup#module-organization and https://flutter.dev/docs/development/add-to-app/android/project-setup#create-a-flutter-module with the ".ios/.android directory is emphemeral, run flutter pub get to generate it.
@stevemessick @xster Does Android Studio handle a missing .android directory (like after a git clone) in the module?  Does it ever run flutter pub get or flutter build in the module, or mostly is just creation supported?
https://flutter.dev/docs/development/add-to-app/android/project-setup#using-android-studio

This is a duplicate of https://github.com/flutter/flutter/issues/25931, I'll take my IDE question over there.

Sorry, I meant duplicate of https://github.com/flutter/flutter/issues/21706.


Problem with file .\track-data*.gpx Error: Cannot find matching path! Wrong vehicle car or missing OpenStreetMap data? Try to increase maxSearchMultiplier

Hello,
I installed the map-matching some days ago and i'm trying to run it on different GPX files of GPS bike data. Some of them work and i get good results some others don't and i'm getting the following error:
"Problem with file .\track-data*.gpx Error: Cannot find matching path! Wrong vehicle car or missing OpenStreetMap data? Try to increase maxSearchMultiplier=100.0"
Even though i set my vehicle to be a bike.
I tried to set the maxSearchMultiplier to 100.0 and it didn't work.
I don't know whats wrong and i couldn't debug the whole algorithm.
Is there any restrictions on the GPX files? Am i missing something here?
Thanks for your help.

In my branch I translated this log into "Check missing OpenStreetMap data". Anyhow, first I would try to set maxSearchMultiplier up to 800.
In case that you speak German you might read up https://github.com/ratrun/map-matching/blob/htmlresultoutput/htmlreport.de.md. If this does not help you please post your GPX. Without a GPX it is next to impossible to guess what might be the reason for your case.

Thanks for your reply.
I tried the maxSearchMultiplier it didn't work as well and unfortunately i don't know much of German, please have a look on the GPX file:
https://www.dropbox.com/s/hx8lz1kfbx4qqr6/trace2ClearGPX22.gpx?dl=0

Your example works for me using the latest Saarland geofabrik extract, vehicle car and following settings:
gpxAccuracy=15 graph.flagEncoders=car forceRepair=true separatedSearchDistance=250 htmlReport=true maxSearchMultiplier=100
I guess that you are using an OSM input file which does not cover your area. This will generate the oberserved error:  "Error: Cannot find matching path! Wrong vehicle car or missing OpenStreetMap data?"
Note: From the beginning of the input GPX in the south of "Prinzenweiher", it looks as if the track was recorded using vehicle=foot, because there are only foothpaths mapped there. As the detour to the Meerwiesertalweg in the north is not too far away, the algorithm snaps in to this segment.

Thanks @ratrun ! @ObadaM maybe you need to remove the graph-cache as your current vehicle is also not the one you specified

Thanks @ratrun i've downloaded the latest Saarland.osm.pbf and with these parameters now it works!


Multi Availability Zone

Should support multi az for all HA components. We must also define a singleton zone

Any progress on this?


Update use-source-control-explorer-manage-files-under-version-control.md

The Note item did not make sense. My proposed change is just a pointer (IMHO) in the right direction.

@EDToft : Thanks for your contribution! The author(s) have been notified to review your proposed change. @MaximMalygin

IMPORTANT: When this content is ready to merge, the reviewer must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @MaximMalygin


newtrify app kills headphone audio

newtrify seems like a great app but I had to uninstall it as I found I could not hear any audio through the headphone socket. Internal speaker was not affected.  My phone is very old so its probably not worth it to fix.
Hardware: Dell Venue
Android version 2.3.3

OK i will have a look at that.

Hardware related.


MentionsLogger isn't storing new mentions?

Existing log files were not being written to due to a pandas mistake that is now corrected.


Use urllib2 instead of urllib

urllib has trouble with certain proxy configurations, for example,
failing to retrieve the https pypi index when the proxy is set as http (see #11).
urllib2.urlopen does not have the same problem, and has the added benefit
of raising URLError when a server responds with a code > 400, which means
we can get rid of our custom error classes.

This is available in jujuresources >= 0.3.0.  All of the the Apache-based big data charms in the ~bigdata-dev namespace have been updated to use this library:
https://jujucharms.com/u/bigdata-dev/#charms
This will make its way from -dev to the recommended namespace during the next promulgation window.


Removing extra hard rule

Pulling hr above Criterion #1 on Quality Summary page

This was fixed in another PR.


HomeAssistant integration?

Is there an opportunity for Home Assistant integration? There are other projects but none that appear to have the flexibility/capability this one does.

I'm looking forward for it too

I'm looking forward for it tool


[Topic] Next Versions release Checklist.

Proposed Changes
Please note that this is a work in progress and is up for debate
The goal is to be able to have a double chest of all items no matter how the settings are setup.
Example: Disabling end city structures. Enables Shulker shells from cleric

One should also be able to complete the blaze and caves advancements  with any settings.

Generator

Make the generator do all the work when it  comes to structures.
Add a way to have custom structures from other mods
Faster generation
New Setting for disable and enabling structures. If we still need the bounding box maybe we can make that happen but I would think that if the end user says disable nether fortress they are not even wanting the bounding boxes.

Trading
There will be a new mod soon that will have new villagers in the game.  Please see this [repo](for more details)

Removal of the following trades.

Reinforced deep slate will be removed from masons. (ancient city)
Heart of the sea from wandering traders. (craft-able now)
Shulker shells from cleric (The end. shulker farms are easy now a days)
Music disks from the wandering trader. (New villager same the record man sells all records)
End portal frames from wandering trader. (No need as stronghold and custom structure now has it)
Lava bucket from wandering trader. (One can get lava from the runned portals. Or we can add the bonus chest)
Sponge from wandering trader (Ocean monuments have these now)

New Trades

64 iron will get you 1 raw iron (Jewelry maker)
64 copper will get you 1 raw copper. (Jewelry maker)
64 gold gets you 1 raw gold .
all four of the two tall flowers can be bought from the wandering trader.
With all the new villager types there is a separate page for all these trades [Add Link]

Loot

Removal

witches drop nether wart.(can get from fortress and custom structures)
drowned's drop gold. (Gold can be made with pigman farms)
phantom -> elytra (You can get from the end)
Piglins drop nether rack. (craft-able)
Piglins ancient debris (withers dropnow)
HOV for diamonds (diamonds can be found in ancient cities and shipwrecks ect )
Budding amethyst (we can make geos spawn now.)

Added

Withers drop ancient debris (33% chance ? maybe should be higher.)
Husks drop sand (50%)
Tropical fish drop coral fans

Crafting
New:

Red sand can be crafted with 8 sand and a red dye
All coral blocks can now be crafted with the 2x2 of the coral fan.
nether rack can now be crafted from 2x2 weeping vines
Heart of the sea can now be crafted from one nether star four sea lanterns and four dark prismarine
Kelp can be crafted from a 2x2 of sea grass.
Spore blossom can now be crafted from flowering azalea and four pieces of moss.

Removed:

mycelium has been removed
grass has been removed. (spawn island, starts out as grass)
toml config has been removed and replaced with json. Includes Gui.
Change default runtime mem to 1G

Misc

New advancements for mcskyblock.
Change license in fabric-mod to be MIT and add m_jimmer as a member.
Added mod menu integrations
New mappings to official mojang mappings
lay foundations for making the mod fabric or forge.
Preset has be renamed to mcskyblock:skyblock_preset
Seek out others that speak other languages and ask for help with translations.
Removal of lib toml4j and added modmenu and cloth config.
Made gradlew executable for mac, gnu/linux and bsd users
Renamed asset folder to match modid
Added misc static variables for better portability.
Create list of mods to add to make mod pack.

Ok, so the reason behind the bounding box was originally a feature request a while back - some users wanted witch huts, fortresses and monuments to spawn without having to mine up all the blocks, while still being able to spawn the relevant mobs. Admittedly the way it currently does that is very hacky, but it does work - possibly room for improvement though.
As for the changes, for the most part I think those are good ideas. However considering the goal is "The goal is to be able to have a double chest of all items no matter how the settings are setup" wouldn't quite apply with some of the removals. Possible we keep as-is (or make harder, but not impossible) if the relevant structures are disabled and remove those options if they're enabled.

"Reinforced deep slate will be removed from masons. (ancient city)", "Sponge from wandering trader (Ocean monuments have these now)", "Lava bucket from wandering trader. (One can get lava from the runned portals. Or we can add the bonus chest)", "phantom -> elytra (You can get from the end)", "HOV for diamonds (diamonds can be found in ancient cities and shipwrecks ect )", "Budding amethyst (we can make geos spawn now.)" - If structures are disabled, you wouldn't be able to get a double chest of those.

"witches drop nether wart.(can get from fortress and custom structures)" and "drowned's drop gold. (Gold can be made with pigman farms)" - Lets say you're playing with no custom structures, how would you get to the nether to be able to get nether wart / gold? Current progression would be splash potion of weakness + golden apple on a witch to get a villager, this enables aquiring emeralds which can be traded with the wandering trader for a lava bucket to make a nether portal.

"mycelium has been removed" - unsure on the rationale of this one, there's no way to get mycelium even if all vanilla structures are enabled (could move to a dedicated villager maybe). Seems antithetical to the double chest of items goal to just remove it completely

"grass has been removed. (spawn island, starts out as grass)" - might be tricky if someone decides to do a one-block start, tree log might cause it to revert to dirt before you have time to cut the log down.

Final thoughts on this, if we're making these new villagers core to being able to get all the items / achievements in the game, would it not make more sense to store them in the mod itself? If not, then it would probably be worth making the base mod have a dependency on that villages mod at least.


I261 update team coordinator index page

Change Summary
[Briefly summarise the changes that you made. Just high-level stuff]
On TC login it displays a list of registered competitions, then links to a list of teams from the institution that is registered to the competition.
Change Form
Fill this up (NA if not available). If a certain criteria is not met, can you please give a reason.

[x] The pull request title has an issue number
[x] The change works by "Smoke testing" or quick testing
[ ] The change has tests
[ ] The change has documentation

Other Information
[Is there anything in particular in the review that I should be aware of?]

In institutions.js on line ~200 (in the where clause) its getting a team count where its only looking for institutionId but it should now be institutionId and tournamentId

Does this mean that we can have teams with the same name on different tournaments? Like Team 1 and Team 2 in multiple tournaments?

In institutions.js on line ~200 (in the where clause) its getting a team count where its only looking for institutionId but it should now be institutionId and tournamentId

Does this mean that we can have teams with the same name on different tournaments? Like Team 1 and Team 2 in multiple tournaments?

Correct :)


Understanding pricing when using DynamoDB Titan

Hi,
this is not an issue, rather a fuzzy question.
I would like to get a good understanding of what would be the price (in terms of $) of using DynamoDB Titan. For this, I need to be able to understand when DynamoDB Titan does reads and writes. Right now I am pretty clueless.
Ideally I would like to run a testcase which adds some vertices, edges and then does a rather simple traversal and then see how many reads and writes were done. Any ideas of how I can achieve this? Possibly through metrics?
if it turns out I can't extract this information myself, I would very much appreciate a first brief explanation about when DynamoDB Titan performs reads and writes.
Thank you!

For all Titan backends, to understand and estimate the number of writes, we rely on estimating the number of columns for a given KCVStore. You can also measure the number of columns that get written using metrics when using the DynamoDB Storage Backend for Titan.
To enable metrics, enable the configuration options listed here:
https://github.com/awslabs/dynamodb-titan-storage-backend/blob/1.0.0/src/test/resources/dynamodb.properties#L6
Specifically, enable lines 7-11.
Note the max-queue-length configuration property:
https://github.com/awslabs/dynamodb-titan-storage-backend/blob/1.0.0/src/test/resources/dynamodb.properties#L45
If the executor-queue-size metric hits max-queue-length
http://docs.aws.amazon.com/amazondynamodb/latest/developerguide/Tools.TitanDBMetrics.html#Tools.TitanDBMetrics.ExpandedMetrics
for a particular tx.commit() call, then you know that the queue / storage.buffer-size were not large enough. Once the executor-queue-size metric peaks without reaching max-queue-length, you know you have captured all the columns being written in a tx.commit() call, so that will give you the number of columns being changed in a tx.commit(). You can look at UpdateItem metrics for edgestore and graphindex to understand the spread of columns between the two tables.
All Titan storage backends implement KCVStore, and the keys and columns have different meanings depending on the kind of store. There are two stores that get the bulk of writes, assuming you have not turned on user-defined transaction logs. They are edgestore and graphindex.
The edgestore KCVStore is always written to, regardless of whether you configure composite indexes. Each edge and all of the edge properties of that edge are represented by two columns (unless you set the schema of that edge label to be unidirectional). The key of edge columns are the out-vertex of an edge in the direct column, and the in-vertex of an edge in the reverse. Again, the column of an edge is the in-vertex of an edge in the direct column, and the out-vertex of an edge in the reverse. Each vertex is represented by at least one column for the VertexExists hidden property (https://github.com/thinkaurelius/titan/blob/1349a5edd1149a00da6a37a4584aed25a8164a14/titan-core/src/main/java/com/thinkaurelius/titan/graphdb/types/system/BaseKey.java#L26), one column for a vertex label (optional) and one column for each vertex property. The key of vertices is the vertex id and the columns correspond to vertex properties, hidden vertex properties, and labels.
The graphindex KCVStore will only be written to if you configure composite indexes in the Titan management system. You can index vertex and edge properties. For each pair of indexed value and edge/vertex that has that indexed value, there will be one column in the graphindex KCVStore. The key will be a combination of the index id and value, and the column will be the vertex/edge id.
Now that you know how to count columns, you can use this knowledge to estimate the size and number of writes to edgestore and graphindex when using the DynamoDB Storage Backend for Titan. If you use the multiple-item data model for a KCVStore, you will get one item for each key-column pair. If you use the single-item data model for a KCVStore, you will get one item for all columns at a key (this is not necessarily true when graph partitioning is enabled but this is a detail I will not discuss now). As long as each vertex property is less than 1kb, and the sum of all edge properties for an edge are less than 1 kb, each column will cost 1 WCU to write when using multiple-item data model for edgestore. Again, each column in the graphindex will cost 1 WCU to write if you use the multiple-item data model.
Lets assume you did your estimation and you use multiple-item data model throughout. Lets assume you estimate that you will be writing 750 columns per second to edgestore and 750 columns per second to graphindex, and that you want to drive this load for a day. You can set the read capacity for both tables to 1, so you know each table will start off with one physical DynamoDB partition to begin with. In us-east-1, the cost for writes is $0.0065 per hour for every 10 units of write capacity, so 24 * 75 * $0.0065 is $11.70 per day for writes for each table. This means the write capacity would cost $23.40 per day for edgestore and graphindex together. The reads could be set to 1 read per second for each of the tables, making the read cost 2 * 24 * $0.0065 = $0.312 for both tables per day. If your AWS account is new, the reads would fall within the free tier, so effectively, you would only be paying for the writes.
Another aspect of DynamoDB pricing is storage (https://aws.amazon.com/dynamodb/pricing/). If you write 750 columns per second, that is 64.8 million items per day to one table, that means 1.9 billion (approximately 2 billion) items per month. The average number of items in the table in a month is then 1 billion. If each items averages out to 412 bytes, and there is 100 bytes of overhead, then that means 1 billion 512 byte items are stored for a month, approximately 477 GB in a month. 477 / 25 rounded up is 20, so storage for the first month at this load would cost 20 * $0.25 dollars a month. If you keep adding items at this rate without deleting them, the monthly storage cost will increase by approximately 5 dollars per month.
If you do not have super nodes in your graph, or vertices with a relatively large number of properties, then the writes to the edgestore will be distributed evenly throughout the partition key space. That means your table will split into 2 partitions when it hits 10GB, and then each of those partitions will split into a total of 4 partitions when they hit 10GB, and so on and so forth. the nearest power of 2 to 477 GB / (10 GB / partition) is 2^6=64, so that means your edgestore would split 6 times over the course of the first month. You would probably have around 64 partitions at the end of the first month. Eventually, your table will have so many partitions that each partition will have very few IOPS. This phenomenon is called IOPS starvation. You should have a strategy in place to address IOPS starvation. Two commonly used strategies are 1. batch cleanup/archival of old data and 2. rolling (time-series) graphs. In option 1, you spin up an EC2 instance to traverse the graph and write old data to a colder store (S3, Glacier etc) and delete it from DynamoDB. In option 2, you direct writes to graphs that correspond to a time period (weeks - 2015W1, months - 2015M1, etc). As time passes, you down provision the writes on the older tables, and when time comes to migrate them to colder storage, you read the entire graph for that time period and delete the corresponding DynamoDB tables. The advantage of this approach is that it allows you to manage your write provisioning cost with higher granularity, and it allows you to avoid the cost of deleting individual items (because you delete a table for free instead of incurring at least 1 WCU for every item you delete).

Wow, thank you so much for all this information Alexander!! It is very nice of you to share so much knowledge with the community, I really appreciate it.
I will now spend some days processing this information and reading up on some of the concepts you mentioned.
Thank you again!

This response goes into DynamoDB pricing considerations in great detail.
I do not want it to get lost in a closed GitHub issue, so can you post it to SO please?
Thanks
Alex


Mongoid truncation broken in 1.4.0

require 'database_cleaner'
DatabaseCleaner[:mongoid].strategy = :truncation
# => DatabaseCleaner::UnknownStrategySpecified: The 'truncation' strategy does not exist for the mongoid ORM!  Available strategies: truncation

Seems to be a regression in 1.4.0. The 1.3 series is working fine.

:+1:
I'm experiencing the same issue after updating the gem.

Same here.

Rolled back to 1.3.0 and works fine. 1.4.0 is broken as above.

I am just addd gem 'mongoid-tree' in my Rails Gemfile. it's ok now.
You can try this .

adding gem 'mongoid-tree' works for me too, if it's a required gem it should be documented.
thanks

just adding a +1 here

this bug should be fixed by this commit:
https://github.com/DatabaseCleaner/database_cleaner/commit/88479b71c9b4b82872fc037d54a26128edab6b7f

please release a new version without the dependency on mongoid-tree, it's been 2 months already since this issue is known.
thx!

:+1:

I believe this is the same as https://github.com/DatabaseCleaner/database_cleaner/issues/299, where @etagwerker just promised a new release. Maybe this one can be closed.

This issue looks to be solved (latest version; 1.4.1). Why don't you close it? @iangreenleaf

or @etagwerker could you please close this issue?


String quotes are swallowed.

Hi,
An invocation command like the following fails on Windows because the escaped quotes are getting swallowed:
rez-env maya -- maya -command "print(\"Hello!\")"
In Maya:
print(Hello!);
// Error: print(Hello!);

I'm being hit by this one again, this time with subprocess/os.system invocation. Anybody can confirm the issue? It is consistent on Windows for me and basically I cannot use any strings in commands.

os.system("rez-env maya -- maya -command "print('Hello!')"")

print(""Hello!");
// Error: print(""Hello!"); //
// Error: Line 1.13: Syntax error //

Summoning Thorsten (@instinct-vfx)! :)

Hm. Not using Maya and have no package yet so can not try this specific case. Generally nested quotes in the shell and in batchfiles are a bit of a mess to get right on windows.
Did you get your first problem fixed then? Because the latter is calling rez i assume it is not being run from within rez. If you are working with rez from python is there any reason why you are not using the API instead of os.system calls. Just curious, not meant as a solution to the problem.

Disclaimer: My testing may be totally not helpful because we are running rez from a central location with a rather complex set of batch wrappers calling each other. Things that work for me do not necessarily work in a standard setup and vice versa :)

Generally nested quotes in the shell and in batchfiles are a bit of a mess to get right on windows.

I think this is what I'm experiencing right now :)

If you are working with rez from python is there any reason why you are not using the API instead of os.system calls

I certainly could, as this is part of a build script that does many command line things, I used command line calls for consistency. I might give it a whirl though as it will probably help.

To escape a double quote you need a second double quote on the Windows command line. The backslash escapes it from rez.
rez-env maya -- maya -command "print(\"\"Hello!\"\"\")"
rez-env python -- python -c "print(\"\"Hello!\"\")"
Maya is even more special it seems because you need triple quotes for some reason.
My approach to figuring things out:
Resolve an environment and test your command in there before running the command through rez-env.
rez-env python -> python -c "print(""Hello!"")"
rez-env maya -> maya -command "print(""Hello!""")"
When this works it's easier to know what has to be escaped for rez.


[CI] Update CI docker and suppress warnings

@JehandadKhan @atamazov
We have seen many issues with this update and some of them look like:
[2023-08-20T06:24:00.398Z] /long_pathname_so_that_rpms_can_package_the_debug_info/src/out/ubuntu-20.04/20.04/build/hip-on-rocclr/hipamd/src/hiprtc/hip_rtc_gen/hipRTC_header.h:1034:49: error: empty expression statement has no effect; remove unnecessary ';' to silence this warning [-Werror,-Wextra-semi-stmt]
[2023-08-20T06:24:00.398Z]       __hip_assert(false && "invalid tile size");
[2023-08-20T06:24:00.398Z]                                                 ^

@junliume BTW I see that submodule fin updated from ebf9b355da09e10cb77e4b163a02e57be954095e to b2f3f4db3c3d7dd757e6d9e68719a780d8114dfa. Diff seems empty. Please revert this change.

We have seen many issues with this update and some of them look like:

@junliume I've been addressing many of the rocm 5.7 related make check issues in an MIOpen-internal branch, I'll send you.

@junliume

@JehandadKhan @atamazov We have seen many issues with this update and some of them look like:
[2023-08-20T06:24:00.398Z] /long_pathname_so_that_rpms_can_package_the_debug_info/src/out/ubuntu-20.04/20.04/build/hip-on-rocclr/hipamd/src/hiprtc/hip_rtc_gen/hipRTC_header.h:1034:49: error: empty expression statement has no effect; remove unnecessary ';' to silence this warning [-Werror,-Wextra-semi-stmt]
[2023-08-20T06:24:00.398Z]       __hip_assert(false && "invalid tile size");
[2023-08-20T06:24:00.398Z]                                                 ^

This seems like a a problem in the implementation of the precompiled HIP headers. I do not know what __hip_assert is, but I suspect that it should not expand to nothing  when NDEBUG is defined. If I am right, then the simplest way to fix this in the HIP compiler/library is to define that macro like this state-of-the-art thing:
#ifndef NDEBUG
#define __hip_assert(ex) ... // normal definition of assert
#else
#define __hip_assert(ex) (void(0))
#endif

The code like (void(0)); should not produce any warning. Alternatively, hipRTC_header.h should observe NDEBUG at line 1034 and do not use __hip_assert when NDEBUG is defined.
Again, AFAICS this happens in the PCH, not in our code. I recommend informing the guys who are responsible for PCH, then silence the warning in MIOpen permanently (I do not think it's worth a full-featured "workaround" stuff) and forgetting about this.

@junliume

@JehandadKhan @atamazov We have seen many issues with this update and some of them look like:
[2023-08-20T06:24:00.398Z] /long_pathname_so_that_rpms_can_package_the_debug_info/src/out/ubuntu-20.04/20.04/build/hip-on-rocclr/hipamd/src/hiprtc/hip_rtc_gen/hipRTC_header.h:1034:49: error: empty expression statement has no effect; remove unnecessary ';' to silence this warning [-Werror,-Wextra-semi-stmt]
[2023-08-20T06:24:00.398Z]       __hip_assert(false && "invalid tile size");
[2023-08-20T06:24:00.398Z]                                                 ^

This seems like a a problem in the implementation of the precompiled HIP headers. I do not know what __hip_assert is, but I suspect that it should not expand to nothing when NDEBUG is defined. If I am right, then the simplest way to fix this in the HIP compiler/library is to define that macro like this state-of-the-art thing:
#ifndef NDEBUG
#define __hip_assert(ex) ... // normal definition of assert
#else
#define __hip_assert(ex) (void(0))
#endif

The code like (void(0)); should not produce any warning. Alternatively, hipRTC_header.h should observe NDEBUG at line 1034 and do not use __hip_assert when NDEBUG is defined.
Again, AFAICS this happens in the PCH, not in our code. I recommend informing the guys who are responsible for PCH, then silence the warning in MIOpen permanently (I do not think it's worth a full-featured "workaround" stuff) and forgetting about this.

@atamazov  I agree, this should be fixed in the runtime headers

@junliume I am working on this. Please expect update soon.

@junliume  I think that https://github.com/atamazov/MIOpen/blob/ci_rocm57_ata1 is ready for merging here.

Fixed potential UB in KernelArgs detected by sanitizer.
WORKAROUND_FAULTY_HIPMEMGETINFO_VEGA_NAVI2X introduced.

If hipMemGetInfo fails, issues warning message and returns "fake" 16 GiB value. For now, this allows for the continued use of gfx906 and Nav21 in development and testing.
W/A is GPU-agnostic because we do not know the GPU type in the place where the W/A is applied.
W/A may be considered as potentially dangerous because it always camouflages hipMemGetInfo failures. However, we're safe as long as hipMemGetInfo works fine on all supported GPUs (as it should, as the HIP runtime must be well tested before use with MIOpen).

[CI] Fix: Applied WORKAROUND_ISSUE_1148 to a 'Fp16 Hip All gfx1030' stage.

This is expected to prevent random failures on Navi21 nodes. My Navi21 system fails quite consistently (but randomly ;) without this W/A.

[CI] Quality: Removed unnecessary params.TARGET_NAVI21 from AnyGPU stages.
Merged recent changes from ci_rocm57
Merged recent changes from develop, conflicts resolved.

🌀 Testing
Tested on Navi21 and gfx906/60 (Radeon VII) systems. Note that my Navi21 system has the "hipMemGetInfo" issue, but my Radeon VII doesn't. Tried the followed tests on both systems:

Debug build with sanitizers, Smoke FP32 test - PASSED
Release build, Full FP16 test - PASSED

@junliume b59403dd0eea0fa36fc6f1e778434104555a0d41 is merged into https://github.com/atamazov/MIOpen/blob/ci_rocm57_ata1.

WORKAROUND_FAULTY_HIPMEMGETINFO_VEGA_NAVI2X

@atamazov  I am not reproducing the hipGetMemoryInfo() error with 5.7 RC3 docker (at least so far), but indeed many CI nodes are not as stable now, and I have asked these base OS to be updated.

@junliume Are you going to merge https://github.com/atamazov/MIOpen/blob/ci_rocm57_ata1 into this branch? If yes, then I will double-check if hipMemGetInfo problem is resolved in RC3 and narrow (or remove) the W/A.

@junliume Are you going to merge https://github.com/atamazov/MIOpen/blob/ci_rocm57_ata1 into this branch? If yes, then I will double-check if hipMemGetInfo problem is resolved in RC3 and narrow (or remove) the W/A.

I wish to and need to figure out how to sync from ci_rocm57_ata1 ...

I see, let me check if merge finished cleanly...

@junliume 🆗

@junliume hipMemGetInfo problem is NOT resolved in 5.7 RC3. This PR can be merged as is.

@junliume @JehandadKhan [Notice] Interesting detail:
Both RC1 and RC3 report HIP version 5.7.31921 in hip_version.h which is not good. However, CMake (hip-config-version.cmake) reports different numbers:

RC1: 5.7.23302
RC3: 5.7.23341

"Thanks" to some problems with hip_version.h in some old versions of ROCm we use CMake machinery as a workaround, and the library is able to distinguish between RC1 and RC3 (even if this is not necessary yet).

Ping @JehandadKhan and @atamazov for review after CI has passed.


Listener: introduce filter chain context

Description: Introduce filter chain context.
Goal: Support the future work of adding filter chain without drain the whole listener, and deleting one filter chain by draining only the connection associated with the deleted filter chain.
The ListenerFactoryContext should cover FilterChainFactoryContext, and filter chain context should cover the life of all the associated connections referring to the filter chain.
In this PR the filter chain contexts are not yet destructed independently. I have follow up PRs to release the power of filter chain contexts.
Risk Level: LOW
Testing: unit test
Addressing https://github.com/envoyproxy/envoy/issues/4540  3/N

/assign @htuch

In this version, network filters doesn't actually need FilterChainContext... Will update the PR.
/wait

/wait

Ack.

I'll leave this PR to @mattklein123 as he is the ListenerManager expert, unassigning.

Needs master merge and then I will take another look.
/wait

@mattklein123  Thanks! Merging offline...

Merged and tests are passing consistently on my machine

@junr03 I'm assigning you to help review this also as per my other comment this change will help with your API listener change. Thank you.

/retest

resolving conflict

/azp run envoy-macos

@junr03  Thank you! I just run the local tests on linux. Comparing with master, this PR doesn't introduce per cluster memory usage bump. List my number here (both master and PR returns the same value)
YMMV
[ RUN      ] IpVersions/ClusterMemoryTestRunner.MemoryLargeClusterSizeWithFakeSymbolTable/IPv4
test/integration/stats_integration_test.cc:284: Failure
Expected equality of these values:
  m_per_cluster
    Which is: 43464
  43413

[  FAILED  ] IpVersions/ClusterMemoryTestRunner.MemoryLargeClusterSizeWithFakeSymbolTable/IPv4, where GetParam() = 4-byte object <00-00 00-00> (1381 ms)
[ RUN      ] IpVersions/ClusterMemoryTestRunner.MemoryLargeClusterSizeWithRealSymbolTable/IPv4
test/integration/stats_integration_test.cc:333: Failure
Expected equality of these values:
  m_per_cluster
    Which is: 35598
  35548


emacs: fix for AppKit detection on Darwin

Emacs wasn't building, because it couldn't detect AppKit/AppKit.h.
It turned out it could find the header, but the configure check was relying on some things that were brought in by cf-private.

Closing: A better way to fix this has recently been merged to master.


Add SSSOM mappings

MOD recommends and/or defines terms for semantic artefact descriptions, and these terms usually have other equivalent or similar terms that may be used in semantic artefacts descriptions in the wild.
This issue is about providing SSSOM mappings between the MOD terms and other related terms, facilitating the conversion between existing semantic artefact descriptions to an MOD view of the description.

The mappings will be made available in the new repo: https://github.com/FAIR-IMPACT/MOD-mappings


Don't rely on the npm_execpath env variable to determine which package manager to use

The npm_execpath env variable isn't set if you install the package manager in a way other than npm i -g [yarn|pnpm].
This PR just checks if pnpm --version or yarn --version successfully runs. If so, it picks whichever command didn't fail. If both commands fail, we fall back to NPM.

Julian, "This is only needed the very first time we create a Webstone app/plugin. Any other time, we can determine the package manager based on the available lock file."


Update nodejs to match staging/prod exactly

Codecov Report

Merging #462 (de6c8e9) into develop (4c696d3) will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #462      +/-   ##
===========================================
+ Coverage    98.96%   98.98%   +0.01%
===========================================
  Files          107      112       +5
  Lines         5819     5897      +78
  Branches         0        8       +8
===========================================
+ Hits          5759     5837      +78
  Misses          60       60


[ICON] WhatsApp

Icon Request

Icon name: WhatsApp
Use case: Sharing content
Screenshots of similar icons:

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

Any updates on this icon? Was surprised that it was not in the library...

Thank you!

Hey, I'll try to include it in the upcoming updates. The request hasn't received any reaction on support, so its priority has decreased over time.


[feature]Is possible to combine all subpages into one pdf?

Just like a pdf book

Not at the moment, there are ways to do this, from other programs like Adobe Acrobat. An older version of this tool, using markdown export has a book function, but it doesn't work well.
https://github.com/ganeshh123/notion-pdf-export/releases/tag/1.1.0

Thanks. By the way, any plan for emoji support?

Depends on how this goes:
https://github.com/wkhtmltopdf/wkhtmltopdf/issues/2913
The conversion is done using another program

Just like a pdf book

A simple script to merge all pages into one pdf after all subpages are exported.
Used https://github.com/mstamy2/PyPDF2
by comment /PyPDF2/pdf.py , line 1512,1513 to aviod the error " PyPDF2.utils.PdfReadError: Unexpected destination '/__WKANCHOR_2' "
https://github.com/Rickyyy-zh/notion-pdf-export/blob/Rickyyy-zh-patch-1/merge_wholedocuments.py


Add some proper closing tests

Codecov Report

Merging #136 into master will increase coverage by 0.24%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #136      +/-   ##
=========================================
+ Coverage   91.26%   91.5%   +0.24%
=========================================
  Files          24      24
  Lines        1602    1601       -1
=========================================
+ Hits         1462    1465       +3
+ Misses         79      77       -2
+ Partials       61      59       -2

Flag
Coverage Δ

#dot
91.5% <100%> (+0.24%)
:arrow_up:

#dot_data
65.78% <ø> (ø)
:arrow_up:

#dot_sync
87.37% <ø> (+0.66%)
:arrow_up:

#dot_undo
100% <ø> (ø)
:arrow_up:

#dot_x_db_mongodb
81.17% <ø> (ø)
:arrow_up:

#dot_x_db_mysql
87.5% <ø> (ø)
:arrow_up:

#dot_x_db_pg
87.95% <ø> (+2.4%)
:arrow_up:

#dot_x_journal
99.28% <100%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

x/journal/subscription.go
100% <ø> (ø)
:arrow_up:

x/journal/model.go
98.5% <100%> (-0.03%)
:arrow_down:

sync/reconciler.go
89.84% <0%> (+1.56%)
:arrow_up:

x/db/pg/pg.go
87.95% <0%> (+2.4%)
: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 5becee7...44708b0. Read the comment docs.


Getting an error using this extension

When I try to Expand or Refresh the Spool File Extension I get the error in the screen shot.  I tried this on both my companies IBMi and on pub400 with 2 different computers.  I tried to look and see if I could see any log that would point me to more information but didn't find one but might not have looked in the correct places.

I'm not a JS/TS person but I was going to try and download the extension and run it in debug to see if I could figure out where it is failing but looks like it might have some pre-reqs as I'm getting several errors.

Thanks guys.


Refactor links generation code

Coverage decreased (-0.03%) to 83.799% when pulling 7acd161500cff818d934efc26ddf47105622a1e2 on 785-content-type-issue into 1fd9cdde3e988403fdcf909cf9fafc41fb49766e on master.

Coverage decreased (-0.02%) to 83.81% when pulling ddde72dd19e850a40abe835ba01954104381fbc3 on 785-content-type-issue into 1fd9cdde3e988403fdcf909cf9fafc41fb49766e on master.

Coverage decreased (-0.02%) to 83.81% when pulling 8dc989fa95df0b246c9c5e4cd270cd97cbd0df17 on 785-content-type-issue into 1fd9cdde3e988403fdcf909cf9fafc41fb49766e on master.


Update FMS: Fix for flush issue on macOS

This is a patch update to FMS which changes all the call flush() in FMS to flush() to allow Intel Fortran on macOS to work (@tclune knows more of the details...something to do with the shared object libraries).
I have tested with AMIP, MOM5, and MOM6 and all run and seem zero-diff. But, it would good if @yvikhlya or @sanAkel can make sure nothing broke in coupled-space. (I'd be very sad if changing flush statements changed the model state. 😄 )

I certainly don't know much beyond this.  Basically Intel appears to share some code between the FLUSH intrinsic and the Intel subroutine extension FLUSH.   Somewhere or another there is a check in their code that disallows negative unit numbers even though those now arise often with the NEWUNIT argument to open.    The exact circumstances appear to be very subtle though.

@mathomp4 , @tclune I can double check this next week

@sanAkel Thanks! We can pull this in as needed, I suppose. Though it's not crucial at the moment.


no data on UART with raspi 400

Hello,
I was having trouble getting UART1 to respond on my raspi 400, so I tried your code to check if I was missing something obvious; but there is still no output.  :-(
Some things I checked:

I used the serial console of the stock raspbian build to verify that the UART is electrically connected properly (the console was usable)
I used /proc/iomem in raspbian to verify that the raspi 400 does list its UART at the same memory locations as the raspi 4.
I added some UART output to your part5-framebuffer example to verify that the kernel does boot up and does execute the relevant instructions (the framebuffer output was visible).
I checked that core_freq_min=500 and enable_uart=1 were included in the config.txt

It's possible that there is some other obscure difference between the raspi 4 and raspi 400, so I am considering splashing out on a raspi 4B to test with....
I wonder if you have any ideas?

That diagram seems to be using wiringpi's pin numbering scheme, which is similar-yet-different to the Broadcom one (see the table on http://wiringpi.com/pins/).  On https://pi4j.com/1.4/pins/rpi-4b.html, they list the same pinout for the 4B model.  Very confusing, but at least the pinouts are the same.
I will have another think today and then decide whether to buy a 4B for testing.


Dependency Issue

Using Java 2.12.8 I'm unable to build the latest milestone release. Looks like maybe the kafka-avro-serializer is no longer available, and needs to be bumped to 5.3.0?
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	::          UNRESOLVED DEPENDENCIES         ::
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	:: io.confluent#kafka-avro-serializer;5.2.2: not found
[warn] 	Note: Unresolved dependencies path:
[warn] 		io.confluent:kafka-avro-serializer:5.2.2
[warn] 		  +- com.ovoenergy:fs2-kafka-vulcan_2.12:0.20.0-M2

You need to add the Confluent resolver to your build.sbt.
resolvers += "confluent" at "https://packages.confluent.io/maven/"

This should be in the getting started documentation, will update.

That got it! Thanks for your time.

https://github.com/ovotech/fs2-kafka/pull/161 has been merged and published to clarify this.


add github action about intellij plugin

add github action to build intellij plugin
result:
the intellij plugin jar will show at the release page.
ps:
install plugin from local disk. depends on local node

does the dist.zip of intellij plugin is build realtime in github action?


Research for Docker base image recommendation

Goals:

Use Docker base images that have an LTS policy that are compatible with EdgeX LTS policy
Use Docker base images that have low "CVE churn" so that we need to update base images less frequently.

Motivation:
CVE's such as CVE-2021-36159, which caused critical vulnerability in Alpine base images.
Most of the time we only need a base image to run the entrypoint scripts.
Research Alpine vs Busybox vs Distroless base images.
Prepare recommendation on preferred base image going forward.

@cloudxxx8  That is a good point.  With this approach, we could simply force a rebuild of the containers, with no changes at all to the underlying github.

Closing this out since all of our dockerfiles now have an apk upgrade when we had to close a CVE for Napa release where the base image hadn't been updated in 2 months.


improvement: allow manual passed key prop from b-table-column in b-table

Overview of the problem
Buefy version: 0.5.0
Vuejs version: 2.4.2
OS/Browser: Various
Description
I am creating a b-table with conditional column, the scopedSlots prop of  is a function that returns different b-table-columns on different status.
e.g. show/hide or change columns on a specific condition.
While I found b-table component doesn't update properly when I am changing scopedSlots  prop passed to b-table.
Steps to reproduce

set b-table's scopedSlots.default to a function getSlots that returns an array of b-table-column
make getSlots return 3 columns at condition A, and return 4 different columns at condition B.
change the condition for getSlots.

Expected behavior
b-table should be updated to match correct columns returned by getSlots function
Actual behavior
b-table is not being updated correctly.
Possible reason and solution
I found b-table uses scopedSlots array's index as key, thus a key prop value is not always binded to the same b-table-column and table columns could not update properly by Vue.
A piece of related buefy source code: https://github.com/rafaelpimpa/buefy/blob/79f7119174946774926679b2ed87898375ecdc11/src/components/table/Table.vue#L44-L49
I am wondering if t-table could accept manual key prop from b-table-column component, and fallback to index when there is no key prop provided. If this is acceptable, I could make a quick PR for it.

Can you make a repro for this? I'm working on something similar and want to test.

Thanks for the response. I'd make one on the weekend.

Here is the repo: https://github.com/frantic1048/buefy-issue-238
The symptom is similar to #239

Just use the visible prop instead of removing or pushing to the slots prop:
slots.push([
    <b-table-column visible={this.slotsType === 'default'} field="b" label="B" width="40">
        { props.row.b }
    </b-table-column>,
    <b-table-column visible={this.slotsType === 'default'} field="c" label="C" width="40">
        { props.row.c }
    </b-table-column>,
    <b-table-column visible={this.slotsType === 'default'} field="d" label="D" width="40">
        { props.row.d }
    </b-table-column>,
    <b-table-column visible={this.slotsType !== 'default'} field="e" label="E" width="40">
        { props.row.e }
    </b-table-column>
])

Just use the visible prop instead of removing or pushing to the slots prop

I know this method. It helps when all columns are predictive, but not with dynamically generated ones (That's the case I concern about).
I just read the fix in 1eb35feb7d76715a82e3b557cb2dc2465ccf4a9b , it works.
Thanks for the sweet UI :)


Secondary Storage server fails if multiple Management servers are defined

ISSUE TYPE

Bug Report

COMPONENT NAME

systemvm agent (secondary storage vm)

CLOUDSTACK VERSION

<IP_ADDRESS>

CONFIGURATION

Global Setting
Key: host
Value: IP_ADDR1,IP_ADDR2 (eg: <IP_ADDRESS>,<IP_ADDRESS>)
OS / ENVIRONMENT

SUMMARY

When setting the host global setting to multiple addresses (separated by a comma), the secondary storage systemvm alerts and fails to further operate.
STEPS TO REPRODUCE

Change the global settings option "host" to multiple addresses as defined in the description field.
Restart the Secondary Storage systemvm.

EXPECTED RESULTS

Secondary Storage systemvm comes online and proceeds with standard operation (template management, etc)

ACTUAL RESULTS

Agent went to alert state and never performed secondary storage operations.

From cloud.log:
2018-11-05 22:07:04,813 WARN  [storage.resource.NfsSecondaryStorageResource] (maa
in:null)  destIp is not a valid ip address or cidr destIp=<IP_ADDRESS>,<IP_ADDRESS>
2018-11-05 22:07:04,854 ERROR [storage.resource.NfsSecondaryStorageResource] (maa
in:null) Couldn't cast null to integer
2018-11-05 22:07:04,860 INFO  [storage.template.DownloadManagerImpl] (main:null))
 DownloadManager: starting additional services since we are inside system vm
2018-11-05 22:07:04,866 ERROR [storage.resource.NfsSecondaryStorageResource] (maa
in:null) Couldn't cast null to integer

/cc @borisstoyanov @nvazquez

@ccheline couple of questions,

have you seen this with just the SSVM?
is the management server running and properly configured to work as a management cluster on all the IPs in the list
is this a clean install or upgrade?

Can you test with <IP_ADDRESS> or latest? I'm unable to reproduce this @ccheline


CABLE PIcontrol suite

|    by<EMAIL_ADDRESS>
u-bf481 is the candidate coupled model PPI control suite with CABLE. This ticket documents the configuration set up and spinup.

Issue migrated from trac:367 at 2024-01-31 18:34:34 +1100

<EMAIL_ADDRESS>changed status from new to accepted

<EMAIL_ADDRESS>changed _comment1 which not transferred by tractive

<EMAIL_ADDRESS>commented

History
u-ax871 started from JULES PI control u-at658 at model time 02010101. D2 iceberg scheme. CABLE code /home/599/mrd599/mosrs/jules/r11864_tvrad_fix_ih_merge. This was a manual merge of branches/dev/ianharman/vn10.6_CABLE_r11864@12302 and branches/dev/martindix/r11864_tvrad_fix@12260.
u-ba648 started from u-ax871 03010101. Configuration as for u-ax871 but with ocean SW changes and CABLE changes

Medlyn stomatal conductance
PFT dependent root depth
Albedo calculated with correct zenith angle rather than timestep lagged.

CABLE code /branches/test/martindix/r12417_albedo_fix, branches/dev/Share/vn10.6_CABLE@12579
u-bb194 started from u-ax871 03010101. Configuration as for u-ba648 but with CABLE changes

Antarctic & Greenland snow albedo increased to NIR=0.65, VIS=0.9.
Increased permanent snow roughness
Vegetation convex parameter

CABLE code is branches/dev/Share/vn10.6_CABLE@13036.
u-bd052 started from u-ax871 03010101. Configuration as for u-bb194 but using NCAR LAI, /projects/access/data/ancil/access_cm2_n96e/O1/cable_vegfunc_ncarlai.anc. CABLE code branches/dev/Share/vn10.6_CABLE@13036
u-be106 started from u-bd052 04010101. Configuration as for u-bd052 but with ocean albedo scaling set to 1.02. CABLE code branches/dev/Share/vn10.6_CABLE@13036, branches/dev/martindix/vn4.7_access_cm2@12754. Note that the latter branch is just to handle the albedo scaling namelist variable.
Restart reproducibility experiments with u-be106 showed a sensitivity to the timestep number. This was traced to CABLE incorrectly scaling precipitation rates by timestep number rather than timestep length in one part of the code.
New suite u-bf481 with this fixed started from u-be106 05010101.
CABLE code branches/dev/martindix/r13036_precip_scaling@13918 branches/dev/martindix/vn4.7_access_cm2@12754

<EMAIL_ADDRESS>set owner to mrd599

<EMAIL_ADDRESS>changed _comment0 which not transferred by tractive

<EMAIL_ADDRESS>changed _comment1 which not transferred by tractive

<EMAIL_ADDRESS>changed _comment2 which not transferred by tractive

<EMAIL_ADDRESS>commented

Candidate PI Control CMIP6 STASH from Roger's u-bf022.
Stash Table
Export STASH from u-bf022 using
cd ~rb4844/roses/u-bf022/app/um
rose macro stash_copy.STASHExport

In target suite, use rose-edit to remove the existing STASH and then run the metadata StashProfilesRemoveUnused macro. Outside rose-edit
rose macro stash_copy.STASHImport

Change domain profiles dsoil to 6 levels, dpft to 13 and dtile to 17 for CABLE.
Add back the CABLE STASH package from the original suite plus a few extra tiled variables.
Set all files to reinitialise every calendar month.
MOM diagnostics changed to save conservative temperature as well as potential temperature and to set diagnostic SST to potential temperature. Also extra daily diagostics and other OMIP diagnostics requested by Abhi.
Run with these diagnostic changes restarted at 07000101.

<EMAIL_ADDRESS>changed _comment3 which not transferred by tractive

<EMAIL_ADDRESS>changed _comment1 which not transferred by tractive

<EMAIL_ADDRESS>changed _comment4 which not transferred by tractive

<EMAIL_ADDRESS>changed _comment0 which not transferred by tractive

<EMAIL_ADDRESS>changed _comment2 which not transferred by tractive

<EMAIL_ADDRESS>commented

As requested by Arnold, change CICE diagnostics from
f_ardg='x'

to
f_ardg='m'

Change applies from the start of year 719.

<EMAIL_ADDRESS>commented

FAFMIP setup changed results so not using this at the moment.

<EMAIL_ADDRESS>changed _comment0 which not transferred by tractive

<EMAIL_ADDRESS>changed _comment3 which not transferred by tractive

<EMAIL_ADDRESS>commented

Extra ocean diagnostics from Abhi saved from start of year 770.

<EMAIL_ADDRESS>commented

CMIP6 requires extra diagnostics for the PI control period equivalent to 1985-2014. This is model years 835-864. These were copied from Roger's u-bd266 and added with a package name PIextra to make it simple to turn them off again later.
Model warm-restarted at year 835 with these changes.

<EMAIL_ADDRESS>commented

Restarted again at 865 with the PIextra package turned off.

<EMAIL_ADDRESS>changed _comment0 which not transferred by tractive

<EMAIL_ADDRESS>commented

Extra ocean diagnostics from Abhi. Extra UM diagnostics for energy fluxes over sea ice and land tiles. Also removed 8,202 LAND SNOW MELT HEAT FLUX because it's not saved correctly by CABLE. It can be calculated from 8,231 LAND SNOW MELT RATE.
Changes apply from start of year 724.
Added ice category fields, SH and LH fluxes over open sea. Also tropopause height and density for ozone re-distribution.
Changes apply from start of year 727.

<EMAIL_ADDRESS>changed _comment0 which not transferred by tractive

<EMAIL_ADDRESS>commented

Found a problem with diagnostic 10m wind speed that affected sea-salt aerosol. New run with this and other sea-salt aerosol fixes u-bi889 started from year 850 of bf481.
Official start of PIcontrol DECK run is year 950.
Full history

u-ax871 from year 201 of at658 (JULES)
u-bd052 from year 301 of ax871
u-be106 from year 401 of bd052
u-bf481 from year 501 of be106
u-bi899 from year 850 of bf481
Official DECK start at 950 of u-bi889

<EMAIL_ADDRESS>changed _comment1 which not transferred by tractive


Please add ModRiskFileTypes

Your project is very useful for me, thanks.
Please add ModRiskFileTypes, the command is follow:
reg add HKCU\Software\Microsoft\Windows\CurrentVersion\Policies\Associations /v ModRiskFileTypes /t REG_SZ /d .bat;.exe;.reg;.vbs /f

Hi @ropean! Thanks for your advice. I checked this value in Win10, it isn't a default options. So, I will not add this. But, you can fork this repo for yours.


Fix: empty payload

Checklist

[x] I have read the contribution guidelines.
[x] I added or modified the existing tests to cover the change (we do not allow our test coverage to go down).
(n/a) If this is a modification that impacts the behavior of a public API

[x] I edited the corresponding document in the devdoc folder and added or modified requirements.

Reference/Link to the issue solved with this PR (if any)
#1081
Description of the problem
Method request with no payload were raising a json decode issue like mentioned in #1081.
Description of the solution
Added a oneliner to manage empty payloads.
MQTT tests updated too.
HTTP tests: I did not find the way to test this specific change.

/azp run


chore(package): update eslint to version 4.4.1

Closes #17

Coverage remained the same at 100.0% when pulling<PHONE_NUMBER>0f1f6a16b0f1ff125b0778c6b19606 on greenkeeper/eslint-4.4.1 into 9dbecc7ac0ceef1a08f290d21dc3b7b28dfc8ba1 on master.

Coverage remained the same at 100.0% when pulling<PHONE_NUMBER>0f1f6a16b0f1ff125b0778c6b19606 on greenkeeper/eslint-4.4.1 into 9dbecc7ac0ceef1a08f290d21dc3b7b28dfc8ba1 on master.

Coverage remained the same at 100.0% when pulling 66c6d57efa482752bd4167ff1d47e8eaf944d992 on greenkeeper/eslint-4.4.1 into 9dbecc7ac0ceef1a08f290d21dc3b7b28dfc8ba1 on master.

Coverage remained the same at 100.0% when pulling 66c6d57efa482752bd4167ff1d47e8eaf944d992 on greenkeeper/eslint-4.4.1 into 9dbecc7ac0ceef1a08f290d21dc3b7b28dfc8ba1 on master.

Coverage remained the same at 100.0% when pulling 66c6d57efa482752bd4167ff1d47e8eaf944d992 on greenkeeper/eslint-4.4.1 into 9dbecc7ac0ceef1a08f290d21dc3b7b28dfc8ba1 on master.


Js api calls

3 APIs are in. One gets the lat and lon, the other gets the weather, and the third gets the user's lat and lon.
Distance calc is in progress.

Wow, very cool parker

dope