[WIP]: Multi-repo support (MVP)

This mega PR is tracking the branch on which stuff is happening. Yes, multi-repo is now number 1 priority for me on Buildasaur. Will take a couple of days (also because I'm building it fully with ReactiveCocoa, which I'm just learning), but is on its way.
Feel free to subscribe to this PR and answer my questions about building apps with AppKit, which is just a PITA for me as an iOS developer.

Question 1: #139.

Result of Integration 1
Duration: 3 minutes and 9 seconds
Result: All 19 tests passed, but please fix 5 warnings.
Test Coverage: 20%.

Result of Integration 2
Duration: 2 minutes and 32 seconds
Result: All 19 tests passed, but please fix 5 warnings.
Test Coverage: 20%.

Result of Integration 3
Duration: 2 minutes and 46 seconds
Result: 1 error, failing state: build-errors

Result of Integration 4
Duration: 35 seconds
Result: -1 error, failing state: trigger-error

Result of Integration 5
Duration: 2 minutes and 20 seconds
Result: All 19 tests passed, but please fix 5 warnings.
Test Coverage: 20%.

Result of Integration 6
Duration: 8 hours, 37 minutes and 2 seconds
Result: -1 error, failing state: trigger-error

Result of Integration 7
Duration: 2 minutes and 47 seconds
Result: All 19 tests passed, but please fix 4 warnings.
Test Coverage: 20%.

Result of Integration 8
Duration: 2 minutes and 25 seconds
Result: All 19 tests passed, but please fix 4 warnings.
Test Coverage: 21%.

Result of Integration 9
Duration: 2 minutes and 38 seconds
Result: All 19 tests passed, but please fix 4 warnings.
Test Coverage: 21%.

Result of Integration 10
Duration: 2 minutes and 16 seconds
Result: All 19 tests passed, but please fix 4 warnings.
Test Coverage: 21%.

Result of Integration 11
Duration: 2 minutes and 41 seconds
Result: All 19 tests passed, but please fix 4 warnings.
Test Coverage: 21%.

Result of Integration 12
Duration: 14 seconds
Result: -1 error, failing state: trigger-error

Result of Integration 13
Duration: 16 seconds
Result: -1 error, failing state: trigger-error

Result of Integration 14
Duration: 15 seconds
Result: -1 error, failing state: trigger-error

Result of Integration 15
Duration: 15 seconds
Result: -1 error, failing state: trigger-error

Result of Integration 1
Duration: 23 seconds
Result: -1 error, failing state: trigger-error

Result of Integration 2
Duration: 2 minutes and 32 seconds
Result: All 19 tests passed, but please fix 4 warnings.
Test Coverage: 21%.

Result of Integration 3
Duration: 5 minutes and 46 seconds
Result: 1 error, failing state: build-errors

Result of Integration 4
Duration: 3 minutes and 6 seconds
Result: 3 errors, failing state: build-errors

Result of Integration 5
Duration: 3 minutes and 36 seconds
Result: 1 error, failing state: build-errors

Result of Integration 6
Duration: 2 minutes and 21 seconds
Result: All 19 tests passed, but please fix 4 warnings.
Test Coverage: 21%.

Result of Integration 3
Duration: 2 minutes and 57 seconds
Result: 1 error, failing state: build-errors

Result of Integration 1
Duration: 3 minutes and 46 seconds
Result: 1 error, failing state: build-errors

Result of Integration 3
Duration: 3 minutes and 10 seconds
Result: Build failed 2 tests out of 21

Result of Integration 3
Duration: 3 minutes and 10 seconds
Result: Build failed 2 tests out of 21

Result of Integration 9
Duration: 7 minutes and 19 seconds
Result: All 23 tests passed, but please fix 2 warnings.
Test Coverage: 21%.

Result of Integration 9
Duration: 7 minutes and 19 seconds
Result: All 23 tests passed, but please fix 2 warnings.
Test Coverage: 21%.

Result of Integration 9
Duration: 7 minutes and 19 seconds
Result: All 23 tests passed, but please fix 2 warnings.
Test Coverage: 21%.

Result of Integration 9
Duration: 7 minutes and 19 seconds
Result: All 23 tests passed, but please fix 2 warnings.
Test Coverage: 21%.

Result of Integration 11
Duration: 2 minutes and 51 seconds
Result: All 23 tests passed, but please fix 2 warnings.
Test Coverage: 21%.

Result of Integration 11
Duration: 2 minutes and 51 seconds
Result: All 23 tests passed, but please fix 2 warnings.
Test Coverage: 21%.

Result of Integration 11
Duration: 2 minutes and 51 seconds
Result: All 23 tests passed, but please fix 2 warnings.
Test Coverage: 21%.

Only took two weeks. :sweat: :laughing: :rocket:


Data access

Hi,
Is there any timeline when this dataset will be available again? Thanks.

Any updates? Thanks in advance.

@Yoo-Hak any updates?


Converting file

Can this be used for converting a .md file to a valid .html file?
I only saw this converting a md strings

Well, yes. You can use the cli tool for that.
In this example showdown is installed globally but you can achieve the same thing with a local installation.
##Windows
echo ^^^^ > otp.html
showdown makehtml -i "path/to/source/file.dm" >> otp.html
echo ^^ >> otp.html

unix
echo "" > otp.html
showdown makehtml -i "path/to/source/file.dm" >> otp.html
echo "" >> otp.html

@tivie I'm using nodejs, so maybe this is not the cleanest way to do it but it works! 👍
thanks 😄


path.join universal front - backslash not implemented

Node's version of path.join converts the front or backslash appropriately depending on the operating system. For example, windows uses a frontslash for its directories. And linux and mac use a backslash for their directories.
The issue is when coding on Windows, you must use a frontslash. But when deploying on Mac, it must be converted to a backslash. This presents a portability issue.
Node's "join" actually handles this quite simply using conditionals to determine which operating system the user is on. But this version does not handle the front/backslash problem. Is it possible to implement Node's version of "join" in this module?

Examples:
path.join("a\b\c") on Windows gives "a/b/c"
path.join("a/b/c") on Linux gives "a\b\c"


Bundle the fonts instead of loading from google fonts (blenderskool#17)

WRT #17

I am really sorry about that, I didn't notice my defaults on prettier. Fixing the same

Fixed :)

Fixes #17


[Question] Expected shape of one element arrays

Question
I am wondering if np.array([1]) should be a valid element of Discrete(2) action space. It currently isn't because shape is expected as (), not (1,): https://github.com/Farama-Foundation/Gymnasium/blob/dcb150e8893161d941f8a3fff1baef232cad4d96/gymnasium/spaces/discrete.py#L95
How do people experienced in gym codebase reason about this? :)

Interesting question, personally, I wouldn't have an issue with modifying the contain function but I think users should require their space to Box(low=discrete_space.start, high=discrete_space.start + discrete_space.n, shape=(1,), dtype=np.int64) which is equivalent to what you want

I think it makes sense as it is, i.e. [1] isn't a valid member of Discrete(2). A discrete state is meant to be a lone integer and not an array. Shape-related errors are common enough as it is, so this extra piece of rigidity can help prevent them.


