injector: add openssl

Signed-off-by: Long<EMAIL_ADDRESS>Description
It is only a POC for Dapr to use OpenSSL via CGO, the sidecar injector is a single component and is used frequently in the cases that lots of services start and stop.
Perf tests increase by about 10% at local, @artursouza @yaron2 please call perf tests with GH to reproduce it as I do not have permission.
Issue reference

Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

Does not build on ARM.

@artursouza @yaron2 still need all CI pass then we can run perf test? I just want to show how we can get benefit from CGO with OpenSSL, so it is not a complete PR, could we run perf tests directly?
If not, I will make it complete.

Can you help us understand why injector performance is worth introducing a CGO dependency? Even on the busiest clusters, pod launch rate is very likely to be <1/second. Our own perf test suite only launches 11 containers over roughly 30 minutes of the test run. I'm not sure how a 10% performance improvement on those 11 requests could even be visible?
Also, the KinD tests are failing, which means perf tests aren't likely to be able to run.

@wcs1only I would fix KinD issues. It is to show we can introduce CGO to optimize SSL related, a try indeed. It does not very obvious like calling something like SHA256 but works in some cases. In Spike/Big promotion case, some clusters need to inject 100+/s pods in responding to big traffic.

Even at a burst of 100/s of pods/s, the injector TLS performance is not at all likely to be a scaling factor. You are free to fix this PR enough that we can run the perf tests against it, but we'll need to prove measurable benefit to actual workloads before accepting a CGO dependency like this.

@wcs1only I would fix KinD issues. It is to show we can introduce CGO to optimize SSL related, a try indeed, not for merge. It is not very obvious like calling something like SHA256 but works in some cases. In Spike/Big promotion case, some clusters need to inject 100+/s pods in responding to big traffic.

@daixiang0 the sidecar injector can be scaled out horizontally, as the k8s API server uses a round-robin load balancing approach for the Injector's k8s service. This is unlike the placement service, for example, that can only have one active instance at all times. So according to this, I'm not sure the CGO drawbacks are worth the single instance performance benefits. What do you think?

Yes, it is not a significant factor I think, maybe I need to choose a better one for POC.

@yaron2 Accelerating the connection to the k8s API server will make the sidecar creation requests get faster response, which will be particularly obvious in large-scale clusters. Although k8s API server uses a round-robin load balancing approach, accelerating each connection response still makes sense.
@beiwei30 WDYT?


Ability to use in plain HTML/CSS

First of all, I want to say stitches looks like one of the most promising libraries I've seen in a while, following styled system/theme UI. I'm looking to adopt a decent scale enterprise app.
Is your feature request related to a problem? Please describe.
I can only use stitches with CSS in JS? No way to export compiled CSS.
Describe the solution you'd like
It would be great if there was a way to compile CSS utilities, tokens and components into a static CSS file.
Describe alternatives you've considered
Style dictionary. Tailwind.

Not sure if this answers your question but you can use getCssText to access the css string

Stitches does not currently offer a way to use it without JS
As @wobsoriano said, you can use the getCssText API to return the CSS string for a page, but even then, your components need to be authored in JS.
Good news is that we will start working on a static extraction feature, which will essentially allow users to use Stitches in a similar fashion than CSS Modules.
So, keep an eye out, and follow us on Twitter! 🚀


[question] applicability to 1D chromatograms

Hi,
Very nice work!
Since you did a partial re-write of peakonly and get to know it well, I would have the following question:
In your opinion, what would be the easiest way to use peakonly to detect peaks, not in a 2D chromatogram (retention time, intensity, m/z), but in a 1D chromatogram (retention time, intensity) coming from another detector than MS. Peak shapes should be similar.
I would love to hear from you about it. If my question is unclear for you, I am happy to develop and/or share .mzML files with multiple spectra/chromatogram objects coming from various detectors in it!
Best,

It is just a wrapper around the code from the original repository. So, I did not rewrite the code or retrain the models. I think, the nature of how neural networks work, you would have to retrain the networks and change the network architecture in order to make it work for 1D case. Did you try the find_peaks function from scipy.signal? You may have to add a bit of preprocessing like smoothing to make it work.


IOS to Android Video call Echo from the iPhone

Describe the bug
When making a video call from android to ios there is an echo produced by the iPhone and is heard on the android device. I tested it in ideal environments and followed agoras recommendations on reducing echo. But I still receive an echo.
When the Android user speaks he can hear himself 1 second later.  When the iOS user speaks he hears no echo.
I have narrowed it down to the iPhone only on speakerphone. Modern iPhones have multiple microphones, I don't know if this would cause any issues.
To Reproduce
Steps to reproduce the behavior:
Make a Video Call from an Android device to an iOS device.
Expected behavior
For the iPhone to apply some sort of Echo Cancelation and the Android user to not hear an echo
Screenshots
Not applicable
Smartphone (please complete the following information):

Device: [e.g. iPhone13]
OS: [e.g. iOS14.8]
agora_rtc_engine 4.0.7

Additional context
I have tested with multiple different iPhones calling android devices and it always an echo coming from the iPhone I have not had any call where this was not the case to me this is a consistent issue that I have no Idea how to solve.

seems related to https://github.com/AgoraIO/Agora-Flutter-SDK/issues/466

How is this related to #466? #466  deals with the speakerphone causing the person not to hear the other person talking. This issue deals with an echo produced by the iPhone.

@louisschprs are two devices too near?

@LichKing-2234 Hi, I have separated the 2 devices into different rooms on the same network, I have also called someone who was 50 km away via the internet. In all my calls the Android user hears an echo produced by the iPhone user. The iPhone user however has no echo on their side.
The echo that you hear is your own voice playing back to you.
When two iPhone users call each other both hear an echo.

I will test it tomorrow.

Okay, thank you. I think it is the iPhones echo cancellation interfering with the Agora Echo Cancelation. But I have no Idea how to switch on and off one or the other.

Btw, which version of our SDK you used?

I'm using the Flutter package agora_rtc_engine version 4.0.7. I'm not sure what the native SDK version is that this package is using.

 agoraEngine.setAudioProfile(AudioProfile.SpeechStandard, AudioScenario.ChatRoomGaming);     agoraEngine.enableAudioVolumeIndication(1, 1, true);
This is my audio profile settings

Hi @LichKing-2234 any news from the testing?

It works for me, could you pls provide the sdk log file?

Sure, how would I find the file on xCode and on android?

Pls refer to the comment on setLogfile method.

Okay. would you require the logfile from the device that produces the echo or the one that hears the echo

@louisschprs Yes

Which one?

Okay. would you require the logfile from the device that produces the echo or the one that hears the echo

Providing both of them is best.

I Also have this issue, seems to be only on iOS 14 devices. On an old iPhone6 Device, for example, that does not happen. But out QA team found 100% of the time an iphone withy iOS14 without headplugs will cause echo.

I use the

    AudioProfile audioProfile = AudioProfile.SpeechStandard;
    AudioScenario audioScenario = AudioScenario.Education;

    await currEngine!.setAudioProfile(audioProfile, audioScenario);

I was thinking that maybe it has to do with this new 'audio profile' feature that iOS has.

I tested the Agora example app, did not find an echo. Then Created a new app and ported only the calling functionality over to see if it is something else in my app potentially causing the issue. The isolated version did not produce an echo.
So there is something inside the app causing issues on iOS and to be honest I cannot find where the problem is.

agorasdk.log
Here is the log of the iphone (a pain in the arse to get it, damn i hate ios) (ps. im mirrorlink, just different account)

Im gonna try a few things:

not setting the audio profile
not using a custom capturer (we do it to render masks on the users faces)

and see if I get any result

After some tests, it seems that this issue is related to use a custom native frame capturer, that implements AgoraVideoSourceProtocol and use a AgoraVideoFrameConsumer

hey @LichKing-2234 do you know if agoraio flutter plugin support adding masks? I know it already supports virtual background, but having masks would be great so I woudlnt need to mess up with native code. I know there is a face detection, but it seems to be missing the face rotation and also an option to render an image over. Thanks!!

@LichKing-2234 it seems now agora has a agora_rtc_rawdata plugin. Does this plugin work alongside with the "enableVirtualBackground" method? or, if I implement the agora_rtc_rawdata , the enableVirtualBackground will not work? thanks!

So, just to close this issue, I think it ended up being a problem when in my code I had a custom AVSession created. It was a very very old code, but the new agora_rtc_rawdata should do the trick much better.

I have an update on my situation. It turns out that the problem was caused by an external package and not Agora itself.
Before implementing Agora I implemented Flutter WebRTC, this proved time consuming and we decided to go with Agora. However, I never removed the package from the pubspec file. Once removed the echo disappeared.


Updated files, added requirements.txt and updated installation instructions in README.md

Hello @vinitshahdeo,

As per the updated Twitter API, some methods have been deprecated, so I've updated the code which has the updated methods.
Removed all the hard-coded API keys which were present in the files of 'hashtag' subdirectory.
Also, I have added requirements.txt so that the necessary libraries could be installed with one command and subsequently updated README.md with added installation instruction.

By the way, this is my first PR and also my first contribution to Open Source Projects. I would greatly appreciate constructive feedback and look forward to contributing value to even more projects in the future.😊
Regards.

Thank You for the reply @vinitshahdeo
Yes, I did run the code locally and got a few errors, which are the ones I resolved and committed to the forked copy of my repo. The only issue I was facing was with the API giving the error with code 453 which I later found out through various discussion on forums that when Twitter was acquired and later changed to X, the Developer Free tier also went through some changes due to which even pulling data from X using the API was not given access to the free tier users. And even for the basic access we need to buy the basic package which costs $100/month. Thus, I was unable to fully test the code, but I went through various discussions which led to my final conclusion that the rest of the code should be working correctly. I've attached relevant screenshots as well.

Also, please refer to this doc file in which I have attached the screenshot of the resolved issues:
https://docs.google.com/document/d/1R8U--17XPKwj3elAIpdurQkAy41kPxjv6J1zCW_ydQ0/edit?usp=sharing
I would greatly appreciate any kind of feedback which would help me grow. 😊
PS: I forgot to save the files before committing to the repository, which I noticed now, thus the search and the tweeperror method are still unchanged in the files inside the hashtags subdirectory. I'll commit the changes to my forked repo now.

Thank you! Merging it to develop.

Hey @vinitshahdeo
Thanks a lot for approving the pull request. This PR approval not only made my day a lot better, but also made the start of my journey even more memorable and this journey even more exciting.😊
With this, I'm officially on the journey of being an open source contributor.
Thanks a lot!

Thank you so much. Welcome to the open-source world.
All the best:)
On Tue, 13 Aug, 2024, 9:43 am Shashank Jangde, @.***>
wrote:

