Disable overview ruler entirely

Can you add an option to disable overview ruler entirely?
It's located underneath the right scrollbar, and I find it distracting, as there's symbols changing all the time.
Version: 1.40.1
Commit: 8795a9889db74563ddd43eb0a897a2384129a619
Date: 2019-11-15T10:46:02.883Z
Electron: 6.1.2
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Linux x64 4.15.0-70-generic
I've consulted the link below, no effect:
https://stackoverflow.com/questions/59002585/how-can-i-remove-symbols-on-right-scrollbar-of-vscode

Looking forward to this feature!

https://stackoverflow.com/questions/50593516/colored-pixels-in-scrollbar-in-vs-code
You could probably use "workbench.colorCustomizations" to make the colors transparent

Played around with it, and reached a config that I like
    "workbench.colorCustomizations": {
        // OverviewRuler
        "editorOverviewRuler.border": "#0000",
        /// Search command
        "editorOverviewRuler.findMatchForeground": "#fff4",
        "editorOverviewRuler.rangeHighlightForeground": "#0000",
        "editorOverviewRuler.selectionHighlightForeground": "#0000",
        "editorOverviewRuler.wordHighlightForeground": "#0000",
        "editorOverviewRuler.wordHighlightStrongForeground": "#0000",
        "editorOverviewRuler.modifiedForeground": "#0000",
        "editorOverviewRuler.addedForeground": "#0000",
        "editorOverviewRuler.deletedForeground": "#0000",
        //"editorOverviewRuler.errorForeground": "#0000ff",
        //"editorOverviewRuler.warningForeground": "#ff00ff",
        //"editorOverviewRuler.infoForeground": "#00ff00",
        "editorOverviewRuler.bracketMatchForeground": "#0000",
    }

I tired this so many ways. It does not seem to work all.
Version: 1.77.3 in apple Mac silicon chip
{
"zenMode.hideLineNumbers": false,
"workbench.colorTheme": "Visual Studio Dark",
"workbench.colorCustomizations": {
// OverviewRuler
"editorOverviewRuler.border": "#ffffff",
"editorOverviewRuler.width": 1, // change the width here
/// Search command
"editorOverviewRuler.findMatchForeground": "#ffffff",
"editorOverviewRuler.rangeHighlightForeground": "#ffffff",
"editorOverviewRuler.selectionHighlightForeground": "#ffffff",
"editorOverviewRuler.wordHighlightForeground": "#ffffff",
"editorOverviewRuler.wordHighlightStrongForeground": "#ffffff",
"editorOverviewRuler.modifiedForeground": "#ffffff",
"editorOverviewRuler.addedForeground": "#ffffff",
"editorOverviewRuler.deletedForeground": "#ffffff",
"editorOverviewRuler.errorForeground": "#ffffff",
"editorOverviewRuler.warningForeground": "#ffffff",
"editorOverviewRuler.infoForeground": "#ffffff",
"editorOverviewRuler.bracketMatchForeground": "#ffffff",
},
"[python]": {
"editor.formatOnType": true
}
}

Please include this feature

still not implemented and still annoying

still not implemented. I'm still waiting

If the workarounds that have been posted here don't work for your use case, maybe you can also give a try to the APC Customize UI++ extension, it allows you to customize the css of every part of the vscode UI, maybe it can help here.
Anyway, pardon me for the "mr know-it-all" comment, but imho "still not implemented"/"please fix" comments on github (like the last 3 comments) generally don't help, especially when done on a feature request. It only bother people who have subscribed to the issue by sending a notification for nothing. Worse, maintainers often end up locking the discussion to prevent useless spam. This would be sad.
In github if we really want a feature, I think all we can do is :

thumb-up the original issue so that it gains attention from the development team :

use the comments section to post additional informations that could either help the development team understand/resolve the issue, or help users finding a workaround
try to fork the repo, develop the feature and submit a PR in the hope it gets merged by the development team

kind regards

Really want this.


Speed hit on version 1.1.1 when using --join

Compiling single files is fine, but I'm noticing a big compiling speed difference between versions 1.0.1 and 1.1.1 when using the --join flag to compile ~45 .coffee files.

@adamyonk: has this issue been resolved?

No longer able to replicate.


Fix DL link

Signed-off-by: Roeland Jago Douma<EMAIL_ADDRESS>

Didn't worked?

Didn't worked?

Didn't worked?

Nope the 302 was not followed

Didn't worked?

Nope the 302 was not followed


Fix #390.

Oh no... I committed some code I only added for debugging, I'm closing the pull request and making another.


[Android] rendering  with Tilesheet.TILE_TRANS_2x2

Rendering of scaled Tilesheet via TILE_TRANS_2x2 with atlases (tested with 2k atlas) results in rendering of parts that are around target rectangle. Made a simple project to test it:
https://dl.dropboxusercontent.com/u/34170815/Projects/_temp/MobileScaleTest.zip
You'll see as white rectangle becomes bigger via scale, there will be red and green borders, which are placed around white pixel on atlas image.

Does this error occur on other targets or just Android?

Does not appears on neko/windows targets. Can't test it on iOS. (and this appeared not only on my device)

Are you able to provide screenshots of the problem? I also tried another two devices with different resolutions and still can't replicate the problem. I've now tried devices with resolutions of 480x800, 800x1280, 1080x1920 and 1440x2560, and Android versions ranging from Lollipop to Marshmallow.

 It supposed to render perfectly white rectangle, but I get adjacent pixels on atlas at corners.
And second device that definitely affected is:
Prestigio Multiphone PAP3350 duo
Android 4.2.2
320х480


Add HTML tile

Addresses #292
The editor works ok but there's some issue with it rendering content once a document is saved.
Possibly due to some configuration issue for the HTML/View.jsx part.
Will be working on this, but any pointers on where I should be looking at would be helpful!

Pull Request Test Coverage Report for Build 1528

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at ?%

Totals

Change from base Build 1525:
0.0%

Covered Lines:

Relevant Lines:
0

💛  - Coveralls

@tisto Yes I missed that. Updated changelog and fixed conflicts

@ajayns @nileshgulia1 I also fiddled around for prettyfy the output, but I'm afraid we would need an external trigger for it. We currently have no handler that executes something on tile blur, that might be handy... /cc @robgietema
FYI I used pretty npm package.

Now I think it's ready for prime time :)
/cc @nileshgulia1 @ajayns

@ajayns @sneridagh functionality looks great!!! Do we currently filter the input in any way or is that something that we have to take care of in the future?

@tisto I already thought about that, we can surely look for it in the future, but for now it's fine if we trust our editors, of course!


There should be an extension for overlays

The management of overlays should be moved to an extension. That would make it a lot easier to add new types of overlays and also keep the code cleaner. The extension should be a part of the current project so that old API wouldn't be broken. Just deprecate the old commands.
Overlays that should be part of this extension at the first stage:

Marker (from original code, see also PR27).
Polyline (from original code).
Polygon (from original code).
Info window (from orignal code.
Circle (see PR19 and PR16).

There should also be support for features missing from the original add-on:

All overlays should be clickable (with proper RPCs).
Investigate if drawing functionality could be a part of this extension too.
Markers: z-index, anchoring of icon (see PR27)
Caption support

Old issues closed when this was created: #6, #7, #9, #13, #14.

This project is no longer being maintained here. It has moved to https://github.com/parttio/Google-Maps-for-Vaadin
Thanks.


Update Documentation

Documentation should be updated to detail the entire use case of each component

Reach out to community to give us examples of instances where the documentation was not clear or complete for them. We could use the monthly forum to address this question.

@alessioventuriniAND @precious-onyenaucheya-ons Is there anything I need to do on this one yet? or is it a case of waiting to see if there's anything people say isn't clear?

We have released the new community pages that can be used to provide feedback and raise issue with the team. We also have the monthly forum meeting where the community can raise queries and issue with the team directly.


Ensure Eloquent\Collection returned after filtering

I encountered an error when the FilterBuilder tried to access Collection::load() on a Support Collection, and not an Eloquent Collection. This is related to #257.
I tracked it down locally and realised that I had indexed IDs, that did not exist in the database. When the ElasticEngine maps the Eloquent Collection data, if a model was not found, null would be returned. If this happens, the Eloquent Collection is converted to a Support Collection, thus causing the error. The Collection::filter() method is called right after map to filter out the nulled values, but the collection has already been converted at this stage.
This PR will ensure the returned collection is an Eloquent Collection.

Nice PR. Could you please update the tests to catch this situation? As soon as possible I'll check this PR in my sandbox.

Thanks @iget-esoares . Just pushed a test case. The Elasticsearch results returned two hits, but the newQuery chain will only return an array of a single model, thus causing the Collection::map() to return nulled values, and cause the conversion to Support\Collection.
If you want to see the conversion happening (and failing), you can return $values; in  ElasticEngine.php:334

Released in v4.2.1


Fix init.d script

fixes #16

Not sure where this goes but this blog post needs to be appended to include your change: http://docs.ghost.org/pl/installation/deploy/


Update cdn build to support typescript

Overview
Updating the cdn webpack config to support typescript files. With this change, JS files in the packages directory can be renamed with the .ts extension and the build will continue functioning as-is.
Related Github Issue
https://issues.newrelic.com/browse/NEWRELIC-5562
Testing
Update one of the files in the browser-agent-core package to have the .ts file extension and run npm run cdn:build:prod or any of the other build tasks. The build should complete with no changes to the output files.

What's the status of this post TS talk?

Since there is already TS in packages/browser-agent we should go ahead with this.

What's the status of this post TS talk?

Since there is already TS in packages/browser-agent we should go ahead with this.
Ok, could you merge/rebase this with the webpack changes from Ben pls

What's the status of this post TS talk?

Since there is already TS in packages/browser-agent we should go ahead with this.

Ok, could you merge/rebase this with the webpack changes from Ben pls

I don't think we will need it with @metal-messiah 's one agent poc.

Closing. This will not be needed with the one-agent work @metal-messiah is doing.


Skip lookup of process definition on key=null

Fixes #50

[ ] avoid calling child process provider twice by caching its result in CallableElement

debugger and logging confirm that the code in edfbac2 invokes the ChildProcessProvider twice for the case that there is a child process. That should be fixed.


[SPARK-36005][SQL] The canCast method of type of char/varchar is modified to be consistent with StringType

What changes were proposed in this pull request?
The canCast method of type of char/varchar is modified to be consistent with StringType
the method cast will change the type char/varchar to StringType
def cast(to: DataType): Column = withExpr {
val cast = Cast(expr, CharVarcharUtils.replaceCharVarcharWithStringForCast(to))
cast.setTagValue(Cast.USER_SPECIFIED_CAST, true)
cast
}
The canCast method of type of char/varchar  must   be consistent with StringType
Why are the changes needed?
Before I used stringType instead of char/varchar, my application code has the logic to judge using canCast. There was no problem before, but now it’s changed to char/varchar, and the judgment of canCast fails. If it doesn’t pass, I Need to change a lot of application code
Does this PR introduce any user-facing change?
no
How was this patch tested?
i add UT。

Can one of the admins verify this patch?

@zheniantoushipashi does it cause any user-facing behaviour change?

@HyukjinKwon
I think it only affects
the canCast method of object Cast, the method cast will change the type char/varchar to StringType, so i think the canCast method of type of char/varchar must be consistent with StringType

@HyukjinKwon   Only affect the rule of  ResolvePivot in Analyzer, the others have no effect

@HyukjinKwon


Spread cache keys when using Redis#mget

The signature of Redis#mget actually requires a splatted array of keys to fetch. Currently, the implementation works fine for the default redis gem, but the current behavior (non-spreading) does not work with redis-store. This patch should fix that.

Tests are just failing because the examples require a secret to run.


Remove requirements for unit testing

Note: Even though I put no-compat label, this PR does not break backward-compatibility. Backward compatibility was broken when testing framework was replaced with PyTest. I put the label simply to signify that.
This PR removes pytest and mock requirements from Chainer normal installation.
That would reduce the possibility of trouble for users resulting from migration to PyTest.
OTOH, some parts of chainer.testing package depend on pytest and mock. Simply removing requirements would result in ImportErrors when importing chainer.testing. To avoid that, I introduced is_available() functions to such modules.
This PR should be backported, because PyTest has introduced to stable branch as well.

LGTM. waiting for the Jenkins test finishes

It passed all Jenkins tests

No backport is made for this? I have posted #3682.

Thank you!


Review Study Investigator links

Some of the Study Investigator links are out of date (pointing to new/different profiles). We need to remove the ones that are no longer valid

Fixed in PR #271


remove unreachable line

This will cause extra warnings on compilers that don't recognize [[no return]], of which IIRC we had at least one of :)

When compiling chrome with default settings, I got:
../../third_party/libjxl/src/lib/jxl/render_pipeline/stage_epf.cc:510:10: error: 'return' will never be executed [-Werror,-Wunreachable-code-return]

When you say "we had at least one of", do you mean in our pipeline? Is that compiler still used?
Let me run the full pipeline...

When compiling chrome with default settings, I got:
../../third_party/libjxl/src/lib/jxl/render_pipeline/stage_epf.cc:510:10: error: 'return' will never be executed [-Werror,-Wunreachable-code-return]

When you say "we had at least one of", do you mean in our pipeline? Is that compiler still used? Let me run the full pipeline...

I don't remember where we got those...
If we have both compilers that complain with the line, and compilers that don't, then we have a bit of a problem :) let's hope not...

full pipeline seems to pass


Class 'app\controllers\Query' not found

when i use select2, there was an error : "Class 'app\controllers\Query' not found".

This appears to be an issue that is not directly related to Select2, but instead is related to an integration or downstream project that is not officially supported by Select2.  It is recommended that you create a ticket at the downstream repository or send a message to the mailing list for help.
https://github.com/kartik-v/yii2-widget-select2
https://github.com/tonybolzan/yii-select2

you must mention namespace like this
$query = new \yii\db\Query;
instead of $query = new Query;


[WIP] complex_webqa - Error zipfile.BadZipFile: Bad CRC-32

Have a string zipfile.BadZipFile: Bad CRC-32 for file 'web_snippets_train.json' error when downloading the largest file from dropbox: https://www.dropbox.com/sh/7pkwkrfnwqhsnpo/AABVENv_Q9rFtnM61liyzO0La/web_snippets_train.json.zip?dl=1
Didn't managed to see how to solve that.
Putting aside for now.

Thanks for your contribution, @thomwolf. Are you still interested in adding this dataset?
We are removing the dataset scripts from this GitHub repo and moving them to the Hugging Face Hub: https://huggingface.co/datasets
We would suggest that you create this dataset there. Please, feel free to tell us if you need some help.


Tools 2979: Add Special Validation for Size Fields

Add Special Validation for Size Fields

Multiple of 128M
Should have Quotient to be in Power of 2

Codecov Report
Attention: Patch coverage is 0% with 26 lines in your changes missing coverage. Please review.

Project coverage is 36.19%. Comparing base (141a52a) to head (47f4045).
Report is 1 commits behind head on master.

Files with missing lines
Patch %
Lines

asconfig/utils.go
0.00%
20 Missing :warning:

asconfig/conffilereader.go
0.00%
6 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #61      +/-   ##
==========================================
- Coverage   36.27%   36.19%   -0.09%
==========================================
  Files          23       23
  Lines        5615     5639      +24
==========================================
+ Hits         2037     2041       +4
- Misses       3439     3462      +23
+ Partials      139      136       -3

Flag
Coverage Δ

?

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

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


small modernizing for Django 3.0

I don't know why tests doesn't pass, but everything works fine.

django.utils.six.moves replaced with six
django.test.simple.DjangoTestSuiteRunner replaced with django.test.runner.DiscoverRunner, but seems didn't work

I need this changes :)

Please, i need this changes too.

Also need this change :)


Support lambda class serialization via hints

Currently lambdas serialization https://github.com/oracle/graal/commit/b455f23b624a1af149c7d5769443a4775bedd087 is supported in GraalVM native-image. This commit add lambdaCapturingTypes and lambdaCapturingTypeNames to conveniently add capturing classes to serialization configuration.
Example:
`
public class LambdaClassSerialization {
private static void serialize(ByteArrayOutputStream byteArrayOutputStream, Serializable serializableObject) throws IOException {
ObjectOutputStream objectOutputStream = new ObjectOutputStream(byteArrayOutputStream);
objectOutputStream.writeObject(serializableObject);
objectOutputStream.close();
}
private static Object deserialize(ByteArrayOutputStream byteArrayOutputStream) throws IOException, ClassNotFoundException {
    ByteArrayInputStream byteArrayInputStream = new ByteArrayInputStream(byteArrayOutputStream.toByteArray());
    ObjectInputStream objectInputStream = new ObjectInputStream(byteArrayInputStream);
    return objectInputStream.readObject();
}

public static void testLambdaClassSerialization() throws Exception {
    ByteArrayOutputStream byteArrayOutputStream = new ByteArrayOutputStream();
    Function<Integer, String> function = (Function<Integer, String> & Serializable) (x) -> "Hello";

    serialize(byteArrayOutputStream, (Serializable) function);
    (Function<Integer, String>) deserialize(byteArrayOutputStream);
}

}
@SerializationHint(lambdaCapturingTypes = {LambdaClassSerialization.class})
@SpringBootApplication
public class LambdaClassSerializationSpringApplication {
public static void main(String[] args) throws Exception {
	SpringApplication.run(LambdaClassSerializationSpringApplication.class, args);
}

}
`

@hanakeichen Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@hanakeichen Thank you for signing the Contributor License Agreement!

Since Spring Boot 3.0 GA has been released, Spring Native is not actively developed, so I will close this PR unmerged. I don't think we support lambda class serialization in Spring Framework 6, and not sure yet if we should support it or not yet. We should be sure to have enough use cases if we want to support it, so if you create a PR/issue on Spring Framework side, make sure to provide more details on your use cases.


Support Dask as an Adapter

Describe the feature
Support Dask just as Spark is supported.
Who will this benefit?
This will benefit realtime / web-request use cases where milliseconds matter.  The same isomorphic Machine Learning model Transform using SQL can then run in bulk (against the datalake) and in the web-request sub ms latency contexts.

Hey @talebzeghmi - cool idea! I've heard many good things about Dask, but I haven't used it before personally. This would be a great opportunity to get some experience with it for sure.
The Spark plugin for dbt is really a SparkSQL plugin -- dbt handles templating SQL and executing it against a remote Spark cluster. All of dbt's existing adapter plugins are databases (eg. BigQuery, Snowflake, Redshift, Presto, etc). Does Dask provide any sort of SQL-interface into working with dataframes? Based on my (admittedly very incomplete) understanding, I don't quite see where Dask would fit into the dbt picture.
I'm super happy to discuss / brainstorm around this issue even if it's not something we'd be able to pick up imminently -- keen to hear what you have in mind & your opinions on the broader Python data space here if you'd care to share them!
Thanks for taking the time :)

Does dbt do simple SQL translation or does it have an executor engine plugin model to execute the query plan produced?  An example would be how Hive supports an HQL to be executed by MapReduce, Tez or Spark.

dbt builds SQL statements and them to a database to be executed. dbt itself does not interpret user-provided SQL!

closing this one. Let me know if you have any further thoughts here @talebzeghmi - happy to re-open and discuss :)

Hi @drewbanin and @talebzeghmi!
Congratulations on this well-done and well-documented package!
I am the main developer of dask-sql, a relatively new extension of Dask that adds SQL capabilities. dask-sql is still a very young project and can definitely not be compared with the very mature SparkSQL or alike, but we are happy to extend the collaboration as much as possible. @rajagurunath brought up a possible adapter in dbt in the referenced issue.
I had only a quick look so far so excuse my naive question: If dbt issues a create table SQL command (in one way or the other), does it rely on the data being actually stored, e.g. on disk? All meta information on the tables in dask-sql lives in memory so far - a possible restart of the dask-sql server will remove all tables again. Is this a possible showstopper for implementing an adapter?
The dask-sql server implementation speaks the presto wire protocol, so apart from some possible SQL incompatibilities (we do not cover the full SQL standard so far, especially not some table-description-commands), I do not see a reason why such an adapter can not be implemented.

@nils-braun That sounds really neat!
That sounds like a serious complication, though perhaps not a showstopper, so long as the meta information about tables persists between sessions/connections. How often do you expect a dask-sql server to be restarted? How would end users, seeking to benefit from the transformed datasets produced by a dbt project, expect to access or query those tables?

Hi @jtcohen6
the idea is that the dask-sql server is restarted as often as things like e.g. a Hive or presto server - and the Dask cluster itself. So in general not very often.
One naive question: What happens if the created tables are lost again? I assume dbt will probably recreate them with the cost of additional processing time?

That makes sense. (Ultimately, I imagine it would be valuable to back up those tables in some kind of metastore, along the lines of Hive or Presto). In the meantime, you've got the right idea: dbt takes the opinionated view that all transformed data objects (models) should be idempotent, such that it can rebuild them from raw data sources, at any time and for any reason, with no data lost—just the cost of the computation.

