Very slow startup with new v2.21.0

I have a system with many cameras connected. Some have an IMU, some do not. With version v2.20.2 and earlier, I would just create the pipeline as if all cameras had an IMU (because there was no way to ask) and the program would start up fine, just printing out a warning.
Now, with v2.21.0, the same program throws an exception if the pipeline is created with an IMU node but the camera has no IMU, which forces the use of the new IMU detection API. That sounds nice, except that querying for IMU existence requires connection to the device, which is always very slow. Having to do it twice (first with the non-IMU pipeline, then check for IMU, then second connection with the IMU pipeline) is 2x very slow, which is unacceptable.
I have a very strong preference for the old method that allows creation of the IMU node without throwing an exception, just a warning. Alternatively, it would help to have a fast way of determining presence of an IMU prior to making the pipeline e.g. have the connection always being very fast, maybe by having an option to connect without creating a pipeline, just enough to query the device properties, then later start the pipeline. As it stands, I cannot upgrade to the new version.

@saulthu

Alternatively, it would help to have a fast way of determining presence of an IMU prior to making the pipeline e.g. have the connection always being very fast, maybe by having an option to connect without creating a pipeline, just enough to query the device properties, then later start the pipeline

This is already possible and was the intended way to use.
See example here

@szabi-luxonis
The linked example does this:
import depthai as dai
device = dai.Device()
imuType = device.getConnectedIMU()
imuFirmwareVersion = device.getIMUFirmwareVersion()

And that may be fine for a single device, yet it is even incomplete for a single device. In that single device case the code must continue by closing that device and reconnecting with a pipeline that either contains an IMU node or does not contain an IMU node, hence more than doubling the connection time. In my case I have many cameras, and connection is already very slow without increasing the connection time even more, which has happened in the transition from 2.20.x to 2.21.x.

_try_2_20_x_method() with depthai==2.20.x:
start: 0.000 s
pipeline: 0.000 s
len(all_infos)=7
getAllAvailableDevices: 0.232 s
[1944301081E8221300] [<IP_ADDRESS>] [1.312] [IMU(0)] [critical] IMU driver failed with error code 1. Your board likely doesn't have IMU!
[1944301031F2221300] [<IP_ADDRESS>] [1.801] [IMU(0)] [critical] IMU driver failed with error code 1. Your board likely doesn't have IMU!
[19443010F125231300] [<IP_ADDRESS>] [1.825] [IMU(0)] [critical] IMU driver failed with error code 1. Your board likely doesn't have IMU!
[1944301091E8221300] [3.1.1] [1.378] [IMU(0)] [critical] IMU driver failed with error code 1. Your board likely doesn't have IMU!
[18443010E1EA3A1300] [3.1.2] [1.845] [IMU(0)] [critical] IMU driver failed with error code 1. Your board likely doesn't have IMU!
len(all_devs)=7
open: 12.355 s
[19443010F128231300] [<IP_ADDRESS>] [1.422] [IMU(0)] [critical] IMU driver failed with error code 1. Your board likely doesn't have IMU!
close: 32.657 s
del: 0.013 s

_try_2_20_x_method() with depthai==2.21.x:
start: 0.000 s
pipeline: 0.000 s
len(all_infos)=7
getAllAvailableDevices: 0.262 s
Traceback (most recent call last):
  File "C:\Users\saul\Documents\test_depthai\try_2.20.x.py", line 52, in <module>
    _try_2_20_x_method()
  File "C:\Users\saul\Documents\test_depthai\try_2.20.x.py", line 35, in _try_2_20_x_method
    dev = dai.Device(pipeline, info)
RuntimeError: IMU(0) - IMU invalid settings!: IMU not detected. Your board doesn't have IMU or requires firmware update (Set imu.enableFirmwareUpdate(True) explicitly.). 1, 321

_try_2_21_x_method() with depthai==2.21.x:
start: 0.000 s
imu_pipeline: 0.000 s
non_pipeline: 0.000 s
len(all_infos)=7
getAllAvailableDevices: 0.238 s
getConnectedIMU: 48.674 s
len(all_devs)=7
open: 12.880 s
close: 32.204 s
del: 0.013 s

To be clear, in these tests I'm connecting to 6x OAK-1-W which do not have IMU, and 1x OAK-D-W which does have IMU.

@saulthu Connecting to the device is separate from starting a pipeline.
You can connect to the device:
device = dai.Device()
and query connected cameras, IMU, and their capability and create the pipeline based on that, then start the pipeline, without rebooting the device.
See example here
In your case it would be something like:
device = dai.Device()
imuType = device.getConnectedIMU()
pipeline = dai.Pipeline()
if imuType != "NONE":
  imu = pipeline.create(dai.node.IMU)
  xlinkOut = pipeline.create(dai.node.XLinkOut)
  xlinkOut.setStreamName("imu")
  imu.enableIMUSensor(...)
....
with device:
    device.startPipeline(pipeline)

OK, this issue appears to be a misunderstanding! I will try this immediately. Thanks.


The README file in this repo has a bad link

The README file in this repo has a bad link
Status code [404:NotFound] - Link: https://github.com/miyes90
This was found by an new experimental hobby project that I have just created: https://github.com/MrCull/GitHub-Repo-ReadMe-Dead-Link-Finder
If this has been in any way helpful then please consider giving the above Repo a Star.

Thanks for the heads up!


Making HttpApp a type-alias to Http

Converting HttpApp to Http and vice-versa has the following limitations

A bit of memory overhead is added when converting a Http to an HttpApp
Composition is also limited in HttpApp, so one needs to convert it to an Http first, apply operators and then convert it back to a HttpApp.

In this PR, we address the above problems. The trade-off is that we end up adding operators that are only applicable if the A is Request and/or B is Response

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/next@13ae231). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff               @@
##             release/next     #531   +/-   ##
===============================================
  Coverage                ?   57.49%
===============================================
  Files                   ?       66
  Lines                   ?     1468
  Branches                ?       45
===============================================
  Hits                    ?      844
  Misses                  ?      624
  Partials                ?        0

Continue to review full report at Codecov.

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


Create Image table

We should omit this panoramaimage, and instead add a Photo object (maybe call it a pano or image or something)
Swift analog: https://github.com/andybaumgar/mesh-db-swift/blob/main/Sources/App/Models/Photo.swift

We may be able to get away with referencing the Github repository for now: https://github.com/nycmeshnet/node-db/tree/master/data/panoramas

Implemented in #216


Add build script for releases

Suggest to add a build script using the build tool which covers the following steps:

Running the example live scripts so they all show their outputs
Bumping the release number in Contents.m (useful for showcasing compute environment in DandiHub)

@ehennestad: one thing discussed in the overview by @bpancras at the recent "campfire chat" was the build tool.
This issue raised last year suggested some low-hanging fruit this system could help to achieve in the BOT context.
Maybe useful as a quick way to get familiar with it.

