I make multi ring chart successfully, but I can not make adjusted legend

Hello I want to make multi ring donut chart.
So I find other chart library.
But I think maybe is it possible make with c3 and css. So I try it.
Here is my result.
https://jsfiddle.net/say8425/xtr6jo5m/
Well, It's not well designed but anyway it works.
However one thing is missed It's the legend.
I hide legend of middle and inner chart.
So only outer legend is shown. And when I hover or click it just works on outer.
I try this legend works on middle and inner chart.
But I don't know well about d3 so it's fail.
How can I apply this legend works with middle and inner chart?

That is absolutely crazy but is also awesome. Props! :+1:
Closing because it seems you've resolved your issue; if not, please ask on the Google Group. Thanks!


[Bug]: Seems it disrepects "@ts-nocheck"

Version
29.1.1
Steps to reproduce
Put // @ts-nocheck and ts-jest will still perform type checking for this file
Expected behavior
Files with //@ts-nocheck aren't type checked.
Actual behavior
Those are typechecked
Debug log

Additional context
No response
Environment
System:
    OS: macOS 13.6
    CPU: (10) arm64 Apple M1 Pro
  Binaries:
    Node: 16.16.0
    npm: 8.11.0
  npmPackages:
    jest: 29.7.0 => 29.7.0

+1

+1


Face detection and emtion recognition for extreme pose

Hi. I am working with videos of children and therapist interaction facing each other. Faces will be of pose > 45. What's your suggestion for above task. Will emotion detector work for children? Also, is Retina Face the best in extreme pose? Thanks

Thanks

Hello, I tested for emotion on side views as well as frontal. Emotion performance is not satisfactory. Any suggestions to improve performance. Can you suggest better emotion detector?


Responsive block groups not changing for mobile

We're having some issues with responsive block groups not updating to phone-block-group-x-up.  When we look at it in Chrome using the device toolbar, it looks fine, but once we open it on an actual mobile device, it doesn't seem that any of the tablet or phone styles are being used.  You can see this on http://doc.arcgis.com/en/arcgis-online/ or on http://docdev.arcgis.com/en/aimeetest/ (where I've been messing around with the HTML a bit to try to get it to work).  On either page, scroll to the Get Started section to see what I'm talking about.  Is there something we're doing wrong, or is this a bug that has been fixed in a more recent version of the code?

@ajbroadbent what browser and version? ios safari/mobile chrome/android browser?
I just tried on my phone in both Safari and Chrome on iOS and they both worked fine. But in my iOS Simulator (which uses a dated version of Safari) it displayed like you're talking about. My guess is that you might be using an older version of Safari and it's not playing nice.
Looks like flex box support in older versions of mobile safari was there but there was no wrapping:

But the good news is that most people who are using mobile safari are using the latest version as it's updated automatically when you update the OS:

That's probably the problem, then.  My phone OS isn't up to date.  Thank you!

It seems like there are still some mobile issues popping up.  This is on an Android:

And this iws on an iPhone 5s with iOS 10.2:

@ajbroadbent I think you are just missing the viewport meta tag in the head of those documents:
<meta name="viewport" content="width=device-width, user-scalable=no">

If I add that to the head of the pages that aren't working it seems to work fine.

Perfect! Thank you!


Missing guidance on adding contributors to licensing headers

The codebase currently uses these copyright headers:
//-------------------------------------------------------------------------------------------------------
// Copyright (C) Microsoft Corporation. All rights reserved.
// Licensed under the MIT license. See LICENSE file in the project root for full license information.
//-------------------------------------------------------------------------------------------------------

Because the Microsoft CLA does not facilitate or require transfer copyright, copyright may belong to a number of contributors. There is no guidance on how contributors should be added to the header.
Do we add them after the Copyright (C)? What ordering scheme do we use? First name/Last name/Significance?
Alternatively, and contributors could be added:
//-------------------------------------------------------------------------------------------------------
// Copyright (C) Microsoft Corporation and contributors. All rights reserved.
// Licensed under the MIT license. See LICENSE file in the project root for full license information.
//-------------------------------------------------------------------------------------------------------

The .NET Foundation takes a similar approach.

Thanks for bringing it up Rafael. Lets go with the same approach as the .Net Foundation team and add "contributors" where appropriate. I will update our guidance.

If I add a completely new translation unit, adding Copyright (c) Microsoft Corporation would be incorrect as no copyright was transferred. I suspect this is one reason why the .NET Foundation uses different language:
// Licensed to the .NET Foundation under one or more agreements.
// The .NET Foundation licenses this file to you under the MIT license.
// See the LICENSE file in the project root for more information.

Looking into few .NET repositories I don't see one standard way.
dotnet/cli uses the format you suggested above;
// Copyright (c) .NET Foundation and contributors. All rights reserved.
// Licensed under the MIT license. See LICENSE file in the project root for full license information.
and dotnet/coreclr uses various formats dependents on the library they use.
Or are you saying we can't use both: "Copyright (c) Microsoft Corporation" and "Copyright (C) Microsoft Corporation and contributors"?

If I create foo.cpp, I implicity own it so I'd be inclined to put only Copyright (c) Rafael Rivera. Just something to be aware of!
Does LCA have recommendations we can apply here?

Let me check with CLA and get back to you.

Checked with legal, we can use this format; in fact it is their guidance.
It also reinforces why we typically need to go through the OSS process when we take back in our own project code (with external modifications) into our own proprietary projects.
//-------------------------------------------------------------------------------------------------------
Copyright (c) Rafael Rivera. All rights reserved.
Licensed under the MIT license. See LICENSE file in the project root for full license information.
//-------------------------------------------------------------------------------------------------------

@vladimp LCA has spoken! Thanks for the quick follow up.

No Problem


don't send unknown signal message to rust sdk with protocol 9

rust sdk can't handle unknown message (ErrorResponse and TrackSubscribed)

Thanks!

@cnderrauber merging this to pick up this version for cloud.


Allow certificate thumbprint to be copied from a table row context menu

I'm not sure how to do this with Prism. It seems kind of hard.

It was kind of hard. The solution came from ContextMenu for WPF DataGrid on row level by Ghassan Karwchan.


Update expectedHash for OAW 3B package

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

#sign-off


TypeError: object NoneType can't be used in 'await' expression

Exception in thread Thread-1 (reader_thread):
Traceback (most recent call last):
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.10_3.10.3056.0_x64__qbz5n2kfra8p0\lib\threading.py", line 1016, in _bootstrap_inner
    self.run()
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.10_3.10.3056.0_x64__qbz5n2kfra8p0\lib\threading.py", line 953, in run
    self._target(*self._args, **self._kwargs)
  File "C:\Users\lking\Documents\Lou's Software\projects\tm-csv-connector\tm-csv-connector\tm-reader-client\app.py", line 162, in reader_thread
    run(reader(port, logging_path))
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.10_3.10.3056.0_x64__qbz5n2kfra8p0\lib\asyncio\runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.10_3.10.3056.0_x64__qbz5n2kfra8p0\lib\asyncio\base_events.py", line 649, in run_until_complete
    return future.result()
  File "C:\Users\lking\Documents\Lou's Software\projects\tm-csv-connector\tm-csv-connector\tm-reader-client\app.py", line 152, in reader
    await send_to_backend(websocket, msg)
  File "C:\Users\lking\Documents\Lou's Software\projects\tm-csv-connector\tm-csv-connector\tm-reader-client\app.py", line 128, in send_to_backend
    await websocket.send(sending)
TypeError: object NoneType can't be used in 'await' expression

needed to use websockets.connect()


fix javadoc formatting in class EnableRabbit

Merged as https://github.com/spring-projects/spring-amqp/commit/9df8c13ee782529142ededffbdbfb8d6afc1a646 with some minor fix for similar formatting problem in the XML sample.
Thank you very much for the contribution.
Looking forward for more!


style: [theme] less saturated colors in dark theme

Description
Colors less saturated and less vibrated improve legibility and reduce visual vibration.
My suggestion is to change the colors current for less colors saturated and less vibrated. It's not the best, but it's a start.
How to test?

Switch theme light/dark
Expected behavior
Old colors:
Print
New colors:
Print

Hey @silvanavlima, Thanks for your contribution!
Can you make a rebase based in develop branch?


Cannot change the value of parameters after creates an instance of Model

In current specification, the instance of Model has an immutable borrow of certain Env, to retrieve error messages after calling C API. It makes be impossible to modify all parameters of a Env until all of instances of Model created from the envinroment.
fn main() {
  let env = Env::new("").unwrap();    // create an environment.

  {
    // Create a model from `env`, and then an `immutable` borrows are holded in `model`.
    let mut model = env.new_model("model").unwrap();

    // A `mutable` borrow cannot create at current scope, because `model` still holds a borrow.
    env.set(param::OutputFlag, 0).unwrap();

    ...
  }

  // `env` can modify the value of parameters if all borrowings of it are removed.
  env.set(param::OutputFlag, 0).unwrap();
}

The instance of GRBenv will copy at model creation (see GRBnewmodel and so on).  It means that all of environments are managed by C API and it is inappropriate  to manage their resource in Rust (except for them created using Env::new()).  Model should contain the body of the instance of Env, not "reference".


Eager mode model.save will not save layer vars if the layer is in list of list

tf 1.10.1
like below model.save will not save CuDNNGRU vars if you use MyLayer in a tf.keras.Model.
class MyLayer(keras.Model):
   def __init__(self):
    super(MyLayer, self).__init__()
      self.encodes =  [(keras.layers.CuDNNGRU(units=100,
                                    return_sequences=True,
                                    return_state=False,
                                    recurrent_initializer='glorot_uniform'),)]

  def call(self, x):
    return self.encodes[0][0](x)

A list works and a list of lists works but a list containing a tuple does not, correct? Tracking tuples would be nice but isn't being actively worked on. Happy to advise/review PRs.

@allenlavoie I think it would be nice to track and record any train able layer variables, not matter what data structure they are in.
I'm also interested in the mechanism of save vars.

Seems if use another layer like B in Layer A(B is a member of A), then we do not save any vars of B.
Then in order to save vars of B I have to make A a Model instead of a Layer, but for Model seems
we can not use add_weight, then I have to wrap members using add_weight using Layer.. A bit complex.

Right, Layer doesn't have a setattr override. Originally they were going to, but just defining it is a performance issue for functional layers (which create Layer objects in a loop and share variables by some other means).
You can add variables to a Model by assigning them to an attribute just like with sub-Layers. add_weight should work too; what's the issue?

