[#369] download button text

Updated the text on the resource content download button.

After merging the develop branch I can no longer see the download button. Is this being addressed as part of [#409]?

Yes, this is addressed as part of [#409], but only when you create an empty resource without uploading files; otherwise, you should see download button. In any case, fix to #409 has been merged into develop, so download button should always be present with that fix.

@mjstealey Is the last commit due to the merge with develop of any problem? Otherwise, +1

@mauriel133, @pkdash - deployed on dev and playground. Button text renders as expected however file size is always listed as 0... not sure if this is an oversight or a rounding thing. (tested on small files)

Took a brief look at the genericresource.html code and found my previous change with new bagit url was overridden by the merge. {{bag_url}} context variable should be used for bag url rather than retrieving it from cm.bags.first.bag.url. Also, ({{ b.bag.size|filesizeformat }}) have to be removed due to the new bagit workflow. @mjstealey @mauriel133 It may need some manual merging to bring the new change in.

@hyi - should this be a new issue?
Also, do you have a reference (previous pull-request) of what the file modifications you made should look like?

@mjstealey Brian's branch appears to have my latest code for genericresource.html: https://github.com/hydroshare/hydroshare/blob/413-update-rest-api-new-bagit-workflow/hs_core/templates/pages/genericresource.html
In particular, look at the lines 686-692, which got overridden by the merge.

@hyi - the only difference I see is the button-label line.
Would the line be fine if the ({{ b.bag.size|filesizeformat }}) references were removed?
Or is there something else that I am missing here?
pre-merge: 686 - 693

  {% for b in cm.bags.all %}

        Download All

   {% endfor %}

post-merge: 686 - 693

  {% for b in cm.bags.all %}

        Download All Content as Zipped BagIt Archive ({{ b.bag.size|filesizeformat }})

   {% endfor %}

@mjstealey Yes, just checked develop branch and it looks like so. For some reason, last time I checked, the href was not assigned to {{ bag_url }}, but now it looks like removing ({{ b.bag.size|filesizeformat }}) should work. The reason we cannot use this line to check for size is that with the new bagit workflow, we don't know the bag size since all bagit operation is done on iRODS side.


Add pizza_job_id to processor kwargs

this is a patch to https://github.com/black-cape/cast-iron-worker/pull/4 where the job_id wasn't passed to processor for processor to fully leverage pizza tracker


Custom icons for custom entities

I have been trying to use some custom icons for new entities that I am creating for the tooltip actions but I haven't been able to get the actions array to accept what I put in for the icon. Is there any requirements that need to be met for custom icons to be displayed properly?

Hey @CraigLu, the only requirement is that icons need to be a React component.
All built-in icons are SVGs converted to components. You should replace the fill attribute of path elements with currentColor to allow icons to inherit CSS colors (light gray as default, white on hover and blue when active).
Here is the delete icon for a reference:
https://github.com/globocom/megadraft/blob/master/src/icons/delete.js
Why did you close the issue? Is it solved already?
Feel free to join us on our #megadraft channel on DraftJS' Slack if you have additional questions:
http://draftjs.slack.com
Register here: https://draftjs.herokuapp.com/


json output for observation data

Since the input format of the observation data is some kind of text file it would be helpful to convert the data into valid json for further work.

Done with #15.


Update the project to Phoenix 1.13

Can we please update the project to the latest Phoenix (and other dependency) versions?
For someone who is completely new to Phoenix, the older versions can be confusing.

Howdy!  Updating this is a little low on the list of priorities for the contributors at the moment.  We're busy working on Guardian v1 so we can shift our focus to giving Ueberauth (and the strategies) some much needed TLC.
That said @rishavs, pull requests are always welcome!  A great way to learn how to use Ueberauth is to give it a shot 😁

Thanks for replying. I am currently trying to get this to work on 1.1.3 but having a weird error
=INFO REPORT==== 4-Aug-2017::02:33:51 ===
    application: logger
    exited: stopped
    type: temporary
** (Mix) Could not start application oauth: could not find application file: oauth.app

i havent written the auth controller code so that might be an issue. but i have not used oauth2 anywhere so far manually (i guess it comes as a dependency for some of the strategies)

Howdy @rishavs!  Do you have this code up on a public repo?

@doomspork  I'm keen to do this if it's still on the table

@zacck please do!  I'm happy to help in a limited capacity 👍

@doomspork  I think the PR for this is done and pushed


fix docker-compose

exposing /app folder makes the container not find the entrypoint.sh

also, when starting the container, it errors the parameter "start_period" in healthcheck. (I don't know why, I don't know how to use healthcheck).

also, when starting the container, it errors the parameter "start_period" in healthcheck. (I don't know why, I don't know how to use healthcheck).

It stills not fixed. Any progress?

It stills not fixed. Any progress?

Check PR #29 . Just expose the volume /images to your host. You can also delete lines start_period, test and interval if you're not using a cluster.

It stills not fixed. Any progress?

Check PR #29 . Just expose the volume /images to your host. You can also delete lines start_period, test and interval if you're not using a cluster.

Yeah, I already mapped the '/images' internal directory, also de '/cache'. Thank you ;)

In case you don't want to clone this repo just use this service in docker-compose:
    imgpush:
        image: hauxir/imgpush:latest
        container_name: imgpush
        volumes:
            - data-images:/images
        ports:
            - "5000:5000"
        environment:
            - MAX_UPLOADS_PER_MINUTE=100
            - MAX_UPLOADS_PER_HOUR=5000
            - MAX_UPLOADS_PER_DAY=100000
            - VALID_SIZES=[150,300,600,900,1200]
            - NAME_STRATEGY=uuidv4

Don't forget to change environment variables and add data-images volume or change it to the bind


Roll engine 4663d357a360..713fe130eb02 (2 commits)

https://github.com/flutter/engine/compare/4663d357a360...713fe130eb02
git log 4663d357a36043b7dcafa01fc88511f457cc1319..713fe130eb0200412b04cc0c3356fce5f6f98153 --no-merges --oneline
713fe130e Roll src/third_party/skia 103288b50fa7..454e5fb7457d (11 commits) (flutter/engine#7792)
87b40ba97 Allow embedders to specify pointer device IDs. (flutter/engine#7790)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (chinmaygarde@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


recover primary shard first and start replica recovery after most of the primary started

In our company, we have 1M shards and 400 nodes within a cluster.
The business is very important and the data availability is very  important. In full cluster recovery process, we want to recover primary shard as soon as possible and the replica recovery can be delayed after most of the primary is started.
Why we don't want to set cluster.routing.allocation.enable to primaries

It is rely on some decider and decider itself will have computation overhead.
It rely on SRE to manually change cluster.routing.allocation.enable to all  after all the primary shards are recovered.
Fetching data of replica shard cost almost half an hour in our environment because the number of shards * nodes is too large. During that period of time, no shard can be recovered at all.

Our solution is to totally ignore replica recovery process including fetchdata and recovery at all until most of the primary are started.
We add a new parameter: cluster.routing.allocation.replica_recovery_wait_primary.enable and cluster.routing.allocation.replica_recovery_wait.threshold and the range is 0 to 1.0.
Replica recovery can only be started if percentage of the active primary shards exceed the threshold.
If the proposal is good, I can submit the MR.

@Bukhtawar Thanks for your reply.
We already tried batch async shard fetch in our environment(https://github.com/elastic/elasticsearch/issues/80694) and the recovery speed improve a lot indeed. But the primary recovery speed is still not as fast as we expected. As I listed, deciders of ReplicaShardAllocator also take a lot of time in unassignedShards loop which slow down the MasterService throughput.
I think batch fetch data and primary recovery first does not conflict and they can work together.

Thanks, Can you please share the stack traces(thread dump or profiler results) for the deciders which based on your analysis is causing the slowdown or additional memory overhead. Wanted to understand if there is a scope to optimize those areas as well.


00668 Add MetaMask wallet to the explorer

Description:
Changes below add support for MetaMask wallet and extend WalletDriver abstraction to support wallet with multiple associated accounts.
Metamask support

it can be enabled using VITE_APP_ENABLE_METAMASK environment variable.
Import to Metamask now automatically associates token if needed

Wallet management

WalletManager now exposes a list of accounts (WalletManager.accountIds)
one account is the active one (WalletManager.accountId)
operations (changeStaking(), approveHbarAllowance() …) applies the active account

Related issue(s):
Fixes #668, #458
Checklist

[ ] Documented (Code comments, README, etc.)
[x] Tested (unit, integration, etc.)

LG.


Add pre-release version changelog to version list

Depends on (and includes) #1766 (due to an unavoidable merge conflict otherwise).

Would be good to build the changelog from the correct branch somehow, I guess.

Yeah, but how would the script know that the "stable-branch-off" has
happened?
On Mon, Jun 26, 2017, 12:22 Vladimir Panteleev<EMAIL_ADDRESS>wrote:

Would be good to build the changelog from the correct branch somehow, I
guess.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dlang/dlang.org/pull/1769#issuecomment-311020871, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEKwdhHj1tJdpt2eaWKv_ez9jbbr2QWBks5sH4ZXgaJpZM4OEkCx
.

(rebased to rekick DAutoTest as the changelog has been manually added now)

SIgh, what happened with the "not yet released" text on http://dlang.org/changelog/2.075.0_pre.html?
@MartinNowak Could we please keep "not yet released" on that page up until the release date? We already had questions about it when someone found the page on Google what was up with the future release date.


docs(guide/Developer Guide): correct broken link

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug fix
What is the current behavior? (You can also link to an open issue here)
Broken external link
What is the new behavior (if this is a feature change)?
Corrected the link
Does this PR introduce a breaking change?
No
Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[x] Fix/Feature: Docs have been added/updated
[x] Fix/Feature: Tests have been added; existing tests pass

Other information:
The blog post "Creating multilingual support using AngularJS" has moved. Corrected the URL.

@Narretz Can you help out here? Iæm just trying to fix a broken link in the docs. Seems like flaky tests are stopping the build.


enable pekko theme in docs

link validator job is hanging - probably an issue downloading stuff to run the job - has been happening for last few hours across all builds - no evidence that the issue is related to this PR - so I will merge this


Webpack 4 support

Webpack 4 is currently in beta (should be released in Feb 2018) and introduce a new plugin system (called tap).

Implemented for v4.0.0.


axisAddMarginMin not working with negative values

Hi.
Thank you for your job building this component, I think it is amazing.
It seems there's an issue when using the axisAddMarginMin property in a range that include negative values.
In this example I have it like this axisAddMarginMin: false but it is still including a margin on the lower bound of the Y-axis as can be seen in this image:

That's not from axisAddMarginMin. If that setting is set to false, then it's just off, regardless of the data range. What you're seeing is just the normal y-axis scaling done by the charting engine. Your data becomes slightly negative at some point, so the scale is adjusted to reflect this. axisAddMarginMin creates problems only when your data doesn't become negative (it stays at 0). Actually axisAddMarginMin is disabled by default since V 1.0.43 and I might remove it entirely from upcoming versions.
How do you see your y-axis look like for data like this ? There are negative parts after all. And keep in mind that the positive and negative parts of the y-axis need to be symmetrical. If you don't want a negative scale to appear at all, then for ymin to 0, that will effectively cut out everything negative.

That's not from axisAddMarginMin. If that setting is set to false, then it's just off, regardless of the data range. What you're seeing is just the normal y-axis scaling done by the charting engine. Your data becomes slightly negative at some point, so the scale is adjusted to reflect this. axisAddMarginMin creates problems only when your data doesn't become negative (it stays at 0). Actually axisAddMarginMin is disabled by default since V 1.0.43 and I might remove it entirely from upcoming versions.
How do you see your y-axis look like for data like this ? There are negative parts after all. And keep in mind that the positive and negative parts of the y-axis need to be symmetrical. If you don't want a negative scale to appear at all, then force it with ymin set to 0, that will effectively cut out everything negative.

Hum so I dind't understand well the expected behaviour for axisAddMarginMin.
I think I can achieve what I need with ymin (although this doesn't seem to be documented), but this way my range will not be dynamic according the values present.
I was expecting that the graphic didn't add unnecessary margin to values present like the following

How it is right now it's decreasing a useful area of the graphic and losing resolution to the values shown.

AxisAddMargin was mostly designed to make room for a curve 'overshooting' into areas above or below the valid range of data. This happens very rarely, it's much more uncommon than originally anticipated and it only happens with spline curve graphs. As such that feature is not very useful and has been disabled by default.
The charting renderer will choose a scale that fits the data and that looks nice. For that it generates the Y ticks so that the tick spacing is even, whenever possible. Specifically it will try to avoid situations like the one in your example (the squashed -0.02 to -0.03 at the bottom). Instead it will create more space for negative values and keep things nice and symmetric. It's a tradeoff.
Use ymin/ymax if you want to limit the range. You will find several usage examples for those in the sample YAML in the readme.


Design services for Spinnaker.io

I'm working on a refresh of Spinnaker.io with the Spinnaker Docs SIG and TOC. The RFC for the refresh has been approved and is documented here.
I have found a contractor who is experienced with the infrastructure and eager to begin work on the development work outlined in the SOW. I'd like to request that the Linux Foundation creative team help with the design work outlined and collaborate with the dev contractor (Amy) on resources such as wireframes as needed.

Working with design team to coordinate

I'm also waiting to hear whether or not I should proceed with paying the dev. contractor with Armory budget (this has been approved by the SIG), or should wait to see if my funds request to the CDF has been approved by the board.
Can you let me know what to do? I don't want to commit an OSS governance faux pas

We have hired the dev contractor and are ready to begin with design services. Please advise on next steps.

I'm talking with the dev contractor now and she needs design input on the work she is doing for the Home page and other pages.
We are unable to wait any longer than about a week to secure design services from the CDF/LF. Moving forward with the dev work that is already underway requires some wireframes to work from. There's a possibility for us to hire a contract designer for the work, but this will make the spinnaker.io refresh project being funded by Armory go over-budget, as we had planned to leverage LF services.
@ccraigross are we still able to collaborate on this? What kind of timeline is possible? What are my next steps at this point given Dan's departure? Thanks!

Hi @brianwarner can you help me with this? The designer is waiting on Hero image, etc.
We can hire out a designer but since Armory is already footing the bill for development of this project, I was hoping for some more LF support for this much-needed re-platforming and update of Spinnaker.io. Either way, we need to move forward. Thanks.
CC Spinnaker Docs SIG @dorbin @brian-armory @aimeeu

Hey @brianwarner, thanks for sharing this. I want to confirm - is the designer who created this engaged with our developer (@amygroshek), and able to work on the project? We need more than just a hero design. Re: choosing between these designs, it is a bit difficult to choose without considering the other elements. This is what the frontend designer from Netflix on our SIG said:

It’s a difficult call to make in isolation, because the hero image is kind of like your opening statement on how the rest of the design will go. The statement should echo throughout the rest of the design. So my answer would be whatever gives the designer enough liberty to play off of their declared visual cues elsewhere, vs a hero image that doesn’t match everything else.
Taken in isolation, I like the first one the best, but I want to see where these color selections, photos of sailboats, and splashes of water go. I’m a sucker for good illustrations too, so if the hero has the same illustrative style as secondary elements, that could be great.

This is fine for a first attempt but we're not super in love with any of these proposed hero's. We need a designer who can get engaged in our project on GitHub/Slack, listen to our requirements and preferences, and iterate with us. Do we have that from the LF team?

So we need design concepts that show us other concepts besides the hero, and that are based on our current mock-up for the new site (not just the home page of the existing site, which is changing significantly with the redesign). The mock-up is linked in the RFC.

Hi @dnilasor, thanks for the feedback, I've passed this to the LF team.

Great! Thanks @brianwarner . I have a bit more, from our dev, about what she is looking for:

...the industry standard is a Sketch file or a design in Figma, something where styles can be easily copied, commented on, revised, etc. You might not have that option… But that’s definitely what I would try for.
b) This is also very far from the Tensorflow site you provide as an example. Ideally I think you would be able to get very close to that, with different color scheme and motifs but a similar aesthetic. I can get you very close without any help, but some of the very specific background patterns & etc we would need a designer for.

So that's the kind of engagement and artifacts we are hoping for - Sketch or Figma files and help with advanced designs

Following up. @brianwarner, we are looking for a yes/no on whether this is the kind of support that an LF designer can offer. @Amygroshek still hasn't heard from them to my knowledge.

All, I chatted with Tracy and Rosalind offline, and due to a variety of constraints we agreed this probably doesn't make sense to come from the LF. I'll go ahead and close out the issue.


⚠ Bofrost.lu has degraded performance

In a72b38b, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1156 ms

Resolved: Bofrost.lu performance has improved in 52b9b78 after 6 minutes.


ATTENTION ScheduleWithCron don't work as expected

The code I use:
task, err = ts.ScheduleWithCron(func(ctx context.Context) {
	fmt.Print("Scheduled Task With Cron\n")
	ierrors.Println(logTag, "Scheduled Task With Cron\n")
}, "* * * * * *", chrono.WithLocation("Etc/UTC"))

I would expect that the cronjob here will run every second, but in real it runs every "tick"
....

Scheduled Task With Cron
2021-12-10T21:48:15.307977068Z (Fight): Scheduled Task With Cron

Scheduled Task With Cron
2021-12-10T21:48:15.308020282Z (Fight): Scheduled Task With Cron

Scheduled Task With Cron
2021-12-10T21:48:15.30807167Z (Fight): Scheduled Task With Cron

Scheduled Task With Cron
2021-12-10T21:48:15.308107834Z (Fight): Scheduled Task With Cron
....

also its not working to use
styles like:
*/5 * * * * *
0 0-59/5 * * * *
0 */5 * * * *
0 1,6,11,16,21,26,31,36,41,46,51,56 * * * *
....

this will result also in the run in "loop" or also like in while true loops.
This will result in a very very bad DoS because the CPU load goes into the sky.

Please have a look to https://github.com/robfig/cron/blob/master/parser.go to get inspired for the fix.

This will work 0/3 0/3 * * * * as expected.

Thank you for reporting these issues, I'm gonna try and reproduce them, if the library does not work as expected, I'm gonna fix them as soon as possible.

When I tried your code snippet above, it worked as expected.
Could you please make sure that the library you are using for output prints the time correctly? As you see, I'm using log.Print function and everything is okay.
ts := chrono.NewDefaultTaskScheduler()
task, err := ts.ScheduleWithCron(func(ctx context.Context) {
		log.Print("Scheduled Task With Cron\n")
}, "* * * * * *", chrono.WithLocation("Etc/UTC"))

However, When I used the following patterns, I confirm that it causes an infinitive loop.
*/5 * * * * *
0 0-59/5 * * * *
0 */5 * * * *
0 1,6,11,16,21,26,31,36,41,46,51,56 * * * *

This bug fix will be delivered in the next release version.

Note: This issue will be closed within 7 days due to inactivity. please give needed information.

I will check this again.

Did you get a chance to check the issue? Due to inactivity, I have to close this issue. If the same issue occurs, could you re-open the issue?


[WIP] Remove argument processing from the tweaker

Umm I didn't intend for this to be merged yet... hence WIP

Well actually I guess Minecraft wouldn't have loaded without them...


Databricks is also supported by FAC

Issue in ./help/start/release-notes.md
The list of supported data warehouses under What cloud warehouses are supported? is outdated. At least Databricks has been added recently. Please update the list.
Ref: FAC October '24 release

Thanks for your feedback @pmonjo, the section has been updated. Changes will be live later today.
Internal reference: DOCAC-11341


Pizza Parties no longer flood the station with more than it could ever eat.

About the PR

Massively decreases the amount of pizza the station gets in both pizza party events, fixing issue #28395. Additionally, the large pizza party event now has a minimum player count to occur.
Why / Balance

The small and large pizza party events give you 16 and 64 pizzas respectively. This is far, far more than the station can actually use and is essentially a chance each round for the chef to be put out of a job.
Technical details

Changes like two numbers, this is very much baby's first howdoicode PR.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

It was always weird that event used emergency pizza crates instead of normal ones, yeah. Alternatively it could just be a single emergency pizza crate instead of multiple.


Elbrus do not publishes the topic /map. How can i use it with nav2?

nav2 uses openvlsam is default. But I want use this project. But I get the message /map topic is not published. Can i use Elbrus and nav2? Or is there other engines for navigation? Are there examples for this? My ros2 version is humble.

We integrate Isaac VSLAM into Nav2 as an odometry source but not as a map/global localization source yet. What does your sensor setup look like and what workflow you're trying to build? The Nav2 community was developing full visual mapping & navigation using OpenVSLAM if I recall but I'm not sure what the status is. OpenVSLAM had shut down in favor of OrbSLAM at some point.

I have to connect up to 4 Zed 2i cameras (instead high-cost lidar)  to Jetson Xavier NX and get the navigation. Because Elbrus don't uses CUDA in Isaac SDK I want to make the pipe: ros2 = Zed 2i + Elbrus + nav2

In our upcoming release for Isaac ROS, we should have an example in Nvblox to use with Isaac VSLAM . Along with Nav2, you should have all of the building blocks for a visual navigation solution. We're still evaluating this on our reference robots, however, and your experience would be greatly appreciated.

Please take a look at the Nvblox/Isaac VSLAM tutorial here: https://github.com/NVIDIA-ISAAC-ROS/isaac_ros_nvblox/blob/main/docs/tutorial-nvblox-vslam-realsense.md.


[bug] Fix broken UI

Notes for Reviewers
This PR fixes the broken ui recently caused by PR
Signed commits

[ ] Yes, I signed my commits.

@theBeginner86


Referral Handler can create a new Task for each Service Request

When the Referral Handler receives a ServiceRequest and all referenced Resources, it should create a Task for the referral.
Task.Status should be "Requested"
Task.businessStatus should be "received"

Functionality is included and it works locally.


Result cell value viewer (value panel) not working for JSON and enum

System information:

Operating system (distribution) and version: Ubuntu 16.04 LTS
DBeaver version: Version <IP_ADDRESS>912011827
Additional extensions: -

Connection specification:

Database name and version: Postgres
Driver name: Postgres
Do you use tunnels or proxies (SSH, SOCKS, etc)? No

Describe the problem you're observing:
Result cell value viewer (value panel) not showing any text area for JSON and drop down for enum which can be edited
Steps to reproduce, if exist:
Select a cell with JSON or enum data type, and show the Value panel
Include any warning/errors/backtraces from the logs
JSON
eclipse.buildId=unknown
java.version=1.8.0_222
java.vendor=Private Build
BootLoader constants: OS=linux, ARCH=x86_64, WS=gtk, NL=en_US
Command-line arguments:  -os linux -ws gtk -arch x86_64
This is a continuation of log file /home/payfazz14/.local/share/.DBeaverData/workspace6/.metadata/.bak_0.log
Created Time: 2019-12-09 10:35:48.530
org.jkiss.dbeaver.model
Error
Mon Dec 09 10:44:34 WIB 2019
Can't determine object class 'org.jkiss.dbeaver.ui.data.managers.ContentValueManager'
Enum
eclipse.buildId=unknown
java.version=1.8.0_222
java.vendor=Private Build
BootLoader constants: OS=linux, ARCH=x86_64, WS=gtk, NL=en_US
Command-line arguments:  -os linux -ws gtk -arch x86_64
This is a continuation of log file /home/payfazz14/.local/share/.DBeaverData/workspace6/.metadata/.bak_0.log
Created Time: 2019-12-09 10:35:48.530
org.jkiss.dbeaver.model
Error
Mon Dec 09 10:43:37 WIB 2019
Can't determine object class 'org.jkiss.dbeaver.ext.postgresql.ui.config.PostgreEnumValueManager'

Screenshot:

Same on Windows 10

Duplicate of #7333
You can re-download 6.3.0 version from dbeaver.io - we have released hotfix version on the last week.


Refactor actions_list for clarity

Addresses #4335
Removes the need for rubocop violation suppressions by refactoring show_details to use some helper methods.
Fixes a problem with trying to show the details of an action that is not yet properly subclassing Action, instead showing the helpful error given during action listing

Looks great, thanks for cleaning up this class! I see in the PR description you explain the change with the not properly subclassing warning. The reason why it might not be a good idea to crash is that it's a breaking change: back in the days fastlane's action didn't have a standardized format. Someone might (not sure if true) still use an action that's using the old format, and hasn't yet had the chance to migrate.
However the warning has been shown for about a year now, so now might be a good time to not support them any more. It's just the documentation, so it doesn't actually break the production system.

@KrauseFx If you would be in favor of removing the checks/errors about improperly defined actions, that's OK too. Removing them would make the logic even simpler. Let me know!

Discussed with @KrauseFx in Slack, where he approved!


feat: Added LabelTextAttribute

Can be used on any serialized field to show a custom label in the inspector.

When Unity's "nicify" makes a mess, e.g. Network IDs (custom) instead of Network Ids (Unity)
When we want a more descriptive label than the actual field name.

Usage:
using UnityEngine;
using Mirror;

public class TestLabelText : MonoBehaviour
{
    [LabelText("Custom Label Here")]
    public int foo;
}

@MrGadget1024 could you post a screenshot?

@MrGadget1024 could you post a screenshot?

Description updated with sample code and screenshot.

Imho this is nice for our own projects but adding a bit unnecessary complexity in a project which is this popular

Imho this is nice for our own projects but adding a bit unnecessary complexity in a project which is this popular

I have a couple places in Mirror to use this, actually, but didn't proceed unless this got merged.

After giving this some more thought, I still think this is a bit too much scope creep.
We don't need to fix Unity itself in this project.
Users are used to 'not 100% ideal labels' and I think they can live with it for Mirror too.


New version: PSFWrite v0.3.1

Registering package: PSFWrite
Repository: https://github.com/ma-laforge/PSFWrite.jl
Created by: @ma-laforge
Version: v0.3.1
Commit: ac95ef775bd1b6d582f0ac0b21d757c0993d0930
Reviewed by: @ma-laforge
Reference: https://github.com/ma-laforge/PSFWrite.jl/commit/ac95ef775bd1b6d582f0ac0b21d757c0993d0930#commitcomment-33873852
Release notes:

Add support for Project.toml
Also:
-Deprecate REQUIRE file.
-Remove support for Julia < 1.0.

Consider enabling TagBot to make sure the release is persistent in your repo, see https://discourse.julialang.org/t/ann-tagbot-creates-tags-and-releases-for-your-julia-packages-when-theyre-registered/23084

I hope you meant the tags would be created in a "consistent" fashion.  I thought releases were always "persistent" - unless somone alters "JuliaRegistry" or deletes their package repo.
I don't particularly like the idea of giving write access to my repo if I don't have to.

Would not mind if only had permissions to create tags --> Github doesn't seem to have that level of granularity on permissions.
Would prefer if the bot could simply create something like a PR (a tag-request??) that you could merge in to tag the repo.


[Support] Add a public API to allow clearing all (static) timer groups.

Cherry-pick of a minor upstream change that lets us turn back off the annoying automatic printout of timer groups when stats-output-dir is active in swiftc (due to an earlier upstream change a few months back that accidentally turned it on).
Summary:
Formerly, all timer groups were automatically cleared when printed out. In
https://reviews.llvm.org/rL324788 this behaviour was changed to not-clearing
timers on printout, to allow printing timers more than once, but as a result
clients (specifically Swift) that relied on the clear-on-print behaviour to
inhibit duplicate timer printing on shutdown were broken.
Rather than revert that change, this change adds a new API that enables
clients that want to clear all timers to do so explicitly.
Reviewers: george.karpenkov, thegameg
Reviewed By: george.karpenkov
Subscribers: llvm-commits
Differential Revision: https://reviews.llvm.org/D50874
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@339980 91177308-0d34-0410-b5e6-96231b3b80d8

@swift-ci please test

(Reader note: assessment when discussing with folks managing the branches is that the CI bot is presently misconfigured for testing this branch, but it's a harmless change and probably ok to land anyways. So landing!)


[BUGFIX] Shows default badge when calling for a non existing repository

Relates #163

Hi @TildBJ,
thanks for your patch.
We stopped the development of version 2. We started with the development of the new version of badge poser on release/v3.0.0 branch.
I have applied your patch on it: https://github.com/PUGX/badge-poser/pull/205
Thank you so much @TildBJ!


WIP: Cross-compile Go executables locally, not in Docker.

Avoid all the complexity of the go-deps docker image by doing the
(cross-)compilation to Linux locally. This will reuse the results of
the dep ensure that everybody already runs after pulling changes
from Git. When Go 1.10 is used then its build cache will cache
dependencies in a much finer-grained way than we got with the
go-deps-based building. Since most Conduit contributors have to run
Linux-taretted Docker builds in a VM, this combination of using the
native Go execuitable + the Go 1.10 build cache should result in
substantially faster incremental rebuilds of these containers.
Expand .dockerignore so that we send even fewer files to Docker
during the build, which speeds thigns up further.
Next we can split the components of the controller into multiple
containers to further improve incremental build performance when
only a single comnponent of the controller is being modified.
Further, this is a step towards using lighter and faster container
building tools (e.g. Smith or Source-to-Image).
This should work just fine even for Go releases prior to 1.10, however
go build won't be incremental in those older versions.
Signed-off-by: Brian Smith<EMAIL_ADDRESS>

I suspect we can do a similar thing for the proxy, as well.

I suspect we can do a similar thing for the proxy, as well.

Yes, I'd like to do that too. However, the proxy, being in Rust, requires a cross-linker, which is harder to set up.

How does this change relate to the go-deps optimization in #332?

When Go 1.10 is released it will change how go install and go build cache dependencies. Some of what I did in #332 and other changes would likely need to change at last a bit for Go 1.10 in order to preserve the performance wins. #332 is a more conservative approach that's less work to finalize so I'm planning to merge #332 and then eventually (hopefully soon) replace it with this. This PR (#341) is an alternative to adjusting what is done in #332 that gets about the same performance when we switch to Go 1.10.

Our existing Docker build setup does not require Go installed on the host system, are we ok moving away from that?

That's true, but in practice everybody hacking on the Go code in Conduit has Go installed and often (usually? almost always?) builds Conduit's Go components natively instead of using bin/docker-build*, until they need to test their changes in Kubernetes. In part this is because an edit-build-test loop using bin/docker-build* and minikube is too slow. My short-term goal is to speed up the edit-build-test loop for components that depend heavily on Kubernetes and can't work outside of Kubernetes, like the TLS infrastructure. My long-term goal is to make the edit-build-test loop using minikube so efficient that we no longer need non-Kubernetes dev environments like the docker-compose environment.
I've already started writing a script (for Linux) that downloads exactly the right version of Go needed, verifies its sha-256, and installs it in a conduit-tools directory. I think this approach can be extended to other tools (e.g. the Rust tools, kubectl, minikube, etc.) and for other operating systems. We can eventually add a Git pre-submit hook that checks that the submitter has the right version of the toolchain installed.

Either they're hacking on the proxy and its libraries independently of the Go components, in which case this PR won't affect them, or they're using the Go components unchanged in which case they can docker pull the images tagged master and use them, or they need to modify the Go components in which case they'll probably appreciate a better edit-built-test cycle for the Go code.

Related to 1, our Docker builds guarantee the Go version, along with the state of dependencies. With this change, we're relying on the Go version of the host system, and whether the user has run dep ensure recently. Is moving away from this an acceptable risk? Are there other ways to guarantee build consistency between contributors?

People hacking on the Go code already need to know how to do dep ensure at the right times already because we often avoid docker-build* in favor of building locally, for performance reasons. So I don't think dep ensure is a burden. We can probably add a Git pre-submit hook to check that Gopkg.lock and Gopkg.toml are consistent. We can also do a similar check in CI.

My primary concern here is when I see a Docker image, I like to know exactly how it was built.

Yes, this is a goal for me too. I think we can get this by having a script (for Linux, at least) that downloads and installs the exact versions of the tools we need and checks the SHA-256 hashes. Then we can run that script + the build in a container (or even in a full VM) like we currently do, to generate the release images.

I'm going to close this and rework things a bit, based on feedback from @siggy and @olix0r.
In the next iteration, i will make it so that one can compile the executables in the the images locally, without doing so in docker build, but the docker build way will still be support and still be the default.
Also, I think I'll do the next iteration of the experiment specifically for the Conduit CA containers, instead of for the entire control plane. That way my changes will be less disruptive to others. Once we've proved the concept with the Conduit CA component, we can apply it to the rest of the control plane and to the proxy.


Python 3.13 migrator fails with KeyError: 'patch' in (recently added) ops code

Am seeing the following error with the polars migration in this bot job:
Traceback (most recent call last):
  File "/home/runner/work/cf-scripts/cf-scripts/cf-scripts/conda_forge_tick/auto_tick.py", line 721, in _run_migrator_on_feedstock_branch
    migrator_uid, pr_json = run_with_tmpdir(
                            ^^^^^^^^^^^^^^^^
  File "/home/runner/work/cf-scripts/cf-scripts/cf-scripts/conda_forge_tick/auto_tick.py", line 445, in run_with_tmpdir
    return run(
           ^^^^
  File "/home/runner/work/cf-scripts/cf-scripts/cf-scripts/conda_forge_tick/auto_tick.py", line 547, in run
    rerender_info = _run_rerender(
                    ^^^^^^^^^^^^^^
  File "/home/runner/work/cf-scripts/cf-scripts/cf-scripts/conda_forge_tick/auto_tick.py", line 252, in _run_rerender
    rerender_msg = rerender_feedstock(str(context.local_clone_dir), timeout=900)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/runner/work/cf-scripts/cf-scripts/cf-scripts/conda_forge_tick/rerender_feedstock.py", line 24, in rerender_feedstock
    return _rerender(feedstock_dir, timeout=timeout, use_container=use_container)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/runner/micromamba/envs/cf-scripts/lib/python3.11/site-packages/conda_forge_feedstock_ops/rerender.py", line 48, in rerender
    return rerender_containerized(
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/runner/micromamba/envs/cf-scripts/lib/python3.11/site-packages/conda_forge_feedstock_ops/rerender.py", line 116, in rerender_containerized
    if data["commit_message"] is not None and data["patch"] is not None:
                                              ~~~~^^^^^^^^^
KeyError: 'patch'

However the error doesn't seem to indicate an issue with polars
Instead it leads back to the code in this repo. It looks like the line with the error was added last week with PR: https://github.com/conda-forge/conda-forge-feedstock-ops/pull/22

cc @beckermr

Another failure is molmod in this job. Perhaps that is an easier case to debug

This has already been fixed. We're waiting for the bot to retry failures. It does this basically last.

See the discussion here: https://github.com/regro/cf-graph-countyfair/issues/35
I rebuilt the container by hand to fix the issue and the sync is now working.

This is fixed. I moved the lock file back to cf-scripts.

Thanks Matt! 🙏


rowSizeList can not open the list menu

like the picture shows. click the arrow button but no response. I don not know why.

That's probably because you might be on React 16 and material ui 0.19.x
I solved this by upgrading react-tap-event-plugin to 3.0.0 and doing this at my entry point
import injectTapEventPlugin from 'react-tap-event-plugin' injectTapEventPlugin()
After doing this the menu works but it doesn't appear at the bottom of the table. It appears at the top of my webpage!


Unable to Import back into REDIS

In redis Commander, I click "More... -> Export". I then choose my connection. I then choose "JSON". I then click "Export". A file named db.[someNumber].redis gets downloaded to my machine. Later, when I go to import this data, I'm unable to do so.
I click "More... -> Import".  I then choose my connection. There is a big text box. I was expecting a file upload button. Still, I open the file that I downloaded, copy the JSON to the clipboard, and paste it into the big text box. I then click "Import". When I do this, I get a success message that says 0 inserted and 0 errors.
It seems that there isn't a way to re-import data after its been exported.

There is a bug (possibly multiple) in the way data is exported
I had to write a script to clean up the output
Also beware: when you reimport the data the whole dataset is dropped before the new import commences. If the latter operation fails you would have lost all the data
I will try and put together a pull request

Still no pull request?

My last pull request https://github.com/joeferner/redis-commander/pull/272 will fix the export/import when you run it inside a docker container. When running with simple git checkout you have to patch the used redis-dump library with the patch file provided (have a look at the Dockerfile line 21 for the exact command to run). I created a pull request for the redis-dump package too, but this is not merged as of now (https://github.com/EvilDevRu/node-redis-dump/pull/1)
At least the REDIS version of the import is working now. The library used by RedisCommander does not support import of JSON format (as far as i understand).

As said JSON import is not available from importer lib used, only REDIS commands are working.
For flushing database before importing, there is a configuration param (redis.flushOnImport) to either check or uncheck this checkbox per default. This can be set via ENV-var or cli param too. Will probably help the one or other...


Querying null or not null values

Hi,
we are evaluating this awesome piece of software ! And it looks really amazing, thank you very much!
I'd like to know if there is a way to query null or not null values.
Something like:

{
	myTestQuery(paramOne: "paramOneValue") {
		totalCount
		nodes {
			consumerCategories(orderBy: position, deletedAt: !null) {
				nodes: {
					rowId
					name
				}
			}
		}
	}
}
```

Thanks

Xavier

Not currently, we are looking at ways to improve our free form filtering if you want to comment/contribute in #47.
However, this can be done using a procedure in the meantime. Something like:
create function person_undeleted_categories(person person) returns setof categories …

See the docs on procedures for more 😊

Thanks for your reply,
yes we will contribuate on #47

Thanks! 😊


Cross-site Scripting issue

Hi mate!
Your app suffer from a Cross-site Scripting (XSS) issue.
File: index.php
Line: 62
Vuln:

You should clean the inputs prior to print them to screen.
Cheers!

Fixed in bbfb2ea


Petkit Fresh Element Gemini NOT SUPPORTED

Please, add support for https://petkit.com/products/fresh-element-gemini (model "D4s").
I tried to get it work with your plugin but I always get the error message:
15/1/2023, 15:23:29 Petkit Bridge starting
15/1/2023, 15:23:41 Petkit Bridge Loaded plugin 'homebridge-petkit-feeder-fresh-element'
15/1/2023, 15:23:41 Petkit Bridge Loading 1 accessories...
15/1/2023, 15:23:41 Petkit Bridge WARNINGunhandled rejection: TypeError: Cannot read properties of undefined (reading 'DropMeal')
15/1/2023, 15:24:46 Petkit Bridge Configuration change
15/1/2023, 15:24:46 petkitbridge stopping
15/1/2023, 15:24:51 Petkit Bridge starting
15/1/2023, 15:24:59 Petkit Bridge Loaded plugin 'homebridge-petkit-feeder-fresh-element'
15/1/2023, 15:24:59 Petkit Bridge Loading 1 accessories...
15/1/2023, 15:27:36 petkitbridge stopping
15/1/2023, 15:38:02 petkitbridge stopping

Also having this issue:

16/1/2023, 10:55:09 Petkit Bridge starting
16/1/2023, 10:55:17 Petkit Bridge Loaded plugin 'homebridge-petkit-pet-feeder'
16/1/2023, 10:55:18 Petkit Bridge Loading 1 platforms...
16/1/2023, 10:55:18 Petkit Bridgepetkit_pet_feeder [INFO] Begin to initialize Petkit feeder platform.
16/1/2023, 10:55:18 Petkit Bridgepetkit_pet_feeder [INFO] Petkit feeder platform loaded.
16/1/2023, 10:55:18 Petkit Bridgepetkit_pet_feeder ERROR [ERROR] TypeError: Cannot read properties of undefined (reading 'forEach')
16/1/2023, 10:55:18 Petkit Bridge Bridge is running on port 51836.

I have not abandoned it but for important reasons I have had to prioritize other things. The first week of February I start with it.
Jubepue

I have not abandoned it but for important reasons I have had to prioritize other things. The first week of February I start with it.
Jubepue

Hello man,
Any news?
Thanks

Hello!
Sorry for the delay but I was working on a new plugin to integrate all the petkit accessories in a single platform, it detects the accessories and creates, deletes and updates them automatically. You can install it on your homebridge by searching for "homebridge-petkit-platform". currently only works with feeder models but soon, eversweet and pura. https://github.com/jubepue/homebridge-petkit-platform

Hello! Sorry for the delay but I was working on a new plugin to integrate all the petkit accessories in a single platform, it detects the accessories and creates, deletes and updates them automatically. You can install it on your homebridge by searching for "homebridge-petkit-platform". currently only works with feeder models but soon, eversweet and pura. https://github.com/jubepue/homebridge-petkit-platform

thank you, so Fresh Element Gemini now being supported in "homebridge-petkit-platform" ?

Yes!jubepueEl 14 abr 2023, a las 5:38, imchesterhk @.***> escribió:
Hello! Sorry for the delay but I was working on a new plugin to integrate all the petkit accessories in a single platform, it detects the accessories and creates, deletes and updates them automatically. You can install it on your homebridge by searching for "homebridge-petkit-platform". currently only works with feeder models but soon, eversweet and pura. https://github.com/jubepue/homebridge-petkit-platform
thank you, so Fresh Element Gemini now being supported in "homebridge-petkit-platform" ?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were assigned.Message ID: @.***>


fix(ast): rename TSIndexSignatureName in JSON AST

Fixes #2656.

#2664  👈
main

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

Join @overlookmotel and the rest of your teammates on  Graphite

I'm going to merge this one as it's uncontentious I think. But leaving the rest of the stack for Boshen to review.


Properly overwrite empty allowed source ranges for load balancers

Fix the bug with the default range value being always overwritten. Even when allowedSourceRanges was unset in a PG manifest, the relevant slice was empty and had zero length, thus causing the overwrite of the supposed default value with the empty slice. Switching to >0  instead of >=0 should fix this.
Log the final source ranges that make it to a service spec for debugging purposes. No that the generateService() responsible for the spec is called in many places, and there is a chance that the desired source range is already applied or will not be applied for some reason such as a Sync failure.
Briefly document the intended behavior of the allowedSourceRanges field

Coverage remained the same at 22.778% when pulling 41c0c547a54091b33d61056eeca9de7f47fc40e6 on fix-default-source-range into 688d252752e8d1240cf8f1fbb23c1ff4fc52ea28 on master.


Any way of debugging the nodejs code?

Hey there,
When writing some nodejs code, how can I debug it as I go along? I need to sync/build/run every time?

Unfortunately, these steps are required to get your node.js project running on mobile devices. And if I'm informed correctly, the node.js mobile toolkit doesn't currently support to attach a debugger.
But you could test/debug your node.js part separately from your app by running it locally as a standalone project. You could then use all Node.js debug features. But you'd need to add some kind of a wrapper bridge to simulate the message exchange between the Node.js project and the application.

Wouldn't it be possible to open the debugger port optionally? Or is that not available via the embedded API?

No, unfortunately not.
The nodejs-mobile toolkit, which is used in this project to embed Node.js in mobile applications, doesn't provide the intl (internationalization) module. And since the V8 inspector depends on this module, the inspector/debugger is currently not available. ^1

I currently use a build with intl (with small icu) so it works fine. The author of nodejs-mobile will likely make a build with intl. It increases the size by 10mb, so he might want to want provide it separately.  If you can make it condtional on intl's existence, then I can try it out.


Rainloop set rights (to be able to change admin password)

Hi,
if you can´t change the admin password in admin panel you have to check the rights.
cd /var/www/
find . -type d -exec chmod 755 {} \;
find . -type f -exec chmod 644 {} \;

chown -R www-data:www-data .

perhaps it is possible to set the rights on docker build.
Cu
Robin

Weird, I remember this working without having to do anything. Anyway, thanks for the info.

I got this problem on normal unchanged configuration. Perhaps the rights has to be set a little bit later in script.

Hello!
You have a new message, please read http://wakokubag.com/accusation.php?yhsqa
<EMAIL_ADDRESS>

Hello!
You have a new message, please read http://gratefulexistence.com/fight.php?gd
<EMAIL_ADDRESS>


Fixing missing parameter error in unused keygen function

This is a typo fix for an unused function that was showing a linting error due to a missing parameter.

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


Could not get tests to pass

Although we lack details of of which errors were faced during the tests, PR #22 and #23 improve them
Please, if you still facing issues do not hesitate to re-open this issue with further details


Update Dev Drive card in whats new page based on Dev Drive feature availability

Summary of the pull request

Dev Drive has now been released in the Dev Channel - See : https://blogs.windows.com/windows-insider/2023/05/24/announcing-windows-11-insider-preview-build-23466/
Updated the min build number for the Dev Channel to be 23466 in the Dev Drive util
Updated the Dev Drive Card to only show the link when the feature is enabled
Updated the Dev Drive Card's button text to show the "Learn more" text when the Dev Drive feature is not available.
Updated button click behavior based on  Dev Drive feature availability

When disabled -> we launch the Dev Drive support page
When enabled -> we launch the disks and volumes settings page.

When feature is disabled:
https://github.com/microsoft/devhome/assets/105318831/f462c5ea-77d4-457b-a1ba-9b95e7db1b05
When feature is enabled.
https://github.com/microsoft/devhome/assets/105318831/f6b9ead4-b561-4491-add0-09ae0773ded3
References and relevant issues
Detailed description of the pull request / Additional comments
Note: I'll need to also submit another PR over at learn.microsoft.com for the Dev Drive support page: with info on which builds we expect Dev Drive to be available on: https://learn.microsoft.com/en-us/windows/dev-drive/
Validation steps performed
Tested locally on machine.
PR checklist

[X] Closes #725
[X] Closes #792
[ ] Tests added/passed
[ ] Documentation updated

/azp run

/azp run


(optional?) Dependencies don't allow perbase to be used or compiled on Ubuntu 18.04.

Dear perbase authors,
thank you for your great tool, I have been looking for a while to find an optimized alignment parser. It works great as is on newer versions of Ubuntu, however on Ubuntu 18.04 neither the pre-compiled version works (due to a GLIBC 2.28+ requirement), nor is it compilable (due to cmake package requirement coming from the rust cmake 0.1.48 package. which uses the --parallel parameter that is not implemented in the cmake package available on Ubuntu 18.04). I could fix this by lowering the rust cmake package to 0.1.45, which as far as I could tell did not affect other things during compiling.
https://github.com/sstadick/perbase/compare/master...patbohn:perbase_ubuntu18.04:master
If you don't mind, you could lower the rust cmake package version to 0.1.45 (or maybe the documentatino in this Issue alone will help others with similar requirements).
Best wishes,
Patrick

Hi Patrick! Thanks for making an issue!
I think my preferred method of fixing this would be to add another target to my CI jobs for Ubuntu 18.04 and get an updated cmake env installed in that environment.
If there is a way to specify a sub-dependancy version that doesn't require altering the lock file I'd also be up for that.
I'm not sure when I'll get to updating CI for this project to do that - but if you get something working I'm totally open to a PR.

@patbohn - could you try out the binary here: https://github.com/sstadick/perbase/releases/tag/v0.8.5? I added Ubuntu-18.04 to my CI release matrix with the blind hope that it's using an older version of glibc and so should work on others Ubuntu 18.04 instances.

Yes, that worked! Thanks :)


Add Key Updates for CosmosClient

Description
Adds the ability to update the account key in a CosmosClient via an AzureKeyCredential.
A client constructed using a AzureKeyCredential will utilize that credential to generate the token hashes. The AzureKeyCredential.Update method allows the key (and the resulting hash function) to be updated so all subsequent requests utilize the new hash function.
The underlying implementation of AuthorizationTokenProviderMasterKey has been updated to use a ComputeHashFunctionProvider instead of keeping its own IComputeHash function. The concrete implementation of the ComputeHashFunctionProvider abstract class based on an AzureKeyCredential will automatically update the hash function if it detects that the key has changed since the most recent access of the hash function.
Type of change
Please delete options that are not relevant.

[X] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

Closing issues
Closes #789

Thanks @prasadu-microsoft for contribution.
Personally, I like AzureKeyCredential module from Python/JAVA: https://docs.microsoft.com/en-us/python/api/azure-core/azure.core.credentials.azurekeycredential?view=azure-python
@j82w any idea of equivalent in .NET?
Storage has its own full type also might mean that .NET didn't have one?

Thanks @prasadu-microsoft for contribution.
Personally, I like AzureKeyCredential module from Python/JAVA: https://docs.microsoft.com/en-us/python/api/azure-core/azure.core.credentials.azurekeycredential?view=azure-python
@j82w any idea of equivalent in .NET? Storage has its own full type also might mean that .NET didn't have one?

I looked at the link you provided - it doesn't seem like it lends itself to our use case since the credential class in our case must update its hashing function when the new key is set. So, even if Azure had its own implementation, it seems like we would need to have our as well?

After our offline discussion, I've changed the implementation to use AzureKeyCredential instead of introducing a new type.

Closing this out in favor of #3276


Introduce auto updates

For auto updates to work, we will need a server running

[ ] https://github.com/ArekSredzki/electron-release-server
or
https://github.com/Aluxian/squirrel-updates-server

[ ] Update the application code acordingly

https://github.com/mattermost/desktop/pull/162

This seems to be an duplicate of #15

That's correct @Razzeee, I'll close this issue as a duplicate of #15 and have carried over your initial comment there.


The ability to transform polygon shape to polygon track.

My actions before raising this issue

[ ] I install CVAT on my Linux Ubuntu 20.04 server by this instruction link to docs. Then create a supervisor user and login. CVAT works perfectly.
[ ] I create a task for labeling from the set of images.
[ ] I read docs about "Track mode with polygons" link.
[ ] Then I create track polygon on frame 1 (step 1) and press N key to finish it (step 2), moved to the frame 3 (step 3), press Shift+N to create the new polygon (step 4) and press N to finish (step 5).
[ ] I read docs about CVAT AI Tools AI tools. I was very interested in Interactors for Semi-automatic segmentation like DEXTR, f-BRS, HRNet, Inside-Outside-Guidance.
[ ] I follow guide in this page guide how install and run AI tools. I successfully launched DEXTR and it worked.
For DEXTR running we have this commands:

nuctl deploy --project-name cvat \
  --path serverless/openvino/dextr/nuclio \
  --volume `pwd`/serverless/common:/opt/nuclio/common \
  --platform local

It is upload a special docker image with DEXTR, run it, and then CVAT can interact with it.
My idea and its description:
I want to track some object, for example car1. I got polygon shape from DEXTR for car1 object on frame 1. Then I got polygon shape from DEXTR for car1 object on frame 7. I want to track car1 from frame 1 to frame 7. But as I understand I can't because DEXTR give me polygon shapes.
Some other intresting example. I have dataset with polygon shapes and I want to change it to tracking objects.
It would be great to be able to simply combine polygons of the same object into one track.
For example set to first polygon on frame1 you are belong to track1, then go to polygon on frame 2 set the same you are belong to track1 and ect.
Expected Behaviour
There is no bug in this case, there is a request to add additional functionality.
Current Behaviour
We can use integrators like DEXTR only for instance segmentation, but not for tracking.
The dataset in which the data is labeled for instance segmentation, as I understand, cannot be converted to tracking.
Possible Solution

Add the ability to convert first polygon shape to first key polygon track and convert second polygon shape to last key polygon track to make interpolation between them.
Add the ability to combine different polygons on consecutive frames in one track.

Steps to Reproduce (for bugs)
There is no bug in this case, there is a request to add additional functionality.
Context
This will allow the use of interactors such as DEXTR for tracking objects.
This will allow to combine different polygons on consecutive frames in one track. I think converting the dataset from instance segmentation(poly shapes) will be faster than labeling it from scratch from source images.
Your Environment
I have everything working according to the instructions. I'm just offering a new functionality.

Hi,
You can use "Merge" feature.
On track one enable merging mode and click the polygon shape.
Go to frame seven and click the polygon there.
Finish merging by clicking Merge button again.
Go to frame two and remove outside keyframe there.  (Outside keyframes are automatically added if there is not shape selected on frame N+1 after selected shape on frame N)
Now you get polygon track on frames 1-7. To correct interpolation set similar start point and direction using these buttons:

Thank you very much for fast answer.
I try it and it works )
It may be good to add a small addition to the documentation that combining tracks works with both tracking polygons and polygon shape.

@zhiltsov-max
Do you have any suggestions about different behaviour in MOT and COCO?

Hope you got your answers. I will close the issue.
Do not hesitate to reopen if you still have questions

Thank you for fast answering and help )
I tried to follow your advice and I get some following problems.
I create a project and a annotation task for it from 10 images. I annotate ine tracking box and one tracking polygon to test export to MOT dataset.
I go to my project and I export it as Datumaro dataset zip archive without images. I unzip it first to the same folder.
I have a "annotations" folder with "default.json" file inside.
I work with anaconda python. To guarantee a clean experiment, I created a new working environment specifically for the package datumaro.
I run command inside my python 3.10.8 virtual environment for datumaro:
pip install datumaro[default]
I have virtual environment with these modules:
# packages in environment at /home/admin-gpu/anaconda3/envs/datumaro:
#
# Name                    Version                   Build  Channel
_libgcc_mutex             0.1                        main
_openmp_mutex             5.1                       1_gnu
aiohttp                   3.8.3                    pypi_0    pypi
aiohttp-retry             2.8.3                    pypi_0    pypi
aiosignal                 1.3.1                    pypi_0    pypi
amqp                      5.1.1                    pypi_0    pypi
antlr4-python3-runtime    4.9.3                    pypi_0    pypi
appdirs                   1.4.4                    pypi_0    pypi
async-timeout             4.0.2                    pypi_0    pypi
asyncssh                  2.12.0                   pypi_0    pypi
atpublic                  3.1.1                    pypi_0    pypi
attrs                     22.1.0                   pypi_0    pypi
billiard                  <IP_ADDRESS>                  pypi_0    pypi
bzip2                     1.0.8                h7b6447c_0
ca-certificates           2022.10.11           h06a4308_0
celery                    5.2.7                    pypi_0    pypi
certifi                   2022.9.24       py310h06a4308_0
cffi                      1.15.1                   pypi_0    pypi
charset-normalizer        2.1.1                    pypi_0    pypi
click                     8.1.3                    pypi_0    pypi
click-didyoumean          0.3.0                    pypi_0    pypi
click-plugins             1.1.1                    pypi_0    pypi
click-repl                0.2.0                    pypi_0    pypi
colorama                  0.4.6                    pypi_0    pypi
commonmark                0.9.1                    pypi_0    pypi
configobj                 5.0.6                    pypi_0    pypi
contourpy                 1.0.6                    pypi_0    pypi
cryptography              38.0.4                   pypi_0    pypi
cycler                    0.11.0                   pypi_0    pypi
datumaro                  0.3.1                    pypi_0    pypi
defusedxml                0.7.1                    pypi_0    pypi
dictdiffer                0.9.0                    pypi_0    pypi
diskcache                 5.4.0                    pypi_0    pypi
distro                    1.8.0                    pypi_0    pypi
dpath                     2.1.1                    pypi_0    pypi
dulwich                   0.20.50                  pypi_0    pypi
dvc                       2.36.0                   pypi_0    pypi
dvc-data                  0.28.3                   pypi_0    pypi
dvc-http                  2.27.2                   pypi_0    pypi
dvc-objects               0.14.0                   pypi_0    pypi
dvc-render                0.0.14                   pypi_0    pypi
dvc-task                  0.1.6                    pypi_0    pypi
dvclive                   1.1.0                    pypi_0    pypi
filelock                  3.8.0                    pypi_0    pypi
flatten-dict              0.4.2                    pypi_0    pypi
flufl-lock                7.1.1                    pypi_0    pypi
fonttools                 4.38.0                   pypi_0    pypi
frozenlist                1.3.3                    pypi_0    pypi
fsspec                    2022.11.0                pypi_0    pypi
funcy                     1.17                     pypi_0    pypi
future                    0.18.2                   pypi_0    pypi
gitdb                     4.0.10                   pypi_0    pypi
gitpython                 3.1.29                   pypi_0    pypi
grandalf                  0.6                      pypi_0    pypi
h5py                      3.7.0                    pypi_0    pypi
hydra-core                1.2.0                    pypi_0    pypi
idna                      3.4                      pypi_0    pypi
iterative-telemetry       0.0.6                    pypi_0    pypi
kiwisolver                1.4.4                    pypi_0    pypi
kombu                     5.2.4                    pypi_0    pypi
ld_impl_linux-64          2.38                 h1181459_1
libffi                    3.4.2                h6a678d5_6
libgcc-ng                 11.2.0               h1234567_1
libgomp                   11.2.0               h1234567_1
libstdcxx-ng              11.2.0               h1234567_1
libuuid                   1.41.5               h5eee18b_0
lxml                      4.9.1                    pypi_0    pypi
matplotlib                3.6.2                    pypi_0    pypi
multidict                 6.0.3                    pypi_0    pypi
nanotime                  0.5.2                    pypi_0    pypi
ncurses                   6.3                  h5eee18b_3
networkx                  2.8.8                    pypi_0    pypi
nibabel                   4.0.2                    pypi_0    pypi
numpy                     1.23.5                   pypi_0    pypi
omegaconf                 2.2.3                    pypi_0    pypi
opencv-python             <IP_ADDRESS>                 pypi_0    pypi
openssl                   1.1.1s               h7f8727e_0
orjson                    3.8.3                    pypi_0    pypi
packaging                 21.3                     pypi_0    pypi
pandas                    1.5.2                    pypi_0    pypi
pathspec                  0.9.0                    pypi_0    pypi
pillow                    9.3.0                    pypi_0    pypi
pip                       22.2.2          py310h06a4308_0
prompt-toolkit            3.0.33                   pypi_0    pypi
protobuf                  3.20.1                   pypi_0    pypi
psutil                    5.9.4                    pypi_0    pypi
pycocotools               2.0.6                    pypi_0    pypi
pycparser                 2.21                     pypi_0    pypi
pydot                     1.4.2                    pypi_0    pypi
pygit2                    1.11.1                   pypi_0    pypi
pygments                  2.13.0                   pypi_0    pypi
pygtrie                   2.5.0                    pypi_0    pypi
pyparsing                 3.0.9                    pypi_0    pypi
python                    3.10.8               h7a1cb2a_1
python-dateutil           2.8.2                    pypi_0    pypi
pytz                      2022.6                   pypi_0    pypi
pyyaml                    6.0                      pypi_0    pypi
readline                  8.2                  h5eee18b_0
requests                  2.28.1                   pypi_0    pypi
rich                      12.6.0                   pypi_0    pypi
ruamel-yaml               0.17.21                  pypi_0    pypi
ruamel-yaml-clib          0.2.7                    pypi_0    pypi
scipy                     1.9.3                    pypi_0    pypi
scmrepo                   0.1.4                    pypi_0    pypi
setuptools                65.5.0          py310h06a4308_0
shortuuid                 1.0.11                   pypi_0    pypi
shtab                     1.5.8                    pypi_0    pypi
six                       1.16.0                   pypi_0    pypi
smmap                     5.0.0                    pypi_0    pypi
sqlite                    3.40.0               h5082296_0
tabulate                  0.9.0                    pypi_0    pypi
tensorboardx              2.5.1                    pypi_0    pypi
tk                        8.6.12               h1ccaba5_0
tomlkit                   0.11.6                   pypi_0    pypi
tqdm                      4.64.1                   pypi_0    pypi
typing-extensions         4.4.0                    pypi_0    pypi
tzdata                    2022f                h04d1e81_0
urllib3                   1.26.13                  pypi_0    pypi
vine                      5.0.0                    pypi_0    pypi
voluptuous                0.13.1                   pypi_0    pypi
wcwidth                   0.2.5                    pypi_0    pypi
wheel                     0.37.1             pyhd3eb1b0_0
xz                        5.2.8                h5eee18b_0
yarl                      1.8.2                    pypi_0    pypi
zc-lockfile               2.0                      pypi_0    pypi
zlib                      1.2.13               h5eee18b_0

Then I run command
datum transform -t shapes_to_boxes -o output_dir annotations
# or datum transform -t shapes_to_boxes -o output_dir annotations/

And I get this error
$ datum transform -t shapes_to_boxes -o output_dir annotations/

ERROR:root:'NoneType' object has no attribute 'commit'
Traceback (most recent call last):
  File "/home/admin-gpu/anaconda3/envs/datumaro/bin/datum", line 8, in <module>
    sys.exit(main())
  File "/home/admin-gpu/anaconda3/envs/datumaro/lib/python3.10/site-packages/datumaro/cli/__main__.py", line 184, in main
    retcode = args.command(args)
  File "/home/admin-gpu/anaconda3/envs/datumaro/lib/python3.10/site-packages/datumaro/util/scope.py", line 158, in wrapped_func
    ret_val = func(*args, **kwargs)
  File "/home/admin-gpu/anaconda3/envs/datumaro/lib/python3.10/site-packages/datumaro/cli/contexts/project/__init__.py", line 645, in transform_command
    dataset, _project = parse_full_revpath(args.target, project)
  File "/home/admin-gpu/anaconda3/envs/datumaro/lib/python3.10/site-packages/datumaro/cli/util/project.py", line 136, in parse_full_revpath
    return parse_revspec(s, ctx_project=ctx_project)
  File "/home/admin-gpu/anaconda3/envs/datumaro/lib/python3.10/site-packages/datumaro/util/scope.py", line 158, in wrapped_func
    ret_val = func(*args, **kwargs)
  File "/home/admin-gpu/anaconda3/envs/datumaro/lib/python3.10/site-packages/datumaro/cli/util/project.py", line 100, in parse_revspec
    if project.is_ref(proj_path):
  File "/home/admin-gpu/anaconda3/envs/datumaro/lib/python3.10/site-packages/datumaro/components/project.py", line 2555, in is_ref
    return self._git.is_ref(ref)
  File "/home/admin-gpu/anaconda3/envs/datumaro/lib/python3.10/site-packages/datumaro/components/project.py", line 1174, in is_ref
    self.repo.commit(rev)
AttributeError: 'NoneType' object has no attribute 'commit'

For what reason does it occur? =)

Seems strange, could you please check with the previous version of Datumaro (v0.3)? You can install it with pip install 'datumaro==0.3'. If not works, it deserves a bug report to the Datumaro repo. In both cases, you can use the following python script as a solution:
import datumaro as dm

dataset = dm.Dataset.import_from('annotations/', 'datumaro')
dataset.transform('shapes_to_boxes')
dataset.export('output_dataset', 'mot_seq') # add ", save_images=True" if you exported with images from CVAT

There seems to be some kind of error either in saving the file or in its path. For some reason, the script that you have given gives out such a strange error in my environment with datumaro == 0.3.1.
Traceback (most recent call last):
  File "/home/admin-gpu/Downloads/cvat_datasets/datumaro_convert.py", line 3, in <module>
    dataset = dm.Dataset.import_from('annotations/', 'datumaro')
  File "/home/admin-gpu/anaconda3/envs/datumaro/lib/python3.10/site-packages/datumaro/components/dataset.py", line 1165, in import_from
    detected_sources = importer(path, **kwargs)
  File "/home/admin-gpu/anaconda3/envs/datumaro/lib/python3.10/site-packages/datumaro/components/extractor.py", line 440, in __call__
    raise DatasetNotFoundError(path)
datumaro.components.errors.DatasetNotFoundError: Failed to find dataset at 'annotations/'

I have tried both relative and absolute paths. Seems strange. Just in case, I attached a file "annotations/default.json" to this message (default.zip).
Just in case, I saved my test dataset to datumaro format again and extracted it. But nothing has changed.
Now I will create a new virtual environment with 'datumaro==0.3' and I'll try both ways with it.

I create new virtual environment and install datumaro==0.3. When running in console mode, I was asked to install git python, then the same error returned as before.
I also ran the python script and got the same error as before. In general, the behavior of version 3.0 repeats the behavior of the new version 3.1.
Python скрипт tells that there is not dataset at 'annotations/'. Perhaps for the same reason, when reading a folder 'annotations/' through the console, the code gets a NoneType and it does not have the necessary 'commit' method that should be used further.
There is a suspicion that I seem to be getting the wrong zip archive of the datumaro dataset. And can I export with the same option of mask to shapes conversion from the cvat video data set format?
 It seems, there are several problems along the path, which need to be fixed. A quick workaround will be to modify the script above this way:
import datumaro as dm

dataset = dm.Dataset.import_from('path/to/default.json', 'datumaro')
dataset.transform('shapes_to_boxes')
dataset.init_cache()
dataset._data._media_type = dm.Image
dataset.export('output_dataset', 'mot_seq_gt')

Here is the output_dataset.zip that I've got.

Ok, I've tested the pipeline. It seems, there are several problems along the path, which need to be fixed. A quick workaround will be to modify the script above this way:
import datumaro as dm

dataset = dm.Dataset.import_from('path/to/default.json', 'datumaro')
dataset.transform('shapes_to_boxes')
dataset.init_cache()
dataset._data._media_type = dm.Image
dataset.export('output_dataset', 'mot_seq_gt')

Here is the output_dataset.zip that I've got.

Thank you very much, I also test it and it works =)
I also wanted to ask where to see which keywords correspond to the names of datasets?
How to find that keyword for intresting dataset ? (MOT challenge is ''mot_seq_gt'') ?
I search for MOT challenge dataset on documentation page link, but there is not MOT.
But I find some other and find that for
KITTI dataset we have keyword 'kitty'
COCO dataset we have keyword 'coco'.

How to find that keyword for interesting dataset ? (MOT challenge is ''mot_seq_gt'') ?

For many datasets there is documentation at link you attached, but not for all. For others, we can check in the format tests and in the full lists available in the datum import --help and datum export --help CLI output. Dataset examples are available in the test resources here, they should have their names aligned, for the most part.

datum import --help

Thank you for information and your help )
Now I understood what the difficulty was.
Keyword mot_seq' for import, but for export we should use keyword mot_seq_gt'.
I have no more questions on this topic.

Hi,
You can use "Merge" feature. On frame one enable merging mode and click the polygon shape. Go to frame seven and click the polygon there. Finish merging by clicking Merge button again. Go to frame two and remove outside keyframe there. (Outside keyframes are automatically added if there is not shape selected on frame N+1 after selected shape on frame N)
Now you get polygon track on frames 1-7. To correct interpolation set similar start point and direction using these buttons:
About merge you can find here: https://opencv.github.io/cvat/docs/manual/basics/track-mode-basics/

Is there CVAT API for doing this?

@gwengusc , it's a UI feature.


[🐞] HTML nesting error should not crash server

Which component is affected?
Qwik Runtime
Describe the bug

Reproduction
n/a
Steps to reproduce
creat invalid HTML (like

)
System Info
na

Additional Information
No response

It will crash most likely in the browser later, in a almost impossible to debug way.

I think the exception is fine. I think the issue is that the server quits and now the Dev has to restart it. So I think what we are looking for is the exception as above, but without the need to have to restart the server.


[Mod] Request Updates to BDB Netkan

Mod info

Mod name: Bluedog Design Bureau: https://github.com/KSP-CKAN/NetKAN/blob/master/NetKAN/BluedogDB.netkan

Hi could you kindly help us with the following updates to the BDB netkan

add Simple Adjustable Fairings as a hard dependency for BDB

https://github.com/KSP-CKAN/NetKAN/blob/master/NetKAN/SimpleAdjustableFairings.netkan

update the author field as follows: Cobaltwolf, JSO, Zorg

Add "Contract Configurator" to recommends
https://github.com/KSP-CKAN/NetKAN/blob/master/NetKAN/ContractConfigurator.netkan

Add the following to Suggests
SCANsat https://github.com/KSP-CKAN/NetKAN/blob/master/NetKAN/SCANsat.netkan
Dock Rotate https://github.com/KSP-CKAN/NetKAN/blob/master/NetKAN/DockRotate.netkan

Add the following to Supports

TACLifeSupport https://github.com/KSP-CKAN/NetKAN/blob/master/NetKAN/TACLS.netkan
EngineIgnitorReignited https://github.com/KSP-CKAN/NetKAN/blob/master/NetKAN/EngineIgnitorReignited.netkan
Thanks very much

Hi @zorg2044, thanks for the updates!
I'll do most of these in a pull request, but for the author change, that field is pulled from the authors list on SpaceDock. I've just added you to the authors list over there, there should be an invitation that you can accept to make it official.

Thanks a lot!


Fire CommandEvent on SimpleCommandService.call()

The SimpleCommandService does not fire a command event when the call method is used.
I don't know if this is intended. When a plugin uses the call method, there would be no way for other plugins to cancel the command execution.

It might make sense, rather than the MC ServerCommandManager calling a CommandEvent which is picked up by the SimpleCommandService, to have the ServerCommandManager directly use the call method, and have the SimpleCommandService call the CommandEvent.
Another issue is integrating the Forge CommandEvent with ours. This may be more complicated because the forge event requires an ICommandSender and an ICommand, which we don't really have if we don't use the MC CommandHandler.


Allow trying multiple entries in a multi cradle if the first one fails

At the moment, the bare Cabal cradle can't handle components other than the main library component. We can fix this with a multi component where we have a mapping for all the paths to their appropriate components. However, it would be nice to be able to just use this for the components that the bare Cabal cradle can't find.
With @fendor 's help, I tried something like this:
cradle:
  multi:
    - path: ./.
      config:
        cradle:
          cabal:
    - path: ./.
      config:
        cradle:
          cabal:
            - path: ./mylib/test
              component: mylib:mylib-tests

However, it turns out that this will only try the first component that matches a path. So depending which one I put first, either the test component or the library component will work, and the other will give an error.
It would be nice to be able to do something like this.

While I like the general idea of it, it will be a breaking change in the hie.yaml file format.
I am not sure if anyone uses cradles such as:

I think this is a suitable issue for Zurihac for whoever is interested!
It should be enough to only modify the function multiCradle
https://github.com/mpickering/hie-bios/blob/master/src/HIE/Bios/Cradle.hs#L276

Since cabal 3.4, this issue is obsolete. Usually cabal repl <filepath> works for every component in a cabal project (assuming build-type: Simple).
Going to close this, feel free to re-open if current situation is unsatisfying.


Do not send voucher when customer signs up if he did not register for newsletter

When signin up, the customer receives a voucher all the time, even if he did not register for the newsletter, this bug was introduced in this PR: https://github.com/PrestaShop/ps_emailsubscription/pull/38
Fixes https://github.com/PrestaShop/PrestaShop/issues/17990

Thanks @matthieu-rolland

Thank you @matthieu-rolland


Support full-blown interactive terminals

Turning this into a new module instead, now that it's outgrown the name "vt100:" https://github.com/vito/midterm
Will archive this repo to avoid renaming hell.


fix(datastore): avoid creating index on foreign key fields

Description of changes:
This PR introduces a change that skip SQL indexes creation when the fields involved are used to represent associations.
Check points: (check or cross out if not relevant)

[x] Added new tests to cover change, if needed
[x] Build succeeds with all target using Swift Package Manager
[x] All unit tests pass
[x] All integration tests pass
[ ] Security oriented best practices and standards are followed (e.g. using input sanitization, principle of least privilege, etc)
[x] Documentation update for the change if required
[x] PR title conforms to conventional commit style
[ ] If breaking change, documentation/changelog update with migration instructions

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1937 (7170796) into feat/datastore-custom-pk (cb5bfea) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@                     Coverage Diff                      @@
##           feat/datastore-custom-pk    #1937      +/-   ##
============================================================
+ Coverage                     59.30%   59.34%   +0.03%
============================================================
  Files                           719      719
  Lines                         22317    22327      +10
============================================================
+ Hits                          13235    13249      +14
+ Misses                         9082     9078       -4

Flag
Coverage Δ

API_plugin_unit_test
62.60% <ø> (+0.18%)
:arrow_up:

AWSPluginsCore
70.29% <ø> (+0.07%)
:arrow_up:

Amplify
48.07% <ø> (ø)

Analytics_plugin_unit_test
71.09% <ø> (ø)

Auth_plugin_unit_test
78.85% <ø> (ø)

DataStore_plugin_unit_test
75.22% <100.00%> (+0.02%)
:arrow_up:

Geo_plugin_unit_test
52.12% <ø> (ø)

Predictions_plugin_unit_test
26.81% <ø> (ø)

Storage_plugin_unit_test
58.04% <ø> (ø)

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

Impacted Files
Coverage Δ

...goryPlugin/Storage/SQLite/ModelSchema+SQLite.swift
94.25% <100.00%> (+0.74%)
:arrow_up:

...gin/Sync/InitialSync/ModelSyncedEventEmitter.swift
89.58% <0.00%> (-1.05%)
:arrow_down:

...gin/Subscribe/DataStoreObserveQueryOperation.swift
74.64% <0.00%> (-0.94%)
:arrow_down:

...eCategoryPlugin/Sync/RemoteSyncEngine+Action.swift
17.64% <0.00%> (ø)

...eAndLocalSave/ReconcileAndLocalSaveOperation.swift
88.27% <0.00%> (+0.73%)
:arrow_up:

...APICategoryPlugin/Operation/AWSRESTOperation.swift
58.22% <0.00%> (+2.53%)
:arrow_up:

...n/Operation/AWSGraphQLOperation+APIOperation.swift
42.85% <0.00%> (+3.57%)
:arrow_up:

...e/AWSPluginsCore/Auth/AWSAuthServiceBehavior.swift
87.50% <0.00%> (+12.50%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Get rid of the archived awesome-wayland

why draft

why draft

idk maybe to replace it idk

BTW, does anybody knows what happened to it? Like, why is it archived?
I've tried writing to @natpen 's mail, but received no response.

conflict

thanks


Cant start electropm

my system environment is windows 10 and node version is v12.14.0, when i start electron then occur the proble in picture : Unable to  parse package.json.

This is not enough information for us to proceed. How did you start electron? Did you follow the instructions provided?

I fllowed the guide, clone the project, cpnm install and npm start, then I have trouble in start electron.I can't locate the problem, and there are no userful information in log. I guess it could be caused by environment  infact but I don't konw how to fix. my computer is HP laptop 14s.

this issue is caused by another encrypt software


image don't show with good shape on iOS

avatarContainer: {
borderColor: '#9B9B9B',
borderWidth: 1 / PixelRatio.get(),
justifyContent: 'center',
flexDirection:'column',
alignItems: 'center',
marginTop:20
},
avatar: {
borderRadius: 100,
width: 80,
height: 80,

}

Border radius should be half of the width (height) if you want to achieve circle. Web css is more forgiving and will just round it up into circle, but in react native you can overshot and create weird shapes.

So, it works on Android exactly like it works on web, right?

@shergin from what I saw yes.

Yeah, cool, I believe we should fix it on iOS!

@shergin there has been some conversation on this topic in the past https://github.com/facebook/react-native/issues/1572  .

Seems that the issue was fixed for custom "border rendering engine", but not for CALayer based one.
Is this happening only if we apply the border radius to <Image> (not <View>) component?

Hi there! This issue is being closed because {:search=>"repo:facebook/react-native is:open is:issue updated:<2017-05-25 -label:"Good First Task" -label:"Help Wanted" ", :message=>"it has been inactive for a while"}. Maybe the issue has been fixed in a recent release, or perhaps it is not affecting a lot of people. Either way, we're automatically closing issues after a period of inactivity. Please do not take it personally!
If you think this issue should definitely remain open, please let us know. The following information is helpful when it comes to determining if the issue should be re-opened:

Does the issue still reproduce on the latest release candidate? Post a comment with the version you tested.
If so, is there any information missing from the bug report? Post a comment with all the information required by the issue template.
Is there a pull request that addresses this issue? Post a comment with the PR number so we can follow up.

If you would like to work on a patch to fix the issue, contributions are very welcome! Read through the contribution guide, and feel free to hop into #react-native if you need help planning your contribution.

We have to fix it. I will try.


How to use a key string instead of a p8 file in config?

Currently. I'm using config bellow with a p8 file (APNsAuthKey_XXXXXXXXXX.p8). I found that in the p8 file, content is a string of a private key. How to use a key string instead of a p8 file?
const options = {
token: {
key: 'path/to/APNsAuthKey_XXXXXXXXXX.p8',
keyId: process.env.APPLE_KEY_ID,
teamId: process.env.APPLE_TEAM_ID,
},
production: process.env.APP_ENV === 'prod',
};

Currently. I'm using config bellow with a p8 file (APNsAuthKey_XXXXXXXXXX.p8). I found that in the p8 file, content is a string of a private key. How to use a key string instead of a p8 file?
const options = {
token: {
key: 'path/to/APNsAuthKey_XXXXXXXXXX.p8',
keyId: process.env.APPLE_KEY_ID,
teamId: process.env.APPLE_TEAM_ID,
},
production: process.env.APP_ENV === 'prod',
};

@lamlejobchat you need to read p8 file as below:
let bufferKey = fs.readFileSync('path/to/APNsAuthKey_XXXXXXXXXX.p8');
const options = {
token: {
key: bufferKey ,
keyId: process.env.APPLE_KEY_ID,
teamId: process.env.APPLE_TEAM_ID,
},

you could do it like below.
const key = `-----BEGIN PRIVATE KEY-----
MIGTAgEAMBMGByqGSM49AgEGCCxxxxxxxxxxxxxxx
-----END PRIVATE KEY-----`

const options = {
  token: {
    key: key,
    keyId: "xxxx",
    teamId: "xxxx"
  }
}


POC: pagination

Proposed changes

Jira ticket: CLOUDP-#

Closes #[issue number]
Checklist

[ ] I have signed the MongoDB CLA
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added any necessary documentation in document requirements section listed in CONTRIBUTING.md (if appropriate)
[ ] I have addressed the @mongodb/docs-cloud-team comments (if appropriate)
[ ] I have updated test/README.md (if an e2e test has been added)
[ ] I have run make fmt and formatted my code

Further comments

This looks very good.
I see this PR introducing two different feature:

Adapts fluent api
Uses right values from structs.

IMHO second target is most improtant as it introduces a lot of bad casts in the CLI right now.
We can do it by extended support in sdk

Since then our approach for SDK have changed. We have more efficient method that involves RequestParams Object.
In future we should look to move mocks to SDK and remove store layer completely.


Create CI build

To automatically build / test the code on commit.

Git hub actions were added to implement build and test.


Support gosimports -version

It would be nice to know which version of gosimports I am using.
Expect something like:
> gosimports -version
v0.1.5

@gtxistxgao -version flag is now supported!

Thanks @rinchsan !


Set compiler-explorer.com to https

This is setup in the console manually and it seems to work

add this to alb.tf to reflect the additional cert added to the listener
resource "aws_lb_listener_certificate" "compiler-explorer-alb-listen-https-ce-cert" {
  listener_arn    = "${aws_alb_listener.compiler-explorer-alb-listen-https.arn}"
  certificate_arn = "arn:aws:acm:us-east-1:052730242331:certificate/7abed4ab-ecfc-4020-8f73-f255fd82f079"
}

(I guessed on the certificate_arn based on cloudfront terraform)

I did add the compiler-explorer.com certificate to the load balancer beforehand, I think that needs to be setup in terraform as well somewhere around here -> https://github.com/mattgodbolt/compiler-explorer-image/blob/f01ab0670fc819af1ef2d4df4d832475aab75747/terraform/alb.tf#L74

add this to alb.tf here to reflect the additional cert added to the listener
resource "aws_lb_listener_certificate" "compiler-explorer-alb-listen-https-ce-cert" {
  listener_arn    = "${aws_alb_listener.compiler-explorer-alb-listen-https.arn}"
  certificate_arn = "arn:aws:acm:us-east-1:052730242331:certificate/7abed4ab-ecfc-4020-8f73-f255fd82f079"
}

(I guessed on the certificate_arn based on cloudfront terraform)

Done, thanks. Arn was correct

Merged: terraform thinks that not all changes have been applied:
An execution plan has been generated and is shown below.
Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  + aws_lb_listener_certificate.compiler-explorer-alb-listen-https-ce-cert
      id:              <computed>
      certificate_arn: "arn:aws:acm:us-east-1:052730242331:certificate/7abed4ab-ecfc-4020-8f73-f255fd82f079"
      listener_arn:    "arn:aws:elasticloadbalancing:us-east-1:052730242331:listener/app/GccExplorerApp/4a4513393260c7c5/1dc0ebb4dd9eeb53"

Plan: 1 to add, 0 to change, 0 to destroy.

This may be due to you having already added it: will adopt it (one thing not held in source control is the mapping between AWS resources and the names in terraform)

Yay! I see you added it...and you fixed the original issue which was the certificates! Hooray!


possible to use esp-01/esp-12?

Is it possible to make an "simpler" version,and use esp 01 or esp12 etc?

Hi, this is very likely. With similar size, prices (around 3$ for the chip alone) and the option to use bluetooth with the ESP32, I was aiming for that chip, though.
Also: a lot of the ESP modules come with on-PCB antenna. Some additionally offer a u.fl to connect but need soldering to actually use it (e.g. D1 Mini Pro)

I've just ordered an ESP32-C3 ( https://wiki.seeedstudio.com/XIAO_ESP32C3_Getting_Started) which I will try out when it arrives (early april).
It is much smaller than the ESP32-wroom32u, more power-efficient and still able to run the core (hopefully ;) ) without changes.
I'll test and update here.

OK, I've tried the code on the ESP32-C3 and it doesn't work right away. The good thing is that it only needs a very small change to work. (https://github.com/atc1441/NETSGPClient/pull/27)
For me, the size was important and the ESP32-C3 is barely larger than the LC12S and does have a ESP32, USB-C and the u.fl on it.

Since the patch was taken info NETSGPClient (yay, my first patch, I'm happy :D ) the ESP32-C3 is supported.
I consider that the best platform for this project for price, size and power usage reasons compared to ESP8266 or ESP32.


API constructor failure not signalled in HTTPServer start

If the API constructor raises an exception at https://github.com/bbc/nmos-common/blob/master/nmoscommon/httpserver.py#L43 then the HTTPServer thread exits but doesn't set started (and failed). A client that is waiting for started to be set will wait forever. A similar issue could arise if the socket calls raise an exception.

Fixed in https://github.com/bbc/nmos-common/pull/107


Add administrative brew locking function

This PR is work towards #3326.
This PR adds a check to the getBrew function that throws an error if the brew stub has a lock object with a state property set to 1 or 2.
State 1: locked, inaccessible.
State 2: locked, only accessible via Edit page.
The intended purpose of this function is to allow administrators to prevent access to brews that have identified issues - for example, DMCA take down requests.

This seems fine, and I would be ok merging this to form a basis for the next steps, since it shouldn't break anything. Do you want me to merge it now or keep tinkering on it?

I'll keep tinkering for now, I've used lock.state as an integer because I anticipated having multiple states (such as completely locked, or locked for editing only). If it will stay as simple as it is, then it may be more legible to future developers if I were to use named boolean states - such as locked and editable.

Not sure if this is useful, but there is a 423 - Locked error code.

MongoDB entry:

Share Page:

Edit Page:

Includes a locked brew test:

I haven’t taken a strong look at this beyond just get comments/screenshots here, so pardon if I missed it:
i think this needs two things:  a list of possible reasons the brew is locked (just a static thing, not anything actually related to a specific brew), and a path to contacting “administrators” like a link to the Reddit mod mail or whatever.
Right now it just looks overly harsh, with no indication of what the issue could be and no further help to get it fixed. Part of this is styling (big red letters) and part of it is content.
Further, it might just be best to say “unavailable” rather than locked, or even just use the same error page as the other “no brew” error. I’m not sure we need to tell all viewers that an author is getting DMCA’d (when we aren’t the ones really arbitrating/deciding that)


Add support for custom nodeRenderer

This PR adds support for a custom nodeRenderer prop for ObjectInspector. This is a change I needed and I saw you talking about it in #19, so I thought I'd give it a shot.
Rundown of the changes:

Add support for nodeRenderer prop on ObjectInspector
Export ObjectLabel, ObjectRootLabel, ObectName, and ObjectValue so that they can be used with the custom node renderer (eg, only making minor modifications to the default renderer.)
Add support for extra styles prop on ObjectValue and ObjectName.

I'm using this library in Assertible's HTTP response viewer and need a way to show validations inline. These changes allow me to do things like this:

Let me know what you think. I'm happy to make some more modifications, if required.

I'd like to also support the nodeRenderer prop on DOMInspector and TableInspector, but I can follow-up with those in another PR.

@xyc bump

Hey if you want this released somewhere, I'm interested.
We're maintaining a fork here:
https://github.com/storybooks/react-inspector

Hey @ndelangen - I'm definitely interested in getting this in, thanks for picking up the slack!
I see from #26 that you were granted access to this repo, so is the best bet to leave the PR here? I can get the branch updated w/ the latest changes.

Hey @ndelangen - got the branch updated with the react and prop-types updates.

Hey @CodyReichert I just released version 2.0.0!
With a bunch of changes, sorry these are causing merge conflicts for you, if you need help resolving them, just ask.
Yes this is now the correct place, we'll be maintaining and releasing from here.

Are you interested in co-maintaining btw @CodyReichert ?

Hey @ndelangen - yes I am. I'd be happy to help out with bugs and PRs, or anything else really. I use this library a lot, so definitely happy to help maintain it.
PS, I messed up that last merge :p -- pushing an update now

Alright, went ahead and just updated these changes from scratch, so everything is up to date and there's not a bunch of merge commits to sift through.

Added new tests for some of the existing components and new props

@CodyReichert How do you want to proceed? I you think this is mergeable, let's do it!
D you want to release this? Is this a major / minor ?

Hey @ndelangen - thanks, that sounds good. This is ready to merge. I just documented the new nodeRenderer prop for ObjectInspector in the README, which is the last thing I wanted to do in this PR.
This is a minor version bump, since there's no breaking changes.
I do want to follow up and add a nodeRenderer prop to the DocumentInspector, but I'll do it in a separate PR so we can go ahead and release this one.

I'm gonna go ahead and pull this in, everything seems to be working well. @ndelangen - when you see this, let me know what you think about cutting a release!