Yes, this should be straight-forward.
A couple of questions:
I know you can run export(..., Run=true) to run livescripts and they will show their outputs. Also, I have found the internal function matlab.internal.liveeditor.executeAndSave which is useful if you don't need to save. Do you know of any non-internal methods for doing this?
Also, maybe more important. I wanted to export a live script using a GitHub action, but got the following error:
 {Error using matlab.internal.cef.webwindow
  DISPLAY environment variable must be set to a valid X11 display.

  Error in matlab.internal.liveeditor.LiveEditorUtilities.createDocument (line 26)
      webWindow = matlab.internal.cef.webwindow(url, matlab.internal.getDebugPort);

  Error in matlab.internal.liveeditor.LiveEditorUtilities.open>openUsingCEF (line 100)
                  LiveEditorUtilities.createDocument(timeout,webWindow);

  Error in matlab.internal.liveeditor.LiveEditorUtilities.open (line 33)
      [javaRichDocument, cleanupObj, webWindow] = openUsingCEF(fileName, reuse, timeout, webWindow);

  Error in export (line 116)
          matlab.internal.liveeditor.LiveEditorUtilities.open(...

The workflow runs on ubuntu-latest
Any ideas if that would be possible?

Thanks @ehennestad  for elaborating. That's what I guessed the application would be!
Generally my opinion is: a build script seems heavyweight solely for the purpose of making live scripts viewable/linkable.
Wrt the File Exchange rendered versions, they are linkable in my hands, by using "Copy Link Address" from the browser. Can you see this on a second look?
That said, I recognize this is not elegant, and it's only sensible to update this link on the README at release versions, not at every commit. Given that there are other reasons for the build script (as listed above), I'm good with exporting to MD and/or HTML as an additional benefit: stable links for each of the rendered examples.

Generally my opinion is: a build script seems heavyweight solely for the purpose of making live scripts viewable/linkable.
Just out of curiosity, heavyweight in what sense?

Wrt the File Exchange rendered versions, they are linkable in my hands, by using "Copy Link Address" from the browser. Can you see this on a second look?

Yes I see it now. Then I would suggest to update the links for the FileExchange badges in the readme tables point directly to the tutorials and not the main FileExchange page for the BOT

Yes I see it now:) Then I would suggest to update the links for the FileExchange badges in the readme tables to point directly to the tutorials and not the main FileExchange page for the BOT

I encourage that update!

Just out of curiosity, heavyweight in what sense?
There's a class of user for whom GitHub is second nature but GitHub Actions is next level. I'm a bit queasy if rendering (which FX provides) is the reason they have to go through that transition. Imagine the experimental scientist (not a toolbox author) publishing the code for their figure publication on GitHub as part of an open science practice. I'd like to encourage them to use a live script to make it expository, without requiring them to learn/use GitHub Actions to make it broadly visible.

Regarding exporting live scripts in GitHub Actions: The export function requires the machine have a display but GitHub-hosted Linux runners do not provide a display by default.
You can set up a virtual display on GitHub-hosted Linux runners by using Xvfb display server to enable functions like export to work. Please see the example below:
name: Use Virtual Display on Linux Runner
on: [push]
jobs:
  my-job:
    name: Test an App
    runs-on: ubuntu-latest
    steps:
      - name: Check out repository
        uses: actions/checkout@v4
      - name: Start virtual display server
        if: runner.os == 'Linux'
        run: |
          sudo apt-get install -y xvfb
          Xvfb :99 &
          echo "DISPLAY=:99" >> $GITHUB_ENV
      - name: Set up MATLAB
        uses: matlab-actions/setup-matlab@v2
      - name: Run MATLAB
         uses: matlab-actions/run-command@v2
         with:
           command: export("sample.mlx");

@ehennestad Can you link me to the full workflow YAML and the job log with the error?

For the record, the solution posted above works, but I had not set the "DISPLAY" environment variable correctly. It is important that "DISPLAY" is set before using the run-command step

Ty @ehennestad for the update and the insight into a subtle failure mode.
Please keep world posted (e.g., on this Issue) if you get a build script going. This would be a good story to share with the campfire cohort.


refactor(variables): update CreateVariableOverlay to use routes

Closes #13243
This PR changes the CreateVariableOverlay to use React Router rather than open/close state in the parent component.

[x] Rebased/mergeable
[ ] Tests pass

I think the failing e2e test is in master


Appearance Opacity Slider

I'm always frustrated when I can't automate the opacity controller and am forced to manually adjust opacity by dragging the control points on the green line with the mouse for individual images, then have to eyeball the values as opposed to inputting exact values of say 90.000 instead of a general 89.972 to 90.156.
I want to be able to save opacity settings the same way I can save transfer function colors so that when I open a new dataset, I can load up an opacity that shows me a preset I've made.
As a secondary quality of life change, being able to change the coordinate numbers and the "Scale Opacity Values" slider by inputting numbers into a box similar to the Min and Max values below the opacity controller would also improve usability.
I want to be able to use this to easily apply opacity changes to multiple storm datasets. I think that this change can affect all datasets
Additional Reference Screenshot:

Fixed


Spatial Ctrl BBox + Pipe Inserter

Something is messed up when the pipe inserter transforms a spatial ctrl bbox.  Looks like the return of the bbox gets moved and there is an inserted pipe that comes after it.

Turns out it is because I accidentally constructed the BlackboxImpl inside the accel which is wrong and now it throws an error if you try to do this.


Should organizations be bundled in contributors

The Data Package property contributors allows to list organizations. I wonder if we shouldn't merge our custom term organizations with contributors, and maybe assign these a special role like publisher?

We would say:

Organizations related to the Data Package can also be included in contributors, for example with role publisher.


Add postcodes to practices in analyse search

From user feedback

Would be easier to find practice in the analyse tool if they had postcode associated in brackets in the search rather than practice code

Another request to search by ODS code. Some people are more familiar/use ODS code as shorthand for their organisation (99p anyone?!).
Also there are situations when ODS is simply the best way to identify due to practices being co-located and having the same name like the two  Chiswich Family Practice
https://openprescribing.net/practice/E85075/
https://openprescribing.net/practice/E85130/


compilation failure in swift-png using Xcode 16 beta 1

I was popping back into working on swift-noise, and opened up the project under Xcode 16 beta 1 to see how it would do. That project is using version 4.2.2 of swift-png, and while it continues to compile fine with Xcode 15, it fails to compile under Xcode 16 beta 1.
I captured a screenshot of the issue in Xcode 16b1, which seems to be an issue with destructuring un-named tuples into a named variant:

known issue with swift, fixed upstream but not yet available with beta1


bug: Node Search doesn't grep whole words

The feature that searches for nodes in the AST breaks after the first letter.
Steps to reproduce

When i type F it greps for FunctionDecl

But when I type FU, it doesn't.

can i get assigned?

!assign @kdb04 120


Recursively Custom commands calling

Required Information

Operating system: Ubuntu 17.10
PHP version: 7.0.1
PHP Telegram Bot version:^0.52.0
Using MySQL database:  no
MySQL version:
Update Method: Webhook
Self-signed certificate: yes
RAW update (if available):

Expected behaviour

Created custom commands as per documentation, whenever it calls it should reply back
Actual behaviour

When custom command called, the command response recursively. I need to stop this. I need response once not many time
Steps to reproduce

Bot Username - @lalaworld_bot
Extra details

Your webserver might be replying to webhook with non-200 reply, your code might be returning PHP warning or error - see your webserver logs to check whenever your answers to Telegram are http code 200.

Can you please share request format to check on postman by hitting api.

I have checked neither error has created nor warnings in web server logs.

@sachin-lalaworld Is this recursion still happening to you?
Sometimes it can happen that Telegram has server issues, which pass by.

Hello, the same problem is happening to me today and I do not know how to stop it

@JONAGUIPER can you share your code? Also, do you use request logging?


Add comment to list when creating a new comment

Description
When Publishing a new comment, it was not appearing on the comment list without a page refresh.
This PR adds reactive comments and the new ErrorPage message to the Wallet list (happen to have stumbled upon it)
Rebased from feat/improve-error-view/81
Screenshots
Comment is added after publish:
https://github.com/paideiadao/paideia-app/assets/2249046/dd7264a6-0549-4d36-941c-a641b0a392c0
If there's a network error we remove it:
https://github.com/paideiadao/paideia-app/assets/2249046/25488f06-5070-4f6a-84da-e38a05308ae8
Error page on Wallet list if address is invalid:

@JoaoFGuiomar were you working on the main branch? We have been working on beta branch for quite some time. I'm hoping we can eliminate any merge conflicts without too much trouble. I apologize for any time wasted, we were trying to keep bounties short partly to avoid any conflicts

@JoaoFGuiomar were you working on the main branch? We have been working on beta branch for quite some time. I'm hoping we can eliminate any merge conflicts without too much trouble. I apologize for any time wasted, we were trying to keep bounties short partly to avoid any conflicts

I was. I can try to rebase and see how many conflicts appear and see if its manageable.

@esot321c also updated this branch
Its rebased from feat/improve-error-view/81 which is rebase from beta
When feat/improve-error-view/81 is merged, this PR will have a lot less changes. This is a small fix

Hi @JoaoFGuiomar, we don't have the manpower to review a PR this big at the moment. Do you mind separating each update into a separate PR, and only linting the files you touched for that update? That way our team can review it properly. At the moment it will take us more time to review the PR than to actually do the bounties, which is counterproductive to the current goal of getting Paideia shipped to public beta. I hope you understand, we definitely appreciate the help!


@conda-forge-admin,please add bot automerge

Additional comment:
@conda-forge-admin,please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/dtools-feedstock#7.


Can't run on Android simulator correctly.

Hi, when I try to run renderdoc on Android simulator(On PC，the CPU structure is x86?), the renderdoc says can't find the abis.so I want to compile from the source code.
So, my question is that can work? If works, where to find librenderdoccmd.so and libVkLayer_GLES_RenderDoc.so(x86) ?
thank you.

Hi there, your issue has been closed due to not correctly following the issue template. Please read the documentation about filing issues and open a new issue which does follow the issue template. Do not reply to this issue with more information or with a correctly formatted issue, as that will only lead to confusion.
I try to address all github issues and fix bugs quickly, but badly formatted issues or poorly described bug reports especially are a significant drain on project resources. Taking a few minutes to file a good issue makes a huge difference and will get it addressed faster - everybody wins!
In github when you first create a new issue the form is pre-populated with a template which you should read over and follow. As linked above there is documentation going into more detail about what makes a good issue.


changes in EMAIL button UI on home page

Made changes in the Email button of the Home page.
[However , GITHUB is showing major changes , maybe because i have formated the code once to look good. Actual changes are made in few classes . which ONLY affect the Email button]
-> Added Arrow icon near "Getting Started text"
-> "Getting started" text was'nt capitalized on the original netflix website. So corrected that.
-> Colour of button chnages on Hover  as in original netflix site.
-> a bit of border radius to the button.

host it and put it here in the comment

host it and put it here in the comment

https://zealous-brown-dbc4b5.netlify.app/

resolve conflicts and add translating transition to arrow in the button

Have resolved the conflicts. Have a look @ahmadsohail404

resolve conflicts and add translating transition to arrow in the button

here is the updated Link https://zealous-brown-dbc4b5.netlify.app/

hey there @ahmadsohail404 . i have resolved all the conflicts. you can check preview here https://zealous-brown-dbc4b5.netlify.app/

![Uploading Screenshot 2021-10-03 at 10.31.56 PM.png…]()
Fix them

hey there @ahmadsohail404. Those were fixed already. I just had'nt updated the netlify site. Nevermind, I've done it now. Have a look https://zealous-brown-dbc4b5.netlify.app/

I think the white and red bars aint aligned properly

dont forget to pull the latest changes

Okay @ahmadsohail404  I've fixed that and pulled the latest changes in too. https://zealous-brown-dbc4b5.netlify.app/

dont forget to pull the latest changes

have a look once

@ahmadsohail404 check once please

Hey @ahmadsohail404 , I have made the changes and also pulled the updates as of 12:30 pm. Please have a look.

index.html is the conflicting file


Support for branches containing capital letters and special characters

Evidence with A/B tests enabled: https://github.com/coiled/coiled-runtime/actions/runs/3030036750

Ready for review and merge


json_name on protobuf message fields not respected

As an example:
https://github.com/akuity/kargo/blob/9097a9efafe14c135526508e532e4aeda6f91ceb/api/service/v1alpha1/service.proto#L381
https://github.com/akuity/kargo/blob/9097a9efafe14c135526508e532e4aeda6f91ceb/pkg/api/service/v1alpha1/service.pb.go#L3111
@devholic you are probably the best person to investigate this.

@krancour
This is because golang protobuf package depends on json field in protobuf struct tag for the backward compatibility (https://github.com/golang/protobuf/issues/1307#issuecomment-809811088)
Using google.golang.org/protobuf/encoding/protojson instead of encoding/json will fix this issue and also can prevent potential failure while encoding well-known types (e.g. google.protobuf.Timestamp).:
https://github.com/akuity/kargo/blob/c6d0a6d600a3a3a5e8b9d04816b60fb8d2fd4b4c/internal/cli/cmd/version/version.go#L143
(Kubernetes types (Kargo API types) should keep use encoding/json since they depend on gogo/protobuf features)

Thanks @devholic. I've put more thought into this and realize we really shouldn't be hung up over field names in protobuf messages anyway, as they're never really supposed to be seen. So... I went and fixed the problem that was making these snake_case fields bleed through in the CLI. See #1917.
I think we can close this.


Have an https server to run the demos

Let's have a Docker based approach to be able to run the demos and test the website. We can use https and Http2 to simulate the real life situation of the keyguard

See https://github.com/nimiq/keyguard-next/pull/299


custom font-family for the labels

The developer can custom font family for the label by using labelFontFmily

I have released v0.0.11 which includes functionality to render custom labels


Discord.net 0.9.5 + Mono - An address incompatible with the requested protocol was used

The bot works very well on my Windows machine, but when I tried to migrate it to a Linux server, it built fine, but when I start it, I get the following exception:
Log 10/2/2016 7:19:48 PM: GET voice/regions: 401.86 ms
Log 10/2/2016 7:19:48 PM: GET gateway: 22.95 ms
Log 10/2/2016 7:19:48 PM: Login successful, gateway: wss://gateway.discord.gg/?encoding=json&v=4
Crashed with exception, restarting.
System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> System.Net.Sockets.SocketException: An address incompatible with the requested protocol was used
  at System.Net.Sockets.Socket..ctor (System.Net.Sockets.AddressFamily addressFamily, System.Net.Sockets.SocketType socketType, System.Net.Sockets.ProtocolType protocolType) [0x0005d] in <6c7c03f48a9747c8a644da4f63a34480>:0
  at System.Net.Sockets.Socket.ConnectAsync (System.Net.Sockets.SocketType socketType, System.Net.Sockets.ProtocolType protocolType, System.Net.Sockets.SocketAsyncEventArgs e) [0x0000b] in <6c7c03f48a9747c8a644da4f63a34480>:0
  at (wrapper managed-to-native) System.Reflection.MonoMethod:InternalInvoke (System.Reflection.MonoMethod,object,object[],System.Exception&)
  at System.Reflection.MonoMethod.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00038] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
   --- End of inner exception stack trace ---
  at System.Reflection.MonoMethod.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00053] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at System.Reflection.MethodBase.Invoke (System.Object obj, System.Object[] parameters) [0x00000] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at SuperSocket.ClientEngine.ConnectAsyncExtension.ConnectAsync (System.Net.EndPoint remoteEndPoint, SuperSocket.ClientEngine.ConnectedCallback callback, System.Object state) [0x00039] in <4ba236fbf29248a4985a92e6e3907e4a>:0
  at SuperSocket.ClientEngine.TcpClientSession.Connect () [0x00074] in <4ba236fbf29248a4985a92e6e3907e4a>:0
  at WebSocket4Net.WebSocket.Open () [0x0003a] in <4ba236fbf29248a4985a92e6e3907e4a>:0
  at Discord.Net.WebSockets.WS4NetEngine.Connect (System.String host, System.Threading.CancellationToken cancelToken) [0x000cc] in <599fbedd978e4fce934746a63a740a87>:0
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at Discord.Net.WebSockets.WebSocket+<BeginConnect>d__34.MoveNext () [0x003b7] in <599fbedd978e4fce934746a63a740a87>:0
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x0004e] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x0002e] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x0000b] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at System.Runtime.CompilerServices.ConfiguredTaskAwaitable+ConfiguredTaskAwaiter.GetResult () [0x00000] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at Discord.Net.WebSockets.GatewaySocket+<Connect>d__12.MoveNext () [0x0013f] in <599fbedd978e4fce934746a63a740a87>:0
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x0004e] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x0002e] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x0000b] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at System.Runtime.CompilerServices.ConfiguredTaskAwaitable+ConfiguredTaskAwaiter.GetResult () [0x00000] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at Discord.DiscordClient+<BeginConnect>d__78.MoveNext () [0x00351] in <599fbedd978e4fce934746a63a740a87>:0
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at Discord.DiscordClient+<BeginConnect>d__78.MoveNext () [0x005a6] in <599fbedd978e4fce934746a63a740a87>:0
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x0004e] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x0002e] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x0000b] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at System.Runtime.CompilerServices.ConfiguredTaskAwaitable+ConfiguredTaskAwaiter.GetResult () [0x00000] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at Discord.DiscordClient+<Connect>d__77.MoveNext () [0x0008b] in <599fbedd978e4fce934746a63a740a87>:0
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x0004e] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x0002e] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x0000b] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at System.Runtime.CompilerServices.TaskAwaiter.GetResult () [0x00000] in <dca3b561b8ad4f9fb10141d81b39ff45>:0
  at PlaneSculptorsDiscordBot.Program+<Main>c__AnonStorey7+<Main>c__async6.MoveNext () [0x00085] in <a353675f20fb49b8b7b05a5467a7fc40>:0

