The report of clocking summary is not right aligned for double width characters.

Describe the bug

There is the same issue with vim-dotoo, the report of clocking summary is not right-aligned for double width characters.
See vim-dotoo issue
Steps to reproduce

* TODO [#A] Chinese for AAA 中文
  SCHEDULED: <2022-04-25 Mon 11:30-12:30 +1w>
  :PROPERTIES:
  :LAST_REPEAT: [2022-04-19 Tue 04:45]
  :END:
  :LOGBOOK:
  CLOCK: [2022-04-25 Mon 21:13]--[2022-04-25 Mon 21:14] => 0:01
  :END:
  - State "DONE" from "TODO" [2022-04-19 Tue 04:45]

* TODO [#A] English for BBB
  SCHEDULED: <2022-04-25 Mon 14:00-15:00 +1w>
  :PROPERTIES:
  :LAST_REPEAT: [2022-04-19 Tue 04:45]
  :END:
  :LOGBOOK:
  CLOCK: [2022-04-25 Mon 21:13]--[2022-04-25 Mon 21:14] => 0:01
  :END:
  - State "DONE" from "TODO" [2022-04-19 Tue 04:45]

Expected behavior
Align double-width characters as single-width characters.
Emacs functionality
No response
Minimal init.lua
Any init.lua.
Screenshots and recordings
No response
OS / Distro
Ubuntu 20.04/ Windows 10/ Mac OS
Neovim version/commit
0.6
Additional context
No response

Should be fixed now.


[Mbtiles Support]

Issue Type
Issue Type
[✔] Question
[✔ ] Bug
[✔] Improvement
[✔ ] Documentation
Description
hi mate. osmdroid version 6.1.18, Load mbtiles files not created by mobac, and the map will show blank when opening it.
1.Can the document indicate the versions supported by mbtiles?
2.Can it support loading mbtiles files of other spec versions?
3.If mbtiles files of different spec versions are converted into .map or gpkg, does osmdroid support loading?
load mbtiles file code samples:

           val simpleRegisterReceiver = SimpleRegisterReceiver(context)
            val tileProvider = OfflineTileProvider(simpleRegisterReceiver, fileArray)
            mapView.setTileProvider(tileProvider)

            val archives = tileProvider.archives
            if (archives.isNullOrEmpty()) {
                mapView.setTileSource(TileSourceFactory.DEFAULT_TILE_SOURCE)
                return
            }
            val tileSources = archives[0].tileSources
            archives.forEachIndexed { index, iArchiveFile ->
                LogUtil.e(tag, "addMultipleMapFile index:$index iArchiveFile:${iArchiveFile.tileSources}")
            }
            if (tileSources.isNotEmpty()) {
                val source = tileSources.iterator().next()
                mapView.setTileSource(FileBasedTileSource.getSource(source))
            }

result tileSources.isNotEmpty() always false. Map display white.

My mbtiles file is opened with the database sql as follows:

Version of osmdroid the issue relates to:
6.1.18
best wish.

does this help?
https://github.com/osmdroid/osmdroid/blob/master/osmdroid-android/src/main/java/org/osmdroid/tileprovider/modules/MBTilesFileArchive.java#L19

@spyhunter99 thank you mate, I found the reason why it cannot be loaded normally. The internal format of mbtiles is different. Files containing metadata and tiles fields in version 1.1 can be displayed normally.
My mbtiles file does not contain a tiles table.
MBTilesTileSource mvtTileSource = new MBTilesMvtTileSource(filePath);

mapsforge-vtm loaded normally.

I think it should be the internal data reading part of MBTilesTileSource that is different, I will refer to it.
Does osmdroid support custom reading of database internal fields?

@spyhunter99 mate, how custom MBTilesFileArchive?  ArchiveFileFactory extensionMap cannot directly remove the "mbtiles" type.

@spyhunter99 It was a problem with the mbtiles file. I re-downloaded the mbtiles file and it works normally now.

hi mate, are vector mbtiles map files supported? @spyhunter99


Non linear blend Tensor impl - HOST

Adds Non linear blend tensor implementation on HOST
Adds relevant unit/perf tests

@rrawther @kiritigowda @r-abishek This PR passes all relevant unit tests.

@r-abishek can you resolve conversations on the PR ?


[HOLD for payment 2024-03-06][$500] [Simplified Collect][Categories] WorkspaceCategoriesPage: base page implementation, displaying categories and its statuses

Implement this section of the Simplified Collect: Profile, Members, Categories design doc.
Issue OwnerCurrent Issue Owner: @ArekChr
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~014e6c1796715bbf98
Upwork Job ID:<PHONE_NUMBER>413355521
Last Price Increase: 2024-02-08

@ArekChr

Sure, please assign me @luacmartins

Not overdue

The draft PR is nearly complete. @luacmartins @jliexpensify, could you please review my latest update in the PR? The issue mentioned in comment is currently a blocker, thanks!

Pulled the Q out to Slack

Yes, it's done

Yeah, we need to switch @parasharrajat for @rushatgabhane for the C+ tracking.

@jliexpensify @luacmartins I have an automatic issue to issue payment for the PR that's linked to this issue. Do we need to issue one or two payments to Rushat for https://github.com/Expensify/App/pull/36324?

@sonialiap thanks for checking. We should only issue one payment for @rushatgabhane's PR review. I think we can close the internal issue in favor of this one.

Thanks! Makes sense to keep it all in one place 👍 Closing out the other issue

Payment Summary

Contributor: @ArekChr (Invoiced)
-C+: @rushatgabhane $500 (New.Dot)

If this is correct, I'll remove the Upworks job. @rushatgabhane anything need to be done for the checklist?

$500 payment approved for @rushatgabhane based on summary.

Upworks job removed, bump on @rushatgabhane for the checklist (if applicable).

@jliexpensify it's a new feature.
So I recommend that we add the test steps in regression suite.

Navigate to the workspace settings as an admin within a paid policy group.
Verify that you can access the categories page.
Verify that categories are listed.
Confirm that you can select all categories by clicking on 'name', and deselect them by clicking on 'name' once more.
Access workspace settings as a member with a paid group policy.
Replace the sub-path /profile with /categories in the URL.
Verify that a 'not found' screen is displayed on the central page.
Repeat steps 4 through 6 as both an administrator and a member under a free policy.
Confirm that you are unable to access the categories page.

Payment Summary:

Contributor: @ArekChr (Invoiced)
C+: @rushatgabhane $500 (New.Dot)


JCon reports back to JOSS reviews.

Instead of reporting that the pipeline failed on the JCon review repository the failure get's reported in joss-reviews.
https://github.com/JuliaCon/proceedings-papers/issues/64#issuecomment-1302437320

This happens because the workflows are configured with the wrong values for repositories and journal data.
For example: GH_REPO here should point to the JCon repo, and journal here and branch_prefix should be jcon instead of joss.
Same with the other workflows.


Feature Request: bosh "archive" deployment functionality

Sometimes its not clear if one of our deployments is a duplicate or something that was used for testing and is no longer needed.  It would be great if we could archive deployments before completely deleting them.  We are using vSphere, but other environments should be able to do something along these lines.
The proposal is to make bosh archive suspend the VMs in their current state saving everything in memory including containers to disk (vmware does this natively, but other hypervisors offer similar capabilities).  Once all the VMs in the deployment are suspended, the deployment effectively frees up its reserved IP range and enables us to deploy over it.  In the event that we need to bring this environment back, we should be able to free up the subnet block allocated to the archived deployment and start it back up without modifying the runtime config, loosing any data and changing the network config of the deployment.

Hey @kitsirota it doesn't look like this issue has gotten much traction. This isn't a feature which is on our near/mid-term roadmap right now, so I'm going to go ahead and close this. If you or others have more thoughts on how this might work in the future, please let us know.


[Feature] Support MinkowskiUnet for semantic segmentation

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
It is a continued integration of MinkowskiEngine framework for sparse tensors processing (#1422). This PR includes implementation of the semantic segmentation methods and color augmentations described in 4D-SpatioTemporal ConvNets: Minkowski Convolutional Neural Networks, CVPR'19.
Modification
Add MinkowskiUnet instance segmentation module and its submodules like:

segmentors/sparse_encoder_decoder.py --> SparseEncoderDecoder3D [inherited from EncoderDecoder3D module]
backbones/mink_unet.py --> MinkUNetBase [inherited from nn.Module module]
decode_heads/mink_unet_head.py --> MinkUNetHead [inherited from Base3DDecodeHead module]

Add color augmentations for point clouds like:

pipelines/transform_3d.py --> YOLOXHSVPointsRandomAug [inherited from YOLOXHSVRandomAug module]
pipelines/transform_3d.py --> ChromaticJitter (originally taken from the code for the paper)

BC-breaking (Optional)
MinkowkiEngine support. Updated Dockerfile was also added.
Use cases (Optional)
Model declaration looks like this:
model = dict(
    type='SparseEncoderDecoder3D',
    voxel_size=0.05,
    backbone=dict(
        type='MinkUNetBase',
        depth=18,
        in_channels=3,
        planes='A',
        D=3,
    ),
    decode_head=dict(
        type='MinkUNetHead',
        channels=96,
        num_classes=20,
        loss_decode=dict(
            type='CrossEntropyLoss',
            class_weight=None,  # should be modified with dataset
            loss_weight=1.0,
        )))

Augmentations can be included into a common pipeline:
# scannet semantic segmentation dataset example
train_pipeline = [
    dict(
        type='LoadPointsFromFile',
        coord_type='DEPTH',
        shift_height=False,
        use_color=True,
        load_dim=6,
        use_dim=[0, 1, 2, 3, 4, 5],
    ),
    dict(type='LoadAnnotations3D',
        with_bbox_3d=False,
        with_label_3d=False,
        with_mask_3d=False,
        with_seg_3d=True,
    ),
    dict(
        type='PointSegClassMapping',
        valid_cat_ids=(1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 14, 16, 24, 28,
                       33, 34, 36, 39),
        max_cat_id=40,
    ),
    dict(
        type='IndoorPatchPointSample',
        num_points=num_points,
        block_size=1.5,
        ignore_index=len(class_names),
        use_normalized_coord=False,
        enlarge_size=0.2,
        min_unique_num=None,
    ),
    dict(
        type='YOLOXHSVPointsRandomAug', # example of new augmentation
        hue_delta=2,
        saturation_delta=15,
        value_delta=15,
    ),
    dict(
        type='NormalizePointsColor',
        color_mean=None,
    ),
    dict(
        type='GlobalRotScaleTrans',
        rot_range=[-0.087266, 0.087266],
        scale_ratio_range=[.9, 1.1],
        translation_std=[.1, .1, .1],
        shift_height=False,
    ),
    dict(
        type='RandomFlip3D',
        sync_2d=False,
        flip_ratio_bev_horizontal=0.5,
        flip_ratio_bev_vertical=0.5,
    ),
    dict(type='DefaultFormatBundle3D', class_names=class_names),
    dict(type='Collect3D', keys=['points', 'pts_semantic_mask']),
]

Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects.
The documentation has been modified accordingly, like docstring or example tutorials.

@HuangJunJie2017 @lindahua plz help me with readthedocs build failed problem. I dont know what to do with this

Hi @Ilyabasharov , thanks for your awesome work! The doc build error is really strange. We will fix the mmcv version problem in the new version release and then have a look at the doc error. Please first fix the preliminary comments and resolve the conflict.

Hi @Ilyabasharov ！We are grateful for your efforts in helping improve mmdetection3d open-source project during your personal time.
Welcome to join OpenMMLab Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us ：https://discord.gg/UjgXkPWNqA
If you have a WeChat account，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
Thank you again for your contribution❤


tags: show tag on pages when tag has space

when I tagged "Github Pages" on my blog, but it does't show on my pages.

Thanks for reporting. This is a caused by #378 and will be fixed soon.

Pushed to the main branch. This will be contained in the next release


Deprecate method that does unexpected things and document alternative

Also minor cleanup.

I started down that path initially, then attempted to write a test for it, which got me thinking about the use cases for it. There is no case today where we call that method with a non-null referencedTypeState, and it doesn't quite fit that a HollowObjectSchema gets a reference to a type state for another type while you're adding a field to it. It makes more sense that the type states get set when deserializing a snapshot with a full type state, and that is already exposed by the public setReferencedTypeState.
On a side note, AFAICT the referenced type state being accessible from the schema is really just a shortcut for internal use - I'm not convinced it's a good idea to have a reference to the data from the schema.

That makes sense, i suspect no one uses this method, and I agree with you about the binding of type state to the schema.


Startup: Python error in main script

After install on first Install, GUI pops up with an error  at the same time, closing the error causes the GUI to disappear and any launches after that have an icon pop up on the taskbar for a second to close immediately after. Uninstall and reinstall leads to the same results.
Guava Version
V1.0.0-Beta

I'm aware of this bug, this is not a problem with my program, it's the library I'm using to compile the code into an executable file, It'll be solved in the next update since I'm gonna move away from cx_Freeze because it has tons of bugs like that, I will add a requirements.txt for the time being for users to able to use the source code until I fix the compiled version and release the new version which will have few more features and will be much more stable.
If you want to use the source code for now and you need any assistance please message me on Discord and I'll help you.

I'll leave this open until I release the new version

[ ] Fix the compiled version in the next release

@karimawii how to fix?
ERROR: Could not find a version that satisfies the requirement pywin32==306 (from versions: none)
ERROR: No matching distribution found for pywin32==306

What did you do for this to happen?

What did you do for this to happen?

I will try to build from source code

can you do:
pip install pywin32

can you do:
pip install pywin32

you're not on windows right?

you're not on windows right?

github codespaces

Well, Codespaces is based on Linux, the while thing won't be able to work on there.

If you're on a Windows device try running it locally.

If you're on a Windows device try running it locally.

I need to clear up the requirement.txt, I'm sorry, it has a ton of extra modules that you won't be using, just delete the line where it says notify-run=0.015 and you should be good to go, you won't need notify-run.

I need to clear up the requirement.txt, I'm sorry, it has a ton of extra modules that you won't be using, just delete the line where it says notify-run=0.0.15 and you should be good to go, you won't need notify-run. 35aa849

so much conflicts =/

oh god https://github.com/karimawii/Guava/commit/4423ac64c7f1ef6aa79196ac4491052eda1b2df3, I hope this solves it

oh god 4423ac6, I hope this solves it

https://github.com/karimawii/Guava/commit/d0b3bbf73d46ccdc5fcdf2ac2e4df0d5d49206ca I think we'll just keep doing this until 0 conflicts, I don't have the proper tools to test and resolve all of the conflicts atm

d0b3bbf I think we'll just keep doing this until 0 conflicts, I don't have the proper tools to test and resolve all of the conflicts atm

There's an open issue for that exact problem over on athrvvvv's AppOpener repo

I temporarily and very quickly fixed the issue in another branch by putting the client path manually in the code at line 1513 replace the existing path with you LeagueClient.exe path following the same format

@Suburbanno refer to #69 for solving this

There's an open issue for that exact problem over on athrvvvv's AppOpener repo, but they don't seem to have figured it out yet, he said it'll be solved with v1.7 but he haven't released it yet

Actually, releasing AppOpener v1.7 this month!!! 🤝👍

@athrvvvv YAY! I will also rewrite this to be somewhat object-oriented not just a messy 2000+ lines spaghetti kingdom. Thanks for taking the time 🤝

AppOpener v1.7 just released!! ✌
@Suburbanno Upgrade AppOpener like:
pip install AppOpener==1.7

I have added an Exception, if the File creation process failed, it will create files using the old method in AppOpener v1.3
Hope soo, this works!! If your Issue is fixed, you may close this now 😊
Lots of love,
athrv ❤


updated star pattern program in C

this program prints left pointing arrow in C.

@AdarshAddee kindly review my PR


Detect which column a div is on

Thanks for submitting an issue to RGL!
Please mark the type of this issue:

[ ] Bug
[ ] Feature Request
[X] Question

If you have a question or bug report, please use the WebpackBin Template
to demonstrate. It is much easier for us to help you if you do.
Working on a thing to help roommates coordinate chores.

Is there a way to see which column a particular chore is on? For example, I have three chores under the "deck" column/column 0, and five chores under the "sunday" column/column 1. I would like to be able to detect which chore is assigned to which day based on the column number.
Thanks!

Yes, use the callbacks as described here. In particular, you can listen to onLayoutChange and correlate an item's i (which is an id) to its position.


[CARBONDATA-4189] alter table validation issues

Why is this PR needed?

Alter table duplicate columns check for dimensions/complex columns missed
Alter table properties with long strings for complex columns should not support

What changes were proposed in this PR?

Changed the dimension columns list type in preparing dimensions columns [LinkedHashSet to Scala Seq] for handling the duplicate columns
Added check for throwing an exception in case of long strings for complex columns

Does this PR introduce any user interface change?

No

Is any new testcase added?

Yes

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12602/job/ApacheCarbonPRBuilder2.3/5392/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12602/job/ApacheCarbon_PR_Builder_2.4.5/3647/

retest this please

Build Failed  with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12602/job/ApacheCarbon_PR_Builder_2.4.5/3649/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12602/job/ApacheCarbonPRBuilder2.3/5394/

retest this please

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12602/job/ApacheCarbonPRBuilder2.3/5402/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12602/job/ApacheCarbon_PR_Builder_2.4.5/3658/

LGTM

retest this please

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12602/job/ApacheCarbonPRBuilder2.3/5415/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12602/job/ApacheCarbon_PR_Builder_2.4.5/3672/


Update for phase 2

Context
Changes proposed in this pull request
Guidance to review

Closing in favour of #192.


How to load the Mali GPU device driver?

There is first step in original readme Load the Mali GPU device driver. But how is it possible to do for example on mainline kernel 4.14 on H3 board with Armbian? modprobe mali say that there is no such module. In case if such module should be build by hands, could you please share any useful documentation/links, please?
Thanks.

Sorry but only in Italian.
See this guide for both kernel and userspace drivers
http://abiuplayer.altervista.org/compilare-i-drivers-kernel-e-userspace-per-gpu-mali-su-allwinner-h3-e-h5/


Readme

README
User Story
As a boot camp student,
I want the prework notes to be structured on a webpage,
So that I can easily find and read the information.
Acceptance Criteria
GIVEN the prework project repository,
WHEN I view the README for the repository,
THEN I can see a professional README that provides information on the project and how to use it

completed module 3.5.3 on 07-30-24


AttributeError: 'dict' object has no attribute 'sort' in /examples/image_to_annotations.py

Hi, when I tried to run my own image by setting up the container and following the tutorial below, it shows an error message.
(animated_drawings) torchserve % cd ../examples
(animated_drawings) examples % python image_to_animation.py drawings/garlic.png garlic_out
Here is the error message:
Traceback (most recent call last):   File "/Users/chen_yenru/Documents/GitHub/DATASCIENCE/ML/AnimatedDrawings/examples/image_to_animation.py", line 41, in <module>     image_to_animation(img_fn, char_anno_dir, motion_cfg_fn, retarget_cfg_fn)   File "/Users/chen_yenru/Documents/GitHub/DATASCIENCE/ML/AnimatedDrawings/examples/image_to_animation.py", line 19, in image_to_animation     image_to_annotations(img_fn, char_anno_dir)   File "/Users/chen_yenru/Documents/GitHub/DATASCIENCE/ML/AnimatedDrawings/examples/image_to_annotations.py", line 59, in image_to_annotations     detection_results.sort(key=lambda x: x['score'], reverse=True)     ^^^^^^^^^^^^^^^^^^^^^^ AttributeError: 'dict' object has no attribute 'sort'
I think it's because of Python's version difference, where .sort() is only for Python 2.
Yet, when I change it to sort(detection_results, key=lambda x: x['score'], reverse=True), it then returns a type error on the indices.
Here is the error message:
Traceback (most recent call last):   File "/Users/chen_yenru/Documents/GitHub/DATASCIENCE/ML/AnimatedDrawings/examples/image_to_animation.py", line 41, in <module>     image_to_animation(img_fn, char_anno_dir, motion_cfg_fn, retarget_cfg_fn)   File "/Users/chen_yenru/Documents/GitHub/DATASCIENCE/ML/AnimatedDrawings/examples/image_to_animation.py", line 19, in image_to_animation     image_to_annotations(img_fn, char_anno_dir)   File "/Users/chen_yenru/Documents/GitHub/DATASCIENCE/ML/AnimatedDrawings/examples/image_to_annotations.py", line 59, in image_to_annotations     sorted(detection_results, key=lambda x: x['score'], reverse=True)   File "/Users/chen_yenru/Documents/GitHub/DATASCIENCE/ML/AnimatedDrawings/examples/image_to_annotations.py", line 59, in <lambda>     sorted(detection_results, key=lambda x: x['score'], reverse=True)                                             ~^^^^^^^^^ TypeError: string indices must be integers, not 'str'
I'd like to ask how to overcome this problem?
Thanks!

Hello,
.sort() is supported in Python 3, so that's not the issue.
The issue is that the response from the torchserver within the docker contain is not in the format expected by the script. Can you inspect or debug the contents of detection_results in the line before the error occurs?

I have the same issue. The problem is that the prediction was failed on the torchserver side. Here is the content of detection_results :
{'code': 503, 'type': 'InternalServerException', 'message': 'Prediction failed'}

Were you using the example garlic image or something else?
This issue from the pytorch github is related to torchserve error debugging https://github.com/pytorch/serve/issues/215
Debugging torchserve is outside my scope of support for now, but if you figure out the cause / resolution please share it.

In my case, switching back to CPU solved the issue. It seems there are some problems related to using GPU on the torchserve side.

Hi thhung, may I ask how you switch back to CPU? Did you mean specifying in Pytorch to use only the CPU?
Regarding that, I'd like to ask whether it's to type export CUDA_VISIBLE_DEVICES="" in the shell?
Thanks!

@chenyenru I think you can tried export CUDA_VISIBLE_DEVICES=" to see if it works. I simply turnoff the connection with GPU to force torchserve use only CPU. Let me know if you manage to do it. You can try some config on the torchserve side. I did but it doesn't work for me.

the same issue.

Solved it after setting docker

Hardware: M2 max .
Python:3.11.2
MacOS

@zsinba Mine was different with yours. Mine have the 503 error code. I think it is related to GPU implementation. Your case probably is related to the resources.

I don't think it has anything to do with GPU. The M2 Max with  mps,  but not GPU;
And I didn't set the parameters. I guess he still used the cpu

it's ok


[Image] Add headers prop in source array type prop

My first contribution 👍
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Bonus points for screenshots and videos!)

If more tests are required for this, will add it as well 😄


Update config files for navigation in simulation

I am opening this draft PR for discussion on how to handle some changes
Features:

added support for Rviz visualization of the robot model
added realsense and lidar nws config files for ros2
remappers for all the joint states for ros2 TF2 robot-state-publisher package
removed manually contacts on all the robot links except for the soles (could be handled differently creating a new model)
changed some lidar values in the urdf to match the real sensor and sat the visualization to false
added all the remaining finger joints of the hands, in the left/right_hand_hardware_device for showing properly the robot model on Rviz (without any errors)

I have reverted the changes of the joints on the hands.
However, just for information, Rviz will not display correctly the hands and throw some errors/warnings, as in this pic:

If someone will need Rviz for some demos, he should be careful to hide the warnings and errors, since it's not pretty.

I have reverted the changes of the joints on the hands.
However, just for information, Rviz will not display correctly the hands and throw some errors/warnings, as in this pic:

This is a known issue, not ergocub-specific, but that you can also find on iCub. Right now on joint_states we are publishing only actuated joints, the tf in order to be computed need the values of the physical joints.
The hands are in general underactuated systems, where actuated joints > phisycal joints.
We are missing a SW layer for handling coupling and decoupling and publish both kind of data, see this discussion https://github.com/orgs/robotology/discussions/609.
If you don't need them you can ignore for now those errors/warnings

Yes the errors could be ignored, so can I convert this draft to PR?

Yes the errors could be ignored, so can I convert this draft to PR?

This point as still to be addressed, have you investigated about the possibility of removing collision to all parts in the generation of the urdf?

@SimoneMic feel free to put it "Ready for review" as soon as the generation of ergoCubGazeboV1_noContacts work 👍🏻
I would call it ergoCubGazeboV1_minContacts or something like that, because if I am not wrong the contacts in the feet are preserved

Now it should be able to auto merge and I have removed the changes to the URDF of ergoCubGazeboV1

Now it should be able to auto merge and I have removed the changes to the URDF of ergoCubGazeboV1

Unfortunately seems that the conflicts are still there, but only the author of the PR can manage them
Could you also remove from the PR the model.urdf of ergoCubGazeboV1_mincontacts? It should be automatically created by the automatic github action

Probably the easiest thing is to squash everything in just one commit, because you added and then removed the urdfs, that in the meanwhile have been changed in origin.

I can't squash, I'll open a new one

Closing, moved to #92


updates for cross domain

I have run this file for single file , it works. But I can not run it for cross domain, as I have no data for cross domain. I try to download it from spieceasi github, but the data is not in tsv format. I hope this will give output for cross domain if the data is in tsv format.

I just took the example data from HMP2 website, but feel free to export the phyloseq objects to TSV if you need for testing, or I could supply them if you'd like.


list of heterogeneous datatypes

test = [1,2,3,"dog"] results in:
Exception: Not a homogeneous array

This is a long standing issue with TOML. Per the spec, arrays can not hold multiple types. There have been proposals to add tuples to the spec but I do not believe that any have made it in yet.
On Wed, Jan 28, 2015 at 06:25:34AM -0800, Darin wrote:

test = [1,2,3,"dog"] results in:
Exception: Not a homogeneous array

Reply to this email directly or view it on GitHub:
https://github.com/uiri/toml/issues/27


use flash attn fuse cross entropy loss to reduce metric memory usage

This PR uses fused cross entropy loss from flash attention in the metric LanguageCrossEntropy (also LanguagePerplexity).
The current torch.nn.CrossEntropyLoss call needs 6 * seq_len * vocab_size GPU memory, and can be the bottleneck memory usage when sequence length is long (where act ckpt is probably used). Using  cross entropy loss from flash attn resolves this problem.
Example test model with long sequence and full act ckpt:
with torch loss fn:

with flash_attn loss fn

@cli99 consider doing it as in https://github.com/mosaicml/llm-foundry/pull/575/files, to avoid introducing flash attn as a composer dependency.

@cli99 should we close or is this still WIP


cmd/link: accepts nonsense ldflags

I don't think this should succeed:
~/go/bin/go build -ldflags=a=b hello.go  && ./hello
hi

I haven't checked the code, but from strace it looks like go build is swallowing ldflags that don't start with a -.

The syntax -ldflags=a=b means "pass b to all packages that match the pattern a".  If there are no packages that match the pattern a, then the flag will never be passed.

Oh. That makes sense, I just didn't even consider that that syntax would apply to ldflags. I'm not sure what good it is, but it's not a bug. Thanks.


[FEATURE REQUEST] chain_property_optional()

So that in a chain like this:
self.tab_view
    .property_expression("selected-page")
    .chain_property::<adw::TabPage>("child")
    .chain_property::<Page>("scale")
    .chain_closure::<String>(closure!(|_: Option<glib::Object>, scale: f64| {
        format!("{:.1}%", scale * 100f64)
    }))
    .bind(&*self.scale_entry, "text", None::<&adw::TabView>);

I can use chain_property_optional(), receive scale: Option<f64> into the final closure and handle the selected-page is None case. Kind of an and_then() for expressions.

cc @SeaDve

This can't be done, any closure expression will never evaluate if a property expression resolves to null

This is a general concern with property expressions BTW, you have to be careful not to let any of the objects become null or the view will become stale

I suppose we can close this for now until something changes upstream.


agent: add several new flags to control Cilium's datapath events notifications

This commit introduces three new configuration flags for the Cilium agent, allowing users to choose the bpf event types they want to expose to Cilium monitor and Hubble.

--bpf-events-drop-enabled                                   Expose 'drop' events for Cilium monitor and/or Hubble (default true)
--bpf-events-policy-verdict-enabled                         Expose 'policy verdict' events for Cilium monitor and/or Hubble (default true)
--bpf-events-trace-enabled                                  Expose 'trace' events for Cilium monitor and/or Hubble (default true)

The default values for these flags remain set to true, not changing the current behaviour.
In our case, we found particularly useful to disable the TraceNotification in order to reduce the CPU overhead on some of our nodes when Hubble is enabled as we were mostly interested into dropped packets.
agent: add several new flags to control Cilium's datapath events notifications

You might need to run  make -C Documentation update-cmdref (again) so that CI is happy :)

indeed, thanks! 🙈

Just realizing that I missed out on @ericmountain's draft PR https://github.com/cilium/cilium/pull/30031
@joestringer, are you happy with the way it looks or would you prefer if we pursue a more generic command-line option?

Thanks for the PR. My gut feeling is it'd be simpler to have a single notify flag that allows defining which notifications to enable (with a default that enables all notification types). I see that @kaworu has expressed interest in this PR and I think he could have some useful input here as well.

@kaworu I like that proposal. This should enable us to subsequently add more specific configurations for each event type.
@mvisonneau What do you think about the Helm configuration proposed by @kaworu above?
One more thing, this PR describes the change as modifying the behaviour for endpoint event notifications, but I think that's incorrect. I believe this impacts all datapath event notifications (via daemon.GetOptions). Please double-check and consider renaming + changing the release note and PR description to reflect this.

👋 hey, thanks a ton for the feedback and additional context! I did my best to update it towards the direction you suggested. I assumed that as we are potentially leaning towards per event type, rate limit capabilities, it would be best to keep the several flags 🤔. (it also seemed to align with the suggested helm implementation). Happy to make further improvements as you feel might be necessary though! 😄

I've squashed the commits 👍

/test

PR looks good. Great work @mvisonneau
I guess as a follow up, the filter added in https://github.com/cilium/cilium/pull/24828 can be deprecated and further removed.

Thanks for the PR. My gut feeling is it'd be simpler to have a single notify flag that allows defining which notifications to enable (with a default that enables all notification types). I see that @kaworu has expressed interest in this PR and I think he could have some useful input here as well.

@joestringer The current implementation for enabling different event types is not unified, and changing it into a single config flag is not a straight forward thing to do. i.e. take a look at https://github.com/cilium/cilium/pull/27885 - I started it some time ago but didn't have time to follow up and finish the implementation.
I believe the best approach is to move forward taking smaller steps and start with merging this as is and only after that take another step to change the configuration API.

I guess as a follow up, the filter added in https://github.com/cilium/cilium/pull/24828 can be deprecated and further removed.

Note that https://github.com/cilium/cilium/pull/24828 allow to disable some user-space events that we can't disable on the BPF side (e.g. MessageTypeAgent) and other that cannot (yet) be disabled from BPF (e.g. MessageTypeTraceSock).
So before deprecating / removing the --hubble-monitor-events flag let's discuss further which events we want to be able to disable long-term.

I believe the best approach is to move forward taking smaller steps and start with merging this as is and only after that take another step to change the configuration API.

I'm fine with this, it matches better with the Helm configuration proposal and configuring multiple aspects of each event category.

The PR already picked up conflicts that need to be resolved :/

argh indeed, I just rebased, hope CI will still pass! 🤞

/test

This needs another rebase, sorry.

apologies for the delay, it should now be rebased! :)

