style: adjust pill spacing

Alaska Airlines Pull Request
Adjusts pill based on pixel pushing feedback given during auro-flight review. We are reducing the space between the IATA code and the pill to 4px, and increasing the min-width of the duration pill to 72px.
Type of change:
Please delete options that are not relevant.

[x] Revision of an existing capability

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


update package lock

Fixes #9522

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[x] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).

Codecov Report

Merging #9523 (4eabc53) into main (677c761) will decrease coverage by 0%.
The diff coverage is n/a.

@@         Coverage Diff          @@
##           main   #9523   +/-   ##
====================================
- Coverage    73%     73%   -1%
====================================
  Files       192     192
  Lines      8881    8881
  Branches   1306    1306
====================================
- Hits       6495    6486    -9
- Misses     1871    1877    +6
- Partials    515     518    +3

Impacted Files
Coverage Δ

src/platform/logging/logger.ts
52% <0%> (-18%)
:arrow_down:

src/platform/logging/transports.ts
78% <0%> (-5%)
:arrow_down:

src/platform/errors/sessionDisposedError.ts
100% <0%> (+40%)
:arrow_up:


Benchmarking 2: Merkle Tree inclusion-proof scenario of depth n

Goal is to provide a Merkle inclusion proof of length n
That means we want to prove that e is an element of a Merkle tree. To prove that, we need to know the path on the Merkle tree to the element. The path consists of hashes and either a 0 or another element e_2, at the end.
Implementation
We also discussed that we don’t actually need to build a large Merkle tree — that it is instead sufficient to submit a Merkle path into the VM, and then re-compute the Merkle root using that path.
The path looks like [h(x1), h(x2), ..., e_2] whereas x1, x2, ... are the hashes of different branches of the tree. We don't need to know anything about that other than the hash.
As with hash chaining, we agreed that it makes sense to allow each system to use the hash function(s) of their choice, Miden will go for Rescue Prime.
As with hash chaining, we can pick 3 or so “job sizes.” In this context, the “job size” corresponds to the length of the Merkle path. Miden will use 10, 100, 1000.
So we will create a vector [0, 1, 2, ..., 999] and hash each element using Rescue Prime to get our full Merkle path. Then for each job size we will use only the first n elements.

@bobbinth @itzmeanjan
Can you help me with a small problem?
I have a vector [0, 1, 2, ...], which I want to use as my Merkle path. So I want to apply Rescue Prime to each element of the vector.
let merkle_path_vector_full: Vec<u64> = (0..1000).collect();

I use miden-core v0.3 and import chiplets for the chiplets::hasher::hash_elements(). Alternatively I can use chiplets::hasher::merge().
So now, hash_elements() only accepts 4 Felts.
That means I must transform every vector number into 4 Felts. So 0 as 4 Felts is unclear to me.
Is there a way to simply hash 0, 1, 2, 3, ... using Rescue Prime?

A couple of comments:

I'm not sure checking Merkle paths of large length (i.e., over 256) makes much sense - this will probably never be used in practice. In fact, I would say that in practice Merkle paths of depth greater than 64 will be rarely used. I would re-define the benchmark to pick a Merkle path of some depth (e.g., something like 32) and then $n$ would be the number of Merkle paths we verify. So, for $n=10$ we verify 10 Merkle paths of depth 32, for $n=100$ we verify 100 Merkle paths of depth 32 etc.
A Merkle paths must consists of digests (outputs of a hash functions) - so, we can't use simple number. What you can do is convert each number to an element (via Felt::new()) and then hash this element using hash_elements()). This will give you a digest for each node in the path.
To compute a root of this path, you'll need to hash the elements of the path using merge() function. The order in which you do the merging will define the index of the leaf in the Merkle tree. It may actually make sense to generate a random 32-bit index, and then use it to compute the root of the path.

Thanks a lot, that helps indeed.

Let me discuss that point with Tim and Delendum.
I did it exactly like you described in the end. The problem is that Felt::new() gives me only 1 element, but hash_elements() consumes a Digest as you say. So my solution to create a mocked Merkle path is now, to quadruple the number that I wanted to hash and then create Felts from those four numbers and hash it using hash_elements().

[1000, 999, ...] -> [1000, 1000, 1000, 1000, 999, 999, ...] -> [Felt::new(1000), Felt::new(1000), Felt::new(1000), Felt::new(1000), 999, 999, ...]

I did use merge() and rphash (Assembly) and had to play around with the order a bit. But now it works.

The problem is that Felt::new() gives me only 1 element, but hash_elements() consumes a Digest

hash_elements() takes a slice of field elements (not a Digest) - so, you should be able to do something like hash_elements(&[Felt::new(x)]);

OK, we change the scenario no and re-define the benchmark to pick a Merkle path of some depth (e.g., something like 32), and then n would be the number of Merkle paths we verify. So, for n=10, we verify 10 Merkle paths of depth 32; for n=100, we verify 100 Merkle paths of depth 32 etc. (as @bobbinth suggested)
To discuss a super simple example, for n=1 and depth=2, that means I create a SparseMerkleTree.
        let merkle_leafs_keys: Vec<u64> = (0..4).collect();
        let mut merkle_leafs: Vec<Word> = Vec::new();

        for i in 0..merkle_leafs_keys.len() {
            merkle_leafs.push([Felt::new(merkle_leafs_keys[i]), Felt::ZERO, Felt::ZERO, Felt::ZERO]);
        }

        // Now we create a Sparse Merkle Tree with the leafs we just created
        let smt = AdviceSet::new_sparse_merkle_tree(merkle_leafs_keys, merkle_leafs, 2).unwrap();

Then the scenario is about verifying that node_0 (the node with index 0) is indeed part of the tree.
I understand that in Miden assembly, we can create a simple program to verify the inclusion of a node into a Merkle Tree.
  begin
  # verify a merkle path
      mtree_get
  end

If this is correct so far, I need to create ProgramInputs for that program, hence for stack_init = [depth, index, root of the tree] and the tree must be present in the advice provider, so I can push the tree smt as advice_sets.
Is this correctly understood @bobbinth ?

Yes, that's correct. Maybe one small comment that we also need to check the state of the stack at the end to make sure we got the expected leaf.

Closed by https://github.com/delendum-xyz/zk-benchmarking/pull/24


California State Data

In contact with a few people as per a state-wide address dataset. No returned e-mails since 2/3/2017... But I did find this; perhaps a state-wide parcel dataset for residential areas?
http://services.gis.ca.gov/arcgis/rest/services/Boundaries/Parcels_Residential/MapServer/0

Just got off the phone with john - he cannot share data as per license agreement. He does have a UCD dataset of parcels with only APN field. Although this will not help us, I asked for it anyway ;)
He said several counties in the state are under agreement w third party vendor http://www.parcelquest.com/

Most the remaining small counties in California are locked up in Parcel
Quest, sadly.
On Mon, Mar 6, 2017 at 11:54 AM, Justin Meyers<EMAIL_ADDRESS>wrote:

Just got off the phone with john - he cannot share data as per license
agreement. He does have a UCD dataset of parcels with only APN field.
Although this will not help us, I asked for it anyway ;)
He said several counties in the state are under agreement w third party
vendor http://www.parcelquest.com/
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openaddresses/openaddresses/issues/2596#issuecomment-284512476,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0EO8SF4uUQSKEXGQnfNjrQNsirsna1ks5rjGSBgaJpZM4MNfus
.

no state wide initiative (according to John) for a pacel/ address dataset.

We have almost all of the data in California anyhow.
For completeness on the 1/2 dozen missing counties it might be interesting
to get a list of addresses from the county in question, and then run them
thru Mapzen search to geocode them using the Tiger information. I suspect a
lot of address point data we get from counties start off life that way, and
then is moved onto buildings in a cleanup step. Right now this isn’t
priority.
On Mon, Mar 6, 2017 at 12:01 PM, Justin Meyers<EMAIL_ADDRESS>wrote:

no state wide initiative (according to John) for a pacel/ address dataset.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/openaddresses/openaddresses/issues/2596#issuecomment-284514460,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0EOwFo4PDKc3-cOlNh6JabUSseZiCNks5rjGYIgaJpZM4MNfus
.

@justinelliotmeyers I'm not sure if this data different than the what you linked to on services.gis.ca.gov so just throwing this into the mix. I've come across this statewide parcel boundary dataset before. Unfortunately, it only only contains the geometries and the parcel ID:
https://egis3.lacounty.gov/dataportal/2015/09/11/california-statewide-parcel-boundaries/
That page points to a ~3.1 GB zip file:
http://egis3.lacounty.gov/dataportal/wp-content/uploads/ShapefilePackages/Parcels_CA_2014.zip
@nvkelso I never thought about reverse geocoding to attach known address points to parcels... that's an interesting idea.