You can find source code of the bot here: https://github.com/mzabsky/webdrafter/tree/master/bot/PlaneSculptorsDiscordBot

Make sure you run certmgr -ssl https://discordapp.com and certmgr -ssl https://gateway.discord.gg otherwise mono wont connect to discord over ssl.

Mono is Stable <IP_ADDRESS>.
I did run the two certmgr commands. Before posting the issue.
I suspect the error has something to do with IPv6 (other google results for that exception suggest as much), but I'm not sure what can I do about it without drilling deep into DiscordNet (or even worse the communication library it uses).

Discord.Net does nothing that specifies networking configurations. If you're machine is specifying IPv6, it's something in your machine.

Exactly the same problem. Imported certs and using 4.6.1 mono.

@mzabsky can you try Eric's fix, and see if that resolves it?

I just started experiencing this issue too after performing an update, so that is the likely cause. I'm not entirely sure how to downgrade to 4.4.2 though.

Depends on your OS. You need to use the system's package manager to uninstall then reinstall the specific version. (I've never had to downgrade, so there might be a specific command/method for that, depending on your OS)

I can confirm my bot works after downgrading to Mono 4.4.2.
I assume this is not a proper solution, so I am not closing yet :)

We use a different networking setup in 1.0, could someone please check if this issue is present there as well?

Expiriencing same after upgrade from 4.2 to 4.6.1.
Connecting using WebSocket4Net to even ws:// not wss://  scheme and got this:
[DEBUG] <DMPCLIENT> 11-02 08:42:16: Starting ws://c1.XXXXXX.com/wsc/
[ERROR] <DMPCLIENT> 11-02 08:42:16: Can't start WSDMPClient - Exception has been thrown by the target of an invocation.,   at System.Reflection.MonoMethod.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x0004d] in /builddir/build/BUILD/mono-4.6.1/mcs/class/corlib/System.Reflection/MonoMethod.cs:313
  at System.Reflection.MethodBase.Invoke (System.Object obj, System.Object[] parameters) [0x00000] in /builddir/build/BUILD/mono-4.6.1/mcs/class/referencesource/mscorlib/system/reflection/methodbase.cs:229
  at SuperSocket.ClientEngine.ConnectAsyncExtension.ConnectAsync (System.Net.EndPoint remoteEndPoint, SuperSocket.ClientEngine.ConnectedCallback callback, System.Object state) [0x00039] in <4ba236fbf29248a4985a92e6e3907e4a>:0
  at SuperSocket.ClientEngine.TcpClientSession.Connect () [0x00074] in <4ba236fbf29248a4985a92e6e3907e4a>:0
  at WebSocket4Net.WebSocket.Open () [0x0003a] in <4ba236fbf29248a4985a92e6e3907e4a>:0
  at DMPUtils.WSDMPClient.OnStart () [0x00046] in <08c32b8c89594a5db0dd068de82c3c6d>:0
[ERROR] <DMPCLIENT> 11-02 08:42:16: InnerException - An address incompatible with the requested protocol was used,   at System.Net.Sockets.Socket..ctor (System.Net.Sockets.AddressFamily addressFamily, System.Net.Sockets.SocketType socketType, System.Net.Sockets.ProtocolType protocolType) [0x0005e] in /builddir/build/BUILD/mono-4.6.1/mcs/class/System/System.Net.Sockets/Socket.cs:212
  at System.Net.Sockets.Socket.ConnectAsync (System.Net.Sockets.SocketType socketType, System.Net.Sockets.ProtocolType protocolType, System.Net.Sockets.SocketAsyncEventArgs e) [0x00000] in /builddir/build/BUILD/mono-4.6.1/mcs/class/System/System.Net.Sockets/Socket.cs:1361
  at (wrapper managed-to-native) System.Reflection.MonoMethod:InternalInvoke (System.Reflection.MonoMethod,object,object[],System.Exception&)
  at System.Reflection.MonoMethod.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00038] in /builddir/build/BUILD/mono-4.6.1/mcs/class/corlib/System.Reflection/MonoMethod.cs:305

