External Torque

For discussion:
Beginning to implement external torque inputs. API might make sense to change, though, and is currently incomplete.
The idea is that the frozen field on the Thruster could indicate whether to freeze either force or torque axes, allowing anywhere from 0/1 to 6 DOF for a Thruster.
Currently this ignores the frozen field and provides all possible DOF. In addition, it might be worth adding a different strength field for torque versus force. However, I'm not going crazy in case a preferred approach is an entirely separate class  for external torques than forces.
This builds on #94 and addresses #61

@cdfreeman-google wondering if you have thoughts/comments on the API here

I REALLY like the choice of using frozen to specify DoF - good thinking.  I'll review this PR more in depth in the next day or so, unless @cdfreeman-google gets to it first.

Agree with Erik, this is great stuff.  Thanks so much for the proposal!  Added one interface comment, and pointed out the one other place in the codebase that I think this touches currently.

@erikfrey @cdfreeman-google thanks for the positive responses! It's still not fully functional as it doesn't properly respect the frozen fields yet. I'm working on this still. I was running into issues with handling a flexible number of channels (particularly in the case of multiple forces). I've done a bit more learning about jax and think I have a valid approach by vectorizing over both a mask array and an index array (padded with zeros for masked out dimensions). I hope to have this updated by the weekend with comparisons on the pendulum channels to ensure no regression.
p.s. there is already a bit of a fudge in that the pendulum classes override action_size to be 1 despite the force using 3 elements. I'll hopefully have it freeze all but one axis and be able to remove that class definition.

@erikfrey I believe all nits should be handled.