@allenlavoie How to add variables to a Model by assigning them to an attribute just like with sub-Layers ?  I could not find any related examples.  Something like self.abc = self.add_variable("abc", [1, 100], initializer=tf.ones_initializer(dtype=tf.float32)) in Model.init() will fail and error say 'add_variable is not supported on Networks. However, you may assign variables to attributes and they will show up in the weights and variables properties'
Another question is if I want to use a Dense layer in a Model,  but the Dense layer out dim is unknown until I see the first input example, see input is [batch_size, dim] then I will use Dense(dim) , if using Layer
I think I can do this using build method, for Model how to do this ? I need to write another Layer to do this first ?

I'd use tf.Variable:
>>> m = tf.keras.Model()
>>> m.v = tf.Variable(1.)
>>> m.trainable_variables
[<tf.Variable 'Variable:0' shape=() dtype=float32, numpy=1.0>]

It's fine to create new Layers in your call() method for subclassed Models. Just make sure they only get created on the first call. I think you can also define a build() method for Models if you'd like.

@allenlavoie I see, thanks!

@chenghuige ok to close?

@drpngx  Fine close this.


Remote Host status bar entry has no max width

see attached screen capture
the remote host entry is too wide, we should crop it

Yes I do agree. However more interesting is why the remote entry looks like that, it should not be so cryptic.
This is what I see:

Was this eventually going away?

Oh I get it, you did not use web but desktop. I wonder if maybe @legomushroom or @srivatsn could comment why desktop is rendered different from web. I suspect a different label formatter maybe?

@aeschli maybe when you detect the remote label to be very long you simply crop it in the middle adding a "..." or so?

@bpasero Not sure what you mean. You suggest we should crop it? I think that should better be done in CSS

Originally the thinking was that in web, the URL has the context but in desktop it's not there but since GH codespaces names are too long, I agree this doesn't look good. @lostintangent @2percentsilk thoughts?

Yeah I'd agree that it's too long and a bit unsightly. Given the name of the codespace is also a part of the title bar, I don't think we need to have it fully duplicated at the bottom. Setting a max width and truncating the codespace name seems reasonable to me. For what exactly the max width would be I'd lean on @misolori :)

I agree on keeping it shorter, just throwing a random number of 40 characters seems to look better. As @2percentsilk mentioned that the name is already repeated in other areas we don't need it as much here. It would also keep the width of the item ~275px.

@aeschli the reason I suggested a manual solution without CSS is that as far as I know there is no CSS rule to crop a label in the middle. But if we think that the label can be cropped at the end, I think a CSS rule could work.
Since each status item has a id, a CSS rule from the remoteIndicator such as:
#status\.host {
	max-width: 40px;
}

Should work. Maybe @misolori could advise for a good rule that works together with flex.
I am not sure I would go as far as to apply this rule to every item, not sure about the consequences for extensions. Related issue is https://github.com/microsoft/vscode/issues/84258

I think in order for this to work via css the remote text needs to be wrapped in it's own span tag, we currently only wrap the codicon in a span. See example.

First of all, the remote extension should use a compact, human readable host name.
Still, I suggest to fix the status bar entry (which already has a max-width) so it gets the ... at the end. That way all status bar entries will profit and it's guaranteed that the maximum space is used. I personally don't think ... in the middle make anything better (really depends on the label), and it's not a goof idea to to have RemoteStatusIndicator do any shorting as it will have to make assumptions on the current width of the status bar entry.

Assigning to @bpasero to consider the change in the status bar as sketched by @misolori .

I ended up pushing a change to trim the remote label to max 40 chars as suggested by Miguel:

I don't to turn this into a general solution for all status bar entries because this is challenging and we have related issues to cover that in particular, namely https://github.com/microsoft/vscode/issues/84258


nixos.tests.containers-imperative fails on master, blocking nixos-unstable channels

Issue description
nixos.tests.containers-imperative is failing on master, see hydra logs. This is a blocker for the nixos-unstable and nixos-unstable-small channels.
I am currently bisecting and analyzing the failure. Not yet sure whether it's the test or the containers that are broken.

was fixed by 0385d69f878a4b9ce4d4735e28dff6ecf7b64507


Include imports in examples

A few of the code examples have things like P.x in them. I had no idea what this was until I opened up the documentation on elm-packages and saw that one of the modules is Simple.Animation.Property. From there I inferred that P was an alias for Simple.Animation.Property.
I think it would be easier if, for example,
animatedDot : Html msg
animatedDot =
    Animated.div expandFade [ class "dot" ] []

expandFade : Animation
expandFade =
    Animation.fromTo
        { duration = 2000
        , options = [ Animation.loop ]
        }
        [ P.opacity 1, P.scale 1 ]
        [ P.opacity 0, P.scale 2 ]

was instead
import Simple.Animated as Animated
import Simple.Animation as Animation
import SimpleAnimation.Property as P

animatedDot : Html msg
animatedDot =
    Animated.div expandFade [ class "dot" ] []

expandFade : Animation
expandFade =
    Animation.fromTo
        { duration = 2000
        , options = [ Animation.loop ]
        }
        [ P.opacity 1, P.scale 1 ]
        [ P.opacity 0, P.scale 2 ]

Good idea, happy for you to make a PR if you have time?

Good idea, happy for you to make a PR if you have time?

closed with #7

closed with #7


Create a CONTRIBUTING.md file for guidelines on contributions

Objective:
To create a CONTRIBUTING.md file that provides clear and helpful guidelines for contributing to this project. This file will outline the process contributors should follow when creating pull requests, reporting issues, or suggesting improvements.
Requirements:

Outline the steps for contributing, including:

Cloning the repository.
Setting up the environment if necessary.
Creating a new branch and making changes.

Include details about code style, commit message conventions, and testing requirements (if any).
Mention the process for reporting issues and feature requests.
Add guidelines for respectful and constructive communication.

Deliverable:
A CONTRIBUTING.md file that is comprehensive, clear, and structured, ensuring a smooth experience for new contributors.

What code style, commit message conventions, and testing requirements did you observe in this repo?

In reviewing the repository, I did not find specific files that define a code style (such as .eslintrc or .prettierrc) or a documented commit message convention. There are also no dedicated testing files or a tests folder that would typically indicate testing requirements. I reviewed the package.json and requirements.txt files, but they did not contain any testing scripts or configurations related to code style. It seems that this project may not enforce strict guidelines for code style, commit messages, or testing.

@anchalchaturvedi08 apologies but we don't need it currently. Closing this now.


RollbarTestingException: Testing rollbar with "rake rollbar:test". If you can see this, it works.

View details in Rollbar: https://rollbar.com/project-playtime/project-playtime/items/12/

RollbarTestingException: Testing rollbar with "rake rollbar:test". If you can see this, it works.
  File "/app/vendor/bundle/ruby/2.4.0/gems/rollbar-2.15.0/lib/rollbar/rake_tasks.rb", line 41, in test_rollbar
  File "/app/vendor/bundle/ruby/2.4.0/gems/rollbar-2.15.0/lib/rollbar/rake_tasks.rb", line 53, in verify
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_controller/metal/basic_implicit_render.rb", line 4, in send_action
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/abstract_controller/base.rb", line 186, in process_action
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_controller/metal/rendering.rb", line 30, in process_action
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/abstract_controller/callbacks.rb", line 20, in block in process_action
  File "/app/vendor/bundle/ruby/2.4.0/gems/activesupport-5.1.1/lib/active_support/callbacks.rb", line 131, in run_callbacks
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/abstract_controller/callbacks.rb", line 19, in process_action
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_controller/metal/rescue.rb", line 20, in process_action
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_controller/metal/instrumentation.rb", line 32, in block in process_action
  File "/app/vendor/bundle/ruby/2.4.0/gems/activesupport-5.1.1/lib/active_support/notifications.rb", line 166, in block in instrument
  File "/app/vendor/bundle/ruby/2.4.0/gems/activesupport-5.1.1/lib/active_support/notifications/instrumenter.rb", line 21, in instrument
  File "/app/vendor/bundle/ruby/2.4.0/gems/activesupport-5.1.1/lib/active_support/notifications.rb", line 166, in instrument
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_controller/metal/instrumentation.rb", line 30, in process_action
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_controller/metal/params_wrapper.rb", line 252, in process_action
  File "/app/vendor/bundle/ruby/2.4.0/gems/activerecord-5.1.1/lib/active_record/railties/controller_runtime.rb", line 22, in process_action
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/abstract_controller/base.rb", line 124, in process
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionview-5.1.1/lib/action_view/rendering.rb", line 30, in process
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_controller/metal.rb", line 189, in dispatch
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_controller/metal.rb", line 253, in dispatch
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_dispatch/routing/route_set.rb", line 49, in dispatch
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_dispatch/routing/route_set.rb", line 31, in serve
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_dispatch/journey/router.rb", line 46, in block in serve
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_dispatch/journey/router.rb", line 33, in each
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_dispatch/journey/router.rb", line 33, in serve
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_dispatch/routing/route_set.rb", line 832, in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/omniauth-1.6.1/lib/omniauth/strategy.rb", line 189, in call!
  File "/app/vendor/bundle/ruby/2.4.0/gems/omniauth-1.6.1/lib/omniauth/strategy.rb", line 167, in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/omniauth-1.6.1/lib/omniauth/builder.rb", line 63, in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/rack-2.0.3/lib/rack/etag.rb", line 25, in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/rack-2.0.3/lib/rack/conditional_get.rb", line 25, in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/rack-2.0.3/lib/rack/head.rb", line 12, in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/rack-2.0.3/lib/rack/session/abstract/id.rb", line 232, in context
  File "/app/vendor/bundle/ruby/2.4.0/gems/rack-2.0.3/lib/rack/session/abstract/id.rb", line 226, in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_dispatch/middleware/cookies.rb", line 613, in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_dispatch/middleware/callbacks.rb", line 26, in block in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/activesupport-5.1.1/lib/active_support/callbacks.rb", line 97, in run_callbacks
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_dispatch/middleware/callbacks.rb", line 24, in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/rollbar-2.15.0/lib/rollbar/middleware/rails/rollbar.rb", line 24, in block in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/rollbar-2.15.0/lib/rollbar.rb", line 145, in scoped
  File "/app/vendor/bundle/ruby/2.4.0/gems/rollbar-2.15.0/lib/rollbar/middleware/rails/rollbar.rb", line 22, in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_dispatch/middleware/debug_exceptions.rb", line 59, in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/rollbar-2.15.0/lib/rollbar/middleware/rails/show_exceptions.rb", line 22, in call_with_rollbar
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_dispatch/middleware/show_exceptions.rb", line 31, in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/railties-5.1.1/lib/rails/rack/logger.rb", line 36, in call_app
  File "/app/vendor/bundle/ruby/2.4.0/gems/railties-5.1.1/lib/rails/rack/logger.rb", line 24, in block in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/activesupport-5.1.1/lib/active_support/tagged_logging.rb", line 69, in block in tagged
  File "/app/vendor/bundle/ruby/2.4.0/gems/activesupport-5.1.1/lib/active_support/tagged_logging.rb", line 26, in tagged
  File "/app/vendor/bundle/ruby/2.4.0/gems/activesupport-5.1.1/lib/active_support/tagged_logging.rb", line 69, in tagged
  File "/app/vendor/bundle/ruby/2.4.0/gems/railties-5.1.1/lib/rails/rack/logger.rb", line 24, in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_dispatch/middleware/remote_ip.rb", line 79, in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_dispatch/middleware/request_id.rb", line 25, in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/rack-2.0.3/lib/rack/method_override.rb", line 22, in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/rack-2.0.3/lib/rack/runtime.rb", line 22, in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/activesupport-5.1.1/lib/active_support/cache/strategy/local_cache_middleware.rb", line 27, in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionpack-5.1.1/lib/action_dispatch/middleware/executor.rb", line 12, in call
  File "/app/vendor/bundle/ruby/2.4.0/gems/actionp