[SuperEditor][Mobile] Fix downstream image caret position (Resolves #1959)

[SuperEditor][Mobile] Fix downstream image caret position. Resolves #1959
Tapping close to the trailing edge of an image places the selection correctly, but the caret isn't displayed. We had the same issue in the past on desktop. It was fixed by https://github.com/superlistapp/super_editor/pull/1820, but mobile doesn't use the same caret layer that desktop uses, so the issue is still present on mobile.
The root cause is the same of the desktop issue:
The downstream caret is positioned at (right, 0), so when the image takes all the available screen width (like in the demo app), the caret is positioned off-screen. Block level elements that have padding don't have this issue, because there is room to render the caret.
The same solution was applied:
This PR changes the caret overlay to check whether the caret rect is placed outside of its bounds at the right side. If it is, the caret position is adjusted.
The golden  tests for Android are still failing due to this exception:
══╡ EXCEPTION CAUGHT BY FLUTTER TEST FRAMEWORK ╞════════════════════════════════════════════════════
The following assertion was thrown running a test:
'package:flutter_test/src/_matchers_io.dart': Failed assertion: line 29 pos 10:
'!renderObject.debugNeedsPaint': is not true.

When the exception was thrown, this was the stack:
#2      captureImage (package:flutter_test/src/_matchers_io.dart:29:10)
#3      MatchesGoldenFile.matchAsync (package:flutter_test/src/_matchers_io.dart:96:21)
#4      _expect (package:matcher/src/expect/expect.dart:109:26)
#5      expectLater (package:matcher/src/expect/expect.dart:73:5)
#6      expectLater (package:flutter_test/src/widget_tester.dart:511:25)
#7      compareWithGolden (package:golden_toolkit/src/testing_tools.dart:296:9)

It seems to be caused by the Follower used by the Android collapsed handle.

@matthew-carroll Can you do a first pass review?
I still don't know what to do with this assertion failure, but it should impact the fix.

@matthew-carroll We had the golden test issue on the past on linux and worked around that with the following code:
 @override
  void markNeedsPaint() {
    super.markNeedsPaint();

    if (kDebugMode &&
        !kIsWeb &&
        Platform.isLinux &&
        Platform.environment.containsKey('FLUTTER_TEST') &&
        !WidgetsBinding.instance.hasScheduledFrame) {
      // We are running on a linux test and we don't have a scheduled frame.
      //
      // We ran into an issue in some golden tests, on linux only, where `debugNeedsPaint`
      // is `true` after the build/layout/phase pipeline. This causes the tests to fail,
      // because trying to capture the image of a `RenderObject` throws an assertion failure
      // if `debugNeedsPaint` is `true`. To avoid that, immediately schedule a new frame,
      // so Flutter pipeline runs again before we try to capture the image.
      WidgetsBinding.instance.scheduleFrame();
    }
  }

Should we also check for Android?

@angelosilvestre I think we mentioned it on our call, but you can go ahead and let that particular check run for all platforms.

CC @BazinC

@matthew-carroll I'm getting a failure on the new goldens on CI only that seems related to some antialiasing on the drag handle.
Test image:

Master image:

Isolate diff:

Any ideas?

@angelosilvestre can you just regenerate that golden?

@matthew-carroll Locally, the regenerated golden is identical to the previous one...

@angelosilvestre if your local Flutter version is identical to the CI Flutter version, then I'm not sure why we're seeing different results. If that's truly the case, then consider using the tooling that we created, which allows a certain number of pixels to be different before failing the test.

After using our tooling that tolerates small differences, I got back to the infamous debugNeedsPaint assertion failure.

@angelosilvestre are the goldens the only remaining problem? If so, let's file an issue to look into it and get this fix merged.

@matthew-carroll Yes, the only problem is the failing goldens on Android. I modified these tests to be skipped so we can try to find out the reason.

@angelosilvestre please be sure to file an issue, if you haven't already


user_id in table

Hi,
In my user_details table, the id is one to one relation with auth.users.id.
when i try to run the test
test('can create table with no dependencies', async ({ supawright }) => {
    await supawright.create('user_details')
  })

i get Error: Error inserting data into user_details: insert or update on table "user_details" violates foreign key constraint "user_details_id_fkey"
How do i fix this?

Hey @rohankm. Can I see the table schema for user_details?

Hey @rohankm. Can I see the table schema for user_details?

Hi,
here is the schema
create table
  public.user_details (
    id uuid not null,
    created_at timestamp without time zone not null default now(),
    updated_at timestamp without time zone not null default now(),
    email text not null,
    photo_uri text null default ''::text,
    name text null default ''::text,
    constraint user_details_pkey primary key (id),
    constraint user_details_id_fkey foreign key (id) references auth.users (id) on delete cascade
  ) tablespace pg_default;

basically, the id of the table is automatically filled from auth.uid() function

Ahhhh if it has a default of auth.uid(), Supawright can't handle it because it uses the service account, which has a auth.uid() of null.
Take a look at #4 for a solution :)

Thanks createUser works
btw does the trigger not work while creating user?
i have set up a trigger when a user sign ups it automatically creates a record in user_details

Does it require any special fields in the metadata etc.? The trigger should still work if it's set up correctly - we use triggers on auth.users inserts too, and it works fine.

no, it just takes the user ID and email and creates a row in user_details.
When I manually sign up the trigger works
I tried to select and it returned no rows
{ data: [] }

I can't see any reason it wouldn't work. Is the auth.users record being created successfully?
The createUser function just calls the Supabase sign up function under the hood:
  public async createUser(attributes?: AdminUserAttributes): Promise<User> {
    const { data, error } = await this.supabase().auth.admin.createUser({
      email: faker.internet.email(),
      password: faker.internet.password(),
      ...attributes
    })
    if (error) {
      log.error('Error creating user', { error, attributes })
      throw new Error('Error creating user: ' + error.message)
    }
    this.record({
      schema: 'auth',
      table: 'users',
      data: data.user as unknown as Select<Database, 'auth', 'users'>
    })
    return data.user
  }

Can I see the code for your trigger?

Closing due to inactivity


TrustManager issue

I'm having an issue running a JUnit test which includes a WireMockRule.
The endpoint is HTTP but I'm getting an error which seems to suggest WireMock wants to use HTTPS:
Caused by: java.security.NoSuchAlgorithmException: class configured for TrustManagerFactory: sun.security.ssl.TrustManagerFactoryImpl$PKIXFactory not a TrustManagerFactory
at sun.security.jca.GetInstance.checkSuperClass(GetInstance.java:258)
at sun.security.jca.GetInstance.getInstance(GetInstance.java:237)
at sun.security.jca.GetInstance.getInstance(GetInstance.java:164)
at javax.net.ssl.TrustManagerFactory.getInstance(TrustManagerFactory.java:138)
at org.apache.http.conn.ssl.SSLContextBuilder.loadTrustMaterial(SSLContextBuilder.java:101)
at com.github.tomakehurst.wiremock.http.HttpClientFactory.buildAllowAnythingSSLContext(HttpClientFactory.java:84)
at com.github.tomakehurst.wiremock.http.HttpClientFactory.createClient(HttpClientFactory.java:63)
at com.github.tomakehurst.wiremock.http.ProxyResponseRenderer.(ProxyResponseRenderer.java:59)
at com.github.tomakehurst.wiremock.WireMockServer.(WireMockServer.java:103)
Googling suggests that adding @PowerMockIgnore("javax.net.ssl.*") at the top of the class should resolve this but this hasn't made any difference for me.
What could I be missing?
Robert

Good chance another dependency in your project is messing with the Java TLS config. No idea what though if it's not Powermock. Suggest disabling other libraries one by one until the problem stops.

Thanks Tom.  I resolved this issue.  It was because I had the ignore annotation on a steps class rather than on the Cucumber test runner class.
Robert


Improve performance hit caused by conversion and resampling on iOS

A loopback test app takes considerable CPU time on iOS (iPhone 5s) due to performance hit caused by audio sample conversion (F32LE to S16LE) and resampling (44,100 to 48,000 for Opus, 44,100 to 8,000 for PCM). Xcode profiler confirms these elements to be the offenders.
I've narrowed it down by transmitting only audio, disabling DTLS and using the PCMA codec with 8000 clock-rate. One would think loopback of 8KHz PCM would be an easy task for modern hardware.

My initial analysis:
On iOS, the audiosrc's src pad offers fixed capabilities of:
audio/x-raw
format: F32LE
layout: interleaved rate: 44100
channels: 2
channel-mask: 0x0000000000000003

From a cursory look at osxaudio (gst-plugins-good/sys/osxaudio/gstosxaudiosrc.c, function gst_osx_audio_src_probe_caps), it returns fixed caps based on the input stream's current format (i.e. querying kAudioUnitProperty_StreamFormat). This is unlike, for example, the ALSA source element which provides a wide range of possible formats.

We're using Remote IO, and Apple says we cannot expect it to convert or resample for us. Therefore, is it the Right Thing for osxaudio source not to offer flexible caps?

If so, what kind of performance hit is expected from a 44,100 F32LE to 8,000 S16LE conversion? What we're seeing now seems way high to me, but I'm inexperienced with this.

In latest git master of gst-plugins-bad, it can output many more formats than F32LE. It will IIRC output whatever the audio session is configured to.
So for 1), yes that's correct. For 2), what are the numbers? How expensive is it?

@sdroege CPU usage outlined in this thread: https://groups.google.com/forum/#!topic/openwebrtc/bZJfu5CQ33U

btw, owr_local_media_source_request_source doesn't use the caps for OWR_MEDIA_TYPE_AUDIO at all. Is it supposed to be like this, i.e. are the caps somehow propagated back to the source as "preferred" caps? (I'm a gstreamer newbie, so I really don't know.)

It appears that osxaudio retrieves zero sampling rate from AU (at least on Remote IO), and so sampling rate ends up being GST_AUDIO_DEF_RATE (= 44100).
Following IRC discussion:

@ford-prefect will implement real probing ("preferred_caps") for Remote IO, based on kAudioSessionProperty_CurrentHardwareSampleRate etc.

It should be documented that OS X and iOS developers should set up AVAudioSession in their app code.

owr should warn when conversion and/or resampling would occur on OS X and iOS, and recommend setting up AVAudioSession.

@ford-prefect I took care of this -- can you review https://bugzilla.gnome.org/show_bug.cgi?id=743758 ?
This reduces iPhone 5s CPU usage to 14% (PCMA) and 35% (Opus 1.0), which is pretty good :)

Sweet! I'll try to get a review of that done today. A quick look suggests
we're still working with F32LE? I guess the next step would be to probe
that too.
On 31 January 2015 at 07:52, Ilya Konstantantinov<EMAIL_ADDRESS>wrote:

@ford-prefect https://github.com/ford-prefect I took care of this --
can you review https://bugzilla.gnome.org/show_bug.cgi?id=743758 ?
This reduces iPhone 5s CPU usage to 14% (PCMA) and 35% (Opus 1.0), which
is pretty good :)
—
Reply to this email directly or view it on GitHub
https://github.com/EricssonResearch/openwebrtc/issues/144#issuecomment-72300271
.

Fantastic!

I'm available for any help on further optimization for iOS audio. Done a iOS 6.x and 7.x port of WebRTC in the past and have a working setup for an earlier version of WebRTC, so I might be able to provide some pointers.

This fix should eliminate the other case of resampling (before osxaudiosink) that happens sometimes (dependent on race):
https://bugzilla.gnome.org/show_bug.cgi?id=743925

Just as a quick status update, with current gst-plugins-base/good master, we no longer do any resampling on a Bowser <-> FF call. Next step is to look at the rest of @ikonst's caps probing work for the next set of improvements.

Excellent!

@ikonst any news on https://bugzilla.gnome.org/show_bug.cgi?id=743758 ?

We had the Purim holiday this weekend which really took all my attention.
:) I hope to work on it today.
On Mar 8, 2015 1:58 PM, "Robert Swain"<EMAIL_ADDRESS>wrote:

@ikonst https://github.com/ikonst any news on
https://bugzilla.gnome.org/show_bug.cgi?id=743758 ?
—
Reply to this email directly or view it on GitHub
https://github.com/EricssonResearch/openwebrtc/issues/144#issuecomment-77745115
.

Just to note that these has been progress in https://bugzilla.gnome.org/show_bug.cgi?id=743758 - great work guys! :smile:


Leverage mypy to check str.format expressions

from loguru import logger

class Foo:
bar = "baz"

foo = Foo()

logger.debug('The bar is "{my_foo.bar}"', my_foo=foo)
logger.debug('The bar is "{my_foo.bar}"', my_foo="not foo")  # error: "str" has no attribute "bar"

Closes #42

@kornicameister How do you track which versions of mypy are compatible? Since I am tapping into mypy's internals I am worried it will break with upcoming versions, so I think we should have a way of making that visible.

@ThibaultLemaire thx for pull request. It looks really nice, I'd like to hear more about that visit_expr you came up with and how did you that. mypy is magical and that's a magic I am seeing :D.
Anyway,I really like your job on error messages although I wonder if having them more accurate wasn't better. Could you elaborate why did you change those?
PS. sorry for being so lazy about reviewing, I had my evenings filed with either micro-management on 2 children or overtime from work :/

Well, I probably don't understand half of the magic of mypy, but the result looks great! Thanks for your work. :)

Hey, looks like it's my turn to apologize for the delay, last week was crazy at work.
Thank you Delgan, I have been focusing precisely on keeping the magic on mypy's side with this PR and not having to understand it :wink: Hopefully we can make that work in satisfying way

@ThibaultLemaire  would you mind checking #31 ? Would you change invalid that PR?