Hi @peabody124, this is a tricky design decision - in general I like the idea of moving to generalized forces that can take a variable number of the 6dofs, and we will in a future version of Brax in which we do reduced coordinates (see #104 )
Looking at this closely now, I can see that it works but is a bit out of step with how we construct our other helper classes.  Joint has Revolute, Universal, Spherical, which each purposefully handle distinct shapes so that we vmap over them individually.  In the same way Actuator devolves under the hood into an Actuator1D, Actuator2D, Actuator3D classes, so that we can vmap over them.
So I think for now, we should stick to that pattern.  In the future when we move to more generalized joint/actuator system we will rip off the bandaid and change all of this at once.  So concretely I think that means: leave Thruster as a dumb/simple class that for now is hard coded to take in 3 positional dimensions, and add a new message type and class called, say, Twister that is also dumb/simple and takes in 3 angular dimensions.
I realize this is a big left turn from what you have so far and apologize for that - Daniel and I should have looked at this more closely early on when you first proposed it and anticipated this design issue.  If you'd like to update this same PR, we'll get this in!

Sure. I can split this into a separate Twister. I don't mind stripping out the action dimension selection too. It's actually not needed for the purpose I'm currently using the residual forces for.
That being said, before doing so I'd just like to play devil's advocate for this implementation with another use case I'd been considering. Say you want to make a quadcopter (semi accurately) then you would want four thrusters with force along the Z axis only and torque around the Z axis only. A 3D twister/thruster format would prohibit making that type of model without an awkward environment that maps the action space onto the system's action dimensions.
The current implementation, by using the additional (precomputed) mask helper variable, still does allow vmap-ing along them.
Anyway, if you want to split into separate Thruster class that is always 3 axes, I can do that.

You are 100% correct that with the proposed design, you'd have to do something like:
def step(self, state: env.State, action: jp.ndarray) -> env.State:
  # quadcopter thrusters only work in +/- Z
  act = jp.array([0, 0, action[0], 0, 0, action[1], 0, 0, action[2], 0, 0, action[3]])
  qp, info = self.sys.step(state.qp, act)
  ...

Which is less powerful than what you have built.  Normally I'd agree with your devil's advocate position, but we are going to soon start working #104 which is going to involve some pretty big changes both under the hood and to the config format.
In this case I'm prioritizing uniformity of the underlying implementation over expressivity - I'd prefer to keep things simple for now to ease the pain of that upcoming refactor.

External torque is now implemented in 3a197dba959901fab16333ccde7e44e8f1981cfd - please try it out and let us know if there are any issues.

@erikfrey is there still external torque in v2? I was working on porting some code to that but I'm not seeing they still exist...

Hi @peabody124 , we don't have external torque in v2, but if you'd like to implement it, I can provide some pointers

@btaba that would be great. happy to work on PR since I'd love to update my code soon

Hey @peabody124
Actuator sites are more likely to be first implemented in MJX (https://github.com/google/brax/discussions/409) following MuJoCo's implementation. Here's where to start:
https://github.com/google-deepmind/mujoco/blob/e57b34303a13195bf5fe87d37cb778321130a229/src/engine/engine_core_smooth.c#L1001
I'm assuming you're using the generalized backend. If not you can follow the implementation in v1 https://github.com/google/brax/blob/main/brax/v1/physics/forces.py


Bump to core 6.0.25

This is now out of date, and there are memory leak fixes in place for HAPI that must use 6.1.2 or higher.
New PR is here: https://github.com/hapifhir/hapi-fhir/pull/5317


AMQNET-735 Add AsyncMessageListener to Consumer API

It depends on https://github.com/apache/activemq-nms-api/pull/26

I thought API was merged?

The API got merged but hasn't been released yet. I was holding out for a solid nms-amqp implementation. Now that I've got one, I've kicked off the release process for nms-api version 2.1.0.

nms.api is merged and released. @michaelandrepearce @lukeabsent could you please take a look?

+1 form me. @lukeabsent ?

@lukeabsent please review also else will look to merge soon (end of week)

Hi @Havret is it possible this change did not make it into the Apache.NMS.AMQP 2.2 package?
https://www.nuget.org/packages/Apache.NMS.AMQP
We're using Apache.NMS.AMQP 2.2 and Apache.NMS 2.1 packages, but receive a System.TypeLoadException:
Method 'add_AsyncListener' in type 'Apache.NMS.AMQP.NmsMessageConsumer' ... does not have an implementation.
It seems there are no corresponding (github) tags associated to check this.
The version at https://activemq.apache.org/components/nms/providers/amqp/downloads/ doesn't contain it.
The version at https://www.nuget.org/packages/Apache.NMS.AMQP/2.2.0 doesn't contain it.
Method 'add_AsyncListener' in type 'Apache.NMS.AMQP.NmsMessageConsumer' from assembly 'Apache.NMS.AMQP, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' does not have an implementation.

System.TypeLoadException
at Apache.NMS.AMQP.NmsSession..ctor(NmsConnection connection, NmsSessionId sessionId, AcknowledgementMode acknowledgementMode)
at Apache.NMS.AMQP.NmsConnection.CreateSessionAsync(AcknowledgementMode acknowledgementMode)
at Apache.NMS.AMQP.Util.Synchronization.TaskExtensions.GetAsyncResult[T](Task`1 task)
at Apache.NMS.AMQP.NmsConnection.CreateSession(AcknowledgementMode acknowledgementMode)


fix: Made metering point type nullable toward bff

Description
References

We're currently processing your upload.  This comment will be updated when the results are available.


Support encoding PNG from BGRA colors

This would be helpful for Servo.

BGRX (little endian XRGB) would be very useful for Wayland (le-xrgb8888 is one of the few mandatory format for wayland compositors).


crio-umount.conf and seccomp.json are present in both cri-o and libpod

RE: $SUBJECT, ETCDIR_LIBPOD in libpod and ETCDIR_CRIO in cri-o are both set to /etc/crio, this causes conflicts with the cri-o package unless of course, I change the install rules for the podman deb package.
Do these 2 files need to be present in both repos?
@rhatdan @mheon @baude

We can drop the umount config here, it's not used elsewhere.
Seccomp should be moved out of both repositories and sourced from somewhere common - it's not specific to any of our tools. I think there's a containers-seccomp repo we created for this?

ETCDIR_LIBPOD is set to /etc/crio in the makefile, apparently, which seems very wrong

@mheon can't seem to find the containers-seccomp repo, do you have a link for that?

It lives here: https://github.com/seccomp/containers-golang
Unfortunately, I think my memory was faulty - it doesn't seem to include the defaults

@mheon I notice the doc mentions it generates a default.json. It's intended to be the same as our seccomp.json, yes? Also, not sure what defaults you mean, but is that something you can get included upstream maybe today? I guess I can create another package for containers-seccomp for ubuntu.


Documentation for AppleID and/or authorization code flows

I can't figure out how to sign in using AppleID with this library, and I can't understand how the authorization code flow is supposed to work in general.
The README explicitly says to use "Resource owner password-based", but that should obviously be "Authorization code" in this case. But what about the client ID and client secret? Do those work just like with convert_token? Can I use a fake URI for redirect URIs? It seems that no matter which way I twist and turn things, I get invalid_client or invalid_grant back.
Much appreciated.

@denizdogan yes Apple Sign in is a little bit of a mistery so far. It's in my list of todos to include it here. Did you find anything interesting yourself that can be included to the docs?

@wagnerdelima Not yet, was hoping that someone else had made it work. The whole chain of dependencies is so convoluted, it's hard to know what library does what. PyCharm goes a long way, but even then it's hard to keep track. :)

Having debugged this one for a little bit, it seems like there's a Grant object missing while validating the authorization code. Line 412 in oauth2_provider.oauth2_validators.OAuth2Validator.validate_code raises Grant.DoesNotExist. It seems to me that this Grant object is meant to be created in oauth2_provider.views.mixins.OAuthLibMixin.create_authorization_response.
From what I've been able to gather, it seems that the idea is that this is created in AuthorizationView with an HTML page and a form. Obviously, this is not suitable for AppleID login, so I'm definitely missing something. (Or some library is missing something.)
Trying a completely different approach, I've tried to POST some authorization code data to /api/auth/complete/apple-id/, but all that does is give me:
django.urls.exceptions.NoReverseMatch: 'social' is not a registered namespace
Internal Server Error: /api/auth/complete/apple-id/

This is the same problem that has been mentioned here two weeks ago: https://github.com/wagnerdelima/drf-social-oauth2/issues/79#issuecomment-1002739804
I'm not sure where to go from here. It seems that something is missing here, but I'm not entirely sure what it is.

Unfortunately I never tried to use the apple sign on. I can try debugging this a little later once I am done with my current task.

I finally managed to get AppleID sign-in working, but not with the authorization code, rather with the identity token (which I didn't realize we receive from the SDK). Here is how it's done in SwiftUI:
SignInWithAppleButton(.continue) { request in
    request.requestedScopes = [.fullName, .email]
} onCompletion: { result in
    switch result {
    case .success(let authResults):
        switch authResults.credential {
        case let credential as ASAuthorizationAppleIDCredential:
            guard let identityToken = credential.identityToken else {
                print("no identity token received")
                return
            }
            let identityTokenString = String(decoding: identityToken, as: UTF8.self)
            // convert the identity token to an access token:
            // POST /auth/convert-token/
            // {
            //     "grant_type": "convert_token",
            //     "backend": "apple-id",
            //     "client_id": "<application ID>",
            //     "client_secret": "<application secret>",
            //     "token": identityTokenString
            // }
        default:
            // failed to get credentials
        }
    case .failure(let error):
        // sign-in failed
    }
}

Thanks to this comment in the deprecated repo: https://github.com/RealmTeam/django-rest-framework-social-oauth2/issues/241#issuecomment-932751527
Close this issue if you'd like, otherwise let's consider this issue to be about documenting this somewhere.

Sorry, maybe I misunderstood your question, was my response what you were looking for? I don't really remember.

The question was not how to set up the configuration. It's just that I have not found a way to create an apple application to get the apple key, apple secret etc. Do you know where to get those variables from?

@wagnerdelima Here's some example values, with some explanation below.
SOCIAL_AUTH_APPLE_ID_CLIENT = "com.foobar.MyApp"
SOCIAL_AUTH_APPLE_ID_TEAM = "FDHPXE33MD"
SOCIAL_AUTH_APPLE_ID_KEY = "3AB82BA21G"
SOCIAL_AUTH_APPLE_ID_SECRET = "-----BEGIN PRIVATE KEY-----\nVdH8so0J1DSjnWydQYEXIgBXJznkeOxL9UwKDuipH4KSDzbBF4ehTlbkui4eo3fB\nDk2qNRWtomLL1Rh0lqtmvQ7BdqqLXA8aYba7p12DWSIqP0ZOZsHPGTOmdSDXfkak\nfW9cT2tS7XxGtqtnm8mpHP9c8WzufCN7Cri6Co27ULUTz8MUPQebws7ugYiFT0Xf\n52F6BIw8\n-----END PRIVATE KEY-----"
SOCIAL_AUTH_APPLE_ID_SCOPE = ["email", "name"]

Obviously all values above are fake.

SOCIAL_AUTH_APPLE_ID_CLIENT – literally just your app bundle ID, you choose this yourself when creating your app
SOCIAL_AUTH_APPLE_ID_TEAM – the team ID, can be found in e.g. App Store Connect, or in Apple Developer, etc.
SOCIAL_AUTH_APPLE_ID_KEY – you need a private key to use Sign in with Apple, just follow the guides linked below. They describe the process of generating a key and where to find its ID. The ID can also be found in the actual filename of the .p8 file, e.g. "AuthKey_3AB82BA21G.p8". Anyway, the private key ID goes in this field.

https://developer.apple.com/help/account/manage-keys/create-a-private-key
https://developer.apple.com/help/account/manage-keys/get-a-key-identifier

SOCIAL_AUTH_APPLE_ID_SECRET – literally the content of your .p8 file, just replace newlines with \n

Additionally, you need to add the "Sign in with Apple" entitlement to your app. It's simple to do via "Signing & Capabilities" in your app project, and when done correctly you will have a .entitlements file in your app project root, and it will look like this:

Hopefully this helps. I'm not 100% sure what is being asked exactly.

Hello @denizdogan I've create an apple sign in with react and I got the id_token after sign in also I have make all the configurations in my settings.py file properly but when trying to convert_token I got this error "token": [
"Ensure this field has no more than 500 characters."
] can you help me here ?

https://prog.world/django-drf-sign-in-with-apple/
This explains how to get the necessary stuff for Apple.


Fixes #39: Limit updated to 50K

Discovered by a dev on Stack Overflow

Would you mind updating the unit test as well?

@thekaveman Give that a try. I'll admit I made both the change and the unit test update fairly blindly, as I don't have a .NET environment on my Mac.
Which begs the question - has SODA.NET been tested with Mono? That'd make it easier for me to contribute.

@chrismetcalf looks good! Nice work for a Mac guy :wink:
You bring up a good point, and I hadn't previously thought about Mono support. I don't have access to a Mac environment, but I'll see about playing around in my Linux VM (maybe even try out the new Visual Studio Code)


Custom SupplierPart object manager and prefetch related models

Should fix #1809 and maybe improve performance a bit when lookups to SupplierPart objects are made :smiley:

Thanks for the improvement. We should try to profile the other admin pages too.


Ordering of time dimension for observations when sequence length is greater than 1

Hi
For an algorithm I am implementing, I am using BC-RNN with a sequence length of 10 and thus the training data is indexed in a fashion where batch is the first dimension and time is the second dimension. I also for this algorithm, however, need a second network that is conditioned only on the current (or most recent observation). It is still very unclear to me whether the data structure containing the data is ordered in such a way that the first index represents the most recent time step or the last index represents the most recent time step.
Thanks

The second dimension of the batch is in time-order. For example, batch["actions"][0] will have a sequence of actions a_t, a_{t + 1}, ..., a_{t + T - 1} where T is the sequence length.


Some android fixes and added additional hardware features.

Use namespace in the app's Gradle file across all the Gradle projects to fix deprecation warnings.
Added Drop file event support by including the intent-filter reference and support for additional hardware features and USB device attachment.

https://github.com/openfl/lime/assets/77043862/3c1c76cc-c4f4-42a6-aadd-7c8379a1d3c5
This shows how the file drop works for android.

screen-20240609-144625.mp4
This shows how the file drop works for android.

pls can u make a haxe package that adds support for microphone capting pls we need it a lot

Use namespace in the app's Gradle file across all the Gradle projects to fix deprecation warnings.

Since this is fixing an issue, I'd say it should go on the develop branch.

-Added Drop file event support by including the intent-filter reference and support for additional hardware features and USB device attachment.

Since this is a new feature, it goes to 8.2.0-Dev. You may want to split this into two PRs.

Updated AndroidManifest.xml to include new hardware features and improve compatibility.
Added optional support for:

Bluetooth
Gamepad
USB host
PC-type external mouse input
Low latency audio
Pro audio

Included a new intent filter for handling USB device attachment events.
Ensured all new features are optional by setting android:required="false".

These changes enhance the application's flexibility and compatibility with various input methods and devices, ensuring a better user experience across different hardware configurations.

I'm not convinced enough Lime apps use these to make them the default. Why not just set them on a per-app basis?

Add Bluetooth permission.

This means apps will be listed as using Bluetooth even when they actually don't. Could turn users off from downloading an app. Again, why not just set it on a per-app basis?

pls can u make a haxe package that adds support for microphone capting pls we need it a lot

This is off-topic for this thread. Please discuss it in the issue instead.

So, should i split the pr?

Well, the third pr mostly adds things that most sdl apps have by default but if it's not suitable for apps, i can revert the commit.


Menu icons don't show up on Linux (.desktop files contain references to .png files we don't ship!)

Solution: include icons in PNG format as well.

Tracking at https://github.com/conda-forge/mne-feedstock/issues/74


Complete CloudApiClient

The CloudApiClient class is missing methods for the following API functions

[x] List a site’s tasks. [See #11]

GET /sites/:site/tasks

[x] Copy files from one site environment to another.

POST /sites/:site/files-copy/:source/:target

[x] Deploy code from one site environment to another.

POST /sites/:site/code-deploy/:source/:target

[x] Add a database.

POST /sites/:site/dbs

[x] Copy a database from one site environment to another.

POST /sites/:site/dbs/:db/db-copy/:source/:target

[x] Delete a database.

DELETE /sites/:site/dbs/:db

[x] Move domain(s) from one site environment to another.

POST /sites/:site/domain-move/:source/:target

[x] Deploy a specific VCS branch or tag to an environment.

POST /sites/:site/envs/:env/code-deploy

[x] Delete a site environment database instance backup.

DELETE /sites/:site/envs/:env/dbs/:db/backups/:backup

[x] Restore a site environment database instance backup.

POST /sites/:site/envs/:env/dbs/:db/backups/:backup/restore

[x] Configure Live Development on an environment.

POST /sites/:site/envs/:env/livedev/:action

[x] List an environment’s domains.

GET /sites/:site/envs/:env/domains

[x] Get a domain record.

GET /sites/:site/envs/:env/domains/:domain

[x] Delete a domain.

DELETE /sites/:site/envs/:env/domains/:domain

[x] Add a domain name.

POST /sites/:site/envs/:env/domains/:domain

[x] Purge the Varnish cache for a domain

DELETE /sites/:site/envs/:env/domains/:domain/cache

#73 fixes the problem with installing from a Drupal project. I think the make file should also work fine.


Remove MKL form macOS tests

Fixes #1080

coverage: 94.73%. remained the same
when pulling a3d3aec24d3ebae436bbe6094ee235f6b2843b09 on bnavigator:osx-no-mkl
into 9fa4aac494776650e82d0c2dbdd36ff330a82fad on python-control:main.


[FLINK-30795][tests] StreamingWithStateTestBase of win not correct

What is the purpose of the change
Fix test case broken in win os.
Brief change log
Change "file://" to LocalFileSystem.getLocalFsURI.toString.
Verifying this change
org.apache.flink.table.planner.runtime.stream.sql.WindowAggregateITCase case work correctly in win os.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Hi @AlanConfluent , could you please help me review this?

Hi @healchow @zentol , could you please review and merge this pr? the ci has passed.

We no longer account for developing Flink on Windows. Only unix environments are supported.


Bug fixes

I don't get it, is there actually a bug fix among these unnecessary formatting cahnages?

No it just makes everything unnecessarily more verbose.
For example what is the point of this:
if (childSupplier != null) {
    return childSupplier;
} else {
    // If the child Value is not found, return null
    return null;
}

Just return childSupplier; ???

Makes it easier to read, along with some better checks and some fixes.

I can assure you that sprinkling {...} blocks does not in fact make it easier to read.
It just adds more indentation levels that are unnecessary for most one liners.
At best I can appreciate some of the variables renamed in here, still tho it's not really fixing any bug.


OpenAPI docs

Are there any plans for supporting automatic OpenAPI docs generation or some other method to quickly generate API clients?

Definitely there are at some point.
You are welcome to contribute OpenAPI module as well.

The work has started on the openapi branch, you are welcome to review

openapi module released in 1.6.0


Always inline Unary Operations

Currently UnaryOperations are not inlined which make compilation to IL a bit more tedious. Consider the following program:
let arr = []
for(let i =0; i< 10; i++){
 arr[i++] = i
}

Which would compile to:
v0 <- CreateArray []
BeginFor v1
    v2 <- UnaryOperation v1, '++'
    StoreComputedProperty v1, v2, v0
EndFor

Without inlining this IL would lift to:
let arr = []
for(let v1 =0; v1< 10; v1++){
 const v2 = v1++
 arr[v2] = v1
}

Which is incorrect. Of course this could be solved by generating additional IL instructions to emit a temporary variable and then reassign the temporary variable. Instead a more elegant solution would be to always inline UnaryOperations which saves us from unnecessary IL gymnastics with temporary variables.

Don't these two JavaScript snippet behave identically though? It shouldn't matter whether you do print(i++) or let t = i++; print(t).
One thing that does matter though:
let a = [1,2,3];
a.length++;

vs
let a = [1,2,3];
let l = a.length;
l++;

I'm not sure if we are handling that correctly currently? Probably it'd need to be compiled to something like
let a = [1,2,3];
let l = a.length;
l++;
a.length = l;

which isn't super nice, either.

Don't these two JavaScript snippet behave identically though? It shouldn't matter whether you do print(i++) or let t = i++; print(t).

You're right, in this instance inlining isn't required since they are semantically identical. However if we have a statement like the following:
let x = 10;
let y = x+(x++)

we would need to use a temporary variable to lift this to:
let x = 10;
let temp = x
x++
let y = temp  + temp

One thing that does matter though: ...

Yeah, so this is an issue that we will have with LoadProp/LoadComputedProp and LoadElement operations :(
The current inlining approach that I've pushed doesn't address this issue. I've yet to think of a solution to handle these ops but I'm leaning in favour of handling this via the Lifter instead of generating additional IL. This is mainly to make compilation easier and have fewer instructions but I'm open to suggestions on other approaches.

Right, thanks for the examples! I'm a bit hesitant to attempt to solve these issues in the Lifter. IMO, the algorithm used by the lifter should have no effect on the semantics of the program. The inlining is only performed to make the samples more readable, but it should ideally also work without any inlining at all. I also think it'd be weird for a generated FuzzIL program like
v0 = LoadProperty array, 'length'
v1 = UnaryOperation v0 '++'

To change the length of array in this example. It's more intuitive if the variable behave like, well, variables, and not potentially as mutable references to object properties :/
So I think even though it adds more instructions, I'm in favor of solving these issues in the compiler... Maybe we could even drop support for ++ and -- entirely since it leads to these kinds of weirdnesses and that way just force any compiler to turn them into += 1 first (which is more or less what should happen in the compilation pipelines of actual JS engines as well, I'd expect). Then we could still add an optimization to the lifter to turn += 1 into ++ if possible, but then it is really just that, an optimization that doesn't change the semantics of the program, just makes it a bit more readable.


Fix tests on bs-platform 4.0.7

With these changes all the tests now pass on bs-platform 4.0.7.
Fixes #33.

Coverage decreased (-0.8%) to 97.881% when pulling 1615dfce437e15dbd137b1780e8adf202b06c0ec on zploskey:fix_callbacks into 879b67fc0f2888059402a2dc8e3a443c1430363a on glennsl:master.

I actually don't know why this would affect test coverage. Essentially it just adds some annotations. What do you think? I won't be back in front of a computer for a few hours so feel free to revert that one nit in the mli file if you like.

It could be because of changing the val back into an external, but I'll have to check. If I hadn't removed the js artifacts from the repo I could have investigated it directly on coveralls, so I think I'll add those back too.


Workflow disruption

Between 8:15 PM - 9:15 PM (EST) newly created applications could not be started with npm run start.
Failed to compile.

Error in ./src/index.js
Module build failed: TypeError: /Users/joe/Desktop/testing/src/index.js: Cannot read property 'scope' of undefined
  @ multi main

One of our dependencies released a backwards incompatible change in a minor release.
We typically try to avoid this by pinning our package versions, but we currently have no way of ensuring deterministic builds.
Few users were affected, but situations like these raise questions about how we can prevent this from happening in the future.
We want to ensure we keep our users' trust in our reliability, and I think this deserves some discussion about what can be done.

Is there anything that prevents us from shipping releases with npm-shrinkwrap.json and yarn.lock files (does yarn try use dependents lock files?)?
Should we start bundling dependencies again? There seems to be hard feelings about them: https://github.com/facebookincubator/create-react-app/pull/1068. Are they better in NPM 4? Can we push to make them better?
Or should we live with that accidents happen? 🤷♀

Yarn only uses the top-level yarn.lock - but that should be enough, right? The lockfile includes direct and transitive dependencies. If you package a version that works, it'll work on the client until they run an upgrade.

That might work if we ship it as part of our template, which seems a bit finicky.

Yeah - keeping it up to date be a manual process then, right? Each release someone would have to regenerate an up-to-date lockfile.

We'd probably just push the responsibility of publishing onto our CI server which would generate a lockfile and run tests against it.

Wouldn't that still break when the user runs yarn upgrade or installs a dependency?

bundledDependencies likely won't get better because they are a fringe feature. They cause other problems, are buggy, and Yarn doesn't support them either.
Shrinkwrap only works on npm but not Yarn.
I think we should just teach people about setting up Yarn and using a lockfile. That solves all problems. We can include this in the User Guide (e.g. “Ensuring Deterministic Builds”).

@wtgtybhertgeghgtwtg - yes and yes, but that's on the user. yarn add shouldn't have much of an impact compared to yarn upgrade (although it could, someone better versed in the ways of Yarn can speak to that).

@wtgtybhertgeghgtwtg I don't believe yarn add upgrades anything unintentionally, but the benefit here is that they can rollback their yarn.lock file (if source controlled) and have a working project instead of manually trying to diagnose and fix errors.

@gaearon adding docs will go a far way, but it still doesn't solve fresh installs unless if we add the yarn.lock to the template

but it still doesn't solve fresh installs unless if we add the yarn.lock to the template

If user doesn't have Yarn then lockfile wouldn't be useful anyway.
If user has Yarn then yarn.lock will get created automatically in the project folder (I just tried it).

The lock file generated on install wont be of use to them if the registry has a bad published version.

Ah, I get it now. Maybe we could bundle last known good yarn.lock for Yarn users?

That's what I was thinking, we can detect if they have yarn and include a yarn.lock file. Should we do the same for npm's shrinkwrap?

Shrinkwraps are a pain to deal with so I don't want to create one by default.

It's odd, npm advises to ship npm-shrinkwrap.json, and Yarn advises to not ship yarn.lock (but doesn't give a reason).

Could we have create-react-app delete the shrinkwrap after initial install? NPM doesn't upgrade sub-dependencies (unlike Yarn) and our pinned versions + initial install with shrinkwrap should be safe.

It doesn't make sense to ship yarn.lock for libraries because only top-level yarn.lock is respected (and this is by design). But in our case we are creating a top-level yarn.lock.

Ah, that's right -- that makes more sense. But yes, we'd be making the top-level yarn.lock. 😄

Could we have create-react-app delete the shrinkwrap after initial install?

Then it wouldn't have any effect because the next person to clone the project (e.g. teammate) will use the regular resolution mechanism. So we can't ship a shrinkwrap unless we actually ask people to use it (which I'm not recommending).

Mmm yeah you're right, the next teammate would potentially have a problem then. Sounds like a yarn-only solution for now if we go through with it. 👍 We can wait to see if anyone else provides some feedback.

Want to send a PR for Yarn-only solution? We can also leave it for contributors if we explain in more detail how it's supposed to work.

I probably wont have time to work on this until the weekend. We could add the help-wanted label for now and I can remove it if no one starts come the weekend (or later)?

Would that get rid of the yarn add react react-dom then? You'd basically copy over yarn.lock (which gets generated / committed from CI after a successful build) and run one yarn install?

Would that get rid of the yarn add react react-dom then? You'd basically copy over yarn.lock (which gets generated / committed from CI after a successful build) and run one yarn install?

Yes. We might want to do it after https://github.com/facebookincubator/create-react-app/pull/1253 which also groups react and react-dom installation together with react-scripts for npm. Can you coordinate with @n3tr on this? I think we could land #1253 first, but maybe it makes more sense to start with it and then add the Yarn thing on top in a single PR.

Per our quick discussion on #1253 - I'll wait until that lands to start the PR for this, if that works.

Closing this as I believe it's pretty inactionable by us.
npm@5 now uses a lock file by default, which should help alleviate this.


Tween onStart does not honor delay (either fired immediately, or not at all) - 2.3.0

When creating a tween with a delay and autoStart set to true, onStart will never be fired.
When autoStart is false and the tween is started by calling the start() method, onStart will be fired immediately and not after the delay has passed. This contradicts the docs.
See http://codepen.io/anon/pen/BNyMPO?editors=001 for an example.

Already fixed in the dev version: https://github.com/photonstorm/phaser/commit/8519c9e39448f29429f12b29bb69a4264cf59fca

Thank you!


bower.json outdated

Hi, i am working to set up an assets management environment including your plugin. Installing it via bower install works. However, your bower.json file is outdated and the reason for grunt to fire the error Fatal error: Can’t detect main file for "bootstrap-modal-carousel" component. You should explicitly define it via bower_concat’s mainFiles option. See Readme for details.Checking your bower.json file i see under"main":`
"docs/css/bootstrap-modal-fullscreen.css",
"docs/css/bootstrap-carousel-fit.css",
"docs/js/bootstrap-carousel-fit.js",
"docs/js/bootstrap-modal-local.js",
"docs/js/bootstrap-modal-fullscreen.js",
"docs/js/bootstrap-modal-carousel.js"

This does not reflect your current repo setup. There is no such directory docs containing these assets. Bower retrieves this structure:
"less/carousel-fit.less",
"less/modal-fullscreen.less",
"js/carousel-fit.js",
"js/modal-carousel.js",
"js/modal-local.js",
"templates/...."

This is obviously not what is defined in bower.json. The script files deployed are also not prefixed with bootstrap- and - most important thing - the modal-fullscreen.js file is missing at all.
Would you please fix that soon?

Hi, thanks for reporting my mistake. I just updated the bower informations with the right libs.

You're welcome. Please tell me, what happened to modal-fullscreen.js?

Bower is just a representation of the packaged lib. If you want the source libs, modal-fulscreen.js in the src/js folder, you have to install the package as a git repo (see http://bower.io/#install-packages).

This is infact what i did when i encountered the first issue for which i opened this issue report. But, check the src folder yourself and see that there is no modal-fullscreen.js file in there. So bower won't check it out at all no matter which way i ask it to install your repo. If the file isn't there at all, it won't find it neither way.

The modal-fullscreen.js is part of 1.x version and the last bower version is 2.0.1.
The new version supports the last version of bootstrap and the new HTML markup of the modal was easier to manipulate in CSS so modal-fullscreen.js became useless.
Which version do you use in your project ?

I switched over to the latest version. However, before i started to set up my assets management environment via bower i always downloaded the *.min.js file from your repo did not watch which files it had merged.

I have to come back to the bower.json issue. I tried to fetch your updated plugin via bower update first and after this has not worked via bower install. However, it always fetches the outdated version. I guess you should update the release number not only in bower.json but probably also in package.json. As you can see you the plugin reports to be v2.0.0. Mabe this is why bower does not load the package with the latest bower.json.

I add to create a new tag for this version. The tags are now updated.


Finish removing support for legacy network protocol

Close #5670
The so-called legacy network protocol was already half-non-functioning. We broke its features over time, such as performing block requests and sending transactions. The reason why it wasn't completely removed earlier is because older versions still open this substream for handshaking purposes, even though they don't make use of it once it's open.
This PR finishes removing it entirely.
The changes were very straight forward. It is purely code removal.
It seems that Rust does not always properly warn about unused code, as it didn't warn me about the upgrade::legacy module being unused even though it was. There might be more old code that can be removed, but I don't know where.

Nothing unusual on the burnin node.

Needs a second approval for merging.

bot merge force

bot merge

bot merge force


Warning message: resemble.outputSettings mutates global state

I get the following output:
resemble.outputSettings mutates global state, and will be removed in a later major version
Can we remove this?

@Kartikeya99 can you look into this?

I found a similar issue https://github.com/rsmbl/Resemble.js/issues/144#issuecomment-405960445 in the 2nd version of resemblejs and it was promised to be fixed upwards the 3.0.0 version.
The tool already uses the 3.0.0 version and so I'm assuming that this issue hasn't yet been fixed and will be fixed in either the 4th or later major versions and it should be safe for us to use until the dependency gets updated to a later version.

My pull request removing the warning just got merged at resemblejs. Now we can update the resemblejs dependency and the warning will not come.

@pablopaul I am closing this issue, if this still happens please feel free to reopen.


[Caddy] allow user to disable server header via the GUI

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[X] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[X] When the request is meant for an existing plugin, I've added its name to the title.

Is your feature request related to a problem? Please describe.
I was configuring Caddy to disable the server header when it was in the community repo but I have not been able to configure Caddy through the plugin to disable the server header.
Describe the solution you'd like
I would like to be able to either check a box to remove the server header or have a way to customize headers in the UI
Describe alternatives you've considered
I have tried using including the options in /usr/local/etc/caddy/caddy.d but it appears this has to be added to each front end

Do you mean this? https://caddyserver.com/docs/caddyfile/directives/reverse_proxy#headers
Wherever possible, I want to adhere to the defaults. https://caddyserver.com/docs/caddyfile/directives/reverse_proxy#defaults
Can you explain the usecase for the header manipulation that can't use these defaults?

Can you explain the usecase for the header manipulation that can't use these defaults?

Not OP, but I had to remove the Authorization header to prevent 400 errors for a Unifi controller. Likely just bad handling on the Unifi side, but as the forum post is 5 years old and I ran into it this year, I doubt it'll be fixed anytime soon.

@justinhorton I'm surprised by that since I have reports of the Unifi controller working. Heres one: https://forum.opnsense.org/index.php?topic=38714.msg190745#msg190745
I guess this kind of jank depends on the unifi controller version/implementation.

When I worked in Web hosting customers requested the header be removed since it is an information disclosure that could help an attacker and I have made a habit of doing this. This an example of of the types of issues customers would report
https://www.tenable.com/plugins/was/98618

Header manipulation won't be easy to template because of all the options it offers, and because there can be multiple header_up and header_down with regex matching and replacement.
Since Caddy uses sane secure defaults, the security risk of exposed headers should be minimal? (I don't know, not a security expert on webservers).
Maybe somebody wants to contribute the headers logic to this plugin. Though since there are options like nginx and haproxy that have specialized on feature richness, maybe they are the better choice for this?

@justinhorton I'm surprised by that since I have reports of the Unifi controller working. Heres one: https://forum.opnsense.org/index.php?topic=38714.msg190745#msg190745
I guess this kind of jank depends on the unifi controller version/implementation.

I just re-tested with my Unifi server (latest version) and it actually has the 400 error message with and without header_up -Authorization and also without Caddy proxying at all. So, I guess this wasn't a valid example and none of my current services need header modifications. Thanks for your work on getting Caddy into OPNsense!
(The "400: Bad Request" message appears very briefly after logging in, before the dashboard loads. So briefly that I had to do a screen recording to discern whether the flash I saw was this message. And also so briefly that I mistakenly thought the header change fixed it in the first place!)

@justinhorton Thank you for the report, I'm glad it can work without header modifications.
If anybody wants to implement it though, it would probably have to be like the basic auth section. It needs an array section in the model and an own bootgrid where all header manipulations can be saved, and then a model relation field inside the handlers where multiple headers can be selected, the templating working kinda like the basic auth example.
It's a bigger addition with intermediate difficulty.

I will take ownership of this issue and implement a flexible header menu, since I have a feeling this will be one of the more often requested features.

Here's a preview of os-caddy-1.5.4 to test the header functionality, any tests are very welcome.
This is a dev version, don't use on mission critical systems
How to install:

opnsense-patch 2d45b78
curl https://github.com/Monviech/opnsense-plugins/blob/os-caddy-1.5.4-pkg/www/caddy/work/pkg/os-caddy-1.5.4.pkg -o os-caddy-1.5.4.pkg
pkg install ./os-caddy-1.5.4.pkg


pull request for e2e test at 12/20/2016 21:45:21

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


What is the use of the TenantObiectId="b6c11183-fc66-xxxx-xxxx-xxxxxxxxxxxx" in the refreshed B2C policies?

@SB-o-matic It returns the current tenantId that is being used. See docs on claim resolver


Implemented cksum

cksum is one of the posix utilities of all time.

The core logic is a bit cryptic (mostly bit manipulation) so I couldn't tell what happening there

hello @cospplredman. thanks for your pr. please test this before we merge it. Also, be sure to provide. if you tasted all the edge cases, mention me so I can merge it. ;)

My testing methodology is as follows: run both my implementation and the gnu implementation on every file in the cksum directory.
$ ./cksum *
175369053 16296 cksum
1855786587 7 cksum.1
4088649404 2179 cksum.c
1192622583 51 Makefile
197436690 157 test.sh
$ cksum *
175369053 16296 cksum
1855786587 7 cksum.1
4088649404 2179 cksum.c
1192622583 51 Makefile
197436690 157 test.sh

the results match as far as i can tell.
One notable difference between this implementation and the gnu implementation is that I do not handle directories, I opt to instead just return EXIT_FAILURE. The reason I prefer to fail is that there isn't a standard C way of checking if something is a directory, so any method of doing so would break me from my (very tasteful) c99 compliance, and cksum is only defined on files anyways.

also pull the latest changes


Import stdout on older D versions too

Thanks!

This is breaking the code.
Probably because you have some static part of the code that using "stdout", while import of the std.stdio now not in the static part

Do you have a pull request that fixes it? The CI passed completely so I'm not sure which part might be affected.


Bump mlflow to v2.14.1

Describe changes
This PR bumps mlflow to the newest version, v2.13.2.
I ran the test pipeline and it works:

Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[x] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] I have based my new branch on develop and the open PR is targeting develop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.
[x] If my changes require changes to the dashboard, these changes are communicated/requested.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Other (add details above)

@strickvl 2.14.0 has been released yesterday. Let me upgrade, test and then adapt the PR.

Ok we'll hold till you check that.

Runs successfully. Adapting the PR now.

@strickvl all is ready for review and tests  :)

@strickvl bumped mlflow to 2.14.1 per their latest release.

@christianversloot thank you so much for your contribution ❤  I will merge this now and run the rest of the test on the release branch. 👍


Feature/metrics assets operator fix

Detailed description:
Remove count metrics & HBAR as bridge account asset metric
Which issue(s) this PR fixes:
Issues #360 #354 #353

Can we remove
	CountAssetMetricNamePrefix           = "count_"
	CountAssetMetricHelpPrefix           = "The count of "

and everything unnecessary in the metrics documentation


Update jsx.d.ts

EventHandlerUnion is missing  EventHandlerWithOptions

Draft, as there's people contributing to this https://discord.com/channels/722131463138705510/783844647528955931/1296957567746048042

closing in favor of https://github.com/ryansolid/dom-expressions/pull/370


Controller is vulnerable to being a confused deputy (without an external ID)

In the current design, the controller creates a Kubernetes Secret object when it notices one or more pods that wish to mount a Secret with the right name. Any pods within that namespace can mount that same Secret object, regardless of governing RBAC permissions, and any pods in any namespace can assume whichever IAM roles trust the controller's initial role. That's a broad entitlement for role delegation.
In kiam, there's a way to limit which IAM roles pod authors can assume within a namespace, but even that doesn't go far enough: It allows the Kubernetes administrator some control over IAM role assumption, but it does not help the IAM roles' owners limit their trust scope. The IAM roles' owners must trust kiam or kube-aws-aim-controller for all the pods in an entire Kubernetes cluster; essentially, the role owners must trust an entire cluster. That's too broad of a scope with a multi-tenant cluster, where an IAM role owner might trust only a namespace, or only particular pods within the namespace.
By forcing the IAM role owners to trust the entire cluster, kube-aws-iam-controller becomes a confused deputy. Pod authors in one namespace may be able to learn the names of Secret objects in another namespace—as some multi-tenant clusters allow inter-namespace observation, if not mutation—and once they have that role name, they too can use it, even if the given IAM role owner only intended pods for one tenant to be able to assume it.
One way around this problem as prescribed by AWS is to use an External ID. When assuming the IAM role nominated by the Secret name suffix, the assuming STS client will include an external ID specific to the requesting party—in this case, a Kubernetes pod—that the IAM role owner can assess in his trust policy for the role being assumed.
Here, kube-aws-iam-controller could supply three pieces of information to the IAM role owner:

pod name
containing namespace
cluster ID (optional)

With these three pieces, an IAM role owner could limit the scope of his trust, down from many Kubernetes clusters—in the case where  kube-aws-iam-controller's initial assumed role is shared among clusters—to a single cluster, further down to a single namespace within a cluster, and even further down to a prefix name of the pods within a namespace.
The external ID syntax would need to preclude false aliasing. We know that neither namespace names nor pod names can contain a forward slash, but we have no idea what constitutes a cluster ID, so we can construct the external ID as
namespace name / pod name [ / cluster ID ]
treating the suffix as optional, only included if kube-aws-iam-controller is configured with a nonempty cluster ID. It may be useful to forbid a cluster ID to include a forward slash character ('/').
With this external ID include in its calls to sts:AssumeRole, kube-aws-iam-controller would allow the IAM role owner to include conditions in its trust policy, using the string condition operators to match expected values:

Constraint
Condition

Within namespace
{"StringLike": {"sts:ExternalId": "trusted-ns/*"}}

Pods with generated name
{"StringLike": {"sts:ExternalId": "trusted-ns/server-*"}}

Within cluster
{"StringLike": {"sts:ExternalId": "*/*/cluster-1"}}

Within specific pod
{"StringLike": {"sts:ExternalId": "trusted-ns/server-0/cluster-1"}}

Note that due to the limited capability of the StringLike operator, the "Within cluster" example above is vulnerable to false aliasing if a cluster ID can contain forward slashes, assuming that the StringLike operator uses greedy matching for wildcards. Likewise, it is difficult to match a pod name prefix of, say, "server-" when another set of pods could use the prefix "server-impostor-."
Some refinement is possible for the external ID format, but I hope this makes the idea clear enough: Without an external ID, kube-aws-iam-controller's trust scope is too broad for some environments, and AWS already has a way of narrowing such a scope.
The main problem I see here is that Kubernetes Secret names are scoped to the containing namespace, so kube-aws-iam-controller can't easily include a pod name in an external ID; any number of pods in the same namespace could mount the Secret, so long as the first pod's projected external ID satisfies the assumed IAM role's trust criteria. Trying to create a Secret that only some pods within a namespace could make use of is complicated, if not impossible. (Perhaps a validating admission control plugin could handle that, though an IAM role's trust criteria could change after the pod is created.) Given that, omitting the pod name and using just the namespace name and optional cluster ID (with the format reduced to namespace name / [ cluster ID ]) would still be an improvement.
I'd like to hear your thoughts on the proposal. In the meantime, thank you for your work on the project!

Thanks a lot for the very detailed feature request!
As you have noticed, until now, this project has solely focused on making a robust alternative to something like kube2iam, but has not touched on topics like multi-tenant clusters.
While I don't have a use case right now, for what you propose, I would be happy to review PRs making this an opt-in feature. :)
I like the idea of using externalID with the namespace and optional clusterID as you propose, however, the pod name part I would leave out because it has the problems you mention and this is more of a general "problem" of how Kubernetes secrets work. I would not try to solve the problem in this project, but rather on a higher level e.g. through an admission controller like you also mention.
Pods are also often dynamic and managed by a controller of some sort, so what you really would want to express is that: "only the pods of this 'application' (e.g. deployment) can read this and that secret", rather than naming the pods explicitly in the access granting feature.

Are you amenable to reviewing a patch that introduces use of an external ID, as described above? I can see a path to introducing it that doesn’t look too invasive to my eye.
Still in need of investigation is how IAM role trust relationship constraint wildcards work. I want to understand that fully before committing to the format for these external IDs, as being able to match their components with wildcards—without falling prey to false aliases—is useful, if not essential.

Yes, I would be happy to review PRs for this. I think it's a good feature, just didn't have any time myself to look at it :)

