Not working from external IP

Hi,
Following #126 and #128.
I'am trying to access OpenDataCam from external IP (4G or even LAN through a router)
Here is my setup :
Adlink Neon camera - USB <- USB/Ethernet Dongle -> Ethernet - Raspberry Pi with OpenWrt (router) - USB <- USB/Ethernet Dongle -> Macbook
No docker used there.
The OpenWrt is configurated with firewall rules :
Any IP:8080 from LAN or WAN -> Adlink camera IP:8080 (on LAN of course)

Addressing other services, on other IP:Port (with other firewall rules) are Ok.
Addressing standalone local server "$ serve -l 8080" is Ok.
Here is the problem :
Addressing < Camera IP >:8080, the request never ends. No output, nothing.
But :
Addressing < Camera IP >:8080/status is OK (JSON received)
Addressing < Camera IP >:8080/console is OK (same output as on console)
I've test the patch provided in #128, no change.(edited)
Any help ?
I can help for tests if needed.
Thanks

Sorry,
Applying patch #128 gives me :

But can't go further.

Can you copy the complete log output here? From this snippet I'm not sure if darknet is running. If darknet can not be started you will be stuck on the init screen forever.

Yes, Darknet is running ok.
Acceding to opendatacam without the router (directly on same LAN) is Ok, and running as expected.
Adding the router cause the problem.
Here is the full log.
adlink@adlink-desktop:~/XXXXXXXX/opendatacam$ npm run start

><EMAIL_ADDRESS>start /home/adlink/XXXXXXXX/opendatacam
> PORT=8080 NODE_ENV=production node server.js

-----------------------------------
-     Opendatacam initialized     -
- Config loaded:                  -
{
  "OPENDATACAM_VERSION": "3.0.1",
  "PATH_TO_YOLO_DARKNET": "/home/adlink/XXXXXXXX/darknet",
  "VIDEO_INPUT": "usbcam",
  "NEURAL_NETWORK": "yolov4-tiny",
  "VIDEO_INPUTS_PARAMS": {
    "file": "opendatacam_videos/demo.mp4",
    "usbcam": "v4l2src device=/dev/video0 ! video/x-raw, framerate=30/1 ! videoconvert ! appsink",
    "raspberrycam": "nvarguscamerasrc ! video/x-raw(memory:NVMM),width=1280, height=720, framerate=30/1, format=NV12 ! nvvidconv ! video/x-raw, format=BGRx, width=640, height=360 ! videoconvert ! video/x-raw, format=BGR ! appsink",
    "remote_cam": "YOUR IP CAM STREAM (can be .m3u8, MJPEG ...), anything supported by opencv",
    "local_cam": "pylonsrc imageformat=rgb8 ! videoconvert ! appsink",
    "remote_hls_gstreamer": "souphttpsrc location=http://YOUR_HLSSTREAM_URL_HERE.m3u8 ! hlsdemux ! decodebin ! videoconvert ! videoscale ! appsink"
  },
  "TRACKER_SETTINGS": {
    "objectMaxAreaInPercentageOfFrame": 80,
    "confidence_threshold": 0.2,
    "iouLimit": 0.05,
    "unMatchedFrameTolerance": 5
  },
  "COUNTER_SETTINGS": {
    "minAngleWithCountingLineThreshold": 5,
    "computeTrajectoryBasedOnNbOfPastFrame": 5
  },
  "VALID_CLASSES": [
    "*"
  ],
  "DISPLAY_CLASSES": [
    {
      "class": "bicycle",
      "hexcode": "1F6B2"
    },
    {
      "class": "person",
      "hexcode": "1F6B6"
    },
    {
      "class": "truck",
      "hexcode": "1F69B"
    },
    {
      "class": "motorbike",
      "hexcode": "1F6F5"
    },
    {
      "class": "car",
      "hexcode": "1F697"
    },
    {
      "class": "bus",
      "hexcode": "1F68C"
    }
  ],
  "PATHFINDER_COLORS": [
    "#1f77b4",
    "#ff7f0e",
    "#2ca02c",
    "#d62728",
    "#9467bd",
    "#8c564b",
    "#e377c2",
    "#7f7f7f",
    "#bcbd22",
    "#17becf"
  ],
  "COUNTER_COLORS": {
    "yellow": "#FFE700",
    "turquoise": "#A3FFF4",
    "green": "#a0f17f",
    "purple": "#d070f0",
    "red": "#AB4435"
  },
  "NEURAL_NETWORK_PARAMS": {
    "yolov4": {
      "data": "cfg/coco.data",
      "cfg": "cfg/yolov4.cfg",
      "weights": "yolov4.weights"
    },
    "yolov4-tiny": {
      "data": "cfg/coco.data",
      "cfg": "cfg/yolov4-tiny.cfg",
      "weights": "yolov4-tiny.weights"
    }
  },
  "TRACKER_ACCURACY_DISPLAY": {
    "nbFrameBuffer": 300,
    "settings": {
      "radius": 3.1,
      "blur": 6.2,
      "step": 0.1,
      "gradient": {
        "1": "red",
        "0.4": "orange"
      },
      "canvasResolutionFactor": 0.1
    }
  },
  "MONGODB_URL": "mongodb://<IP_ADDRESS>:27017",
  "PORTS": {
    "app": 8080,
    "darknet_json_stream": 8070,
    "darknet_mjpeg_stream": 8090
  }
}
-----------------------------------
Process YOLO initialized
> Ready on http://localhost:8080
> Ready on http://<IP_ADDRESS>:8080
Success init db
Process YOLO started
 CUDA-version: 10020 (10020), cuDNN: 8.0.0, GPU count: 1
 OpenCV version: 4.1.1
 0 : compute_capability = 620, cudnn_half = 0, GPU: NVIDIA Tegra X2
   layer   filters  size/strd(dil)      input                output
   0 (node:8600) [DEP0001] DeprecationWarning: OutgoingMessage.flush is deprecated. Use flushHeaders instead.
First request on console stream
conv     32       3 x 3/ 2    416 x 416 x   3 ->  208 x 208 x  32 0.075 BF
   1 conv     64       3 x 3/ 2    208 x 208 x  32 ->  104 x 104 x  64 0.399 BF
   2 conv     64       3 x 3/ 1    104 x 104 x  64 ->  104 x 104 x  64 0.797 BF
   3 route  2                                  1/2 ->  104 x 104 x  32
   4 conv     32       3 x 3/ 1    104 x 104 x  32 ->  104 x 104 x  32 0.199 BF
   5 conv     32       3 x 3/ 1    104 x 104 x  32 ->  104 x 104 x  32 0.199 BF
   6 route  5 4                                    ->  104 x 104 x  64
   7 conv     64       1 x 1/ 1    104 x 104 x  64 ->  104 x 104 x  64 0.089 BF
   8 route  2 7                                    ->  104 x 104 x 128
   9 max                2x 2/ 2    104 x 104 x 128 ->   52 x  52 x 128 0.001 BF
  10 conv    128       3 x 3/ 1     52 x  52 x 128 ->   52 x  52 x 128 0.797 BF
  11 route  10                                 1/2 ->   52 x  52 x  64
  12 conv     64       3 x 3/ 1     52 x  52 x  64 ->   52 x  52 x  64 0.199 BF
  13 conv     64       3 x 3/ 1     52 x  52 x  64 ->   52 x  52 x  64 0.199 BF
  14 route  13 12                                  ->   52 x  52 x 128
  15 conv    128       1 x 1/ 1     52 x  52 x 128 ->   52 x  52 x 128 0.089 BF
  16 route  10 15                                  ->   52 x  52 x 256
  17 max                2x 2/ 2     52 x  52 x 256 ->   26 x  26 x 256 0.001 BF
  18 conv    256       3 x 3/ 1     26 x  26 x 256 ->   26 x  26 x 256 0.797 BF
  19 route  18                                 1/2 ->   26 x  26 x 128
  20 conv    128       3 x 3/ 1     26 x  26 x 128 ->   26 x  26 x 128 0.199 BF
  21 conv    128       3 x 3/ 1     26 x  26 x 128 ->   26 x  26 x 128 0.199 BF
  22 route  21 20                                  ->   26 x  26 x 256
  23 conv    256       1 x 1/ 1     26 x  26 x 256 ->   26 x  26 x 256 0.089 BF
  24 route  18 23                                  ->   26 x  26 x 512
  25 max                2x 2/ 2     26 x  26 x 512 ->   13 x  13 x 512 0.000 BF
  26 conv    512       3 x 3/ 1     13 x  13 x 512 ->   13 x  13 x 512 0.797 BF
  27 conv    256       1 x 1/ 1     13 x  13 x 512 ->   13 x  13 x 256 0.044 BF
  28 conv    512       3 x 3/ 1     13 x  13 x 256 ->   13 x  13 x 512 0.399 BF
  29 conv    255       1 x 1/ 1     13 x  13 x 512 ->   13 x  13 x 255 0.044 BF
  30 yolo
[yolo] params: iou loss: ciou (4), iou_norm: 0.07, obj_norm: 1.00, cls_norm: 1.00, delta_norm: 1.00, scale_x_y: 1.05
  31 route  27                                     ->   13 x  13 x 256
  32 conv    128       1 x 1/ 1     13 x  13 x 256 ->   13 x  13 x 128 0.011 BF
  33 upsample                 2x    13 x  13 x 128 ->   26 x  26 x 128
  34 route  33 23                                  ->   26 x  26 x 384
  35 conv    256       3 x 3/ 1     26 x  26 x 384 ->   26 x  26 x 256 1.196 BF
  36 conv    255       1 x 1/ 1     26 x  26 x 256 ->   26 x  26 x 255 0.088 BF
  37 yolo
[yolo] params: iou loss: ciou (4), iou_norm: 0.07, obj_norm: 1.00, cls_norm: 1.00, delta_norm: 1.00, scale_x_y: 1.05
Total BFLOPS 6.910
avg_outputs = 310203
 Allocate additional workspace_size = 26.22 MB
Loading weights from yolov4-tiny.weights...Demo
net.optimized_memory = 0
mini_batch = 1, batch = 1, time_steps = 1, train = 0
nms_kind: greedynms (1), beta = 0.600000
nms_kind: greedynms (1), beta = 0.600000
Done! Loaded 38 layers from weights-file

 seen 64, trained: 0 K-images (0 Kilo-batches_64)
video file: v4l2src device=/dev/video0 ! video/x-raw, framerate=30/1 ! videoconvert ! appsink
[ WARN:0] global /home/nvidia/host/build_opencv/nv_opencv/modules/videoio/src/cap_gstreamer.cpp (933) open OpenCV | GStreamer warning: Cannot query video position: status=0, value=-1, duration=-1
setvideoresolution
Video stream: 2592 x 1944
Objects:

 JSON-stream sent.

FPS:0.0          AVG_FPS:0.0
Objects:

Thank you for the logs. I have a theory why Opendatacam is not handling your NAT setup correctly. I believe what happens is the following:

