Could you please provide more guidance on reproducing Aachen for SIFT?

Hi!  I would like to ask for guidance on reproducing Aachen for SIFT and SGMNet+SIFT?
At a starter, I have tried to run MNN+SIFT for Aachen day-night...but I get a much worse result for 8196 keypoints... 23.5 / 33.7 / 41.8
I am guessing that maybe because I am using SIFT instead of RootSIFT... But I am not sure about other settings that could be different...
So, as I saw the result of SGMNet+SIFT as well as MNN+SIFT on both papers and on https://www.visuallocalization.net/details/17655/, I am wondering how to reproduce them (especially Table 4. in the paper)...
Could you please share the setting? I would like to learn how to get a similar result...
So I would like to scope down my question as follows:

Firstly, it is RootSIFT not SIFT...where self.root == True (in  https://github.com/vdvchen/SGMNet/blob/main/components/extractors.py#L43 ) ? Is that correct? Do you have to further normalize the features (to keep the norm of feature dimension to 1) ?
Did you restrict image size?  For example, in D2-Net, the maximum size was restricted to 1600 ? Did you do something like that too?
For MNN+SIFT, is there any thresholding applied for MNN matching?
Is the setting for SGMNet+SIFT for Aachen Day-Night similar to the following?  Also, as I read the name ...rootsift8k_upright_512_0.2_SGMNet... What do you mean by 512 and 0.2?

matcher:
  name: SGM
  model_dir: ../weights/sgm/root
  seed_top_k: [256,256]
  seed_radius_coe: 0.01
  net_channels: 128
  layer_num: 9
  head: 4
  seedlayer: [0,6]
  use_mc_seeding: True
  use_score_encoding: False
  conf_bar: [1.11,0.1] #set to [1,0.1] for sp
  sink_iter: [10,100]
  detach_iter: 1000000
  p_th: 0.2

Also, is the setting for SG+SIFT for Aachen Day-Night similar to ?

matcher:
  name: SG
  model_dir: ../weights/sg/root
  net_channels: 128
  layer_num: 9
  head: 4
  use_score_encoding: True
  sink_iter: [100]
  p_th: 0.2

Hi,

Set self.root=True means using RootSIFT instead of SIFT. RootSIFT is l2-normalized by definition and SIFT extracted by opencv is also normalized by default (with a norm of 256 as I recall). So if you perform direct NN matching, the normalization should not be an issue. As a safe guard, you can l2-normalize these features manually.

For Aachen day-night I use the original img resolution.

Sorry but I think MNN doesn't a thresholding. You can apply Lowe's ratio test(https://stackoverflow.com/questions/51197091/how-does-the-lowes-ratio-test-work) together with MNN to achieve better results. For RootSIFT, I think 0.8 is a good threshold for ratio test.

512 means setting seed_top_k as [512,512], which is proportionally increased with keypoint number(8k). 0.2 means setting p_th as 0.2.

Yes, I think that's a correct setting for SG+RootSIFT.

One last thing, for Aachen, using upright RoofSIFT/SIFT generally improves the performance. I used the following code to produce upright features.
def run(self, img_path):  sift = cv2.xfeatures2d.SIFT_create(nfeatures=self.num_kpt, contrastThreshold=self.contrastThreshold)  img = cv2.imread(img_path,cv2.IMREAD_GRAYSCALE)  cv_kp= sift.detect(img, None)  score=np.asarray([_kp.response for _kp in cv_kp])  if self.upright:      cv_kp=[cv2.KeyPoint(x=kp.pt[0],y=kp.pt[1],_size=kp.size,_angle=0) for kp in cv_kp]  cv_kp,desc=sift.compute(img,cv_kp)  desc=desc/np.linalg.norm(desc,axis=-1)[:,np.newaxis]  if self.root:      desc=np.sqrt(abs(desc/(np.linalg.norm(desc,axis=-1,ord=1)[:,np.newaxis]+1e-4)))  kp = np.array([[_kp.pt[0], _kp.pt[1]] for _kp in cv_kp]) # N*2  index=np.flip(np.argsort(score))  kp,desc,score=kp[index],desc[index],score[index]  return kp, desc,score

Hi! Thanks so much. I have tried... Yes, for RootSIFT,  0.8 is a good threshold for ratio test.
Thank you very much....
Also, I have another question... I am using your work with my dataset ....
The value of seed_top_k: [256,256] has to be higher than the number of keypoints? Is that right?

Hi,
For aachen, using upright features should further boost performance.
We assume the input kpt number should be larger than seed number. If you are using it with very low kpt number(eg ~200), we suggest to sample fewer seeds, while the performance is not guaranteed.

Hi! Thank you so much... Could you help explain how to extract the upright feature? I am very sorry for this question.. I am very new to this field...

We assume the input kpt number should be larger than seed number. If you are using it with very low kpt number(eg ~200), we suggest to sample fewer seeds, while the performance is not guaranteed.
I see thank you very much for confirmation.. I will try accordingly.

Hi,

Set self.root=True means using RootSIFT instead of SIFT. RootSIFT is l2-normalized by definition and SIFT extracted by opencv is also normalized by default (with a norm of 256 as I recall). So if you perform direct NN matching, the normalization should not be an issue. As a safe guard, you can l2-normalize these features manually.
For Aachen day-night I use the original img resolution.
Sorry but I think MNN doesn't have a thresholding. You can apply Lowe's ratio test(https://stackoverflow.com/questions/51197091/how-does-the-lowes-ratio-test-work) together with MNN to achieve better results. For RootSIFT, I think 0.8 is a good threshold for ratio test.
512 means setting seed_top_k as [512,512], which is proportionally increased with keypoint number(8k). 0.2 means setting p_th as 0.2.
Yes, I think that's a correct setting for SG+RootSIFT.
One last thing, for Aachen, using upright RoofSIFT/SIFT generally improves the performance. I used the following code to produce upright features.

def run(self, img_path):
    sift = cv2.xfeatures2d.SIFT_create(nfeatures=self.num_kpt, contrastThreshold=self.contrastThreshold)

    img = cv2.imread(img_path,cv2.IMREAD_GRAYSCALE)
    cv_kp= sift.detect(img, None)
    score=np.asarray([_kp.response for _kp in cv_kp])
    if self.upright:
        cv_kp=[cv2.KeyPoint(x=kp.pt[0],y=kp.pt[1],_size=kp.size,_angle=0) for kp in cv_kp]

    cv_kp,desc=sift.compute(img,cv_kp)
    desc=desc/np.linalg.norm(desc,axis=-1)[:,np.newaxis]
    if self.root:
        desc=np.sqrt(abs(desc/(np.linalg.norm(desc,axis=-1,ord=1)[:,np.newaxis]+1e-4)))
    kp = np.array([[_kp.pt[0], _kp.pt[1]] for _kp in cv_kp]) # N*2
    index=np.flip(np.argsort(score))
    kp,desc,score=kp[index],desc[index],score[index]
    return kp, desc,score

You may use codes in this reply.

Thank you so much !


Prl 3140 child relation ship mapping new

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)
[ ] the keep-helm label has been added, if the helm release should be persisted after a successful build

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

These changes will be merged as part PR https://github.com/hmcts/prl-cos-api/pull/1599
Closing this PR.


GI: move FST Docker containers to their own repo

new repo location: https://github.com/hashgraph/solo-containers

Done


SdkBody callbacks for progress reporting

Responding to the removal of SdkBody callbacks in #2065.
I'm currently using with_body_callback() for one thing only: to implement progress reporting for upload operations (e.g. S3's put_object). In this case for retryability, ByteStream::from_path() is used, which precludes sneaking in a hook for progress otherwise. For data going in the other direction (e.g. get_object) it's easy to put progress reporting in stream consumption so no such need for a hook exists.
It seems there are enough escape hatches to implement this manually in an API consumer, e.g. SdkBody::retryable(). But it's a little bit more unclear what the recommended and hopefully somewhat minimal and future proof method would be for doing so.

The way to accomplish this going forward is with "body wrappers."
I've put an issue on the example writer's backlog to add an example of how to do this.
Depending on your familiarity with Rust, you may be able to accomplish this on your own. You'd need to create a struct that looks similar to this:
pin_project! {
    pub struct ProgressBody<InnerBody> {
        #[pin]
        inner: InnerBody,
        bytes_written: u64,
        content_length: u64,
    }
}

impl ProgressBody<SdkBody> {
    /// Given an `SdkBody`, a `Box<dyn HttpChecksum>`, and a precalculated checksum represented
    /// as `Bytes`, create a new `ChecksumBody<SdkBody>`.
    pub fn new(
        body: SdkBody,
        content_length: u64,
    ) -> Self {
        Self {
            inner: body,
            content_length,
        }
    }

    fn poll_inner(
        self: Pin<&mut Self>,
        cx: &mut Context<'_>,
    ) -> Poll<Option<Result<Bytes, aws_smithy_http::body::Error>>> {
        use http_body::Body;

        let this = self.project();

        match this.inner.poll_data(cx) {
            Poll::Ready(Some(Ok(data))) => {
                this.bytes_written += data.len() as u64;
                let progress = *this.bytes_written as f64 / *this.content_length as f64;
                tracing::trace!(
                    "Read {} bytes, progress: {:.2}%",
                    data.len(),
                    progress * 100.0
                );

                Poll::Ready(Some(Ok(data)))
            }
            // Once the inner body has stopped returning data, check the checksum
            // and return an error if it doesn't match.
            Poll::Ready(None) => {
                tracing::trace!("done");
            }
            Poll::Ready(Some(Err(e))) => Poll::Ready(Some(Err(e))),
            Poll::Pending => Poll::Pending,
        }
    }
}

(I didn't compile this to see if it actually works, but it should get you very close to what you need.)
And then call SdkBody.map() to wrap the body in your ProgressBody:
let body = ByteStream::from(SdkBody::from(<your file>).map(move |body| {
    let body = ProgressBody::new(body, <body content length>);
    aws_smithy_http::body::SdkBody::from_dyn(aws_smithy_http::body::BoxBody::new(body))
}));

I'll leave this issue open until we have an example.

Understood, and thankyou! Is there a concise version of SdkBody::from(<your file>) recommended? As far as I can see the smithy implementation (PathBody) isn't exported, and only indirectly available via FsBuilder, which only provides ByteStream, not SdkBody.
I had a stab at doing it via hyper::Body::wrap_stream, and seems to provide everything except size_hint, but unsure if I'm losing anything else this way. Setting length on the put operation itself solves the need for size_hint.
use futures::future::TryFutureExt;

fn path_to_sdk_body(path: PathBuf, progress: cli::ProgressFn) -> SdkBody
{
    let open_fut = async move {
        let file = tokio::fs::File::open(path).await?;
        Ok(tokio_util::io::ReaderStream::new(file))
    };
    let flattened = open_fut.try_flatten_stream();
    let inspected = flattened.inspect_ok(move |bytes| progress(cli::Update::StateProgress(bytes.len())));
    let hyper_body = hyper::body::Body::wrap_stream(inspected);
    SdkBody::from(hyper_body)
}

fn path_to_bytestream(path: PathBuf, progress: cli::ProgressFn) -> ByteStream
{
    let retryable = SdkBody::retryable(move || {
        progress(cli::Update::StateRetried);
        path_to_sdk_body(path.clone(), progress.clone())
    });
    ByteStream::from(retryable)
}


[Device Support Request] Skydance WZ-SPI RGBW LED Strip Controller

Problem description
The device connects to Home Assistant correctly. Firmware and diagnostic information is shown. It is, however, not possible to control the color/brightness/pattern of the LED strip.
Product Page: https://iskydance.com/index.php?c=product_show&a=index&id=2439
Solution description
Control LED strips in the same way as other LED strip controllers already supported by ZHA.
Screenshots/Video
No response
Device signature
Device signature
{
  "node_descriptor": {
    "logical_type": 1,
    "complex_descriptor_available": 0,
    "user_descriptor_available": 0,
    "reserved": 0,
    "aps_flags": 0,
    "frequency_band": 8,
    "mac_capability_flags": 142,
    "manufacturer_code": 4417,
    "maximum_buffer_size": 66,
    "maximum_incoming_transfer_size": 66,
    "server_mask": 10752,
    "maximum_outgoing_transfer_size": 66,
    "descriptor_capability_field": 0
  },
  "endpoints": {
    "1": {
      "profile_id": "0x0104",
      "device_type": "0x0051",
      "input_clusters": [
        "0x0000",
        "0x0004",
        "0x0005",
        "0xef00"
      ],
      "output_clusters": [
        "0x000a",
        "0x0019"
      ]
    },
    "242": {
      "profile_id": "0xa1e0",
      "device_type": "0x0061",
      "input_clusters": [],
      "output_clusters": [
        "0x0021"
      ]
    }
  },
  "manufacturer": "_TZE204_8fffc3kb",
  "model": "TS0601",
  "class": "zigpy.device.Device"
}

Diagnostic information
No response
Logs
No response
Custom quirk
No response
Additional information
No response

We can't know the dates for the updates? We can't know the dates for the updates?

+1

+1

+1


Add error message when "crypt.exe" is missing

I suppose this file should not be public for now, or the user has to manually pick a version from the cpp_build folder.
if so, tell me and I'll update the message (or you after merging).

I've been meaning todo this, thanks man!
I appreciate it!

For some reason I only got focused on the else statement, this PR is actually very invalid. I'll make a fix right away lol.

#8


Merged commit to support other query types, starting with 'Or'

When using retrieve_multiple.

Ok, new PR squashed and addressed comments, should be good?

ahh looks like tests are failing now, looking into it.

should pass now - needed ||= so a passed nil from above doesn't override the param default

Great! Thanks @mbhnyc . I appreciate you working through this.


Added missing newline

cc: @cartermp

Open Publishing Build Service: The pull request content has been published and here are some changed files links of this time:

Docset 1 - core-docs
https://stage.docs.microsoft.com/en-us/dotnet/core/api/system.net.http.headers.viaheadervalue?branch=pr-en-us-1023
https://stage.docs.microsoft.com/en-us/dotnet/core/api/system.net.http.headers.warningheadervalue?branch=pr-en-us-1023
https://stage.docs.microsoft.com/en-us/dotnet/core/api/system.net.http.headers?branch=pr-en-us-1023
Please see 3429 more published files of this docset in report.

Status: Succeeded.
Open Publishing Report.


Please support for firebase_database 11.x

As realtime database version has upgraded to 11, please support this.

Thanks for the remainder, I will work on it as soon as I get some free time. PRs are also welcome :)

#19

Any updates?

✅  https://github.com/sitatec/firebase_database_mocks/releases/tag/v0.7.0


Generated test module will be included by default in sphinx docs

I'm sure that, most of the time, developers don't care much about having user facing documentation for tests. With the recursive autodocs, these modules are included by default. We should be able to remove them with something like
autodoc_mock_imports = [
    '{{mykit}}.tests',
]

to conf.py.

Thanks for raising this @ianmkenney! sounds like a good solution -- could you please open a PR?


Dart build is killed

Hello,
I'm trying to set up Security Monkey in EC2 following the guide at http://securitymonkey.readthedocs.io/en/latest/quickstart.html#clone-the-security-monkey-repo but I'm getting stuck after: "sudo /usr/lib/dart/bin/pub build". I receive the following error messages:
ubuntu@ip-172-31-66-15:/usr/local/src/security_monkey/dart$ sudo /usr/lib/dart/bin/pub get                     [81/1910]
Resolving dependencies...
Got dependencies!
ubuntu@ip-172-31-66-15:/usr/local/src/security_monkey/dart$ sudo /usr/lib/dart/bin/pub build
Loading source assets...
Loading dart_to_js_script_rewriter, di/module_transformer, initialize/build/loader_replacer, observe, smoke/src/default$
transformer and initialize transformers...
Loading angular transformers... (2.4s)
Building security_monkey... (14.5s)
[Info from Dart2JS]:
Compiling security_monkey|web/main.dart...
[Warning from Dart2JS on security_monkey|web/main.dart]:
web/packages/angular/core/annotation_src.dart:3:44:
Library 'package:di/di.dart' doesn't export a 'Visibility' declaration.
Try removing 'Visibility' from the 'show' clause.
import "package:di/di.dart" show Injector, Visibility, Factory;
                                           ^^^^^^^^^^
[Warning from Dart2JS on security_monkey|web/main.dart]:
web/packages/angular/core/annotation_src.dart:3:56:
Library 'package:di/di.dart' doesn't export a 'Factory' declaration.
Try removing 'Factory' from the 'show' clause.
import "package:di/di.dart" show Injector, Visibility, Factory;
                                                       ^^^^^^^
[Warning from Dart2JS on security_monkey|web/main.dart]:
web/packages/angular/core/annotation.dart:11:5:
Library 'package:angular/core/annotation_src.dart' doesn't export a 'ShadowRootAware' declaration.
Try removing 'ShadowRootAware' from the 'show' clause.
    ShadowRootAware,
    ^^^^^^^^^^^^^^^
[Warning from Dart2JS on security_monkey|web/main.dart]:
web/packages/angular/core/module.dart:70:5:
Library 'package:angular/core/module_internal.dart' doesn't export a 'CacheStats' declaration.
Try removing 'CacheStats' from the 'show' clause.
    CacheStats,
    ^^^^^^^^^^
[Warning from Dart2JS on security_monkey|web/main.dart]:
web/packages/angular/core/module.dart:71:5:
Library 'package:angular/core/module_internal.dart' doesn't export a 'ComponentCssRewriter' declaration.
Try removing 'ComponentCssRewriter' from the 'show' clause.
    ComponentCssRewriter,
    ^^^^^^^^^^^^^^^^^^^^
[Warning from Dart2JS on security_monkey|web/main.dart]:                                                       [40/1910]
web/packages/angular/core/module.dart:50:5:
Library 'package:angular/core_dom/module_internal.dart' doesn't export a 'NgAnimate' declaration.
Try removing 'NgAnimate' from the 'show' clause.
    NgAnimate,
    ^^^^^^^^^
[Warning from Dart2JS on security_monkey|web/main.dart]:
web/packages/angular/core/module.dart:54:5:
Library 'package:angular/core_dom/module_internal.dart' doesn't export a 'PendingAsync' declaration.
Try removing 'PendingAsync' from the 'show' clause.
    PendingAsync,
    ^^^^^^^^^^^^
[Dart2JS]:
web/main.dart:

****************************************************************
* WARNING: dart:mirrors support in dart2js is experimental,
*          and not recommended.
*          This implementation of mirrors is incomplete,
*          and often greatly increases the size of the generated
*          JavaScript code.
*
* Your app imports dart:mirrors via:
*   main.dart => package:security_monkey/security_monkey.dart => package:security_monkey/model/hammock_config.dart => da
rt:mirrors
*   main.dart => main_generated_type_factory_maps.dart => package:security_monkey/security_monkey.dart => package:securi
ty_monkey/model/hammock_config.dart => dart:mirrors
*   main.dart => main_static_metadata.dart => package:security_monkey/security_monkey.dart => package:security_monkey/mo
del/hammock_config.dart => dart:mirrors
*   main.dart => main_static_type_to_uri_mapper.dart => package:security_monkey/security_monkey.dart => package:security
_monkey/model/hammock_config.dart => dart:mirrors
*
* You can disable this message by using the --enable-experimental-mirrors
* command-line flag.
*
* To learn what to do next, please visit:
*    http://dartlang.org/dart2js-reflection
****************************************************************
[Warning from Dart2JS on security_monkey|web/main.dart]:
web/main_generated_type_factory_maps.dart:368:58:
Additional argument.
  import_44.Rect: (a1, a2, a3, a4) => new import_44.Rect(a1, a2, a3, a4),
                                                         ^^
[Info from Dart2JS on security_monkey|web/main.dart]:
web/packages/angular_ui/utils/position.dart:100:3:
This is the method declaration.
  Rect({this.left:0, this.top:0, this.width:0, this.height:0});
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[Warning from Dart2JS on security_monkey|web/main.dart]:
web/main_generated_type_factory_maps.dart:368:62:
Additional argument.
  import_44.Rect: (a1, a2, a3, a4) => new import_44.Rect(a1, a2, a3, a4),
                                                             ^^
[Info from Dart2JS on security_monkey|web/main.dart]:
web/packages/angular_ui/utils/position.dart:100:3:
This is the method declaration.
  Rect({this.left:0, this.top:0, this.width:0, this.height:0});
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[Warning from Dart2JS on security_monkey|web/main.dart]:
web/main_generated_type_factory_maps.dart:368:66:
Additional argument.
  import_44.Rect: (a1, a2, a3, a4) => new import_44.Rect(a1, a2, a3, a4),
                                                                 ^^
[Info from Dart2JS on security_monkey|web/main.dart]:
web/packages/angular_ui/utils/position.dart:100:3:
This is the method declaration.
  Rect({this.left:0, this.top:0, this.width:0, this.height:0});
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[Warning from Dart2JS on security_monkey|web/main.dart]:
web/main_generated_type_factory_maps.dart:368:70:
Additional argument.
  import_44.Rect: (a1, a2, a3, a4) => new import_44.Rect(a1, a2, a3, a4),
                                                                     ^^
[Info from Dart2JS on security_monkey|web/main.dart]:
web/packages/angular_ui/utils/position.dart:100:3:
This is the method declaration.
  Rect({this.left:0, this.top:0, this.width:0, this.height:0});
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[Dart2JS on security_monkey|web/main.dart]:
4 warning(s) and 1 hint(s) suppressed in package:security_monkey.
[Dart2JS on security_monkey|web/main.dart]:
4 warning(s) suppressed in package:angular.
Killed
ubuntu@ip-172-31-66-15:/usr/local/src/security_monkey/dart$

Any idea what might be going on here? Thanks!

Check that you have enough memory.  I've tried installing SM on a t2.micro (or whatever the smallest instance type is) in the past and ran into the same thing due to lack of resources.

That seems to have been the problem, had to change my instance from t2.micro to t2small. The original documents mention m1.small with 1.7GB RAM, but that image doesn't seem to exist anymore.

You can just pull the compiled version and cp that to security monkey. That works in my environment.

@hobbits49 Pull the compiled code from where?


Fixed some vulnerability and upgrading the bootstrap to new version for fix a type of attack

Hi @MKokeshi, there was a previous attempt at this change here. Do you know if simply upgrading it actually works?

yeah, there are definitely changes required to upgrade to 4.0.0. i'm not sure #7285 even gets them all but didn't have confidence that change was fully tested...


From KerasNLP to KerasCore : Add RMSLayerNormalization

Context: kers-team/keras-nlp/#1207
KerasNLP's T5 (now Llama too) uses a custom layer norm which is indeed RMSLayerNormalization. It's being used as it is from here to here
As it is getting popular, it would be good to officially support it as a Keras layer.
I can open a PR!

As @mattdangerw says here, we could either have a new layer RMSLayerNormalization or an option to the existing layer, something like LayerNormalization(rms_scaling=True).

something like LayerNormalization(rms_scaling=True).

I think that would be preferable. Thanks for the suggestion!

Something like this?
Arguments

    Args:
        ...
        center: If True, add offset of `beta` to normalized tensor. If False,
            `beta` is ignored. Defaults to `True`.
        scale: If True, multiply by `gamma`. If False, `gamma` is not used.
            When the next layer is linear (also e.g. `nn.relu`), this can be
            disabled since the scaling will be done by the next layer.
            Defaults to `True`.
        rms_scaling: If True, `center` and `scale` are ignored, and the
            inputs are scaled by `gamma` and the inverse square root
            of the square of all inputs. This is an approximate and faster
            approach that avoids ever computing the mean of the input.

Sure, sounds good to me. Thanks!


Dell 3050 sff

Hello and thank you for your great job
I tried your EFi to install sequoia but I was unable to do so in 3050sff
I have a working EFI for Sonoma,can you kindly help me modify it so I  can achieve to install sequoia please?

I have same the same issue

@R3n09 @ryanhaven
List your specs or I'm closing it.
It's not proper way to report issues, use your words and RTFM.

Issue resolved.
Updated kexts and core to latest versions.


Address some lint errors

Addresses some lint errors in the go report card.

/assign mattmoor

/ok-to-test

/lgtm
/approve

/test pull-knative-pkg-unit-tests


eCarePlanner 3.1.1 - no goals displayed

goals seems to be operating weirdly.  Christina reported that the Goals section "just spins":

but I see that it appears to complete (no spinner), it's just empty:

here's what's displayed for this patient in the MyCarePlanner app:

It also appears that there aren't any errors related to getting the goals, although I do see 405 errors in log-service.ts that occur after getSummaryGoals calls.  I suspect these 405 errors may be related to #346 but I can't be sure.

Keep this card (close #342 as duplicate).


Link to Pico pinout broken (in gp2040-ce.info)

This link is broken:
https://gp2040-ce.info/#/configs/Pico/assets/PinMapping.png

Hi @aserrallerios  - Thank you for the report.  I am not quite sure why the link is broken but we will get it fixed up.

This has been resolved in a PR


Update helper-eachIndex.md

this is an object. To print the value the keyword item must be used instead.
Reference: code

@AurelioDeRosa thanks for the fix. Sorry there was no reply, the project was neglected for a while. We recently refactored the code and we're writing the docs, but we do appreciate the pr. thanks


Add MI manual health check

Current dependencies on/for this PR:

main

PR #3559   👈

This comment was auto-generated by Graphite.


Fix broken build caused by PR#1251

Fix the broken build caused by adding the OrbitClientGgp to the collector.
Update conanfile  so the ggp client is also available in bin/.

This should fix the broken build caused by the previous PR. I have included a temp commit so the code is checked in the presubmit before merging.

Build seems to work now :)

Merging the fix. Thanks a lot!


Adding Additional Features

Good Afternoon -
Have you thought of, or tried to add additional characters and perhaps make them in-app purchases?
Thanks,
Hayden

Hi @hgrieder,
we did not consider in-app purchases as the app was developed with the aiming of learning how to make small games on SpriteKit.. (I was teaching my students some basic concepts of game development and Objective-C). Although, it is easy to add extra characters (not only that, but take a look to BEEPlayer, BEEMonster and perhaps BEEItem).
Thanks,
Luiz


Wrong date format in function app settings when set by az cli

Describe the bug
We are currently developing a simple Azure Function App that utilizes a Cosmos DB Change Feed Trigger. This trigger includes an option to set a StartFromTime for the cursor, which is intended to prevent the processing of all data within the container from the beginning. The recommended format for the StartTime is ISO 8601.
The issue arises when configuring the service using Azure CLI. Despite entering the StartFromTime in the correct ISO 8601 format, the configuration value format appears to be altered upon execution of the CLI command. This unexpected change in the timestamp format leads to an issue where the Function App is unable to correctly map the trigger endpoint.
Related command
az functionapp config appsettings set --name $name --resource-group $group --settings "test_StartFromTime=2023-01-03T12:12:23Z"
Errors

Issue script & Debug output
the problem is not with the script, only with the result
Expected behavior
Az Cli should set the value in the defined format
Environment Summary
azure-cli                         2.53.1
core                              2.53.1
telemetry                          1.1.0
Dependencies:
msal                            1.24.0b2
azure-mgmt-resource             23.1.0b2
Python location 'C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\python.exe'
Extensions directory 'C:\Users\pawel.azure\cliextensions'
Python (Windows) 3.10.10 (tags/v3.10.10:aad5f6a, Feb  7 2023, 17:05:00) [MSC v.1929 32 bit (Intel)]
Legal docs and information: aka.ms/AzureCliLegal
Additional context
No response

Thank you for opening this issue, we will look into it.

Additional investigation needed


test_linspace error with numpy

FAILED array_api_tests/test_creation_functions.py::test_linspace - AssertionError: out[0]=0.0, should be -0.0 [linspace()]
Falsifying example: test_linspace(
    num=1,
    dtype=numpy.float32,
    endpoint=False,
    data=data(...),
)
Draw 1 (start): -0.0
Draw 2 (stop): -1.0
Draw 3 (kw): {'dtype': numpy.float32, 'endpoint': False}

Not clear if this is actually required by the spec or not. See https://data-apis.org/array-api/latest/API_specification/generated/array_api.linspace.html

This doesn't look like a very useful failure, and I don't think the answer is required to be -0.0. I suggest avoiding the use of -0.0 or check with an assert function that has behavior like:
>>> -0.0 == 0.0
True


Column with dots support.

This pr adds scol_for in util, and related methods in _InternalFrame to get Spark Column with dots in the name.
Resolves #217.

Codecov Report

Merging #490 into master will decrease coverage by 0.58%.
The diff coverage is 98.66%.

@@            Coverage Diff             @@
##           master     #490      +/-   ##
==========================================
- Coverage   93.71%   93.13%   -0.59%
==========================================
  Files          28       28
  Lines        3803     3816      +13
==========================================
- Hits         3564     3554      -10
- Misses        239      262      +23

Impacted Files
Coverage Δ

databricks/koalas/plot.py
93.77% <ø> (ø)
:arrow_up:

databricks/koalas/series.py
93.65% <100%> (ø)
:arrow_up:

databricks/koalas/indexing.py
93.2% <100%> (ø)
:arrow_up:

databricks/koalas/indexes.py
89.88% <100%> (-0.44%)
:arrow_down:

databricks/koalas/ml.py
96.15% <100%> (ø)
:arrow_up:

databricks/koalas/generic.py
94.28% <100%> (ø)
:arrow_up:

databricks/koalas/utils.py
97.43% <100%> (-2.57%)
:arrow_down:

databricks/koalas/mlflow.py
94.73% <100%> (-0.14%)
:arrow_down:

databricks/koalas/internal.py
98.27% <100%> (+0.19%)
:arrow_up:

databricks/koalas/frame.py
92.96% <97.22%> (-2.15%)
:arrow_down:

... and 4 more

Continue to review full report at Codecov.

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

I think I don't have a better idea than this ...

Thanks! Let me merge this for now and let me think about the better way.

I would expect tests to cover scenarios I'd mentioned in the ticket. What about the case of complex field a.b in a DataFrame? Does it still work?

In the current way, the column resolution is delegated to Spark as is. So, the tests added in this PR should be enough.
Currently complex type support itself isn't supported properly mainly because pandas doesn't have a nested type.

All good, thank you.


[Bug] The sourceFilter option seems to not be honored

Describe the bug
The sourceFilter option seems to not be honored - if I understand well its meaning that is "the pattern used to identify the files that will be part of the coverage report".
To Reproduce
I created a repository to easily test and reproduce the issue: https://github.com/ericmorand/c8-with-bundle
Just execute npm mcr and wait for the report to be displayed: it includes every file that is referenced in the V8 coverage data (that is node_modules files), plus src/main/lib/foo.ts but excluding all the other files present in src/main that also matches the passed sourceFilter pattern src/main/**/*.ts (like src/main/lib/bar.ts).
Expected behavior
The report only contains all the files that match the passed sourceFilter pattern. In this case, all the files matching src/main/**/*.ts

I can't pass the npm run build:main, could you please help to check it:
> build:main
> node build.mjs src/main/index.ts dist --treeshake

\c8-with-bundle\node_modules\typescript\lib\typescript.js:113963
    Debug.assert(contains(commandLine.fileNames, inputFileName), `Expected fileName to be present in command line`);
          ^

Error: Debug Failure. False expression: Expected fileName to be present in command line
    at Object.getOutputFileNames (\c8-with-bundle\node_modules\typescript\lib\typescript.js:113963:11)

Sorry, you need to build the test suite with npm run build:test. I assmue my main build script is flawed but we dont need it here. :)