I have this implemented. It needs further testing with real STS calls to exercise the IAM policy condition operators, and then documentation. Once I'm sure that it works as intended, I'll post the PR, with documentation coming along once we agree on the detailed design choices.

Cool! Looking forward to that!


With ffmpeg crate, Failed to load dynamic library 'hub.dll'

Report

I'm making windows desktop app using ffmpeg crate.
Excately there is "hub.dll" with compiled .exe file
Steps to Reproduce
I added ffmpeg-next = "6.1.1" at \native\hub\Cargo.toml to use ffmpeg.
Cargo.toml
ffmpeg-next = "6.1.1"

when i use ffmpeg-next crate, above problem which load fail hub.dll.
For example just (Example Code for using ffmpeg-next)
extern crate ffmpeg_next as ffmpeg;
let _ = ffmpeg::init();

just remove which using ffmpeg code, the error would disappear.
System Information
rustc --version
rustc 1.76.0 (07dca489a 2024-02-04)

flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.19.2, on Microsoft Windows [Version 10.0.19045.4170], locale ko-KR)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[X] Android toolchain - develop for Android devices
    X Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.7.4)
[!] Android Studio (not installed)
[√] VS Code (version 1.87.2)
[√] Connected device (3 available)
[√] Network resources

! Doctor found issues in 2 categories.

