Question: CSS class names (.mantine-######) for custom CSS

First of all: thanks for all the work, the new version is really nice and fast!
I've been playing with custom CSS and JS and I saw those hash-looking CSS class names and, although they seem consistent between sessions, I'm afraid they will change in future bulds.
For most of the cases I'm able to avoid using them directly but there are a few cases in which I haven't found an alternative. One example are the CSS classes for read and unread entry items. The classes are:

.mantine-1cyhnjt, .mantine-1rebq3j (unread)
.mantine-13mxilh, .mantine-1byd8cf(read)

In case these class names are reliable and will be consistent in future versions, then there is no problem and you can ignore and close the issue. Otherwise, do you know if there is a good way to handle this?
Thanks again!

You should not rely on those, as you suspected they are generated and may change from version to version.
I can however add classes manually to elements to help with custom CSS rules. Do you need something else besides a "read" or "unread" class on entries?

I did a deep review and there are 4 CSS classes for entries that would solve all my issues:

read / unread
open
active (or focused)

I think the place would be line 98 of FeedEntry.tsx:
<Paper withBorder radius={borderRadius} className={classes.paper}>
If I were familiar with react & mantine I would submit a PR with the changes, but it's probably better for me to not touch the code.
More thanks!

I added the following classes in 412fc52f1cc442ca6047c9ae0982a8fe9da24811

read / unread
expanded if the entry is open
selected if the entry is the current selection
show-selection-indicator if the indicator is shown (left border is orange)

Does that work for you?

That's more than perfect!
Thank you very much!

The changed is now deployed on commafeed.com :)


uBlock Origin is working now

uBlock Origin is now working well again on YouTube without being detected. You might want to try it considering the recent issues we've encountered. Given YouTube's frequent updates against ad blockers, it might be better to keep both Remove Adblock Thing and uBlock Origin handy so we can switch if one gets blocked.
Thanks to all the devs for your hard work. Hoping to see the recent issues resolved soon!

Firefox user.
After the update, sometimes overlap video sound it like you play 2 video in the same time
ublock on if ublock off ads and video  still overlap

Yep guys uBlock is working still

Hw is itworking now? that's good to know

Dont know what you're talking about, just updated all filters and its still broken.

Dont know what you're talking about, just updated all filters and its still broken.

Dont know what you mean broken.

Dont know what you're talking about, just updated all filters and its still broken.

Dont know what you mean broken.

As in still not working correctly. The only thing that works is dropping down to 5.3.

ublock origin workst a treat


bpf: xdp: fix coccicheck warning about DROP_MISSED_TAIL_CALL

The coccicheck workflow strictly wants us to throw the drop notification right away. It's not enough to fall through to the drop notification below.

/test


undefined:1

为什么会undefined啊（

...??
What happened to it...??

QAQ
I can't use it on my laptop,it says "undefined" even when just inputing "xepub"!

Is there any screenshots you could provide?
P.S. I'm gonna make a huge update on it... Use material designed...

Try to install the alpha branch with
npm install -g xepub@alpha

Tell me if problem still exists.

???

Well, I have no idea with that error...
Could you please provide your EPUB file here?
Or it throws error every book?