Hm, I can imagine why my work would conflict this PR, but I'm not sure how exactly the two would interact.
Also, what exactly do you mean by "invalid[ate]" ? If you mean make obsolete, I don't think so: my changes have no way of knowing if record=True has been passed, but if you mean conflict in a way that would require to update #31, I'm afraid so, yes.
I only glanced at it though, so I'll take the time to investigate properly.

Okay, so, after further investigation, I reckon you could adapt your work on #31 to make it work side by side, but then you'd have a bit of an overlap.
To take your example code from #18, and compare the (theoretical) results of #31 and #43 (this PR):
logger.info('{record["msg"]}') # loguru -> ❌, #31 -> ❌, #43 -> ❌
logger.opt(record=True).info('{record["msg"]}') # loguru -> ✔, #31 -> ✔, #43 -> ❌

As you can see, there would be duplicate error messages, as well as contradicting messages where my work has shortcomings and can't understand that record is a valid keyword argument.
The issue is that I rely entirely on mypy's internals and have no control over them, so I couldn't silence that error even if I wanted to.
What we could do, though, -- and, to me, that would be the best solution -- would be to pass the record argument to mypy to validate it as well. In the same spirit to what I've done here, this would mean making mypy validate
logger.opt(record=True).info('{record["msg"]}')

as if it were something like
logger.info('{record["msg"]}'.format(record={}))

Unfortunately, I don't know how much of your work we could reuse if we go down that path.

Also related to that, I have noticed that you store the loggers the plugin has previously encountered in the self._known_loggers dict, to deal with .opt() configuration, but this is discouraged by the mypy documentation (emphasis mine):

Plugin developers should ensure that their plugins work well in incremental and daemon modes. In particular, plugins should not hold global state due to caching of plugin hook results.

I wonder if we could rather play on the type we return from the plugin to store that configuration, which should save us a lot of trouble in the future as well.
To illustrate, I was wondering if we could make things work a bit like this:
from loguru import logger  # type is inferred by mypy to be Logger

with_record: LoggerWithRecord = logger.opt(record=True)
# Those would be custom types that we return, in order to know what to do next time we encounter it
lazy: LazyLogger = logger.opt(lazy=True)
with_ip: BoundLogger = logger.bind(ip="<IP_ADDRESS>")
with_ip_in_record: BoundLoggerWithRecord = with_ip.opt(record=True)

# And here mypy should pass us all the info we need to validate it properly
with_ip_in_record.info("Your local ip is {record["ip"]}")

@ThibaultLemaire for the global state, I was not aware of that. If could drop the issue for that matter, I'd appreciate. Just put a link to your comment.
As for the #31 I think I'd prefer to have mypy deal with string formatting if possible. I imagine them doing far better job.
This is rather interesting glitch, I certainly need time to even get my head around. I mean, don't get it wrong, your idea is far nicer compared to mine but this record thingy makes it hard to just accept.
As you fairly pointed out it simply won't work now.
Unless, we can get our hands on record=True and somehow push into kwarg arguments of formatting call a type containing all fields from a record loguru defines. Anyway, record=True is not a point of interest on this PR. It is for #18 to resolve.
I will play around with your solution and I believe in the end of the week, unless something pops out, we can merge that.
@Delgan do you have any interesting cases we could use to challenge this PR? If you do, please put a comment :)

I tried hard but could not generate a error specific to the _check_str_format_call function. Congrats for mastering mypy internals!
However,  some tests led me to open #49. :p

@thx @Delgan , I will try to look at this if possible..or rather if I find time...it is horrible how little of that I have :D

@ThibaultLemaire I am merging this. Thanks a lot for patience and waiting for me here :)


Added a onLogin callback

This PR adds an onLogin that is needed to add Nextjs preview mode.
See changset for implementation.

Probably best to demo this for office hours but looks good to me! Just a leftover console.log to remove

Removed them


Issue: MAGMA variable batch LU solver example

This is PR holds an example of trying to exercise MAGMA's variable batch LU factorization and solver routines.
At the moment I have the factorization working; however since there is no direct companion solver routine I am trying to use the magmablas_dtrsm_vbatched routines to form a solver. Unfortunately I am not sure how to call the function properly. I'm hoping if someone from the MAGMA team has some time the could take a look at main.cpp.

Thanks @nbeams! and to confirm, we have to apply the pivoting ourselves to the RHS prior to making the solve call?

Correct.


Display nav_roll and nav_pitch in telemetry

Hi
I need to see navigation roll and pitch angle for tuning PID during flight but I can not find it. I think this parameters not send with Mavlink.

We need more information to help you.

What angle exactly do you need to see?
What ground station software are you using?
What PX4 version are you using?
What telemetry link are you using?

I need to see navigation angle of roll and pitch . i use QGroundcontrol and latest px4 v1.9.2 . for telemetry i use 3dr telemetry or RFD900 .

So you mean you need to see the setpoint of the roll and pitch angle?

I suspect the request is to be able to see both the roll/pitch setpoints and also actual roll/pitch values and display them live (using the same units and scale) in the QGC Analyze Widget for purposes of tuning.  I recall being able to do this with older versions of PX4, but with more recent releases of PX4 I too have been unable to figure out if both of these values (setpoints and actual) are still able to be displayed by the widget.

Ok, got it. I wonder if QGC has the functionality to request custom messages at a certain rates.
What you could try is to add an extras.txt as described here with this line:
mavlink stream -r 50 -s ATTITUDE_TARGET -u /dev/ttyS1

I'm not sure if it is /dev/ttyS1, this depends on the hardware that you're using and wiring setup.

currently we can see roll and pitch angle in QGC Analyze Widget but they are not in degree.

currently we can see roll and pitch angle only in QGC Analyze Widget but they are not in degree.

Are they in radians?

yes , they are radians.

What about the field roll sp on the screenshot above? Isn't that what you want together with ATTITUDE.roll? It's both in rad but at least both have the same unit.

"roll sp" is roll rate . i noticed during flight roll set point appears (in degree). so just roll angle change to degree .

"roll sp" is roll rate .

Really? I don't think so:
https://github.com/mavlink/qgroundcontrol/blob/4b6163ecfd35aeefa45391dcc477122177bea50a/src/uas/UAS.cc#L301

i was wrong . it is roll setpoint ( rad)


grantPermission for Android

Need a method for granting push notifications from within the app on Android.
My app has a feature/service that requires Push to be enabled.  If the user manually turns off app permissions for push notifications I need a way of asking/granting push notifications again from within the app.  At best, it seems all I can do is test to see if it is enabled and if not alert the user that they need to go into the phone settings -> notifications -> "app" and enable it that way.  Its def not ideal.

We really need the same thing. We need to be tactful with our ask on android, and right now the prompt just pops up whenever the user logs in for the first time.

please confirm whether this issue persists with v1.1.3

@soumak77 This is not related the the v1.1.0 update.  This is a limitation on the Android platform.
@rolinger @drewctate If you can submit the code that implements said functionality, we can definitely review the Pull Request.  I think the only way to achieve this is to open up the settings page as I don't believe Android allows apps to prompt users who explicitly block them.


two word components get renamed as hyphen separated

When running npx acme pull command two name components such as "contentfragment" get renamed as a hyphen separated name like "content-fragment".

