Problem with Google Maps in iOS, app crash after switch to another page from map page

I couldn't get any error message on this crash.
My steps was:

open app,

navigate to map page,

zoom out,

go back to home page or other page,

then crash..

If I remove google map page, this app will never crash. Xcode and VS code don't show any  error message. So I have no idea how it happened.
I tested this on iPhone6 , 7, and X. This app will crash when using these phones, but it never crash on any Android phone.
I am stacking on this bug for days, so please me.
My map page code:
import 'package:flutter/material.dart';
import 'package:google_maps_flutter/google_maps_flutter.dart';

class Tester extends StatefulWidget {
  Tester({Key key}) : super(key: key);

  _TesterState createState() => _TesterState();
}

class _TesterState extends State<Tester> {
  GoogleMapController mapController;
  @override
  Widget build(BuildContext context) {
    return Scaffold(
        body: Container(
      height: MediaQuery.of(context).size.height,
      width: MediaQuery.of(context).size.width,
      child: GoogleMap(
        onMapCreated: (GoogleMapController controller) {
          mapController = controller;
        },
        initialCameraPosition:
            const CameraPosition(target: LatLng(43.6332, -79.3932), zoom: 12),
      ),
    ));
  }
}

My Flutter doctor info:
[✓] Flutter (Channel stable, v1.5.4-hotfix.2, on Mac OS X 10.14.5 18F132, locale zh-Hans-CA)
    • Flutter version 1.5.4-hotfix.2 at /Users/shijiesun/flutter
    • Framework revision 7a4c33425d (7 weeks ago), 2019-04-29 11:05:24 -0700
    • Engine revision 52c7a1e849
    • Dart version 2.3.0 (build 2.3.0-dev.0.5 a1668566e5)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/shijiesun/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.2.1, Build version 10E1001
    • ios-deploy 1.9.4
    • CocoaPods version 1.6.2

[✓] Android Studio (version 3.4)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 35.3.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

[✓] VS Code (version 1.35.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.1.0

[✓] Connected device (1 available)
    • iPhone • 322f1e976614e48eeca824746453ccecd4645d2d • ios • iOS 12.1.4

Here is a video to show how it happened.
https://youtu.be/18An4uYjF1I

Facing the same issue!! Couldn't solve it :(

I managed to solve this issue by using the MaterialPageRoute for navigation I was using a custom PageRoute.

To resolve this we will need a complete application that shows the problem; the code above is incomplete (no runApp, etc).

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of 'flutter doctor -v'  please.
All system setups can be slightly different so it's always better to open new issues and reference related issues.


Fix cloud specific attributes type.

Code Coverage is 79.5%, details at https://s3.amazonaws.com/rs-code-coverage/rsc/2015-04-18_193_Coverage-79.5.txt (sometimes the URL can take a few minutes to be available)


Section needs clarification regarding Microsoft Surface devices...

This section should address Microsoft Surface Pro devices. It says to reach out to the hardware OEM but in this case it would be MS. Google searches have come up fruitless so I'm hoping maybe I just missed the documentation on how to do this somewhere. Should I expect Hybrid AAD joins to fail on all of my Surface devices? Per the spec sheet for my own Surface Laptop 2, the TPM is FIPS compliant but I can't find instructions anywhere on how to disable this.
Hybrid Azure AD join is not supported for FIPS-compliant TPMs. If your devices have FIPS-compliant TPMs, you must disable them before proceeding with Hybrid Azure AD join. Microsoft does not provide any tools for disabling FIPS mode for TPMs as it is dependent on the TPM manufacturer. Please contact your hardware OEM for support.

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

ID: 72cdc251-618a-e84e-f40d-960b0ac69f99
Version Independent ID: 8cbaba3c-e883-2c71-28fb-f88db3fcaf7d
Content: How to plan hybrid Azure Active Directory join implementation in Azure Active Directory (Azure AD)
Content Source: articles/active-directory/devices/hybrid-azuread-join-plan.md
Service: active-directory
Sub-service: devices
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@IXbaalque Thanks for your feedback! We will investigate and update as appropriate.

@lXbalanque We have engaged the content authors to have this reviewed with engineering to see how best the details on the page can be updated for our customers.

Sent follow-up email to product group for response

We'd have to review this with the Surface team. Could you ask a similar question on the Surface documentation, perhaps it's not documented publicly.

I think any additional information would require a support case to be opened. I don't think this is purely documentation a request at this point.

#please-close


 I used the following code.
CollectionReference instance = MockFirestoreInstance().collection("orders");
    await instance.document(order2.orderId).setData(order2.toJson());
    DocumentSnapshot snapshot = await instance.document(order2.orderId).get();
    Order order = Order.fromSnapshot(snapshot);

Encountered a NoSuchMethodError an error while saving an instance of Timestamp by converting it into a DateTime object using toDate() method.
this.timeStamp = (map["timeStamp"]).toDate();

Error encountered
NoSuchMethodError: Class 'DateTime' has no instance method 'toDate'.
  Receiver: Instance of 'DateTime'
  Tried calling: toDate()

Am I doing something wrong or is the Mock instance actually returning an instance of DateTime instead of Timestamp?

Seems like it has since been fixed. The following test works:
  test('returning Timestamp', () async {
    CollectionReference instance = MockFirestoreInstance().collection("orders");
    await instance.document(uid).setData({
      'time': DateTime.now(),
    });
    DocumentSnapshot snapshot = await instance.document(uid).get();
    final timestamp = snapshot['time'] as Timestamp;
    expect(timestamp.toDate(), isNotNull);
  });


Add GroupBy.shuffle_to_chunks()

This adds some new API to shuffle an Xarray object. Shuffling means we sort so that members of a group occur in the same chunk, with the possibility of multiple groups in a single chunk.
gb = ds.groupby(..)
shuffled : Dataset = gb.distributed_shuffle()
shuffled.groupby(...).quantile()

I've also added shuffle_by to DataArray and Dataset. This generalizes sortby, and lets you persist a shuffled Xarray object to disk.
ds.shuffle_by(grouper)

[x] xref #9220, xref #9546
[x] Tests added
[ ] User visible changes (including notable bug fixes) are documented in whats-new.rst
[x] New functions/methods are listed in api.rst
[x] wait on dask 2024.08.1
[x] upstream flox release
[x] add chunks to signature.
[x] error if group is chunked

cc @phofl

@aulemahal using shuffle should massively improve your "Padded DOY Grouper" thing. Can you let us know what API would work for your use case? Trying it out would also be quite valuable.

Looks great overall!
Since the original, looks like we added .shuffle_by. What's the role of .shuffle_by('foo') relative to .groupby('foo').shuffle()?

ATM shuffle_by returns the shuffled Dataset or DataArray. groupby.shuffle() returns a new Groupby object that uses the shuffled data.
I am not very attached to this. FYO Stephan mentioned sortby as a possible alternate entrypoint instead.

We could also just have GroupBy.shuffle() -> Dataset | DataArray.
Then
# Save the groupby kwargs since we need it twice
groupers = {"label": UniqueGrouper()}

# get the shuffled dataset
shuffled: Dataset = ds.groupby(groupers).shuffle()

# Now group the shuffled dataset the same way and map a UDF that expects all members of a group in memory at the same time.
shuffled.groupby(groupers).map(UDF)

Alternative names could be GroupBy.as_sorted() -> Dataset | DataArray, GroupBy.sorted_object() -> Dataset | DataArray.

After thinking about it a while, sortby is similar but different w.r.t chunking. It's vectorized indexing with an index you get from calling argsort, and dask/cubed is free to rechunk as they please.  (for e.g. sortby(time=UniqueGrouper()) when all(chunksizes["time"] == 1) is basically free in terms of rechunking, but with a shuffle, you'd get the same time labels in the same chunk. We could have sortby a grouper object basically provide np.concatenate(_group_indices) as the indexer.
This is all fairly advanced stuff, so IMO we can start with the smallest possible API surface, which would be GroupBy.shuffle IMO (or any of the other names above)
cc @shoyer

Very much agree with all of your message — sortby being a bit different, starting with GroupBy.shuffle

We could also just have GroupBy.shuffle() -> Dataset | DataArray.

(If it were GroupBy.shuffle() -> GroupBy, then I guess .shuffle_by would be equivalent to .groupby(groupers).shuffle().map(lambda x: x)...)

Would it be too verbose to call this distributed_shuffle? Shuffling just has a very specific meaningful in distributed computing that is quite different from the generic meaning of "shuffle" for rearranging data.

Would it be too verbose to call this distributed_shuffle? Shuffling just has a very specific meaningful in distributed computing that is quite different from the generic meaning of "shuffle" for rearranging data.

For the method is on the GroupBy object, is shuffle less ambiguous? FWIW I had correctly presumed it meant its actual meaning given it was associated with group by
Otherwise we could workshop distributed_shuffle to something terser :)

OK I went with distributed_shuffle returning a shuffled Xarray object. Any further groupby operations will need to repeat the groupby statement:
e.g.
gb = array.groupby_bins("dim_0", bins=bins, **cut_kwargs)

shuffled: DataArray = gb.distributed_shuffle()

gb = shuffle.groupby_bins("dim_0", bins=bins, **cut_kwargs)

how about shuffle_to_chunks?

Looks good! (No strong view on the exact name...)

OK I'm going with shuffle_to_chunks. Will merge in ~12 hours. It's been around a while now :)

Sounds good!
On Wed, Nov 20, 2024 at 3:58 PM Deepak Cherian @.***>
wrote:

OK I'm going with shuffle_to_chunks. Will merge in ~12 hours. It's been
around a while now :)
—
Reply to this email directly, view it on GitHub
https://github.com/pydata/xarray/pull/9320#issuecomment-2489782096, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJJFVXAWGTXX62ULVGF6L32BUOZVAVCNFSM6AAAAABMDO3MWGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOBZG44DEMBZGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Really like the new name!

Thanks for the input!


Test 11 OIDC4VP , when we check a box we dont get the correct disclosure in the vp_token

when we check the firt box we get
EJhG0SYM1JjG_PAkpZsO3qrc2esDhp2236g-v4vxSQmKnHUjehDQ~~eyJhbGciOiJFUzI1NiJ9.
there are 2 ~ and nothing in between