same errors
> build:test
> node build.mjs src/test/index.ts dist --source-map

\c8-with-bundle\node_modules\typescript\lib\typescript.js:113963
    Debug.assert(contains(commandLine.fileNames, inputFileName), `Expected fileName to be present in command line`);
          ^

Error: Debug Failure. False expression: Expected fileName to be present in command line

Im using Node.js v20, Windows 10

Very interesting. Windows 10 being the difference betzeen our setups, I assume my build tool doesn't support it properly. Let me fix this.

So back to you issue. I think your sourceFilter pattern should be **/src/**/*.ts, it used the tool minimatch see https://github.com/isaacs/minimatch

It doesn't remove node_modules from the coverage but adds src/test/index.ts, which seems to demonstrate that the pattern is honored.
By the way, I pushed a new version of the project that comes with the dist folder already generated. You can now execute npm run mcr, without having to build the test suite before.

Thanks, I got it. The entry file is dist/index.mjs not src/**, so we need both entryFilter and sourceFilter
and It's recommended to use a config file mcr.config.js
// mcr.config.js
module.exports = {
    // logging: "debug",
    name: 'My Coverage Report',
    reports: [
        'v8',
        'console-details'
    ],
    outputDir: './coverage-reports',
    entryFilter: {
        "**/node_modules/**": false,
        "**/dist/**": true
    },
    sourceFilter: {
        "**/src/test/**": false,
        "**/src/**": true
    }
};

Then run npx mcr npm t again. Let me know if it works.

BTW, the sourceFilter dose not support multiple patterns for now, and I will fix it in next version:
--entryFilter "{'**/node_modules/**': false, '**/src/*.js': true}"

I'm not sure I understand the point of having an entry filter, but I probably misunderstand the meaning of the source filter. My asumption was that it filtered the files that are considered as "sources" - that is, the files that we want to get the coverage. With this asumption, there is no need for an entry filter because none of the files that are hit during the execution of the command should be emitted in the report, except if they have been added explicitely to the source filter.
Typically, in my case, using sourceFilter "src/main/**/*.ts", I would only get those files in the report, regardless of the files that were hit when executing command. Would I want node_modules files to be included in the report, I would just have to add them to the sourceFilter pattern.