Hey @vinitshahdeo https://github.com/vinitshahdeo
Thanks a lot for approving the pull request. This PR approval not only
made my day a lot better, but also made the start of my journey even more
memorable and this journey even more exciting.😊
With this, I'm officially on the journey of being an open source
contributor.
Thanks a lot!
—
Reply to this email directly, view it on GitHub
https://github.com/vinitshahdeo/jobtweets/pull/52#issuecomment-2285305753,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE5D5FS3WYPEBWCOFUMIWUTZRGBWVAVCNFSM6AAAAABMMMZMTGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBVGMYDKNZVGM
.
You are receiving this because you were mentioned.Message ID:
@.***>


Composer.json aborts the installation

PHP Version
8.1.10
Shopware Version
dev-trunk
Expected behaviour
Running bin/setup the composer.json will install the packages
Actual behaviour
Execution is aborted with the error:
Your requirements could not be resolved to an installable set of packages.
Problem 1

Root composer.json requires shopware/platform 6.4.x@dev || dev-trunk -> satisfiable by shopware/platform[dev-trunk].
shopware/platform dev-trunk requires php ~8.1.0 || ~8.2.0 -> your php version (7.4.3; overridden via config.platform, actual: 8.1.10) does not satisfy that requirement.

If I change line #31, in root composer.json, to:
"php": "8.1.10"
The error is:
Problem 1

Root composer.json requires shopware/platform 6.4.x@dev || dev-trunk -> satisfiable by shopware/platform[dev-trunk].
shopware/platform dev-trunk requires league/flysystem-memory ~3.10 -> found league/flysystem-memory[3.10.3, 3.x-dev] but it conflicts with your root composer.json require (1.0.2).

So, how to solve this?
How to reproduce
I'm following the installation (for MAMP) here:
https://developer.shopware.com/docs/guides/installation/mamp

league/flysystem-memory v3 seems to new for 6.4. 6.4 still uses flysystem v1


Tree: speed up uniformChunk tests by making the polygon test have fewer sides

Description
This speeds up correctness and performance tests for uniformChunk cursor.
Note that the performance tests for polygon data will not be comparable across this change since its measuring something very different.

@CraigMacomber you will also need to re-add it to the test data, as it was removed to unblock the build failures

@CraigMacomber you will also need to re-add it to the test data, as it was removed to unblock the build failures

I'll make sure a followup with that happens.


Grammatical fixes and adjustments for readability.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@loridean Awesome! Thanks for making these improvements to the readme! Congrats on your first open source contribution to this repository!

 :)


Defaults in Trello getBatch are 'undefined'

e.g.
  /batch:
    get:
      operationId: getBatch
      parameters:
        - default: undefined
          description: 'list of API v1 GET routes, not including the version prefix'
          in: query
          name: urls
          required: true
          type: string

@bobby-brennan Can you make PR that removes them?


Satisfaction survey - put in place a process to review findings

@georginaplatt, shall we just review any findings from satisfaction survey in upcoming sprint, then close this?

@georginaplatt, did you ever get the survey data? Was there anything of interest in it?

@georginaplatt, Ana tells me you okayed us removing the survey so I've prepped it for removal in Monday's release. Shall we close this ticket?


Feature: Improve Dio exception reports

:scroll: Description
Adds an event processor, which add chained exceptions for the inner exception of DioError,
see https://sentry.io/share/issue/638a3758a43d4f61bc2a0c62acdefd69/
:bulb: Motivation and Context
Fixes https://github.com/getsentry/sentry-dart/issues/715
:green_heart: How did you test it?
:pencil: Checklist

[ ] I reviewed submitted code
[ ] I added tests to verify changes
[ ] I updated the docs if needed
[ ] All tests passing
[ ] No breaking changes

:crystal_ball: Next steps

I never get any DioError in the FailedRequestClient , the whole block here: https://github.com/flutterchina/dio/blob/develop/dio/lib/src/dio_mixin.dart#L620. is executed later.

This works, I have the correct in-app grouping.

I basically copied this and and added some more code from FailedRequestClient in order to add a SentryRequest to the event. And I disabled the FailedRequestClient. This combination now gives me a pretty good result.

That's actually a pretty good idea.

The only problem I see, is that the error is not tied to the transaction in any way.

As this integration doesn't start any transactions, you basically have to write code like the one below anyway
  final transaction = Sentry.getSpan() ??
      Sentry.startTransaction(
        'dio-web-request',
        'request',
        bindToScope: true,
      );
  try {
    final response = await dio.get<String>('https://flutter.dev/');
    transaction.spanStatus = SpanStatus.ok();
  } catch (exception, stackTrace) {
    transaction.throwable = exception;
    transaction.spanStatus = SpanStatus.internalError();
    Sentry.captureException(exception, stacktrace);
  } finally {
    await transaction.finish();
  }

and then it is tied together.

As this integration doesn't start any transactions, you basically have to write code like the one below anyway

True, it is all tied to my Flutter route transactions.

Codecov Report

Merging #718 (61a5b31) into main (6206e85) will decrease coverage by 2.35%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #718      +/-   ##
==========================================
- Coverage   90.70%   88.34%   -2.36%
==========================================
  Files         104        6      -98
  Lines        3336      103    -3233
==========================================
- Hits         3026       91    -2935
+ Misses        310       12     -298

Impacted Files
Coverage Δ

dart/lib/src/protocol/mechanism.dart

.../lib/src/integrations/debug_print_integration.dart

dart/lib/src/sentry_transaction_context.dart

dart/lib/src/integration.dart

dart/lib/src/protocol/sentry_trace_context.dart

dart/lib/src/protocol/breadcrumb.dart

dart/lib/src/sentry_client.dart

dart/lib/src/protocol/sentry_trace_header.dart

dart/lib/src/sentry_envelope_header.dart

dart/lib/src/platform/platform.dart

... and 88 more

Continue to review full report at Codecov.

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

When a description is set on the mechanism, the type is not shown in Sentry UI. Not sure why.

When a description is set on the mechanism, the type is not shown in Sentry UI. Not sure why.

Yeah, that's also true for FlutterError which sometime have the description set.
I don't know whether that's working as expected or not. Maybe @marandaneto know.
Probably this is worth an issue at the Sentry repo itself.

When a description is set on the mechanism, the type is not shown in Sentry UI. Not sure why.

Yeah, that's also true for FlutterError which sometime have the description set. I don't know whether that's working as expected or not. Maybe @marandaneto know. Probably this is worth an issue at the Sentry repo itself.

I agree, an issue there would be better, I don't know the answer either.

There are a lot of trade-offs here and I'd need some time to review this and decide how we could improve the chained exceptions and grouping for dio, the discussions are insightful tho, thank you @ueman and @kuhnroyal

@ueman and @kuhnroyal when it throws e.g. SocketException, the stackTrace has only one frame and it is low level, those events are grouped together, I don't see a big problem for that since it does not matter where it comes from, there's no network or you can't do much about it anyway.
If it's about failedRequestStatusCodes, it uses the StackTrace.current and it does not contain the App's frames, fair, events may be grouped together since the frames are almost always the same, but the important bit here is, which URL is throwing? and this is answered by the events, does it matter if the same API is called from different places?
DioError seems to be richer in both cases.
If you have access to the DioError and it throws because of e.g. 404, you have access to the stack trace with App's frames.
If you have access to the DioError and it throws because of e.g. no network, you have access to dio.error which is the wrapped SocketException, also dio.stackTrace has Apps frames.
so DioError is always better, DioError.stackTrace is always better than the captured stackTrace via the catch block, dio.error could be a chained exception.
Do you know if using Dio Interceptors, do we have access to the DioError?
What was the trade-off between the Adapter vs the Interceptor?

so DioError is always better, DioError.stackTrace is always better than the captured stackTrace via the catch block, dio.error could be a chained exception.

Yes thats right.
Adapter was easier to implement and may be easier to maintain, as the implementation is similar to the one for the http package.
But an adapter doesn't know about the DioError, an interceptor does.
The interceptor is harder to configure correctly, as interceptors actually can prevent following interceptors from running or modify the response/error.

so DioError is always better, DioError.stackTrace is always better than the captured stackTrace via the catch block, dio.error could be a chained exception.

Yes thats right.
Adapter was easier to implement and may be easier to maintain, as the implementation is similar to the one for the http package. But an adapter doesn't know about the DioError, an interceptor does. The interceptor is harder to configure correctly, as interceptors actually can prevent following interceptors from running or modify the response/error.

Indeed, we do something similar for Java-OkHttp and it was quite straightforward -> https://github.com/getsentry/sentry-java/blob/main/sentry-android-okhttp/src/main/java/io/sentry/android/okhttp/SentryOkHttpInterceptor.kt
I don't know the internals of dio or how it handles interceptors, and errors, but it should not be that different.
We could try since I don't see any other way to improve the current impl. without DioError.
@ueman solution with the EventProcessor also seems to be a good idea, I'm wondering which one is better.

The interceptor is harder to configure correctly, as interceptors actually can prevent following interceptors from running or modify the response/error.

That's the point I'm concerned about.
Depending on the configuration we might report errors even if they're later discarded and we could also get the data wrong as it might be modified after recording. Also we might not know about errors because a potential SentryDioInterceptor wasn't even called.
I'm not quite sure how hard or error prone it is to get it right, so I'm gladly hear your thoughts on it.
We probably shouldn't promote the dio integration to stable until this discussion is solved tho 😄

I have used an interceptor based approach so far and it worked pretty well, but the configuration problems are a point of concern.
Maybe it just needs to be documented, I mean now we have documented that ..configureSentry() needs to be called last. With an interceptor we just leave that and default just add the Sentry interceptor it at index 0? Dunno.
On the other hand this "feature" may be what we want. Let's says you have an interceptor which handles OAuth refreshes when there is a 401/whatever response. I can then choose to not forward this response error because I am instead refreshing the token and re-execute the original request. I don't want to see this in Sentry.

If we don't capture events because any other interceptor has discarded it, well, it really depends on the setup and what the other interceptors do, it's a best-effort approach, if the data comes in handy to our interceptor, we're able to enrich and capture errors, otherwise not.
For 401 errors, you can either disable not passing to the error code range or even use the beforeSend callback to drop it.
The interceptor should be configurable enough and fit most of the "normal" use cases, users are free to either configure accordingly or just not use it.
I don't think there's a solution for every edge case, but it should be possible for the majority that does simple REST calls, etc...
What do you think?

For performance tracing it would be very bad to have a span/transaction which is not closed. So performance tracing is going to stay as is.

Is it that bad? Or is it the responsibility of the developer to adapt his code?

The DioEventProcessor is still useful for chained stack traces, so it's also going to stay.

We can get chained exceptions from interceptor as well but then it doesn't work for try/catch... And do you need to capture with try/catch when it was already captured in an interceptor?
How does it work with the http package? Uncaught vs adapter/client?