/test

/test

@mvisonneau ipsec upgrade can be flakey, and seems unlikely to be related to these changes.
For the Travis failure, it looks like a similar flake issue/fix was closed recently .  Perhaps try rebasing to ensure your branch has those changes?

thanks for your guidance @tommyp1ckles, I've once again rebased! 🤞

/test

Heads up that the main blocker on this PR is getting the required jobs to pass. The L4LB test has been retriggered five times over the past couple of weeks and seems to be consistently failing in this way:
curl: (28) Failed to connect to <IP_ADDRESS> port 80 after 135242 ms: Connection timed out
+[18:53:45] echo 'Failed 1'
Failed 1
+[18:53:45] exit -1
Error: Process completed with exit code 255.

It's not clear to me why this PR would be triggering such a failure, but it's also hard for us to ignore a consistent signal from CI that the tests are failing and then merge the PR. It may be worth looking around in the issues to see if there are similar failures reported by others, and worst case rebase again to see if it was a problem already fixed by another PR in the main branch.
And yes I recognize that asking to rebase again is a bit tedious, unfortunately it looks like this PR may have been adversely impacted by the elevated background failure rate in the tree since late Jan / early feb (via the CI dashboard). We're gradually improving some of the individual workflows that have been failing more often, but a couple of the workflows still seem to have elevated failure rates:

👋 hey @joestringer, thanks for the feedback! I've just stumbled upon https://github.com/cilium/cilium/issues/31167, going to attempt rebasing as well as I am also unsure where to start looking otherwise 🙈

/test

seems like it did the trick! 🙌

@mvisonneau thanks for the contribution!


Launch Meltano v1.0

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1096
Originally created by @dmor on 2019-10-06 23:32:42

[x]  @dmor draft announcement email for newsletter subscribers
[x]  send announcement email via Substack
[x]  @dmor draft v1.0 announcement blog post
[x]  publish on Meltano.com blog
[x]  publish on Meltano Medium blog
[x]  Tweet blog post
[x]  Publish post on LinkedIn

View 10 previous comments from the original issue on GitLab


Return clear dial errors

It is just WithBlock?

yes it is. Unit test emulates problem behavior. Explained in the commit


Hope to integrate mqtt-cli command line mode, support import and export commands, similar to postman's export API

As the title, I hope to add this function so that a use case can be formed for testing.

Hello, thanks for your suggestion. The following is my point:

But I think MQTT X is originally a graphical tool, and the newly added CLI mode is not incompatible with software design

Currently MQTTX already supports import and export functions, the current version only supports JSON files, the next version already supports XML, CSV and Excel.

We can continue the discussion here.