Opendatacam listens on any IP address/hostname for incoming connections on Port 8080.
The first client connecting to <odc>:8080/ or <odc>:8080/start starts the Darknet process (see server.js)
Opendatacam then tries to connect to Darknet using the <odc> IP address/hostname from your browser (see Opendatacam.js). This works fine unless you use NAT.
Because now Opendatacam tries to connect to Darknet on your public IP/hostname ports 8070 and 8090. However as there are no port forwarding rules setup, the connection will fail.

Therefore, to fix this you have two options:

Setup port forwarding for 8070 and 8090
Fix the Darknet hostname in https://github.com/opendatacam/opendatacam/blob/development/server/Opendatacam.js#L792 to localhost (I have not tried this, but something like this should work)

Thanks for your answer.
Option 1 :
Here is the port forwarding rules :

Addressing < IP >:8070 : Ok, I can see JSON output from Darknet
Addressing < IP >:8090 : Ok, I can see the video stream
But, the opendatacam UI still does not show up :(
Option 2 (Option 1 is still on):
The opendatacam UI show up and working correctly, but can't see the video stream :(
I'am sure we're close :)
May be we need to do the same fix for the video stream ?
Thanks,

I modified server.js with :
express.get('/webcam/stream', (req, res) => {
const urlData = getURLData(req);
// Proxy MJPEG stream from darknet to avoid freezing issues
return new MjpegProxy(http://localhost:8090).proxyRequest(req, res);
});
And it works.
But I'am sure there is a better way to do that !!!

Sorry I forgot about the MJPEG Proxy! For now I believe we could fix it to localhost, like you did, to get Opendatacam working behind NAT.
Long term we should consider making this configurable for more elaborate setups where Darknet is running on a different machine.

@bobmorane83 I've created a follow up issue to fix this. If your problem is now solved feel free to close this here.

Thanks for your support.


Introduce new commitment format to enforce channel lease expirations in scripts

Replaces https://github.com/lightningnetwork/lnd/pull/5549
See PR body there

Rebased again, itests got borked during the original rebase, looking into the failure, prob a stray guard based on the new commitment types somewhere.

Rebased once again! Found an issue in the OG rebase thanks to @guggero


[sw/dif_pwrmgr] Power Manager DIF Library Header

This change introduces the header of the power manager DIF library that defines its public interface.
Notes

The specification for power manager is not published yet but you can find it in lowRISC/opentitan#3119.
The number of wake-up and reset sources are yet to be finalized. This DIF library can currently support upto 32 but we expect to have a small number of sources for each, e.g. 3 to 5.
Wake-up and reset sources don't have any names yet. Once the design is finalized, we plan to define them (auto-generated) in hw/top_earlgrey/sw/autogen/top_earlgrey.h (lowrisc/openTitan#3138).
Thus, the interface for interacting with sources may (probably will) change based on the final design.
Reset reasons are recorded by reset manager, not power manager.
ABORT wake-up reason may go away (depends on if we really need to protect any peripherals from an illegal low power entry).
@tjaychen please let me know if I missed anything.

Signed-off-by: Alphan Ulusoy<EMAIL_ADDRESS>

Thanks for your reviews @jon-flatley, @sriyerg, @tjaychen, @lenary! I implemented your comments in the last push. Looking forward to more feedback.

I think it is worth convert this header API to the new DIF template by @mcy . It will avoid churn, as it would be unfortunate to implement this API just to do name refactoring on both .cand  .h.

Last update:

Updated definitions to comply with new naming guidelines.
Addressed comments.
Updated API to use bitmasks and removed some redundant functions.
Added DIF checklist.

@gkelly or @silvestrst can you take a quick look and let me know if you have any concerns?

Last update: Incorporated @silvestrst's and @lenary's comments.
I will merge this once the CI is done.


Update the privacy center docker build/push to follow the same patterns as the main application

Is your feature request related to a specific problem?
The privacy center image publishing is a bit different and less robust than the main image that is leveraging the nox session.
Describe the solution you'd like
The privacy center docker publish should be moved into the same file and follow the same patterns as the main application

Important to complete after merging #1508


NIFI-6714 Corrected OSUtils to use Process.pid interface method

Description of PR
NIFI-6714 Corrects OSUtils to use the java.lang.Process interface pid() method instead of the implementation class pid() method, which avoids illegal reflective access warnings during startup on Java 11, and allows NiFi to be started on Java 13.  The getProcessId() method retains the platform-specific logic to determine a child Process ID on macOS, Linux, and Windows for Java 8.  This update introduces a unit test for getProcessId() that succeeds on currently supported platforms, and also removes the unused getJavaVersion() method.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically main)?

[X] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[X] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[X] Have you written or updated unit tests to verify your changes?
[X] Have you verified that the full build is successful on JDK 8?
[X] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

Reviewing..

I can confirm that the "Illegal reflective access" error when building with Java 8 and running on Java 11 are now gone with these changes. Awesome, thanks, exceptionfactory! +1 will merge.


{Packaging} Fix DEB install script

Resolve #28984.
There is no tty in GitHub runner. This error is raised when run gpg --dearmor -o /etc/apt/keyrings/microsoft.gpg:
gpg: cannot open '/dev/tty': No such device or address
Revert to the save method from the previous commit.
Related PR: #28250

Packaging

The same change should be made on https://learn.microsoft.com/en-us/cli/azure/install-azure-cli-linux?pivots=apt.

The document is used in an interactive environment, and the user can handle it when File '/etc/apt/keyrings/microsoft.gpg' exists. Overwrite? (y/N) appears.


Updated cheatsheet.md with 'get taints' command

Fix #34450

Hey @taniaduggal,
The content in this PR is same as @Kwazimolo's PR #34450, could you explain a little bit about what's the purpose of this PR?

@taniaduggal I appreciate your first time effort for the contribution to Kubernetes Docs. Please look out for some good-first-issues and you can work on that. Happy to help 🙂
Closing this PR as it's duplicate of #34450
Thank you!
/close


Always respond with a referer after a logout request, successful or unsuccessful.

@hongsudt this is something we want reviewed ASAP so I'll put it in current milestone...

@svoinea can you review and close this feature from @ljpearlman ?

The feature was tested on rbk-dev and works as expected.
The ermrest_config.json file needs to have these two lines:
  "default_logout_path": "/chaise/logout",
  "globus_auth_logout_redirect_name": "RBK"


Refreshing JWT

Is it possible to refresh tokens? In documentation, it provides an example of how to create a bearer token with a expire time but not how to refresh the token.
In flask it has a package for it: Flask-JWT-Extended
I looked up source code of  FastAPI and Python-JOSE couldn't find anything.

As I can see - it's impossible now. But you can recreate token by the same data and it will work like update.
For example:
import datetime
from time import sleep

from jose import jwt
from jose.exceptions import ExpiredSignatureError

if __name__ == "__main__":

    data = {"bearer": "It's an example of bearer"}
    secret = "Secret key"
    expired = 5

    data["exp"] = datetime.datetime.utcnow() + datetime.timedelta(
        seconds=expired
    )

    token = jwt.encode(data, secret)

    print(token)

    sleep(expired + 1)

    try:
        decrypted = jwt.decode(token, secret)
    except ExpiredSignatureError:
        data["exp"] = datetime.datetime.utcnow() + datetime.timedelta(
            seconds=expired
        )
        token = jwt.encode(data, secret)

        print(token)

        decrypted = jwt.decode(token, secret)

    print(decrypted)

Thanks for the help @shidenko97 and thanks for closing the issue @ycd! 🍰

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


How to disable gestures on iOS?

I need to disable gestures on iOS, but as I see on the code, they are enabled by default if we are on that platform:
https://github.com/fram-x/FluidTransitions/blob/6797b810f36f61db10cbb8fffcb241079e295d4a/lib/FluidTransitioner.js#L244

Should be working - we're using this in our customer apps. Use navigationOptions´ and set gesturesEnabled` to false.


dask dashboard does not produce the correct/accessible link in ml-notebooks

The dask dashboard is not properly configured on the ml-notebook. When I do the following to be able to view the dashboard
import distributed

client = distributed.Client()
client

I get the dashboard link http://<IP_ADDRESS>:46197/status, which not accessible on the cloud. It should instead be something like: https://m2lines.2i2c.cloud/user/dhruvbalwada/proxy/8787/status (as is the case for the regular notebooks).

Probably caused by https://github.com/2i2c-org/infrastructure/issues/1530

@dhruvbalwada try now?

This works.


Structured logging and log levels

We need to improve MCAD logging in particular switching to structured logging and revisiting log levels and log verbosity to facilitate log analysis.

Relates to project-codeflare/codeflare-operator#113.
/cc @KPostOffice

@tardieu Great :-) I think that this will be a perfect alignment with the metrics work that @dimakis started here: https://github.com/project-codeflare/multi-cluster-app-dispatcher/pull/486
@tardieu as part of the log levels setting, can you split between (a) operational logs (which are more important) and (b) debugging/developer logs (which are also very important) --- from what I saw when I scanned the code a lot of the logs are developer-centric and less "operations" centric.


Add option for running the logger with flushing to disk after each log call

It is currently very difficult to troubleshoot crashing extension host issues because the logs could be missing data that was not synchronously flushed to disk after each log call.

@joaomoreno @sandy081 Could you please squeeze this in for September? I would like to enable the extension host to always use synchronous flushing to disk after each log call so I can troubleshoot crashes...

I think we should do this with an env variable or cli argument, since as a setting we might not be able to read it as early as we'd like at startup and we'd only want this for troubleshooting. My preference would be with an env variable, since that propagates effortlessly. Is that OK @alexandrudima?

@joaomoreno I would hard-code the extension host to always use synchronous flushing.
TBH I don't understand why one would ever use logging without synchronously flushing ^^

Because this will block every time a log statement is written. I cannot say how much it will impact the perf but why do not this is optional and enable this only when needed?

Anyways, I have added a constructor option sync in SpdLogService which is false by default. Passing true will flush the log after each call - d34109e2ba87ce3df3e7a77a8beccf1bd606357f

As discussed exposed flush method on logService to flush logs.

Thank you! :heart:


HBASE-24960 reduce invalid subprocedure task

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 40s
Docker mode activated.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 24s
the patch passed

+1 :green_heart:
compile
1m 11s
the patch passed

+1 :green_heart:
javac
1m 11s
the patch passed

+1 :green_heart:
shadedjars
7m 16s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 41s
hbase-server in the patch failed.


mailto for e-mails appearing in webchat

Is your feature request related to a problem? Please describe.
When showing an e-mail address, we need to make it clickable and it needs to open a mailto the like of Outlook mailto.
Describe the solution you'd like
when I click an e-mail in the webchat, I want it to redirect me to a mail window with it in send parameters.

Mailto already work on QA.
Just use markdown syntax to make it work
The response<EMAIL_ADDRESS>


