Replace True Breaking Pristine Class

I'm not sure if this is a bug, or if I missed something in the release notes for upgrading to 1.3 .
Using replace:true on a directive with an input element such as a checkbox in Angular 1.2 sets the classes ng-valid and ng-pristine on the parent element.
However, in Angular 1.3.6, only the ng-valid class is being set on the parent element.
Reproducible: always
Browsers: Chrome 39, Firefox 34, and IE 10
Operating system: Windows 7
Plunkers Demonstrating the Difference in Behavior:
Angular 1.3.6 : http://plnkr.co/edit/VES7ZZXfqrpXU4gVwmYs?p=preview
Angular 1.2.9 : http://plnkr.co/edit/DPAfwHjGOfMw3grYiTxf?p=preview

Note: since replace is deprecated, this is unlikely to be get fixed.
This one's actually different than all the other merge-attributes bugs with replace. Sometime in 1.2.x, the initial setting of classes for ngModel was moved to the compile function. addClass is directly called on the element. Afterwards, replace replaces the old with the new node, and merges the attributes (including classes). But it only merges the attributes that are set on the template, not on the element! Since we called addClass on the node that is getting replaced, these are lost.
Now ng-valid is on the element because it gets set after validation passes, too.
A fix would be to call $addClass on the attrs object which gets passed to the compile function, but I don't know if this would eliminate the performance gains.

Yeah, it's not a huge deal for me, the simplest work around for us was to just not use replace : true, because it's deprecated anyways.


Testing

Fix intro test
MVP for FindMeetingQuery, code passes all tests

I was working through eclipse and I think I accidentally added everything, but the only two I really need to merge with master are Greeter.java and FindMeetingQuery.java. Is there a way to not merge the rest? I don't think there should be an issue if I do merge everything though

You could submit a commit that removes all of those files, though it's fine if you merge them in as well.


I have some problems when using the api

Your Feature Request
when I use the function GetComponentImages,this function returns a pointer to Boxa type.when I want to operate on that pointer,visual studio prompts the following error：
pointer to incomplete class type "Boxa" is not allowed.  （The same problem also occurs in types such as Pixa, Boxaa, etc..I checked using visual studio and find that Boxa,Pixa type have only type declarations but no definitions.The api interface does not appear to be the cause of the problem.Mainly these of the types tesseract uses are not defined and implemented.I don't know how to fix it）
Here is my code：
#include
#include <leptonica/allheaders.h>
#include "opencv2/opencv.hpp"
#include <tesseract/baseapi.h>
#include <tesseract/publictypes.h>
#include <opencv2/imgproc.hpp>
using namespace cv;
int main()
{
std::string filename = "3.png";
Mat img;
img = imread(filename);
cv::cvtColor(img,img,COLOR_BGR2RGB);
if (img.data==nullptr)
{
    printf("No image data \n");
    return -1;
}
int w = img.cols;
int h = img.rows;
tesseract::TessBaseAPI api;
// Initialize tesseract-ocr with English, without specifying tessdata path
if (api.Init(NULL, "eng"))
{
    std::cout << stderr << std::endl;
    exit(1);
}
api.SetImage((uchar*)img.data, img.cols, img.rows, img.channels(), img.step);

//Pix* pix=api.GetInputImage();
Boxa* boxes = api.GetComponentImages(tesseract::RIL_TEXTLINE, true, NULL, NULL);
boxes->         //try to do something,Here is where it went wrong

api.End();

return 0;

}

Please respect the guidelines for posting issues: use tesseract user forum for asking questions/support.


 With BT tethering enabled on rig but iPhone hotspot shut off, anytime the rig moves out of range from Wi-Fi and attempts to connect to disabled iPhone hotspot, the rig locks up and ceases to work. Always requires a hard reset to reboot rig. We keep the iPhone hotspot on all the time but there are random times when iOS seems to take it upon itself to disable the hotspot. I have read that this may be the result of "low power mode" being activated in iOS. Unfortunately, I cannot provide any logs because the rig is locked up once this happens and there is no papertrail to pull up. Everything looks normal right up to the transition out of Wi-Fi then rig goes silent.
https://gitter.im/nightscout/intend-to-bolus?at=59f894aad6c36fca31bc1627

What do syslog and kernel.log show? Do you have a kernel panic / trace dump?

Syslog and kernel.log show nothing during the time of the problem.  The next event for both is when the rig went through the hard reset. What is a kernel panic / trace dump?  I'm guessing if I don't know what these are then I don't have them?

When I've seen something similar it has been accompanied by scary messages in the logs indicating a kernel bug. If you don't have those it's probably a different issue.

If anyone else is able to reproduce this, please speak up.  We don't see the same issue on our rigs running jubilinux 0.1.1.

My rigs both have the same behavior mentioned above- if they're unable to gain network, they eventually panic. Plugging them in for serial comms pre-hard-reset results in nothing sent (putty connects, but no commands sent/received). Happens on both jubilinux 0.1.1 and 0.2.0.
I did find #803 earlier- is this an attempted fix for this bug?

The symptoms I’m trying to fix with 803 include a kernel panic in syslog when unable to get a bnep0 IP with DHCP, which I normally observe while flying. Does your syslog show anything like that right before the rig dies?

Syslog ends before the restart with " DHCPDISCOVER on bnep0 to <IP_ADDRESS> port 67 interval 13", but doesn't report errors. The next thing is the @^@^@^ of the restart. I've see it truncate lines too, but I can't find one in recent logs.
There is a stack and a call trace in kern.log:
Nov 20 04:00:51 localhost kernel: [128650.987535] BUG: unable to handle kernel NULL pointer dereference at 00000010 Nov 20 04:00:51 localhost kernel: [128650.987657] IP: [<c17d507a>] l2cap_chan_send+0x9a/0xac0 Nov 20 04:00:51 localhost kernel: [128650.987746] *pdpt =<PHONE_NUMBER>301001 *pde =<PHONE_NUMBER>000000 Nov 20 04:00:51 localhost kernel: [128650.987826] Oops: 0000 [#1] PREEMPT SMP Nov 20 04:00:51 localhost kernel: [128650.987893] Modules linked in: bcm4334x(O) kvm_intel kvm gpio_pca953x ti_ads7955 iio_basincove_gpadc ftdi_sio uvcvideo videobuf2_vmalloc videobuf2_memops videobuf2_core usb_f_acm u_serial g_multi li$ Nov 20 04:00:51 localhost kernel: [128650.988180] CPU: 1 PID: 25195 Comm: kbnepd bnep0 Tainted: G        W  O 3.10.98-jubilinux-edison #13 Nov 20 04:00:51 localhost kernel: [128650.988274] Hardware name: Intel Corporation Merrifield/BODEGA BAY, BIOS 466 2014.06.23:19.20.05 Nov 20 04:00:51 localhost kernel: [128650.988365] task: f4bc2760 ti: f4b34000 task.ti: f4b34000 Nov 20 04:00:51 localhost kernel: [128650.988430] EIP: 0060:[<c17d507a>] EFLAGS: 00010246 CPU: 1 Nov 20 04:00:51 localhost kernel: [128650.988499] EIP is at l2cap_chan_send+0x9a/0xac0 Nov 20 04:00:51 localhost kernel: [128650.988556] EAX: 0000069b EBX: 00000000 ECX: 0000001c EDX: f4abad6c Nov 20 04:00:51 localhost kernel: [128650.988625] ESI: f5138c00 EDI: f5138e80 EBP: f4b35de4 ESP: f4b35d80 Nov 20 04:00:51 localhost kernel: [128650.988695]  DS: 007b ES: 007b FS: 00d8 GS: 0000 SS: 0068 Nov 20 04:00:51 localhost kernel: [128650.988757] CR0: 8005003b CR2: 00000010 CR3: 359ee000 CR4: 001007f0 Nov 20 04:00:51 localhost kernel: [128650.988826] DR0: 00000000 DR1: 00000000 DR2: 00000000 DR3: 00000000 Nov 20 04:00:51 localhost kernel: [128650.988893] DR6: ffff0ff0 DR7: 00000400 Nov 20 04:00:51 localhost kernel: [128650.988939] Stack: Nov 20 04:00:51 localhost kernel: [128650.988970]  00000246 f5138e84 f4b35d94 c184726d f5138e80 0000001c c18415c0 00000000 Nov 20 04:00:51 localhost kernel: [128650.989109]  f6ffe690 f4b35dbc c12722cd f5138e94 f5138c00 f4b34000 f4abad6c 22222222 Nov 20 04:00:51 localhost kernel: [128650.989245]  22222222 22222222 22222222 f5138e80 f5138c00 f5138e80 f513b000 f5138c00 Nov 20 04:00:51 localhost kernel: [128650.989383] Call Trace: Nov 20 04:00:51 localhost kernel: [128650.989442]  [<c184726d>] ? sub_preempt_count+0x3d/0x50 Nov 20 04:00:51 localhost kernel: [128650.989519]  [<c18415c0>] ? __mutex_lock_slowpath+0x250/0x370 Nov 20 04:00:51 localhost kernel: [128650.989599]  [<c12722cd>] ? __enqueue_entity+0x6d/0x80 Nov 20 04:00:51 localhost kernel: [128650.989680]  [<c17d9ee7>] l2cap_sock_sendmsg+0x67/0xd0 Nov 20 04:00:51 localhost kernel: [128650.989758]  [<c16b46cb>] sock_sendmsg+0x7b/0xb0 Nov 20 04:00:51 localhost kernel: [128650.989827]  [<c18429bf>] ? __schedule+0x3bf/0x7c0 Nov 20 04:00:51 localhost kernel: [128650.989897]  [<c1275b0a>] ? enqueue_task_fair+0xf0a/0x1610 Nov 20 04:00:51 localhost kernel: [128650.989988]  [<c16b472d>] kernel_sendmsg+0x2d/0x40 Nov 20 04:00:51 localhost kernel: [128650.990060]  [<c17e98ee>] bnep_session+0x3ee/0x800 Nov 20 04:00:51 localhost kernel: [128650.990132]  [<c126f340>] ? try_to_wake_up+0x190/0x240 Nov 20 04:00:51 localhost kernel: [128650.990210]  [<c126f450>] ? wake_up_state+0x20/0x20 Nov 20 04:00:51 localhost kernel: [128650.990280]  [<c1269f48>] ? complete+0x48/0x50 Nov 20 04:00:51 localhost kernel: [128650.990347]  [<c17e9500>] ? bnep_rx_control+0x3d0/0x3d0 Nov 20 04:00:51 localhost kernel: [128650.990418]  [<c1260784>] kthread+0x94/0xa0 Nov 20 04:00:51 localhost kernel: [128650.990484]  [<c1840000>] ? pskb_trim_rcsum.part.9+0xf/0x38 Nov 20 04:00:51 localhost kernel: [128650.990567]  [<c18490f7>] ret_from_kernel_thread+0x1b/0x28 Nov 20 04:00:51 localhost kernel: [128650.990640]  [<c12606f0>] ? kthread_create_on_node+0xc0/0xc0 Nov 20 04:00:51 localhost kernel: [128650.990708] Code: 8d 65 f4 5b 5e 5f 5d c3 90 8b 74 24 30 0f b7 46 2e 39 44 24 14 0f 87 5f 08 00 00 f6 05 2a 76 b9 c1 04 8b 5e 04 0f 85 0f 09 00 00 <8b> 43 10 8b 5c 24 14 8b 7c 24 30 8b 74 24 38 83 e$ Nov 20 04:00:51 localhost kernel: [128650.991373] EIP: [<c17d507a>] l2cap_chan_send+0x9a/0xac0 SS:ESP 0068:f4b35d80 Nov 20 04:00:51 localhost kernel: [128650.991470] CR2:<PHONE_NUMBER>000010 Nov 20 04:00:51 localhost kernel: [128651.249089] ---[ end trace 61e8318898b62114 ]---

That's the same kernel bug / panic that I see every time right before mine locks up.
Anyone care to do some kernel hacking / debugging?  ;-)

