subscriber level healthchecks

Adding delay consumer errors to subscriber err channel
Restart subscriber for any error on its err channel
Restart stream for any error while running stream

Codecov Report

Merging #345 (e52294d) into master (256902c) will decrease coverage by 1.66%.
The diff coverage is 22.22%.

@@            Coverage Diff             @@
##           master     #345      +/-   ##
==========================================
- Coverage   57.21%   55.55%   -1.67%
==========================================
  Files         123      123
  Lines        6385     6423      +38
==========================================
- Hits         3653     3568      -85
- Misses       2417     2549     +132
+ Partials      315      306       -9

Flag
Coverage Δ

integration
?

unittests
55.55% <22.22%> (-0.18%)
:arrow_down:

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

Impacted Files
Coverage Δ

internal/subscriber/core.go
0.00% <0.00%> (ø)

internal/subscriber/retry/builder.go
0.00% <0.00%> (ø)

internal/subscriber/retry/retrier.go
18.08% <0.00%> (ø)

internal/subscriber/subscriber.go
48.29% <ø> (+1.58%)
:arrow_up:

internal/stream/stream.go
77.73% <11.53%> (-8.12%)
:arrow_down:

internal/tasks/subscriptiontask.go
70.50% <37.50%> (-2.02%)
:arrow_down:

internal/subscriber/retry/delayconsumer.go
45.45% <50.00%> (+0.07%)
:arrow_up:

internal/stream/metrics.go
100.00% <100.00%> (ø)

pkg/messagebroker/kafka.go
18.75% <0.00%> (-22.00%)
:arrow_down:

Continue to review full report at Codecov.

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


Update tox and travis configs.

Test on Python 2.5.
Run tests from travis exclusively with tox.
Set the Python version correctly for travis.
Conistently indent travis.yml.

Coverage remained the same at 100.0% when pulling 46739a431deb716156632105f8632fb2cf80e3a5 on benkonrath:update-travis-config into 03cfe0ad72a6f4a34cbe76de3edb2fc1fdd1731b on WoLpH:develop.

I still have to fix something because travis is running all the tests in one of the runs which shouldn't happen.

Coverage remained the same at 100.0% when pulling 536e726b35e6406fd3587b5d0ed8a9342e8a3e49 on benkonrath:update-travis-config into 03cfe0ad72a6f4a34cbe76de3edb2fc1fdd1731b on WoLpH:develop.

Coverage remained the same at 100.0% when pulling 795b09f43e7e46f2fa4f08689949a23f3c86bae7 on benkonrath:update-travis-config into 03cfe0ad72a6f4a34cbe76de3edb2fc1fdd1731b on WoLpH:develop.

Coverage remained the same at 100.0% when pulling 1b6937972f6cdcd8158c394d0a1df967ad4655f2 on benkonrath:update-travis-config into 03cfe0ad72a6f4a34cbe76de3edb2fc1fdd1731b on WoLpH:develop.

This is now ready. I think this is a much improved travis config.

Wow, indeed much better. Thank you!
Since I'm currently moving I don't have my systems up and running yet but I'll push out a new release soon.

Apparently Github won't allow me to merge yet: Pull requests that have a failing status can’t be merged from a phone

There's no rush on getting this merged. Whenever you have time is good. Good luck with your move.

Thanks for all the help!


Add ops_coverage python script

@ricmk Do you want to support me to integrate this script in the CI?
Please check https://github.com/tensorflow/tensorflow/issues/14798#issuecomment-1047796247
/cc @andrew-leaver @cheshire

Check also my "extra" comments:
https://github.com/tensorflow/tensorflow/commit/8fccf8a7728315d1cf2c538a3bb55029103addac#commitcomment-75913049
https://github.com/tensorflow/tensorflow/commit/a099818c97085a7174da52dc05cf5b49ec7105cf#r76511015

/cc @dathudeptrai

@cheshire @ricmk Can you share a little bit the plan/roadmap after you have removed all the support protos with this commit
https://github.com/tensorflow/tensorflow/commit/38582d06e9acd5108ce943af1e793aa493282cad ?

/cc @paynecl

@paynecl Any news on this?

Can we use/rely on XlaOpRegistry to generate this table?
https://github.com/tensorflow/tensorflow/blob/814e437df95b8b1455452feb47eee5c5e37ce229/tensorflow/compiler/tf2xla/xla_op_registry.cc#L57
If this could be ok Is this only valid for the "standard" TF brdige?

Yes.  I actually have an internal change for this that I'll send for review
that integrates the information with the raw ops table (sorry for the long
delay).
Current approach would use the opset defined by the registry of the non
milr bridge.  That is currently the default bridge for GPU/CPU so we'd use
that as ground truth in the docs. We'd switch over to the MLIR opset
construction when that bridge becomes default.
On Mon, Jan 23, 2023 at 12:27 PM bhack @.***> wrote:

Can we use/rely on XlaOpRegistry to generate this table?
https://github.com/tensorflow/tensorflow/blob/814e437df95b8b1455452feb47eee5c5e37ce229/tensorflow/compiler/tf2xla/xla_op_registry.cc#L57
If this could be ok Is this only valid for the "standard" TF brdige?
—
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/56510#issuecomment-1400936009,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQLFIWQ7MJZIALMTQ4A3KFLWT3SSTANCNFSM5ZITLAQQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks for the update, as it seems an internal change I suppose that it will be pushed directly on GitHub without a PR.
I will close this PR when you are ready as I cannot interact with a CL right now.

I don't know. @paynecl What is the status of your mentioned CL?

I think @paynecl is looking into it.

Sorry for the delay.  Draft is written and I've synced with Mark on the
documentation updates; I need to get someone to help with an internal issue
but can submit after it is resolved.
On Tue, Mar 21, 2023 at 11:59 AM Mark Daoust @.***>
wrote:

I think @paynecl https://github.com/paynecl is looking into it.
—
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/56510#issuecomment-1478432493,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQLFIWRSIXP37EESX5LCJQLW5H3B3ANCNFSM5ZITLAQQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @paynecl Any update on this CL? Please. Thank you!

@paynecl submitted the changes to the doc generator, I'm not 100% sure why this hasn't showed up on the raw_ops page yet:
https://www.tensorflow.org/api_docs/python/tf/raw_ops

Hi @paynecl Any update on this PR? Please. Thank you!

Fixed: https://www.tensorflow.org/api_docs/python/tf/raw_ops

@MarkDaoust thanks. It would be really nice to know what is planned to be covered and what it will be never covered by design. I think it will help a lot to choice on what ops to rely if you need to compile your whole graph.
Do you think is it possible to add this info in the page?

Hi @MarkDaoust Any update on above comment from @bhack? Please. Thank you!

Do you think is it possible to add this info in the page?

No that information is not available.


Track expense - Error when submitting track distance expense to new workspace

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.64-1
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5240743
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Go to staging.new.expensify.com
Go to self DM.
Track a distance expense from + menu.
Click Categorize it from the actionable whisper.
Click New workspace button (if there is existing workspace).
Select a category.
Submit the track distance expense to the workspace.

Expected Result:
The track distance expense will be submitted to the workspace without issue.
Actual Result:
Error shows up when categorizing track distance expense and submitting it to the workspace.
It shows
"Unexpected error submitting this expense. Please try again later.The workspace is no longer accessible. Please try again on a different workspace."
The newly created workspace chat also disappears after the error appears.
Workaround:
Unknown
Platforms:

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

Screenshots/Videos
https://github.com/user-attachments/assets/150131fd-58c4-44b4-a08d-dc82075a9740

View all open jobs on GitHub

@dukenv0307 @rlinoz @narefyev91 - Do you think this is related to https://github.com/Expensify/App/pull/52482?

@dukenv0307 @rlinoz @narefyev91 - Do you think this is related to #52482?

Yeah it should be related - @rlinoz i double checked what cause the issue. If we use QA steps from this issue - CategorizeTrackExpense API call returns an error: "The workspace is no longer accessible. Please try again on a different workspace". If i manually remove waypoints from CategorizeTrackExpense - API call returns no errors - and flow is working fine. Could you please double check

Ah yes, this is happening because we will create the policy only after calculating the rate, not sure how we will fix this in the backend yet, but meanwhile let's revert that PR

Asked for CP here https://expensify.slack.com/archives/C07J32337/p1732105497368409

Issue fixed
https://github.com/user-attachments/assets/6b3f2f7d-3db1-4cc4-9da0-5aa1aaea9388

Thanks for the quick fix all!


LOG-5287: Update the Admin > Logs page to pull the available pod list from the Loki API

This PR looks to ensure that pods which are no longer available are able to have their logs be selected as a filter.
This does create a behavior change on the Admin > Logs filter, as before even if there were no logs for a pod it would still be displayed in the list, while now only pods which have logs will be displayed. I believe that this makes sense and will make our software easier to use, but if this isn't desired behavior, I can add in an array merge to make sure that the pods that haven't sent logs still show up to filter to.

Thnx @PeterYurkovich, I think it would be better to include pods that don't have logs yet. Admins will have permission to list pods. Worst case if they don't, we can show the list coming from Loki API. This will help troubleshoot issues for those pods

/lgtm

/cherry-pick release-5.9


Where is source code of the doc of spring-boot-3.x ?

Backgroud
I  have compiled this code in my local and see the document like Spring-Boot Document 2.x`
And I see there is a new version with greater lookout named  Spring-Boot Document 3.x`
Issue: Where the source is?
More:
I'm a Spring fans, I want join the devlopment or fix issue together! 🚀

已收到~感谢来信！


Trees appear too bulky in some places on the map

I wanted to take a look of how Streets GL rendered the Gundam (F4 for comparison here):

But trees cover it:

Based on aerial imagery F4 looks more similar to reality (Bing imagery):