Every book has this problem... :(

I realized what had happened on Windows...
path.join() used \ as the separator on Windows...
And then it crashed...
I'll fix it and publish new version soon.

Fixed, if this problem STILL EXISTS, please give a screenshot using xepub with --debug flag.

How's it going? (；･`д･´)

All right, I regard it has been fixed...


feat: add option to override base url for GHES

Adds an option for --octoherd-base-url to configure underlying Octokit client for GHES. The URL passed to --octoherd-base-url will be passed as baseUrl to the underlying Octokit client.
There isn't a current testing pattern in place for the existing options. Open to feedback/ideas on how to add tests for options if desired.

we've verified locally that the change works as intended. happy to discuss ideas for further automated testing as a follow up


Proposal for Halyard to have a installer script

About the tool

Halyard is the CLI administration tool that manages the lifecycle of  Spinnaker deployments.
Refer https://www.spinnaker.io/setup/install/halyard/
How the "cURL & bash" command be

curl https://installer.to/halyard | bash

Checklist

[x] I checked the Issues list and I'm sure I'm not duplicating an existing request
[x] I checked the Pull Request list and I'm sure I'm not proposing for a tool that is about to get added

I request the community to consider my proposal and cast your votes by commenting,

+1 if you like to see an installer script for this tool in this repo
-1 if you do not like to see an installer script for this tool in this repo

+1

Can I work on this sir?

Yes, go ahead


Latest versions of mailhog and mongodb incompatible (docker-compose)

Replicate using this docker-compose.yml:
version: "3.9"
services:
  mailhog:
    image: "mailhog/mailhog:latest"
    ports:
    - "50001:8025"
    - "50002:1025"
    environment:
      MH_STORAGE: mongodb
      MH_MONGO_URI: "mongo:27017"
    depends_on:
    - mongo

  mongo:
    image: "mongo:latest"

I went back to Mongo 4.20 to make it work.
If you need a more recent Mongo, just add a second mail_mongo service with version 4.20 or so

See https://github.com/mailhog/storage/pull/12
You can compile a working version with: https://github.com/vmary2014/MailHog


Digital Scrapbook with Animated Stickers

Digital Scrapbook with Animated Stickers
issue no. : #1861
https://github.com/user-attachments/assets/fdbbd168-0463-4ed5-a0a9-0c893ad55182

@shriyadindi there is a conflict

@iamrahulmahato resolved


Fix PM sorting.

Automated message from Dropbox CLA bot
@showell, it looks like you've already signed the Dropbox CLA. Thanks!

@timabbott It looks like you merged these, so closing.

I did.  Man, have not been doing well at closing all the PRs today :/.


The editor under the context view 'Active and soft deleted blobs' always keeps loading when soft deleting one ADLS Gen2 item

Storage Explorer Version: 1.28.0
Build Number: 20230131.13
Branch: rel/1.28.0
Platform/OS: Windows 10/Linux Ubuntu 22.04/MacOS Ventura 13.2 (Apple M1 Pro)
Architecture: ia32/x64/arm64
How Found: From running test case
Regression From: Previous release (1.27.2)
Steps to Reproduce

Expand one ADLS Gen2 storage account -> Blob Containers.
Create a blob container -> Create a folder.
If the soft delete is enabled -> Disable the soft delete.
Then enable the soft delete.
Soft delete the blob -> Switch to the context view 'Active and soft delete blobs'.
Check the editor shows well.

Expected Experience
The editor shows well.
Actual Experience
The editor alway keeps loading on the current context view.

Verified this issue on the build rel/1.28.0 20230202.3. Fixed. So, we are going to close it.


Add link to native mobile maps apps

Could be done similar to the Foursquare App on iOS. One link for external routing in the card and a popup that asks which service you want to use.


Persian

Language
Persian aka Farsi
Can you help with the translation?
Yes.

Thanks for your interest, can you follow up #98?


SPIKE: Survey bundled packages

AC:

Survey libraries to see if any can be deleted from previous unbundling (e.g. libpng) and write follow-up tickets
Create a table on GitHub wiki to track remaining bundled libraries and indicate LOE to unbundle

what is LOE?

@tst-awankhede: Level Of Effort.


About the instable performance on voxelrcnn-focalsconv-lidar.

I test cfgs/kitti_models/voxel_rcnn_multiclasses_focalsconv_lidar.yaml  and get follows:
2023-06-06 18:18:16,228   INFO  *************** EPOCH 80 EVALUATION *****************
2023-06-06 18:19:39,205   INFO  *************** Performance of EPOCH 80 *****************
2023-06-06 18:19:39,205   INFO  Generate label finished(sec_per_example: 0.0220 second).
2023-06-06 18:19:39,205   INFO  recall_roi_0.3: 0.961773
2023-06-06 18:19:39,205   INFO  recall_rcnn_0.3: 0.962228
2023-06-06 18:19:39,206   INFO  recall_roi_0.5: 0.894875
2023-06-06 18:19:39,206   INFO  recall_rcnn_0.5: 0.903806
2023-06-06 18:19:39,206   INFO  recall_roi_0.7: 0.643893
2023-06-06 18:19:39,206   INFO  recall_rcnn_0.7: 0.682803
2023-06-06 18:19:39,208   INFO  Average predicted number of objects(3769 samples): 9.748
2023-06-06 18:19:58,413   INFO  Car<EMAIL_ADDRESS>0.70, 0.70:
bbox AP:90.5178, 88.2612, 87.4234
bev  AP:89.9737, 87.4929, 87.0976
3d   AP:87.4831, 76.6078, 74.1041
aos  AP:90.46, 88.09, 87.19
Car<EMAIL_ADDRESS>0.70, 0.70:
bbox AP:96.2130, 90.8027, 88.7787
bev  AP:95.3868, 90.0077, 88.2213
3d   AP:89.0930, 76.6307, 74.3588
aos  AP:96.14, 90.61, 88.54
Car<EMAIL_ADDRESS>0.50, 0.50:
bbox AP:90.5178, 88.2612, 87.4234
bev  AP:97.7484, 89.1943, 88.9649
3d   AP:97.5897, 89.0465, 88.7367
aos  AP:90.46, 88.09, 87.19
Car<EMAIL_ADDRESS>0.50, 0.50:
bbox AP:96.2130, 90.8027, 88.7787
bev  AP:98.7406, 94.1972, 94.0597
3d   AP:98.6287, 93.7526, 91.7435
aos  AP:96.14, 90.61, 88.54
Pedestrian<EMAIL_ADDRESS>0.50, 0.50:
bbox AP:70.2669, 65.7228, 63.5041
bev  AP:63.5311, 57.8132, 53.9084
3d   AP:57.2681, 50.8920, 46.5279
aos  AP:65.89, 60.98, 58.70
Pedestrian<EMAIL_ADDRESS>0.50, 0.50:
bbox AP:70.7022, 66.2482, 63.3979
bev  AP:62.9012, 56.7882, 52.7481
3d   AP:56.0274, 49.2045, 44.7342
aos  AP:66.55, 61.67, 58.77
Pedestrian<EMAIL_ADDRESS>0.25, 0.25:
bbox AP:70.2669, 65.7228, 63.5041
bev  AP:76.8723, 73.1268, 70.4608
3d   AP:76.6298, 72.6185, 69.9068
aos  AP:65.89, 60.98, 58.70
Pedestrian<EMAIL_ADDRESS>0.25, 0.25:
bbox AP:70.7022, 66.2482, 63.3979
bev  AP:78.7871, 74.4536, 71.2797
3d   AP:78.4024, 73.9311, 70.2335
aos  AP:66.55, 61.67, 58.77
Cyclist<EMAIL_ADDRESS>0.50, 0.50:
bbox AP:89.8355, 82.4624, 76.8221
bev  AP:92.7843, 77.7540, 72.8453
3d   AP:85.6997, 70.2179, 65.0887
aos  AP:89.56, 81.88, 76.31
Cyclist<EMAIL_ADDRESS>0.50, 0.50:
bbox AP:94.9215, 84.0335, 79.5658
bev  AP:95.0312, 78.6199, 73.0521
3d   AP:88.5547, 70.5502, 66.0223
aos  AP:94.54, 83.40, 78.93
Cyclist<EMAIL_ADDRESS>0.25, 0.25:
bbox AP:89.8355, 82.4624, 76.8221
bev  AP:93.5421, 80.2735, 76.8016
3d   AP:93.5421, 80.2735, 76.8016
aos  AP:89.56, 81.88, 76.31
Cyclist<EMAIL_ADDRESS>0.25, 0.25:
bbox AP:94.9215, 84.0335, 79.5658
bev  AP:96.0182, 82.0864, 77.6258
3d   AP:96.0182, 82.0864, 77.6258
aos  AP:94.54, 83.40, 78.93

I modified it to a single class version, and the performance still is not stable. Like this:
2023-06-06 13:57:47,331   INFO  *************** Performance of EPOCH 80 *****************
2023-06-06 13:57:47,332   INFO  Generate label finished(sec_per_example: 0.0151 second).
2023-06-06 13:57:47,332   INFO  recall_roi_0.3: 0.949410
2023-06-06 13:57:47,332   INFO  recall_rcnn_0.3: 0.949207
2023-06-06 13:57:47,332   INFO  recall_roi_0.5: 0.935033
2023-06-06 13:57:47,333   INFO  recall_rcnn_0.5: 0.936661
2023-06-06 13:57:47,333   INFO  recall_roi_0.7: 0.761427
2023-06-06 13:57:47,333   INFO  recall_rcnn_0.7: 0.805981
2023-06-06 13:57:47,336   INFO  Average predicted number of objects(3769 samples): 5.945
2023-06-06 13:57:54,136   INFO  Car<EMAIL_ADDRESS>0.70, 0.70:
bbox AP:96.8045, 89.0459, 88.6606
bev  AP:89.9699, 87.5767, 87.2567
3d   AP:88.4307, 77.7115, 77.0925
aos  AP:96.73, 88.91, 88.44
Car<EMAIL_ADDRESS>0.70, 0.70:
bbox AP:98.3765, 93.6633, 91.6879
bev  AP:93.1352, 90.3745, 88.4254
3d   AP:89.4289, 79.5083, 77.1856
aos  AP:98.31, 93.48, 91.44
Car<EMAIL_ADDRESS>0.50, 0.50:
bbox AP:96.8045, 89.0459, 88.6606
bev  AP:97.1384, 94.8830, 89.2238
3d   AP:97.0505, 89.4147, 89.1669
aos  AP:96.73, 88.91, 88.44
Car<EMAIL_ADDRESS>0.50, 0.50:
bbox AP:98.3765, 93.6633, 91.6879
bev  AP:98.6951, 96.2895, 94.5248
3d   AP:98.6632, 94.5457, 94.3302
aos  AP:98.31, 93.48, 91.44

Here is my single class version yaml file:
CLASS_NAMES: ['Car']

DATA_CONFIG:
    _BASE_CONFIG_: cfgs/dataset_configs/kitti_dataset.yaml
    DATA_AUGMENTOR:
        DISABLE_AUG_LIST: ['placeholder']
        AUG_CONFIG_LIST:
            - NAME: gt_sampling

              USE_ROAD_PLANE: True
              DB_INFO_PATH:
                  - kitti_dbinfos_train.pkl
              PREPARE: {
              filter_by_min_points: ['Car:5'],
                 filter_by_difficulty: [-1],
              }

              SAMPLE_GROUPS: ['Car:20']
              NUM_POINT_FEATURES: 4
              DATABASE_WITH_FAKELIDAR: False
              REMOVE_EXTRA_WIDTH: [0.0, 0.0, 0.0]
              LIMIT_WHOLE_SCENE: False

            - NAME: random_world_flip
              ALONG_AXIS_LIST: ['x']

            - NAME: random_world_rotation
              WORLD_ROT_ANGLE: [-0.78539816, 0.78539816]

            - NAME: random_world_scaling
              WORLD_SCALE_RANGE: [0.95, 1.05]

MODEL:
    NAME: VoxelRCNN

    VFE:
        NAME: MeanVFE

    BACKBONE_3D:
        NAME: VoxelBackBone8xFocal
        USE_IMG: False
        ENLARGE_VOXEL_CHANNELS: 64
        KERNEL_SIZE: 5
        USE_STAGES: [1,]

    MAP_TO_BEV:
        NAME: HeightCompression
        NUM_BEV_FEATURES: 256

    BACKBONE_2D:
        NAME: BaseBEVBackbone

        LAYER_NUMS: [5, 5]
        LAYER_STRIDES: [1, 2]
        NUM_FILTERS: [64, 128]
        UPSAMPLE_STRIDES: [1, 2]
        NUM_UPSAMPLE_FILTERS: [128, 128]

    DENSE_HEAD:
        NAME: AnchorHeadSingle
        CLASS_AGNOSTIC: False

        USE_DIRECTION_CLASSIFIER: True
        DIR_OFFSET: 0.78539
        DIR_LIMIT_OFFSET: 0.0
        NUM_DIR_BINS: 2

        ANCHOR_GENERATOR_CONFIG: [
            {
                'class_name': 'Car',
                'anchor_sizes': [[3.9, 1.6, 1.56]],
                'anchor_rotations': [0, 1.57],
                'anchor_bottom_heights': [-1.78],
                'align_center': False,
                'feature_map_stride': 8,
                'matched_threshold': 0.6,
                'unmatched_threshold': 0.45
            },
        ]

        TARGET_ASSIGNER_CONFIG:
            NAME: AxisAlignedTargetAssigner
            POS_FRACTION: -1.0
            SAMPLE_SIZE: 512
            NORM_BY_NUM_EXAMPLES: False
            MATCH_HEIGHT: False
            BOX_CODER: ResidualCoder

        LOSS_CONFIG:
            LOSS_WEIGHTS: {
                'cls_weight': 1.0,
                'loc_weight': 2.0,
                'dir_weight': 0.2,
                'code_weights': [1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0]
            }

    ROI_HEAD:
        NAME: VoxelRCNNHead
        CLASS_AGNOSTIC: True

        SHARED_FC: [256, 256]
        CLS_FC: [256, 256]
        REG_FC: [256, 256]
        DP_RATIO: 0.3

        NMS_CONFIG:
            TRAIN:
                NMS_TYPE: nms_gpu
                MULTI_CLASSES_NMS: False
                NMS_PRE_MAXSIZE: 9000
                NMS_POST_MAXSIZE: 512
                NMS_THRESH: 0.8
            TEST:
                NMS_TYPE: nms_gpu
                MULTI_CLASSES_NMS: False
                USE_FAST_NMS: False
                SCORE_THRESH: 0.0
                NMS_PRE_MAXSIZE: 2048
                NMS_POST_MAXSIZE: 100
                NMS_THRESH: 0.7

        ROI_GRID_POOL:
            FEATURES_SOURCE: ['x_conv2', 'x_conv3', 'x_conv4']
            PRE_MLP: True
            GRID_SIZE: 6
            POOL_LAYERS:
                x_conv2:
                    MLPS: [[32, 32]]
                    QUERY_RANGES: [[4, 4, 4]]
                    POOL_RADIUS: [0.4]
                    NSAMPLE: [16]
                    POOL_METHOD: max_pool
                x_conv3:
                    MLPS: [[32, 32]]
                    QUERY_RANGES: [[4, 4, 4]]
                    POOL_RADIUS: [0.8]
                    NSAMPLE: [16]
                    POOL_METHOD: max_pool
                x_conv4:
                    MLPS: [[32, 32]]
                    QUERY_RANGES: [[4, 4, 4]]
                    POOL_RADIUS: [1.6]
                    NSAMPLE: [16]
                    POOL_METHOD: max_pool
        TARGET_CONFIG:
            BOX_CODER: ResidualCoder
            ROI_PER_IMAGE: 128
            FG_RATIO: 0.5

            SAMPLE_ROI_BY_EACH_CLASS: True
            CLS_SCORE_TYPE: roi_iou

            CLS_FG_THRESH: 0.75
            CLS_BG_THRESH: 0.25
            CLS_BG_THRESH_LO: 0.1
            HARD_BG_RATIO: 0.8

            REG_FG_THRESH: 0.55

        LOSS_CONFIG:
            CLS_LOSS: BinaryCrossEntropy
            REG_LOSS: smooth-l1
            CORNER_LOSS_REGULARIZATION: True
            GRID_3D_IOU_LOSS: False
            LOSS_WEIGHTS: {
                'rcnn_cls_weight': 1.0,
                'rcnn_reg_weight': 1.0,
                'rcnn_corner_weight': 1.0,
                'rcnn_iou3d_weight': 1.0,
                'code_weights': [1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0]
            }

    POST_PROCESSING:
        RECALL_THRESH_LIST: [0.3, 0.5, 0.7]
        SCORE_THRESH: 0.3
        OUTPUT_RAW_SCORE: False

        EVAL_METRIC: kitti

        NMS_CONFIG:
            MULTI_CLASSES_NMS: False
            NMS_TYPE: nms_gpu
            NMS_THRESH: 0.1
            NMS_PRE_MAXSIZE: 4096
            NMS_POST_MAXSIZE: 500

OPTIMIZATION:
    BATCH_SIZE_PER_GPU: 8
    NUM_EPOCHS: 80

    OPTIMIZER: adam_onecycle
    LR: 0.01
    WEIGHT_DECAY: 0.01
    MOMENTUM: 0.9

    MOMS: [0.95, 0.85]
    PCT_START: 0.4
    DIV_FACTOR: 10
    DECAY_STEP_LIST: [35, 45]
    LR_DECAY: 0.1
    LR_CLIP: 0.0000001

    LR_WARMUP: False
    WARMUP_EPOCH: 1

    GRAD_NORM_CLIP: 10

Performance on single-modal pvrcnn is stable.
Is this normal?

Can you share the following yaml files that detect the three categories

@lmn1516 I use the owner-released one.
https://github.com/dvlab-research/FocalsConv/blob/master/OpenPCDet/tools/cfgs/kitti_models/voxel_rcnn_multiclasses_focalsconv_lidar.yaml

Hi,
because voxel r-cnn is originally designed to be single class. I modify it to be this config file just as a trial. I think the single-class version might be more stable.
Regards,
Yukang Chen

Voxel r-cnn在单模态情况下的结果是多少，我尝试复现，得到的3D ap≈80%

@Karkers 在KITTI上复现Voxel R-CNN(不论是Voxel R-CNN还是Focals Conv-Voxel R-CNN)，很容易得到一个和论文里差不多的结果，具体的日志文件我找不到了，不好意思。


Duplicate heading text produces duplicate id attributes in HTML output

If you have multiple headings with the same text for whatever reason, the corresponding heading tags will also have the same id attribute. They can also conflict with other elements on the page.
Input:
# Example
## Example

Output:

Example

Example

I'd suggest both prefixing the IDs with something to sort of namespace them (e.g. mde-h1) and possibly suffixing with an incrementing integer (either for any duplicate headings, or globally for all headings).
Suggested Output:

Example

Example

This must be a setting with marked. Not sure if it's configurable.

I am not sure if this will help you guys but I just post my investigations about this issue here:
I installed the package via bower so I got all files to debug a bit arround.
The problem is the the following function in the file simplemd.js:
Renderer.prototype.heading = function(text, level, raw) {
  return '<h'
    + level
    + ' id="'
    + this.options.headerPrefix
    + raw.toLowerCase().replace(/[^\w]+/g, '-')
    + '">'
    + text
    + '</h'
    + level
    + '>\n';
};

it seems that this function comes from marked as @WesCossick already said.
To "fix" that bug just replace the function by the following code:
Renderer.prototype.heading = function(text, level, raw) {
  return '<h'
    + level
    + '>'
    + text
    + '</h'
    + level
    + '>\n';
};

I am not familiar with this modules so I dont know if that has any side effects but it works for me. I just downloaded it. I hope my post will push one of the developers who are familiar with this entire Library into a direction that helps. I hope for a fix, then I can remove my dirty hack ;)
I think the ids are used for a special feature but I dont know which function needs that ids. At the moment I dont have any errors when I just remove them...