Update to 10.0.0-M5, 9.0.35, 8.5.55

This release adjusts the Apache Tomcat Native library output to no longer include "APR based":
Loaded APR based Apache Tomcat Native library
->
Loaded Apache Tomcat Native library

(That textual adjustment is the reason https://doi-janky.infosiftr.net/view/flat/job/update.sh/job/tomcat/ is failing.)


Sound card configurable. Localizable for zh-Hans.

Please git rebase to master instead of merging master.

ok

The PR is good, you just need to make some changes.


Problem with build on raspberry pi3b+

Look in picture. How to resolve?

Latest version of ocamlfuse is 2.7.1-cvs6. You should run opam update && opam upgrade to get the latest version.


Extension host terminated unexpectedly

VSCode Version: 1.27.2
OS Version:

Steps to Reproduce:
Add this following extension in your VS code studio and do some work you will get popup of extension host terminated unexpectedly in within 10 mins
1.

Does this issue occur when all extensions are disabled?: Yes/No
No

I suggest that you report this to https://github.com/angular/angular. They have a bunch of issues for its language service: https://github.com/angular/angular/issues?q=is%3Aopen+is%3Aissue+label%3A"comp%3A+language-service"


How to Improve Training Parameters.

Thanks for building such an awesome tool!
I'm training a “robot” wake word detection model, but the results are not as good as I expected. I'd like to ask for some advice on how to adjust the training parameters to improve accuracy and recall. I have an Nvidia RTX 4090 graphics card to use. Here are my current parameters and effects.
Current Parameters:

n_samples: 50000
steps: 100000
target_accuracy: 0.7
target_recall: 0.9
max_negative_weight: 1500
augmentation_rounds: 3
model_type: dnn
layer_size: 128
target_false_positives_per_hour: 0.9

Results

Final Model Accuracy: 0.7114999890327454
Final Model Recall: 0.42500001192092896
Final Model False Positives per Hour: 2.4778761863708496

Hi tebie, you may find the docs/models folder useful, it describes the steps to train good models.


V almonacid/ch2323/add wallet transfer support

EDIT: removing WIP label after some additions/fixes/refactoring though not all functionalities  are ready. Remaining issues will be addressed in follow-up PR(s).
This PR removes the wallet check button and adds the shelley wallet upgrade flow and hopefully all  the functions required to generate the wallet migration transaction. However, the transaction construction will fail as there are still some issues that need to be fixed in the rust bindings.
Restore wallet on Shelley button:

Modification on restore screen
The restore wallet screen is modified by just passing a isShelleyWallet navigation parameter, which changes the navigation theme ad-hoc.

Verify restored wallet

Wallet upgrade

Confirm wallet upgrade

When no upgrade tx is required

https://github.com/Emurgo/yoroi-frontend/pull/1287 make sure this is included also


MultiLine string literal fix

Figured out that the reason why Literals wasn't recognizing the string as a multiline string was because I accidentally put the wrong sign for the newline character using "/n" instead of "\n". The tests are working on my end now.

Ha! Of course it's this simple, lol! We can merge this anytime you want, but it's probably safer to not make any more changes until after your presentation.


Make waitForInput in Http4sUtils public

For use in quasar-advanced

Current coverage is 79.31%

Merging #1090 into master will not affect coverage as of 4122ae0

@@            master   #1090   diff @@
======================================
  Files          173     173
  Stmts         9692    9692
  Branches       314     314
  Methods          0       0
======================================
  Hit           7687    7687
  Partial          0       0
  Missed        2005    2005

Review entire Coverage Diff as of 4122ae0

Powered by Codecov. Updated on successful CI builds.


YETUS-963. prototool-basedir should be a cwd

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/yetus-github-multibranch/job/PR-107/1/console in case of problems.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m 39s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

_ Patch Compile Tests _

+1 :green_heart:
shellcheck
0m  2s
There were no new shellcheck issues.

+1 :green_heart:
shelldocs
0m  0s
There were no new shelldocs issues.

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

_ Other Tests _

+1 :green_heart:
asflicense
0m 10s
The patch does not generate ASF License warnings.

3m  6s

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://builds.apache.org/job/yetus-github-multibranch/job/PR-107/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/yetus/pull/107

Optional Tests
dupname asflicense shellcheck shelldocs

uname
Linux 1055845f2da6 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
precommit/src/main/shell/personality/yetus.sh

git revision
master / 9f99186

Max. process+thread count
46 (vs. ulimit of 2000)

modules
C: precommit U: precommit

Console output
https://builds.apache.org/job/yetus-github-multibranch/job/PR-107/1/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.7.0

Powered by
Apache Yetus 0.12.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.


Dragging onto layer will still emit a Click event

layer = new Layer
layer.on Events.Click, -> print 'issue'

... then drag from outside the layer, onto the layer: issue appears in the console.
I presume that a standard Click event should work only when the touch starts and ends over the same layer. Is this a bug or there's a better approach to restrict the Click events?

Events.Click is just indeed a shorthand for "mouseup" on desktop and "touchend" on touch devices. If you want to have the native "click" event, you can use:
layer = new Layer
layer.on "click", -> print 'issue'

I agree that this is counterintuitive, but there's probably a good reason for this. @koenbok can you elaborate?


pyclass: refactor tp_new / tp_dealloc / remove PyCellInner

This PR is a bunch of cleanups to PyCell and PyClassInitializer code for the slot implementations of tp_new, tp_alloc, tp_dealloc and tp_free:

tp_alloc and tp_free are only implemented for freelist classes, where they provide custom memory allocation / release
tp_new now does the same thing for all #[pyclass] (call super tp_new and initialize the PyCell contents)
tp_dealloc does the reverse of tp_new (drop the PyCell contents and then call super tp_dealloc)

This work seems necessary to me as a precursor to #1647, #1637, #1344 and #991.
While doing this I also found a nasty bug that we don't call tp_dealloc of inherited native types, which means that dict subclasses never decrease reference counts of all their contents on drop (for example).
I added a benchmark for class instantiation. Sadly these cleanups don't seem to have meaningfully changed the benchmark. I did notice that we're a lot faster than the numbers reported in #661, so there must have been progress over time.

Oooh nice, a segfault purely on windows 32-bit 👀

Turns out I forgot #[repr(C)] from PartiallyInitializedPyCell 😬

If nobody reviews of raises any concerns by the weekend I plan to merge this.


Persisting data and distributing apps using a registry should not be in the tutorials section

To me, the pages around persisting data and distributing applications should not be in the same section as a tutorial about building a URL shortener.

I was just revisiting the Hosting Spin Apps in OCI Registries page mentioned here and was about to suggest a few updates.  However, I'm wondering if we might consider removing this tutorial especially now that the Publishing and Distribution guide appears to cover most, if not all, of the demonstrated functionality?  cc @tpmccallum @itowlson

This is being updated as part of the V3 ToC work.


Consolidate verification of entries when we have multiple verifiers

Add a description

The entry verify flow is repeated in very similar ways across the different verify funcs. We should see if there are ways to consolidate these so they don't diverge in subtle ways.

From @wlynch in https://github.com/gittuf/gittuf/pull/372#discussion_r1647670092
Relevant log output if the discussion pertains to existing gittuf functionality
No response
Code of Conduct

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

This has been addressed with the consolidation into verifyGitObjectAndAttestationsUsingVerifiers.


listObjects Exception

Hi, this looks quite promising.
I'm using it to download files from a storage bucket for now.
for the configuration I used:
bucket: "$awsBucket.s3.$awsRegion.amazonaws.com"
(notice the .s3)
s3client.getObject // works :)
s3client.buildSignedGetParams // works :)
ListBucketResult listBucketResult = await s3client.listObjects(prefix: "/", delimiter: "/");
Exception has occurred.
DeserializationError (Deserializing '[Error, {Code: AccessDenied, Message: Access Denied, RequestId: 6AF7623EE000E...' to 'ListBucketResultParker' failed due to: Tried to build class "ListBucketResultParker" but nested builder for field "result" threw: Tried to construct class "ListBucketResult" with null field "name". This is forbidden; to allow it, mark "name" with @nullable.)

Hi,
There seems to be 2 problems:

You don't have the permission to list objects in this bucket. Make sure the IAM user has the following rights:

ListBucket
ListBucketVersions (I think, might work without that)

This library doesn't handle permisson errors at all at the moment :) I'll try to reproduce this error and release a new version that returns a better error description.

You are so right, I think that must be it.

Version 0.2.0 throws an exception with a description what happened in such cases.

Nice, thanks. Wonderful plugin btw.


Fix off-by-one error for empty strings

Previously parsing this would break: {"key":""}
jsonExtract("{"key":""}", "key")  -> ""}"

Thank you @softplus !


Error

Please tick all applicable boxes.

[x ] I am using Python 3.5.3 or higher (run python --version on the command line)
[ x] I have followed the official guides to install the bot for my system
[x ] I have updated my dependencies to the latest version using the appropriate update script

Which version are you using?

[x ] The latest master version (release-260419)
[ ] The latest review version

What type of issue are you creating?

[x] Bug
[ ] Feature request
[ ] Question

Description of issue
Steps to reproduce

Error queuing playlist:
float division by zero
when trying to use youtube auto gen playlist

Log file
Please attach your MusicBot log file (located at logs/musicbot.log) to this issue. You can do so by dragging and dropping the file here. If you do not include your log file, you WILL be asked to provide one.
musicbot.log

Last I checked youtube playlist aren't supported by ytdl.

Oddly enough I've always been able to play auto gen playlist from youtube till this last update. but okay.


azure-cli: add tests for cli with extensions/with mutable config

Description of changes
Adding some tests so I don't have to check things manually each time...
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

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

@katexochen Do you want this to be merged as it is?

I'm going to merge this to proceed with a minimal azure-clirefactor. Initially by-name and nixfmt. Then, tweaking dependencies. And think of what else can be done to make upkeep of this package easier. (I don't have a solution yet.) But I find this package tricky as it is.


Material UI Icons loses displayName property in production environment?

Material UI Icons loses displayName property in production environment.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

I'm drilling down Material UI Icons from parent Sidebar component to children SideBarIcon components. When working in a developer environment, I have access to the displayName property. This conveniently disappears when I create a production build.
Expected Behavior 🤔

In developer environment I should still have access to the displayName property.
Steps to Reproduce 🕹

https://i.stack.imgur.com/QddKv.png
Display Name Property is gone in npm run build. Picture with displayName is what I need
Code is seen here
https://stackoverflow.com/questions/63421869/material-ui-icons-loses-displayname-property-in-production-environment
Context 🔦

Building a portfolio
Your Environment 🌎

Tech
Version

@material-ui/core
v4.11.0

@material-ui/icons
v4.9.1

React
v16.8.4

display names should only be defined in development, not in production.


the game didnt crack

0xe0000010  error apears

Please send a screenshot of the error.
Also please tell me the game name.

the game is city driver

Please send screenshot and give me steam link of the game.