Any other investigation anyone wants to do on this?  If not, I think I'll consider the BT crashes fixed by #803 and continue investigating the more general issue under https://github.com/openaps/oref0/issues/809


Downloads with the main page

Due to an error updating the main repository this pull request was inadvertently closed and will need to be resubmitted.


[ballast-saved-state] add option for a minimum delay between "save" requests

For ViewModels that are updated very quickly, it may be desirable to skip some State emissions in order to not overload the disk and make sure that writes to the disk don't slow down the main ViewModel's processing.
To that end, all writes should be persisted asynchronously, and a configurable delay between emissions. By default, the delay will be 0. If there is a delay, then that much time will wait before sending the state to the adapter. If a new State is emitted before that delay completes, then the latest State value will be persisted. This needs to be more clever than a simple .mapLatest, as we don't want to reset the timer because emissions may continually prevent the adapter from ever being called.

bufferFlow parameter added to BallastSavedStateInterceptor in 2.0.0, which allows user-specified buffering/filtering on the States as they are sent to be saved


When redirected to login, upon successful login, redirect to the originally requested url that needed you to login

For normal users this works but not for staff that login via Google. This pr fixes that.


Model rendering does not work in browsers on iOS versions below 16.4

Hi,
First of all, I want to say that your model library is fantastic, and I really enjoy using it. However, I have encountered an issue where the model can not rendering on iPhones running iOS versions below 16.4.
Could you please look into this issue and let me know what might be causing it?
Thank you!

I'll see if I can find a device with an appropriate iOS version and I'll get back to you :)

I reproduced this issue on an iOS 16.1 simulator, as shown in the screenshots below (Figures 1 and 2 are the exception captures I added in the source code, and Figure 3 is the console error). The error occurs in the WebAssembly.compile(sorterWasmBytes) section. Could you take a look and see how this issue can be resolved?

I should have asked this earlier, but does this error occur only in specific browsers (like Safari) or all browsers?

I made an update that forces a non-SIMD version of the splat sort when using iOS < 17.0 here: https://github.com/mkkellogg/GaussianSplats3D/tree/general_updates. Would you be able to try that out?

Thank you very much for your response and the provided solution. I tested it on an iOS 16.1 simulator and it works fine under normal circumstances. However, when I set sharedMemoryForWorkers to false or when I render the model using an iOS 15.0 simulator, I encounter the following error: "WebAssembly.Module doesn't parse at byte 67shared memory is not enabled". I am not sure how to resolve this issue. Could you please take another look?

Ok I've made a few more tweaks targeting iOS versions below 16.0 to that same branch, want to give it another try?

Great, glad it worked!

Hello, after conducting my tests, there still seems to be an issue with iOS versions below 16.4. I think the following changes can be made to the code in the file /src/worker/SortWorker.js:
    // iOS makes choosing the right WebAssembly configuration tricky :(
    let iOSSemVer = isIOS() ? getIOSSemever() : null;
    if (!enableSIMDInSort && !useSharedMemory) {
        sourceWasm = SorterWasmNoSIMD;
        if (iOSSemVer && iOSSemVer.major < 16) {
            sourceWasm = SorterWasmNoSIMDNonShared;
        }
    } else if (!enableSIMDInSort) {
        sourceWasm = SorterWasmNoSIMD;
    } else if (!useSharedMemory) {
        if (iOSSemVer && iOSSemVer.major < 16) {
            sourceWasm = SorterWasmNonShared;
        }
    }

In the code, change the condition
if (iOSSemVer && iOSSemVer.major < 16) {
to
if (iOSSemVer && iOSSemVer.major <= 16 && iOSSemVer.minor < 4)
Moreover, according to the previous logic, the if inside the last else if should not be reachable.
Therefore, the modified code segment would be:
 // iOS makes choosing the right WebAssembly configuration tricky :(
 const iOSSemVer = isIOS() ? getIOSSemever() : null
 if (!enableSIMDInSort && !useSharedMemory) {
   sourceWasm = SorterWasmNoSIMD
   if (iOSSemVer && iOSSemVer.major <= 16 && iOSSemVer.minor < 4)
     sourceWasm = SorterWasmNoSIMDNonShared
 }
 else if (!enableSIMDInSort) {
   sourceWasm = SorterWasmNoSIMD
 }


Merge Dev

@mnordsletten any ideas as to why it fails?

It passed when I tried to build it again: https://jenkins.includeos.org/job/pull_build/325/console
There is definitely some issues with these larger merges.
Jenkins test please


Implement flake8-newspaper-style

Fixes #2436

What happens if two functions or classes call each other?

RecursionError :D
Yeah, I'm going to have to think about solving that.

What happens if two functions or classes call each other?

Should be fixed now

I'd love to get the ecosystem CI check running on this, I wonder why it's not appearing.

I'd love to get the ecosystem CI check running on this, I wonder why it's not appearing.

The diff is too large and exceeds the GitHub summary... https://github.com/charliermarsh/ruff/actions/runs/4457214954/jobs/7828206645?pr=3561

Thanks, really sorry for the double review, the comments helped me understand the code better so have one or two more questions!

No thanks for your comments, it made me understand a bit more about the projects structure and Rust in general

@charliermarsh @MichaReiser  Is there anything I can do to advance this PR?

@sasanjac - No, really sorry, this is just on me/us to define a formal policy around which plugins we include in Ruff or don't, and why -- which we're working on. Some plugins are obvious or easy to include, but in this case I'm moving slowly because it's a very opinionated rule that will trigger a lot of violations on any project using "ALL" (which is discouraged, but does have usages out there), and so I want to have a clear decision-making process in place before we merge this.
I'm happy to be responsible for rebasing etc. though I again am having some trouble pushing to this branch, maybe because of the backticks, I'm not sure.

I'm happy to be responsible for rebasing etc. though I again am having some trouble pushing to this branch, maybe because of the backticks, I'm not sure.

That's really strange. I can't edit any settings on this PR and afaik edits are allowed for maintainers by default. Did you check out the branch in my fork?

Thank you again for all your work here. I'm sorry to have let this one linger, but... I've made the decision to close it out. I don't think this plugin has the right characteristics to be part of Ruff right now -- it'd be one of the least-downloaded existing plugins we've implemented (~5k downloads per month), but with a high flag rate, and a complex interface + implementation. It would also conflict with ssort, which there's been interest in implementing, and has about 10x the downloads in the wild. None of these characteristics would be individually disqualifying, but as a whole, I don't think it makes sense to merge.
I apologize for not making this call sooner, especially before you'd started on the implementation -- I'm trying to get better at making those kinds of decisions earlier in the development process, but am still growing into my responsibilities as a maintainer.


Duplicate results

The following query should work:
{
    "message": {
        "query_graph": {
            "nodes": {
                "a": {
                    "id": "CHEBI:89564",
                    "category": "biolink:ChemicalSubstance"
                },
                "b": {
                    "category": "biolink:Disease"
                }
            },
            "edges": {
                "ab": {
                    "subject": "a",
                    "object": "b",
                    "predicate": "biolink:related_to"
                }
            }
        }
    }
}

This returns an answer as of #83, but it's actually duplicated 12 times... I'm repurposing this issue to address that. I suspect Strider and Plater are both traversing the Biolink hierarchy, and Strider is not properly deduplicating at the end.

Fixed by #89.


Migrating to AngularJS 2.0

Open for discussions -
As angularjs 2.0 near to final release, we're considering to use this as the primary javascript framework.
Initially, we'd like to stick with standard razor mvc. We love to hear suggestion/comments on using angularjs vs standard razor+jquery.
Cheers,

I think AngularJS 1.x is good for the next 1 year.

I mean if we are going to use angular we must start on version 2 once released. We'd like to eliminate the burden of migrating again from 1.x to 2.0 as version 2 is a complete rewrite.

Good call Marvs

Makes sense :+1:

I would love it.


Add RAII helper class to add/remove an element from an std::list

Description: Add an RAII helper class that adds an element to a list on construction and removes it on destruction.
Risk Level: low
Testing: unit tests
Fixes: #7020
Signed-off-by: Elisha Ziskind<EMAIL_ADDRESS>

/retest


Repeating section default value not working (when selecting from async select options)

I'm submitting a ...  (check one with "x")
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request

Current behavior
Default selected value of a select with asynchronous options is not working, when defined in repeating section. I have tried the same defined in normal (non-repeating section) fields and it works as expected.
Minimal reproduction of the problem with instructions
I used repeating section component provided in example (well, outdated one, where repeating section type is extending FieldType, not FieldArrayType). Select is populated through API call.

Angular version: 5.2.5
Browser: Chrome
Language: TypeScript 2.4.2
Node (for AoT issues): node --version =  v9.5.0

can you provide me a reproduction example using stackblitz, here is a starter example: https://stackblitz.com/edit/angular-ngxformly-repeat-3ewxjc

also I highly suggest to use the new FieldArrayType

Actually, after tinkering for a while with the stackblitz I have found, that my problem lies elsewhere. The problem is, when model has desired key set as null. Then the default value is ignored. Here is example of my problem: https://stackblitz.com/edit/angular-ngxformly-repeat-evkbvm (bitcoin and bitcoin3 should both get option 'A' pre-selected).

defaultValue is assigned only when the model value is undefined. so if you want to take account of defaultValue pass an undefined value instead.

I see. Thanks for your help!


[IA-3483] Fix for location not being set correctly in WorkspaceContainer

There is currently a bug where we're Unable to create cloud environments outside of US even when the workspace bucket is outside of US.
To reproduce:

Create/Load a workspace that has a bucket outside of the US (Montreal for example).
Attempt to create a cloud environment in this workspace. Click customize
Notice that the dropdown to choose a location is stuck on US.
Expectation: Location dropdown should be set to Montreal, with the option to change to US.

This happens because the isGoogleWorkspace prop doesn't get set properly when we first try to call loadBucketLocation. However, the googleProject is, so we can use that to determine that we are not an Azure workspace and can safely attempt to load the bucket location.

Nice, this resolves this comment on #3037.

Thanks @cahrens and @nawatts !
When this feature was first being developed, I suggested that we add an integration test similar to the analysis integration test. It would create a workspace, launch a cloud environment, and run a notebook, but just all in a non-US region. It would be straightforward to implement. Unfortunately at the time, tests were very flaky and we didn't want to add another flaky test. It was decided that manual testing would be sufficient.
But this was a bug that could have been caught easily with that integration test. Is now a good time to revisit that decision?

Yes, I agree that we are now more open to adding new tests, as long as you are willing to keep on top of it and help address issues that arise. Note that you can mock out specific AJAX calls if it makes your test easier to write/maintain.


Add IT-Schulungen.com as Training Partner

with this pull request, I would like to add IT-Schulungen.com as a training partner to the Prometheus website.
If you have any questions in this regard, please feel free to reach out to me!

Oh, another note: the DCO CI check is failing, you'll need to include Signed-off-by: Author Name<EMAIL_ADDRESS>in every commit message, see https://github.com/prometheus/docs/pull/2306/checks?check_run_id=12424657802

Sorry for the late reply!
Just added the signed-off-by

ufff.. sorry. should be fixed now.

Thanks :) :+1:


Improve responsive design of radio buttons and checkboxes

Fixes #73
Overwrote bootstrap styling to fix radio buttons & checkbox behaviour.
Added some vertical spacing as well.