Hi, I'm trying to reproduce the error, but it looks like ffmpeg-next requires some specific system librareis.
  thread 'main' panicked at C:\Users\temed\.cargo\registry\src\index.crates.io-6f17d22bba15001f\ffmpeg-sys-next-6.1.0\build.rs:734:22:
  called `Result::unwrap()` on an `Err` value:
  pkg-config exited with status code 1
  > PKG_CONFIG_PATH=C:\cfg PKG_CONFIG_ALLOW_SYSTEM_LIBS=1 PKG_CONFIG_ALLOW_SYSTEM_CFLAGS=1 pkg-config --libs --cflags libavformat

  The system library `libavformat` required by crate `ffmpeg-sys-next` was not found.
  The file `libavformat.pc` needs to be installed and the PKG_CONFIG_PATH environment variable must contain its parent directory.
  PKG_CONFIG_PATH contains the following:
      - C
      - \cfg

  HINT: you may need to install a package such as libavformat, libavformat-dev or libavformat-devel.

  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

I'm on Windows 11 just like you. Could you describe how you installed ffmpeg system libraries for me?

if vcpkg installed on you, Only need following code.
vcpkg install ffmpeg --triplet=x64-windows
(maybe installing with around 30 min)
After installed ffmpeg to use vcpkg, if appear the following error, you can install MSYS2 and following gcc compiler
Unable to find libclang: "couldn't find any valid shared libraries matching: ['clang.dll', 'libclang.dll'], set the `LIBCLANG_PATH` environment variable to a path where one of these files can be found (invalid: [])"