Add react-emoji-react

Not sure if is the best section for that but it's totally worth it

Hi, thanks for the PRs! Is there a reason you didn't use devarchy to add the libraries? It's then easier for to manage addition requests

what is devarchy?

https://devarchy.com/react-components
Would be awesome if you could add them there. Requests are easier to manage on devarchy.

It is included in the catalog. Thanks for the PR! Let me know if there is a reason you didn't use devarchy to add it.


doc: Fix typo in api-extensions.md

Signed-off-by: KATOH Yasufumi<EMAIL_ADDRESS>

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

jenkins: test this please

Thank you!


--dry-run prints even executed migrations

When I run 'db-migrate up:mydb' I get:
[INFO] loaded extra config for migration subfolder: "mydb/config.json"
[INFO] No migrations to run
[INFO] Done
When I run 'db-migrate up:mydb --dry-run' I get:
[INFO] dry run
[INFO] loaded extra config for migration subfolder: "giga/config.json"
................
all the previous migrations are printed

Please always provide everything needed to reproduce your issues.
This includes:

a verbose run
used db-migrate version
used db-migrate driver
used node version
used npm version

And if necessary also migrations amd config details to reproduce.

Version: 0.10.0-beta.12
I expect to see only SQL migrations that should be performed when running with --dry-run option
May be I am doing smth wrong?

Still missing the verbose run, used driver and so on. Without any information I can't help you and currently I have exactly nothing to see any behavior or do have steps to reproduce it.

Attached test: db-migrate-issue-377.zip
I use:

Node.js 4.4.3
NPM 2.15.1
MySQL 5.5.49 (please update database JSON - I use non standard port)

Top run test: npm run up
I expect dry run to say: No migrations to run, however, I get SQL printed

By the way - thanks for quick previous responses :)

Got it thanks, tracked as upstream issue here:
https://github.com/db-migrate/mysql/issues/5

I tried the workaround https://github.com/db-migrate/mysql/issues/5 (using all) but it did not help.

@SilentScope It is not a workaround but a fix suggestion which needs a change at the driver level. I will look at this in a few hours.

fixed, published as

<EMAIL_ADDRESS>
You can upgrade to this new driver version as of now.

It works as expected now - thanks a lot! :)


fix atomics memory ordering for sharedptr

Consume was used in the original pr nim-lang/Nim#10485, I haven't questioned it before nim-lang/Nim#19212

my memory fools me, it first appeared in https://github.com/nim-lang/fusion/pull/8

Ready. Followed mratsim's comment.


Graceful app shutdown hook

Currently Revel performs a process kill on the app. This doesn't give the app a chance to perform any actions before shutting down.
harness/harness.go:153-159
SHA: abc464838b1e45be6563b72b75f8b85a1bf6f771
        // Kill the app on signal.
        ch := make(chan os.Signal)
        signal.Notify(ch, os.Interrupt, os.Kill)
        <-ch
        if h.app != nil {
                h.app.Kill()
        }
        os.Exit(1)

Which eventually leads to
harness/app.go:83-92
// Terminate the app server if it's running.
func (cmd AppCmd) Kill() {
        if cmd.Cmd != nil && (cmd.ProcessState == nil || !cmd.ProcessState.Exited()) {
                revel.TRACE.Println("Killing revel server pid", cmd.Process.Pid)
                err := cmd.Process.Kill()
                if err != nil {
                        revel.ERROR.Fatalln("Failed to kill revel server:", err)
                }
        }
}

I'm thinking something like the following code used instead of AppCmd.Kill() could solve the problem. I don't know if it would be possible to make the timeout configurable.
// Terminates a server
func (cmd AppCmd) Terminate() error {
        revel.TRACE.Println("Terminating revel server pid: ", cmd.Process.Pid)
        cmd.Cmd.Signal(os.Interrupt)
        select {
        case <-cmd.waitChan():
                return
        case <-time.After(10 * time.Second):
                revel.TRACE.Println("Timeout; Force kill revel server pid: ", cmd.Process.Pid)
                cmd.Process.Kill()
                return
        }
        panic("Impossible")
}

Considering this still has not been implemented yet, are there any plans to do so?

@notzippy I assume it's possible to check go version from within the code. Perhaps integrate the graceful shutdown, but gate it behind a minimum version check?  That way the overall package is still usable for older environments, while at least getting the functionality implemented.


Add evaluation stats to file load screen

average percent complete & complete laps percent

Hi @dmh23, Can i pick this one?


Single Marker BrAPI call

We need a call to allow one to query the intersection between accession and marker. So, rather than getting a whole markerprofile, ask "What is the genotype of accession X at marker position Y"

Or also filter by marker state... give me the lines that have this state etc. Maybe should be another ticket.

New GA4GH schema allows for this


⚠ Bofrost.de has degraded performance

In 0b13620, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 860 ms

Resolved: Bofrost.de performance has improved in d02dfdd after 9 minutes.


Setup-node for release workflow

#575  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @CatChen and the rest of your teammates on  Graphite


Can't un-mute or change volume on muted HTML5 playsinline video on iPad

Expected behaviour
Ability to un-mute audio on a muted, playsinline video on iPad
Actual behaviour
No un-mute button or volume slider is available
Steps to reproduce

open https://codepen.io/anon/pen/dwqPzO on iPad device or simulator
start video

Environment

Browser: iOS Safari
Version: 12
Operating System: iOS
Version: 12

Console errors (if any)
Link to where the bug is happening
https://codepen.io/anon/pen/dwqPzO (this is just the basic HTML5 video test case, with the muted attribute added to the video tag)

The same thing is happening when i'm using 3.5.6.
I need the un-mute button.
on this issue https://github.com/sampotts/plyr/issues/656, @sampotts says this issue got fixed in 3.2.4. (I confirmed that although version 3.2.4 throws an error when the page is server-side rendered on nuxt...)

I'm also having this issue, there is no option to mute or unmute a video on iOS.


Flag is not updated with the correct country when disableAreaCodes = true

When the disableAreaCodes is enabled, the flag is not been updated according to the area code typed.
Steps to reproduce the bug:

Type +1 438

Expected result:

The Canada flag should show up since that area code belongs to Canada

Actual result:

The US flag show up

I create the PR #64 to solve this bug

with the implementation of onlyCountries, this is not needed any more


generate signature for SparseTensor failed

