Fatal Exception: NSInvalidArgumentException when retrieving health data

Thanks for creating this library.
Describe the bug
After migrating an app from rn-apple-healthkit to react-native-health there have been a large number of crashes reported in Crashlytics. The crash seems to occur when health data is received. However I have been unable to reproduce this error myself on an iPhone simulator or a physical device.
This is the error:
Fatal Exception: NSInvalidArgumentException ***[__NSPlaceholderDictionary initWithObjects:forKeys:count:]: attempt to insert nil object from objects[1]
Which seems to happen inside the completionHandler in the method fetchSumOfSamplesOnDayForType. This method is called from many other methods that retrieve various types of health data such as getStepCountOnDay.
Where one of these values in *response is nil, which due to the if-statement should be startDate or endDate. And buildISO8601StringFromDate can return nil if there is an error.
Inside the completionHandler:
if (!value && value != 0) {
            callback(@[RCTJSErrorFromNSError(error)]);
            return;
        }

         NSDictionary *response = @{
                 @"value" : @(value),
                 @"startDate" : [RCTAppleHealthKit buildISO8601StringFromDate:startDate],
                 @"endDate" : [RCTAppleHealthKit buildISO8601StringFromDate:endDate],
         };

        callback(@[[NSNull null], response]);

Do you have an idea what might cause this error?
These are the permissions that I use:
const appleHealthKitOptions = {
    permissions: {
        read: [
            AppleHealthKit.Constants.Permissions.StepCount,
            AppleHealthKit.Constants.Permissions.SleepAnalysis,
            AppleHealthKit.Constants.Permissions.HeartRate,
            AppleHealthKit.Constants.Permissions.DistanceCycling
        ],
        write: []
    }
};

And these AppleHealthKit methods:
AppleHealthKit.getDailyStepCountSamples
AppleHealthKit.getDailyDistanceCyclingSamples
AppleHealthKit.getSleepSamples
AppleHealthKit.getHeartRateSamples

To Reproduce
Unfortunately I cannot reproduce it myself.
Smartphone (please complete the following information):

Device: It happens on many different types of iPhones
OS: Both iOS 13 and 14
Version 1.7.0

Thanks a lot.

Hey @rnbrk, really weird, I could not reproduce this as well, please share more information if you have

Also running into this. Seems to only be when fetching StepCount
*** -[__NSPlaceholderDictionary initWithObjects:forKeys:count:]: attempt to insert nil object from objects[1]

@macelai  We are also having the same exact issue and not able to reproduce it. But this can potentially be fixed by just having default values for the cases when start or end date are null in the buildISO8601StringFromDate assignment right? Since this is in the method fetchSumOfSamplesOnDayForType, maybe using the "day" as the start and end date? Wdyt?

We are facing the same issue, any news on this? We are also using Version 1.7 so is there a chance that this is fixed in the latest release?

@lucaspbordignon I am using 1.12 version of this library and my users get the same crashes every day. I don't understand the reason. I can't reproduce this bug but see them on bugsnag. Any solution, advice? Maybe these bugs are related to iOS 15 (and later) updates?
@c-goettert @rnbrk  Did you find any working solution to prevent this bug?

@gigby unfortunatley no, we couldn't find a solution / reason for this bug yet..

@c-goettert  I found out. It happens on iOS 15.2 or later due to prewarming feature. You can easily awoid these bugs if you don't call Healthkit from a background app state. No background calls no these bugs

@gigby thanks for letting me know, we indeed use background calls to fetch health data. We'll deactivate those and see if this will solve the issue.

@gigby unfortunatly we still see this issue after disabling our background task. How did you make sure that Healthkit is not called from a background state? Did you use AppState-Events from React Native or did you just remove any existing Background Observers? Or did you remove Background-Fetch Entitlement from your app completely?