pacman -S mingw-w64-i686-gcc
pacman -S mingw-w64-x86_64-gcc

I also tried tons of thing to solve installing ffempg problem. it could be not working on your computer.
if not working these solutions, i can say more options to install ffmpeg

I found the solution!!!
the error is generated at load_as.dart>>loadRustLibrary() function.
the function cant find hub.dll, but can't find other dll which related ffmpeg.
/avcodec-60.dll
/avdevice-60.dll
/avfilter-9.dll
/avformat-60.dll
/avutil-58.dll
/swresample-4.dll
/swscale-7.dll

located at (Path to ffmpeg)\ffmpeg_x64-windows\bin
So, you can solve this error by putting the above dll file in the exe file location (e.g. \build\windows\x64\runner\Debug).

Congratulations! Thank you so much for sharing the info :)


Unable to build Dockerfile-alpine: "/tmp/setup_agent.sh: line 477: pushd: not found"

Build Environment:

Ubuntu 16.04 LTS
Docker: 17.05.0-ce, build 89658be

Steps to reproduce:
$ git clone git@github.com:DataDog/docker-dd-agent.git
Cloning into 'docker-dd-agent'...
remote: Counting objects: 2081, done.
remote: Compressing objects: 100% (16/16), done.
remote: Total 2081 (delta 6), reused 9 (delta 2), pack-reused 2063
Receiving objects: 100% (2081/2081), 2.46 MiB | 1.75 MiB/s, done.
Resolving deltas: 100% (1209/1209), done.
Checking connectivity... done.
$ cd docker-dd-agent/
$ docker build -f Dockerfile-alpine -t docker-dd-agent .
Sending build context to Docker daemon  2.808MB
Step 1/17 : FROM alpine:3.6
 ---> e2cd449cde75
Step 2/17 : MAINTAINER Datadog<EMAIL_ADDRESS> ---> Running in 9ca479a43c59
 ---> 413da77f4f79
Removing intermediate container 9ca479a43c59
Step 3/17 : ENV DD_HOME /opt/datadog-agent DD_START_AGENT 0 DOCKER_DD_AGENT yes PYCURL_SSL_LIBRARY openssl AGENT_VERSION 5.21.1 DD_ETC_ROOT "/opt/datadog-agent/agent" PATH "/opt/datadog-agent/venv/bin:/opt/datadog-agent/agent/bin:$PATH" PYTHONPATH "/opt/datadog-agent/agent" DD_CONF_LOG_TO_SYSLOG no NON_LOCAL_TRAFFIC yes DD_SUPERVISOR_DELETE_USER yes DD_CONF_PROCFS_PATH "/host/proc"
 ---> Running in cc59463f6aa8
 ---><PHONE_NUMBER>3b
Removing intermediate container cc59463f6aa8
Step 4/17 : RUN apk add -qU --no-cache coreutils curl curl-dev python-dev tar sysstat tini
 ---> Running in cd161f2a17bb
 ---> 807d276a938d
Removing intermediate container cd161f2a17bb
Step 5/17 : ADD https://raw.githubusercontent.com/DataDog/dd-agent/master/packaging/datadog-agent/source/setup_agent.sh /tmp/setup_agent.sh
Downloading  22.75kB
 ---> abcfea8d8a56
Removing intermediate container e8e0d7b8963c
Step 6/17 : RUN apk add -qU --no-cache -t .build-deps gcc musl-dev postgresql-dev linux-headers     && sh /tmp/setup_agent.sh     && apk del -q .build-deps     && rm /tmp/setup_agent.sh
 ---> Running in 4879d043c272
Checking that logfile is writable
OK
Checking installation requirements
* uname Linux
* sysstat is installed
* python found, using `python2.7`
* downloader found, using `curl -k -L -o`
* sed found, using `sed`

Installing Datadog Agent 5.21.1
Installation is logged at /opt/datadog-agent/ddagent-install.log

* Setting up a python virtual env
Done
* Activating the virtual env
Done
* Setting up setuptools
Done
* Setting up pip
Done
* Installing requirements
Done
* Downloading agent version 5.21.1 from GitHub (~5 MB)
Done
* Uncompressing tarball
Done
* Downloading integrations from GitHub
Done
* Uncompressing tarball
Done
* Setting up integrations
It looks like you hit an issue when trying to install the Datadog agent.
###
Done
* Setting up integrations
/tmp/setup_agent.sh: line 477: pushd: not found
It looks like you hit an issue when trying to install the Datadog agent.
###
###

Do you want to send a failure report to Datadog (Content of the report is in /opt/datadog-agent/ddagent-install.log)? (y/n)
The command '/bin/sh -c apk add -qU --no-cache -t .build-deps gcc musl-dev postgresql-dev linux-headers     && sh /tmp/setup_agent.sh     && apk del -q .build-deps     && rm /tmp/setup_agent.sh' returned a non-zero code: 1
$

While debugging this issue, I was able to solve the issue for me, by updating the Dockerfile-alpine like this:
diff --git a/Dockerfile-alpine b/Dockerfile-alpine
index 0ab2933..4ede1f9 100644
--- a/Dockerfile-alpine
+++ b/Dockerfile-alpine
@@ -23,6 +23,9 @@ RUN apk add -qU --no-cache coreutils curl curl-dev python-dev tar sysstat tini
 ADD https://raw.githubusercontent.com/DataDog/dd-agent/master/packaging/datadog-agent/source/setup_agent.sh /tmp/setup_agent.sh
 RUN apk add -qU --no-cache -t .build-deps gcc musl-dev postgresql-dev linux-headers \
     # Install the agent