We will end. up with duplicate events If we capture all errors in an interceptor and then automatically have the same errors captured in the zone...

@ueman I didn't get around to testing again yet, but have a slightly related question.

Nothing much has changed regarding the functionality of this PR. Mostly clean up. There's another one in https://github.com/getsentry/sentry-dart/pull/728. Regarding https://github.com/getsentry/sentry-dart/pull/718#discussion_r794086467, I'll raise another PR which should enable that for all errors, even non DioErrors.

What do you think about adding a way to pass a custom parent span to a request via Dio's extra map, that the TracingClientAdapter will use by default and fallback to Sentry.getSpan() if it is not set?

That should be doable, but better in another PR 😅

await dio.get<String>('https://flutter.dev/asdfasg');, throws because 404 status code: https://sentry.io/organizations/sentry-sdks/issues/3007144862/events/31495690091f4d7684c339e6fd9a6ebc/?project=5428562
await dio.post<String>('https://flutter.dev/', data: Exception());, throws because Exception can't be serialized: https://sentry.io/organizations/sentry-sdks/issues/3009008663/events/fcb1d27e11be4af4b71b25b6c04128c6/?project=5428562

@marandaneto, as you said, code looks good.
But I don't really know how EventProcessor works in the Dart SDK, need to get familiar with this.

@marandaneto, as you said, code looks good. But I don't really know how EventProcessor works in the Dart SDK, need to get familiar with this.

Feel free to ask (here or on Discord), I'll probably can answer at least some questions.

@ueman sentry-dio / package-analysis is unhappy, otherwise LGTM.

@kuhnroyal also thanks for the valuable tests and insights.

I would guess it's because pana doesn't check the dependency overrides? The getter which pana complains about isn't available in the latest stable but on main. I don't think I can do anything about that.
Pana isn't happy because of:
[x] 0/30 points: code has no errors, warnings, lints, or formatting issues
Found 3 issues. Showing the first 2:
ERROR: The getter 'exceptionFactory' isn't defined for the type 'SentryOptions'.
ERROR: The method 'shouldAddBody' isn't defined for the type 'MaxRequestBodySize'.

Indeed, I believe pana should respect dependency_overrides or at least offer an option to respect it since multi-module projects would run into this issue very often.
Maybe worth filing an issue on https://github.com/dart-lang/pana/ as a feature request.
Let's merge and not block on this then :)

I'm fine if you want to address https://github.com/getsentry/sentry-dart/pull/718#discussion_r807145080 or keep as it is, your call.
I'm good to merge as it is.


How can I allow only one client to connect to the server?

I am a beginner. ㅜㅜ

The library doesn't provide a facility to permit/deny based on address/port/netmask.


[feature]: add docs for ev and sample_weight

add docs for ev and sample_weight

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


Update Symbols Rendering Correctly

All Symbols should be rendering correctly

@Dbarnes1 - can you re do/open this pull request - but into v.next (vs. master - sorry I missed that when we went over) - thanks


Add the proximity info on ranging response

From #5.
Some useful stuff:

StackOverflow: estimating-distance-to-ibeacon-on-ios
[StackOverflow: understanding-ibeacon-distancing]
(http://stackoverflow.com/questions/20416218/understanding-ibeacon-distancing)
StackOverflow: measure-distance-to-ibeacon-from-android-device

Gonna implement this in a minute.

Landed on 1.1.5, might need some tuning.

hi,
I actually test react-native-ibeacon on IOS and this one on android.
I have access to and estimote beacons.
and the result from both app are really different.
With my phone next to each other and the beacons 5meter away approximately, i get this result :
//IOS:
[ { major: 32404,
    minor: 62922,
    uuid: 'B9407F30-F5F8-466E-AFF9-25556B57FE6D',
    accuracy: 10.83739327988672,
    rssi: -95,
    proximity: 'far' } ]

//ANDROID
distance: 1.1567290440348383
major: 32404
minor: 62922
proximity: "Near"
rssi: -80
uuid: "b9407f30-f5f8-466e-aff9-25556b57fe6d"

Do you know if we can use the same SDK on IOS and Android in order to have the same result, or at least something similar. :)

Do you know if we can use the same SDK on IOS and Android in order to have the same result, or at least something similar. :)

I'll do some research on the Estimote SDK, but in my opinion this library should continue using the Alt Beacons SDK since it supports almost every kind of beacons, Estimote included.
I also still need to do a lot of testing on aligning the results of this library with the iBeacons one (hope to do them this week).
However please keep in mind that the result will never be the same because it is strictly tied to the Android device and its specs (you can have different result even comparing two Android devices).
Regarding your test, the only two info you need to compare are rssi and proximity.
The rssi one shows the signal strength, the proximity one is an estimation of proximity that is calculated differently in iOS and Android. Take a look at the links in my first post for more info.
If you have any suggestion on how to tweak the proximity value don't hesitate and send a pull request/write a post here :)
P.S.: The distance parameter on Android seems strange... make sure to wait at lease 20/30 seconds before taking notes of the parameter because that result is "smoothed" and require some time to settle :)

I'm closing, feel free to re-open if needed!


Roughness automatic texture packing