Also check policies:
    aemStyleSystem: {
      policy: '/policies/content-fragment.json'
    }
  }```

Policy file name: `/policies/contentfragment.json`

The problem in detail.
acme pulls the policies exactly as they are named in AEM.
When creating the stories, in the part that references the component's policy, acme uses the name of story as the policy name (which comes from the name of the page in AEM). In most cases they match:
button.stories.js and /policies/button.json
But sometimes they don't--especially when the component/page is two words:
experience-fragment.stores.js and /policies/experiencefragment.json
When generating the stories it would be better to get the actual name of the policy as it is saved in the policies directory and use that.


plugin grpc Unary

You need to update docs of supported list accordingly.

And it seems you don't add your new test in the CI files, therefore, your cases haven't been verified by CI.


Inconsistencies in the spec

In CreateSession, 400 error response the following example is provided.
                UePortsRequired:
                  summary: uePorts is required
                  value:
                    code: INVALID_INPUT
                    message: "Expected property is missing: uePorts"

This meant to say that uePorts is a required param. However, the CreateSession definition mentiones uePorts as optional param.
Which one is correct?
Same way in the below example, it is mentioned port ranges are not allowed for uePorts. However, uePorts is of type PortSpec and there is no restriction on specifying ranges.
                UePortsRangesNotAllowed:
                  summary: Ranges at uePorts are not allowed
                  value:
                    code: INVALID_INPUT
                    message: "Ranges not allowed: uePorts"

If these examples are not upto date, could we please remove them?

@SfnUser yes, some of the error response examples are not consistent and make no sense within v0.8.0. We had some other examples during the review of the documentation (cf for example https://github.com/camaraproject/QualityOnDemand/pull/71#discussion_r1038167486). Luckily they are only examples and need not to be implemented. And yes, we should be more consistent here in the upcoming version.
If I'm not mistaken the two error responses you mentioned are both related to the initial thought that the uePort is needed if a public IPv4 address should to be resolved to identify an UE. In this case the uePort is needed in addition to the IPv4 address and it can't be a range. My current hope is that @eric-murray plans to clean up the error message examples in his planned PR for #34. It should then also be clearer in the spec what is expected from the API caller.
Beyond that: if an operator does not implement a certain parameter or option of the spec, the 501 code "Not Implement" might be more appropriate. As we want to use this error code I will open up a new issue to introduce it into the spec.

Closed as discussed in call on Jan 27th.


64bit arguments passing with 68k .library ABI

We're trying to provide some support for AmiSSL in Free Pascal Compiler. This involves creating our own headers for AmiSSL, to talk to the ABI directly, based on the C includes. However, when it comes to the Motorola 68k/AmigaOS3 version, it looks like some of the C inline macros only specify a single register (say, d0) to pass uint64_t or int64_t parameters to the function. One simple example is OPENSSL_init_ssl(), where the flags parameter is uint64_t, yet if I check the inline macros, I only see d0 mentioned. This obviously cannot work properly, however, in the amissl_lib.sfd, clearly (d0-d1) as register pair is specified to be used, but I can't see any trace of this in the inline macros.
So, what's the deal here? The .sfd file is ignored, the inline macros are wrong, did we find an issue, or it works regardless through some C compiler magic I we are not aware of? Thanks.

@chainq Compiler magic :-)
Basically, for GCC/OS3, it is implicit that the register specified is the first of the pair if the type is 64-bit. I had to double-check this by disassembling as I couldn't remember, but I can see that GCC is correctly passing the high bits in d0 and the low bits in d1, for OPENSSL_init_ssl(). If you look at the LP2() macro itself, it puts the uinit64_t macro parameter into a variable of that type in register d0 (and d1 implicitly).  I don't think GCC has a way to explicitly specify d0-d1, as you would do in VBCC for example.

@Futaura thanks for the answers. Things are more clear now I think. We're going to add support to specify register pairs for 64bit values in Amiga-style .library calls in Free Pascal, so the AmiSSL headers can be completed.
Actually, meanwhile I learned, that one needs this for some of the double precision math libraries as well. We didn't support this, because Free Pascal provides its own soft-float facilities instead for FPU-less systems, so supporting the old and fairly limited math libs wasn't a priority. So yeah, that could then work too. Closing the ticket, thanks.


Tappable for Label

👋  Hello
Can you please add Tappable to  /TABTestKit/Classes/Elements/Label.swift?

Heya @bartzet  - I'll endeavour to take a look into this or possibly @roger-tan or @annapiktas. In the mean time do you have any updates?

That should be possible if Label is conform to the protocol with Tappable.  Have you got chance to get this working by chance?
When I have some free time, I could have a look to make conform to Tappable and update our example test against that.

That should be possible if Label is conform to the protocol with Tappable.

That's correct. It needs to be implemented on the framework level though.
When I applied the change to the framework's code I managed to make it work but it would be great if the framework could support it.
Thanks

@bartzet That's not really how Apple intended you to use labels.
try setting accessibilityTraits to .button
And use TABKit's Button()
More information: https://developer.apple.com/documentation/uikit/uiaccessibility/uiaccessibilitytraits

That's fine. This is what I've been doing so far. Thank you for your comments, appreciate it.


App shortcut/activity to launch dashboard

I don't see anything apart from WelcomeActivity in Tasker. This requires a further click on Launch Dashboard to open the dashboard. Can we have an activity to LaunchDashboard?

WelcomeAxtivity has logic to open the dashboard after automatically if you’ve already been to the settings.  It’s the same as if you clicked on the wall panel app icon.  So if you’ve already gone through the settings, any subsequent launches of the application will redirect you to the dashboard automatically.   The welcome page is only shown on intial install and launch but you need to first visit the settings page before the logic will take you to the dashboard.


Make pluginId field uniform

There is some confusions about the pluginId field. Indeed for out coming event the field name is pluginId but for incoming event it is plugin_id.
Uniform name would be better (it leads to huge bug confusion if you miss this difference).

We've now brought these API's to be internal-only, so closing for now.  Thanks!


Seeing some warnings on 811 fresh install jetty startup.

Describe the bug
There are a lot of warning messages in Jetty log on 811 fresh install startup.
To Reproduce
Steps to reproduce the behavior:

Install 811
Start Jetty and you will see following warnings:- Please see attached server.log for full log.

_09:00:05,744 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1001,ContentType:1015}, because ContentType is not found.
09:00:05,783 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1003,ContentType:1016}, because ContentType is not found.
09:00:05,784 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1003,ContentType:1023}, because ContentType is not found.
09:00:05,784 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1003,ContentType:1022}, because ContentType is not found.
09:00:05,785 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1003,ContentType:1021}, because ContentType is not found.
09:00:05,786 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1003,ContentType:1017}, because ContentType is not found.
09:00:05,786 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1003,ContentType:1020}, because ContentType is not found.
09:00:05,787 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1003,ContentType:1019}, because ContentType is not found.
09:00:05,787 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1003,ContentType:1018}, because ContentType is not found.
09:00:05,789 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1003,ContentType:1017}, because ContentType is not found.
09:00:05,790 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1003,ContentType:1016}, because ContentType is not found.
09:00:05,790 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1003,ContentType:1023}, because ContentType is not found.
09:00:05,790 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1003,ContentType:1022}, because ContentType is not found.
09:00:05,791 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1003,ContentType:1020}, because ContentType is not found.
09:00:05,791 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1003,ContentType:1019}, because ContentType is not found.
09:00:05,792 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1003,ContentType:1021}, because ContentType is not found.
09:00:05,792 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1003,ContentType:1018}, because ContentType is not found.
09:00:05,853 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1007,ContentType:1024}, because ContentType is not found.
09:00:05,854 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1007,ContentType:1018}, because ContentType is not found.
09:00:05,854 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1007,ContentType:1025}, because ContentType is not found.
09:00:05,854 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1007,ContentType:1024}, because ContentType is not found.
09:00:05,855 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1007,ContentType:1018}, because ContentType is not found.
09:00:05,855 WARN  [PSSlotDefDependencyHandler] Removing Slot <==>Template/ContentType relationship{Slot:0-5-1007,ContentType:1025}, because ContentType is not found.
09:00:06,030 WARN  [PSTemplateDependencyHandler] The template specified by 1,068 was not found C:com.percussion.services.assembly.impl.PSAssemblyService:L:1453
09:00:06,031 WARN  [PSTemplateDependencyHandler] The template specified by 1,067 was not found C:com.percussion.services.assembly.impl.PSAssemblyService:L:1453
09:00:06,031 WARN  [PSTemplateDependencyHandler] The template specified by 1,062 was not found C:com.percussion.services.assembly.impl.PSAssemblyService:L:1453
09:00:06,032 WARN  [PSTemplateDependencyHandler] The template specified by 1,066 was not found C:com.percussion.services.assembly.impl.PSAssemblyService:L:1453_

Expected behavior
Server should startup clean
server.log

These warnings are coming because we are not installing FastForward objects as part of 8.1.1 .Thus, it's OK. They show up first time we start server after install. Restarting the server doesn't show these warnings.


3.14 compatibility. Update patches from pierstitus, zapashcanon and polter-rnd.

I've rebased patches from #102 to the actual master branch. Also compatibility with gnome-shell 3.14 was added for support of the CentOS 7.2

I don't think it is appropriate to merge this into master, but in its own branch, it's fine. There is a branch for backport3.14 where it would be more apropriate.
Also please note that the gone extension website rejected the submissions I made for 3.14 so this won't end up on extension.gnome.org .

Also please note that the gone extension website rejected the submissions I made for 3.14 so this won't end up on extension.gnome.org .

What about f#ckin CentOS support, gnome team? Rhetorical question

Also please note that the gone extension website rejected the submissions I made for 3.14 so this won't end up on extension.gnome.org .

Which review was this? From what I can see, your last review request was from November 16 and was accepted.


[WIP] Check bump dcos-test-utils in 1.11.

High-level description
Investigating https://github.com/dcos/dcos-test-utils/pull/70 and  https://github.com/dcos/dcos/pull/3843 and https://github.com/mesosphere/dcos-enterprise/pull/3905
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

https://jira.mesosphere.com/browse/DCOS-45738

Related tickets (optional)
Other tickets related to this change:

DCOS_OSS- Foo the Bar so it stops Bazzing.

Checklist for all PRs

[ ] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change:
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[ ] Read the DC/OS contributing guidelines
[ ] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and review process
What is the review process and when will my changes land?
All PRs require 2 approvals using GitHub's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

We are now up to date so I am closing this, thanks @ArmandGrillet !


deployment failed while deploying to kuberntes

error gotten when i tried to deploy a service fails at deployment to Kubernetes through Jenkins CI/CD:
Error: Get https://46FD186CCDF8FB42C11F1F7A6A72B7F8.sk1.eu-west-1.eks.amazonaws.com/api/v1/namespaces/kube-system/pods?labelSelector=app%3Dhelm%2Cname%3Dtiller: dial tcp: lookup 46FD186CCDF8FB42C11F1F7A6A72B7F8.sk1.eu-west-1.eks.amazonaws.com on <IP_ADDRESS>:53: no such host
[Pipeline] }
[Pipeline] // node
[Pipeline] End of Pipeline
GitHub has been notified of this commit’s build result
ERROR: script returned exit code 1
Finished: FAILURE
Output of helm ### version:
Client: &version.Version{SemVer:"v2.13.1", GitCommit:"618447cbf203d147601b4b9bd7f8c37a5d39fbb4", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.13.0", GitCommit:"79d07943b03aea2b76c12644b4b54733bc5958d6", GitTreeState:"clean"}
Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.12", GitCommit:"c757b93cf034d49af3a3b8ecee3b9639a7a11df7", GitTreeState:"clean", BuildDate:"2018-12-19T11:16:52Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.12", GitCommit:"c757b93cf034d49af3a3b8ecee3b9639a7a11df7", GitTreeState:"clean", BuildDate:"2018-12-19T11:04:29Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform (AKS, GKE, Minikube etc.): AWS/kubernetes

@eumoh1601 This seems to imply that there is a connection issue to the cluster. Is there a way you can troubleshoot the connection to the cluster? Can you run the deployment manually? Can you run any kubectl commands against the cluster?
Trying to narrow down where the issue might be.

I'm going to close this ticket due to inactivity, but please re-open if this still needs to be addressed. Thanks!


always backing off

i have no seman API

The SS API should be 1 call limit per second without a key, are you running this in parallel?

You mean even i do not have ss api i should export S2_API_KEY=""  ?

That shouldn't matter, the default value will likely be fine. The only thing that matters is whether you are trying to call this more than 1/s.

No. I didn’t.
Cong Lu @.***>于2024年8月29日 周四下午4:15写道：

That shouldn't matter, the default value will likely be fine. The only
thing that matters is whether you are trying to call this more than 1/s.
—
Reply to this email directly, view it on GitHub
https://github.com/SakanaAI/AI-Scientist/issues/78#issuecomment-2316794844,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARQNVA423CXKGMDK6X4LXJTZT24BVAVCNFSM6AAAAABNHOPT22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJWG44TIOBUGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

I am experiencing the same issue. I just ran my template; I didn't specify anything regarding parallel running or the number of API calls. It is stuck for hours in this process.
Anytahing to change in terms of configuration to account for this?

Can you check if you are able to make an API request at all? https://www.semanticscholar.org/product/api%2Ftutorial#request
It may be a firewall issue.

Same issue boring me......

Same issue boring me......

Do you have a firewall that blocks this?

I opened all port
Cong Lu @.***>于2024年8月30日 周五下午4:54写道：

Same issue boring me......
Do you have a firewall that blocks this?
—
Reply to this email directly, view it on GitHub
https://github.com/SakanaAI/AI-Scientist/issues/78#issuecomment-2320258516,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARQNVA6RP23XPGFJ4HC6QPTZUAJJLAVCNFSM6AAAAABNHOPT22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRQGI2TQNJRGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

I opened all port Cong Lu @.>于2024年8月30日 周五下午4:54写道：
…
Same issue boring me...... Do you have a firewall that blocks this? — Reply to this email directly, view it on GitHub <#78 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARQNVA6RP23XPGFJ4HC6QPTZUAJJLAVCNFSM6AAAAABNHOPT22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRQGI2TQNJRGY . You are receiving this because you authored the thread.Message ID: @.>

Have you tried the tutorial here:  https://www.semanticscholar.org/product/api%2Ftutorial#request

I opened all port Cong Lu @.>于2024年8月30日 周五下午4:54写道：
…
Same issue boring me...... Do you have a firewall that blocks this? — Reply to this email directly, view it on GitHub <#78 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARQNVA6RP23XPGFJ4HC6QPTZUAJJLAVCNFSM6AAAAABNHOPT22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRQGI2TQNJRGY . You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
Have you tried the tutorial here: https://www.semanticscholar.org/product/api%2Ftutorial#request

Yes. I did.

Hmm interesting, this looks like an issue on the Semantic Scholar side, or other users on your network using SS. I would comment out uses of this API while you still don’t have a key and try get one!

thank you conglu, let me know when you fix this!


Propper short integer checks

The range for short integers is -32768 to 32767 instead of -32767 to 32767, isn't it? The frame.py file checks for the second (wrong?) range.

Hey @SophieAltenburg,
We have implemented this by following the C32.118.2 standard. Also please check out this article.


upgrade eureka to 1.1.152

https://github.com/Netflix/eureka/releases/tag/v1.1.152
Will require changes because json is no longer rendered using xstream.

https://github.com/Netflix/eureka/releases/tag/1.1.153
Add's DI support!

Hi Spencer,
Have you already started working on this?
It would be nice to upgrade at least up to 1.1.156 (and possibly even more recent versions) to benefit from other bug fixes that could provide a better user experience. I'm thinking about:

https://github.com/Netflix/eureka/issues/539
Not a big issue but it introduces extra communication between the client and the server which makes hard to understand what is happening.

https://github.com/Netflix/eureka/issues/533
Again, not a show stopper. But with this fix, we can now rely on a public method to feed the UI and report accurate status.

https://github.com/Netflix/eureka/commit/0db1f58de2681cbddee84e6cac7e432c34aefe8c
They introduced a way to trigger the InstanceInfo replication immediately without having to wait for the next "InstanceInfoReplicator" run. This could help answering questions like https://github.com/spring-cloud/spring-cloud-netflix/issues/377

In general, they seem to have refactored their code in a way that could make the integration into SpringBoot a bit more easier. Logging has been improved as well. Not perfect though, but apparently slightly better...

Hmmm... was a bit too fast: YES you already started working on the issue ;-)
So, when do you plan to release a new version with these changes?

I'm working on refactoring using their new refactor for dependency injection, that might be a while, but upgrading will probably be pretty simple in and of itself

actually I forgot the change to use jackson for json will require some work, so not quite as simple.  Upgrading and refactor to use DI should be separate issues.

https://github.com/Netflix/eureka/releases/tag/v1.1.157 has @brenuart's CacheRefreshedEvent and some other DI improvements.  This will make the change from xstream to jackson less painful because we won't have to try and shoehorn custom events in.


added Troubleshooting.md and some extra documentation

Maybe a bit wordy in some parts. Might also need a spell check;

Running a test to make sure that submitting through a sbatch job actually works. Other than that I think I'm done with the updates to this PR.

Test complete - seems that submitting Nextflow in a sbatch job works just fine.


fix: fix adding large keys truncates

fixes #84
Appears to be an issue with darwin interactive mode of /usr/bin/security, where the key is limited in size. Using non-interactive and invoking command directly functions correctly.

This way we revert a previous change intended to prevent exposing passwords in the system process list see: https://github.com/zalando/go-keyring/pull/65
I think supporting such large keys should only be done if it can also be done in a way which doesn't expose the passwords.

thanks @mikkeloscar for pointing that out. I can close this as its not required, I also found out that windows is limited to 512 byte values, so to keep platform functionality similiar I see no reason to support larger values anymore.


Move log timestamp to Foundation Date

Summary
This pull request is aimed to the issue #129 for using Foundation.Date for log timestamps
Design decisions
I've add a private time formatter shared instance for keeping the initialisation cost low, and set the format template to print hours, minutes, seconds and milliseconds.
I don't know if you prefer additional information on the date or the time is enough.

Current coverage is 82.92% (diff: 100%)

Merging #184 into master will decrease coverage by <.01%

@@             master       #184   diff @@
==========================================
  Files           179        179
  Lines         13257      13255     -2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits          10994      10992     -2
  Misses         2263       2263
  Partials          0          0

Powered by Codecov. Last update c5425a9...d500a2d

Nice job, thanks! However, I think it is more appropriate to set date format like this:
formatter.dateFormat = DateFormatter.dateFormat(fromTemplate: "HH:mm:ss.SSS", options: 0, locale: .current)

This way, the user's locale will be respected. For example, for me it prints time like this:

13:52:19,075

(with comma after 19, not the dot), which is right in my region (Ukraine).

Oh, ok, I've done that at first but then I've gone to do a fixed format.
The dateFormatter property is outside Logger for having it shared across all Logger instances and not create a new one every time a Logger object is created.
I will push the changes when I will be back home.

While I'm at it, you think I have to change the HH part of the template with jj to adapt dinamically to 12/24 hour clock?
Or at least is what I can understand from this documentation even if I don't know if you have to add the a to add AM/PM...

No problem, for the mont and day I'm sure it can do if you use the enum specifiers, I don't know if it will do the same with the localised template...

Done, I've even removed the separator from the template because they aren't needed, foundation is figuring out everything by itself.
And yes, the order of year, month and day are based on the locale 👍🏻

Awesome! Just one more thing - could you write a unit test that ensures the formatter spits out the right string? I believe what you say about the unnecessary separators but I'm worried that it might be different on Linux for whatever reason, etc.

The tests are no problem, but it possible to cheat a bit? 😁
Because the actual lotTimeFormatter is a fileprivate property I can't access directly inside the tests and in addition to that I have to set a fixed sets of locale and time zone to know how the strings would come out and so reset the dateFormat to it.
My idea is to tests on a brand new DateFormatter with a couple of different locale and time zone and two or three different dates to see if the format string without any separator is always equals to something we will know.
Is it ok for you?

If you use @testable import you should be able to access the fileprivate property. If this does not work, maybe make the formatter public? That could provide other benefits (user customization) anyway.
I agree that you should use several constant dates with locales and just plug them into the formatter.

@testable importallow only internal properties to be accessible. I've moved the logTimeFormatter property to be public so people can changes all the properties as they like.
The tests will check against the en_US and it_IT locales, I have to add another one?

I'm start to thinking that on Linux the unicode library is parsing the date skeleton template in a totally different way than on macOS.
It seems to completely ignore the forced 24 hour marker and for the Italian locale the two platform don't agree on the coma separator between date and hour...

Ok, I've spin up an ubuntu 12.04 server and I've found a super weird bug. On Linux if you add anyone of the time zone specifier to the skeleton format will disable the forced 24h format, as soon as you change the skeleton from yyMMdd HHmmssSSSz to yyMMdd HHmmssSSS the printed hours will revert correctly to the force 24h format.

Interesting... so what you're saying is that removing the z will fix it? Can you try committing that change? If it fixes CI, we can happily merge :)

Oh, so removing Z removes the time zone? That's a shame... @paulofaria @robertjpayne what do you think is the right thing to do here?

@JGiola could you please raise a bug at bug.swift.org? I think the formatting itself is secondary in regards to correctness. Timezones should be respected above all. If this means different formatting, I'm fine with that.

Yeah, I've filed it when I've been able to check what was going on on a Ubuntu server.
If you want to join in and write it down better is here https://bugs.swift.org/browse/SR-2906

In regard of the pull request what you want me to do? Going back on adding the time zone to the skeleton format and comment out the tests for now? Or leave things as they are now?


gw mute command not working properly (possibly)

typed .gw start mute TenSura s2 ep8 did not mute the voice channel with the format of .gw start mute episode title
This suggestion was created by a user and sent by the bot:
jukebox-botto#8602

this problem is on discord's side because it only mutes people who newly join not people who are already are in the voice channel.


Add .snyk file to ignore examples directory

Can't ignore all vulnerabilties with a wild card on the examples directory. @dobegor Feel free to reopen if you feel its worth it to add an ignore rule for each K8s vulnerability to the .snyk file.
Might be an easier piece of work to fix the issues in the manifests :D


sboms.tar being created with extra empty file

Describe what should be investigated or refactored
The sboms.tar file is being created with extra empty file when doing a zarf package create. Can be recreated by
Links to any relevant code
https://github.com/defenseunicorns/zarf/blob/193314b69b9c7eb1a62100c31c82a45bb4725aeb/src/pkg/utils/io.go#L461
(Potential fix) In the CreateReproducibleTarballFromDir function a check can be made if the dirPath and filePath are equal to not write the empty header.
Additional context
It was mentioned that there are other parts of the code base where empty/unnecessary files are being written to zarf packages.

We should also address the empty zarf-<number> folders as well during this.


Wait for proxy to propagated to cluster

Since we are not disabling the CVO so in case of proxy, CVO
take more time to propagate it to different operators, at this
moment we are checking operator status in every 2 sec (60 times)
till all are available.

retest this please


Eloquent doesn't check deleted_at column in join clauses (inner join, outer join etc.)

Laravel Version: 7.7.1
PHP Version: 7.4.3
Database Driver & Version:

Description:
When I use an inner join clause on a eloquent model It retrieves all columns include the rows that their deleted_at is NOT NULL.
Assume that we have 2 model State and City and these tow have one to many relation, when I run State::query()->join('cities', 'cities.state_id','states.id')->where('id',1)->get() query, It It retrieves state 1 with its cities included deleted_at=2020-0x-xx xx:xx:xx cities.
Steps To Reproduce:
We try to develop a package to solve this problem by checking deleted_at rows in join clauses:
https://github.com/BlackPlatinum/laravel-softdelete-fixer

You're not referencing a model here. You're using a raw join. So the query builder can't apply any soft deleting behavior because it can't know that the table is a model with the SoftDeletes trait applied.

So if I use pure eloquent it will check for deleted_at rows in join clauses automatically?


make docker-build command not found

when I follow instructions I get:
make docker-build docker-push IMG=/perses-operator:tag
make: *** No rule to make target 'docker-build'.  Stop.

moving ahead doing make deploy I get:
containerStatuses:

containerID: cri-o://8e65aff7b0cf237c2598df12acecded18c1432a536b94adaf466c6344d1e18b4
image: gcr.io/kubebuilder/kube-rbac-proxy:v0.13.1
imageID: gcr.io/kubebuilder/kube-rbac-proxy@sha256:18f12fc3a0085bbc303cfbd2ad1151127c1c959c4641e36027547b6b66eb4c92
lastState: {}
name: kube-rbac-proxy
ready: true
restartCount: 0
started: true
state:
running:
startedAt: "2024-12-19T10:32:00Z"
image: docker.io/persesdev/perses-operator:v0.1.0
imageID: ""
lastState: {}
name: manager
ready: false
restartCount: 0
started: false
state:
waiting:
message: |
container create failed: time="2024-12-19T10:38:52Z" level=error msg="runc create failed: unable to start container process: exec: "/manager": stat /manager: no such file or directory"
reason: CreateContainerError
hostIP: <IP_ADDRESS>

tested with crc locally

container create failed: time="2024-12-19T10:38:52Z" level=error msg="runc create failed: unable to start container process: exec: "/manager": stat /manager: no such file or directory"

mmm actually that's because the binary in the perses-operator is /bin/operator. Should we change the binary and the place in the docker image @jgbernalp. Or should we just update the file https://github.com/perses/perses-operator/blob/main/config/manager/kustomization.yaml ?

Adjusted in #41


Return type for Redis#exists is wrong

Right now, the return type is marked as 0 | 1 when it should in fact simply be number:

Relevant Redis docs: https://redis.io/commands/exists

Correct, my bad. I'll roll out a fix tonight.
Thanks for reporting it.


Update sponsorship listing

Update Instabug sponsorship listing
• Modify content
• Modify picture
• Modify URL

@vsouza Hi Vinicius, Thanks for updating Instabug's listing. I was about to get in touch with you regarding that.
Thanks, 🙏

@azamzamy Can we merge?

@vsouza Yes, please.

Thanks @azamzamy


Bump to version 1.10.0 for Release

Codecov Report
Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 67.37%. Comparing base (5d1d82a) to head (71c4a57).

Files with missing lines
Patch %
Lines

dbt_common/__about__.py
0.00%
1 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #198   +/-   ##
=======================================
  Coverage   67.37%   67.37%
=======================================
  Files          52       52
  Lines        3372     3372
=======================================
  Hits         2272     2272
  Misses       1100     1100

Flag
Coverage Δ

unit
67.37% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fixes for protonation of protein and binding site detection

Description
Small fixes for T018 pipeline issues. Namely:

commands in obabel helper function are run in the wrong order and result in docking to a structure that has no hydrogens
issue with number of positional arguments when running binding site detection class.

Status

[ ] Ready to go

@corey-taylor, did the results change? If so, we need to rerun the notebook, right?

Hi @corey-taylor, thanks a lot for fixing this - I reran the CI a couple of times, seems to be URL timeouts here and there. Let's merge this and check if the website will be rendered alright (do not quite understand the Docs fail).

Yep, agreed with this. If the site renders fine then we'll just put it down to gremlins in the machine or something.

@corey-taylor, did the results change? If so, we need to rerun the notebook, right?

The results changed for me, yes. I didn't actually run the input ligand in the notebook, I ran 3W32's co-crystallised ligand but you'd imagine they'd be quite different. I'll re-run it and commit.

@dominiquesydow Oh wait a sec, didn't the notebook work better when you or @AndreaVolkamer ran it last time, given I use Linux?

@corey-taylor, on second thought: in the notebook, we are testing for the resulting compounds from projects 1 and 2:
project1.OptimizedLigands.output
# NBVAL_CHECK_OUTPUT
---
[<Ligand CID: 11292933>]

and
project2.OptimizedLigands.output
# NBVAL_CHECK_OUTPUT
---
[<Ligand CID: 28693>]

With your changes, we are still getting the same compounds (we know that because the CI is not failing except for the Python 3.9 setup). Does this make sense to you? To be honest, I would have expected to see changes - but hey, happy about stable results :)

With that said, running the notebook under 3.9 indeed reproduces the CI error also locally. I won't be able to look into this tonight but will try to pick this up tomorrow.

@corey-taylor, on second thought: in the notebook, we are testing for the resulting compounds from projects 1 and 2:
project1.OptimizedLigands.output
# NBVAL_CHECK_OUTPUT
---
[<Ligand CID: 11292933>]

and
project2.OptimizedLigands.output
# NBVAL_CHECK_OUTPUT
---
[<Ligand CID: 28693>]

With your changes, we are still getting the same compounds (we know that because the CI is not failing except for the Python 3.9 setup). Does this make sense to you? To be honest, I would have expected to see changes - but hey, happy about stable results :)

Ordinarily I'd be quite worried about getting consistent results when changing something as major as the protonation of the entire protein. Certainly the ranking of compounds changed for me, although this is obviously a very small library and a fairly apolar pocket compared to other kinases so maybe that's why.

With that said, running the notebook under 3.9 indeed reproduces the CI error also locally. I won't be able to look into this tonight but will try to pick this up tomorrow.

For your info, nothing special about my setup. I run 3.9.7 in BunsenLabs Linux (used theTeachOpenCADD yaml). I certainly don't see the AttributeErrors so I'm not sure why the Ubuntu version is being tripped.

@jaimergp could you please take a look at why conda is failing for the Python 3.7 unit tests?
E.g. https://github.com/volkamerlab/teachopencadd/runs/5716919708?check_suite_focus=true

Merging this PR - T018 runs successfully under Py3.7 and 3.9 after workaround as outlined in PR description.
Remark on CI:

Docs CI still failing, addressed in https://github.com/volkamerlab/teachopencadd/pull/217
Unit tests CI still fails for T018, I think with a connection error (in the SI section), most probably not an issue on our end:  ValueError: Fetching results from DoGSiteScorer API failed.
  Expected key id not found in the response.
  The response message is as follows: {'message': 'Loading PDB file'}

Unit tests CI for MacOS: Full conda install failed (at least the imports seem to not exist) - probably rerun will resolve this. If not, open new issue and PR to fix this.


add llvm-cov coverage

Fixes #107

Codecov Report

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

@@           Coverage Diff           @@
##             main     #136   +/-   ##
=======================================
  Coverage        ?   86.69%
=======================================
  Files           ?       19
  Lines           ?     5373
  Branches        ?        0
=======================================
  Hits            ?     4658
  Misses          ?      715
  Partials        ?        0

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[Security sprint] Update sensitive GUIDs

Remove senstive GUIDs and replaces with dummy values

Changes addressed. #sign-off


self in setget

I have the feeling that using self. is kind of expensive but I like using self. because it gives me clear code. Anyway, I wanted to use setget for some experimentation and I found that I am unable to use self. in these functions giving an error such as Parser Error: Must use 'vel_max' instead of 'self.vel_max' in setter.. Why is that? And can this be "fixed"? I say "fixed" cause I see this is a GDScript compilation error so I guess this is the intended behavior but it doesn't make sense to have self. available everywhere except in this one place. I think it's a usability flaw. I know it's minor thing :D

If you use self.variable = value, it triggers the setter of the variable,
so if you use it inside the setter it'll be an infinite loop

I think someone optimized the use of self actually, so it's as fast as not
using it
On Sun, Aug 28, 2016 at 7:48 PM, punto-<EMAIL_ADDRESS>wrote:

If you use self.variable = value, it triggers the setter of the variable,
so if you use it inside the setter it'll be an infinite loop
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/6313#issuecomment-243004400,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z20G9afCa_641RGtuCmSTPzMZVnALks5qkhAvgaJpZM4Ju9n-
.

@punto- well it does make sense and I know that Python also uses a hidden variable when working with @property. But I still feel like it's inconsistent. Why does it trigger an infinite loop when using self., but it doesn't without? I mean, even without self. you're still accessing and setting the same variable are you not?
@reduz that's great :). Yay improvements!

Because there needs to be a way to be able to use the variable from the
setter without triggering an infinite loop, this is as good as any :) The
setter actually sets the value of the variable, so most of the time it
needs to manipulate it, especially if there's no getter.
On 29 August 2016 at 06:50, Răzvan Cosmin Rădulescu <
<EMAIL_ADDRESS>wrote:

@punto- https://github.com/punto- well it does make sense and I know
that Python also uses a hidden variable when working with @property. But
I still feel like it's inconsistent. Why does it trigger an infinite loop
when using self., but it doesn't without? I mean, even without self.
you're still accessing and setting the same variable are you not?
@reduz https://github.com/reduz that's great :). Yay improvements!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/6313#issuecomment-243031599,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGVmPeIVaWCBByLbLaQu6njbRv6BQZ-Dks5qkmUggaJpZM4Ju9n-
.

I understand that, except I don't understand why adding self. would trigger the loop while not adding it will not, since in both cases you're still refering to the same class variable, that's what I don't get. With or without self. it's the same variable, unless... in the background the engine makes a hidden variable that isn't accessible from GDScript directly or something on these lines. Now that I look at the docs there are two things:
var x = 'something' setget set_x, get_x
and then there's
var x setget set_x, get_x
Alright, I was using the first version and I just couldn't get why you have to use x inside of set_x for example, without self., but now I see that it isn't mandatory to use it, you can very well use a different variable such as self.__x for example which you would treat as being "hidden". I propose to make this work as in Python and only have var x setget set_x, get_x allowed and use some other variable inside of set_x & get_x, not x itself. Or maybe it's just me and I'm the only one confused about it :)...

I'm closing this since there does not seem to be a clear consensus as to whether this is a bug.


sb show: Add support for certificate chains and variable RSA key sizes

The sb show command only accepted sb2 files with a single certificate with an RSA 2048bit key. This PR adds basic supports for files using a chain of certificates and other RSA key sizes.

Could you rebase on main and force-push this again please?

For testing I used the sb2 for Nitrokey firmware. It might be interesting to generate sb2 files specifically for testing.

BTW if you have spare energy, it would be great to have a SignedSb21File::read<P: AsRef<Path>>(path: P) -> Result<Vec<u8>> method which did the loading, parsing, and importantly checked the signature that's squirreled into the sho method. This way, instead of publishing firmware with SHA256 hashes, we could use the native integrity check.


Probleme with providing room code

this is a probleme im facing ...
thnx in advance

if you already in a game (since it isn't detecting you) try spam changing your in-game color until it does detect your game

Sorry bro but it didnt work ... Ps: im running a cracked version of the game and did not buy it from steam .. so any other solution ?

We do not support cracked, read the readme


differentiate between syntax errors and style errors

Could you underline proper syntax errors in say red and style errors in a different colour like yellow?
That would be massively helpful as the red underline pops up for example when you declare a parameter and before you use it.

This is something you can control in your .eslintrc. For every check you can define whether it will be an error or a warning. We render warnings with a green underline but are considering to change this to yellow.


Webpage blocked by major search engines and security software.

I could not find any other way to contact you regarding this.
Your website looks like it has a serious issue. Google and Microsoft prevent access to the page from browsers. Even when I tell Google to go there anyway my anti-virus flips out.
I was not sure you were aware of this but someone might have buried something in your webpage which needs to be addressed.

Hey, thank you for taking the time to inform me of this. I kinda left some Advertisement code on the webpage way back, and it seems to have gone bad lately. I also got popups and stuff when I visited just now =/
I have removed the advertisement code now, so it should all work well.


Adding validation decorators to plain DTO and Entity

Currently, validation decorators are only added to CreateDTO and UpdateDTO, the types of DTOs which are used to send data to the API, unlike Entity and plain DTO, which are returned by the API and therefore normally do not need validation decorators.
Scenarios
Usage of Query() together with a DTO
I use the plain DTO to send data to the API, but since it lacks validation decorators the necessary type transformations are not applied, forcing me to transform the values manually.
The plain DTO is used to type the query parameters of my GET endpoints. This allows consumers of the API to filter results using a combination of any of the DTO's fields. This is done by mapping the plain DTO to a QueryDTO using Nest's PartialType() utility function, which creates a copy of the plain DTO with all fields now optional.
// query-profile.dto.ts
import { PartialType } from '@nestjs/swagger';
import { ProfileDto } from '../generated';

export class QueryProfileDto extends PartialType(ProfileDto) {}

// profile.controller.ts
@Get()
get(@Query() query: QueryProfileDto) {
   return this.profileService.get(query);
}

Since QueryDTO is now based on the plain DTO it always reflects the table and the API consumer can apply whatever filter combination they want.
Other possible scenarios
Nest's mapping utility functions can also be used to create more fine-grained DTOs or Entities for specific scenarios, for instance by using PickType() to trim the DTO down to what is needed.
For this to work well though, the base DTOs/Entities that are used together with these utility functions need to have class validators.
I propose an additional configuration flag that, if set to true, causes these validators to be added to both the plain DTO and Entity.
Problem when using Query() together with a DTO
Transformation using validation decorators only works when using Query(), if enableImplicitConversion enabled:
// main.ts
app.useGlobalPipes(
   new ValidationPipe({
      transform: true,
      transformOptions: { enableImplicitConversion: true }
   })
);

This is not optimal, as there are downsides to using enableImplicitConversion.
Solution
Explicitly adding a transform decorator to the property does the trick:
// profile.dto.ts
import { ApiProperty } from '@nestjs/swagger';
import { IsInt, IsOptional } from 'class-validator';
import { Transform } from 'class-transformer';

@ApiProperty({
   type: 'integer',
   format: 'int32',
   required: false,
   nullable: true
})
@IsOptional()
@IsInt()
@Transform((params) => Number(params.value))
age?: number | null;

Note
Instead of adding these decorators to plain DTO and Entity, a new DTO type could be introduced, which includes all fields the plain DTO possesses, but also adds the required validation and transform decorators.

Interesting use case. I think a separate QueryDTO would be the cleaner solution. Especially if it needs extra type conversion.
So, query params are all strings by default. For basic types, type conversion should be simple.
?q=foo&q=bar&bool=true&count=10

should give
{
  q: ['foo', 'bar'],
  bool: true,
  count: 10,
}

But what if the user defines more complex types, e.g., relations etc.?
model Profile {
  user  User
}

I too think that a dedicated QueryDTO would make a lot of sense.
I do not however think, that the QueryDTO has to cover relations, as those would be available under their own path using a dedicated QueryDTO.
/profiles/  would accept query parameters defined in QueryProfileDTO
/profiles/123/followers would  also accept QueryProfileDTO (followers are other profiles/users)
/profiles/123/galleries would accept QueryGalleryDTO
The /profiles/  endpoint can return a ResultDTO that includes the actual data requested, information about pagination and sorting and also additional links, pointing to the linked resources that can be queried separately.
But I would consider all of this API implementation details, that do not directly concern this library.
Mapping multiple DTOs into a single QueryDTO can get quite messy if the DTO properties are not unique.
Sometimes you want endpoints that take relations into consideration of course, as this would reduce the amount of queries needed, but in this case I would manually create a dedicated DTO, which can also be done using Nest's PartialType() mapping utility function, based on the QueryDTOs in question.
Since the QueryDTOs, the new QueryDTO is based on, have all the transformation decorators in place these custom QueryDTOs should be able to make use of them, too.

My question above was about how to handle relations and complex types. Since query params can only be simple strings, I think the fields meta and user in the example below should just be ignored. So, only fields of basic types (string, number, boolean) can be queried.
The following model:
model Profile {
  id      String @id
  age     Int
  isAdmin Boolean
  tags    String[]
  meta    Json
  user    User @relation(...)
}

could result in something like this:
export class QueryProfileDto {
  @ApiProperty({
    required: false,
    nullable: true
  })
  @IsOptional()
  @IsString()
  id?: string;

  @ApiProperty({
    type: 'integer',
    format: 'int32',
    required: false,
    nullable: true
  })
  @IsOptional()
  @IsInteger()
  @Type(() => Number)
  age?: number;

  @ApiProperty({
    required: false,
    nullable: true
  })
  @IsOptional()
  @IsBoolean()
  @Transform(({ value }) => value === 'true' || value === '1')
  isAdmin?: boolean;

  @ApiProperty({
    isArray: true,
    required: false,
    nullable: true
  })
  @IsOptional()
  @IsArray()
  @IsString()
  tags?: string[];
}

Possible query url:
?tags=foo&tags=bar&age=40&isAdmin=1

I think  | null is not necessary, because unlike in the request body you can't pass null. Either the query param is defined or not.
Maybe tags also needs some custom @Transform, because a single ?tags=foo may result only in a string, but not string[].


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v4.4.296",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=a48f6a2bf33734ec5669ee03067dfb6c5b4818d6"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "a48f6a2bf33734ec5669ee03067dfb6c5b4818d6",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "dm btree remove: fix use after free in rebalance_children()\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.4.296 by commit a48f6a2bf33734ec5669ee03067dfb6c5b4818d6. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2021-1002613


Question: How To Access Files/Folders In The Root DirectoryI want to Access Files in Root Directory With Superuser Access

I want to Access Files in Root Directory but This app I can't give Root permission, is there any way to do it?

No. I'll try to add this in a future version. For now I'm closing this issue as I can't fix it.


Should threejs be changed to a peer dependency?

#728, #755, and related issues discuss the use case of using NGL in downstream projects that also need to use threejs as a dependency, eg because NGL is embedded into an existing threejs app or a downstream app wants to extend NGL somehow (eg, adding a custom representation). To further support this use case, I'm wondering if it would make sense to make threejs a peer dependency instead of a regular dependency. This would ensure that only one copy on threejs is in play, and its version is always synced with the one required by NGL.

Honestly, I don't know....
I've spent lots of time recently trying to migrate the Three.js version (still an ongoing project) and realized that this library makes breaking changes in the context of NGL every 5-10 minor version bump.
In your experience, how using peerDependencies instead of dependencies will help solve the problem you are describing?
A user who needs an updated version of Three.js, can they make it work with NGL if we are using peerDependencies?

In your experience, how using peerDependencies instead of dependencies will help solve the problem you are describing?

Right now, since I have lint rules set up forbidding importing dependencies that are not in my package.json, I have to add three as a dependency to my package.json. If NGL upgrades its three version and I don't, there will be two conflicting versions of three installed at use at the same time. Same if I accidentally change the version and get out of sync with NGL. Making it a peer dependency would enforce that if I add it to my package.json, it has to use the same version of three as NGL, preventing the two from getting out of sync.

A user who needs an updated version of Three.js, can they make it work with NGL if we are using peerDependencies?

That user shouldn't be using an updated version of three since it is liable to break NGL though, right? Or are you arguing for the case of the user creating a separate three app that never interacts with NGL? In the latter case I suppose that would be an issue.
I suppose you could make an argument that you should just import ngl's version without adding it to the main package.json, and in my case that just means adding lint rule exclusions. Though it could still create problems if another dependency is used which also has three as a dependency or peer dependency of a conflicting version


Rename total_documenttype_counts to documenttype_counts

No other counts field prefixes total.

Related: #140
Added to sheet: https://docs.google.com/spreadsheets/d/1UdPZXmiuir_mFQDYJHTWbwgdWnORzMTlbKUEsspxK54/edit#gid=855843256


Add a function to zlib data compression

One use case is to generate Kroki.io diagrams without requesting APIs, and reduce build time and avoid network issues.

You can POST diagrams to Kroki, see https://docs.kroki.io/kroki/setup/usage/#post-request -- I'm not sure your other arguments are very convincing. Build time should be the same. What network issues?

You can POST diagrams to Kroki

Yes, I can, I'm implementing it via GetRemote.

Build time should be the same

APIs should take more time than local compression. With this feature, there won't be HTTP requests during the build stage.

What network issues?

One example is that the API may crash sometime, which could cause the build to fail.

It seems to me that calling the Kroki API once with a POST request (and caching the returned image) makes more sense that to have every client call the Kroki API with a GET request.
I would rather know about a Kroki API crash during a build than have clients see broken images.

APIs should take more time than local compression. With this feature, there won't be HTTP requests during the build stage.

OK, then I misunderstood. I will keep this proposal open. The proposed use case isn't convincing enough alone for this to be added to Hugo.
If someone asked me how to Integrate Kroki with Hugo, I would agree with @jmooring and say that the best/recommended way would be to do POST request during builds.


Two formattings inside word cause wrong output

In case of applying two formattings inside word output is wrong.
eg.
ab*~~cd~~*ef

permalink to example

Not sure if this can be fixed fast, but at first glance really may be error.

This is intended, add spaces like this: ab *~~cd~~* ef if you want two formattings.
Reason is: as per spec, * cannot open a tag if it is preceded by a letter (b) and succeeded by a punctuation (~).
Commonmark standard (the one we follow) doesn't define strikethrough, but if you replace it with __, you get similar results:
https://spec.commonmark.org/dingus/?text=ab*__cd__*ef


Update to Godot 3.1

This doesn't work on 3.1. Hopefully it require just small fix.

You should set Mouse -> Filter to Pass. Othervise GUI event won't get passed into _gui_input handler.

Thanks, updated Readme.


Add apk with control plane QSG

 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.


Fixed #26561 -- Fixed unusably slow SelectBox performance with large lists

Discussion on ticket 26561. Patch to at least make it usable with large lists in 1.10, with the long-term rewrite on ticket 15220 still to come.

Using a jQuery instead of raw JavaScript doesn't serve your optimization purposes very well. I believe this can be achieved without using it.

@charettes, the bulk of the speed boost comes from the changes to redisplay, as that was the biggest slowdown culprit. Unfortunately there's an IE bug with setting innerHTML on a select (which I originally did to clear it out), so I figured $.empty would handle the cross-browser compatibility problems there.

@charettes, I've redone the commit to address your comments. Now the only use of jQuery is for $.empty. If there's a quick and easy cross-browser compatible alternative we can drop it all together.
The main performance gains (in order of impact) are from:

Refactoring redisplay to rebuild the select in a string instead of in DOM
Using splice in delete_from_cache
Stopping filter for an option after the first token miss

The rest of the changes, using local variables to avoid re-executing code inside of loops, offer a very small performance impact compared to the above and could be dropped if there is a desire to keep the patch smaller and more focused.

Buildbot, test on selenium.

The selenium test results are a bit confusing. Looks like there was at least one test mishap (address already in use during setUp), a couple failures for JS i18n which this shouldn't affect at all, and then the seemingly relevant failures are for failing to activate the 'select' icon when clicking inside the from box, but that should be entirely handled by SelectFilter2.js, since it binds a change handler and refreshes the icons on change, never calling SelectBox.js during that process.
¯\_(ツ)_/¯

@charettes, I found the cause of the selenium test failure, it was that the event listeners for click events on the select boxes were being lost since now the entire select is being recreated. I fixed the issue by moving the binding up further in the DOM and checking that the events were for select/option.
I also added another performance fix as another commit (slight scope creep) because I found a good fix for the very slow (I was seeing 5 seconds for a list of 20k options) SelectFilter.refresh_icons on Chrome and Safari. It's a little clever trick that momentarily marks the select as required and asks the browser if it's valid via the :valid to see if there are any selected options. This was much, much faster in my testing. The is(':valid') could be changed to checkValidity() if preferred, but the latter fires an event according to the documentation so I thought a quick check of the CSS selector would be cleaner.

merged in fb68674ea4b05280e3deb2ed23602038755092bc, thanks!

Thanks for your work on this David.

@charettes, no problem. Happy it was able to make it in for 1.10, that'll make some people over at django-newsletter happy.
@timgraham, of course right after you merged this I spotted one line that could be improved, but it's not a big deal. Line 166 in SelectBox.js would be better moved outside of that else so that it is always run even if is(':valid') throws an error (which it will on IE 9). If an error throws there the required attribute will be left on the field, but I don't think it's a big deal because if the error occurs then the browser doesn't support HTML5's required to begin with, so in all likelihood it will entirely ignore the required attribute. Just a hygiene thing with it being left there inadvertently.


Veracode warning on 'product-agents-edc'

Please report undisclosed or confidential vulnerabilities here: https://www.eclipse.org/security/
Topics (Please mark an [x] to your Topic):

[ ] KICS
[ ] Invicti
[ ] GitGuardian
[ ] Pentesting
[ ] Security assessment (threat modeling, code reviews)
[ ] Trivy
[x] Veracode (initial setup, GitHub integration)
[ ] Other

Please describe your request:
For 'product-agents-edc' we now successfully setup the veracode workflow (adopted from 'tx-tractusx-edc' https://github.com/catenax-ng/tx-tractusx-edc/blob/main/.github/workflows/veracode.yaml)
https://github.com/catenax-ng/product-agents-edc/actions/runs/5195436041
But the uploading of the artifact jars seems to run into a warning, e.g.
[23.06.07 02:49:50]
[23.06.07 02:49:50] Application profile "knowledge-agents-edc/controlplane-memory" (appid=1748166) was located.
[23.06.07 02:49:50]
[23.06.07 02:49:50] Attempting to create a new build with name "controlplane-memory-c5511f6e17788b497f14b3271d67112d05a92ff8".
[23.06.07 02:49:51]
[23.06.07 02:49:51] * Action "UploadAndScan" returned the following message:
[23.06.07 02:49:51] * App not in state where new builds are allowed.
[23.06.07 02:49:51]
[23.06.07 02:49:52]
[23.06.07 02:49:52] * A scan is in progress or has failed to complete successfully. Wait for the current scan to complete or delete the failed scan from the Veracode Platform and try again.
Is that because the application profile name does not match/prefix with the 'product-agents-edc' name or do we have to setup some additional things there?

Hi @drcgjung , Veracode can't scan your uploaded files. Please check your uploaded files.
The projects at the bottom of the picture have the same problem.
Do you need all these projects? If no, please let me know - I will remove them.
Removed all running scans (also knowledge-agents-edc/controlplane-memory" (appid=1748166)), you can retest. Means, trigger your Veracode pipeline first and then see which jobs still go wrong.

@drcgjung Please review, the comment from @the-tatanka

Hi @drcgjung , Veracode can't scan your uploaded files. Please check your uploaded files. The projects at the bottom of the picture have the same problem.
Do you need all these projects? If no, please let me know - I will remove them.
Removed all running scans (also knowledge-agents-edc/controlplane-memory" (appid=1748166)), you can retest. Means, trigger your Veracode pipeline first and then see which jobs still go wrong.

Excuse me for the late answer, was on holiday.
Since I have not worked with veracode before, It was more of a copy&paste thing from tractusx-edc
(see for example https://github.com/eclipse-tractusx/tractusx-edc/actions/runs/5238983165/jobs/9458401715)
The repo results in 6 docker images (4 extended control planes, 2 additional data planes)
I presumed that we need to setup a veracode module per resulting image.
What I did not notice (until now) was that tractusx-edc moved from a classpath/lib-based approach to an uber-jar approach in the 0.4 versions. So it is clear now that veracode cannot find an endpoint in the target jar.
Should I rather:

add the classpath libs to the tgz as well or rather
move to an uber-jar approach (and put several uber-jars in a single veracode module tgz)

Thx for caring,
CGJ

resolved


Anyway to see debug sql in the logs?

Or do we just follow the example and prey?

https://stackoverflow.com/questions/568564/how-can-i-view-live-mysql-queries#7470567

I have autoloaded the plugin
cat data/loaded_plugins
emqx_management.
emqx_recon.
emqx_retainer.
emqx_dashboard.
emqx_rule_engine.
{emqx_auth_mysql,true}.
I can see there are no acl calls - even though I have configured
auth.mysql.acl_query = select allow, ipaddr, username, clientid, access, topic from mqtt_acl where ipaddr = '%a' or username = '%u' or username = '$all' or clientid = '%c'
2024-02-13T06:16:25.389504Z	 6305 Prepare	select password from mqtt_user where username = ? limit 1
2024-02-13T06:16:25.389620Z	 6305 Execute	select password from mqtt_user where username = 'gkx' limit 1
2024-02-13T06:16:25.389978Z	 6303 Prepare	select is_superuser from mqtt_user where username = ? limit 1
2024-02-13T06:16:25.390101Z	 6303 Execute	select is_superuser from mqtt_user where username = 'gkx' limit 1
2024-02-13T06:16:30.441864Z	 6298 Prepare	select password from mqtt_user where username = ? limit 1
2024-02-13T06:16:30.442004Z	 6298 Execute	select password from mqtt_user where username = 'gkx' limit 1
2024-02-13T06:16:30.442395Z	 6303 Execute	select is_superuser from mqtt_user where username = 'gkx' limit 1
2024-02-13T06:16:30.701130Z	 6299 Prepare	select password from mqtt_user where username = ? limit 1
2024-02-13T06:16:30.701273Z	 6299 Execute	select password from mqtt_user where username = '03000001' limit 1
2024-02-13T06:16:35.505184Z	 6300 Prepare	select password from mqtt_user where username = ? limit 1
2024-02-13T06:16:35.505422Z	 6300 Execute	select password from mqtt_user where username = 'gkx' limit 1
2024-02-13T06:16:35.505803Z	 6305 Prepare	select is_superuser from mqtt_user where username = ? limit 1
2024-02-13T06:16:35.505930Z	 6305 Execute	select is_superuser from mqtt_user where username = 'gkx' limit 1
2024-02-13T06:16:40.554297Z	 6302 Prepare	select password from mqtt_user where username = ? limit 1

If superuser SQL is not enabled or the client is not a superuser, ACL SQL is used to query the client's ACL rules in the database.
Ok, I choose a non super user and see the acl query in the mysql logs.
2024-02-13T07:38:01.895679Z	  111 Execute	select allow, ipaddr, username, clientid, access, topic from mqtt_acl where ipaddr = '<IP_ADDRESS>' or username = 'gkx123' or username = '$all' or clientid = ''


Include notebook data in save

Our group uses the TTS notebooks to record progress on personal goals, plans for future enhancements, etc. It would be really nice to have this persist along with the saved data. Maybe also the options notebook so the values don't have to be reset when importing into a new game.

Good point. Do you have other notebook as as well or one per player? Options might be tricky. Simply saving and loading would be easy. But if a newer mod version has new options those would be overwritten. I'll have to think of something. :-)

Just one notebook per player. Not renamed from default colors.
Options point is good. Maybe it gets saved but not loaded into options
notebook? That way it's at least in the game for reference.
On Sun, Feb 21, 2021, 1:39 AM Sebastian Stern<EMAIL_ADDRESS>wrote:

Good point. Do you have other notebook as as well or one per player?
Options might be tricky. Simply saving and loading would be easy. But if a
newer mod version has new options those would be overwritten. I'll have to
think of something. :-)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Sebaestschjin/gloomhaven-campaign-manager/issues/70#issuecomment-782809230,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHLY42B7L4HEZPG4JVBGTKTTACTDFANCNFSM4X6FD44A
.

Yeah, good idea. It could create a new notebook "Previous Options" or something which can be checked. This would be very simple to implement, so I guess I'll start from there. :-)