Sweet, thanks @zawarudo! 🎉


"Show TLs" button in chat window overlapping with Chat Selector

Not sure if this should be an issue or how you look at this problem since Chat Selector is a third party extension for youtube. Still, it is a handy feature to use when there are alot of spam messages.
The button "Show TLs" is overlapping with Chat Selector for Youtube (making it impossible to interact with) like so:

It is nothing but a minor inconvenience. Love your work either way.

Same with ytcFilter.

Same with HyperChat

The Show TLs button is inside the chat's overlay. Supporting extensions which rely on monkey patching would require either monkey patching the Show TLs button, or moving the button elsewhere.

supporting them would require either injecting the Show TLs button once the chat has loaded, or moving the button elsewhere.

I completely understand. If moving or injecting the button would mess up other things in the app, then we can just close this issue.

It doesn't mess things up, the button just has to be moved. 🙂 Overall, youtube chat is poor in many ways (resource usage, spam filtering, etc.), so I think making this change to support extensions which modify the youtube chat is justified.


Doubt about behavior of dft

Great thing that you guys have added inverse ffts! However, I can't figure out how to exactly preserve the original coordinates of the data though. I was able to do that with ffts so I'm guessing there should be also a way to do this with xrft. Here's a MWE:
In [105]: import xarray as xr

In [106]: import xrft

In [107]: T = xr.tutorial.open_dataset("air_temperature").air.isel(time=[0, 1])

In [108]: T_f = xrft.dft(T, dim="lat", true_phase=True, true_amplitude=True)

In [109]: T2 = xrft.idft(T_f, dim="freq_lat", true_phase=True, true_amplitude=True,)

In [110]: T.lat
Out[110]:
<xarray.DataArray 'lat' (lat: 25)>
array([75. , 72.5, 70. , 67.5, 65. , 62.5, 60. , 57.5, 55. , 52.5, 50. , 47.5,
       45. , 42.5, 40. , 37.5, 35. , 32.5, 30. , 27.5, 25. , 22.5, 20. , 17.5,
       15. ], dtype=float32)
Coordinates:
  * lat      (lat) float32 75.0 72.5 70.0 67.5 65.0 ... 25.0 22.5 20.0 17.5 15.0
Attributes:
    standard_name:  latitude
    long_name:      Latitude
    units:          degrees_north
    axis:           Y

In [111]: T2.lat
Out[111]:
<xarray.DataArray 'lat' (lat: 25)>
array([-30. , -27.5, -25. , -22.5, -20. , -17.5, -15. , -12.5, -10. ,  -7.5,
        -5. ,  -2.5,   0. ,   2.5,   5. ,   7.5,  10. ,  12.5,  15. ,  17.5,
        20. ,  22.5,  25. ,  27.5,  30. ])
Coordinates:
  * lat      (lat) float64 -30.0 -27.5 -25.0 -22.5 -20.0 ... 22.5 25.0 27.5 30.0
Attributes:
    spacing:  2.4999999999999964

Using the xrft.fft option also doesn't produce what I expected:
In [112]: T_f = xrft.fft(T, dim="lat",)

In [113]: T2 = xrft.ifft(T_f, dim="freq_lat")

In [114]: T.lat
Out[114]:
<xarray.DataArray 'lat' (lat: 25)>
array([75. , 72.5, 70. , 67.5, 65. , 62.5, 60. , 57.5, 55. , 52.5, 50. , 47.5,
       45. , 42.5, 40. , 37.5, 35. , 32.5, 30. , 27.5, 25. , 22.5, 20. , 17.5,
       15. ], dtype=float32)
Coordinates:
  * lat      (lat) float32 75.0 72.5 70.0 67.5 65.0 ... 25.0 22.5 20.0 17.5 15.0
Attributes:
    standard_name:  latitude
    long_name:      Latitude
    units:          degrees_north
    axis:           Y

In [115]: T2.lat
Out[115]:
<xarray.DataArray 'lat' (lat: 25)>
array([-30. , -27.5, -25. , -22.5, -20. , -17.5, -15. , -12.5, -10. ,  -7.5,
        -5. ,  -2.5,   0. ,   2.5,   5. ,   7.5,  10. ,  12.5,  15. ,  17.5,
        20. ,  22.5,  25. ,  27.5,  30. ])
Coordinates:
  * lat      (lat) float64 -30.0 -27.5 -25.0 -22.5 -20.0 ... 22.5 25.0 27.5 30.0
Attributes:
    spacing:  2.4999999999999964

I feel like I'm missing something obvious but can't quite figure it out...
How can I preserve coords so that I get the same result going to fourier space and back?

fft drops the information regarding the phase so ifft will only give back the original data but not coordinates as it is lost in that process. Try using dft and idft if you want to get the coords going back and forth.

Some examples are give in the documentation page: https://xrft.readthedocs.io/en/latest/DFT-iDFT_example.html

Hi, @roxyboy, thanks for the quick answer. However, I also can't figure out how to that with dft. The first example I gave used dft and the phase information still seemed to get lost. I read through the example in the docs but it still seems that that example loses the phase information, no?

Ah, that's because you're data isn't centered. Try setting the lag flag in idft to 45? I think since your coordinates is starting from 15.

Hi @tomchor, @roxyboy ,
When T_f(...,true_phase=True) is evaluated, the phase is preserved but the coordinates of T are lost in the transformation.
T_f is in the Fourier domain and is not associated to any direct (lat) coordinates. Thus, when idft() is called on T_f, idft has no idea on which range of coordinates you need the result. The default is on centered coordinates  with the same range of your inital coordinates ([-30,30] in your example). If you want your data back in the good range of coordiantes, you need to shift the centered standard output coordinates. As your initial coordinates are around 45 degree, you have to call idft with the lag argument:
T2 = xrft.idft(T_f, dim="freq_lat", true_phase=True, true_amplitude=True, lag=45)
However, looking to you example, I see that the returned result is the same as the orignal T but reversed. This is because signal is on decreasing "lat" axis. This is a xrft bug !
For now, as a workaround, use:
T = T.sortby('lat')
just after loading T to get the expected behaviour.

Thanks, @lanougue, that's super helpful!


Fix "mentionBg" and add "dndIndicator"

The "mentionBg" used to be "mentionBj" (Mattermost bug): https://mattermost.atlassian.net/browse/MM-7399

Done!

All is good, thank you very much!

Wow, I just saw that I lost write access to this repository… I'll try to gain them back and merge ASAP. Sorry for the delay.


Refactor coroutines