1, Due to the characteristics of v8 coverage data, we need to use two filters entryFilter and sourceFilter see here for more infomation
2, src/main/index.ts should be a untest file, which is not included in the bundle and coverage data, we can try all option see here

Yes, the CLI arguments only can pass the string and number, so some of the options not supported by CLI, please use config file for CLI.

About source and entry filter, I have the feeling that the second one is not needed and that every file (present in the V8 report or found in the source map) can be filtered against only the source filter - which is ultimately the declaration of the files that we want to cover.
I'll try locally to remove the option, and will create a PR only for discussion and proof of concept, if you agree.

No, I don't think so.
The entryFilter is used to filter all entry files from the V8 coverage data. like huge node_modules, otherwise, performance would suffer due to the parsing too many meaningless files.
The sourceFilter is only used to filter source files from the sourcemap of the entry file.
In other words, if an entry file does not have a sourcemap, then the sourceFilter will not be executed.

I've thought about it, and perhaps it's feasible to merge the entryFilter and sourceFilter into one filter. I will continue to think about it.


Session Pool Management

Hi, this issue is just to say I wrote a session pool management crate based on r2d2: https://github.com/spruceid/r2d2-cryptoki.
I believe it's the right way of handling concurrent requests for service HSMs like cloud-hosted ones, but I thought maybe you'd have an opinion.
On a side note, I added a Zeroize wrapper for pins to ensure users understand how sensitive they are, and unless you think it's unnecessary I will create a PR as it would belong in this crate.