Thank you for your support of MQTTX, I hope your problem has been solved. Next, the import and export of payload will be implemented. I will close this issue. :)

Thank you for your support of MQTTX, I hope your problem has been solved. Next, the import and export of payload will be implemented. I will close this issue. :)


[Bug]: Three players joining a game simultaneously can cause all three to join

Bug Summary
A race condition in requests to join a game can cause too many players to join the game because the game doesn't appear full at the time the multiple requests to join are processed. We should fix this by utilizing the new lock-game helper in the subscribe game endpoint.
Detailed Description
#783 introduced the lock-game and unlock-game helpers which the backend can use to force requests dealing with a particular game are processed in sequence, rather than in parallel. #783 applies these helpers to the ready.js api endpoint in order to prevent race conditions in requests to ready/unready for a game. We should apply the same locking/unlocking the game techniques in api/controllers/game/subscribe.js to prevent a similar race condition in the subscribe action.
Current Behavior
Multiple players joining a game simultaneously sometimes causes all of them to 'successfully' join, resulting in more than 2 players in the game.
Expected Behavior
Only two players should be able to join a game at a time.
Steps to Reproduce

Sign in with three accounts in three windows
Create a game
Join it as one player
Hit the join button with the two other players at the same time. They might both be admitted into the game, resulting in 3 total players in the game, instead of 2.

I`ll take it :)


Support specifying generics to tagged template string functions

Flow version: All
Expected behavior
Tagged template strings functions should support specifying generics at call time
Actual behavior
Tagged template strings functions does not support specifying generics at call time

Link to Try-Flow or Github repo:
https://flow.org/try/#0MYewdgzgLgBAtgTwCoEMDmMC8MA8AlAUwgFcAbKAPgApoAnASzDQgBoYA6TgBxVpTggBKAFwxCJclgowA3gCgYi0JFi0iZKKPEasMKsTAATAgDNGBQ6JRgEggNxyFitVGK0wMNRKgOAvg7llaE91SWxEVDQcMGI4ACMCWgoAA2S7IA

Is anyone aware of any workaround for this?

To give some context, this affects Emotion styled components Flow types.
type Props = { color: string };
const Foo = styled.div<Props>`
  color: ${props => props.color};