(Note that OpenAddresses is looking for primary sourced data, so I wouldn't want us to ingest generated data like geocoded address lists)

@iandees yes, I veered the discussion off course there... didn't mean to imply that was suitable for ingestion into openaddresses, but instead as a potential use of openaddresses' sourced data.

No worries. I didn't mean to halt conversation.

Stale


Zend\Db\ResultSet\ResultSet resets objects on each iteration

I have a TableGateway that is set to give me a custom row object (via a RowGatewayFeature). My custom row extends Zend\Db\RowGateway\RowGateway and adds just one property and its getter and setter (no other logic is included in that class). The problem is that when I iterate through a resultset a second time, my custom property gets reset.
In the following code, $result is an instance of my custom RowGateway class.
EDIT: fixed typo
$resultset = $table->select();
$resultset->buffer();

foreach ($resultset as $result) {
    $result->setMediaFile('foo');
    echo $result->getMediaFile(); // 'foo'
}

foreach ($resultset as $result) {
	echo $result->getMediaFile(); // null
}

This issue has been closed as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html


Request for linking Phone icon to huawei phone app(com.android.incallui)

Link com.android.incallui to @drawable/dialer please. Thank you

Hello! I’ve just linked the dialer icon to the “com.android.incallui” package.

Thank you

Please also link
dialer icon to "com.android.phone" & "com.android.contacts.activities.DialtactsActivity"
Thank you

Done!

Thanks


Feature/165-Recurring donations

Is your feature request related to a problem? Please describe.
Not applicable
Describe the solution you'd like
The register of a partner role needs to be implement it. If you want to do more than one time donations, the partner register screen should be done.
After the register it is done, the partner should be navigate to partner data personal to finish the register steps
Additional context
More information can be seen in figma

Story Points: 8
See the discussion in Frontend S3
Powered by Parabol


chore(deps): upgrade compiler dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-compiler-dependencies-k8s-28-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3838

Questions ?
Please refer to the Backport tool documentation


Allow customers to change number of days on terminate workspaces feature

This feature has been added to the latest (v2.6.0) release.


TypeError: Cannot read property 'id' of undefined

https://sentry.io/organizations/impactasaurus/issues/1466677006/?referrer=github_plugin
TypeError: Cannot read property 'id' of undefined
  at setMeetingState (./src/app/containers/Meeting/index.tsx:131:33)
  at componentDidUpdate (./src/app/containers/Meeting/index.tsx:80:12)
  at Array.forEach (<anonymous>)
  at Array.forEach (<anonymous>)
  at Array.forEach (<anonymous>)
...
(17 additional frame(s) were not displayed)

This occurred when continuing an incomplete record which belonged to a questionnaire with no questions. I believe creating a record to a questionnaire without questions is not possible, but continuing obviously is allowed.


[WIP] Eventing TLS: Pass CA certs to the data plane

Fixes #3118

Proposed Changes
WIP: Needs tests

:gift: Add new feature
Pass CA certs to the data plane

Release Note

Eventing TLS: Pass CA certs to the data plane

Docs

-     (Apache License 2.0) native-linux-x86_64 (com.aayushatharva.brotli4j:native-linux-x86_64:1.8.0 - https://github.com/hyperxpro/Brotli4j/natives/native-linux-x86_64)
+     (Apache License 2.0) native-linux-aarch64 (com.aayushatharva.brotli4j:native-linux-aarch64:1.8.0 - https://github.com/hyperxpro/Brotli4j/natives/native-linux-aarch64)

How do I fix this error, It is probably because I am using ARM64
/cc @matzew @pierDipi @creydr

@pierDipi sorry for the delay on this, I have exams right now, I will be free by Friday evening

Most of the failing tests are because I cant generate getReplyUrl() function in java. It was removed by this https://github.com/knative-sandbox/eventing-kafka-broker/pull/3124#discussion_r1212607715 and I cant regenerate it
/cc @pierDipi

@Vishal-Chdhry I think you might need to run hack/update-codegen.sh https://github.com/Vishal-Chdhry/eventing-kafka-broker/pull/1

Yes, I have actually tried to do that, but it doesn't add anything for some reason. One reason I can think of is that I am on a Mac and bash4 is not directly available on Mac.
So I am using Lima to run a WSL-like Ubuntu setup on my Mac. That might be causing update -codegen to not work properly.
@pierDipi Can you please run it on top of my PR and commit it here, that will be really helpful.

@Vishal-Chdhry yes, that's what I did in this PR: https://github.com/Vishal-Chdhry/eventing-kafka-broker/pull/1 you can merge it and it will be added to this PR

Thank you pier!

/retest-required


add CastImageFilter to support output writing

to solve issue #35

Great, I was about to comment that the output image was of type float and a cast just needed to be added!
Please only make the change to correct the issue and not the other off topic style changes.
Thanks you very much for your contribution.


Fatal Exception: GoogleApiHandler on launch

Having problems starting the app (restructuring branch) on our dev tablet (Android 6.0.1, Kernel 3.14.55), it immediately fails with exception:
org.radarcns.android.android E/AndroidRuntime: FATAL EXCEPTION: GoogleApiHandler
    Process: org.radarcns.android.android, PID: 11930
    java.lang.NoSuchMethodError: No static method zzz(Ljava/lang/Object;)Ljava/lang/Object; in class Lcom/google/android/gms/common/internal/zzaa; or its super classes (declaration of 'com.google.android.gms.common.internal.zzaa' appears in /data/app/org.radarcns.android.android-2/base.apk)
        at com.google.android.gms.measurement.internal.zzx.zzbd(Unknown Source)
        at com.google.android.gms.measurement.AppMeasurement.getInstance(Unknown Source)
        at com.google.android.gms.internal.zzadd.zzaR(Unknown Source)
        at com.google.android.gms.internal.zzadd.zzaS(Unknown Source)
        at com.google.android.gms.internal.zzade$1.zza(Unknown Source)
        at com.google.android.gms.internal.zzade$zzb.zza(Unknown Source)
        at com.google.android.gms.internal.zzade$zzb.zza(Unknown Source)
        at com.google.android.gms.internal.zzzv$zza.zzb(Unknown Source)
        at com.google.android.gms.internal.zzzq$zzb.zza(Unknown Source)
        at com.google.android.gms.internal.zzaap$zza.zzb(Unknown Source)
        at com.google.android.gms.internal.zzaap$zza.zzwb(Unknown Source)
        at com.google.android.gms.internal.zzaap$zza.zzvZ(Unknown Source)
        at com.google.android.gms.internal.zzaap$zza.onConnected(Unknown Source)
        at com.google.android.gms.common.internal.zzl$1.onConnected(Unknown Source)
        at com.google.android.gms.common.internal.zzf$zzj.zzwZ(Unknown Source)
        at com.google.android.gms.common.internal.zzf$zza.zzc(Unknown Source)
        at com.google.android.gms.common.internal.zzf$zza.zzu(Unknown Source)
        at com.google.android.gms.common.internal.zzf$zze.zzxa(Unknown Source)
        at com.google.android.gms.common.internal.zzf$zzd.handleMessage(Unknown Source)
        at android.os.Handler.dispatchMessage(Handler.java:102)
        at android.os.Looper.loop(Looper.java:148)
        at android.os.HandlerThread.run(HandlerThread.java:61)
It works fine on an Udoo Neo (Android 6.0.1, Kernel 3.14.52, No Google Play Services installed), however that is a bit cumbersome for testing things out on.
The app has worked fine on the tablet, before the restructuring changes.
I've googled a bit and found that this error might happen when google play services dependencies aren't the same version throughout the projectm however I have not found a solution.
Also getting an error in app/build.gradle at

compile 'com.android.support:appcompat-v7:22.2.0'

telling me that "All com.android.support libraries must use the exact same version specification..." however building works without a hitch. Not sure if that is related.

@blootsvoets and myself were checking the issue on app/build.gradle. It could be related to some hidden dependencies.

I've added an explicit dependency to firebase 10.0.1 in f29877f, before gradle was complaining that google couldn't detect the firebase version, and this seems to fix that. Could you check if that fixes your issue as well? (not the Android Studio warning, but the exception)
When we go to ambulatory trials, I think we should just move to the latest version of Android for our build target. That would mean that users get notifications based on the permissions that we require, BUT, at least the code is then well supported. There isn't any firebase version available anymore that depends on Android API 22. The oldest version I have available is 9.0.0, which depends on Android API 23.

https://github.com/RADAR-CNS/RADAR-AndroidApplication/commit/f29877f68cf560d69222677b0d8f71c2e6985de5 fixed the exception, thanks!


[zero-copy] elide map_with_state

Make map_with_state non-fallible, and elide it in check mode like map or map_with_span
This may be more iffy, as the user may mutate the state. One option would be to make map_with_state pass in a &S instead of &mut S, to indicate that it is supposed to read-only the state, and force try_map_with_state if you rely on the output.

Hmm, I'm not convinced that this is a good idea, at least by default. Having this fail could potentially result in very surprising behaviour, and it would be difficult for users to determine why it happened, or even search for the problem at all. I would rather that it worked by default. Do you need this for something in particular?

No, I was just considering which places explicitly called Emit, and I noticed this idiosyncracy between this and other maps. Maybe it would be better to add some map_with_state_pure which behaves like this? Not including it at all is fine, I was just looking for a chance to improve efficiency. Even without the emission change, I think the signature change on its own is good, and that lack of &mut may actually be a bug.

Updated to only include the signature change

Thanks!


API Methods insert()

Inserting a html string with whitespaces and/or newline can cause failure in creating the new element.
I have not tested if it is whitespace or newline characters, but stripping them both out solved my issues.
Workaround: create a jQuery element from string, then insert jQuery element,

Fixed in https://github.com/patrickkunka/mixitup/releases/tag/v3.2.2.
Whitespace is now trimmed from all HTML strings sent to .insert()


Add package description

Package.json is missing description and link to repository, therefore, there is no direct link from npm and package description looks crazy

@lundeeell it's fixed now! will be included in the next release


Reselect Dart2 xxx  is not a subtype of type '(R1) => T' of 'func'

Hello,
i have an issue where everytime i try to use reselect with a StoreConnector i get the following error:
I/flutter ( 7219): The following assertion was thrown building StoreConnector<AppState, List<LibraryMovie>>:
I/flutter ( 7219): type '(List<LibraryMovie>) => List<LibraryMovie>' is not a subtype of type '(R1) => T' of 'func'
I/flutter ( 7219):
I/flutter ( 7219): Either the assertion indicates an error in the framework itself, or we should provide substantially
I/flutter ( 7219): more information in this error message to help you determine and fix the underlying cause.
I/flutter ( 7219): In either case, please report this assertion by filing a bug on GitHub:
I/flutter ( 7219):   https://github.com/flutter/flutter/issues/new
I/flutter ( 7219):
I/flutter ( 7219): When the exception was thrown, this was the stack:
I/flutter ( 7219): #0      createSelector1 (package:reselect/reselect.dart:235:27)
I/flutter ( 7219): #1      moviesIs4KSelector (file:///C:/xxx/selectors.dart:12:65)

But i followed the Readme (adding types only):
Widget connection part:
return new StoreConnector<AppState, Iterable<LibraryMovie>>(
        converter: (store) => moviesIs4KSelector(store.state),
        builder: (context, movies) {
          return new GridView.count(
...

Selectors:
typedef R Func1<A, R>(A a);

final Func1<AppState, MoviesState> moviesStateSelector = (AppState state) => state.moviesState;
final Func1<AppState, Iterable<Library>> librariesSelector = (AppState state) => moviesStateSelector(state).libraries;
final Func1<AppState, List<LibraryMovie>> moviesSelector = (AppState state) => librariesSelector(state).expand((f) => f.folders).expand((m) => m.movies).toList();

final  Func1<AppState, List<LibraryMovie>> moviesIs4KSelector = createSelector1<AppState, List<LibraryMovie>, List<LibraryMovie>>(
    moviesSelector,
    (products) => products
    .where((m) => m.video== null ? false : m.video.quality == "1080p").toList(),
);

If however i don't use reselect but a simple selector everything works fine:
final Func1<AppState, Iterable<LibraryMovie>> moviesIs4KSelector = (AppState state) => moviesSelector(state).where((m) => m.video== null ? false : m.video.quality == "1080p");
Any idea where this is coming from?
thanks in advance for your help!

I am seeing a very similar issue also with Dart 2.
The following assertion was thrown building StoreConnector<AppState, AppointmentListViewModel>:
type '(List, List) => List' is not a
subtype of type '(R1, R2) => T' of 'func'
The code looks like
final Selector<AppState, List<AppointmentModel>> appointmentsSelector = (AppState state) => state.appointments;

final Selector<AppState, List<ClientModel>> clientsSelector = (AppState state) => state.clients;

final Selector<AppState, List<ApppointmentWithUserModel>> appointmentsWithUserSelector =
  createSelector2(appointmentsSelector, clientsSelector, (List<AppointmentModel> appointments, List<ClientModel> clients) {
      final List<ApppointmentWithUserModel> list = new List<ApppointmentWithUserModel>(appointments.length);

      return list;
    }
  );

I can call appointmentsSelector and clientsSelector individually and they work fine, but combining the two raises the above error.
Is this a Dart 2 issue?

Hey all, sorry about that trouble! This appears to be a Dart 2 issue. I'll try to patch it up in the next couple of days.

And, actually just made some time to hopefully fix it :) Could you please try the following version of the lib and see if it solves the problems you've encountered:
dependencies:
  reselect:
    git:
      url: git://github.com/brianegan/reselect_dart.git
      ref: dart-2

If it works out, I'll publish it as a new release.

Seems to have fixed it for me - thanks!!

I still have the issue with this dumbed down selector:
final  Func1<AppState, Iterable<LibraryMovie>> distinctMoviesSelector = createSelector1<AppState, Iterable<LibraryMovie>, Iterable<LibraryMovie>>(
    moviesSelector,
    (movies) => movies
);

I/flutter (11360): The following assertion was thrown building StoreConnector<AppState, Iterable<LibraryMovie>>:
I/flutter (11360): type '(Iterable<LibraryMovie>) => Iterable<LibraryMovie>' is not a subtype of type '(R1) => T' of
I/flutter (11360): 'func'
I/flutter (11360):
I/flutter (11360): Either the assertion indicates an error in the framework itself, or we should provide substantially
I/flutter (11360): more information in this error message to help you determine and fix the underlying cause.
I/flutter (11360): In either case, please report this assertion by filing a bug on GitHub:
I/flutter (11360):   https://github.com/flutter/flutter/issues/new
I/flutter (11360):
I/flutter (11360): When the exception was thrown, this was the stack:
I/flutter (11360): #0      createSelector1 (package:reselect/reselect.dart:235:27)
I/flutter (11360): #1      distinctMoviesSelector (file:///C:/Devs/rs-flutter/redseat/lib/selectors.dart:23:73)

Package being: C:\Devs\flutter.pub-cache\git*reselect_dart-349bf201a2e3bf54fb2fc5579ce440516b4aea14*\lib\reselect.dart

@neckaros Hrm, dang... Would you mind providing a bit more code or be willing to share your project? I'm no longer seeing that error with my test cases, but perhaps I missed something.

Not too sure if its relevant but I would also avoid the typedef as well - typedef R Func1<A, R>(A a);
Also the createSelector1 function is taking 3 parameters instead of 2 - is this correct?
As input it should take a selector which is Iterable and not two of them.
final  Func1<AppState, Iterable<LibraryMovie>> distinctMoviesSelector = createSelector1<AppState, Iterable<LibraryMovie>, Iterable<LibraryMovie>>(

Yep, it requires 3 types since you've got 2 functions! The flow works like this:
Your selector function goes from S -> R1, the combiner function goes from R1 -> T. Therefore, you've got 3 types: S, R1, T.
If you don't need a cominber function, you could consider simply memoizing your function like so:
final memoizedSelector = memo1<S, T>(selector)
memo1 comes from the memoize package that reselect depends on.

@neckaros Were you able to play with this any more and find the issue? I'd like to release a new version for Dart 2, and I think the branch I've got up should do the trick, but want to be sure what I've done fixes the problem :)

I will give it another try and get back to you
i should use:
git:
      url: git://github.com/brianegan/reselect_dart.git
      ref: dart-2

?

@brianegan Hi! I'm migrating the project to dart 2.0 now and encountered the same problem. I see there is a fix merged in master but why didn't you publish this to the pub? I have to use an override to git branch because of this, could you publish fix?

Oh dang, that's my bad -- totally thought I'd pushed it! Now published as 0.3.0. Thanks for the heads up.


add analysis tools first click and path history

Two features added by blindguradian50 (Keiths IA Group 2022).

Please resolve merge conflicts before submitting a pull request

These changes are redundant and are already merged into upstream.


unixPB: Install Python2.7 on C6; Use it to install JDKs

Fixes: #1877
IT WORKS! (in VPC at least: https://ci.adoptopenjdk.net/job/VagrantPlaybookCheck/1067/ )
Installs Python 2.7 on CentOS6 and then uses that Python to run the adoptopenjdk_install tasks. See the referenced issue for what other things were tried or suggested. This solution also adds some future proofing somewhat, in that we can just use the Python2.7 whenever we have another unarchive / get_url cert issue because CentOS6 is old.
Checklist

[x] commit message has one of the standard prefixes
[ ] FAQ.md updated if appropriate
[ ] other documentation is changed or added (if applicable)
[x] playbook changes run through VPC or QPC (if you have access) : See above
[ ] for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

How long does this role to execute? Could be preferable to use the pre-built one I put in place last week instead which would also avoid having to install the extra prereq packages on the machine.

About 10 minutes in VPC , not sure on the GHCheck as it doesn't have timestamps. But yeah, okay, I'll try prebuilt :-)

Well it works! :tada:


Binance alternative urls for better performance

Hi!
Have you considered to support alternative binance api urls?
Like
API_URL = 'https://apiX.binance.{}/api'
where X belongs to ["", "1", "2", "3"]
Here is the example on how it works for me:
cmach@cmach1:~$ ping api.binance.com
PING d3h36i1mno13q3.cloudfront.net (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=1 ttl=237 time=99.9 ms
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=2 ttl=237 time=100.0 ms
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=3 ttl=237 time=100 ms
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=4 ttl=237 time=100.0 ms
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=5 ttl=237 time=100 ms
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=6 ttl=237 time=100.0 ms
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=7 ttl=237 time=99.9 ms
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=8 ttl=237 time=99.9 ms
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=9 ttl=237 time=100.0 ms
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=10 ttl=237 time=100 ms
^C
--- d3h36i1mno13q3.cloudfront.net ping statistics ---
10 packets transmitted, 10 received, 0% packet loss, time 9010ms
rtt min/avg/max/mdev = 99.911/99.965/100.016/0.034 ms
cmach@cmach1:~$ ping api3.binance.com
PING d1rtiksidvgi3p.cloudfront.net (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from server-52-84-226-14.sin2.r.cloudfront.net (<IP_ADDRESS>): icmp_seq=1 ttl=247 time=0.894 ms
64 bytes from server-52-84-226-14.sin2.r.cloudfront.net (<IP_ADDRESS>): icmp_seq=2 ttl=247 time=0.473 ms
64 bytes from server-52-84-226-14.sin2.r.cloudfront.net (<IP_ADDRESS>): icmp_seq=3 ttl=247 time=0.380 ms
64 bytes from server-52-84-226-14.sin2.r.cloudfront.net (<IP_ADDRESS>): icmp_seq=4 ttl=247 time=0.362 ms
64 bytes from server-52-84-226-14.sin2.r.cloudfront.net (<IP_ADDRESS>): icmp_seq=5 ttl=247 time=0.491 ms
64 bytes from server-52-84-226-14.sin2.r.cloudfront.net (<IP_ADDRESS>): icmp_seq=6 ttl=247 time=0.385 ms
^C
--- d1rtiksidvgi3p.cloudfront.net ping statistics ---
6 packets transmitted, 6 received, 0% packet loss, time 5085ms
rtt min/avg/max/mdev = 0.362/0.497/0.894/0.183 ms
I have tried from several locations, including Singapore, Tokyo and Amsterdam) and every time alternative urls work much faster.

+1

Does this work for websockets too?

+1
and what is difference v1 and v3?

its not a version, they are different sites

I know that is different sites. I mean api version. when I check the official binance docs, I see api v1, v2, v3

+1
and what is difference v1 and v3?

your question does not relate to this topic.
Though, as far as I remember, v3 is for spot trading  (api moved on v3 two years ago from v1). So so its just a matter of version number.

If you want to improve your ping, use local instances and run your execution there.
I did extensive testing and here are the results.
It looks like they have local servers in Tokyo and New Jersey.
So you can get as low as 10ms including already the calculation from the server. It's not just ping. 100ms for just a ping is definitely too high.

Tokyo - 0.01s*
Seul - 0.042s*
New Jersey - 0.01s*
Los Angeles - 0.125s*
Frankfurt - 0.245s*
London - 0.24s*
Singapore - 0.095s*
Sydney - 0.126s*

time for price request
Using this server provider: https://www.vultr.com/?ref=8815565-6G. You get 100$ credits with them. Much simplier to use than AWS.


knot-resolver: fix cpu usage in kres-gc-cache

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@vcunat ping me when fix hits the knot-resolver repository.

Note for others: so far noticeable issues have only been confirmed for encrypted ZFS, not for what I'd call "usual configurations".

I'm not sure about 20.03.  I didn't push anything in there (for now, at least).

Probably should backport, imo.

Agreed.

I normally backport minor releases of knot-resolver, so this would probably get in anyway.  The change is (slightly?) beneficial to all users that have the cache on a non-tmpfs, not just ZFS.

Pushed to 20.03 as well.


Igapp 383 additional field for feedback contact mail

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword IGAPP.
Feedback-View in the webapp:

Feedback in CMS:

I updated the tests but for some reason flow declares an error for on file:
Error ----------------------------------------------- src/modules/feedback/components/tests/FeedbackBox.spec.js:25:7
Cannot create FeedbackBox element because property onContactMailChanged is missing in props [1] but exists in
PropsType [2].
src/modules/feedback/components/tests/FeedbackBox.spec.js:25:7
v-----------
25|       <FeedbackBox
26|         isPositiveRatingSelected={false}
27|         comment=''
29|         selectedFeedbackOption={feedbackOptions[0]}
30|         onCommentChanged={onCommentChanged}
31|         onFeedbackOptionChanged={onFeedbackOptionChanged}
32|         onSubmit={onSubmit}
33|         sendingStatus='SUCCESS'
34|         t={t}
35|         closeFeedbackModal={() => {}}
36|         theme={theme} />
---------------^ [1]
Can someone help me with this issue?

I updated the tests but for some reason flow declares an error for on file:
Error ----------------------------------------------- src/modules/feedback/components/tests/FeedbackBox.spec.js:25:7
Cannot create FeedbackBox element because property onContactMailChanged is missing in props [1] but exists in
PropsType [2].
src/modules/feedback/components/tests/FeedbackBox.spec.js:25:7
v-----------
25| <FeedbackBox
26| isPositiveRatingSelected={false}
27| comment=''
29| selectedFeedbackOption={feedbackOptions[0]}
30| onCommentChanged={onCommentChanged}
31| onFeedbackOptionChanged={onFeedbackOptionChanged}
32| onSubmit={onSubmit}
33| sendingStatus='SUCCESS'
34| t={t}
35| closeFeedbackModal={() => {}}
36| theme={theme} />
---------------^ [1]
Can someone help me with this issue?

You need to pass the onContactMailChanged prop to fix the flow error.

Tested on firefox, looking good 👍 You could look into whether it makes sense to generalize FeedbackComment as @Taggotty suggested, but I fear it does not make sense as one is a TextInput whereas the other is a TextArea. Your choice I'd say @Cmd190

Tested on firefox, looking good 👍 You could look into whether it makes sense to generalize FeedbackComment as @Taggotty suggested, but I fear it does not make sense as one is a TextInput whereas the other is a TextArea. Your choice I'd say @Cmd190

Yes, that's the point. I'll leave it as it is.


add coase.rst in main lecture_source_py

thanks @AnjuJoon -- @jstac I will wait for your review before merging this. Cheers.

@jstac this is the pull request.

@AnjuJoon Thanks Anju, much appreciated.  Did you see my email about the need to get the latest from the master branch of the sandpit, where I fixed the red symbols?  Does this PR include that fix?

@jstac  Sorry, I haven't fixed the red symbols yet. I will take the latest version from sandpit in a while and will update you. Also in which index file should I add this lecture?

Thanks @AnjuJoon .
Please add it to the "Multi-agent Models" section and in the graduate course but not the undergraduate course.

@jstac I have made the required changes. Please let me know if more changes are needed. Thanks

Great work, thanks @AnjuJoon!


Extend job to upload schema artifact to release

Adds a workflow that generates and upload the openapi.yml file to a new release after it is published.
Fixes https://github.com/snypy/snypy-backend/issues/77

Codecov Report

Merging #79 (116c4c7) into master (4f51ec4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #79   +/-   ##
=======================================
  Coverage   94.23%   94.23%
=======================================
  Files          60       60
  Lines        1508     1508
=======================================
  Hits         1421     1421
  Misses         87       87

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Add license to README to make it consistent with other PWA libraries.

Ping!

I changed the year.


Wrap "ERR max number of clients reached" to JedisConnectionException

Maybe we may want to convert JedisDataException which message is "ERR max number of clients reached" to JedisConnectionException since Redis will close the socket immediately.

@HeartSaVioR let's continue discussion in #1036


A required package for installing Kong on Ubuntu is undocumented

Summary
This requirement isn't listed in the documentation, so I'm not sure if this was intentional.  I'd prefer if it weren't required since it would make it much easier to use this package when creating a docker container.  If sudo is actually needed a recommended package can be used instead like gosu.  If the requirement is going to be left in place the documentation should be updated.
Steps To Reproduce

Download the .deb package
Attempt to install it without the sudo package installed

Additional Details & Logs

Kong version (0.9.3)
Kong error logs

Selecting previously unselected package kong.
(Reading database ... 9941 files and directories currently installed.)
Preparing to unpack kong-0.9.3.deb ...
Unpacking kong (0.9.3) ...
dpkg: dependency problems prevent configuration of kong:
 kong depends on sudo; however:
  Package sudo is not installed.

dpkg: error processing package kong (--install):
 dependency problems - leaving unconfigured
Errors were encountered while processing:
 kong

Operating System Ubuntu Xenial

This could be a legacy dependency - let me double check.

I have updated the distributions for 0.9.3 - can you please download it again?

@thefosk, it looks like that did the trick.  Thank you for the quick turn around, I appreciate it.


Final polishing on #90

ACs

Update CHANGELOG to reflect UI additions
CSS cleanup to fit in with our guidelines
Dicrease padding-left in filterbox
Search icon is a tad too small
A subtle hover effect on the “x” would be nice
If there are no results for the keyword filter, some written feedback in the apps list area would help for communication
ESC key should reset the filter

After merging: https://github.com/mesosphere/marathon-ui/pull/90

I am a bit concerned about performance. Shouldn't we throttle the user keystrokes?
I can see a huge performance hit when the list grows to larger numbers (we are triggering a re-render on each keypress here).

Yeah, this is a good point. As long as we keep the debounce rate low enough to make it feel snappy, I'm in favour.


Relative imports in python?

Is there any interest in adjusting the code for Python so it can handle relative imports? It would be great if the plugin could link back to files within the current repo. Ideally recognizing the follow constructions:
from . import module
from .module import object
from ..parent_module import object
import .module
from packagename import module # I recognize this one would be trickier

Hi @JordanReiter, thanks for the report!

from . import module
from .module import object
import .module

I think these cases are already covered by the isLocalFile/isInit logic in the python plugin. I might be wrong about this though, so if you find a counter-example, please post it here.

from ..parent_module import object

This one probably wouldn't be too hard to support, by extending the logic linked above. Would you like to try it? I'd be happy to answer any questions you have about the development process, or help out with bugs along the way.

from packagename import module # I recognize this one would be trickier

Yeah, this one definitely would be trickier, especially since we don't know whether packagename refers to another file in the repository, or an external package from e.g. PyPI. I'm a little rusty on exactly how Python imports get resolved, but from the documentation, it sounds like we should check for a package in the current directory, then fallback to PyPI. If you'd like to tackle this one as well, I'd again be happy to help.

Awesome, I'd love to give it a shot.
On May 4, 2017 5:52 PM, "Joseph Frazier"<EMAIL_ADDRESS>wrote:

Hi @JordanReiter https://github.com/JordanReiter, thanks for the report!
from . import modulefrom .module import objectimport .module
I think these cases are already covered by the isLocalFile/isInit logic
in the python plugin
https://github.com/OctoLinker/browser-extension/blob/a9aff9125c1ef3f54b42db74a546b27f7a181c98/lib/plugins/python.js#L9-L25.
I might be wrong about this though, so if you find a counter-example,
please post it here.
from ..parent_module import object
This one probably wouldn't be too hard to support, by extending the logic
linked above. Would you like to try it? I'd be happy to answer any
questions you have about the development process, or help out with bugs
along the way.
from packagename import module # I recognize this one would be trickier
Yeah, this one definitely would be trickier, especially since we don't
know whether packagename refers to another file in the repository, or an
external package from e.g. PyPI. I'm a little rusty on exactly how Python
imports get resolved, but from the documentation
https://docs.python.org/3/tutorial/modules.html#the-module-search-path,
it sounds like we should check for a package in the current directory, then
fallback to PyPI. If you'd like to tackle this one as well, I'd again be
happy to help.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OctoLinker/browser-extension/issues/357#issuecomment-299319789,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAN-TjvpENk0ab8Rk_TeKR7-RpcAp9p5ks5r2ki5gaJpZM4NPkjm
.

That's great to hear. The Contributing Guide should get you up-and-running. Note that you'll need the Yarn package manager to install all the dependencies properly. Feel free to ask any questions that come up!


Waiting for gc IP timeout

It is hoped that the IP of the pod will be recycled from the IP pool within the expected time.
Copy-paste the link of the CI build where that test failure has happen.
https://github.com/spidernet-io/spiderpool/actions/runs/5181484876/jobs/9337328015
Upload the zip file generated from that test failure.
ipv4Only-debuglog.txt.zip
Copy-paste the output the test failure.
  [FAILED] Expected success, but got an error:
      <*errors.errorString | 0xc000adb670>:
      time out to wait ip reclaimed finish
      {
          s: "time out to wait ip reclaimed finish",
      }
  In [It] at: /home/runner/work/spiderpool/spiderpool/test/e2e/reliability/reliability_test.go:157 ****

We can make the IP-GC scan-all frequently to release the leak IPs


Update README.md: optional clipboard feature not necessary anymore.

current issue:

using "fltk-bundled" feature not work on x64-windows-msvc due to rust-lld: error: undefined symbol: Fl_compose,  Fl_compose_reset,  Fl_compose_state,  Fl_reset_marked_text,  Fl_insertion_point_location,  Fl_copy,  Fl_add_clipboard_notify and  Fl_remove_clipboard_notify. (something ain't right inside lib_x64-windows-msvc.tar.gz, will be fixed on the next update maybe?).

anyway, on linux working perfectly fine. great work thank you @MoAlyousef :)

Thank you.
I've rechecked the lib_x64-windows-msvc.tar.gz and it appears to work. I can still create a new one. But just to check, can you run cargo clean && cargo build?

"cargo clean && cargo build?" yup, @MoAlyousef linking error still appear on my windows (even using default linker cl.exe) , i've no idea why.

Hmm I’ll look into it and upload a new archive as well.

I've uploaded a new archive. I've also publish a new version of fltk-egui with a feature fltk-bundled as well. Those missing symbols are in the most recent version of fltk-sys, so I think it's cargo that's probably resolving different versions of fltk.


Base os image file name mismatches in ubuntu-xenial.meta4 and Rakefile

I'm building ubuntu xenial stemcell 456.40 for SoftLayer and found the following issue.
The file name of base os image specified in https://github.com/cloudfoundry/bosh-linux-stemcell-builder/blob/ubuntu-xenial/v456.40/bosh-stemcell/image-metalinks/ubuntu-xenial.meta4#L2 has been changed to ubuntu-xenial.tgz from previous bosh-ubuntu-xenial-os-image.tgz. But in the Rakefile https://github.com/cloudfoundry/bosh-linux-stemcell-builder/blob/ubuntu-xenial/v456.40/Rakefile#L61, the resolved meta4 download filename specified after --file is still bosh-ubuntu-xenial-os-image.tgz which makes the download failed due to error File does not exist.
Would you please fix the issue? Thanks.

Any update?

I pushed a fix for this today.  Let us know if you still have issues.


Allow selection of the default beacon in the Settings pane

Is your feature request related to a problem? Please describe.
I'm at the point in my Space Exploration run where I'd like to select the Wide area beacon as my default beacon in the Settings pane.
Describe the solution you'd like
I think the best solution is to make the non-clickable beacon image on the Settings pane into a clickable button, like the factory-selection and module-selection buttons.
Describe alternatives you've considered
The workaround is for me to simply select the Wide area beacon for every recipe after setting up my desired products.

This sounds more like a bug because as far as I know it should be configurable in settings like it is in the products list.


Document preference for Put(, )

See #492

We no longer need to document this.  We automatically set timestamp if it's not set already.

This is obsolete.  Closing this.


When a keyref'ed element is directly flagged by DITAVAL, its contents disappear

Expected Behavior
If I define a myvar variable key in my map:
<keydef keys="myvar">
    <topicmeta>
      <keywords>
        <keyword>***SOME VALUE***</keyword>
      </keywords>
    </topicmeta>
  </keydef>

then apply @rev to an instance of this keyref'ed variable in a topic:

"myvar" is set to <ph keyref="myvar" rev="1.0"/>
then publish the map with a DITAVAL that applies flagging to @rev:
<val>

   <revprop action="flag" backcolor="#FEFEE1"/>
</val>

then I expect the keyref'ed value to be both shown and highlighted in all transformation formats.
Actual Behavior
Somehow the keyref'ed contents disappear if the element is also directly flagged by a DITAVAL. This occurs in all transformation formats. This appears to happen during preprocessing, as the normalized DITA from the DITA-OT demonstrates the issue:

"myvar" is set to <ph keyref="myvar" rev="1.0"><ditaval-startprop style="background-color:#FEFEE1;"><revprop action="flag" backcolor="#FEFEE1"/></ditaval-startprop><ditaval-endprop><revprop action="flag" backcolor="#FEFEE1"/></ditaval-endprop></ph>.

Although plenty of additional flagging elements and attributes added, the value ***SOME VALUE*** is gone.
If I publish the map without DITAVAL flagging, the substituted value is kept in the normalized DITA:

"myvar" is set to <ph keyref="myvar" rev="1.0">***SOME VALUE***</ph> (with @rev).

As this bug occurs in preprocessing, it affects all transformation formats.
Steps to Reproduce
Testcase here:
ditaot_flagged_keyref.zip
To demonstrate the bug,
dita -i map.ditamap -f dita --filter flag_rev.ditaval
To demonstrate that the bug only occurs when DITAVAL flagging is applied,
dita -i map.ditamap -f dita
Environment

DITA-OT version: 3.6
Operating system and version: Ubuntu Linux 18.04 LTS, running in Windows 10 via WSL
How did you run DITA-OT? dita command
Transformation type: all

This seems to work properly with DITA OT 3.5.4.

I think the problem is that the "ProfilingFilter.startElement(String, String, String, Attributes)" adds extra flagging elements inside the ph element which has a keyref. And later on the module which needs to expand the keyref looks at the "ph" element which is not empty anymore (has the flagging elements inside) so it decides to use the existing content of the "ph" instead of the keyref.

@raducoravu - thanks for the quick analysis! We will use the suggested workaround.


[Fabric]: Remove cxx from iOS project

Summary
Remove cxx from iOS project, because we have the corresponding implementation in iOS. Otherwise, it would break the logical handling of iOS part.
Changelog
[iOS] [Fixed] - Remove cxx from iOS project
Test Plan
Fabric enabled Tester can work like loading image.

Those path fragments in exclude_files (sic!) sections.

@shergin ? emm, I don't get what your point.

Oops, seems I was very wrong about that. The changes actually make a lot of sense: we don't need cxx and android folders to compile those targets on iOS.


vendor:publish Tag input is string explode it seperated by comma to c…

When calling vendor:publish with the call method it is easy to create an array for tag(s). But raw on the commandline it is a bit harder.
If the parameter value is separated by a comma (like --tag=resources,public) it will now create an array with two items.
$this->call('vendor:publish', ['--tag', => ['resources', 'public']]);
or
$this->call('vendor:publish', ['--tag' => 'resources,public']);

You can just pass multiple --tag arguments.


synology-drive-client: 3.3.0 -> 3.4.0

Description of changes
Release Notes can be found here: https://www.synology.com/en-global/releaseNote/SynologyDriveClient
I've got the changed URL from https://www.synology.com/en-global/dsm/feature/drive
(yeah, the new URL looks quite phishy but it's what i've got from the official site and the old returned 404).
Haven't done anyhing regarding darwin despite hoping that i've calculated the correct hash.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

1 package built:

synology-drive-client


help(youtube-player): not able to play any videos from YouTube Movies channel

What are you trying to do?
Play videos from this channel with youtube-player component
What troubleshooting steps have you tried?
youtube-player component is not able to play any videos from this channel in production mode, but able to play them in development mode (ng serve)
Reproduction
Here is the example - https://stackblitz.com/edit/angular-ivy-emrpyh
Steps to reproduce:

add youtube-player component to the project
as videoId  set any video from this channel

Environment

Angular: 10.0.8
CDK/Material:
Browser(s): Google Chrome Version 85.0.4183.121 (Official Build) (64-bit)
Operating System (e.g. Windows, macOS, Ubuntu): macOS Catalina Version 10.15.7 (19H2)

I get a message that the author has restricted access to the video in my country. There's also a setting that prevents videos from being embedded that could be blocking it.

There's also a setting that prevents videos from being embedded that could be blocking it.

But it works on my local computer with ng serve

I think this is just a Chrome (or maybe Chromium) issue and doesn't have anything to do with the cdk/youtube-player. Both videos in your StackBlitz work fine in FireFox, for example.

There’s a project that also stopped working today: https://github.com/youtube/youtube-ios-player-helper. I wonder if YouTube itself changed something with the iframe API that broke everything that depends on it for embedding YouTube videos

I think this is just a Chrome (or maybe Chromium) issue and doesn't have anything to do with the cdk/youtube-player. Both videos in your StackBlitz work fine in FireFox, for example.

Interesting observation, I've tried it on Safari and indeed it works

Closing this since I don't think it has anything to do with the Angular wrapper; it seems like this is something with the underling YouTube iframe embed.


Support storing shallow objects as part of the deepsignal

What
Fixes https://github.com/luisherranz/deepsignal/issues/36.
Support storing shallow objects as part of the deepsignal with shallow.
It accepts this signatures:
import { deepSignal, shallow } from "deepsignal";

const shallowObj = { a: 1 };
const store = deepSignal({ b: 2, shallowObj: shallow(shallowObj) });

import { deepSignal, shallow } from "deepsignal";

const shallowObj = { a: 1 };
shallow(shallowObj);
const store = deepSignal({ b: 2, shallowObj });

import { deepSignal, shallow } from "deepsignal";

const store = deepSignal({ b: 2, shallowObj: { a: 1 } });
shallow(store, "shallowObj");

Why
To prevent deepSignal from deep observing objects that don't need to be observed.
How
By exposing a shallow function that adds references to the ignore set used by the shouldProxy function.

@wittjosiah, I haven't figured out the TypeScript part yet, but I've released this with a different tag so you can test it out and let me know what you think:
https://www.npmjs.com/package/deepsignal/v/1.4.0-shallow.0
npm install deepsignal@shallow

Thanks @wittjosiah.

I have some concerns about the code in this pull request: the shallow function does not remove any already existing signals for the given object - which could exist after "deepSignal"ling a class instance (my code currently crashes because of such a situation)...

Well, I might have misunderstood the "shallowing" mechanism: it seems to ignore complete objects only - not just properties.
Even in the case shallow(object,key) it is the current object referred to by object.key which is ignored. As soon as a different object is assigned to object.key, it will be deeply observed again...

When you mention "I haven't figured out the TypeScript part yet", does this refer to the type returned by shallow() being <T> and state  expects the assignment to be <DeepSignalObject<T>>?
This is what I see, but I'm not sure I am implementing it correctly.
store.state.categoryLookup = shallow(
  new CategoryLookup(categories),
) as DeepSignal<CategoryLookup>;  // Have to cast as shallow is returning <CategoryLookup>

Perhaps I am setting the state's property incorrectly, since it is an object versus a primitive?

the shallow function does not remove any already existing signals for the given object

Yes, that's it. You need to use shallow before adding that property to the state.

it seems to ignore complete objects only - not just properties

Yes, that's how it works and hence the name "shallow": it will only observe the reference to that object, but not the object itself.

Have to cast as shallow is returning <CategoryLookup>

It's not a DeepSignal. So in that case, I believe CategoryLookup should be correct, isn't it?

I'm happy with TypeScript now. I've also removed the shallow(store, "shallowObj") API and only shallow(shallowObj) will be officially supported.


workaround for ie8 and uglified compilations of the code.

By giving a name to a previously anonymous function, /lib/createAction.js
can be minifized safely for ie8.
More info: https://github.com/reflux/reflux-core/issues/16

I can confirm that after applying this patch I was able to revert back to { mangle: true } in my uglify step and have it working correctly in IE8. However, still resisting to use unreleased non-upstream version of refluxjs in production.

My concern is that this will set a precedent that we will write code in a way that supports IE8.  My gut check says that we should be able to use anonymous functions without thinking about IE8 and uglification.

Hi devinivy,
I share your concern, as I mentioned I do think this is an ugly workaround. The only argument in favor for it is that this is rather special case. This is the only case I've seen an anonymous function causing problems with IE8, and it must be related to the fact that it's a functor using itself...
I haven't been able to find any alternative :S. Perhaps
var functor = function F () {

is a bit more elegant and shows to some extend that it's not just an anonymous function.

reflux-core is not stated to support IE8.


Metrics Server Deployment Failing: Certificate Validation Error

Please provide an in-depth description of the question you have:

What do you think about this question?:
Hello, @warjiang @XiShanYongYe-Chang , Please advise on how to resolve this certificate validation issue and successfully deploy the metrics-server
Environment:

Karmada version: latest
Karmada Dashboard version: 0.1.0
Others: N/A

@axif0 can you show your karmada.config file

@axif0 Where you did you deploy the metrics-sever,  on the control-plane or on the member cluster
If you start the environment by local-up-karmada.sh, then it will install metrics-server on member cluster
https://github.com/karmada-io/dashboard/blob/d23208ae9cd99dcf9f258140991856c4cf6d7fbf/hack/local-up-karmada.sh#L240C1-L250C1

@warjiang  Can you please how can i access cluster_sync_status_duration_seconds for member1 cluster?
Karmada Metrics Reference

@warjiang  Can you please tell how can i access/see in command terminal cluster_sync_status_duration_seconds or different Karmada components which mentioned in Karmada Metrics Reference for member1 cluster?  Should we use Prometheus for monitoring ?
i installed metrics-server as running hack/local-up-karmada.sh, i'm sorry i didn't check the file. i instaled metrics-server on member clusters

You can execute the following instructions:
kubectl --kubeconfig ~/.kube/karmada.config --context karmada-host  -n karmada-system get po -l app=karmada-controller-manager

it will output like this:

Things you should be careful are that karmada-controller-manager worked in master-slave architecture, only one pod will output the related metrics.
pod_name=karmada-controller-manager-54d9c78689-2ft65
kubectl --kubeconfig ~/.kube/karmada.config --context karmada-host   get --raw /api/v1/namespaces/karmada-system/pods/${pod_name}:8080/proxy/metrics|grep cluster_sync_status_duration_seconds

in my case, the first one is master, so it output:

the cluster_sync_status_duration_seconds metric has label cluster_name, so you can filter it by label_name

@axif0 i metioned that you seems have great interest about the metrics scraping, can you tell me more about your use-case?

Some metrics like histogram/Gauge/Counter are not visualized in dashboard. I'm trying to implement those Karmada Metrics in the dashboard.
@warjiang I wanted to contribute in karmada. Later I seen this is selected by LFX.

can you tell me how to resolve this error ? or what ever step you follow to deploy the api-server or control plane or dashboard.

hello .. @jayesh9747 did you configure first?
export KUBECONFIG=${HOME}/.kube/karmada.config

hello @axif0   I have tried using the given document by you

but still this waiting for ok... is shown to me  after run  this command bash hack/local-up-karmada.sh

hello .. @jayesh9747 did you configure first?
export KUBECONFIG=${HOME}/.kube/karmada.config . you can check this out here

and it is the next step after running this command but i stuck before execute this command

Hello, @jayesh9747 Are you prepare and download all required images in your local machine ?
if not then please try,
cp hack/images/image.list.load.online.example hack/images/image.list
bash hack/ops/load-images.sh hack/images/image.list


Integration Test - Release - 2018/07/13 06:56:27

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

@PRMergerTest3 : Thanks for your contribution! The author, @v-caxian, has been notified to review your proposed change.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

csidevdocs-public/IntegrationTest/BuildIntegartionTestFiles/ChangeSentToAuthor.md
:white_check_mark:Succeeded

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.


Simple scaffolding not possible anymore on meteor 1.3 with 0.5.5

I'm having this prototype app which was running with meteor 1.2 and angular2 beta.16. Now i've updated everything to meteor <IP_ADDRESS> and @angular rc1, and all other packages (including this one to 0.5.5).
Now i'm shortly away from the point i turn completely insane: I'm having my /client/app.ts which boots everything:
import 'reflect-metadata';
import 'zone.js/dist/zone';
import {Component} from '@angular/core';
import {bootstrap} from 'angular2-meteor-auto-bootstrap';
import {Socially} from './social';
bootstrap(Socially);
This variant above is working. As you can see, the "Socially" class is imported from his own file, laying directly in /client folder (social.ts). Now i move the file "social.ts" into a subfolder called "app". The import changes accordingly:
import {Socially} from './app/social';
As soon as we changed that and everything reloaded, we start getting this exception:
EXCEPTION: No Directive annotation found on App
As i could only barely believe that the folder structure is counting for this - i changed it back in structure and bam, works again... -.-
So i thought i'm "smart" and, so i ended up creating a "local" class directly in app.ts, just extending from the class i actually want:
import {
  AppMeteor, AppMeteorDecorator
} from './app/components/app-meteor/app-meteor.component';
@Component(AppMeteorDecorator)
class AppFireball extends AppMeteor implements OnInit, AfterViewChecked {
}
And well.. the app itself works, but the exception is just moving around, now he's complaining:
EXCEPTION: No Directive annotation found on HeaderMeteor
Well yeah, HeaderMeteor is just another custom component i'm adding as a directive - coming from another sub-folder.... -.-
The strange thing is - leaving meteor out the door, doing everything with webpack and no backend - the exact same app is working!
So i can't help myself but it must be related to the meteor-angular packages, maybe this one here or the "auto-bootstrap" one...
Can anyone help? Does anybody see the same behavior?

Ah damn.. i just found the new part in the tutorial about the special "imports" folder... -.- That would be the solution right...?

yes :)

hehe thanks for the answer anyways 😄  it was a classic murphys-law case.. whenever i try to explain my problem to somebody else (or to github) - it happens quite often that i suddenly found the solution on my own...

it a good thing, now your solution is out in the open for others to learn from


fix: 5+ LGTM warnings/alerts

Things added/changed:

Fixed 5+ LGTM similar warnings/alerts.

Hmm, I have seen the errors, but not sure how to fix them. Can someone provide any help? Thanks. 🙂

@Panquesito7 CI may fail if you force-push too often. This is because GitHub Actions doesn't have enough time to initialize the CI environment.

@akien-mga?

This would need to be redone, as there are now merge conflicts on nearly every file touched. These might have been fixed already in the master branch.


[sal/wifi] fix long time initalizing sal/wifi issue for developerkit/…

…starterkit
1, Add independent bk7231 wifi module instead of previous reuse mk3060
2, Remove unnecessary atcmd to fix long time wifi initialization issue
3, add wifi bk7231 FW version print

 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.Jianlong seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Fixing autorest page reference

I think this link should point to AutoRest page to continue with SDK generation. Looks like the repo was changes but the link was not updated.
This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ ] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

This commit was treated and no generation was made for Azure/azure-sdk-for-python

This commit was treated and no generation was made for Azure/azure-sdk-for-go

From the doc looks like it is a private repo.

This commit was treated and no generation was made for Azure/azure-sdk-for-python

Was unable to create SDK Azure/azure-sdk-for-python PR for this closed PR.

This commit was treated and no generation was made for Azure/azure-sdk-for-go

Swagger to SDK encountered an unknown error: (Azure/azure-sdk-for-go)
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/github_tools.py", line 29, in exception_to_github
    yield context
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/restapi/github.py", line 180, in rest_handle_action
    return rest_pull_close(body, github_con, restapi_repo, sdk_pr_target_repo, sdkbase)
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/restapi/github.py", line 311, in rest_pull_close
    rest_pr.create_issue_comment("Was unable to create SDK %s PR for this closed PR.", sdkid)
TypeError: create_issue_comment() takes 2 positional arguments but 3 were given


audio still sounds like eatin rust

make it not ?? ig /??? tea periodt um miss gurl purr

Your Emulator is pretty cool, but the AudioContext doesn't offer too much performance to the audio stuff, i'm moving on to this guy here: https://boytacean.joao.me/ that is written in WebAssembly.
But very cool project nonetheless, i'm still using as a second option to my project in browsers that don't support asm


getter setter checks for a, b, c for a superellipsoid

Added getter and setter tests for the super-ellipsoid for a,b,c.
Description

Added getter and setter tests for the super-ellipsoid for a,b,c.
Motivation and Context

There were no getter and setter tests for a,b,c.

Types of Changes

[ ] Documentation update
[ ] Bug fix
[ ] New feature
[ ] Breaking change1

1The change breaks (or has the potential to break) existing functionality.
Checklist:

[x] I am familiar with the Contributing Guidelines.
[x] I agree with the terms of the Contributor Agreement.
[x] The changes introduced by this pull request are covered by existing or newly introduced tests.
[x] I have updated the changelog and the credits.

@Tobias-Dwyer looks like validation still needs to get added on this PR.


pages: add "how to svelte" document

Adds a Svelte + SvelteKit how-to guide for Pages

looks good to me. only comment I have is that it might be worth mentioning the cloudflare adapter if only to say that it's meant for cloudflare workers which this tutorial doesn't cover so that if people see that adapter they won't be confused as to why it's not being used

@benmccann Thanks! That will be included in the upcoming Workers How-to series. This is specifically the pages.dev website

I don't have full context here but it is expected that user will know that pages is only for static sites currently (which isn't a constraint Svelte/ SvelteKit has, hence the potential for confusion), or is it worth adding a note about that when discussing changing the adapter?
Other than that, it looks good to me!

@pngwn Yes, this is the pages.dev website, which already covers the product details. It's static content only, and all How-To guides are leading each framework in a direction that's static-only.


Add support for other console methods

Currently, only console.log statements work. It would be nice to provide support for methods like console.table, console.time, console.timeLog and console.timeEnd.

Good idea!

Thankyou for your patience, here it is, it now supports other console methods also:
https://marketplace.visualstudio.com/items?itemName=KaranJanthe.purgeConsole
ps: I was busy with react-native and regex bug, there for it got delay little :)

No worries, thanks a lot Karan!


Add Myne & Update links for GreenStash

P.S. I'm the developer of both of these apps :)

P.S. I'm the developer of both of these apps :)

Thank you for these fantastic apps! You can place a ❤ by then if you love using them too or developing.

So I came back to the repo and I see no changes to your pull request, and I think I know why. I merged this, but I had a mirror setting with the codeberg repo: https://codeberg.org/Psyhackological/AAA
Does anyone know how to merge again? I deleted this push sync, but I don't know how to merge this again... weird problem.

Hmm, weird. Unfortunately, I don't have any experience with Codeberg, as I've never used it myself, so I can't really help with that :(

I disabled "codeberg" part (still learning). Now I want to remerge your Pull Request, but I don't know how. I cannot comprehend how remote things like Discussions, Releases, Pull Request, Issues are separated from git itself. For now, I installed GitHub CLI and will try to work with that...

For anyone curious, here are the workaround with GitHub CLI:

cd AAA/

Make sure you have proper remotes with git remote -v

codeberg        git@codeberg.org:Psyhackological/AAA.git (fetch)
codeberg        git@codeberg.org:Psyhackological/AAA.git (push)
origin  git@github.com:Psyhackological/AAA.git (fetch)
origin  git@github.com:Psyhackological/AAA.git (push)

gh pr checkout 79

remote: Enumerating objects: 14, done.
remote: Counting objects: 100% (14/14), done.
remote: Compressing objects: 100% (12/12), done.
remote: Total 12 (delta 8), reused 0 (delta 0), pack-reused 0
Unpacking objects: 100% (12/12), 3.84 KiB | 393.00 KiB/s, done.
From github.com:Psyhackological/AAA
 * [new ref]         refs/pull/79/head -> starry-shivam/main
Switched to branch 'starry-shivam/main'

git switch main

git merge starry-shivam/main

Updating 76fd89f..632c8fe
Fast-forward
 README.md | 17 +++++++++++++----
 1 file changed, 13 insertions(+), 4 deletions(-)

git log --oneline

632c8fe (HEAD -> main, origin/main, origin/HEAD, starry-shivam/main) Address Review
2d6d88e Update app counter
ed8c69d Add Myne in e-book readers section
204a5bb greenstash: Update links
76fd89f Add Fossify Notes < Notes (250)


Fix DatetimeIndex and other docstring for missing period at the end of summary

Same as #132 but for the next docstrings:
pandas.IndexSlice: Summary does not end with a period
pandas.MultiIndex.names: Summary does not end with a period
pandas.MultiIndex.is_lexsorted: Summary does not end with a period
pandas.MultiIndex.reorder_levels: Summary does not end with a period
pandas.DatetimeIndex.snap: Summary does not end with a period
pandas.DatetimeIndex.to_perioddelta: Summary does not end with a period
pandas.DatetimeIndex.to_pydatetime: Summary does not end with a period
pandas.DatetimeIndex.to_series: Summary does not end with a period
pandas.TimedeltaIndex: Summary does not end with a period
pandas.PeriodIndex.is_leap_year: Summary does not end with a period
pandas.api.extensions.ExtensionArray._concat_same_type: Summary does not end with a period
pandas.api.extensions.ExtensionArray.dropna: Summary does not end with a period

I would like to work on this

Closing this issue because it was merged in pandas #28031


Add User Arjun-Kadam

Related issue

What changes does this PR do?
Screenshots (If any)
Questions

Hello @aparna2071 I have placed name in right alphabetical order.

Hello @aparna2071 I have placed name in right alphabetical order.

Hello Arjun, I have already approved your PR, you have to wait for one more approval from our maintainers to get this PR merged 😄


Use Stripe CardElement to display existing card information

Is there a way to use the CardElement to display existing card information rather than using it purely as an input field? Something to this effect...
class SomeComponent extends React.Component {
  render() {
    const cardDetails = {
        last4 : 1234,
        brand : 'Visa',
        name : 'John Smith'
    }
    return (
      <CardElement details={cardDetails} />
    )
  }
}

Hey @mitchswaldman! This is a really cool idea! Unfortunately, it's not something that's implemented right now, mostly by design. If users were to have control over the actual data inside an element, there'd be a greater chance they accidentally put themselves into PCI scope.
In this particular example with last4 and brand, we could support something like that because these pieces of information aren't as sensitive as others, but we have not and are not currently planning on building an Element that renders this information.

@jez For sure. Thanks for the response!


Roll Flutter Engine from 3d1faae1bc2f to 241774c29344 (3 revisions)

https://github.com/flutter/engine/compare/3d1faae1bc2f...241774c29344
2023-08-29<EMAIL_ADDRESS>Roll Dart SDK from 60a3bb84fa6d to f6c11801db41 (1 revision) (flutter/engine#45207)
2023-08-29<EMAIL_ADDRESS>Roll Skia from b27fecd52899 to f576668bfe99 (1 revision) (flutter/engine#45206)
2023-08-29<EMAIL_ADDRESS>Bump actions/checkout from 3.5.3 to 3.6.0 (flutter/engine#45205)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC aaclarke@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


New Modal with useRef hook generates a warning/error in console

First, I would like to say thank you for sharing this awesome UI library to the world.
I was reading/coding a tutorial about react-daisyui modal. After going over the issues thread, it was suggested to use the useRef hook in the example link
import { useState, useRef, useCallback } from "react";
import {
  Button,
  Modal,
} from "react-daisyui";

import './App.css';

function App() {
  const ref = useRef<HTMLDialogElement>(null);

  const handleShow = useCallback(() => {
    ref.current?.showModal();
  }, [ref]);

  return (

DaisyUI Modal

      <Button className="w-32" onClick={handleShow}>
        Open Modal
      </Button>
      <Modal
        backdrop={true}
        ref={ref}
      >
          <Modal.Header className="font-bold">Hello!</Modal.Header>
          <Modal.Body>Press ESC key or click outside to close</Modal.Body>
      </Modal>

  );
}

export default App;

The codes above generate a warning: 'A string ref, "false", has been found within a strict mode tree' in the browser console. Thank you for the reply.

I have updated the codes above to the one below and got rid of the error. But the Modal does not close when user clicks outside of it.
import { useRef, useCallback } from "react";
import {
  Button,
  Modal,
} from "react-daisyui";

import './App.css';

function App() {
  const ref = useRef();

  const handleShow = useCallback(() => {
    ref.current?.showModal();
  }, []);

  return (

DaisyUI Modal

      <Button className="w-32" onClick={handleShow}>
        Open Modal
      </Button>
      <Modal
        backdrop={true}
        ref={ref}
      >
        <Modal.Header className="font-bold">Hello!</Modal.Header>
        <Modal.Body>Press ESC key or click outside to close</Modal.Body>
      </Modal>

  );
}

export default App;

Thank you for the reply.

I have updated the codes above to the one below and got rid of the error. But the Modal does not close when user clicks outside of it.
import { useRef, useCallback } from "react";
import {
  Button,
  Modal,
} from "react-daisyui";

import './App.css';

function App() {
  const ref = useRef();

  const handleShow = useCallback(() => {
    ref.current?.showModal();
  }, []);

  return (

DaisyUI Modal

      <Button className="w-32" onClick={handleShow}>
        Open Modal
      </Button>
      <Modal
        backdrop={true}
        ref={ref}
      >
        <Modal.Header className="font-bold">Hello!</Modal.Header>
        <Modal.Body>Press ESC key or click outside to close</Modal.Body>
      </Modal>

  );
}

export default App;

Thank you for the reply.


update ansible-lint to 6.11 for pre-commit hooks

ansible-lint 6.8 behave differently than CI locally for some reason. with ansible 6.11 (same version as latest release of the github action) this is fixed now
fix #207

Merging since this is very safe.


Problem with npm install

Hello,
I've problem with the installation : I did  npm install and I get this :
npm ERR! git clone git@github.com:github:whitef0x0/wildcard-subdomains Clonage dans le dépôt nu '/home/pi/.npm/_git-remotes/git-github-com-github-whitef0x0-wildcard-subdomains-318de7d0'
npm ERR! git clone git@github.com:github:whitef0x0/wildcard-subdomains fatal: remote error:
npm ERR! git clone git@github.com:github:whitef0x0/wildcard-subdomains    is not a valid repository name
npm ERR! git clone git@github.com:github:whitef0x0/wildcard-subdomains   Email<EMAIL_ADDRESS>for help
npm ERR! notarget No compatible version found: wildcard-subdomains@'github:whitef0x0/wildcard-subdomains'
npm ERR! notarget Valid install targets:
npm ERR! notarget ["0.0.1","0.0.2","0.0.3","0.1.0","1.0.0","1.1.0"]
npm ERR! notarget
npm ERR! notarget This is most likely not a problem with npm itself.
npm ERR! notarget In most cases you or one of your dependencies are requesting
npm ERR! notarget a package version that doesn't exist.

npm ERR! System Linux 4.9.17-v7+
npm ERR! command "/usr/bin/nodejs" "/usr/bin/npm" "install"
npm ERR! cwd /var/www/my_webapp/www
npm ERR! node -v v0.10.29
npm ERR! npm -v 1.4.21
npm ERR! code ETARGET
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /var/www/my_webapp/www/npm-debug.log
npm ERR! not ok code 0

Could you help me please ? I'm building it on a Raspberry Pi 2 B+ (ARMv7; Raspbian)

@frju365 What version of npm do you have?

pi@yunohost:~ $ npm -v
1.4.21

@frju365 you are running a old version of npm and node. You should be using node 5.0.0-6.4.0 with npm 3.x


revert: preserves tools inner in local drawer in AppLayout toolbar variant

Reverts cloudscape-design/components#2816
There is a regression: https://d21d5uik3ws71m.cloudfront.net/components/2dc1dbac64920a30fa81c761749f6ca825d93410/dev-pages/index.html#/light/app-layout/with-split-panel?appLayoutToolbar=true
This page renders an empty space on the right now

Can we merge this instead? https://github.com/cloudscape-design/components/pull/2828

We found a fix: https://github.com/cloudscape-design/components/pull/2828


Remove ruiaraujo from CODEOWNERS

I have not really contributed in a few years.

👍

Should MAINTAINERS also be modified?

:+1:


Fixed incompatibility with DVS agent

DVS agent is supporting only VLAN networks, but in case when APIC creates networks,  we need only to pass the type check in DVS agent. So, one way is to fake network_type as a VLAN.

I'm not aware that this is a problem -- it's worked fine with the DVS agent in our testing. Also, the apic-ml2-driver is deprecated in newton, which is now EOL. Unless there is a compelling reason I think we can abandon/close this.

No longer outstanding issue -- closing.


Add Non-Linear Causal Relationships

To provide more data characteristics beyond the conditional Gaussian distribution model, MANM-CS should be extended by non-linear causal relationships.

Currently, we have "Defines the percentage of nodes that shall be of discrete type. Depending on its value the appropriate model (multivariate normal, mixed gaussian, discrete only) is chosen.".
Should non-linear causal relationships exist independent of the model (see quoted sentence)?
Further, do we want to support a mixture of types of relationships?

As noted from the discussion. We want to remove the beta values for continues to continues variables relationships. And rather have functions e.g., linear, quadratic, power of, tanh, betrag|exp|, ... .

Proposed Strucuter:- Continuou

As noted from the discussion. We want to remove the beta values for continues to continues variables relationships. And rather have functions e.g., linear ( ), ( )², ( )³, tanh( ), sin( ), ... .

Proposed Structure with Parameters: (4 possible models to choose)

Continuous:
1.
Discrete:

Proposed Structure Including Parameters:
Overall

num_processes

Graph Sampling

num_nodes
edge_density
num_samples

Distribution Model
Continuous
1.	List of possible functions from {linear ( ), ( )², ( )³, tanh( ), sin( ), ...}
2.	Sample probability for each function
3.	continuous_noise_std [0, Inf)
Discrete
1.	min_discrete_value_classes
2.	max_discrete_value_classes
3.	discrete_signal_to_noise_ratio
Conditional Linear Gaussian
1.	discrete_node_ratio
2.	min_discrete_value_classes
3.	max_discrete_value_classes
4.	discrete_signal_to_noise_ratio
5.	continuous_noise_std
6.	continuous_beta (tbd)
Mixed
1.	discrete_node_ratio
2.	min_discrete_value_classes
3.	max_discrete_value_classes
4.	List of possible functions from {linear ( ), ( )², ( )³, tanh( ), sin( ), ...}
5.	Sample probability for each function
6.	discrete_signal_to_noise_ratio
7.	continuous_noise_std


Model checkpoints from Table 3

Hi,
Thanks for releasing the code, especially in PyTorch. I was wondering if you are also planning to release the checkpoints for models from Table 3?

These models are now listed at the bottom of the README. Let us know if anything doesn't work!

Thanks a lot for releasing multiple checkpoints. I wonder if the checkpoints correspond to the ema version (since its not clear from their naming whether its ema or non-ema).


Add more options; change encode to skip non-printing ASCII characters

Support more HTML escape options (allowUnsafeSymbols, encodeEverything)
Support global HTML escape option overrides
Change default value of useNamedReferences from true to false
Update README, improve tests

@youming-lin Shmuel merged this but I'd still appreciate you taking a look at my review comments :)

@ianpartridge I didn't realize there's an OptionSet protocol, I'll read up about it and implement it :)

@ianpartridge Actually, now that I've read about OptionSet, I'm not sure if there's a simple way to replicate the ability to set individual options globally using OptionSet. Also, it's not clear what the absence of an option in an option set means: does it mean that the default value (could be either true or false) for the absent option should be used, or does it mean that the absent option should be considered to be in a false state?

Ah I take your point about setting global options. I hadn't clocked that you were setting those via String.HTMLEscapeOptions.

Would it be cleaner to change to an options: [String: Any] argument like in the recent changes to the plugin constructor in Kitura-CredentialsFacebook?

Hmm I feel like that loses us type-safety... Maybe what you have is OK for now?

@ianpartridge That's OK with me. Can I go ahead and tag what's currently in master as a new 3.0.0 release?

Sure, unless you know of any outstanding API-breaking changes we want to make?

Nothing in plan right now. I'll go ahead and tag a new release.


Improved performance

Methods to improve performance
Trying to improve further

I think this is a good improvement, we can do better, but we would need to change hound, as the slowness right now is from hound reading samples.
Encode:

Decode:

As you can see, most of the time is spent on reading the samples, for encoding, its 50/50 between reading from hound and writing to the file, so I think its okay.
But we can make this very fast by using a simple wav writer, and we can do optimization when reading/writing

But we can make this very fast by using a simple wav writer, and we can do optimization when reading/writing

I'll check it

Another interesting detail/cheat, we can do this extremely easily and make it insanely fast.
the wav file stores the PCM samples directly as bytes, without any compression, so it means the file content are actually stored in bytes, and this is true, as u can see here.
original file

wav

So we can add just a header for wav and just copy the whole file, which should be fast

So we can add just a header for wav and just copy the whole file, which should be fast

Wow, is that really possible!! How can we get header? Is it variable or constant in all files?

Hound uses this library: https://docs.rs/wav/latest/wav/index.html
this library allows u to write the header, so yeah, would be interesting to try


[Snyk] Security upgrade bootstrap from 3.4.1 to 4.0.0

Snyk has created this PR to fix 2 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

spiderfoot/static/package.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Cross-site Scripting (XSS) SNYK-JS-BOOTSTRAP-7444593
  688

Cross-site Scripting SNYK-JS-BOOTSTRAP-7444617
  688

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Cross-site Scripting (XSS)

Checkmarx One – Scan Summary & Details – 74877892-581b-4765-ab0d-18ff8081e124
New Issues

Severity
Issue
Source File / Package
Checkmarx Insight

CVE-2024-6531
Npm-bootstrap-4.6.2
Vulnerable Package

Fixed Issues

Severity
Issue
Source File / Package

CVE-2007-2379
Npm-jquery-3.7.1

CVE-2014-6071
Npm-jquery-3.7.1


alfabetical

var catNames = require('cat-names');

var sorted = catNames.all.slice().sort(function (a, b) {
  return a.localeCompare(b);
});

Is it not alphabetical ?

Well. It's more about… README.md you know.
Top 100 cat names in alfabetical order.

❯❯❯ http://www.thefreedictionary.com/alphabetical

Oh, I thought I was on the dogs one.
And yeah, I can't spell: https://github.com/sindresorhus/cat-names/commit/28d0225c94b764fcd5aafab1e597de79598b527f

Thank you @sindresorhus ;)


Adding enum option fails with weird error

I'm upgrading a schema:
scalar type TicketStatus extending enum<
        'new',
        'closed'
>;

To:
    scalar type TicketStatus extending enum<
        'new',
        'assigned',
        'closed'
    >;

Using CREATE MIGRATION .. TO (I mean schema is bigger, but I'm just adding an option)
And I get this error:
InternalServerError: ALTER TYPE ... ADD cannot run inside a transaction block

I assume that that alter type is generated somewhere internally. Anyway, it should either be fixed or some workarounds proposed in the error message.
(removing an option works, though)
Server version 1.0-alpha.2+dev.333.gf30744d

Hm. This is weird.  Postgres documentation states that ALTER TYPE .. ADD can run inside a transaction block, but the new value cannot be used until the transaction is committed (https://www.postgresql.org/docs/current/sql-altertype.html#id-<IP_ADDRESS>.7)

The error seems to be PostgreSQL 11 behavior.  You need to rebuild your checkout (drop build/postgres) and run pip install -e .)

I use a nightly package 1.0a2.dev333+gf30744d-2019113000~xenial.nightly and don't install postgres myself. So if it's old postgres the nightly package must be fixed.

But yes, postgres is old:
/usr/lib/x86_64-linux-gnu/edgedb-1-alpha2/bin/postgres -V
postgres (PostgreSQL) 11.2

Ah. Right. That's a packaging bug. Will fix

Ah. Right. That's a packaging bug. Will fix

@elprans Is this now resolved?

Yes, this is fixed.


Stop all executing blocks on play selected

Quick bugfix: If there is another block executing, and then you use the "play selected" command on a non-executing block, both blocks will execute at the same time. I changed it so that it will stop all executing blocks before executing the selected block.

Hmm.. I can see why this is a good default, but you might want parallel executing Blocks though sometimes? Fungus is designed so that you can have multiple Blocks executing in a Flowchart at the same time, it's handy for lots of things.
What about if there were two menu options? "Play from selected" and "Stop all and play"? That way you can choose which you want.

Woops didn't know that was the case. I like the idea of the two different options. I'll get to work on that. Thanks!

So like this?

Perfect - thanks!


chore: redirect degen frame to thirdweb contract page

PR-Codex overview
The focus of this PR is to enhance the functionality of the Thirdweb Degen Engine by adding a new action validation method and updating the minting process for NFTs.
Detailed summary

Added validateAction method to validate actions
Updated NFT minting process with new action handling
Refactored API handler to use NextRequest
Improved response handling in the API handler

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

📝 Changed routes:

/frame/degen/mint

(view full size)

And 1 other route:

/api/frame/degen/mint

Commit a941872f63a58ccf47bb27069d914a6b0ffd85c7 (https://thirdweb-m4ggmjc1g.thirdweb-preview.com).