Hey! Thanks for the heads-up, the r2d2 crate does look pretty nice. I'm considering adding some links, perhaps on the README, to crates that have useful "3rd party" functionality. 🤔
In terms of a PR for zeroizing PINs - yes, please, that would be appreciated.

Sounds like a good idea! Added some info about r2d2 in the README in https://github.com/parallaxsecond/rust-cryptoki/pull/242
A question, is there any security implications in always having a session open, ready to use?


Plotting Age specific outcomes

I have pulled the recent version of the FPsim and ran it on my laptop. When it comes to plotting the age specific results, it gives me the following error:
mcpr_fig = scens.plot('as_mcpr')
File ~\OneDrive - Bill & Melinda Gates Foundation\Adolesents-FPsim\fpsim\fpsim\fpsim\scenarios.py:428 in plot
return self.msim_merged.plot(to_plot=to_plot, plot_sims=plot_sims, **kwargs)
File ~\OneDrive - Bill & Melinda Gates Foundation\Adolesents-FPsim\fpsim\fpsim\fpsim\sim.py:1318 in plot
sim.plot(to_plot=to_plot, **kw, **kwargs)
File ~\OneDrive - Bill & Melinda Gates Foundation\Adolesents-FPsim\fpsim\fpsim\fpsim\sim.py:704 in plot
raise ValueError(f"Age specific plot selected but sim.pars['track_as'] is False")
ValueError: Age specific plot selected but sim.pars['track_as'] is False
I appreciate any help with fixing it.

Hi @nnoori-IDM
I think it should work if you set pars['track_as'] = True
For instance

import fpsim as fp

n_agents   = [10_000, 100][debug]
start_year = [1980, 2010][debug]
track_as = True  #Track age-specific channels
pars = fp.pars(n_agents=n_agents, start_year=start_year, end_year=2040, track_as=track_as)

# Define scenarios, etc ...

Thanks @pausz for your response. I tried it and it gave me the following error:
AttributeError: module 'fpsim.defaults' has no attribute 'age_specific_results'
Below is my code:
pars = fp.pars(location='ethiopia')
pars['n_agents'] = 50_000 # Small population size
pars['start_year'] = 2010
pars['end_year'] = 2040
repeats = 10
youth_ages = ['<18', '18-20']
year     = 2022
pars['low_age_short_int'] = 10
pars['high_age_short_int'] = 20
pars['track_as'] = True
pp_method= ['Implants','Condoms','Pill']
kw1 = dict(method=pp_method[1],ages=youth_ages,matrix='pp0to1')
kw2 = dict(method=pp_method[2],ages=youth_ages,matrix='pp0to1')
kw3 = dict(method=pp_method[3],ages=youth_ages,matrix='pp0to1')
implant1_4 = fp.make_scen(
label = '1.4 times pp Implants initiation rate in <18 and 18-20',
year=year,
probs = [
dict(init_factor=1.4, **kw1)
]
)
condom1_4 = fp.make_scen(
label = '1.4 times pp Condoms initiation rate in <18 and 18-20',
year=year,
probs = [
dict(init_factor=1.4, **kw2)
]
)
pill1_4 = fp.make_scen(
label = '1.4 times pp Pill initiation rate in <18 and 18-20',
year=year,
probs = [
dict(init_factor=1.4, **kw3)
]
)
s1 = implant1_4 + condom1_4 + pill1_4
#%% Create sims

scens = fp.Scenarios(pars=pars,repeats=repeats)
scens.add_scen(label='Baseline')
scens.add_scen(s1,label="1.4 times PP Initiation Rate")
  # Run scenarios

scens.run(serial=debug)


Chrome 30 (desktop): drag with right mouse button not detected

When developing a web application that shall work in desktop and mobile browsers we wanted to have shortcuts with right mouse button on desktop browsers.
We use dragstart/drag/dragend events and detected desktop browser by verifying ev.gesture.srcEvent object type. Mouse button is then read from the srcEvent (evt.buttons or evt.button or evt.which).
This works as expected in Firefox and IE10, however it does not work in Chrome 30: dragstart is not fired at all on right button.
I inspected the code of Hammer and found out that detection is based on property e.which (=== 1). FF and IE10 set e.which to 1 and e.button or e.buttons accordingly.
Chrome sets e.which to 3 and e.button to 2.
I propose to change the button detection to use event properties in this order:

e.buttons > 0 (or selectively) (DOM level 3)
e.button
e.which (non-standard)

Reference:
https://developer.mozilla.org/en-US/docs/Web/API/MouseEvent.buttons
https://developer.mozilla.org/en-US/docs/Web/API/MouseEvent.button
http://msdn.microsoft.com/en-us/library/ie/ff974884(v=vs.85).aspx

Closing this since its about drag which was removed in v2


Doc - Cites Kubernetes in the upgrade guide

Build:

http://jenkins01.arangodb.biz:8080/job/arangodb-documentation/292/

Preview:

http://web01.arangodb.biz/documentation/upgrade-guide-imp/Manual/Upgrading/

Indeed. For now I believe we have to survive it this way - one generally speaking would love to have everything related to upgrade into a single chapter
I believe the placeholder is a temporary measure - let's see what the future brings. Merging for now. Thanks for the review..


Add PR preview to this docset

We might need to do some more config changes if these docs are supposed to have Edit button, feedback, etc.

OPS Build status updates of commit a4f3e8b:
:clock10: Preparing: average preparing time is 35 sec(s)

/cc @Rick-Anderson @shirhatti

OPS Build status updates of commit a4f3e8b:
:clock10: Incremental building: average incremental building time is 1 min(s)

OPS Build status updates of commit a4f3e8b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

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


Run on node16

I'm getting this warning on my builds starting a couple days ago:
[setup-job](https://github.com/Bandwidth/iris/actions/runs/3245261849/jobs/5325182399)
Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/. Please update the following actions to use Node.js 16: tony84727/changed-file-filter

From my testing in a private repository, it seems to run just fine just changing this one line.

Thanks


Use new Union interface in MergeSketchUDAF

Union now supports a new interface, allowing MergeSketchUDAF to do all its computations with Union rather than having to have an UpdateSketch and a Union sketch. This should simplify the code even though it won't reduce the memory footprint (union and update sketches are currently lazily constructed and only one ever exists at a time).

This was in DataToSketchUDAF, and it was done


Error Trace

Following our conversation, the colslice is not in GLM. It's in MatDS.scala. I still think it's worth implementing, but I'm open to a workaround.
java.lang.RuntimeException: operator colslice not implemented for TMat
  at BIDMat.Mat.notImplemented0(Mat.scala:23)
  at BIDMat.Mat.colslice(Mat.scala:80)
  at BIDMach.datasources.MatDS$$anonfun$next$1.apply$mcVI$sp(MatDS.scala:43)
  at scala.collection.immutable.Range.foreach$mVc$sp(Range.scala:160)
  at BIDMach.datasources.MatDS.next(MatDS.scala:41)
  at BIDMach.models.Model.bind(Model.scala:81)
  at BIDMach.Learner.init(Learner.scala:42)
  at BIDMach.Learner.train(Learner.scala:52)

I've got a version of colslice working. sum is the next operator of this nature to implement (or, really, a generalized reduce). I also think it's worth it, but just FYI


Update @dimforge/rapier3d-compat to 0.13.0

@dimforge/rapier3d-compat has been released - creating this issue to track updating rt/rapier and exposing new functionality.
https://github.com/dimforge/rapier.js/pull/273
Looking at the release notes, these look like the main things we'll want to expose in the existing react APIs:

[ ] Collider contact skin - https://github.com/dimforge/rapier/pull/625
[ ] Rigid body soft ccd - https://github.com/dimforge/rapier/pull/625
[ ] World.lengthUnit - "which can be used to indicate the typical size of dynamic objects (e.g. 100 pixels instead of
1 meter)"

What about ActiveCollisionTypes so we can enable contacts between static/kinematic bodies.

From a breif look doesn't look like that was added in 0.13.0, but for sure we should make sure it's exposed.