Without looking at the code, this.options.headerPrefix looks possibly useful.

Without looking at the code, this.options.headerPrefix looks possibly useful.
this.options.headerPrefix is already set to nothing at default.

if you would set a this.options.headerPrefix the output just results in something like this.

Test

Test

This would still not turn off any duplicate ids in the output ;)

@TatzyXY it definitely looks like this is coming from marked. I would recommend submitting a PR on their repository so that it eventually makes its way into SimpleMDE.


Intersection Point in Y Shapped Linked Lists

💥 Proposal
(Suppose there are given two singly linked lists of different  size N and M, Now get the point where two linked lists intersect each other)
Have you read the Contributing Guidelines on Pull Requests?
(yes)

I would like to do this in c++

I would like to do this in Python

I would like to this solve this in C ,please assign it to me.

I would like to work with this issue in java.. kindly, assign me..

@siddharth25pandey, @plazzy99 , @iamrajiv  .
Please assign this Issue. My all assigned work are done. I am waiting.

@Rishikeshrajrxl go  with c++
@pradeep98  go with python
@SandipNayak go with c
@viveksahu26 go with java


feat: Added optional start process at elementId

closes #581
Background
We have maintained a separate fork og this library that has the ability to start a process at a specific elementId. To avoid keeping this fork up to date manually, we'd like to share the functionality as it seems more people are missing it and at the same time bring the functionality into the main line of the client.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


⚠ Bofrost.fr has degraded performance

In ebe1425, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1114 ms

Resolved: Bofrost.fr performance has improved in 3cb6873.


Misspelled words are not shown as such.

Got it to work in the console and started trying random words, and I'm getting the wrong results.
checker = require('spellchecker')
checker.isMisspelled('helo')
> false
checker.isMisspelled('hel')
> false
checker.isMisspelled('hello')
> false
checker.isMisspelled('helasdfasdf')
> true

Cheking the en_US.dict file I can see those words don't exist, but both hel and helo are suffixes for words that do exist in the dictionary.

I'm also seeing this. Everything is spelt correctly which is great for my bad spelling but bad in terms of functionality. I'm using osx, electron 1.2 with en_GB. I think this could be related to #49


Milestone 3: Report feedback

Reasoning:

Good notes giving context for the Sept/Aug 2005 repayment status!
Explanation on Fig 7 is an improvement, but it would still be better to have a more precise description of your model's feedback than "fairly good". (This may be tricky, we can talk about this in lab.)
Have you considered creating features relative to credit limit, as indicated by your peer feedback?
Peer feedback: "Explains precesion/recall and how it's related to default"

You do this in the summary ("Due to the risk associated with customers failing to pay, the model was designed to maximize the recall rate, identifing customers that will default to the greatest extent.") but it would also be good to include it in the conclusions - especially since you note that your best-case model actually has lower recall than the base-case. I'd like to to briefly discuss that observation!

We can see that some of the features have stronger correlation with the response varibale than others, for example LIMIT_BALANCE and Age.

Do you mean that both age and limit balance have stronger correlations with the response variable, or that limit balance has a stronger correlation than age?
Writing:

You're missing references to values from your report, eg: {{auc}}.
Some grammatical issues, eg: "It shows that that mean, standard deviation, min, max etc." "It's interesting that The medians" "Summary the data used in this study"

I will be doing the report modification

You're missing references to values from your report, eg: {{auc}}.

The final report is the html file and not the ipynb file since we could not we render the variables in Jupyter .
https://github.com/UBC-MDS/DSCI_522_group-314/blob/master/doc/final_report.html

Other issues were fixed as suggested

I see, the html was not displaying well on github so I was looking at the notebook.
I am noticing now that the project I cloned from github did not produce an html - when re-trying make, I note the error:
ValueError: Unknown exporter "html_embed", did you mean one of: asciidoc, custom, html, latex, markdown, notebook, pdf, python, rst, script, slides?

Do you know if this is a dependency issue?

@ksedivyhaley , hmm we ran into this error on linux machines before. But we changed the makefile and it seemed to resolve it.
#create final report .html
doc/final_report.html : results
	jupyter nbconvert --execute  doc/final_report.ipynb --to html

I see no html_embed in the current makefile, is it possible your local copy is out of sync?

We addressed this issue here. It was only for linux system and fixed it
https://github.com/UBC-MDS/DSCI_522_group-314/issues/51

I see no html_embed in the current makefile, is it possible your local copy is out of sync?

I was testing your Milestone 3 version, as I needed to ensure that the Makefile as submitted for the Milestone would result in a properly rendered report. Hadn't noticed/remembered the previously resolved issue with the same system-dependent bug when responding to this issue.
Confirmed that the properly rendered html has the values in the correct place.


comparison between  output of SID(see in the dark) and photoshop

I tried to compare the output generated by SID (O-sid) and photoshop (O-ps, with all auto from DNG to PNG), and according to what I saw:
1/ when the light is extremely low, O-sid is better than O-ps.
2/ but when the light is low and not extremely, O-ps is better than O-sid.
Also, O-sid loses quite a few texture compared to O-ps.
I wonder is this as expected?

For 2/, can you let me know which image you used? I can take a look.

I see. You are testing our model on your own data. This is very possible as our model is not expected to work on every camera sensor. We only train it on the Sony set in this case. I do not know which camera you used to capture this image. You may need data from your camera to finetune or retrain the model. This seems to be relevant: #7

That's interesting. The sensor we used is Sony IMX 117, which is quite popular for cameras.
As we can see, the output PNG brighten of whole photo at the cost of losing textures and mixing colors.
It resembles the deep network perform a blur filter (even not a edge-preserving filter) on photos.

Another reason may be that the input image is too good, not dark at all. The exposure time is not the only factor to make the image dark or not. If you take the image in daytime with enough light, it will not be dark at all. What is the amplification ratio you used for the result? If the input is too good, there is no problem with existing methods/software. Then this case will be far from our training range.

@shockjiang
I have tried the pre-trained model and the dataset provided by the author, some results are not good.
https://drive.google.com/open?id=18ilE6dmhXTpVD7ETjbK2_1ih78rxdePU
I think the photo you provide in the first time is comparably good. Why the output image became worse in the second time?

which two files did you refer to? @yueruchen

@shockjiang
You gave several images at the first time, the output is comparably lighter than one in the second time.
fist time:https://drive.google.com/file/d/1wOZGKXVsp5WdqW-aeGeBS9XSIrwsceA_/view
second time:

@shockjiang Hi, I have tested your DNG file , but the sid result is very pool. is the bit length 12? what is the black level ?Thanks !

@shockjiang @cchen156  hi, Have you solved the problem of losing textures and details?


BasePath not generated with JAX-RS interfaces

I am currently generating JAX-RS annotated interfaces. I'd like to have all the swagger-generated enpoints have a base path, for example, of /api.  I tried adding basePath: "/api" to the root of my swagger.yaml, but it does not look like this is picked up during code generation (and I do not see anything referencing basePath in the mustache templates.  Is this something that can be done?
I found a related issue in swagger codegen: 1425
Thanks

Hi @jkerak, thanks for the comment. @jbellmann, do you agree with the comment? If so, maybe @jkerak would be interested in helping with this.