Describe the bug
Roughness texture is packed with other empty channels (I'm guessing Metalness and AO?) and becomes pink at export with a drastic increase in weight (2.7Mo from 500Ko).
If forcing JPEG texture export, the texture is still pink and still receive a big increase in weight for no reason whatsoever.
The original roughness texture is a 8 bit grayscale image.
This is bizarre because I remember making a bug report on a similar subject explaining exactly why automatic packing is a bad idea so I don't understand why this bug/feature made it's way into 3.3 (or 3.2 because I upgraded from 3.1)? A .glb that was exported at 1Mo a few months ago is now exported at 4Mo, you can see how bad this is for the web.
So is it just me missing a new export option or are we stuck with this unwanted behavior for now?
I don't want a gltf + bin + texture export by the way, no need to suggest the "Keep Original" option, I know about it but I need the same for .glb.
Thanks!
To Reproduce
Add grayscale roughness texture to material, export to gltf + bin + textures and see the roughness texture increase in weight as well as becoming RGB and pink and a PNG disguised as a JPG.
Expected behavior
Export textures as they are originally instead of forcing a packing of roughness with other (non-existing) textures.
Screenshots

.blend file/ .gltf
khronos_test.zip
Version

OS: W10
Blender Version 3.3

Confirmed.
There is an issue with image size.

No issue on 3.2.1
Issue on 3.3.x
No issue on 3.2.1 with addon code from master (modulo renaming ShaderNodeSeparateRGB/ShaderNodeSeparateRGB)

--> This let me think the problem may come from a change in Blender or Blender api, somewhere between 3.2 and 3.3

I am starting a git bisect to try to identify the Blender issue

Got it:
https://developer.blender.org/T102421

Problem is solved in https://developer.blender.org/rB5f4afecbf3559a7ddcc2621d1e863a5425859404
This should be backported in 3.3.2 (that will be released sometime in december).
I will also add an option (that is added for 3.4 in bpy) to choose the jpeg quality level at export


DE: 3 images redirect to English page instead of German

Go to the following page: https://www.netcentric.biz/de/insights/whitepapers
The top 2 and bottom right image link to the English version of the pages they should be linked to.
Note: the buttons below the images are correctly linked.

Fixed in author. https://www.netcentric.biz/de/insights/whitepapers


Failed to execute 'define' on 'CustomElementRegistry'

I've been using Toggle element for a while now without making any changes to my backend, and today they don't work anymore and I get this error Failed to execute 'define' on 'CustomElementRegistry': the name "bx-checkbox" has already been used with this registry
My product depending on Carbon design is now facing issues that I can't control, this has to be fixed

Can you provide a working example?


Resizable Sidebar

Changes

The sidebar can be now be resized by dragging the divider (closes #3447)
Dragging the sidebar below a minimum size collapses it
A button in the top toolbar lets you uncollapse the sidebar, complete with icon and tooltip
Double-clicking the divider toggles the sidebar collapse
Collapse-toggling is animated
Selecting a feature or ending an add-feature mode auto-opens the sidebar
Photoviewer UI has been moved from init.js to photoviewer.js
The photoviewer now has a slight margin to set it away from the sidebar
The undo/redo, save, and progress indicator controls are now floated to the right
The save count has been redesigned to visually integrate with the save button
When the sidebar is collapsed, the mode buttons are centered
Full right-to-left layout support for all of the above

Uncollapsed

Collapsed

Resize and auto-uncollapse demo

Note
As you can see above, making the sidebar resizable took me somewhat beyond the scope of the original issue. Namely, I thought we could make better use of space in the top bar now that it can take up the full width of the window, so I moved some buttons to the right side. This caused the save count to be cut off, so I redesigned it to be incorporated within the button rather than outside it. I'm open to exploring other solutions if these changes are controversial.

This looks great @quincylvania 🎉
I love that you took the time to do separate LTR and RTL icons.
It looks like the loading spinner might be gone?
Aside from that I didn't see any major issues with the code, so let's just merge as-is and we can tweak it if we find any issues.

@bhousel Thanks!! The spinner shouldn't be gone. I moved it to the left of the undo/redo buttons. I did add display: none to the fullscreen button so it won't take up space in the layout and since it wasn't shown anyway.
FYI: The reason I added the wip tag was that I wanted to give a little more attention to the top bar buttons, likely implementing #4356, but you're right that can be a future change.

I got the following issue with the current Firefox on Windows 10:
When making the sidebar very wide, iD gets into a state where undo and save buttons are out of view, even after making the sidebar smaller again. When the sidebar is collapsed, the buttons are fully visible. When the sidebare is very small, the buttons are partially visible.
This state is somewhat permanent. Restarting iD wasn't sufficient. I had to clear the browser cache to clear the error state.
I wasn't able to reproduce the error with Edge.
Btw, the delete and info buttons in the all tags section should not scale with the sidebar width to provide more space for tag values.


vue-touch-keyboard

The README should be updated since I'm guessing you took the README from another project.
vue-touch-keyboard is available under the MIT license.

Thanks!


aln, baseline

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[a,b]

/trunk merge


Nearly same function name (IDFGH-3690)

https://github.com/espressif/esp-idf/blob/cf056a7d0b90261923b8207f21dc270313b67456/examples/bluetooth/esp_ble_mesh/ble_mesh_sensor_model/sensor_server/main/main.c#L339
Hello,
In the above example, the two function name "example_ble_mesh_send_sensor_setting_status" and "example_ble_mesh_send_sensor_settings_status" having the almost same name, only difference is the 's' in the settings. Please avoid these similar names of the function/variable as they create the confusion in the readers mind.
Thanks!

Thanks for reporting and letting us know, we will look into.

Hi @ankit-thealchemist
Sorry for the confusion caused.
Sensor Settings Status & Sensor Setting Status are just two SIG-defined model status messages, so in the example I just named the functions based on the names of these two messages.
And if some better ideas come on your side, please feel free to submit a PR for the changes.
Thanks.

@Campou thanks for the reply!
I can understand it now! As I am new to the ble mesh, I don't know it. Do we have any resource that have all the models related knowledge such as model status messages, structure, working etc.

Hi @ankit-thealchemist
You can go to the bluetooth official website and download the Mesh Model specification, which includes all the model related definitions (i.e. messages, behaviors, etc.)
Thanks.


era5: back extension to 1940

https://cds.climate.copernicus.eu/cdsapp#!/dataset/reanalysis-era5-single-levels?tab=overview

Confirmed, it works.


Salsa: Require of module shows type as "typeof "

TypeScript Version:  nightly
Code
I've written a fourslash test:
/// <reference path="fourslash.ts" />

// @allowJs: true
// @Filename: a.js
////const /**/x = require("./b");

// @Filename: b.js
////exports.x = 0;

goTo.marker();
verify.quickInfoIs("const x: typeof ");

Expected behavior:
Ideally, const x: { x: 0 }, but at least something that doesn't look like a bug.
Actual behavior:
We get "typeof " as the type.
Note that we don't get any such error when the imported module uses module.exports = 0.

it should be typeof ./b since the module does have a name.


Vue Router does not match some paths containing "." in last path.

Version
4.1.6
Reproduction link
codesandbox.io
Steps to reproduce

Open the Vue.js project in your CodeSandbox browser.
Click on "users/user.id" to go to "https://xxxxxxxx.preview.csb.app/users/user.id".
Confirm that the page is displayed.
Reload the CodeSandbox browser.
Nothing show on the screen.

What is expected?
Matches /users/:id.
What is actually happening?
Nothing is displayed.
The browser's default 404 is displayed.

It does not reproduce when  is clicked. It is reproduced by reloading the browser or accessing the URL directly.
Below is a list of the results we have tried with the same source code.

/users/user-id -> match /users/:id
/users/user_id -> match /users/:id
/users/user.id -> not match
/users/user..id -> not match
/users/user.-id -> not match
/users/user-id/foo -> match /users/:id/foo
/users/user_id/foo -> match /users/:id/foo
/users/user.id/foo -> match/users/:id/foo
/users/user..id/foo -> match /users/:id/foo
/users/user.-id/foo -> match /users/:id/foo

That's a codesandobox bug, it's not showing you the application as if it was trying to get a static file. The . doesn't have any special meaning in vue router params


Configure Dr Elephant to publish artifacts to Jfrog Bintray

Configuring Dr Elephant to publish artifacts to Jfrog Bintray using sbt-bintray plugin.

Add a document on how to publish and release the latest version of Dr.Elephant to Bintray which is the aim for this PR.

Document on how to publish and release the latest version of Dr.Elephant to Bintray:
https://docs.google.com/document/d/1gVHm58ncLYT2YQfZtAxJ5jXGGCc-5GKJKlmnwEP5LvA/edit?usp=sharing

Document on how to publish and release the latest version of Dr.Elephant to Bintray:
https://docs.google.com/document/d/1gVHm58ncLYT2YQfZtAxJ5jXGGCc-5GKJKlmnwEP5LvA/edit?usp=sharing
It would be better to attach this doc with the PR description as Usage Instruction etc.


Take example screenshots in CI

Objective

I want to take screenshots of examples in CI to help with validation of changes

Solution

Can override how much time is updated per frame
Can specify on which frame to take a screenshots
Save screenshots in CI

I reused the TimeUpdateStrategy::ManualDuration to be able to set the time update strategy to a fixed duration every frame. Its previous meaning didn't make much sense to me. This change makes it possible to have screenshots that are exactly the same across runs.
If this gets merged, I'll add visual comparison of screenshots between runs to ensure nothing gets broken
Migration Guide

TimeUpdateStrategy::ManualDuration meaning has changed. Instead of setting time to Instant::now() plus the given duration, it sets time to last update plus the given duration.

Example screenshots can be found in the artefacts section of https://github.com/bevyengine/bevy/actions/runs/4800302635

Would be awesome to use these CI-generated images in the examples page of the website :)

That will be the plan after the next release!


fix ie10+ hangs indefinitely

see https://github.com/flowjs/flow.js/issues/55

Thanks, but try not to include merge commits in pull request next time.

OK.Sorry about it.


implement burning

TODO in this PR:

[x] fix burning dmg pyro app icd (should be global 2s icd shared between characters, but not between enemies)
[x] fix burning damage sequence (there seems to be no limit to the amount of burning dmg that can be taken by a target in 2s)

TODO in later PR:

[ ] write tests
[ ] add OnBurning/Burning check to existing code where necessary (Dendro Resonance +30 EM, 4p CWoF, Collei A1+A4, Tighnari C4)

Yoooooolo


TASK: Set correct dependencies

Hello @jobee ,
I relaxed the version constraint to allow Neos 8 and made a release. What's the reason you wanted to explicitly define the minimal PHP version, I think that dependency should be handled Neos itself.

Hi @rolandschuetz , I noticed the following dependency issues while trying to install your package in a Neos 3.3 / PHP 7.3 project.

remove Neos 3.2 - because it's not compatible for various reasons (Settings YAML filenames, Fusion prototypes, ...)
set the minimum PHP version to 7.4 - because you use typed properties

https://github.com/code-q-web-factory/neos-instagram/blob/48abecd985b91ea7a02578a7245b420a3bbbc6d9/Classes/CodeQ/Instagram/Domain/Service/InstagramService.php#L20
I'm also fine with just relaxing the dependency for Neos 8.x, like you already did. Neos 3.3 / PHP 7.3 shouldn't be used in production anyway ;-)

Thank you for your contribution, @jobee !


Not getting any Login or installation form. Just plain HTML

I installed pathfinder on myserver/pathfinder and got the this issue

Error - 404
Status: "Not Found". Use your browsers Back button to navigate to the page you have previously come

But I solved with help from #149
Since then I'm just getting like a plain white HTML Site <- Link is a picture of what I get.
I could not find any solution to it yet. If I set  BASE = (blank)  again. I'm getting the black styled 404 Error.

I am stuck here too :/  @exodus4d any way to debug what is going on?
i don't have a php log since with php_flag and php_value it just error out with internal error

I still couldn't resolve it. I guess that I miss some settings or something like that. Just can't figure out what.

Are you running the site on a subdomain?
Have you looked in the webserver log to see the error message when you hit that page?
Have you doublechecked so that it's trying to work out of the correct path?
That to me seems like it's trying to grab the css files from the wrong directory.
If you can provide the URL you host the webpage on that'd help to pin-point the issue.
I'd also suggest joining Slack and asking in the support channel there.
Click here to join the pathfinder Slack.

is that slack open? I have been unable to sign in to it to ask for assistance.
Trying to install on https://sub.domain.org/pathfinder
the index page is loading, but /setup, /login etc are 404ing.
running on Apache on centos 7 with php7 installed.

@toro- You can request Slack access hier. Please send me your mail, Ill send you an invite<EMAIL_ADDRESS>I´m not an Apache expert, but I´m sure someone else is able to help you there.

@toro- Did you ever figure this out? I have the same setup as you and am experiencing identical issues...


Use kustomize's output to generate CRDs for helm

Ideally, we'd want all chart resources to be generate from the kustomize output.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/lgtm


Auto resize images before upload

My label studio instance runs locally while users access it over internet. But it takes really long time for a user to upload all tasks in the project since many images are of high resolution and weigh up to 10 Mb. I wonder what is the normal workaround for this. Is there a way for newly imported images to get auto-resized so as not to put a heavy load when a new user accesses a project?

Hello Fox111,
Usually, you would want to use either cloud storage or local storage that will allow for the faster upload. Maybe the API or SDK can help with upload as well


cmd/compile: mark pointer types pointing to noalg types noalg too

Similar problem to #32595.
Found this bug while investigating avoiding to create algs for array types that are only used as backing array for slices.
It seems this is the reason our initial attempts at introducing that optimisation in https://go-review.googlesource.com/c/go/+/151497 failed with test errors.
Arrays marked noalg are created by the compiler to hold keys and values
to initialize map literals. The ssa backend creates a pointer type for the
array type when creating an OpAddr while processing the loop that
initializes the map from the arrays. The pointer type does not inherit
the noalg property but points to the noalg array type.
This leads to strange problems were the same values of a type dont compare equal anymore if created through reflect:
package a

func A() {
   var m map[int]int = map[int]int{
      0: 0, 1: 0, 2: 0, 3: 0, 4: 0, 5: 0, 6: 0, 7: 0, 8: 0, 9: 0,
      10: 0, 11: 0, 12: 0, 13: 0, 14: 0, 15: 0, 16: 0, 17: 0, 18: 0, 19: 0,
      20: 0, 21: 0, 22: 0, 23: 0, 24: 0, 25: 0, 26: 0, 27: 0, 28: 0, 29: 0}
   if len(m) != 30 {
      panic("unepexted map length")
   }
}

package b

import "reflect"

func B() {
   t1 := reflect.TypeOf([30]int{})
   t2 := reflect.TypeOf(new([30]int)).Elem()
   if t1 != t2 {
      panic("[30]int types do not match")
   }
}

package main

import (
   "a"
   "b"
)

func main() {
   a.A()
   b.B()
}

Im sending a CL to fix this and also a test case. But first need an issue number for the file name.

This appears to be a pre-existing issue since at least Go 1.16.5: https://play.golang.org/p/DFjkXhMtEzO
This is an unfortunate issue, and it should be fixed. But I think fixing it will be too invasive for Go 1.17.
As mentioned on CL 333030, I think the right fix here is to teach the linker to realize alg and noalg type descriptors can be deduplicated, but to prefer the alg variants when present. There's no need for alg and noalg types to have different descriptors; noalg types only exist as a space optimization to avoid creating the equality and hash functions for generated types that we know will never need those. And we can save even more space by getting rid of the redundant noalg variants when they're not needed.
(I'm sure there's some trickiness with dynamic linking here too. If we don't already, maybe we should just turn off noalg for dynamic linking. I imagine there are / will be similar issues there, and I think we have less control over symbol definition priority in the OS's dynamic linker.)


chore(package): update mocha to version 6.1.2

closes https://github.com/sifrr/sifrr/issues/96

Pull Request Test Coverage Report for Build 1878

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 92.576%

Totals

Change from base Build 1876:
0.4%

Covered Lines:
1671

Relevant Lines:
1805

💛  - Coveralls


@JobWorker: Pass custom args to business method

With Camunda 7 we managed to pass custom arguments to business methods (CamundaDelegate) to hide secondary business aspects from the implementation of a e.g. service-task. As we are now moving to Camunda 8 we try to do the same and also found a way:

We implement a bean of type ZeebeWorkerValueCustomizer which will be picked up to adopt the ZeebeWorkerValue object.
In the ZeebeWorkerValue we replace the original MethodInfo by a derived class which overrides the "invoke" method to inject the null-arguments unknown to the Spring Zeebe client.

To do so, we needed to add a copy-constructor to MethodInfo since there is a Builder and a private constructor which prevents the class to be extended: https://github.com/stephanpelikan/spring-zeebe/commit/6f50f13a0e266c463fcc084027bde99c2a1d8dd1
Before we provide this a PR we would get confirmed that this is the right way to achieve this and its stable to use ZeebeWorkerValueCustomizer for this.

@stephanpelikan would this feature solve the issue? https://github.com/camunda-community-hub/spring-zeebe/pull/656


[Bug] Vagrant disksize plugin doesn't resize root filesystem

Describe the bug
We're using the vagrant plugin disksize to increase the disk storage, but within the VM, the actual filesystem remains the same
To Reproduce
Steps to reproduce the behavior:

Launch the ts.elastomic VM per the instructions
SSH into the system
Run lsblk
See root partition is only 50G

NAME            MAJ:MIN RM  SIZE RO TYPE MOUNTPOINT
sda               8:0    0  256G  0 disk
├─sda1            8:1    0    1G  0 part /boot
└─sda2            8:2    0   79G  0 part
  ├─centos-root 253:0    0   50G  0 lvm  /
  ├─centos-swap 253:1    0  3.9G  0 lvm  [SWAP]
  └─centos-home 253:2    0 25.1G  0 lvm  /home

Expected behavior
I expected the filesystem to have grown to the configured value in the Vagrantfile like so:
NAME            MAJ:MIN RM  SIZE RO TYPE MOUNTPOINT
sda               8:0    0  256G  0 disk
├─sda1            8:1    0    1G  0 part /boot
└─sda2            8:2    0  255G  0 part
  ├─centos-root 253:0    0  226G  0 lvm  /
  ├─centos-swap 253:1    0  3.9G  0 lvm  [SWAP]
  └─centos-home 253:2    0 25.1G  0 lvm  /home

Proposed fix
The following script could be used as a systemd service on boot to grow this. Fortunately, XFS can be resized while online, but that should probably happen before you launch Elasticsearch or other services.
#!/bin/bash

the_root_device='/dev/sda'
the_dynamic_partition='2'
the_dynamic_partition_path="${the_root_device}${the_dynamic_partition}"
the_root_vgname='centos'
the_root_lvname='root'
the_root_lvpath="/dev/${the_root_vgname}/${the_root_lvname}"

# Grow the partition table
growpart "${the_root_device}" "${the_dynamic_partition}"
sync; sync; sync

# Reload the partition table
partprobe
sync; sync; sync

# Resize the physical volume group
pvresize "${the_dynamic_partition_path}"
sync; sync; sync

# Resize logical volume to the full disk, then grow the filesystem
lvresize -l +100%FREE --resizefs /dev/centos/root
sync; sync; sync

The above depends on the following
sudo yum install cloud-utils-growpart

Better fix
To further simplify your life, update the Elastomic box using standard partitions instead of LVM, then all you have to do is install the following and reboot. The root partition will auto-resize any time there is free space when booted.
sudo yum install cloud-utils-growpart cloud-init

Included sudo yum install -y cloud-utils-growpart into the above bash script and added it to the vagrant file just before elastic provisioning. initial tests look good. rebuilding a clean env and testing changes again.
Branch: https://github.com/thremulation-station/thremulation-station/tree/kirk-vagrant-disksize

Build is Good

Fixed by Kirk.


pickle.load ran out of input at dataset.py

pickle.load ran out of input during training. Did I do something wrong？

Could you provide the error message and the memory size of your machine?

It seems that some of the dataset size is empty then result in pickle.load turn out to be error. How can I fixed it?

This could be a mistake when i finally organizing the data, i have already update the data. Could you please try again？

hello
Seems that the problem still remain!

I'm not really sure why the promblem still exist. The file size is correct this time...

Sorry, it seems that 7084db*.p is still incomplete. You can download this single pickle file (updated) from the attachment.
I have checked that all the pickle files should be able to load from my end this time. Please feel free to let me know if the problem still exists. I will update the dataset file later (only change one file).
7084db2193bc9ee43d113b8340bd5029.p.zip

It happens to be successful since I changed the file 7084db*.p only. I'll keep aware if there is other problem. Thank you.


Deploy new & updated contracts

Description
Changes from #23 require a deployment of RandomConsumer.cdc and a redeployment of CoinToss.cdc. The relevant PR should also include updates to the README pointing to the updated deployments.

Running into failures deploying CoinToss to Testnet using CLI, reported in Discord thread.
Will pick up after issue has been identified and resolved.

CoinToss.cdc

Testnet - 0xb6c99d7ff216a684
RandomConsumer.cdc
Testnet - 0xed24dbe901028c5c
Mainnet - 0x45caec600164c9e6


Intern new

A small short PR that enables a new-factory to instantiate itself.
previously there was no way to do that. The following code will stack-overflow:
class A
   new do
      var res = new A
      res.do_thigns
      return res
   end
end
var a = new A # infinite recursion of `new A`

Note: this is a very bad example as what is done in the previous new should be done in an init instead since it is related to the initialization of the object. A new factory should be exclusively used for some factory-concern like returning a more specialized object or returning an already existing object.
With this PR, the primitive constructor is available and is called intern, as the annotation that indicates things that cannot be programmed in Nit but are provided by the execution engine.
So in the previous example, just use this primitive constructor instead of doing a recursive call:
      var res = new A.intern

This intern constructor just do the allocation and the initialization of attributes with default values.
init is not called.
Maybe it should not do the initialization of attributes neither but if I skip them, there is no way for the programmer to ask for them manually. whereas init in not automatically called but can be called manually.
note: this PR is a small step toward the conclusion of the constructor saga, more will come soon (I hope)

I would have used the init with attributes, for visibility concerns. Unless... do we force autoinit setters to be public?
+1

@xymus wut? I am not sure I understand what you are saying.

My point was that I would have expected intern to refer to the anonymous init. But by rereading the description of the PR I understand the intention. How can we call init from a new?
new do
    var a = new A.intern
    a.init # The KISS way?
    return a
end

The fact is that there is to much thing called constructor and init in this language :/

The basic allocator.
It just allocates a block of memory and setup minimal things.
Basically it returns a new empty object. mostly a larva still in the egg.
It is not accessible from the language.

The internal constructor.
This one allocates the object then sets up the default values of attributes. This is one designated by the present PR.
In the generated C code it is called NEW_cnameofclass.
Basically, it returns a larva that can crawl.

The primitive anonymous init.
It is the init with no parameter. Object::init.
It is just a convenient place to put initialization things that will be usually called at the end of the whole construction.
Because there is no parameter, it is easy to combine them in multiple inheritance.

The autoinit.
This is not a real thing and correspond to:

the execution of all autoinit annotated parts (mostly attribute without default value), then
the execution of the primitive anonymous init.

While not a real thing you can use it: indirectly in a natural anonymous new or directly in a named constructor since is the init you can call when you delegate a part of the construction to the standard process.

init named do
   things
   init(1, "hello") # call of the whole autoinit, not just Object::init
   more.things
end

The natural anonymous new.
It is when you do new A(args) and correspond to the internal constructor followed by an execution of the autoinit.
It returns a beautiful adult object not in a larvae state.
I suspect this is the one you refereed to?

The named inits.
They contain a full initializers and are used trough named construction (new A.foo(1)) and can use used in other named inits.
Such a new correspond to the internal constructor followed by an execution the initializer.
They are the old-style constructors and because each one is a full initialiser that works by delegation, they are hard to combine in multiple inheritance.

The new-factories
Introduced by the FFI, they are a way to hide a factory function under a new ClassName syntax.
new-factories in Nit, or in FFI, are in fact analogous to top-level methods that just return an object.
When a new-factory is anonymous it shadows the natural anonymous constructor, this issue was the point of the PR.

To answer your specific question, init can be called on the result of the new.intern but, as with named constructor the init is in fact the autoinit with its parameters if any. So:
class A
	var i: Int
	new do
		var res = new A.intern
		res.init(5) # The KISS way!
		return res
	end
end
var a = new A
print a.i # 5

+1, I like it.
This factorize the classic scheme in which we create and object and immediately after we call a method to do things (like memoize the object or whatever).


Decode barcodes on cropped specimen images

Metadata fields need to be populated from data decoded from barcodes on cropped specimens images. Decoding should be robost and require as little human intervention as possible.