Ah thanks for opening an issue 👍 (https://github.com/pmndrs/react-three-rapier/issues/674)


Javascript Parameters with properties and dotted names

From @Beng89 on March 8, 2016 2:39
Currently intellisense does not detect parameter properties see http://usejsdoc.org/tags-param.html for details.

Is this something that support will be added for?
Copied from original issue: Microsoft/vscode#3798

This is a valide feature request, moving to TypeScript.

Seems working in latest.


Feature request - ability to localise has_one relationships

Maaaybe being a little cheeky here :wink: but thought I might suggest it, as it's come up as a requirement.
Basically rationale behind this is that I am linking to a video that has an English audio track on the English site, but for the Arabic version of the site, English is kinda useless - so a separate version is needed.
I tried my luck with also adding to the $translate array the FileID, but that doesn't seem to have worked with an UploadField.

CASE WHEN "SiteTree"."ClassName" IN ('MediaNode', 'FileNode', 'ImageNode', 'VideoNode')
		THEN "MediaNode"."FileID_en"
			WHEN "SiteTree"."ClassName" IN ('ImageNode')
				THEN "ImageNode"."FileID_en"
				ELSE NULL

This looks like a core bug not just a fluent one. ImageNode should not be in the first list, only the second.


Changed type of truncated result error from `StreamExecuteScanQuery…

…` to retryable error
I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Codecov Report

Merging #375 (11cfae5) into master (510ab4b) will decrease coverage by 9.61%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##           master     #375      +/-   ##
==========================================
- Coverage   59.62%   50.00%   -9.62%
==========================================
  Files         216      215       -1
  Lines       22916    22908       -8
==========================================
- Hits        13663    11455    -2208
- Misses       8420    10760    +2340
+ Partials      833      693     -140

Flag
Coverage Δ

1.16.x
49.64% <57.14%> (-9.67%)
:arrow_down:

1.18.x
47.00% <57.14%> (-12.54%)
:arrow_down:

1.19.x
34.72% <0.00%> (-24.81%)
:arrow_down:

connection
?

database_sql
?

discovery
?

e2e
33.43% <57.14%> (-14.96%)
:arrow_down:

integration
?

macOS-latest
34.72% <0.00%> (-0.04%)
:arrow_down:

ratelimiter
11.28% <0.00%> (+0.19%)
:arrow_up:

scripting
11.77% <0.00%> (-0.32%)
:arrow_down:

table
30.37% <57.14%> (-0.26%)
:arrow_down:

topic
?

ubuntu-latest
49.96% <57.14%> (-9.62%)
:arrow_down:

unit
34.79% <0.00%> (-0.04%)
:arrow_down:

windows-latest
34.36% <0.00%> (-0.40%)
:arrow_down:

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

Impacted Files
Coverage Δ

internal/table/scanner/scanner.go
38.47% <0.00%> (-1.73%)
:arrow_down:

internal/table/session.go
76.51% <ø> (ø)

internal/table/scanner/result.go
83.16% <100.00%> (-1.37%)
:arrow_down:

internal/xsql/valuer.go
0.00% <0.00%> (-100.00%)
:arrow_down:

internal/topic/convertors.go
0.00% <0.00%> (-100.00%)
:arrow_down:

internal/topic/retriable_error.go
0.00% <0.00%> (-100.00%)
:arrow_down:

internal/grpcwrapper/rawtopic/drop_topic.go
0.00% <0.00%> (-100.00%)
:arrow_down:

internal/grpcwrapper/rawydb/operation_params.go
0.00% <0.00%> (-100.00%)
:arrow_down:

...nternal/grpcwrapper/rawtopic/controlplane_types.go
0.00% <0.00%> (-90.33%)
:arrow_down:

internal/grpcwrapper/rawtopic/create_topic.go
0.00% <0.00%> (-86.37%)
:arrow_down:

... and 90 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


URLs update

To allow catalogartifact.azureedge.net the document says you can use AzureCloud service tag.
Is this up to date?
Because I tested in 3 different tenants and subscriptions and allowing AzureCloud service tag connectivity does not allow catalogartifact.azureedge.net.

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

ID: 406f8b6d-6471-c983-c102-9d60a57964cd
Version Independent ID: 98fa8587-36e1-690c-12c5-b865955ada0b
Content: Azure Virtual Desktop required URL list - Azure
Content Source: articles/virtual-desktop/safe-url-list.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

@jorge-ataide Thank you for your feedback! We will review and update as appropriate.

@Heidilohr can you please check if this needs an update on the doc?

Closing the loop on this - the service tag was changed from AzureCloud to AzureFrontDoor.Frontend on January 13th.
#please-close


What is the cause of this problem?

This error occurred after starting the project, but it also happens occasionally ---Error: ENOENT: no such file or directory, stat 'D:\WorkSpace\IWorkSpace\webssh2\app\client\public\client.htm'

Not sure I've seen that before. the file exists?

I found the problem. After I modified the project, public can be lost, only src is left, thank you.

I want to embed this project into the Vue project. Do you have any good suggestions? I have tried it and there is no result.

i also want to embed this project into the Vue project. :)

it would be nice if there was an easy way to just drop this in a project, but code probably has to be refactored and this more than likely turned into a module...


Shell script to set up server as systemd managed daemon on port 80

There should be a shell script that sets up the server as a systemd managed daemon. Systemd can manage the socket, allowing port 80 operation.
Example as ansible script: https://github.com/tkurki/marinepi-provisioning/tree/master/roles/node-app

Fixed by #116


fixed the date/time issue on posts

Now its showing dates for older post and posts that are uploaded on the same dates it will show the time of upload else dates from the next day.

I didn't change anything else other than the function.
On 01-Dec-2018 10:28 PM, "Mohit Kumar Yadav"<EMAIL_ADDRESS>wrote:

@mohitkyadav requested changes on this pull request.
@sudipt1999 https://github.com/sudipt1999 please do not change the part
which is not relevant to the issue in a single PR. If you find some issues,
feel free to create new issues and then work on them. As of in this case
don't change any formatting.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NJACKWinterOfCode/cruzz-web/pull/6#pullrequestreview-180530418,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AlIE0RAOSOytApJyl-3uoByFRXY2aKxvks5u0rUigaJpZM4Y8-Gb
.

Take a look in the diff https://github.com/NJACKWinterOfCode/cruzz-web/pull/6/files you changed 461 lines in this PR.

I m really sorry , it's actually I use a extension in my IDE so it aligns
and add ; or things like that automatically, I will remove it , but I
didn't made any actual change to code I only changed the function
On 01-Dec-2018 10:36 PM, "Mohit Kumar Yadav"<EMAIL_ADDRESS>wrote:

Take a look in the diff https://github.com/NJACKWinterOfCode/cruzz-web/
pull/6/files you changed 461 lines in this PR.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NJACKWinterOfCode/cruzz-web/pull/6#issuecomment-443440947,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AlIE0XjlkZ1G8mFCNWwLnjZyC4PSJzSIks5u0rcPgaJpZM4Y8-Gb
.


Final people content needed

I just noticed that there is no content for people at the moment. There is only a placeholder example at the moment. Any idea when this content might be ready? @Qianqianye @limzykenneth @davepagurek
We are moving quite fast this week so we have limited options. Unless the content is ready but not merged, I propose that we move forward with placeholder content, develop the layout, and hide the links for the people content for release.

Hi @stalgiag, we will make a small adjustment on the design that we will have around 10 people in the list with all same image hierarchy. Each person will have name, profile pic, title, and a short bio. Could you make a template in the 'People' mdx file, so I can add the content by Tuesday?

Example content is here. The layout has already been coded so it should work as expected even after the final content is in without additional code updates.

I will work on the people content today. In the 'About' page 'People' preview section, can we adjust the order of profiles?

I have added some content for the people page in the PR #259.
We outlined the people page strucutre in this doc, where we include 4 chapters: Project Leads, Project Mentors, p5.js Alumni, and p5.js contributor. We are planning to have same treatment for Project Leads, Project Mentors, p5.js Alumni, which means including same image size and bio. For the p5.js contributor, we are planning to get the name from the Contributors session.
Please let us know if you have the capacity to implement this. Thanks!
cc @stalgiag @outofambit

Hi just to follow up on this. We'd like to add the content if you could provide the page structure with chapters (Project Leads, Project Mentors, p5.js Alumni, and Contributors), we can add content from our side.
@outofambit @stalgiag @ccanash

@Qianqianye for now I think you can add everyone who is not a contributor as featured or alumni with the current schema. I can split featured into mentors and leads and adjust the styling. You would like the People page to have a header for each chapter and use the current styling for the leads for mentors, leads, and alumni within each of their respective chapters. Is that right?

Yes, that's right. We also want to follow up with a previous comments in this thread that in the 'About' page 'People' preview section, the order of the profiles didn't follow the order in the 'People' page.

@Qianqianye this is done and is currently deploying on main.

Thanks @outofambit. I added the alumni info.  We will keep this issue open until we add the contributor list. Does the Bocoup team has the capacity to add the Contributors list, as indicated in the final design?

Thanks @outofambit. One last note about people content: The 'About' page' is currently showing all people content items (see screenshot below), and I believe we are only showing the first row according to the design. Is it something you can take a look? I can open an new issue if it's easier for organizational purpose.

hi @Qianqianye, i didn't realize that 6 was a hard limit and not just the number of alumni + leads together available in the assumed content. i've limited it to 6 in 0e42fcc141e01600e2d4a613594500445a28f6f9.