maybe trees should be smaller by default? In my city also they look too bulky
p.s. have you considered enabling "Discussions" (e.g. https://github.com/streetcomplete/StreetComplete/discussions/) so we can discuss things there are not proper bugs?

Is the height  - value on trees evaluated in streets.gl?

Is the height - value on trees evaluated in streets.gl?

Yes, it is supported for individual trees and tree rows.

A 953m tree compared to normal trees

Who actually maps trees and rows with (estimated) heights and diameters? I don't, never ever, not encountered a natural / monumental tree that warrants the effort. Species could tell a little, orchard olives are 5-6m at best. Those street side certainly are pruned, all branches chopped to at least 5m from ground. Pines in the city usually have only the crown and in fact the Riserva Naturale Pineta here shows old to very old, only crown. Just added levels to apartment blocks on Via del Santuario, 7 at left, 5 at right, the pines lining the streets are reaching the top of the 4th floor.

A 953m tree compared to normal trees

It's the mana tree from Secret of Mana! ;-)

Anyway, I compiled a small list of tags interesting for rendering in streets-gl from the information in the wiki in the hopes that this summary will be useful:
Size
Your tree models are roughly 1:2, i.e. double as high as wide. So:

use height, if not set est_height for height
use diameter_crown for width (height ~= width * 2)

circumference of the trunk is set on a lot of trees, because it is easy to measure on foot (may even be a StreetComplete quest one day). diameter also exists. diameter = circumference / PI. A very rough estimate would be diameter_crown = diameter * 30. Hence:

otherwise use diameter to estimate width
otherwise use circumference to estimate width

Type of tree

for basic appearence: leaf_type=needleleaved or leaf_type=broadleaved
use leaf_cycle to havve deciduous trees not have leaves in winter
use leaf_cycle to fall back to a best guess if leaf_type is not specified

Specific models for specific trees
(Over the top / blue sky:)
For certain (families of) trees with specific appearance that are often used as , I guess you need to look in any of genus, species, species:wikidata, taxon and even species:<language>, genus:<language> to find which tree is which with best coverage. Examples of trees with a special look and that are used often in cities from the top of my head:

palm trees
cherry trees (pink blossom in spring)
apple trees (white blossom in spring)
orange, lemon trees (etc.) (quite distinct due to the many fruits in the tree)

However, I think such detection would warrant a separate library project that heavily makes use of wikidata data best written in a language that can compile to various targets (JS, native, ...), e.g. in Kotlin or Rust. Such a library project could then maybe also make predictions on the width, height, relation to trunk diameter etc.

I think that most tree height are integrated from opendata bases (for exemple in Paris: this tree  comes probably from this database)
Another funny example nearby:

https://www.openstreetmap.org/node/5234550067#map=19/48.89506/2.31063

key
value

height
60

leaf_type
broadleaved

min_height
57

natural
tree

It has probably been tagged so to look fine in f4map. but it's probably a pretty rare case
I think location=roof+height=3 would be a better tagging and that it should give the same result.
About sekerob comment on solar panel, I think location=roof should be used to put whatever (trees, pools, pitches,solarpanels?) on top of roofs (should it be a separate issue)?

Units of measurement seem to be ignored
diameter=15 cm
https://www.openstreetmap.org/node/6788395443
https://streets.gl/#59.43304,24.75283,29.75,132.25,1319.15

Units of measurement seem to be ignored
diameter=15 cm

Thank you for reporting this. I've added support for "cm" units, it's fine now.

Now:

Seems fixed, thanks!


xwin32 issues

Describe the bug
Order of init of GL was move earlier in new versions Qt, casuing a crash b/c the opengl context isn't ready uyet.
Qt 5.6 was last known working.
This could be the source of the crash for XWin 32 18 users.

Note: XWin32 21 does not have this issue.

this may also be the cause of #19418

I've made one last attempt to reach out to the technician RITM1001439 has been assigned to. If I don't hear back by next Monday, I will escalate.

thanks @markcmiller86 !

This has been resolved by updating XWin32 to version 21 on iSRD.


Webpack build error

Hi with recent changes , when I try to build keycloakify , I get this error :
[webpack-cli] Invalid configuration object. Webpack has been initialized using a configuration object that does not match the API schema.
 - configuration.output.path: The provided value "E:keycloakify\node_modules.cachescripts\u000bendorFrontendDependencieswebpack_output" is not an absolute path!
   -> The output directory as **absolute path** (required).
node:internal/errors:984
  const err = new Error(message);
              ^

Error: Command failed: npx webpack --config node_modules\.cache\scripts\vendorFrontendDependencies\webpack.config.js

To reproduce ( on windows machine )
git clone https://github.com/keycloakify/keycloakify.git .
yarn
yarn build

Hello @uchar,
Sorry about that, it's an update I made to the build process of Keycloakify to vendor in dompurify so that the lib can remain free of dependencies.
I forgot to think about Windows. It's fixed


manage multiple namespaces

make changes to manage multiple namespaces

thanks for that, and sorry for the late merge!


Cannot publish pagelayout

I try to publish an existing custom pagelayout. The pagelayout is a html file related to an aspx file. I still see a minor version checked in. Is there some special function to publish a html pagelayout file?

Are you using the following command:
Add-SPOFile  -Publish

In my current script I do a check out an check in with pnp powershell. And then I do a publish with normal powershell because I didnt find it in pnp powershell yet.

I found this work around:
# update list item to create assocation (aspx) file
				$masterpageGalleryList = Get-SPOList -Identity /_catalogs/masterpage

				if($masterpageGalleryList)
				{
					Set-SPOListItem -List $masterpageGalleryList -Identity $_.ListItemAllFields.Id -Values @{}
				}

				# Publish the file
				Set-SPOFileCheckedIn -Url $_.ServerRelativeUrl -CheckinType MajorCheckIn


Retrieve Address on load

Hi,
congrats for one more time for your plugin. Great work.
I want to ask you if it's possible the following case. What I did so far is that by clicking on a link I get the current user lat, long. I load a new php where I Get the values and set map's lat long so the marker is properly appears on the map on the correct point. So, beside this, I'm wondering if I can trigger any function and fill in the input type field with the address based on the lat long values while the page is loaded? (like when you drag the marker and the field fills in with the address of that point)
Thanks in advance.

You could ask again google to get places at lat/lng but I'm wondering why you don't save in your DB more than lat/lng. You should store all info you have lat/lng/address/... Like this you don't have to request something

Sorry maybe I didn't explained it correctly. I described a case when a user wants to find where is he and save this point as his place. To do that I need two things. The lat long and the address. Then
the user could give a name to this place and store it into my DB.
"Visualise" the steps:

User clicks on a link and I get the current Lat,Long <-- Done
Based on the Lat, Long we've just found, we want the address of this point while the page is loading (just like when you drag the marker) <-- Help :)
Finally, user types a name and save it to the DB <-- Done

I want to know how to accomplish the 2nd step.
Thank you.

OK don't mind, I parsed google map JSON based on the Lat, Long on my own.


Include hidden quotas

Hidden quotas are not accessible using cloud monitoring APIs. How can we make these quota metrics available on QMS?

QMS has dependency on the the Cloud Monitoring APIs. We are in touch with the Product management to bring support for hidden quotas.
If a hidden quota metric is available using any other public APIs, customers can extend QMS to include support for these quota metrics.


apache fastcgi protocol

nodejs can receive apache fastcgi requests like this
process.stdin.on('data', function (it) {

how do i decode the fastcgi request data? that's what this library is for, but it's very unclear how to use it.

Hiya - this library is a fairly low-level implementation of the FastCGI protocol. It's not particularly concerned about where the FastCGI interactions are coming from, just in decoding/encoding them.
This lib is used in https://github.com/samcday/node-fastcgi-application to make a Node process available to a FastCGI manager.


java.lang.RuntimeException: createContext failed: EGL_BAD_CONFIG

I'm seeing this issue / crash on some devices:
java.lang.RuntimeException
in android.opengl.GLSurfaceView$EglHelper.throwEglException

Zenfone2 (Z00A)
ZenFone 5 (ASUS_T00J)
Galaxy S4 (ja3g)
Galaxy Tab2 7.0 (espressorf)
HTC One M9+ (htc_hiau_ml_tuhl)
Galaxy Tab3 10.1 (santos10lte)
Galaxy Note4 (trltechn)
java.lang.RuntimeException: createContext failed: EGL_BAD_CONFIG
at android.opengl.GLSurfaceView$EglHelper.throwEglException(GLSurfaceView.java:1228)
at android.opengl.GLSurfaceView$EglHelper.throwEglException(GLSurfaceView.java:1219)
at android.opengl.GLSurfaceView$EglHelper.start(GLSurfaceView.java:1069)
at android.opengl.GLSurfaceView$GLThread.guardedRun(GLSurfaceView.java:1442)
at android.opengl.GLSurfaceView$GLThread.run(GLSurfaceView.java:1281)

screenshot from the play store

potentially relevant:
https://code.google.com/p/webrtc/issues/detail?id=1875
http://stackoverflow.com/questions/26504742/open-gl-bad-config-error-on-samsung-s4
to do:

try OF 0.9 and test on a device that this happens (a few people have emailed me about crashes and offered to test apks)
get a device that this crash happens on and look at the gl allocation on 0.8.4

this is the commit to look at, apparently newer phones don't want this color mode (and just want 8,8,8):
https://github.com/openframeworks/openFrameworks/commit/38a40427

Got this down to just Redmi Phones across multiple OS. Not sure why, maybe stencil still debugging


Ensure resource argument is mandatory

Background
Currently we have an argument named resource that’s used in methods across the extension.
This is used to ensure we access the workspace related to the resource passed in.
Currently this is an optional parameter (everywhere) resource?: Uri, hence sometimes values are not being passed into these methods when they should.
Not doing, results in errors where extension doesn’t differentiate between multiple workspace folders in a multi-root environment.
We need to ensure:

This argument is mandatory,
Authors of the code are forced to pass in a resource identifier

The solution is easy,: Change the signature of the argument to ‘resource: Uri | undefined’
What is the cost/benefit associated with each approach?
Option 1: Make this change across the board in on PR
Cost: A week, Modify extension code, unit tests and all functional tests.
Benefit: It gets done in one go, as opposed to working on this bit by bit.
Risk: High, if done in a single PR.
If done in multiple (smaller) PRs, then we can manage the risk (lower).
Personally I prefer option 1 although a week seems high.
We have to change multiple methods (that accept this optional argument),
I.e.: Multiple methods = multiple classes = multiple files = lots of changes.
Option 2: Make this change incrementally
Cost: Minimal, Fix files you modify/create.
No different from what we have today for (strict typescript compilation checks).
Benefit: It gets done in one go, as opposed to working on this bit by bit.
Risk: Low.

Dupe of https://github.com/microsoft/vscode-python/issues/3005?

Yes


db.on('ready', ...) not called if autoOpen: true

for dexie@2, the on ready event is fired just fine with autoOpen: true (and also with autoOpen: false + db.open())
however, for<EMAIL_ADDRESS>the ready event is not fired with autoOpen: true.
the workaround is to set autoOpen: false, then call db.open(), and the event will fire correctly.

as a sidenote, dexie is super cool! I'm really liking the new 3.0 db.use() functionality. thanks so much for this library! it's really really awesome

actually, that workaround doesn't seem to work now. I don't know why it stopped working either. it only works some times and I'm trying to isolate where it fails to give a small reproducible case.
I'm trying the db.use you posted in #836 and if I comment that out or the 'populate' event, then it loads up, but not with both.

omg, I feel kind of silly, but in the process I learned a lot about dexie.
so, in the example code in #836, it should read:
            return table.mutate(req).then((res) => {
              const queue = (tx['_myQueue'] = tx['_myQueue'] || [])
              queue.push({req, res})
              return {...res} // this line was missing!
            })

everything seems to be working. I'll reopen if there's a problem, but looks good so far :) !

Thanks @heavyk for correcting the sample in #836. My bad to forget returning the res. I will update #836 accordingly. Do I understand it right, that the issue with 'ready' event not firing was a result of the faulty middleware? So with a corrected middleware or no middleware at all, the ready event works as expected in<EMAIL_ADDRESS>?

correct, if the mutate function does not return, then the ready event will never fire if also the populate event also happens happening.. so...
db.on('populate', ...) + db.use({ ... }) without a return event from the mutate will cause the ready event never to fire.
the reason is, when the populate event is called, any modifications that are made inside the populate event will go through the mutate event, yet because the mutuate does not return the response, it'll wait forever before calling the ready event.

I think the solution to this problem is to check to see when calling the mutate function to ensure that something is returned. I couldn't figure out where this was getting lost in dexie when I looked into it yesterday. wherever this is, if the mutate event doesn't return a response, then the original res should be used (eg. if ((res = this.mutate()) == null) res = originalRes)

Looking at the code, it seems as, in the case a middleware does not return a response, res.numFailures will throw with TypeError: Cannot read property 'numFailures' of undefined, making the promise reject and the calling flow abort.
I don't think we could (or should) cover up in this scenario in case middleware returns undefined. We don't even have a default res to return, we only a req (request), which is of a different shape. The middlware stack is the sole source of a response, and the nature of a middleware is that it is free to do whatever it likes. It is expected though to return something that corresponds to the MutateResponse interface. To really check each middleware, it would be needed to put an additional sanity-checker middleware in front of each user-registered middleware, checking that the response is truthy and has the required properties numFailures, failures and lastResult. But I would not give that the highest prio right now. However very glad that you took the time to debug this and find the fix! Thanks again!


Reinstate Windows CI to use Github Actions

This issue is based on the need to temporarily disable MLIR-Windows-CI but we want to reinstate the check for the near future. We want to follow the precedent set by ONNX: https://github.com/onnx/onnx/pull/6075

FYI- @AlexandreEichenberger

I communicated with folks from Microsoft, and they have no-one currently working on this. If someone is using onnx-mlir on Windows, now would be the time to volunteer a bit of your time to re-instate the CIs.


Focus on ion-textarea in front of bound text (iOS)

Ionic version:  (check one with "x")
[ ] 1.x
[x] 2.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
When clicking on a textarea the focus is in front of the bound text with no way to change it to behind the text
Expected behavior:
Like in ion-input: Focus is after the text
Steps to reproduce:
Insert text into an ion-textarea, click on the text area again (on an iphone, tested on device and elevator (iOS 10.2)
<ion-textarea [(ngModel)]="description"></ion-textarea>

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.4
Ionic CLI Version: 2.1.17
Ionic App Lib Version: 2.1.7
Ionic App Scripts Version: 0.0.47
ios-deploy version: 1.9.0
ios-sim version: 5.0.11
OS: macOS Sierra
Node Version: v6.9.2
Xcode version: Xcode 8.2 Build version 8C38

Hello! Would you mind trying the latest nigthly (npm install ionic-angular@nightly --save) and trying this again? Thanks!

Hi,
just tested it with the recent nightly (Ionic Framework Version: 2.0.0-rc.4-201701101946), issue seems to be the same still :(

turns out I had the same issue as #9960 and did not put it inside ion-item tags


Associate votes with Discord servers + users

Pass in Discord server ID and user IDs with votes so we can generate Discord server leaderboards and show Discord users on Leaderboards.

Done!


fixed dark mode in review section

Related Issue
Fixes:  #166
Description
The dark mode was not working in review section. I fixed it.
Type of PR

[ x] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots
Before:

After:

Checklist:

[x ] I have made this change from my own.
[ ] I have taken help from some online resources.
[ x] My code follows the style guidelines of this project.
[ x] I have performed a self-review of my own code.
[ x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ x] My changes generate no new warnings.
[ x] I have tested the changes thoroughly before submitting this pull request.
[ x] I have provided relevant issue numbers and screenshots after making the changes.

@subin-shk Your pull request has been successfully merged! Don't stop here—continue contributing by addressing more issues and aiming to become a top contributor. Exclusive swag awaits for those who go the extra mile!
Great contribution to GSSOC'24 EXTENDED! 🎉


Destroy do not show flash message when object.destroy returns false

Using ActiveRecord we can return false from any callback to stop the destroy but IR thinks it's was destroyed and show the success message.
The issue could be simplified by this:
class SomeController < InheritedResources::Base
  def destroy_resource(object)
    false
  end
end

Currently I'm using this to workaround the issue:
def destroy(options={}, &block)
  object = resource

  if destroy_resource(object)
    options[:location] ||= smart_collection_url
  else
    options[:location] ||= edit_resource_path
    options[:alert]    ||= t("flash.actions.destroy.alert", :resource_name => resource_class.model_name.human)
  end

  respond_with_dual_blocks(object, options, &block)
end

Paaaaaaaatch!

I trieeeeeeeeeed! :P
But something on responders or maybe with respond_with didn't work.
Note the options[:alert] is a POG to fetch the flash message but do not provide any expected fallback.
Removing this gambeta we receive the success message for destroy instead of failure.

I believe that the current implementation expects some error to be added to the model, so IR would consider it a failing destroy case. Just to confirm, are we going to change this?

Charles is correct. I have forgot about this case. Closing.

Ok, it seems like we're having some problems with the way it works today, by requiring errors to be added to the model to consider a failure destroy scenario. The issue #185 was opened by @derekprior a few hours later relating the same problem.
I don't actually remember why we decided to let it this way, whether there was some drawback or not. After discussing with @josevalim, we believe that there seems to be no real problem to change this to check for destroy return value, so if you or @derekprior want to try out a patch, it'd be welcome :)
Thanks.

I'll give it a shot. I'm trying to wrap my head around what respond_with_dual_blocks is actually doing.


generate example via @react-native-community/cli command function

based on #52, with the following major benefits:

no need for the user to install react-native CLI tool
avoid issues due to differences between react-native-cli & @react-native-community/cli as discussed in brodybits/create-react-native-module#312

This is now accomplished by importing the init function from commands subdirectory of @react-native-community/cli, with a couple advantages over using the main API module as I tried in PR #52:

using template option to support flexible react-native version seems to work properly
no need to add react-native as a peer dependency

The template is hard-coded to use react-native-tvos@latest in order to support tvOS.
A couple of other related fixes are included:

use await to wait for createReactNativeLibraryModule to finish
determine example app subdirectory name & subdirectory path before generating the example app, which seems to be needed to avoid a path issue due to an apparent quirk with using the init command function

I was able to generate a module working on Android, iOS, and tvOS with this proposed update.
Some TODO items:

[ ] update test suite
[ ] update to support flexible template, in order to support flexible React Native version
[ ] update comments & output to remove installed react-native CLI requirement
[ ] apply some additional cleanup

Closing in favor of PR #64, which is basically a successor to this proposal.


Is MARC field 260 subfield $c (publication date) repeatable?

Pulling out a TODO #1599.
See https://github.com/wellcometrust/platform/pull/1599/files#diff-98e5e1eb54a097b178c6b358fcf4ca6bR8 – I made an educated guess about whether we'd see it more than once, and if so, how to handle it.
Qs for @silveroliver (or possibly the librarians):

Can a MARC record have more than one instance of field 260 subfield $c?
If so, how should we join multiple instances for our publicationDate field?

In MARC, the 260|c is repeatable (R): http://www.loc.gov/marc/bibliographic/bd260.html

So then the notes on $c say:

Only one subfield $c may be recorded in multiple publishing statements of a record.

Hope this helps - see attached document.
260 subfield c.docx


Tests failing

https://github.com/aurimas13/Solutions-To-Problems/blob/d621eaa33b943250834ef477e87cdfd0459981ed/LeetCode/Python Solutions/Number of Subsequences That Satisfy the Given Sum Condition/number.py#L24C56-L25
These two tests fail.
...     assert s.numSubseq([1,1,2,2,3,3,4,4,5,5], 8) == 147

I believe the correct answer is 1011.
...     assert s.numSubseq([1,2,3,4,5,6,7,8,9,10], 10) == 15

I believe the correct answer is 341.

Thank you for pointing it. Fixed it.

I don't see a code change on master.


Switching mail provider but continues to use previous

Laravel Version: 5.3 ^
PHP Version: 7.0
Database Driver & Version: Not Applicable
Mail Providers: Mailgun & Mailtrap

Description:
Whilst testing locally with Mailtrap (SMTP) and then switching drivers to another (in this instance Mailgun) sending mails still uses the Mailgun settings.
Even an artisan clear config does not fix the issue.
Currently only way found to get it to recognise the change is to restart the server / local machine.
Steps To Reproduce:

Build basic Laravel App
Setup SMTP credentials for Mailtrap, and configure Mailgun service credentials.
Set smtp/mailtrap as the mail driver in config
Test Sending a mail (received at Mailtrap)
Change driver to mailgun
Test sending mail (still received at Mailtrap)

Laracasts Discussion Link
https://laracasts.com/discuss/channels/laravel/mail-not-sending-with-mailgun-but-sends-with-mailtrap

@davidpiesse How do You serve the app locally?

Valet - but also tried with artisan serve
Live server is your bog standard DO, 16.04, PHP 7 box - using Forge.
On 31 March 2017 at 09:46, Sebastian Nowak<EMAIL_ADDRESS>wrote:

@davidpiesse https://github.com/davidpiesse How do You serve the app
locally?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/laravel/framework/issues/18588#issuecomment-290654435,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAw3iu_OrGfuHs3UjXNpp3I5SCmfFgIvks5rrL19gaJpZM4MvXbF
.

@davidpiesse I do not know how the Valet works but if You are using artisan serve or php built-it server, then You need to restart it on every change in .env file because all entries are read only once on the start.

I get that sort of local issue (I assume valet may have a similar
limitation) - but I have had the issue in prod as well..
On 31 March 2017 at 09:56, Sebastian Nowak<EMAIL_ADDRESS>wrote:

@davidpiesse https://github.com/davidpiesse I do not know how the Valet
works but if You are using artisan serve or php built-it server, then You
need to restart it on every change in .env file because all entries are
read only once on the start.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/laravel/framework/issues/18588#issuecomment-290657227,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAw3itC8BAnH5d1oMolpVDhZBJKUweolks5rrL_IgaJpZM4MvXbF
.

have you tried clearing the config cache?

I did - I gradually escalated the changes - clearing caches, etc. but they
only thing that eventually worded was a system reboot.
On 31 March 2017 at 11:07, Dylan DPC<EMAIL_ADDRESS>wrote:

have you tried clearing the config cache?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/laravel/framework/issues/18588#issuecomment-290673379,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAw3ijZzlhThk9_z4z0uAE47-Wum2bVzks5rrNB3gaJpZM4MvXbF
.

Looks to me as a system caching issue or something, closing this issue then.
Anyone please feel free to ping me if you can replicate.


Cannot paste responses to prompts into PowerShell host

Without PSReadline, it's possible to paste both commands and responses to interactive prompts in the same block, like so:
New-Item -Type File foo.txt
Remove-Item -Confirm foo.txt
Y
# (invisible newline here)

With PSReadline, the 'Y' is not passed to the interactive prompt, but is parsed as a separate command after the interactive prompt is finished.

Closing in favor of #579 which covers all of the known issues with right-click paste.


Please update version in PyPI

If using pip install, version 1.0.0 (Jul 31, 2018) will be installed.
I think this version is outdated. People who install this package by pip  cannot run examples. (Because of import error on multi_labeled)
Therefore, I'd like ask for you to update.

thanks for the feedback @masakiaota - I just pushed the updated version (1.1.0) to PyPI.


Allow setup the test execution order

Are you requesting a feature or reporting a bug?
Requesting a feature
What is the current behavior?
You can execute tests concurrently without restrictions
What is the expected behavior?
Although you guys encourage people to make test cases atomic, there are instances where having multiple test cases that execute in a specific order is needed. For instance: first "Creating a user", then "Removing a user", etc.
If you keep them in order in your fixture, it works well as long as you execute them in one browser, but if you execute them with concurrency, there's no guarantee that the tests will be executed in that specific order.
Can you give us an option to prevent test cafe from running some tests in parallel?
I was thinking maybe a fixture option like
fixture(Fixture name)
.page(page)
.parallel(false);
Or something along those lines. The idea is that that specific fixture should be run serially.
If that's not feasible, maybe we can specify the order and prevent a test case X to enter the pool until some other test case Y is done, that way we can force test case order execution.
I hope that makes sense.
Thanks a lot!
How would you reproduce the current behavior (if this is a bug)?
Provide the test code and the tested page URL (if applicable)
Tested page URL:
Test code

Specify your

operating system:
testcafe version:
node.js version:

I looked up how other test frameworks do it. Nightwatch runs tests in alphabetical order apparently and will run them in the order they were specified in a test file. I recall selenium webdriver was similar. I wonder if testcafe is the same.
https://stackoverflow.com/questions/32703989/how-can-i-run-nightwatch-tests-in-a-specific-order

@charlieg-nuco, AFAIK Nightwatch and Selenium don't have a feature that is similar to the concurrency mode. In the regular mode, TestCafe executes tests preserving the order they were described in fixture files.

I have also run into cases where I would like something like this. My tests are usually grouped into suites (groups of files, each file with its own fixture). Each suite must run in order, but different suites can be run parallel to each other.
I think a good way to accomplish this would be to allow the user to specify atomic groups of files that can not be run in parallel. For example, if I say run all tests is tests/**, but pass an atomic block of ['tests/a.js', 'tests/b.js'], one browser window might run all of the tests in a.js and then b.js while the other browser windows run the remaining tests in whatever way is most efficient.
This might be a little awkward to specify on the command line API, but I think this is a more advanced case and it's fine to require that you use the node API if you need this functionality.

We have the same issue. We have an online file system of sorts and are testing the trash functionality. There are tests for deleting and restoring single files etc. and there is also a test to empty the entire trash. The problem is that without being able to control the order in a fixture, the empty entire trash test will randomly interfere with the other tests causing them to fail.

@alexschwantes thank you for sharing your use case. Currently, you can execute tests that can't work in the concurrency mode in a separate TestCafe session.

Thanks @AndreyBelym. We run a continuous integration server that then reports all the results together via a junit report plugin. Can the different testcafe sessions join the results or do you mean to just run testcafe twice separately? And if so then how do you prevent the first session from running a specific test that you want to run in the second session without hardcoding files that you want to run?

@alexschwantes
I think you can try to use a programming interface to run tests separately. Please see the example in this answer on StackOverflow
In this case, you need to write some custom code to join the reporter results or implement your own reporter somehow.
As for  hardcoding the files, you can use the metadata mechanism, which is more suitable for this purpose.

Thanks @AlexKamaev.
Yes I can see that writing my own harness with the programming interface to filter tests with the new metadata functionality and run them separately and then join the results could be a way to do it. But it gets messy very quickly. Even just updating the correct values in the junit results for number of tests run and the time it took etc. Additionally it doesn't solve the issue others have described where it would be useful to simply run tests sequentially in a fixture.  But it does come part of the way.

@alexschwantes
I understand that it would be better for you to have the functionality out of the box than using workarounds. However, I cannot give you any estimates on when the feature will be implemented, so at this moment I recommend you use the workaround described above to achieve the desired behavior.

I'm having the same issue. I know Protractor shards (their term for concurrency) tests at the file level. Ie. tests in a single file run in one browser instance, and in the order they are written in the file (though this can also be randomized). Having some kind of flag that would allow concurrency at the file level would likely solve this issue in Testcafe... 🤔

I agree with @qualityshepherd having concurrency at file level gives you better control and you are assured that tests are guaranteed to execute in certain order

Is there any news on when this feature will approximately be available?

Another use case for this proposal is when you have tests that include some long waiting, and you want these particular tests to run in parallel with the rest of your test suite.

https://supportcenter.devexpress.com/internal/ticket/details/T1062344

On its own, being able to parallelize tests at the fixture or file level would greatly increase the stability of my test runs.
As @qualityshepherd noted, running several tests at the same time on the same feature can cause race conditions with test logic. Doing so also interacts very poorly with features that are resource-intensive or are otherwise not designed for significant parallel usage.

IMO, there are really only two problems with Testcafe; that keep it from easily being the best e2e solution:

This one: concurrency at the test level instead of file level
lack of beforeAll which of course is THE BIGGIE

Hi guys,
We will keep this feature in mind when discussing future development. Thank you for your request.
@qualityshepherd what do you mean beforeAll? We already have before and after hooks for testRun.

@Aleksey28 we've been over this MANY times :D https://github.com/DevExpress/testcafe/issues/3517

I see that we already delivered the feature you requested in a different thread.
Those who also require to handle the beforeAll problem with context for a specific fixture can refer to this package: testcafe-once-hook.

Yeah... sadly, the once-hook doesn't work for me... but I appreciate the effort.

Could you please clarify whether testcafe-once-hook does not work in your usage scenario or it does not meet your requirements?

Do you have an open issue for this problem? If not, please create one and share a simple sample where we could reproduce the problematic behavior.

Hello everyone,
We have added a new fixture method to disable the global concurrency setting for a specific fixture in the latest release.


Missing workspace level setting

Please add the workspace setting for ArcGIS visuals too, many people dont know where that setting is located.

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

ID: acf2c48a-66f7-6c38-286e-ae429fc34883
Version Independent ID: 033ec0fc-5c72-8cf2-20fb-1c7a36ebcd5f
Content: Interacting with an ArcGIS map that has been shared with you - Power BI
Content Source: powerbi-docs/visuals/power-bi-visualizations-arcgis.md
Service: powerbi
Sub-service: pbi-visuals
GitHub Login: @mihart
Microsoft Alias: mihart

Thanks for your comment, @Hikmer . This issue has been assigned to the doc author for review.

Hi @Hikmer ,
Thank you for your suggestion.  Would you please clarify what you mean by "workspace settings" for ArcGIS? If an ArcGIS map is one of the visuals on a page in a report in a workspace, the "workspace settings" are the same as the settings for the rest of the pages in the report.  If I'm not understanding your questions, please let me know.
Take care,
the DOCS team

This is mentioned in the "Manage the use of ArcGIS for Power BI within your organization" section of the doc. Also this is a duplicate (or the other is a duplicate)

This is missing from the documentation
[image: image.png]
On Wed, Apr 7, 2021 at 3:44 PM msftrien @.***> wrote:

This is mentioned in the "Manage the use of ArcGIS for Power BI within
your organization" section of the doc. Also this is a duplicate (or the
other is a duplicate)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/powerbi-docs/issues/2855#issuecomment-815308944,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFMTYD7ZJGC3KACZ34EDMYTTHTN3VANCNFSM4YC3JZKA
.


Update variables to work with the ansible setting 'inject_facts_as_vars' set to False

Some variables are referenced by the name ansible_ which is only set when the config option 'inject_facts_as_vars' is set to True (which is is by default). If this option is set to false this role stops working. The solution is to reference the variable by their full fact name (ansible_fact.<variable name>).
https://docs.ansible.com/ansible/latest/reference_appendices/config.html#inject-facts-as-vars
https://docs.ansible.com/ansible/latest/reference_appendices/faq.html#when-is-it-unsafe-to-bulk-set-task-arguments-from-a-variable

Thanks for the PR. There seems to be some problem with the build but it's not on your side. I'm trying to fix it - when I'm done with it, we can rebase and get this tested.

Sounds good! Just write here when you are done and I will rebase :+1:

You can already do it :)

Ok, DigitalOcean is acting weird, I'll need to have a further look.

No problem, just write here when it's ready for another rebase.

I think you can now rebase again

Still seems like the test does not work

Oh, I see, the issue was this all along 🤦  https://github.community/t/make-secrets-available-to-builds-of-forks/16166/32
Since the local tests are passing, I'll go ahead and merge them. For the external test, I'll later try to exclude them from running if PR is coming from a fork.
Thanks for the contribution!


Accordion for the program page

Before sending your pull-request 💌 make sure :

[ ] The tests are up-to-date and your code is tested

I have a problem of having a flatlist inside a scrollview. If anyone can have a look


#15117 fix drivers for folder without parents

Closes #15117

Drivers need to be downloaded into C:\Users\usr_name\AppData\Roaming\DBeaverData\drivers but now they are downloaded into C:\Users\usr_name\AppData\Roaming\DBeaverData\workspace6\drivers which can lead to excessive storage use as well as other hidden issued.

Verified


feat(ONYX-1169): Implement submission status in My Collection artwork grid

This PR resolves ONYX-1169
Description

Display submission status in MyCollection artwork grid
Hide High demand icon if artwork has been submitted for sale

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[ ] I added an app state migration, or my changes do not require one.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

Implement submission status in My Collection artwork grid (behind AREnableSubmitArtworkTier2Information) -daria

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Cross-platform user-facing changes (Implement submission status in My Collection artwork grid (behind AREnableSubmitArtworkTier2Information) -daria)

  Generated by :no_entry_sign: dangerJS against f77944bc6c2534751c80f99590b1fbf40f66ff73


NCD - Labs - Pregnancy [8h]

Issue
We need to test women of child bearing age for pregnancy in NCD as it creates other complications and will be handled by other services.
Notes:

this test only populates for women of child bearing age as we have previously defined in the app
only the first questions should populate - if answer is yes, record and move on. If answer is no, then the rest of the question will populate.

Lab Order

Lab Results

To Do

Style screens per design
create the logic for it to only appear for certain patients
adjust any other necessary logic

@jeanette-pelizzon
Isn't pregnancy an RDT test, similar to HIV?

https://nutrition-ihangane.pantheonsite.io/
QA Review
Checked and approved:

Questions regarding pregnancy are populated per the conditions.

@anvmn Seems like something is weird with adding a new participant. Moving this back to To Do for QA followup

@adamhstewart Fixed and depoyed to Nutrition env for retesting

https://nutrition-ihangane.pantheonsite.io/app/#individual-participants/ncd
QA Review
Checked and approved:

The register a new participant link under NCD encounter is correctly working, I have created a new patient.


bpf: overlay: remove unused code

The hdrlen calculation in handle_ipv6() isn't needed any longer. Remove it.

/test


Update spark-mlcontext-programming-guide.md

Added code for Spark 1.6 since the example breaks on Spark 1.6

Can an Admin verify this patch?

Yes, just run the code on Spark 1.6.X and it breaks, use my two lines and it works on 1.6.X

Hi @romeokienzler. In PR245 you also included the import and creation of the sqlContext. Should this be included or should this PR be merged as is?
Thank you for committing to the master branch.

add to whitelist

Actually, looking at the usage, the current commit looks great since the sqlContext is used in the commented code. LGTM. Thank you @romeokienzler. I will merge.

This update is now available publicly here: http://apache.github.io/incubator-systemml/spark-mlcontext-programming-guide.html

@deroneriksson I've double checked, the SQL-Context is needed indeed, so I've created a new pull request

@deroneriksson Here the sqlContext is needed again, that's the reason I've created it - now I've commited a better solution by importing import sqlContext.implicits._ I don't know (since this tutorial is written for Zeppelin and I'm using jupyther on IBM Data Science Workbench if sqlContext and the import for sqlContext.implicits._ is done by Zeppelin itself, do you know that? But anyways, this solution works for this but this definitely doesn't work in Jupyter. Once this pull request is merged I can work on the fix for this  section as well


Update to Inline Style

Update in order to Fix Code Splitting Issue with Laravel Mix, Laravael Mix has an issue with Styles when code-splitting, this is a Webpack related Issue, Having Inline Styles creates a situation where the component loads sometimes after compilation and does not most times. Inlining the styles and removing the Style tag from the component, Fixes the issue.

Sorry for the late reaction and thank you!


Steph's proofreading

Proofreading complete! This is really excellent and I'm really excited about it.

[ch45573]


Rename EMARDD Syndrome to MEGF10-Related Myopathy

Mondo term (ID and Label): Early-onset myopathy-areflexia-respiratory distress-dysphagia syndrome (MONDO:0013731)
Suggested new label: MEGF10-Related Myopathy, MEGF10 Myopathy
Optional: Any additional information (like supporting evidence, PubMed ID, etc.) The ClinGen Congenital Myopathy GCEP reviewed this classification and determined this naming schema does not reflect the phenotypes of all probands with variants in MEGF10, particularly as a later-onset milder variant is reflected under the same disease entity (PMIDs: 22371254, 27460346 for examples). These phenotypes also appear individually or in various combinations in other myopathies (Hereditary myopathy with early respiratory failure PMID: 15802564, TPM2-Related Myopathy (Historically Nemaline/Cap Myopathy) PMID: 19047562) which could be confusing for clinicians.
@nicolevasilevsky : This is a ClinGen request

Thanks @RedRiver1816, I revised the label.
I added this text definition as well, does this look okay?
A congenital myopathy caused by mutations in the multiple epidermal growth factor-like domains 10 (MEGF10) gene, which causes early-onset myopathy characterized by severe weakness, scoliosis, joint contractures, areflexia, respiratory distress, and dysphagia, and a milder phenotype of minicore myopathy.

@nicolevasilevsky That all looks fine! Thanks for your assistance.


Propshaft ESM Modules chunks not resolved correctly (works with sprockets-rails)

application.js:

app/views/layouts/application.html.erb:

Google Chrome console / networks tab

Scenarios:
Gemfile: Propshaft uncommented, sprockets-rails commented
ESBUILD_MODULES=1 ./bin/dev
Output: 🔴
Gemfile: Propshaft commented, sprockets-rails uncommented
ESBUILD_MODULES=1 ./bin/dev
Output: 🟢
Gemfile: Propshaft commented, sprockets-rails uncommented
./bin/dev
Output: 🟢
Gemfile: Propshaft uncommented, sprockets-rails commented
./bin/dev
Output: 🟢
Minimalist application to reproduce the issue: https://github.com/navidemad/PropshaftApp

If you precompile your assets you will probably see that sprockets breaks too, because it only adds digests to the file names after precompilation.
I think we are going to need a jscompiler like we have for csscompiler to solve this.

Hi
By changing the regex in asset.rb to match the one in the this sprockets pull request
and using the following esbuild configuration from the sample repo above:
"use strict";

import esbuild from "esbuild";
import path from "path";
import glob from "glob";

const __dirname = path.resolve();

var entryPoints = glob
  .sync(path.resolve(__dirname, `app/javascript/*.js`))
  .reduce(function (map, filepath) {
    map[path.basename(filepath, ".js")] = [filepath];
    return map;
  }, {});

const esbuildOptions = {
  bundle: true,
  logLevel: "debug",
  entryPoints: entryPoints,
  format: "esm",
  outdir: path.resolve(__dirname, "app/assets/builds/js/modern"),
  chunkNames: '[name]-[hash].digested',
  splitting: typeof process.env.ESBUILD_MODULES != "undefined",
  treeShaking: true,
  sourcemap: false,
  minify: false,
  metafile: false,
  platform: "browser",
  watch: true,
};

esbuild
  .build(esbuildOptions)
  .catch((e) => {
    compilationError(e.message);
    process.exit(1);
  });

everything seems to be working normally:

@thanosbellos Thanks for the heads up. https://github.com/rails/propshaft/pull/55 is up.

Thanks a lot for your work. This is the only thing, that kept me from upgrading to propshaft and Rails 7.0. I hope that the sprockets pull request, is also merged into the master so that any legacy app can use the same configuration.
Thanks again.

@navidemad #55 was merged. Using the chunkNames config demonstrated by @thanosbellos should be enough to get chunks working in your app. Could you check if everything is ok, so we can close this issue and the other PR?

@brenogazzola Confirmed, it's working well when I specify in my esbuild config:
chunkNames: '[name]-[hash].digested',
What about https://github.com/rails/propshaft/pull/49, does it still necessary ?

Nope. I've closed that one and left a note. If it's working well, then I'll consider this one solved.


Generate 128x128 images

Hi,
thank you very much for this code. I have only one question: how may we generate 128x128 images instead 64x64 using test.py? It is possible to use the pretrained model publaynet_netG.pkl or it is necessary to perform another training modifying the network?
Best regards

Were you able to get meaningful generations (for the original 64x64 case) using the pretrained publaynet_netG.pkl model?


Add @pending test support

In vanila mocha it is:
it("test name");

@suite class Suite1 {
    @test @pending test1() {}
    @pending test2() {}
}

Both test1 and test2 will be considered pending tests.


undefined symbol: PyByteArray_Type

When issuing the following command:
$ vim --cmd 'py3 import struct'

The following error is thrown:
Error detected while processing pre-vimrc command line:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/local/lib/python3.7/struct.py", line 13, in <module>
    from _struct import *
ImportError: /usr/local/lib/python3.7/lib-dynload/_struct.cpython-37m-x86_64-linux-gnu.so: undefined symbol: PyByteArray_Type
Press ENTER or type command to continue

The error only occurs when running the internal vim Python 3 interpreter.
vim was built from source and configured with the following:
./configure --enable-python3interp=yes --with-python3-command=python3 --enable-fail-if-missing

vim version command output:
$ vim --version
VIM - Vi IMproved 8.1 (2018 May 18, compiled Nov 21 2018 21:36:44)
Included patches: 1-535
...
Huge version without GUI.  Features included (+) or not (-):
+acl               +extra_search      +mouse_netterm     +tag_old_static
+arabic            +farsi             +mouse_sgr         -tag_any_white
+autocmd           +file_in_path      -mouse_sysmouse    -tcl
+autochdir         +find_in_path      +mouse_urxvt       +termguicolors
-autoservername    +float             +mouse_xterm       +terminal
-balloon_eval      +folding           +multi_byte        +terminfo
+balloon_eval_term -footer            +multi_lang        +termresponse
-browse            +fork()            -mzscheme          +textobjects
++builtin_terms    +gettext           +netbeans_intg     +timers
+byte_offset       -hangul_input      +num64             +title
+channel           +iconv             +packages          -toolbar
+cindent           +insert_expand     +path_extra        +user_commands
-clientserver      +job               -perl              +vartabs
-clipboard         +jumplist          +persistent_undo   +vertsplit
+cmdline_compl     +keymap            +postscript        +virtualedit
+cmdline_hist      +lambda            +printer           +visual
+cmdline_info      +langmap           +profile           +visualextra
+comments          +libcall           -python            +viminfo
+conceal           +linebreak         +python3           +vreplace
+cryptv            +lispindent        +quickfix          +wildignore
+cscope            +listcmds          +reltime           +wildmenu
+cursorbind        +localmap          +rightleft         +windows
+cursorshape       -lua               -ruby              +writebackup
+dialog_con        +menu              +scrollbind        -X11
+diff              +mksession         +signs             -xfontset
+digraphs          +modify_fname      +smartindent       -xim
-dnd               +mouse             +startuptime       -xpm
-ebcdic            -mouseshape        +statusline        -xsmp
+emacs_tags        +mouse_dec         -sun_workshop      -xterm_clipboard
+eval              -mouse_gpm         +syntax            -xterm_save
+ex_extra          -mouse_jsbterm     +tag_binary
   system vimrc file: "$VIM/vimrc"
     user vimrc file: "$HOME/.vimrc"
 2nd user vimrc file: "~/.vim/vimrc"
      user exrc file: "$HOME/.exrc"
       defaults file: "$VIMRUNTIME/defaults.vim"
  fall-back for $VIM: "/usr/local/share/vim"
Compilation: gcc -std=gnu99 -c -I. -Iproto -DHAVE_CONFIG_H     -g -O2 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
Linking: gcc -std=gnu99   -L/usr/local/lib -Wl,--as-needed -o vim        -lm -ltinfo -lnsl  -lselinux  -ldl     -L/usr/local/lib/python3.7/config-3.7m-x86_64-linux-gnu -lpython3.7m -lpthread -ldl -lutil -lm

The error is also reproducible using the non-stripped version of vim.
However, the symbol in question is defined in the non-stripped version (I'm assuming it also exists in the stripped/installed version - just hard to confirm with no debugging symbols):
$ nm --defined-only vim | grep PyByteArray_Type
0000000000aeac00 D PyByteArray_Type

It does not occur when running the installed (system) Python 3 interpreter.
As expected, the same symbol is also defined in the system interpreter:
$ nm --defined-only /usr/local/bin/python3 | grep PyByteArray_Type
00000000008f7140 D PyByteArray_Type

The vim interpreter was linked with the same Python 3 library as used by the system interpreter.

What answer do you get to python3 --version at the shell prompt? (It should be 3.7 with possibly a third number.)
Do you get the same error if you type :py3 import struct after Vim has started?
If you get 3.7[.nn] at step 1 and the same error at step 2, try recompiling with make reconfig, which will rerun configure then recompile everything, so you should set your configure arguments in advance by means of environment variables, as explained at http://users.skynet.be/antoine.mechelynck/vim/compunix.htm (Caution: if you set them by means of a bash script, as I do, that script must be sourced, not just run, in the same shell where you will then compile.)

If after step 3 then the usual make install both run successfully you still get the same error with the newly compiled Vim, then I don't (yet?) know what to try next.
Best regards,
Tony.

Thanks Tony.

Python 3 version:

$ python3 --version
Python 3.7.1

Yes, the same exact error.

Unfortunately, no joy. Same problem

Okay, so it appears the issue can be resolved by pre-setting LDFLAGS with the following and then rebuilding vim:
$ export LDFLAGS="-rdynamic"

It appears to successfully resolve the symbol reference I was having issues with.
Curiously, a brief look at if_python3.c implies the Python 3 library is treated as a shared library (which could be totally wrong as I haven't dug any deeper).
However, the actual library that's linked is static (although, this may be quite normal).

For completeness, this is the current source I have checked out:
$ git log -n 1
commit 6b731886ca94d66b9bdedfb7e603af44a6400399
Author: Bram Moolenaar<EMAIL_ADDRESS>Date:   Fri Nov 16 20:54:47 2018 +0100

    patch 8.1.0535: increment/decrement might get interrupted by updating folds

    Problem:    Increment/decrement might get interrupted by updating folds.
    Solution:   Disable fold updating for a moment. (Christian Brabandt,
                closes #3599)

Bram's current source is actually 8.1.0541 as of this writing, but the latest patchlevel is Windows-only AFAICT so I'm using 8.1.0540 for the time being. I don't see anything related to solving your problem in patches 536 to 541 however.
My Huge build of (g)vim (the only one of my builds which has interpreters, not python3 however: it is built with +lua -mzscheme +perl +python -python3 +ruby +tcl) gets linked with -rdynamic -Wl,-export-dynamic near the start of its link command-line without the need for me to specify them explicitly.
@brammool @chrisbra : maybe auto/configure forgets to include these switches when building with python3 and no other interpreters? Or is it something else?

Closing as environmental issue.
Simple workaround is mentioned above.

I was hitting the same problem, me too on CentOs 7. Are we sure it's not a bug in the build scripts?

Same problem here, but inside a virtualenv. The workaround fixed only outside virtualenvs.

Same problem here when compiling on Manjaro kde.
$ git log -1
commit 38854b565acba39eff36cf3c6396c911bf072bdc (HEAD -> master, tag: v9.0.0962, origin/master, origin/HEAD)
Author: porygonisaduck<EMAIL_ADDRESS>Date:   Sun Nov 27 20:55:05 2022 +0000

patch 9.0.0962: virtual text below cannot be placed below empty lines

Problem:    Virtual text below cannot be placed below empty lines.
Solution:   Add one character. (James Alvarado, closes #11606, closes #11520)

The workaround helps.
My command is:
LDFLAGS="-rdynamic" ./configure --prefix=/home/yggdroot/.vim9 --enable-luainterp  --enable-python3interp --enable-cscope --enable-gui


Change Update Location

After my user account migration I tried to open a single project. Everything is OK except: I cannot see ANY Update because every update is saved in %AppData%.
I would like you to change that behaviour:
Either let the user specify where to store update files, make ovious where update files are stored or use a more intuitive path (i.e. in the project file directory under updates).

I will add the possibility to change the path where the package files are stored.


Switched rvm out for rbenv - now nothing works...

Hi,
I switched rvm out for rbenv - installed and looks to be ok according to the rbenv doctor script, but I can't do anything now, trying to run rails s:
/.rbenv/versions/2.3.0/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in require: cannot load such file -- rails/cli (LoadError)
trying to run bundler:
/.rbenv/versions/2.3.0/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in require: cannot load such file -- bundler (LoadError)
any idea what I can do to fix?
J.

Hello McFly!
I shouldn't do these things so late in the evening - had to reinstall rails and bundler...
J.


Distributed zqd on K8s using HAProxy with "leastconn" load balancing

Moving forward with Noah's idea to use an out-of-the-box least-connection load balancing algorithm rather than implement custom job scheduling for now.

I copied the changes I needed from this PR into #1695.


Prevent plugin binaries from releasing

WHY
WHAT
REF

Codecov Report

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

@@           Coverage Diff           @@
##           master     #188   +/-   ##
=======================================
  Coverage   61.05%   61.05%
=======================================
  Files          39       39
  Lines        1407     1407
=======================================
  Hits          859      859
  Misses        488      488
  Partials       60       60

Continue to review full report at Codecov.

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


Roll Engine from 2440ad6d39a5 to 10f06b5abf84 (3 revisions)

https://github.com/flutter/engine/compare/2440ad6d39a5...10f06b5abf84
2020-11-06<EMAIL_ADDRESS>[web] Implement style inheritance during paragraph construction (flutter/engine#22365)
2020-11-06<EMAIL_ADDRESS>Revert "Rasterizer is initialized with an external view embedder (#22348)" (flutter/engine#22367)
2020-11-06<EMAIL_ADDRESS>Roll Skia from e886b8e8b10b to 86d4cfdf8edc (2 revisions) (flutter/engine#22364)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


Infinite loss value when training under amp

Hi, I encounter the infinite loss value assertion failure when training using mixed precision.
The trackback like this:
Traceback (most recent call last):
  File "main.py", line 498, in <module>
    main(args)
  File "main.py", line 409, in main
    train_stats = train_one_epoch(
  File "ConvNeXt/engine.py", line 63, in train_one_epoch
    assert math.isfinite(loss_value)
AssertionError

I wonder how I could fix this problem. Thanks very much!

Hi,
This happened to me as well, did you find a solution to this problem?


Most problems deploying/upgrading BinderHub can be solved by checking indentation

Hello!
So 9 times out of 10 when my BinderHub is acting weird, it's because I've indented something incorrectly in config.yaml. There was a (brief) discussion in #835 around trying to implement some kind of linting check to catch these types of mistakes, but there isn't a concrete plan yet.
For the time being, I suggest updating the Debugging BinderHub docs to advise users to thoroughly check their config files before diving into k8s debugging.
Are there any thoughts or further ideas around this?

This is a general problem with helm, silently ignoring misspelled or mis-structured config instead of warning about it. It's feasible to handle certain common mistakes like verifying the list of top-level keys in values.yaml, but general linting is going to be tough without a dedicated command that's not run on helm upgrade, which is of significantly reduced utility.

Agreed. I guess what I want to do is raise awareness that this is probably the most likely cause of a problem and to check that before deep-diving into any k8s stuff.
This might be an ok resource to link to. It doesn't seem perfect, but better than nothing. I might play with it over the next week or so and see how I get along with it.

Yeah. It would be really awesome if helm could record which items in values.yaml were visited and then  show you what's been ignored. That might be pretty hard, though.
helm lint is useful, but I think it doesn't help with the ignored-config case. It does help catch things like producing invalid templates. If you are exploring it, maybe check out the jupyterhub lint script, which we use to run yamllint, helm lint, and kubeval to test the validity of the chart.

Ah, what might be useful for ignored config is helm diff, which shows you what would change. If you think you're changing something and the diff is empty, that's probably a good clue!

Thanks, those look like great resources!

I think adding some text to the documentation (or even just linking to this issue) would already help get people unstuck. Should we do that while we investigate more automatic/tooling based approaches?
Another thing to do is add a paragraph to the docs about the difference between using the BinderHub chart (in which case you (probably) have a config.yaml) and the case where you make a chart that uses the BinderHub chart (a la https://github.com/jupyterhub/mybinder.org-deploy/). Because now you need an extra level of indentation.
I think our docs are consistent in using the config.yaml approach but people might find snippets around the internet or look at the mybinder.org config files and then end up stumped.

Another thing to do is add a paragraph to the docs about the difference between using the BinderHub chart (in which case you (probably) have a config.yaml) and the case where you make a chart that uses the BinderHub chart (a la https://github.com/jupyterhub/mybinder.org-deploy/). Because now you need an extra level of indentation.

The third flavour being you have a BinderHub config.yaml file and you are customising it using the JupyterHub guide. In which case, everything you see on those pages will need to go under the jupyterhub key in config.yaml. But it's not correct to change those docs as they are correct for people deploying a JupyterHub only.

@minrk just gave a great answer here that could be written up

Yeah, docs on common gotchas (e.g. extra indent) and troubleshooting (helm diff) would be a great start.

I'm having a headache with connecting an Azure container registry to my BinderHub. So if I don't have any breakthroughs with that, I can start a PR for this 😄

@sgibson91 :heart: :tada: you are great! :D

PR open at #847 🎉


Start Kitematic with Virtualbox automatically

Expected behavior
Start Kitematic with Virtualbox automatically.
Actual behavior

Everytime I start the Kitematic and I got the error above.
If I click on "Use Virtualbox", everything works as expected.

Steps to reproduce the behavior

Start "Docker Quickstart Terminal"
Start "Kitematic"
Got error as image above
Click on "Use Virtualbox"
Everything works

Other information
I'm using VirtualBox because I need the VMware for other personal project and I can't keep Hyper-V and VMware installed the same time.

Please upgrade to the latest release where this option can be set in the preference panel.

I got the Kitematic version from latest Docker Toolbox.
Now I update the Kitematic manually and everything works.
Thanks a lot!


Warn users that figure caption is required

@wrynearson @naomatheus @deborahUAH
Description:
All figures should include a caption created by the user.  To ensure this happens, a warning message is needed to tell users that a figure caption is required.
Tasks:

[ ] Provide the user with a warning message when an image (figure) is added without a caption.  This warning should be triggered for both PDF types. This warning should state "A figure caption is required."

Acceptance Criteria:

[ ] A warning appears when a user adds an image but no caption (need to determine when this warning should occur).

###Example:

@frozenhelium we have many possible ways to implement this (a check at the time of image upload, a check on edit page change, a check on draft -> closed review, etc.


[Bug] Crashlytics plugin - keys don't work

I am using the latest version firebase_crashlytics: 0.0.4+3, but there is a issue in the _prepareKeys() function , could you please confirm and fix it ? thank you.
@Hixie @kroikie

I've created a PR (https://github.com/flutter/plugins/pull/1799) that fixes this problem.

FYI @kroikie
I've added some tests for this feature to your PR.

Although this was closed and there has been a change for fixing the problem, still something is wrong as keys still are not reported!
There is a related open issue here: https://github.com/flutter/flutter/issues/31371
Please upvote this to make sure it gets fixed sooner. I don't want to use other unofficial plugins!

@realcarlos
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/1144. Any further collaboration will be done there.


Add support to rails 5

This is related to issue #447 and PR #486.
In #447 I made a full log of what I did, also commented in #486, since it will fix mime-types but will not work in rails 5.
Also this PR, is not to merge yet. We should wait sinatra release the 2.0 stable version.
Thanks

I am closing this PR, since this is not compatible with previous versions of rails.
I tried to test it locally, but no success.
So the best solution is to use gem "sinatra", "~> 2.0" instead of putting the version as dependent, otherwise will break older version of rails.

I am getting this error trying to include sinatra as well.
Could not find gem 'sinatra (~> 2.0)' in any of the gem sources listed in your Gemfile or available on this machine.


[BUG] ariaExtract: multi-looking results in all nans in the geometry files

Describe the bug
Running ariaExtract.py with -ml option set to 2 results in all invalid pixels in the geometry files (incidenceAngle, lookAngle, azimuthAngle, bParallel, bPerpendicular)
To Reproduce
Steps to reproduce the behavior:

Download a set of ARIA interferograms and put them in products/
Create an 'output' directory to store products
Command used: ariaExtract.py -f 'products/S1-GUNW*.nc' -w output -l 'all' -d Download -ml 2
Output files listed above do not contain any valid pixels, and the DEM contains all zeros.

Running the same command without the -ml argument works fine.
Expected behavior
Get valid output geometry files that correspond to the multilooked IFGs
Desktop (please complete the following information):

ARIA-tools git tag: [e.g. git show --summary]: 0bff9fc6568e7856606fad82e74547f8cfcc623b
OS: Mac OS Big Sur 11.2.3 (20D91)

@jlmaurer following up the discussion we had in the meeting today, an easy test to see if the issue is the overlap is to just run without multi-looking and see if the invalid pixels are still present.

Hi @bbuzz31 thanks for that! I checked both versions (no multi-looking and with multi-looking). The image shows that the full-res file seems to work just fine (with of course invalid pixels outside the bounds of the frame). Not sure whether this makes sense based on what you were saying?
Command for hi-res version:
ariaExtract.py -f 'products/S1-GUNW-A-R-077-tops-2020*.nc' -w output_hires -l 'all' -d Download

Command for lo-res version:
ariaExtract.py -f 'products/S1-GUNW-A-R-077-tops-2020*.nc' -w output -l 'all' -d Download -ml 2

Thanks for sharing. I think this means that the minimum overlap is NOT the problem, perhaps @sssangha can comment.
The next easiest place to check for bug would probably be the masking. The value 0 was recently introduced to represent nans following MintPy, which I noticed caused a bug in tsPlot

@jlmaurer @bbuzz31 in my latest PR #275 , I am able to confirm that the multilooked fields are no longer empty:

@sssangha that's great! What was causing the issue?

@jlmaurer I should clarify that while I wasn't able to replicate your exact issue. There were problems with attempting to download and manage large DEMs because the product shapefiles were being leveraged as opposed to the actual broader, grid extent of the region. It is possible this may translate to empty fields downstream, but I wasn't able to replicate it after my fixes.

@jlmaurer could you use PR #275 to see if this is still an issue?

@jlmaurer we will cut a release for ARIA-tools, but want to make sure this is not an issue. Could you give it another test when you are back from holiday?

@sssangha @dbekaert I pulled the latest changes and also using the later version of GDAL. I think the issue has something to do with the DEM; the uncropped DEM is fine but the cropped and multi-looked DEM is all NaNs.

@jlmaurer Brett appears to have replicated your issue with an older version of gdal (3.1.4), so I did a completely new install where I rebuilt the environment from scratch (ended up with goal 3.2.1), and find that this issue does not arise. See DEM and incidence angle below:

I'm thinking that this may not be just a problem associated with an older version of gdal per se, but perhaps a combination of packages which result in this issue. I would suggest to completely rebuilt a new conda environment from scratch as I did and hopefully you can confirm my findings. Would you or @bbuzz31 have some bandwidth for this? @rzinke did you recently encounter such an issue?
Many thanks!

@sssangha that is a good point, I will try rebuilding the environment from scratch and see if that works.

@sssangha I rebuilt from scratch with gdal v3.1.2.
I'm now encountering an error message.
Also, I've noticed you aren't running on the same track. May be better to focus on one before testing on others.
ariaDownload.py -t 077-s 20200101 -e 20200201
ariaExtract.py -f 'products/S1-GUNW-A-R-077-tops-2020*.nc' -l 'all' -d Download -ml 2

Errors:
ARIA-tools Version: 1.1.0
*****************************************************************
*** Extract Product Function ***
*****************************************************************
Multi-core version
All (8) GUNW products meet spatial bbox criteria.
Group GUNW products into spatiotemporally continuous interferograms.
All (1) interferograms are spatially continuous.
All layers are to be extracted, pass all keys.
Thread count specified for gdal multiprocessing = 2
WARNING: User-defined bounds results in an area of 684150 km which exceeds the maximum download area of 400000; downloading in chunks
^[Warning 1: the input vector layer has a SRS, but the source raster dataset does not.
Cutline results may be incorrect.
Applied cutline to produce 3 arc-sec SRTM DEM: ./DEM/SRTM_3arcsec.dem
Generating: unwrappedPhase - [==================================================] 20200124_20200112NETCDF:"/Volumes/BB_4TB/data/VLM/Sentinel1/track_077/products/S1-GUNW-A-R-077-tops-20200124_
20200112-231342-33720N_31670N-PP-3e2b-v2_0_2.nc":/science/grids/data/unwrappedPhase is GDAL compatible
NETCDF:"/Volumes/BB_4TB/data/VLM/Sentinel1/track_077/products/S1-GUNW-A-R-077-tops-20200124_20200112-231342-33720N_31670N-PP-3e2b-v2_0_2.nc":/science/grids/data/connectedComponents is GDAL compatible
(many more of these omitted for brevity)
Traceback (most recent call last):
  File "/Users/bb/Miniconda3/envs/ARIA-tools-test/bin/ariaExtract.py", line 21, in <module>
    main(inps)
  File "/Users/bb/Miniconda3/envs/ARIA/lib/python3.8/site-packages/ARIAtools/extractProduct.py", line 1193, in main
    export_products(standardproduct_info.products[1], standardproduct_info.bbox_file,
  File "/Users/bb/Miniconda3/envs/ARIA/lib/python3.8/site-packages/ARIAtools/extractProduct.py", line 657, in export_products
    product_stitch_overlap(unw_files,conn_files,prod_bbox_files,bounds,prods_TOTbbox, outFileUnw=outFileUnw,outFileConnComp= outFileConnComp, mask=mask,outputFormat = outputFormat,verbose=verbose)
  File "/Users/bb/Miniconda3/envs/ARIA/lib/python3.8/site-packages/ARIAtools/unwrapStitching.py", line 1458, in product_stitch_overlap
    unw.UnwrapOverlap()
  File "/Users/bb/Miniconda3/envs/ARIA/lib/python3.8/site-packages/ARIAtools/unwrapStitching.py", line 340, in UnwrapOverlap
    self.__calculateCyclesOverlap__()
  File "/Users/bb/Miniconda3/envs/ARIA/lib/python3.8/site-packages/ARIAtools/unwrapStitching.py", line 387, in __calculateCyclesOverlap__
    connCompFile1 = gdal.Warp('', self.ccFile[counter], options=gdal.WarpOptions(format="MEM", cutlineDSName=outname, outputBounds=polyOverlap.bounds, dstNodata=connCompNoData1))
  File "/Users/bb/Miniconda3/envs/ARIA/lib/python3.8/site-packages/osgeo/gdal.py", line 677, in Warp
    return wrapper_GDALWarpDestName(destNameOrDestDS, srcDSTab, opts, callback, callback_data)
  File "/Users/bb/Miniconda3/envs/ARIA/lib/python3.8/site-packages/osgeo/gdal.py", line 4319, in wrapper_GDALWarpDestName
    return _gdal.wrapper_GDALWarpDestName(*args)
RuntimeError: Cannot find coordinate operations from `GEOGCRS["unknown",DATUM["unnamed",ELLIPSOID["Spheroid",6378137,298.257223563,LENGTHUNIT["metre",1,ID["EPSG",9001]]]],PRIMEM["Greenwich",0,ANGLEUNIT["degree",0.0174532925199433,ID["EPSG",9122]]],CS[ellipsoidal,2],AXIS["latitude",north,ORDER[1],ANGLEUNIT["degree",0.0174532925199433,ID["EPSG",9122]]],AXIS["longitude",east,ORDER[2],ANGLEUNIT["degree",0.0174532925199433,ID["EPSG",9122]]]]' to `GEOGCRS["WGS 84",DATUM["World Geodetic System 1984",ELLIPSOID["WGS 84",6378137,298.257223563,LENGTHUNIT["metre",1]]],PRIMEM["Greenwich",0,ANGLEUNIT["degree",0.0174532925199433]],CS[ellipsoidal,2],AXIS["latitude",north,ORDER[1],ANGLEUNIT["degree",0.0174532925199433]],AXIS["longitude",east,ORDER[2],ANGLEUNIT["degree",0.0174532925199433]],ID["EPSG",4326]]'

Note the black squares in this UNCROPPED DEM and the weird black line through the middle. Is something not getting stitched correctly?

@bbuzz31 I tried it again using your track and commands, and didn't encounter the same issue nor found a gap in my DEM. Below are the DEM and azimuth angle field:

It is possible that perhaps our environments are slightly different. Excuse me for spamming the thread with a long list, but I will share my conda environment below.

Thanks for confirming @bbuzz31


Adding @scottnicolson as a project collaborator

Hi @mimshwright and @DJTB,
I want to open the discussion of adding @scottnicolson to the project team as a collaborator. He would act as a representative on our end (@tofugu).
We recently have been active in our development which involves the integration of WanaKana. The work has surfaced some issues and things we would like to adjust. Some of this is already happening now with https://github.com/WaniKani/WanaKana/pull/153 and https://github.com/WaniKani/WanaKana/pull/151. I also think it’ll be positive to add another helping hand to assist in any goals this project has and process community submitted issues & PRs.
The main purpose is to shorten the feedback loop; working together to discuss these changes, how it fits with the goals of the project, and to get these changes out.
I am not super familiar at the moment with the current governance or working arrangement of the project. I (and @scottnicolson) want to respect what is in place if there is one well established. If there are any concerns please don’t hesitate to voice them.
I appreciate your involvement and views @mimshwright and @DJTB. I'll leave @scottnicolson to do their intros.
Mims, if you feel good about whats being proposed, you should have the necessary permissions to be able to add Scott to the project team. If not, I can do it.

Completely fine with bringing Scott on board.
There hasn't really been any specific governance, I've taken the helm by default since I was the only maintainer particularly active.
We'll also need to invite scott to the npm team for publishing permissions (you, myself, or mims can action this).

@DJTB thanks for the support.

Is there a particular way that you would like to work together on changes? and are there changes that do need attention?
When it comes to changes do we just review each others PRs? Do all PRs need to be reviewed?
Is there a guiding principle for what this project is and what it is not that will help determine whether changes should be made?  For example, there was talk about moving this project to typescript, was there a reason for this?  i.e. is the project considered feature complete is now just about fixing any issues? if so, is switching to typescript very high value for a project that is in maintenance mode?

I think it's fine to propose a feature/change in an issue if you want feedback, but if it's a bug fix feel free to go ahead and raise a PR without preamble.
Review each others PRs - for the past five or so years it's basically been me alone so I've pushed changes through without reviews due to radio silence. I'm happy to review your PRs and have you review mine though.
I think the guiding principle at this stage is to avoid adding unnecessary changes / overcomplicate further. Personally I think the project is faily feature complete for what it intends to provide. I think any major additions / diversions should probably be done in another package (e.g. if we chose to provide a Wanakana react component, it should be separate and use Wanakana as a peer-dep).
Moving to Typescript I believe is considered a nice-to-have by @mimshwright and I. If this project was started today, that's how we'd go about it for ease of development / maintenance / consumption.

Review each others PRs - for the past five or so years it's basically been me alone so I've pushed changes through without reviews due to radio silence. I'm happy to review your PRs and have you review mine though.

This works for me if you are happy with this too.

I think the guiding principle at this stage is to avoid adding unnecessary changes / overcomplicate further. Personally I think the project is faily feature complete for what it intends to provide. I think any major additions / diversions should probably be done in another package (e.g. if we chose to provide a Wanakana react component, it should be separate and use Wanakana as a peer-dep).

Sounds good.

Moving to Typescript I believe is considered a nice-to-have by @mimshwright and I. If this project was started today, that's how we'd go about it for ease of development / maintenance / consumption.

I agree if it was started today ..., my question was more about the how much value would be added if the project is in maintenance mode.  I wonder whether it would be considered a "major additions / diversion" at this point.  I am guessing though that due to time constraints of those involved that this particular issue parked for now.

In terms of bringing me on board, could you please add me to the NPM team and provide any roles I might need in GitHub?

Hi guys,
I feel like a real grandpa to this project today and I'm so glad it remains relevant to people.
I'm adding @scottnicolson to the project and I just made @DJTB an admin too. I think @DJTB might need to add you to NPM.
Governance & Guiding Principles
First, I should say, for all practical purposes, I'm not really involved anymore. Whomever is willing to put in the work should get to make the big decisions AFAIAC. Duncan's suggestion sounds fine to me.
The project was originally started as a commission by WaniKani / Tofugu. We made an agreement that I would develop it as an open source project that was sponsored by the company. The business needs of WaniKani are the major driver for all of the original features and are as good of a guiding directive as any. So I think giving them direct representation via Scott makes sense.
Typescript
The original code was written in CoffeeScript and later ported to ES6+ by @DJTB.
I discussed porting it to TypeScript and even tried to do it but it was not as easy as I expected it to be and I lost motivation to finish. And to be totally honest, I was just kind of looking for something to keep busy during the pandemic. So we could consider that parked / abandoned.
BTW, If I were to change anything at this point it would be 1. to separate the whole DOM binding functionality from the core library since it has historically been a source of issues and confusion and 2. to refactor some of the code to be more DRY. There are loads of functions that are essentially doing the same thing with minor tweaks and a functional-programming-oriented overhaul might make the code a lot tighter. But again, there is probably no real need for this since it's stable and not under active development.
Welcome to the team!

Thanks @mimshwright for sorting that and the guidance on where you would make changes.


VisIt 3.1.4 Unexpcted Message?

Describe the bug
Open DB and see:
Due to the large number of variables, 1091, in this database, VisIt has
automatically disabled speculative expression generation of ALL forms
(e.g. vector-magnitude, mesh-quality, time-derivative, operator-created)
because the large number of expressions bogs down GUI performance.
You may override this behavior by exiting VisIt and restarting with the
environment variable...
    "VISIT_FORCE_SPECULATIVE_EXPRESSION_GENERATION"
set. Be aware that doing so will likely result in sluggish GUI performance.
If you need one or more of the expressions VisIt's speculative expression
generation process provides and are unwilling or unable to manually create
the expression(s) you need, you have no choice in this version of VisIt
except to override this behavior and suffer any GUI performance issues.
Unfortunately, if you are running in client-server mode, you will have to
take action to set this enviornment variable on both the server and client
machines. This will only be necessary in VisIt version 2.12.3. An entirely
different mechanism for controlling this behavior will be introduced in 2.13.

Should we remove info about 2.12.3 and 2.13?
And are we in OK shape now with respect to this behavior.
I haven't checked for 3.3.0, we should.
Helpful additional information

Did VisIt crash:
Did you get wrong results:

Desktop

OS and version: toss3
VisIt Version: 3.1.4
Server info (if applicable):

Additional context
Add any other context about the problem here. Attach any (zipped) relevant files, such as data, configurations, sessionfiles, etc.

This is being reported on 3.1.4 here and at that time, it was still in this state. It was fixed on 3.2RC and on develop.


Add testing for data races to CI builds

Do we want a separate debug build for cli with race detector? I assume it incurs some performance penalty. It doesn't take too long to build another cli/ConsoleClient so we can just add another go build command that outputs to *-debug

I assume it incurs some performance penalty.

I also assume so, but I doubt that CPU performance penalty is going to matter in a proxy client application.


Renaming split-names response to splits

Fix content response name in split_names_from_streaming and split_names_from_dataset_info from "split_names" to "splits" so that it is consistent with the response of splits worker

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


[Testing] moveos_stdlib::raw_table

I'll do this one.

raw_table tests are covered by table and type_table


Icons to add to Clarity iconography

Select one ...  (check one with "x")
[ ] bug
[ x ] feature request
[ ] enhancement

Please add the attached three cloud icons to Clarity Iconography
cloud icon set.zip
Would be great if it's added in time for VMworld - @nbewley can provide more specific timeline.
Requested  for use in the Cloud Services Marketing website.

@Shijir
These are for the Technology category.


A function that helps you understand the functions of tools and options in .cfg file.

What
A function that helps you understand the functions of tools and options in .cfg file.
Why
It is difficult to easily understand the functions of many tools and options when writing .cfg files.
How
Use codelens to describe the functions of tools and options.
Here's an example of a function.

This is a sample of codelens and description of tools.

If you place the mouse on this comment, you will find the required options.

If you click on this comment, a message will be output.

It's a bit hard to understand where this issue came from.

It would help if you add a link if there is another related issue
I can imagine your difficulties but that can bring errors in communication

At least

We don't have any UI in our code so it's hard to catch what or how your capture came from.

Codelens is a function provided by vscode.
Here's the url of example.
https://github.com/microsoft/vscode-extension-samples/tree/main/codelens-sample
And some guide.
https://code.visualstudio.com/blogs/2017/02/12/code-lens-roundup

@emost22 , if I understand correctly, you are working on an text editor for .cfg file that shows helper balloons (codelens) ?

I don't know understood correctly "text editor" in your question, but I'm working on VSCode.

you are working on an text editor for

The text editor window inside VS code...

That's right. May I develop for this issue?

May I develop for this issue?

I think you are already on this :)
Or I didn't quite understand your question?

No, I think you understand :D
Thank you!

Closing as not active. Feel free to reopen if needed.


--terminate flag not working correctly

The --terminate flag seems not work as expected.  Looks like you may need to invert the logic and use --noterminate since it should terminate by default to match dsdgen defaults.  Currently there seems no way to set it to false given when the flag is present on the command line it is set to true state.
If I change the default to false, it works correctly, but does not match dsdgen defaults.
https://github.com/Teradata/tpcds/blob/master/src/main/java/com/teradata/tpcds/Options.java#L29

@gregrahn thanks for pointing this out! I just put up a pull request addressing this issue https://github.com/Teradata/tpcds/pull/14.


Only validate images once

This fixes a bug where the validation job could be handled more than
once which could result in a duplicate EMR cluster spinning up for the
same layer.
Connects #318

Spinning up a second local worker to test...

You could also send duplicate image validation messages to SQS to verify this.

+1. Looks good. Tested.


Github Automated Analysis

1.An AI tool for analyzing candidate’sGitHub repositories to find the most challenging project they have done.
2.Used prompt engineering and OpenAI api to get the results. Can help to automate verifying skills of a candidate in the recruitment
process.

Please assign me this issue under GSSOC'24