That is nice to hear @jtcohen6 - thanks for the clarification.
I think in this case it makes sense to continue our work (or currently, @rajagurunath's work) with the integration of dbt and dask-sql. Maybe it is to some use for some users! Thanks for your help so far - we might come back with more questions :-)

Hi guys, since dbt adopted pure Python models, do you think there is a pure python implementation for Dask and dbt?


[CP Stg] Fix assignment of selectedItemIndex in PopoverMenu

cc: @roryabraham

Details

This PR fixes the issue of selecting list items in PopoverMenu using the enter button on the keyboard. Because the assignment of a value to selectedItemIndex was done after onItemSelected() was called, the value of selectedItemIndex in onModalHide() was null. Assigning a value to selectedItemIndex before calling onItemSelected solves the problem.
Fixed Issues

$ https://github.com/Expensify/App/issues/40399
PROPOSAL:
Tests

[ ] Verify that no errors appear in the JS console

Log in to New Expensify.
Navigate to any chat
Click on + button
Navigate via + menu using Up/Down keys and press Enter on any button.
Verify whether the action assigned to the button has been performed.

Offline tests

QA Steps

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/Expensify/App/assets/47774969/c85b55ef-8297-455a-acf0-2c4f0b35e246

Android: mWeb Chrome

https://github.com/Expensify/App/assets/47774969/21c8b028-78dc-49f9-89f4-6babfb94cc07

iOS: Native

https://github.com/Expensify/App/assets/47774969/9de7a6d8-0513-49dd-affc-f1a30659aeab

iOS: mWeb Safari

https://github.com/Expensify/App/assets/47774969/7a6e85b0-e1a1-4d88-a73a-7e28ea4e28f6

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/47774969/0a71a31c-845a-4a7e-840e-afa3ba3d1859

MacOS: Desktop

https://github.com/Expensify/App/assets/47774969/ca443b10-4d0e-426a-aebb-07b4c021b7de

🚀 Deployed to production by https://github.com/mountiny in version: 1.4.63-21 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Define Project Architecture

It would be nice to have a descriptive diagram, much like the overall AIDE diagram, that describes the various components of the GUI.
These are my initial thoughts on the implementation of the GUI:

The GUI should be able to inspect what designs aide_draw can draw, and which aide_design can design and auto-generate the relative GUI buttons.
The overall flow of the AIDE plugin's GUI should be intuitive and simple. There are three distinct tasks that the GUI aids in:

initiating designing of the plant.
start drawing the plant from either the designed JSON or another one someone sent
start documenting the plant (choose a template/type of document, and send request off to aide_document.

In one sense, the GUI does the same thing over and over again. It creates a button, prompts and collects data from the user, calls the relevant function with said data, and returns the result. The more abstractly you can define these tasks, the less replication you will have.

The architecture drawing should propose large-level components of the GUI package. Think of it as the GUI- zoomed in part of the overall AIDE design diagram. It should answer the following equations:

What major sub-packages are needed? What are the vague boundaries and responsibilities of those sub-modules?
How will the sub modules generate the user experience?
How does data flow throughout the system?
How does the GUI call external AIDE packages?

I would love to work on this with y'all. I recommend starting a google drawing and sharing me. I can make comments and suggestions.

Hey Ethan! We made a google drawing and shared that with you if you wanna take a look and give us some suggestions

@ach238 Where is it? I just need the file name if you don't want to link it!

here's the link: https://docs.google.com/a/cornell.edu/drawings/d/1Wx0-HbvOSo6qYDUNG_nqSEbtwN5-B-4QEhm20BV67jw/edit?usp=sharing

This looks like a good start. You'll want to expound on these tasks. What module is responsible for creating the buttons? What does it need to know, and what should it return?
Think about the interfaces between each major function. ie, the parameters called and what it will return. There are two types of interfaces to consider. Those outside your package and those within. External or outside interfaces are the functions your allowing draw, document, etc. to use from you and all the functions you are hoping to use of theirs. I currently see no reason you need to expose any functions for others to use, but it is possible. The internal interfaces only require conversation within your team to develop. Once you develop them well, you can all work on separate components without getting in one another's way.


Support for azurerm_postgresql_flexible_server

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Anything in works to support postgres flexible server?
New or Affected Resource(s)/Data Source(s)
azurerm_postgresql_flexible_server
Potential Terraform Configuration
No response
References
No response

I'm wondering the same. +1

Especially since single server is on a deprecation-path: https://learn.microsoft.com/en-us/azure/postgresql/single-server/overview-single-server


Some country flags bigger than others

Hi,
So basically I am using this lib like this:
<Grid item xs={12}>
    <FormControl sx={{ width: '100%', mt: 2}} variant="outlined">
        <MuiTelInput value={phoneNumber} onChange={setPhoneNumber} required defaultCountry="FR" label="Phone number" />
    </FormControl>
</Grid>

And there are 2 countries (Ascencion Island and Tristan Da Cunha) with a very large flag image. Any fix for this ?

Hello !
Could you please share a codesandbox plz ?
Victor


Tracking unique project keys

This allows us to track unique project identifiers so we know how many projects are being used, and allows for cross referencing any identifiers the integration has seen that may not exist on Jira (e.g. we might see UTF-8, but that's likely not a valid Jira project).
/cc @annafil

🙌yay!!


AngularJs 1.x integration

Hello all,
someone tried to integrate openjscad with an angularjs project? i need to know if there's some documentation/plunkr example for this...
thanks in advance!

Nope. Nothing.


How to execute Package SQL in schemachange

Hi Team,
In SnowSql we have command to execute the master script to execute internal scripts using !load command. when we are trying to execute that from Schemachange its failing getting following error :
snowflake.connector.errors.ProgrammingError: 000900 (42601): SQL compilation error:
Empty SQL statement.
Could you suggest Us Ho to use Load in schemachange

I have the same problem. I tried to use the include functionality of jinja but it did not help

Hey all. The primary issue here is that !load command is a snowsql specific thing. If you have suggestions for how schemachange can be updated to support this (with Jinja) I would welcome the PR!

Hello @lakshmiprasannaguntuka, @romaain
Could you take a look at #110 and see if the approaches discussed there will help run your master script using schemachange similar to snowsql's !load command.


No virtual method getOpenSourceSoftwareLicenseInfo

Plugin Version: 0.16.1
React Native Version: 0.47.1
When using this module with latest play-services 11.2.0, there's no longer a method GoogleApiAvailability#getOpenSourceSoftwareLicenseInfo:
  compile "com.google.android.gms:play-services-base:11.2.0"
  compile "com.google.android.gms:play-services-maps:11.2.0"

GoogleApiAvailability
08-18 11:40:12.159 18012 18074 E AndroidRuntime: java.lang.NoSuchMethodError: No virtual method getOpenSourceSoftwareLicenseInfo(Landroid/content/Context;)Ljava/lang/String; in class Lcom/google/android/gms/common/GoogleApiAvailability; or its super classes (declaration of 'com.google.android.gms.common.GoogleApiAvailability' appears in /data/app/com.transistorsoft.backgroundgeolocation.react-1/base.apk)
08-18 11:40:12.159 18012 18074 E AndroidRuntime: 	at com.airbnb.android.react.maps.AirMapModule.getConstants(AirMapModule.java:50)
08-18 11:40:12.159 18012 18074 E AndroidRuntime: 	at com.facebook.react.bridge.JavaModuleWrapper.getConstants(JavaModuleWrapper.java:140)
08-18 11:40:12.159 18012 18074 E AndroidRuntime: 	at com.facebook.react.bridge.queue.NativeRunnable.run(Native Method)
08-18 11:40:12.159 18012 18074 E AndroidRuntime: 	at android.os.Handler.handleCallback(Handler.java:751)
08-18 11:40:12.159 18012 18074 E AndroidRuntime: 	at android.os.Handler.dispatchMessage(Handler.java:95)
08-18 11:40:12.159 18012 18074 E AndroidRuntime: 	at com.facebook.react.bridge.queue.MessageQueueThreadHandler.dispatchMessage(MessageQueueThreadHandler.java:31)
08-18 11:40:12.159 18012 18074 E AndroidRuntime: 	at android.os.Looper.loop(Looper.java:154)
08-18 11:40:12.159 18012 18074 E AndroidRuntime: 	at com.facebook.react.bridge.queue.MessageQueueThreadImpl$3.run(MessageQueueThreadImpl.java:194)
08-18 11:40:12.159 18012 18074 E AndroidRuntime: 	at java.lang.Thread.run(Thread.java:761)

AirMapsModule#getConstants

Apparently this method should no longer be used. See here.

Same issue, there are currently 3 PRs open - out of the 3, I think #1612 has the better implementation since it provides a more meaningful message.

Yeah I agree. I don't know who to poke to get one of these merged... This is a pretty big issue and a very small fix...

I've already pinged the maintainer in the PR, he apparently (usually) does a new release every 2 weeks so it will hopefully be fixed before next week :)
And yeah, I agree, it's an important & easy fix to set in place :)

Fantastic thanks mate!

Same issue, Same solution

Any news on this ?

None on my side - no commits in the last 20 days :/
@gpeal @christopherdro sorry to ping you guys but it's kind of a necessary bug to fix

This seems merged. Any chance to get a release?

I'm sure @christopherdro & @gpeal will take care of it soon :)
In the meantime you can use master:    "react-native-maps": "airbnb/react-native-maps#9b11f7c",

Just published a new version. https://github.com/airbnb/react-native-maps/releases/tag/v0.17.0


Add redirect options for Google and GitHub sign-in

Notes for this fix

We changed the template email in supabase to remove the /welcome in the email that is sent to new uers
We added the "has_onboarded" check for new users to the WithAuthSSR on dashboard
I change the check_onboarded route to only check the current org (before it was checking if ANY orgs you have been added to were onboarded)
Made it so that any new orgs a user creates is automaticlaly "onboarded"
We set all current users to has_onboarded true, this way they arent thrown to /welcome

Action items:

If there are no requests on the dashboard we should ask users to go through the onboarding


How to save transfered results as .obj files?

Thanks for your work, How to save transfered results as .obj files?

def save_to_mesh(vertices, faces, output_mesh_path=None):
assert output_mesh_path is not None
with open(output_mesh_path, 'w') as fp:
for vert in vertices:
fp.write('v %f %f %f\n' % (vert[0], vert[1], vert[2]))
for face in faces+1:
fp.write('f %d %d %d\n' % (face[0], face[1], face[2]))
print('Output mesh save to: ', os.path.abspath(output_mesh_path))


Introduce unknown type concept to support any data type

We no longer require this unknown type. hence closing the PR


About refPE on sparse detectors

Hi @ChenhongyiYang, thanks for your great work. May I know when will the code about StreamPETR with refPE be publicly released? Thanks.

Hi, thank you for being interested in our codebase. We have open-sourced the code for StreamPETR. For BEV transformation using WidthFormer, we will open-source it later because we need some time to clean up the codebase.

Hi, thank you for being interested in our codebase. We have open-sourced the code for StreamPETR. For BEV transformation using WidthFormer, we will open-source it later because we need some time to clean up the codebase.

How long is the code open source?


Progress bar

Just a progress bar to give you company while waiting for the updater to download the update.

Thank you as always!

Thank you as always!


M9-3-3:  Ignore uninstantiated templates

Affected rules

M9-3-3

Description
Query reports cases in uninstantiated templates.

Use unsubstantiated templates instead.

Affected rules

M9-3-3

Description
Query reports cases in uninstantiated templates.

Unsubstantiated templates


RegEx.Email not working on client

Im having an issue when trying to validate Email's on the client, the email is just not validated, if value is present the validation doesnt throw an error. I have created a repro for this.
https://runkit.com/pmogollons/5ef51b97822e3e001395eca0
Hope this helps, thanks.

Hey, this is an issue with the schema definition. To specify an string which matches a regular expression you would define it like so:
const schema = new SimplSchema({
    contact: { type: Object },
    'contact.email': {
        type: String,
        regEx: SimplSchema.RegEx.Email
    },
})

Which can be shortened with the shorthand:
schema = new SimplSchema({
    contact: {type: Object },
    'contact.email': SimplSchema.RegEx.Email,
})

As shown here:
https://github.com/aldeed/simpl-schema#more-shorthand
Runkit example:
https://runkit.com/coagmano/5ef57c16bf7bce001bb88039

Thanks @coagmano, his will solve it.
But it is still weird that { type: SimplSchema.RegEx.Email } works on server but not on client.

Not sure why that would work, unless the server has a different older version of this package. Seems this is solved enough to close, though.


cqframework groupid/namespace doesn't correspond to a working web domain

On https://github.com/cqframework/clinical_quality_language/blob/master/Src/java/cql-to-elm/OVERVIEW.md we see you use the groupid info.cqframework however if I go to cqframework.info in my web browser it brings up some Japanese-language page that doesn't seem related to the project.
Did you lose control of your groupid domain or should org.cqframework be used in the above document?
For that matter, trying to visit cqframework.org doesn't bring up a server at all.

Thank you for bringing this to our attention. We'll look into it.


Test/carrot

Description

Motivation and context

Related issue

Types of changes

[ ] Patch (bug fixes, typos, mistakes; non-breaking change which fixes an issue)
[ ] Minor (add a new token, changing a value; non-breaking change which adds functionality)
[ ] Major (deleting a token, changing token value type; fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] I updated the token in all applicable sets. This applies if updating, adding, or deleting a token that has data across different sets (for example, if the value differs across color themes.)

SYNCING spectrum-non-colors
Using ./spectrum-tokens-studio-data/src/tokens-studio/spectrum-non-colors/ as SOURCE
Using ./spectrum-tokens/packages/tokens/src/ as DESTINATION
READING spectrum-tokens: /home/runner/work/spectrum-tokens-studio-data/spectrum-tokens-studio-data/spectrum-tokens/packages/tokens/src/
READING  tokens studio: /home/runner/work/spectrum-tokens-studio-data/spectrum-tokens-studio-data/spectrum-tokens-studio-data/src/tokens-studio/spectrum-non-colors/
TOKENS STUDIO DATA IS NON-COLOR ONLY
LOADED SPECTRUM TOKENS: 2685
LOADED STUDIO TOKENS:   1484
NEW TOKEN VALUES:     1
RENAMED TOKENS: 1
DEPRECATED TOKENS: 1
WROTE: /home/runner/work/spectrum-tokens-studio-data/spectrum-tokens-studio-data/spectrum-tokens/packages/tokens/src/color-aliases.json
SYNC COMPLETE
Spectrum Tokens Pull Request URL

./logs/deprecatedTokens.json

[
  {
    "uuid": "51f78c7f-995e-4c63-8393-044b80f0be42",
    "token": "background-opacity-down-potato"
  }
]

./logs/newTokens.json

{
  "background-opacity-down-carrot": {
    "$schema": "https://opensource.adobe.com/spectrum-tokens/schemas/token-types/opacity.json",
    "value": "0.1",
    "uuid": "e71fe292-a61b-4d01-8770-b57699df3d99"
  }
}

./logs/renamedTokens.json

{
  "background-opacity-down-potato": "background-opacity-down-carrot"
}


HTML Report cannot be displayed

Prerequisites

[X] I am using the latest version of Locust
[X] I am reporting a bug, not asking a question

Description
The HTML report cannot be displayed on the browser for downloading, when a large number of failure/exception are present in the test.

Run a test with a high volume of failures
After the test is completed
Go to DOWNLOAD DATA tab and click Download Report
A blank page is displayed and the content cannot be displayed.

It would be great if a max default value can be set here
https://github.com/locustio/locust/blob/master/locust/html.py#L108
So the page does not try to load tons of failures/exceptions that crashes the UI
Command line
-f my_test.py --autostart --modern-ui
Locustfile contents
class FastUser(FastHttpUser):
    tasks = [FastUserTasks]

class FastUserTasks(TaskSet):
    @tag("Task1")
    @task(1)
    def task1(self):
        """
        I omitted how below variables are generated. Overall, there is some logic that generates protobuf messages
        that are sent as binary over http post action when executing the tasks.
        """
        self.client.post(
            url,
            name=f"/{message_type}",
            data=message,
            headers={
                "Content-Type": "application/octet-stream"
            }
        )

Python version
3.11
Locust version
2.24.0
Operating system
MacOS

Please re-test using latest version (2.27.0)

Please re-test using latest version (2.27.0)

Hi @cyberw,
Thanks for the suggestion,
I upgraded and tested locally,
It got unresponsive around 60K failures,
In my opinion, I would like to disable showing failures/exceptions or at least set a default num of items, I do not see much value on trying to render all rows in the report, it turns into a really big scrollable page that cannot be easily navigated,
Thanks!

Ah, I didnt read your report closely.
The problem is with unique error messages, right?
I’ll look into this. Perhaps we can do something on the UI side as well, but we could/should probably group any new exceptions after 1000 unique ones as just ”other” or something.
For now you should be able to do this manually by just setting the error message to something less unique.

Ah, I didnt read your report closely.
The problem is with unique error messages, right?
I’ll look into this. Perhaps we can do something on the UI side as well, but we could/should probably group any new exceptions after 1000 unique ones as just ”other” or something.
For now you should be able to do this manually by just setting the error message to something less unique.

That's a great idea, thanks for looking into it
Can you please point me to the docs/code on how to set a the error, to avoid producing tons of them?
right now I am not catching the exception, so I am doing
        self.client.post(
            url,
            data=message,
            headers={
                "Content-Type": "application/octet-stream"
            }
        )

Use catch_response=True and response.failure(”something went wrong, but lets keep the error message the same for all”) as documented here:
https://docs.locust.io/en/stable/writing-a-locustfile.html#validating-responses


Array#each_index returns an enumerable of integer type

The Array#each_index method returns an Enumarable of Integer type, not the type of the elements of the Array
Motivation

Resolves https://github.com/sorbet/sorbet/issues/3110
Test plan

When a block is supplied to the Array#each_index method the types of the returned elements are the same as the input array. See this example:
list = ['hello', 'bye']
list.each_index{ |i| puts i.is_a?(Integer) }

results in this output:
true
true
=> ["hello", "bye"]

This tells us that the Array rbi currently has the correct element type specified for the return type when a block is specified.
However, when there is no block specified the Enumerable returned by the Array#each_index method does not enumerate over elements that match the type of elements in the array. Instead it
enumerates over Integers as shown by the two true values printed from the i.is_a?(Integer) code.

Thanks I'm testing this change on Stripe's codebase. The link for Stripe employees to view it is:
https://go/builds/bui_HMuueTrqpMXjJ0

Thanks!


Small errror in locale ru-ru

Hello!
 "ERROR: Cannot load locale /w2ui/src/locale/1.5/ru-ru.json"
thanx

Fixed


cadvisor is using 15-25% CPU with 54 containers

Watching top on my Docker host, the .cadvisor process is using anywhere between 15 to 25% CPU all the time. I have 54 containers, not very active.
I notice that the version of cadvisor used by rancher is quite old -- version 0.4.0 released September 2014. The latest version is 0.18.0, released September 2015. I don't know how much performance has improved, but it may be worth a look.
I'm guessing that rancher is also using the default sampling interval of 1 second. For demo purposes this is nice and shows a pretty graph, but for production an interval of 5 or 10 seconds would have less overhead. It should be possible to configure the sampling interval in the Rancher UI.

We are actually using version v0.16.0 of cadvisor. The release number in the link you sent is the release for rancher/cadvisor-package repo.
https://github.com/rancher/cadvisor-package
@cloudnautique Can you look into the default sampling?

I read some comments in a cadvisor issue on this subject and apparently it's usage of du for measuring disk usage on aufs-backed containers seems to be the cause.

It looks like the underlying problem has been fixed in a newer version of cadvisor. It's a little hard to tell because they don't keep a particular accurate changelog, but judging from commit dates, I think it's in the v0.19.2 release.

Thanks! Will take a look at the new release.

+1

+1 @cloudnautique :smile:

Testing out 0.19.3

With v0.46.0-rc3 , we are packaging v0.19.3:
agent.package.cadvisor.url=https://github.com/rancher/cadvisor-package/releases/download/v0.19.3/cadvisor.tar.gz

The issue wasn't fixed yet. I simulated the environment when I have 50 not very acrive containers (most of them run /bin/bash). The .cadvisor process is using about 10-20% CPU.
In my real environment I have similar ~330 containers. According to htop sensors the .cadvisor process is using 60-100% CPU.
I tried to reduce the scheduling priority for the .cadvisor process: renice -20 CADVISOR_PID. That didn't make any significant impact on CPU usage. I also tried to pin the rancher-agent container to a particular CPU core to not overload others. Still it's not a solution.
 I don't need any metrics for those containers. Is there any way to just disable cadvisor in rancher-agent container?

Using Rancher v0.49.0, I can confirm this. cadvisor is still using 15-20% CPU on a host with 12 active containers. Which version of rancher/agent-instance or rancher/agent has this new version of cadvisor?

@gordontyler the package is updated via Rancher server and not the rancher/agent version. Most of our agent code is updated this way.
@psviderski, there is no way to disable it right now. We have it in the roadmap to allow passing options so you could adjust the sample rate. We rely on cAdvisors metadata information for a few things, so Im not sure how easy it would be to completely turn it off.

Same problem here. 18 containers not very active and cadvisor is taking about 10-15% CPU constant usage.
rancher-server v1.0.0.

Any updates for this issue?

Same problem here. Only 1 container active apart from the Rancher agents, and consistently seeing 20%+ CPU usage. This is on a hardware server with 4 cores & 8GB RAM.

v1.0.2 with 15+ containers per host, cadvisor is registering ~10% cpu usage

I have 1.1.x version - and .cadvisor is taking 50% CPU on one of my hosts (ubuntu LTS 16.04)

v1.0.2 with 11 container, .cadvisor is taking 5% ~ 10% CPU. But docker stats shows rancher-agent container taking 300%+ CPU.
Another host seems fine.

@snipking In our next release (v1.2.0-pre4), we have removed cadvisor from Rancher and will be using docker stats to report stats.


User expiration date

Hi
can you please add a user expiration date and time to set on peers?

Hello, can we have expire date and data usage on panel? More like a counter and wearing in the panel.. it is better to these options in automatic, but warning can do the job also.
And it would be nice to have like a sort and group management for users also.
Like group and sort users into data limit and ip speed limit or expire limit.

I need the expiration date to remove the peer or disable it after the expiration date also being able to limit monthly traffic would be great!

Hello Dear
Please add expireation date for peers .
Thanks in advance

Hello Dear
Please add expireation date for peers .
Thanks in advance

Hello Dear
Please add expireation date for peers .
Thanks in advance

Hello Dear
Please add expireation date for peers .
Thanks in advance

Will do

Will do
Please add expireation date for peers

Will do

Please add expireation date for peers

azumi has done it , but its is on persian language

azumi has done it , but its is on persian language

That project has many bug and gets no update. That project is not suitable in production.


Flow aiding - Reset state when flow is enabled only if it is the only position/velocity aiding sensor

I'm currently working on a drone with an optical flow sensor and is has a nasty attitude glitch at each stop. After analyzing the log, I found out that the vxy and xy reset counters are incremented every time the drone stops. The reset occurs at each stop because it goes in and out of flow aiding and that the condition for a reset is always true "if the vehicle doesn't have GPS or external vision aiding" (my vehicle has GPS but no EV); the correct check should be "if it doesn't have gps and doesn't have external vision aiding" or -as I implemented it here- "if flow is the only velocity/position aiding source".
Below a log showing that the reset counter is incremented every time the height source switches from baro (during motion) to range aid (after a stop).

@jkflying This should also fix the glitches you see on the Eiger

A flight with that fix:

This is awesome! We've seen those twitches for a while but never tracked it down. Didn't think to look for resets!


Event Title is not Clickable

Is your feature request related to a problem? Please describe.
I was confused since the title of an event at /events/ is not clickable. I clicked it multiple times until I found out I had to use the "Show" button.

Figure 1: Me selecting the event title because I tried to use it as a button.
Describe the solution you'd like
Clicking the name of an event has the same effect as using the "Show" button
Describe alternatives you've considered
Alternatively, move the show button to the left to make it more obvious.
Additional context
n/a

It is clickable on a smaller screen. I agree though we could make it more like GitHub notifications and ditch the show button.


No SDK support for the Update Workflow Variables Endpoint?

There appears to be an endpoint to update Workflow variables on the Swagger API page, but it's not (from what we could find) supported in the SDK, we had to add a separate method in our project to call the API directly via HttpClient rather than using the SDK.  Can this be added to the SDK please?  Thanks!

Hi @mross002 the new version expected to be available soon should have this.  stay tuned!

Hi @mross002 this API is now available in the latest version that was released.

Hi @mross002 closing this as this was already released.  Please feel free to open the issue again if you need further help.


Fix errors for forwarder.pp using rpm package provider

I tried to install splunk::forwarder using the default package provider (rpm), but I ran into errors. This PR resolves those issue for me on a new RH7 machine.

The first commit solves this error message: Execution of '/bin/yum -d 0 -e 0 -y list splunkforwarder' returned 1: Error: No matching Packages to list, which happens because the pkg_provider was "undef" instead of $splunk::params::pkg_provider.
The second commit solves this error message: Error: /Stage[main]/Splunk::Forwarder/Package[splunkforwarder]/ensure: change from absent to present failed: Execution of '/bin/rpm -i /opt/staging/splunk/undef' returned 1: error: open of /opt/staging/splunk/undef failed: No such file or directory which was happening because $staged_package was being used without being assigned.

:+1:

:+1:

:+1:

+1

@hunner Is this module being supported anymore?

@kyledecot puppetlabs doesn't use gh for bug tracking. it should probably be filed in their jira and subject to their triage.
or just fork it like everybody else :D

@asquelt I ask because there are open pull requests (some as old as 12 months). I have filed jira tickets which have been marked as "Major" but have sat for months. If the repo is no longer supported I feel it should be marked as such.

may be. you might want to try to bring it to their attention during modules triage.

Didn't know about that @asquelt Thanks!

If this isn't being supported any longer i'll fork it and put it up on the forge with the merged PRs.

@discreet Saw that you forked. Do you think you can make a PR to resolve these outstanding issues?Hoping we can get this resolved in this original repo. Sorry for slow response. Can we take a look at this @kfjohnson or @hunner ? cc: @ajjohnsonpdx

@jpadams i just forked and merged the outstanding PRs with the exception of #15 into my fork. I'm going to be deploying to my environment today/tomorrow for full testing. If you would like i can PR back to this repo with all changes.

@discreet Yes, please, submit a PR with the merges you've made. I'm sure the team would like to take it if possible. Thanks for leading on this!

@jpadams will do.  I'm also going to include a custom fact for the forwarder version for linux (windows too if i have time).


Error building debug java_binary: "Extension file not found. Unable to load package for '@runtime_classpath//:java_classpath.bzl': The repository could not be resolved."

I get the following output (or similar) any time I try to execute a java_binary target with a debug run configuration via the plugin:
Building debug binary
Command: /Users/rg/bin/bazel build --tool_tag=ijwb:IDEA:ultimate --curses=no --color=no --experimental_ui=no --progress_in_terminal_title=no --aspects=@runtime_classpath//:java_classpath.bzl%java_classpath_aspect --override_repository=intellij_aspect=/Users/rg/Library/Application Support/IntelliJIdea2017.2/ijwb/aspect --output_groups=runtime_classpath --experimental_build_event_binary_file=/var/folders/4q/rm3flsfj6qdd31h6ljx0264h0000gp/T/intellij-bep-3c2f5063-7a0f-4976-ad17-57b250c26730 --noexperimental_build_event_binary_file_path_conversion -- //tomcatRunner:foo

____(11-20 14:33:28.440) Loading complete.  Analyzing...
ERROR: (11-20 14:33:28.443) Extension file not found. Unable to load package for '@runtime_classpath//:java_classpath.bzl': The repository could not be resolved.
ERROR: (11-20 14:33:29.124) Analysis of aspect '@runtime_classpath//:java_classpath.bzl%java_classpath_aspect of //tomcatRunner:foo' failed; build aborted: Extension file not found. Unable to load package for '@runtime_classpath//:java_classpath.bzl': The repository could not be resolved.
____(11-20 14:33:29.124) Elapsed time: 1.953s
____(11-20 14:33:29.125) Waiting for build event protocol upload: 0s____(11-20 14:33:29.125) Waiting for build event protocol upload: 0s
failed (1): bazel build --tool_tag=ijwb:IDEA:ultimate --curses=no --color=no --experimental_ui=no --progress_in_terminal_title=no --aspect...

I get this error even with the simplest target I could come up with:
java_binary(
  name='foo',
  main_class='com.redfin.tomcatRunner.FooMain',
  srcs = [ 'src/main/java/com/redfin/tomcatRunner/FooMain.java'],
)

FooMain.java:
package com.redfin.tomcatRunner;

public class FooMain {
    public static void main(String[] args) {
        System.out.println("Hello, World");
    }
}

Info:

bazel 0.7.0
IntelliJ IDEA (Ultimate) 2017.2.6
Plugin v 2<IP_ADDRESS>.5
A colleague on IntelliJ CE is reporting the same behavior.
"Run" instead of "Debug" works fine.

I saw these message in the bazel console in IntelliJ w/ IntelliJ v2017.2.5 and a 2017.10.x version of the plugin, but they were just messages. I just upgraded to try and get rid of them, and it prevents running the targets completely.
LMK if there's any other information I can provide.
Thanks!

Thanks for letting us know. This is fixed for the next release.

Workaround:
In IDEA, go to run configurations. Add "Remote". Copy the line labeled "Command line arguments for running remote JVM".
In BUILD, add a copy of the java_binary rule, name it e.g. idea_debug. add javacopts = ["-g"] and jvm_flags = [" copied line from above "].
Start your binary with bazel run idea_debug -- ARGS_FOR_YOUR_PROGRAM and then start the debug task in IDEA.


Updating to RTM builds of xunit

cc @dougbu

:shipit: (like the cleanup of copy-pasta dependency-mismanagement)


Add workaround for missing REPLACE_INVALID_UTF8

v8::String::REPLACE_INVALID_UTF8 was added in node.js v0.10.29.
Check the version macros to see if it's available.
Fixes: https://github.com/nodejs/nan/issues/394
Fixes: https://github.com/nodejs/nan/issues/396
R=@kkoopa, /cc @rvagg @Raynos
I think we should address this, I'm getting complaints from people about broken builds separate from the filed issues.

What with < 0.8.27 then? And what are everybody doing on 0.10.26?
-0

Didn't realize this was also released in v0.8.  Fixed up, PTAL.

And what are everybody doing on 0.10.26?

I can attest from first-hand experience that upgrades in large companies are often painful and prolonged.  Big businesses don't move fast (to their detriment but hey.)

Code looks ok. The proper thing to do would be add 0.8.26 and 0.10.28 to CI so this does not regress, but that will slow down runs even more. I think the risk of regression is fairly small.


tmux_cwd_sync_now optionally select a pane

Hey. What do you think about having an option for a command tmux_cwd_sync_now?
I often press xc and always go to a pane i synced and think there might be an option something like tmux_cwd_sync_now select=True or tmux_cwd_sync_now focus=True to reduce the repeatable action.
Then this option can be set up as default in a config or one more shortcut is added.

Yes, I think this is a good idea and something I would probably use myself!

Ok, it will take some time to figure out how to implement it, that's a standard feature i mean a command option so shouldn't be complicated.
I'll send a PR with a solution.

Hey - I was looking to see how difficult this would be to implement, but it turned out to be really simple (hope you don't mind).
Do you mind testing #7 for me?

Hey - I was looking to see how difficult this would be to implement, but it turned out to be really simple (hope you don't mind).
Do you mind testing #7 for me?

Tested, it's working well. At the moment, i don't see any issue...

This is now in v1.0.8


Return output when call Tell function

Hi. Thanks for a great library. Very helpful for me 😄
When call Tell function, there is a possibility that return some output.
For example,
tell application "iTunes"
    return name of current track
end

It returns current track name.
So, I modified return values to returns output and error.

Awesome thanks for contributing!


abacus-learning fails sporadically

The abacus-learning tests fail sporadically. This happens when the tests are run on a loaded CPU core, so I guess this is due to some synchronization issues.
We might consider using generator functions in learning.
Note: The tests were failing due to different result in Math.exp (see #436)

Thanks for reporting this. Will take a look weekend.

Deleted the code. We should get back to this when we have a better idea how v2 pipeline will look like.


Rewrite CompartmentSearch to main search index in Cosmos

Description
Rewrite CompartmentSearch to main search index in Cosmos.
Example of expanded rewritten expressions:

Rewriter uses type filtering information to only include the relevant search params.
Related issues
Addresses AB#87577.
Testing

Existing Unit and E2E tests
Manual test cases

FHIR Team Checklist

[x] Update the title of the PR to be succinct and less than 50 characters
[ ] Add a milestone to the PR for the sprint that it is merged (i.e. add S47)
[ ] Tag the PR with the type of update: Bug, Dependencies, Enhancement, or New-Feature
[ ] Tag the PR with Azure API for FHIR if this will release to the Azure API for FHIR managed service (CosmosDB or common code related to service)
[ ] Tag the PR with Azure Healthcare APIs if this will release to the Azure Healthcare APIs managed service (Sql server or common code related to service)
[ ] CI is green before merge
Review squash-merge requirements

Semver Change (docs)
Patch|Skip|Feature|Breaking (reason)

I noticed we are still storing the compartment indices in the Cosmos DB resources - is this intentional?

I noticed we are still storing the compartment indices in the Cosmos DB resources - is this intentional?

Good question, happy to null them out first if it feels like the right thing to do. Since this doesn't apply to SQL yet, it might make sense to remove all this code with that as a second part. Also, if we were forced to roll-back for any reason, I'd rather still have them there :)


Added CLI tools for Red Hat OpenShift on AWS

Added the Red Hat OpenShift on AWS (ROSA) cli tool
Description
The ROSA cli is a command line too for administering OpenShift clusters on the managed AWS service. This merge request adds additional functionality to arkade to allow simple installation.
Motivation and Context
This allows the simple installation of the ROSA cli client tools, avoiding the process of having to log into the Red Hat portal and choose the appropriate binary from a drop down selection. Having this functionality in arkade will improve my workflow.
How Has This Been Tested?
make e2e and hack tool.
+ ./arkade get rosa --arch arm64 --os darwin --quiet
+ file /home/matt/.arkade/bin/rosa
/home/matt/.arkade/bin/rosa: Mach-O 64-bit x86_64 executable
+ rm /home/matt/.arkade/bin/rosa
+ echo

+ ./arkade get rosa --arch x86_64 --os darwin --quiet
+ file /home/matt/.arkade/bin/rosa
/home/matt/.arkade/bin/rosa: Mach-O 64-bit x86_64 executable
+ rm /home/matt/.arkade/bin/rosa
+ echo

+ ./arkade get rosa --arch x86_64 --os linux --quiet
+ file /home/matt/.arkade/bin/rosa
/home/matt/.arkade/bin/rosa: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), statically linked, Go BuildID=UJy8CZP35000NA1DroaT/lO2jEUUnFH0Raz-VQItd/YE6IzU1BAWt0YE-B0f9M/7MqygW3g8mr-PRlw8ubZ, not stripped
+ rm /home/matt/.arkade/bin/rosa
+ echo

+ ./arkade get rosa --arch aarch64 --os linux --quiet
+ file /home/matt/.arkade/bin/rosa
/home/matt/.arkade/bin/rosa: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), statically linked, Go BuildID=UJy8CZP35000NA1DroaT/lO2jEUUnFH0Raz-VQItd/YE6IzU1BAWt0YE-B0f9M/7MqygW3g8mr-PRlw8ubZ, not stripped
+ rm /home/matt/.arkade/bin/rosa
+ echo

+ ./arkade get rosa --arch x86_64 --os mingw --quiet
+ file /home/matt/.arkade/bin/rosa.exe
/home/matt/.arkade/bin/rosa.exe: PE32+ executable (console) x86-64 (stripped to external PDB), for MS Windows, 13 sections
+ rm /home/matt/.arkade/bin/rosa.exe
+ echo

If updating or adding a new CLI to arkade get, run:
go build && ./hack/test-tool.sh TOOL_NAME

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Documentation

[x] I have updated the list of tools in README.md if (required) with ./arkade get --format markdown
[ ] I have updated the list of apps in README.md if (required) with ./arkade install --help

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s

[ ] I have tested this on arm, or have added code to prevent deployment

Thanks for the PR. @Jasstkn do you have a moment to review?

@mkimberley hey! why did you close the PR?


Split the Abatement Cost component

There is a natural division in the Abatement Costs component, whereby a large number of parameters get translated into 4 key parameters.  Splitting the component at this point also allows the marginal abatement costs to be equated to a tax rate, to determine the emissions reduction process.

Hi @davidanthoff: This is the first part of allowing tax-driven policy.  I'll follow up on the other necessary components over email.

Hi @lrennels, I think this got missed along the way, but should have been merged before any updates.  Can you please update to this version of the code?

@jrising my version is here: #164 , I'll finish checking it tomorrow, the goal being for that one to replace this one.


Add breadcrumbs to admin program image upload page

Description
Please explain the changes you made here.
Release notes
The title of the pull request will be used as the default release notes description. If more detail is needed to communicate to partners the scope of the PR's changes, use this release notes section.
Checklist
General
Read the full guidelines for PRs here

[ ] Added the correct label: < feature | enhancement | bug | dependencies | infrastructure | ignore-for-release | database >
[ ] Assigned to a specific person, civiform/developers, or a more specific round-robin list.
[ ] Created unit and/or browser tests which fail without the change (if possible)
[ ] Performed manual testing (Chrome and Firefox if it includes front-end changes)
[ ] Extended the README / documentation, if necessary

User visible changes

[ ] Followed steps to internationalize new strings
[ ] Wrote browser tests using the validateAccessibility method
[ ] Tested on mobile view. See mobile device mode
[ ] Manually tested at 200% size
[ ] Manually evaluated tab order

New Features

[ ] Add new FeatureFlag env vars to server/conf/helper/feature-flags.conf
[ ] Conditioned new functionality on a FeatureFlag
[ ] Wrote browser tests with the feature flag off and on, etc.

Instructions for manual testing
If instructions are needed for manual testing by reviewers, include them here.
Issue(s) this completes
Fixes #<issue_number>; Fixes #<issue_number>...

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #6041

PR #6042

PR #6049

PR #6050

PR #6200

PR #6218

PR #6227

PR #6254   👈

PR #6043

This stack of pull requests is managed by Graphite.


latest 2019-04-08-raspbian-stretch issue

My phat beat has been working fine on the prior release of stretch (2018-11-13-raspbian-stretch), but there appears to be an issue with the latest version.
It's a new install, ran 'sudo curl -sS get.pimoroni.com/phatbeat  | bash' without issue, and the speaker-test from the command line works great. But in the desktop environment , the audio device is disabled (the task bar speaker icon has a red 'X', and the volume control is disabled). When clicking on the speaker icon, it shows snd_rpi_hifiberry_dac with a green checkmark, but when I attempt to open device settings via the task bar or via Audio Device Settings in Preferences, a popup window appears with the error "No ALSA Devices Found".
Not sure if this is a pulseaudio vs alsa issue, but any suggestions would be welcome.

What do you see if you aplay -l and aplay -L in terminal?

Here is the the output for each command:
pi@rpi-r:~ $ aplay -l
**** List of PLAYBACK Hardware Devices ****
card 0: sndrpihifiberry [snd_rpi_hifiberry_dac], device 0: HifiBerry DAC HiFi pcm5102a-hifi-0 []
  Subdevices: 0/1
  Subdevice #0: subdevice #0

pi@rpi-r:~ $ aplay -L
null
    Discard all samples (playback) or generate zero samples (capture)
pivumeter
softvol_and_pivumeter
default
    Playback/recording through the PulseAudio sound server
sysdefault:CARD=sndrpihifiberry
    snd_rpi_hifiberry_dac,
    Default Audio Device
dmix:CARD=sndrpihifiberry,DEV=0
    snd_rpi_hifiberry_dac,
    Direct sample mixing device
dsnoop:CARD=sndrpihifiberry,DEV=0
    snd_rpi_hifiberry_dac,
    Direct sample snooping device
hw:CARD=sndrpihifiberry,DEV=0
    snd_rpi_hifiberry_dac,
    Direct hardware device without any conversions
plughw:CARD=sndrpihifiberry,DEV=0
    snd_rpi_hifiberry_dac,
    Hardware device with all software conversions

After a bit more research, I've determined that the problem is with pulseaudio. More specifically, the problem is with pulseaudio when connecting to the pi via VNC... which was not a problem prior to the April '19 release of stretch.
Initially, I discovered the following errors in syslog:
Apr 16 15:40:51 rpi-r pulseaudio[967]: Using device: phat-beat
Apr 16 15:40:51 rpi-r pulseaudio[967]: [pulseaudio] server-lookup.c: Unable to contact D-Bus: org.freedesktop.DBus.Error.NotSupported: Unable to autolaunch a dbus-daemon without a $DISPLAY for X11
Apr 16 15:40:51 rpi-r pulseaudio[967]: [pulseaudio] main.c: Unable to contact D-Bus: org.freedesktop.DBus.Error.NotSupported: Unable to autolaunch a dbus-daemon without a $DISPLAY for X11

I was able to address this by adding:
Environment=DISPLAY=:0
to the [Service] section in file: /etc/systemd/system/pulseaudio.service.
Now I see the following errors:
Apr 16 15:46:19 rpi-r pulseaudio[949]: Using device: phat-beat
Apr 16 15:46:19 rpi-r pulseaudio[949]: [pulseaudio] server-lookup.c: Unable to contact D-Bus: org.freedesktop.DBus.Error.Spawn.ExecFailed: /usr/bin/dbus-launch terminated abnormally with the following error: No protocol specified
Apr 16 15:46:19 rpi-r pulseaudio[949]: [pulseaudio] server-lookup.c: Autolaunch error: X11 initialization failed.
Apr 16 15:46:19 rpi-r pulseaudio[949]: [pulseaudio] main.c: Unable to contact D-Bus: org.freedesktop.DBus.Error.Spawn.ExecFailed: /usr/bin/dbus-launch terminated abnormally with the following error: No protocol specified
Apr 16 15:46:19 rpi-r pulseaudio[949]: [pulseaudio] main.c: Autolaunch error: X11 initialization failed.

From what I've uncovered so far, this should be fixed by insuring that this:
load-module module-native-protocol-unix auth-anonymous=1 socket=/tmp/pulseaudio.socket
is in file: /etc/pulse/default.pa
and that this:
default-server = unix:/tmp/pulseaudio.socket
is in file: /etc/pulse/client.conf.
The problem is, these are already in place with the default files. So at the moment, I'm stuck.
It appears that the phatbeat script is only uninstalling and then reinstalling pulseaudio. Since stretch is no longer distributed with pulseaudio installed by default, the net effect of the script is to do a new install of pulseaudio on a newly installed pi. Unfortunately, this disables audio in the desktop environment if accessing via VNC with the newest release (I've confirmed that it works fine if I access via x11 using a local x-server, and I suspect that it would be fine if a monitor/keyboard/mouse are plugged into the pi as well).
Is pulseaudio required for the phatbeat? Thanks...

pulseaudio was required for earlier versions of pHAT BEAT where ALSA alone would end up running the PCM clock without sending any data- causing the speakers to just sit and heat up. We added a hardware system to prevent this on subsequent versions though, so it's very likely you can run without pulse.

pulseaudio was required for earlier versions of pHAT BEAT where ALSA alone would end up running the PCM clock without sending any data- causing the speakers to just sit and heat up. We added a hardware system to prevent this on subsequent versions though, so it's very likely you can run without pulse.

Can I ask what hardware system was added? (and anything else useful you might remember about this problem). We think we are encountering something similar with our custom MAX98357A board.

We added this on a later revision of pHAT Beat but I can’t remember which one exactly.
We used a transistor/resistor/capacitor setup which charged the cap as the left/right clock line wiggled and pulled the enable for the DAC via the transistor. Something like that, anyway.
The problem was - IIRC - specifically the PCM clock running without the LRClock. I’m not sure if there’s a more correct way to work around this.


Backport "[spec] remove implementation details from enum spec" to LTS

Backports #18579 to the LTS branch.
PR submitted by the release tooling.
[skip ci]

No regressions detected in the community build up to lts-18731.
Reference


Implement secrets-pull logic

Features:

Implement secrets-pull logic for services to make backup of them

Testing:

 Implement secrets-pull logic

 Implement secrets-pull logic

Wow, it's pretty slow :-D

Wow, it's pretty slow :-D

So it doesn't seem to differentiate parameters vs secrets, though? I think we should probably only download the secrets, but not the parameters... Let's discuss on Monday

So it doesn't seem to differentiate parameters vs secrets, though? I think we should probably only download the secrets, but not the parameters... Let's discuss on Monday

Also, if possible, I would prefer to change the logic to:

If local *.json file exists, make a backup of it,
Store the secrets under .json

Also, if possible, I would prefer to change the logic to:

If local *.json file exists, make a backup of it,
Store the secrets under .json


Call to a member function setNode() on null

In file '/var/www/html/lib/private/Share20/Manager.php' line 822.
This is because I inverted create and createInternal.
The flow is:

files_sharing ShareAPIController calls our ShareAPIHelper to create $oldShare
ShareAPIController calls the built-in ShareManager, which calls our SSP to create $newShare
data from $oldShare is merged into $newShare in https://github.com/nextcloud/server/blob/v25.0.1/lib/private/Share20/Manager.php#L821

Question is: should our SAH or our SSP make the call to our RHC and thus to Reva?
More detailed:

NC GUI
file_sharing app ShareAPIController
our ShareAPIHelper

Reva

addSentShare
newShare
createInternal
SSP create
createScienceMeshShare
addSentShareToDB

This is where the share gets created!

Reva

empty response from /ocm/send
ShareAPIController is done calling our ShareAPIHelper
now the shareManager gets involved
lib/private/Share20/Manager
Manager calls SSP::create but this gets suppressed and null is returned
Manager tries to set the data from the first share into the second one, but fails because the second one is null.

One thing is certain: SSP create should return the share.

The share should have a full id and a share time. Fixed in nc-sciencemesh dev branch now.

The problem is, when I move this code from SSP create to SSP createInternal (where it should be because there we can fill in the true id), then the error is back. So should we maybe make the call to RHC from  SSP create, then?

Tracing again:...

it starts at files_sharing ShareAPIController line 486, createShare
straight away, ShareManager line 1780 NewShare is called, setting the rootFolder and the userManager
some inputs are checked (permissions, path, user permissions on the resource)
the node is set on the share
the node is locked
some more things related to permissions
now createShare is called on the helper, but I'm not sure whether we actually need to do anything here?
our helper calls SAH-createShare but I'm not sure whether this is smart because the helper cannot return any data
shareType and sharedBy are set
now all the preparation is done, and control is passed from the ShareAPIController to the shareManager
canShare, generalCreateChecks, pathCreateChecks
if the storage is external, the owner is set to the mounter. If not, the owner is set to the file owner, or to the sharer
the target is set
a preShare event is triggered
now control is passed to the provider
back in the ShareManager, a postShare event is triggered
if configured, the ShareManager sends out an email (but only if the share type is local user-to-user)
back in the ShareAPIController, the output is formatted

OK, moved the call to reva from the helper to the provider, great progress!
Now that the helper almost does nothing we could also maybe avoid having to register it (only register the provider, not the helper)
Anyway, next step: pass the ID through reva.

OK! fixed on the NC side, now the reva side...

Fixed in the dev branch of nc-sciencemesh and the efss-backend-fixes branch of reva


emule-community: update to 0.60d; add x64 architecture

Closes #4992

The PR is marked as a draft because auto-update needs a fix

Please don't remove the trailing newlines from the other files.

Please don't remove the trailing newlines from the other files.

Hi, @rashil2000 sorry for the change. The other changes are not shown on my machine.

/verify


Add new v1 Alaska DEM data

2 meter strip DEM data for Alaska (or 5 meter composite), hot off the presses. #thanksobama
Because Alaska is so far north, in web Mercator space this amounts to A LOT OF TILES to update.

Press release
Data portal

License and Usage
ArcticDEM data is an unlicensed product and may be used, distributed, and modified without permission.
It looks like there are known holes in data coverage that are scheduled to be filled.
ArcticDEM data encompasses all land area north of 60° north latitude. In addition, coverage will include all territory of Greenland, the entire state of Alaska, and the Kamchatka Peninsula of the Russian Federation. The project commenced with Alaska and is expected to create a comprehensive elevation model of the Arctic within the two-year term of the U.S. chairmanship of the Arctic Council, which began in April 2015.
Here near Anchorage and Sarah Palin's Russian suburbs cottage:

But at it's highest quality it looks like:

There is a lot of data here. I was going to start mirroring the 2m strips to the sources bucket, but there are well over 30k images, each in the 100's to 1000's of MB each. I'm still trying to understand what's in each of these, but it might make more sense to use the 5m mosaic tiled product they have. That's still a lot of data, but there's (a) much less data and (b) less overlap for us to process.

And considering our zoom 15 is nominally 3 meters I think that's totally fine :)

On Jun 5, 2017, at 19:51, Ian Dees<EMAIL_ADDRESS>wrote:
There is a lot of data here. I was going to start mirroring the 2m strips to the sources bucket, but there are well over 30k images, each in the 100's to 1000's of MB each. I'm still trying to understand what's in each of these, but it might make more sense to use the 5m mosaic tiled product they have. That's still a lot of data, but there's (a) much less data and (b) less overlap for us to process.
―
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Going with the 5m data.
Listing files
lftp \
    -c 'find' \
    ftp://ftp.data.pgc.umn.edu/elev/dem/setsm/ArcticDEM/mosaic/v2.0/ \
  | grep tar.gz > pgdc_5m_mosaic_listing.txt

Mirroring files to openterrain S3
cat pgdc_5m_mosaic_listing.txt \
    | xargs \
      -I {} \
      -P 24 \
      -n 1 \
      sh -c 'export f="{}"; export s3="s3://elevation-sources-prod/pgdc_5m_mosaic/`basename $f`"; curl -s $f | AWS_DEFAULT_PROFILE=openterrain aws s3 cp - $s3; echo $s3' \
    tee pgdc_5m_mosaic_s3_objects.txt

Transcoding files
....

Mirroring is in progress.

Transcoding files
aws s3 ls s3://elevation-sources-prod/pgdc_5m/ --recursive | \
grep tar.gz$ | awk '{print $4}' | \
while read filename; do
    bn=$(basename ${filename%.tar.gz})
    make submit-job job=aws/transcode-job.json.hbs input=s3://elevation-sources-prod/${filename} output=s3://elevation-sources-transcoded/$(dirname $filename)/${bn} name=$(sed 's/\./_/' <<< $bn)
done


Set value + expiration in one trip

Small optimization.  I'm not sure if this is less ideal due to the expiration saved potentially being +/- a second from the desired expiration due to I/O. Let me know.

That seems like an acceptable trade-off to make :+1: One fewer roundtrip at the cost of possibly being off by 1 RTT seems like a good optimization

One of the specs seems to be broken, any idea what might've happened there?

I came across the same sort of thing when working on https://github.com/hummingbird-me/sidekiq-debounce/pull/17 . It was due to an interaction with the stubbed scheduled set object. In that case I got past it by pulling in timecop and not using the stub, but there'll be a less drastic solution in there.


Added ABL kinases

The ABL Kinsases were one of the examples in the MITRE parts list from the eval. Added these.

Thanks @mwptrsn!


DM-38279: Wait for lab deletion before returning

If we wait for namespace deletion in the background and return immediately to JupyterHub, JupyterHub assumes that we have deleted the lab and may immediately attempt to create a new one, which then fails.

What happened without this is that if mobu had to delete a lab at startup, the lab deletion would complete immediately and then it would try to create a new lab and that would immediately fail, so after each error it had a round of failing before it could successfully restart. Maybe there's something trickier we can do; I'll add this to my to-do list to look at in more detail.


Automated Package Upgrade

:tada: This PR is included in version 5.0.166 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Confusion about some codes

Hi, Thank you for sharing the codes!
I find the codes below (in def sp_attn_head.py) is a bit hard to understand, since they are not directly corresponding to the original description in text:
f_1 = tf.layers.conv1d(seq_fts, 1, 1)
f_2 = tf.layers.conv1d(seq_fts, 1, 1)

f_1 = tf.reshape(f_1, (nb_nodes, 1))
f_2 = tf.reshape(f_2, (nb_nodes, 1))

f_1 = adj_mat*f_1
f_2 = adj_mat * tf.transpose(f_2, [1,0])

logits = tf.sparse_add(f_1, f_2)

lrelu = tf.SparseTensor(indices=logits.indices,
values=tf.nn.leaky_relu(logits.values),
dense_shape=logits.dense_shape)

coefs = tf.sparse_softmax(lrelu)

seq_fts = tf.layers.conv1d(seq, out_sz, 1, use_bias=False) can be regarded as multiplying a weight matrix W (converting F features to F’ features). However, I do not understand the following steps:
f_1 = tf.layers.conv1d(seq_fts, 1, 1)
f_2 = tf.layers.conv1d(seq_fts, 1, 1)
f_1 = adj_mat*f_1
f_2 = adj_mat * tf.transpose(f_2, [1,0])
logits = tf.sparse_add(f_1, f_2)

I am aware that the codes are different since it is conducted on a “matrix” level instead of node level. But I cannot see how the attention mechanism is achieved by these steps. Can you help explain a bit?
Many thanks,
Weixin.

Hi Weixin,
Thanks for your interest in the GAT work and for reaching out. I'll try to elaborate on those lines in more detail.
As our layer is a single-layer NN, we can decompose it from W[h_i||h_j] to W_1h_i + W_2h_j (with W_1 and W_2 being separate linear transformations).
As W_1 and W_2 are pointwise applied to each node, we may express them as a 1D convolution with kernel size 1 across the node feature matrix. Applying these two convolutions gives us f_1 (which is W_1h_i) and f_2 (which is W_2h_j).
In the dense GAT layer (https://github.com/PetarV-/GAT/blob/master/utils/layers.py#L13-L17), we compute W_1h_i + W_2h_j for all (i, j) pairs, by simply adding f_1 to transpose(f_2). This implies adding something of shape (batch, nodes, 1) to something of shape (batch, 1, nodes), and through the power of broadcasting we obtain the full (unnormalised) attention coefficient matrix of shape (batch, nodes, nodes), with TensorFlow automatically performing this sum for all pairs of positions. You can then normalise the coefficients by applying LeakyReLU, bias and softmax.
For the sparse layer, the idea is pretty much the same, only we do not wish to use more memory than is necessary. This is why we multiply the values with the sparse matrix's entries, to preserve only the positions i and j that are actually to be used.
Does that help? Feel free to reach out if you need more info.
Thanks,
Petar

Hi Petar:
Thank you for your detailed explanation, which helps a lot!
Many thanks,
Weixin.

Hi, Petar, thanks for your explanation! However I have some questions about the bias matrix. How to generate a bias matrix,  what's the physical meaning of bias, and why you choose this generation setting?
This looks like the default bias matrix generation for dense adjacency matrix.
https://github.com/PetarV-/GAT/blob/ea3aeafb782e20738a090dbf7914aa6e5a1a137e/utils/process.py#L14
https://github.com/PetarV-/GAT/blob/ea3aeafb782e20738a090dbf7914aa6e5a1a137e/utils/process.py#L25
def adj_to_bias(adj, sizes, nhood=1):
    nb_graphs = adj.shape[0]
    mt = np.empty(adj.shape)
    for g in range(nb_graphs):
        mt[g] = np.eye(adj.shape[1])
        for _ in range(nhood):
            mt[g] = np.matmul(mt[g], (adj[g] + np.eye(adj.shape[1])))
        for i in range(sizes[g]):
            for j in range(sizes[g]):
                if mt[g][i][j] > 0.0:
                    mt[g][i][j] = 1.0
    return -1e9 * (1.0 - mt)

Hi, Petar, thanks for your explanation! However I have some questions about the bias matrix. How to generate a bias matrix, what's the physical meaning of bias, and why you choose this generation setting?
This looks like the default bias matrix generation for dense adjacency matrix.
https://github.com/PetarV-/GAT/blob/ea3aeafb782e20738a090dbf7914aa6e5a1a137e/utils/process.py#L14
https://github.com/PetarV-/GAT/blob/ea3aeafb782e20738a090dbf7914aa6e5a1a137e/utils/process.py#L25
def adj_to_bias(adj, sizes, nhood=1):
    nb_graphs = adj.shape[0]
    mt = np.empty(adj.shape)
    for g in range(nb_graphs):
        mt[g] = np.eye(adj.shape[1])
        for _ in range(nhood):
            mt[g] = np.matmul(mt[g], (adj[g] + np.eye(adj.shape[1])))
        for i in range(sizes[g]):
            for j in range(sizes[g]):
                if mt[g][i][j] > 0.0:
                    mt[g][i][j] = 1.0
    return -1e9 * (1.0 - mt)

until mt[g][i][j] = 1.0, it is to add self-loop to the adj matrix.


Better data structures for types

Right now, types are represented as strings in our AST. For example, the type of [1, 2, 3] is the string "List(Int)". Right now this is fine because type checking is as simple as a string comparison with the expected type, but soon this will be insufficient because we'll have union types. It's a good idea to move to better data structures for holding types before we merge in union types.
Here's the list of all the types we currently have and the proposed data structure to hold them in the AST. So instead of just strings, we'll now have atoms, strings and structs representing types.
Booleans - atom :Bool
Atoms (:foo)- string ":<atom>"
Integers - atom :Int
Strings - atom :String
Lists (List(<type>)) - %List{type: <type>}
Records ({foo: <type>, ...}) - %Record{fields: [{<key_atom>, <value_type>}, ...]}
Tuples {<type>, ...} - %Tuple{elements: [<type>, ...]}
Function references - %FunctionRef{arg_types: [<type>, ...], return_type: <type>}
Nothing - atom :Nothing

I'll take this on!

I just noticed all the <> were missing because i didn't use the right formatting 🤦  . Fixed it.


Support other Email Services by leveraging config.js Host

Support additional email services by allowing users to setup Host and Port via app settings instead of requiring one of the 3 services. I use SparkPost as my mailer and would like to set it up like so: https://vinaygopinath.me/tech/2016/06/14/sending-emails-from-ghost-with-nodemailer-and-sparkpost/

It's a good idea. Do you want make a PR?

Yeah I'll see if I can put one together

Any suggestions on how to implement it?


Typo fixed

@mp-loki Yesterday we have done installation on windows and noticed that the instruction misses "--loadDemo" option. For example on Linux:
# --loardDemo option will load demo data: users, devices, assets, rules, widgets.
/usr/share/thingsboard/bin/install/install.sh --loadDemo

Please fix this and I will merge the PR

Done, also found another one


Reproducible SIGSEGV in query

postgres 16.2.
debian bookworm
ARM64 (AWS, t4g.xlarge, Graviton2, 64bit)
latest docker image 16.2 from https://hub.docker.com/_/postgres
on a specific query there is a SIGSEGV, resulting in two core files in /var/lib/postgresql/data
-rw------- 1 postgres postgres<PHONE_NUMBER> Apr 30 13:09 core.32
-rw------- 1 postgres postgres<PHONE_NUMBER> Apr 30 13:09 core.33

installed symbols
find-dbgsym-packages /usr/lib/postgresql/16/bin/postgres
sudo apt install postgresql-16-dbgsym

then tried to get a stacktrace:
gdb -q /usr/lib/postgresql/16/bin/postgres ./core.33

[Thread debugging using libthread_db enabled] Using host libthread_db library "/lib/aarch64-linux-gnu/libthread_db.so.1".
Core was generated by `postgres: parallel worker for PID
32                                          '.
Program terminated with signal SIGSEGV, Segmentation fault.
--Type <RET> for more, q to quit, c to continue without paging--
#0  0x0000fffdf6f1d708 in ?? ()
(gdb) bt full
#0  0x0000fffdf6f1d708 in ?? ()
No symbol table info available.
#1  0x0000fffdf96a6338 in __gthread_mutex_lock ()
     at
/usr/lib/gcc/aarch64-linux-gnu/12/../../../../include/aarch64-linux-gnu/c++/12/bits/gthr-default.h:749
No locals.
#2  lock () at
/usr/lib/gcc/aarch64-linux-gnu/12/../../../../include/c++/12/bits/std_mutex.h:100
No locals.
#3  lock_guard () at
/usr/lib/gcc/aarch64-linux-gnu/12/../../../../include/c++/12/bits/std_mutex.h:229
No locals.
#4  remove_fatal_error_handler ()
     at
build-llvm/tools/clang/stage2-bins/llvm/lib/Support/ErrorHandling.cpp:76
No locals.
#5  0x0000fffdf7114918 in ?? ()
No symbol table info available.
Backtrace stopped: previous frame inner to this frame (corrupt stack?)

and
gdb -q /usr/lib/postgresql/16/bin/postgres ./core.32

[Thread debugging using libthread_db enabled] Using host libthread_db library "/lib/aarch64-linux-gnu/libthread_db.so.1".
Core was generated by `postgres: sa bi <IP_ADDRESS>(55185) SELECT                                   '.
Program terminated with signal SIGSEGV, Segmentation fault.
--Type <RET> for more, q to quit, c to continue without paging--
#0  0x0000fffdf59aa708 in ?? ()
(gdb) bt full
#0  0x0000fffdf59aa708 in ?? ()
No symbol table info available.
#1  0x0000fffdf8736338 in __gthread_mutex_lock ()
     at
/usr/lib/gcc/aarch64-linux-gnu/12/../../../../include/aarch64-linux-gnu/c++/12/bits/gthr-default.h:749
No locals.
#2  lock () at
/usr/lib/gcc/aarch64-linux-gnu/12/../../../../include/c++/12/bits/std_mutex.h:100
No locals.
#3  lock_guard () at
/usr/lib/gcc/aarch64-linux-gnu/12/../../../../include/c++/12/bits/std_mutex.h:229
No locals.
#4  remove_fatal_error_handler ()
     at
build-llvm/tools/clang/stage2-bins/llvm/lib/Support/ErrorHandling.cpp:76
No locals.
#5  0x0000fffdfe8e8e28 in ?? ()
No symbol table info available.
Backtrace stopped: previous frame inner to this frame (corrupt stack?)

please let me know if there is something useful in this data to analyze, or if you want me to execute other steps to get more useful data. the issue can be reproduced on demand.

Our packages here are stock installed packages from upstream, so I think https://wiki.postgresql.org/wiki/Apt is probably the best place to chase information about what's happening here. :sweat_smile:
From that page, my best guess would be one of:

Mailing list<EMAIL_ADDRESS>(Archives)
IRC channel: #postgresql-apt @ irc.libera.chat