What related GitHub issues or StackOverflow threads have you found by searching the web for your problem?
I can not find any posts about how to export a sparse model in tensorflow. I searched for hours.
Environment info
Operating System:
centos7 without gpu
If possible, provide a minimal reproducible example (We usually don't have time to read hundreds of lines of your code)
export function:
# Read TFRecords files for training
filename_queue = tf.train.string_input_producer(
    tf.train.match_filenames_once(FLAGS.train),
    num_epochs=epoch_number)
serialized_example = read_and_decode(filename_queue)
batch_serialized_example = tf.train.shuffle_batch(
    [serialized_example],
    batch_size=batch_size,
    num_threads=thread_number,
    capacity=capacity,
    min_after_dequeue=min_after_dequeue)
features = tf.parse_example(
    batch_serialized_example,
    features={
        "label": tf.FixedLenFeature([], tf.float32),
        "ids": tf.VarLenFeature(tf.int64),
        "values": tf.VarLenFeature(tf.float32),
    })
batch_labels = features["label"]
batch_ids = features["ids"]
batch_values = features["values"]

        print("Start to run export model")
        start_time = datetime.datetime.now()

        ckpt = tf.train.get_checkpoint_state(checkpoint_dir)
        if ckpt and ckpt.model_checkpoint_path:
            print("Use the model {}".format(ckpt.model_checkpoint_path))
            saver.restore(sess, ckpt.model_checkpoint_path)
            export_version = FLAGS.export_version
            export_path = "./export/"+job_name
            print('Exporting trained model to %s' % export_path)
            model_exporter = exporter.Exporter(saver)
            model_exporter.init(
                sess.graph.as_graph_def(),
                named_graph_signatures = {
                  'inputs': exporter.generic_signature({'ids': features['ids'], 'values':features['values']}),
                  'outputs': exporter.generic_signature({'labels': features['labels']})},
                init_op=init_op)
            model_exporter.export(export_path, tf.constant(export_version), sess)
            print('Done exporting!')

Logs or other output that would be helpful
(If logs are large, please upload as attachment or provide link).
Traceback (most recent call last):
  File "run.py", line 343, in <module>
    'inputs': exporter.generic_signature({'ids': features['ids'], 'values':features['values']}),
  File "/home/serving/anaconda2/lib/python2.7/site-packages/tensorflow/contrib/session_bundle/exporter.py", line 121, in generic_signature
    signature.generic_signature.map[name].tensor_name = tensor.name
AttributeError: 'SparseTensor' object has no attribute 'name'

@ericyue Could you find a solution?

Also meet this problem. Any suggestion? @ericyue

I'm getting the same error when calling tf.python.saved_model.utils.build_tensor_info() on a sparse_placeholder (TF v1.0). Anyone have a solution yet?

can we use like this?
inputs = {'source': utils.build_tensor_info(pred_data[0]),               'target_sparse': (utils.build_tensor_info(pred_data[1].indices), utils.build_tensor_info(pred_data[1].values), utils.build_tensor_info(pred_data[1].dense_shape)),               'source_len': utils.build_tensor_info(pred_data[2])               }

Why this issue was closed? I am experiencing the same problem.

I think the details of the original issue are out of date, but it still doesnt seem possible to use sparse tensors as part of a signature.  Is this intended behavior?
From this doc:
https://github.com/tensorflow/serving/blob/master/tensorflow_serving/g3doc/signature_defs.md
"Note that TensorInfo itself requires specification of name, dtype and tensor shape. While tensor information is already present in the graph, it is useful to explicitly have the TensorInfo defined as part of the SignatureDef since tools can then perform signature validation, etc. without having to read the graph definition."
(linked from here: https://github.com/tensorflow/tensorflow/tree/master/tensorflow/python/saved_model)
This seems consistent with the code -- looking here (in  def _validate_signature_def_map(self, signature_def_map)):
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/saved_model/builder_impl.py#L202
The definition of TensorInfo seems to show that sparse will not have a name:
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/core/protobuf/meta_graph.proto
So basically, when using a VarLenFeature, export fails:
AssertionError: All TensorInfo protos used in the SignatureDefs must have the name field set: dtype: DT_STRING

This fix from a few days ago in saved_model.utils.py seems to imply that using a SparseTensor should be possible, but maybe I'm misinterpreting:
https://github.com/tensorflow/tensorflow/commit/eb27277809c038d93edcfb7221295608654589d7

I also meet the similar problem. Solutions about prediction using sparse tensor are really really rare.

I am having the same problem. I can't find a way to export using SavedModelBuilder with signatures generated from SparseTensors. Is there really no way of achieving this?

I have the same issue, I am working with sparse_placeholder and I get this error:
AssertionError: All TensorInfo protos used in the SignatureDefs must have the name field set: dtype: DT_FLOAT
tensor_shape {
unknown_rank: true
}
coo_sparse {
values_tensor_name: "input/values:0"
indices_tensor_name: "input/indices:0"
dense_shape_tensor_name: "input/shape:0"
}

@MatteoGlarey I "solved" the problem by building tensor infos from the 3 individual Tensors that make up a SparseTensor (*/indices, */values, */shape) and then save the model using these tensor infos. The downside is that when the model is being deployed using Tensorflow Serving, the value to be scored has to be supplied in that same format, meaning as incides/values/shape individually.

Thank you for the fast reply, I will try it as soon as possible!


Added app id to retail mobile qr code

WHY are these changes introduced?
This PR adds the appId to the mobile QR code generated for POS. The appId is required for certain APIs that POS makes available to UI extensions. In a dev environment, this is the only way for POS to access the appId for a dev app.
WHAT is this pull request doing?
It checks to make sure the app exists, and if so adds the appId in the URL QR code.
Checklist

[ ] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[ ] I've considered possible documentation changes
[ ] I've made sure that any changes to dev or deploy have been reflected in the internal flowchart.

@vividviolet interesting. @henrytao-me do you know how Shopify mobile gets the appId? The reason why we need this is because the adminGenerateSession endpoint requires the app ID as a parameter. So as we're writing out the API for the app extension to be able to call that, we're trying to figure out a way to get that app ID in the dev environment.


A question about INFO - 2023-10-10 21:01:41,619 - test - [03201|05263],mdx:0.0624. what is mdx?

Please refer to
https://github.com/gitouni/CalibNet_pytorch/blob/25fdfa519ea0963d80ad76a0817af886cb144442/test.py#L67-L71
It is the mean of the defined residual vector res_npy.

Thanks for the answer, I would like to ask why the sequences 11, 12 and 13 must be present at the same time when testing, can't we just test a particular sequence?


 To read more about doctor command visit: https://github.com/react-native-community/cli/blob/main/packages/cli-doctor/README.md#doctor
error Failed to install the app.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.`
which is FALSE : the message was more explicit on version 0.71 ! (saying that gradlew cannot be run)
You can search the problem a long time with such a message !!
React Native Version
0.72.5
Output of npx react-native info
System:
OS: Linux 6.1 Debian GNU/Linux 12 (bookworm) 12 (bookworm)
CPU: (8) x64 Intel(R) Core(TM) i7-6920HQ CPU @ 2.90GHz
Memory: 52.71 GB / 62.62 GB
Shell:
version: 5.2.15
path: /bin/bash
Binaries:
Node:
version: 18.18.0
path: /usr/local/bin/node
Yarn:
version: 1.22.19
path: /usr/local/bin/yarn
npm:
version: 10.1.0
path: /usr/local/bin/npm
Watchman: Not Found
SDKs:
Android SDK: Not Found
IDEs:
Android Studio: Not Found
Languages:
Java:
version: 11.0.20
path: /usr/bin/javac
Ruby: Not Found
npmPackages:
"@react-native-community/cli": Not Found
react:
installed: 18.2.0
wanted: 18.2.0
react-native:
installed: 0.72.5
wanted: 0.72.5
npmGlobalPackages:
"react-native": Not Found
Android:
hermesEnabled: true
newArchEnabled: false
iOS:
hermesEnabled: Not found
newArchEnabled: Not found
Steps to reproduce
on linux : simple !

remove the exec permissions of gradlew
yarn run android

Snack, screenshot, or link to a repository
you can init a blank project on linux (if you can because of other bugs !)
and you make the 2 simple steps to reproduce.

I upgraded to 0.72.6
same issue , same repro

so you can see that android/gradlew execution permissions are none (-rw-r--r--)

We use the default permissions that Gradle creates for the Gradle Wrapper.
I can see that the execution bit is set correctly:
$ ls -la
drwxr-xr-x   9 ncor  staff   288 Oct 13 16:10 .
drwxr-xr-x  20 ncor  staff   640 Oct 13 16:10 ..
drwxr-xr-x   6 ncor  staff   192 Oct 13 16:10 app
-rw-r--r--   1 ncor  staff   601 Oct 13 16:10 build.gradle
drwxr-xr-x   3 ncor  staff    96 Oct 13 16:10 gradle
-rw-r--r--   1 ncor  staff  1902 Oct 13 16:10 gradle.properties
-rwxr-xr-x   1 ncor  staff  8497 Oct 13 16:10 gradlew
-rw-r--r--   1 ncor  staff  2868 Oct 13 16:10 gradlew.bat
-rw-r--r--   1 ncor  staff   249 Oct 13 16:10 settings.gradle

yes, I forgot to tell you but you can imagine :

I cannot on Linux create (init ) a project, so they come through Github from a Windows init
that is to say that at Gthub clone from a Windows initied project, that we still not have the adapted message to solve our issue.
so I will come back and close as soon I can init a project on Linux.

This and other similar issues related to the init command shoudl be opened against the CLI as that's where the init code lives.
https://github.com/react-native-community/cli/issues

Let's open another issue there and let's continue the conversation there

hi , on new 0.72.7 created , gradlew has the exec permission on linux - good - thks


Update SVT-AV1 to 1.2.1

SVT-AV1 1.2.1 changelog
[1.2.1] - 2022-08-15

Fix a crash at the end of the encode that may occur when an invalid metadata packet is sent with the EOS packet

Should be part of the latest set of auto-updates


lib-bucket-provision update image to quay.io/noobaa/pause

This image was taken from the "upstream" docker.io kubernetes/pause
image in order to remove the need to open also docker.io routes for
disconnected clusters.
See https://bugzilla.redhat.com/show_bug.cgi?id=1808401
Signed-off-by: Guy Margalit<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.

Flat operator directory structure is obsolete from 23-rd of October 2019, only nested directory structure will be accepted.

New Submissions

[ ] Has you operator nested directory structure?
[ ] Have you selected the Project Community Operator Submissions in your PR on the right-hand menu bar?
[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Is your submission signed?

Updates to existing Operators

[ ] Is your new CSV pointing to the previous version with the replaces property?
[ ] Is your new CSV referenced in the appropriate channel defined in the package.yaml ?
[ ] Have you tested an update to your Operator when deployed via OLM?
[ ] Is your submission signed?

Your submission should not

[ ] Modify more than one operator
[ ] Modify an Operator you don't own
[ ] Rename an operator - please remove and add with a different name instead
[ ] Submit operators to both upstream-community-operators and community-operators at once
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description about the managed Application and where to find more information
[ ] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[ ] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file a PR against this repo and explain your need
2 For more information see here

Hi @guymguym, please check if image is public.

@J0zi oops, changed to public. thanks.

/test CVP

@guymguym, please add a 'status' field to your Custom Resource. It should be mandatory now.

@J0zi I don't think I am missing a status.
The issue is the same everytime for this CRD-only operator - that the operator itself is a pod that does nothing and scorecard does not see the operator setting a status.
See the previous PR too - https://github.com/operator-framework/community-operators/pull/1289#issuecomment-592191700

@guymguym, you are right, there is a dummy pod. It will help us to mention it bellow PR form next time.
Please consider bumping patch version. Because it can affect users in some cases after deleting image from old location.

Hi @guymguym, do you mind bumping patch version? I think we are close to merge.

I think it's ok to merge this without bumping the patch version - the functionality hasn't changed.

Thanks @J0zi @dmesser for following up on this, I dropped this ball for a bit.


add Taylor Thomas as wg wasm chair

In light of the work that Taylor Thomas has been leading the Wasm Working Group and his contributions to the broader community in Wasm and Kuberentes, we would like to add Taylor as a chair for the WG.

/cc @nikhita @raravena80
related to: https://github.com/cncf/toc/issues/1237

Enthusiastic +1 from me. Thank you Taylor!

+1


Exception handling in transition

Hi there and thanks for the great library.
I'm experiencing a little problem with exceptions occurring during state transition functions. At the moment - at to the point of my understanding - If an exception is thrown within a transition an event is fired which at least allows us to log the error, but the transition is still executed.
Is there a possible way to reject a transition in case of an exception? I.e. the state does not change?
Thank you very much.
Merlin

Hi Merlin,
Thanks!
The state machine was designed so that the caller of the state machine does not have to deal with exceptions. Therefore exceptions are all caught and never handed over to the caller.
When I designed the state machine, I decided that the state machine always change state on an event when there is a suitable transition, regardless of whether there is an exception in an action or not. There is no simple way around this without changing some internal code. I always fired an error event onto the state machine so that it entered an error state on an exception. You can queue a new event during executing a transition action. I did this because we used to model external devices with this state machine and an exception is always an error in some way.
If this is not possible in your situation, then it probably would be best to change the code to not switch state in case of an exception.
That would require that you would have to return whether an exception occurred on line
https://github.com/appccelerate/statemachine/blob/6ae943467fca0472887aefeeeb9d387cb17354f4/source/Appccelerate.StateMachine/AsyncMachine/Transitions/TransitionLogic.cs#L78
and if so not executing the following line.
Feel free to copy the code into your solution and change it.
I hope that helps you.
Happy coding
Urs

Hi Urs,
Thanks for the reply.
I see your point here. And having looked at the code I understand that it is not a "quick one" to implement this.
For the record, my suggestion here is: In the Exception-Event handlers, introduce a flag in the EventArgs to reject the transition.
Handling of this, however, is not trivial, as the Exit-Action is usually already performed when the transition Action is executed. I see where this is going :-)
Having a transition to an error state does seem like a suitable workaround. I will see if that works for me.
Thank you again.
Merlin


Jlaney patch 1

Pull Request Test Coverage Report for Build 168

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 59.768%

Totals

Change from base Build 163:
0.0%

Covered Lines:
271

Relevant Lines:
429

💛  - Coveralls


[cairo] update to 1.7.6

Describe the pull request

What does your PR fix?
Fixes #23848

Which triplets are supported/not supported?

Yes

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

This error is not introduced by this pr, it is related to ci basline

All feature are tested successfully in the following triplet:

x86-windows
x64-windows


Correct a misspelt word

Once you've confirmed you are targeting the correct daemon, run the container with the following command:
                                            ^^^^^^^
                                            this word was misspelt

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

ID: 51faeed1-77c1-c8c6-c097-6806a3bbdf85
Version Independent ID: 84597429-d2e1-71ed-ee5d-fd2d66dcbff7
Content: Set up Linux containers on Windows 10
Content Source: virtualization/windowscontainers/quick-start/quick-start-windows-10-linux.md
Product: windows-server
Technology: windows-containers
GitHub Login: @taylorb-microsoft
Microsoft Alias: jgerend

@emanuelen5 Thank you for catching that, we will get this fixed right away.


Redirect looping

When I was using this:
RewriteEngine On
RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule ^(.*)$ index.php [QSA,L]

I got 500 error. error_log shows:
[core:error] [pid 39496:tid 2116] [client <IP_ADDRESS>:63803] AH00124: Request exceeded the limit of 10 internal redirects due to probable configuration error. Use 'LimitInternalRecursion' to increase the limit if necessary. Use 'LogLevel debug' to get a backtrace.

And this works:
RewriteEngine On
AddType application/x-httpd-php5 .php
RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule ^(.*)$ /index.php?/$1 [L]

Do you still have this issue?

I have tried both of your examples. Last documented variant should work, Check Readme.
RewriteEngine On
RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule ^(.*)$ index.php [QSA,L]

If you still have this issue, please create new issue ticket.


Guess punctuation userscript doesn't work under Greasemonkey in Firefox

I'm using Firefox 92.0 with Greasemonkey and don't get the "Guess punctuation" button on edit pages. I've tried with Tampermonkey in Chrome and it works fine.

I tried Violentmonkey in Firefox instead and it works, so either you can close this or you can take a look into why it doesn't work with Greasemonkey if you care. Thanks.

I just tried it myself (using Firefox 92.0 and Greasemonkey 4.11) and indeed the buttons did not show up:

Script error in [Greasemonkey script https://github.com/kellnerd/musicbrainz-bookmarklets/MusicBrainz: Guess Unicode punctuation; version 2021.8.27]:
ReferenceError: $ is not defined

This means that the script does not find the jQuery library which is bundled with MB's code.
Dropping the jQuery dependency is already on my todo list, so this userscript might work with Greasemonkey in the future. Personally I am using Violentmonkey on Firefox since Greasemonkey 4 caused problems with some of my most used userscripts.


Loop improvemenet

Import Pull-Request sisimai/rb-Sisimai#111 as possible.

Coverage remained the same at 95.296% when pulling 2fedf957189d889fae62e18444004ecc50f05e4d on loop-improvemenet into decc05ba9536eec59656bd2e40bc38549f441ef5 on master.


datetime support for min and max functions

It looks to me I'm doing something wrong, but I can't find the reason.

It seems that when I select the row using min / max functions, an additional hour is added to the date. The same example using pandas works fine. Could you please help me?

Hi @franperezlopez, Thanks for the report!
Actually I couldn't reproduce the behavior in my local, but I had something in mind and submitted a PR #2077.
Could you help test the PR in your end?

Hi, I'm sorry but I had no time for testing your branch ... did you try updating the local time of your machine? I live in UTC+1 so I presumed the difference was because that

Hi @francopatano, I merged the PR. Could you try with the latest master? Thanks!

works on my machine 👍
thanks for the fix. keep up the good work!!

Thanks again for the report and confirming!


anytype: 0.35.4 -> 0.35.25

Description of changes
Updating to the most current version.
First PR request for nixpks. Let me know of issues.
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

[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 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

[ ] Fits CONTRIBUTING.md.

Welcome to our small community :-) I'm not familiar with Anytype, but I just tested running the executable and this worked. I would merge this PR if I had the permissions. :-)

Thank you for taking the time to check. @iblech
(Einen schönen Sonntag wünsche ich :smile:)


feat: migrate to new calldata

Fixes <no issue>
What

implemented new calldata encoding

Why

first step for migration to new genvm

Testing done

python file is taken from genvm, untested here
ts parser is tested to some extent
some manual testing was done

Decisions made

Checks

[x] I have tested this code
[x] I have reviewed my own PR
[ ] I have created an issue for this PR
[x] I have set a descriptive PR title compliant with conventional commits

Reviewing tips

For now there is a problem with "json" input by user for obvious reasons. I can allocate some time to fix them, but I am not great in reactive ui's
User facing release notes
New calldata string representation is slightly different from json:

it has bytes type (encoded in text as b#<hex bytes>)
it has address type (encoded in text as addr#<64 hex digits>
keys can omit "

Actually I found out that probably what we mean by address is 20 bytes and not 32 as in this PR. It is sort of confusing:

account address is 20 bytes
word size (in eth) is 32 bytes, slot addresses in eth are also 32 bytes

If I fix that right now, will it break js sdk? @cristiam86


closes #81 Run tests randomly

Codecov Report

Merging #82 (d61b9a1) into main (84c244b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #82   +/-   ##
=======================================
  Coverage   99.21%   99.21%
=======================================
  Files           9        9
  Lines         380      380
=======================================
  Hits          377      377
  Misses          2        2
  Partials        1        1

Continue to review full report at Codecov.

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


⚠ ClamAV has degraded performance

In e0801ed, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 81 ms

Resolved: ClamAV performance has improved in 09e06be.


format codes

What this PR does:
format pixiu codes

fix https://github.com/apache/dubbo-go-pixiu/issues/215

fix https://github.com/apache/dubbo-go-pixiu/issues/214

Codecov Report

Merging #216 (4cadf9a) into develop (c5b3b0b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #216   +/-   ##
========================================
  Coverage    39.71%   39.71%
========================================
  Files           40       40
  Lines         2397     2397
========================================
  Hits           952      952
  Misses        1343     1343
  Partials       102      102

Impacted Files
Coverage Δ

pkg/client/dubbo/dubbo.go
23.20% <ø> (ø)

pkg/client/mapper.go
80.95% <ø> (ø)

pkg/config/config_load.go
62.61% <ø> (ø)

pkg/filter/metric/metric.go
37.03% <ø> (ø)

pkg/filter/ratelimit/init.go
72.72% <ø> (ø)

pkg/pixiu/listener.go
19.64% <ø> (ø)

pkg/registry/util.go
0.00% <ø> (ø)

pkg/registry/zookeeper.go
63.63% <ø> (ø)

Continue to review full report at Codecov.

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


Strong name validation failed for package OpenCvSharp3-AnyCPU <IP_ADDRESS>70110

I have some problems with running OpenCvSharp3 using IIS because strong name validation fails:
sn -vf OpenCvSharp.Blob.dll
Microsoft (R) .NET Framework Strong Name Utility  Version 4.0.30319.0
Copyright (c) Microsoft Corporation.  All rights reserved.
Failed to verify assembly -- Strong name validation failed.

However, it still working well for package OpenCvSharp3-AnyCPU <IP_ADDRESS>61105

Thanks! This problem is fixed in the next release.
https://github.com/shimat/opencvsharp/commit/61563222640e4c758538eefe53629d6331c2900a
sn -vf OpenCvSharp.Blob.dll

Microsoft(R) .NET Framework Strong Name Utility Version 4.0.30319.0
Copyright (c) Microsoft Corporation. All rights reserved.

Assembly 'OpenCvSharp.Blob.dll' is valid


[Bug]: Error in resuming groupchat.

Describe the bug
i am getting error while resuming the group chat.
i have a function name get_policy_details
chat_history :
[{'content': 'i want to know policy details', 'name': 'chat_manager', 'role': 'user'}, {'content': 'Sure, I can help with that. Could you please provide me with your policy number?', 'name': 'policy_details_agent', 'role': 'user'}, {'content': 'its PDPAC045795', 'name': 'customer', 'role': 'assistant'}, {'content': '', 'function_call': {'arguments': '{"policy_number":"PDPAC045795"}', 'name': 'get_policy_details'}, 'name': 'policy_details_agent', 'role': 'assistant'}, {'content': "{'api_result_type': 'Success', 'concepts': {'policyDetails': {'policyDetails': {'account': {'name': 'test policy', 'coApplicantNames': '', 'email'<EMAIL_ADDRESS>'geographicalArea': 'The Cayman Islands', 'peroidOfInsurance': '08/14/2024 to 08/14/2025', 'totalPremium': '958.05'}, 'vehicleCoverageDeductibles': {'vehicleDetailsList': {'Items': [{'vehicle': {'insuredValue': '7001-10000', 'year': '2025', 'makeCode': 'Brightdrop', 'modelCode': 'EV600', 'vIN': '444', 'registrationNo': '', 'makeOther': None, 'modelOther': None, 'vehicleType': 'Private Car', 'authorizedDrivers': ''}, 'coverages': {'Items': [{'caption': 'Collision Coverage:', 'value': 'Yes'}, {'caption': 'Fire and Theft Coverage:', 'value': 'Yes'}, {'caption': 'Catastrophe Coverage:', 'value': 'Yes'}, {'caption': 'Rental Coverage:', 'value': '7Days'}]}, 'deductibles': {'Items': [{'caption': 'Collision', 'value': '750'}, {'caption': 'Fire and Theft', 'value': '500'}, {'caption': 'Catastrophe', 'value': '1000'}]}, 'limitsOfLiabilities': {'thirdPartyPropertyDamage': '$250,000', 'perPersonThirdPartyBodilyInjury': '$1,000,000', 'perEventThirdPartyBodilyInjury': '$5,000,000'}}]}}}}}}", 'name': 'get_policy_details', 'role': 'function'}, {'content': 'Here are the details for the policy number **PDPAC045795**:\n\n### Account Information:\n- **Policy Name:** Test Policy\n- **Email:**<EMAIL_ADDRESS>**Geographical Area:** The Cayman Islands\n- **Period of Insurance:** 08/14/2024 to 08/14/2025\n- **Total Premium:** $958.05\n\n### Vehicle Coverage and Deductibles:\n- **Vehicle Type:** Private Car\n- **Make:** Brightdrop\n- **Model:** EV600\n- **Year:** 2025\n- **Insured Value:** $7001-10000\n- **VIN:** 444\n\n#### Coverages:\n- **Collision Coverage:** Yes\n- **Fire and Theft Coverage:** Yes\n- **Catastrophe Coverage:** Yes\n- **Rental Coverage:** 7 Days\n\n#### Deductibles:\n- **Collision:** $750\n- **Fire and Theft:** $500\n- **Catastrophe:** $1000\n\n#### Limits of Liabilities:\n- **Third Party Property Damage:** $250,000\n- **Per Person Third Party Bodily Injury:** $1,000,000\n- **Per Event Third Party Bodily Injury:** $5,000,000\n\nIf you have any more questions or need further details, feel free to ask!', 'name': 'policy_details_agent', 'role': 'user'}]
when i try to resume this chat  it gives the error as
  File "/Users/shubhamgilada/Developer/scripts/autogen/mapping/autogen/Flask/Extented_framework/usecases/nested_chat/run.py", line 104, in <module>
    result = policy_group_chat("abc", abc)
  File "/Users/shubhamgilada/Developer/scripts/autogen/mapping/autogen/Flask/Extented_framework/usecases/nested_chat/group_chat_get_policy_details.py", line 323, in policy_group_chat
    last_agent, last_message = manager.resume(messages=messages_json)
  File "/Users/shubhamgilada/miniconda/lib/python3.10/site-packages/autogen/agentchat/groupchat.py", line 1202, in resume
    self._valid_resume_messages(messages)
  File "/Users/shubhamgilada/miniconda/lib/python3.10/site-packages/autogen/agentchat/groupchat.py", line 1391, in _valid_resume_messages
    raise Exception(f"Agent name in message doesn't exist as agent in group chat: {message['name']}")
Exception: Agent name in message doesn't exist as agent in group chat: get_policy_details```

### Steps to reproduce

_No response_

### Model Used

gpt4o

### Expected Behavior

it should resume.

### Screenshots and logs

_No response_

### Additional Information

_No response_

update:
i have added one more condition to _valid_resume_messages
    def _valid_resume_messages(self, messages: List[Dict]):
        """Validates the messages used for resuming

        args:
            messages (List[Dict]): list of messages to resume with

        returns:
            - bool: Whether they are valid for resuming
        """
        # Must have messages to start with, otherwise they should run run_chat
        if not messages:
            raise Exception(
                "Cannot resume group chat as no messages were provided. Use GroupChatManager.run_chat or ConversableAgent.initiate_chat to start a new chat."
            )

        # Check that all agents in the chat messages exist in the group chat
        for message in messages:
            if message.get("name") and message.get("role") != "function": # here
                if (
                    not self._groupchat.agent_by_name(message["name"])
                    and not message["name"] == self._groupchat.admin_name  # ignore group chat's name
                    and not message["name"] == self.name  # ignore group chat manager's name
                ):
                    raise Exception(f"Agent name in message doesn't exist as agent in group chat: {message['name']}")```

 not sure if its a correct fix

Can I just check that when you resume the group chat has the same agents as the original group chat?

Can I just check that when you resume the group chat has the same agents as the original group chat?

I also have this problem.
It occurred when a external UserProxyAgent initial_chat with a group_chat_manager. Becuase the UserProxyAgent is from external user to talk with the group_chat_manager, when resume, the message histories contain the external UserProxyAgent'name which is not in the group_chat internal agents.
But I thank this should be supported to work fine.Because UserProxyAgent is always from external when I set the group_chat as an indenpendent workflow


Some security issues about this project.

I am a security researcher and recently used our designed Golang code scanning tool during security audits. During this process, I regretfully discovered some vulnerabilities in your project that may affect security. I've described these issues in the attachment.Looking forward to your reply.
details.pdf

You clearly took no time to understand the context of what you are reporting. Please stop wasting our time.


virtual-scroll: Change detection not triggered when using DataSource that uses viewChange.

Bug, feature request, or proposal:
Change detection is not triggered when you scroll if you are using a custom DataSource that utilizes viewChange.  I have no idea what the appropriate way to fix this is, but this is what I did and it solved my problem:
const onRenderedDataChange = CdkVirtualForOf.prototype['_onRenderedDataChange'];
CdkVirtualForOf.prototype['_onRenderedDataChange'] = function (this: any) {
    onRenderedDataChange.apply(this, arguments);
    if (!this._changeDetectorRef) {
        this._changeDetectorRef = this._viewContainerRef.injector.get(ChangeDetectorRef);
    }
    this._changeDetectorRef.detectChanges();
}

What is the expected behavior?
Change detection should occur when the underlying DataSource's data changes.
What is the current behavior?
Change detection does not occur when the underlying DataSource's data changes.
What are the steps to reproduce?
This demo is almost identical to the one that I created for #11289.  In that demo I used setInterval(noop, 1000) to suppress this problem.  That is obviously a horrible solution, but I was trying to focus on that particular bug.  In this demo, just scroll to the bottom and the words "Loading..." will not disappear like they should.  If you put setInterval(noop, 1000) in the constructor or apply my patch mentioned above, the problem will go away.
https://stackblitz.com/edit/angular-material2-issue-czyxyd
What is the use-case or motivation for changing an existing behavior?
It's broken.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
@angular/cdk-experimental 6.4.6
Is there anything else we should know?
No.

Hmm it looks like this may be caused by the fact that cdkVirtualFor is emitting on viewChange outside the NgZone. If I create an Observable based on viewChange and put it back inside the zone, change detection seems to trigger: https://stackblitz.com/edit/angular-material2-issue-sgwfxs?file=app/virtual-scroll/virtual-data-source.class.ts (of course there's still a mess of errors, I'll need to look into that too). I'll create a PR to emit in the zone so this workaround isn't needed

Awesome!  Thank you.


Add support for importing schematic files

And it seems that the Minetest plugin has already some support for it:
https://forum.minetest.net/viewtopic.php?p=192890#p192890

Analysis of Minetest supporting of Schematics:
https://forum.minetest.net/viewtopic.php?f=53&t=24651

Pretty related to: https://github.com/juntosdesdecasa/mcthings/issues/18

Start from here: https://github.com/arpruss/raspberryjammod/blob/master/mcpipy/import.py

So next steps:

Download and try to read some schematics to understand the contents and recheck that they are in NBT format
Try to use standard libraries so all will work in the future
In there are specific logic like the one in setBlockWithNBT include it in our library

Ok, let's use as reference for getting schematics: https://www.planetminecraft.com/projects/?share=schematic It seems to be the place with more contents (worlds, schematics ...). But let's try better the same schematics than alan in: https://www.instructables.com/id/Star-Wars-and-Star-Trek-in-Minecraft-with-Raspberr/

The first step is just to download it and to try to load it using the NBT python library in order to take a look to the contents.

Ok, first step done:

The NBT contents are basically tags (The format is designed to store data in a tree structure made up of various tags).  And yes, we have TAG_compound with tags inside it (leaves):

Ok, next step is to:

Identify blocks: individual or groups? type of block? position?
Draw the blocks inside MC/MT

After a first analysis of the data, this format is not the same than the one supported in https://github.com/arpruss/raspberryjammod/blob/master/mcpipy/import.py. So time to do our own analysis.
Taking a look to the tags, the dimension of the schematic is

so we have: 17229 = 3366 blocks. And this is the size of:

So it is clear where the blocks are defined. And for each block in the above list, we have a value from 0 to 13. And we have a palette field with schematic['Palette'].tags with values from 0 to 13. So it seems it is defining the kind of block in minecraft. So we just need to understand the order iin which blocks_data is filled.

Ok, the format inside the schematic files is different. So we need to detect the version inside the file. And for the old version, there is no info about the version.

So let's focus first in the old format, the one supported in https://github.com/arpruss/raspberryjammod/blob/master/mcpipy/import.py

And the format description is at: https://minecraft.gamepedia.com/Schematic_file_format.

Blocks: Block IDs defining the terrain. 8 bits per block. Sorted by height (bottom to top) then length then width—the index of the block at X,Y,Z is (Y×length + Z)×width + X.
Data: Block data additionally defining parts of the terrain. Only the lower 4 bits of each byte are used. (Unlike in the chunk format, the block data in the schematic format occupies a full byte per block.)

But, how is the Raspi Python working in 1.15.2? Because of:
[04:46:45] [Server thread/WARN]: Initializing Legacy Material Support. Unless you have legacy plugins and/or data this is a bug!
[04:46:49] [Server thread/WARN]: Legacy plugin RaspberryJuice v1.11 does not specify an api-version.
[04:46:49] [Server thread/INFO]: [RaspberryJuice] Loading RaspberryJuice v1.11

This is why it works no matter that we are using blocks ids pre-flattening.

The implementation it is easy: just read the blocks (Blocks), get the additional data (Data) (only 4 bits) and show it when setBlock at the right position. We can improve the rendering if we detect Y-stripes (or group of Y-stripes) that can be drawn together using setBlocks because they share the material.

O, my first schematic drawn:

Compare with

The above schematic was created in 2012 and we are recreating it 8 years later. So all the creations for 8 years can be reused with current status of the technology. I am not using yet the Data info.

Another one:

Compare with https://www.minecraft-schematics.com/schematic/11813/:

This one needs more care with the missing data info not supported yet.

Ok, time to add support for the data field to build schematics. According to the doc about the format:
https://minecraft.gamepedia.com/Schematic_file_format
«Data: Block data additionally defining parts of the terrain. Only the lower 4 bits of each byte are used.» So we need to extract this 4 bits and convert them (to what type? int?) to the data of the block. If the 4 higher bits are 0, it should work without any additional change. But let's try.

Much better now using the data info:

Ok, let's do a test with a complex schematic before closing this issue:
https://www.minecraft-schematics.com/schematic/9676/#

It needed around 15-20s to do it but it is awesome. So with the current code, we have a decent support for loading schematics!

Closing this issue!


PHPVMS not working behind reverse proxy

Describe the bug
links in reverse proxy still contain internal address
Version
version 7.0.0-dev
To Reproduce
Steps to reproduce the behavior:
1 setup a server. and check if it is working (http://internal.local works correctly)
2. Setup a Reverse proxy server (I use Ngingx) and add a proxy path

location /phpvms {
proxy_pass http://internal.local/phpvms/;
proxy_set_header X-Real-IP $remote_addr;
proxy_set_header X-Forwarded-For   $proxy_add_x_forwarded_for;
}

Open External link
Most part of pages still openned via  http://internal.local which is not accessible from the internet. External link is not working
See error

Expected behavior
Site opens correctly from external and internal web pages
Screenshots

Additional context
I tryed this manual, but it is not working.
https://lavalite.org/blog/using-laravel-routes-in-reverse-proxy

Laravel does not work well in subdirectories.  You'll need to point it either a subdomain or a root domain. There is an example vhost for nginx here:
https://github.com/nabeelio/phpvms/blob/dev/resources/docker/nginx/default.conf

No this is not subdomain. this is REVERSE PROXY

You've set the location block to a subfolder - Laravel doesn't work right in subfolders. The vhost needs to be at the location / of either a domain or subdomain or the assets won't be found. You also need to point the vhost to the /public folder. Refer to the vhost that I've posted above.


Add a test device for YARP based ROS Publisher

This PR adds a test device for YARP based ROS Publisher added in https://github.com/dic-iit/bipedal-locomotion-framework/pull/156.
Current implementation cannot fit in to a CI testing framework, since the process requires roscore to run (which is not our dependency), further it requires an additional subscribing mechanism to check if the data is published properly or not.
So we will stick to manually running the test device, at the moment. We could fix it when we have more time dedicated for software-engineering tasks.

I have moved this device into devices/examples folder and made the compilation of the example devices optional.

This PR adds a test device for YARP based ROS Publisher added in https://github.com/dic-iit/bipedal-locomotion-framework/pull/156.
Current implementation cannot fit in to a CI testing framework, since the process requires roscore to run (which is not our dependency), further it requires an additional subscribing mechanism to check if the data is published properly or not.
So we will stick to manually running the test device, at the moment. We could fix it when we have more time dedicated for software-engineering tasks.

Can you open an issue on this? I may have a few ideas on how to do this easily in the future.

If you agree we can merge it

If you agree we can merge it

I have squashed the commits, once the CI passes, we can merge! Thank you! :)

I have squashed the commits, once the CI passes, we can merge! Thank you! :)

macOS is failing because of a known issue not related to this PR https://github.com/dic-iit/bipedal-locomotion-framework/pull/167#issuecomment-756859285

macOS is failing because of a known issue not related to this PR https://github.com/dic-iit/bipedal-locomotion-framework/pull/167#issuecomment-756859285


[FLINK-5991] [state-backend, streaming] Expose Broadcast Operator State

This PR exposes broadcast operator state through the CheckpointedFunction interface, by adding broadcast state access methods to OperatorStateStore.
Since the functionality was already internally available and had test workarounds using casting, this PR simply removes those casts and properly tests the functionality through the vanilla OperatorStateStore interface.

R: @aljoscha @StefanRRichter

The PR re-introduces methods that have been removed from the public interface before the release. Reason the remove the methods was that we had too little time to make a final decision on the API. As this matches my suggestion, +1 from me. However, @aljoscha should also give his thumps up because I think is this is the point where we should catch up with our API discussion.

Looks good in general. Some thoughts on refinement:

I think we should not really encourage the use of Java Serialization, so I would suggest to remove the serializable shortcut for broadcast state. We have it for the default operator state only as a bridge to mimick the previous behavior (easier migration)

Since broadcast state already defines its redistribution pattern (broadcast), the shape of the datastructure is orthogonal. We can for example also have broadcast map state, broadcast value state, etc. Hence, I would suggest to take "list" somewhere into the name of the methods.

I would like us to take some time and think about the name. We are about to introduce a thing called "broadcast state" somewhat soon in the effort to make streaming joins possible. This broadcast state will provide an interface very similar to the current keyed state (we'll probably reuse the descriptors and state interfaces) but be checkpointed only on one operator because we only allow modifications based on broadcast input.
I propose to rename the state we're talking about here to UnionState (or something) similar because what it does is take the snapshotted state from all operators and when restoring sends the union of that to all operators.

+1 on not exposing the Java Serialisation shortcut, btw. I was very unhappy that we even have it for the normal operator state. 😃

I wonder if there could also exist a case for broadcasting operator state (non-keyed), where only one operator instance is selected as sender and all others receive on restore. Furthermore, the union aspect may (or may not) happen at restore time, but not at the time that a user requests this state. For what this currently does, I think ReplicatingState describes it pretty well. Broadcast would be a good description from the operator's perspective: it broadcasts the generated data to all peers on restore.

@StefanRRichter I think I was faster ... 😉

Nice following the discussion here :)
I think I'm also leaning a bit more towards naming it UnionListState.
From the user's prospective, it seems to be more clearer if the name of the type of state describes how the state will be restored.
This would imply that it would be ideal to rename the repartitionable list state methods, though.
Another follow up question regarding the Java serialization shorcuts:
should we also deprecate the getSerializableListState method to prepare it for removal?

I think @StephanEwen wanted to keep a simple, intuitive way for users to register their state that does not require them to think about serializers etc.. While I understand this point, I am also leaning towards @aljoscha 's feeling that using this should at least be disencouraged.

We can deprecate, because it was added to provide a smooth path from Flink 1.1.

@StefanRRichter and @tzulitai what's the state of this? Can we go forward with calling it union state?

Fine with me.

@aljoscha @StefanRRichter I'll update the PR today, thanks for the reminder.

@aljoscha @StefanRRichter Rebased on master to resolve conflicts.
Also, the follow-up commits addresses the following:

Remove / deprecate the Java serialization shortcuts. We have some connectors that use the deprecated shortcut method, but I think since changing those will require some more involved migration work, I propose to fix those as a separate JIRA / PR.

According to Stephan's suggestion, refine the current naming of the operator state methods. getOperatorState is renamed to getListState, and the new union state is named as getUnionListState.

Further question regarding deprecating Java serialization shortcuts: do we then also have a cause to deprecate the ListCheckpointed interface, since that itself is a shortcut also?

This looks very good. 👍
And yes, I would be in favour of deprecating ListCheckpointed but let's see what @StefanRRichter and @StephanEwen think.

Isn't the 90% cases already covered with StateDescriptor(stateName, typeClass)?

Well, this way is easy, but still slightly more complicated that the original Checkpointed. Personally, I would also consider that an easy-enough replacement. That is why my vote was also on deprecation.

I've opened a separate JIRA to deprecate ListCheckpointed. Lets keep this PR self-contained in just refining the OperatorStateStore interface.
I think this PR is still lacking an update to https://ci.apache.org/projects/flink/flink-docs-release-1.2/dev/stream/state.html#using-managed-keyed-state before its good to go. Adding this ;)

Updated managed operator state docs.
@aljoscha @StefanRRichter could you have a final look at it? If there's no other problems I'll proceed to merge this. Thanks :)

Nice update to the docs! And yeah, a separate issue for deprecation is very good. 👍

Very good work and thanks for improving the documentation. I like the update. From what I have seen in the past, some user have mistaken the list-nature of the operator state and simply dumped lots of small elements in the list, that should not actually be the unit of repartitioning and sometimes even logically belonged together. I wonder if the different semantics in list state between the operator state and the keyed state can be confusing and error-prone for users and what we could do about this? A method called getListState might be a step in the wrong direction.
Besides this, +1 from me.

I'm not so sure, but I think Stefan has a point with the potential confusion. Especially if in the future, if both the keyed state and the operator state are registered through the initialization context, it can definitely be confusing if both states have the getListState or even getMapState ... methods, but actually have completely different semantics.
To address @StefanRRichter's concerns, perhaps we could

Try to emphasise the semantic differences in the docs / Javadocs.
Consider renaming getListState to getXXXListState? "XXX" needs to imply the round-robin redistribution scheme.
I was also wondering about another possibility, which follows a completely different approach: getRepartitionableListState(descriptor, REDISTRIBUTE_MODE).

I would stick to getListState() and rely on users knowing on what kind of state store they are calling the method.
In general, users should not use this interface too much, I think it's mostly there for sources, such as Kafka.

In general, I agree with @aljoscha, however I have already heard at least one case where this confusion was leading to problems. Given the short time that we provide this feature, this could also come up more often in the future and is not so nice to change after the fact.

I'd say lets keep the naming as is, since in general all of us agree on that already.
I'll try to do some more work on the docs and Javadocs to make sure that at least the documentation clearly explains and emphasizes the semantical differences.

Merging after Travis turns green


Can't install package

i am trying to install the packages, but not installing
react-native-cli: 2.0.1
react-native: 0.57.8
npm: 6.6.0
yarn: 1.12.3
command : npm install react-native-audio-recorder-player --save

You should link the library. This is native dependent!


New Material 3 InkFeature

Android 12 introduces a new patterned touch effect which looks different to the current InkWell implementation.

Flutter will match this with the upcoming Material You, but will we still be able to use the old InkWell once updated? Will we be able to use the old InkWell for older platforms and the new one starting with Android 12?

InkWell doesn't even yet support the Android 9 ripple effect. https://github.com/flutter/flutter/issues/49957

CC @clocksmith

blocked by https://github.com/flutter/flutter/issues/93498

I'm in progress on a proposal (see: the issue) for some changes that would improve the fidelity of the current InkRipple/InkSplash animations to the implementations for Android versions 11 and below (which I'm assuming we'll be continuing to support for the foreseeable future?). It occurred to me recently that it would be worth building an understanding of how we're planning on implementing the Material 3 version of the ripple animation, so I can identify where the changes I'm proposing may become irrelevant or unhelpful. So I've been unsuccessfully looking to find some detail on that—has a plan been decided/shared publicly yet?
For example, I see that this is blocked by an issue with the shader-related stuff—is there a plan to add a new splash factory for the new animation, or is that going to be solved by some other means?

Any update?

Hi, this is ready for the C++ engine. Going to work with https://github.com/flutter/flutter/issues/73163 for testing

@willlockwood I added some notes for the proposal to revamp the ripple. It doesn't appear that it will change the m3 ripple since its a separate splash factory.
On the subject of splash factories, one additional suggestion is to rename splashFactory to inkFactory so InkSplash is better understood as a "splash" type of ink, etc.

WIP https://github.com/flutter/flutter/pull/96598

blocked by https://github.com/flutter/flutter/issues/99884

I'm having issues with the ripple (and its green) effect being applied to both NavigationBar and NavigationRail, image. Any solution?


MW 1.28 Undefined variable error

Notice: Undefined index: loop in /extensions/Sm2Shim/Sm2Shim.Class.php on line 75
And autostart on line 70

Fixed via f7654f68eb46ff3b0014855656b3cbcb9d651ab5.


Refactoring

Note: Migrating this thread from email to GitHub Issues for transparency.
@rachlim wrote:

Hey all!

So sorry to work-email you guys during your holidays- really hope you are enjoying the break!

Here are several things that we are going to work on for the new term, that hopefully minimise pressure/workload on all of you by being more efficient and ensuring more effective learning on the part of the students. The main objective of this is to improve the skills/learning experience of the struggling students, teaching/instruction as you normally go about it will continue, Seb! I'm aware that some of it is not ideal but it's what we have to work with and I'm really looking forward to working as closely with you guys as possible on this!

Half-half Teaching/Labs Split
As a result of a lot of feedback indicating that they would like more labs and hands-on activity, class will be divided into teaching in the morning based on objectives set for the day (plus whatever cool live coding/activities Seb has in mind) and labs/hands-on or wax-on, wax-off activities in the afternoon that are tied to those objectives.

Class Split
During labs, students will be split into 2 groups, the struggling students and the faster students. The struggling students will go into a circle at the side of the room where Zell will go over the days objectives + basic knowledge (e.g functional Javascript) again, in step-by-step, structured way. I know Seb and Cheryl, you are concerned about managing the rest of the class, but this is the best way we can cater to the different learning styles without leaving people significantly behind. Let's try this for 2-3 days first, and then check-in to see how we can improve it?

Project Structure
We'll start implementing more structure to the projects (for the weaker students). I think Seb's style of 'anything goes' worked super well for the faster students and if you'd like, we can maintain that around certain parameters. For the struggling students, we'll pair them up with one another and have a very specific project they'll be working on (maybe even the same project) that they can be helped with along the way. We'll share out the project guidelines and parameters on Friday mornings after which they can start thinking about it and planning with your guidance, going back on the weekend to experiment and really getting into it on Monday. Fridays will be for presentations for all, no matter where they are on their projects.

Planning
We'll sit down on Monday or Tuesday (if you guys are free?) to talk about the different objectives Seb wants to cover for the remaining three weeks and to plan out how it will look with the inclusion of labs. Things we will consider: objectives not yet covered + objectives to recap/revise. We have roughly 13 more lessons, counting project week and project prep Fridays!

Speakers
While the speakers so far have been amazing and really added breadth to what the students are learning, we need to focus more on labs and classes for the next few weeks so the students get as much guided hands-on experience as needed, and if speakers do come in, it should ideally happen over lunch/for shorter time periods, once/twice a week. If speakers have already been arranged for class time and it's not an absolutely need-to-know skill, we can structure it such that the struggling students go out for their labs during the sessions.

Again, I'm so sorry to disturb you guys on holiday, and I really do hope that this is okay for all of you. Do let me know if there are aspects you think can be done better or more efficiently and let's talk about that! We're all on the same side and the goal at the end of this is to equip our students with developing skills, even if we may not necessarily be creating developers immediately.

Seb, if you could think about what you want to do next week, maybe for the first two-three mornings and what exercises you'd like them to do in the afternoons (if you have more time, maybe recap exercises or codealongs?), I will add that into the calendar (:

Thank you, everyone!

Regards,
Rachel

@cbas wrote:

Hi Rachel,

Appreciate the suggestions. Always keen to try more experiments on the class and see what works.

This worked great in the last week or two. Glad the feedback confirms our intuition. Would like to split each day into quarters: lesson/lab & lesson/lab. Doing a lesson for more than an hour tends to lose a lot of the students. The more granular the lessons, the better IMO. Have taught many mini-lessons during exercises and projects and they seem to stick with the students.

Parity is not my concern. That means I'm happy to split the class in half (or even smaller segments, even down to the individual level). Also means I'm happy the faster students go waaaay beyond the rest (as they have). Let's help everyone learn as much as they can. Hard to do this during a lecture-style class. Works well during labs with many smaller impromptu lessons based on missing skills we discover in specific students. One thing to watch out for is the self-fulfilling prophecy of dividing people into "weaker" and "stronger" groups. This is detrimental to both groups.

That's almost exactly much what we did on the second project. What totally failed was pairing weaker students together. No amount of coaching other than 100% dedication was enough. We don't have enough coaching bandwidth so I'd like to try pairing weaker and stronger students instead. This worked very well for some pairings. As far as "structure" is concerned, it’s important that students learn to solve problems, not just follow instructions. Programming requires an enormous capacity for creative thinking.

What do you mean by roughly 13 more lessons? There are a number of important topics I'd like to cover in the coming 5 weeks. These would certainly be useful to anyone who wants to be a programmer. Can you mark on the shared calendar which days are not available for lessons? I thought we'd have project week 9 & 12 but the remainder would be lesson weeks. I'll work on a list of topics.

Speakers have been hit & miss. Mostly hit, which is why I continued inviting them. I think the last one had students sleeping in their chairs, watching videos, and completely uninterested. Frankly I didn't enjoy that session either. However a few of the students looked extremely interested. Most of the other speakers have been amazingly well received by the class. Could this be an issue with the content?

@rachlim wrote:

Hey Seb!

Thanks for the email and thought put into it- I do genuinely feel the majority of the class is doing amazing and I'm always so blown away by how inspired and encouraged they are in the class.

LOVE this suggestion! Let's do this!
Yeah- I'll talk to the students about following Zell during the labs as a 'structured learning' group, and those who feel they might benefit from it can sit in if they so wish. It won't be segregating, more of differentiated learning.
I think pairing weaker and stronger students can result in the weaker student not really trying as much as the stronger student doesn't have the time/level of understanding to stop and explain things; particularly if the weak/strong divide is too large. Do you think it might work better if we paired the struggling students up, and for the other students paired weak/strong but gave them closer attention and made the objectives easier/more scaffolded? We can assign pairings next week, say Wednesday before we give them the brief on Friday?
Oh, when I said 13 lessons I assumed it would be chunked into morning teaching/afternoon labs. But breaking days into quarters would give us 26 lessons! I'll mark into the calendar right now- but for clarification, non-lesson days would the two Fridays before project week (as pre-project prep and discussion) and the project weeks. Which gives us 3 weeks - 2 days remaining, so 13 days.
I think speakers have been great, and most of the students agree but in their 1:1 feedback many have said they would rather spend the time practising/coding- perhaps we can find a happy balance for this by having fewer, and also I think making certain speakers' sessions optional especially for the struggling students. What do you think?

Regards,
Rachel

@rachlim Thanks for the calendar updates. :heart:
Why change project presentations to Friday? Last time we did presentations on Monday and Tuesday mornings. Worked quite well, I thought, as it allows an extra weekend to polish the project.
The thinking behind this was twofold,

That we are introducing projects in the week before (Friday) so they now have that weekend to work on it after initial advice and discussion, giving them a total of 7/8 days to work on the projects- same time! Presentations could be done in the afternoon after lunch, perhaps?
Given we have only got 3 weeks remaining (and 13 days of classes) it would be best to maximise those times for classes and keep projects to project weeks (particularly as the final week ends on a Friday and they will have to present on that Friday itself!

Also, I think confining projects to project week make the whole process clearer and more structured- after that week, if students want to work on their projects on their own time to further polish etc. that's fine but class time will be for class. Sound okay?

@cbas I've also done a draft schedule on the calendar for the quarter lesson/lab on Monday 4th Jan- is this time division okay by you? I can just replicate it across the rest of the days and you can fill in objectives when you decide what they are (: Let me know if you think the timings ought to be changed. Aziza also needs a 1.5 hour block for Outcomes sessions (talking CVs/Portfolios etc) on Thursday/Friday- if you think cannot afford to give up lesson time on Thursday we can do Friday after project brief!

Calendar
Reasoning makes perfect sense now. Hadn't realised the last project week doesn't have a Monday to follow up. :confounded: :cry:
I further divided the mornings to start with a retrospective (daily news, show & tell, etc). Also works as a buffer but I'll make sure it starts on time and adds value to prevent slippery slope of latecomers.
Project Week / Hackathon
What we did on project 2 was to turn the Friday before project week into a "hackathon." Teams were adjusted a bit on Monday but it was useful to iterate the project.

Outcome Sessions
Please mark these slots on the calendar if/once they're confirmed.

@cbas @rachlim
Hi All,
I noticed this thread is only 6 days old. Wondering if you guys have discussed anything regarding course materials given out in advanced before or during the break? I don't see them being discussed here. And I see MVC is added in the calendar after someone requested for it.
Seb, you mentioned Parity is not your concern. While I understand that all of us are of different programming calibre and it is easier to teach those who have programming background, I sincerely hope you rethink your stance. I decided to come to a reputable school like GA with certain expectations that I will get a reasonable education in web development in a structured way. I fully agree that everyone wants to learn as much as they can if not they wont be spending full day at the school. But I am overwhelmed by the sheer amount of unstructured information given to us. I'm sure Rachel can elaborate on this as I have discussed with her at length.
I am afraid I don't understand what Seb's wrote: "As far as "structure" is concerned, it’s important that students learn to solve problems, not just follow instructions. Programming requires an enormous capacity for creative thinking."  I am wondering whats the underlying message here?
We are all working adults and exercise certain amount of creative thinking and use problem solving skills
during the course of our professional lives. We were methodically trained / drilled in school or on the job to approach any problems with a certain set of best practises involved. Yes, there are times when emergencies arise, that is when our strong foundation in whichever subject matter comes into play.
Thanks and good night.
T

@Tianster Of course. Plan for change. Gradual introduction of concepts. We've all come very far over the past few weeks, not least of all you. I'm very proud of the progress, especially yours.
Everyone is in the same class. How long one takes to grok information depends on curriculum & teaching style but also on students' prior experience & knowledge. I have not yet seen a school where every student graduates with the same exact, absolute level of experience and knowledge. Rather than paralysis or retreat, continue to work and learn at your own pace. Myself and the team are always there for you. Thanks for making use of the additional classes provided over the holidays.
There are no silver bullets, no panacea. What's being refactored here is the structure of the class, to better align with this specific and varied group of individuals at this specific intersection of time in their lives.
Keep up the hard work!

Further refactoring:
CSS Breakfast (& Lunch!)
Every time we come back to class we'll start off with a bite-size exercise. Rewards punctuality. Maximises available time. Gets everyone in the right frame of mind to code & learn.
Prep work for the exercises consists of small screenshots and a GitHub Issue on this repo labelled exercise. Anyone is welcome to propose exercises. See example: #11

Review: Seems to work well so far this week.


Knowledge State Depictions

Thank you for providing this useful collection of models! DKVMN seems to perform best on my dataset. Yet, I am having trouble to display the knowledge states as described in their paper. Are you aware of any source where the code for creating knowledge states for DKVMN or SKVMN are available?
Thank you very much in advance!

hi, sorry for the late reply. Unfortunately, we have not visualized the knowledge state depiction for DKVMN. But we also have similar experiments in our work ( https://arxiv.org/pdf/2302.06885.pdf). In this work, we treat the probability of the model prediction as the knowledge state.

Thank you for providing this useful collection of models! DKVMN seems to perform best on my dataset. Yet, I am having trouble to display the knowledge states as described in their paper. Are you aware of any source where the code for creating knowledge states for DKVMN or SKVMN are available?
Thank you very much in advance!

hi, sorry for the late reply. Unfortunately, we have not visualized the knowledge state depiction for DKVMN. But we also have similar experiments in our work ( https://arxiv.org/pdf/2302.06885.pdf). In this work, we treat the probability of the model prediction as the knowledge state.


Inline query rendering broken when using Tasks plugin

What happened?
inline Dataview query is not rendered when Task plugin is turned on.
For example, the following set up won't render the actual file name when Task plugin is on; however turning off the task plugin (and restart Obsidian) can have it rendered correctly.

[ ] #task =this.file.name

DQL
No response
JS
- [ ] #task `=this.file.name`

Dataview Version
0.5.47
Obsidian Version
1.0
OS
MacOS

Isn't this more of a conflict with tasks? This isn't particularly easy to solve from the DV side I imagine.

The thing is, it works fine in editing mode but it breaks in reading mode. It looks like somehow the inline expression wasn't processed at all. Because Tasks plugin should not care about the dataview syntax, I wonder if this is something at Dataview side.


Fixed incorrect imports (Docs)

The import was defined as DataTable instead of datatable, also missing a comma after State.

Thanks for your help


dataZoom.startValue - Type 'string' is not assignable to type 'number'

Version
5.0.2
Steps to reproduce
Using datazoom option for line chart similar to https://echarts.apache.org/examples/en/editor.html?c=line-aqi inside angular.
import { EChartsOption } from 'echarts';
{
....
dataZoom: {
type: 'inside',
startValue: '2014-06-01'
}
....
}
What is expected?
As per documentation number, string, Date are allowed in dataZoom.startValue.
What is actually happening?
Type error is getting thrown Type 'string' is not assignable to type 'number'. node_modules/echarts/types/dist/echarts.d.ts:8213:5 startValue?: number;

Hi @pissang
I tried to fix it and create PR. But it is not that easy of changing to number | string | Date. It has other dependancies and leading to option: Opt;    // TODO Opt should only be object.
I shouldn't fix this TODO as it may cause some other problems.


Feature request for useSSR hook

Is your feature request related to a problem? Please describe.
Quickly know where your code will be executed;

In the server (Server-Side-Rendering)
In the client, the navigator

Describe the solution you'd like
I want a hook that will give me information if I run my code in the browser or server
A clear and concise description of what you want to happen.
Describe alternatives you've considered
https://usehooks-ts.com/react-hook/use-ssr
https://github.com/alex-cory/use-ssr

this is a very good utility and definitely a good idea, but should it be a hook really? if so why not just a common utility?
I mean... we can definitely provide a utility checking if it is server side or client side rendering and we can possibly export it from the library, something like this:
const isServerSideRendering = () => typeof window !== 'undefined';

// import something like this maybe?

import { isServerSideRendering } from 'beautiful-react-hooks/commons';

or we can improve and extend (and rename) the already existing isClient function.
I generally tend to consider hooks pieces of React-related business logics that help the devs to use (or hook into) React features, but this is just a check that's not strictly related to any React feature...
to be fair such a utility can be used in many other context...
I know a lot of other React hook libraries have such a hook, but in fact one of the reason I wrote this library is to enforce good practices using React hooks (which are very often overlooked) and I completely disagree with this approach for the instance:
https://usehooks-ts.com/react-hook/use-ssr
Please let me know what you think

I've made some small research about that and I came to the conclusion that It's quite tricky :P I've found this in docs (https://reactjs.org/docs/hooks-custom.html#extracting-a-custom-hook): A custom Hook is a JavaScript function whose name starts with ”use” and that may call other Hooks. So may use other hooks, but doesn't have to. The only difference between the function and the hook, in this case, is that this automatically forces it to use rules of hooks. From the same doc above: [...] it’s just like a normal function. Its name should always start with use so that you can tell at a glance that the rules of Hooks apply to it.. But I also understand your perspective and I think it does not make sense as far as isCliect function is more appropriate and reusable. To close! Thanks for this brainstorm! It's always good to learn something new: P


Improve non-follower selection behavior

Just some small improvements.

Just some small improvements.

some conflicts

some conflicts

Yeah, I'll resolve them and merge when I get home

Looks like it needs another approval since I amended the conflicts and force-pushed. @davidarroyo1234


Updated from Jade to Pug

I have updated the Jade parts of Yeogurt to now reference Pug.  This will make the generator easier to update and use moving into the future.
Slight backwards compatibility issue with the sub-generators. Not sure if this is really an issue or just perceived issues.

Hey @benjamin-richardson,
Thanks for the PR! Apologies for the late response.
If you could fix the conflicts with new release, that would be awesome. As for merging this in, I'm waiting for pug to come out of beta. Until it's full release, I will keep this PR open and unmerged.
Let me know if you have any questions. Nice work 👍

I think this issue on the Pug GitHub Repo says that it has officially been released now:
https://github.com/pugjs/pug/issues/2776
Does that mean that this Pull request can be merged now?

@Dan503, Looks like it's still in rc-4, so I'm still going to hold off on updating from Jade for a bit longer. Not sure why it's taking so long for a stable release, as it is quite frustrating for this project. Hopefully a stable release will be pushed soon and I will get it integrated with Yeogurt quickly.

Maybe it`s time? Already 2.0.3 version.
https://github.com/pugjs/pug/releases

Closing this in favor of #226 as the code required some pretty substantial surgery to get things up to date and working.


Failing to complete the run

[2023/07/25 01:29:10] Starting Execution
[2023/07/25 01:29:10] Disabling Internet Explorer Enhanced Security Configuration
[2023/07/25 01:29:10] Installing essential Apps and Tools (i.e.: PowerShell Core, 7zip, AzCopy, AzCLI, VMRC)
[2023/07/25 01:29:19] |--Install-Applications - Used Choco to install: PowerShell-Core, AzCopy, AzCli, 7zip and VMRC
[2023/07/25 01:29:19] Check if C:\temp exist. If it did not exist, then create it
[2023/07/25 01:29:19] Downloading bootstrap-nestedlabs.ps1 script
[2023/07/25 01:29:19] |--Get-BootstrapScript - Nested labs bootstrap script 'bootstrap.ps1' already exists
[2023/07/25 01:29:19] |--Get-BootstrapScript - Nested labs bootstrap script 'bootstrap-nestedlabs.ps1' already exists
[2023/07/25 01:29:19] Creating a Windows Scheduled Task to register bootstrap-nestedlabs.ps1 script, and trigger it on Jumpbox VM startup
[2023/07/25 01:29:20] |--Set-BootstrapScheduledTask - bootstrap-nestedlabs.ps1 will be deploying 1 nested labs for group 1
[2023/07/25 01:29:20] Downloading nested labs Zip package
[2023/07/25 01:29:21] |--Get-NestedLabPackage - Failed to download nested labs zip package. Please run bootstrap.ps1 again to make sure package is downloaded properly before you proceed
[2023/07/25 01:29:21] Concluding Execution
[2023/07/25 01:29:21]

[2023/07/25 01:29:21] |--Get-NestedLabPackage - Failed to download nested labs zip package. Please run bootstrap.ps1 again to make sure package is downloaded properly before you proceed

Thanks @jnordsving for reporting this issue. The issue was a result of the nestedlabs being deleted at its location. The issue is now  addressed by re-uploading the nestedlabs package.


v0.7.0 API Review

What type of PR is this?
/kind feature
What this PR does / why we need it:
This PR is for the purpose of reviewing the v0.7.0 release. It compares the /apis dir from the main branch to the /apis dir on the release-0.6 branch. This is to avoid distracting the PR with all the docs, GEP, and test changes that have also occurred in the same time. This PR is not intended to merge, instead is meant purely to facilitate API review.
/cc @bowei @shaneutt @youngnick
/assign @khenidak @thockin

/approve
/lgtm
Thanks for this.

/lgtm
to make it real

Thanks everyone for the reviews! v0.7.0 has been released, closing this out.