This issue also exists for Spring annotated interfaces. I just checked with version 0.4.34 and the springinterfacesResponseEntityNoSwaggerAnnotations Template. The following swagger:
"swagger": "2.0",
"info": {
"basePath": "/api",
"paths": {
"/dependencies": {
...
produces this Interface:
public interface DependenciesApi {
@RequestMapping(value="/dependencies",method=RequestMethod.GET , produces={ "application/json" })
ResponseEntity dependenciesGet( @RequestParam(value="names")
List names);
}
I think, the base Path should be added as Class Annotation:
@RequestMapping("api")
public interface DependenciesApi {
@RequestMapping(value="/dependencies",method=RequestMethod.GET , produces={ "application/json" })
ResponseEntity dependenciesGet( @RequestParam(value="names")
List names);
}
to make this work.
I compared it with swagger-codegen-maven-plugin (spring profile for version 2.2.1), but they don't generate the basePath either.

Thank you for the fix,any ideea when this will be released ?

Oh, I missed the existing PR. But I will have a look today.


chapter 9, 10 , 11

all work done

I've added some general suggestions, Along with them can you please add some function to test how 'pass' works?

i have updated the changes and also added a use case for pass. kindly check


Update to newest code

I need to bump that thread for server permissions handling.
On Sat, Apr 25, 2015, 10:26 AM rourke750<EMAIL_ADDRESS>wrote:

Merged #3 https://github.com/Civcraft/modmail_ticketmanager/pull/3.
—
Reply to this email directly or view it on GitHub
https://github.com/Civcraft/modmail_ticketmanager/pull/3#event-289981178
.


replanter-1.19.2-1.4.jar not working

Feature is not working, and i cant see the mod in mod menu with replanter-1.19.2-1.4.jar.
All is fine with replanter-fabric-1.19-1.3.jar.

because it's for forge not fabric.


Can not deploy the code

Deployment Started
Status: Queued
Status: InProgress
Status: InProgress
Status: InProgress
Status: Completed
Deployment Complete
Failures:
classes/LeadConversionTriggerHelper.cls(142,18):Variable does not exist: Account.sObjectType
classes/LeadConversionTriggerHelperTest.cls(38,60):Method does not exist or incorrect signature: Account.sObjectType.getDescribe()
triggers/LeadConversionTrigger.trigger(34,5):Variable does not exist: LeadConversionTriggerHelper

Hi @rsmithdev6,
Thanks for the heads up. I did a test in a new developer org and was able to reproduce the problem. My unscientific testing makes me think that the recordTypeId field is not available on sobjects unless a custom record type exists.
Per that hypothesis, the problem is that the org you're deploying to likely does not have custom record types yet created for the Account object. Also, I imagine you may be trying to deploy into a new org and so there are no custom record types created for Contacts, Leads, or Opportunities either.
To fix the problem, please:

Make sure you have a Lead Process created (Setup | Leads | Lead Processes )
Make sure you have at least one Lead record type created other than 'master' (Setup | Leads | Record Types)
Make sure you have at least one Contact record type created other than 'master' (Setup | Contacts | Record Types)
Make sure you have at least one Account record type created other than 'master' (Setup | Accounts | Record Types)
Make sure you have at least one Opportunity Sales Process created (Setup | Opportunities | Sales Processes)
Make sure you have at least one Opportunity record type created other than 'master' (Setup | Opportunities | Record Types)

After I performed the above six steps in my developer org then I was able to deploy this code successfully.


freqz for sos input

I saw there is some discussion going on in #3259 about supporting other input formats in addition to b, a. I couldn't figure out if something has already been decided, but I think supporting sos filters is really important. I already implemented a sosfreqz function that accepts a sos matrix and basically forwards the calculations to freqz. Are you guys interested? If yes, I could create a PR.

cf https://github.com/scipy/scipy/pull/4465

Ah, I didn't see that - thanks, I'll comment over there.


Cannot find module './zlib_bindings'

node   v0.6.1
pdfkit   v0.1.6
Code:
var pdf = require('pdfkit');

Error log:
Error: Cannot find module './zlib_bindings'
    at Function._resolveFilename (module.js:334:11)
    at Function._load (module.js:279:25)
    at Module.require (module.js:357:17)
    at require (module.js:368:17)
    at Object.<anonymous> (/home/user/share/test_pdfkit/node_modules/pdfkit/node_modules/flate/lib/zlib.js:1:80)
    at Module._compile (module.js:432:26)
    at Object..js (module.js:450:10)
    at Module.load (module.js:351:31)
    at Function._load (module.js:310:12)
    at Module.require (module.js:357:17)

i'm getting the same error in version v0.6.2 any help?

I had the same problem and fixed.
Goto node_modules/pdfkit/node_modules/flate/ directory.
Copy the file zlib_bindings from build/Release directory to lib directory.

@jacksctsai
Thanks, it worked :)

I don't object, but felt obligated to mention this.
My personal stumbling block is that I'm using this on heroku right now, which is still at node 0.4.7, but honestly that's starting to create a lot of problems a lot of places now, and I'd rather just figure out some way to move up to 0.6.x for everything.
Utilizing the built in bindings sounds like a good direction from my naive view ... ...

Alright... took a look at this today, but found out that the zlib bindings in Node 0.6 are all asynchronous and the module I was using before was synchronous.  So I'm going to have to do a larger refactoring of the code to make it work if I'm to use the built in bindings.  I'd like to do that, but I'm busy with lots of other things so I don't really know when I'll get to it.  If you feel motivated please send me a pull request.  Sorry about the wait.

A bit more accurate how-to:

Goto node_modules/pdfkit/node_modules/flate/build/Release directory.
Copy the file zlib_bindings.node to ../../lib directory (lib directory of flate!).

Hi there. Thanks for the hard work on this great module. Are there any updates on this issue? The manual file copy works, but wondering if anyone is looking to tackle the proper fix.

So the proper fix for this will be a refactor of most of the codebase to be asynchronous rather than synchronous as Node 0.6's built in bindings are async.  I have been meaning to do this for a while anyway for performance reasons, and this issue compounds the problem.  I'll be working on PDFKit more over the next few weeks so expect a fix for this and other issues shortly!
Sorry about the wait, and if you think you can tackle any of the open issues here it would be greatly appreciated as I am only one person! :)
Devon

That's great news. Many thanks for your effort!

any way to get the workaround working on cloud9. (I can't see the zlib_bindings file there)

@jacksctsai thank you!

Hi

Hi i have installed zlib and pdfkit as well but not able to see ./zlib_bindings file in pdfkit

Hi

Hi i have installed zlib and pdfkit as well but not able to see ./zlib_bindings file in pdfkit

i'm with you on this one. still figuring out how to solve it.

Hi

Hi i have installed zlib and pdfkit as well but not able to see ./zlib_bindings file in pdfkit

i'm with you on this one. still figuring out how to solve it.

I am having the same problem too. Any ideas?

To improve the likehood of getting support, more info should de provided:
What environment ? Node? browser?
What bundler? Webpack? Rollup?
Provide a minimal reproduction snippet

To improve the likehood of getting support, more info should de provided:
What environment ? Node? browser? What bundler? Webpack? Rollup? Provide a minimal reproduction snippet

I got the error and mentioned how to reproduce here, Kindly have a look.
https://stackoverflow.com/questions/69805191/what-is-the-proper-way-to-use-pdfkit-in-react-native

@jacksctsai Please help me i did not get your this solution please explain it
Goto node_modules/pdfkit/node_modules/flate/ directory.
Copy the file zlib_bindings from build/Release directory to lib directory.
I could not find any folders in node_modules. I installed pdfkit from npm and please tell me now what i have to do? where is the flate? and all the other zlib+bindings? I am using react. Please elaborate your solution here


TF 2.3 fails to build with CUDA 11.0 using only 7.5 compute

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10 Pro x64 2004 Edition
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): source
TensorFlow version: 2.3.0
Python version: 3.8.5
Installed using virtualenv? pip? conda?: No, a direct accessible installation
Bazel version (if compiling from source): 3.4.1
GCC/Compiler version (if compiling from source): MSVC 2019 C++ compiler
CUDA/cuDNN version: 11.0/ cuDNN ver 8.0
GPU model and memory: RTX 2080 GPU memory : 8 GB , system memory 32 GB

Describe the problem
ERROR: An error occurred during the fetch of repository 'local_config_cuda':
Repository command failed
'C:/Program' is not recognized as an internal or external command, operable program or batch file.
Provide the exact sequence of commands / steps that you executed before running into the problem
Step 0 : git cloned the master, cd tensorflow, git checkout r2.3
Step 1 : python ./configure.py
-> specified : (1) Python and default library path to the installed python 3.8.5 64 bit with all requisite keras packages.
-> yes to CUDA : Found CUDA 11.0 and Found cuDNN 8.0, specified a compute capability of 7.5 (only one)
-> optimization flags : /arch:AVX2 and /std:c++17
-> eigen inline : N (longer compile time acceptable).
-> no to ROCm, and no to android.
Step 2 :  bazel build --config=opt --config=cuda --define=no_tensorflow_py_deps=true //tensorflow/tools/pip_package:build_pip_package
Proceeded a few steps for one minute and then failed.
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.
PS C:\Users\gautam\tensorflow> bazel build --config=opt --config=cuda --define=no_tensorflow_py_deps=true //tensorflow/tools/pip_package:build_pip_package
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
WARNING: The following configs were expanded more than once: [cuda, using_cuda]. For repeatable flags, repeats are counted twice and may lead to unexpected behavior.
INFO: Options provided by the client:
Inherited 'common' options: --isatty=1 --terminal_columns=120
INFO: Reading rc options for 'build' from c:\users\gautam\tensorflow.bazelrc:
Inherited 'common' options: --experimental_repo_remote_exec
INFO: Options provided by the client:
'build' options: --python_path=C:/Program Files/Python3/python.exe
INFO: Reading rc options for 'build' from c:\users\gautam\tensorflow.bazelrc:
'build' options: --apple_platform_type=macos --define framework_shared_object=true --define open_source_build=true --java_toolchain=//third_party/toolchains/java:tf_java_toolchain --host_java_toolchain=//third_party/toolchains/java:tf_java_toolchain --define=use_fast_cpp_protos=true --define=allow_oversize_protos=true --spawn_strategy=standalone -c opt --announce_rc --define=grpc_no_ares=true --noincompatible_remove_legacy_whole_archive --noincompatible_prohibit_aapt1 --enable_platform_specific_config --config=v2
INFO: Reading rc options for 'build' from c:\users\gautam\tensorflow.tf_configure.bazelrc:
'build' options: --action_env PYTHON_BIN_PATH=C:/Program Files/Python3/python.exe --action_env PYTHON_LIB_PATH=C:/Program Files/Python3/lib/site-packages --python_path=C:/Program Files/Python3/python.exe --config=xla --action_env CUDA_TOOLKIT_PATH=C:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v11.0 --action_env TF_CUDA_COMPUTE_CAPABILITIES=7.5 --config=cuda --action_env TF_CONFIGURE_IOS=0
INFO: Found applicable config definition build:v2 in file c:\users\gautam\tensorflow.bazelrc: --define=tf_api_version=2 --action_env=TF2_BEHAVIOR=1
INFO: Found applicable config definition build:xla in file c:\users\gautam\tensorflow.bazelrc: --action_env=TF_ENABLE_XLA=1 --define=with_xla_support=true
INFO: Found applicable config definition build:cuda in file c:\users\gautam\tensorflow.bazelrc: --config=using_cuda --define=using_cuda_nvcc=true
INFO: Found applicable config definition build:using_cuda in file c:\users\gautam\tensorflow.bazelrc: --define=using_cuda=true --action_env TF_NEED_CUDA=1 --crosstool_top=@local_config_cuda//crosstool:toolchain
INFO: Found applicable config definition build:opt in file c:\users\gautam\tensorflow.tf_configure.bazelrc: --copt=/arch:AVX2 --copt=/std:c++17 --define with_default_optimizations=true
INFO: Found applicable config definition build:cuda in file c:\users\gautam\tensorflow.bazelrc: --config=using_cuda --define=using_cuda_nvcc=true
INFO: Found applicable config definition build:using_cuda in file c:\users\gautam\tensorflow.bazelrc: --define=using_cuda=true --action_env TF_NEED_CUDA=1 --crosstool_top=@local_config_cuda//crosstool:toolchain
INFO: Found applicable config definition build:windows in file c:\users\gautam\tensorflow.bazelrc: --copt=/w --copt=/D_USE_MATH_DEFINES --host_copt=/D_USE_MATH_DEFINES --cxxopt=/std:c++14 --host_cxxopt=/std:c++14 --config=monolithic --copt=-DWIN32_LEAN_AND_MEAN --host_copt=-DWIN32_LEAN_AND_MEAN --copt=-DNOGDI --host_copt=-DNOGDI --linkopt=/DEBUG --host_linkopt=/DEBUG --linkopt=/OPT:REF --host_linkopt=/OPT:REF --linkopt=/OPT:ICF --host_linkopt=/OPT:ICF --experimental_strict_action_env=true --verbose_failures --distinct_host_configuration=false
INFO: Found applicable config definition build:monolithic in file c:\users\gautam\tensorflow.bazelrc: --define framework_shared_object=false
DEBUG: Rule 'io_bazel_rules_docker' indicated that a canonical reproducible form can be obtained by modifying arguments shallow_since = "1556410077 -0400"
DEBUG: Repository io_bazel_rules_docker instantiated at:
no stack (--record_rule_instantiation_callstack not enabled)
Repository rule git_repository defined at:
C:/users/gautam/_bazel_gautam/4wn3vkrp/external/bazel_tools/tools/build_defs/repo/git.bzl:195:33: in
INFO: Repository local_config_cuda instantiated at:
no stack (--record_rule_instantiation_callstack not enabled)
Repository rule cuda_configure defined at:
C:/users/gautam/tensorflow/third_party/gpus/cuda_configure.bzl:1399:33: in
ERROR: An error occurred during the fetch of repository 'local_config_cuda':
Traceback (most recent call last):
File "C:/users/gautam/tensorflow/third_party/gpus/cuda_configure.bzl", line 1369
_create_local_cuda_repository(<1 more arguments>)
File "C:/users/gautam/tensorflow/third_party/gpus/cuda_configure.bzl", line 1051, in _create_local_cuda_repository
_find_libs(repository_ctx, <2 more arguments>)
File "C:/users/gautam/tensorflow/third_party/gpus/cuda_configure.bzl", line 598, in _find_libs
_check_cuda_libs(repository_ctx, <2 more arguments>)
File "C:/users/gautam/tensorflow/third_party/gpus/cuda_configure.bzl", line 500, in _check_cuda_libs
execute(repository_ctx, <1 more arguments>)
File "C:/users/gautam/tensorflow/third_party/remote_config/common.bzl", line 208, in execute
fail(<1 more arguments>)
Repository command failed
'C:/Program' is not recognized as an internal or external command,
operable program or batch file.
ERROR: Skipping '//tensorflow/tools/pip_package:build_pip_package': no such package '@local_config_cuda//cuda': Traceback (most recent call last):
File "C:/users/gautam/tensorflow/third_party/gpus/cuda_configure.bzl", line 1369
_create_local_cuda_repository(<1 more arguments>)
File "C:/users/gautam/tensorflow/third_party/gpus/cuda_configure.bzl", line 1051, in _create_local_cuda_repository
_find_libs(repository_ctx, <2 more arguments>)
File "C:/users/gautam/tensorflow/third_party/gpus/cuda_configure.bzl", line 598, in _find_libs
_check_cuda_libs(repository_ctx, <2 more arguments>)
File "C:/users/gautam/tensorflow/third_party/gpus/cuda_configure.bzl", line 500, in _check_cuda_libs
execute(repository_ctx, <1 more arguments>)
File "C:/users/gautam/tensorflow/third_party/remote_config/common.bzl", line 208, in execute
fail(<1 more arguments>)
Repository command failed
'C:/Program' is not recognized as an internal or external command,
operable program or batch file.
WARNING: Target pattern parsing failed.
ERROR: no such package '@local_config_cuda//cuda': Traceback (most recent call last):
File "C:/users/gautam/tensorflow/third_party/gpus/cuda_configure.bzl", line 1369
_create_local_cuda_repository(<1 more arguments>)
File "C:/users/gautam/tensorflow/third_party/gpus/cuda_configure.bzl", line 1051, in _create_local_cuda_repository
_find_libs(repository_ctx, <2 more arguments>)
File "C:/users/gautam/tensorflow/third_party/gpus/cuda_configure.bzl", line 598, in _find_libs
_check_cuda_libs(repository_ctx, <2 more arguments>)
File "C:/users/gautam/tensorflow/third_party/gpus/cuda_configure.bzl", line 500, in _check_cuda_libs
execute(repository_ctx, <1 more arguments>)
File "C:/users/gautam/tensorflow/third_party/remote_config/common.bzl", line 208, in execute
fail(<1 more arguments>)
Repository command failed
'C:/Program' is not recognized as an internal or external command,
operable program or batch file.
INFO: Elapsed time: 55.288s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (0 packages loaded)
currently loading: tensorflow/tools/pip_package
19:14:49 PS C:\Users\gautam\tensorflow>

@quasar66
Can you try with tested build configurations from here and try building TensorFlow with CUDA 10.1, and check if it works.
Please, refer #42084 (comment)  for cuda 11.0 support. Thanks!

Hi...
Update - the build process fails for CUDA 10.2, and 10.1 with identical errors at same stage. The input parameters remain the same : namely, use GPU, use AVX2, use c++17.
An observation : The error seems to be a string parsing error as the software seems to be searching for 'C:\Program' which is probably incorrect use of ' '(space character) for parsing a execution string (short for C:\Program Files...)
The source is release 2.3.

Try changing your python path since the python path created similar problem for me

--python_path=C:/Program Files/Python3/python.exe -> this to C:/Python3/python.exe somethinglike this

Hi...
The build process kickstarted properly with the change in Python installation directory .. However, fails to handle the cudnn with the error as shown below.
ERROR: C:/users/gautam/tensorflow/tensorflow/stream_executor/cuda/BUILD:319:11: undeclared inclusion(s) in rule '//tensorflow/stream_executor/cuda:cudnn_stub':
this rule is missing dependency declarations for the following files included by 'tensorflow/stream_executor/cuda/cudnn_stub.cc':
'tensorflow/stream_executor/cuda/cudnn_version.h'
Target //tensorflow/tools/pip_package:build_pip_package failed to build
INFO: Elapsed time: 2652.569s, Critical Path: 377.54s
INFO: 5670 processes: 5670 local.
FAILED: Build did NOT complete successfully
The opt : AVX2, dropped the other opt (c++17 as most of the compilation overwrote the flag to c++14 - the default C++).
But thanks @wolwire for the idea regarding the python installation path !! Wondering why I didn't think of it myself...
@ravikyram - I need to recompile from source as I need to generate the C++ library for running loadsavedmodel() in a C++ program.
Any advice please?

Hi..
Further attempt to compile completely without CUDA support, and taking ONLY the default options also failed. This time the error is not very clear, but the only error line seems to be (not sure why we need b_float compatibility)
ERROR: C:/users/gautam/tensorflow/tensorflow/python/BUILD:501:11: C++ compilation of rule '//tensorflow/python:bfloat16_lib' failed (Exit 2): cl.exe failed: error executing command

Target //tensorflow/tools/pip_package:build_pip_package failed to build
INFO: Elapsed time: 6344.527s, Critical Path: 390.88s
INFO: 8875 processes: 8875 local.
FAILED: Build did NOT complete successfully
Environment : bazel 3.4.1 - all Windows C++ specifications installed, MSYS2 installed, MSVC 14.27.29110, Windows SDK : 10.0.19041.1
running on windows 10 Pro 64 bit updated to 2004 Edition.
Thanks..

@meteorcloudy could you take a look?

Hi @quasar66 , can you rebuild  with --verbose_failures and provide the error message here? Thanks!

Hi @meteorcloudy
I am currently attempting to build with Bazel 3.1.0, AVX2, c++17, the master branch (not r2.3), CUDA 11.0, cuDNN 8 , build for tensorflow.dll (which is what I actually need) - and its been running for last 16 hours (my PC is little old) but has not yet shown any errors. 14107 of 14375 .. 100% CPU for last 16 hours :-) and for some time it indeed had used up all 32 GB of RAM.
If this succeeds, I will most certainly let you know. If it fails, I shall run with the flag you mentioned and will share the result.
Thanks...