@c-goettert
And somewhere in code I use previous func to get some data like this:
export async function getSleepHours(timeFrom?: number | Date, timeTo: number | Date = Date.now()) {
  const result: Array<TSleepEventRawIos> = [];

  const isGranted = await initHealthkit([AppleHealthKit.Constants.Permissions.SleepAnalysis]);

  if (isGranted) {
    let timeStart;
   // code here
  }

@c-goettert Were you able to look into the thing mentioned above by @gigby?
I'm getting this same issue but not on every phone that runs this code. I'm not sure if it's something related to initializing apple health after checking if the app isn't in the background as our code runs when the app is open.

Any update on this bug? We are receiving this error mostly on iOS 15, and few in iOS 14. We do not intentionally call any method from background state.

Having the same issue

+1 Looks like mostly users on iOS 15

We are also still facing this issue. For the next release I will try @gigby's solution to check if the app is in foreground before interacting with HealthKit.

We're getting the same error/crash when calling AppleHealthKit.getStepCount. Did anyone manage to find a fix/workaround?

I'm having this issue, did somebody figure it out?

Hey guys, I plan to take a deep look into this crash in a few days. If you have any additional information it would be very welcome, thanks

Thank you for looking into this, we are still regularly getting lots of these in our error reporting. For us I can say that it only seems to occur when the app runs in background mode. It might be somehow related to the iOS rights system, e.g. that values may not be accessed while the phone is locked or similar. This would fit the error messages we get from the lib before the exception:

Error while trying to get blood pressure samples from healthkit: , error getting blood pressure samples:Error Domain=com.apple.healthkit Code=6 "Protected health data is inaccessible" UserInfo={NSLocalizedDescription=Protected health data is inaccessible}

This is what an example stack-trace looks like (probably not very helpful):
libobjc.A.dylib                     0x000000019a6c4f54 objc_exception_throw + 60
CoreFoundation                      0x0000000182159530 B2D21CFD-378C-36D5-BAF7-3F70599CFEFC + 1713456
CoreFoundation                      0x00000001821646ac B2D21CFD-378C-36D5-BAF7-3F70599CFEFC + 1758892
CoreFoundation                      0x000000018200d464 B2D21CFD-378C-36D5-BAF7-3F70599CFEFC + 353380
CoreFoundation                      0x000000018202be0c B2D21CFD-378C-36D5-BAF7-3F70599CFEFC + 478732
App                                 0x00000001022e4744 _ZN5folly6detail15str_to_integralIxEENS_8ExpectedIT_NS_14ConversionCodeEEEPNS_5RangeIPKcEE + 161376
App                                 0x00000001022f1eb8 _ZN5folly6detail15str_to_integralIxEENS_8ExpectedIT_NS_14ConversionCodeEEEPNS_5RangeIPKcEE + 216532
libdispatch.dylib                   0x0000000181cc0914 A5CBAAB3-E389-3548-BAAC-FAB18411B94A + 10516
libdispatch.dylib                   0x0000000181cc2660 A5CBAAB3-E389-3548-BAAC-FAB18411B94A + 18016
libdispatch.dylib                   0x0000000181cc9de4 A5CBAAB3-E389-3548-BAAC-FAB18411B94A + 48612
libdispatch.dylib                   0x0000000181cca98c A5CBAAB3-E389-3548-BAAC-FAB18411B94A + 51596
libdispatch.dylib                   0x0000000181cd51a8 A5CBAAB3-E389-3548-BAAC-FAB18411B94A + 94632
libsystem_pthread.dylib             0x00000001f23620f4 _pthread_wqthread + 288
libsystem_pthread.dylib             0x00000001f2361e94 start_wqthread + 8```

Fixed by #280

I fixed #97 and I believe it will fix this issue too. Please update to v1.13.6 and let me know if the issue persists.


How this player works?

Hey! May you explain to me how this player works? what happens when I insert a link to player?

Hi @INTERNALINTERFERENCE, most of the work is done in Sync.razor in the client.
When you add a video, the first thing we do is call Send()
https://github.com/brasky/OpenSync/blob/master/OpenSync/Client/Pages/Sync.razor#L416
This does a few things, but most importantly it adds the video to a playlist and tells the other clients about the new video by telling the server (also called "Hub") about the new video https://github.com/brasky/OpenSync/blob/master/OpenSync/Client/Pages/Sync.razor#L443

thanks! btw you can check my version  :3


Implement the op Add and Cast

Also create the script to generate the Boilerplate of ONNX op implementation.

@jinning-li  PTAL


as_ctypes function for Lambdify

@richardotis, can you use these ctypes functions? You shouldn't need to access symengine Cython layer and can do it in pure python.

Can you post a simple example of how to use this? My earlier solution looked similar, but I couldn't figure out how to get the function pointer back in the downstream code.

Here's how to do this in pure python.
In [1]: import numpy as np

In [2]: from symengine import *

In [3]: var("x y z")
Out[3]: (x, y, z)

In [4]: l = Lambdify([x, y, z], [x+y+z, x*y*z+1])

In [5]: addr1, addr2 = l.as_ctypes()

In [6]: import ctypes

In [7]: inp = np.array([1,2,3], dtype=np.double)

In [8]: out = np.array([0, 0], dtype=np.double)

In [9]: addr1(out.ctypes.data_as(ctypes.POINTER(ctypes.c_double)), inp.ctypes.data_as(ctypes.POINTER(ctypes.c_double)), addr2)

In [10]: out
Out[10]: array([6., 7.])

To do this in cython,
In [11]: %load_ext cython

In [12]: %%cython
    ...:
    ...: ctypedef void (*math_function_t)(double*, const double*, void* user_data) nogil
    ...:
    ...: import ctypes
    ...:
    ...: def call_lambdify(double[::1] out, double[::1] inp, l):
    ...:     addr1, addr2 = l.as_ctypes()
    ...:     cdef math_function_t cy_f_ptr = (<math_function_t*><size_t>ctypes.addressof(addr1))[0]
    ...:
    ...:     cdef void * data =  (<void**><size_t>ctypes.addressof(addr2))[0]
    ...:     with nogil:
    ...:         cy_f_ptr(&out[0], &inp[0], data)
    ...:
    ...:
warning: /home/isuru/.cache/ipython/cython/_cython_magic_b3ffdcd6b0cb2c599014b3a4acdd137a.pyx:12:21: Use boundscheck(False) for faster access
warning: /home/isuru/.cache/ipython/cython/_cython_magic_b3ffdcd6b0cb2c599014b3a4acdd137a.pyx:12:30: Use boundscheck(False) for faster access

@isuruf Thanks, this works great! I'd support merging this and reverting #291 in light of the footguns involving build-time paths to symengine.
Here is the downstream implementation I've put together which uses this:
.pxd
ctypedef void (*math_function_t)(double*, const double*, void* user_data) nogil

cdef class FastFunction:
    cdef readonly object _objref
    cdef math_function_t f_ptr
    cdef void *func_data
    cdef void call(self, double *out, double *inp) nogil

.pyx
cdef class FastFunction:
    def __cinit__(self, object func):
        if func is None:
            self.f_ptr = NULL
            self.func_data = NULL
            return
        # Preserve reference to object to prevent garbage collection
        self._objref = func
        addr1, addr2 = func.as_ctypes()
        self.f_ptr = (<math_function_t*><size_t>ctypes.addressof(addr1))[0]
        self.func_data =  (<void**><size_t>ctypes.addressof(addr2))[0]
    cdef void call(self, double *out, double *inp) nogil:
        if self.f_ptr != NULL:
            self.f_ptr(out, inp, self.func_data)

Upon re-reading I think we need to keep addr1 and addr2 around as well:
.pxd
cdef class FastFunction:
    cdef readonly object _objref, _addr1, _addr2
    cdef math_function_t f_ptr
    cdef void *func_data
    cdef void call(self, double *out, double *inp) nogil

.pyx
cdef class FastFunction:
    def __cinit__(self, object func):
        if func is None:
            self.f_ptr = NULL
            self.func_data = NULL
            return
        # Preserve reference to objects to prevent garbage collection
        self._objref = func
        self._addr1, self._addr2 = func.as_ctypes()
        self.f_ptr = (<math_function_t*><size_t>ctypes.addressof(self._addr1))[0]
        self.func_data =  (<void**><size_t>ctypes.addressof(self._addr2))[0]
    cdef void call(self, double *out, double *inp) nogil:
        if self.f_ptr != NULL:
            self.f_ptr(out, inp, self.func_data)

Upon re-reading I think we need to keep addr1 and addr2 around as well

Are you sure? These are pointers to the function (which is not deleted) and the symengine object (which is kept alive by _objref)

I am not sure. Some tests with trying to force deletion and collection of addr1 and addr2 seem to work fine, so I may just misunderstand how the pointer extraction code works (which isn't surprising, since I couldn't figure it out earlier on my own)

I think I get it now. The [0] at the end immediately dereferences the pointer to addr1/addr2 (returned by ctypes.addressof()), so now you just have a pointer to the same memory that those objects pointed to, so they can be safely deleted.

Yes, see https://stackoverflow.com/a/49635841/4768820 for a detailed answer

What's needed to merge this? A test?

Yes, and a revert of last PR


Add logger implementation

Closes #2

@radu-matei I cannot seem to run the integration test for the logger. Not sure what I am missing?!
dev (default) github/nitishm/wasi-experimental-toolkit on 🌱 nitishm/feat/2/implement-logger [📝🤷] is 📦 v0.1.0 via 🦀 v1.57.0 took 12s
🕙[ 16:04:00 ] [🔴 101] ✗  cargo test --tests wasi_log_tests
   Compiling wasi-experimental-toolkit v0.1.0 (/Users/nitishm/github/nitishm/wasi-experimental-toolkit)
    Finished test [unoptimized + debuginfo] target(s) in 7.97s
     Running unittests (target/debug/deps/wasi_experimental_toolkit-742a05f25e940d6e)

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

     Running tests/integration.rs (target/debug/deps/integration-0f8c97e522fe45c3)

running 1 test
test wasi_log_tests::test_rust_log ... FAILED

failures:

---- wasi_log_tests::test_rust_log stdout ----
thread 'wasi_log_tests::test_rust_log' panicked at 'called `Result::unwrap()` on an `Err` value: unknown import: `wasi-log::info` has not been defined', tests/integration.rs:143:63
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

failures:
    wasi_log_tests::test_rust_log

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 5 filtered out; finished in 0.31s

error: test failed, to rerun pass '--test integration'

Hi, @nitishm, thanks a lot for your contribution!
In the current model, there are two ways an interface can be satisfied:

through another component — this requires us to link the two together (and currently generate a statically linked Wasm module) before instantiating it
through a host import — this requires us to setup the runtime to handle the imports from the guest.

And this is how the existing implementations are currently split, some are Wasm components, some are host implementations.
The tests simply import interfaces — so before instantiating them, we must either satisfy the imports through the host runtime, or link them with other Wasm components that implement those interfaces.
In your case, you implemented the log interface as a Wasm component, but did not link it before actually instantiating your tests — see the cache tests for how to set the linking for the test.

Hi, @nitishm, thanks a lot for your contribution!
In the current model, there are two ways an interface can be satisfied:

through another component — this requires us to link the two together (and currently generate a statically linked Wasm module) before instantiating it
through a host import — this requires us to setup the runtime to handle the imports from the guest.

And this is how the existing implementations are currently split, some are Wasm components, some are host implementations.
The tests simply import interfaces — so before instantiating them, we must either satisfy the imports through the host runtime, or link them with other Wasm components that implement those interfaces.
In your case, you implemented the log interface as a Wasm component, but did not link it before actually instantiating your tests — see the cache tests for how to set the linking for the test.

Thanks for explaining the subtle differences. I completely glazed over the fact that I was using rust-bindgen and not wasmtime bindgen in the log crate. Nevertheless it's been good exercise and learning opportunity to understand and try the two models. I opted for the wasmtime implementation. But if required I can either switch to the manual linkage option or add that to this existing option. LMK what you think, which option is better?

This mostl LGTM.
Manually updating the revision and running the tests passes:
running 1 test
[2022-01-12T17:13:46Z ERROR log_wasmtime] To err is human to rub it in is divine
test wasi_log_tests::test_rust_log ... ok

This is great, once you clean it up, update the revision, and (please) sign your commits, this should be good to go.
Thanks a lot!


Didn't create Stripe Plan?

Hi,
Trying this out.  I got this error when trying to subscribe to my first plan:
AppPlans: There was a problem creating Stripe customer: No such plan: betaStarter [Service subscription error trying to add plan betaStarter for user WKP8qqrxxxxT7sMQT

It's from here.
Looking at the code, I think it's expecting the plan to already be made.   Even though payOptions needs an amount.
On first use, I thought the plan data would be stored in the Meteor code.  Plans could be setup with the Stripe API.
My use case was something like this (I wanted the limits to be stored with the price):
AppPlans.plans = {};

// Active Plans

AppPlans.plans.betaFreemium = {
  name: 'Freemium',
  amount: 0,
  limit: 5,
  description: 'Fremium Plan',
  action: 'Try Risk-Free',
};
AppPlans.define('betaFreemium');

AppPlans.plans.betaStarter = {
  name: 'Starter',
  amount: 1500,
  limit: 15,
  description: 'Starter Plan',
};
AppPlans.define('betaStarter', {
  services: [
    {
      name: 'stripe',
      planName: 'betaStarter',
      payOptions: _.pick(AppPlans.plans.betaStarter, 'name','description','amount'),
    }
  ],
});

Is the project under active development?

Currently you have to go into Stripe and define the plan yourself first. It may be possible to have the package create it for you. I'd accept a PR if someone figures that out.
This package is used in some production apps so the pieces used by those apps are "active" in that we would fix any issues affecting the apps that use it. No one is actively adding new features, but I can look at PRs and/or work with additional maintainers if anyone is interested.

lol, I set out to automate this, then made them manually.
There's are good business reasons to separate the plans from the code.
Also, I see now that payOptions needs the amount to get the stripe token.  Anyways, all is well :-)


fixed atrous_conv2d to work with unknown shape of the input tensor

Can one of the admins verify this patch?

I signed it!

This is great. Could you add a test to make sure it keeps working?

The test should still work. I ran them and fixed some imports. Still they are failing, because the calculated padding is somehow wrong. Currently I have no time to investigate. Maybe someone sees the cause immediately.

The current atrous_conv2d seems requiring image size should be divisible by the rate.
InvalidArgumentError: Image height 232 and width 248should be divisible by block_size: 16
	 [[Node: atrous_conv2d_7/SpaceToBatch = SpaceToBatch[T=DT_FLOAT, block_size=16, _device="/job:localhost/replica:0/task:0/cpu:0"](transpose_21, atrous_conv2d_7/concat)]]
Caused by op u'atrous_conv2d_7/SpaceToBatch', defined at:

Thanks @shampool! Works for me.

sorry, some conflicts have sprung up, can you rebase?

Seems like somebody else applied a fix in the meanhwile: https://github.com/tensorflow/tensorflow/commit/3a744a417178646712e9cda0a85655d5d8968e19


Add check for installed gh cli

This adds a check for whether the gh cli is installed before trying to use it

cc @stefanv I think this would be a more helpful error in case it's not installed, sorry about that!


Fix #598: Start with/without debugging fails if Solution Explorer has never been opened

Gracefully handle exceptions from EnvDTE.ActiveSolutionProjects.

+1


improve delete namespace

Coverage decreased (-0.05%) to 48.867% when pulling e533198e2e7f4c6061254931ee5ee0e79fd85c17 on lepdou:namespace_delete into 509ef4cf6d1c41eb443bd301aef92189de2f78c1 on ctripcorp:master.


overriding rendering methods in devise_token_auth

I am currently building a rails api and I am using your devise_token_auth.
I’ve been stuck for a while on how to render custom data for user model. Rather than getting a ison like this:
{
"data": {
"id": 316,
"provider": "email",
"uid"<EMAIL_ADDRESS>"name": "olahlou",
"nickname": "olahlou",
"email"<EMAIL_ADDRESS>"score": 2,
"image": null
}
}
Let’s say the user has_many :posts, I would like to apply a user serializer(I am using active model serializers) to render something like (without the “data” key):
{
“id”:316,
“name”:”olahlou”,
“posts”:[{
#first post,
#second post...
}]
}
I’ve read the documentation an I guess I have to override the SessionController#render_create_success but I really don’t know how to do it.
Thank you in advance for your  help.

@omar28
in your "User" model, try something like this:
def token_validation_response
  UserSerializer.root = false
  UserSerializer.new(self).as_json
end

And in your user_serializer. you can set up whatever you want to render.
Please let me know if that helps.

@omar28
def token_validation_response
  UserSerializer.root = false
  UserSerializer.new(self).as_json
end

This will render your serializer.
Now for the data thingy, you need to look at session_controller:
def render_create_success
  render json: {
    data: resource_data(resource_json: @resource.token_validation_response)
  }
end

Look at:
# config/routes.rb
Rails.application.routes.draw do
  ...
  mount_devise_token_auth_for 'User', at: 'auth', controllers: {
    sessions_controller:  'overrides/sessions_controller'
  }
end

# app/controllers/overrides/sessions_controller.rb
module Overrides
  class SessionsController < DeviseTokenAuth::TokenValidationsController
    def render_create_success
      render json: resource_data(resource_json: @resource.token_validation_response)
    end
  end
end

@omar28 Please close the issue if that worked for you. Otherwise, let me know so I can help you out ^_^

why you inheriting
class SessionsController < DeviseTokenAuth::TokenValidationsController
to override SessionsController ?
it should be class SessionsController < DeviseTokenAuth::SessionsController ?

Why resource_data method is not available when I override SessionsController?
I have this error:
NoMethodError (undefined method `resource_data' for #<Overrides::TokenAuth::SessionsController:0x007fc21abe6958>):
  app/controllers/overrides/token_auth/sessions_controller.rb:6:in `render_create_success'

@kjakub sorry! You are right. I didn't pay attention to that.
@adisos This is what I have:
class Api::V1::SessionsController < DeviseTokenAuth::SessionsController

  def resource_data(opts = {})
    opts[:resource_json] || @resource.as_json
  end

  def render_new_error
    render json: {
      errors: [I18n.t("devise_token_auth.sessions.not_supported")]
    }, status: 405
  end

  def render_create_success
    render json: {
      sponsor: resource_data(resource_json: @resource.userable.token_validation_response)
    }
  end

  def render_create_error_not_confirmed
    render json: {
      errors: [I18n.t("devise_token_auth.sessions.not_confirmed", email: @resource.email)]
    }, status: 404
  end

  def render_create_error_bad_credentials
    render json: {
      errors: [I18n.t("devise_token_auth.sessions.bad_credentials")]
    }, status: 402
  end

  def render_destroy_success
    render json: {
    }, status: 200
  end

  def render_destroy_error
    render json: {
      errors: [I18n.t("devise_token_auth.sessions.user_not_found")]
    }, status: 404
  end
end

It works fine. I made the mistake @kjakub mentioned. Use
DeviseTokenAuth::SessionsController

instead of
DeviseTokenAuth::TokenValidationsController

My bad!

I know, I solved by defining method resource_data in my controller. But this method was already defined in DeviseTokenAuth::ApplicationController from which DeviseTokenAuth::SessionsController was inherited. Sorry for offtopic, but could somebody explain me?

Well, the "why this works like that", only the creators/maintainers know. I'm here because I use it and maybe I might be useful.
I would have done somethings in a diff way, though.

Well, I found out. I'm using rubygems version, which doesn't have resource_data method.

Me too! This is strange... any one?

I think it's simple because gem author didn't update rubygems for a long time.

Take a look to this
ROUTES
mount_devise_token_auth_for 'User', at: 'auth', skip: [:omniauth_callbacks], controllers: { sessions:  'overrides/sessions' }
Controller overwrite
module Overrides
class SessionsController < DeviseTokenAuth::SessionsController
protected
def render_create_success
render json: @resource
end
end
end
Credits to Doncote, here original post https://gitter.im/lynndylanhurley/devise_token_auth/archives/2015/12/18

I solved this by:
# user.rb
def token_validation_response
    self.as_json(include: :posts)
end

This is pretty easy. The documentation at https://github.com/lynndylanhurley/devise_token_auth#overriding-rendering-methods makes it sound WAY more confusing, it would be great to add a section describing this common and much simpler case (i.e. don't need custom JSON per controller method, just in general for devise-token-auth, but don't want to change it everywhere else in the app).

@Meekohi your answer is working perfectly and solved it for me.
(Unfortunately, I tried all the other ones before scrolling all the way down )
Really hope, people will read the full thread. Thank you!


macOS-13 support

Hi is available the macos 13 support with new xcode 15?
thanks

Hi, This actions should work with all macOS images and all xcode versions which are pre-installed on GitHub Hosted images.
As I can see only Xcode 15 beta is pre-installed right now: https://github.com/actions/runner-images/blob/main/images/macos/macos-13-Readme.md#xcode
I think GitHub Actions team is going to deploy new image with Xcode 15 stable within a couple of days. As soon as it is deployed, this doc will be updated.
After that, this action will work automatically with new version.

it has been deployed: https://github.com/actions/runner-images/issues/8282#issuecomment-1733633274

Find a simpler way, in .github/actions/configure_xcode_version/action.yaml:
name: "Configure: Xcode version"

runs:
  using: "composite"
  steps:
    - name: "Configure: Xcode version"
      shell: bash
      # for Xcode path, see https://github.com/actions/runner-images/blob/main/images/macos/macos-13-Readme.md#xcode
      run: echo "DEVELOPER_DIR=/Applications/Xcode_15.0.app" >> $GITHUB_ENV


AI search targets,allies

if you want use AI search path,enemies,allies, maybe need search with options .
For example nearest target or ally, perhaps search target(as item like chest,coins,weapons).

So, the purpose of this package is only give a A* algorithm. Maybe this ideia could be a new package tht use this package.

So, the purpose of this package is only give a A* algorithm. Maybe this ideia could be a new package tht use this package.

but create grid is reusing in other type of search,
maybe diffrent packages for Grid , AStar , tbs finding, ai search

So, the purpose of this package is only give a A* algorithm. Maybe this ideia could be a new package tht use this package.

some example extension optimization when u import only one and two funtions

Astart+ findPath + findSteps this is 16 kb
Astar + findPath 14kb
why need more packages for search algoritmis ?


Clippy: too many arguments - gltf_loader: load_node

Description
Clippy Too-Many arguments for load_node (8/7)
  --> hotham/src/gltf_loader.rs:88:1
   |
88 | / fn load_node(
89 | |     node_data: &gltf::Node,
90 | |     gltf_buffer: &[u8],
91 | |     vulkan_context: &VulkanContext,
...  |
96 | |     images: &[gltf::image::Data],
97 | | ) -> Result<()> {
   | |_______________^
   |

TODO

[ ] Build up node_entity_map
[ ] Add mesh
[ ] Add children

From: #59 Discussion
It seems like there's an element of load, spawn, and insert for each node, with the latter two aspects being involved with the world

Not high priority; we can always come back to this.


Weird issue with multi socket CPUs

Describe the bug
When running si.cpu() on a multi-socket server the results are incorrect.
To Reproduce

Get multi-socket server
Run command
Profit?

Current Output
{ manufacturer: 'Intel®',
  brand: 'Xeon® X5650',
  vendor: 'GenuineIntel',
  family: '6',
  model: '44',
  stepping: '2',
  revision: '',
  voltage: '',
  speed: '2.67',
  speedmin: '',
  speedmax: '',
  cores: 24,
  physicalCores: 3,
  processors: 2,
  socket: 'LGA1366',
  cache: { l1d: 393216, l1i: 393216, l2: 3, l3: 24 } }

Expected behavior
I think this is the correct result. The server is using 2x 6 core CPUs with 2 threads per core.
{ manufacturer: 'Intel®',
  brand: 'Xeon® X5650',
  vendor: 'GenuineIntel',
  family: '6',
  model: '44',
  stepping: '2',
  revision: '',
  voltage: '',
  speed: '2.67',
  speedmin: '',
  speedmax: '',
  cores: 24,
  physicalCores: 12,
  processors: 2,
  socket: 'LGA1366',
  cache: { l1d: 393216, l1i: 393216, l2: 3, l3: 24 } }

Environment (please complete the following information):

systeminformation package version: latest
OS: macOS and Unraid
Hardware: MacBook Pro 2010 and Dell R710

Additional context
For reference the same version of si works perfect on my single-socket Macbook.
{
  manufacturer: 'Intel®',
  brand: 'Core™ i5 M 520',
  vendor: 'GenuineIntel',
  family: '6',
  model: '37',
  stepping: '5',
  revision: '',
  voltage: '',
  speed: '2.40',
  speedmin: '2.40',
  speedmax: '2.40',
  cores: 4,
  physicalCores: 2,
  processors: 1,
  socket: '',
  cache: { l1d: 32768, l1i: 32768, l2: 262144, l3: 3145728 }

@OmgImAlexis can you provide the output of the following:
cat /proc/cpuinfo | grep -E "physical id|core id"

as well as
lscpu

Thank you in advance!

Let me know if you need anything else.
physical id     : 1
core id         : 0
physical id     : 0
core id         : 0
physical id     : 1
core id         : 1
physical id     : 0
core id         : 1
physical id     : 1
core id         : 2
physical id     : 0
core id         : 2
physical id     : 1
core id         : 8
physical id     : 0
core id         : 8
physical id     : 1
core id         : 9
physical id     : 0
core id         : 9
physical id     : 1
core id         : 10
physical id     : 0
core id         : 10
physical id     : 1
core id         : 0
physical id     : 0
core id         : 0
physical id     : 1
core id         : 1
physical id     : 0
core id         : 1
physical id     : 1
core id         : 2
physical id     : 0
core id         : 2
physical id     : 1
core id         : 8
physical id     : 0
core id         : 8
physical id     : 1
core id         : 9
physical id     : 0
core id         : 9
physical id     : 1
core id         : 10
physical id     : 0
core id         : 10

Architecture:                    x86_64
CPU op-mode(s):                  32-bit, 64-bit
Byte Order:                      Little Endian
Address sizes:                   40 bits physical, 48 bits virtual
CPU(s):                          24
On-line CPU(s) list:             0-23
Thread(s) per core:              2
Core(s) per socket:              6
Socket(s):                       2
NUMA node(s):                    2
Vendor ID:                       GenuineIntel
CPU family:                      6
Model:                           44
Model name:                      Intel(R) Xeon(R) CPU           X5650  @ 2.67GHz
Stepping:                        2
CPU MHz:                         1595.525
BogoMIPS:                        3191.91
Virtualization:                  VT-x
L1d cache:                       384 KiB
L1i cache:                       384 KiB
L2 cache:                        3 MiB
L3 cache:                        24 MiB
NUMA node0 CPU(s):               0,2,4,6,8,10,12,14,16,18,20,22
NUMA node1 CPU(s):               1,3,5,7,9,11,13,15,17,19,21,23
Vulnerability L1tf:              Mitigation; PTE Inversion; VMX conditional cache flushes, SMT vulnerable
Vulnerability Mds:               Vulnerable: Clear CPU buffers attempted, no microcode; SMT vulnerable
Vulnerability Meltdown:          Mitigation; PTI
Vulnerability Spec store bypass: Mitigation; Speculative Store Bypass disabled via prctl and seccomp
Vulnerability Spectre v1:        Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:        Mitigation; Full generic retpoline, IBPB conditional, IBRS_FW, STIBP conditional, RSB filling
Flags:                           fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc arch_perfmon pe
                                 bs bts rep_good nopl xtopology nonstop_tsc cpuid aperfmperf pni pclmulqdq dtes64 monitor ds_cpl vmx smx est tm2 ssse3 cx16 xtpr pdcm pcid dca sse4_1 sse4_2 popcnt aes lahf_lm pt
                                 i ssbd ibrs ibpb stibp tpr_shadow vnmi flexpriority ept vpid dtherm ida arat flush_l1d

These are identical CPUs as well. You may want to display the output differently for multi-socket systems as they may have different CPUs in each socket.

@OmgImAlexis ok, I know where the problem is ... will provide a solution mid of next week as I need to finish a project till Tuesday

No worries at all. Thanks for the quick updates as usual.

@OmgImAlexis I changed the code now (not yet released on npm). Can you test it on your side? you only need to grab lib/cpu.js from github and replace your existing one. If this returns good results, I will publish this on npm. Thank you in advance!!

New output looks better.
{ manufacturer: 'Intel®',
  brand: 'Xeon® X5650',
  vendor: 'GenuineIntel',
  family: '6',
  model: '44',
  stepping: '2',
  revision: '',
  voltage: '',
  speed: '2.67',
  speedmin: '',
  speedmax: '',
  cores: 24,
  physicalCores: 12,
  processors: 2,
  socket: 'LGA1366',
  cache: { l1d: 393216, l1i: 393216, l2: 3, l3: 24 } }

@OmgImAlexis - thank you for testing! New version published on npm v4.14.12


Remove dependency override for analyzer

Remove dependency_overrides for analyzer 3.3.1

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 91.841%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
394

Relevant Lines:
429

💛  - Coveralls

I would prefer you simply reverted 32ac88d

I would prefer you simply reverted 32ac88d

I wanted to be sure that all the tests will pass with their fix. That's way created a new PR.


Add visualization function

You can visualize pointcloud result when you test the model. In tools/pointcept/engines/test.py，you can read the motifications I made. Besides, the pointcloud is colored with its class.

May I ask the reason why I put a copy of the whole Pointcept in the tool folder? (Check the "files changed")

May I ask the reason why I put a copy of the whole Pointcept in the tool folder? (Check the "files changed")

Oh, that may be my mistake. Perhaps I installed pointcept for environment setting in a wrong working directory.
Whats more, i found that I need to write a function instead of realizing it in test.py. I will modify it when I get time.


Replace unsigned long with uint32_t

Replace unsigned long with uint32_t
Connects to ament/ament_lint#34

+1


Add winAutoSuggestBox directive

Add winAutoSuggestBox directive to make it easy to use new
AutoSuggestBox from WinJS 4.0 preview. Also add tests and an example
to README.md.

Hi @tro, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, MSBOT;

Hey @tro, thanks for this pull request! It's looking great. If you could make the two changes I've called out here, I'll be happy to merge this in

Thanks for the review @TheWrathOfConst. I did the changes you requested.

Merged. Thanks for contributing!


cant proceed

zefoy.com just loops pn the captcha

Change DNS brother

Hello. A new update was just pushed. Please check it and let me know if everything's working good.


Broken links in CONTRIBUTING.md

Context
Running Safari 11.0.2 on macOS
Expected Behavior
The Issue Template and Pull Request Template should go to the correct page on Github
Current Behavior
What is the current behavior?
These links:
Issue Template/Checklist
Pull Request template/Checklist
are outdated, leading to a 404 error.

I will update these as they are a simple fix requiring the new link.

Thanks for your help!It's quite interesting as I thought the links were well-formed from a Markdown perspective. I made the mistake of using local paths, but forgot the document was stored in the docs folder :)
The CODE_OF_CONDUCT.md link is also broken, could you include that as well?

I will fix other repositories with the same failure


Title in Xwindows seems to be set after window is realized

FVWM does it's window 'styling' (decorations and other window specific config) based on the discoverable data when a window is realized (or when FVWM is forced to reprobe via a config reload).
When freerdp is called with /t:'something' it realizes the window first and then changes the window title to match 'something'  This causes FVWM to be unable to match 'something' in order to behave differently according to different freerdp sessions with different window titles without also forcing FVWM to reload the config every time freerdp is called.  That's annoying and innefficient and means certain behaviors are very difficult to force without an inordinate amount of scripting.
Please set the X window title prior to realizing the freerdp window.
I'm running freerdp on Arch:  ~> xfreerdp -version
This is FreeRDP version 1.2.0-beta1 (git n/a)

Fixed with #5023


Target net461 and netstandard2.0 in addition to net6.0

Dropping this as it's going to take some effort to setup tests for .NET Framework in GitHub Actions. If anyone needs to use this while targeting something besides .NET 6.0, feel free to continue from here.


Do not assume a JobLogger receives a logger

In sidekiq/sidekiq#6200, the constructor for JobLogger was changed to accept an entire Sidekiq::Config rather than a logger.
To maintain compatibility with previous versions of Sidekiq, the constructor for Logcraft's Sidekiq::JobLogger has been changed to handle both cases by checking the type of the argument before assigning the logger.

@menyhertfatyol has tried this PR on one of our internal repositories, and has confirmed that it fixes compatibility with Sidekiq 7.3.0.

Thanks for the PR, @wfarr . I'm sorry I haven't had much time to deal with this and I see now that the tests are failing. It seems to be a trivial fix. I can take a look this weekend and hopefully merge this (and the other PR as well).
@barna-emarsys thank you for confirming. 👍

Thanks for the PR, @wfarr . I'm sorry I haven't had much time to deal with this and I see now that the tests are failing. It seems to be a trivial fix. I can take a look this weekend and hopefully merge this (and the other PR as well).

No worries at all! I definitely just tested this on Sidekiq 7.3, threw the PR out there, and assumed all would be good.
I've pushed a provisional change that I expect may fix the issue, but otherwise I'm happy to wait for you to take a look when time allows. 🙇

I ended up reimplementing this because I wanted to add some tests, add Sidekiq 7.3 to the CI matrix, etc. In any case, thanks a lot for contributing! ❤


changing defaults for parsemode, unfurllinks, unfurlmedia

We should revisit our default settings for ParseMode, UnfurlLinks, and UnfurlMedia. Since RoboSlack specifies all Slack Markdown for a Slack message, we should not need Slack to perform any additional parsing on messages coming from the library.

See https://api.slack.com/docs/message-formatting#parsing_modes for more details on parsing modes.
See https://api.slack.com/docs/message-link-unfurling for more details on unfurling links and media.

The current defaults mean:

ParseMode=FULL - anything that looks like a URL to Slack will be processed with additional Markdown. This breaks Roboslack's LinkDecorator's <link|text> Markdown, as Slack will change this to something like <<link>|text> when parsing the message on FULL mode. We end up seeing this in Slack:

unfurl_links=true and unfurl_media=true - this expands any URLs/images in a Slack message to show link/image previews. In my opinion, this behavior should be set to false unless explicitly asked for otherwise, as it ends up generating link/image previews unintentionally:

LGTM


How do I revoke NPM credentials?

I clicked the "Generate NPM credentials" button that is listed on this page, which created a token.  How do I later revoke this token?  The documentation does not explain how to do that.

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

ID: cb7c2ab6-4ba1-51d5-96bb-38a9ab6b50c3
Version Independent ID: 415de875-ba1b-476b-f08b-1a61dc7a2747
Content: Set up your client's npmrc
Content Source: docs/artifacts/npm/npmrc.md
Product: devops
GitHub Login: @elbatk
Microsoft Alias: elbatk

Maybe each time you generate new NPM credentials it invalidates the old one?

Hi @octogonz , currently there is no way to manage the token that gets created in this method. We've heard this as a pain point and we're currently taking a look at the entire Connect to Feed dialog to get this onboarding flow correct. I'm going to close this as a docs issue as this is something we're tracking on the product side of things, but I would be interested to know your opinion on how you'd like to see this work - more configuration options when creating token? A link to manage the token with the token name after creation? Let us know :)

@elbatk GitHub's Settings -> Developer settings -> Personal access tokens is a pretty decent design.

currently there is no way to manage the token that gets created in this method

What happens if the token is accidentally leaked?  (For example, what if a person accidentally pushes their npmrc file to Git?)  Is there no way to revoke the token?


s3 listObjects error: "The data couldn’t be read because it isn’t in the correct format"

Hey, I am using the s3 client and am running
let request = S3.ListObjectsRequest(bucket: bucketName)
let response = try self.s3.listObjects(request)

The second line throws the following error:
"The data couldn’t be read because it isn’t in the correct format"
The error looks like it might be from parsing the XML that comes back in the response from S3?

I am getting a similar error when I try to get a single object. Specifically, the ServiceProtocol for S3 is specified as .restxml:
https://github.com/noppoMan/aws-sdk-swift/blob/master/Sources/AWSSDKSwift/Services/s3/S3_API.swift#L19
Given that that is the case the AWSClient tries to parse the response data as XML:
https://github.com/noppoMan/aws-sdk-swift-core/blob/master/Sources/AWSSDKSwiftCore/AWSClient.swift#L453
This fails, however, since the data is just the raw S3 object data and not XML. As far as I can tell this looks like correct behavior for S3:
http://docs.aws.amazon.com/AmazonS3/latest/API/RESTObjectGET.html

This is the code that I'm using for the get object request.
let getObjectRequest = S3.GetObjectRequest(bucket: bucket, key: hash)
let getObjectOutput = try s3.getObject(getObjectRequest) // <-- Error is thrown here.

thanks for reporting the issue !
I'll fix it asap.

@aasoni @cloutiertyler
I just fixed these bugs by https://github.com/noppoMan/aws-sdk-swift-core/pull/14 and wrote tests for getObject and getObjects in #63 . It seems to work fine.
I also published version 1.1.2 that includes this fix. I hope it works out.
And thanks again to Tyler for your polite report. it was helpful to me to fix program quickly.


pysftp.CnOpts() gives No Host Keys Found

Hello
I use the latest pysftp 0.2.9.
I run the following code:
import pysftp
cnopts = pysftp.CnOpts()
cnopts.hostkeys = None

And receive the following error:
---------------------------------------------------------------------------
HostKeysException                         Traceback (most recent call last)
<ipython-input-10-7c5a03189319> in <module>
      1 import pysftp
----> 2 pysftp.CnOpts()

~\Anaconda3\envs\controller2\lib\site-packages\pysftp-0.2.9-py3.7.egg\pysftp\__init__.py in __init__(self, knownhosts)
     62         else:
     63             if len(self.hostkeys.items()) == 0:
---> 64                 raise HostKeysException('No Host Keys Found')
     65
     66     def get_hostkey(self, host):

HostKeysException: No Host Keys Found

Ultimately I want it to read from ~/.ssh/known_hosts or just leave bypass the host keys but I get the error even before I am able to set it as such.
Please advise

Hello
I have met the similar problem today. It seems that your ~/.ssh/known_hosts is empty. To overcome the issue I have created fake known_hosts file with one row of some host and added this file to a parameter at the cnopts initialization as so:
cnopts = pysftp.CnOpts(knownhosts=os.path.expanduser(os.path.join('~', '.ssh', 'fake_known_hosts')))   cnopts.hostkeys = None

Im flattered that people are finding this repo. It's very old work from a long time ago.  I'm happy to merge any fixes or improvements if you haven't already just forked, but there might be some more mature packages out that to help you faster.

Im flattered that people are finding this repo. It's very old work from a long time ago.  I'm happy to merge any fixes or improvements if you haven't already just forked, but there might be some more mature packages out that to help you faster.


Document options in assets

fix #3950

[x] Read and sign the [CLA][1]. PRs that haven't signed it won't be accepted.
[x] Make sure your PR complies with the [writer's guide][2].
[x] Review the diff carefully as sometimes this can reveal issues.
[x] Do not abandon your Pull Request: [Stale Pull Requests][3].

This only removes the options, are you going to commit the new ones as well?

This only removes the options, are you going to commit the new ones as well?

WIP


issue with installing request package

after
npm install --save request
and
import request from 'request'
I get this error:
 ERROR  Failed to compile with 6 errors                                          12:47:13 PM

These dependencies were not found:

* fs in ./~/request/lib/har.js
* net in ./~/forever-agent/index.js, ./~/tough-cookie/lib/cookie.js and 1 other
* tls in ./~/forever-agent/index.js, ./~/tunnel-agent/index.js

To install them, you can run: npm install --save fs net tls

I'm sorry but this has nothing to do with this template. I suggest to ask the node / npm guys on StackOverflow or a similar place.


Android: Issue: Violation of Interfering with Apps, Third-party Ads, or Device Functionality policy

As it seems google claims that the ads are triggered after the application is closed. The update i did had no changes in the code about the triggering of the ads.
Probably something changed at the algorithm of google store. I also cannot recreate the claims of google.
The ad i use is an interstitial. It is triggered from a user action. Also i check from the platform { App }, the state of the application before i show the ad.
Those are the dependencies i use
"dependencies": {
    "@angular/common": "^9.0.1",
    "@angular/core": "^9.0.1",
    "@angular/forms": "^9.0.1",
    "@angular/platform-browser": "^9.0.1",
    "@angular/platform-browser-dynamic": "^9.0.1",
    "@angular/router": "^9.0.1",
    "@capacitor-community/admob": "^1.0.1",
    "@capacitor/android": "^2.2.0",
    "@capacitor/cli": "^2.2.0",
    "@capacitor/core": "^2.2.0",
    "@capacitor/ios": "^2.2.0",
    "@ionic/angular": "^5.2.2",
    "@ionic/app-scripts": "^3.2.4",
    "chart.js": "^2.8.0",
    "core-js": "^2.5.4",
    "css-loader": "^3.4.2",
    "jetifier": "^1.6.6",
    "karma-coverage": "^2.0.1",
    "karma-phantomjs-launcher": "^1.0.4",
    "rxjs": "^6.5.4",
    "to-string-loader": "^1.1.5",
    "tslib": "^1.9.0",
    "webpack": "^4.41.6",
    "zone.js": "^0.10.2"
  },

I believe i tested everything that will fix the problem at the typescript code, except from changing the type of ad itself.

As it seems google claims that the ads are triggered after the application is closed.
What is mean this?? If possible, I would like to get more information.(ex: error log).
Thanks.

There is not error log. When testing everything is working fine. When uploading a new version of the app it is rejected by the store. The only response from the google store is the following email:

Hi Developers at Greasidis Dimitris,
Thanks for submitting {your-app} to Google Play. After review, your app has been rejected due to a policy violation. If you submitted an update, the previous version of your app is still available on Google Play.
Issue: Violation of Interfering with Apps, Third-party Ads, or Device Functionality policy
Ads associated with your app must not interfere with other apps, ads, or the operation of the device, including system or device buttons and ports. This includes overlays, companion functionality, and widgetized ad units. Ads must only be displayed within the app serving them.
Next steps:

Read through the Interfering with Apps, Third-party Ads, or Device Functionality policy and make appropriate changes to your app. Be sure to remove any ads that appear after the user has exited the app, or after the user has pressed the back button to exit the app.
Make sure that your app is compliant with all other Developer Program Policies. Additional enforcement could occur if there are further policy violations.
Sign in to your Play Console and submit the update to your app.

If you’ve reviewed the ads policy and feel this decision may have been in error, please reach out to our policy support team. One of my colleagues will get back to you within 2 business days.
Best,
Google Play Review Team

Issue: Violation of Interfering with Apps, Third-party Ads, or Device Functionality policy

Where did you display AdMob Ad position? You can't display ads on the content. In reality, it will be displayed at the top or bottom of the app, like this:
https://i.stack.imgur.com/OR76l.png

At first i had the following setup:
const options = {
  adId: **********,
  adSize: AdSize.SMART_BANNER,
  position: AdPosition.BOTTOM_CENTER,
  isTesting: false,
}

This was working obviously but it was rejected and then i tested the same options as the example in the documentation for an interstitial ad.
const options = {
  adId: **********,
  isTesting: false,
}

Same thing. It worked but still rejected.
So probably the buttons of the phone should be always visible even in an interstitial app?

Sorry, I'm not know. I'm using this plugin now, right.
You can reply this email to ask the AdMob team for more specifics. This issue may not a plugin, but an AdMob convention issue.

Is there a way to set margin to an interstitial ad throught the adOptions. I believe this is the solution of the problem

I understood! You don't said about banner. This issue is about interstitial.
Certainly we need the option of ensuring the height of the header and footer. Can you create pull request about this??


Refactor to RTK Query

Which issue(s) this PR fixes
Fixes #173
Fixes #112
Fixes #113
Fixes #114
Fixes #126
Fixes #168
Fixes #28
Replaces #201
Additional comments
It fully replaces the old way of querying the server and eliminates using OpenAPI generator on Java.
Agreements

[x] I hereby declare this contribution to be licensed under the Apache License Version 2.0, January 2004.
[x] I hereby agree to grant TOSIT a copyright license to use my contributions.

Just descovered a few issues:

change env vars to config from the react state here
when switching between companents on the service page, variables are not correctly changing in the Monaco edditor

We should add yarn.lock and package-lock.json to the .gitignore until the first release at least.

added lock files to gitignore

Looks promising!!
If I'm not mistaken, it's missing:

Authentication
Remove the need for a NEXT_PUBLIC_TDP_API_URL=http://localhost:8000 variable in the .env file

Do you see anything else to add / do you need some help?

Also, we may want to refactor it a bit (code is spited between src/scripts/, src/features and src/store.ts). Maybe we could put all Redux related code in a src/store/ directory?

I've checked previously opened issues. Your PR will:
Fixes #112
Fixes #113
Fixes #114
Fixes #126
Fixes #168
Fixes #173 (current)

I'm not sure that we should commit the /src/features/tdpApi.ts file as it is generated. We could generate it programmatically from the schema to ensure that the file is not altered.

I tried the programmatic usage, but it doesn't work for in browser environment. Check this issue.
I think it is doesn't make a big sence commiting the genereated API in src/features/api/tdpApi.ts or not. We have a full control over the schema and it is stored in the same repository. Once it is changed, we regenerate the API and commit both. Git-ignoring it would just make the source code less dependent on instructions in the doc.
If we ignore, I would suggest:

keeping generate script
adding postinstall script calling generate

Example:
    ...
    "generate": "npx @rtk-query/codegen-openapi ./scripts/openapi-config.ts",
    "postinstall": "npm run generate"
    ...

I'm not sure to understand the issue but you're right, that's not very important. The solution that you propose using the postinstall script would be enought!
Idk if we could have another opinion but IMO we shouldn't commit the generated output.

Done:
[x] We may want to refactor it a bit (code is spited between src/scripts/, src/features and src/store.ts). Maybe we could put all Redux related code in a src/store/ directory?
[x] I'm not sure that we should commit the /src/features/tdpApi.ts file as it is generated.

when switching between components on a service page, variables are not correctly changing in the Monaco edditor

Fixed this. However, swithing beteween components is not smooth anymore. Didn't have time to improve it.
The was a problem of react rerendering when variables changes.
There is now an issue with validating. It is not working as expected. I'll continue next week on fixing it.

However, swithing beteween components is not smooth anymore.

Done

There is now an issue with validating. It is not working as expected.

Done

@PaulFarault
review please, it is ready

I'll have a look at it. You still have some conflicts to resolve before merging it to master.

rebased

Seems like you involuntarily "broke" some previous features while rebasing, pagination doesn't seem to work as expected: https://github.com/TOSIT-IO/tdp-ui/pull/185

Seems like you involuntarily "broke" some previous features while rebasing, pagination doesn't seem to work as expected: https://github.com/TOSIT-IO/tdp-ui/pull/185

fixed


API - documents are out of date, and can we use as fix for NHSE doing simple analyses

Our API page on OpenPrescribing is out of date: it doesn't give details of a number of API calls that are available to users, including measures data, tariff or PPU.
I'm sure where we are with regards to https://github.com/ebmdatalab/openprescribing/issues/1993, but we should consider updating the website accordingly.
In addition, could better use of APIs reduce our reliance on NHSE analysts needing access to BQ to undertake relatively simple analyses on prescribing data?

I'll add to the Data Team backlog. If we're going to be pointing more people to the API we should probably also look at:

https://github.com/ebmdatalab/openprescribing/issues/3640

Been discussing this at the "other" job, some small changes to the API call may help reducing questions, and improve usefulness of site.


No connection could be made because the target machine actively refused it.

+0800 2023-09-19 23:55:19 ERROR [2571389966 1ms] inbound/http[http]: process connection from <IP_ADDRESS>:55993: exchange6: dial tcp <IP_ADDRESS>:7090: connectex: No connection could be made because the target machine actively refused it. | exchange4: dial tcp <IP_ADDRESS>:7090: connectex: No connection could be made because the target machine actively refused it.
+0800 2023-09-19 23:55:20 ERROR [553288773 1ms] dns: lookup failed for gae2-spclient.spotify.com: exchange6: dial tcp <IP_ADDRESS>:7090: connectex: No connection could be made because the target machine actively refused it. | exchange4: dial tcp <IP_ADDRESS>:7090: connectex: No connection could be made because the target machine actively refused it.
+0800 2023-09-19 23:55:20 ERROR [553288773 1ms] inbound/http[http]: process connection from <IP_ADDRESS>:56000: exchange6: dial tcp <IP_ADDRESS>:7090: connectex: No connection could be made because the target machine actively refused it. | exchange4: dial tcp <IP_ADDRESS>:7090: connectex: No connection could be made because the target machine actively refused it.
+0800 2023-09-19 23:55:20 ERROR [3931544707 1ms] dns: lookup failed for gae2-spclient.spotify.com: exchange6: dial tcp <IP_ADDRESS>:7090: connectex: No connection could be made because the target machine actively refused it. | exchange4: dial tcp <IP_ADDRESS>:7090: connectex: No connection could be made because the target machine actively refused it.
+0800 2023-09-19 23:55:20 ERROR [3931544707 1ms] inbound/http[http]: process connection from <IP_ADDRESS>:56007: exchange6: dial tcp <IP_ADDRESS>:7090: connectex: No connection could be made because the target machine actively refused it. | exchange4: dial tcp <IP_ADDRESS>:7090: connectex: No connection could be made because the target machine actively refused it.
参考不良林的教程配置，报错如上
https://bulianglin.com/archives/hysteria2.html
服务器配置：
listen: :443 #监听端口
#使用自签证书
tls:
cert: /etc/hysteria/server.crt
key: /etc/hysteria/server.key
auth:
type: password
password: huang@123 #设置认证密码
masquerade:
type: proxy
proxy:
url: https://bing.com #伪装网址
rewriteHost: true
客户端配置：
server: xxx.xxx.xxx.xxx:443
auth: huang@123
bandwidth:
up: 50 mbps
down: 50 mbps
tls:
sni: bing.com
insecure: true #使用自签时需要改成true
socks5:
listen: <IP_ADDRESS>:7090
http:
listen: <IP_ADDRESS>:8080

你先看看你的端口开没开，然后看dns配置
resolver:   type: udp   tcp:     addr: <IP_ADDRESS>:53     timeout: 10s   udp:     addr: <IP_ADDRESS>:53     timeout: 10s   tls:     addr: <IP_ADDRESS>:853     timeout: 4s     sni: cloudflare-dns.com     insecure: false   https:     addr: <IP_ADDRESS>:443     timeout: 4s     sni: cloudflare-dns.com     insecure: false

哈哈哈哈一样的问题：

用的acme证书
服务器listen: :443
启动log一切正常
netstat看根本没有443端口占用

http inbound 端口错了，跑到 socks5 的了

http inbound 端口错了，跑到 socks5 的了

这个教程，v2rayN的配置就是到socks5端口的

Is the problem solved?

Is the problem solved?

Not yet...

Make sure the server hysteria has root privileges

I seem to have solved the problem.
1. Make sure the server hysteria has root privileges

2. Make sure the udp ingress  rule in the server's security list is turned on.

+1哈，应该和我一样是Oracle吧，最终解决办法是在Oracle的网络设置里打开443的upd ingress。
Same here, if you are on Oracle don't forget to add an ingress rule for 443 udp.

I seem to have solved the problem.
1. Make sure the server hysteria has root privileges

2. Make sure the udp ingress  rule in the server's security list is turned on.

+1哈，应该和我一样是Oracle吧，最终解决办法是在Oracle的网络设置里打开443的upd ingress。
Same here, if you are on Oracle don't forget to add an ingress rule for 443 udp.

我这样设置了，还是不行

I seem to have solved the problem.
1. Make sure the server hysteria has root privileges

2. Make sure the udp ingress  rule in the server's security list is turned on.

+1哈，应该和我一样是Oracle吧，最终解决办法是在Oracle的网络设置里打开443的upd ingress。
Same here, if you are on Oracle don't forget to add an ingress rule for 443 udp.

我这样设置了，还是不行

那有没有可能你还和我一样用的是ubuntu22+？那就需要在机器上 sudo ufw allow 443 ; sudo ufw enable

我的之前都是正常的，用了BitTorrent之后会出现这个问题。

ensuring the server has root privileges and that the UDP ingress rule in the server's security list is turned on.
If you're using Oracle,  adding an ingress rule for port 443 in Oracle's network settings.

我的已经解决了
昨天刚部署的hysteria2，也是跟着不良林的步骤弄的，过了一晚上，今天再看v2rayN就各种refused it了，报错跟你的一模一样。

重启电脑，如果重启电脑能解决，说明是进程占用端口问题
重启电脑也解决不了的，先确定服务端没问题后，去到hysteria客户端存放的目录，用cmd运行hysteria，查看报错信息来推断配置哪里有问题。记得运行前先把配置文件放到hysteria目录下，命名为config.yml

我遇到的问题是server配置的ip和域名的问题，我发现hysteria服务端如果用的是ca证书，那客户端的server就不能是ip:端口格式，不然会报错说客户端初始化失败，得改成域名:端口格式。别人我不知道，我用ip就报错，改域名后好了

我开启全部端口就好了, 最后发现是UDP的443只要放开就行

请问解决了吗？遇到同样的问题折腾了很久还是不行


Enable IPv6 on compose networks where enable_ipv6 is true

There is a compose feature called enable_ipv6 that can be set on a network.
When this is currently set on a compose network, it has no effect. Compose does let us know that the EnableIPv6 flag is ignored, though.
However, nerdctl network create does support IPv6, it can be set using the --ipv6 flag. When the --ipv6 flag is set, IPv6 IPAM is added just like IPv4 IPAM to the network. However, when --ipv6 is not set, the IPv6 IPAM is ignored and only the IPv4 settings are added.
We should add support for enable_ipv6. This PR aims just to do that.
Please note that this is not enough for compose to support IPv6... see issue https://github.com/containerd/nerdctl/issues/3637. I'm not yet familiar enough with the codebase to add such a feature. But this should allow manual configuration.
Added two tests:

one test to test enable_ipv6 true, the IPv6 subnet should stay in the IPAM configuration after creation
one test to test enable_ipv6 false, the IPv6 subnet is filtered out of the IPAM configuration after creation and only IPv4 remains

Seems like the second test: "test to test enable_ipv6 false, the IPv6 subnet is filtered out of the IPAM configuration after creation and only IPv4 remains" fails on the Docker integration test.
Should we remove this test and only test if enable_ipv6: true works fine?

Seems like the second test: "test to test enable_ipv6 false, the IPv6 subnet is filtered out of the IPAM configuration after creation and only IPv4 remains" fails on the Docker integration test.
Should we remove this test and only test if enable_ipv6: true works fine?

I think we need to figure out why nerdctl runtime do not consider enable_ipv6 false, it may point to an underlying problem.


Instructions are invalid

Instructions are invalid - needs updating to direct to 'Calendar' instead - searching for Fiscal wont work

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

ID: a55675e3-1a31-5f8f-0fe0-d3f4456216b7
Version Independent ID: dd99a009-617e-af6f-0706-44233b7fb596
Content: Manage fiscal year settings (Dynamics 365 Marketing)
Content Source: ce/marketing/fiscal-year-settings.md
Service: dynamics-365-marketing
GitHub Login: @alfergus
Microsoft Alias: alfergus

Thanks for bringing this to our attention. The location of some of our settings changed in the February update. I've updated the documentation with the text below.

Make sure you have the System Administrator security role or equivalent permissions in Dynamics 365 Marketing.
Open the Settings menu  at the top of the page and select Advanced settings. The advanced-settings area then opens in a new browser tab. Note that this area uses a horizontal navigator at the top of the page instead of a side navigator.
The new tab will open the Business management settings by default. On the Business management page, select Fiscal year settings.
Enter information in the text boxes.

In the Start Date box, select the date to start the fiscal year.
In the Fiscal Period Template drop-down list, select how your fiscal year is divided.
In the Fiscal Year drop-down list, select how you want to display the fiscal year.
In Name Based On drop-down list, select whether the fiscal year name is displayed on the start or end of the fiscal year.
In the Fiscal Period drop-down list, select how you want to display the fiscal period.
In the Display As drop-down list, select how you want the fiscal year abbreviation and the year to appear.

Select OK.


Automated cherry pick of #3230: gpu mutil attach by model type, fix network hint name

Cherry pick of #3230 on release/2.11.
#3230: gpu mutil attach by model type, fix network hint name

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@407ba69). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.11   #3247   +/-   ##
==============================================
  Coverage                ?   7.11%
==============================================
  Files                   ?     476
  Lines                   ?   77312
  Branches                ?       0
==============================================
  Hits                    ?    5503
  Misses                  ?   71347
  Partials                ?     462

Flag
Coverage Δ

#aFlag
7.11% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/networks.go
0.65% <0%> (ø)

pkg/compute/models/guest_actions.go
0% <0%> (ø)

pkg/compute/models/isolated_devices.go
2.5% <0%> (ø)

Continue to review full report at Codecov.

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

/lgtm

/approve


BUG: CircleCI 1.0 is retiring

We need to move to 2.0 by August.
We should also switch to using environment.yml, which means having a newer version of Mesa (the one of 14.04 is too old to support the necessary OpenGL extensions for VTK7+).

here is an example of the new API we could reuse:
https://github.com/NeuralEnsemble/python-neo/tree/master/.circleci

SciPy also just transitioned. The annoying part for us will be getting a
usable docker image with OSMesa for VTK rendering.

see if you find any repo on github that uses vtk with circleci

Maybe this one
https://github.com/conda-forge/pyvtk-feedstock
I'll try to look this week


Implement custom pilots UI

Done in https://github.com/tablelandnetwork/rigs/pull/463 and https://github.com/tablelandnetwork/rigs/pull/481


Fix --time when there are VIR errors, add option for machine-readable json output of verification outcome and timing

This didn't work for me at first because it was using an older version of serde_json that didn't support u128. Can you constrain the version to be after v1.0.93? Perhaps just take 1.0.95, the current version?


TaxonName - cached_nomenclature_date not checking year_of_publication

When I have year_of_publication filled, then it is not referenced in cached_nomenclature_date.  Probably a prioritization issue.
Example is after a resave:

Sorry.  Issue was bad data.  Behaviour was correct.


0.6 not reading custom/app.ini

when I try running a new install using ./gogs web, it is trying to read from <IP_ADDRESS> and not from the correct IP address. This seems to be because it is not reading custom/app.ini (hard to tell for sure).

Sorry, the error message was in yellow and hard to read on a white background; you might want to make it a better colour.

Sorry, the error message was in yellow and hard to read on a white background; you might want to make it a better colour.

Hmm, which error message? Give a screenshot would be cool.


Add 'Generate SSHA Hash' operation

This operation generates an RFC 2307-compliant, cryptographically-secure SSHA (Salted SHA1) password hash - commonly used for storing passwords in systems such as OpenLDAP.
More detail here on SSHA:
https://www.openldap.org/faq/data/cache/347.html
Some things to note:

Contrary what the similar-sounding name may suggest, this is different to the existing SHA1 operation (although does use SHA1 in part) and despite the code being very simple, it is actually a surprisingly difficult recipe to build in CyberChef using the existing set of operations
No automated test is applicable for this. This is because the output (although derived from the input) is not deterministic, as it involves a random IV
Given the intended use case of this involves entering a password into the input field, it would have been great to include a 'Mask Input' checkbox in the operation which causes the input field to mask, however I could not find any way to programmatically do this without attempting something extremely hacky... which just didn't seem right. Happy to consider any alternative options anyone may know of for that if it sounds like a good idea though.

Hey @a3957273, hope you're well. Is there anything further I can do to get this one over the line?
Thanks!


refactor: use std::min(), std::max()

Dusting off an older PR. This is a followup to https://github.com/transmission/libutp/pull/7 and is described in that PR description.
Part two of a three-part series to use std:: tools instead of bespoke ones. Part three will fix #6.

[x] Part 1: remove Array; use std::vector instead (#7)
[x] Part 2: remove min, max; use std::min, std::max instead (this PR)
[ ] Part 3: remove utpHashTable; use std::unordered_map instead

Unfortunately there don't seem to be any real tests for libutp, so the need to manually test is even greater than usual.

whoops, targeted to the wrong branch. Sorry for the noise; will resubmit


Compendium Not Merging With Updates

For every update I have had to recreate the custom classes because they do not carry over to the newest update.  I've had to create the Knight of St. Ydris, Witch, and Warlock three times now.
The class options remain in my items, but they cannot be opened because the compendium entries are deleted.

The default shadowdark compendiums will be replaced entirely on new updates. This is normal behavior. All custom classes should be stored in a separate world level or shared level compendium for this reason.

The default shadowdark compendiums will be replaced entirely on new updates. This is normal behavior. All custom classes should be stored in a separate world level or shared level compendium for this reason.

Any guidance, or a how-to link on how to make sure I do this appropriately?

Just create a new world level compendium outside of the shadowdark default and shadowdark system folders in the compendium tab. See this article for more details. https://foundryvtt.com/article/compendium/
It's helpful to look at the icons that appear on the compendium in the tab:

As a best practice, avoid modifying system or module compendiums as they might be overridden by updates.

That's very helpful, thank you!  I'm working on porting all of the system compendium to a world compendium before I update to the latest version.


Nearby option relative to first task

Basically, I would like to be able to fix each task within a certain area - let's say within a 10 mile radius of the first task.
However what usually happens is that if you select "next task: nearby" on each task, it is likely you will end up miles away in one direction from your first task.
Instead it would be good if there was a "nearby first task" option, which would find unresolved task nearby your first task.
Hope I've explained this correctly! 😄

You could create a virtual challenge for that purpose. In the Find Challenge screen, zoom and pan the map to the area you want to work in, use the filters to narrow down to the challenge(s) you want to work on, draw a lasso on the map, and start your challenge.
I realize the documentation about virtual challenges is pretty minimal.. I opened a ticket on the docs repo to remind us to do something about that.

Wow, interesting - didn't know this featured existed. The +/- lasso on the right hand side of the map is not obvious at all, only now that you have pointed those buttons out did I realise they were there!
However once draw with the lasso, click work on selected tasks then add a name - I get the following error:
Oops!
Unable to create a virtual challenge{details}


Minor memory fix

PR intention
Fixes a minor memory leak.

Take a fix from the bitcoin code instead
https://github.com/bitcoin/bitcoin/blob/master/src/httprpc.cpp#L114

They have static keyword for the buffer. Our code does not have anything to prevent the race condition on the array. I understand why you requested this change but I don't know what is better for you in this situation - leave it as implemented (without unnecessary copying from char array to vector), or apply their code without the static keyword. Please let me know.

Look carefully at their code
        static const unsigned int KEY_SIZE = 32;
        unsigned char out[KEY_SIZE];

static is only for size (rightfully so though I'd prefer constexpr). Buffer is not static


refactor: rename connection.RateLimit to connection.RateLimitPerHour

Summary
Description
rename connection.RateLimit to connection.RateLimitPerHour
Does this close any open issues?
close #2360

@phzbbbbbbbb  can you change the version to lastest time in plugin init_schema. such as<PHONE_NUMBER>1138 --><PHONE_NUMBER>1138 .  if not,  it might fail. someone who had run the devlake service, due to the database model updated.

@phzbbbbbbbb Hi, thank you for your contribution to Apache Devlake, Devlake community specially invites you to join our contributor group and issue you a contributor certificate. If you are willing, please contact<EMAIL_ADDRESS>


Added - Multiple URLs using TLS

To improve overall privacy/security. Also fixed some minor 404 redirects.
I'm hoping this is the active branch?
While the CONTRIB reference states using the "v2_fixes" branch it seems largely stale?

@shitwolfymakes Could you review/assist please? I'm not familiar with the proper procedure.

I'm not a maintainer or anything, I'm just a guy sharing what worked for me.
I can't approve or merge, but I'd recommend fixing the errors that the automation tools are reporting. It looks like secure security implementation might be flawed. Once you've got that fixed, I'd be happy to give your code a sanity check!

By reverting the changes in utils.py it now passes correctly.

Oh, by the PR title I thought you were implementing TLS for connections to ARM. It looks like all you're doing here is changing urls to https from https. As long as the api calls still return properly over https I can't see a reason to not merge this.
Two things though:

Please change the title of the PR to better reflect what you're doing here
Please test the new APIs and let us know if they work the same as the current branch

This is a solid first PR!

In that case I'll wait till the devel branch settles with your forks proposed changes to re-open the PR.
I'm also working an updated Debian installer for the wiki.


Fix the handling of project paths containing spaces

As requested, this branch has been rebased and squashed.
This provides a fix for the external runner, allowing to have path with spaces in the external runner executable path.

:+1:


Change default subreaper setting

This subreaper should always be turned on for containerd unless
explicitly needed for it to be off.
Signed-off-by: Michael Crosby<EMAIL_ADDRESS>

Codecov Report

Merging #1822 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master    #1822   +/-   ##
=======================================
  Coverage   49.39%   49.39%
=======================================
  Files          82       82
  Lines        8362     8362
=======================================
  Hits         4130     4130
  Misses       3573     3573
  Partials      659      659

Flag
Coverage Δ

#linux
49.45% <0%> (ø)
:arrow_up:

#windows
45.89% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

server/config.go
0% <ø> (ø)
:arrow_up:

server/server_linux.go
0% <0%> (ø)
:arrow_up:

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 967caee...6e9f24b. Read the comment docs.

@mlaventure said:

Should not forget to add this to the release notes though

Oops :(

:joy:, I guess we should add labels for that


gltf animation

[x] Review the documentation: https://virocore.viromedia.com/
[x] Search for existing issues: https://github.com/viromedia/virocore/issues
[x] Use the latest ViroCore release: https://virocore.viromedia.com/docs/releases
[x] Turn off Instant Run in Android Studio

Environment
Please provide the following information about your environment:

OS: andorid
Version: ViroCore 1.12.0
Device(s): pixel

Description
Load GLTF animation and fill the display, I through
Set animationKeys = object.getAnimationKeys();Get the animation Take 001,
but GetAnimation ("Take 001");
Animation. SetLoop (true);
Animation.  play ();
This doesn't work. Why is that?

@dthian the new version if fixed animation question ?

Hey @544531645, apologies for the delay, the Panda model should render with animation in the next update.

Closing as this should be fixed in the latest release.


Gb postgres

This won't work for Production though. Right now the whole database is lost every time our docker server shuts down.


N23: Added bevy_midi and bevy_config_cam V2

Part of #677
CC: @17cupsofcoffee
V2 due to git and IDE issues. V1 is #682
Added <!-- no toc --> to prevent auto-gen from markdown all in one.

Merged, thanks!


fix(button): buttons don't get activated when ion-label contains some elements

Short description of what this resolves:
If a button contains <p>, <h1> or any exotic element, the button is not properly activated if the user taps under such elements. This feels very unresponsive and laggy.
I am not sure making <ion-label> pointer-events: none is the correct solution, but this bug is real and it should be fixed somehow.
Ionic Version: 2.x

This PR should close:
#6192
#6201
#6192

pointer-events: none; seems like the more performant solution, since we get around traversing up the DOM tree for ion-label content. Good thinking!

woot! Thanks for testing this all out!

Thank you @manucorporat

@bryandh I added the CSS rule you said and it's still not working for me with the ion-item ion-label bug.  Any idea what's going on?


Part C and part D video

User Story
As a student, I'd like to have a video for part D so I can pass the class.
Detailed Description
Provide additional details and context.
Subtasks (Please also link task PRs here)

[]
[]
[]

This is done, so we can close this


Production release

Yup, I mentioned this also to Justin. We need more content.
@jmervine: how do the appendLocals.js changes look to you? At first I had an issue with us overriding the whole res object the way I implemented so I ended up using Object.assign. It should be OK, but please confirm :)

@jmervine I agree 100%, I will be working on the content now before pushing the Books and Jobs pages


Q: how does this compare with simdjson?

Specifically https://github.com/minio/simdjson-go has iterator and find methods as well.

It seems simdjson does not provide a nice query language? gjson might be possible to implement on top of simdjson.

simdjson does not support AArch, so it doesn't work on mobile devices, macOS, Raspberry Pi, etc.


Robot should stop when battery is discharged.

Environment

OS Version:  Ubuntu 20.04
Source build of Gazebo Garden

Description
In the example file linear_battery_demo.sdf, this line (https://github.com/gazebosim/gz-sim/blob/9370793caf94e96a49ba8103c6c950644a573f46/examples/worlds/linear_battery_demo.sdf#L31) says that the blue vehicle should stop when the battery runs out. I tried running the example and letting the battery run out, but the blue vehicle did not stop.

Expected behavior: The blue vehicle should stop after the battery is drained, respective of the velocity command.
Actual behavior: The vehicle continues to move.

Steps to reproduce

In the file linear_battery_demo.sdf change the line 476 to have a high power load, say 660 instead of the default 6.6, so that the battery drains quicker.
Run the file using ign gazebo -v 4 linear_battery_demo.sdf -z 1000000
In a separate terminal, run : ign topic -t "/model/vehicle_blue/cmd_vel" -m ignition.msgs.Twist -p "linear: {x: 6.0} angular: {z: 0.4}"
Check the battery status in a separate terminal : ign topic -e -t /model/vehicle_blue/battery/linear_battery/state.

The blue vehicle continues to move even after the battery is fully drained.

Suggestions
If it is not possible to resolve this behavior for now, the documentation in the example sdf file should be updated.

I did a quick search of the usages of the components::Battery. https://github.com/gazebosim/gz-sim/search?q=components%3A%3ABatterySoC
Seems like systems should read the BatterySoC component like the Sensor system does: https://github.com/gazebosim/gz-sim/search?q=components%3A%3ABatterySoC

This section in Physics.cc is checking for the battery status and is supposed to stop the joints from rotating : https://github.com/gazebosim/gz-sim/blob/f4198be19ae432bbcc584b73887178320e69ed90/src/systems/physics/Physics.cc#L1802-L1807

Makes sense for the case of the example above. However, there are many systems which apply forces/set velocities directly on links. These will not avail of the joint force property.

This PR fixes it for this particular example : https://github.com/gazebosim/gz-sim/pull/1500


Add makefile.

This will make build and test locally easier.

This change is

Maintenance plan:
Update HUGO_VERSION in Makefile and build the Dockerimage again if we want to use different version of hugo.

/assign @hongye-sun

/assign

@irvifa Please would you explain the reason for this PR in more detail.
If you're proposing a new way of building the website locally (that is, different from what's in the README) please would you include some documentation in the PR. Prerequisites, and how to use the MAKE file.
It will also be good to include a maintenance plan - what needs to change when the website is updated to different versions.
Note that the process needs testing before we can merge this PR.

/hold

@sarahmaddox can you explain how the test works for the makefile? Thanks!

@irvifa We'll need to manually test the process after you've documented it.

@sarahmaddox okay i’ll update the readme and how we maintain it. Thanks for the advice..

@sarahmaddox I added the how to's in the README.md and plan of its maintenance.

I don't think we need to complicate the website preview any more than it needs to be. The current prices is fairly straightforward. Either run hugo locally to create a preview, or create a PR and the current repo setup automatically creates a preview for you.
If you'd still like your Docker setup, I suggest you commit to your own GitHub repository.


Refactor workspace setup step

@shyba @acamino Please review this one. A benefit of splitting this into more steps, is that we can show more code.


Does tangram supports objects such as trees etc?

Hi,
It is for a while that I am busy with finding a way to illustrate trees or other objects using tangram, I have seen some examples of webgl libraries such as threejs that they have created such objects, But I have no idea whether or not tangram supports these kind of shapes?Can some one gives me some hints?thanks

Hi @am2222 – we don't support external 3D objects at this time but hope to in the future!


Updated the uploaded rule to make use of a built in function

Check this @adelowo, maybe I'm missing something

I have manually test this with code here, and it seems works properly.

Sorry for the late reply..Like i said the other time. I have been extremely busy

Its ok @adelowo :smiley:


CLOUD-42848 Recipe integration test

@martonsereg @schfeca75 plz review

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1771/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1772/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1777/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1797/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1798/


fix(testing): remove datetimepicker extraneous logs

add date-fns for locales, import and register enUS and ptBR localeObjects, set defaultLocale to enUS
and update test to match.
@jackcmeyer I think this is the fix for this issue:
https://github.com/HospitalRun/hospitalrun/issues/146
but I wasn't sure since it was in the main repo issues.
If anything it fixes these extraneous logs in the components library
Newly added dependencies with Bundlephobia links:

"date-fns": "^2.9.0",
<EMAIL_ADDRESS>

LGTM


Use sudo for privileged process checks on filedescriptors

What does this PR do?
Repackaged the patch I suggested at https://github.com/DataDog/dd-agent/issues/2033#issuecomment-325657568 in a dd-agent-omnibus compatible way (no additional python file, only process/check.py is modified).
Motivation
Datadog does not provide out of box support for monitoring file descriptors used by processes not running under the same user (dd-agent by default).
The suggested work-around of running the datadog agent as root is explicitly not recommended which is probably reasonable: https://help.datadoghq.com/hc/en-us/articles/115000066506-Why-don-t-I-see-the-system-processes-open-file-descriptors-metric-
Additional Notes
This requires the addition of the following sudo policies in the deb and rpm packages:
dd-agent ALL=NOPASSWD: /opt/datadog-agent/embedded/bin/python /opt/datadog-agent/agent/checks.d/process.py num_fds *
dd-agent ALL=NOPASSWD: /opt/datadog-agent/embedded/bin/python /opt/datadog-agent/agent/checks.d/process.pyc num_fds *

Usage of sudo generates frequent logging in /var/log/auth.log. That may be silenced with additional PAM rules in /etc/pam.d/sudo if desired.
Resolves DataDog/dd-agent/issues#2033
The same may be done for the io_counters method.

Hey @pdecat! Thanks a lot for your contribution!
This is a good idea but I am not sure that it should be turned on by default in the package. I am wary about giving our agent any sudo power that it doesn't absolutely need. So I don't think this should be turned on by default in the packages. Can you add instructions to turn it on in the readme and in the case of failure after attempting to use sudo.
Thanks a lot, again, for your contribution! 😄

Hi @gmmeyer , I'll document the sudoers part in the README.
Also, what would you think about adding an option to the process check's configuration to enable the use sudo, disabled by default? That would avoid failed sudo executions in case the sudoers part is not set up.

I think that's very wise, it shouldn't fail when it doesn't have the proper permissions. It should log something on info but not fail.

Thanks! :champagne:

Hi @hush-hush / @ChristineTChen, I get that this was reverted for 5.22.
What issues did you have?
Did you enable the sudo part to actually get open_file_descriptor metrics?

Bump ^ @hush-hush @ChristineTChen


Set up the edit profile page layout

https://github.com/sopra-fs23-group-32/SoPra23_Client/commit/4329e4ef37fb26e7574fba21c8ba8eddd717b385
https://github.com/sopra-fs23-group-32/SoPra23_Client/commit/f533ed35a7e4420b0ec246107dd3bb0404e501d8

https://github.com/sopra-fs23-group-32/SoPra23_Client/commit/0b50a71254df0d052b467f45a2fa0b4f6c32c88d


#27 rest connector for Trades API

@davsclaus @rhuss @jimmidyson can you take a look?
This creates a connector for a single operation on the Trades API REST
resource (day-trade-api[1]), this is done by delegating to Camel REST
component that in turn uses Camel Undertow component to perform the
actual interaction. Camel Jackson component is used to unmarshal the
response received from the service.
This has support for only one REST operation (getTrade[2]), but can be
used as a template to create connectors for other operations.
To initiate the exchange, the incoming message needs to have orderId
header set to the ID of the order that needs to be fetched.
The Java class org.example.ipaas.Trade has been vendored within the
connector as com.redhat.ipaas.connector.daytrade.Trade.
[1] https://github.com/kcbabo/ipaas-demo/tree/master/day-trade-api
[2] https://github.com/kcbabo/ipaas-demo/blob/master/day-trade-api/src/main/java/org/example/ipaas/OrderDesk.java#L33
[3] https://github.com/kcbabo/ipaas-demo/blob/master/day-trade-api/src/main/java/org/example/ipaas/Trade.java

Great work.
You have the spring-boot stuff turned off in the pom.xml, was there a problem with that? AFAIR it helps generate the spring boot autoconfiguration so we can configure component level options on that level, which in this example is the host option.

@davsclaus yeah I commented that out, I can't really remember why, could be that my IDE was bugging me. And thanks for cleaning this up 👍


When track is too long (25 minutes) playing from the middle calls state "ended" and track doesn't play

Describe the Bug
after I add a track (which is very long ~25minutes). I call TrackPlayer.seekTo to put let's say to 02:00 minute and then press play, player fires "buffering" then "playing" then "ended" then "paused" (all right after each other) and user can't play it anymore.
Steps To Reproduce
try to add a track with 25minutes long and right after playing seek to at the middle of it.
Environment Info:
react-native-track-player` version - 4.0.0-rc07 (but I've faced with this on 3.2.0 versions as well)
platform - iOS

I'm having a problem that may be related.
If you call TrackPlayer.getDuration(), does it give you the correct duration for the track?
I'm having a problem where my mp3 file actually has the incorrect duration embedded it.  I do know what the correct duration is and I set that duration when adding the tracks to the queue.
The issue is that TrackPlayer does not seem to use the track metadata for determining the duration when using seekTo or useProgress.
I'm working if that is the issue you are running into and that your audio file is returning the wrong duration.
I'm only using v3.2, but if you are using v4, maybe you could try setting the problem duration of the track when adding the track to your queue and see if the newer version has "fixed" the duration issue.

@markmccoid thanks and you are right the getDuration() function returns incorrect value, and when I set duration in metadata it still returns wrong value

How do you add tracks? I've had this wrong with TrackPlayer.add when adding more than one track. Now I use TrackPlayer.load instead and its working correctly.


Move validation of connection headers in HTTP/2 back to HpackDecoder

Motivation:
#12755 added validation for presence of connection-related headers while HpackDecoder decodes the incoming frame. Then #12832 moved this validation from HpackDecoder to DefaultHttp2Headers. As the result, existing use-case that could use DefaultHttp2Headers for HTTP/2 and HTTP/1.X broke when users add  any of the mentioned prohibited headers. The HTTP/1.X to HTTP/2 translation logic usually has sanitization process that removes connection-related headers. It's enough to run this validation only for incoming messages and we should preserve backward compatibility for 4.1.
Modifications:

Move isConnectionHeader and te validations from DefaultHttp2Headers back to HpackDecoder;
Add tests to verify HpackDecoder fails incoming headers as expected;
Add tests to verify mentioned headers can be added to DefaultHttp2Headers;

Result:
Backward compatibility is preserved, while validation for connection-related headers is done in HpackDecoder.
Backport of #12975.

@chrisvest do you know why java17-graal pipelined failed 2 times in a row at "check_build_result.sh" task? how can I check the build-leak.output file?

@idelpivnitskiy it failed due a timeout:
https://pipelines.actions.githubusercontent.com/serviceHosts/b6067acc-bb69-4f41-849f-a8c55fdca4e1/_apis/pipelines/1/runs/17834/signedlogcontent/10?urlExpires=2022-11-10T07%3A42%3A49.2416345Z&urlSigningMethod=HMACV1&urlSignature=5S6Fp5p8Ty3rnNvq6kx7s1mTEIGFOBRxtOdqY7lq75k%3D
Looks not related at all.


hack to fix #41

Description in issue.

Tested on the WS7 at OxIonics. I vote we merge this into master and test on the assorted wavemeters at OITG.


Separate settings prompt, clustering and server

bin/serve to handle prompting.
server/index.js to contain a single pipeline server.
server/cluster.js (for example) to contain clustering logic (used from bin/serve)

We should also allow for opting in/out of clustering, in case this cases issues on certain platforms.

Implemented in 245ba4309da0420019e04a9cf5259d82ae9e8a19.


Controlling function renaming

Is it possible to turn off Closure Compiler manipulation of function names, either via a compiler flag or an annotation, while still using ADVANCED_OPTIMIZATIONS?
Apologies in advance if this question has been answered elsewhere. I'm quite familiar with exports and externs (as well as when to use which), however it seems that exporting is only possible for properties and there's no way to tell the compiler to preserve function names (even globally)?

You can preserve a function in a fashion by overwriting a function defined in an extern.
// ==ClosureCompiler==
// @compilation_level ADVANCED_OPTIMIZATIONS
// @output_file_name default.js
// @warning_level VERBOSE
// ==/ClosureCompiler==

alert = function(msg) {
  console.log(msg);
};

alert('hello world');

However, the compiler will still change the argument names.

Sorry, I should have been more clear. I meant function names as in an actual function object's .name property.
In the same example:
alert = function alert(msg) {
  console.log(msg);
};

alert('hello world');

While the global alert is preserved here the function name from function alert() is stripped away. Is there any way to preserve the function's name property itself?

No - that is not supported. See #942 for more discussion on the issue.

Ah ok... I was assuming that it wasn't possible but I didn't manage to find this issue. Thanks!


deis-store-metadata won't start (missing ceph-mds)

Hi,
I'm using a fork of Deis based on 1.6.0-dev on a fresh cluster.
http://imgur.com/hfOrz9a

I "fixed" the problem by building a new cluster.


[Postgresql Connector] TestAT

Coverage decreased (-0.1%) to 35.022% when pulling b1bb1d61ebf06b82225fda694edafbac1030466e on pmadrigal:feature/postgresql-testAT into 6225af29b1a2d81c237eb5fc152a3dff86cf39a7 on Stratio:master.

Coverage decreased (-0.03%) to 35.129% when pulling b1bb1d61ebf06b82225fda694edafbac1030466e on pmadrigal:feature/postgresql-testAT into 6225af29b1a2d81c237eb5fc152a3dff86cf39a7 on Stratio:master.

Coverage remained the same at 35.159% when pulling 756b8790491b3755299f6ad17e0b7a01db6bdeb7 on pmadrigal:feature/postgresql-testAT into 6225af29b1a2d81c237eb5fc152a3dff86cf39a7 on Stratio:master.

Coverage remained the same at 35.159% when pulling 6edcef881773866b8130c84731493089a33872bb on pmadrigal:feature/postgresql-testAT into 6225af29b1a2d81c237eb5fc152a3dff86cf39a7 on Stratio:master.

Coverage remained the same at 35.159% when pulling 6edcef881773866b8130c84731493089a33872bb on pmadrigal:feature/postgresql-testAT into 6225af29b1a2d81c237eb5fc152a3dff86cf39a7 on Stratio:master.

Coverage remained the same at 35.159% when pulling 0ad3e428619afb2ed50e17cf16f04bdfd0210b4c on pmadrigal:feature/postgresql-testAT into 6225af29b1a2d81c237eb5fc152a3dff86cf39a7 on Stratio:master.

Coverage remained the same at 35.159% when pulling 8fa97e16b74bde74658f6f267952be67a72f73c9 on pmadrigal:feature/postgresql-testAT into 6225af29b1a2d81c237eb5fc152a3dff86cf39a7 on Stratio:master.


Listening for specific paths on FirestackModule

Hi,
I'm trying to listen to a specific path changing via FirestackModule.

import { FirestackModule } from 'react-native-firestack'
const users = new FirestackModule('users', {firestack})

export default combineReducers({
	users: users.reducer
})

Is there a way to just listen to /users/1 for example if I just need to listen to changes on that path?  Can't find much documentation on FirestackModule, so hopefully someone can point me in the right direction. Thanks in advance.

@prescottprue How's integration with your redux module?


Generate sample client side queries based on schemas.

We can use the graphQL code generator to generate client-side typings and queries. Additionally, we can investigate generating react based forms based on schema. For that we can use https://uniforms.tools/

I have done some playing with this and the way I see it is that.
On Init, we will ask if we want to generate client-side code.
We will have a template for the client-side.
graphback init

Do you want to generate client? Yes/No

This puts flag in config and client folder in server.
Client should contain:

Queries and mutations (typesafe)
Very very simple test that will allow people to use client and leave query/mutate unimplemented.

Bonus - we can link queries into the playground (server)
Follow up. Blog post - end to end development experience on graphback

Update. Uniforms on it's own provides really good solution, however we need to generate sample queries and mutations for GraphQL first including input types. Then we will be able to feed then into uniforms or other elements.
I have done POC as part of the https://github.com/aerogear/modern-appdev-workshop/

Generally generating sample queries and mutations that accept parameters like here:
https://github.com/aerogear/modern-appdev-workshop/tree/master/template/client/src/graphql
will be the most critical step. Then we can get fragments and other things

Do we generate a fixed query and mutation like findAllTasks and createTask for example?

Take look into the example linked above. Variables are passed from client

Going to generate CRUD queries for each type with arguments as variables into specific folders like in the example.

generate sample queries. We will follow up with uniforms tool to pick that up


PacketTracerPatcher.exe

Where is PacketTracerPatcher.exe？

@WinniyGD build it in Visual Studio using .NET Framework 4.6.1.

but App.config

how to download PacketTracerPatcher.exe