Sorry about all that; I was configuring something!


Empty result, send messenger to the user

Hey guys,
if a result is empty, using is code
(events | filter: terms).length === 0
for exemple

     not events result

Thanks a your plugin! it`s amazin :)

Does this actually work? I cant seem to make it work.

@Alex61NN5 can you provide some parts of your code?

not events result

Worked for me!

for future you, use the as keyword to alias the results and avoid double data filter processing. e.g.

<ng-container *ngIf="isSearching && rows.length==0">

        No data matching the filter criteria

   </ng-container>


fix(shipper): update DHL for Germany

Fixed German DHL sensor
Breaking change
none
Proposed change
German DHL sensor did not work. I adjusted the patterns so hopefully it will now detect mails correctly.
Type of change

[x] Bugfix (non-breaking change which fixes an issue)
[x] Update existing shipper

Codecov Report

Merging #766 (da7d3cf) into dev (b1630d4) will increase coverage by 0.06%.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##              dev     #766      +/-   ##
==========================================
+ Coverage   97.14%   97.21%   +0.06%
==========================================
  Files           6        8       +2
  Lines        1297     1508     +211
==========================================
+ Hits         1260     1466     +206
- Misses         37       42       +5

Impacted Files
Coverage Δ

custom_components/mail_and_packages/const.py
100.00% <ø> (ø)

custom_components/mail_and_packages/__init__.py
95.90% <0.00%> (ø)

Thanks!


Update webpack-hot-client and koa-webpack

This PR contains a:

[ ] bugfix
[x] new feature
[ ] code refactor
[ ] test update
[ ] typo fix
[x] metadata update

Breaking Changes
This PR updates 2 mayor libraries to latest version and adds new options in configuration.
Additional Info

ohh i forgot to push all my changes :+1:

No worries. We're making this update as part of a large codebase refactor. Hang tight, it won't be long.


Anatomize.py Will Show as Match When Matched String is Just Whitespace

Anatomize.py will still record a regex match as true even if the string it matched is only whitespace. There is no use case where this would be needed and is a bug as a blank string is considered not a match.

Fixed with commit https://github.com/magneticstain/Inquisition/commit/fd7a15c917aec11ab030b8a3e69793b7acafc89c


HUD Status bars

Implement HP, TP, SP, and TNL status bars in UI. Remove old code.
ToDo list for Status Bars:

Convert old status bars to new XNAControls (see EndlessClient.Hud.StatusBars directory)
Create status bar controls in HudControlsFactory.CreateHud(). Add them to the return collection with their associated HudControlIdentifier
a. A factory may be necessary if the dependency list for the status bars becomes too long

Completed in referenced commits.


Network blocks table shows header before any blocks are loaded

Describe the bug

The headers for the blocks table will be displayed for a couple seconds before the blocks are displayed.
To Reproduce

Hit refresh on the blocks page
See the table headers floating without blocks

Expected behavior

Show a loading graphic while the blocks are loading

I don't think this is an issue. It's actually better to show the header than nothing. The only thing that I'd fix it the transaction counter, which is blank before blocks are displayed

i have a simple solution.


high execution times of rmse method and trustworthiness and density metrics at dimensionality reduction

rmse method and trustworthiness and density metrics require a long execution time and a lot of memory to be executed.
Possible common cause might be the fact that the three processes calculate under the hood the pairwise (euclidean) distances for the full count matrix (stored in .layers['counts']).
Would it make sense to create a novel component which precomputes the pairwise distances?
Related to #46

closing this for now, will re-evaluate when necessary


Made two, one works 100%, other only basic functions works

So I made two identical ones for my two garage doors by soldering myself (had the W1 mini's laying around from before). The first one works 100%. The second one, only door open close works so it is obviously connected to some level. but none of the other functions works to report status - like door status shows OPEN while it is closed, I lot of other stuff is in UNKNOWN or NA state. What could be possibly wrong with the second one?

It would be useful to know your system configuration (e.g. V1.0 or V2.0), the type of mosfet you're using, and a link to the schematic you've used.
My guess is you're in emulation mode because you're not passing data.  I would try changing the mosfet associated with the received data path (Q1).  The gate threshold voltage could be too low.

This is the schematic I am using using 2n7000

so, I made a fresh third one since I had more spare parts thinking one of the mofset or resistor may be bad. Interestingly, it has the same exact issue. The first one works great but the second and third one has the same issue - can open and close but not able to read any data. I connected the second and third to the first garage door opener thinking may be my other garage door opener is not sending serial data or something but they don't work connected to the "good" garage door opener either. Hmm, I may have to live with just one for now so I am unsure how to troubleshoot any further.

Does the good unit work with the second GDO?  Does the obstruction circuit work?  Can I assume you're using through-hole parts?  If so, you should double check the orientation of the mosfets as there can be a different lead orientation if BS170 variants are somehow mixed in with the 2N7000s.
https://www.vishay.com/docs/70226/70226.pdf
Unfortunately, 2N7000 mosfets can have a fairly broad variation of Vgs.  If you have  a 100k resistor, you might try wiring it between the gate of Q1 and ground.  That would indicate the data bus is not getting close enough to ground, or the mosfet isn't totally turned off (or not discharging fast enough) even if it is getting close enough to ground.  If it still doesn't work, try two 100k resistors in parallel.
What kind of test equipment do you have?  A simple logic probe and a multi-meter would be useful.  At least you might be able to test the parts to see if they're usable in the circuit.
https://www.amazon.com/Elenco-Electronics-LP-560-Logic-Probe/dp/B000Z9HAP4

Yeah, the good unit works with the second GDO. I didn’t connect the
obstruction wire with any of the units. Yes, thru hole components and
orientation of transistors is exactly the same for all the units.
I will try by using a 100k resistor instead of 10k.
I have a multimeter but not sure what to test with it. The ESP is obviously
working. The resistors read 10k on multimeter. ESP reads the GDO to some
extent to not say that no GDO is connected. It can send open close commands
but not read any of the other sensor data. Well I don’t need much of the
other sensor data but it keeps saying that the door is open when it’s
closed which is not good.
On Sun, Mar 3, 2024 at 1:22 AM Sudo-Rob @.***> wrote:

Does the good unit work with the second GDO? Does the obstruction circuit
work? Can I assume you're using through-hole parts? If so, you should
double check the orientation of the mosfets as there can be a different
lead orientation if BS170 variants are somehow mixed in with the 2N7000s.
https://www.vishay.com/docs/70226/70226.pdf
Unfortunately, 2N7000 mosfets can have a fairly broad variation of Vgs. If
you have a 100k resistor, you might try wiring it between the gate of Q1
and ground. That would indicate the data bus is not getting close enough to
ground, or the mosfet isn't totally turned off (or not discharging fast
enough) even if it is getting close enough to ground. If it still doesn't
work, try two 100k resistors in parallel.
What kind of test equipment do you have? A simple logic probe and a
multi-meter would be useful. At least you might be able to test the parts
to see if they're usable in the circuit.
https://www.amazon.com/Elenco-Electronics-LP-560-Logic-Probe/dp/B000Z9HAP4
—
Reply to this email directly, view it on GitHub
https://github.com/Kaldek/rat-ratgdo/issues/91#issuecomment-1975059738,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH7MJVXXWGL54FARUWBP6RLYWK6TPAVCNFSM6AAAAABECDXN36VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZVGA2TSNZTHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

The mosfets are probably the problem since the 2N7000s are so finicky. The current recommendation is to use BS270s, but you may be able to try all of the 2N7000s you have around and get lucky finding another one that works. Since commands work, the problem should be on the RX line and if nothing else, this more up-to-date version of the schematic uses additional resistors, which is the only version that works for some people. I'll be curious if the 100k resistor works since that's a new suggestion.

I didn’t connect the obstruction wire with any of the units.

This isn't necessarily a problem, especially if you don't have obstruction sensors attached to your GDO, but it should be noted that there is a bug where this sometimes causes the "close" command to be nonfunctional. If you find that to be the case for you, "toggle" can still be used to close the door but would obviously be more efficacious if you have open/close status.

@foreverimagining beet me to this, but here's my reply anyhoo.

Yes, thru hole components and orientation of transistors is exactly the same for all the units.

Anything in a TO-92 package will look similar.  Check the markings on the transistor.

I will try by using a 100k resistor instead of 10k.

That's not the mosfet of concern.  I'm referring to Q1, which looks at the data on the red wire.   When the voltage on the gate crosses its threshold when going from high to low, it should change state.  That is, the voltage at D2 should "open" and momentarily go to 3.3 volts.  I'm suggesting you connect a pulldown resistor (or two) between the gate (a.k.a. the data bus) and ground.  This will help the transistor by marginally changing the gate voltage when the data line is low.
By the way, this is what others have been using.  Look at R6 and R7.  Also note that Q1 is a 2N7002, which has a slightly higher minimum Vgs.  (FWIW, my opinion is the threshold should ideally be between 2 and 3 volts).
https://github.com/Kaldek/rat-ratgdo/pull/49

I have a multimeter but not sure what to test with it.

You might try to put it on the AC setting and probe between ground and the mosfet drain connected to D2. If the transistor is not changing state, it should be a fairly stable reading of 0 volts.  You can also try the DC setting, which should jump around a bit if the transistor is working.

@foreverimagining beat me to this

Lol, this seems to be a regular occurrence here. Still, you had good additional input that added value to the conversation. It can just take time to type everything up and a lot can happen in between. XD

@foreverimagining, it's a bit of a 'hail Mary.'
If you're interested, look here:  https://github.com/Kaldek/rat-ratgdo/pull/75

@foreverimagining, it's a bit of a 'hail Mary.'
If you're interested, look here: #75

Ah, from the alternative one. I'll have to look into it more sometime. I'm always interested in learning new things, and I've pretty much jumped into electronics with 3 feet recently, so it ought to be something good to learn from.


No examples or libraries to convert mainstream data to TimeStream

Hello,
The CRUD sample is sending data that is constructed by hand. Do we have any automated way of building of payload without we building the payload. Cant this be automatically done without users defining the data in this format ?
const cpuUtilization = {
    'Dimensions': dimensions,
    'MeasureName': 'cpu_utilization',
    'MeasureValue': '13.5',
    'MeasureValueType': 'DOUBLE',
    'Time': currentTime.toString()
};

const memoryUtilization = {
    'Dimensions': dimensions,
    'MeasureName': 'memory_utilization',
    'MeasureValue': '40',
    'MeasureValueType': 'DOUBLE',
    'Time': currentTime.toString()
};

Hi @IrfanSyed-PSC,
Could you please provide more information about the specific scenarios you have in mind for automation? Additionally, sharing your use case will help us better understand your requirements and enable us to propose the most suitable solution.
I understand that this ticket was raised quite some time ago. If it is no longer relevant or if you have found a solution, please let us know so that we can close this issue appropriately.

We are not longer using it, so no update on that. Will close the ticket.


cargo-spellcheck: 0.8.13 -> 0.8.14

Motivation for this change
Update cargo-spellcheck, a tool to check rust documentation for spelling and grammar mistakes.
This release updates the dependencies to fix any potential vulnerabilities (see #141368).
Commit range (no changelog available).
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 141510 run on x86_64-linux 1

1 package built:

cargo-spellcheck


TypeError: params.then is not a function

Saw that #14 also had this problem. But I installed the latest npm package for hubot-yelp so your fix should have worked. Tried removing the parens around "params" but that still says error. Any ideas?
ERROR TypeError: params.then is not a function   at queryYelp (/home/ec2-user/hiphubot/node_modules/hubot-yelp/src/lunchtime.coffee:41:3, <js>:27:31)   at TextListener.callback (/home/ec2-user/hiphubot/node_modules/hubot-yelp/src/lunchtime.coffee:59:5, <js>:50:14)   at executeListener (/home/ec2-user/hiphubot/node_modules/hubot/src/listener.coffee:65:11, <js>:53:19)   at allDone (/home/ec2-user/hiphubot/node_modules/hubot/src/middleware.coffee:44:37, <js>:34:16)   at /home/ec2-user/hiphubot/node_modules/async/lib/async.js:274:13   at Object.async.eachSeries (/home/ec2-user/hiphubot/node_modules/async/lib/async.js:142:20)   at Object.async.reduce (/home/ec2-user/hiphubot/node_modules/async/lib/async.js:268:15)   at /home/ec2-user/hiphubot/node_modules/hubot/src/middleware.coffee:49:7, <js>:37:22   at _combinedTickCallback (internal/process/next_tick.js:131:7)   at process._tickCallback (internal/process/next_tick.js:180:9)

Did you restart your hubot after installing the latest? I know that hubot scripts won't pick up the latest unless the bot is turned off and on again.

Yeah, it's been on and off since yesterday when I installed hubot-yelp for the first time. Tried toying with it to see if I could fix it but nothing yet.

What JS does coffeescript compile the script to? It appears to be correct when I past it into the coffeescript website.
Have you tried upgrading your version of coffeescript?

Oops, looks like updating coffeescript fixed it. My b
Although I'm still not getting any messages printed out, that is a separate issue that I will try to troubleshoot before I create a new issue here. Thanks!

@aysong please make sure your API tokens are registered with Yelp's new API (Fusion).
Yelp's V2 API will be deprecated soon: https://engineeringblog.yelp.com/2017/06/upcoming-deprecation-of-yelp-api-v2.html


Adjust analog FM radio bandwidth

The actual bandwidth of an analog FM radio channel (at least in the US) is 258.722KHz (-129.361Khz - 129.361Khz), not 200KHz.  While there shouldn't be any audio data out that far from the center, it does make the implementation more accurate and allows the signal meter to provide a better SNR reading.
The problem, at least here in the US, is that HD Radio screws with this. Hybrid channels can (and seemingly always) truncate the analog bandwidth back to 200KHz. A quick test bumping analog to 258.722KHz shows that I need to do some work on the signal meter before changing this in the code, since it would give very inaccurate information for Hybrid channels, the calculated "noise" frequency moves into the digital signal, which is pretty much always around 10dB lower than the carrier, making the displayed SNR worthless.
Leave it as-is for now. The only real downside to the current implementation is that users may not be getting the "best" SNR possible from the signal meter for analog-only channels. The process of maximizing the SNR they do see by adjusting the tuner gain is still a very valid and correct process to follow.
When DAB(+) comes into play with it's crazy 1500KHz bandwidth, the entire concept of the signal meter needs to change anyway :)

FYI for myself: https://patents.google.com/patent/US9350471

Also I noted that the analog DSP doesn't accommodate the different de-emphasis for Europe, it's 50us instead of 75us.  Not sure what effect that may be having.

After analyzing this, I think it's a bad idea. Both North America and Europe define channels with 200MHz separation. While I have access to analog channels with > 200MHz bandwidth, I cannot hear or see any difference.  I think HD Radio needs adjustments to figure out how much of the "extended" bandwidth the channel uses, but that wouldn't affect the analog component.
Leaving open since I do want to check/address the de-emphasis concern.

The de-emphasis is handled by the demodulator, but it probably needs to be made more clear in the settings. Today this will be handled by ensuring the addon is set to RBDS instead of RDS, might want to consider having a more clear "Region" setting that allows for North America or World and then just handle RBDS/RDS based on that.


[Sync] Double IBD problem

Could be related to #938
Node 1 is at block 1900 (consensus), Node 2 is at block 2000 (consensus), best chain height is 2100 (both nodes headers tip).
Node 1 will use LookaheadBlockPuller to ask for blocks from Node 2. Node 1 will use "getdata" message from BlockPullerBehavior.StartDownload to ask for block 2001. Node 2 will receive this in BlockStoreBehavior where is this sequence:
                // TODO: check if we need to add support for "not found"
                Block block = await this.blockStoreCache.GetBlockAsync(item.Hash).ConfigureAwait(false);

                if (block != null)
                {
                    this.logger.LogTrace("Sending block '{0}' to peer '{1}'.", item.Hash, peer?.RemoteSocketEndpoint);

                    //TODO strip block of witness if node does not support
                    await peer.SendMessageAsync(new BlockPayload(block.WithOptions(peer.SupportedTransactionOptions))).ConfigureAwait(false);
                }

This means that if Node 2 does not have this block, it will ignore "getdata" message.
Node 1 will have this block assigned to the Node 2 and marked as "pending download".
If Node 2 is the only peer that Node 1 is connected to, Node 1 is stuck and won't advance in consensus.
There are three things that should be implemented to fix this:

node 2 should be polite and not to ignore "getdata" message, it should reply with "notfound" message
node 1 should be able to understand "notfound" message and remove the assignment from the peer based on that
node 1 should have timeout on block download, the mechanism is there in form of quality score, but it is not used if there is only one peer connected, therefore in this case (which can be well intended setup), it is normal that node 1 will stop syncing at some point. it should eventually timeout, it could be after couple of minutes, but it should be possible

Investigate how this is done in bitcoin core.

This will be fixed by the design of the refactor.

Fixed with activation merge.


WIP - refactor tests for features to be declarative style and smaller classes

If this is a WIP, poss best to make the PR draft.

Nice work @shahnaazr. I like the way you've organised things by elements and tasks and the tests seems a bit more readable.

Closing and deleting branch as per Slack: https://mojdt.slack.com/archives/C0223AGGQU8/p1643805667164229


Feature Request: Provide HTTP Passthrough to OpenTSDB

In some cases, I would like to be able to call other URIs to call oepntsdb directly, such as submitting annotations, getting the server version and updating/querying metric meta-data. It would be convenient if all OpenTSDB api endpoints were exposed by bosun in some generic fassion such as /api/tsdb/[open-tsdb-api]

You can use the undocumented relayListen setting in your conf file. It will listen on the given address (i.e., set to :4242) and will pass through all /api/X calls to your OpenTSDB server.

Needs Documentation.


Recolorize NaN error after alignment

Hi Hannah,
I am using Recolorize to analyze Heliconius wings and I am following the recolorize -> patternize pipeline using the recolorize_examples. I have created a similar file structure as the one used in the recolorize examples files. The images have had the background removed using rembg and are png's. The images are available here:
erato_lativitta.zip
The images align well, and I am able to generate reconstructions in the R-plot space. However, when I go to plot them all using the lapply function, I get an error: error in rgb color intensity nan not in (0,1). And upon further inspection, there are NaN's in the sample_raster_data attribute. Like so.
Have you encountered this error before or have any suggestions for how to proceed?
Thank you,
Christopher

Hi Christopher,
Thanks for bringing this bug to my attention! I was able to reproduce the same problem on my computer using your attached files. It seems that sometimes in the alignment step, patternize ends up generating 'NaN' values, and the brick_to_array function in recolorize doesn't correctly convert those into 0's to represent background. It looks like this is largely the result of rotating images to be aligned (you can see that the NaN values are all the result of rotating a rectangle):
# load raster library
library(raster)

# read in raster list from erato_lativitta
rasterlist <- readRDS("erato_lativitta/fws_imageList_aligned.rds")

# generate plots of where the NaN values are (in green):
layout(matrix(1:10, nrow = 2))
par(mar = c(2, 3, 3, 1))
b <- lapply(1:length(rasterlist), \(x) plot(merge(is.nan(rasterlist[[x]])),
                                            main = names(rasterlist)[x]))

If you look at the brick_to_array function in recolorize, you'll see that the only way we identify background pixels is by looking for pixels with an alpha value of 0. So by default, the NaN values get treated as being in the region of interest, carry through to the converted array as if they're regular RGB pixels, and then confuse the downstream functions that are expecting RGB values in the 0-1 range.
You could in theory fix that by just converting all the NaN and NA values in each raster to a 0 before running them through brick_to_array:
for (i in 1:length(raster_list)) {
    r <- raster_list[[i]]
    r[is.na(r)] <- 0
    raster_list[[i]] <- r
}

...but, that feels silly when I could fix this problem entirely by just adding a single line (r[is.na(r)] <- 0) to the brick_to_array function, so that's what I did. This converts all NaN and NA values to 0 before looking for pixels with an alpha of 0 to treat as background. This seems to fix the problem, because after updating recolorize the following code runs with no problem:
a_list <- lapply(rasterlist, brick_to_array)
lapply(a_list, plotImageArray)

The obvious caveat here is that if you want your NaN/NA values to represent something other than background, this will eliminate them, so it's a pretty patternize-specific solution. That said, recolorize doesn't really have a way of handling NA/NaN values besides ignoring them, so in this case it seems appropriate.
TL;DR if you're having the same problem
Update recolorize from source by running devtools::install_github("hiweller/recolorize"), restart your R session, and try converting from raster bricks to arrays again.
Let me know if this resolves the issue for you!


Update PR and Bug templates

What does this pull request do?
We've already discussed removing "Why is this important?" from the
PR template. I went a step further and removed "Expected behavior"
from the Bug_report template, as I think it's almost always addressed in
the "Describe the bug" section.
@beniwohli WDYT?

@beniwohli Any thoughts here?

Oh sorry, forgot to submit my 👍


When predicting on batch, is it possible to use a worker to load images into disk while running forward pass for last batch?

I packed my testing images into batches and use model.predict_on_batch to predict on batch. However, I have 40 images per batch, loading them into disk needs a second, this is a lot of time compared to the forward time, so is there any tool I can use to load next batch into disk while predicting on a batch? Thanks!

Yes, what you want to use is predict_generator. See the docs.


Updated installation via pnpm

Please describe the changes this PR makes and why it should be merged:
The use of pnpx is now deprecated and usage will result in an error, it is recommended to use [pnpm dlx](https://pnpm.io/cli/dlx) instead
Status

[x] Code changes have been tested against prettier, or there are no code changes
[x] I know how to update typings and have done so, or typings don't need updating

Semantic versioning classification:

[ ] This PR changes the codebase

[ ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)
[ ] This PR changes the internal workings with no modifications to the external API (bug fixes, performance improvements)

[x] This PR only includes non-code changes, like changes to documentation, README, etc.

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

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1257

Relevant Lines:
5082

💛  - Coveralls

Hmm, thank you @RayyanNafees; but, I'm unsure as to whether this is a major addition for now.


[BUG]Cannot resolve com.azure.spring:azure-spring-boot-starter-active-directory:3.0.0-beta.1 on azure-spring-boot-sample-active-directory-stateless project

Describe the bug
Cannot resolve com.azure.spring:azure-spring-boot-starter-active-directory:3.0.0-beta.1
and I can not find this dependency at maven repository
Code Snippet
<dependency>
      <groupId>com.azure.spring</groupId>
      <artifactId>azure-spring-boot-starter-active-directory</artifactId>
      <version>3.0.0-beta.1</version> <!-- {x-version-update;com.azure.spring:azure-spring-boot-starter-active-directory;current} -->
    </dependency>

Expected behavior
Screenshots
Setup (please complete the following information):

IDE : IntelliJ Idea

That project isn't in maven central yet, so you have to build from source: https://github.com/Azure/azure-sdk-for-java/blob/master/sdk/spring/CONTRIBUTING.md#building-from-source

@liuziyuan
Here it is: https://repo.maven.apache.org/maven2/com/azure/spring/azure-spring-boot-starter-keyvault-certificates/
And if you are using latest master branch, please build by the following command:
cd azure-sdk-for-java
mvn clean install -pl .\sdk\spring\azure-spring-boot-samples\azure-spring-boot-sample-active-directory-stateless -am

Refs: https://stackoverflow.com/questions/4588956/what-exactly-happens-during-this-command-mvn-pl-project-list

@chenrujun

Here it is: https://repo.maven.apache.org/maven2/com/azure/spring/azure-spring-boot-starter-keyvault-certificates/

No - that's keyvault-certificates - azure-spring-boot-starter-active-directory is missing.

@derkoe
Oh, my mistake. You are right.
We didn't release azure-spring-boot-starter-active-directory:3.0.0-beta.1.
Refs:
https://github.com/Azure/azure-sdk-for-java/blob/master/sdk/spring/azure-spring-boot-starter-active-directory/CHANGELOG.md
Please use 2.3.5 instead. or build azure-spring-boot-starter-active-directory:3.0.0-beta.2 by yourself.  (check my last comment about how to build by yourself).

Hey guys, azure-spring-boot-starter-active-directory 3.0.0 has been released now, here is the maven link. Closing this issue now.

Hey guys, azure-spring-boot-starter-active-directory 3.0.0 has been released now, here is the maven link. Closing this issue now.


Previous 5 boardings not sorted properly

When I look at the recent boardings I can see the following vessels: Orca - Magic - Location Test - TestVessel - Jenny. But in the database the most recent boardings are Orca - Predator - Magic - Location Test - TestVessel.

What I see on the screen (ios is the same even though this is an android screenshot):

From the database (the query limits the timestamp so there aren't ALL the records shown):
MongoDB Enterprise atlas-l4abbo-shard-0:PRIMARY> db.BoardingReports.aggregate([{$match: {   agency:'Ecuadorian Galapagos',   timestamp: {$gt : ISODate("2020-07-25 00:00:00.000Z")} }}, {$sort: {   timestamp: -1 }}, {$project: {   'vessel.name': 1,   timestamp: 1, _id: 0 }}]);
{ "timestamp" : ISODate("2020-07-28T17:17:43.407Z"), "vessel" : { "name" : "Orca" } }
{ "timestamp" : ISODate("2020-07-28T17:16:56.318Z"), "vessel" : { "name" : "Predator" } }
{ "timestamp" : ISODate("2020-07-28T17:16:22.183Z"), "vessel" : { "name" : "Magic" } }
{ "timestamp" : ISODate("2020-07-28T16:55:34.351Z"), "vessel" : { "name" : "Location Test" } }
{ "timestamp" : ISODate("2020-07-28T16:21:17.338Z"), "vessel" : { "name" : "Location Test" } }
{ "timestamp" : ISODate("2020-07-28T16:19:35.063Z"), "vessel" : { "name" : "Location Test" } }
{ "timestamp" : ISODate("2020-07-28T14:11:02.583Z"), "vessel" : { "name" : "Location Test" } }
{ "timestamp" : ISODate("2020-07-28T09:04:12.183Z"), "vessel" : { "name" : "TestVessel" } }
{ "timestamp" : ISODate("2020-07-27T14:27:32.455Z"), "vessel" : { "name" : "Jenny" } }
{ "timestamp" : ISODate("2020-07-27T14:23:28.354Z"), "vessel" : { "name" : "Jenny" } }
{ "timestamp" : ISODate("2020-07-27T12:21:27.275Z"), "vessel" : { "name" : "Location Test" } }
{ "timestamp" : ISODate("2020-07-27T12:20:05.059Z"), "vessel" : { "name" : "Location Test" } }
{ "timestamp" : ISODate("2020-07-27T12:03:34.775Z"), "vessel" : { "name" : "Jenny" } }
{ "timestamp" : ISODate("2020-07-27T11:43:04.691Z"), "vessel" : { "name" : "Jenny" } }

@Sheeri  it happens, because "Orca" and "Predator" both don't have permit number  (or have the same - empty).

Bob from WildAid says. "Most records will have permit number, but we should not hide records without them."
Please show boardings with permit numbers. It is OK if boardings with the same vessel name and blank permit number (e.g. "test" and blank permit number) are grouped as the same vessel.

@Sheeri, for now it works in the same way. If vessel has permit number, we use permit number as unique identifier and group vessels by permit number. If vessel don't have permit number, we use name of the vessel as a unique identifier and group this vessel by name.

Confirmed this works as you say. Closing!


Create FunctionContext extension method to retrieve HttpContext

Issue describing the changes in this PR
resolves #1531
Pull request checklist

[x] My changes do not require documentation changes

[ ] Otherwise: Documentation issue linked to PR

[ ] My changes should not be added to the release notes for the next release

[x] Otherwise: I've added my notes to release_notes.md

[x] My changes do not need to be backported to a previous version

[ ] Otherwise: Backport tracked by issue/PR #issue_or_pr

[ ] I have added all required tests (Unit tests, E2E tests)

Manually tested but looking to add to the test suite


Ignore hidden field if an input with the same name already exists

I use Asp.net MVC, when I use the @Html.EditorFor (or CheckboxFor) the Razor view engine generates two inputs, a checkbox and a hidden field. Both of these have the same id & name. (http://stackoverflow.com/questions/2860940/why-does-the-checkboxfor-render-an-additional-input-tag-and-how-can-i-get-the-v)
When I submit the form through the typical <form type='post' action='myurl'> ... <input type='submit'></form> then the MVC controller gets all the correct information, however when I use jquery ajax and this plugin (which I love btw), then the value for my field is overwritten by the hidden field's value.
I have put together a jsfiddle to repo this issue.
https://jsfiddle.net/8df7fc9k/1/
Thanks,
Nathan

Hey Nathan,
Thanks for the detailed information and the fiddle. It allows me to help you a lot better.
typical POST request

checkbox checked test=true&test=false
checkbox unchecked test=false

jQuery serialize

checkbox checked test=true&test=false
checkbox unchecked test=false

jQuery serializeObject

checkbox checked {test: "false"} (first value is overridden by previous value)
checkbox unchecked {test: "false"}

This plugin was originally designed to operate this way, but in hindsight it could be considered a flaw. The 3.x branch will implement the W3C HTML JSON form submission spec which operates more in accordance with your expectations.
In 3.x you can expect
<input name="test" value="true">
<input name="test" value="false">

To serialize as
{test: ["true": "false"]}

Until then, I'm afraid this plugin may not be very useful for your particular MVC framework.

Thanks for the quick reply and the great plugin.
I have hacked my code right now to make the checkbox update the hidden field so I am still able to use your serializer.
Is there an ETA on 3.0?

@nathankoop
Sadly I've been really busy and haven't had a lot of time to direct at the 3.x code.


Feature req.: support all whatlang languages

As I understand, whichlang is based on whatlang which currently supports 68 languages. Can whichlang be extended to support them also?

That being said, the language I miss the most is Estonian.


Why is it shitty :D

A?

Because it's missing pretty much every feature that makes it preferable over marching cubes. It has no support for sharp features, can't fix edge ambiguities and isn't adaptive. Right now it pretty much generates the same output as marching cubes.


[BUG]: fix failed

Description of the bug
vercel not deploying bruh
Steps To Reproduce
https://vercel.com/account/tokens
create a token from here (hope its unlimited)
go to your github repo >> settings >> secrets and variables
make a VERCEL_TOKEN with the token content
Additional Information
No response

@LelouchFR

@hawl1 yup working on it


Print issue in chrome

I have a print css that sets .sb-left to display:none. This works fine in firefox but in chrome if you print a page with the left slidebar open the slidebar still prints.
I also have this in my print css
#sb-site {min-height:0 !important;transform:none !important;box-shadow:none !important;}
Have you come across this before? Have any suggestions?

I've managed to get rid of the slidebar content now but am still left with the space where it was.

Hey,
I've not actually looked into print styles with Slidebars, I'll definitely make sure I look into it for a future version.
When you tried to print and the space still remained, was the Slidebar open or closed?
Thank,
Adam

Open
--- Original Message ---
From: "Adam Smith"<EMAIL_ADDRESS>Sent: 30 March 2015 18:45
To: "adchsm/Slidebars"<EMAIL_ADDRESS>Cc: "barbmc"<EMAIL_ADDRESS>Subject: Re: [Slidebars] Print issue in chrome (#175)
Hey,
I've not actually looked into print styles with Slidebars, I'll definitely make sure I look into it for a future version.
When you tried to print and the space still remained, was the Slidebar open or closed?
Thank,
Adam

Reply to this email directly or view it on GitHub:
https://github.com/adchsm/Slidebars/issues/175#issuecomment-87581490

My css works fine I'm firefox but not in chrome
--- Original Message ---
From: "Adam Smith"<EMAIL_ADDRESS>Sent: 30 March 2015 18:45
To: "adchsm/Slidebars"<EMAIL_ADDRESS>Cc: "barbmc"<EMAIL_ADDRESS>Subject: Re: [Slidebars] Print issue in chrome (#175)
Hey,
I've not actually looked into print styles with Slidebars, I'll definitely make sure I look into it for a future version.
When you tried to print and the space still remained, was the Slidebar open or closed?
Thank,
Adam

Reply to this email directly or view it on GitHub:
https://github.com/adchsm/Slidebars/issues/175#issuecomment-87581490

Chrome might be printing the site at the transform: translate() amount too. Try a print style like this:
@media print {
  .sb-slidebar {
    display: none;
  }

  #sb-site. .sb-site-container {
    -webkit-transform: translate( 0px );
    transform: translate( 0px );
    left: 0;
  }
}

That worked - thanks!!!
Regards
Barbara
Date: Mon, 30 Mar 2015 01:11:56 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [Slidebars] Print issue in chrome (#175)
Chrome might be printing the site at the transform: translate() amount too. Try a print style like this:
@media print {
.sb-slidebar {
display: none;
}
#sb-site. .sb-site-container {
-webkit-transform: translate( 0px );
transform: translate( 0px );
left: 0;
}
}
—
Reply to this email directly or view it on GitHub.

I spoke too soon, it didn't work on my menu planner page
Regards
Barbara
Date: Mon, 30 Mar 2015 01:11:56 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [Slidebars] Print issue in chrome (#175)
Chrome might be printing the site at the transform: translate() amount too. Try a print style like this:
@media print {
.sb-slidebar {
display: none;
}
#sb-site. .sb-site-container {
-webkit-transform: translate( 0px );
transform: translate( 0px );
left: 0;
}
}
—
Reply to this email directly or view it on GitHub.

Yes it did sorry I didn't see the second bit. Funny I needed that for one page and not another :)
Regards
Barbara
Date: Mon, 30 Mar 2015 01:11:56 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [Slidebars] Print issue in chrome (#175)
Chrome might be printing the site at the transform: translate() amount too. Try a print style like this:
@media print {
.sb-slidebar {
display: none;
}
#sb-site. .sb-site-container {
-webkit-transform: translate( 0px );
transform: translate( 0px );
left: 0;
}
}
—
Reply to this email directly or view it on GitHub.

That's great news, glad it worked.

I know this is an IE issue, but it wass sopposed to be fixed in ie7 (I've got ie11). I've done a lot of research and tried lots of fixes that have worked for others with no joy. There's no issue in firefox or chrome.
So I thought i'd ask, just in case you had come across this.
The issue is - I have a page with a pdf in an iframe. When I scroll down, the iframe sits over the top of my navbar. See attached screenshot.
Regards
Barbara
Date: Mon, 30 Mar 2015 01:11:56 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [Slidebars] Print issue in chrome (#175)
Chrome might be printing the site at the transform: translate() amount too. Try a print style like this:
@media print {
.sb-slidebar {
display: none;
}
#sb-site. .sb-site-container {
-webkit-transform: translate( 0px );
transform: translate( 0px );
left: 0;
}
}
—
Reply to this email directly or view it on GitHub.

Sorry I can't see your attachment, but it sounds like an issue with your navbar's z-index.

oops - here it is
Regards
Barbara
Date: Sat, 25 Apr 2015 06:59:52 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [Slidebars] Print issue in chrome (#175)
Sorry I can't see your attachment, but it sounds like an issue with your navbar's z-index.
—
Reply to this email directly or view it on GitHub.

Yes I played around with the z-index thing until it was so frustrating that I gave up. Apparently IE doesn't treat z-index quite like any other browser. What a surprise :)
Regards
Barbara
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Subject: RE: [Slidebars] Print issue in chrome (#175)
Date: Sun, 26 Apr 2015 15:42:29 +1000
oops - here it is
Regards
Barbara
Date: Sat, 25 Apr 2015 06:59:52 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [Slidebars] Print issue in chrome (#175)
Sorry I can't see your attachment, but it sounds like an issue with your navbar's z-index.
—
Reply to this email directly or view it on GitHub.

Did you ever get a chance to look into this?
Cheers
Barbara
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Subject: RE: [Slidebars] Print issue in chrome (#175)
Date: Sun, 26 Apr 2015 15:45:08 +1000
Yes I played around with the z-index thing until it was so frustrating that I gave up. Apparently IE doesn't treat z-index quite like any other browser. What a surprise :)
Regards
Barbara
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Subject: RE: [Slidebars] Print issue in chrome (#175)
Date: Sun, 26 Apr 2015 15:42:29 +1000
oops - here it is
Regards
Barbara
Date: Sat, 25 Apr 2015 06:59:52 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [Slidebars] Print issue in chrome (#175)
Sorry I can't see your attachment, but it sounds like an issue with your navbar's z-index.
—
Reply to this email directly or view it on GitHub.

Nope, I still can't see your screenshot, I think you;ll need to post it here on GitHub rather than by email reply. Also Slidebars doesn't actually have a navbar, so it's hard for me to support something I didn't create.

Do you have a demo url? There is no way I can possible see whats causing that issue from the screenshot, other than I'm sure its something to do with the z-index.

Ok if you go to http://webrecipemanager.com/mywrm using internet explorer
Log in with
user mywrm
p/w mywrmpass
The recipe with the pdf should load, but if not - open the left menu and click on the mimosa salad recipe in the favorites section.
THen just scroll down the page and watch the joy :)
Cheers
Barbara
Date: Thu, 7 May 2015 05:58:56 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [Slidebars] Print issue in chrome (#175)
Do you have a demo url? There is no way I can possible see whats causing that issue from the screenshot, other than I'm sure its something to do with the z-index.
—
Reply to this email directly or view it on GitHub.

Ahh I'm sorry I don't have access to a windows computer. As far as I can see from the markup it looks all ok.
Does this link help at all?
http://stackoverflow.com/questions/12911428/z-index-does-not-work-in-ie7-ie8-with-pdf-in-iframe

Yeah tried all that - no joy unfortunately. It's driving me crazy :)
    Sent by Outlook for Android

On Fri, May 8, 2015 at 1:40 AM -0700, "Adam Smith"<EMAIL_ADDRESS>wrote:
Ahh I'm sorry I don't have access to a windows computer. As far as I can see from the markup it looks all ok.
Does this link help at all?
http://stackoverflow.com/questions/12911428/z-index-does-not-work-in-ie7-ie8-with-pdf-in-iframe

Reply to this email directly or view it on GitHub:
https://github.com/adchsm/Slidebars/issues/175#issuecomment-100154494

It didn't work for other people either. NOt sure what the difference is - that's a really simple example, and there's something somewhere, but I've run out of ideas.
Cheers
Barbara
Date: Fri, 8 May 2015 01:40:25 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [Slidebars] Print issue in chrome (#175)
Ahh I'm sorry I don't have access to a windows computer. As far as I can see from the markup it looks all ok.
Does this link help at all?
http://stackoverflow.com/questions/12911428/z-index-does-not-work-in-ie7-ie8-with-pdf-in-iframe
—
Reply to this email directly or view it on GitHub.

This one? http://stackoverflow.com/questions/9074365/youtube-video-embedded-via-iframe-ignoring-z-index

Videos are working fine for me, it's just PDF I have an issue with - do you think that is relevant?
    Sent by Outlook for Android

On Fri, May 8, 2015 at 5:58 AM -0700, "Adam Smith"<EMAIL_ADDRESS>wrote:
This one? http://stackoverflow.com/questions/9074365/youtube-video-embedded-via-iframe-ignoring-z-index

Reply to this email directly or view it on GitHub:
https://github.com/adchsm/Slidebars/issues/175#issuecomment-100225023


Tile Work Services

PORCELAIN TILE WORK IN DUBAI
We create quality from the inside out. Our ceramic and porcelain tiles installation services are superior because of our work quality and  service into each and every project we handle. To denote this internal quality and to identify our superior tile installation service, we have created marks of distinction in UAE.

GLASS TILE WORK IN DUBAI
Whether you are looking for a simple glass subway tile for your kitchen back splash or a unique glass mosaic tile accent for bathroom walls, We offer best installation service to create a striking statement in your home.

CEMENT TILE WORK IN DUBAI
Cement tiles are environmentally friendly and sustainable, just like many centuries ago. No heating is involved during the production as the tiles are not kiln-fired, but left to dry in fresh air for 21 to 28 days. We provide you with installation service for your desired designs


[8.x] Generate docblocks for real-time facades

Realtime facades are introduced way back in laravel 5.4 I think, but they are not used that often.
One of the main reasons that people get peace off is that they immediately lose IDE auto-completion when switching to a real-time facade.
This PR makes docblocks available for public non-static methods on the generated facade class in the cache folder.

We can have an artisan command to remove the generated facades to refresh the docblocks when the underlying class evolves over time.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Add support for resolvers option

Closes #85

Codecov Report

Merging #87 (d309aa3) into master (04b67c9) will decrease coverage by 1.38%.
The diff coverage is 33.34%.

@@            Coverage Diff             @@
##           master      #87      +/-   ##
==========================================
- Coverage   71.62%   70.24%   -1.37%
==========================================
  Files           8        8
  Lines         546      561      +15
==========================================
+ Hits          391      394       +3
- Misses        125      136      +11
- Partials       30       31       +1

Impacted Files
Coverage Δ

attacker/attacker.go
68.63% <ø> (ø)

main.go
70.51% <33.34%> (-4.49%)
:arrow_down:

gui/drawer.go
70.87% <0.00%> (-1.57%)
:arrow_down:

Continue to review full report at Codecov.

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

@matsuyoshi30 Hi, thanks for your contribution! I'll take a look later 😄

Sorry for the delay. I'm in the middle of something, so I'll take a look this weekend.

Thanks for great opportunity 👍


Update{Occurrence,Note} functionality with FieldMask support

Fixes #312.
I actually decided to use github.com/mennanov/fieldmask-utils for managing the field mask operations until golang has a native solution in their protobuf library. It's a well-tested package that others have had success with, but let me know if I should use something else.
I'm open to any comments or concerns!

I've been running into some weird build issues last night which started after I added new packages in (and had a cyclic dependency, which should be fixed now though). Is there a specific procedure you guys follow to vendor new packages? The go.mod isn't in the root of the project so I guessed things would be done differently

re: adding new packages
The go.mod file in the tools directory is a bit deceptive. It's used to install some proto tools as binaries. If you need to add a new package, I believe we use dep to manage the vendor directory. There are two PRs to move over to go modules, but neither have been accepted yet. In the short term I would encourage you to use dep to add your new dependency to the vendor folder.

Also we should probably add some tests for this new update behaviour, a table with a few different starting occurrences, update masks, and output occurrences (and notes) would probably be sufficient

For this, I could add unit tests to the new functions I built and update the existing Update{Occurrence, Notes} tests as integration tests?

Also we should probably add some tests for this new update behaviour, a table with a few different starting occurrences, update masks, and output occurrences (and notes) would probably be sufficient

For this, I could add unit tests to the new functions I built and update the existing Update{Occurrence, Notes} tests as integration tests?

That'd be great, thanks!

Hey @ItsMeWithTheFace, the status of the PR is shown as Draft, which means that it's not ready for full review yet? Feel free to ping this thread when you'd like us to take another look. Thanks!

Yeah I'm currently working on a more robust implementation that I'll push out very soon. I'll let you guys know once that's done :)

@aysylu, @vtsao, @clrprod, I'm very close to a full review (just need to sort out this locking issue in memstore.go) but feel free to leave any feedback on this PR!

Everything should be done now! Ready for review

Hi @ItsMeWithTheFace, thanks for your contribution! I apologize for the delay in the review, I'll be able to get to it next week. Thanks for your patience!

I wrote this for a CRUD back-end it's quite small and specific to get either map[string]interface{} or JSON.
https://github.com/mickep76/update_mask

Closing this due to inactivity since September 2019. Please feel free to open a new PR with the comments addressed from this PR.


THRIFT-5560: use junit 5 for all java unit tests

use junit 5 for all java unit tests

[x] Did you create an Apache Jira ticket?  (not required for trivial changes)
[x] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[x] Did you squash your changes to a single commit?  (not required, but preferred)
[x] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[x] If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

honestly i have no idea why just switching to JUnit 5 results in this error:
TestTSSLTransportFactoryCustomClient2 > testIt() FAILED
    org.apache.thrift.transport.TTransportException at TestTSSLTransportFactoryCustomClient2.java:32
        Caused by: java.io.IOException at TestTSSLTransportFactoryCustomClient2.java:32

but there's no stacktrace and I can't reproduce it locally.

turns out it was a race condition: some unit tests were modifying system property while others read from it. this was not an issue before because forkEvery=1 is enabled for gradle which fully isolate these side effects but the correct way is to get rid of such side effects in unit tests and then fully parallelize the unit test.

had to rebase to pickup fix from #2585

test failures seem flaky - the C++ ones are failing...

hi @ctubbsii do you still have any questions regarding this pull request? if no possibly this can be merged before:

#2581
because otherwise it'll introduce a lot of hard to resolve conflicts with this; on the other hand i can always just re-format if this pull request goes in first

hi @ctubbsii do you still have any questions regarding this pull request? if no possibly this can be merged before:

THRIFT-5568: use spotless plugin and google-java-format to enforce a consistent code format #2581

because otherwise it'll introduce a lot of hard to resolve conflicts with this; on the other hand i can always just re-format if this pull request goes in first

@Jimexist I merged this first, because I thought you were saying this PR should be merged before that one. But, now I see conflicts in that one. So, I'm wondering if you meant that one could be merged first. Apologies if I misunderstood. I can merge that one in next once the conflicts are resolved.


feat(api): added methods to launch the image picker and camera as promise

[X] Explain the motivation for making this change.
[X] Provide a test plan demonstrating that the code is solid.
[X] Match the code formatting of the rest of the codebase.
[X] Target the master branch, NOT a "stable" branch.

Motivation (required)
Easier API for those who want to work with promises instead of callbacks. I have found myself moving 'promisifying' code across multiple projects which use this library.
Test Plan (required)
New additions so only basic testing is needed as there is no risk of regression. We have ran tests within out application from our fork: https://github.com/Qeepsake/react-native-image-picker

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Force rebuild for composer 2

Composer 2 was released a few weeks ago. I think your repository doesn't need any updates for this, but a rebuild of the images should be enough. Is this something that will happen soon? Thanks!

Hi there 👋
Thanks for letting me know, the images have been rebuilt successfully.
Could you please let me know if that resolves the issue? 🙂

@lorisleiva How to use composer V1 via your images?

I suppose you would need to run the script composer self-update --1 before your script.

Sorry for my late response. The composer steps in my builds are going a lot quicker since the rebuild of the image, so I think is a success. Thanks for the quick response!


Adds set_pod_key action

Fixes https://github.com/fastlane/fastlane/issues/4408

Looks great, thanks for the PR 👍 I added one comment before it's ready to be merged

Done! ❤

Looks great, thanks for updating. One more comment. Also, could you squash the commits to one? 👍

Oh and I see we have conflicts 😢

Fixed the conflicts and rebased 💯

This is great, thanks @marcelofabri 👍

Awesome contribution, this will be in the next release 🚀


I need help merging other ogar servers into eachother to create a amazing server will anyone help?

Invalid and nothing to do with project. Please actually make a real issue.

I did before with by adding gamemodes from other servers
You have to make sure they use the same sort of structure or you have to go about re-writing the whole thing


Fixed Multiple Tabs Issue

Added Z-Index as well

@abeersaqib Thanks for your effort. It seems logical to listen cy.on('resize', function(){ instead of  $(window).bind('resize', function () { to me. I asked Max if there is any  inconvenience of this change. Please see my comment in #23.
In this extension we build cytoscape-expand-collapse.js automatically from the source files (by running gulp build command). Therefore, we need to make the changes in source files instead of  cytoscape-expand-collapse.js and then build that file.
I think that in your commit, named as 'Fix of Issue-23', the main change is 'https://github.com/iVis-at-Bilkent/cytoscape.js-expand-collapse/pull/24/commits/9e200fc632ca9471d00f64fe31a00d1f894f3324#diff-3fb9b12b337f91715905334068592531R81'. However, do we need the other changes there? Does your problem reappear when  you undo the other changes in that commit.

Yes we need the other changes since when ever there are multiple tabs other canvas become of size 100*100 since our multiple tabs are the instances of our desktop based application so window.resize wont trigger for us what triggering for us is the cy.resize due to layout resizing

@abeersaqib OK, I understand why we need to call sizeCanvas() on cy.resize event. However, what I mean by other changes are

The one in the following line 'https://github.com/iVis-at-Bilkent/cytoscape.js-expand-collapse/pull/24/commits/9e200fc632ca9471d00f64fe31a00d1f894f3324#diff-3fb9b12b337f91715905334068592531R43'
I think that we have one canvas instance here which is the varaible '$canvas' and handling it in sizeCanvas() it enough. I could not figure out why we should traverse each of $('.expand-collapse-canvas') instead.

The other one is the one in the following lines
https://github.com/iVis-at-Bilkent/cytoscape.js-expand-collapse/pull/24/commits/9e200fc632ca9471d00f64fe31a00d1f894f3324#diff-3fb9b12b337f91715905334068592531R153 and https://github.com/iVis-at-Bilkent/cytoscape.js-expand-collapse/pull/24/commits/9e200fc632ca9471d00f64fe31a00d1f894f3324#diff-3fb9b12b337f91715905334068592531R605. I think that we already have access to cy instance on these functions and do not need to parametrize it.

Yeah these lines can be ignored they were the changes made before cy is parametrized

@metincansiper I am preparing the jsfiddle it will take some time

@abeersaqib okey thaks.

@metincansiper I've just checked this problem is occurred when we use cytoscape as a Desktop Application
i am currently using the Electron if i wont add that line the children wont be removed and still clickiable and visible even after the collapse
While on web version the children are removed properly.
if you need the video demo / electron app demo which ever suits you ?

Implemented in https://github.com/iVis-at-Bilkent/cytoscape.js-expand-collapse/pull/52.


update version bump script by using "sed"

Signed-off-by: YCShen1010<EMAIL_ADDRESS>Now the version bump script next-csv.sh is supported on both linux and mac

/cherry-pick release-ltsr

/lgtm

/cherry-pick release-ltsr


Use re2 for regex evaluations

Fixes #918

It's worth noting that this increases the size of the image quite substantially. Need to look into whether some of these things can be done in another step of a multi step docker build perhaps?
> docker inspect -f "{{ .Size }}" flagsmith/flagsmith-api:main
433365795

> docker inspect -f "{{ .Size }}" flagsmith/flagsmith-api:improvement-use-google-re2
712845968

We cant we use pyre2? It installs without needing to be compiled when I build the docker image?

We cant we use pyre2? It installs without needing to be compiled when I build the docker image?

You mean just pip install pyre2 ? Does it actually work though?

The root Dockerfile needs updating too

Why are we not just putting this in requirements.in?

Why are we not just putting this in requirements.in?

Because it doesn't install on mac.


feat: Add customization point for footer logo title

Summary of Changes
In addition to a customization point for the header logo title (58472343), the footer's logo title should also be customizable.
Requested by: https://github.com/amundsen-io/amundsenfrontendlibrary/pull/953#issuecomment-831803247
Tests
Tests for the getter method are added.
CheckList
Make sure you have checked all steps below to ensure a timely review.

[ ] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[ ] PR includes a summary of changes.
[ ] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

@Golodhros No problem. That sounds reasonable. I just thought my Pull Request was too ad-hoc, and adding more custom points makes Amundsen complex unnecessarily. It does not make users happy, as well as contributors.
One thing I am a little worried about is the similar Pull Requests will be made again by other contributors because many users have similar needs... We need some statements from project owners about frontend customization.

Thanks for your understanding!

One thing I am a little worried about is the similar Pull Requests will be made again by other contributors because many users have similar needs... We need some statements from project owners about frontend customization.

That's a good point! I will try to find a spot to mention this in our docs or in the code itself.


Updated package version to 0.1.16

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Fixing old 404 pages [Removed]

Please fix the rest of the comments i gave

i did

Finally it gets merged!

Yeah lol


Merge pull request #1 from apache/master

update

Can one of the admins verify this patch?

Hi @hxfeng did you mean to file this?  If not, mind closing it?

It looks like this was opened by mistake; do you mind closing this issue?


Client verbose logging does not log sufficient information to diagnose issues interacting with rest sources.

In addition to logging that the client is sending GET and POST requests to specific endpoints and the http response code received back, we should log the full request response when verbose logging is enabled.  This will allow us to diagnose issues purely from logging and not require live repros of any issue.

I've been having a bunch of issues trying to figure out what command hits what endpoint on the current release of winget. Better logging would help a ton.

FYI "--verbose-logs" is the flag to turn verbose logs on per command. We're aiming to get another build out soon[tm].


allow events to be canceled before coming into PIXI

This fixes the issue reported here: #1763
Basically I'm checking the originalEvent.defaultPrevented to flag events as stopped and binding mousemove events on interactionDOMElement  instead of window (which is the first level of event propagation and can't be prevented/stopped)
Live example of this patch: http://jsfiddle.net/endel/xnmy239h/1/

Closing this based on my comment feedback.

Got it, thanks for your feedback!


please provide a way to configure up front so timezone defaults to

(in my case, Pacific)

Expr should just have a key/value store with things like TimeZone. Then Expr.get and Expr.set could access them.

It looks like time zones are pretty much impossible to deal with is standard JS. There are libraries that purport to Do The Right Thing, but I don't know if we can use them in GAS.

This was rather scary, but it looks like GAS is now JS compliant (instead of Excel compliant, which is what that 1899 business is all about...)

Maybe the best we can do here is make_timestamp and make_timestamptz The former would use UTC and the latter would use a specific time zone setting by  default.

One way to skin this would be to use Date string parser and feed it ISO times with offsets:
const actual = new Date('2022-08-31 08:59:51 -0700');
const expected = new Date(Date.UTC(2022, 7, 31, 15, 59, 51));
expect(actual).to.deep.equal(expected);

Unfortunately, this relies on Date.parse, which is known to be inconsistent across browsers.

Wow Apps Script support for dates is so bad people suggest writing your own parser! Easy enough with a RegExp but DST is still an issue without a library.


Review tagui update command to automatically fetch latest version of TagUI - done

This could be very useful. So we can make a cron and fetch updates more regularly.
In addition i think we could have a tagui -version or even a tagui command alone give back the version number as a response

Thanks @steewel, oh yes, when you type tagui without options and parameters it will show the version and options list.

Added your notes from telegram group chat -

about overwriting custom codes over /src... is really something to worry about, but I think if there is a warning or a backup / restore command, that possible problem can be mitigated

Added PHP code to compare versions and download from master, and also initial entry points from tagui / tagui.cmd -
// return TagUI version of given runner file
function tagui_version($tagui_runner_file) {
$tagui_runner = file_get_contents($tagui_runner_file);
$left_anchor = 'echo "tagui v'; $right_anchor = ': ';
$version_start_pos = strpos($tagui_runner, $left_anchor) + strlen($left_anchor);
$tagui_version = substr($tagui_runner, $version_start_pos);
$version_end_pos = strpos($tagui_version, $right_anchor);
$tagui_version = substr($tagui_version, 0, $version_end_pos); return $tagui_version;}

$tagui_master_version = tagui_version('https://raw.githubusercontent.com/kelaberetiv/TagUI/master/src/tagui');
$tagui_local_version = tagui_version(dirname(__FILE__) . DIRECTORY_SEPARATOR . 'tagui');

if (floatval($tagui_local_version) == floatval($tagui_master_version))
echo "You are already using the latest version of TagUI - v" . $tagui_master_version . "\n";
else {echo "Downloading and updating to latest version of TagUI - v" . $tagui_master_version . "\n";

file_put_contents(dirname(dirname(__FILE__)) . DIRECTORY_SEPARATOR . "TagUI-master_v" . $tagui_master_version . ".zip",
fopen("https://github.com/kelaberetiv/TagUI/archive/refs/heads/master.zip", 'r'));

Next would be

unzipping and setting file permissions (at least for macOS and Linux, to have executable flag)
iterating and validating entry points to call php -q tagui_update.php in tagui and tagui.cmd

need to find another path, default PHP does not have the ZipArchive class
<?php

// return TagUI version of given runner file
function tagui_version($tagui_runner_file) {
$tagui_runner = file_get_contents($tagui_runner_file);
$left_anchor = 'echo "tagui v'; $right_anchor = ': ';
$version_start_pos = strpos($tagui_runner, $left_anchor) + strlen($left_anchor);
$tagui_version = substr($tagui_runner, $version_start_pos);
$version_end_pos = strpos($tagui_version, $right_anchor);
$tagui_version = substr($tagui_version, 0, $version_end_pos); return $tagui_version;}

// retrieve TagUI versions for local and master
$tagui_master_version = tagui_version('https://raw.githubusercontent.com/kelaberetiv/TagUI/master/src/tagui');
$tagui_local_version = tagui_version(dirname(__FILE__) . DIRECTORY_SEPARATOR . 'tagui');

// download and update if there is a new version
if (floatval($tagui_local_version) == floatval($tagui_master_version))
echo "You are already using the latest version of TagUI (v" . $tagui_master_version . ")\n";
else {echo "Downloading and updating to latest version of TagUI (v" . $tagui_master_version . ")\n";
$zip_file = dirname(dirname(__FILE__)).DIRECTORY_SEPARATOR."TagUI-master-v".$tagui_master_version.".zip";
if (file_put_contents($zip_file,fopen("https://github.com/kelaberetiv/TagUI/archive/refs/heads/master.zip",'r')) == false)
echo "ERROR - cannot download new TagUI version from below GitHub URL\n" .
"https://github.com/kelaberetiv/TagUI/archive/refs/heads/master.zip\n";
else {$zip = new ZipArchive; if ($zip->open($zip_file) === true) {$zip->extractTo(dirname($zip_file)); $zip->close();
echo "TagUI updated (v" . $tagui_master_version . ")\n";} else echo "ERROR - cannot unzip ".basename($zip_file)."\n";}}

?>

Seems to find a working solution by doing the entry in shell, version detection and downloading in php, and unzipping in shell. checklist of test cases before this feature can be considered done.

[ ] version older
[ ] version newer
[ ] version same
[ ] no web connection
[ ] can't access URL
[ ] files already exist
[ ] folders already exist
[ ] post update can run
[ ] repeat above for linux
[ ] repeat above for windows

working implementation for mac and linux fully in shell in tagui/src/tagui -
note that this version of update command does not handle major upgrade where a full package zip (100+ MB) is needed including dependencies like SikuliX. this handles version updates directly retrieving from GitHub source code.
# go into download and update process if tagui update is given
if [ "$1" = "update" ]; then
SOURCE="${BASH_SOURCE[0]}" # slimerjs's implementation instead of cd "`dirname "$0"`"
while [ -h "$SOURCE" ]; do TAGUI_PARENT_DIR="$( cd -P "$( dirname "$SOURCE" )" && pwd )"
SOURCE="$(readlink "$SOURCE")"; [[ $SOURCE != /* ]] && SOURCE="$TAGUI_PARENT_DIR/$SOURCE"; done
TAGUI_PARENT_DIR="$( cd -P "$( dirname "$SOURCE" )/.." && pwd )"
if [ -f "$TAGUI_PARENT_DIR/TagUI-master.zip" ]; then rm "$TAGUI_PARENT_DIR/TagUI-master.zip"; fi
if [ -f "$TAGUI_PARENT_DIR/src/tagui_master" ]; then rm "$TAGUI_PARENT_DIR/src/tagui_master"; fi

curl -s -o "$TAGUI_PARENT_DIR/src/tagui_master" "https://raw.githubusercontent.com/kelaberetiv/TagUI/master/src/tagui"
if [ -f "$TAGUI_PARENT_DIR/src/tagui_master" ]; then
if ! grep -iq 'echo "tagui v' "$TAGUI_PARENT_DIR/src/tagui_master"; then
rm "$TAGUI_PARENT_DIR/src/tagui_master"; fi; fi
if ! [ -f "$TAGUI_PARENT_DIR/src/tagui_master" ]; then
echo "ERROR - internet connection to access GitHub is required to update TagUI"; exit 1; fi

TAGUI_MASTER_VERSION="$(grep 'echo "tagui v' "$TAGUI_PARENT_DIR/src/tagui_master" | cut -d' ' -f 3 | head -n 1)";
TAGUI_LOCAL_VERSION="$(grep 'echo "tagui v' "$TAGUI_PARENT_DIR/src/tagui" | cut -d' ' -f 3 | head -n 1)";
rm "$TAGUI_PARENT_DIR/src/tagui_master"
if [[ "$TAGUI_MASTER_VERSION" == "$TAGUI_LOCAL_VERSION" ]]; then
echo "You are already using the latest version of TagUI (${TAGUI_LOCAL_VERSION%?})"; exit 0; fi

echo "Downloading and updating to latest version of TagUI (${TAGUI_MASTER_VERSION%?})"
curl -s -L -o "$TAGUI_PARENT_DIR/TagUI-master.zip" "https://github.com/kelaberetiv/TagUI/archive/refs/heads/master.zip"
if [ -f "$TAGUI_PARENT_DIR/TagUI-master.zip" ]; then
if grep -iq "not found\|bad request" "$TAGUI_PARENT_DIR/TagUI-master.zip"; then
rm "$TAGUI_PARENT_DIR/TagUI-master.zip"; fi; fi
if ! [ -f "$TAGUI_PARENT_DIR/TagUI-master.zip" ]; then
echo "ERROR - cannot download from https://github.com/kelaberetiv/TagUI/archive/refs/heads/master.zip"; exit 1; fi

unzip -qq "$TAGUI_PARENT_DIR/TagUI-master.zip" -d "$TAGUI_PARENT_DIR"; \cp -R "$TAGUI_PARENT_DIR/TagUI-master/src" "$TAGUI_PARENT_DIR/."; \cp -R "$TAGUI_PARENT_DIR/TagUI-master/flows" "$TAGUI_PARENT_DIR/."; \cp -R "$TAGUI_PARENT_DIR/TagUI-master/docs" "$TAGUI_PARENT_DIR/."; \cp "$TAGUI_PARENT_DIR/TagUI-master/README.md" "$TAGUI_PARENT_DIR/."; \cp "$TAGUI_PARENT_DIR/TagUI-master/LICENSE.md" "$TAGUI_PARENT_DIR/."; chmod -R 755 "$TAGUI_PARENT_DIR"; if [ -d "$TAGUI_PARENT_DIR/TagUI-master" ]; then \rm -R "$TAGUI_PARENT_DIR/TagUI-master"; echo "TagUI successfully updated at $TAGUI_PARENT_DIR"; fi; exit 0

# end of if block to handle download and update process
fi

Above commit adds this. Users can download the latest copy of TagUI with this and unzip to overwrite existing installation (please drag the folders under tagui\src to overwrite your existing installation) - https://github.com/kelaberetiv/TagUI/archive/master.zip
In the next release, this will become part of the packaged zip files.

Closing issue, change included in latest packaged release - https://github.com/kelaberetiv/TagUI/releases/tag/v6.46.0