fix: The problem of picture flickering after switching to the background (#793, #1598)

An attempt was made to add a WeakCache solution for Kingfisher to switch to background image flicker. For specific reasons, please refer to: https://github.com/onevcat/Kingfisher/issues/1598#issue-784729434, #793, Referenc:  https://github.com/SDWebImage/SDWebImage/blob/8d558cb79f44c00bc75e71a509021b9551941aed/SDWebImage/Core/SDMemoryCache.m#L24 .

I am not a fan of adding another hash map to the memory storage backend. To prevent the background evict, maybe we have other options, such as implementing NSDiscardableContent to the storage object. I will see...

Implemented in a better way. See #1890


Staging/deploy

Thank you for contributing to the Cohere Toolkit!

[ ] PR title: "area: description"

Where "area" is whichever of interface, frontend, model, tools, backend, etc. is being modified. Use "docs: ..."  for CI changes.
Example: "deployment: add Azure model option"

[ ] PR message: Delete this entire checklist and replace with

Description: a description of the change
Issue: the issue # it fixes, if applicable
Dependencies: any dependencies required for this change

[ ] Add tests and docs: Please include testing and documentation for your changes

[ ] Lint and test: Run make lint and make run-tests

AI Description

This PR makes changes to multiple files, primarily involving updates to code structure, dependencies, and configuration settings.
Code Changes:

src/backend/config/auth.py: Enabled OpenIDConnect as the default authentication strategy.
src/backend/main.py: Removed the verify_migrate_token import and updated the /migrate endpoint to use Alembic migrations directly. Added error logging for migration failures.
src/backend/model_deployments/cohere_platform.py: Updated error logging to include the HTTP response text when retrieving models fails.
src/backend/tools/google_drive/tool.py: Modified the is_available method to check for CLIENT_SECRET instead of CLIENT_ID.
src/interfaces/assistants_web/.env.{development,production}: Updated the NEXT_PUBLIC_FRONTEND_HOSTNAME to use port 4000 instead of 4444.
src/interfaces/assistants_web/package.json: Changed the port used in the "dev", "start", and "start:single-docker" scripts from 4444 to 4000.
src/interfaces/assistants_web/src/constants.ts: Added a new field is_private to the BASE_AGENT constant, setting it to false.
src/backend/routers/tool.py: Added an import for Settings and included a log statement to display the Redis URL.
.github/workflows/docker_push_frontend.yml: Modified the context path from ./src/interfaces/coral_web to ./src/interfaces/assistants_web.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 74.60%. Comparing base (ee97d0f) to head (8cb4ba3).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #639   +/-   ##
=======================================
  Coverage   74.60%   74.60%
=======================================
  Files         256      256
  Lines       11054    11054
=======================================
  Hits         8247     8247
  Misses       2807     2807

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


ui part updated

@AnandBaburajan  please review my code. updated the UI part

Thanks for the contribution! I'm merging this assuming that the UI will be further improved as the work progresses :D


Adjust git url validation

Description
So far, the method to prevent the input of malicious URLs has been to use a blacklist of sequences that basically prohibit the use of % or ... However, this method is too restrictive since there are urls that use for example %20. This PR revises the url checking by removing the sequence blacklist and now always resolves the given url first (e.g. https://test.com/test/%2e%2e%2f -> https://test.com/test/../ -> https://test.com/) before checking if there exists a git setting with a url that is a prefix of the given resolved url. This way we no longer disallow special characters in the URL, but still only allow URLs that are a prefix of a given git setting.
Resolves #419
Testing
First, entering a URL with a special character (%20) works when no git settings are specified, but since I do not have access to a repository using special characters in the url, I cannot test the actual creation of the git source. Second, if a Git setting is specified (e.g. https://github.com/DSD-DBS/capella-collab-manager%20) and this is entered in the absolute URL field, the correct base integration URL is selected and one can also further extend the url with a relativ path, but again I cannot test whether the actual creation works.
Checklist

[x] My code follows the guidelines of this project: CONTRIBUTING.md
[x] I updated the documentation with the new functionality
[x] I went through the code and removed all print statements, breakpoints and unused code
[x] Error handling for all possible scenarios has been implemented
[x] I've add logging for all relevant operations
[x] I added the appropriate labels to this PR (enhancement/bug/documentation, effort, priority)
[x] @MoritzWeber0 Testing the creation of Git sources using a repository with special characters

Done

[x]  Not directly related to this PR, but There exists no git setting being a prefix of the resolved url should be renamed to The resolved URL doesn't match with one of the allowed Git instances.

[x]  When the prefix doesn't match anymore, there is not error message in the frontend.

[x]  Deletion of Git instances doesn't work anymore

I can confirm that URLs with %20 work again.

Done


Could not connect to WebDriver  if trying to run tests on browser stack

What are you trying to achieve?
I want to test site on mobile device, using browserstack
What do you get instead?
Can't connect to WebDriver

Provide console output if related. Use --verbose mode for more details.

# paste output here

Can't connect to WebDriver.
Error: getaddrinfo ENOTFOUND http http:80

Please make sure Selenium Server (ChromeDriver or PhantomJS) is running and accessible

**output with verbose:**

  [1] Starting recording promises
    Emitted | suite.before ([object Object])
  "Previous" CTA on Select Shop page to become Back Button @mobile @BKM-223
    Emitted | test.before ([object Object])
    [1] Error | Error: Can't connect to WebDriver.
Error: getaddrinfo ENOTFOUND http http:80

Please make sure Selenium Server (ChromeDriver or PhantomJS) is running and accessible
    Emitted | test.failed ([object Object])
  × "before each" hook: codeceptjs.before for ""Previous" CTA on Select Shop page to become Back Button @mobile @BKM-223" in 9049ms
    [1] Error | Error: Can't connect to WebDriver.
Error: getaddrinfo ENOTFOUND http http:80

Please make sure Selenium Server (ChromeDriver or PhantomJS) is running and accessible
Error: Can't connect to WebDriver.
Error: getaddrinfo ENOTFOUND http http:80

Please make sure Selenium Server (ChromeDriver or PhantomJS) is running and accessible
    Emitted | test.after ([object Object])
    Emitted | suite.after ([object Object])

Details

CodeceptJS version: 1.4.4
NodeJS Version: 8.11.3
Operating System: Windows 10
Protractor: 5.4.1
Configuration file:

appium.conf.js:
exports.config = {
output: "./output",
helpers: {
REST: {
endpoint: "",
defaultHeaders: {
appId: "***********************************"
}
},
Appium: {
browser: "iPhone",
device: "iPhone 8 Plus",
realMobile: true,
os_version: "11.0",
user: "",
key: "********",
host: "http://hub-cloud.browserstack.com/wd/hub",
baseUrl: '*******',
waitForTimeout: 60000
},
MyHelper: {
require: "./page_objects/helpers/MyHelper.js"
}
},
include: {
    I: "./steps/custom_steps.js",
    ....
},
mocha: {},
bootstrap: false,
teardown: null,
hooks: [],
gherkin: {},
plugins: {
    allure: {
        enabled: true
    }
},
tests: "./tests/*/*_test.js",
timeout: 10000,
name: "ui"

}

@KaterynaGubar can we close the issue?


bug: standard styling for code

Problem illustrated on a blog post:

Note: This only happens when there's no language specified.
Might as well use the standard BG colors and gaps from prisma CSS Lib


Cannot login

I am self hosting. I created an account but am getting an error logging in. In the lower right hand corner there is a red message box saying:

An error occurred while logging in. Please contact support for more information.

In case I was using the wrong password I initiated the password reset process, got the email, click the link, and reset my password... after which I still was not able to login.
Other users can log in fine.
Any ideas as to what I can do to fix this?

Try again in a half hour, we have security that kicks in after a few failed logins and it doesn't matter what you do, it won't be reset. If however, you do a password reset with a valid token we may want to lift this. Did that fix it?

Were you able to resolve this issue? Please let me know if you are still running into this behavior and we'll reopen this issue


Failed to execute 'postMessage' on 'DOMWindow': The target origin provided ('file://') does not match the recipient window's origin ('null').

My code is as follows:
<!DOCTYPE html>
<html>
	<head>
		<script<EMAIL_ADDRESS>	</head>

	<body>
		<textarea></textarea>

		<script>
			const el = document.querySelector('textarea');
			const stackedit = new Stackedit();

			// Open the iframe
			stackedit.openFile({
				name: 'Filename', // with an optional filename
				content: {
				  text: el.value // and the Markdown content.
				}
			});

			// Listen to StackEdit events and apply the changes to the textarea.
			stackedit.on('fileChange', (file) => {
				el.value = file.content.text;
			});
		</script>
	</body>
</html>

It worked for a while but then the following error started showing up:

Failed to execute 'postMessage' on 'DOMWindow': The target origin provided ('file://') does not match the recipient window's origin ('null').

It appears in the console as soon as the page loads. It also appears for every key that I type in the markdown section in the editor

It will probably fail with protocol file://. Are you running your site from your hard drive?

Problem solved. Thanks.


Jump lists support

I was trying to implement jump list in my app, but I was unable to get arguments at startup that are required to interact with user. I found out that method OnLaunched is overrided and sealed in BootStrapper class. Is it possible to fix it?

Yes. Fixing now. Nuget update asap. Also about to check-in JumpList sample just for you!


Add 8M/16M support

rboot code supports 8M/16Mbyte operation but the makefile and esptool2 don't have the requisite bits.
Not tested on hardware as I've no Esp8266 devices handy with 8Mbyte or 16Mbytes!

Thanks, merged.


Make path to OpenSSL on macOS version-agnostic

Fixes #12

Tests failed, but from what I can see tests failed for your last few commits, including the last release. 😒

PR is not editable, will merge and see it tests work then.

Sorry, it's been a too long since I last used GitHub to submit an open source PR and I didn't remember that edit access to PRs was even a thing. :(
Thank you for taking care of it.


Failed to install custom protoc version in pre-build

Checklist

[X] I've looked through the issues and pull requests for similar reports

Describe your issue
I tried to compile a project requiring protobuf-compiler >=  3.15 to aarch64-unknown-linux-gnu. Currently cross-rs uses ubuntu 20.04 so the bundled protobuf-compiler is 3.6.1, so I have to manually download and install protobuf in pre-build stage. The Cross.toml looks like:
[build]
image = "ghcr.io/cross-rs/aarch64-unknown-linux-gnu:main"
pre-build = [
    "apt update && apt install -y unzip",
    "curl -LO https://github.com/protocolbuffers/protobuf/releases/download/v3.15.8/protoc-3.15.8-linux-x86_64.zip && unzip protoc-3.15.8-linux-x86_64.zip -d /usr/"
]

And when I run cross build --target aarch64-unknown-linux-gnu, from the docker buildx logs, the protobuf package is downloaded and unzipped to /usr/bin/protoc, but Rust compiler complains "Cannot find protoc":
Caused by:
  process didn't exit successfully: `/target/debug/build/greptime-proto-080c7657f1b449ef/build-script-build` (exit status: 101)
  --- stdout
  cargo:rerun-if-changed=proto/greptime/v1/database.proto
  cargo:rerun-if-changed=proto/greptime/v1/health.proto
  cargo:rerun-if-changed=proto/greptime/v1/meta/common.proto
  cargo:rerun-if-changed=proto/greptime/v1/meta/heartbeat.proto
  cargo:rerun-if-changed=proto/greptime/v1/meta/route.proto
  cargo:rerun-if-changed=proto/greptime/v1/meta/store.proto
  cargo:rerun-if-changed=proto/greptime/v1/meta/lock.proto
  cargo:rerun-if-changed=proto/greptime/v1/meta/cluster.proto
  cargo:rerun-if-changed=proto/prometheus/remote/remote.proto
  cargo:rerun-if-changed=proto/substrait_extension/promql_plan.proto
  cargo:rerun-if-changed=proto

  --- stderr
  thread 'main' panicked at 'Could not find `protoc` installation and this build crate cannot proceed without
      this knowledge. If `protoc` is installed and this crate had trouble finding
      it, you can set the `PROTOC` environment variable with the specific path to your
      installed `protoc` binary.If you're on debian, try `apt-get install protobuf-compiler` or download it from https://github.com/protocolbuffers/protobuf/releases

  For more information: https://docs.rs/prost-build/#sourcing-protoc
  ', /home/lei/.cargo/registry/src/index.crates.io-6f17d22bba15001f/prost-build-0.11.9/src/lib.rs:1457:10
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...

What target(s) are you cross-compiling for?
aarch64-unknown-linux-gnu
Which operating system is the host (e.g computer cross is on) running?

[ ] macOS
[ ] Windows
[X] Linux / BSD
[ ] other OS (specify in description)

What architecture is the host?

[X] x86_64 / AMD64
[ ] arm32
[ ] arm64 (including Mac M1)

What container engine is cross using?

[X] docker
[ ] podman
[ ] other container engine (specify in description)

cross version
cross 0.2.5
Example
git clone https://github.com/GreptimeTeam/greptimedb
cd greptimedb
cat <<EOF > ./Cross.toml
[build]
image = "ghcr.io/cross-rs/aarch64-unknown-linux-gnu:main"
pre-build = [
"apt update && apt install -y unzip",
"curl -LO https://github.com/protocolbuffers/protobuf/releases/download/v3.15.8/protoc-3.15.8-linux-x86_64.zip && unzip protoc-3.15.8-linux-x86_64.zip -d /usr/"
]
EOF

cross build --target aarch64-unknown-linux-gnu

Additional information / notes
I'm also confused that when I target aarch64 platform, looks like cross-rs created an x86_64 Ubuntu container for cross compilation. So which protoc version should I use? aarch64 or x86_64?

This is a particular issue with a hard to understand error message. Thankfully I've seen it before here.
The solution should be to chmod the binary so that it has the correct permissions. so add chmod 755 /usr/bin/protoc, does that work better?
as for your question, yes, all our current images are for a x86_64 container platform.

This is a particular issue with a hard to understand error message. Thankfully I've seen it before here.
The solution should be to chmod the binary so that it has the correct permissions. so add chmod 755 /usr/bin/protoc, does that work better?
as for your question, yes, all our current images are for a x86_64 container platform.

Thanks a lot! That did solve the problem. The error message Could not find protoc is a bit confusing so I didn't expect it to be a permission issue.


Rayon Threadpool

I added Rayon to the crate. Rayon::spawn is actually being used in several place at the moment. This can be closed.


Update and rename LICENSES to LICENSE .xml

The pull request does not conform with our quality standards: Missing link to issue and missing description etc. In addition, we are unable to see how this pull request would improve SecHub.

Oh sorry!! I'll check it out again when I'll be free for it.
Thankyou.
On Mon, Nov 6, 2023, 1:51 PM Jeeppler @.***> wrote:

Closed #2649 https://github.com/mercedes-benz/sechub/pull/2649.
—
Reply to this email directly, view it on GitHub
https://github.com/mercedes-benz/sechub/pull/2649#event-10869200930, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BDORJRIUGGYWOGSAPXCJYW3YDCMZVAVCNFSM6AAAAAA66YBTDOVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJQHA3DSMRQGA4TGMA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Added Contribution to #131 - VerletDVDLogo

I've used my verlet.js physics engine to create Bouncing DVD Logo with verlet physics.
I love :heart: verlet physics.
demo : https://anuraghazra.github.io/CanvasFun/VerletDVDLogo

Awesome ❤


SLIM-1365 Configure Definable Load Profile SLIM-1344

SonarQube analysis reported 2 issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 ActionMapperService.java#L159: Private method 'postConstruct' is never used.
 ActionMapperService.java#L110: Private method 'postConstruct' is never used.

All is well.


Update Readme to conform to current owltools build

We changed a little of where/how owltools is built, but this is not reflected in the readme.

I think this is fixed with https://github.com/owlcollab/owltools/commit/4a34c35e41fe2d074e642195aa7daee4a8ab13f7


Coronavirus updates

CHANGED
You should register the death within 5 days.
You can contact any register office but if you use the one in the area where the person died you’ll be given the documents you’ll need on the day.
If you use a different register office the documents will be sent to the office in the area where the person died before they’re issued to you. This means you’ll usually wait a few days.
TO
You should register the death within 5 days.
Contact a register office to register the death. You can contact any register office but it will be quicker if you use the one in the area where the person died.
BECAUSE
Conversation with Austin Hayes, the SME at General Register Office (GRO). They have advised register offices what to do but different register offices are doing different things - depending on where they are and how badly their area has been affected by coronavirus.
The advice is still that you should register the death within 5 days. It’s not that you ‘must’ register the death. So didn’t change that sentence.
The law was changed in the Coronavirus Act to allow death registrations to be done by phone but they don’t have to be done by phone. But people are unlikely to get the documents they need on the day because of the sheer number of deaths that are expected.
REMOVED
Registering the death will take about 30 minutes - you might need to make an appointment.
BECAUSE
We don’t know how long the phone call will take if someone is registering the death over the phone. People won’t be able to make an appointment in lots of areas.

TO
The register office will tell you what you need to do when you contact them.
BECAUSE
Medical certificates will probably be emailed by the hospital directly to the register office but that will depend on individual register offices and hospitals. Agreed with the GRO that the best thing for people to do is ask the register office what they are doing.

###If form 100A or 100B is issued (ie there is no inquest)
CHANGED
If available (but don’t worry if not), also take the person’s:

birth certificate
marriage or civil partnership certificate
NHS medical card

TO
The register office may also want to see the persons:

birth certificate
marriage or civil partnership certificate
NHS medical card

Ask the register office if they need them when you contact them.
BECAUSE
Many death registrations are going to be done over the phone, so the user won’t ‘take’ any documents to the register office. They might have to send, scan and email - but it will be up to each register office.
REMOVED
You should also take supporting documents that show your name and address (eg a utility bill) but you can still register a death without them.
BECAUSE
Many register offices will be registering deaths over the phone and will tell people what to take with them. Further up the page, we are telling people to contact the register office and that the register office will tell them what they need to do.

Our checks won't run on this because it's a PR from a fork. @NikkiAlbon We added your @NicolaAlbon account to alphagov last week, but you're using a different one. If we need to switch the account around we can do.
On the subject of the actual PR, I think this is the most justification we've ever had for making a Smart Answers content change, so that's great - thanks!

@issyl0 Do I need to do it again?

Hi @issyl0
Whenever I sign in with my Cabinet Office email, it comes up as 'Nikki Albon'.
I found the email you send to a personal email address last week but it won't let me in.
I get a message saying "Invitation not found".
Could you change them over so that I can make these edits? This is a very urgent coronavirus update

@NikkiAlbon Sure, I'll swap them around for next time! For now, I've fixed it so that the checks run on this so this can be merged as soon as those are green!

This needs to be on a branch of the main repo from the start to trigger a Heroku build, it turns out. Nikki's going to re-do it and #4412. 😭


Show a posts exact date and time

I would like to request a feature where one can hover on the relative date/time and it shows the exact time of posting.
Also, do you accept PRs? I might work on this myself if I can find some time 😀

This is a great idea, thank you!
Yes, PRs are welcome, only I'm now migrating the project to TypeScript (will likely finish today). Once that is completed, feel free to contribute, here are the brief recommendations (I think what you're planning to do would be a Watcher). I can also implement that myself if you don't have much time for that.


Update easy coding standard to ^5.1

Thanks.

Great job :+1:

Great job :+1:

Great job :+1:

@marmichalski Nice job :+1:

@marmichalski Nice job :+1:

@marmichalski Nice job :+1:

@marmichalski Nice job :+1:


Update nf-core tutorial

Updated nf-core tutorial

[x] Split into nf-core usage tutorial and contributor overview
[x] Updated nf-core usage tutorial
[x] Updated nf-core usage tutorial slides
[x] Updated nf-core contributor overview
[x] Updated nf-core contirbutor overview slides

Pending points is where to link these tutorials now 😄

Looks really great I think 🌟 . Something small for the contributing PDF:

To ask in #new-pipelines comes on the slide about releasing pipelines. Maybe this point can (also) go before the create pipelines slides?

The failing links are because the material is not yet merged!

TESTING - Summary
GitPods looks like a step towards a modern, fancy and catchy, ligth-weighted and ‘all-possible-things-integrated’ IDE. It is loaded with tons of enthusiasms of young software engineers.
To the nf-core tutorial...
What I liked:
all the url links worked
integration with VSC desktop worked
the browser extension for GitPod nested in my GitHub account
the Terminal was connected to nf workspace
What I did not like:
#Too colloquial language of the tutorial, too many "stuffing" sentences. I understand, you like to spread awesome attitude, but if there is too much text with no factual information, it causes disorientation in the text.
#The definition of Gitpod - be clear and concise. Mention IDE.
#What confused me, that I was reading the tutorial of nf-core, but was constantly redirected to external services. You might stress that supporting material is not (y)our own.
I got two error messages when playing around, but they might be irrelevant:
A)
https://8888-ggabernet-nfcorewebsite-cehmcxutnmn.ws-eu38.gitpod.io/
Port 8888 Not Found
Please make sure this port is exposed and your application is up and running.
B)
Failed to connect to Gitpod workspace ggabernet-nfcorewebsite-cehmcxutnmn: Error: Response closed without headers
Thank you for familiarising myself with a new tool!
Hanka


using const may cause problems if fundamental constants are switched on the fly.

The problem will arise if fundamental constants are switched on the fly. In this case, functions that have compiled methods may retain the old values of the const globals. This needs to be checked. One possible solution is to not use const but rather fix the type of the globals using a ::XXX suffix in the definition. This should ensure type stability.

Why would any of the __b_ values be non-constant? I thought the point of the __b_ was so that we had some constant base unit that didn't change?

We want to be able to use, say, CODATA 2014. But do we want the ability to
do that “on the fly”?
On Thu, Sep 26, 2024 at 05:53 mattsignorelli @.***>
wrote:

Why would any of the _b values be non-constant? I thought the point of
the _b was so that we had some constant base unit that didn't change?
—
Reply to this email directly, view it on GitHub
https://github.com/bmad-sim/AtomicAndPhysicalConstants.jl/issues/67#issuecomment-2376726139,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACXTUHGAOIJGKMJTNXHAZPLZYPYT7AVCNFSM6AAAAABO37FGMKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNZWG4ZDMMJTHE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***
com>

We've resolved this using functions for the physical constants that are defined using eval in setunits


Bracket matching indicator is almost no visible

Hello,
I am using Atom (0.165.0) on my Ubuntu 14.04 system (64-bit) with the solarized-dark theme and the bracket matching indicator is almost no visible at all.
I've added this
.bracket-matcher .region {
  border-bottom: 5px solid white;
  position: absolute;
}

to my style sheet to make it more visible, but failed.
It seems as if there is another layer above the indicator, that's making the indicator obscure.
This is how it looks:

and this is how it looks in the Atom Dark theme:

As you can see, it's much much clearer.
If there's somewhere I can look for fixing it and submitting a pr, just tell me so.
Regards,
Chris

Happy New Year to all!
@clhynfield Thank you for providing the link to the commit. It helped me resolve my issue!
Only problem I had, was not finding how to update the solarized-dark-syntax package.
I've tried cloning it into the ~/.atom/packages directory, but it didn't show up as a new syntax theme.

Only problem I had, was not finding how to update the solarized-dark-syntax package.

It is bundled with Atom so if you are on version 0.166 (the latest version) of Atom then then latest version of this theme, 0.30.0, will be included with it.


Style-CI Update: unnecessary parameter default value

How should we handle default values of parameters which are are not "the rightest ones".
https://styleci.io/analyses/8wEdZq
Style-CI seems to be updated with a feature to fix this parameters and remove the default value. should we go this way and remove them? this seems to be the "rightest" way because this default values has no effect on the code.
But i think the developer which has added this values has some intentions. so where can we define some "example" data - phpdoc for example but in which format should we do it there?
I have created a PR with the resolved style-ci patch https://github.com/sulu-io/sulu/pull/1913

Merged the PR #1913, is resolved now.


[Suggestion] Add Typescript definition file

First off, thanks for the project!
Thought I'd share this and hope to make this package typescript friendly. Not sure if this is all that's required to get this working, but it did a lot for me. This assumes that the 'name' option is left at 'ls'. I still had to import like const Storage = require('vue-ls') - not sure if the fix is with a definitions file or modification of the shipped package.
import _Vue, { PluginFunction } from 'vue';

declare module 'vue/types/vue' {
  interface Vue {
    $ls: WebStorage;
  }

  interface VueConstructor {
    ls: WebStorage;
  }
}

declare class WebStorage {
  public length: number;

  constructor(storage: Storage);
  get(name: string, def?: any): any | null;
  set(name: string, value: any, expire?: number): void;
  key(index: number): string | null;
  remove(name: string): void;
  clear(): void;
  on(name: string, callback: Function): void;
  off(name: string, callback: Function): void;
}

declare interface StorageOptions {
  namespace?: string;
  name?: string;
  storage?: StorageTypes;
}

export enum StorageTypes {
  Session = 'session',
  Local = 'local',
  Memory = 'memory',
}

declare const _default: {
  install: PluginFunction<StorageOptions>;
};

export default _default;

Thank you!
I will soon study the question and think about what can be done

Yeah, TS support is important... I am getting erros while using it with TS

INE/timeline-vue/src/main.ts
3:21 Could not find a declaration file for module 'vue-ls'. 'C:/xx/_xx/xx-vue/node_modules/vue-ls/dist/vue-ls.js' implicitly has an 'any' type.
Try npm install @types/vue-ls if it exists or add a new declaration (.d.ts) file containing declare module 'vue-ls';

@RobinCK any plans for this? would be much appreciated

At the moment I don't have plans to add typings.
But PR is always welcome

@RobinCK Do you have any idea about below error?
Yes need typescript support, i have also facing that issue only

https://github.com/RobinCK/vue-ls/issues/23#issuecomment-460181232

I did that now im not getting that issue after declaring like  declare module 'vue-ls'; But below error still getting
147:9 Element implicitly has an 'any' type because expression of type '"$ls"' can't be used to index type BannerComponent

Property '$ls' does not exist on type 'BannerComponent'.


Application label causing manifest merger issues

Gradle is failing when trying to merge manifest files. I try to use the replace attribute as suggested but it does not work. Have you come across this at all? What is the purpose of the label being defined in your library's manifest? Comparing to other popular third-party android libraries, the label attribute is not usually defined in the library manifest.
Attribute application@label value=(ClinicalKey DEBUG) from AndroidManifest.xml:20:9-41
is also present at [com.btkelly:gandalf:1.3.0] AndroidManifest.xml:15:9-41 value=(@string/app_name).
Suggestion: add 'tools:replace="android:label"' to  element at AndroidManifest.xml:15:5-19:11 to override.

I just stumbled upon this https://code.google.com/p/android/issues/detail?id=193679. Looks like there is a bug in the build tools with no workaround. I believe removing your label should fix the problem. I can submit a pull request with the proposed change.

Ah, this is a good catch. This is caused by the fact that you have hard coded the label value inside your application block. I will include a fix for this in the next release, in the meantime you can create a string resource like so <string name="app_name">ClinicalKey DEBUG</string> and setting your application label to android:label="@string/app_name".
Great find, thanks for reporting this.

Feel free to submit a PR, always appreciate the help.

The hardcoded value is on in my debug/AndroidManifest.xml. I get the same error when building release which uses a string resource:
Attribute application@label value=(@string/clinicalkey_title) from AndroidManifest.xml:20:9-50
is also present at [com.btkelly:gandalf:1.3.0] AndroidManifest.xml:15:9-41 value=(@string/app_name).
Suggestion: add 'tools:replace="android:label"' to  element at AndroidManifest.xml:15:5-417:19 to override.

Seems like the issue is I am using the tools:ignore attribute with the tools:replace and that is the bug in the linked android bug report. Only workaround I have found is removing the tools:ignore attribute.

The issue comes from the fact that the label is already defined and cannot be different. If you update your label string resource name to app_name instead of clinicalkey_title for all build types this should resolve the issue. When the merging occurs it will replace the library version of app_name with your version. This is still a work around and I would like to resolve it by removing the label from the lib itself.

Ahh, you are right, that did the trick. Thanks! Great work on the library by the way. Working on a prototype to build this into our app and stumbled upon this library. Need to convince the team its worthy of using and hopefully the iOS guys like iOS version as well. Anyway, I am going to submit a PR. Do you think any of that application tag is necessary? Most libraries don't have one at all.

Probably not, you can pull everything and just test compilation with the built in example app. If that works fine should be good to go.


Suggestion to make fn's in TickDataProvider trait mutable to fetch tick data during calculation

I am working on a library (rethdb-dexsync) where I would like to fetch tick data on demand from a Reth node database. However, it seems that the TickDataProvider requires prefetching all tick data during initialization, as the data cannot be modified during the calculation.
Since &self is not mutable, I currently don't see a way to dynamically fetch and return a reference to new fetched ticks. Is this a use case you would like to support? Or am I missing something, perhaps there is a deref workaround that I haven't considered?
pub trait TickDataProvider {
    type Index: TickIndex;
    fn get_tick(&self, tick: Self::Index) -> Result<&Tick<Self::Index>, Error>;
    fn next_initialized_tick_within_one_word(
        &self,
        tick: Self::Index,
        lte: bool,
        tick_spacing: Self::Index,
    ) -> Result<(Self::Index, bool), Error>;
}

Are you trying to implement TickDataProvider on your own type? Have you tried wrapping your tick data provider in RefCell? Making TickDataProvider use &mut self will likely incur a larger refactor, requiring Pool and Position to be mutable as well.
I also don't see why reading from Reth requires mutability, unless you cache the result inside get_tick.

Exactly, I really like the possibility of bringing my own provider and have tried to do so. And you're right, I want to cache the result in the provider to reuse it for multiple calculations. The problem I'm facing is returning a reference. My current code looks like this, but the RefCell approach fails to compile, and I get the following error:
error[E0515]: cannot return value referencing local variable `ticks_ref`
  --> examples/univ3_sdk_pool.rs:44:24
   |
43 |         match ticks_ref.get(&tick_id) {
   |               --------- `ticks_ref` is borrowed here
44 |             Some(v) => Ok(v),
   |                        ^^^^^ returns a value referencing data owned by the current function

code:
fn get_tick(&self, tick_id: Self::Index) -> Result<&Tick<Self::Index>, Error> {
    ...
    let tick = Tick::new(tick_id, tick_info.liquidity_gross.to(), i128::try_from(tick_info.liquidity_net).unwrap());
    self.ticks.borrow_mut().insert(tick_id, tick);

    let ticks_ref = self.ticks.borrow();
    match ticks_ref.get(&tick_id) {
        Some(v) => Ok(v),
        None => Err(Error::InvalidTick(tick_id)),
    }
}

The problem is how I can store somewhere a reference in this function and return it. I got the idea that the return value is of course a reference to not clone the tick every time. But make returning the fn just a Tick would also help me to move forward. I don't know if it makes sense somehow to have a get_tick() and get_tick_ref() where the ref function is always evaluated first. I would appreciate any hints or suggestions to resolve this issue.

Exactly, I really like the possibility of bringing my own provider and have tried to do so. And you're right, I want to cache the result in the provider to reuse it for multiple calculations. The problem I'm facing is returning a reference. My current code looks like this, but the RefCell approach fails to compile, and I get the following error:
error[E0515]: cannot return value referencing local variable `ticks_ref`
  --> examples/univ3_sdk_pool.rs:44:24
   |
43 |         match ticks_ref.get(&tick_id) {
   |               --------- `ticks_ref` is borrowed here
44 |             Some(v) => Ok(v),
   |                        ^^^^^ returns a value referencing data owned by the current function

code:
fn get_tick(&self, tick_id: Self::Index) -> Result<&Tick<Self::Index>, Error> {
    ...
    let tick = Tick::new(tick_id, tick_info.liquidity_gross.to(), i128::try_from(tick_info.liquidity_net).unwrap());
    self.ticks.borrow_mut().insert(tick_id, tick);

    let ticks_ref = self.ticks.borrow();
    match ticks_ref.get(&tick_id) {
        Some(v) => Ok(v),
        None => Err(Error::InvalidTick(tick_id)),
    }
}

The problem is how I can store somewhere a reference in this function and return it. I got the idea that the return value is of course a reference to not clone the tick every time. But make returning the fn just a Tick would also help me to move forward. I don't know if it makes sense somehow to have a get_tick() and get_tick_ref() where the ref function is always evaluated first. I would appreciate any hints or suggestions to resolve this issue.

Try
        let ticks_ref = unsafe { self.ticks.as_ptr().as_ref() }.unwrap();

Thank you very much! This solution is now working for me. I will proceed with using unsafe for the time being, but I would prefer to eliminate it in the future. If you ever consider changing the trait or have a different approach in mind, please let me know. I would be happy help with the development or create a PR for that.


Fix Cross-Origin issue for Camps-Index-API

*Error when trying to call api

Not an issue yet,
Need to implement JSONP.


DD-330

Proposed changes

Related issues

Checklist

[x] I consider the submitted work as finished
[x] I tested the code for its functionality
[ ] I wrote test cases for the relevant uses case
[ ] I added/update the relevant documentation (either on github or on notion)
[ ] Where necessary I refactored code to improve the overall quality

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

@r-joyce I've tested things in UI with the issue you've addressed. But it doesn't seem to be duplicating fields in the UI. If you check Risk > Remediation where we're not using any pagination, we still see this error but it doesn't seem to be duplicating fields in the UI.


Bump Python and RAPIDS versions

NeMo only supports python>=3.10 so we need to bump our version too. Only bumping the Python version gives this error during installation:
cugraph 23.12.0 requires dask-cuda==23.12.*, but you have dask-cuda 23.10.0 which is incompatible.
cugraph-service-server 23.12.0 requires dask-cuda==23.12.*, but you have dask-cuda 23.10.0 which is incompatible.
cuml 23.12.0 requires dask-cuda==23.12.*, but you have dask-cuda 23.10.0 which is incompatible.
raft-dask 23.12.0 requires dask-cuda==23.12.*, but you have dask-cuda 23.10.0 which is incompatible.
rapids-dask-dependency 23.12.1 requires dask==2023.11.0, but you have dask 2023.9.2 which is incompatible.
rapids-dask-dependency 23.12.1 requires distributed==2023.11.0, but you have distributed 2023.9.2 which is incompatible.

By bumping the RAPIDS packages too, this error goes away. GPU Unit tests pass as well. Addresses #17

@ryantwolf I suggest updating README.md and explicitly mentioning the minimum required Python version.


Using userspace buffers

Is it possible to use buffers provided by the user to avoid a copy? Right now it seems that the buffers are allocated by library itself.
https://github.com/katyo/linux-video-rs/blob/51dc1b6bc2642db2c1dbfb73b3fb01f76b4e2689/core/src/impls/buffer.rs#L187

@FallingSnow It's possible by using Linux Video API, but the transferring external buffers to kernel currently hasn't implemented yet.
In other words currently you can use userspace buffers which created by this crate not youself.


Extracting the call handler for Spark commands from kernel.

Description
This is the concept of extracting the call to Spark commands from processing via routing. Commands are called directly, as when using the command() function. This will reduce the load on the console application in the form of Request, Response, Filters and Routes services and more.
In the beginning, the idea of creating an Application class was considered, from which the CodeIniter and Console classes would be inherited. This was necessary to initialize the application. But due to the fact that, at the request of one person, the core of the application turned into a service, there were difficulties with the implementation.
Therefore, the initialization of the application through the CodeIgniter class is saved.
The Console class now uses the Services::commands() service directly instead of using the CommandRunner class.
The CodeIgniter class has been cleared of processing Spark commands, and the Console class no longer processes Response.
Although this is not within the purview of the PR, I added the StreamFilterTrait trait, which correctly adds and removes filters for capturing stream output, since the current implementation in the tests is not correct enough in my opinion.
Depends on StreamFilterTrait https://github.com/codeigniter4/CodeIgniter4/pull/6112
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[x] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

I have not looked into this PR yet, but this is a big change.
It would take some time to review.
So could you please split StreamFilterTrait into a separate PR?
It is small and will be merged sooner than this.

@kenjis StreamFilterTrait https://github.com/codeigniter4/CodeIgniter4/pull/6112

I'm still in favor of this. The rest of the team is oddly quiet 😅 Let's get the User Guide and changelog updates done and then I will give them one last chance.

@iRedds Thank you for updating the docs.
One last thing, Upgrading Guide also needed.
Like this: https://codeigniter4.github.io/CodeIgniter4/installation/upgrade_420.html#mandatory-file-changes
See https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/pull_request.md#upgrading-guide

@kenjis I have doubts about the quality of the upgrade manual.

Please add.
--- a/user_guide_src/source/installation/upgrading.rst
+++ b/user_guide_src/source/installation/upgrading.rst
@@ -12,6 +12,7 @@ upgrading from.
 .. toctree::
     :titlesonly:

+    upgrade_430
     upgrade_421
     upgrade_420
     upgrade_418

I think the upgrading is okay. Thanks.