Memtracking stuff

Built on top of #5965.

LGTM

May want to figure out what's up with the tests...

Good now ?

LGTM


⚠ Forum - Funcraft has degraded performance

In ce9d68b, Forum - Funcraft (https://community.funcraft.net/) experienced degraded performance:

HTTP code: 200
Response time: 712 ms

Resolved: Forum - Funcraft performance has improved in f11fbbb after 11 minutes.


Retry tavily outside of model parsing and allow to not generate subqueries

Is this just in case there is a bug inside the transformed_results = [...] bit of the function?

yeah, if they change output model, it won't help to retry the call


When importing Flink SQL into Starlocks, if the data format is JSON, the default value cannot be triggered, while if the data format is CSV, the default value can be triggered

Steps to reproduce the behavior (Required)

CREATE TABLE send_coupon_info (
dt date NOT NULL COMMENT "分区时间，取自drawTime字段",
primary_id varchar(50) NOT NULL COMMENT "主键id,上游表名和主键id拼接而成",
insert_time datetime NULL DEFAULT CURRENT_TIMESTAMP COMMENT "插入数据库时间"
) ENGINE=OLAP
PRIMARY KEY(dt, primary_id)
COMMENT "领券记录表"
PARTITION BY RANGE(dt)
(
PARTITION p20231112 VALUES [("2023-11-12"), ("2023-11-13")),
PARTITION p20231113 VALUES [("2023-11-13"), ("2023-11-14")),
PARTITION p20231114 VALUES [("2023-11-14"), ("2023-11-15")),
PARTITION p20231115 VALUES [("2023-11-15"), ("2023-11-16")),
PARTITION p20231116 VALUES [("2023-11-16"), ("2023-11-17")))
DISTRIBUTED BY HASH(primary_id) BUCKETS 1
PROPERTIES (
"replication_num" = "3",
"dynamic_partition.enable" = "true",
"dynamic_partition.time_unit" = "DAY",
"dynamic_partition.time_zone" = "Asia/Shanghai",
"dynamic_partition.start" = "-1",
"dynamic_partition.end" = "2",
"dynamic_partition.prefix" = "p",
"dynamic_partition.buckets" = "1",
"dynamic_partition.history_partition_num" = "0",
"dynamic_partition.replication_num" = "3",
"in_memory" = "false",
"storage_format" = "V2",
"enable_persistent_index" = "true",
"compression" = "LZ4"
);
Flink SQL DDL and insert statements:
CREATE TABLE send_coupon_info (
dt              date,
primary_id      STRING,
PRIMARY KEY(dt,primary_id) NOT ENFORCED
) WITH (
'connector' = 'starrocks',
'jdbc-url' = 'jdbc:mysql://',
'load-url' = '',
'database-name' = 'ods',
'table-name' = 'send_coupon_info',
'username' = 'root',
'password' = '',
'sink.semantic'='at-least-once',
'sink.label-prefix'='send_coupon_info',
'sink.properties.format'='json'
);
insert into send_coupon_info
select
TO_DATE(DATE_FORMAT(create_time,'yyyy-MM-dd'),'yyyy-MM-dd'),
CONCAT(table_name,'_',cast(id  as string)),
from send_coupon_info_source ;
select current_version()
mysql> select current_version();
+-------------------+
| current_version() |
+-------------------+
| 2.5.13 a3b58a0    |
+-------------------+
1 row in set (0.01 sec)
View insert_ All values in the time column are null, and it was found that the default current was not triggered_ TIMESTAMP value
Modify the DDL 'sink. properties. format'='json 'in Flink SQL to' sink. properties. format '='CSV', which triggers the default current_ TIMESTAMP value, which can obtain the normal storage time

I experience the same thing.


Can't find the material-design-fonts-extended

I have yay installed
I use arch btw
btw another question
Where did you get the arch logo at the papirus theme?

Update: I found the real command.
It's not showing the pacman icons still tho
([https://github.com/Chick2D/neofetch-themes/issues/2])


[6340] Refactor Hesa::RetrieveJob job to fan out

Context
The Hesa::RetrieveJob job runs as one giant job and if any of the inline Trainees::CreateFromHesa.call fails it halts processing of subsequent records.
This PR refactors Hesa::RetrieveJob to fan out the Trainees::CreateFromHesa calls into their own jobs. This will allow our collection process to continuously run and not get coupled to problematic records.
Changes proposed in this pull request

Updates Hesa::RetrieveJob
Move the mapping logic from XML to a hash outside of Trainees::CreateFromHesa so it can be serialised properly when passed into the job
Update specs

Guidance to review
This will now look like so:

 Modify HESA import process so that it continues with remaining trainees upon encountering an error


Support tail-based sampling from OTEL Collector

Jaeger v2 can now support tail based sampling that exists in OTEL Collector as an extension.

[ ] include loadbalancing exporter and tail_sampling processor in components.go
[ ] create sample configuration and docker-compose file utilizing it
[ ] create e2e integration test (script and GH actions workflow)
[ ] create README documenting the setup

As a single task it's too large for good-first-issue, but can be done incrementally

@yurishkuro I'm interested in working on this but have never contributed to this repository before. Would you be able to guide me on how to tackle this issue?

@mahadzaryab1 I would start with

reading the blog post I linked
reproducing it in a local setup via docker compose
then swapping the final stage collectors with jaeger-v2 collector (it will require importing the tail sampling processor into components.go)

sounds good! i'll give this a shot

@yurishkuro I got the first item done in https://github.com/jaegertracing/jaeger/pull/5878 and read the blog post. Are there any instructions/examples on how I can can create a sample configuration and and docker-compose file. Thank you so much for your time and help!

Sample configurations are provided in the blog post. Example of docker compose using Jaeger is in docker-compose/monitor/docker-compose-v2.yml (but you will need to build the Docker image locally so that the code recognizes tail sampling processor, because officially published image will not)

@yurishkuro thank you! as a follow-up, i'm trying to build the image locally by running make build from jaeger/docker-compose/monitor but am running into the following error. am i missing a step in between?
+ cd packages/jaeger-ui
./scripts/rebuild-ui.sh: line 35: cd: packages/jaeger-ui: No such file or directory
make[2]: *** [rebuild-ui] Error 1
make[1]: *** [jaeger-ui/packages/jaeger-ui/build/index.html] Error 2
make: *** [build] Error 2

@yurishkuro this is how i've modified the docker setup to reproduce tail based sampling - let me know if you have any feedback

you need to do initialize & update git submodules in order to access UI dir

@yurishkuro awesome! thank you. I was able to get the setup going based on the configuration I linked above. Here's the output I'm seeing from the different policies that are being evaluated:
jaeger-1          | 2024-08-23T05:14:01.932Z	debug	sampling/string_tag_filter.go:95	Evaluting spans in string-tag filter	{"kind": "processor", "name": "tail_sampling", "pipeline": "traces", "policy": "string_attribute"}
jaeger-1          | 2024-08-23T05:14:01.932Z	debug	sampling/latency.go:34	Evaluating spans in latency filter	{"kind": "processor", "name": "tail_sampling", "pipeline": "traces", "policy": "latency"}
jaeger-1          | 2024-08-23T05:14:01.932Z	debug	sampling/probabilistic.go:46	Evaluating spans in probabilistic filter	{"kind": "processor", "name": "tail_sampling", "pipeline": "traces", "policy": "probabilistic"}
jaeger-1          | 2024-08-23T05:14:01.932Z	debug	sampling/status_code.go:54	Evaluating spans in status code filter	{"kind": "processor", "name": "tail_sampling", "pipeline": "traces", "policy": "status_code"}

I suggest now to think about what e2e integration test could look like for this

@yurishkuro will do - thanks for all your help and guidance so far! should the sample configuration/docker compose/readme go in https://github.com/jaegertracing/jaeger/tree/main/examples?

docker-compose/tail-sampling

@yurishkuro I completed the second task of creating a sample configuration. I'm looking for a bit of guidance on the e2e integration tests. I see that we have some integration tests but those mostly seem to be for the storage collectors. Would those same tests apply here? Or do we want to maybe test the traces are being sampled according to the policies in our config?

You can use the tests we have as inspiration, but don't try to fit your test to them. How would you e2e test tail sampling if starting from scratch? Try explaining the whole setup and test process.

@yurishkuro I am envisioning something like this:

Set up the jaeger-v2 collector with a tail-sampling processor with a string_attribute policy that matches on a particular tag
Start the load balancing collector and jaeger-v2 collector from the docker-compose setup
Send spans with various attributes to the otel-collector with load balancing
Test that only the tags that match the ones listed in the policy are sampled/stored by the jaeger-v2 collector

Let me know what you think and if you have any feedback!

SGTM. A couple thoughts:

Such test would depend on your ability to generate very specific traces, how are you planning to achieve that? E.g. both microsim and tracegen utils generate nearly identical traces, although you have more control with tracegen where, for instance, you could control how many service names it generates
How will you verify that the right traces are sampled?
Most importantly, you need to make sure that the test is robust, e.g. that the reason some traces are not sampled is due to tail sampler, not due to some other condition. One way to ensure this is to run the A/B test where the only change between A and B is the configuration of the tail sampler. E.g. in A you only configure it to sample service-a, but not service-b, and you verify that you can observe that. Then in B you flip that condition and again verify that you get expected results.

BTW, in order to perform this test you do not need load generator running continuously, it's better if you just generate a fixed number of traces. You also need to make sure the storage is purged between A and B.

@yurishkuro Thanks for the follow-up. I'm going to try playing around with tracegen to begin with to see what kind of traces I can generate. I'm currently trying to replace mirosim with tracegen to the docker-compose.yml with the following configuration.
  tracegen:
    image: jaegertracing/jaeger-tracegen:latest
    environment:
      - OTEL_EXPORTER_OTLP_TRACES_ENDPOINT=http://jaeger:4318/v1/traces
    command: ["-duration", "10s", "-workers", "3", "-pause", "250ms"]
    depends_on:
      - jaeger

This doesn't seem to work and gives me the following error:
tracegen-1        | 2024/08/24 20:42:32 traces export: Post "http://jaeger:4318/v1/traces": dial tcp: lookup jaeger on <IP_ADDRESS>:53: no such host
tracegen-1 exited with code 0

Do you know what I'm missing? Here is the full docker-compose set up if that helps.

You're missing network setting on tracegen compose config

@yurishkuro thank you so much! As a follow up, I was wondering if you had any guidance (or if there are already places we are doing this) on how I can query the spans  that are stored? If we set up a filter on the service name, then we can do an A/B test with and without sampling as you suggested by querying the spans from store.

@yurishkuro I've been reading through the storage integration tests. If we add a filter on the service name in our tail sampling processor config, then can we do something similar to https://github.com/jaegertracing/jaeger/blob/main/plugin/storage/integration/integration.go#L136 to get all the services in the span and ensure that we only have the ones we listed in our policy?

As well, do you have any thoughts on where the integration tests should live? It seems like the storage ones are in cmd/jaeger/internal/integration.

to get all the services from the traces we have collected (in memory) and ensure that we only have the ones we listed in our policy?

Conceptually yes, but sampling works at the trace level (all spans or nothing), so still need a bit more thought how you'd test them

The test needs a new workflow yaml file and a new shell script to orchestrate. Then depending on how you want to do the analysis you might need more - at this point I don't know what you have in mind. If you need go code then cmd/jaeger/internal/integration would be a good place.

That's fine as a first version, but it's not quite a robust test because it doesn't prove that service02 was actually generated in the first place. An A/B test wouldn't have that problem.

@yurishkuro I see. How about something like?

Start jaeger backend with batch processor
Use tracegen to generate traces for 5 services
Query the data store to see that traces are present for all 5 services
Flush the data store
Start jaeger backend with tail sampling processor that has a policy to only sample service.name=tracegen-02
Query the data store to see that traces are present only for tracegen-02

@yurishkuro would you be able to help me with the setup of the test? i'm stuck on the following:

how do I query the datastore?
can all of this be done through the shell script or should I be writing go code?

It is easiest to query data store if you write Go code in cmd/jaeger/internal/integration, because that's exactly what the tests located there are doing - they are using an RPC implementation of SpanReader which proxies the requests to the query service running in the jaeger-v2 collector.
Load balancer is not necessary since you will only be running a single instance of the collector. The objective of the load balancer is to ensure that all spans for the same trace ID end up in the same instance of the collector that runs tail sampling logic.

Got it! A couple of follow-ups I had:

Can we simply use the SpanWriter to manually send spans? Is there a reason why we want to use tracgen?
If i write the tests in cmd/jaeger/internal/integration, will they be run as part of https://github.com/jaegertracing/jaeger/blob/main/Makefile#L140-L146? Is that fine?

Yes if you have to write code to read you might as well save whichever traces you need via Span Writer.
The test won't automatically run because those integration tests all require an environment variable to activate, otherwise they will all run at once and most of them fail because storage backend won't be available.

@yurishkuro I've completed the integration test and pushed it to https://github.com/jaegertracing/jaeger/pull/5878. Working on the final README task - should this go in jaeger/docker-compose/tail-sampling?

yes please

@yurishkuro Thanks for all your help and guidance in helping me complete this issue. I learnt a lot about OpenTelemtry, Jaeger, and Distributed Tracing. I would like to keep contributing to this project. Are there any other issues that I can pick up next?

@mahadzaryab1 appreciate the help. The top priority is completing the work on Jaeger-v2, which you can see in the project board https://github.com/orgs/jaegertracing/projects/3/views/2


Object Orientation Doc - Comment Incorrect

The problem
https://docs.perl6.org/language/objects

say @words.join: '--';
# OUTPUT: «Abe--Lincoln--said--Fourscore--and--seven--years--ago␤» --

The comment line should be:

# OUTPUT: «Abe--Lincoln--said--Fourscore  and  seven  years  ago␤» --

without last four --'s

Suggestions
Change comment to:
OUTPUT: «Abe--Lincoln--said--Fourscore  and  seven  years  ago␤» --
without last four --'s

The colon in "separating the argument list with a colon" must a be typo.
@JJ Do you think the following statement seems clearer and the example given more in line with it?

Similarly, multiple arguments can be specified by placing a colon after the method and separating the argument list with a comma:

say @words.join('-').subst: 'years', 'DAYS';
# OUTPUT: «Abe--Lincoln--said--Fourscore and seven DAYS ago␤»


modules: canopennode: Updated CANopenNode driver to v2.0

This commit updates the existing zephyr driver for canopennode from v1.3 to v2.0 and adds support for all new upstream driver features. The motivation for this was that I needed the ASCII gateway feature, which is not available in v1.3.
I ran checkpatch on my own code, but the stuff in the objdict-directory of the example is autogenerated and won't pass. But I hardly even touched that, so it should be fine.

The following west manifest projects have been modified in this Pull Request:

Name
Old Revision
New Revision
Diff

canopennode
https://github.com/zephyrproject-rtos/canopennode/commit/53d3415c14d60f8f4bfca54bfbc5d5a667d7e724
0c000b59242e006630fe326eb3ef5a8b78ada38f
<EMAIL_ADDRESS>

Note: This message is automatically posted and updated by the Manifest GitHub Action.

What is the reason to do it for NCS?

Actually I wasn't sure where to submit this. Should this go directly into upstream Zephyr?

What is the reason to do it for NCS?

Actually I wasn't sure where to submit this. Should this go directly into upstream Zephyr?

Yes, please.

Alright, I'll close this then.


let not allowed as a bound identifier in for-in and for-of loops

> require('esprima').parse('var func = let => 0')
Error: Line 1: Unexpected token let
    at constructError (esprima.js:2388:21)
    at createError (esprima.js:2407:17)
    at unexpectedTokenError (esprima.js:2481:13)
    at throwUnexpectedToken (esprima.js:2486:15)
    at parsePrimaryExpression (esprima.js:3255:13)
    at inheritCoverGrammar (esprima.js:2662:18)
    at parseLeftHandSideExpressionAllowCall (esprima.js:3387:20)
    at inheritCoverGrammar (esprima.js:2662:18)
    at parsePostfixExpression (esprima.js:3463:16)
    at parseUnaryExpression (esprima.js:3523:20)

As far as I can tell, let is a valid Identifier and BindingIdentifier under non-strict mode, and therefore it should be allowed as a parameter.

ES2015 14.2:

ArrowFunction[In, Yield] :
ArrowParameters[?Yield] [no LineTerminator here] => ConciseBody[?In]
ArrowParameters[Yield] :
BindingIdentifier[?Yield]
CoverParenthesizedExpressionAndArrowParameterList[?Yield]

ES2015 [12.2](http://www.ecma-international.org/ecma-262/6.0/#sec-primary-expression):

> ```
CoverParenthesizedExpressionAndArrowParameterList[Yield] :
    ( Expression[In, ?Yield] )
    ( )
    ( ... BindingIdentifier[?Yield] )
    ( Expression[In, ?Yield] , ... BindingIdentifier[?Yield] )

ES2015 12.1:

BindingIdentifier[Yield] :
Identifier
[~Yield] yield

Identifier :
IdentifierName but not ReservedWord

ES2015 [12.1.1](http://www.ecma-international.org/ecma-262/6.0/#sec-identifiers-static-semantics-early-errors)

> ```
BindingIdentifier : Identifier
   * It is a Syntax Error if the code matched by this production is contained in strict mode code and the StringValue of Identifier is "arguments" or "eval".
>
Identifier : IdentifierName but not ReservedWord
    * It is a Syntax Error if this phrase is contained in strict mode code and the StringValue of IdentifierName is: "implements", "interface", "let", "package", "private", "protected", "public", "static", or "yield".
    * It is a Syntax Error if StringValue of IdentifierName is the same String value as the StringValue of any ReservedWord except for yield.

 I believe this should covered when we tackled #1289.

Seems like that was fixed, but the following two cases are not:
for (let in a) {}
for (let of a) {}

@TimothyGu The first is a valid Script, the second is not.

Ah okay. Didn't see "[lookahead ≠ let ]" in the spec.


Bug: Reordering Sortable Posts not updating in Dashboard

When reordering sortable posts in the dashboard, the website updates accordingly, but the list in the dashboard will revert to the previous order when refreshed. However, the order on the rendered page will not revert.
To Reproduce
Steps to reproduce the behavior:

Open the dashboard
Go to a section with sortable posts
Reorder the posts
View the rendered page
Refresh the dashboard

Expected behavior
The order in the dashboard is the same as on the rendered page.
Your Environment
Include relevant details, so others can replicate the issue.

OS: macOS Catalina 10.15.3
Browser: Safari 13.0.5
Node Version: 8.16.2
Vapid Version: 0.9.5

Same problem.
My Environment is

OS: Ubuntu 18.04
Browser: Safari 13.0.5, Firefox Developer Edition 75.0b5
Node Version: 10.19.0
Vapid Version: 0.9.5

Thanks for letting me know. I'm a bit overloaded with personal things right now, so I may not be able to get to this for a bit. Would either of you be willing to help jump in and submit a pull request?

I’ve never worked on a Node.js project before, but I’ll try my best to find and fix the bug.


filetransfer: can we merge transfers with ach.MergeFiles helper?

ach.MergeFiles is a helper for combining the Batch records between two files. This is useful for optimizing files to reduce costs from the Federal Reserve.
Can we use that in paygate's filetransfer Controller.mergeTransfer? Right now we add/remove the batches, but the helper should do this for us (and split over the line limit).
We might need an option to merge files according to a config (custom line limits, for example).

Yep!


FR: Auto-detect if CSV has header

Version
All versions
Feature
It would be nice to have an option to try to automatically detect if a newly-opened CSV has a header row or not and enable the Read options -> Has header checkbox automatically if so. This might not always be straightforward and may be infeasible, but I thought there could be a place to at least discuss it. There would be tons of edge cases, but I wonder if there's a way to guess the format of a large majority of normal csv files by seeing if e.g. the entropy or word-like structure of the first row of text differs significantly from the next e.g. 10 rows averaged, and possibly that the next 10 rows are all much more similar to each other compared to the first row. There is likely a much better method but I wonder if there could be a simple, lightweight algorithm to guess and get it right 95% of the time.
As a simpler first step, there could be an icon by the first row or right-click context menu to disable/enable treating it as a header row. Even if the automatic algorithm is infeasible, this would be helpful.

Personally, I don't care about headers, I hardly ever use them.
The only thing that works well with headers is if the table has many rows.
In this case, you can make the first row sticky and basically have the same effect.

As a simpler first step, there could be an icon by the first row or right-click context menu to disable/enable treating it as a header row.
Even if the automatic algorithm is infeasible, this would be helpful.

I don't think I have understood this. Via "Read options -> has header -> reset data" you can enable/disable that this row is treated as a header row.
If you only work with csv files that have a header row, you can activate the "has header" option in the settings to apply it automatically as soon as the table is opened.
That said, it's an interesting question.
I don't know much about entropy, but I do know that similarly structured data has low entropy.
And low entropy data can be compressed well...
Maybe something with a size ratio of compressed/uncompressed 10 lines + header row...
We can open this up for discussion.

I don't think I have understood this. Via "Read options -> has header -> reset data" you can enable/disable that this row is treated as a header row.

You're right, you can already do this. I instinctively looked for a way to activate this (or essentially 'freeze first row' as Google Sheets and Excel offer) from the header row directly, then eventually found this option. Maybe it's just me that thinks some interaction directly with the rendered header row would be easier, not a high priority.
And yes, something along the lines of that with entropy and data size. You wouldn't need to store any state, just have some quick calculation when opening the document to decide whether to render or not. Granted, I'm not sure if there's already state stored to retain resized column widths / past decision to enable/disable a particular csv's header row... but if that existed I don't think it's stored in the same folder anyway.

If you only work with csv files that have a header row, you can activate the "has header" option in the settings to apply it automatically as soon as the table is opened.

This is perfect. But I looked for this a lot and couldn't find it until I found this post. Perhaps this can be made easier to find. Maybe a tooltip next to the checkbox to give it as a suggestion. Or have a way to make it the default from the UI?
Anyway, that option saves me a lot of time (it was annoyting to do three clicks every time I opened a file

Expand "Read options"
Tick "Has header"
Collapse "Read options"

I can add a tooltip and point out that there is such a setting.

That would be great!

The next version will have a setting tryToGuessHasHeader and will do what it says. I will set it to false by default (may change in the future).
I thought about the entropy approach again, but I don`t think that will work well.
Here are my thoughts:

the header row is just one row, calculating the entropy of the other rows (or just some rows) is much more data

so the header will only change the entropy a little bit

headers are usually like headings, which are different from the cell values

header data will probably not be included in the cell values
numbers are usually not used as headers

the first row is a data row (not a header row) if

we find at least 3 columns in which

the header value is also contained in the cell values (of this column)
the header value is only a number

I also added the setting names to the has header tooltip.


[Docs]: README.md - link architecture diagram

What type of suggestions?
Unclear Documentation
What page is concerned?
https://github.com/kubefirst/kubefirst#readme
What is the issue?
The image link to the Kubefirst Architecture Diagram need to be fixed.
How it looks now: (../docs)/
Fix: (./images/kubefirst-arch.png)
Author: Guilherme Dias<EMAIL_ADDRESS>Code of Conduct

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

Thanks @Siluryan, sorry for the delay, I'm fixing it now.


[Enhancement] Module 0: Move setup files into a public bucket to eliminate bucket creation step

For a simpler participant experience, it would be great if AWS published the deployment files to a public S3 bucket that could be used to deploy the CloudFormation stacks. Many other workshops and AWS solutions take this approach and it saves setup time and eliminates potential issues during deployment (i.e. accidentally copying the files into a /deploy prefix in the bucket, which  then causes an Access Denied error when attempting to launch with the URL supplied in the instructions.

Good suggestion.  This is something that we are actively working on so stay tuned for an update that addresses this in the near future.

An update has been pushed so that the templates and the functions for this workshop are now sources from a central S3 bucket.  There is no longer the need to download and stage the additional assets.


Joint genotyping qc to multicohorts

Simple uplift of ClinvArbitration workflow to use Cohorts
Slightly more involved uplift of JointVcfQC and JointVcfMultiQC (note, these are not terminal stages in any workflow, so we... don't run them in production)

Cohort -> MultiCohort
Corrections to imports
Only call self.expected_outputs() once
Use built-in self.prefix and self.web_prefix (these Stages seem to predate the builtin Stage methods)

All tests and linting pass locally for me

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

Project coverage is 78.42%. Comparing base (1126264) to head (af3cc50).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #955   +/-   ##
=======================================
  Coverage   78.42%   78.42%
=======================================
  Files          10       10
  Lines        1794     1794
=======================================
  Hits         1407     1407
  Misses        387      387

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


Numpy Concatation Error on S2COGREADER.part

Hey all, I am trying to use S2COGREADER to read part of a raster according to a bbox..
here is my geojson courtesy of geojson.io
geojson = {
  "type": "FeatureCollection",
  "features": [
    {
      "type": "Feature",
      "properties": {},
      "geometry": {
        "type": "Polygon",
        "coordinates": [
          [
            [
              35.013427734375,
              2.0869407308811065
            ],
            [
              37.46337890624999,
              2.0869407308811065
            ],
            [
              37.46337890624999,
              4.850154078505659
            ],
            [
              35.013427734375,
              4.850154078505659
            ],
            [
              35.013427734375,
              2.0869407308811065
            ]
          ]
        ]
      }
    }
  ]
}

I then use featureBounds to get the bbox and pass it to the reader with a band expression:
geom = geojson["features"][0]["geometry"]
with S2COGReader("S2A_29RKH_20200219_0_L2A") as sen:
    data = sen.part(bbox=featureBounds(geom), expression="(B03-B11)/(B03+B11)")

I get this error
ValueError: all the input array dimensions for the concatenation axis must match exactly, but along dimension 1, the array at index 0 has size 618 and the array at index 1 has size 555
I have tried different geometries and different ID numbers. Any help?

Maybe related to the fact that Band 3 has 10m resolution and Band 11 has 20m resolution? Try with B03 and B04?

Kyle, cheers. That seemed to be the problem.

I'd argue this is still a bug, or at a minimum there should be a better error message. What do you think @vincentsarago?

It's a tricky problem and will only happens if the bbox in the raw resolution result in a array > 1024px (I think)
The problem is that each band can have different resolution and we can't really now it in advance
To fix this you can either limit the 'max-size' to a smaller value or to fix the width and height.
We could maybe resolve this in the 'multi_array' function by resembling each array to the same size!

I'm surprised the arrays (size 618 and 555) aren't a factor of two

I reopened the issue, let me know if you want me to close it. I'll do what @vincentsarago suggested and report back

Why it should be factor of two @kylebarron?
This is for the 'part' method (would be the same for preview if one file is smaller than 1024px).

I don't know the internals of part well enough but I just assumed a factor of two because the resolutions are a factor of two apart

This is fixed in rio-tiler now.


Amplify previews build failed after @aws-amplify/cli 4.8.0

Hi!
We found that the Previews failed to build when the @aws-amplify/cli 4.8.0+. We have a project deployed in the Amplify console and use @aws-amplify/cli 4.5.0 last year. Our build settings using Amplify CLI latest version.
We found that the build failed in Previews when we submit new PR due to Current environment cannot be determined when using @aws-amplify/cli  is 4.11.0. We change the build setting back to @aws-amplify/cli 4.5.0 and the build is successful. After testing each different versions. We found that till @aws-amplify/cli 4.7.0 our build is successful and build is failed from version 4.8.0 to 4.11.0.
Please check below messages and let us know what changes from our end need to be done in order to use latest version. Thanks!
See below error message using 4.9.0.
                                 # Starting phase: build
                                 # Executing command: amplifyPush --simple
2020-01-10T14:34:17.925Z [INFO]: # Getting Amplify CLI Cloud-Formation stack info from environment cache
2020-01-10T14:34:17.935Z [INFO]: # Start initializing Amplify environment: devupdate
2020-01-10T14:34:17.937Z [INFO]: # Initializing new Amplify environment: devupdate (amplify init)
2020-01-10T14:34:18.400Z [INFO]: Note: It is recommended to run this command from the root of your app directory
2020-01-10T14:34:20.616Z [INFO]:
2020-01-10T14:34:20.621Z [INFO]: For more information on AWS Profiles, see:
                                 https://docs.aws.amazon.com/cli/latest/userguide/cli-multiple-profiles.html
                                 init failed
2020-01-10T14:34:20.625Z [INFO]: { UndeterminedEnvironmentError: Current environment cannot be determined
                                 Use 'amplify init' in the root of your app directory to initialize your project with Amplify
                                 at getEnvInfo (/root/.nvm/versions/node/v10.16.0/lib/node_modules/@aws-amplify/cli/lib/extensions/amplify-helpers/get-env-info.js:19:15)
                                 at AmplifyToolkit.getProjectDetails [as _getProjectDetails] (/root/.nvm/versions/node/v10.16.0/lib/node_modules/@aws-amplify/cli/lib/extensions/amplify-helpers/get-project-details.js:14:26)
                                 at Object.run (/root/.nvm/versions/node/v10.16.0/lib/node_modules/@aws-amplify/cli/node_modules/amplify-provider-awscloudformation/lib/amplify-service-migrate.js:9:42)
                                 at Object.run (/root/.nvm/versions/node/v10.16.0/lib/node_modules/@aws-amplify/cli/node_modules/amplify-provider-awscloudformation/lib/initializer.js:80:33)
                                 at process._tickCallback (internal/process/next_tick.js:68:7)
                                 at Function.Module.runMain (internal/modules/cjs/loader.js:832:11)
                                 at startup (internal/bootstrap/node.js:283:19)
                                 at bootstrapNodeJSCore (internal/bootstrap/node.js:622:3) name: 'UndeterminedEnvironmentError' }
2020-01-10T14:34:20.666Z [ERROR]: !!! Build failed
2020-01-10T14:34:20.761Z [ERROR]: !!! Non-Zero Exit Code detected
2020-01-10T14:34:20.761Z [INFO]: # Starting environment caching...
2020-01-10T14:34:20.762Z [INFO]: # Environment caching completed
Terminating logging...

See below message for the successful using 4.7.0
                                 # Starting phase: build
                                 # Executing command: amplifyPush --simple
2020-01-10T14:53:29.689Z [INFO]: # Getting Amplify CLI Cloud-Formation stack info from environment cache
2020-01-10T14:53:29.698Z [INFO]: # Start initializing Amplify environment: devupdate
2020-01-10T14:53:29.700Z [INFO]: # Initializing new Amplify environment: devupdate (amplify init)
2020-01-10T14:53:30.215Z [INFO]: Note: It is recommended to run this command from the root of your app directory
2020-01-10T14:53:32.701Z [INFO]:
2020-01-10T14:53:32.706Z [INFO]: For more information on AWS Profiles, see:
                                 https://docs.aws.amazon.com/cli/latest/userguide/cli-multiple-profiles.html
2020-01-10T14:53:32.718Z [WARNING]: - Initializing your environment: devupdate
2020-01-10T14:53:32.992Z [WARNING]: ✔ Initialized provider successfully.
2020-01-10T14:53:33.023Z [INFO]: The following types do not have '@auth' enabled. Consider using @auth with @model
2020-01-10T14:53:33.024Z [INFO]: - Collection
                                 - Archive
                                 Learn more about @auth here: https://aws-amplify.github.io/docs/cli-toolchain/graphql#auth
2020-01-10T14:53:33.165Z [INFO]: GraphQL schema compiled successfully.
                                 Edit your schema at /codebuild/output/src967976075/src/iawa_v2/amplify/backend/api/collectionarchives/schema.graphql or place .graphql files in a directory at /codebuild/output/src967976075/src/iawa_v2/amplify/backend/api/collectionarchives/schema
2020-01-10T14:53:33.170Z [WARNING]: - Updating resources in the cloud. This may take a few minutes...
2020-01-10T14:53:34.312Z [WARNING]: ✔ All resources are updated in the cloud
2020-01-10T14:53:34.314Z [INFO]:
2020-01-10T14:53:34.386Z [INFO]: Initialized your environment successfully.
2020-01-10T14:53:34.388Z [INFO]: Your project has been successfully initialized and connected to the cloud!
                                 Some next steps:
                                 "amplify status" will show you what you've added already and if it's locally configured or deployed
                                 "amplify <category> add" will allow you to add features like user login or a backend API
                                 "amplify push" will build all your local backend resources and provision it in the cloud
                                 “amplify console” to open the Amplify Console and view your project status
                                 "amplify publish" will build all your local backend and frontend resources (if you have hosting category added) and provision it in the cloud
                                 Pro tip:
                                 Try "amplify add api" to create a backend API and then "amplify publish" to deploy everything
2020-01-10T14:53:34.425Z [INFO]: # Environment devupdate details:
2020-01-10T14:53:34.926Z [INFO]:
2020-01-10T14:53:34.927Z [INFO]: devupdate
....
2020-01-10T14:53:35.464Z [INFO]: # Completed phase: build

@yinlinchen can you share your build settings?

Just experienced the same problem now. I had updated to 4.11 a week ago, and did a ton of builds, but now I have this error too.

@swaminator Please see the build settings below
version: 0.1
backend:
  phases:
    build:
      commands:
        - '# Execute Amplify CLI with the helper script'
        - amplifyPush --simple
frontend:
  phases:
    preBuild:
      commands:
        - yarn install
    build:
      commands:
        - yarn run build
  artifacts:
    baseDirectory: build
    files:
      - '**/*'
  cache:
    paths:
      - node_modules/**/*
test:
  artifacts:
    baseDirectory: cypress
    configFilePath: '**/mochawesome.json'
    files:
      - '**/*.png'
      - '**/*.mp4'
  phases:
    preTest:
      commands:
        - npm install
        - npm install wait-on
        - npm install<EMAIL_ADDRESS>mochawesome mochawesome-merge mochawesome-report-generator
        - 'npm start & npx wait-on http://localhost:3000'
    test:
      commands:
        - 'npx cypress run --reporter mochawesome --reporter-options "reportDir=cypress/report/mochawesome-report,overwrite=false,html=false,json=true,timestamp=mmddyyyy_HHMMss"'
    postTest:
      commands:
        - npx mochawesome-merge --reportDir cypress/report/mochawesome-report > cypress/report/mochawesome.json

Hello @yinlinchen , do you still face this issue?

Hi @Joycehao19, I didn't have this issue after I upgraded to Amplify/CLI 4.13+


Should en be tested instead of all?

In discussion of https://github.com/phetsims/QA/issues/330, it was noted that sims were tested with the all link, but sims are published with the en link.  We are looking into en sims on phet-dev will work, but either way, many developers wondered if testing en would work better.  At the moment, all is tested for consistency, but I am unaware of any other reason it would be preferred.  @phet-steele do you know any deeper reasons for this/any reason using en might be detrimental?

@phet-steele do you know any deeper reasons for this/any reason using en might be detrimental?

It was my understanding that _all.html is a better catch all? We use those versions in offline products (ie. the installer and the app [which would probably be used on an iPad Pro!!!]), and it shouldn't be different enough from sims built for specific locales, right?
If we are encountering an issue with the _all.html on one platform, does that really mean we should stop testing it all together? I would think we would either a). figure out why the _all.html is broken on the one platform and fix it, or b). just use the _en.html on the one platform and none of the others. Otherwise, this sounds like one of those sledgehammer and nail situations. What are advantages to testing only the .en_html?
Regarding point a)., I think this is exactly why we test the _all.html file: to find bugs with it. It's weird to me that we are advocating to change the testing protocol so quickly. This is similar to if we found a bug in a sim and just said, "Eh, let's just not test that version of the sim and ignore the bug".
Disclaimer: testing both of these versions wouldn't be "detrimental" to anything other than time.

To be clear, I don't think we should stop testing all or that the solution to it potentially not working is to stop supporting it.  A few developers thought it would be more useful to test en and I didn't have enough information to say why we use all.  Thanks for the input.

Sounds good, I think we can certainly continue testing with the _all.html, just wanted to know all the previous history and reasoning. @KatieWoe feel free to close


feat(api-reference): create workspace store in references

This is the prequel to merging auth. This just imports the client workspace store into references and starts using it for the modal directly. Allows us to delete a few other files like the event bus etc since we can now manipulate the workspace store directly
Previously we didn't worry too much about SSR on the client since it was client only, now the JS will be loaded in references so it will be SSR'ed. However the client is still initialized on mounted, only the JS is loaded in SSR. So I had to create the last router factory
I think down the road we can instantiate the workspace store earlier and only create the router on mounted. That should be SSR safe and would give us the workspace store throughout references. To do this we would need to split out the parts of the store that require a router as we cannot create the router on the server. We would then be able to access useWorkspaceStore in references. What do you think?

@coderabbit review


helm chart supports more repo related to observability

now helm chart support

grafana [metric/dashboard]
prometheus [metric]
jaeger  [trace]

Based on the discussion in slack, we plan to add more support

loki [log]
Tempo [trace]

Pending options

clickhouse [log/trace]

Can you add this to the Helm Charts repo?

Closing this to move discussion to #503


cli push: creating strings that shouldn't be

In a .ts file the following code is resulting in the creation of a new translation with string value "someInput".
import { Component, ElementRef, ViewChild } from '@angular/core';

@Component({
  selector: 'some-feature',
  templateUrl: './some-feature.html'
})
export class SomeFeature {
  @ViewChild('someInput', { static: true })
  someInputRef: ElementRef<any>;
}

@jdewit We released a patch for this issue in 2.2.0, can you check if it's working for you now?

Yeah, that fixed it. Thanks


"Sending fatal alert BadCertificate"

v1.0.0-x86_64-pc-windows-msvc

I've had this issue previously, seems like it originates from rustls. Does it happen everytime, or does it work sometimes?

it turned out the root cause is my company proxy. It works fine from normal network.

Ok, good to know. I'll close the issue. If this happens in some other way, feel free to reopen/open new one.


Dropdown component has keyboard accessibility issues

The Dropdown component doesn't work very well with the keyboard: in particular, when it's closed, then using the arrow keys leads to strange behaviour and will typically make the component crash with the following error:
Cannot read properties of null (reading 'scrollIntoView')

To reproduce:

Press the up/down arrow keys when the Dropdown is closed;
Press enter or space.

Here is a REPL.

Thanks for the detailed repro – fixed in v0.70.10.

@metonym Thank you for swiftly addressing this, as always.
I think the latest changes address the critical issue of the component crashing (reading 'scrollIntoView'), but the keyboard behaviour is still a bit mysterious.
On the same REPL, if you hit the up/down arrows on the Dropdown when it's closed, then hit enter, it will choose a new element, but it isn't really clear how this happens.
For example:

Suppose the currently selected element is A.
With the Dropdown closed, hit [up].
Hit [enter].

Now the selected element will be C: in other words, hitting [up] has jumped from the top to the bottom of the list. Furthermore, there's no indication of a change until the user hits [enter].

Similar examples to show how the behaviour is mysterious:
Again on the same REPL,

Start at A.
Hit [up] twice, then [enter].
Now the result is B.

or:

Start at A.
Hit [down] once, then [enter].
The dropdown opens.

etc.


Updating datastore URI template for v1beta3.

Also updating the docs link in the README.
NOTE: This should not be merged until v1beta3 is launched but it should be straightforward to review.
This partially crosses with #1330 (use of project in Connection.build_api_url)

LGTM, pending release of v1b3.

@tseaver It seems we can't change the target branch, so I'm closing this and will be re-sending against the v1beta3 branch.
Ref: https://github.com/isaacs/github/issues/18


Attrs linting error

I can't tell if this fixes anything. it changes the name of the project and adds linter and stuff like that. I see you've closed it anyway, but please make PRs that are easy for maintainers to choose to merge or not. especially, include a description of what it's intended to improve...

Hi @dominictarr I opened this PR by mistake (GH UI automatically opens PR against the upstream repo) and was not intended to introduce changes to this repo, which is why I closed it.


Enabled word_boundaries

In order to use in languages other than English (Chinese or Japanese with tokenized), it is now possible to select words to be separated by word boundaries instead of non-wordboundary when separating words .

Coverage decreased (-2.3%) to 97.049% when pulling 0ff4e77aec656e9b61a08abce94f999daee165fb on mkt3:master into 50c45f1f4a394572381249681046f57e2bf5a591 on vi3k6i5:master.


Rename babylon to babel

We are currently throwing a depreciation warning due to the recent changes in Babel.

Related: https://github.com/babel/babel/pull/7937
Renaming babylon to babel will solve this.

This is fixed the the latest alpha, 1.31.0-alpha.3, please install it via npm install -g prisma@alpha

This is available in the latest stable 1.31.1. Please install this via npm install -g prisma.
I am closing this issue, please open a new issue again if necessary.


_389-ds-base: 2.4.2 -> 2.4.3

Description of changes
https://github.com/389ds/389-ds-base/compare/389-ds-base-2.4.2...389-ds-base-2.4.3
This fixes a compilation error against Rust 1.71 in staging-next (https://github.com/NixOS/nixpkgs/pull/245935).
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Duplicate of #247324.

Whoops, I missed that. Thank you!


Scrolling input field

Closes #94

Ok. Now selecting all of the last query on open doesn't work anymore. Someone should check up on that.