+    && sed -i "s:=\$(ls \$DD_HOME/integrations/):=\$(ls -l \$DD_HOME/integrations/ | awk '/^d/ { print \$9 }'):" /tmp/setup_agent.sh \
+    && sed -i "s:pushd $INT_DIR:cd $INT_DIR:" /tmp/setup_agent.sh \
+    && sed -i "s:popd:cd ..:" /tmp/setup_agent.sh \
     && sh /tmp/setup_agent.sh \
     # Clean build dependencies
     && apk del -q .build-deps \

Hi @dirkmoors
It does seem like it indeed. Thank you for bringing this up to our attention!
We'll investigate it further into this in the ticket that was created.
Regards,

@antoinepouille Alright, you're welcome! Keep me posted! :)


Upgrade local(osmosis|relayer) go images and dazel

Closes: #XXX
What is the purpose of the change
Go is bumped to 1.21. Now local testnet uses 1.21 images as well. Changes are:

bump go image to 1.21 in localrelayer and localosmosis
fix dasel related issues. I'm assuming dasel related errors occured because the newer image probably installs a newer version of dasel which contains breaking changes.

Testing and Verifying

localosmosis runs with no issue.
when running localrelayer hermes throws a transport error.  I'm not sure if this error was there in the previous versions. I'm investigating.

Documentation and Release Note

[ ] Does this pull request introduce a new feature or user-facing behavior changes?
[ ] Changelog entry added to Unreleased section of CHANGELOG.md?

Where is the change documented?

[ ] Specification (x/{module}/README.md)
[ ] Osmosis documentation site
[ ] Code comments?
[ ] N/A

closing and reopening to get workflows to run


update projections

Original issue: https://charm.cs.illinois.edu/redmine/issues/637

The last version of projections available for download is 6.5.1 from July 2013.  While it works fine, there are some issues I would hope are resolved by now that a new release would take care of.
Specifically, when tracing messages back through a CkLoop or periodic callback function (which show up as untraced background in projections), causality is assigned to the most recent entry point on the pe rather than, e.g., the origin of the CkLoop call or the entry point that scheduled the periodical callback.

Original date: 2015-03-12 20:17:55

Note that charmplusplus.org currently links to projections 6.2.1.

Original date: 2016-02-15 21:29:05

CkLoop issues fixed in ~~https://charm.cs.illinois.edu/gerrit/#/c/512/~~ https://github.com/UIUC-PPL/charm/commit/2e2760a1a525cb666131f6c5f8214e0b4b136b30.


Broken link in readme

404 in this link provided with the text "this repository"
https://github.com/LiquidGalaxyLAB/reforestation-assistant/blob/main

Fixed on commit 6a0c302a958941ec89fd8b559a04be96f71e9198


[vi] chore: fix broken link in README-vi