That is aderow drm, not crackable by my app. Find crack elsewhere.


update init.sh to use telegraf directories

A recent change (120218f) causes the packages to use the telegraf directories instead of the influxdb directories.  The init.sh script is still using the influxdb directories, however causing the packaged service to not start.

+1 This bit me today as well

ah, sorry about that - this one is my fault. thanks for the fix, @jseriff!
i'll get this merged in now and cut another release shortly.


[Bug] Settings Page Unresponsive in Homebridge

Please use the template and fill in all the data requested below!
Describe the bug

I have installed the z2m plugin 3 times already to see if the issue goes away, but it doesn't. Once the plugin installed, I click on SETTINGS for this plugin and I get "Page Unresponsive" after about 5-10 seconds trying to click and update the values such as Server, etc. The only options given are "Exit Page" or "Wait". Clicking EXIT, crashes the page. Clicking WAIT, does nothing.

Related devices
Are you experiencing this behavior with a particular device? Yes
Please list which device(s) you experience this with.

In Homebridge VM hosted on Proxmox

To Reproduce
Steps to reproduce the behavior, for example:

Install Zigbee2mqtt from homebridge UI
Go to Plugins page and click on settings for z2m
click around on the screen for few seconds

Expected behavior

It should allow me to update the server in the settings page and save it, however, it hangs and rashes.

MQTT messages
If possible (and applicable), please provide the following:

Nothing shows up in the messages

Versions used
Please provide the version of the following pieces of software:

homebridge-z2m v1.6.0
Homebridge v1.3.4
Zigbee2MQTT 1.21.1-1

Screenshot

Might be similar to #169.

Which browser are you using?
Have you tried a different one?
Does the console of the browser show anything particular?
You can also use the settings view as a workaround and/or edit the config file directly.

@itavero: Thank you for your quick response. I am using the latest version of Chrome on Mac: 94.0.4606.61
I do have the uBlock Origin installed on Chrome, not sure if that messes with it or not, just tried Safari and everything is working fine. I should've have know this and tried a different browser.
It doesn't show anything particular, it just shows what you see in the screenshot.

It might be that extension. I'm not sure, as I don't use Chrome anymore or that extension.
You can try to temporarily disable it of course.
Regardless, it sounds like it's a problem with homebridge-config-ui-x, rather than with this plugin.

same issue here but on safari, I realized that Adguard (extension) was causing that.

Closing this issue due to lack of activity.


Unzip with SAF

Hello, can you make this library work with SAF or DocumenfFile methods?

The implementation has been done though very late I know, but as they say, better late than never. Thanks for giving the DocumentFile suggestion. The changes have been done and released as part of 1.2.0. @warr24-tech


Creating a subfolder module

Hello,
I don't know if this is a bug or that's how it should be.
I want to have my modules to be created in subfolders with the folder names in the module's classes namespace.
But when trying to create a module within a subfolder php artisan module:make Store\Product it will throw this error:
Created : C:\xampp\htdocs\bnaturalia\bnaturalia-api\Modules/Store\Product/module.json
Created : C:\xampp\htdocs\bnaturalia\bnaturalia-api\Modules/Store\Product/Routes/web.php
Created : C:\xampp\htdocs\bnaturalia\bnaturalia-api\Modules/Store\Product/Routes/api.php
Created : C:\xampp\htdocs\bnaturalia\bnaturalia-api\Modules/Store\Product/Resources/views/index.blade.php
Created : C:\xampp\htdocs\bnaturalia\bnaturalia-api\Modules/Store\Product/Resources/views/layouts/master.blade.php
Created : C:\xampp\htdocs\bnaturalia\bnaturalia-api\Modules/Store\Product/Config/config.php
Created : C:\xampp\htdocs\bnaturalia\bnaturalia-api\Modules/Store\Product/composer.json
Created : C:\xampp\htdocs\bnaturalia\bnaturalia-api\Modules/Store\Product/Resources/assets/js/app.js
Created : C:\xampp\htdocs\bnaturalia\bnaturalia-api\Modules/Store\Product/Resources/assets/sass/app.scss
Created : C:\xampp\htdocs\bnaturalia\bnaturalia-api\Modules/Store\Product/webpack.mix.js
Created : C:\xampp\htdocs\bnaturalia\bnaturalia-api\Modules/Store\Product/package.json

   Nwidart\Modules\Exceptions\ModuleNotFoundException

  Module [Store\Product] does not exist!

  at C:\xampp\htdocs\bnaturalia\bnaturalia-api\vendor\nwidart\laravel-modules\src\FileRepository.php:396
    392▕         if ($module !== null) {
    393▕             return $module;
    394▕         }
    395▕
  ➜ 396▕         throw new ModuleNotFoundException("Module [{$name}] does not exist!");
    397▕     }
    398▕
    399▕     /**
    400▕      * Get all modules as laravel collection instance.

  1   C:\xampp\htdocs\bnaturalia\bnaturalia-api\vendor\nwidart\laravel-modules\src\Traits\ModuleCommandTrait.php:16
      Nwidart\Modules\FileRepository::findOrFail("Store\Product")

  2   C:\xampp\htdocs\bnaturalia\bnaturalia-api\vendor\nwidart\laravel-modules\src\Commands\SeedMakeCommand.php:85
      Nwidart\Modules\Commands\SeedMakeCommand::getModuleName()

this package not work with php artisan module:make Store\Product
Please use php artisan module:make Product
if You want create package Product inside folder Store, please config
 'paths' => [
        /*
        |--------------------------------------------------------------------------
        | Modules path
        |--------------------------------------------------------------------------
        |
        | This path used for save the generated module. This path also will be added
        | automatically to list of scanned folders.
        |
        */

        'modules' => base_path('Modules'), // change to base_path('Modules/Store')

or change scan config
    'scan' => [
        'enabled' => false,
        'paths' => [
            base_path('vendor/*/*'),
        ],
    ],

or register autoload to Modules/Store