Small refactor that prepares background for future work (especially #532 )

Codecov Report

Merging #671 into master will decrease coverage by 0.13%.
The diff coverage is 36.11%.

@@             Coverage Diff              @@
##             master     #671      +/-   ##
============================================
- Coverage     77.52%   77.39%   -0.14%
  Complexity      604      604
============================================
  Files            99       99
  Lines          2541     2544       +3
  Branches        338      344       +6
============================================
- Hits           1970     1969       -1
- Misses          357      358       +1
- Partials        214      217       +3

@jan-gogo
Thanks for comments, I've implemented changes.


Reinstate the proper sandbox landing page URL

At the moment, we're using a test url behind auth. We need to remember to reinstate the page at the proper url, for when we're ready to launch.

The url should be changed to sandbox.aztec.network

The docs link on the page should point to the 'netlify dev' version.


Use default vCPU and MemorySize when such annotations are not provided

Use default vCPU and MemorySize when such annotations are not provided
Re-vendor hcsshim

cc @jterry75, @kevpar , @katiewasnothere

This PR is not needed. Silly me


Neato App on IOS fails when plugin is used

I have a D3. With the plugin installed, everything works fine. But my ability to use the app from my iPhone now fails. Prior to the homebridge-neato plugin, I could start the app and connect to the D3 from anywhere (home or out and about using 3g/4g/LTE). I also got notifications on my phone when cleaning completed and an updated coverage map. With the homebridge plugin enabled, I can control it from the Home app in IOS, but not from the Neato app. Further, while it cleans, etc I get no notifications when cleaning is done or maps (which are tied to the IOS app). Is there a login limitation? I know with Arlo's you can only connect from the app or the plugin, not both at the same time. Is there something similar with this plugin? Are others experiencing the same issue?

I've noticed this too, and it happens even when I don't use Homekit to turn on my D7. The wifi indicator will flash for quite a while, until it then goes solid and re-establishes its connection to the Neato app. The Vacuum remains responsive in Homekit though.

I'm not sure that this is realted to the plugin. I use the plugin and the app at the same time for over a year now. First with a Botvac Connected and now a Botvac Connected D5.
Does the problem dissolves as soon as you stop homebridge (or remove the plugin from homebridge).

I can't confirm any issues using the plugin and the Neato App with my D7. Controlling with Plugin/HomeKit and App works together here. Including all notifications.
I use the plugin from the first day on and also the Neato App on iOS.
Firmware on the D7 always the latest. Now @4.2.3-153
The only thing I noticed, but this has nothing to do with the plugin, that the Neato from time to time loses Wifi connection but after a few seconds all is back to normal.
@naofireblade Hope you are already received also the firmware update for your D5 with no-go lines.
I noticed that the problem starting the Neato from the app always "without" no-go lines, is probably an issue or not supported by Neato and not by your plugin. If I start the Neato with IFTTT the same happens. Always starts without no-go lines. :(
But more this ongoing here. ;) https://github.com/naofireblade/homebridge-neato/issues/13

It appears to work fine now that I've reset the wifi settings on my D7.

Sorry to piggy-back on this issue, but can someone tell me how to actually add my D7 to the home app? I've got the plugin installed and the log said that it found my D7 Robot so I think everything is good there.
I'm just unsure if I need to wait for Homekit to sync with homebridge or if there is sommething additional I need to do to add my D7 robot to the Home app. Thanks!

Restart your Homebridge.
On Sun, Oct 28, 2018 at 2:05 PM Jordan Godwin<EMAIL_ADDRESS>wrote:

Sorry to piggy-back on this issue, but can someone tell me how to actually
add my D7 to the home app? I've got the plugin installed and the log said
that it found my D7 Robot so I think everything is good there.
I'm just unsure if I need to wait for Homekit to sync with homebridge or
if there is sommething additional I need to do to add my D7 robot to the
Home app. Thanks!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/naofireblade/homebridge-neato/issues/17#issuecomment-433741338,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aeyz4OmznIlT7WJrN1VHPtcSjzsbGnm4ks5uphwKgaJpZM4VNe_u
.

Yes, please post your config without any personal info. This is a very simple plugin to use, mine looks like this:
        {
            "platform": "NeatoVacuumRobot",
            "email"<EMAIL_ADDRESS>            "password": "password"
        }

You'll need to ensure your D7 is registered with your Neato account first.

I've noticed this too, and it happens even when I don't use Homekit to turn on my D7. The wifi indicator will flash for quite a while, until it then goes solid and re-establishes its connection to the Neato app. The Vacuum remains responsive in Homekit though.

I can confirm this behavior now. While cleaning the app can't connect to the robot. About 60s after cleaning, the app can establish a connection again.
I have 4 systems that connect to the robot and so I might reach the limit: Homekit, Home Assistant, Alexa, Neat App.

The newest release v0.5.0 has an option to use the refresh mode "auto" (updates the robot state only while cleaning) or to disable background refreshes completly with "refresh": 0.


Add support to VMRC Console  when using VMware hypervisor

Author Name: Carlos  Cesario (Carlos  Cesario)
Original Redmine Issue: 4785, https://dev.opennebula.org/issues/4785
Original Date: 2016-09-15

Hello all,
Would be interesting and secure provide virtual machine console access using VMRC API (When using VMware).
With it will not needed change firewall rules, run vnc service proxy, will needed only needed vrmc plugin in browser.
There is one good example how to use API - Perl
http://www.virtuallyghetto.com/2013/02/how-to-access-vsphere-remote-console.html
Ruby code
https://github.com/vmware/rvc/blob/master/lib/rvc/modules/vmrc.rb
-Carlos

Original Redmine Comment
Author Name: Ruben S. Montero (@rsmontero)
Original Date: 2016-09-16T11:14:00Z

Thanks, very interesting indeed!

Original Redmine Comment
Author Name: OpenNebula Systems Support Team (OpenNebula Systems Support Team)
Original Date: 2017-05-04T17:25:30Z

Ideally read-only user run VMRC.
https://code.vmware.com/web/sdk/55/vmrc
http://www.virtuallyghetto.com/2014/10/standalone-vmrc-vm-remote-console-re-introduced-in-vsphere-5-5-update-2b.html
https://www.vmware.com/support/developer/vmrc/index.html
New HTML console: https://www.vmware.com/support/developer/html-console/index.html

duplicated by #2312


Add X and SSRN social networks

Re-opening since #91 and #101 were closed.

Hello, we can still comment under the issues even if they are closed.  I am closing this one to keep the discussion in the same place.


Extricate Pux as much as possible

Anything in scope as Projector.Html.Runtime will be used.
A valid runtime is anything that exports types Html ev, Attribute ev, and the functions described below.
! @charleso @sphvn

This means you can't use two separate runtimes at once, though I'm sure you could quickly figure out how to hack in a new import string if you really needed to.

👍
I wouldn't mind chatting about how much work it would be (for me) to just make it a typeclass library at some point in the future.

OK - in case we forget to have that chat: you would need to thread two extra type parameters around, one representing Html and one representing Attribute. This would mean replicating the current type parameter-tracking code in Purescript.hs to instead figure out if any types in a signature use Html. I don't think it buys a whole lot, but it's tractable.


[#881] fix: Ensure LocalStorageMeta disk size is correctly updated when events are processed

What changes were proposed in this pull request?
Ensure all events are marked as understorage, this will result to the LocalStorageMeta being updated when events are processed.
Why are the changes needed?
Currently LocalStorageMeta is only update with metrics from the first event in a given shuffleId and partitionId, the first event updates metrics because there is no entry in partitionsOfStorage and the event get marked as underStorage, however, for future events in the same shuffleId and partitionId selectStorage returns the storage and does not mark the event as underStorage so when updateWriteMetrics is called, event.getUnderStorage() returns null and storage.updateWriteMetrics(metrics); is skipped.
As metrics are not updated correctly, LocalStorage.canWrite will not return the correct result.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Added unit test which covers multi events for the same shuffleId and partitionId

Codecov Report

Merging #902 (d262b70) into master (3e58805) will increase coverage by 1.84%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##             master     #902      +/-   ##
============================================
+ Coverage     55.24%   57.09%   +1.84%
- Complexity     2201     2202       +1
============================================
  Files           333      313      -20
  Lines         16449    14091    -2358
  Branches       1307     1308       +1
============================================
- Hits           9087     8045    -1042
+ Misses         6851     5605    -1246
+ Partials        511      441      -70

Impacted Files
Coverage Δ

...he/uniffle/server/storage/LocalStorageManager.java
86.85% <66.66%> (+0.72%)
:arrow_up:

... and 21 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@zuston Could you review this pr? Because you are more familiar with this part of code.

@zuston Could you review this pr? Because you are more familiar with this part of code.

Oh, yes, let me take a look in the next few days

I noticed this whilst benchmarking Uniffle. It makes the settings related to disk capacity almost void

@awdavidson Could you tell me your company? Do you use Uniffle in your production environment?

@awdavidson There is conflict in the branch 0.7. Could you raise a pr for branch 0.7?


String Enum values compile to the key rather than source value

Describe the bug
In typescript, we can represent string enums like so:
export enum EnvironmentName {
  DEV = 'dev',
  QA = 'qa',
  FOO = 'bar'
}

However, in all languages besides js, jsii compiles these so that the key name is used. For instance, python output looks like:
<EMAIL_ADDRESS>class EnvironmentName(enum.Enum):
    DEV = "DEV"
    QA = "QA"
    FOO = "FOO"

When using the enum as a type, this doesn't matter, as the literal value is unimportant. But when outputting a string, Python developers are now outputting FOO whereas TS/JS users of the same construct output bar. This breaks interoperability.
Expected Behavior
The string representation of an enum value in all languages should match the source code. As noted in the jsii design tenants:

jsii applications behave identically regardless of the language they are written in. It favors correctness over performance.
Unsupported idioms will cause a compile-time error to be emitted.
When prohibiting an idiom, jsii strives to provide an error message that gives the user insight into why the pattern cannot be supported.

Applications should behave identically, and if that is not possible, there should be a compiler error for string enums where the key and value do not match.
Current Behavior
An application which uses runtime validation of enum values will fail in languages other than JS. For instance:
export const validateEnv = (env: EnvironmentName) => {
  if (Object.values(EnvironmentName).includes(env)) {
    throw new TypeError(`Value "${env}" is not a valid environment name.`);
  }
}

I believe this is because the JS assembly is used when calling Object.values(), and the value passed in doesn't match JS values due to the serialization differences.
Reproduction Steps

Create a construct library which uses a string enum's values inside functional code.
Publish package to NPM and another supported target.
Import the package in the target language and call the function.
Observe output.

Possible Solution
Rework how JSII models string enum values. Not sure how complicated this is, but it seems important for correctness.
Additional Information/Context
No response
SDK version used
jsii=5.2.8; jsii-pacmak=1.89.0
Environment details (OS name and version, etc.)
MacOS 11.7.10

Not entirely sure about the implications here. We might need to enforce that always enum keys == value


Not automatically scroll when the new line in the list block type

Hi.
When a new line in ordered-list-item andunordered-list-item
The input area does not scroll automatically. It is scrolled When you start typing.
https://github.com/facebook/draft-js/blob/master/src/component/contents/DraftEditorBlock.react.js#L110
Seems blockNode.offsetTop is not able to get properly.
(Perhaps offsetParent is I think that's because it becomes the parent element of the ul or ol )
Is there a solution?

Can you provide a video capture of the bug?

I've noticed this as well. It only updates to the proper scroll position when I enter content into the list item. Here's a gif:

Faced with the same issue. Are there any updates regarding bug?

@hellendag Is this issue on track?

Anything new on this??  I'm having the same issue

pageage.json
"resolutions": { "draft-js": "0.11.7" }


Video games own/made by Microsoft?

Should video games made/owned by Microsoft be added?  There were a lot of first party games on Windows Phone that Microsoft made and killed.

I may add few videogames, you may write few of them. However, regarding old PC and console games (the ones not depending on remote servers) I think they shounldn't be listed.


navigator.pop() followed by navigator.push(screen) does not dispose the ScreenModel from the popped screen

Describe the bug
I have been trying to figure out why my screenModel retains its state and data after the screen is popped and the screenModel should be disposed, I have my screen defined as this:
class ScreenB(
	private val data: BDataClass
) : Screen {
	@Composable
	override fun Content() {
		val navigator = LocalNavigator.currentOrThrow
		val viewModel = getScreenModel<ScreenBViewModel>(
			parameters = {
				ParametersHolder().apply {
					add(data)
				}
			}
		)
		ScreenBContent(
			viewModel = viewModel,
			onNavigateBack = {
				navigator.pop()
			},
			onNavigateScreenC = { id ->
				navigator.pop()
				navigator.push(ScreenC(id))
			}
		)
}

//ScreenModel
class ScreenBViewModel(
	private val data: BDataClass
) : ScreenModel {
	init {
		Log.v("testing", "Created ScreenBViewModel")
	}

	override fun onDispose() {
		super.onDispose()
		Log.v("testing", "Disposed ScreenBViewModel")
	}
}

//Koin Module:
val viewModelModule = module {
	factory { params ->
		ScreenBViewModel(params.get())
	}
}

Then if I navigate from ScreenA to ScreenB, I see the "Created ScreenBViewModel" Log
Then if I navigate back (either with system back button or my onNavigateBack callback I do see the "Disposed ScreenBViewModel" (this is as expected)
however if, while on ScreenB, I call my onNavigateScreenC callback I do not see the "Disposed ScreenBViewModel" and then if I navigate back to ScreenA and then forward into ScreenB again I do not see the "Created ScreenBViewModel" Log meaning that this is still using the previous instance that should have been disposed.
in Testing I have found that if I replace the
navigator.pop()
navigator.push(ScreenC(id))

with
navigator.replace(ScreenC(id))

then the ScreenModel is disposed correctly which is fine for this example but in other places I may need to pop 2 or 3 times before pushing a new screen, and I need to screenModel(s) to be disposed correctly in those instances
To Reproduce
Steps to reproduce the behavior:

Set up App with ScreenA, ScreenB(with ViewModel) and ScreenC
from ScreenA push ScreenB (with some data for the ScreenModel)
from ScreenB pop and then push ScreenC
from ScreenC pop (you should now be back on ScreenA)
from ScreenA push ScreenB (with different data than on step 1)
See that ScreenB's ScreenModel still has the data from step 1

Expected behavior
I would expect that is I pop a screen, then it's associated ScreenModel should be disposed. Even if that pop is immediately followed by a push.
Voyager module and version:

voyager-navigator:1.0.0
voyager-screenmodel:1.0.0
voyager-tab-navigator:1.0.0
voyager-transitions:1.0.0
voyager-koin:1.0.0

Koin module and version:

koin-bom:3.5.3
koin-core
koin-compose
koin-android

Snippet or Sample project to help reproduce
See code snippet above

I will have to take a look on this. We have a popUtil function, we could have a replaceUntil as well, this would combine multiples pops and the push at the same time.

The way that Voyager disposes the Screens and ViewModel/ScreenModel is the next tick of the composition when the event is a dispose event, when you do pop+ push at the same time, the last event will be a push, so voyager can't dispose the screen that you have done pop. This is the nature of the library being built fully with Compose.
We have introduce a new way of handling disposing for Transition API that is still experimental, it does not relay on the lastEvent, I have not tested, but maybe it could work with pop + push, can you have a look?
https://github.com/adrielcafe/voyager/pull/436
The docs are here: https://voyager.adriel.cafe/transitions-api/
Is the disposeScreenAfterTransitionEnd = true
We are considering replace the StepDisposable with this approach if is stable enough.

I tried upgrading to 1.1.0-beta02 and using disposeScreenAfterTransitionEnd = true as you suggested but I am now getting the Screen was used multiple times crash mentioned in the docs when I use either pop + push or just replace when navigating from ScreenB to ScreenC
I have tried setting a ScreenKey on the screen using the class name + data ID but I still get the crash...

Try tô use uniqueScreenKey instead

ah I didn't realise uniqueScreenKey was an actual val...
So that stops the crash, but now it causes ScreenC's ScreenModel to be re-created in an infinite loop for some reason, I'm using replace to go from ScreenB to ScreenC

Can someone write the steps to solve this

The way that Voyager disposes the Screens and ViewModel/ScreenModel is the next tick of the composition when the event is a dispose event, when you do pop+ push at the same time, the last event will be a push, so voyager can't dispose the screen that you have done pop. This is the nature of the library being built fully with Compose.
We have introduce a new way of handling disposing for Transition API that is still experimental, it does not relay on the lastEvent, I have not tested, but maybe it could work with pop + push, can you have a look? #436
The docs are here: https://voyager.adriel.cafe/transitions-api/ Is the disposeScreenAfterTransitionEnd = true
We are considering replace the StepDisposable with this approach if is stable enough.

This really helped me, thank you. If I add a delay in between the pop and push calls, it will work:
val coroutineScope = rememberCoroutineScope()
...
Button(
    onClick = {
        navigator.pop()
        // don't push immediately, wait for Compose tick to happen, and for the screen model to be disposed
        coroutineScope.launch {
            // the tick can be just 1 millisecond, it doesn't have to be long
            delay(1)
            navigator push MyAwesomeNextScreen()
        }
    }
)
...

This works, but because of the delay, even thought it's tiny (just 1 millisecond), the screen flickers, at least in my use case, it doesn't look very nice.
@DevSrSouza Is there an API we could use to dispose a screen model manually? Without needing a delay? I have tried using the link you provided, but I don't use transitions on Desktop, I present my screens side-by-side instead. I also tried setting
disposeBehavior = NavigatorDisposeBehavior(disposeSteps = false), in the Navigator, but that didn't help.

Hello! A little late to the discussion but I've found a solution that acts as replaceAll:
import cafe.adriel.voyager.core.stack.Stack

fun <Item> Stack<Item>.replaceUntil(item: Item, predicate: (Item) -> Boolean) {
    val items = items.takeUntilInclusive(predicate) + item
    this.replaceAll(items)
}

fun <T> List<T>.takeUntilInclusive(predicate: (T) -> Boolean): List<T> {
    val result = mutableListOf<T>()
    for (item in this) {
        result.add(item)
        if (predicate(item)) break
    }
    return result
}

Due to how voyager works internally the items inserted into the replaceAll call that already exist won't be recreated, all new ones will be created and all items that are no longer there will get disposed!

@hristogochev Sadly, this doesn't dispose of the screen view model correctly. I tried it, and it shows me the old data. The only thing that works is adding a delay.


ipcache: keep upserted prefixes from being deleted by InjectLabels

The "new" ipcache API (InjectLabels()) will delete a prefix if there are no metadata entries left. However, this behavior is not desirable when a user of the old APIs has taken control of this prefix. In order to prevent this, the field "createdFromMetadata" is set on all "new-style" prefixes to indicate they are safe to delete.
However, in the case where a prefix is first injected via the metadata layer, then transitions to the old APIs, it should not be deleted. In other words:

UpsertMetadata(A)
UpsertGeneratedIdentities(A)
RemoveMetadata(A)

should not delete A from the bpf ipcache, just the metadata layer.
This change prevents that by ensuring the bit is always set.

Split out from #28673 on @christarazi's request.

Requesting @gandro's review, as he approved the original commit.

/test


Bump framework version number in Dockerfile examples

The Dockerfile examples uses .NET 5.0 image when viewing docs for .NET 6.0.
This PR adds the correct image versions for version 6.0.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: bvda sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the very valuable update. Be sure to check out your name as a contributor about the 3rd of next month on the What's new page in the Community contributors section at the bottom.


Sometimes end call is not reported to other phone

Using

IOS & Android
react-native-voip-push
react-native-callkeep

Sometimes I will make a call, the call rings on other phone. If I end the call on initiator phone then sometimes the other phone keeps ringing.
User A calls User B
User A ends call before answer
User B phone still rings
OR
The opposite does not work sometimes
User A calls User B
User B rejects the call
User A is still calling
(Both work sometimes, and sometimes don't work)

I am aiming for 100% reliability.
My code is below:
https://pastebin.com/6Zv8Jg1e

@stephanoparaskeva  do you have any specific day hours where it happens? If so - please report

@stephanoparaskeva  do you have any specific day hours where it happens? If so - please report

@stephanoparaskeva do you have any specific day hours where it happens? If so - please report

This happens almost any time per day I believe, sometimes things are just delayed

@stephanoparaskeva do you have any specific day hours where it happens? If so - please report

This happens almost any time per day I believe, sometimes things are just delayed

Closed due to inactivity. Please create a new issue if needed.


Rename commander.cpp to Commander.cpp and update CMakeLists.txt.

Hi,
This PR replaces PR #9860 for the rationale discussed in that PR and aims to unify the naming convention between commander.cpp and Commander.hpp.  This PR reinforces convention within that module such that files containing classes reflect the name (with respective capitalization) of the class contained within that file.
Let me know if you have any questions on this PR.
-Mark

@hamishwillee , should I close out this PR?  Thanks for your advice!
-Mark

@mcsauder I am OK with this, but my field is docs, not code. @dagar - OK to merge?

Conflicts at the moment, but I'm fine with the rename.

Thanks @dagar , merge conflicts resolved and pushed up!


[FEATURE REQUEST] Access binary data of file without decoding

std::vector<unsigned char> pixels (xres*yres*channels);
in->read_image (TypeDesc::UINT8, &pixels[0]);
in->close ();
The basic example for reading or writing a file using oiio offers a read or write method that returns the decoded binary image. For certain image processing, it may be necessary to access the encoded binary data inside the jpeg or png.
A read_binary() and write_binary() method can be offered that will extract just the binary data in a buffer and write it back. Of course, the size of the binary buffer using read_binary() and write_binary() must be exactly the same.
It will be easy to do this within oiio as the library already is aware of the header and data sections due to the read and write functionality.
Alternatively, a data_offset() method can return the number of bytes in the header. And the user can modify the binary data on their own. Combined with the ability to write jpeg to memory, this will be quite useful.

I'm embarrassed that there was no reply to this right away.
I agree in principle -- there is a case to be made that ImageInput/ImageOutput should have ways to read tiles/scanlines/stripes of raw compressed data. I can think of at least two important use cases:

Times when you want to copy a file (to the same file format type) with only changes in metadata. This shouldn't require a full decompress/recompress cycle.
Future cases where we want to do decompression on GPUs.

If somebody wants to take this on, the first thing this would need is a detailed proposal for how the ImageInput and ImageOutput APIs ought to be modified to accommodate this.


TypeError: 'coo_matrix' object is not subscriptable

am encountering the same issue that was noticed by another user.
my input is two matrices, AD and DP
mquad -m cellSNP.tag.AD.mtx,cellSNP.tag.DP.mtx -o ./mquad/ --minDP 5 -p 20
output
[MQuad] Loading matrix files ...
16569 variants detected...
Variant names detected...
CPUs used: 20, batch size: 128
Fitting in sparse mode...
Initializing fit(mode: deltaBIC) on 16569 variants...
Traceback (most recent call last):
File "/Users/nikolay/anaconda3/envs/python-3.11.3/bin/mquad", line 8, in
sys.exit(main())
^^^^^^
File "/Users/nikolay/anaconda3/envs/Python-3.11.3/lib/python3.11/site-packages/mquad/mquad_CLI.py", line 132, in main
df = mdphd.fit_deltaBIC(
^^^^^^^^^^^^^^^^^^^
File "/Users/nikolay/anaconda3/envs/Python-3.11.3/lib/python3.11/site-packages/mquad/mquad_batch_mixbin.py", line 142, in fit_deltaBIC
results = pool.starmap_async(fit_batch, self._batchify(batch_size, x, y, valid_row_sizes)).get()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/nikolay/anaconda3/envs/Python-3.11.3/lib/python3.11/multiprocessing/pool.py", line 382, in starmap_async
return self._map_async(func, iterable, starmapstar, chunksize,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/nikolay/anaconda3/envs/Python-3.11.3/lib/python3.11/multiprocessing/pool.py", line 478, in _map_async
iterable = list(iterable)
^^^^^^^^^^^^^^
File "/Users/nikolay/anaconda3/envs/Python-3.11.3/lib/python3.11/site-packages/mquad/mquad_batch_mixbin.py", line 80, in _batchify
_ad = np.squeeze(np.asarray(self.ad[_x, _y]))
TypeError: 'coo_matrix' object is not subscriptable

Hi @drSnuce,
Can I please have a brief look at the dimensions of the input files? For example something like
head cellSNP.tag.DP.mtx
will really help me debug the issue.
Aaron

Aaron,
I was able to use mquad by using a folder argument instead of passing just two matrices (-c instead of -m). However, it is worth to troubleshoot the issue, since I am second user encountering the problem. Here is the output of head cellSNP.tag.DP.mtx:
%%MatrixMarket matrix coordinate integer general
%
16569	10556	36003472
1	189	1
1	855	1
1	994	2
1	1159	1
1	1278	1
1	1414	1
1	1605	1
Output of head cellSNP.tag.AD.mtx
%%MatrixMarket matrix coordinate integer general
%
16569	10556	304384
11	3595	1
25	10273	1
32	1848	1
32	9473	1
41	2988	1
42	10295	1
53	3735	1.
Both matrices are produced by cellsnp-lite.
Nikolay.

Hi Nikolay,
Thanks for reporting the issue and sorry for the late reply. I have made a new release (0.1.7) which should fix this issue. You can update via pip or cloning the newest version of this repo. Let me know if it still persists.
Aaron


Wording: Update "Charge" to "Virtual Card Charge"

Surfaced in https://opencollective.slack.com/archives/C05V96RERKL/p1704478810607729
OCF Host admins did not realize that "Charge" (the expense type) was meant for virtual card charges. To provide more clarity, we've decided to rename it globally. While doing so, we should ensure that a longer string does not break the UI.

+1 Wanted to rename this for a long time.


Error when deploying: submodel's internal model has errors

When I deploy this app and try to enter data into it, it gives me an error at the end when I try to run it.

Closing as this seems to work now with the fix from #40.


fix(apigw): update service subDomain undefined

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Allow secure config

Add the ability to allow secure connections to temporal
quarkus.temporal.service.secure=true

@selfeky what is the state of your PR?

@ivelinyanev-ship-cars its on production!

@selfeky we have an official Quarkiverse extension going and are looking for adopters to help us improve the product:
https://github.com/quarkiverse/quarkus-temporal


Add infracost integration

Add infracost integration instructions.
Uses job executor service.
Signed-off-by: agardnerit<EMAIL_ADDRESS>

@agardnerIT one tiny suggestion: add --no-color to the Infracost CLI commands that are run should remove the funky color characters in this screenshot


⚠ Bofrost.nl has degraded performance

In b3ff194, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 3514 ms

Resolved: Bofrost.nl performance has improved in 01ffb95 after 6 minutes.


Add interrupt detection and Ctrl-C handling/filtering

See example - but basically allows to interrupt the sleep command for instance (will be used in grol to interrupt long running scripts)
$ go run ./example
11:01:34.002 r1 [INF] Loaded 9 history entries from .history
Terminal is open
is valid true
use exit or ^D or 3 ^C to exit
Use 'prompt <new prompt>' to change the prompt
Try 'after duration text...' to see text showing in the middle of edits after said duration
Try <tab> for auto completion
Terminal demo> after 5s a message
11:01:50.502 r1 [INF] Read line got: "after 5s a message"
11:01:50.502 r1 [INF] Will show "a message" after 5s
Terminal demo> sleep 10s
11:01:53.078 r1 [INF] Read line got: "sleep 10s"
11:01:53.078 r1 [INF] Sleeping for 10s (^C to interrupt)
a message
11:01:56.592 r3 [INF] Ctrl-C found in input
11:01:56.592 r3 [INF] Context done
11:01:56.592 r1 [INF] Sleep interrupted: context canceled
11:01:56.592 r1 [INF] Interrupted, resetting, use exit or Ctrl-d. to exit.
Terminal demo>

funny all the eyes on this and nobody noticed (and I didn't... until now) that it's missing signaling/blocking in the regular reader


cmd/go: allow versioned 'go list' commands

https://github.com/golang/go/issues/40276 allows using go install path@version to install a package at a specific version, ignoring the current module.
https://github.com/golang/go/issues/42088 is an accepted proposal to allow go run path@version with similar semantics - run a main package at a specific version, ignoring the current module.
While I agree that these semantics are most often useful for go install and go run, I argue that they would also be useful for other cmd/go commands. In this proposal, I want to make a case for go list path@version.
The main use case I want to bring forth is to list the packages included in a specific version of a module. Right now, I can only accomplish this via:
go mod init tmp
go get -d ${module}@${version}
go list ${module}/...
rm go.mod go.sum

It could instead be just go list ${module}/...@${version}.
For reference, I'm using this to generate static HTML pages to serve go get and browser requests for vanity import paths; see https://github.com/mvdan/mvdan.cc/blob/572593964513b20649c474cc9a6559405ec9cc29/gen-go-modules.bash#L49-L53.
I think this feature would also make go list a bit more consistent, since go list -m path@version currently works just fine.
/cc @dominikh @bcmills @jayconrod @myitcv @eliasnaur

I can see some use for this feature, it's consistent with the recent change to go install and the approved go run proposal, and I don't see anything fundamentally problematic about it.
I'm curious to see what @jayconrod and @matloob think, but at this point it seems ok to me.

Seems fine to me. Though I think we should talk about allowing @version on all commands that take packages (where it makes sense; not go generate).
The main difference is that go install pkg@version and go run pkg@version both require main packages, and they require those packages to come from the same module at the same version. That would be overly restrictive for go list. So how should the arguments be interpreted?
As a starting point, I think it should do what go get does when run outside of a module: start with an empty build list, add those specific versions, run MVS, and verify we didn't upgrade beyond those versions.

Though I think we should talk about allowing @version on all commands that take packages (where it makes sense; not go generate).

I think that makes sense too, and @dominikh seemed to also agree on Slack. I thought we could start with just go list, as it seems like the next clear win to me, after go install and go run, but I'm also fine with doing many at once.

As a starting point, I think it should do what go get does when run outside of a module: start with an empty build list, add those specific versions, run MVS, and verify we didn't upgrade beyond those versions.

Sounds good to me. From the caller's point of view, if they want to do MVS resolution separately for a number of arguments, they could just use separate go list calls. The module download cache should keep that reasonably fast.

Somewhat related to the version suffix support:
As some tools age and go through many releases, the output length of go list -m -versions is getting longer.
Today, I wished I could use the version query to limit the number of versions.
For example, if I query go list -m -versions<EMAIL_ADDRESS>I hope to get only the list of available versions that satisfy the version query.

The main difference is that go install pkg@version and go run pkg@version both require main packages, and they require those packages to come from the same module at the same version. That would be overly restrictive for go list. So how should the arguments be interpreted?

If the required semantics of the go list pkg@version differ from that of install and run, then I'd hope go list doesn't share the @version syntax for consistency of UX.

If the required semantics of the go list pkg@version differ from that of install and run, then I'd hope go list doesn't share the @version syntax for consistency of UX.

I think it's okay for go list to share the @version syntax, as long as the arguments would be interpreted the same way for all commands.
go list would allow non-main packages, so I don't think go list pkg@version does anything unexpected; that's an error with go install.
In @hyangah's example of go list -m -versions<EMAIL_ADDRESS>there's nothing equivalent to the go list -m flag in go install, so it doesn't seem inconsistent. That said, allowing a version query like @>v0.6.5 to match multiple versions is pretty different than any command today. I wonder if we should track that separately and keep this issue focused on the equivalent forms in go install and go run?


Fix developer portal not starting with npm start

Purpose
This PR fixes the issue where dev server is not spinning up with npm start command when executed within the Developer Portal webapp.

Fixes : https://github.com/wso2-enterprise/wso2-apim-internal/issues/5299


Bump actions version in workflow files

e.g. actions/checkout@v2 generates deprecation warnings in CI

Closing as resolved in #511.


Issue in IE 11: template v-if in table row leads to fragment instance

Vue.js version
1.0.16
Reproduction Link
https://jsfiddle.net/pgunya/v8ndoocg/
Steps to reproduce
Add <template v-if="..."> element in table row and check in IE.
What is Expected?
No fragment instance and correct behaviour.
What is actually happening?
In Internet Explorer 11 when <template v-if="..."> element is present inside template's table row then vue gives a warning about fragment instance and template v-if doesn't work as expected. In latest Chrome and Firefox this works fine. At the same time <template v-if="..."> works fine if placed inside other tag, example: https://jsfiddle.net/pgunya/sa7vtgtt/

IE does not support <template> inside elements like <table>, <select>.. You may want to read through this post: http://forum.vuejs.org/topic/2088/table-and-template-on-ie

Yes, now I see.
But then I have a silly question: template is located inside a <script> tag, I thought that it content should not be parsed by browser until vue compile it and insert into target element. Am I wrong? Thanks!

Vue renders the template into real html before compiling it, so the same html restrictions apply for Vue templates, no matter how you define it.


Change assault borg modules texture

About the PR

This PR changes assault borg modules texture to weapon cyborg module texture from uplink
Why / Balance

Old textures was just recolored tool module texture
Technical details

No
Media

[ ] I have added screenshots/videos to this PR showcasing its changes ingame

Breaking changes
No

Changelog

:cl:

tweak: Changed textures of the assault borg modules

I don`t know why tests failed

I also removed some spaces in the file


trivy-operator: add plural to configaudit CRD

The configaudit reports are namespaced, and they are created for several resources, we should have a plural for it. Currently it only supports singular:
configauditreports                configaudit                    aquasecurity.github.io/v1alpha1        true         ConfigAuditReport

@chen-keinan any point against?

I would like to work on it.

@chen-keinan any point against?

All good !!!

@pradeepnnv Let us know if you have any questions. Thank you!

When I describe the configauditreport, I do see a plural for it.
 k describe crd configauditreports.aquasecurity.github.io

Status:
  Accepted Names:
    Kind:       ConfigAuditReport
    List Kind:  ConfigAuditReportList
    Plural:     configauditreports
    Short Names:
      configaudit
    Singular:  configauditreport

@josedonizetti , looks like I'm missing something. Can you please clarify?

@pradeepnnv Yes, I was referencing about the short names.


Image rebase: rename base-notebook to datascience-notebook

Now we have datahub-base-notebook + datascience-notebook contents all in datascience-notebook.
datascience-notebook has become the root image

Update June 6: merged in changes from PR #66 to #72 into this branch


Support constant names for extended enums

Based on archboard feedback during review of Search's FieldBuilder, we discussed support for constant names for extended enums, which use a nested static class with public constants (similar to some track 1 instances). It would be great to have generator support for this, though in the meantime we can write tests using reflection to make sure that extended enum static properties and constant names match exactly.
For should follow:
public readonly partial struct ExEnum
{
  private const string FooValue = "foo";
  public static ExEnum Foo  { get; } = new ExEnum(FooValue);
  public static class Values // Values here is fixed; if collision exists,
                            // people can use something like ValuesAsString,
                            // but we suspect the chances of this are very small
                            // since this feature should only be used when constant expressions are needed.
  {
    public const string Foo = ExEnum.FooValue;
  }
}

The XML doc comment should be the same on both members.
I'll be opening a PR against azure/azure-sdk for this as well.

We should also have a higher bar for using these extensible enums. They cause lots of problems in UX.

As discussed internally, we'll use Values instead.
As for a higher bar, what should we use instead? We already state they have to be used for input or roundtripping. .NET without unions gives us no other options unless we want to overload parameters (enum or string), and use only strings in model properties, but then how would people know they can use enum values (via ToString())?

With https://github.com/Azure/azure-sdk/pull/1610 merged, this is now part of the guidelines: https://azure.github.io/azure-sdk/dotnet_implementation.html#dotnet-enums-values
Having an attribute or something as opposed to plumbing through an AutoRest directive would be fine, especially since this may only affect C# anyway.

Make this conditional on Values type defined under the extensible enum.

How would that work, though? Can the nested class also be partial?

Yup, https://sharplab.io/#v2:EYLgtghgzgLgpgJwDQBMQGoA+BYAUAAQGYACfARgDZiAHCBGASwgBtSAmUsgdmIG89ig0iXJVa9Jq3wcACggD2AcwQQwHXgF88WgiMo06jFu049+uIcM5jDkk3KUq1fHRqA=


[copy_central] paths in the project need to be made location agnostic

Paths should not need to change based on where the script is executed from.

@manavgarg-discovery i can work on it, can you briefly explain the issue also provide path to the file.


Add asciidoc source

This PR addresses issues #1230, #92, #93. @pvlugter, the Java and Javascript sections still need work to convert the markdown @@snip and @@javadoc references to includes that Antora understands. This can be done in one of two ways: 1) create a submodule that pulls in the source code from the sample repo into the asciidoc tree, or set up the sample repos so that they can be  pulled in by Antora. Merging this PR will not effect the current build since the asciidoc source is in it's own tree. And, before it is published by Antora, a CI/CD process needs to be set up.
To build locally to test Antora, follow these steps:
You will need Node.js and Antora installed as described in the Antora doc.
This Cloudstate repository contains two files for building the docs, site.yml, which builds a production-ready tree of HTML, and author-mode-site.yml, which builds HTML containing reviewer and todo notes.
When your environment is ready, in a command prompt:

Change into the git directory that contains site.yml.

Run one of the the following commands:

antora site.yml
or
antora author-mode-site.yml
Antora generates the output in a build\site folder. Open index.html to view the output.
This information will be added to the readme once the process has been validated.

@rstento You should be able to integrate the Antora build into the travis.yml to supersede Paradox, right?

Thanks @rstento. All looks okay. On the remaining conversions:
Can we not include the snippets from their current locations? We'll still need some way to have these compiled and tested for whatever language they're written in.
For javadocs, it would be useful to have something equivalent to the paradox support for linking to API docs. I see there's an open issue for Antora: https://gitlab.com/antora/antora/-/issues/439 and a recent plugin: https://gitlab.com/djencks/antora-javadoc
Similarly with the external refs to jsdocs that we're using currently.

@rstento, I'm targeting this PR against the antora branch instead of master, so that this can be developed incrementally.  Probably easier than me pushing to your PR branch. I'll squash and merge this one now so we can build on it.


Add Url Dissect extension

Description
I did this extension for me and I use it a lot so I want to share it.
Basically breaks down the URL and you can easily can copy each part of it
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi 👋
Thank you for your contribution 🔥
I am currently reviewing this extension: #11011. Do you think it will suit your needs? We could enhance that one with features from yours

Hi 👋
Thank you for your contribution 🔥
I am currently reviewing this extension: #11011. Do you think it will suit your needs? We could enhance that one with features from yours

Maybe can be another command from that extension, but I built this as a quick way to manipulate an URL and copy and paste fast, I can add it, or see if it can fit in that extension!

We just released the other with copy paste functions, could you check it out again?

I will close this since it no makes more sense!


Scoped change to have enums implement IPropertyValue to be consistent with coreclr and cppwinrt.

Need to make sure this doesn't regress #605

This indeed breaks #605 again because TypeNameSUpport.GetNameForType returns null for enums.


Basic Photometric and Astrometric Validation

Compute and analyze overall astrometric and photometric repeatability of Twinkles PhoSim/DM L2 runs as part of Twinkles Run 1 and Run 2.
Tools will include validate_drp
https://github.com/lsst/validate_drp
and related tests, such as those @SimonKrughoff did in https://github.com/DarkEnergyScienceCollaboration/Twinkles/issues/70#issuecomment-168073498
This Issue is not directly looking at light curves of known variable objects.  Light curve analysis and tests may likely fall more appropriately under Issues for Monitor discussion.
See posts below for various results during development. We will need to parcel up the tests we want so that they can be run as a matter of course as part of the Twinkles workflow.

[ ] Assemble repeatability script and check in #221
[ ] Include or discard old homegrown script #173
[ ] Add repeatability script to workflow

Let's aim to have this in place for Run 2, after some iteration in Run 1.1!

Here are some figures of the astrometric and photometric repeatability as compared to the LSST Science Requirements Document.
u: PA1(IQR) = 82.61+-1.53 mmag
i: PA1(IQR) = 21.95+-1.57 mmag
z: PA1(IQR) = 16.69+-0.57 mmag
y: PA1(IQR) = 25.54+-1.21 mmag

Cooooooool! Nice work @wmwv :-) I'm going to need many of these panels
unpacking for me, I guess others might be in the same boat. Could I please
ask you to find a good place to put them in our Twinkles session slides
here?:
https://docs.google.com/presentation/d/1MdGGDrITW4-n04goJNYBAoVwnQBjRkibxuJY8EJZWXc/edit#slide=id.p
Real quick: how many visits went into these plots? Which dataset are you
working from?
The script that made these, by calling the validate_drp tools - is it
checked into the Twinkles repo? I'm not tracking commits very well, only
issues and pull requests.
Hooray validation! :-)
On Sat, Mar 5, 2016 at 12:16 PM, Michael Wood-Vasey <
<EMAIL_ADDRESS>wrote:

Here are some figures of the astrometric and photometric repeatability as
compared to the LSST Science Requirements Document.
u: PA1(IQR) = 82.61+-1.53 mmag
i: PA1(IQR) = 21.95+-1.57 mmag
z: PA1(IQR) = 16.69+-0.57 mmag
y: PA1(IQR) = 25.54+-1.21 mmag
[image: twinkles_run1_output_i_am1_d_5_arcmin_17 0-21 5]
https://cloud.githubusercontent.com/assets/3201257/13550091/7301be60-e2e3-11e5-930a-c9f9ba25259d.png
[image: twinkles_run1_output_i_check_photometry]
https://cloud.githubusercontent.com/assets/3201257/13550092/7d80f298-e2e3-11e5-8eaf-11db9279f6a1.png
[image: twinkles_run1_output_i_pa1]
https://cloud.githubusercontent.com/assets/3201257/13550094/8bbbb1c2-e2e3-11e5-89e9-1146f93f9b1e.png
[image: twinkles_run1_output_i_check_astrometry]
https://cloud.githubusercontent.com/assets/3201257/13550093/82c9ae2a-e2e3-11e5-8440-b77cbddf460e.png
—
Reply to this email directly or view it on GitHub
https://github.com/DarkEnergyScienceCollaboration/Twinkles/issues/167#issuecomment-192725535
.

These are from just 10 visits from the first sample run of data processing for Twinkles.  My goal today is to understand what has now been processed all the way through processCcd.py (I'm not thinking about coadd stuff yet).  I will update the slides in a few hours with wherever things stand.
These plots and results were generated by simply running:
setup validate_drp
validateDrp.py Twinkles/run1/output

Where you can install validate_drp through the "standard" LSST DM tools (I know they can be frustrating).  For more details see the validate_drp repository README.md:
https://github.com/lsst/validate_drp
or the DM Tech Note at
http://dmtn-008.lsst.io/en/latest/

r-band results from 100-visit run:

Those don't look that great.   Maybe we can chat tomorrow about what might be going on.  Do these plots exclude galaxies by using the classification_extendedness value?

Yeah, lets make sure we leave plenty of time for these. I saw you put a
couple of figures in the slides at
https://docs.google.com/presentation/d/1MdGGDrITW4-n04goJNYBAoVwnQBjRkibxuJY8EJZWXc
, great if you can update them with the most problematic of the new plots!
Thanks Michael, this is great.
On Mon, Mar 7, 2016 at 11:38 PM, SimonKrughoff<EMAIL_ADDRESS>wrote:

Those don't look that great. Maybe we can chat tomorrow about what might
be going on. Do these plots exclude galaxies by using the
classification_extendedness value?
—
Reply to this email directly or view it on GitHub
https://github.com/DarkEnergyScienceCollaboration/Twinkles/issues/167#issuecomment-193650653
.

@wmwv and I talked about this a little bit.  I was wrong that they don't show what I expect.  These are more clear than the repeatability figures I've been producing, but the two are consistent.

Mostly it was a misinterpretation of the 1.2% floor.  My floor of 10 mmag is similar to that seen by Michael.

@wmwv @TomGlanzman @tony-johnson I have elevated this issue to epic status, and edited the all-important first comment to include an initial checklist for what we need to do to get this validation analysis into the workflow and run automagically. No doubt I have done this inadequately, it would be great if you could help improve it. The idea is that each bullet point will point to its own issue - which will also need to be created, and owned - possibly by you! :-)

@wmwv I'm trying to run the current master of validate_drp on the Run1.1 data (using v12_0_rc1 of the Stack).  I'm seeing this:
bash$ validateDrp.py output/
CameraMapper: Loading registry registry from output/_parent/registry.sqlite3
CameraMapper: Loading registry registry from output/_parent/registry.sqlite3
CameraMapper: Loading registry registry from output/_parent/registry.sqlite3
CameraMapper: Loading registry registry from output/_parent/registry.sqlite3
Traceback (most recent call last):
  File "/nfs/farm/g/desc/u1/users/jchiang/desc_projects/twinkles/Run1.1/validate_drp/validate_drp/bin/validateDrp.py", line 86, in <module>
    validate.run(args.repo, **kwargs)
  File "/nfs/farm/g/desc/u1/users/jchiang/desc_projects/twinkles/Run1.1/validate_drp/validate_drp/python/lsst/validate/drp/validate.py", line 460, in run
    **kwargs)
  File "/nfs/farm/g/desc/u1/users/jchiang/desc_projects/twinkles/Run1.1/validate_drp/validate_drp/python/lsst/validate/drp/validate.py", line 525, in runOneFilter
    allMatches = loadAndMatchData(repo, visitDataIds, verbose=verbose)
  File "/nfs/farm/g/desc/u1/users/jchiang/desc_projects/twinkles/Run1.1/validate_drp/validate_drp/python/lsst/validate/drp/validate.py", line 126, in loadAndMatchData
    print(len(oldSrc), "sources in ccd %s  visit %s" % (vId[ccdKeyName], vId["visit"]))
KeyError: 'ccd'

Any suggestions?

Yes, validate_drp master doesn't work on Twinkles.  I've never successfully integrated the hacks I had to do to get it to work on the LSST simulated.  They're really awful.
There's no general way to get the concept of a 'ccd' from the Butler.  Different cameras use different kesy.  There's 'ccd', 'ccdname', and for LSST camera you need the {'raft', 'sensor'} combination.

Here are the repeatability numbers

filter
PA1(IQR) [mmag]
AM1 [mas]

u
321.16+/-77.44
0.00

g
694.16+/-57.66
27.26

r
563.19+/-38.28
26.86

i
544.99+/-56.31
27.75

and the plots for i-band:

@jchiang87 Ah, yes.  If there's just 1 CCD, your dataIdFormat={'visit': int} hack will work.

Hmmm... the photometric repeatability numbers are really bad.
I bet we need to figure out how to separate stars and galaxies much better.

I realize it's worse that star/galaxy separation.  Something just going really funny with the photometry.

I'm investigating this using the files on NERSC in
/global/homes/t/tony_j/Twinkles/Run1.1
(which points to)
/global/cscratch1/sd/tony_j/Twinkles/501

@jchiang87   The NERSC output data repo needs a _parent that points to the correct input data repo before the DM Butler can successfully and easily read files.  I've assigned #263 @tony-johnson to fix this.  Once #263 is closed I can take a look further at investigating the repeatibility in this present issue #167.

Right.  I forgot about that.   To get around this for the db stuff, I created a directory in my area with symlinks so that the butler will work.  That directory is here:
/global/u1/j/jchiang8/twinkles/Run1.1

Ah, very good solution.
Could you kindly update the permissions so I can use that set of symlinks?

So I just did this:
[edison01] pwd
/global/homes/j/jchiang8/twinkles
[edison01] chmod -R go+rx inputs/
[edison01] chmod -R go+rx Run1.1/

Is that sufficient? (It looked to me that you already had access.)

Yes, thank you!
[The previous permissions were:
ls -l /global/u1/j/jchiang8/twinkles/
[...]
drwxr-x---  2 jchiang8 jchiang8  4096 Apr 28 10:54 Run1.1
[...]

]

@jchiang87 Ran and got the following error message.  Have you seen this before?
I suspect there may be some versioning inconsistency between the obs_lsstSim I have from the 2016-04-15 conda build installed on Cori/Edison.  Will think about how to investigate further.  Suggestions welcome.
AttributeError: 'module' object has no attribute 'getKeys'

[edison02 tmp] validateDrp.py /global/u1/j/jchiang8/twinkles/Run1.1
/project/projectdirs/lsst/lsstDM/Cori/conda-env/2016-04-15/LSST_STACK/lib/python2.7/site-packages/matplotlib/font_manager.py:273: UserWarning: Matplotlib is building the font cache using fc-list. This may take a moment.
  warnings.warn('Matplotlib is building the font cache using fc-list. This may take a moment.')
Traceback (most recent call last):
  File "/global/u1/w/wmwv/local/src/validate_drp/bin/validateDrp.py", line 79, in <module>
    kwargs['dataIds'] = util.discoverDataIds(args.repo)
  File "/global/u1/w/wmwv/local/src/validate_drp/python/lsst/validate/drp/util.py", line 147, in discoverDataIds
    thisSubset = butler.subset(datasetType='src', **kwargs)
  File "/project/projectdirs/lsst/lsstDM/Cori/conda-env/2016-04-15/LSST_STACK/opt/lsst/daf_persistence/python/lsst/daf/persistence/butler.py", line 425, in subset
    return ButlerSubset(self, datasetType, level, dataId)
  File "/project/projectdirs/lsst/lsstDM/Cori/conda-env/2016-04-15/LSST_STACK/opt/lsst/daf_persistence/python/lsst/daf/persistence/butlerSubset.py", line 77, in __init__
    keys = self.butler.getKeys(datasetType, level)
  File "/project/projectdirs/lsst/lsstDM/Cori/conda-env/2016-04-15/LSST_STACK/opt/lsst/daf_persistence/python/lsst/daf/persistence/butler.py", line 232, in getKeys
    return self.repository.getKeys(datasetType, level)
  File "/project/projectdirs/lsst/lsstDM/Cori/conda-env/2016-04-15/LSST_STACK/opt/lsst/daf_persistence/python/lsst/daf/persistence/repository.py", line 361, in getKeys
    return self.doParents(Repository.doGetKeys, *args, **kwargs)
  File "/project/projectdirs/lsst/lsstDM/Cori/conda-env/2016-04-15/LSST_STACK/opt/lsst/daf_persistence/python/lsst/daf/persistence/repository.py", line 325, in doParents
    res = func(parent, *args, **kwargs)
  File "/project/projectdirs/lsst/lsstDM/Cori/conda-env/2016-04-15/LSST_STACK/opt/lsst/daf_persistence/python/lsst/daf/persistence/repository.py", line 374, in doGetKeys
    return self._mapper.getKeys(*args, **kwargs)
AttributeError: 'module' object has no attribute 'getKeys'

No, I didn't see that error running with v12_0_rc1.  I have a conda install of the Stack of the same vintage at slac.  I'll try with that Stack to see if I get the same error.

Using the conda distribution I installed at SLAC on 2016-04-12, everything seems to be working fine for validateDrp.py.  It finished g-band and is working the next one.  Here is some provenance info on that Stack:
bash-4.1$ eups list lsst_apps
   w.2016.15    current b2003 conda setup
bash-4.1$

@wmwv That is very strange.  That very line is still in master.

Here is the contents of _mapper:
[edison12] cat _parent/_mapper
lsst.obs.lsstSim.LsstSimMapper
[edison12]

BTW, that output repo worked fine for the pserv ingest (which uses the butler to find the files) into the DESC_Twinkles_Level2 db.  For that, I used the most recent Stack that Heather installed for cori.

@SimonKrughoff @jchiang87   Apologies!  Problem existed between keyboard and chair!

@wmwv Can this issue be closed, or is it still active? Moving to "Post Run 2 tasks for now".


Error Loading Data

Hi,
We are using Hadoop with elasticsearch on a cluster of 5 nodes and we are using : CDH 5.8.0, eastic search 2.3.5 and elasticsearch-hadoop-2.3.4.
We created a table on elastic search using hive and it worked and we are trying to load the data from hive :
INSERT overwrite TABLE cdr_es SELECT ....
FROM cdr_link
But we have the following error :
Diagnostic Messages for this Task:
Error: java.lang.RuntimeException: java.lang.Error: Multiple ES-Hadoop versions detected in the classpath; please use only one
jar:file:/data/yarn/nm/usercache/cloudera-scm/filecache/938/elasticsearch-hadoop-cascading-2.3.4.jar
jar:file:/data/yarn/nm/usercache/cloudera-scm/filecache/955/elasticsearch-hadoop-hive-2.3.4.jar
jar:file:/data/yarn/nm/usercache/cloudera-scm/filecache/931/elasticsearch-storm-2.3.4.jar
jar:file:/data/yarn/nm/usercache/cloudera-scm/filecache/940/elasticsearch-hadoop-mr-2.3.4.jar
jar:file:/data/yarn/nm/usercache/cloudera-scm/filecache/926/elasticsearch-spark_2.11-2.3.4.jar
jar:file:/data/yarn/nm/usercache/cloudera-scm/filecache/937/elasticsearch-spark-1.2_2.10-2.3.4.jar
jar:file:/data/yarn/nm/usercache/cloudera-scm/filecache/929/elasticsearch-spark-1.2_2.11-2.3.4.jar
jar:file:/data/yarn/nm/usercache/cloudera-scm/filecache/950/elasticsearch-hadoop-pig-2.3.4.jar
jar:file:/data/yarn/nm/usercache/cloudera-scm/filecache/959/elasticsearch-spark_2.10-2.3.4.jar
jar:file:/data/yarn/nm/usercache/cloudera-scm/filecache/943/elasticsearch-hadoop-2.3.4.jar
    at org.apache.hadoop.hive.ql.exec.mr.ExecMapper.map(ExecMapper.java:179)
    at org.apache.hadoop.mapred.MapRunner.run(MapRunner.java:54)
    at org.apache.hadoop.mapred.MapTask.runOldMapper(MapTask.java:453)
    at org.apache.hadoop.mapred.MapTask.run(MapTask.java:343)
    at org.apache.hadoop.mapred.YarnChild$2.run(YarnChild.java:164)
    at java.security.AccessController.doPrivileged(Native Method)
    at javax.security.auth.Subject.doAs(Subject.java:415)
    at org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1693)
    at org.apache.hadoop.mapred.YarnChild.main(YarnChild.java:158)

Caused by: java.lang.Error: Multiple ES-Hadoop versions detected in the classpath; please use only one
How can we solve the problem ?
Thanks

This question is better suited for our discuss forum. Github is for confirmed bugs and actionable feature/improvements only.
You seem to be using all of the jars when you should be using only one. We provide the full package of features in one jar (elasticsearch-hadoop-2.3.4.jar), but we also provide dedicated jars for each technology should users desire to use them in a standalone fashion. Keep the elasticsearch-hadoop-2.3.4.jar and remove the others.


allow access to swagger.json file to specific users

Hi Team,
I am using swagger 2 with jersey 2 .
Is there any way to restrict access to swagger.json file for specific users. Currently the swagger.json file is accessible publicly.

The swagger.json is exposed via a regular jax-rs endpoint. You can write and/or apply any security filter on top of that to control access.

Thanks @webron . that solved the issue.


Changing multi-distro vars to facts

Before jumping into it: really appreciate the work done here together with the GitHub actions pipeline + Molecule action. Thank you for sharing these gems!
Debian doesn't (strictly) depend on the values coming from
https://github.com/ome/ansible-role-postgresql-client/blob/master/vars/main.yml#L5-L21
but it will still fail at
https://github.com/ome/ansible-role-postgresql/blob/ffb2effedc5514bfc0fbac2be72fad223cfa8168/tasks/initialise.yml#L14-L26
since that will try to load default values for RedHat (+ versions).
Setting Postgres versions that aren't previously defined on PostgreSQL client (e.g. 13, 14) are working just fine on Debian after implementing these changes.
To segregate both installations, the values are now set using set_fact inside its own distro's tasks file, which corresponds to the conditional on tasks/main.yml and ensures consistency.
Some bonus changes:

included a block on tasks to have become+become_user defined less
times;
added a variable to change the user that it will become when asked.

thank you for pointing that out @sbesson.
The tests are now passing, let me know in case it is necessary to change something else.

are you considering the postgresql_dist_* variables as part of the public API of this role i.e. is it something that not only the role but also the playbooks could start depending on?

From my perspective, keeping the variables as part of the public API would be preferable to enable customizations by users (when/if one day necessary), it allows the role to be future-proof without requiring constant maintenance. e.g.: a user could parameterize the role to have Pg 13+ running on RedHat.
If that sounds plausible, I would be glad to write the README breaking change instructions.
What do you think?

Sorry for the delay and happy new year. Yes please push the related README changes. Also you are mentioning breaking changes: is there anything else than the postgresql_dist_{debian,redhat} internal variables?

Happy new year! :tada:

[...] you are mentioning breaking changes: is there anything else than the postgresql_dist_{debian,redhat} internal variables?

Sorry about the misunderstanding, I used the same term (now I realize it was misleading) from https://github.com/ome/ansible-role-postgresql/pull/25#pullrequestreview-831686128

[...] in the README so that we would consider modifications as a breaking change.

Just pushed some instructions to README + comments on defaults/main.yml @sbesson

Thanks @macunha1. Looks good and I would propose to get this released as 5.1.0 i.e. a minor version increment. Before merging and tagging this, could you sign and return the OME CLA - see the instructions in the contributing documentation?

Thanks @macunha1. Looks good and I would propose to get this released as 5.1.0 i.e. a minor version increment. Before merging and tagging this, could you sign and return the OME CLA - see the instructions in the contributing documentation?

Sorry about the delay, just signed and sent it to the mentioned e-mail.

@macunha1 thanks for your contribution! These changes have now been released in a minor release of this role and are available from Galaxy https://galaxy.ansible.com/ome/postgresql


Help with TextDocumentContentProvider

VSCode Version: 1.7.x
OS Version: OS X 10.10.5

Trying to figure out how to make TextDocumentContentProvider work!
Here is the code from extension.ts
    let provider = new SnippetProvider();
    let registration = vscode.workspace.registerTextDocumentContentProvider('foo', provider);

    let disposable = vscode.commands.registerCommand('extension.showFoo', () =>{
        return vscode.workspace.openTextDocument(vscode.Uri.parse('foo://auth/path')).then((doc) => {
            console.log('doc ', doc);
        });
    });

   context.subscriptions.push(disposable, registration);

And here is a simple class that provides snippets.
export default class SnippetProvider implements vscode.TextDocumentContentProvider {
    private templateUtil = new TemplateUtil();
    public provideTextDocumentContent (uri : vscode.Uri) : Thenable <string> {
        let data = this.templateUtil.loadTemplates();
        return data;
    }
}

I keep seeing cannot open foo://auth/path. Detail: Document with resource foo://auth/path does not exist. Of course, I know that path does not exist. It's virtual. Where am I wrong?
Any thoughts?
Thank you,
Ritesh

There is a complete working sample at https://github.com/Microsoft/vscode-extension-samples/tree/master/contentprovider-sample

The snippet looks ok. One question: are you sure this this.templateUtil.loadTemplates() returns a string? That's the only thing I see there

@jriken good catch. I swear that return value from this.templateUtil.loadTemplates() was screaming at me saying please take care of me I am an object and not a string! I guess not loud enough for me to hear it! ;) ;)
stringify with pretty print took care of it
Thank you!

awesome. thanks for getting back to me.

@jrieken here is a tutorial that should help anyone trying to implement this functionality.
Content Provider Tutorial


[Request] Join latest server/world action

Hey there,
It would be great to add an action to join the latest session. Either latest server or latest sp-world. And "disable" the button if there is no stored last server/world.
And additionally, a placeholder-string with the latest server/world name.
Currently, I am using the "connectToServer"- action, but since it is a private server, I don't want this on the menu when I set the pack to the public. So the idea came up: "why a specific server, why not the last session" - it would be really neat - since most people play mostly a single world/server inside a pack.

@mysticdrew any way you can do this? :3
Would round up my current Setup!

no


.reactserverrc is malformed json

It would seem undefined is not a valid symbol in json, and the debug log-level needed quoting
Seems strange that what one would assume to be generated JSON isn't actually valid
.reactserverrc
 {
-  "routes": "./routes.js"
+  "routes": "./routes.js",
   "host": "localhost",
   "port": 3000,
   "js-port": 3001,
   "hot": false,
   "minify": false,
   "long-term-caching": false,
   "compile-only": false,
-  "js-url": undefined,
   "https": false,
-  "https-key": undefined,
-  "https-cert": undefined,
-  "https-ca": undefined,
-  "https-pfx": undefined,
-  "https-passphrase": undefined,
-  "log-level": debug,
+  "log-level": "debug",
   "env": {
     "staging": {
       "port": "4000"
     },
     "production": {
        "port": "80"
     }
+  }
}

Changing this file and copying the .gulpfile from the repo as noted in #254 seems make the server work for me.

Ouch.  Thanks for reporting @DanielFGray.


add db.fn.coalesce(...)

[x] implement coalesce type extract.
[x] implement coalesce parser.
[x] implement FunctionModule.coalesce(value, ...otherValues).
[x] unit tests.
[x] typings tests.
[ ] FunctionModule.coalesce ts docs.

Don't know if we should go for a new builder and node, so went with raw for now.
If we go for new builder and node.. not sure what terminology to use..
In mysql docs it is grouped with comparison operators & functions.
In postgres docs it is grouped with conditional expressions
In sqlite docs it is grouped with built-in scalar functions

This is good to go once you rebase! Nice work once again 👍

Thanks! Seems ready now.. also went over 5c2c190 (👏 btw ) just to be sure.


add test for require_variable()

This PR:

tests the behaviour of using a list of years as an argument for require_variable()

updates the docstring

[x] Tests Added

[x] Documentation Added

[ ] Description in RELEASE_NOTES.md Added  -> no changed behaviour, so not required

Coverage remained the same at 85.645% when pulling f91afb7c442e25aae2ac9278bb3a77ab5ce52ee7 on require_variable-behaviour into ebe40a1c3abf09a19590810718e5731905e9e780 on master.


Unable to encrypt message when provided address is with dashes

Steps:

Go to transfer view
Put address (address must have at least one outgoing tx) with dashes.
Put message
Click "Encrypt message"

Result: "Invalid recipient public key" message pop-up
Expected result: Encrypt message (if recipient account has outgoing txs - public key known)

@NikolaiB please test when you'll have some time. It's merged into dev branch.

Works fine in dev branch.