Fixed the broken link.
the ones using travis-ci api
[![Build Status](https://api.travis-ci.org/kubernetes/website.svg?branch=master)](https://travis-ci.org/kubernetes/website)

were replaced with netlify link
[![Netlify Status](https://api.netlify.com/api/v1/badges/be93b718-a6df-402a-b4a4-855ba186c97d/deploy-status)](https://app.netlify.com/sites/kubernetes-io-main-staging/deploys)

#41241

/close
Given the inactivity on this PR due to a very low reviewer pool in our Vietnamese docs, and the rebase needed from the author that has yet to be completed, I am closing this PR.


specifying subsets doesn't really work

Hey dude,
I tried specifying fonts and subsets just like Google recommends, however this doesn't work for me, in the sense that ext-latin for example is always loaded on document ready which is too late for me.
The great thing about this library in my opinion is that I can dl the fonts and load them prior to document ready.
Using it just like Google suggests from google fonts :+1:
WebFont.load({
google: {
families: [ 'Lora:400,700,400italic,700italic:latin,latin-ext,cyrillic' ]
}
});
however, all fonts load before document ready except for latin-ext, which only loads on document ready . and I often get a FOUT
http://imgur.com/Gb8UXPI
thank you,
Ilan

BTW,
If I manually specify characters with the text field, I get a custom set which does include all of the characters and it IS loaded early as I wish, but I don't know whether I need to /can specify all (don't even know what is the complete set).
Thanks

Are you using the Google APIs CDN? I was having this problem using that one. Then I realized Google's hosted library is several versions old. I installed locally and it now works much better for me.

The Google module inserts a stylesheet link, which contains external references to font files. These will only be downloaded by the browser if they are needed. This depends on the DOM and CSSOM, so the browser will wait until those are constructed. If you want to download fonts earlier, you'll need to preload them manually.


Fix for #993 LIST_BRANCHES without admin

Replaced the canAccess() method in RpcFilter with !adminRequest,
that should solve the problem.

Looks good.


Duplicates of files/folders with Cache enabled

-->
Describe the bug
If the "Cache Files and Folders for Better Performance" is enabled under experimental features, it duplicates all files and folders within the app. Note: they are not actually duplicating. Opening "File Explorer" shows only one copy of each. So the there is an issue with cache in the "File" app. Also, this so far only seems to impact network attached drives, and does not effect the local "C:" Drive.
To Reproduce
Steps to reproduce the behavior:

Launch App
Click on any network drive
Click on a separate drive or location
Go back to original drive and everything is doubled, then tripled, and so on.

Expected behavior
I expect to not see multiples of files and folders.
Screenshots

Desktop (please complete the following information):

OS Version: [e.g. Windows 10 [19042.685]
App version: [v<IP_ADDRESS>]

Additional context
Closing the app clears out the issue, as does turning off the feature.
Log file
debug.txt

Thanks for the report. The issue has been resolved and the fix will be included in the next version

Thanks for the report. The issue has been resolved and the fix will be included in the next version


Confused Deputy Attack finding for AWS EKS service

Steps to Reproduce
Run scan, inspect finding.
Expected behavior
I want for prowler to detect defects in configuration, and it does an excellent job on this front.  However, I'm getting a "confused deputy" finding on our AWS EKS cluster role.  The remediation guidance suggests following AWS guidance, to include "aws:SourceAccount" and/or "aws:SourceArn" condition keys.  This was not successful.  Opening a ticket with AWS produced language from Omar M.:

You mentioned that you were following the AWS recommendations by adding the “aws:SourceAccount” and “aws:SourceArn” condition keys and that you were still getting the error. I was able to reproduce this similar issue from my end as well and I have check and verified with our internal team that as of now EKS services does not support these conditions to prevent cross-account confused deputy attacks. Also, currently there are no other condition that you can use. It seems that there is currently no way to get around this finding for your use case (with EKS).

I reached out to our service team to provide this feedback but as a support engineer we do not have any ETA when will EKS provide support to these conditions. I was able to find an active feature request to add support for these conditions. I have gone ahead and have added a +1 and have added your case to the request in support of it. Unfortunately, I do not have any ETA for when/if this feature will be released. However, I do recommend keeping an eye on the AWS What’s New page [2] and AWS News Blog [3] for information on new feature releases.

I think it would be helpful to add an explanation to the finding guidance that it is not possible (according to AWS) to clear the finding using the AWS guidance.
Actual Result with Screenshots or Logs

How did you install Prowler?
From pip package (pip install prowler)
Environment Resource
EC2 instance
OS used
RHEL 9
Prowler version
Prowler 4.3.1 (latest is 4.3.3, upgrade for the latest features)
Pip version
pip 21.2.3 from /usr/lib/python3.9/site-packages/pip (python 3.9)
Context
No response

Hi @woodyweaver,
I will add a note in the finding's metadata to indicate that the AWS guidance involving aws:SourceAccount and aws:SourceArn conditions is not applicable to EKS, as confirmed by AWS. This will clarify that there is currently no way to clear the finding for EKS.
In the meantime, you could mute the finding using the Prowler mutelist since it's not remediable for now. I'll make a PR to address this soon. Thanks for your suggestion and for using Prowler! 🚀


update docker setup and add demo data

@jarkenau  What do I need to review here?
@dobro929 The demo data you would have to take a look if you can work with it.

@lucaslootalot I just wanted to check, if your approval would be enough so that I could mege it.

@lucaslootalot i have already checked those in #204.


lasso regression results do not match with scikit-learn or R when lambda is bigger

The lasso regression from breeze does not match with scikit-learn or R (glmnet).
Testing code:

breeze

import breeze.linalg.{norm, DenseMatrix => BDM, DenseVector => BDV}
import breeze.stats.regression.lasso

val a = BDM((1.0, 1.0), (2.0, -2.0), (3.0, 3.0), (4.0, 5.0))
val b = BDV(1.01, 1.98, 3.03, 4.05)
val lambda = 0.5d
val result = lasso(a, b, lambda)
println(result.coefficients)

scikit-learn

import numpy as np
from sklearn import linear_model
X = np.array([[1, 1], [2, -2], [3, 3], [4, 5]])
y = np.array([1.01, 1.98, 3.03, 4.05])
clf = linear_model.Lasso(alpha=0.5, fit_intercept = False)
clf.fit(X, y)
print(clf.coef_)

R

library(glmnet)
X = matrix(c(1, 1, 2, -2, 3, 3, 4, 5), nrow = 4, ncol=2, byrow = TRUE)
y = c(1.01, 1.98, 3.03, 4.05)
coef(glmnet(X, y, alpha=1, lambda=0.5, intercept=TRUE))

Results:

lambda
breeze
scikit-learn
R

0.01
0.9968421052631582, 0.0021
0.99905151, 0.00960668
1.002109388,0.005941413

0.05
0.9586666666666666, 0.0
0.99482704, 0.00832043
1.001213, 0.0

0.1
0.9086666666666666, 0.0
0.98959963, 0.00667717
0.9937595, 0.0

0.5
0.5086666666666666, 0.0
0.942, 0.0
0.9341311, 0.0

1.0
0.0086666666666666, 0.0
0.87533333, 0.0
0.8595955, 0.0

scikit-learn more or less agrees with R, but breeze's results differ quite a lot when lambda is bigger.

In your R example, did you mean intercept=FALSE, consistent with the Python example?

I looked at this, and am not sure what objective function Breeze's lasso is using. I am not familiar with that implementation of Lasso shrinkage, but maybe it's somehow equivalent to the usual form (from scikit-learn)
?


ANDROIDX_VERSION issue

This is amazing, thank you so much for the video and the code.
I was straggling with one thing and I think it can save lots of time to others if you mention this.
The pacakge.json file has this code:
  "cordova.plugins.diagnostic": {
    "ANDROIDX_VERSION": "1.+"
  }

The "+" means it will use the latest version available, in my case that was a newer version than the one you used when recording the video, this created lots of issues and errors (the newer versions have other dependencies).
To fix this the  code should be replaced with:
  "cordova.plugins.diagnostic": {
    "ANDROIDX_VERSION": "1.0.0"
  }

You're very welcome, and glad you found it helpful!
Thanks a lot for mentioning this issue @PlayRobotics.
I have updated it.
On Thu, Jan 27, 2022 at 1:25 PM PlayRobotics @.***>
wrote:

This is amazing, thank you so much for the video and the code.
I was straggling with one thing and I think it can save lots of time to
others if you mention this.
The pacakge.json file has this code:
"cordova.plugins.diagnostic": {
"ANDROIDX_VERSION": "1.+"
}
The "+" means it will use the latest version available, in my case that
was a newer version than the one you used when recording the video, this
created lots of issues and errors (the newer versions have other
dependencies).
To fix this the code should be replaced with:
"cordova.plugins.diagnostic": {
"ANDROIDX_VERSION": "1.0.0"
}
—
Reply to this email directly, view it on GitHub
https://github.com/rmptxf/nrf52_ble_tester_project/issues/1, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/APQS4MTZ4PNBPCQTO4CKERDUYE2U7ANCNFSM5M5YE5OQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thanks again,
another small questions, can you please tell me where I can edit the "intro" screen shown in the screenshot:


fix: typo of wheights

fix typo

 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.


[SPARK-3182][Streaming]: Add geolocation bounding for Twitter Streaming

This PR adds an additional capability to the Twitter Streaming function allowing a user to filter based on a series of latitude-longitude pairs. Currently I've added to the test suite for the Twitter Streaming functionality for both the Java and Scala-based capabilities, updated the Spark documentation to include the lat-lon filter, and provided a lat-lon filter example to further test the functionality given that the test suite cannot accurately test run-time exactness.

Same comment here; we have a bunch of JIRAs and PRs for the same thing. I suggest we converge on SPARK-2788 / https://github.com/apache/spark/pull/1717 as it has the most activity. Could we close this PR?

@srowen I can close this PR if we want to move to #1717. Can you close the JIRA (SPARK-3182) for this though?? Thanks!


QoS Setting for Client Update Request

Recommend updating this document to reflect QoS port configurations for both Hybrid/Online/Teams since the SfB Server documentation points back to this Lync Server 2013 documentation for QoS.
Port range - Modality.
50000-50019 Audio
50020-50039 Video
50040-50059 App/Desktop Sharing
Also the existing cmdlet examples when copied and run have a space between the hypen - and the parameter which need to be removed (this - ClientFileTransferPort 42020 to this -ClientFileTransferPort 42020) for the cmdlet to run properly with copy/paste.

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

ID: e11441ac-62cb-9e72-9069-f7e61c8c02a4
Version Independent ID: 70619c9b-aae7-277c-3d3a-35d82379f19b
Content: Lync Server 2013: Configuring Quality of Service policies for clients running on Windows 7 or Windows 8
Content Source: Lync/LyncServer/lync-server-2013-configuring-quality-of-service-policies-for-clients-running-on-windows-7-or-windows-8.md
GitHub Login: @dstrome2

@greganth1 Thank you for this feedback
I agree this should be updated to reflect the QOS port configurations. I am going to get this over to the Teams/ Skype for Business team so we can get that added and the cmdlet updated as well.
The team will get this prioritized into a sprint plan. Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

Thanks for your feedback and your patience. This content has been updated and published in the 2015 doc set: https://docs.microsoft.com/en-us/SkypeForBusiness/manage/network-management/qos/configuring-port-ranges-for-your-skype-clients#configure-quality-of-service-policies-for-clients-running-on-windows-10


split-changemonitor and split-changemonitorsilent equivalents?

Does this plugin provide the ability to move a single window to a different monitor/workspace on a different monitor?

hyprland can do this out the box with movewindow: bind = SUPER, S, movewindow, mon:+1


Add steps about adding column ContentTitle in the Metadata section

It is now possible to add column Content title and we can add the below steps for that under the steps for ContentDescription in the metadata section as follows:
To add a title column to the document library page, follow these steps:

Follow the initial steps to create a column.
Choose Multiple lines of text.
Name the column ContentTitle.
Add custom title for each item. If no title is supplied, Viva Learning will pick the file name as the title.

Note:The changes will take 24-48 hrs for the change to reflect in Viva Learning.

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

ID: 3055d301-6a49-ab29-5fd0-2bdf1b4076d6
Version Independent ID: 3055d301-6a49-ab29-5fd0-2bdf1b4076d6
Content: Add SharePoint as a learning content source for Microsoft Viva Learning
Content Source: Viva/learning/configure-sharepoint-content-source.md
Service: viva
Sub-service: viva-learning
GitHub Login: @bhaswatic
Microsoft Alias: bhaswatic

If you have a Microsoft alias, you can link your accounts in GitHub so that a blue "Microsoft submitter" label appears on any PRs or issues you submit—guidance on it here: https://docs.opensource.microsoft.com/tools/github/index.html.
Here is the site you can use to see the mappings between GitHub and Microsoft aliases: https://repos.opensource.microsoft.com/people?q=vstanche

Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


[Bug]: Lenra-CI does not return error when pipeline crashes

What happened?
When deploying an application, the lenra-ci rarely crashes but when it happens, it can break your overview page by saying your app is still building.
Link to the pipeline : https://gitlab.com/lenra/platform/lenra-ci/-/jobs/3538148182

Version
No response
Relevant log output
No response

For this we probably add a timeout on the build on the server that say failed after some time.
This may be fix when we use our own runner and get agnostic of GitLab-CI.

May be fixed by lenra-io/server#412


sap.m.Switch should only respond to drag event initiated from within the control area

OpenUI5 version: 1.60.2
Browser/version (+device/version):
Chrome Version 73.0.3642.0 (Official Build) dev (64-bit)
Ubuntu 18.04.1 (Desktop)
Any other tested browsers/devices(OK/FAIL): Firefox 64.0 (FAIL)
URL (minimal example if possible): http://jsfiddle.net/zypA1310/wzjs3g41/
Steps to reproduce the problem:

Create a simple project, default namespace sap.m
Add a SimpleForm with a Label, a Switch and an Input
Run it in any browser
Click the Switch until it's ON (do this at least once)
Click the Input to give it focus
Hold the left mouse button down from within the Input, drag it all the way across the Switch, and then release the left mouse button.

What is the expected result?

Text within the Input will be selected according to the mouse position when the left mouse button is released.
The state of the Switch will not be altered, its appearance should not change for the entire duration of step 6.

What happens instead?

Text within the Input will be selected according to the mouse position when the left mouse button is released.
When the mouse is dragged across the Switch, its appearance is gradually changing, and its state will be changed if the mouse is released at the other side.

Any other information? (attach screenshot if possible)

Hi @zypA13510,
it took a while and we finally managed to make a fix for the reported issue from you.
The issue is now fixed by the change https://github.com/SAP/openui5/commit/e56c725eba5f816af0ab62f753b142e6f4e69207. And will be released with openui5 version 1.77. You can already retest with the openui5 nightly build. https://openui5nightly.hana.ondemand.com/.
Best regards,
Jiawei

I can confirm that the nightly build fixed this. Thanks. 👍


Scroll View Percentage Problem

I have a parent component which is a scrollView inside of it the children component are all percentage, the scroll is not working, is it cause of the percentage? Can we use ScrollView with percentage?

Reason is because percentages are based on the parent container, and in a ScrollView you have the actual container (size of the view port), and then a contentContainer inside (size of all elements) inside that.
You can make it work like this: https://snack.expo.io/@jkcooper/rn24146---scrollview-percentage, but if you want to use things like scrollTo, it's a lot easier just using fixed heights for elements.


LockedComponentunaware oftool` section

I'm trying to enable a component with a "introspection" capability via a factor. My idea was to use the tools section as it's one of the few places in a spin manifest where I have the freedom to pass data. However, inside of the prepare function of the factor, I'm only able to access a selection of a few common fields and even using MetadataKeys, the tools section is not exposed.
Did I overlook something obvious? Could a AppComponent via LockedComponent expose tools directly or via metadata like it does with build?

The tool sections are discarded by spin itself - they are intended to be parsed directly from TOML by external tools.
Could you say more about what you're trying to do?

I'm trying to have a arbitrary data object (toml::Table, serde_json::Value, ...) defined in the manifest and accessible by a factor for a component and by my TriggerApp. The trigger app leverages it to understand the worker configuration for the components. That works well using app().trigger_configs(...). However for the components I haven't found a good way to access some typed data that's component related because the only flexible field in a manifest is tool. So for now the factor looks up the manifest path and loads it again to get the tool information there.
I imagined it could either be something like tool being accessible in PrepareContext.app_component() or something more akin to how the built-in factors use their MetadataKey in app_component).get_metadata(...).
I'm fine with reloading the manifest to get component-level metadata that's specific to my weird usage scenario. I was just wondering if there's a better way to get to the data.

for now I think you're running into an intentional design constraint.

This is what I thought and that's also fine. What I'm doing might fall more into the "custom-runtime-instead-of-spin" world. Yet building onto Spin makes everything so much easier for me as solo dev. I'm leveraging the config format, the whole loading of components (from oci), setting up/configuring of wasmtime and custom imports but everything else is very different. No triggers in that sense, the app is deployed once, runs and exits. A rather weird hybrid of a command trigger where the host environment provides a DAG execution model.


Exclude deleted cards from history

Issue/Bug Description
It could have a heatmap-like button that restricts the math to only existing cards.
I recreated my deck and they are totally egg cards, but they are being considered cards that I no longer have.
Screenshots/Visuals
Main page

When you finish all the cards

Heatmap Parameter

it's just a suggestion. could be a check.

it's just a suggestion. could be a check.

Oh yeah, could see that being useful, actually. Will look into it for the next update, thanks!


Error with modules 'newsfeed' and 'calendar'

I'm not sure if it's even an error.
I've recentrly build my own magic mirror. Everything works fine except for the 2 modules named in the title. When adding the modules to 'config.js' i recieve an error. Im not well known with coding so i would appreciate all tips and hints.
The error:
Uncaught TypeError: Cannot read property 'getElementsByClassName' of null (Main.js:130)
which keeps stacking every second.
Code of 'config.js':
https://ghostbin.com/paste/drcdo
The modules itself are left in their original state.
Link of the website:
http://magic.riksnoeicomputerservice.nl/
I will leave the error on the page for now. If somebody knows what im doing wrong i would like to hear the tips.
Thanks in advance.

Hi,
your url is not a rss feed (http://www.riksnoeicomputerservice.nl) try this one : http://www.tagesschau.de/xml/rss2


你好，不支持Swift吗？

`Dependency "STPickerView" has no shared framework schemes
If you believe this to be an error, please file an issue with the maintainers at https://github.com/STShenzhaoliang/STPickerView/issues/new`

platform :ios, "8.0"
use_frameworks!
target 'TestPicker' do
pod 'STPickerView', '~> 2.1'
end
在Podfile中添加use_frameworks!


eslint-loader should not cache, when process.env.EXTEND_ESLINT === 'true'

Is your proposal related to a problem?

When I change my .eslintrc.js the changes are not reflected in react-scripts start. I found a workaround in #9007 described by @Josema to disable the cache in node_modules/react-scripts/config/webpack.config.js:

this is caused by how react-scripts handles ESLint cache. By default, cache is set to true. If you set it to false, any change to eslint rules will be honoured every time. The setting is in node_modules/react-scripts/config/webpack.config.js

Describe the solution you'd like
Similar to what @Josema wrote (sadly this issue #9007 got closed)

So my suggestion is to make this cache option configurable, and by default set it to false, to avoid confusion.

Currently the webpack.config.js looks like that:
// First, run the linter.
// It's important to do this before Babel processes the JS.
{
  test: /\.(js|mjs|jsx|ts|tsx)$/,
  enforce: 'pre',
  use: [
    {
      options: {
        cache: false,
        formatter: require.resolve('react-dev-utils/eslintFormatter'),
        eslintPath: require.resolve('eslint'),
        resolvePluginsRelativeTo: __dirname,
        // @remove-on-eject-begin
        ignore: isExtendingEslintConfig,
        baseConfig: isExtendingEslintConfig
          ? undefined
          : {
              extends: [require.resolve('eslint-config-react-app')],
            },
        useEslintrc: isExtendingEslintConfig,
        // @remove-on-eject-end
      },
      loader: require.resolve('eslint-loader'),
    },
  ],

I suggest:
cache: !isExtendingEslintConfig,

As it only needs to be disabled when isExtendingEslintConfig is true (process.env.EXTEND_ESLINT === 'true')

Agreed! I'm also having this problem, and additionally would like to turn off caching because I'm using yarn 2 and don't want to have a node_modules directory automatically created just for the cache files.

I like the workaround described here: https://github.com/facebook/create-react-app/issues/9154#issuecomment-662297620 (to delete the eslint cache file). Perhaps this can be automated so that any time the eslint config is updated the cache is deleted. But what if the extended config is updated (eslint-config-react-app). Perhaps there is a proper solution.
Also, let me mention EXTEND_ESLINT so the issue search picks it.

Just wondering if this has been fixed in v4 or not? We're about to migrate to v4 because of various issues we encountered in v3 but not sure if this issue is also included?


An EKS cluster is not imported into Rancher sometimes

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

Create an EKS cluster (specify all the input values)
Create 2 node groups for the cluster.
Import this EKS cluster in rancher.
Cluster is stuck in Waiting state
Phase of the cluster

kubectl get ekscc -n cattle-global-data c-4wclb -o yaml | grep phase
        f:phase: {}
  phase: active

Rancher logs:

2020/08/03 23:11:31 [ERROR] error syncing '_all_': handler user-controllers-controller: failed to start user controllers for cluster c-4wclb: ClusterUnavailable 503: cluster not found, requeuing
2020/08/03 23:11:33 [ERROR] error syncing '_all_': handler user-controllers-controller: failed to start user controllers for cluster c-4wclb: ClusterUnavailable 503: cluster not found, requeuing
2020/08/03 23:11:35 [ERROR] error syncing '_all_': handler user-controllers-controller: failed to start user controllers for cluster c-4wclb: ClusterUnavailable 503: cluster not found, requeuing
2020/08/03 23:11:40 [ERROR] error syncing '_all_': handler user-controllers-controller: failed to start user controllers for cluster c-4wclb: ClusterUnavailable 503: cluster not found, requeuin

Expected Result:
cluster should be imported successfully and should be seen as active
Other details that may be helpful:

Able to import an EKS cluster with 1 nodegroup

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): master-head - latest- commit id: 30e77d238
Installation option (single install/HA): HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Imported EKS
Kubernetes version (use kubectl version):

1.16

This is also seen in an imported EKS cluster with 1 node group.
Cluster is stuck in waiting state.
Logs:
2020/08/04 00:27:11 [ERROR] Failed syncing peers [{<IP_ADDRESS> [<IP_ADDRESS>] true false}]: failed to start user controllers for cluster c-9vw8s: ClusterUnavailable 503: cluster not found
2020/08/04 00:27:11 [INFO] error in remotedialer server [400]: websocket: close 1006 (abnormal closure): unexpected EOF
2020/08/04 00:27:11 [ERROR] Failed to serve peer connection <IP_ADDRESS>: read tcp <IP_ADDRESS>:35150-><IP_ADDRESS>:443: read: connection reset by peer
2020/08/04 00:27:13 [INFO] Adding peer wss://<IP_ADDRESS>/v3/connect, <IP_ADDRESS>
2020/08/04 00:27:13 [INFO] Stopping cluster agent for c-md4wn
2020/08/04 00:27:13 [ERROR] failed to start cluster controllers c-md4wn: context canceled
2020/08/04 00:27:13 [ERROR] Failed syncing peers [{<IP_ADDRESS> [<IP_ADDRESS> <IP_ADDRESS>] true false}]: failed to start user controllers for cluster c-9vw8s: ClusterUnavailable 503: cluster not found
2020/08/04 00:27:13 [INFO] Stopping cluster agent for c-md4wn
2020/08/04 00:27:13 [ERROR] failed to start cluster controllers c-md4wn: context canceled
2020/08/04 00:27:14 [INFO] Handling backend connection request [<IP_ADDRESS>]
2020/08/04 00:27:14 [ERROR] error syncing '_all_': handler user-controllers-controller: failed to start user controllers for cluster c-9vw8s: ClusterUnavailable 503: cluster not found, requeuing

Rancher was unable to communicate with the EKS cluster. The EKS cluster that I observed had private and public access enabled. This means that traffic from the internet can communicate with the cluster and that resources that share a VPC will communicate privately. There were no networking rules to enable this communication. We were able to resolve this by adding the cluster security group to each node of the Rancher management cluster. Adding a rule to allow inbound traffic from a security group associated with the management cluster's nodes to the EKS cluster's security group would likely also work.

Closing this issue - Logging an issue for a Ui doc reference issue - https://github.com/rancher/rancher/issues/28369
Documentation issue is tracked here - https://github.com/rancher/rancher/issues/26930


[BUG] letter-spacing not recognized

Description
I have an SVG with the css style letter-spacing.  This style is not recognized. The png is produced, but without the correct spacing.
Code
The best way to share code for larger projects is a link to a GitHub repository: https://github.com/user/repo/tree/bug-123
But, you can also share a short block of code here:
// some C# code here

You can also share some XAML:
<!-- xaml code here -->

Expected Behavior
I expected to see the png match the SVG.
Actual Behavior
No response
Version of SkiaSharp
2.88.3 (Current)
Last Known Good Version of SkiaSharp
2.88.2 (Previous)
IDE / Editor
Visual Studio (Windows)
Platform / Operating System
All
Platform / Operating System Version
Windows 11
Linux
Devices
Windows 11 laptop
Relevant Screenshots
No response
Relevant Log Output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

What library are you using, can you share your code? Are you using Svg.Skia library or plain SkiaSharp?
This link is not working https://github.com/user/repo/tree/bug-123


'AuthGoogle.get_access_token' was never awaited

[...]/python3.5/site-packages/pogo_async/auth_google.py:72: RuntimeWarning: coroutine 'AuthGoogle.get_access_token' was never awaited
self.get_access_token()

Oops, c6b6ffd should fix that. Thanks.


Validate fails if EquivalentAddins node appears before Hosts node in VersionOverrides

Sample files
manifest-sample-fail.xml.txt
manifest-sample-pass.xml.txt
Using version 0.3.0, this node structure results in a failure:
<VersionOverrides ...>
  <EquivalentAddins>
  </EquivalentAddins>
  <Hosts>
  </Hosts>
  <Resources>
  </Resources>
</VersionOverrides>

Validate output:

Validation: Failed
Error #1:
XML Schema Violation: Your manifest does not adhere to the current set of XML schema definitions for Office Add-in manifests. (link: https://aka.ms/add-in-manifest-schema-violation)

Details: The element 'VersionOverrides' in namespace 'http://schemas.microsoft.com/office/taskpaneappversionoverrides' has invalid child element 'Hosts' in namespace 'http://schemas.microsoft.com/office/taskpaneappversionoverrides'. List of possible elements expected: 'ConnectedServiceControls' in namespace 'http://schemas.microsoft.com/office/taskpaneappversionoverrides' as well as any element in namespace 'http://schemas.microsoft.com/office/taskpaneappversionoverrides/1.1'.

Yet this succeeds:
<VersionOverrides ...>
  <Hosts>
  </Hosts>
  <Resources>
  </Resources>
  <EquivalentAddins>
  </EquivalentAddins>
</VersionOverrides>

If a certain ordering is required for VersionOverrides child elements, it would be helpful to specify that in the VersionOverrides documentation.

See documentation: https://docs.microsoft.com/en-us/office/dev/add-ins/develop/manifest-element-ordering


Padding property causes panel border distortion

Describe the bug
The panel border looses it's original wrap when you apply padding
Steps to reproduce the behavior:

Click on https://function-adminportal-test.shesha.dev/dynamic/Boxfusion.SheshaFunctionalTests.Common/styling-details?id=d0563790-615f-44ac-bb7d-06a8489f63c2
Compare the first panel with the second panel
Actual Results: The side borders of the panel are removed

Expected behavior
The padding should not affect the border of the panel

Related Item

#592

Awaiting for Alex Slavchov to confirm with Ian the current implementation

I'm closing this card as the main item will be modified to include additional requirements: #592


Calling-multiple functions

Hello,
I want to ask please how to call an application which is composed from two functions?? because most of examples in the kubeless directories there is only indications on how to call a function but no in case of having multiple functions?
Thank you.

you can package the kubeless-cli in the application and execute the cli command in that pod. That's one easy way to call, if that's a HTTP trigger, you can configure ingress to trigger.
Note: You could also post this question in slack under kubeless-users chanel.

Can you give me an example please? how we can packege the kubeless-cli??

How are you composing the application? Can you put the docker file please?

Here it is the log of the function:
kubectl logs add-596c47579f-s4gnw
Bottle v0.12.13 server starting up (using CherryPyServer())...
Listening on http://<IP_ADDRESS>:8080/
Hit Ctrl-C to quit.
<IP_ADDRESS> - - [17/Jul/2018:13:22:59 +0000] "GET /healthz HTTP/1.1" 200 2 "" "kube-probe/1.10" 0/255
<IP_ADDRESS> - - [17/Jul/2018:13:23:29 +0000] "GET /healthz HTTP/1.1" 200 2 "" "kube-probe/1.10" 0/156
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/bottle.py", line 862, in _handle
return route.call(**args)
File "/usr/local/lib/python2.7/dist-packages/bottle.py", line 1740, in wrapper
rv = callback(*a, **ka)
File "/kubeless.py", line 76, in handler
raise res
RuntimeError: Working outside of request context.
This typically means that you attempted to use functionality that needed
an active HTTP request.

Sorry, this seems to be a Python related error, probably a runtime error. It seems that your code is trying to access resources that are not available in the request context.