this package not work with php artisan module:make Store\Product
Please use php artisan module:make Product
if You want create package Product inside folder Store, please config
 'paths' => [
        /*
        |--------------------------------------------------------------------------
        | Modules path
        |--------------------------------------------------------------------------
        |
        | This path used for save the generated module. This path also will be added
        | automatically to list of scanned folders.
        |
        */

        'modules' => base_path('Modules'), // change to base_path('Modules/Store')

i don't want to change the Modules path, I just want to add some modules in system-based folders and namespace

or change scan config
    'scan' => [
        'enabled' => false,
        'paths' => [
            base_path('vendor/*/*'),
        ],
    ],

or register autoload to Modules/Store

will that allow me to add \Store in the namespace or it will just look for Modules\Product instead of having \Modules\Store\Product ?

or change scan config
    'scan' => [
        'enabled' => false,
        'paths' => [
            base_path('vendor/*/*'),
        ],
    ],

or register autoload to Modules/Store

will that allow me to add \Store in the namespace or it will just look for Modules\Product instead of having \Modules\Store\Product ?

You can try,
    'scan' => [
        'enabled' => true,
        'paths' => [
            base_path('Modules/*/*'),
        ],
    ],

It will scan package insilde Modules/Store/Product
Default laravel-packages only scan package inside Modules folder

or change scan config
    'scan' => [
        'enabled' => false,
        'paths' => [
            base_path('vendor/*/*'),
        ],
    ],

or register autoload to Modules/Store

will that allow me to add \Store in the namespace or it will just look for Modules\Product instead of having \Modules\Store\Product ?

You can try,
    'scan' => [
        'enabled' => true,
        'paths' => [
            base_path('Modules/*/*'),
        ],
    ],

It will scan package insilde Modules/Store/Product
Default laravel-packages only scan package inside Modules folder

I have tried adding it to the scan config, but still not able to use php artisan module:make Store/Product
I only can create php artisan module:make Product and move it into the Store folder but this not just enough since the Product module's classes namespace will remain as Modules\Product\Http\Controllers\ProductController instead of having  Modules\Store\Product\Http\Controllers\ProductController which is what is needed to be.
Adding a module in the folder without having the folder name in the classes namespace is not a good code structure

why you not rename Modules to Store and put Prodcut inside Store

why you not rename Modules to Store and put Prodcut inside Store

Renaming the namespace in the module's files manually should not be an option at a 2021 package.

No no, rename folder Modules to Store and config namespace and Modules path
<?php

use Nwidart\Modules\Activators\FileActivator;
use Nwidart\Modules\Commands;

return [

    /*
    |--------------------------------------------------------------------------
    | Module Namespace
    |--------------------------------------------------------------------------
    |
    | Default module namespace.
    |
    */

    'namespace' => 'Store',
    'paths' => [
        /*
        |--------------------------------------------------------------------------
        | Modules path
        |--------------------------------------------------------------------------
        |
        | This path used for save the generated module. This path also will be added
        | automatically to list of scanned folders.
        |
        */

        'modules' => base_path('Store'),

when you run php artisan module:make Product, It will create Package Product with namespace is Store/Product inside folder Store

No no, rename folder Modules to Store and config namespace and Modules path
<?php

use Nwidart\Modules\Activators\FileActivator;
use Nwidart\Modules\Commands;

return [

    /*
    |--------------------------------------------------------------------------
    | Module Namespace
    |--------------------------------------------------------------------------
    |
    | Default module namespace.
    |
    */

    'namespace' => 'Store',
    'paths' => [
        /*
        |--------------------------------------------------------------------------
        | Modules path
        |--------------------------------------------------------------------------
        |
        | This path used for save the generated module. This path also will be added
        | automatically to list of scanned folders.
        |
        */

        'modules' => base_path('Store'),

when you run php artisan module:make Product, It will create Package Product with namespace is Store/Product inside folder Store

as i said i don't want to change the modules base path, since there will be other system-based folders this is not a good idea
--- Modules
----- User
----- Account
----- Comment
----- Store
------- Category
------- Product
------- Option
----- Courses
------- Course
------- Registration

your solution doesn't work with the example of the paths above since your solution is just replacing Modules with Store but nothing has really changed

First of all this method does not exist
php artisan module:make Store\Product
One solution is to create a command and do the necessary logic required to implement it.
The second solution is to create all files manually and register them in the service provider of the created module.
the third one is to pass it subfolder I don't know if it is what you need example:
php artisan module:make-controller Product/ProductController Store
will result in the main module Store subfolder Product inside will be your controller,
php artisan module:make-model Product/Product Store
will have result Main module Store subfolder Product inside will be your Model, check the documentation of the web or pass helper help in your terminal to see the commands and how they work.
What you want to do can not be done you must do well first step or second step
reviewing its structure as you want to have it would be something like this.
https://nwidart.com/laravel-modules/v6/advanced-tools/artisan-commands
--- Modules
----- User
----- Account
----- Comment
----- Store Main Module
------- Category don't create subfolder the correct thing would be model and controller I will attach image
------- Product
------- Option
----- Courses
------- Course
------- Registration

First of all this method does not exist
php artisan module:make Store\Product
One solution is to create a command and do the necessary logic required to implement it.
The second solution is to create all files manually and register them in the service provider of the created module.
the third one is to pass it subfolder I don't know if it is what you need example:
php artisan module:make-controller Product/ProductController Store
will result in the main module Store subfolder Product inside will be your controller,
php artisan module:make-model Product/Product Store
will have result Main module Store subfolder Product inside will be your Model, check the documentation of the web or pass helper help in your terminal to see the commands and how they work.
What you want to do can not be done you must do well first step or second step
reviewing its structure as you want to have it would be something like this.
https://nwidart.com/laravel-modules/v6/advanced-tools/artisan-commands
--- Modules
----- User
----- Account
----- Comment
----- Store Main Module
------- Category don't create subfolder the correct thing would be model and controller I will attach image
------- Product
------- Option
----- Courses
------- Course
------- Registration

I think the method should be created and give the ability to create submodules inside a module, it will be much better than having a whole project(e-commerce) logic inside one module. It can't be called a modular app if each module includes 50% of the website logic.
So if it is not a bug could be a feature request?

I would recommend you to make Pull requests, and that you add that functionality to the package, but as I said before you must make a function to create sub folders and the necessary command for terminal.
I recommend you to see asgard
https://github.com/AsgardCms/Platform

I would recommend you to make Pull requests, and that you add that functionality to the package, but as I said before you must make a function to create sub folders and the necessary command for terminal.
I recommend you to see asgard
https://github.com/AsgardCms/Platform

might do it after i finish up my current project, thx for the tips


Continuous build for Windows

Add continuous build for Windows via AppVeyor.

[x] Create AppVeyor project
[x] Add appveyor.yml to repo
[x] Fix test.cmd to pick up RID
[x] Version packages with build number
[x] Push packages to myget.org

AppVeyor project created.
Tests are not using the test.cmd because it breaks on the build machine, which has a different RID for the full framework build and so win7-x86 isn't found. I'm not sure how to discover the acutal RID automatically. Apparently, the one reported by dotnet --info is not necessary the one that may be used by dotnet build. For example, on my Windows 10 machine, dotnet --info reports an RID of win10-x86 but builds into a directory named win7-x86.

Closed by 96d3ac2ea4428b7e752a9afc91408f950b43da49.


fix(pipe): change static pipe overload to reflect source observable t…

Description: add overloads to static pipes functions to enable typing to reflect the correct type from source Observable.
Related issue (if exists):  #4110

Coverage increased (+0.2%) to 96.96% when pulling b73d4e8d5e98231ff2479fbb8a528a088d1a260f on dkosasih:static_pipe_typing into 6a475ccf6276625dd5655479b89be249008d76f4 on ReactiveX:master.

@cartant  what would you suggest about the commit type? I personally don't think it's a chore, not sure of it's feat.

fix is fine, but the explanation needs to be like the related issue, IMO:
fix(pipe): static pipe should return generic functions when possible


I was using this PS since long time and it works perfect, recently when I tried to run the PS " Remove-AzVM -ResourceGroupName $resourceGroup -Name $vmName" I deleted the disk with the VM itself which is very risky, please have a look at this.

[Enter feedback here]

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

ID: 7eed234f-9a8d-4c1b-2368-fe4c8fbe8299
Version Independent ID: 876980d5-e42c-b070-c0a6-73ed3f2c91d1
Content: Change a VMs availability set using Azure PowerShell - Azure Virtual Machines
Content Source: articles/virtual-machines/windows/change-availability-set.md
Service: virtual-machines
GitHub Login: @cynthn
Microsoft Alias: cynthn

Thanks for your feedback. Please provide us with the link to the documentation you are referring to so we can better assist.

Here you go: Change a VMs availability set using Azure PowerShell - Azure Virtual Machines | Microsoft Docshttps://docs.microsoft.com/en-us/azure/virtual-machines/windows/change-availability-set
Thanks,
Suliman Abu Kharroub
Support Engineer
Microsoft Azure Technical Support CSS
Office: +962 (6) 5504290
Mobile:<PHONE_NUMBER>321
<EMAIL_ADDRESS>9 AM - 6 PM UTC+2 Sun-Thu
@.***
 If you need to work with another Azure Technical Support Engineer outside of my working hours, please email APTS @.***> with your case number, availability and email thread.  One of my colleagues will contact you.
If you have any feedback about my work, please let either me or my manager Mohammad Zahran<EMAIL_ADDRESS>From: Karishma Tiwari - MSFT @.>
Sent: Thursday, February 10, 2022 1:11 AM
To: MicrosoftDocs/azure-docs @.>
Cc: Suliman Abu Kharroub @.>; Author @.>
Subject: Re: [MicrosoftDocs/azure-docs] I was using this PS since long time and it works perfect, recently when I tried to run the PS " Remove-AzVM -ResourceGroupName $resourceGroup -Name $vmName" I deleted the disk with the VM itself which is very risky, pleas...
Thanks for your feedback. Please provide us with the link to the documentation you are referring to so we can better assist.
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F87724%23issuecomment-1034295022&data=04|01|Suliman.AbuKharroub%40microsoft.com|48b4bf94cdfe44313a0408d9ec216af2|72f988bf86f141af91ab2d7cd011db47|1|0|637800450578452323|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=%2F4CEHt4OtapI3RR7V5BrBkoxuXKWT%2BlditHXOkNqfPE%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FARGENMTV2DTWTUME4WOQKN3U2LX7XANCNFSM5N5KYGPA&data=04|01|Suliman.AbuKharroub%40microsoft.com|48b4bf94cdfe44313a0408d9ec216af2|72f988bf86f141af91ab2d7cd011db47|1|0|637800450578501730|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=p31J2V0lvAckpv9OvUrbtWxYW2SNVOKBMfxUxQOzfFs%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|Suliman.AbuKharroub%40microsoft.com|48b4bf94cdfe44313a0408d9ec216af2|72f988bf86f141af91ab2d7cd011db47|1|0|637800450578501730|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=5bdmgIdi5z3rFFiu%2FWmkuqo0Zh%2BRt0dcAuheJZ3OtDw%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|Suliman.AbuKharroub%40microsoft.com|48b4bf94cdfe44313a0408d9ec216af2|72f988bf86f141af91ab2d7cd011db47|1|0|637800450578501730|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=9bShGS2KnvZr5DVwhL5teeJH91b%2Bx7xipFPrK4QKeY4%3D&reserved=0.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

More details obtained offline from the original poster of this GitHub issue:
"The expected result when using remove-azvm cmdlet is to delete the VM but keep the OS disk untouched but recently an update were done on the cmdlet behavior, which make to delete the VM and the OS disk together without any warning or confirmation message about the data in the OS disk which is risky"
Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

I added in a warning that you should make sure that the disks are set to detach instead of delete. Thank you!
#please-close


In-tree autotools files cleanup

I'm not sure what we should include in the git tree vs. in the dist tar balls regarding autotools files. In theory, the dist tarball should simply be complete enough to execute ./configure && make to build it, so most stuff would be needed (all the way op to configure and Makefile.in). For the Git repo however, it makes sense to have a developer at least refresh it once when working on the code.
Looking at http://i.stack.imgur.com/SlURG.png it seems to make sense to me to cut off at the configure.ac and Makefile.am line and have people use the bootstrap.sh script as-is to generate their aclocal.m4, config.h.in and Makefile.in on the fly. Since a developer should have all tools anyway to build this stuff, it wouldn't be much of a problem.
Anyone here have any input on this?

#213 closes this.


Remove algorithm factory

Some methods has been removed from _cv::Algorithm) class:

various get/set accessors
static create(string)
virtual info

removed init_ModuleName()
ml module interfaces has been changed:

replaced Params classes with virtual get/set methods

Needs Itseez/opencv#3679 to work.

:+1:


[CP] dart2js test and builder configuration changes

Commit(s) to merge
d77e22a20bcf11f6d2107af9971edbe434cfe54b
Target
beta
Prepared changelist for beta/stable
https://dart-review.googlesource.com/c/sdk/+/287905
Issue Description
We updated the configurations for running dart2js test builders in the SDK repo. There are accompanying changes in the infrastructure but that config is outside of the SDK repo and is the same across main, beta, and stable branches.
What is the fix
Changes to the dart2js test and builder configurations that work with the infrastructure changes.
Why cherry-pick
Allows dart2js test builders to run on the beta branch. Otherwise they will fail on CQ/CI.
Risk
low
Issue link(s)
https://github.com/dart-lang/sdk/issues/51481
Extra Info
No response

Approved @athomas has already given a review on the CL.

Submitted.


Update Dart grammar

Description
Reconfigure the highlighter for Dart away from Dart Atom which is no longer being maintained to a new highlighter under the dart-lang org.
Checklist:

[X] I am changing the source of a syntax highlighting grammar

Old: https://github-lightshow.herokuapp.com/?utf8=✓&scope=from-url&grammar_format=auto&grammar_url=https%3A%2F%2Fgithub.com%2Fdart-atom%2Fdart%2Fblob%2Fmaster%2Fgrammars%2Fdart.cson&grammar_text=&code_source=from-url&code_url=https%3A%2F%2Fgist.githubusercontent.com%2Fmit-mit%2Fb8549d317f24e8b9cdd618fc92c11aa7%2Fraw%2Fe13f789d71b55c9767030bd448eefc2d49e4ff23%2Fnewfeatures.dart&code=

New: https://github-lightshow.herokuapp.com/?utf8=✓&scope=from-url&grammar_format=auto&grammar_url=https%3A%2F%2Fgithub.com%2Fdart-lang%2Fdart-syntax-highlight%2Fblob%2Fmaster%2Fgrammars%2Fdart.json&grammar_text=&code_source=from-url&code_url=https%3A%2F%2Fgist.githubusercontent.com%2Fmit-mit%2Fb8549d317f24e8b9cdd618fc92c11aa7%2Fraw%2Fe13f789d71b55c9767030bd448eefc2d49e4ff23%2Fnewfeatures.dart&code=

There are not a lot of changes right now (if any), but we're about to deprecate the dart-atom repo, and it will no longer be maintained.

Thanks for the approval, and thanks @rkirsling for the thumbs-up.
@pchaigno I don't seem to be able to merge. Will you do that?


@line-62 : webpack has first letter capitalised.

describe your changes...
@line-62 : webpack has first letter capitalised. (documentation guides/hot-module-replacement)

Closing in favor of #1655 which I'll merge momentarily.


Test co-existence of KServe and ModelMesh

[ ] that following installation sequence works: KServe --> MM
[ ] that reverse installation sequence works: MM --> KServe
[ ] MM+KServe: model can be deployed and inferenced with KServe
[ ] MM+KServe: model can be deployed and inferenced with MM
[ ] model can be deployed and inferenced with KServe after MM is uninstalled
[ ] model can be deployed and inferenced with MM after KServe is uninstalled
[ ] MM+KServe is uninstalled fine and re-installed again with no errors

[ ] model inferences with KServe and MM

@vaibhavjainwiz this is the test list

The scope of this issue is to test the KServe and MM could co-exist in the same namespace

I faced below issue while testing the KServe and ModelMesh in same namespace:
https://github.com/opendatahub-io/odh-model-controller/issues/125


Update proof verification

Summary of changes
Changes introduced in this pull request:

Fixes winning post proof check (and sub functions)

Bumps proofs version

Implements new ProofVerifier trait to allow for verification to be swapped out for a mock verifier for testing

There was also a bunch of duplicated and incorrect logic, so consolidating was necessary
This trait just has associated functions, and is just used with PhantomData so it has no runtime cost

I'll get ahead of a few questions since I assume it might not be intuitive from scanning changes:

Why not have the runtime Syscalls trait a supertrait (yes that's the name) of ProofVerifier?

While there is overlap, syscalls does not need the functionality of verifying winning post proofs or generating sector challenge ranges, so that is not a direct subset and would require implementing extra logic for other runtimes/VM.

Why not have StateManager own the ProofVerifier generic?

This is a bit more opinionated, but it made more sense to me to only require the generic type when calling a function that uses the VM. Negative to this is that it opens up a possibility of calling a function with a full and mock verifier, and they could theoretically disagree on state because the full verifier can fail where the mock verifier would succeed. This can always change when #713 is implemented (and our strategy of swapping this out is clear)

Why was the default proof verification removed from Syscalls trait?

Didn't want to add the generic bound on the trait, as only the DefaultSyscalls type has the option of using the generic. Also because the logic was moved into the ProofVerifier trait.

Why were the state_manager::utils functions changed to implementation of StateManager?

Idk, why was it setup the way it was in the first place? Got very verbose quick with introducing the extra generic.

Why do the RPC functions that use the VM default to FullVerifier implementation?

The mechanism for swapping verifier implementation at runtime isn't decided, and didn't want to add another generic bound to the RPC state when the node also defaults to FullVerifier.
Reference issue to close (if applicable)

Closes #712
Other information and links

Why were the state_manager::utils functions changed to implementation of StateManager?
Probably because they are really DB wrappers was the original reason they weren't in the SM impl.

Why were the state_manager::utils functions changed to implementation of StateManager?
Probably because they are really DB wrappers was the original reason they weren't in the SM impl.

I don't know what you mean, the Lotus implementation separates into own functions not associated, which I assume is the reason, but it makes no sense to do in Rust and adds a lot of clunkyness with generics. (https://github.com/filecoin-project/lotus/blob/bc495a5c8cdf3934b4f559f735141bdeaf06c64d/chain/stmgr/utils.go)


shred: improve some output to match GNU's

Still fail on other issues after this
FAIL: tests/shred/shred-passes
==============================

--- exp 2023-12-24 11:41:06.832370909 +0100
+++ out 2023-12-24 11:41:06.848370855 +0100
@@ -1,3 +1,6 @@
+shred: f: pass 1/3 (random)...
+shred: f: pass 2/3 (random)...
+shred: f: pass 3/3 (random)...
 shred: f: removing
 shred: f: renamed to 0
 shred: f: removed
1+0 records in
1+0 records out
102400 bytes (102 kB, 100 KiB) copied, 6.59261e-04 s, 102 MB/s
--- exp 2023-12-24 11:41:06.876370762 +0100
+++ out 2023-12-24 11:41:06.876370762 +0100
@@ -1,23 +1,7 @@
-shred: f: pass 1/20 (random)...
-shred: f: pass 2/20 (ffffff)...
-shred: f: pass 3/20 (924924)...
-shred: f: pass 4/20 (888888)...
-shred: f: pass 5/20 (db6db6)...
[...]


Infinite Scroll works on desktop, breaks on mobile.

Desktop works as expected, however on mobile the loader is at the bottom and the scroll never triggers the next function.  I also tested on just a smaller viewport, not a mobile impersonation and the same issue occurs.  Any advice on how to fix?

I'm also looking for a fix

Check this #87 issue

Here is an example https://codesandbox.io/s/peaceful-banzai-9sqfd
Test it on mobile. It works correctly on desktop but not on mobile

Also check this one https://codesandbox.io/embed/adoring-dhawan-js373?file=/src/index.js&codemirror=1
Only reverse infinite scroll is not working. Normal infinite scroll is working

Here is a quick fix fix
If you want to install this:
npm i vj-abishek/react-infinite-scroll-component#mobile
The issue is, when we use flexDirection: 'column-reverse' . The scroll bar is set to the bottom. In desktop the scrollTop ends with 0 if it reaches the bottom. But it mobile scrollTop behaves oppositely  due to flexDirection: 'column-reverse. The scrollTop starts from 0 at top in mobile.
In desktop scrollTop starts with 0 at bottom and end with element height at top (scrollTop goes negative)
In mobile scrollTop starts with the element height at bottom  and end with 0 at top(scrollTop goes positive)

Mine works on mobile (emulated and android) but is breaking specifically on iOS devices... (both chrome and safari)
Unlike with your problem I am not using reverse.

So, I was facing a similar issue of data not being fetched on mobile, it worked for me by adding/increasing the threshold for mobile.

Mine works on mobile (emulated and android) but is breaking specifically on iOS devices... (both chrome and safari)
Unlike with your problem I am not using reverse.

Same issue here


DateTime and Timezone handling in Association Message

Like the datetime & timezone handling in ScheduleMessages, we need to figure out what to do with the time fields in Association messages.
Is it actually possible to figure this much out, or would it need to be left to the user to do in conjunction with the relevant Schedule messages? If the latter, we probably need to save the timezone offset of the Schedule Message as an attribute of it, to allow for the library user to look that up in order to know the correct timezone treatment of the Association message.

This can't actually be done at the library level as it requires looking up the corresponding schedule messages.
Probably the best way to do this is to implement a TimezoneAwareAssociationMessage class which takes the AssociationMessage and relevant ScheduleMessages and can then build an AssociationMessage with the proper timezones and date/times in the properties, and the library user is responsible for constructing these objects.


Implement medperf-compatible mlcube

Proposed Changes

Provide additional CLI argument names for parsing medperf data
Implement a medperf-specific mlcube.yaml file

Checklist

[x] I have read the CONTRIBUTING guide
[ ] My PR is based from the current GaNDLF master
[x] Non-breaking change (does not break existing functionality): provide as many details as possible for any breaking change
[ ] Function/class source code documentation added/updated
[ ] Code has been blacked for style consistency
[x] If applicable, version information has been updated in GANDLF/version.py
[ ] If adding a git submodule, add to list of exceptions for black styling in pyproject.toml file
[ ] Usage documentation has been updated, if appropriate
[ ] History has been updated, if appropriate
[ ] Tests added or modified to cover the changes; if coverage is reduced, please give explanation
[x] If customized dependency installation is required (i.e., a separate pip install step is needed for PR to be functional), please ensure it is reflected in all the files that control the CI, namely: python-test.yml, and all docker files [1,2,3,4]

Hey @aristizabal95, thanks for the changes. I can review them later tonight and get back to you. Does this meet the interface as-is or is anything else needed?


HDDS-5449 Recon namespace summary 'du' information should return replicated size of a key

What changes were proposed in this pull request?
In addition to actual key size, Recon 'du' should also return the replicated size of a key.
For example, for a 3 way replicated key of size = 1MB, Recon 'du' should return '1MB' as the size of the key, and '3MB' as the replicated size if the key is fully replicated.
[Refactoring]

DU response now shows a "structure"
DU key response shouldn't have subpaths

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-5449
How was this patch tested?
Since this is an improvement/refactoring feature on top of HDDS-5378, no additional unit test is implemented. I tested remotely on the cluster endpoint with the following configs:

All keys are three-way replicated except key3
[Root DU]
http://localhost:9888/api/v1/namespace/du?path=/&replica=true

[Volume DU]
http://localhost:9888/api/v1/namespace/du?path=/vol&replica=true

[Bucket DU] &files=true enabled
http://localhost:9888/api/v1/namespace/du?path=/vol/bucket&files=true&replica=true

[Directory DU] &files=true enabled
http://localhost:9888/api/v1/namespace/du?path=/vol/bucket/dir1&files=true&replica=true

(Key3 is one-way replicated.)
[Nested Directory DU]
http://localhost:9888/api/v1/namespace/du?path=/vol/bucket/dir1/dir2&files=true&replica=true

[Key DU]
http://localhost:9888/api/v1/namespace/du?path=/vol/bucket/dir1/dir2/dir3/key6&files=true&replica=true

cc @smengcl @avijayanhwx. Please review

[Fix]

Addressed comments.
Optimized listKeys by seek() and DFS. No full iteration on FileTable except root level
Added a unit test for the case that a key spanning across multiple blocks and some of containers are under/over-replicated

[Future works]

https://github.com/yuangu002/ozone/blob/HDDS-5449/hadoop-ozone/recon/src/main/java/org/apache/hadoop/ozone/recon/api/NSSummaryEndpoint.java#L228
https://github.com/yuangu002/ozone/blob/HDDS-5449/hadoop-ozone/recon/src/main/java/org/apache/hadoop/ozone/recon/api/NSSummaryEndpoint.java#L907
Change numOfFiles in NSSummary from int to long

Merged. Thanks @yuangu002 for the patch. Thanks @avijayanhwx for the review.


Tighten up the Archivematica permission for Elasticsearch

Currently the Archivematica user has superuser privileges in the cluster. We should fix that.

Closing; we'll be ditching Archivematica + Elasticsearch as part of https://github.com/wellcomecollection/platform/issues/4794


Support que 1.0.0

hirefire can't fetch the job queue for que 1.0.0beta3
Jun 30 12:32:37 prod heroku/router: http_error="Bad chunk" at=error code=H17 desc="Poorly formatted HTTP response" method=GET path="/hirefire/TOKEN/info" host=www.****.com request_id=069f7d28-*********** fwd="54.*.*.*" dyno=web.1 connect=0ms service=5ms status=503 bytes= protocol=https
Jun 30 12:32:37 prod app/postgres.22998: [OLIVE] [569-1]  sql_error_code = 42703 ERROR:  column "job_id" specified in USING clause does not exist in left table
Jun 30 12:32:37 prod app/postgres.22998: [OLIVE] [569-2]  sql_error_code = 42703 STATEMENT:
Jun 30 12:32:37 prod app/postgres.22998: [OLIVE] [569-3] 	SELECT count(*)                                                          AS total,
Jun 30 12:32:37 prod app/postgres.22998: [OLIVE] [569-4] 	       count(locks.job_id)                                               AS running,
Jun 30 12:32:37 prod app/postgres.22998: [OLIVE] [569-5] 	       coalesce(sum((error_count > 0 AND locks.job_id IS NULL)::int), 0) AS failing,
Jun 30 12:32:37 prod app/postgres.22998: [OLIVE] [569-6] 	       coalesce(sum((error_count = 0 AND locks.job_id IS NULL)::int), 0) AS scheduled
Jun 30 12:32:37 prod app/postgres.22998: [OLIVE] [569-7] 	FROM que_jobs LEFT JOIN (
Jun 30 12:32:37 prod app/postgres.22998: [OLIVE] [569-8] 	  SELECT (classid::bigint << 32) + objid::bigint AS job_id
Jun 30 12:32:37 prod app/postgres.22998: [OLIVE] [569-9] 	  FROM pg_locks WHERE locktype = 'advisory'
Jun 30 12:32:37 prod app/postgres.22998: [OLIVE] [569-10] 	) locks USING (job_id)
Jun 30 12:32:38 prod app/web.1: [hirefire:router] queue=0ms
Jun 30 12:32:38 prod app/web.1: 2018-06-30 10:32:37 +0000: Read error: #<PG::UndefinedColumn: ERROR:  column "job_id" specified in USING clause does not exist in left table
Jun 30 12:32:38 prod app/web.1: >

I'm not very familiar with Que, and the macro was contributed by @mossgross-cf, so I'm not entirely sure how to resolve this one. I've taken a look at the changelog and it says that the job_id column was renamed to simply id.
I've done a quick patch on a separate branch, doing the same for the query that's being executed by the Que macro. Perhaps you could give this branch a try and see if it works. Note that other internals might've changed that affect the query, so the above-mentioned patch might not work.
If it doesn't work and you're (more) familiar with the Que internals, feel free to fork and patch the Que macro to conform to the new Que table structure.

Note that the branch is que-1.0.0, so you'll want to add the following to your Gemfile:
gem "hirefire-resource",
  git: "https://github.com/hirefire/hirefire-resource",
  branch: "que-1.0.0"

We're using hirefire with Que (1.0.0.beta4) with gem version >= 0.7.5 and it works fine. This can be closed


Instructions for Private Packages for NPM Don't Work as Expected

Referencing the previous issue, which was closed. https://github.com/cloudflare/cloudflare-docs/pull/1334
Expected Behavior
By using a preinstall script you should be able to copy an .npmrc.pages over the existing .npmrc file in order to authenticate to private packages on GitHub.
Actual Behavior
npm does not reload the .npmrc settings after the preinstall script so copying the .npmrc.pages file to .npmrc has no effect.
Replicating
To avoid any issues with permissions issues with tokens and specific environments I've replicated this issue in a new environment with a validated token.
I've done three different tests do demonstrate the issue. Each test used the same TOKEN_FOR_GITHUB environment variable.
Test One ✅
Directly commit the .npmrc with the ${TOKEN_FOR_GITHUB} placeholder to the repository and deploy.
.npmrc:
@tailwind:registry=https://npm.pkg.github.com
//npm.pkg.github.com/:_authToken=${TOKEN_FOR_GITHUB}

This test successfully installed npm packages from a private repository using the token.
This validates that the token is set correctly and that the configuration of the .npmrc file is correct.
Unfortunately this is not a tenable option, since it breaks all local development unless the environment has a TOKEN_FOR_GITHUB set.
Test Two ❌
Commit an  .npmrc with only the registry information, and create a .npmrc.pages file that has the ${TOKEN_FOR_GITHUB}.
Deploy these using the instructions provided is-pages.js and prepublish script from the documentation, along with using cat to dump the contents of the file.
.npmrc:
@tailwind:registry=https://npm.pkg.github.com

.npmrc.pages:
@tailwind:registry=https://npm.pkg.github.com
//npm.pkg.github.com/:_authToken=${TOKEN_FOR_GITHUB}

is-pages.js:
if (!process.env.CF_PAGES) process.exit(1);

package.json:
  "preinstall": "node is-pages && mv .npmrc.pages .npmrc || echo \"Skipping Cloudflare Pages\" && cat .npmrc",

This failed to build because npm could not authenticate with Github. The contents of the .npmrc were correct and matched the contents of the checked in .npmrc.pages.
This shows that npm is not respecting changes to .npmrc after it has started the install process.
Test Three ❌
Do not create an .npmrc and create a .npmrc.pages file that has the ${TOKEN_FOR_GITHUB}.
Deploy these using the instructions provided is-pages.js and prepublish script from the documentation, along with using cat to dump the contents of the file.
.npmrc.pages:
@tailwind:registry=https://npm.pkg.github.com
//npm.pkg.github.com/:_authToken=${TOKEN_FOR_GITHUB}

is-pages.js:
if (!process.env.CF_PAGES) process.exit(1);

package.json:
  "preinstall": "node is-pages && mv .npmrc.pages .npmrc || echo \"Skipping Cloudflare Pages\" && cat .npmrc",

This failed to build because npm could not authenticate with Github. The contents of the .npmrc were correct and matched the contents of the checked in .npmrc.pages.
This validates that there is no difference between having an existing .npmrc in the repository and having none.
Section that requires update
The "Managing Multiple Environments" section of the documentation should be replaced with a disclaimer that this is not possible
https://developers.cloudflare.com/pages/how-to/npm-private-registry#managing-multiple-environments
This should be added to the "Known Issues" section.
https://developers.cloudflare.com/pages/platform/known-issues
Additional information
I am able to replicate this consistently and reliably, and I believe the extend of my testing has covered all possibilities.
I would be happy to provide access to a private github repository with private packages that is experiencing this issue if further evidence is needed.
Thanks.

Got a workaround .npmrc not being reloaded:

Create bash script (/bin/is-pages):

#!/usr/bin/env bash

[ -z ${CF_PAGES+x} ] && exit 0;

if [ -f ".npmrc.pages" ];
then
  mv .npmrc.pages .npmrc;
  exit 1;
fi

Change preinstall in package.json

"preinstall": "bash ./bin/gh-pack && echo \"Running normally.\" || yarn install"

If CF_PAGES env variable is set and .npmrc.pages exists bash file will move .npmrc.pages to .npmrc and will exit bash script with error.
When preinstall picks up that bash script exited with error - it will stop execution and gets instruction to run yarn install again - when it's being runned second time .npmrc.pages doesn't exists anymore so bash script exits without error and installing starts normally.

Putting the access token definition in the .npmrc is bad, since it's a tracked file it's inevitable that someone will commit it with their token. Would be better to place it there with a env variable. And let the user set it on their machine. Also the .env.local files shouldn't be tracked for the same reason.
//npm.pkg.github.com/:_authToken=$GITHUB_NPM_PKG_TOKEN

Same issue here. Thanks, @flohaider and @grinry for the workaround!

Another workaround is to set the environment variable NPM_CONFIG_USERCONFIG to /opt/buildhome/repo/.npmrc.pages - then no scripts are needed.
This solution also has the nice property that any config settings in .npmrc will override .npmrc.pages, so you don't need to have duplicated settings in both npmrc files.

@ghughes thank you, much better (and faster) :)

We are currently trying to solve this problem for our repo as well, but none of the mentioned solutions here seems to work for us.
We use Yarn Berry (3.2.0) and are trying to install scoped packages from Github (npm.pkg.github.com).
In our .yarnrc.yml we have the following
npmScopes:
  ourcompany:
    npmPublishRegistry: "https://npm.pkg.github.com"
    npmRegistryServer: "https://npm.pkg.github.com"

I tried it with different env vars, with the .npmrc.pages copying, with preinstall scripts, but it seems yarn berry is either ignoring all of these.
Did someone else had issues with yarn berry and found a way around them?
Thanks in advance

@ghughes thanks for the solution, this works in my case. Can I suggest this get added to the docs so people like me don't have to crawl through google results to find the solution? https://developers.cloudflare.com/pages/how-to/npm-private-registry/#custom-registry-endpoints

@danawoodman it's already documented on that page (as of #5894)

Lol, not sure how I missed that 🤦

Now I'm getting this on a project without a custom registry, just a private npm package 🤣
npm ERR! 404 Not Found - GET<EMAIL_ADDRESS>- Not found

(package def exists, using it in multiple other projects)
But NPM_TOKEN is set:

No .npmrc file.

Funny when you hit the same issue 5 months later and see that you were the last one to comment on the top google result...


Update archive.md fix suffixed space character typo

@good92 Thank you for your contribution!  Oracle does not allow us to accept contributions without a signed Oracle Contributor Agreement.  We also have some internal issues right now that prevents us from directly merging PRs in GitHub right now.  We are working through those issues and hope to be able to revert to the normal PR approach in the near future.  We apologize for any inconvenience this may cause.
Since this is such a small change, I will fix the issue myself and get it merged.

great thanks

Fix made and documentation updated.


upcoming: [DI-18311] - Add CloudPulse feature flag and landing page to Cloud Manager

Description 📝
Added cloudpulse section to cloud manager
Added feature flag control for cloudpulse section
Added an empty paper component in the cloudpulse section
Changes  🔄
List any change relevant to the reviewer.
-Added cloudpulse section cloud manager
-Added feature flag control for cloudpulse section
-Added empty paper component in the cloudpulse section
Preview 📷
Include a screenshot or screen recording of the change
:bulb: Use <video src="" /> tag when including recordings in table.

Before
After

How to test 🧪
Prerequisites
(How to setup test environment)

Follow cloud manager standard procedures

Reproduction steps
(How to reproduce the issue, if applicable)

N/A

Verification steps
(How to verify changes)

Navigate to cloud manager and once feature flag of CloudPulse is enabled then navigate to the Monitor section in the sidebar for the CloudPulse section.

As an Author I have considered 🤔
Check all that apply

[x] 👀 Doing a self review
[x] ❔ Our contribution guidelines
[x] 🤏 Splitting feature into small PRs
[x] ➕ Adding a changeset
[ ] 🧪 Providing/Improving test coverage
[ ] 🔐 Removing all sensitive information from the code and PR description
[x] 🚩 Using a feature flag to protect the release
[ ] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

If you're just stubbing this in, it's fine. If not, there are some issues, such as the left nav saying "Monitor" and the page title saying "Akamai Cloud Pulse".

The nav saying "Monitor" and page saying "Akamai Cloud Pulse" is intentional as per our Product Manager and UI mockups.

@santoshp210 Please merge in the latest from develop (you'll have to resolve the conflicts in MainContent, PrimaryNav, and featureFlags) and then CI should run and we can confirm this is good to go.

Hi @mjac0bs , resolved the commits for those files.


Debugging in STM32CubeIDE

Hello
I am a newbie to ROS and currently studying it. I followed tutorials on github that you guys published.
I want to debug it using STM32CubeIDE to the Olimex STM32F407 board.
I've imported Posix, Micro-xrce-dds, rcl ... libraries to my workspace. But when I try to build the project, it gives the errors related with dependancy and configurations.
If it is possible, could you please provide a tutorial or any documentation about debugging micro-Ros app from IDE(freeRTOS).
Kind regards

Hello @Altaibaatar, can you provide some traces of your error? Currently, we are not integrated with STM32CubeIDE. Have you tried to follow the tutorial as they are? Did you achieved ROS 2 communication?

Thank you for your reply @pablogs9
I couldn't achieve the ROS2 communication. Recently I am failing to compile the project. Most of the errors are related to dependancies and linker errors. For example:
Not found file or directory: rmw_microxrcedds_c/config.h
(Btw, I really didn't find the config.h in your repo hehe)
To build my project, I am importing and adding C source files in accordance with diagrams in the link below. As a base of my project, #https://github.com/micro-ROS/freertos_apps/tree/foxy/microros_olimex_e407_extensions is being used.
micro-ros-on-freertos
About the tutorials, I followed the core tutorials that you guys are providing(Building and flashing micro-ROS to uController on Linux machines).
Kind regards

Please follow the instructions in this tutorial: https://micro-ros.github.io/docs/tutorials/core/first_application_rtos/freertos/
And if there is any error copy-paste here the whole terminal output,

Hello please check https://github.com/micro-ROS/micro_ros_stm32cubemx_utils#using-this-package-with-stm32cubeide


Contribution page: taxes not displayed on refunds

Actually, it seems to be displayed. Will check with reporter to understand what happened


You call this a framework?

📝 Description
Is your feature request related to a problem? Please describe.
I'm always frustrated when a developer has an inflated ego.
Describe the solution you'd like
Please rename this to MAYA NODEJS BOILERPLATE
Describe alternatives you've considered
Delete the whole repo
⚙ Component
All of them
📋 Todo:

[ ] Rename to  MAYA NODEJS BOILERPLATE
[ ] Delete the whole repo

🔍 Additional Info
Be humble.

Trolls are not allowed to contribute to an open-source project. Please read the GitHub policy first.


Add reference translation for Authorization template in Policies and telemetry.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: fleeto
If they are not already assigned, you can assign the PR to them by writing /assign @fleeto in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content_zh/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@fanfanbj: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Hi @fanfanbj. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Headless HTML to PDF

Description
Headles PDF generation is not working
Who's implementing?

[ ] I'm willing to implement this feature myself

The problem
Using headless/CLI chromium/chrome used to be able to save a website into PDF or image by providing a url argument but it is no longer working. I have not tried it for a long time so I don't know when or why it stopped working. Is there a workaround? I tried ungoogled chromium, chrome and chrome canary(all should be the same core) but none are working.
Possible solutions
Missing documentation?
Alternatives
No response
Additional context
No response

Running:
ungoogled-chromium --headless --disable-gpu --window-size=1920,5400 --screenshot https://somesite.com
saves it as a png image. To output to pdf:
ungoogled-chromium --headless --disable-gpu --print-to-pdf https://somesite.com

unfortunately it does not work. it seems to be a windows issue since people on linux say it works. ff is the same along will all flavours of chrome.

Using ungoogled-chromium v97.0.4692.71 (win 32-bit) this works:
chrome.exe --headless --dissable-gpu --window-size=1920x5400 --screenshot="C:\\Users\User\\Pictures\\shot.png" "https://somesite.com"
Without a path it just saves it to the chromium directory (...\Appdata\Local\Chromium\Application\97.0.4692.71\shot.png) (also doesn't want to save it as c:\shot.png).

Thank worked, thanks!

One more thing - is there a way to get the cli configuration? chrome help, chrome -help nor chrome --help does not work. Also, is it possible to pipe the output into a socket or network connection instead of a file? Possibly getting some progress update as well in case of a big file or an error?


Approval

Follow up to https://github.com/inventree/InvenTree/pull/5095
Simplified implementation of approvals for orders. Includes building blocks for a more generalized rules and template-driven approach.
Function
This PR adds approvals (optionally) for POs. You can enable them in settings (including minimum approvers and maximum rejections). After that POs can only be placed once they are approved.
Notifications
Notifications will be sent out to the responsible user/group:

when approval starts
on each approval decision
after being rejected (PO will be cancelled)
after being approved (PO will be moved to placed)

UI
As an experiment, I just implemented this in P UI. While the order views are not ready in P UI, approvals are.
Next steps
This is an MVP, I opened an epic for further improvements #5097.
While this PR adds a business logic and UI for PO approvals, approvals are a first-class citizen implementation-wise. They can be used as-is for any model and adding UI elements for that should be simple. The default rules engine is also simple right now but designed for complex requirements. If there is interest this can be extended.
## Still to address
- [ ] Add a general approval flow (creation/viewing/approval/declining)
- [x] Make orders a bit more helpful
- [x] Jump-of-point to the general approval thing in C UI
- [x] Migrate PO stuff into its own PR
- [x] Add auto-generation of approval names
- [x] Add UI for choosing name for approval
- [ ] Add docs
- [ ] Add info-bubble with link to epic for feedback

Closes #3501

@matmair I was under the impression that we were pretty good to go on this one - just waiting on #5597 which was only just merged.
Are you happy for me to re-open and have a proper review? It has only been through lack of time that I have not done this yet

Hi,
Just pitching in here to bring to notice, I am very much interested in this feature coming to picture.
In case additional support is needed, I can pitch in to build the features as well.

Thanks for the interest @imsoumyaprakash. I have already lost interest in pursuing this in the public and started work polishing and moving the code into a closed plugin. You are more than welcome to use the ideas in this PR, but I won't focus on bringing this into core anymore.


Can i host this to my own site?

i wonder if i can host this to my own site.

@0x00chad yes you can
theres a folder backend
upload it to your server, edit config same as firebase config
and change application id in android gradle

Thank you. ^^ is there a way where you can edit configuration in the android app interface? that you dont need to recompile the application. like theres a textbox in application where you can input your own webserver. thanks for this wonderful app. :)

@0x00chad Android only need to add firebase json configuration and change the aplication_id in gradle.
no need server configuration, because it just listen to push notification send by your server.
server send push notification, android will listen and capture push, then change it to sms
as long the firebase project match, android and backend code will connect

can you help me with this i create this php but i want to send this to many number like looping im not good in coding php like it will send to +9234893849,+9349839434 or next line
+9234893849
+9349839445

Phone Number

Message

here is the link https://pastebin.com/raw/UHDA4iub

this will work

<meta name="viewport" content="width=device-width, initial-scale=1">

<form method="POST">

Phone Number
<input style="width:100%" type="text" name="number" placeholder="number1,number2,number3"/>
Message
<input cols="50" type="text" name="message"/>
<input type="submit"  name="send" value="Send Message"/>

</form>

<?php

//Set to your time zone in phone
date_default_timezone_set('Asia/Taipei');

$deviceID = "DEVICE ID";
$secret = "SECRET";
//you can hash to md5 to protect your secret, or you just send the secret

if(isset($_POST['number']) && isset($_POST['message'])){
    $numbers = explode(",",$_POST['number']);
    foreach($numbers as $number){
        echo sendMessage($number, $_POST['message']);
        echo "\n\n";
    }
}

function sendMessage($number, $message){
    global $secret,$deviceID;
    $time = time();
    $secretMd5 = md5($secret.$time);
    return file_get_contents("https://sms.ibnux.net/?to=".urlencode(trim($number))."&text=".urlencode($message)."&secret=$secretMd5&time=$time&deviceID=".urlencode($deviceID));
}
?>

Wow thank you very much. your a life saver. :) is there a chance you will add retrieving message from the phone?

I will, if i need to create it, for now I don't need that
Is hard, because to read sms, android apps need to be SMS client, maybe it can if apps read notification.
But is hard to get phone number except don't save the phone number to phonebook, it will show phone number in notification

@0x00chad now the app can read incoming sms and send it to server

@ibnux thank you.. is there a way to edit config without compiling the whole apk? like you can edit the config in settings in the app to put your own firebase config..

@0x00chad no it can't
Because it need to change Application ID too


None of the genes were predicted when running celldancer.velocity_estimation.velocity

Dear cellDancer team,
When I run the celldancer.velocity_estimation.velocity function, I keep getting an empty celldancer_df. When the function finishes running this is the printed message "None of the genes were predicted. Try visualizing the unspliced and spliced columns of the gene(s) to check the quality". I have tried both sub-selecting genes or providing the top 2000 hvgs and played a bit with the parameters (e.g. setting the norm_u_s=False and norm_cell_distribution=False) without any lack. The spliced and unspliced columns look ok and I can run normal velocity analysis. Could you please advice how maybe I can relax the parameters to see if any of the genes will make it through? Any advice is welcome!
Best,
Anastasia

As an add-on to this issue, I have downloaded the PancreaticEndocrinogenesis_cell_type_u_s.csv tutorial file and run the following commands:
cell_type_u_s_path='PancreaticEndocrinogenesis_cell_type_u_s.csv'
cell_type_u_s=pd.read_csv(cell_type_u_s_path)
cell_type_u_s
cellDancer_df = cd.velocity(cell_type_u_s, permutation_ratio=0.5, n_jobs=7)
And I am still getting the same error.
I am attaching here the details of my conda environment for assistance.
env.txt
Thanks again,
Anastasia

Dear Anastasia,
Thank you for reaching out to us. I noticed that you are using Python v3.8.13. Is it possible for you to create a conda environment following the versions on our installation page? For the conda environment version on the installation page, we've verified the usage multiple times in different devices for all our case study data. If this problem still exist, could you maybe send us sample data with a subset of your gene following the code block below to let us test and replicate the problem (sli5@houstonmethodist.org)? Thank you!
# select genes by specifying genes
gene_list=['gene1','gene2','gene_3']
# or just select first n genes by
n=10 # could be customized
gene_list=list(cell_type_u_s.gene_name.drop_duplicates())[0:n]

# keep the subset of data
cell_type_u_s _subset=cell_type_u_s[cell_type_u_s.gene_name.isin(gene_list)].reset_index(drop=True)
# save the csv file to current directory
cell_type_u_s _subset.to_csv('cell_type_u_s _subset.csv', index=False)

Best,
Shengyu

Dear Shengyu,
thanks so much for your response. I have managed to narrow down the error to a special character in my full_path of where the pipeline was running and trying to store the results of the cd.velocity function.
Once I included within the function a relevant path to the current directory, that completely resolved the issue.
Pasting this here as an example.
loss_df, cellDancer_df=cd.velocity(cell_type_u_s,\
                                   gene_list,
                                   permutation_ratio=0.125,\
                                   norm_cell_distribution=False,
                                   n_jobs=8,
                                   save_path='./')

Best,
Anastasia


Add airtap + Sauce Labs

I'm closing this for now, running airtap --local is good enough for now. I'm thinking level will take over level-browersify later on anyway and should be the one tested on Sauce Labs.
@vweevers WDYT?

SGTM


Define consistent folder layout

ToDo: define a recommended file layout, and describe it in the README / apply across this and other repos.

There is now a path approach implemented in convnwb
ToDo: update here with respect to what has already been done in ConvertTH and ConvertTH