The open-source decoders are very slow and/or fail to find barcodes - commercial libraries give much better results. Inselect is currently hard-coded to use the [Inlite ClearImage library] (http://www.inliteresearch.com/). The library only runs on Windows - if you are using a Mac or Linux then we currently have no good solution. ClearImage supports a wide range of barcode types but Inselect is hard-coded to detect only Data Matrix.  See also  #130.

NaturalHistoryMuseum/gouda@f8f229435eb1c5765291b8f0a5013c5abda2484d adds supports for libdmtx - tested on Mac OS X only.

libdmtx via pydmtx also tested on Linux and 32-bit Windows. 64-bit windows build of libdmtx fails with core dump.

libdmtx via pydmtx working on Mac OS X, Linux and 32-bit and 64-bit Windows.

v0.1.8 (f8f0667) introduces support for the open-source libdmtx decoder on all platforms. See
NaturalHistoryMuseum/libdmtx and NaturalHistoryMuseum/dmtx-wrappers.

Adding support for the zbar open-source library would allow 1D barcodes and QR codes to be decoded. The last release (0.10) was on 2009-10-27. There has been no development since and not likely to be any more. zbar is available for all three major OSes but, frustratingly, there is no 64-bit build for Windows - a show stopper.
We would need to

add support for 64-bit Windows
add a user interface config box that allows the user to choose between

libdmtx
Inlite
zbar

User interface for config will be handled by #156

Support for QR requested by AMNH.

Inselect now supports Inlite, zbar and libdmtx. The only gap is that zbar is not available on 64-bit Windows, for reasons given on 2015-04-20. Edit, Configure Read barcodes shows a box allowing the user to choose between the three decoders; options are disabled where a decoder is unavailable:

Our fork of ZBarWin64 adds support for zbar on 64-bit Windows.


Increase diagnostics

What
Increase the ability to diagnose problems when the changes feed throws an IllegalStateException.
How

Name threads created by thread pools for SQLDatabaseQueue and for pull replications.
Name the ReplicatorThread
Provide the SQLite version and include it in the IllegalState thrown by the changes feed.

Tests
No new tests, manually checked that the threads get renamed using the debugger in idea.

+1

+1


Doc: separate data cleaning

Codecov Report

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

@@           Coverage Diff           @@
##           master      #55   +/-   ##
=======================================
  Coverage   94.38%   94.38%
=======================================
  Files          10       10
  Lines         534      534
=======================================
  Hits          504      504
  Misses         30       30

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 401c653...b758880. Read the comment docs.


Nan when training optical flow channels with kinetics pretrained weights

When training the optical flow channel of SSN, the loss become nan after several iterations. Do you have any idea why this happens? Is there any difference between the training settings with imagenet pretrained weights and Kinetics pretrained weights?

I don't think there is any difference.  This problem only happens when I use BN-Inception network with optical flow as input

Are you exactly following the training parameters in the README?

@mangdian Hi, I met the same problem, did your solve it ?

@happygds No, I have not solve this problem yet

with BNInception. I solved this problem by decrease the initial learning rate.
lr=1e-3 --> nan
lr=1e-4 --> ok

See if #89 will solve your problem especially if you have fewer GPUs. It is merged to the master branch.


Can't build CesiumGltf.lib when I cmake for cesium-native

when I exec "cmake --build build --config Release --target install" , No 'CesiumGltf.lib' build under 'extern\build\cesium-native\CesiumGltf\Release'

And No 'cesium-unreal-samples\plugins\cesium-unreal\Source\ThirdParty' was created，that means it build failed ?
because of above， I can't  BuildPlugin with UE 'RunUAT.bat'

what am i supposed to do ?
ENV:
Unreal Engine 5.2.1
cesium-unreal   main branch
cesium-native   main branch
cmake version 3.20.21032501-MSVC_2
visual studio 2019

Sounds like it must have failed. I'd have to see the entirety of the cmake output to take a guess what might have gone wrong.

Oh I just noticed you're using VS 2019 and cmake 3.20. I'd recommend upgrading both (to VS2022 and cmake 3.30).

thanks!
I just change the files to utf-8 bom which were got error above ,such as 'cesium-native\CesiumGltf\ClassProperty.h' and other several files
redo build ,and it works for me

similar to "https://github.com/CesiumGS/cesium-unreal/issues/997"

I think that telling MSVC that the files are UTF-8 should work as well:
https://learn.microsoft.com/en-us/cpp/build/reference/utf-8-set-source-and-executable-character-sets-to-utf-8?view=msvc-170
MSVC's default seems crazy here:

By default, Visual Studio detects a byte-order mark to determine if the source file is in an encoded Unicode format, for example, UTF-16 or UTF-8. If no byte-order mark is found, it assumes that the source file is encoded in the current user code page, unless you've specified a code page by using /utf-8 or the /source-charset option.

In any case, I don't think there's any action for cesium-unreal here, so I'm going to close this.


Not able to send Messages

Since around 12 hours ago i m not able to send messages to non contact by using the contact.chat.id function.
I m still collecting data but it seems whatsapp changed something on their interface and the object we are obtaining has changed.

Yes i did but it seems whatsapp changed their object names and compositions

Try to use other function like sendMessageToID

Why don't we add following if condition in send_message_to_id() available in init.py
initialAttempt = self.wapi_functions.sendMessageToID(recipient, message)
if not initialAttempt:
return self.wapi_functions.sendMessage(recipient, message)
else:
return True
As i have observed sometime sendMessage() works, sometimes not, so it's better to have a fallback if one function fails , then one auto attempts could be made.
Rest, open to your feedback.


Color code the REPL left margin

The regular editor tracks changes in the left margin (e.g. "saved", "same as saved", "changed", etc.).

We could extend the interactive window to colour code inputs/outputs/errors in a similar way.
This would make it easy to scan through and identify sections in the overall log.

hey @zooba, i am a newcomer and would like to take this issue, if no one is assigned. Any code pointers for me. Thanks.

We're closing old issues with no recent activity. If this is still important, please open a new issue.


Shared Headers fields are broken

Unable to add headers to explorer/sandbox configuration. Issue exists in incognito mode and no plugins enabled.
Browser: Firefox 112.0.1
OS: MacOS Ventura 13.2.1

Similar issue reported also on SO https://stackoverflow.com/questions/75981765/why-are-shared-headers-disabled-in-apollo-sandbox

Issue was not present on Apr 12th 2023.

There is no documented way of enabling setting shared headers from withing the sandbox. However, you can pass your shared headers when initializing the Apollo Server landing page this way :
const landingPagePlugin = ApolloServerPluginLandingPageLocalDefault({
       embed: {
         initialState: {
           sharedHeaders: {
             foo: 'foo',
           },
         },
       },
    })

const apolloServer = new ApolloServer({
  schema,
  plugins: [landingPagePlugin]
})

And you should see your shared headers in the sandbox :

Ref : https://www.apollographql.com/docs/apollo-server/api/plugin/landing-pages/#initialstate-options

To clarify: those fields used to be user editable.

Same issue here. Text inputs are disabled in the DOM.
Only solution is to inspect them and remove the "disabled" attribute.

hi! You can actually pass your shared headers in Sandbox via initialState.sharedHeaders. I'll add this to the documentation, thanks for calling it out.
The disabled input was a bug that should be fixed now, can you confirm? Thanks!

hi! You can actually pass your shared headers in Sandbox via initialState.sharedHeaders. This is documented here https://www.apollographql.com/docs/graphos/explorer/sandbox/#sharedheaders
The disabled input was a bug that should be fixed now, can you confirm? Thanks!

I confirm that it's working again on my side. Thanks for your service!


RBGW/RBGWW light control

currently, an echo device only returns RGB colours with this - it is missing the white channel. It works fine directly between Alexa and the MagicHome controllers, but when I integrate with HA, I have no value to pass to white_value in flux_led.
/
Can you add that to the message properties? Should be no issue since Alexa is clearly sending it.

Can you give an example of what exactly Alexa is sending to the hub and is not returned in the output payload?

I am seeing if I can get that - but essentially, RGBW lights have an
extra channel/value for white. So if I have the lights on, the white is
not controlled so that colour is wrong - eg, asking the lights to turn
red leaves white at full brightness, so it's not correct. I imagine that
there is an extra value being sent, which would then be passed to
flux_led as white_value.
I have done a pull request and was going to look into it - but given I
don't know javascript, it might be a tall order! Do you have a
definition of the complete schema for what Alexa send out when
communicating to Philips Hue? I can probably identify point in the right
direction.
------ Original Message ------
From: "Teodor Ivanov"<EMAIL_ADDRESS>To: "datech/node-red-contrib-amazon-echo"
<EMAIL_ADDRESS>Cc: "andymarden"<EMAIL_ADDRESS>"Author"<EMAIL_ADDRESS>Sent: 23/04/2020 11:04:07
Subject: Re: [datech/node-red-contrib-amazon-echo] RBGW/RBGWW light
control (#120)

Can you give an example of what exactly Alexa is sending to the hub and
is not returned in the output payload?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/datech/node-red-contrib-amazon-echo/issues/120#issuecomment-618309034,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APEGM6S3JOMINFE3TQUZW4LROAHBPANCNFSM4MO4ZPXA.

I believe one of the other alexa node red packages deals with it - the
ones that need AWS. Maybe have a look there. I will see what I can find
out.
------ Original Message ------
From: "Andy Marden"<EMAIL_ADDRESS>To: "datech/node-red-contrib-amazon-echo"
<EMAIL_ADDRESS>"datech/node-red-contrib-amazon-echo"
<EMAIL_ADDRESS>Cc: "Author"<EMAIL_ADDRESS>Sent: 23/04/2020 11:19:40
Subject: Re[2]: [datech/node-red-contrib-amazon-echo] RBGW/RBGWW light
control (#120)

I am seeing if I can get that - but essentially, RGBW lights have an
extra channel/value for white. So if I have the lights on, the white is
not controlled so that colour is wrong - eg, asking the lights to turn
red leaves white at full brightness, so it's not correct. I imagine
that there is an extra value being sent, which would then be passed to
flux_led as white_value.
I have done a pull request and was going to look into it - but given I
don't know javascript, it might be a tall order! Do you have a
definition of the complete schema for what Alexa send out when
communicating to Philips Hue? I can probably identify point in the
right direction.
------ Original Message ------
From: "Teodor Ivanov"<EMAIL_ADDRESS>To: "datech/node-red-contrib-amazon-echo"
<EMAIL_ADDRESS>Cc: "andymarden"<EMAIL_ADDRESS>"Author"
<EMAIL_ADDRESS>Sent: 23/04/2020 11:04:07
Subject: Re: [datech/node-red-contrib-amazon-echo] RBGW/RBGWW light
control (#120)

Can you give an example of what exactly Alexa is sending to the hub
and is not returned in the output payload?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/datech/node-red-contrib-amazon-echo/issues/120#issuecomment-618309034,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APEGM6S3JOMINFE3TQUZW4LROAHBPANCNFSM4MO4ZPXA.

Here you can find the Philips Hue API, which is emulated by this module. https://developers.meethue.com/develop/hue-api/
You have two modes

"HS" for Hue and Saturation
"CT" for Color Temperature

Mixed mode  - RGB + White is not supported, but you can easily implement it as custom logic for your specific case.

Thanks - i tried various combinations but I think flux_led is broken for rgbw lights. I can get what I need, in theory, I think, by detecting hs or ct and setting various parameters. But i get strange behaviour with white-value and kelvin.
So I think you have done all you can - I'll see whether I can get something sensible worked out!
Cheers
Andy
On 23 Apr 2020, 12:42, at 12:42, Teodor Ivanov<EMAIL_ADDRESS>wrote:

Here you can find the Philips Hue API, which is emulated by this
module. https://developers.meethue.com/develop/hue-api/
You have two modes

"HS" for Hue and Saturation
"CT" for Color Temperature

Mixed mode  - RGB + White is not supported, but you can easily
implement it as custom logic for your specific case.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/datech/node-red-contrib-amazon-echo/issues/120#issuecomment-618353516


Use cell and feature filtering

We (I) have left out cell and feature filtering while creating the Seurat object. It's probably good to have some minimal filtering in order to e.g. remove genes for which no expression was measured. This also would solve the backward compatibility issue with trajectory analysis with monocle3.

fixed


Allow admins ability to read targets+images.

Turns out for review we need an interface for requesting imagery. Rather than duplicate it, give admins access.

LGTM


Use background thread to pin snapshots, rather than pin/unpin snapshot each time.

[x] add unpin_snapshot_before RPC: #2964
[x] use this rpc in frontend hummock snapshot manager. It only affects unpin. #2993
[ ] meta auto pin the epoch and push to all FEs (switched to push-based)

Will this solve issue #2356?

Will this solve issue #2356?

Yes

This issue may divide into some parts

[ ]: add unpin_snapshot_before RPC
[ ]: use this rpc in frontend hummock snapshot manager. It only affects unpin.
[ ]: meta auto pin the epoch and push to all FEs (switched to push-based)

for now, we have a new implementation of HummockSnapshotManager after https://github.com/singularity-data/risingwave/issues/3551.
Using background thread to pin snapshots seems not a good idea (it will cause consistency problem #3559).
To prevent pin/unpin snapshots each time, the next implementation is the 'push-base'.  So I think we can close this issue now?


[SPARK-49947][SQL][TESTS] Upgrade MsSql docker image version

What changes were proposed in this pull request?
The pr aims to upgrade the MsSql docker image version from 2022-CU14-ubuntu-22.04 to 2022-CU15-ubuntu-22.04.
Why are the changes needed?
This will help Apache Spark test the latest MsSql.
https://hub.docker.com/r/microsoft/mssql-server

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GA.
Was this patch authored or co-authored using generative AI tooling?
No.

@milastdbx Could you review this PR, please.

Thanks @MaxGekk @milastdbx

Merged to master.


Unable to remove getChildContext() from root component after update to 0.12.0

I get these warnings for almost all the material-ui components I use.
"Warning: owner-based and parent-based contexts differ (values: undefined vs [object Object]) for key (muiTheme) while mounting Toolbar (see: http://fb.me/react-context-by-parent)"
I don't have multiple reacts in node_modules or using multiples. I am not using something like react-router that uses context or did I think. I am only using material-ui components.
Is there an easy way to see what is causing this? I thought with v0.12.0 I would be able to eliminate the context in my root component.
Thanks for the help.

@AddictArts v0.12.0 internally uses context to pass down the default theme (or the custom theme in case you define one in your outermost app component). All material-ui components use context for this purpose.
Can you share your code that shows you the warnings so I can run it and investigate?

Yes absolutely. The project is on github at
https://github.com/SRI-SAVE/react-components
The root component is named EUi.jsx and I use a mixin to put the above code
into it. It is a webpack project. Thank you
On Sep 25, 2015 8:29 PM, "Shaurya Arora"<EMAIL_ADDRESS>wrote:

@AddictArts https://github.com/AddictArts v0.12.0 internally uses
context to pass down the default theme (or the custom theme in case you
define one in your outermost app component). All material-ui components use
context for this purpose.
Can you share your code that shows you the warnings so I can run it and
investigate?
—
Reply to this email directly or view it on GitHub
https://github.com/callemall/material-ui/issues/1728#issuecomment-143394392
.

@AddictArts I ran your project but couldn't see the warnings. Here's the workflow I followed:

Am I missing something?

Thanks for looking. My root component "EUI" had this
  childContextTypes: {
    muiTheme: React.PropTypes.object,
  },

  getChildContext() {
    return {
      muiTheme: ThemeManager.getMuiTheme(LightRawTheme),
    };
  },

So, when you tested it you didn't see the issue. My apologies for that. I found the answer though.
See https://medium.com/@skwee357/the-land-of-undocumented-react-js-the-context-99b3f931ff73
The issue is that my root component "EUI" and owner is not a material-ui component, so the owner and parent don't match. I can live with this and React 0.14 should resolve it with parent-parent checks instead of owner checks. Which in my case the parent would have the context. Thanks

:+1: glad you figured it out

Seems like this issue has been resolved.


Support non packed objects

Support non packed objects for v4 branch.
Enable to get objects from .git/objects/HASHDIR/HASHFILE in func (s *ObjectStorage) Get(h core.Hash) (core.Object, error).
Now not support Iterator in func (s *ObjectStorage) Iter(t core.ObjectType) (core.ObjectIter, error) , but we should support iterator for func (r *Repository) Commits() (*CommitIter, error).
I'm considering how to implement iterator.
#51

Current coverage is 67.42% (diff: 29.41%)

No coverage report found for v4 at 0fa637d.

Powered by Codecov. Last update 0fa637d...86eb298

https://github.com/src-d/go-git/pull/68/commits/70741202d6c50f285c529b54d347a66db2cb1e6b support iterator. but it has a problem.
Commits() iterator isn't sorted.

@alcortesm PTAL

Thanks for your contribution @sona-tar, There are some issues in you PR, I have leave you some comments about them:

minor style changes
bugs in your code (wrong regexps)
it is missing tests
and I think you have break the FsSuite.TestIterCompareWithMemoryStorage test.

It is ok if the commit iterator does not return commits sorted by Commiter.When.

Thanks for your review.

minor style changes
bugs in your code (wrong regexps)
it is missing tests
and I think you have broken the FsSuite.TestIterCompareWithMemoryStorage test.

I fixed it for https://github.com/src-d/go-git/pull/68/commits/25e4a41cdf8980cc8ed97c15880b3d5ae25ea75f and https://github.com/src-d/go-git/pull/68/commits/d346525b08eca8491b43a36601d19198ddd5dec4 .

Also, using regexp can be quite CPU expensive for such a simple task, we are not sure we can afford to waste so much CPU.
I didn't fix regexp problem.

It is difficult to ignore illegal object. like objects/gg/46525b08eca8491b43a36601d19198ddd5dec4.
If we need not ignore illegal object directory. It is only necessary to check object directory name size which is 2 character, and we require all object files in object directory.
- if f.IsDir() && objDirRegExp.MatchString(f.Name()) {
+ if f.IsDir() && len(f.Name()) == 2 {

- if objFileRegExp.MatchString(obj.Name()) {

Could you give me some advice?

Could you give me some advice?

Of course.
if f.IsDir() && len(f.Name) == 2 && isHex(f.Name) {
...
}

func isHex(s string) bool {
	for _, b := range []byte(s) {
		if isNum(b) {
			continue
		}
		if isHexAlpha(b) {
			continue
		}

		return false
	}

	return true
}

func isNum(b byte) bool {
	return b >= '0' && b <= '9'
}

func isHexAlpha(b byte) bool {
	return b >= 'a' && b <= 'f' || b >= 'A' && b <= 'F' // not sure if uppercase is allowed by git
}

This is at least 10 times faster than a regexp, according to my benchmarks.

Thanks for advice!!
I fixed it. 😄
Please let me know if there is a problem.

Should I rebase v4 and resolve conflicts?

https://github.com/src-d/go-git/pull/68#issuecomment-241699379:

Should I rebase v4 and resolve conflicts?

Yes please.

This PR is starting to look good, I've left your some more comments.

I rebased to v4.
Can I fixup this PR commits to one commit?
rebase is so hard...


Correctly handle unassigned counties

Summary
Correctly check if a county is (unassigned), unknown, or some other variation that indicates we don't know which county the case is in, don't error if it's passed.
Fixes #28
Seems to work fine for the us-wa branch:
  {
    county: '(unassigned)',
    cases: 151,
    deaths: 0,
    country: 'iso1:US',
    state: 'iso2:US-WA',
    location: 'iso1:us#iso2:us-wa#(unassigned)',
    date: '2020-03-19',
    source: 'us-wa',
    priority: 0
  },

@jzohrab how would I go about writing tests for this?

@ryanblock i think the location: 'iso1:us#iso2:us-wa#(unassigned)' string is less than ideal, maybe location: 'iso1:us#iso2:us-wa#unassigned' would look nicer.
Also, what do put for countyId when county: '(unassigned)'? @hyperknot?

It's tricky. In CDS we are skipping them now, as the summing happens inside the scrapers, there is no point in processing or exporting them.
But in li we'll calculate summing in a separate step, so we'll need this somehow. I guess (unassigned) is the best guess.


Merged groupby_agg and groupby_dict_agg to implement dictionary functions aggregations

What do these changes do?

 Let's wait other reviewers, if there are no objections we can merge the changes.


[#187] Initial set of encode/decode functions

 Here is an overview of what got changed by this pull request:

Clones added
============
- include/shad/extensions/data_types/data_types.h  8

See the complete overview on Codacy


MP cell surface receptor signaling pathway

GO:0007166 - cell surface receptor signaling pathway
should this have parent
GO:0035556 intracellular signal transduction

The prefix 'intra' means within, while the term for 'cell surface receptor signaling pathway (GO:0007166)' clearly refers to 'extracellular' so I do not think this makes sense.
cell surface receptor signaling pathway (GO:0007166)
Def: A series of molecular signals initiated by activation of a receptor on the surface of a cell. The pathway begins with binding of an extracellular ligand to a cell surface receptor, or for receptors that signal in the absence of a ligand, by ligand-withdrawal or the activity of a constitutively active receptor. The pathway ends with regulation of a downstream cellular process, e.g. transcription.
intracellular signal transduction (GO:0035556)
Def: The process in which a signal is passed on to downstream components within the cell, which become activated themselves to further propagate the signal and finally trigger a change in the function or state of the cell.

I'm a bit confused by this.
Even though a signalling pathway originates at a cell surface receptor it is still "intracellular signal transduction"  isn't it? If it wasn't what would it be called instead. That would make the
signalling steps initiated by the receptor  "intracellular signal transduction", even though it is activated by an extracellular ligand?
or are there signalling pathways which activate a cell surface receptor, and then transmit a signal between cells?
Does anyone else  know?
@Antonialock
@RLovering  your group does a lot of signalling annotation in multicellular?

Oh I think I get it.
So intracellular signal transduction (GO:0035556) is a term that can be used to annotate any part of a signalling pathway excluding receptor. If so that seems a bit  of a strange "grouping" term.
I always though this term was to distinguish from extracellular signalling, like circulating growth factors etc....

I guess cell surface receptor signaling pathway applies to receptors that span the plasma membrane? Not all receptors do...lipids (cholesterol), hormones (estrogen, cortisol) and small uncharged stuff (CO2, O2, NO) can diffuse freely can cross the PM without the needs for an extracellular receptor sensing them, .
I think** all "cell surface receptor signaling pathway (GO:0007166)" would have an "intracellular signal transduction" component, but not all intracellular signalling have an extracellular component.
** I don't know if there are exceptions....for example consider apoptosis...I guess some apoptosis might be due to extracellular cues to die (e.g. in development) but some apoptosis must be due to intracellular cues (ageing cells....build up of buggered mitochondria, intracellular ROS...)
Also  there might be odd stuff going on in the immune system mechanisms. How does directed chemotaxis work?

extracellular signaling sounds a bit weird...how about signals that are sent by one cell and then internalized by other cells? Wouldn't it make more sense to annotate e.g. growth factors, peptide hormones etc to the BP "cell-cell signaling" + the MF "Extracellular signal" (sort of similar to the MF "protein tag"?)

(ah, there is actually no term for "extracellular signal transduction")

So what I am trying to establish is, in yeast we would annotate everything signalling somewhere under "intracellular signal transduction" (because this is true, whether the signal originates inside or outside the cell). But we would not annotate any of the cell surface receptors to "intracellular signal transduction"  (according to the above).
However, because   when annotating individual pathways, we include the receptor (e.g. the git  GPCR  adenylate cyclase pathway includes the GPCR),  This means that effectively these intracellular pathways which originate from a GPCR cannot be placed under "intracellular signal transduction" because one component (the receptor) is on the cell surface. So somehow the logic seems flawed.....

I don't understand Val,
e.g. a GPCR has both an extracellular component (senses ligand) and an intracellular component (binds Gprotein)

Yes but it has an extracellular component. So you could not place the
http://www.ebi.ac.uk/QuickGO/GTerm?id=GO:0007186#term=ancchart
(for instance)
under intracellular signalling.
So, I'm trying to establish the purpose of the term "intracellular signalling"
if GO:0007166 - cell surface receptor signaling pathway cannot be part of it.
I think part of the problem is due to attempts to represent all of the functions as processes.
There are lots of them, like:
GO:0010578 | regulation of adenylate cyclase activity involved in G-protein coupled receptor signaling pathway
(which is another way of describing the GPCR)
I think this is an issue to discuss at the signalling meeting.

Hi @krchristie
I had the same issue in #13903. Finally I made the suggested changes.

Someone should claim one of these pathways for the signaling meeting.

Well we already claimed
adenylate cyclase-activating glucose-activated G-protein coupled receptor signaling pathway
which is  a
GO:0007166 - cell surface receptor signaling pathway
and a
GO:0035556 intracellular signal transduction
which is what causes us the problem....

that's this pathway
https://github.com/pombase/curation/issues/1582

I think I should undo the edits that I did on #13903. I think that the idea of intracellular signal transduction was introduced so that we could represent the conserved downstream modules of signaling pathways, whether those pathways were downstream (or part of) of cell surface receptor signaling pathways or not. That is why I was very reluctant to make the edits that you wanted in the other ticket. So for example in the pathway I have claimed (MAPK cascade), as currently represented (but not necessarily annotated) the cascade lies downstream of many different upstream bits of signaling that activate it.

Hi @ukemi
Can you explain what other types of ' GO:0007165 signal transduction' would exist in addition to 'intracellular signal transduction' ? Not 'extracellular signal transduction', I suppose. Looks like the other main class is 'cell surface receptor signaling pathway' ?
The siblings seem mostly intracellular signaling pathways, such as 'ER-nucleus signaling pathway'. Perhaps this branch needs some cleaning up ?
Thanks,
Pascale

Closing as "out of date"


Hitting Back on Login screen - then Login again stalls the app

In Android.
Steps:

Sign In
https://www.dropbox.com/s/3u3w85kjz7tckre/Screenshot 2015-09-24 16.43.11.png?dl=0

Hit HW Back Button.
Debugger catches a java.lang.reflect.InvocationTargetException
Not much to go beyond that.

On the side - is it possible to copy the username into the Clipboard?  I have a very hard time remembering this Login name.

@away168: Thanks for the report. We're actually going to implement a second button on the login page that completely bypasses the Azure AD login. That's something that Nat asked for too. Great minds think alike, right? ;)

:+1:


Yatin: Watch samcovina.com

Yatin requests the watch of the watch_keyword samcovina\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by cigien in Charcoal HQ

Approved by cigien in Charcoal HQ

samcovina\.com has been seen in 2 true positives, 0 false positives, and 0 NAAs.

samcovina\.com has been seen in 2 true positives, 0 false positives, and 0 NAAs.


Style/EmptyCaseCondition removes method call taking case as argument

I'm running corrections over some older code that hasn't been looked after by Rubocop before, and I've just had the Style/EmptyCaseCondition cop auto-correct a case statement that was being passed as an argument into a method call in such a way that the method call was removed.
Expected behavior
Change a unnecessary case being passed into a method:
  do_some_work case
               when object.nil?
                 Object.new
               else
                object
               end

Into an if statement, maintaining the call to do_some_work
Actual behavior
The call to do_some_work is unexpectedly remove.
Steps to reproduce the problem
Given this contrived eg.rb file:
def some_bad_code(object)
  do_some_work case
               when object.nil?
                 Object.new
               else
                object
               end
end

Run Rubocop corrections:
$ bundle exec rubocop --safe-auto-correct eg.rb
Inspecting 1 file
C

Offenses:

eg.rb:1:1: C: [Corrected] Style/FrozenStringLiteralComment: Missing magic comment # frozen_string_literal: true.
def some_bad_code(object)
^
eg.rb:2:16: C: [Corrected] Style/EmptyCaseCondition: Do not use empty case condition, instead use an if expression.
  do_some_work case
               ^^^^
eg.rb:4:1: C: [Corrected] Layout/IndentationWidth: Use 2 (not 15) spaces for indentation.
               if object.nil?
^^^^^^^^^^^^^^^
eg.rb:6:16: C: [Corrected] Layout/IndentationWidth: Use 2 (not 1) spaces for indentation.
                object
               ^

1 file inspected, 4 offenses detected, 4 offenses corrected

The eg.rb file is left in this state:
# frozen_string_literal: true

def some_bad_code(object)
  if object.nil?
    Object.new
  else
    object
  end
end

RuboCop version
$ bundle exec rubocop -V
0.60.0 (using Parser <IP_ADDRESS>, running on ruby 2.3.7 x86_64-darwin18)

Thanks for the feedback. This issue is a regression by #6257 💦 I opened a PR #6513. Thank you!


Kube-VIP does not work as DaemonSet with interfaces configured by altname property

Describe the bug
Maybe it is a bug, maybe it is an edge case. I am also interested in (different) solutions of other users.
Given are 3 nodes with different interface names, e.g. eth0, enp0s31f6 and eno1. Kube-VIP should be created via DaemonSet.
Potential solution: give all vip interfaces the same altname (e.g. kubevip) and use it during config creation.
To Reproduce
Steps to reproduce the behavior:
VIP_INTERFACE_ALT_NAME=kubevip

# on node1
VIP_INTERFACE=eth0
ip link property add dev $VIP_INTERFACE altname $VIP_INTERFACE_ALT_NAME

# on node2
VIP_INTERFACE=enp0s31f6
ip link property add dev $VIP_INTERFACE altname $VIP_INTERFACE_ALT_NAME

# on node3
VIP_INTERFACE=eno1
ip link property add dev $VIP_INTERFACE altname $VIP_INTERFACE_ALT_NAME

# get interface details (execute on all nodes)
ip addr show $VIP_INTERFACE_ALT_NAME

create kube-vip config (DaemonSet)
kube-vip manifest daemonset \
    --interface $VIP_INTERFACE_ALT_NAME \
    --address $VIP \
    ...

error message in kube-vip pod
time="2023-09-05T11:16:42Z" level=warning msg="failed to get interface \"kubevip\": route ip+net: no such network interface"
time="2023-09-05T11:16:45Z" level=warning msg="failed to get interface \"kubevip\": route ip+net: no such network interface"
time="2023-09-05T11:16:48Z" level=warning msg="failed to get interface \"kubevip\": route ip+net: no such network interface"

Expected behavior
Expect the IP to be provisioned on interface with altname kubevip
Environment (please complete the following information):

OS/Distro: Debian 12
Kubernetes Version: v1.27.4+k3s1
Kube-vip Version: 0.6.2

Kube-vip.yaml:
apiVersion: apps/v1
kind: DaemonSet
metadata:
  creationTimestamp: null
  labels:
    app.kubernetes.io/name: kube-vip-ds
    app.kubernetes.io/version: v0.6.2
  name: kube-vip-ds
  namespace: kube-system
spec:
  selector:
    matchLabels:
      app.kubernetes.io/name: kube-vip-ds
  template:
    metadata:
      creationTimestamp: null
      labels:
        app.kubernetes.io/name: kube-vip-ds
        app.kubernetes.io/version: v0.6.2
    spec:
      affinity:
        nodeAffinity:
          requiredDuringSchedulingIgnoredDuringExecution:
            nodeSelectorTerms:
            - matchExpressions:
              - key: node-role.kubernetes.io/master
                operator: Exists
            - matchExpressions:
              - key: node-role.kubernetes.io/control-plane
                operator: Exists
      containers:
      - args:
        - manager
        env:
        - name: vip_arp
          value: "true"
        - name: port
          value: "6443"
        - name: vip_interface
          value: kubevip
        - name: vip_cidr
          value: "32"
        - name: cp_enable
          value: "true"
        - name: cp_namespace
          value: kube-system
        - name: vip_ddns
          value: "false"
        - name: svc_enable
          value: "true"
        - name: vip_leaderelection
          value: "true"
        - name: vip_leasename
          value: plndr-cp-lock
        - name: vip_leaseduration
          value: "5"
        - name: vip_renewdeadline
          value: "3"
        - name: vip_retryperiod
          value: "1"
        - name: address
          value: <IP_ADDRESS>
        - name: prometheus_server
          value: :2112
        image: ghcr.io/kube-vip/kube-vip:v0.6.2
        imagePullPolicy: Always
        name: kube-vip
        resources: {}
        securityContext:
          capabilities:
            add:
            - NET_ADMIN
            - NET_RAW
      hostNetwork: true
      serviceAccountName: kube-vip
      tolerations:
      - effect: NoSchedule
        operator: Exists
      - effect: NoExecute
        operator: Exists
  updateStrategy: {}
status:
  currentNumberScheduled: 0
  desiredNumberScheduled: 0
  numberMisscheduled: 0
  numberReady: 0

I think this might be fixable by changing https://github.com/kube-vip/kube-vip/blob/063cc2fb8d76301d910e755fb2527761c038a193/pkg/vip/address.go#L71-L74 to first call netlink.LinkByAlias(iface) before returning an error.

Actually, discussion in https://github.com/vishvananda/netlink/pull/862 seems to indicate that that would not work out after all :(

@sdwilsh seems to be merged this year, maybe you do a follow up on this issue?

I'm pretty sure I ended up going a different route than depending on altname since it wasn't clear when this would be resolved.  I agree that it might be much easier now that the upstream library works in a way that would better support this, however!


Black blink before page rendering

Link to PDF file (or attach file here):
pdf.pdf
Configuration:

Web browser and its version: android webview (chromium)
Operating system and its version: Android (all versions)
PDF.js version: 1.5.222
Is an extension:
HTML https://github.com/jamesvanhallen/TryXwalker/blob/master/app/src/main/assets/index.html
JS https://github.com/jamesvanhallen/TryXwalker/blob/master/app/src/main/assets/pageviewer.js

Steps to reproduce the problem:

follow youtube link

What is the expected behavior? (add screenshot)
No blink
What went wrong? (add screenshot)
https://www.youtube.com/watch?v=dXfIoJ_96tQ
Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

fixed! the problem was in incorrect css