This is a live issue (see #442 ) and needs quick resolution.

@mcmonkey4eva Use mono 4.4.2 - people are reporting that downgrading fixes the issue.

It isn't really a big problem, especially when 0.9 isn't even being actively maintained anymore. If the problem exists in 1.0, then it may be addressed.

It isn't really a big problem, especially when 0.9 isn't even being actively maintained anymore. If the problem exists in 1.0, then it may be addressed.

It won't be recommended to use Mono, anyways, for 1.0. .NET Core works for most platforms, now, all but ARM, if you counter Docker Deployment, but even with ARM, that's scheduled for .NET Core this Quarter.

It won't be recommended to use Mono, anyways, for 1.0. .NET Core works for most platforms, now, all but ARM, if you counter Docker Deployment, but even with ARM, that's scheduled for .NET Core this Quarter.

...  I didn't realize there was a 1.0 available. Thanks, I'll see if I can get that one working...

...  I didn't realize there was a 1.0 available. Thanks, I'll see if I can get that one working...

... new question, how do I acquire Discord.NET 1.0? Nuget does not have it as the readme claims (nuget is at 0.9.6), and compiling it from .NET Core CLI just gives failure messages :(
Just a lot of
MSBUILD : error MSB1009: Project file does not exist.
Switch: src\Discord.Net
Microsoft (R) Build Engine version 15.1.458.808
Copyright (C) Microsoft Corporation. All rights reserved.

under dotnet: 1.0.0-preview5-004275 which is latest currently.

... new question, how do I acquire Discord.NET 1.0? Nuget does not have it as the readme claims (nuget is at 0.9.6), and compiling it from .NET Core CLI just gives failure messages :(
Just a lot of
MSBUILD : error MSB1009: Project file does not exist.
Switch: src\Discord.Net
Microsoft (R) Build Engine version 15.1.458.808
Copyright (C) Microsoft Corporation. All rights reserved.

under dotnet: 1.0.0-preview5-004275 which is latest currently.

Check the README in the dev branch. If you're still having issues, open up a new issue as 1.0 is unrelated to this issue.

Check the README in the dev branch. If you're still having issues, open up a new issue as 1.0 is unrelated to this issue.


Update selenium-standalone

grunt-webdriver doesn't work with Firefox 36.0 as it is incompatible with v2.44 of selenium.
Selenium 2.45 has been released which resolves this issue.
selenium-standalone has released v4.2.0 which uses this new version of selenium. There appears to be some migration necessary for grunt-webdriver to work with the new version (Its quite a jump v2.44.0 to v4.2.0).
Hopefully I will get a chance to look at this soon, putting an issue up for anyone else who is having the same issue.

+1

It doesn't work with phantomjs either. The API for selenium seems to have changed

+1

I will merge #53 in the next couple of days and will make couple of options from selenium-standalone available to the grunt interface, so the user can decide which selenium version should get installed


d3.interpolateHcl(a,b)(1) doesn't return 'b'

Apologies if I'm misunderstanding how these interpolators should work, but I would have assumed that the end point should always be reached when t = 1.
A call to:
d3.interpolateHcl('red', 'white')(1)
returns:
"rgb(255, 178, 123)"
when I would expect it to return:
"rgb(255, 255, 255)"
Please see https://observablehq.com/@akrawitz/d3-interpolatehcl for a demo.

https://observablehq.com/@d3/achromatic-interpolation

Thank you! That does provide a work around. However, note that I left a comment in that notebook about an inconsistency between the text and the code (or maybe I'm still not quite getting it).

The text was wrong and I’ve corrected it.


RHSTOR-6195: Bucket - list page

https://issues.redhat.com/browse/RHSTOR-6195
ToDo bucket sync button after merging: https://github.com/red-hat-storage/odf-console/pull/1591

no need for extra "bucket" directory here: /s3-browser/bucket/buckets-list-page/bucketsListPage.tsx... we can directly do packages/odf/components/s3-browser/buckets-list-page/bucketsListPage.tsx
Also, bucketsListPage.tsx should be BucketsListPage.tsx...
IIRC that's what we decided, cc @alfonsomthd to keep me honest.

@SanjalKatiyar yes, we finally agreed  on this:
    └── s3-browser/
        └── create-bucket/    (CRUD-action (use case) folders)
         ...

probably we can add a bit of margin to pagination icons, to make it aligned with search bar and refresh icon, wdyt ??

/test odf-console-e2e-aws

/lgtm


Installation issue on Google Colab

Hi,
Thanks for providing a Cuda optimized implementation. While building the lib I encountered an issue with "inf" at limits.cuh.
CUDA/limits.cuh(119): error: identifier "inf" is undefined

CUDA/limits.cuh(120): error: identifier "inf" is undefined

CUDA/limits.cuh(128): error: identifier "inf" is undefined

CUDA/limits.cuh(129): error: identifier "inf" is undefined

4 errors detected in the compilation of "CUDA/adapool_cuda_kernel.cu".
error: command '/usr/local/cuda/bin/nvcc' failed with exit status 1
Makefile:2: recipe for target 'install' failed
make: *** [install] Error 1

The following notebook provides more details with environment informations:
https://colab.research.google.com/drive/1T6Nxe2qbjKxXzo2IimFMYBn52qbthlZB?usp=sharing

Hi @okbalefthanded , can you try to change the lines to use C's +/- INFINITY micro.?

Hi @alexandrosstergiou , the cmath micro works fine.
the test passed.
Thanks.


RFC: add settings tutorial

Provide up to date and basic troubleshooting info for new users. New wiki page can be read here: https://github.com/Artefact2/ALVR/blob/master/wiki/Settings-tutorial.md
Not for merging yet as the "sync files" action is doing unwanted things.

Waiting for the page split as discussed on Discord.


Changed reference for AppForSharePointWebToolkit

According to guidance there shouldn't be a reference in packages to AppForSharePoint16WebToolkit instead AppForSharePointWebToolkit.
Read more here: https://github.com/OfficeDev/PnP/blob/master/Guidance/ConfigSpPHAppsForDistro.md

Thanks. These AppForSharePoint16WebToolkit package references keep coming back, nice catch :)


Upgrade to springboot 3

Upgraded to SpringBoot 3
AWS to 3.0.0-M3, added depenedency BOM and using autoconfiguration
SpringCloud to 2022.0.0-RC2 (latest available as of now)
Removed zalando, to have minimal changes to application introduced @ControllerAdvice
Adds ServiceTest
Using new sequence generator mappings

@sivaprasadreddy , #49 and #51 will lead to merge conflicts. So if #49 is merged first then 51 should be rebased and then merge.

Amazing work @rajadilipkolli

Thank you, I have fixed merge conflicts. Can you check and merge the PR


Change git clone command in README

GIT protocol might not work everywhere. HTTPS is guaranteed to work if the user can connect to the internet.

thank you for the suggestion!


make index page optional

The short story: it allows you to not generate index page and associated cover images, and use Documenter's sidebar for navigation.
Usage:
# themed version -- the old way
card_templates, card_assets = cardtheme("grid")
makedemos(pagedir, card_templates)
preview_demos(demo; theme="grid")

# themeless version -- this PR adds this new usage
makedemos(pagedir)
preview_demos(demo)

I call this themeless, or "Theme: None":
preview: https://johnnychen94.github.io/DemoCards.jl/previews/PR64/
Note how it is different to navigate "Theme: None" and the other two themes.

@timholy we could use this to rebuild the packages part of JuliaImages's doc, e.g., https://juliaimages.org/latest/pkgs/axes/ to better organize the contents, or rewrite them with Julia scripts via Literate.
closes #62
closes #27

You're really pushing this to new heights! I need to get serious about using this...


feature request: scrollBehavior save and restore scroll for designated  element.

we have a project, html document is 100% height,overflow hidden,the main scroll part is a child div not the html document.
currently scrollBehavior  save and restore scroll for document,can not designate a element.

Closing in favour of #1187


Deploy a mCCF

This PR shows an initial way of how to create a Managed CCF instance in our subscription. At the moment, this can run on request, or for every push to main.

Create mCCF
Setup the App and govern it
Test the banking sample
Destroy

We need to document our release strategy. However, it will allow us to discuss how to proceed with the Data Reconciliation Application.
Relates to #66

Sorry for the delay, I am reviewing it.


Cannot Merge (concat) Dense and Embedding Layers

from keras.models import Sequential
from keras.layers.core import Dense, Merge
from keras.layers.embeddings import Embedding
from keras.optimizers import SGD
DENSE_INPUT_SIZE=9
DENSE_OUTPUT_SIZE = 3
EMBEDDING_INPUT_SIZE = 22
EMBEDDING_OUTPUT_SIZE = 5
TOTAL_HIDDEN_OUTPUT_SIZE = DENSE_OUTPUT_SIZE + EMBEDDING_OUTPUT_SIZE
e = Sequential()
e.add(Embedding(EMBEDDING_INPUT_SIZE, EMBEDDING_OUTPUT_SIZE))
d = Sequential()
d.add(Dense(DENSE_INPUT_SIZE, DENSE_OUTPUT_SIZE))
model = Sequential()
model.add(Merge([e,d], mode='concat'))
model.add(Dense(TOTAL_HIDDEN_OUTPUT_SIZE, 1))
sgd = SGD()
model.compile(loss='mean_squared_error', optimizer=sgd)

TypeError                                 Traceback (most recent call last)
 in ()
23
24 sgd = SGD()
---> 25 model.compile(loss='mean_squared_error', optimizer=sgd)
/home/user/anaconda/lib/python2.7/site-packages/keras/models.pyc in compile(self, optimizer, loss, class_mode, theano_mode)
285         self.X_test = self.get_input(train=False)
286
--> 287         self.y_train = self.get_output(train=True)
288         self.y_test = self.get_output(train=False)
289
/home/user/anaconda/lib/python2.7/site-packages/keras/layers/containers.pyc in get_output(self, train)
49
50     def get_output(self, train=False):
---> 51         return self.layers[-1].get_output(train)
52
53     def set_input(self):
/home/user/anaconda/lib/python2.7/site-packages/keras/layers/core.pyc in get_output(self, train)
339
340     def get_output(self, train=False):
--> 341         X = self.get_input(train)
342         output = self.activation(T.dot(X, self.W) + self.b)
343         return output
/home/user/anaconda/lib/python2.7/site-packages/keras/layers/core.pyc in get_input(self, train)
29     def get_input(self, train=False):
30         if hasattr(self, 'previous'):
---> 31             return self.previous.get_output(train=train)
32         else:
33             return self.input
/home/user/anaconda/lib/python2.7/site-packages/keras/layers/core.pyc in get_output(self, train)
144         elif self.mode == 'concat':
145             inputs = [self.layers[i].get_output(train) for i in range(len(self.layers))]
--> 146             return T.concatenate(inputs, axis=-1)
147         else:
148             raise Exception('Unknown merge mode')
/home/user/anaconda/lib/python2.7/site-packages/Theano-0.7.0-py2.7.egg/theano/tensor/basic.pyc in concatenate(tensor_list, axis)
3860             "or a list, make sure you did not forget () or [] around "
3861             "arguments of concatenate.", tensor_list)
-> 3862     return join(axis, *tensor_list)
3863
3864
/home/user/anaconda/lib/python2.7/site-packages/Theano-0.7.0-py2.7.egg/theano/gof/op.pyc in call(self, *inputs, **kwargs)
507         """
508         return_list = kwargs.pop('return_list', False)
--> 509         node = self.make_node(*inputs, **kwargs)
510
511         if config.compute_test_value != 'off':
/home/user/anaconda/lib/python2.7/site-packages/Theano-0.7.0-py2.7.egg/theano/tensor/basic.pyc in make_node(self, *axis_and_tensors)
3510
3511         return self._make_node_internal(
-> 3512             axis, tensors, as_tensor_variable_args, output_maker)
3513
3514     def _make_node_internal(self, axis, tensors,
/home/user/anaconda/lib/python2.7/site-packages/Theano-0.7.0-py2.7.egg/theano/tensor/basic.pyc in _make_node_internal(self, axis, tensors, as_tensor_variable_args, output_maker)
3576         if not python_all([x.ndim == len(bcastable)
3577                            for x in as_tensor_variable_args[1:]]):
-> 3578             raise TypeError("Join() can only join tensors with the same "
3579                             "number of dimensions.")
3580
TypeError: Join() can only join tensors with the same number of dimensions.

Try upgrading Theano to the latest version (get it from the Github repo).

Just reinstalled via "sudo pip install --upgrade git+git://github.com/Theano/Theano.git"
Same error message

And just for the record, my Theano was already pretty recent.  I set up Ubuntu from scratch and installed Theano directly from git about 2 weeks ago.

As your probably realize, your post is completely unreadable, which is why the cause of the error was not immediately obvious. Looking at the first lines of your code, it seems you are attempting to merge the output of Embedding with the output of Dense. According to the documentation, the former is 3D while the latter is 2D, hence it cannot work.
If you know what you are doing, you can try to use a Reshape or a Flatten layer.

Sorry about that - I've never raised an issue on github before.  How can I improve the formatting?
Also, why is the output of the Embedding 3-d?  It's a mapping from sparse inputs to a vector - why isn't it 1-d or 2-d?

It is 3D because it is mapping sequences of integer indices to sequences of dense vectors. Sequences of vectors are 3D tensors. In the indices representation, the last dimension (features) is implicit (the indices!), hence it fits in a 2D tensor.
Here's a guide about Github markdown: https://help.github.com/articles/github-flavored-markdown/


Added wrapper methods for all console methods and fixed misc lint errors

What we've done in other projects is to configure lint to allow some methods (e.g. 'warn', 'error', etc.) that were intended to be left in place but to warn for others that were assumed to be temporary for debugging only (e.g. log). If the goal is to leave console.logs in place permanently, then I'd agree that the lint rule isn't serving much purpose.

I think this is still a good way to go.  The linter will still find console.logs added by devs as they are testing and will leave in those meant to run in production.

I'm going to close this PR and just turn off the warnings for console.logs in the lint config and put in another PR.


lookAt callback

It would be great if lookAt also accepts a callback that is invoked when the animation finished.

I just added this in<PHONE_NUMBER>db4a09dca8e1b39b1e6268084366d1.

Thanks! Could this also be added to loadScene(...) when the fade-effect finished?
This would be also really helpful! There is the load event, it fires when the scene loaded, but not necessarily when the scene finished fading?

I justed added a scenechangefadedone event for this in<PHONE_NUMBER>7b02fa2a8a8405f508f01da8beb52c.


AVRO-3251 AVDL files containing fields called "uuid" no longer compile.

Jira

[X] My PR addresses the following AVRO-3251 issue

Tests

[X] My PR updates the unit tests

Commits

[X] My commits all reference Jira issues in their subject lines.

Documentation

[X]  No new functionality. Only bug fixes!

Looks good, but incomplete.
There are 31 keywords in the IDL grammar, and only 25 of them are accepted as identifier.
Fields named idl, import, oneway, null, local_timestamp_ms or uuid are all ignored. This commit only fixes the last one.
To verify, see lines 235-264, and lines 1582-1607.

Thanks for the review!

Fields named idl, import, oneway, null, local_timestamp_ms or uuid are all ignored.

Are you sure about local_timestamp_ms ?
It is at https://github.com/apache/avro/blob/99547e0ee2e0a4a839253848d7b34bac7f5e4cea/lang/java/compiler/src/main/javacc/org/apache/avro/compiler/idl/idl.jj#L263 and https://github.com/apache/avro/blob/99547e0ee2e0a4a839253848d7b34bac7f5e4cea/lang/java/compiler/src/main/javacc/org/apache/avro/compiler/idl/idl.jj#L1605

Absolutely right Martin! I've done the comparison again, but now with sort&diff instead of by hand... there were only 5 omissions (including uuid), instead of 6.


Error during compilation on Release Linux Mint 18.1 Serena 64-bit

When i run this command : make -f minicargo.mk
I got this error :
output/libstd.hir.o.c:49611:9: error: ‘_ZRK$aLfn_1_$Rs__ZN3std9panicking9PanicInfo_$T0_as__ZN4core3ops5FnMut$aL$T1_$Rs__ZN3std9panicking9PanicInfo$aR$aRcall_mut’ undeclared here (not in a function)   (void*)_ZRK$aLfn_1_$Rs__ZN3std9panicking9PanicInfo_$T0_as__ZN4core3ops5FnMut$aL$T1_$Rs__ZN3std9panicking9PanicInfo$aR$aRcall_mut
My active rustup toolchain is : nightly-x86_64-unknown-linux-gnu (default)  rustc 1.27.0-nightly (8ff4b4206 2018-05-08)
My OS and kernel : Linux RomMint 4.4.0-116-generic #140-Ubuntu SMP Mon Feb 12 21:23:04 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
Thank you for your help

Hmm... works for me on both 17.3 and 18.3. What version of gcc do you have? and did you custom-download the rustc source to build (or did you use make RUSTCSRC?)
Other questions: What commit of mrustc are you building?

I currently test this project : https://github.com/emosenkis/esp-rs that execute both command make RUSTCSRC and make -f minicargo.mk.
I use gcc (Ubuntu 5.5.0-12ubuntu1~16.04) 5.5.0 20171010.
this is the last commit i get when i run git log --oneline : ba9b4c4 Docs

It looks like that script attempts to check out and use bffed50, but appears to do so in a way that doesn't change the current commit (as reported by git).
Failure confirmed with the commit used by esp-rs, current head works. Maybe open an issue with the esp-rs project requesting an upgrade?

Closing as this isn't a mrustc issue, it's esp-rs pinning to a non-functional revision (which failed CI - see https://travis-ci.org/thepowersgang/mrustc/builds/303911135)


Safely Set Project Environment Variables

Purpose
This PR introduces functionality to create new environment variables via the CMLBootstrapAPI without unsetting existing environment variables.
How it works

A new method has been added called .get_environment_variables() that retrieves project level env variables.
Another method has been added called .add_environment_variable() that first retrieves existing first gets existing project level vars, and then appends to new variables to them before submitting the API request.

@victordibia I think this should fix the issue we were running into with the Churn AMP. I've left the original create_environment_variable() method as it was and just added a new one with the modified behavior per Yuya's comment in the JIRA.
However, now that I think about it, it's probably safe to just overwrite the existing create_environment_variable() with the updated functionality because I can't think of a case where this would break any code that is already using that method....What are your thoughts?

@victordibia I think this should fix the issue we were running into with the Churn AMP. I've left the original create_environment_variable() method as it was and just added a new one with the modified behavior per Yuya's comment in the JIRA.
However, now that I think about it, it's probably safe to just overwrite the existing create_environment_variable() with the updated functionality because I can't think of a case where this would break any code that is already using that method....What are your thoughts?

This looks great @andrew! I think you can ovewrite create_environment_variable() . I also dont see any strong reasons not to.

This looks great @andrew! I think you can ovewrite create_environment_variable() . I also dont see any strong reasons not to.


Update about page link

can go here: https://github.com/goinvo/ListofDemands/wiki/About-List-of-Demands

Done in #43


Update project page with new docs server path

Dependent on #72

Done: https://cloud.spring.io/spring-cloud-open-service-broker/


Explore using ROOT for graph generation

Turning graph saving off makes ctbench-grapher-plot execution a lot shorter (a several seconds vs several minutes). Generating graphs using ROOT instead of Sciplot might be a good way to cut graph generation times by a significant factor.

ROOT doesn't support C++20 yet, closing this issue for now
https://github.com/root-project/root/issues/11587#issuecomment-1281150322


Add youtube gallery component

We need a new component to be able to showcase videos from our youtube channel.
We need to be able to read the data from a Youtube sheet in our workbook [1] and create a gallery of those videos.
When a user clicks on each gallery we need a pop up so that user can play the video while on the website.
We already have a "card layout" so it can be an elaboration of that.

References
[1] https://docs.google.com/spreadsheets/d/1-5LXO4Uoay0X9TX2WeaCmc7FF9jq7X1QHiJRB-5O08c/edit?resourcekey#gid=544201594

Do we need the YouTube JS API for this?


Launch Smoke test fails

Testing https://github.com/microsoft/vscode/issues/79731
Happens for Insider and Electron 6 exploration
 Launch
** Timeout!
Error: Active element not found. Current active element is 'HTML > BODY.windows.file-icons-enabled.vs-dark.vscode-theme-defaults-themes-dark_plus-json > DIV.monaco-workbench.windows.nopanel > DIV.monaco-grid-view > DIV.monaco-grid-branch-node > DIV.monaco-split-view2.vertical > DIV.split-view-container > DIV.split-view-view.visible > DIV.monaco-grid-branch-node > DIV.monaco-split-view2.horizontal > DIV.split-view-container > DIV.split-view-view.visible > DIV.monaco-grid-branch-node > DIV.monaco-split-view2.vertical > DIV.split-view-container > DIV.split-view-view.visible > DIV#workbench.parts.editor.part.editor.has-watermark > DIV.content > DIV.grid-view-container > DIV.monaco-grid-view > DIV.monaco-grid-branch-node > DIV.monaco-split-view2.vertical.separator-border > DIV.split-view-container > DIV.split-view-view.visible > DIV.editor-group-container.active > DIV.editor-container > DIV.editor-instance > DIV.monaco-scrollable-element > DIV'. Looking for .editor-instance[id="workbench.editor.walkThroughPart"] > div > div[tabIndex="0"]
    at g.<anonymous> (file:///C:/Users/dirkb/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1212:821)
    at Generator.next (<anonymous>)
    at file:///C:/Users/dirkb/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:35:566
    at new Promise (<anonymous>)
    at n (file:///C:/Users/dirkb/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:35:343)
    at g.isActiveElement (file:///C:/Users/dirkb/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1212:513)
    at l.call (file:///C:/Users/dirkb/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1217:653)
    at C.onPromise (file:///C:/Users/dirkb/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:339:361)
    at C.onRawMessage (file:///C:/Users/dirkb/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:338:1012)
    at C.protocolListener.protocol.onMessage.e (file:///C:/Users/dirkb/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:338:531)
    at d.fire (file:///C:/Users/dirkb/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:107:468)
    at EventEmitter.n (file:///C:/Users/dirkb/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:104:788)
    at EventEmitter.emit (events.js:182:13)
    3) verifies that application launches when user data directory has non-ascii characters

and
 1) Running Code
      "before all" hook:
    Error: Timeout: is active element '.editor-instance[id="workbench.editor.walkThroughPart"] > div > div[tabIndex="0"]' after 20 seconds.
     at C:\Users\dirkb\Projects\mseng\VSCode\vscode\test\smoke\out\vscode\code.js:175:23
     at Generator.next (<anonymous>)
     at fulfilled (out\vscode\code.js:8:58)

 2) Running Code
      "after all" hook:
    TypeError: Cannot read property 'stop' of undefined
     at Context.<anonymous> (out\main.js:242:28)
     at Generator.next (<anonymous>)
     at C:\Users\dirkb\Projects\mseng\VSCode\vscode\test\smoke\out\main.js:11:71
     at new Promise (<anonymous>)
     at __awaiter (out\main.js:7:12)
     at Context.<anonymous> (out\main.js:241:16)

 3) Launch
      verifies that application launches when user data directory has non-ascii characters:
    Error: Timeout: is active element '.editor-instance[id="workbench.editor.walkThroughPart"] > div > div[tabIndex="0"]' after 20 seconds.
     at C:\Users\dirkb\Projects\mseng\VSCode\vscode\test\smoke\out\vscode\code.js:175:23
     at Generator.next (<anonymous>)
     at fulfilled (out\vscode\code.js:8:58)

 4) "after all" hook:
    Error: Timeout of 60000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

It seems like the smoke test is not updated to the new grid-view (looking at the errors). Thus forwarding this to @sbatten
also fyi @deepak1556 as the author of the testplan

It seems like the app does not get properly intiailized here
Feels like some electron update caused this

I also saw this testing current insider.

@dbaeumer @sbatten @isidorn The error says:
    Error: Timeout: is active element '.editor-instance[id="workbench.editor.walkThroughPart"] > div > div[tabIndex="0"]' after 20 seconds.

So, it can't find that element in the DOM when the Walkthrough page is showing. I just tried that on dev and the test is indeed correct:

Started breaking since https://github.com/microsoft/vscode/commit/fc4fd26b1293fca68ecf41f35b5933fb9ff91e63 cc @bpasero

AH yeah, thanks!


React Native support

Adds the necessary scripts to build the C bindings for iOS and Android which the in turn only need to be dropped into iOS/Android projects to be used via Obj-C++ or Android's JNI.
For my specific use case I use them in op-sqlite.

Some CI checks are failing @ospfranco

Thanks for taking a look, however, I'm currently swamped with work and next week I'm going on vacation for several months. Feel free to take over this PR and modify it as you see fit:

The changes on build.rs are reversible
Encryption seems to be compiling sqlcipher (?) via cmake, which also includes OpenSSL flags, the problem the compilation step is hard to integrate with Android (and that's why I had to disable it). Someone else that knows the internals of libsql should give it a try.
The rest seem trivial

After all is done and said, you only need to follow the guide I left you on notion to produce a static lib and run that against the opsqlite sample app. Once the emulator starts and everything is green you are ready to go.

Thanks for taking a look, however, I'm currently swamped with work and next week I'm going on vacation for several months. Feel free to take over this PR and modify it as you see fit:

I'm not sure I will have capacity to take this PR over. Could you please finish it to the state that's mergable. Right now multiple CI checks are failing for it so it is impossible to accept such work and integrate it into the repository.

The changes on build.rs are reversible

What does that mean? They are not needed? The changes that are required by Android and iOS need to be merged and if they are not compatible with other targets then they should be guarded with android or ios feature.

Encryption seems to be compiling sqlcipher (?) via cmake, which also includes OpenSSL flags, the problem the compilation step is hard to integrate with Android (and that's why I had to disable it). Someone else that knows the internals of libsql should give it a try.

It's ok to disable the encryption for Android and maybe iOS for now but it should be done only for that targets not for all targets which breaks them.

The rest seem trivial

After all is done and said, you only need to follow the guide I left you on notion to produce a static lib and run that against the opsqlite sample app. Once the emulator starts and everything is green you are ready to go.

Could you paste a link to that guide please? I haven't seen it yet.

P.S. if you do turn on encryption, it will require a modification to the bindings to send the encryption key from JS, so, open a PR when the time comes and I will take a look.

Leaving the guide to submitting updates to op-sqlite here for future reference:
https://discord.com/channels/@me/1243157733574840351/1255132819668209724


injector error for facebook connect plugin

I am trying to inject $cordovaFacebookProvider into my config of my app but I am getting an injector error. Is there a new name I am supposed to use?

I think the new name is $cordovaFacebook @Duovili.


Could you try to clarify the process of training the model with coco?

thx.

Hi chenyanyin, it's been a while since I did anything with this, and most of the stuff I added in this repo you probably do not want.
I think training from here: https://github.com/TheLaurens/ThesisProject-SCFasterRCNN9000/tree/1f0a9b4fc380ebd9253b775d1ee06634117b9e72
should allow you to train faster rcnn on ms coco.
If your goal is object detection you should probably just use this:
https://github.com/fizyr/keras-retinanet/tree/master/keras_retinanet

ok, i will try to it. Thank you for your quick reply.

Hi chenyanyin, it's been a while since I did anything with this, and most of the stuff I added in this repo you probably do not want.
I think training from here: https://github.com/TheLaurens/ThesisProject-SCFasterRCNN9000/tree/1f0a9b4fc380ebd9253b775d1ee06634117b9e72
should allow you to train faster rcnn on ms coco.
If your goal is object detection you should probably just use this:
https://github.com/fizyr/keras-retinanet/tree/master/keras_retinanet

hello, i have tried retinanet on my dataset, but I found that accuracy is not as good as faster rcnn. The reason you recommend it to me is because it is fast?


Block Matrix / Block Diagonal Matrix

Please make sure that this is a feature request. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:feature_template
System information

TensorFlow version (you are using): python 1.12.0
Are you willing to contribute it (Yes/No): I would but I have very little knowledge of the TF internals

Describe the feature and the current behavior/state.
I am requesting the development of a block diagonal function for tensors. In scipy this is included in the scipy.linalg.block_diag function. Converted to tensors, it makes sense that given a [M, K, X, ...] tensor, when calling the proposed tf.block_diag it would return a [M*K, M*K, X, ...] matrix with the diagonal elements containing the elements from the [K, X, ...] submatrix. This seems to be implemented as a LinearOperator but it is not conducive to the tensor-level functions that exist for the rest of the linear algebra domain.
Will this change the current api? How?
No, this should not change current api, only add an extra function
Who will benefit with this feature?
I will benefit from this feature, as well as anyone else trying to convert code from numpy/scipy to TF. It is a seemingly standard operator in linear algebra and so it makes sense for it to exist in Tensorflow.
Any Other info.
I have a cludgy working version that uses padding and concatenation
Given Tensor input of shape (B, K, K, ...) will return a Tensor of shape (K*B, K*B) zero filled except for the KxK diagonal elements
import tensorflow as tf
B, K, = input.shape[:2]
paddings = tf.constant([[0, (B-1)*K)],[0, 0]])
diag = tf.pad(input[0], paddings)
for i in range(1, B):
    paddings = tf.constant([[i*K, (B-i-1)*K],[0, 0]])
    zeros = tf.pad(input[i], paddings)
    diag = tf.concat([diag, zeros], 1)

@mileslucas  Please fill this template for a new feature request.

@harshini-gadige I have edited my comment to fit new template, thank you.

+1

bump

@mileslucas,
Can you please refer the documentation of tf.linalg.LinearOperatorBlockDiag and check if this is what you are looking for? Thanks!

If I am not mistaken, unlike scipy.linalg.block_diag, tf.linalg.LinearOperatorBlockDiag does not support non-square sub-matrices.

Okay, if you want to close the issue that's fine but it's clearly not resolved. It's been 3 years since I opened this and I am no longer using TF or working on the same project (we settled on numpy/scipy for all the math) so I don't want to be the lynch pin for this feature.

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.
Thank you!


Add logic to use different install command with tools version 10

The method of piping echo 'yes' into the installer does not work on
version 10 of the installer.  Instead, supply the '-f' flag to the
installer, which was added in 10 to force an install.

Had erroneously closed.  Re-opening.


Update binlog.json

just copy from 3.0
mainly for https://github.com/pingcap/tidb-ansible/pull/604 the missing
two dashboard in 2.1

@liubo0127 PTAL


Change "ports" to "interfaces" in REST API

Will not be done to avoid confusion with OpenFlow spec that uses "port".


rolled logs don't end with .log

Right now when the ziti-tunnel.log file rolls, it doesn't roll over with a .log ending making it 'annoying' to open in windows. add the .log

will be fixed in https://github.com/openziti/desktop-edge-win/pull/74/files


elementary tests: add support for dbt "where" parameter to the tests

Looks good to me.
I think you can just add where: 1=1 config to one test of each type in the schema.yml and that will give us confidence.
Also we should deprecate where_expression and just guide users to use where. No?


fix(controller): use double quotes to escape ENV values

This fixes an issue with Docker's build system.
The following does not work:
><> cat Dockerfile
FROM busybox
ENV POWERED_BY='can\'t compute'
><> docker build .
Sending build context to Docker daemon 109.6 kB
Error response from daemon: Syntax error - can't find = in "compute'". Must be of the form: name=value

However, subbing the single quotes for double quotes works:
><> vim Dockerfile
><> docker build .
Sending build context to Docker daemon 109.6 kB
Step 1 : FROM busybox
 ---> 307ac631f1b5
Step 2 : ENV POWERED_BY "can\"t compute"
 ---> Using cache
 ---> 7b505fc9ce12
Successfully built 7b505fc9ce12
><> docker run 7b505fc9ce12 env
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
HOSTNAME=c4d9a9278acd
POWERED_BY=can"t compute
HOME=/root

fixes #4796

note that this isn't an issue on v2 since we're no longer relying on this logic to inject envvars any more.

Also for completeness, single quote in the double quotes:
><> docker build .
Sending build context to Docker daemon 109.6 kB
Step 1 : FROM busybox
 ---> 307ac631f1b5
Step 2 : ENV POWERED_BY "can't compute"
 ---> Running in 3df68f456364
 ---> 6df7c159a84d
Removing intermediate container 3df68f456364
Successfully built 6df7c159a84d

I tested this with a few config:set variations and can't find any problems. Nice work!
$ deis config
=== classy-question Config
POWERED_BY      can't compute
$ echo "MAILER_SUBJECT=ISP cron can't create booklet" > .env
$ deis config:push
Creating config... done, v5

=== classy-question Config
MAILER_SUBJECT      ISP cron can't create booklet
POWERED_BY          can't compute


Scale of reconstruction is off when no GPS information is available (but GCP is)

Hello :hand:
While trying to process a set of images flown from a kite with no GPS information but with a corresponding GCP file I've noticed that the output reconstruction is in miniature size.
I think this is normal in absence of georeferencing information (no GPS, no GCP), but the GCP has some information about distances, so it might be possible to use it to set a proper scale?
To reproduce:

Use the images in https://github.com/OpenDroneMap/odm_data_copr
Use the following gcp_list.txt file:

WGS84 UTM 11N
235277.61	3811190.36	0	3483.00	729.00	IMG_0031.jpg
235281.01	3811195.14	0	731.00	1017.00	IMG_0031.jpg
235269.88	3811198.11	0	3608.00	2292.00	IMG_0037.jpg
235269.88	3811198.11	0	586.00	1028.00	IMG_0121.jpg
235269.89	3811203.16	0	2040.00	1017.00	IMG_0046.jpg
235262.54	3811203.50	0	3472.00	1002.00	IMG_0052.jpg
235264.49	3811213.70	0	1472.00	1110.00	IMG_0067.jpg
235258.91	3811213.60	0	3910.00	916.00	IMG_0073.jpg
235257.74	3811222.65	0	2020.00	2148.00	IMG_0073.jpg
235246.25	3811220.60	0	3570.00	2593.00	IMG_0082.jpg
235246.25	3811220.60	0	3564.00	1929.00	IMG_0085.jpg
235246.25	3811220.60	0	3407.00	995.00	IMG_0088.jpg
235248.03	3811227.25	0	1082.00	764.00	IMG_0091.jpg

And this config.yaml:
align_method: naive
bundle_use_gcp: true
bundle_use_gps: false
feature_min_frames: 8000
feature_process_size: 2048
matching_gps_distance: 0
matching_gps_neighbors: 8
optimize_camera_parameters: true
processes: 4

After the reconstruction and export_ply command the resulting PLY looks like:

The reconstruction actually looks great, it's just the scale that is off.

(Distance reads about ~5 meters)
I thought it might have been a local vs. world coordinates fault on my end, so I ran export_geocoords odm_data_copr/opensfm --proj "+proj=utm +zone=11 +ellps=WGS84 +datum=WGS84 +units=m +no_defs" --reconstruction and received:
Traceback (most recent call last):
  File "/code/SuperBuild/src/opensfm/bin/opensfm", line 34, in <module>
    command.run(args)
  File "/code/SuperBuild/src/opensfm/opensfm/commands/export_geocoords.py", line 77, in run
    self._transform_reconstruction(r, transformation)
  File "/code/SuperBuild/src/opensfm/opensfm/commands/export_geocoords.py", line 138, in _transform_reconstruction
    shot.pose.set_rotation_matrix(np.dot(R, A1))
  File "/code/SuperBuild/src/opensfm/opensfm/types.py", line 85, in set_rotation_matrix
    raise ValueError("Determinant not 1")
ValueError: Determinant not 1

(which might a different problem)
So I exported the transform with export_geocoords odm_data_copr/opensfm --proj "+proj=utm +zone=11 +ellps=WGS84 +datum=WGS84 +units=m +no_defs" --transformation and the scale values seem like they wouldn't be affected by it:
1.0000598981278017 0.028431922721210867 0.0<PHONE_NUMBER>1039622
-0.02843192033469677 1.0000599017366767 0.0 3811208.6790932035
7.82310962677002e-08 7.82310962677002e-08 0.9999999990686774 9.313225746154785e-10
0.0 0.0 0.0 1.0

I think this has been a problem in earlier version of OpenSfM (~1 year ago) as well.

(Distance reads about ~143 meters)
Whereas the actual distance should be roughly 60-65 meters.

I've tried to change align_method to orientation_prior and align_orientation_prior to vertical (but I'm pretty sure that skips the GCP alignment code), without success.
Looking for some pointers in the areas of the code that I should be focusing my attention to, or if this is a known limitation and/or possible ideas on how to fix it?
Thank you in advance.

Hi,
Running with current master, I end up with the same ValueError: Determinant not 1 as you had. This is because there is no EXIF in shots so a [0, 0, 0] value is used for all shots.
I rebased the branch feat-gcp-align on master we made last time for propagating use of GPS. Using it as is, I had slightly better results. After a couple of commits for generating proper topocentric frame, results are OK.
Made a PR here for feat-gcp-align, so we can push this when @paulinus is back . There is might be some discussion about use_gcp/use_gps and align behaviour, your input might be welcome.
For the align step, I'm curious of any registration any behaviour it would change (two last commits 42cd57c and f28f97a)
Yann

Thanks @YanNoun !! :clinking_glasses:
This is great. I will check the new branch changes in the next few days and re-run our test suite to compare results.
I think a person needs to be part of the organization (mapillary) to be added as a reviewer in PRs. It's a limitation of GitHub (I don't know why they don't let external contributors be reviewers).

This is now fixed! :clinking_glasses:


Split the retrieve testcases command

Codecov Report

Merging #3 into master will not change coverage.
The diff coverage is 0.00%.

@@          Coverage Diff           @@
##           master      #3   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files          16      19    +3
  Lines        2096    2192   +96
  Branches      979    1015   +36
======================================
- Misses       2086    2181   +95
- Partials       10      11    +1

Impacted Files
Coverage Δ

src/commands/open.rs
0.00% <0.00%> (ø)

src/commands/retrieve_testcases.rs
0.00% <0.00%> (ø)

src/commands/submit.rs
0.00% <0.00%> (ø)

src/lib.rs
0.00% <0.00%> (ø)

src/open.rs
0.00% <0.00%> (ø)

src/project.rs
0.00% <0.00%> (ø)

src/testing.rs
0.00% <0.00%> (ø)

src/web/mod.rs
0.00% <ø> (ø)

src/web/retrieve_testcases.rs
0.00% <0.00%> (ø)

... and 4 more

Continue to review full report at Codecov.

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

bors r+


On "fatal errors" stop serving files

When developing, if I make a syntax error and am using qx serve there is no indication in the browser that the compilation failed.
Would it be possible to instead print an error in the browser this way if the developer is not closely monitoring the console where the qx serve is running gets an earlier indication that their compile failed?

i completely agree; I would also like it if there are just warnings, those warnings are picked up and output into the console during app startup


fix parameter error

PR checklist:

Did you run ClangFormat ?
Did you separate headers to a different section in existing community code base ?
Did you surround proton: starts/ends for new code in existing community code base ?

Please write user-readable short description of the changes:

add a test? either in stateless or in smoke.


Fix installation script for RHEL

the previous fix changed the lsb_dist variable too early. We only need to normalize to "centos" for the repository-location, so changing it just before that.
/cc @kencochrane @mlaventure
fixes https://github.com/docker/docker/issues/23787

LGTM, thanks for tracking that down.

Spoke to soon, saw @Purple90 comment
Oh, its because you need brackets around lsb_dist
So change it to this

            if [ "${lsb_dist}" = "redhat" ]; then
                # we the centos repository for both redhat and centos releases
                lsb_dist='centos'
            fi

thanks @kencochrane @Purple90, updated my PR, PTAL

I still have the same error as reported in #23787

@tiborvass the original error, or the one that I introduced?

@tiborvass that's on RHEL? Is there an active subscription?

@thaJeztah oh that must be why lol

I tested an upgrade on RHEL7.2 and things work well.
[ec2-user@ip-172-31-39-24 yum.repos.d]$ curl https://raw.githubusercontent.com/thaJeztah/docker/73d5f0648e430a74cd05ff9fd335611ddc6dd2a1/hack/install.sh | sh
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100 14604  100 14604    0     0  22218      0 --:--:-- --:--:-- --:--:-- 22228
Warning: the "docker" command appears to already exist on this system.
If you already have Docker installed, this script can cause trouble, which is
why we're displaying this warning and provide the opportunity to cancel the
installation.
If you installed the current Docker package using this script and are using it
again to update Docker, you can safely ignore this message.
You may press Ctrl+C now to abort this script.

sleep 20
sudo -E sh -c 'sleep 3; yum -y -q install docker-engine'
Package docker-engine-1.12.0-0.2.rc2.el7.centos.x86_64 already installed and latest version
sudo -E sh -c 'docker version'
Client:
Version:      1.12.0-rc2
API version:  1.24
Go version:   go1.6.2
Git commit:   906eacd
Built:
OS/Arch:      linux/amd64
Cannot connect to the Docker daemon. Is the docker daemon running on this host?

If you would like to use Docker as a non-root user, you should now consider
adding your user to the "docker" group with something like:
sudo usermod -aG docker ec2-user
Remember that you will have to log out and back in for this to take effect!

LGTM

👍


Minimise instead of shaking

Dear Jonathan,
Once again thank you for this amazing work.
Do you think there is a way when pressing CLOSE to minimise the app to just the 3 buttons so that the screen is interactive and when pressing a button it comes back up instead of completely going away and then having to shake the device to return?
Best regards,

The screen is only interactive when the entire view / layout has setVisibility set to GONE
board.setVisibility(View.GONE);
Because of that detail, I had to improvise a way to hide the view in order to use the game's features
If I find a better solution for this, I will update the project
But before that, there will be new updates

Cool thanks. What updates are you thinking about?

@Redmanskoop

Translation ( pt-BR, en-US )
Second guide line in Mode: Table
Maybe ... a solution for the app to fit any screen size

The second guidline is actually a very good idea.

@Redmanskoop
I'm going to post an update today
Now the layout adjustments will be made in just one .xml file
I couldn't get the layout to adapt automatically
In the future I will add the second guide line


add initial content and scripts

add scripts to do load testing for POST requests with a same url but different JSON payload

looks good, ship it!

:shipit:


Using continue in else block of for else

I am not sure for else is fully supported, but here is the one I noticed while fixing re bugs.
for i in range(2):
  print 'i', i
  for j in range(2):
    print 'i', i, "j", j
  else:
    print "continue", i, j
    continue

CPython
i 0
i 0 j 0
i 0 j 1
continue 0 1
i 1
i 1 j 0
i 1 j 1
continue 1 1

grumpy
i 0
i 0 j 0
i 0 j 1
continue 0 1
TypeError: int object is not an iterator
exit status 1

May be else's scope is bind to inner loop, not outer loop?

Good find! It never occurred to me that a continue could appear in an else clause :\
It seems like the correct behavior is that continue applies to the outer loop because if you continue in a non-nested loop you get:
>>> for j in range(2):
...   print "j", j
... else:
...  print "continue", j
...  continue
...
  File "<stdin>", line 5
SyntaxError: 'continue' not properly in loop

I never used too but, that re module use it in awkward way, and outer loop is while True.


Fail to Install UTM on iPadOS 13 or iOS 12.4

AltServer:
Could not save ***.dylib in Framework folder.

Xcode with re-signed ipa:
Unable to install.

Xcode compiling source code:
Build failed: Could not find UTM/sysroot*/***.cpp.

I've tried with a free developer account and a paid account.
I'd appreciate any clues or tips on installing.

If you have a paid account, then generate a provisioning profile, use iOS App Signer to sign the app with that profile, and install the profile and the app with Apple Configurator. This process works 100%.

If you have a paid account, then generate a provisioning profile, use iOS App Signer to sign the app with that profile, and install the profile and the app with Apple Configurator. This process works 100%.

Thanks a lot! It finally worked, with the profile, App Signer, and Apple Configurator.


Refactor loader to decoupe from progressbar

#87
@NicolasRannou hi, I have refactored loader by decouping from progressbar.
Now loader.base extends event-emitter, and emits events at special hooks.
However, the current work need your review and suggestion, I don't want to merge this PR immediately, Let's base current work, and have more discussion. 😄
Example at fiddle

Sounds good!
Couple of comments:

Do we need to include events in the package.json as it is part of npm? (https://nodejs.org/api/events.html#events_events)
Just setting the npm minium version should be enough right?

In the event-progessbar bar, would it make sense to also accept "ID" as "Dom Container"?
Renaming "DomContainer" to "DomTarget" and if it "DomTarget" is a string, get the related to element. If "DomTarget" is already a "Dom ellement", just pass it to the "DomContainer."

Rename curTime to date or time.

parseing  should be parsing

It looks good in general I'm really happy, will let you know when more comments after having deeper look!

I think we should make the event name consistent with the XMLHTTPREquest Standards (https://developer.mozilla.org/en-US/docs/Web/API/XMLHttpRequest)

begin-fetch -> fetch-start
fetching -> fetch-progress
fetch-success -> fetch
and also something like
fetch-end would be good, similar to https://developer.mozilla.org/en-US/docs/Web/Events/loadend
May not be too pretty but might be easier for people to use. WDYT?

@NicolasRannou thank you for advise.
yes, event is a inner package of node.js, i will remove it from the package.json. i'm glad to adhere to any standard 😄 , and give a more beautiful progress bar.

[ ] remove event-emitter from package.json
[ ] fix the event name according to xml standard
[ ] adjust the selection logic of dom container
[ ] do a more beautiful progress bar (oh. css is my weakness 😢, though i always want to get it )

Thanks! Do not worry about the beautiful progress bar, I'm sure we can get something very cool.
Another thing that would be very useful (I know it is a lot to ask but we are now in the process of making AMI robust) would be to add testing to test all functions in the loader...
It implies:
1- Creating specs files such as: https://github.com/FNNDSC/ami/blob/dev/specs/models/models.base.spec.js
2- Test all functions (test for success and failure)
During development, I usually modify https://github.com/FNNDSC/ami/blob/dev/karma.conf.js#L16 and https://github.com/FNNDSC/ami/blob/dev/karma.conf.js#L27 to only run tests over the part I am working on.
Then npm run test.
WDYT?
Thanks!

@NicolasRannou Yes, Testing is necessary, I will give enough test case for loader. But what sample data i should use ? maybe files at /data/dicom/ , i find settings at karma.conf.js?

Yes exactly those files, that should be enough! Thanks!

@NicolasRannou ok! Let me have a try~ 😄

now the fetch events as follow(according to XMLHttpRequest Draft):
loadstart => fetch-start
progress => fetch-progress
error => fetch-error
load => fetch-success [fetch success]
timeout => fetch-timeout
loadend => fetch-end  [The fetch completed (success or failure)]
abort => fetch-abort  [almost not appear here]

Cool! Sounds like you are getting there! :)
Sorry I keep adding things to the todo list but another thing I'd like is to change the name of the EventBasedProgressBar.
Ideally the filename and the class name would match, and I'd like to name it the other way around:
src/helpers/helpers.event_progressbar.js -> src/helpers/helpers.progressbar.eventbased.js
And the class name inside the helpers.progressbar.eventbased.js would be HelpersProgressBarEventBased.
The main reason for that is it allows all the progressbar helpers to be close alphabetically which is handy while development. Also it helps us know what the file does at first glance. Finally will be easy to group files together later on if that makes sense. (i.e. all progressbars in a progressbar directory)
WDYT?

@NicolasRannou just ok, i have written the tests, however need some helper, i will refresh the PR.

Looking really good @chenkaiC4 :)

@NicolasRannou thanks~
however, the event have not been test for now and test cases may not enough, suggestion may be well~  :)

phantomjs have been removed ?
16 03 2017 10:01:31.816:ERROR [plugin]: Error during loading "/home/travis/build/FNNDSC/ami/node_modules/karma-phantomjs-launcher" plugin:
  Path must be a string. Received null
16 03 2017 10:01:40.831:ERROR [karma]: Found 1 load error

Looking into the karma issue - I wanted to switch to yarn and didn't realize it would affect the pull request... on  it...

@NicolasRannou yarn is better than npm. Let's get it.
but, it seem the phantomjs not found.

Should be fixed now - re-building the pull request

Finally, in the specs/loaders/loaders.volume.spec.js, we should remove the .tar suffix for all files. Ia used to be required but it is not anymore, that will make testing much better :)
After that I think we are very close to merging - need to review it all more in depth.

just remove the .tar ?  this will cause 'Not Found'~

if change the source file name, some test case at pasrers.dicom.spec.js should be modify.

Yes it is fine you can just remove the .tar in loaders.volume.spec.js, I already modified things in parsers.dicom.spec.js. When you will update the pull request, the Travis build (which run the tests will pass)

https://github.com/FNNDSC/ami/commit/640cde3bb31f5a3c6d79a92c0208f7f85e4d9d3a

@NicolasRannou 🥂 , more suggestions alway welcome.
i will take them later for now i have to go home :)

@NicolasRannou Hi, I have test the events sent by loader module using sinon spy. :)

Thanks @chenkaiC4 - I'll test it to make sure all demo and examples work then merge it!

@NicolasRannou cheer, if any something i need to do, just tell me, :)

Added 2 minor comments @chenkaiC4 :)

Thanks @chenkaiC4 !


fix output order in sortedBy example

There is an error in example output of .sortedBy() :)

Whoops!


Hardcoded color in ViewFinderView?

Description of the problem:
PS: If this have any similar issue that can help me please just closed this issue
Hi I tried to customize the color attribute in DecoratedBarcodeView from the xml layout.
Even if I have the access to zxing_possible_result_points, zxing_viewfinder_laser and etc,
I can't seems to change the color even if I assign color code to it
Is there anyway I can change the color (Such as laser color, mask color and etc) without tempering with the library files?

I have the same problem , mask color & laser  attributes seem no useful

Laser's alpha value is being animated in values {0, 64, 128, 192, 255, 192, 128, 64}. See field SCANNER_ALPHA in ViewfinderView.
What's actually happens is this alpha value is "set" to transparent color (#00000000) and it changes to black (#FF000000).
The only workaround I can think is to change this SCANNER_ALPHA array values to have only one 0 value.
    private fun disableLaser(decoratedBarcodeView: DecoratedBarcodeView) {
        val scannerAlphaField = ViewfinderView::class.java.getDeclaredField("SCANNER_ALPHA")
        scannerAlphaField.isAccessible = true
        scannerAlphaField.set(decoratedBarcodeView.viewFinder, intArrayOf(0))
    }

Thanks for the pointer into the correct direction, I had to change the code a bit until it worked for me:
        barcodeView = (DecoratedBarcodeView) findViewById(R.id.barcode_scanner);
        barcodeView.decodeContinuous(callback);

        ViewfinderView viewFinder = barcodeView.getViewFinder();
        Field scannerAlphaField = null;
        try {
            scannerAlphaField = viewFinder.getClass().getDeclaredField("SCANNER_ALPHA");
            scannerAlphaField.setAccessible(true);
            scannerAlphaField.set(viewFinder, new int[1]);
        } catch (NoSuchFieldException e) {
            e.printStackTrace();
        } catch (IllegalAccessException e) {
            e.printStackTrace();
        }

I would like to work on this bug, any objection?

Pull requests for this would be appreciated.

Will do, can you assign the issue to me?

Thanks for the pointer into the correct direction, I had to change the code a bit until it worked for me:
        barcodeView = (DecoratedBarcodeView) findViewById(R.id.barcode_scanner);
        barcodeView.decodeContinuous(callback);

        ViewfinderView viewFinder = barcodeView.getViewFinder();
        Field scannerAlphaField = null;
        try {
            scannerAlphaField = viewFinder.getClass().getDeclaredField("SCANNER_ALPHA");
            scannerAlphaField.setAccessible(true);
            scannerAlphaField.set(viewFinder, new int[1]);
        } catch (NoSuchFieldException e) {
            e.printStackTrace();
        } catch (IllegalAccessException e) {
            e.printStackTrace();
        }

This worked for me, Thanks

Description of the problem:
PS: If this have any similar issue that can help me please just closed this issue
Hi I tried to customize the color attribute in DecoratedBarcodeView from the xml layout.
Even if I have the access to zxing_possible_result_points, zxing_viewfinder_laser and etc,
I can't seems to change the color even if I assign color code to it
Is there anyway I can change the color (Such as laser color, mask color and etc) without tempering with the library files?

I want to put border in the Barcode corners. How did you do?


Botframework-connector conversations missing members object id

Versions
botframework-connector 4.1.5
Describe the bug
9 days ago the function getActivityMembersWithHttpOperationResponse was removed from the botframework-connector conversations.ts and only function getActivityMembers was left.
(node:4568) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 2): TypeError: connectorClient.conversations.getConversationMembersWithHttpOperationResponse is not a function
Now there is no way to get member objectId which is returned by conversations API.
getActivityMembers only returns JSON with member id and member name.
How should I get member object id now?

This SDK is built using swagger generated clients around the ms-rest-js package.  The property you are talking about is not part of the Activity protocol, so I suspect that you are looking for an unpublished property of the raw payload.
The ms-rest-js library was changed so that the response is a polymorphic response that includes both the shape of the data as specified in the swagger file (in this case an array of channelAccount objects) as well as a property which represents the raw HTTP response, accessible off of a "_response" property.
_response.bodyText has the raw response of the request and
_response.parsedBody has the channelAccount[]


Vec doesn't serialize properly

You have to explicitly construct Bytes for it to work.
To make that work within data structures,  Bytes needs to be Clone.
I have made a PR for that here :
https://github.com/AlexPikalov/cdrs/pull/178
But we should figure out how to solve the broader problem that we need to treat Vec differently from Vec

Good catch! Currently I'm trying to fix Travis CI. Once it's done I'll merge your PR and we could close this issue.
Thanks, @rrichardson

As a solution for differentiating blob from Vec, there is a PR with Blob type that has similar purposes as Map, List, UDT.
#181


Reduce number of requested permissions

We ask for a lot of permissions at install time, and we've noted before that it's not necessary to do so, e.g. there are ways of accessing the calendar without needing to request those permissions on install.

Sorry, but, which way are you talking about to access calendar without using these permissions?
http://developer.android.com/guide/topics/providers/calendar-provider.html

@hasitpbhatt http://developer.android.com/guide/topics/providers/calendar-provider.html#intent-insert

Using the INSERT Intent lets your application hand off the event insertion task to the Calendar itself. With this approach, your application doesn't even need to have the WRITE_CALENDAR permission included in its manifest file.


New Cookie support doesn't set multiple cookies

Qwik Version
0.13.1
Operating System (or Browser)
Chrome
Node Version (if applicable)
No response
Which component is affected?
Qwik City
Expected Behaviour
I expect to be able to set/delete multiple cookies at once.
Actual Behaviour
It only sets the last one.
Additional Information
Here is a simple test.  Only "test3" will be set.
import { RequestHandler } from<EMAIL_ADDRESS>
export const onGet: RequestHandler = ({ cookie, response }) => {
  cookie.set("test1", "value3");
  cookie.set("test2", "value2");
  cookie.set("test3", "value3")
  return response;
}

idk how this is the case, implementation definively keeps all keys:
  set(
    cookieName: string,
    cookieValue: string | number | Record<string, any>,
    options: CookieOptions = {}
  ) {
    const resolvedValue =
      typeof cookieValue === 'string'
        ? cookieValue
        : encodeURIComponent(JSON.stringify(cookieValue));
    this[RES_COOKIE][cookieName] = createSetCookieValue(cookieName, resolvedValue, options);
  }

ah i see:
          for (const header in userResponse.cookie.headers()) {
            res.setHeader('Set-Cookie', header);
          }


Stop tracking python tests swagger client

https://www.pivotaltracker.com/story/show/155182731

+3 −26,493

😍


[Pulsar Admin] support config request timeout

Motivation
Currently, pulsar admin cli tool request timeout is 300 seconds, this pr is to make it configurable.
Modifications
add option request-timeout to config pulsar admin cli tool request timeout, default is 300 seconds.
add unit test.

/pulsarbot run-failure-checks