`;

@SamChou19815  any explanation?

Sorry this one is closed by accident, but there is still no plan to support this


Bug: LGU Save Files not able to sync with LCBetterSaves

I've noticed that the LGU save files are not syncing with the save files when using LCBetterSaves.
It's not actually that big of an issue, but it has to actually do with the way the actual Game save files and your LGU save files are named. Your LGU save files work perfectly fine EXCEPT for the fact that they aren't matching up with certain save files when one gets deleted.
So for example: the creator of LCBetterSaves has made his save files renamed whenever a save file gets deleted.
My personal example is here: LCSaveFiles 0-4, and your LGU save files 3 and 4. We'll only look at 3 and 4 specifically to recreate this problem. LGU_3 has the Running Shoes Upgrade, while LGU_4 does not. (Ignore everything past LGU_4.json)

Now, if I go into LC and delete LCSaveFile3 (AKA the 4th Save Slot), all the files AFTER the original LCSaveFile3 get their save number subtracted by 1. The Original LCSaveFile3 = Gone, The Original LCSaveFile4 = The New LCSaveFile3, etc.

NOW, the original LGU_4 is not being applied to the New LCSaveFile3. The original LGU_3 is now being applied to the New LCSaveFile3.
In simple terms: when I delete the 1st Save slot, the 2nd Save slot becomes the new 1st Save. BUT, now the new 1st Save (2nd Save) has the original 1st Upgrade Save, which is being caused by the names of the save files. Oh, and the original 2nd Upgrade Save is just kinda there.

I see it has a repository on GitHub as well. I can look into this and see if the owner would be receptive to me making a PR changing the delete method of their mod to be a safer operation. Thanks.

@Uncryptical Paul merged my pull request a few days ago and said he will update it on thunderstore soon here.

should be updated in thunderstore now, LMK if there are further issues


Determine doc impact of P2101R0

Determine doc impact of P2101R0
Occasionally, a paper increases restrictions on users (either at compile-time or run-time) without requiring implementations to diagnose improper usage.
This has no implementation impact, but may have doc impact.
Determine the doc impact of this paper.

moved to quest


ISD dictionaries are not JSON serializable if the filename has a POSIX path

Currently if a filename has a path like "../../my-file.txt", the ISD dictionary is not JSON serializable. The goal of this issue is to only include the filename and not the full path in the metadata so that the ISD dictionary is JSON serializable.

Hi MthwRobinson, I'm a new contributor to this project and I'm currently working on this issue. I'm a bit stuck and not sure how to proceed to resolve the issue. Could you please provide me with some guidance on how to move forward? Any help would be greatly appreciated!

Thanks @firedragonironfist for your interest in contributing! #233 will close this issue, but maybe checkout #228 if you'd like to make a first contribution. Just make sure to assign the issue to yourself so we know you're working on it.

Thank you so much I will check it out

Oh nevermind that was you! I think I misread the username. Just assigned you #228 .


add option to make postgresql.conf unmanaged

add option to make postgresql.conf file unmanaged by puppet.
at this time it's impossible to use own config file.

@frconil to be precise, postgresql_conf is its own type, which parses/inserts into the file.
it's not based on augeas.

@CharlieR-o-o-t thanks for the contribution, but closing since it sounds like this isn't necessary from the comments.


tests: fix handling of tmp dir

If the tests are run in a fresh git checkout where the tests/data/tmp directory doesn't exist, the test_with_output_dir() case would create it as a file instead of a directory, and then the test_with_output_file() case would subsequently fail.
Ensure the tmp directory is actually a directory in each case.
Also ensure the tmp directory itself is removed at the end of the tests, which reveals this issue.

Thank you, good catch!


Make Horizontal Bar chart available again

Make horizontal bar charts available again:
Since we support chart.js in version 3 and above, Better Access Charts has lost the ability to create horizontal bar charts

indexAxis: 'y' in the options section should do the trick

This solution will also work for line charts

Released with version 4.04.12


Remove link to Google Bucket

I heard that we are planning to remove the link to the Google Bucket in the Workspace Summary. This issue is to actually do it, when we're ready.

Until we have a better way of previewing and accessing data from Google buckets, this link should remain. I'm going to close this and reopen it at that later time. Thanks for thinking ahead!


softcover check doesn't notice epubcheck installation

epubcheck was falsely said to be unavailable when it had been installed via Homebrew

Thanks for the contribution. This solution breaks on my system because I didn't use brew. Can you confirm that this works for you?
     # Finds EpubCheck anywhere on the path.
     version_3 = path('epubcheck-3.0/epubcheck-3.0.jar')
     version_4 = path('epubcheck-4.0.1/epubcheck.jar')
     first_path(version_4) || first_path(version_3) || get_filename(:'epubcheck') || ""

If so, please include this code and re-push your PR.

Yep, that worked with my brew installation. Will re-push.


Changed the default value of 'scrollByPage' in the scorable items

This is a possible PR for #145458

I'm not sure changing the default is a valid solution.
Yes, it makes all scrollbars go by page, but it still does not read the config so you still cannot handle it from the configs.
It's enabled everywhere with no control whatsoever.

I'm not sure changing the default is a valid solution. Yes, it makes all scrollbars go by page, but it still does not read the config so you still cannot handle it from the configs.
It's enabled everywhere with no control whatsoever.

The options are here: https://github.com/microsoft/vscode/blob/79dae3daa8cb3d138bdd352de5540488c0425f8b/src/vs/editor/common/config/editorOptions.ts#L3282

We need a workbench.list.scrollByPage setting, I will clarify this in the original issue.


Improve logging of Data creation

User Story
As a user I would like to have insight where my Metal3Data creation is and where it might be stuck.
Detailed Description
Right now the Creation of Metal3Data out of Templates doent indicate the exact point where it is and what resources it is still waiting for.
This makes it really hard to find issues in your data templates.
Increased logging with info like "waiting for ipAddressesFromIPPool ip_1 from testpool would help a lot in that regard,
Anything else you would like to add:
/kind feature

/retitle Improve logging of Data creation

/assign

/remove-lifecycle stale.

/remove-lifecycle stale

/triage accepted

/assign


[crowd]: add missing JVM_SUPPORT_RECOMMENDED_ARGS env

What this PR does
add the missing link from the configmap to the ENV JVM_SUPPORT_RECOMMENDED_ARGS
Current behaviour
All configurations in the values crowd.additionalJvmArgs are written in the configmap, but not applied to the environment
Exepted behaviour
the ENV JVM_SUPPORT_RECOMMENDED_ARGS is set correctly
PS
Can you please make new chart releases for any product?
The current versions are from november/december 2021 and I really need to test all the new features (around 9 PRs) with a stable chart.
thanks!

Hi @janfuhrer , thanks for the contribution. Good catch!
We will work on a new release soon!

Hi @janfuhrer , just a heads up that chart 1.2.0 has now been released~


Custom Codec code table

This PR introduces a custom Codec code table (instead of using the one from rust-cid) which contains only the supported codecs.
This PR can be reviewed as individual commits, I bundled them into one PR to make it clearer what this is leading to.

In general I have the feeling that multihash, cid and ipld crates are exploding in complexity, I don't feel too happy about that, I'd prefer keeping things simple. But I guess it's ok.

The other issue is I'm finding it unfortunate that multihash/cid aren't stack allocated, as someone could always Box them. And parsing a Sha2256Digest would know the size exactly, and a Box could parse any multihash

I'll probably also implement a codec based on parity-scale-codec and scale-info crates. parity-scale-codec is more efficient than cbor for native data types and scale-info would allow adding a metadata section that could allow decoding it to an ipld representation.

The other issue is I'm finding it unfortunate that multihash/cid aren't stack allocated, as someone could always Box them. And parsing a Sha2256Digest would know the size exactly, and a Box<dyn Digest> could parse any multihash

Friedel hacked something together (based on https://github.com/multiformats/rust-cid/pull/52#discussion_r420293672):
https://play.rust-lang.org/?version=nightly&mode=release&edition=2018&gist=296c1d3f20591440cf67ae65db0534e9
to show were things could go in the future. I still haven't had a closer look. But is this what you are after?

Yes, something like that. It's unfortunate that const generics aren't a thing yet, as the proposed solution is basically a workaround.


Error when Epub has not cover

Description
When i try to read epub that has not cover image and any tag for cover in content.opf, throws an exception and can't read epub. If there is image and added in content.opf, it is ok but if there is no cover tag in content.opf, throws an exception
Exception
This exception was originally thrown at this call stack:     VersOne.Epub.Internal.BookCoverReader.ReadEpub2CoverFromGuide(VersOne.Epub.EpubSchema, System.Collections.Generic.Dictionary<string, VersOne.Epub.EpubByteContentFileRef>)     VersOne.Epub.Internal.BookCoverReader.ReadEpub2Cover(VersOne.Epub.EpubSchema, System.Collections.Generic.Dictionary<string, VersOne.Epub.EpubByteContentFileRef>)     VersOne.Epub.Internal.BookCoverReader.ReadBookCover(VersOne.Epub.EpubSchema, System.Collections.Generic.Dictionary<string, VersOne.Epub.EpubByteContentFileRef>)     VersOne.Epub.Internal.ContentReader.ParseContentMap(VersOne.Epub.EpubBookRef, VersOne.Epub.Options.ContentReaderOptions)     VersOne.Epub.EpubReader.OpenBookAsync.AnonymousMethod__1()     System.Threading.Tasks.Task<TResult>.InnerInvoke() in Future.cs     System.Threading.Tasks.Task..cctor.AnonymousMethod__272_0(object) in Task.cs     System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(System.Threading.Thread, System.Threading.ExecutionContext, System.Threading.ContextCallback, object) in ExecutionContext.cs     System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() in ExceptionDispatchInfo.cs

@oguzhandurmaz, thank you for reporting this. It might be the same issue as this one which will be fixed in the upcoming 3.3 release. Could you please check if the 3.3.0-alpha1 version works for you?

Yes it worked. Thank you.

Fixed in the 3.3.0 release.


AssertionError [ERR_ASSERTION]: No parent symbol found

code -
import { sum } from "lodash-es";

function sayBye(message: string): ReturnType<typeof sum> {
  return sum([1, 1]);
}

export { sayBye };

tsconfig.json -
{
  "include": ["."],
  "compilerOptions": {
    "target": "ES2021",
    "module": "ESNext",
    "moduleResolution": "node",
    "esModuleInterop": true,
    "strict": true,
    "sourceMap": true
  },
  "typedocOptions": {
    "entryPoints": ["index.ts"],
    "out": "docs",
    "json": "docs/out.json",
    "plugin": ["typedoc-plugin-missing-exports"],
    "pretty": true
  }
}

I am getting this error :
This happens when ReturnType is used in the code.
Info: Loaded plugin typedoc-plugin-missing-exports
TypeDoc exiting with unexpected error:
AssertionError [ERR_ASSERTION]: No parent symbol found for sum in <internal>
    at isInherited (/Users/k/p/tal/node_modules/typedoc/dist/lib/converter/symbols.js:534:5)
    at Object.convertFunctionOrMethod (/Users/k/p/tal/node_modules/typedoc/dist/lib/converter/symbols.js:210:9)
    at convertSymbol (/Users/k/p/tal/node_modules/typedoc/dist/lib/converter/symbols.js:123:88)
    at Converter.convertSymbol (/Users/k/p/tal/node_modules/typedoc/dist/lib/converter/converter.js:45:37)
    at onResolveBegin (/Users/k/p/tal/node_modules/typedoc-plugin-missing-exports/index.js:45:47)
    at triggerEvents (/Users/k/p/tal/node_modules/typedoc/dist/lib/utils/events.js:191:43)
    at triggerApi (/Users/k/p/tal/node_modules/typedoc/dist/lib/utils/events.js:167:13)
    at eventsApi (/Users/k/p/tal/node_modules/typedoc/dist/lib/utils/events.js:60:18)
    at Converter.trigger (/Users/k/p/tal/node_modules/typedoc/dist/lib/utils/events.js:389:13)
    at Converter.resolve (/Users/k/p/tal/node_modules/typedoc/dist/lib/converter/converter.js:199:14) {
  generatedMessage: false,
  code: 'ERR_ASSERTION',
  actual: undefined,
  expected: true,
  operator: '=='
}

I can't seem to reproduce this with TypeScript 4.7.4, TypeDoc 0.23.1 if you could send a PR to https://github.com/TypeStrong/typedoc-repros with a reproduction, that would be great!


fix:协程模式onOpen事件触发延迟

[This message is created by hyperf-bot]
Hi @boyfoo, this is a READ-ONLY repository, please submit your Pull Request to hyperf/hyperf repository, this Pull Request will close automatically.


Swift Package Manager & C Bindings including Makefile

Swift Package Manager
Swift's package manager can build C/C++ sources and use headers to expose functions to Swift. The standard tree-sitter parser project layout just requires a little extra configuration to make it happy.
Grammars with SPM support to this date:

https://github.com/tree-sitter/tree-sitter-javascript
https://github.com/tree-sitter/tree-sitter-php
https://github.com/tree-sitter/tree-sitter-c
https://github.com/tree-sitter/tree-sitter-java
https://github.com/tree-sitter/tree-sitter-rust

C Bindings & Makefile
This change adds a Makefile and bindings to more easily build static/dynamic libraries for use in C-based languages. The implementation is intended to be as generic as possible, to make for easier adoption and diffing.
This comes from discussion in the main tree-sitter repo https://github.com/tree-sitter/tree-sitter/issues/1488.

Is there anything that's holding this up?

@lukepistrol if you're interested still (like w/ python), just make it a swift bindings addition PR (and let me know for other langs that interest you)

I've added swift bindings now


Paragraph styles disappear when applying something like backColor

You always have to set lineSpacing and other paragraph styles after setting some other substring styles.

Hi, can you provide a real-world example?


feat: custom front-layer shape in BackdropScaffold

added frontLayerShape in-place of frontLayerBorderRadius
deprecating frontLayerBorderRadius in back-compat way

Closes #115

tested with example app with following code:
BackdropScaffold(
    ...
    frontLayerShape: const BeveledRectangleBorder(
          borderRadius: BorderRadius.only(
            topLeft: Radius.circular(16),
            topRight: Radius.circular(16),
          ),
        ),
    ...
)


feat: polish the Python binding exception

For now, we have 15 different error codes in the rust core. FYI https://github.com/apache/incubator-opendal/blob/main/core/src/types/error.rs#L52-L108
But in Python bindings, we mixed all the errors using an error message. It will make the binding easier when people want to handle those errors differently. FYI https://github.com/apache/incubator-opendal/blob/main/bindings/python/src/utils.rs#L87-L96
Let's polish those error codes in the Python binding. Here are two way to make it done

Make a single Exception for one error code.
Store the error code in exception like we have made it done in the Java bindings. FYI https://github.com/apache/incubator-opendal/blob/main/bindings/java/src/error.rs#L48-L64

I may perfer the first way.

Committers should reach consensus before marking this issue is ready for implementation.

Committers should reach consensus before marking this issue is ready for implementation.

Ops, my bad

I'm not familiar in Python's error handling. What method do Python users typically prefer? Also, how will users utilize our exception?
AWS S3 pyhon's sdk seems have different exceptions:
import botocore
import boto3

client = boto3.resource('s3')

try:
    client.create_bucket(BucketName='myTestBucket')

except client.meta.client.exceptions.BucketAlreadyExists as err:
    print("Bucket {} already exists!".format(err.response['Error']['BucketName']))
    raise err

ref: https://boto3.amazonaws.com/v1/documentation/api/latest/guide/error-handling.html#catching-exceptions-when-using-a-resource-client

IMO the UX of having an reasonable exception hierarchy is better than plain error code in Python.

IMO the UX of having an reasonable exception hierarchy is better than plain error code in Python.

+1.
Let's rock!


No git.OpenRepository() without context

as title

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@889a8c2). Click here to learn what that means.
The diff coverage is 73.62%.

@@           Coverage Diff           @@
##             main   #19260   +/-   ##
=======================================
  Coverage        ?   46.40%
=======================================
  Files           ?      870
  Lines           ?   123401
  Branches        ?        0
=======================================
  Hits            ?    57268
  Misses          ?    59178
  Partials        ?     6955

Impacted Files
Coverage Δ

cmd/admin.go
0.00% <0.00%> (ø)

models/organization/team_repo.go
76.19% <ø> (ø)

models/organization/team_user.go
74.57% <ø> (ø)

modules/doctor/misc.go
21.55% <0.00%> (ø)

modules/git/diff.go
76.22% <0.00%> (ø)

routers/web/repo/branch.go
57.32% <0.00%> (ø)

routers/web/repo/commit.go
26.16% <0.00%> (ø)

routers/web/repo/lfs.go
0.00% <0.00%> (ø)

services/pull/check.go
28.35% <0.00%> (ø)

services/pull/review.go
51.45% <0.00%> (ø)

... and 39 more

Continue to review full report at Codecov.

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


Can TypeScript refer to an instance type over a namespace type?

I have a library whose .d.ts file defined something like:
declare class Logger extends EventEmitter {
    constructor();
    info(message: string);
    error(message: string);
}

declare namespace Logger {
    const INFO: number;
    const ERROR: number;
}

export = Logger;

I'm accessing it with require() and loading it in at runtime:
import LoggerType = require("mylogger");

...

let logger: typeof LoggerType;
try {
    logger = require("mylogger").createLogger(...);
}
catch (err) {
    ...
}

if (logger) {
    logger.info("hello world");
}

But this doesn't work because TypeScript doesn't see logger as an instance of Logger but as the namespace Logger.  Can it instead refer to the instance of Logger so logger.info() will not be a type violation?

let logger: LoggerType;
typeof LoggerType gives you the type of the module's export, which is a class. LoggerType gives you the type exported by the module, which is the type of instances of Logger.

Thanks.


fix(runtime-core): handle children of boolean value inside create vnode

fix https://github.com/vitejs/vite/issues/809

https://github.com/vitejs/vite/issues/809#issuecomment-692398088


Feature request #  Bird Species Identification using Deep Learning #Streamlit

Is your feature request related to a problem? Please describe.
Current bird species identification processes can be time-consuming and error-prone. There is a need for an efficient and accurate method to identify bird species from images.
Describe the solution you'd like
Implement a deep learning model to identify bird species from images. Integrate this model with a Streamlit application to provide a user-friendly interface for uploading images and viewing identification results in real-time.
Describe alternatives you've considered
Using traditional image processing techniques for bird identification.
Developing a web-based application using other frameworks such as Flask or Django.
Approach to be followed (optional)
Collect and preprocess a dataset of bird images labeled with species names.
Train a convolutional neural network (CNN) model on this dataset.
Develop a Streamlit application that allows users to upload bird images and view the model's predictions.
Deploy the application on a cloud platform for easy access.
Additional context
This feature will leverage the power of deep learning to improve the accuracy and efficiency of bird species identification. The Streamlit interface will ensure a seamless user experience.

pls assign me and add labels


add more distribution builds

adding centos and ubuntu-20.04.4 to the list of distribution builds we
want to test to make sure it compiles with their shipped cargo version.
Signed-off-by: Brent Baude<EMAIL_ADDRESS>

/lgtm
/approve


Hypervisor mode

The microkit changes for system call/invocation labels are currently hardcoded and won't work for hypervisor mode.
Making it work for hypervisor mode isn't hard per se, but it is ugly (see what had to be done for non-hypervisor mode and imagine even more manual, confusing duplication). It is also quite error-prone and not a maintainable solution in general. This is a known problem and the plan is to eventually export these from the seL4 kernel instead of hardcoding them in microkit. After this, hypervisor mode should just work.

Solved by https://github.com/seL4/microkit/pull/225. Will close this issue when this is merged.


how to dismiss

i want use Progress Dialog , but i can't dismis this.
please help me
my code :
m_dialog.title("بارگذاری")
                    .titleColorRes(R.color.accentColor)
                    .content("لطفا شکیبا باشید")
                    .contentColorRes(R.color.activity_bg_color)
                    .backgroundColorRes(R.color.dialog_color_black)
                    .progress(true, 0)
                    .show();

This is a duplicate of many other issues.

I added a new section to the README since so many cannot figure it out: [Dismissing Dialogs[(https://github.com/afollestad/material-dialogs#dismissing-dialogs)

I added a new section to the README since so many cannot figure it out: Dismissing Dialogs


Fallthrough Statement is not excepted

What version of Go are you using (go version)?

go version go1.18.1 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/g2academy/Library/Caches/go-build"
GOENV="/Users/g2academy/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/g2academy/go/pkg/mod"
GONOPROXY="bitbucket.org/gotoko"
GONOSUMDB="bitbucket.org/gotoko"
GOOS="darwin"
GOPATH="/Users/g2academy/go"
GOPRIVATE="bitbucket.org/gotoko"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.18.1/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.18.1/libexec/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.18.1"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/g2academy/go/course/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/wv/nv0yzj291515sqxgmy7nqjrm0000gn/T/go-build4033544454=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
package main

import (
	"fmt"
)

func main() {
	var pi float32 = 3.14
	switch {
	case pi == 3.14:
		fmt.Println("pi == 3.14")
		fallthrough
	case pi == 11.00:
		fmt.Println("pi == 11.00")
	default:
		fmt.Println("i'm here")
	}
}

What did you expect to see?
The pi == 11.00 block is executed
What did you see instead?
The pi == 11.00 block is not executed

looks like it's working as expected
Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions

In the future, please paste plain text as plain text, not as an image.  Thanks.


chore: add tests for utility functions for upsert env & truncate strings

This PR adds unit tests for the utils pkg functions:
TruncateString and UpsertEnvironmentStatus

The TestTruncateString function validates different truncation scenarios, including handling empty, short, and truncated strings, as well as an invalid negative length.
The TestUpsertEnvironmentStatus function tests the upsert logic for appending new elements, handling empty slices, and managing edge cases with a single element.

Will need to fix the lint issues, and then also add a new Make target that runs go test but excludes the TestControllers function which could actually probably be renamed to TestControllersGinkgo. Then we would probably need to add the new Make target to the github ci action.

and then also add a new Make target that runs go test but excludes the TestControllers function which could actually probably be renamed to TestControllersGinkgo. Then we would probably need to add the new Make target to the github ci action.

I dont know my way around this codebase well, can you show me an example of that, I know gingko does that BDD style of tests

@zachaller ive updated the func name & created the make target, before adding it to CI, I want to decide on what to name the make target, do you have any suggestions?

go-unit-test for the make target I think makes sense

@zachaller its ready for review

Codecov Report
Attention: Patch coverage is 77.77778% with 4 lines in your changes missing coverage. Please review.

Project coverage is 49.75%. Comparing base (4dab5ba) to head (5ecb63b).
Report is 13 commits behind head on main.

Files with missing lines
Patch %
Lines

...nternal/controller/promotionstrategy_controller.go
82.35%
2 Missing and 1 partial :warning:

internal/utils/utils.go
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #161      +/-   ##
==========================================
- Coverage   54.65%   49.75%   -4.90%
==========================================
  Files          10       12       +2
  Lines        1257     1437     +180
==========================================
+ Hits          687      715      +28
- Misses        481      633     +152
  Partials       89       89

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

The coverage is still broken need to merge reports not sure how because ginkgo doesn't seem to support the new go binary format

I wonder if you could do two uploads under different codecov tags, or whatever they call them.

hm... I would have to look into that guessing its more than just https://github.com/argoproj-labs/gitops-promoter/pull/161/files#diff-944291df2c9c06359d37cc8833d182d705c9e8c3108e7cfe132d61a06e9133ddR53


[paconn] Add support for logout

Problem:
Sometimes we use a different accounts to deploy a connector across different environments (a PROD/TEST environment vs a community environment). The paconn utility, once logged in, cannot be used to target the other environment. The only option seems to wait out until the token expires - which is extremely painful.
Describe the solution you'd like
Add a paconn logout command.
Describe alternatives you've considered
Another window/session.

paconn login -f will force a login to a different account.

Logout command for paconn has been added in https://github.com/microsoft/PowerPlatformConnectors/pull/290 for version 0.0.14
This new functionality can be used via paconn logout
Closing issue since feature request was implemented.


Add --max-pods param for bootstrap script

Maximum number of pods per node specified at /etc/eks/eni-max-pods.txt
and it is the limit of network interfaces for the instance type. This
number form an amount of memory in Mi kubernetes reserved with formula
11 * $max_num_pods + 255.
Nodes could have a setup when maximum number of pods defined and
it is less than value at the file. Here I add an ability to use
command line value to cover such cases.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@mogren @natherz97 PTAL

@alesavin Is there a reason you can't use the --kubelet-extra-args parameter to specify max pods? Looks like that should have the same effect. We try to avoid adding new params to the bootstrap script

@mmerkes there are reasons why I see extra args for kubelet not the best choice:

--max-pods, --kube-reserved are deprecated (... parameter should be set via the config file specified by the Kubelet's --config flag. at kubelet params info), config is generated by bootstrap script here
with --kube-reserved I have to specify not only memory but cpu, ephemeral-storage limits; but only reserved memory is the thing I want to touch

just to be clear, I want to change not only maxPods but kube reserved memory as well (as last one correlate to maxPods as 11 * maxPods + 255). I would be glad if you provide any other workaround to tweak these values, I probably missing some parts.

@alesavin you probably want to limit the max-pods value passed as parameter to the value specified in the eni-max-pods.txt file. I believe the number contained there is the max number of network interfaces for the given instance type. You probably do not want to allow users asking for a max-pods value beyond that number

done, add upper boundary for cmd line param

Any progress on this? I'd like to see this functionality added

@mmerkes any feedback for my PR vs "--kubelet-extra-args"?

I, too, would like to see this flag added. The method to use --kubelet-extra-args should be avoided as the kubelet docs indicate the max-pods flag is deprecated (can't deep-link on that page). While it could work, I feel it would be best not to encourage support through a deprecated option.

But, I'd also like to see a value that can go above what the ENI-based max allows, as not everyone that uses the EKS-optimized AMI is using the VPC CNI (like myself).

Hi @alesavin
We encountered (other) deprecated kubelet flags too.
Looking at the bootstrap script and PR #90, it appears that a flexible solution would be to transform kubelet-config.json before calling bootstrap.sh (via cloudinit or other means).
For example, a (terraform-templated) cloud-init.yaml file could look like:
runcmd:
  - echo "$(jq "${kubelet_config_overrides_json} /etc/kubernetes/kubelet/kubelet-config.json)" > /etc/kubernetes/kubelet/kubelet-config.json
  - [
      /etc/eks/bootstrap.sh,
      --apiserver-endpoint,
      "${cluster_endpoint}",
      --b64-cluster-ca,
      "${cluster_ca}",
      "${cluster_id}",
    ]

A module which creates the EKS cluster will also export the node userdata, which could look like this:
output "node_userdata" {
  value = templatefile("${path.module}/cloud-init.template.yaml", {
    cluster_endpoint = aws_eks_cluster.cluster.endpoint
    cluster_ca            = aws_eks_cluster.cluster.certificate_authority.0.data
    cluster_id             = aws_eks_cluster.cluster.id
    kubelet_config_overrides_json = {
      maxPods = 42
  })
}

@yanilov files/bootstrap.sh changes /etc/kubernetes/kubelet/kubelet-config.json directly like you mentioned, not passing kubelet args. If you transform it before the file data (like memory) will be over-written https://github.com/awslabs/amazon-eks-ami/blob/master/files/bootstrap.sh#L440. Am I missing something?

We need to offer a way to specify kubelet config parameters, as we have with --kubelet-extra-args. Modifying the default kubelet config file before or after the bootstrap script executes is fragile, I wouldn't recommend that approach. I'm going to close this, we'll continue tracking in #873.


[Docs] DoubleArray and FloatingPointArray are missing from generated Python docs

Describe the bug, including details regarding any error messages, version, and platform.
https://arrow.apache.org/docs/python/api/arrays.html lists the concrete array types available in Pyarrow. But some are missing: https://github.com/apache/arrow/blob/205ceb99389d2d5060551ebcba17635185c74d9d/python/pyarrow/array.pxi#L1870-L1885 lists the float16/float32/float64 concrete array types, and they don't appear in the generated table, nor do their direct links (eg https://arrow.apache.org/docs/python/generated/pyarrow.DoubleArray.html) work.
Component(s)
Documentation, Python

It appears to me that this is because those types are not in the big list of exported types in https://github.com/apache/arrow/blob/205ceb99389d2d5060551ebcba17635185c74d9d/python/pyarrow/init.py#L200C1-L202. Is that an oversight, or intentional?

Thank you for reporting! I think this is not intentional.
In the PR you could add a comment to generate the preview of the docs: @github-actions crossbow submit preview-docs.


DEFINE ATTRIBUTE doesn't find LET/D variables

Reported by steven c hankin on 21 Jan 2014 18:57 UTC
This script shows two problems

unable to assign an attribute to a LET/D variable
off-target error messages

yes? use levitus_climatology
yes? let/d=1 myvar = 1
yes? define attrib/d=1 temp.myatt = "in dataset 1"   ! works as it should
yes? define attrib/d=1 myvar.myatt = "in dataset 1"  ! bug
 **ERROR: command syntax: myvar.myatt
          enclose coordinate variables in parentheses
 **ERROR: invalid command: variable or axis does not exist XAXLEVITR
yes? define attrib myvar.myatt = "in dataset 1"
 **ERROR: command syntax: myvar.myatt
          enclose coordinate variables in parentheses
 **ERROR: invalid command: variable or axis does not exist XAXLEVITR

Discussion:
I may not be fully understanding the code, but it appears the problem is that in the linked list code that handles attributes all user variables are stored under dset=-1.  The value of -1 is used as a stand-in for the category, cat_user_var, that would indicate a user-defined variable.  Thus there is no way to represent a user variable and a dataset at the same time, as is needed for LET/D definitions.
Fixing this seems likely to require pretty extensive changes.  Since LET/D is not so commonly used, the cost benefit to fixing this may not be high enough to push it to the top of the list.  That is, unless it turns out to prevent the Ferret attribute-defining code from being usable as the means to link sigma and curvilinear variables with their auxiliary variables which may commonly be defined as LET/D variables.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2134

Comment by steven.c.hankin on 21 Jan 2014 19:34 UTC
The fix the you have recommended -- to allow the final command above
DEFINE ATTRIBUTE myvar.myatt = "in dataset 1"
to assign the attribute to the LET/D value -- is worth doing, but only if quite simple to do. It will still leave the underlying problem untouched.   Namely, in cases where LET/D has been used to define the same variable name in multiple datasets, this DEFINE ATTRIBUTE syntax cannot differentiate which is the intended dataset.  The attribute will be arbitrarily assigned to the first LET variable in the linked list that matches the name, "myvar".

Comment by steven.c.hankin on 21 Jan 2014 20:48 UTC
(this comment entered without looking at the code .. so possible over-simplifications)
Correcting the problem is in principle vary simple.

add "cat" as an argument to CD_GET_VAR_ID
inside of CD_GET_VAR_ID search the dset=1 linked list if cat is cat_user_var
whenever a uvar is found check that it is consistent with the dset value

The messy part comes in the changes to the great many places where CD_GET_VAR_ID is called.  The surrounding logic at each of those, including the places where dset gets set to -1 as a surrogate for cat=cat_user_var, all need to be carefully re-examined and modified.

Comment by steven.c.hankin on 18 Feb 2016 18:46 UTC
When looking into this trac ticket be sure to look also in SET_AGG_VAR_ATTS
In that routine I am skipping over the inheritance of attributes that belong to LET/D members of aggregations, because (per this ticket) those attributes cannot be set.  When addressing this trac ticket the routine SET_AGG_VAR_ATTS should be brought up to snuff.

Comment by ansley.b.manke on 18 Feb 2016 19:13 UTC
This comes from what is described in #2352, See also possibly #2352 and #2349

Comment by ansley.b.manke on 26 Feb 2016 19:19 UTC
This is done, with a new benchmark script bn_letd_attributes.  LET/D= variables are stored in the linked-list structure with the dataset they're assigned to, so having multiple LET/D= variables for different datsets with the same name works correctly.


Local IP blacklist bypass

Issue
It appears that we can bypass the local IP blacklist (implemented in #16) by replacing decimal characters with hex. I have tried with octal, and that didn't appear to work. Furthermore, it seems to pass some type of malformed request through when I use hex as well; this may be an issue with Gin, but I'm not 100% sure. EDIT: Turns out this was because I was trying to use HTTPS with a listener that didn't support it
BeePing listener:

HTTP listening service:

Curl requests:

Fix
My suggested fix is to cast the destination IP to an integer in the validateTarget() function, before parsing the IP with net.ParseIP(), because that function is unable to parse hex values. However, I want to be sure that we catch all test cases before doing so.
Other  Notes
It's also worth tracking down what's happening with the data as it's passed through, as it appears to be corrupted or malformed somehow. When I debugged the request, the req value seemed fine (in the CheckHTTP() function), but there were two other weird values that probably shouldn't have been so off: EDIT: Ignore this, see edit above

Thanks to @jimen0 for bringing this to my attention. He may be able to chime in here as well.
Cheers,
Aaron (insp3ctre)

Hello.
As Aaron and I were discussing through DMs, there are some points that we should think about.
First, SSRF is really hard to fix using blacklists, but I think that the idea of converting it to decimal number notation and doind the check is a really cool idea.
There are still a few scenarios that we should be concerned about:

SSRF can be performed by creating a request to, for example, https://mylegitdomain.com and then, redirecting visitor to http://<IP_ADDRESS>:80/, if the software follows the redirect, the attack will take place
There are tons of notations for IP addresses and we can mix them (hex + decimal number, hex + decimal octets, etc), IMO we should do as many tests as possible for this and then fuzz it to check that we fixed as many scenarios as possible

Hope you keep up the work on this project. I think that it's a great idea!
Kind regards,
M. Ángel

Very interesting discussion here ! I'm very amazed of all the ways you can find for breaking Beeping security.
So, for fixing breachs:

Check if the IP address is integer based
Disable HTTP redirect by default
Disallow reserved IP

For the first bullet, are IPv6 addresses OK with this check?
Thank you guys !

No. Our idea (or at least mine), is to convert the IP address to decimal number so that we just need to do a check for the forbidden ranges.
So, for example, if you receive http://<IP_ADDRESS>/ you convert the IP address to it's decimal number representation, which is<PHONE_NUMBER> and then you perform the valid ranges check.
Another example could be receiving  http://0x7F000001/, which is also valid (you can test it with curl if you want), which we could just convert to decimal number and then perform the check. The same goes with IP addresses like 0x1F.0.0.1, 127.1, 0<IP_ADDRESS> (which is a valid octal representation for <IP_ADDRESS>), etc
In my opinion, we should first fix this before we start implementing IPv6 support, there are tons of ways to represent the same IP address in IPv6.
Also, if you're familiar with gin, disabling the HTTP redirect should be easy for you.
Regards.

Disabling HTTP redirect would be done with the http.Client, rather than with Gin. It's pretty straightforward to do, here's the implementation in one of my own projects: https://github.com/insp3ctre/race-the-web/blob/master/main.go#L349.
I like the idea of converting the whole thing to decimal representation, just be sure that you check for strings first, without trying to convert those (as ASCII will convert directly to decimal).
I should also point out that reserved IP addresses are mostly blocked already with the blacklist (https://github.com/yanc0/beeping/blob/master/beeping.go#L106). There are some that aren't, if I recall correctly, but it was because they wouldn't have had any impact on security anyway.
@jimen0 did you get octal working in your testing? For me, it didn't seem to work:

Cheers,
Aaron (insp3ctre)

Hey, @insp3ctre
I did not managed to use octal representations through beeping. I thought it could work because curl http://017700000001:50000/  is the same as curl http://<IP_ADDRESS>:50000, sorry, it was my fault.
If I'm not wrong, the next step then, is blocking HTTP redirections follow by default, right? cc/ @yanc0
Regards.

@jimen0 The first piece of code should fix it, yes. @yanc0, you good with that? Also, I was going to suggest you opening another issue for redirect, but it looks like you have with #21!
As for this issue- I think the solution we had discussed about casting it to an integer before running the local network checks on it should work.
Cheers,
Aaron (insp3ctre)


text-DaVinci-003 vs gpt-4

Hey, awesome project.
just curious about the use of text-DaVinci-003 over gpt-4.
would you still recommend using text-DaVinci-003 over gpt-4 if it’s available to the user?
If so, would you be so kind as to explain why text-DaVinci-003 is a better choice?
Thanks

Hey @ollieanwyll ,
This was developed in Feb,23 when GPT4 wasn't launched. For some reason, I am still in the waitinglist of GPT4 API so I could not test the GPT4 model and compare its results to text-DaVinci-003. However,  I am sure that GPT4 will give improved output and has longer context of 32K tokens compared to the 4096 tokens on text-DaVinci-003. Some contributor can submit a PR for this change.

I tried 3.5 and 4, they work great. They feel more alive when giving responses. 3.5 is also faster and cheaper. I also manage to give them conversation memory by using 3.5 or 4. You can check out my repo: https://github.com/Yiiipu/SimplePDF

@Yiiipu - This is great!
Also, I have added support for all Turbo models now including GPT4:
https://huggingface.co/spaces/bhaskartripathi/pdfGPT_Turbo


feat: Adding icon_url property for team.Application

About
This Pull Request adds a property method for getting the Application icon url in api.model.team.Application.
Checklist

[x] I've ran pre-commit to format and lint the change(s) made.
[x] I've checked to make sure the change(s) work on 3.8.6 and higher.
[x] This fixes/solves an Issue.

(If existent): https://github.com/interactions-py/library/issues/655

[x] I've made this pull request for/as: (check all that apply)

[ ] Documentation
[ ] Breaking change
[x] New feature/enhancement
[ ] Bugfix

OK, I added those to the .pyi.

LGTM 👍🏻 ty for contributing!

No problem. Glad I can contribute something to make the library better.


[vcpkg repo] Open github discussions?

Should we open github discussions? What are the benefits and the drawbacks? cc @AugP

This sounds like a good idea to me as well, it would help us separate out questions or general discussion from future work.

I think an extra discussion place would be could. For example just recently #16974 was opened which could be discussed there instead. I agree with @Hoikas here that the Issues page has to many No-Issues/Questions and needs to be better cleaned up for real or fixable issues. I would also suggest a more strict issue policy which closes issues which don't have a repro or a clean description within a month or two after opening.
The main drawback of course is it will need extra attention

Alright, the team has agreed to open discussions. I will look at opening up discussions with @AugP over the next week or so

Discussions are now open! Please introduce yourselves in the pinned post :)


Can you possibly give me flutter 3.3.3 version?

Can you possibly give me multi_trigger_autocomplete widget for Flutter 3.3.3 version.

@Govindbetterlives I already supports the range flutter: ">=3.3.0". Can you specify your problem a bit more?

Thanks for the quick reply sir,
It's showing an error like this.
/D:/flutter_sources/flutter_3.3.3/.pub-cache/hosted/pub.dartlang.org/multi_trigger_autocomplete-1.0.0/lib/src/multi_trigger_autocomplete.dart:420:15: Error: The method 'TextFieldTapRegion' isn't defined for the class 'MultiTriggerAutocompleteState'.

'MultiTriggerAutocompleteState' is from 'package:multi_trigger_autocomplete/src/multi_trigger_autocomplete.dart'
('/D:/flutter_sources/flutter_3.3.3/.pub-cache/hosted/pub.dartlang.org/multi_trigger_autocomplete-1.0.0/lib/src/multi_trigger_autocomplete.dart') .
Try correcting the name to the name of an existing method, or defining a method named 'TextFieldTapRegion'.
? TextFieldTapRegion(
^^^^^^^^^^^^^^^^^^

Hey @Govindbetterlives you can use the previous version for flutter 3.3.0.
dependencies:
  multi_trigger_autocomplete: ^0.1.1

Hi sir,
The error came like this.
Another exception was thrown: Error: Could not find a Portal above this PortalTarget(debugName: null, portalCandidateLabels=[PortalLabel.main]).

Hey, looks like you missed this step.

Sorry for disturb,
The options list is to overlay another widget.

@Govindbetterlives you need to add the Portal at the very top of the widget tree. Can you share your code somewhere i can look?

Hi,
Now is working well. And also can you show the chip widget inside the textfield?
Something like this

Hi sir,
Can you help me how to show list and display chip input inside the textfield.

@Govindbetterlives I guess you can follow this video.
https://www.youtube.com/watch?v=_X9WOy5F1Lo&ab_channel=SuperDeclarative!


[Suggestion]Donate Button for Extra Features

Upscaling images that are already large may seem to be a bit wasteful, but I'm sure some people (me) have always wanted to do it with some images... but that, of course, is restricted due to it probably being very resource-intensive.
So I suggest a donate button that then removes some of the restrictions (to an extent) that will allow donors to upscale larger images.

You can use desktop version of waifu2x. It is no limit.
See VideoHelp Forum .
I recomment waifu2x-caffe or tanakamura's waifu2x-converter-cpp.

You can use desktop version of waifu2x

I tried building it a while back, but it's very bulky (GB's of data) and confusing (setting up CUDA and torch) to set up, so I gave up.
That's why I would suggest an alternative...

I have no plan to provide a pay service.
waifu2x-caffe and tanakamura's waifu2x-converter-cpp are windows application. There is not need to install Torch and CUDA.


chore: first step

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


MWPW-147127 Scrolling on the gnav in mobile portrait and landscape view is not working as expected ,navigation menu when opened is not seen rendered completely in MILO Pages

The content in the gnav menu (in mobile/tablet viewports, when it's open) was being cut-off on the bottom due to an incorrect height value in the wrapper. Correcting the height value for the wrapper in non desktop viewports fixes the issue
Resolves: MWPW-147127
Test URLs:
Before: https://main--milo--adobecom.hlx.page?martech=off
After: https://mobile-gnav--milo--sharmrj.hlx.page?martech=off
Before: https://main--cc--adobecom.hlx.page/creativecloud?martech=off
After: https://main--cc--adobecom.hlx.live/creativecloud?milolibs=mobile-gnav--milo--sharmrj&martech=off

Please provide a testing URL

I've tested on https://main--cc--adobecom.hlx.live/creativecloud?milolibs=mobile-gnav--milo--sharmrj&martech=off
You're right . I was under the impression that setHamburgerPadding was being called but was accounting for items above it (I misinterpreted the bottom). The problem was in await loadSearch, which was throwing an error that was caught elsewhere (and the error is lana logged so it doesn't appear on the console), so execution was never returned to our bit of code. I imagine we might need to log this error or should the catch I put in be empty as it is now? I think we don't use search anywhere currently (?) so is the error in loading search a problem?
The padding is set only after we close the hamburger for the first time, not when we open it for the first time (this is in the original logic, prior to #2184 being merged into stage), so I've changed that. Should I add some logic so we don't get needless repeat invocations of this method? The padding being removed in the desktop check is overridden by a padding: 0 !important in the min-width: 900px media query, do we want both?

@sharmrj The mention about search is a great one - it might be that we are able to get rid of it completely. That would deserve a due diligence story and checking if any consumers still use it.
Blog currently uses a search variation, but they also use the old gnav and an incompatible search

I've added MWPW-147270 as a followup

Verified the changes on below url :
https://main--cc--adobecom.hlx.live/creativecloud?milolibs=mobile-gnav--milo--sharmrj


Fix applied dates and amount format in calculator views

Description :sparkles:
Issues :bug:
Testing :alembic:
Screenshots :camera_flash:

Additional notes :spiral_notepad:

Codecov Report

Merging #760 (c6f1d10) into develop (20551a9) will decrease coverage by 27.75%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##           develop     #760       +/-   ##
============================================
- Coverage    62.89%   35.14%   -27.76%
============================================
  Files          780       68      -712
  Lines        19741     1363    -18378
  Branches      1924      275     -1649
============================================
- Hits         12417      479    -11938
+ Misses        6955      856     -6099
+ Partials       369       28      -341

Impacted Files
Coverage Δ

...ymentAppliedMoreView/EmploymentAppliedMoreView.tsx
0.00% <0.00%> (ø)

...efitCalculatorView/SalaryBenefitCalculatorView.tsx
0.00% <0.00%> (ø)

.../benefit/messages/migrations/0003_add_seen_flag.py

frontend/kesaseteli/employer/src/pages/index.tsx

frontend/kesaseteli/handler/start.js

...ackend/benefit/messages/migrations/0001_initial.py

...tend/benefit/applicant/src/hooks/useLogoutQuery.ts

frontend/shared/src/components/table/utils.tsx

...d/benefit/applicant/src/backend-api/backend-api.ts

.../migrations/0005_employee_is_living_in_helsinki.py

... and 704 more

Continue to review full report at Codecov.

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


G94 and EMSL Basis Number Issue

Ran across this while closing tickets. Is this relevant and/or is this fixed in the BSE?
https://github.com/psi4/psi4/issues/628

Good catch. I also use hik.
I will add the ability to get hijk convention from the look-up routines, and use that for G94.
I will keep this open until I can verify with a copy of gaussian, though

Confirmed with a copy of G09


Invalid index when bastion is disabled

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Terraform Version and Provider Version
0.12

Affected Resource(s)

data "oci_core_images" "autonomous_images" and locals in bastion module
Terraform Configuration Files
datasources.tf and locals.tf in bastion module

# Copy-paste your Terraform configurations here - for large Terraform configs,
# please use a service like Dropbox and share a link to the ZIP file.
# Please remove any sensitive information from configuration files before sharing them.

Debug Output
Error: Invalid index
on modules\bastion\locals.tf line 12, in locals:
12:   bastion_image_id = var.oci_bastion.bastion_image_id == "Autonomous" ? data.oci_core_images.autonomous_images[0].images.0.id : var.oci_bastion.bastion_image_id
|----------------
| data.oci_core_images.autonomous_images is empty tuple
The given key does not identify an element in this collection value.

Panic Output

Expected Behavior

Actual Behavior

Steps to Reproduce

terraform plan

Important Factoids

References

fixed in #28


Issues with Blazor 0.8.0-preview-19104-04

Trying to use the NuGet package with Blazor 0.8.0-preview-19104-04 results in build errors...
Error	CS0433	The type 'IWebAssemblyHostBuilder' exists in both 'Microsoft.AspNetCore.Blazor.Browser, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60' and 'Microsoft.AspNetCore.Blazor, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60'
Error	CS0433	The type 'BlazorWebAssemblyHost' exists in both 'Microsoft.AspNetCore.Blazor.Browser, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60' and 'Microsoft.AspNetCore.Blazor, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60'

This is caused by the major refactor in Blazor 0.8 and the release of Razor Components. Should be a simple fix of just updating the type and package names, I will release a fix sometime this week.

@csnewman Just did that ^^ and opened a PR for it.

@AndreasTruetschel Thanks for the PR, I've merged it.
I will create a new NuGet release later today.

I will create a new NuGet release later today.

This would be awesome as I need this for my own project ;)

I have published version 0.4.0 on NuGet. It is still indexing, however should be available soon.
dotnet add package BlazorSignalR --version 0.4.0
Let me know if you experience any issues via a new issue.

I have published version 0.4.0 on NuGet. It is still indexing, however should be available soon.
dotnet add package BlazorSignalR --version 0.4.0
Let me know if you experience any issues via a new issue.


Improving parameter estimates/next steps

Easiest to justify if we can add in custom control and demographic data onto a model structure which has been tested more extensively/closer to published research (although not sure if the LSHTM Africa paper** has been peer reviewed/published yet?). This would give us a framework that could be applied to any camp/low resource setting in any country.
As it stands, many of the parameters are unjustified. Most that we do have references for are sourced from:

LSHTM Africa
https://cmmid.github.io/topics/covid19/reports/LSHTM-CMMID-20200419-Covid19-Africa-strategies.pdf

Age-dependent effects in the transmission and control of COVID-19 epidemic
https://www.medrxiv.org/content/10.1101/2<IP_ADDRESS>43018v2.full.pdf

Projecting social contact matrices in 152 countries using contact surveys and demographic data
https://www.ncbi.nlm.nih.gov/pmc/articles/PMC5609774/#!po=63.3333

[ ] R0 calculation needs updating now have asymptomatic transmission at 50%. Currently wrong!

[ ] Paper (3.) has contact matrix for Greece (and 152 countries!). See new stuff in Parameters/ContactMatrices. Currently not happy with how this is implemented - would need to compare to LSHTM code to get right method of using this data (think need - beta S_i c_ij I_j /N_j instead of - beta S_i c_ij I_j and also need right method of changing from 16 x 16 matrix to 8 x 8 *** )

[ ] Need to find appropriate distributions to sample from for proper uncertainty estimates. As yet only sampling epidemiological parameters. Possibly worth sampling over death/hospitalisation probabilities too, and also control efficacies.

[ ] Need solid reference for as many params as possible (most likely all except the custom control ones which can be a bit more ad hoc)

[ ] Need to write up model in full

** Note LSHTM has much longer infectious period (still infectious in hospital etc) than we do. Since we calculate infection rate to give an appropriate R0... upshot is that our epidemic most likely has a comparable size but happens faster than it would in their model
*** unless we'd prefer to scale up rather than scale down... possibly better but will be slower. Likely unimportant decision since other uncertainties significant, so suggest go for faster model and run over a larger number of sampled runs

can add the implementation in splitting the critical compartment in two