Update the directory structure after like 7 months

it had been severely outdated so just updated it

O ok checks passed

why do we even need a directory tree?

why do we even need a directory tree?

Yeah you know what, I thought about it for a little while. and I think we do not need one
I think I should close the PR and the Directory Structure should be removed


huawei & ucloud bugfix

这个 PR 实现什么功能/修复什么问题:

华为云安全组同步 出方向支持空规则
Ucloud默认实例类型变更为N2
Ucloud镜像状态bugfix

是否需要 backport 到之前的 release 分支:

release/2.9.0

Codecov Report

Merging #886 into master will increase coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #886      +/-   ##
=========================================
+ Coverage    7.62%   7.64%   +0.02%
=========================================
  Files         355     355
  Lines       55100   55100
=========================================
+ Hits         4201    4213      +12
+ Misses      50501   50490      -11
+ Partials      398     397       -1

Flag
Coverage Δ

#aFlag
7.64% <0%> (+0.02%)
:arrow_up:

Impacted Files
Coverage Δ

pkg/util/ucloud/host.go
0% <ø> (ø)
:arrow_up:

pkg/util/ucloud/securitygroup.go
0% <0%> (ø)
:arrow_up:

pkg/util/ucloud/image.go
0% <0%> (ø)
:arrow_up:

pkg/util/seclib2/seclib.go
91.48% <0%> (-4.26%)
:arrow_down:

pkg/util/rbacutils/rabc.go
73.86% <0%> (+5.3%)
:arrow_up:

Continue to review full report at Codecov.

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

/cherrypick release/2.9.0

/lgtm
/approve


NameError: name 'fm' isn't defined

trying to run this code

from Maix import GPIO
fm.register(board_info.LED_B, fm.fpioa.GPIO0)
led_r=GPIO(GPIO.GPIO0,GPIO.OUT)
led_r.value(0)

But it am getting this error. Any help will be highly appreciated. Thanks in advance. I am using MaixGo.

Traceback (most recent call last):
File "", line 2, in
NameError: name 'fm' isn't defined
MicroPython v0.5.0-38-g4c4d5b6 on 2020-04-10; Sipeed_M1 with kendryte-k210

from fpioa_manager import fm


csipfs tool

Codecov Report

Merging #140 into master will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #140      +/-   ##
==========================================
+ Coverage   92.53%   92.57%   +0.04%
==========================================
  Files          47       47
  Lines        2345     2345
==========================================
+ Hits         2170     2171       +1
+ Misses        175      174       -1

Impacted Files
Coverage Δ

src/CoreApi/BlockApi.cs
97.27% <0%> (+0.68%)
:arrow_up:

Continue to review full report at Codecov.

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


Adapt to Godot 3

Some of the API has changed. I can help maybe I guess. Let me know!

Hi! Yeah, I had this in mind since they released 3.0. I would be great if you can migrate the code to use 3.0 API. Just create a pull request when you are ready. Let me know if you need some help!

Started #18  :)


Add option to disable logs generated by morgan

Morgan Express middleware now pipes messages into Debug()
Fixed via https://github.com/edrlab/r2-streamer-js/commit/6049b22fea38701c600f4c94c5a622708d4c6b97


WebUI is broken

Hi, I tried to run tribeca with docker using the instructions from README. It seems like it can't establish some connections (the console logs keep saying "connect to ..." and "disconnected from ...").
Host os: Arch Linux (Linux 4.13.12-1)
Docker version 17.10.0-ce
Browser: Google Chrome

try
$ docker logs tribeca
for more detailed logs on whats going wrong. This could happen for a whole variety of reasons. Bad API keys, bad set up on your DB, etc.

Had the same issue, turns out I didn't have the right api key for sandbox.
Changed to the live trading and it works for me now

@gleb-dianov did @heckthor92's solution work for you? If so, please close this issue - or update it if you need further help.

@ariejan @heckthor92 I can't try it at the moment, but I'll check if it works on Wednesday and I'll tell you how it goes.

@ariejan @heckthor92 @CCKRK I checked the logs, and for some reason, it can't match pair to a hitbtc symbol. In my env file I have
TradedPair=BCH/ETH
The error message is "Error: unable to match pair to a hitbtc symbol undefined/ETH\n"
I think it's because hitbtc changed the key for bitcoin cash to BCH and tribeca wasn't updated.
Also, hitbtc's demo account doesn't work at the moment, so I can't just change the pair and test it again.


gh-2885 integration test removing groupBy that does not exist in the …

…properties.
Related Issue

Resolve #2885

Codecov Report

:exclamation: No coverage uploaded for pull request base (v2-alpha@9002d7e). Click here to learn what that means.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##             v2-alpha    #2899   +/-   ##
===========================================
  Coverage            ?   73.62%
  Complexity          ?      228
===========================================
  Files               ?       19
  Lines               ?      728
  Branches            ?       57
===========================================
  Hits                ?      536
  Misses              ?      143
  Partials            ?       49

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


Hand state classifier and fully articulated hand tracker

Is your feature request related to a problem? Please describe.

Hi,
This is the most requested body tracking feature since months now (apart from MacOS and ARM support): https://feedback.azure.com/forums/920053-azure-kinect-dk/suggestions/38933653-fully-articulated-hand-tracking-and-classifiers-fo
It stays in 'Need feedback' state since a long time. Nobody replies to the user comments, too. The user experience in the feedback forum is not great overall. The only recent change is that the feature request was renamed that way. Until then it was only a request for hand state classifier.
Describe the solution you'd like

Do you plan to add any of these features to the Body Tracking SDK any time soon?
Describe alternatives you've considered

Additional context

@rfilkov why did you close this? Did it get implemented?

@RobJellinghaus There is nothing new implemented in the body tracking since v1.0. My patience has some limits.

Dop you think this is going to be implemented soon?
The hancd traking is now reaching the hand tip
https://microsoft.github.io/Azure-Kinect-Body-Tracking/release/1.1.x/group__btenums.html


Buildbucket v2 Migration

Buganizer tracking issue: https://buganizer.corp.google.com/issues/268253534

[x] Publish a new buildbucket package to pub.dev to expose the v2 protos.
buildbucket v1.0.4 has been published: https://pub.dev/packages/buildbucket/versions/1.0.4

Needed to add a significant number of packages to replace all of the objects we are using to reduce code.
All request/response objects are now from protos as well.

[ ] Integrate the buildbucket v2 api into the scheduler.

There are 3 apis that are currently using the v2 endpoint.
[x] (dart_internal_subscription.dart) Start with Dart internal for testing since it is already using the v2 api.
[x] batch_request_subscription.dart needs to be updated.
[ ] (luci_postsubmit_subscription.dart) postsubmit luci subscription - note that some fields have been changed and other removed or renamed.
[ ] (luci_presubmit_subscription.dart) presubmit luci subscription
[ ] github_checks_service.dart

[ ] Update luci_build_service.dart.
[ ] Remove the existing push_message and buildbucket dart code.
[ ] Update the topics/subscriptions currently used as the ones we are pointing to are allowlisted.
[ ] Validate the changes before releasing to prod.

Notes:

Results field from v1 has been deprecated, all status conversions must come from Build.Status.
Build_parameters field may also be deprecated.

The following validations have been made via the current apis:

Validated calls to buildbucket v2 to fetch builds.
Validated calls to buildbucket v2 to cancel builds.


Add Ceprotec MS

@Solangegmachado Could you please send your request by email to<EMAIL_ADDRESS>I can not add your school to this repository because the website suggests that it is neither a university, nor a college, but more like a training center. I am closing the request without merging. Thank you for your understanding.

INGLÊS
the school is an institution that provides courses, from technical,
technological and graduations. such as ongoing analysis systems lasting 3½
years. If necessary, I can send the documentation cnpj, and the like.
Em 11/10/2016 07:39, "Philip Torchinsky"<EMAIL_ADDRESS>escreveu:

@Solangegmachado https://github.com/Solangegmachado Could you please
send your request by email to<EMAIL_ADDRESS>I can not add your
school to this repository because the website suggests that it is neither a
university, nor a college, but more like a training center. I am closing
the request without merging. Thank you for your understanding.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/2374#issuecomment-252893868, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVjB8ptZwJCME5dmvBXs7KZVvXtFvOUWks5qy3VqgaJpZM4KTC5V
.

@Solangegmachado I did not find the quote which you have cited above at the website. Moreover, if you click "CURSOS" you see just "Veja abaixo uma lista com detalhes sobre nossos cursos ativos e futuros lançamentos." and no futher details.  The problem is that the school is not a university and not a college. This is why I strongly suggest to email to<EMAIL_ADDRESS>and explain all the relevant details to my colleague. The classroom licenses are also free, but they are not based on an email from a school.


Production workflow

Hi, i know i tried this before with issue number 82, which was closed, but i struggle to understand the purpose of pattern lab when it come to creating real applications. maybe you can shed some light on this for me. where i work we got exited about this project and decided to use it as a static site generator which was really helpful. now we are at a point where we want to deploy our apps for production. and the only way i see this possible right now is by manually editing the public folder files, copying and pasting and so on.. I can manage that by myself alright, but this way it makes the whole workflow a bit tedious especially if one needs to go back and make changes inside of pattern lab and then go through the whole manual deployment again without some automation tools. This got me thinking maybe we misunderstood the purpose of pattern lab from the first place.
I'd appreciate to hear more from you regarding this issue. thanks!

@dmolsen
HI, please suggest me how to work with pattern lab  and angular js .  Actually I'm building a websites/product  in angular that consumes dynamic data through web Apis.  In this i want to use template/pattern  generated by Pattern lab but i have no idea how to do this. Also I want to use angular routing instead of pattern lab routing. Please provide step by step guide to integrate angular with pattern lab.
I have download and configure this sample https://github.com/bmuenzenmeyer/patternlab-node-angular-example, but this doesn't have much to  solve my problem.
Thanks


livereload not compatible with<EMAIL_ADDRESS>because of tiny-lr dependency

Looks like metalsmith-watch's live-reload capability isn't working properly with npm v3.x because of dependencies on tiny-lr (see issue below). It also looks like some folks are considering forking tiny-lr because it hasn't been updated in a long time.
See https://github.com/mklabs/tiny-lr/issues/91

hey @juliepagano I'm running into this issue as well. Did you figure out workaround?

@jamiewilson For the moment, I've been using a 2.x version of npm. Looks like 2.14.6 is what I'm using, but I imagine any 2.x version would work.
It looks like the issue was fixed in tiny-lr v0.1.7, so this could be resolved by upgrading to that version. If nobody else gets to it before me, I'll see if I can make a pull request some time in the next week.
https://github.com/mklabs/tiny-lr/issues/91#issuecomment-143514008

I'm wondering if this problem is related to #35.


Error: Lost connection to server

I'm running node-red in a raspberry pi 2 with raspbian.
I've installed node-red and a few nodes, modbustcp and others... (Don't know if it matters)
The web works, but few minutes go by, I get that error, and it disconects. (The connection does not stop working really, I'm SSH and does not disconnect) but Node-red conection stops.
Any ideas?
Thanks btw.

pics relateds.

Can you add the node-red console output so we can see if there are any error messages that might point us in the right direction

If I understanded well, this is the messages that I've got.

I see the errors, but don't know what can I do. :(

I just realiced that when I get the error, I can still work, I can add nodes and deploy, but I got "Error: no response from the server" And a few seconds later I try again and it works. (Or it seems like works)
But constantly Im getting this error, lost conection. :(

From that screenshot, it appears the freeboard node is crashing node-red.
The quickest solution will be to remove that node (delete node-red-contrib-freeboard from ~/.node-red/node_modules).
I would suggest raising an issue against that node so they can have a look at what caused this. The key part to include is the 'Uncaught Exception...' message (and the lines that follow) - https://github.com/urbiworx/node-red-contrib-freeboard/issues

Sounds possible, I'll try it.
I restarted node and got this, perhaps clarifies something.
Thanks a lot for the responsing


Enable OpenSSF Scorecard

Overview of feature request
OSSF Scorecard automatically runs several checks, and enabling it in Radius can help identify security vulnerabilities. Full list of checks it runs can be found at https://github.com/ossf/scorecard/#checks-1 and https://github.com/ossf/scorecard/blob/main/docs/checks.md.
We can leverage its GitHub action to enable the scorecard https://github.com/ossf/scorecard/#scorecard-github-action.
Acceptance criteria
The Scorecard GitHub action is enabled and is passing.
AB#9987

This looks like a great thing to add! Their process allows for a just a couple clicks to open a PR with the GitHub Action already pre-generated and ready. Should just take a minute or two. And as part of the same PR we can add the badge to our README.

@AaronCrawfis Thanks for adding details. Yeah it is fairly quick to setup, I'll try this right now. Main thing to that might require further investigation is if any of the checks fail after this is enabled.


Improvements

hi Katerina,
just listing a few things that I think would be improvements to the system. (but not finding the "improvement" tag)
let me know your thoughts on them, if you can.

I think a different colour when the content word is not found in WN would help to see the coverage of PWN.

I wonder how difficult it would be to implement a module dealing with of nominalizations, like the Bridge system had.

I wonder if you think  changing from Stanford CoreNLP to Stanza is very complicated:we (Livy and I) would like to see how SICK-BR compares to SICK.

I wonder if a different glossary of specific terms could be used as well as PWN or instead of it. Thinking of Wikidata, for instance.

There is the issue of PWN3.0 vs PWN3.1 and the new versions of WordNet. two issues here actually, one is that only PWN3.0 is connected to SUMO and two is which of the new wordnets (if any) will become the facto heir to PWN3.0.

I believe Stanza is the way forward, because of multilingualism. But the issue of what exactly is needed from the enhanced dependencies is an open question, I think.


Add functionality to track online users

Added logic to track users connections to fill who is online on chat.
Addresses #313

@gbdrm,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

Thats part of it, how about if there are already 10 users and someone joins, now the other 10 users see they are online, but the new user doesn't see that anyone else is online.

@gbdrm - can you rebase and squash? I have some ideas about making it work with scaleout I would like to try on top of your change.

@moozzyk Do you want me to create a new branch with rebased and squashed changes? If yes - here it is:
https://github.com/gbdrm/SignalR/tree/users-online-chatsample

I thought you would just rebase and squash this branch and force push...

@moozzyk Sure. Done.

@gbdrm - I finally got to it. Here is what I ended up doing. In the PR I added support for presence for single server and distributed scenarios.

Closing in favor of https://github.com/aspnet/SignalR/pull/391


Add new test in the Isogram exercise

éléphant is not an isogram, but some solutions using a
simple regexp replace of non-word characters will fail
at it.
Addresses #287

@matthewmorgan, can you take a quick look at this? Just say ok or not, and I'll take care of the rest.

Well, the PR was so small and inocent that I merge it myself.


Update helm to v3.14.1

What this PR does / why we need it:
Updated helm version introduces security fix.
Remove PSP logic along the way as types from policy/v1 packages has been removed.
Which issue(s) this PR fixes:
Replaces #3044
What type of PR is this?
/kind regression
Special notes for your reviewer:
Does this PR introduce a user-facing change? Then add your Release Note here:

Update helm to v3.14.1

Documentation:

NONE

/retest


waiting for write JWTs to expire

Hey @PierreAndreis Thanks for that neat little action!
I encountered an issue where the garbage collection locks my regestry after running your action with the status: waiting for write JWTs to expire. I'm using v1.0.1.

Also did it not remove anything at all, even it had shown a lot of prune ones in the output.

Seems that the status for the JWTs comes from the registry login before building. Thats okay but after that it did not clean up still:

Hey @appinteractive, thanks for writing our first issue! 🥳
Garbage collection does not remove the images but cleans up all the artifacts not associated with any manifest anymore and still occupy space.

I encountered an issue where the garbage collection locks my registry after running your action with the status: waiting for write JWTs to expire.

DigitalOcean specifies in their docs that it needs to wait for all the tokens that have to write permission to expire first before running it.
Tokens with write permissions are issued mostly right before you push a new image to a container registry. If you are using their cli, the command would be doctl registry docker-config --read-write, and they last 10 minutes.

Also, did it not remove anything at all, even it had shown a lot of prune ones in the output.

Regardless of GC, if the token given to the action has permission to write, it should have pruned the images tags listed on the table from each of your repositories, and you should be able to confirm that by either using their CLI or their portal.
Can you ensure that the token you have given has permission to write and tags are still there?

Hey @PierreAndreis I resolved it by running it as a crone job with GitHub actions.

name: Clean Docker Registry

on:
  workflow_dispatch:

  schedule:
    # run daily on 2 AM
    - cron: "0 2 * * *"

jobs:
  clean-docker-regestry:
    name: Clean Docker Registry

    runs-on: ubuntu-latest

    steps:
      - name: Install doctl
        uses: digitalocean/action-doctl@v2
        with:
          token: ${{ secrets.DIGITALOCEAN_ACCESS_TOKEN }}

      - name: Login to DO Container Registry
        run: doctl registry login --expiry-seconds 30

      - name: DigitalOcean Registry Prune
        uses<EMAIL_ADDRESS>        with:
          token: ${{ secrets.DIGITALOCEAN_ACCESS_TOKEN }}


Add new compression algorithm and models

Add new id encoding method to MARK-V codec.
Add three shader compression models:
lite - fast, poor compression
mid - balanced
max - best compression

Rebased and pushed into master as 250a235a


Use Hamcrest assertions instead of JUnit in examples/integrations/micronaut/data (#1749)

Part of #1749
Tests are disabled :)
Backport 2.x

/trigger


Custom Stacking Behavior

It would be useful to have some sort of property on the event that you can set that would override the default stacking behaviour and let you stack things in custom ways.
Currently, as a workaround, if you want to override default exception stacking behavior, you can do this:
ExceptionlessClient.Default.CreateEvent().SetType(Event.KnownTypes.Error).SetSource("customSource");

And "customSource" will serve as a new stacking. But then you lose the Exception tab and you have to manually translate the Exception information.
But if instead you do
exception.ToExceptionless().SetSource("customSource");

it still uses a default stacking behavior (exception type + method).
Not sure if it would be better to add a completely new method to set custom grouping (e.g., SetCustomStack("customStackName)") or just add a property (e.g., IsCustomGrouping) and just use the Source property to group the exceptions.

We had a quick talk about this over the weekend and we like this idea, would you be willing to take a stab at implementing this in a pull request?
Here is some of our thoughts on this:

Create EventBuilder and Event extension methods on the client to handle this, but place them under a different extensions namespace (maybe Exceptionless.Advanced??). The reasoning for putting them in a different namespace is we want to make it crystal clear that this is something advanced and not a normal scenario and you understand what you are doing. This extension method should set an extended property on the event and call it something like @StackSignature with a string value that you want to stack on. The extension method that we create should validate that this value is set.
Update the pipeline/AssignToStack pipeline action to use this value for the StackSignatureData. Optional/Feedback Should we remove this extended property from the event once we use it on signature data?

@ejsmith can you provide any additional feedback?

So 1st off, the fact that we aren't showing the error tab when there is an error object on the event is a bug. Event type shouldn't matter for showing that tab. Only that the data is present.
I am definitely on board with allowing custom stacking and agree that I am a little worried that people would see something like SetStackingKey and think they needed to set it. But I think if we do something like SetManualStackingKey then I think it is obvious enough that we can just include it with the other extension methods.  We would also want a good XML doc description saying what the method does.

Add SetManualStackingKey(string stackingKey) that sets an extended property called @StackingKey (need to verify that this name matches our other @ names).
Update the pipeline to add this as the only item in the StackSignatureData if it exists and is not empty.
We would want to show the stacking key on the event to make sure that people know why it got stacked the way it did.

If you look at his first sample, he's only setting the error type and not an actual error model on the event.

@niemyjski Ahh, yeah. I missed that. It doesn't have any error object to add to the event.

Thanks! I'll look into it.
And yes, I didn't submit any error object to the event. If I did, then I wouldn't be able to use "Source" as my custom grouping.

@adamzolotarev were you able to get a chance to look into this? I would like to have this for something I am doing as well.

Not yet. I'm going to look at it today/tomorrow.

Awesome! Let me know if you need any help. Here are some pointers:
This is where the current error stacking happens.
https://github.com/exceptionless/Exceptionless/blob/master/Source/Core/Plugins/EventProcessor/Default/20_ErrorPlugin.cs
I think we should have a step that happens before this that checks for the existence of @StackingKey in the event data dictionary. If that exists then we add it to the context.StackSignatureData and then we need to check to make sure that we don't have any StackSignatureData before we add our normal stacking info.

I updated Exceptionless and Exceptionless.Net.
Tried it locally - looks like it's working. The stacking key is shown at stack level:

I didn't get to the part of displaying the stacking key at the event level. Should I submit a pull request for what I have now? And which branch should I be using then?
Changesets:
https://github.com/adamzolotarev/Exceptionless.Net/commit/a1bcc6c2d22fa58bac228bbdf78c5a6ecbcc889f
https://github.com/adamzolotarev/Exceptionless/commit/d9dc8387734b743943ef16540b554c91f3669665

Awesome! Yeah, create a branch and submit a pull request and we will check it out.

This has both been deployed on clients and on the server. Are we 100% happy with this as I'm in the process of writing a blog post for this.

.NET
https://github.com/exceptionless/Exceptionless.Net/blob/master/Source/Shared/Extensions/EventBuilderExtensions.cs#L63
try {
    throw new ApplicationException("Unable to create order from quote.");
} catch (Exception ex) {
    ex.ToExceptionless().SetManualStackingKey("MyCustomStackingKey").Submit();
}

Or you can set the stacking directly on an event (e.g., inside of a plugin).
https://github.com/exceptionless/Exceptionless.Net/blob/master/Source/Shared/Extensions/EventExtensions.cs#L288
event.SetManualStackingKey("MyCustomStackingKey");

JavaScript
https://github.com/exceptionless/Exceptionless.JavaScript/blob/master/src/EventBuilder.ts#L119
var client = exceptionless.ExceptionlessClient.default;
// Node.Js
// var client = require('exceptionless).ExceptionlessClient.default;

try {
  throw new Error('Unable to create order from quote.');
} catch (error) {
  client.createException(error).setManualStackingKey('MyCustomStackingKey').submit();
}

This feature has been implemented.


For large amounts of coffee script files running on 1.9.3 we see strange issues

Hi there,
We are running 1.9.3 and rails 3.2.21. We have ALOT of coffeescript files and we are seeing very strange errors coming out when the coffeescript is valid (when we compile on 2.1.5 we have no problems). When debugging, if I let the system sleep for 5 - 10 seconds all would be well.  My thoughts are around a V8 memory issue where a pause allows clean up / synching.
Best Regards,
Brendan
The args below when passed to 'unbox @v8_context.eval(properties).call' would cause a compiler error but pass when we wait for 10 seconds:
["App.i18n = new Jed\n  locale_data:\n\n    domain: "messages"\n\n    # messages (translation strings) will be requested from the server for the specified lang using AJAX (for full site i18n)\n    messages:\n      "":\n        domain: "messages"\n        lang: "en"\n        plural_forms : "nplurals=2; plural=(n != 1);"\n\n    # latino_messages is used by the latino page, even when a different language is set for the user\n    latino_messages:\n      "":\n        domain: "latino_messages"\n        lang: "es"\n        plural_forms : "nplurals=2; plural=(n != 1);"\n\n      # Homepage\n      "Movies"              : [ null, "Películas"]\n      "Shows"               : [ null, "Programas"]\n      "View All"            : [ null, "Mostrar más"]\n      "Latino Networks"     : [ null, "Cadenas nacionales"]\n\n      # Navbar\n      "Navigation"          : [ null, "Menú"]\n      "HOME"                : [ null, "Casa"]\n      "SHOWS"               : [ null, "Programas"]\n      "MOVIES"              : [ null, "Películas"]\n      "Sports"              : [ null, "Deportes"]\n      "Guide"               : [ null, "Guía"]\n      "My DVR"              : [ null, "Mi DVR"]\n      "Live TV"             : [ null, "TV Vivo"]\n      "Watchlist"           : [ null, "Mi Lista"]\n\n      "Featured Shows"      : [ null, "Programas Destacados"]\n      "All Shows"           : [ null, "Todos los Programas"]\n      "Networks"            : [ null, "Las Redes"]\n\n      "Featured Movies"     : [ null, "Películas Destacados"]\n      "All Movies"          : [ null, "Todos los Películas"]\n      "My Recordings"       : [ null, "Mis Grabaciones"]\n      "Daily Schedule"      : [ null, "Programación Diaria"]\n      "My Timers"           : [ null, "Mis Temporizadores"]\n\n      # Subnav\n      "Kids"                : [ null, "Niños"]\n      "Showcase"            : [ null, "Mostrar"]\n\n      # Account Navbar\n      "Login"               : [ null, "Entrar"]\n      "Log Out"             : [ null, "Salir"]\n      "Account"             : [ null, "Cuenta"]\n      "Account and Devices" : [ null, "Mi Cuenta"]\n      "Manage Profiles"     : [ null, "Administrar Usuarios"]\n      "Parental Controls"   : [ null, "Controles Parentales"]\n      "Enable Kids Mode"    : [ null, "Activar el Modo de Niños"]\n      "Help"                : [ null, "Ayuda"]\n      "Recents"             : [ null, "Recientes"]\n\n      # Carousels\n      "Refresh network"     : ["Refresh to validate network subscription",\n                              "Actualiza para validar suscripción a la red"\n                              ]\n\n\n# translation helper function\nApp.i18n.t = (string, vars...) ->\n  @translate(string).fetch(vars...)\n\n# Manages the correct translation domain based on the route\nApp.i18n.setDomain = (route) ->\n  if route.match("^latino") then @textdomain "latino_messages" else @textdomain "messages"\n", {:bare=>false}]
For large amounts of coffee script files running on 1.9.3 we see strange issues where a file will error on the call when it is a perfectly good coffeescript file. As a result I am adding a second retry to allow the system to recover. Unfortunately, I could not dig deep enough to find out what the issue is. There does not seem to be a good way to see the internals when the script is sent to V8

This repository has been deprecated and has moved to https://github.com/rails/execjs.


feat: handle fields other than text inputs

[ ] textarea
[ ] radio
[ ] checkbox
[ ] file uploader

Actually, should we be responsible for handling this, or is this up to the developer to pass the necessary data?

We don't need to since we are just serializing props on a POCO.

Alright, closing the issue then


standardisation of optical data

I was wondering why the optical data requires standardisation and therefore the additional computation of the mean and std - An additional step that could be avoided if the optical data is radiometrically corrected. So for sentinel-2 divide by 10000 to get the digital values to reflectance which by default is in the range of 0-1.
Moreover, if the data is standardised, one usually needs to do a running mean/std computation over batches which if I understand correctly is an approximation of the actual mean/stdv. If I am not missing anything it makes more sense to use reflectance?
Second to this, if one had to fine-tune clay on reflectance data instead of standardised data would you foresee any issues?

Yes, I agree that using reflectance would be better, and have thought about this at https://github.com/Clay-foundation/model/issues/94. I can reopen that thread, or continue the discussion here?

@weiji14 I suggest we continue the discussion on the other thread.


Console error - Warning: Extra attributes from the server: class

What version of Remix are you using?
1.3.1
Steps to Reproduce

Generate remix app
Add the following to root.tsx under ScrollRestoration tag


Allow SubBrickKwargsHyperparameter to directly specify a subbrick class

When only one class is possible, instead of having a dedicated hyperparameter with only 1 choice, we directly specify the class in the SubBrickKwargsHyperparameter.
We implement that with GA_RCPSP_Solver and ParametersGa.

Split this PR into 2 PRs.


Support Whitelist Of Action Dispatch Exception

This PR introduces the configuration of exception_class_action_dispatch_whitelist. The problem this PR is trying to solve is that JSON API Resources does a great job of serializing errors into a JSON API response to the client. However, certain alerting frameworks report on the error placed in action_dispatch.exception. For example Rollbar does this here.
The only way to handle the exception is to add it to exception_class_whitelist, but then we must basically re-implement what JSON API Resources already does so well in our rescue_from in the Controller.
By introducing this notion of whitelist, we allow JSON API to continue to respond to the error, but we can hide it from Rails and other downstream middleware, i.e. Rollbar. This lets us keep the baby while we toss out the bath water.
Related Issue
https://github.com/cerebris/jsonapi-resources/issues/1151
All Submissions:

[x] I've checked to ensure there aren't other open Pull Requests for the same update/change.
[x] I've submitted a ticket for my issue if one did not already exist.
[x] My submission passes all tests. (Please run the full test suite locally to cut down on noise from travis failures.)
[ ] I've used Github auto-closing keywords in the commit message or the description.
[x] I've added/updated tests for this change.

New Feature Submissions:

[x] I've submitted an issue that describes this feature, and received the go ahead from the maintainers.
[x] My submission includes new tests.
[x] My submission maintains compliance with JSON:API.

Bug fixes and Changes to Core Features:

[ ] I've included an explanation of what the changes do and why I'd like you to include them.
[ ] I've provided test(s) that fails without the change.

Test Plan:
Reviewer Checklist:

[ ] Maintains compliance with JSON:API
[ ] Adequate test coverage exists to prevent regressions

I am going to close this. Given the chat on the issue, it may not make sense.


citations whose id starts with a digit are not processed

Example:
---
title: Test
references:
- id: 1657:huygens
  publisher-place: Lugd. Batav.
  page: 517-524
  container-title: 'Exercitationum mathematicarum libri quinque: Quibus accedit
    Christiani Hugenii tractatus de ratiociniis in aleæ ludo'
  author:
  - family: Huygens
    given: Christiaan
  issued:
    year: 1657
  title: De ratiociniis in ludo aleæ
  type: chapter
  publisher: Ex officina Johannis Elsevirii
  editor:
  - family: Schooten
    given: Franciscus á
...

[@1657:huygens]

renders - without error messages - as
[@1657:huygens]

instead of
(Huygens 1657)

Huygens, Christiaan. 1657. “De Ratiociniis in Ludo Aleæ.” In Exercitationum Mathematicarum Libri Quinque: Quibus Accedit Christiani Hugenii Tractatus de Ratiociniis in Aleæ Ludo, edited by Franciscus á Schooten, 517–24. Lugd. Batav.: Ex officina Johannis Elsevirii.

Changing the id to, for example, x1657:huygens, makes it work again, so the issue appears to be that the id starts with a digit.
In bibtex, it is perfectly valid for citation ids to start with the year, so it would be nice if pandoc-citeproc could support this as well.

That's intentional (see pandoc User's Guide under citations).
I worried that if we allowed citations to start with digits, we'd
falsely capture some things that weren't intened as citations,
e.g. 15 apples @5ea.  Maybe this is overthinking, though.
+++ Matthias C. M. Troffaes [Jan 09 15 01:50 ]:

Example:
---
title: Test
references:
- id: 1657:huygens
 publisher-place: Lugd. Batav.
 page: 517-524
 container-title: 'Exercitationum mathematicarum libri quinque: Quibus accedit
   Christiani Hugenii tractatus de ratiociniis in aleæ ludo'
 author:
 - family: Huygens
   given: Christiaan
 issued:
   year: 1657
 title: De ratiociniis in ludo aleæ
 type: chapter
 publisher: Ex officina Johannis Elsevirii
 editor:
 - family: Schooten
   given: Franciscus á
...

[@1657:huygens]

renders - without error messages - as
[@1657:huygens]

instead of
(Huygens 1657)

Huygens, Christiaan. 1657. “De Ratiociniis in Ludo Aleæ.” In Exercitationum Mathematicarum Libri Quinque: Quibus Accedit Christiani Hugenii Tractatus de Ratiociniis in Aleæ Ludo, edited by Franciscus á Schooten, 517–24. Lugd. Batav.: Ex officina Johannis Elsevirii.

Changing the id to, for example, x1657:huygens, makes it work again, so the issue appears to be that the id starts with a digit.
In bibtex, it is perfectly valid for citation ids to start with the year, so it would be nice if pandoc-citeproc could support this as well.

Reply to this email directly or view it on GitHub:
https://github.com/jgm/pandoc-citeproc/issues/97

Thanks for your quick response. Indeed this is mentioned in the User's Guide. I thought I had read it, but I missed this detail.
Nevertheless, personally, I find the distinction between digits and letters surprising. The standard way in markdown to prevent special characters to act their magic is to backslash them. This already works e.g. \@x1657:huyghens is indeed rendered correctly as @x1657:huyghens and not as a citation. The same could work for \@5ae.
A change would of course break backwards compatibility. It's hard to tell how many people would rely on @5ae not being a citation in documents that use citeproc; probably not many?

Closed by jgm/pandoc#1954.


Feature: Add Progress bars

https://www.google.com/design/spec/components/progress-activity.html
http://g.recordit.co/NEQuvntLcF.gif

Progress bars as a standard component would be really nice, I would definitely make use of this if the feature was added.

+1

+1

Finally for our case we will use progress of html5. We just have to design  it for each platform

Hello! We are moving our feature requests to a new feature request doc. I have moved this feature request to the doc and because of this I will be closing this issue for now. Thanks for using Ionic!

It would be nice to have an option to integrate it with the navbar, seen in Safari iOS, for example.

+1

https://github.com/valor-software/ng2-bootstrap
this can be easily included into your ionic2 project and it gives you some decent consistent progress bar styling across platforms.  Still a native ionic component would be nice but this definitely works, I have rolled it into my own project so that I could have some nice progress bars for downloads.

+1

+1

Hello all! While this is an awesome feature request it is not something that we plan on doing anytime soon. Because of this I am going to move this to our internal feature tracking repo for now as it is just "collecting dust" here. Once we decide to implement this I will move it back. Thanks everyone for using Ionic!

This issue was moved to driftyco/ionic-feature-requests#38

This is something I've been looking around at for a while and looking for a good implementation, out of the box would be amazing.

meanwhile we waiting for the ionic progress bar component, I leave this simple pen with a material design looks, inspired in this css-tricks tutorial and the materialize css indeterminate progress bar
<progress max="100" value="50"></progress>

.progress, progress[value] {
  -webkit-appearance: none;
  appearance: none;
  border: none;
  display:block;
  height: 10px;
  &::-webkit-progress-bar {
    background-color: lighten( map-get($colors, primary), 35% );
  }
  &::-webkit-progress-value {
    background-color: map-get($colors, primary);
  }
}

in the next days I'll try to make it a sass mixin to improve the use

@brandyscarney I am willing to create a progress bar for android, just want a heads up whether anyone on the team is already working on one before starting!

@realappie It is not currently being worked on. I started progress on it awhile ago but it is so outdated now that it would probably be easier to start from scratch. 🙂

@brandyscarney beside the progress bar having four modes, are there any other requirements? Also the progress bar in the gif is a material design one, are we going to be re-using that for iOS and windows as well or are they supposed to get a different looking loading indicator.

@jgw96 Would you mind re-opening this as your link is invalid ? :(

@realappie I would say the progress bars should follow the Material Design requirements: https://material.io/guidelines/components/progress-activity.html
The first iteration doesn't have to do everything, but eventually we want to get there. I think for now we can make them all look the same and then tweak them for iOS / Windows later. The Material Design team has some good documents to refer to: https://docs.google.com/document/d/1lBJkORFeB7y_RMRtXfT4bIdz9PnUYLn0HYtSClayXt4/preview
I was thinking we could have one component ion-progress and then have the following inputs:
type: 'bar' | 'circle' = 'bar';
indicator: 'determinate' | 'indeterminate' | 'buffer' | 'query' = 'determinate';

Since we already use mode for our theming. I'm open to ideas though. 🙂

@brandyscarney No that sounds like a great idea! Also don't we already have an ion-spinner which would be the same as the progress of type circular?
I'm currently busy with the determinate version of the progress bar, the rest will be easy to implement but the animation of this one is tricky !

@brandyscarney I finished the progress bars, but I'm getting some linting errors because I'm using the property left is prohibited. Should I refactor my code or would my code be accepted?

@brandyscarney How do the linting rules actually work? Because the output of the linter is somewhat verbose and I want to understand it so I can contribute to ionic in the future as well.
If you would rather fix it, I will submit a PR :D

@realappie Each linter rule and a description of what they do can be found here: https://github.com/brigade/scss-lint/blob/master/lib/scss_lint/linter/README.md
For right to left support we have just disabled certain properties from being used via PropertySpelling in favor of the mixins. We could probably enhance the linter to provide more information, but this is why it's happening.

@brandyscarney Ah I see, thank you! I will give RTL support a shot but if it takes me too long I will just submit the PR considering I have never done anything with RTL.

Sounds good, feel free to submit the PR and either @AmitMY or myself can help out. :smile:

@brandyscarney Maybe change the v2 label to v4? 👍

Any updates to this? :)

@brandyscarney @realappie  Is there working indeterminate progress bar? Can you please share details


docs(subject): Subject documentation for new docs app.

Description:

Updated subject.md from old docs for consistency with the new API
Copied and linked this to the new docs app

I'm not sure if we want to remove things that were copied over from doc. I'm also not sure whether the migration / update was done properly. I'm happy to address any feedback.
Related issue (if exists): #3953 (partial)

Coverage remained the same at 96.798% when pulling 2e99f040a3df019da0a29aed50cd1679afcf7439 on ajcrites:ajcrites/docs/3953 into e3911bc081999c3a8b39011831bdd6e527e62f40 on ReactiveX:master.

This looks good. One note is that the newer examples are showing the output in the same code example markdown rather than a newer one below. Example: https://github.com/ReactiveX/rxjs/blob/master/src/internal/observable/forkJoin.ts#L85

Comments addressed. I didn't really know where to put the style. I also changed it to just italicize the text instead of use the cursive font since it seems like Kalam isn't included. The cursive is a little too informal.

Updated for requested changes.


"notification-content-deployment" Application Start Failed due to insufficient space quota

Hi,
I am trying to deploy incident management application, the notification-content-deployment is failing due to insufficient space quota.
Here's the log
2024-01-29T11:55:12.915+0000 [CELL/0] STDERR Copying into the container failed: stream-in: nstar: error streaming in: exit status 2. Output: tar: ./app/node_modules/@sap/cds-dk/node_modules/<EMAIL_ADDRESS>Cannot write: No space left on device
2024-01-29T11:55:12.915+0000 [CELL/0] STDERR tar: ./app/node_modules/@sap/cds-dk/node_modules/@sap/hdi-deploy/node_modules/@sap/hana-client/prebuilt/darwinarm64-xcode12: Cannot mkdir: No space left on device
2024-01-29T11:55:12.915+0000 [CELL/0] STDERR tar: ./app/node_modules/@sap/cds-dk/node_modules/@sap/hdi-deploy/node_modules/@sap/hana-client/prebuilt/darwinarm64-xcode12: Cannot mkdir: No space left on device
2024-01-29T11:55:12.915+0000 [CELL/0] STDERR tar: ./app/node_modules/@sap/cds-dk/node_modules/<EMAIL_ADDRESS>Cannot open: No such file

Increasing the space disk quota to Instance Disk500 MB starts the application.
Currently, the mta generated by cds add mta has default disk-quota: 256MB. It needs to be increased.
Here's the current mta
  - name: notification-content-deployment
    type: nodejs
    path: gen/srv
    parameters:
      no-route: true
      no-start: true
      memory: 256MB
      disk-quota: 256MB
      tasks:
        - name: notification-content-deployment
          command: "node<EMAIL_ADDRESS>          memory: 256MB
          **disk-quota: 256MB**
    requires:
      - name: incident-management-destination
      - name: incident-management-auth
      - name: incident-management-db
      - name: incident-management-alerts

Will be fixed with next cds-dk release - than it should generate 1GB of disk-quota


Add badges to repo README

Description
Adds badges to the REAME showing status of unit test workflow, codecov, etc.
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Is it possible to squash those merge commits or maybe do hard reset first and re-create PR with only your commit on top?

Sure @martin-gaievski. I didn't realize it had all the merge commits from the past 😅 Also, all these commits get squashed when we merge, but let me create a cleaner commit for this one.


Enable using 'ts-jest' package name as transform (Take 2) + fix #367

@kulshekhar @screendriver @GeeWee I dug into https://github.com/kulshekhar/ts-jest/issues/367
The bug is in babel-core: https://github.com/babel/babel/pull/6524
What changed was that require('ts-jest') would end up requiring babel-core (it's used by the process() function), whereas before my patch it did not do that. To avoid hitting the babel-core bug, I changed the code to lazily require() the preprocessor, so that babel-core does not get dragged in when require('ts-jest').install() runs.
I changed the scripts/tests.js to copy the package sources, replacing the require('../../../../etc') hack. For some reason, I could not get #367 to happen in the integration test that I added with that hack in place.
I think it's better now because 1) the thing under test is not modified, but instead the code as it will be distributed gets tested 2) the Jest "transform" configuration in the package.json does no longer refer to a parent path, but the config that one would normally use in a real project.
In this PR I re-reverted #368 and added the bugfix on top of it.

I changed the title because "Change entry to preprocessor.js" is not longer true.

Cool thanks 👍

Something seems to have broken the build

Something seems to have broken the build

@kulshekhar Fixed it!

@kulshekhar could you take another look? Thanks!

I'm not sure I'm gonna have the time needed to look again closely at this in the next few days.
So I'm gonna go ahead and merge and publish this now and can revisit this in case this breaks things for someone!

@martijnthe this has been published. Thanks for the PR!

@kulshekhar thanks!


[bug] Plugin webview does not work on first run

Describe the bug
I created a tauri plugin that gets correctly built. However, when first starting tauri (yarn tauri dev), an error is returned on first run saying:
Uncaught SyntaxError: The requested module '/node_modules/.vite/deps/tauri-plugin-zaid-plugin.js?v=2a118ae6' does not provide an export named 'test' (at App.tsx:5:1)

But if one comments out lines 11-19 in App.tsx, reload the page, uncomment them and reload the page, everything works as expected.
Reproduction

clone this repo to reproduce the bug
yarn install
yarn tauri dev

Expected behavior
Plugin to work on first run
Platform and versions
$ tauri info

Environment
  › OS: Windows 10.0.19044 X64
  › Webview2: 101.0.1210.53
  › MSVC:
      - Visual Studio Build Tools 2019
  › Node.js: 16.14.1
  › npm: 8.5.0
  › pnpm: Not installed!
  › yarn: 1.22.17
  › rustup: 1.24.3
  › rustc: 1.61.0
  › cargo: 1.61.0
  › Rust toolchain: stable-x86_64-pc-windows-gnu

Packages
  › @tauri-apps/cli [NPM]: 1.0.0-rc.13
  › @tauri-apps/api [NPM]: 1.0.0-rc.6
  › tauri [RUST]: 1.0.0-rc.14,
  › tauri-build [RUST]: 1.0.0-rc.12,
  › tao [RUST]: 0.9.1,
  › wry [RUST]: 0.17.0,

App
  › build-type: bundle
  › CSP: unset
  › distDir: ../dist
  › devPath: http://localhost:3000/
  › framework: React

App directory structure
  ├─ node_modules
  ├─ src
  └─ src-tauri

Stack trace
No response
Additional context
No response

Tried running on MacOS, works flawlessly, but not on Windows.

I couldn't reproduce this. Also it's not a Tauri issue but a possible configuration issue on your React app or the plugin build.


Inline diff has no horizontal scroll bar if only "before" lines exceed width of editor

monaco-editor version: 0.X.Y
Browser: Chrome, edge
OS: Windows
Steps or JS usage snippet reproducing the issue:
In an inline diff, the horizontal scrollbar does not appear until added or existing text hits the edge of the window. Removed seems to be ignored. Also, as the window is resized I can see the horizontal scrollbar appearing but it immediately disappears. Are removed lines in the diff factored into whether or not this scrollbar should be displayed?

This has been fixed in the meantime.


Socket onDisconnected() not called fast enough if server suddenly loses internet connection

Hey,
I am having an issue with the websocket onDisconnected() not being called fast enough if someone literally pull out the ethernet cable from the server that the app is connected to. As of now, my setConnectionTimeout() is 30 seconds and setPingInterval() is 30 seconds. The time it takes for the onDisconnected() method to be fired is roughly 2 minutes.
I tried setting setConnectionTimeout() to 1 and setPingInterval() to 1 to test but the delay is still a good 2 minutes before onDisconnected() gets called.
Can you point me to the right direction on how to address this issue? I need to get the time down to something around 30 seconds or less if possible.

Hi,
I have the similar issue. Problem is when I have wifi connection on, but don't have internet access. Error callbacks don't execute at this case at all. I debug code and find problem in method WriteThread.flush(). It doesn't throw exception when wifi is on, but doesn't have internet access. My question is: maybe it is good idea to add approach of throwing exception if ping frame is send but pong frame isn't received. Or maybe there is more elegant solution of my problem?

this is a problem of the websocket protocol as far as I was aware.
as you need to send a closing message, so if you kill the connection you will instead wait for the tcp/ip timeout.

I'm sorry for not responding long time.
setConnectionTimeout() has a meaning only when a WebSocket connection is established. Therefore, lowering the value of connection timeout won't work for your use case.
According to the specification, upon receipt of a Ping frame, an endpoint MUST send a Pong frame in response (RFC 6455, 5.5.2. Ping), and the Pong frame must have identical "Application data" as found in the message body of the Ping frame being replied to (RFC 6455, 5.5.3. Pong). Therefore, it is possible to detect disconnection by (1) sending a Ping frame, (2) setting a timer that waits for a Pong frame that has the same application data, and (3) firing an exception or calling a callback if the timer has expired before receiving the expected Pong frame. This behavior can be implemented outside nv-websocket-client. If this behavior were implemented as a feature of nv-websocket-client, it would be helpful. However, I'm sorry I won't be able to spare time in the near future to implement the feature.
Some platforms such as Android provide a mechanism to notify applications of network disconnection. See CONNECTIVITY_ACTION for details.


add sudo calls & tests

Description
Done:

Added sudo call allowing subnet owners to enable or disable commit/reveal
Added sudo call allowing subnet owners to modify the commit/reveal interval
Added checks to enforce enabling/disabling commit/reveal in set_weights, commit_weights, and reveal_weights.
Added tests for all these.
Ran benchmark tests and set the correct  weight values for sudo_set_commit_reveal_weights_interval and sudo_set_commit_reveal_weights_enabled

Related Issue(s)

Closes #[issue number]

Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Other (please describe):

Breaking Change
If this PR introduces a breaking change, please provide a detailed description of the impact and the migration path for existing applications.
Checklist

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have run cargo fmt and cargo clippy to ensure my code is formatted and linted correctly
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Screenshots (if applicable)
Please include any relevant screenshots or GIFs that demonstrate the changes made.
Additional Notes
Please provide any additional information or context that may be helpful for reviewers.

@JohnReedV merge conflict


Enable GTS DA cycles every 6 hours

Describe your changes
Summarise all code changes included in PR:

Fix bugs for $WINLEN
Correct typos in the do_landDA.sh
Add Jiarui's current GTS yaml and add the elevation check (to be consistent with GHCN.yaml filters)
Enable 6-hr cycle for GTS DA

List any associated PRs in the submodules.
N/A
Issue ticket number and link
List the git Issue that this PR addresses:
https://github.com/NOAA-PSL/land-DA_update/issues/26
Test output
Is this PR expected to pass the DA_IMS_test (ie., does it change the output)?
Yes.
Does it pass the DA_IMS_test?
Yes.
If changes to the test results are expected, what are these changes? Provide a link to the output directory when running the test:
N/A
Checklist before requesting a review

[x] My branch being merged is up to date with the latest develop.
[x] I have performed a self-review of my code by examining the differences that will be merged.
[x] I have not made any unnecessary code changes / changed any default behavior.
[x] My code passes the DA_IMS_test, or differences can be explained.

Please review: @ClaraDraper-NOAA @jiaruidong2017, Thank you!


Fix a test error for DB13.3 [databricks]

Fix https://github.com/NVIDIA/spark-rapids/issues/10797
This PR uses a new config relevant to arrow batch slicing for the arrow python runner pick, and applies the pick rule of the arrow python runner to GpuAggreagteInPandasExec in addition to GpuFlatMapGroupInPandasExec.

build

build


WIP: http: strip etags from error responses

This PR currently demonstrates that strong ETags are set on error responses, and that if these are stripped, express still sets weak ETags on those responses.
Setting strong ETags which correspond with OK responses on error responses with non-matching content is not useful, and may cause caching issues.
Adding a test for this without making invasive changes to production code seems tricky.
Running the test locally:

STRIP_ERR_ETAGS=yes make test-integration
STRIP_ERR_ETAGS=no  make test-integration

Closing as this PR just demonstrates the issue.  Fix at https://github.com/getodk/central-backend/pull/1103


Add the text

Text added: "Some people invest in cryptocurrency on exchanges such as Binance, Bitrex, or Coinbase. The market peeked in January of 2018."

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you @DiggyShants for your contribution and congratulations on your first PR merge to this repo! 🎉


Can metaPayloadLength reach to OffsetMapCodecManager.DefaultMaxMetadataSize (4KB)?

Hi team
<IP_ADDRESS>
PartitionState.java 275
private boolean updateBlockFromEncodingResult(String offsetMapPayload) {         int metaPayloadLength = offsetMapPayload.length();         boolean mustStrip = false;         if (metaPayloadLength > OffsetMapCodecManager.DefaultMaxMetadataSize) {             mustStrip = true;             this.setAllowedMoreRecords(false);             log.warn("Offset map data too large (size: {}) to fit in metadata payload hard limit of {} - cannot include in commit. Warning: messages might be replayed on rebalance. See kafka.coordinator.group.OffsetConfig#DefaultMaxMetadataSize = {} and issue #47.", new Object[]{metaPayloadLength, OffsetMapCodecManager.DefaultMaxMetadataSize, OffsetMapCodecManager.DefaultMaxMetadataSize});         } else if ((double)metaPayloadLength > this.getPressureThresholdValue()) {             this.setAllowedMoreRecords(false);             log.warn("Payload size {} higher than threshold {}, but still lower than max {}. Will write payload, but will not allow further messages, in order to allow the offset data to shrink (via succeeding messages).", new Object[]{metaPayloadLength, this.getPressureThresholdValue(), OffsetMapCodecManager.DefaultMaxMetadataSize});         }
for messages keep failed scenario,I found when metaPayloadLength reached to 75% of limit, no more records would be took for processing, framework would stop encoding(cause there's no succeed messages), so I think metaPayloadLength wouldn't getting larger.
So I think  metaPayloadLength would never greater than OffsetMapCodecManager.DefaultMaxMetadataSize
It's  " if (metaPayloadLength > OffsetMapCodecManager.DefaultMaxMetadataSize) " necessary here ?
BRs,
Dumont

Hi @dumontxiong - it is a defensive check - as payload is encoded on commit - encoded size can jump from < pressureThresholdValue to > DefaultMaxMetadataSize between two commits - if we try to send payload over the maximum - the commit will fail - so we do have to trim the encoded data in that case.
The second check blocks new messages from being processed until pending messages drain sufficiently to reduce required encoded payload size to reduce that likelihood.

@rkolesnev thanks!


[Web] Named Routes as page URL

Is possible to get named routes to be the page URL? If possible, how do I achieve that?

This is a question better suited for StackOverflow, since there doesn’t seem to be anything wrong with Flutter itself. @Akora-IngDKB see also https://flutter.dev/community for support options.

@BondarenkoStas sure👍


batch api test failing

Am trying to use batch operations in Seraph and they are failing with neo4j 2.1.5. Neo4j returns a java.lang.NullPointerException for any of the tests in /test/batching.js
The error seems to occur on the commit() method.
The only work around at the moment is to use raw queries.
Thoughts?

This does not seem to be happening for me anymore. Can you confirm that it is happening for you still? Sorry for the delay.

Closing because of inactivity, and I can't reproduce it myself. Batching tests seem to be passing for me.


test(extension): test maintenance 30 December - remove 'next to' check

coverage: 53.248%. remained the same
when pulling f3a9510b738ad913b78869a417b0553f36ce2919 on test/test_maintenance_13_Dec
into 483c812f5d5cca8395ddf468b7123be11ec0e9bd on main.


Madara Kit: A Web Application for Interacting with Madara Chain

Feature Description
We are looking for contributors to create the "Madara Kit", a web application designed to interact with a Madara chain.
Requirements

Chain Information: The web application should display basic information about the Madara chain, such as the details of the latest block.

Transaction Capabilities: The application should enable users to perform various actions, including:

Perform ERC20 transfers.
Mint NFTs (Non-Fungible Tokens).

Deploy Cairo Contract: Optionally, the application could include the ability to deploy a Cairo contract directly through the interface.

Preferred User Flow

User visits the Madara Kit web application.
User views the basic information about the Madara chain, including the latest block.
User chooses to perform an action (ERC20 transfer, NFT minting, or Cairo contract deployment).
The web application guides the user through the necessary steps and performs the chosen action on the Madara chain.

Who Can Help?
Contributors with experience in web development and interacting with blockchain networks, particularly the Madara chain, are highly encouraged to participate. Knowledge in ERC20 transfers, NFT minting, and Cairo contract deployment would be beneficial for this project.
If you want to build it in Rust for the fun feel free, there is an example here: https://github.com/keep-starknet-strange/madara-kit
However, it's not mandatory at all. You can of course go with a more traditional stack like React + TailwindCSS, or something else.
Thank you!

It should be like an explorer but with an interface like OZ Wizard?

Yes

Hey there!
As discussed on TG, here is a first draft of what it could looks like: https://www.figma.com/file/6AyuP4IqYQaCFZ949bu0pv/Madara-Webkit?type=design&node-id=0%3A1&mode=design&t=pHmXvkiihQfXZlUx-1
Lmk what you think & if you need owner/editor access to the Figma :) I will iterate on it based on feedback.

Where do I set up the repository for the project and is there an available doc about  APIs to get details on blocks and transactions from the sequencer?

There is already a repo: https://github.com/keep-starknet-strange/madara-kit
You can ignore what is inside it was just an example to use Rust to build the frontend but it was just an experiment. Let’s build it with a more classical stack such as React with Chakra UI as suggested by @0xlny

Alright, will get started with the project.

@abdelhamidbakhta The UI templates should be ready by tomorrow, I have started working on it. I bootstrapped the project with nextjs and tailwind CSS on a separate branch. I will raise a PR soon.

@abdelhamidbakhta The UI templates should be ready by tomorrow, I have started working on it. I bootstrapped the project with nextjs and tailwind CSS on a separate branch. I will raise a PR soon.

Awesome 😎


JQuery $.when is typed incorrectly

It's using generics to define the done values that come back. The problem is that it's assuming all promises passed into it will be of the same type which is not valid. The types can all be different.

@johnnyreilly

Hi @JohnGalt1717,
In which case presumably you can type assert with any or similar to cater for those edge cases?  If you've a better proposal for how to type this then feel free to submit a PR and we can go from there.
Closing for now.

Either put it back the way it was where the result types were  so that it works properly and doesn't require a hack, or make every return type a different Type based on the parameters passed into the function. (don't know if this is possible)
But the way it is right now would not work for 99% of the uses of $.when so you're requiring it to be hacked essentially all of the time.

Hi @JohnGalt1717,
You may well be right - I don't use the promises in jQuery myself as they are "broken" / not Promises/A+ compliant.
You suggest that a problem that has been introduced into the typing recently.  That being the case please feel free to submit a PR with your alternative implementation (the previous implementation?) and we'll look at it and discuss.
Thanks.

What's a PR? I've submitted this bug already. Just put it back the way it was because your change isn't a valid assumption.

PR = Pull Request.  I don't know exactly what the change you are referring to is - but you can help me there.  If you submit a PR with the correct typing as you see it.
Given what you've said about it being correct in the past you should be able to go through the jquery.d.ts's history and find what you are looking for which will make creating the PR pretty straightforward.


problems in e2e-test

gobgp container cannot start because of a running instance
CRD is terminating

/kind bug
/priority backlog


File unable to be downloaded

Skyrim.Priority.SE was updated today to 3.0 and V2.2.1 is no longer available on github so wabbajack fails to install the pack.

update 0.57 fixed this


Add special dautotest target

The idea is to make it:

easy to modify in case of future updates
easy to spot and understand by contributors to see what's going on with DAutoTest

This is trivial so I'm gonna merge without waiting for further input.

Oh we kinda need @CyberShadow here as he's the maintainer of DAutoTest...

Oh, I thought this was just adding the target.

BTW the respective code for DAutoTest is here: https://github.com/CyberShadow/ae/blob/bdc41d1669264104ae38c5919c9a1560d685c4f9/sys/d/manager.d#L1718
So it should have been:
dautotest: all verbatim pdf diffable-intermediaries

Sorry.

Anyhow no harm done:
https://github.com/dlang/dlang.org/pull/2035
I actually prefer action with 1 in 20 cases being a bit too eager instead of letting PRs stall for months.

Yeah, sorry, I'm on mobile so I'm trying to quickly review PRs that look trivial and not let things sit around.


LC0005 doesn't work in XmlPost

LC0005 doesn't check the fieldelement in an XmlPort.
tableelement(Item; Item)

I would expect an warning for the two item lines (200, 212)

This repo is only for the VS Code extension to download the LinterCop.
I can't move an issue between the repositories unfortunately. Please close this issue here and create a new one here: https://github.com/StefanMaron/BusinessCentral.LinterCop

Will do


chore: archive notice in readme

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


fix rake tests to run under windows

Please, consider merging fix for running test rake tests under windows .
This change should be safe for linux.
This fixes:
\r\n line ending in windows

:=>; File.pathSeparator under windows

 Issue with File.unlink when files locked by antivirus

Hi,
Thanks for the patch.
Could you file an issue against JRuby too for the file lock thing? Do you get that with CRuby as well?
Instead of using File.pathSeparator, could you use the Mirah::Env.encode_paths method? You'd need to break up the class paths into arrays, but I think it would make things clearer.

I'd like to merge the windows build fixes, and to look at the other feature additions, but could you break them out into separate pull requests?

Sorry, missed the fact that I'm developing on a single master branch. Will try to split it somehow.

closed to split into separate pull requests


Auto-suggest search blocks UI on mobile with poor connectivity

Seen on mobile Safari, iOS 8 / iPhone 4S.
On a mobile with poor Internet connectivity, the UI becomes blocked and unusable when typing in a search term. It seems to be triggering many requests in quick succession. It should be throttled and wait for some delay after user input has ended or paused.

Given there was at least one overhaul of the site and the time passed by now, is this issue still relevant or can it be closed? 0:-)
If it turns out to be still relevant, would you have a screenshot or a recording?


#14 Drive: create endpoint

Closes #14.
Only happy path, minimal validation. 500 thrown when car does not exist and passengers will not be saved if wrong ids are supplied.

Please resolve conflict.


test: add tiny wait not to lose globbed entries

Which issue(s) this PR fixes:
Related #3263
What this PR does / why we need it:
In busy loop with wait_flush, there may be a case that Dir.glob() lose
existing entries under target directory in some test cases.
As a result Timeout::Error: execution expired will be raised.
These failures in test/plugin/test_output_as_buffered_backup
are observed with Ruby 3.0 on Windows.
Docs Changes:
N/A
Release Note:
N/A

Timeout::Error: execution expired
<internal:dir>:135:in `glob'
./test/plugin/test_output_as_buffered_backup.rb:175:in `block in wait_flush'
./test/plugin/test_output_as_buffered_backup.rb:112:in `block in waiting'

It seems that the above error doesn't occur.


Add proper support for syntax errors

We could also consider adding the GraphQLError functionality through a trait.

Pull Request Test Coverage Report for Build 493

9 of 41 (21.95%) changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 79.929%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Language/Parser.php
8
12
66.67%

src/Language/Lexer.php
0
5
0.0%

src/Language/Reader/BlockStringReader.php
0
5
0.0%

src/Language/Reader/NumberReader.php
1
8
12.5%

src/Language/Reader/StringReader.php
0
11
0.0%

Totals

Change from base Build 486:
-0.4%

Covered Lines:
2262

Relevant Lines:
2830

💛  - Coveralls


Select element by attribute

Guys,
I'm getting Invalid Selector ussing:
by.css('.xxxx .select-option[data-value=3]')
and
by.css('.xxxx .select-option["data-value"=3]')
Why is that? Is it Ok?
Is there a way to select by class+attr?

You can see that these are not valid selectors by using them in chrome or firefox dev tools console, such as document.querySelector('.xxxx .select-option["data-value"=3]').
The problem is that the number is not a valid comparison. Fix: .xxx select-option["data-value"="3"]

See also
http://stackoverflow.com/questions/6247221/css-selectors-cannot-match-numerical-attribute-values-why
Closing as an answered question.


Hotfix skipping the first chunks of the artifacts

Quick hotfix.
fixes: #4986, #4987
The stream starts being consumed once we put a handler for data event.
When extracting stream.pipeline() under withTempFile in #4963 I didn't move it into the scope too, which due to preliminary awaiting for async operations with the file system allowed for the first chunks of the file to be skipped

@matklad sorry for that, for some reason this doesn't error out on my laptop, we should publish a hotfix for this

That code is so weird. Does attaching the data handler start the download? Why does it fix the problem?

@lnicola
The first chunks of the file are received asynchronously after the current call stack is empty.
Before the hotfix we had:
stream.on("data", () => { reportProgress() });
await withTempFile(x => stream.pipeline(x));
// Inside of `withTempFile() we await for async operations with the file system,
// whils we have already put the first handler for `data` event, this means
// that the stream strats being consumed and while we are waiting for async file system operations
// the handler with `reportProgress()` starts consuming the first chunks of the stream
// after the async operations for creating the temp file are done `stream.pipeline()` internally puts
// its own `data` handler but this happens only after the part of the stream is consumed.

After the hotfix:
await withTempFile(x => {
    // we have put our own handler, the stream will be consumed strating from the next spin of the event loop
    stream.on("data", { reportProgress() });
    // Now stream.pipeline() has put its own handler (now there are 2 handlers total)
    // But no bytes have been consumed, since no handlers are called only on the next spin of the event loop
  stream.pipeline(x);
})

The first chunks of the file are received asynchronously after the current call stack is empty.

Ohh..

In pseudocode:
- registrer on("data", chunk => reportProgress(chunk.length))
await withTempFile() -> await mkdirTemp() // empties the current async callstack
- reportPorgess() handler consumes the first chunk
- mkDirTemp is finshed, await is resumed
- await stream.pipeline() // registers own on("data") handler, empties the current async callstack
- reportProgress() and internall stream.pipeline's handler consume the **second** chunk

I got it. I was missing the "stream starts on the next event loop after the first handler is attached".

bors r+
Thanks for looking into this @Veetaha !

Yup, this hotfix is sufficient as a fix, but the core problem was in mixing temp dir logic with the download logic, I'll submit a PR to decouple them to further prevent such mistakes

Release is published!


Namespaces for conda-store environments

Enviroment created from filesystem what namespace? system or library are often used with others.

Enviroment created from filesystem what namespace? system or library are often used with others.

Admin only environments that cannot be edited by other users.

Admin only environments that cannot be edited by other users.


Fix rack-mini-profiler conflit

Hello,
Thx for you work !
I made this patch so that peek-ng works with other pg monitoring tools.

This solve a stack level too deep error on boot on Rails 4.2.11 with bootsnap:

Called from ~/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/bootsnap-1.4.4/lib/bootsnap/load_path_cache/loaded_features_index.rb:92:in register' Exiting Traceback (most recent call last):         8484: from bin/rails:4:in '
8483: from ~/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:274:in require'         8482: from ~/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:240:in load_dependency'
8481: from ~/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:274:in block in require'         8480: from ~/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/bootsnap-1.4.4/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:30:in require'
8479: from ~/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/bootsnap-1.4.4/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:21:in require_with_bootsnap_lfi'         8478: from ~/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/bootsnap-1.4.4/lib/bootsnap/load_path_cache/loaded_features_index.rb:92:in register'
8477: from ~/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/bootsnap-1.4.4/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:22:in block in require_with_bootsnap_lfi'          ... 8472 levels...            4: from ~/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/rack-mini-profiler-1.0.2/lib/patches/db/pg.rb:110:in async_exec'
3: from ~/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/peek-pg-1.3.0/lib/peek/views/pg.rb:21:in async_exec'            2: from ~/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/peek-pg-1.3.0/lib/peek/views/pg.rb:19:in ensure in async_exec'
1: from ~/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/activesupport-<IP_ADDRESS>/lib/active_support/core_ext/time/calculations.rb:251:in minus_with_coercion' ~/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/activesupport-<IP_ADDRESS>/lib/active_support/core_ext/time/calculations.rb:237:in minus_with_duration': stack level too deep (SystemStackError)

Can someone please merge?

Any likelihood of this getting merged?


Polygon collision failure.

demo

I can confirm this is a problem with Body.scale values that are less than 1.
https://codepen.io/anon/pen/zzQOZM

It has been repaired in the latest version.


Add timeouts

log exceptions

@DanielElisenberg Yes, this was tested with local Microdata stack.

Looks good :)


#855: Ban users without requiring them to be in the server

Based on some testing I did with a few friends, this should still purge the messages of banned users who left before they were banned.

yeah i'm not really sure what to do about the type errors here


Feature: Add an extension popup to show cookie summary.

Description
Adds an pop up accessible by clicking on the popup icon. This pop up shows 2 piecharts ( one for 1p cookies and other for 3p ) giving information about what category of use they lie in.

Screenshot/Screencast

Fixes #5

@ayushnirwal

Would you like to show the cookies count in this PR or later in a separate PR?
When we click on the already opened tab, can we at least show a message asking to refresh the page in v0.1 instead of not opening the popup?

@ayushnirwal Just noticed that we are using the word "Uncategorised" in DevTool while we are using the word "Unknown" in the popup. Let's only use one of them.

@ayushnirwal

Would you like to show the cookies count in this PR or later in a separate PR?
When we click on the already opened tab, can we at least show a message asking to refresh the page in v0.1 instead of not opening the popup?

@sayedtaqui

I have added badge text in this PR since that is related to pop up.
Also added messages "Please refresh the page"in the case you mentioned and "No Cookies" in the case if a tab does not use cookies.

@ayushnirwal It shows "no cookies" instead of asking to refresh page on already loaded tab.

and on a new tab


Fix bug in Eq instance for List

https://github.com/agda/agda2hs/blob/1b350f85ed02f44dd24c07bf7f147055cac6da2e/lib/Haskell/Prim/Eq.agda#L67-L70
Pretty sure it should be
iEqList ._==_  []  [] = true

Oops

Could laws be included as a bunch of @0 field? It'd even be possible to
postulate function extensionality to prove them given that the proofs will
never end up in the compiled code!

I guess they could, but it would mean you would always have to prove them. Also they wouldn't even have to be @0 since agda2hs erases all proofs (or is supposed to: #73).


CHEBI:64306, Wrong structure

Hi Adnan,
The current structure of CHEBI:64306, O-5''-β-D-mannosylqueuosine, has recently been proved wrong in pmid:34880214.
I have attached the new structure that shows that CHEBI:64306 is a O-4''-α-D-mannosylqueuosine.
You will have to update the ontology to CHEBI:64306 is an α-D-mannoside (CHEBI:27535)
Cheers,
Kristian
CHEBI_64306.txt

Now updated, great find.
Thanks


Active Effects from Item Qualities don't work

If you have an item (say, a weapon) with an Item Quality that has an Active Effect, that Active Effect does not work.

Similar to #24 - since the Item Qualities aren't actually being added as embedded items, right now there's no way to get this to function (sadly, not even a workaround).
If I can get items-embedded-in-items working in the future, I'll be able to support this though.

I should probably disable ActiveEffects on Item Qualities until I can support this..


Name of group Scenes will be overwritten

Happy Easter Erik,
short info to you. The problem with the overwritten scene names in the group also exists in homebridge-deconz.
If I change the name of the scene switch in the group in Homekit, this is overwritten after a homebridge restart again with the name value that is stored in deCONZ.
If I understood you correctly in Issue https://github.com/ebaauw/homebridge-hue/issues/1150, the problem should no longer exist with homebridge-deconz?
Best regards

Sorry didn't have time to look into this yet.  Does it still go wrong with v0.1.12 (or v0.1.11)?

No ding. I myself have not yet come to investigate the deCONZ/Ambilight thing further. Wanted me with the result still report to you the days 😉.
Yes, the problem is still there. However, it can be easily readjusted. Create a group with one scene. Rename the switch for the scene to HK then restart homebridge. Now the self created name is gone and the original scene name is displayed again.
I have this behavior only with the scene switches in groups.
Ps. After two days of using homebridge-deconz I have to say that the plugin really runs great. Very good work. Thank you 👌!

There's actually some leftover code from when I tried and sync the HomeKit names to the resource names and vice versa.  This actively overwrites the HomeKit name, but apparently HomeKit only picks this up, after restarting Homebridge.
There's even a bigger issue, that brightness and colour control are missing from the group main service.

In v0.1.13.

Good morning Erik,
very good that you could understand the problem and have already implemented a fix 👌. I am currently not at home but have installed the update and made a few successful test remotely. I think now everything works as expected. If I notice anything else about this case when I'm back home I'll report again.
Thank you very much!


Add findings enabled flag in datasources

Signed-off-by: Surya Sashank Nistala<EMAIL_ADDRESS>By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #99 (9e8bf29) into main (65395db) will increase coverage by 0.07%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##               main      #99      +/-   ##
============================================
+ Coverage     43.13%   43.20%   +0.07%
- Complexity      855      859       +4
============================================
  Files           167      167
  Lines          5826     5828       +2
  Branches        734      734
============================================
+ Hits           2513     2518       +5
+ Misses         3079     3078       -1
+ Partials        234      232       -2

Impacted Files
Coverage Δ

...lytics/transport/TransportIndexDetectorAction.java
0.00% <0.00%> (ø)

...g/opensearch/securityanalytics/model/Detector.java
71.27% <0.00%> (+1.81%)
:arrow_up:

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


Semantic HTML style

Issue: Initial file contains only non-semantic elements (div). Best practice is to use semantic elements.
Advice:
Semantic elements - elements with a meaning, which describes its content. Some of the examples are:
article
aside
details
figure
footer
header
main
Requirement: Convert all div elements into semantic elements.

Commit 931ad73 fixed this issue

Completed in commit 9c45f75


Problem in training YOLOv3

Training problem
I was training yolov3 with batchsize=2. I got the following error:
RuntimeError                              Traceback (most recent call last)
/usr/local/lib/python3.6/dist-packages/torch/utils/data/dataloader.py in _try_get_data(self, timeout)
    760         try:
--> 761             data = self._data_queue.get(timeout=timeout)
    762             return (True, data)

11 frames
/usr/lib/python3.6/queue.py in get(self, block, timeout)
    172                         raise Empty
--> 173                     self.not_empty.wait(remaining)
    174             item = self._get()

/usr/lib/python3.6/threading.py in wait(self, timeout)
    298                 if timeout > 0:
--> 299                     gotit = waiter.acquire(True, timeout)
    300                 else:

/usr/local/lib/python3.6/dist-packages/torch/utils/data/_utils/signal_handling.py in handler(signum, frame)
     65         # Python can still get and update the process status successfully.
---> 66         _error_if_any_worker_fails()
     67         if previous_handler is not None:

RuntimeError: DataLoader worker (pid 2546) is killed by signal: Bus error. It is possible that dataloader's workers are out of shared memory. Please try to raise your shared memory limit.

During handling of the above exception, another exception occurred:

RuntimeError                              Traceback (most recent call last)
<ipython-input-37-5dcbfe173d75> in <module>()
----> 1 gtf.Train(num_epochs=1);

/content/Monk_Object_Detection/7_yolov3/lib/train_detector.py in Train(self, num_epochs)
    386                     # Train mutation

Inference problem
I tired to train yolov3-tiny instead for 1 epoch. With this trained yolov3-tiny, I tried to infer the model
img_path = "/content/drive/My Drive/Monkai/PRImA Layout Analysis Dataset/images/Train/00001317.tif";
gtf.Predict(img_path, conf_thres=0.3, iou_thres=0.5);

I got the following error
AssertionError                            Traceback (most recent call last)
<ipython-input-20-c510a274f973> in <module>()
      1 img_path = "/content/drive/My Drive/Monkai/PRImA Layout Analysis Dataset/images/Train/00001317.tif";
----> 2 gtf.Predict(img_path, conf_thres=0.3, iou_thres=0.5);
      3 #Image(filename='/content/drive/My Drive/Career/output.jpg')

1 frames
/content/Monk_Object_Detection/7_yolov3/lib/infer_detector.py in Predict(self, img_path, conf_thres, iou_thres)
    100         self.system_dict["local"]["dataset"] = LoadImages(source,
    101                                                             img_size=self.system_dict["params"]["img_size"],
--> 102                                                             half=self.system_dict["params"]["half"]);
    103
    104         # Get names and colors

/content/Monk_Object_Detection/7_yolov3/lib/utils/datasets.py in __init__(self, path, img_size, half)
     65         else:
     66             self.cap = None
---> 67         assert self.nF > 0, 'No images or videos found in ' + path
     68
     69     def __iter__(self):

AssertionError: No images or videos found in tmp

Please help me with these problems.

Please check the path to dataset if it's proper or not

Let us know your code and directory structure if the error persists

I am also getting the ' No images or videos found in tmp' error,

Can you please share the code and folder directory details?

Hi
The link of my code: https://colab.research.google.com/drive/14A3siVGbRn3x-AbSwAJhThEu9iURlVFK?authuser=3#scrollTo=jHxFzDF5pABG
In this notebook, I have created a markdow as "error" the cell under that markdown has the error.
Also, I have solved this problem by copying the images to monkai folder the following command tells that
!cp "/content/drive/My Drive/Monkai/input.tif" "Monk_Object_Detection/example_notebooks/sample_dataset/ship/test/"

Ok. So it was more like a file not found error

yes, inside the yolov3 implementation there is a line
os.system("cp"  "<the link of image or video">  /tmp)

This particular line has some problem in execution. I also checked the /tmp/ folder !ls /tmp. It returned empty list. So what I think was the copy command which is being executed in os.system is working so files weren't copying to tmp folder and error has been thrown.

Try to change the .cfg file as mentioned in this link

We have made the necessary changes, please check if the error is now resolved for you?

Closing due to inactivity.

@abhi-kumar : I am getting an error while using pretrained yolov3 for Document Layout Analysis i.e.
AssertionError: No images or videos found in tmp
I am following this tutorial: https://pub.towardsai.net/object-detection-document-layout-analysis-using-monk-object-detection-toolkit-6c57200bde5
Also, are there any tutorials to fine tune the yolov3 models, that will be of great help! I have worked with darknet and ultralytics for yolov5. Din't see any information about fine-tuning on the Github page.
Thanks in advance!


Bug 1967933: Save logs to file as well in addition to stdouting

Adding provision to download logs from each script in addition to std-outing. This is to maintain compatibility with must-gather way of things.

@mrobson @davemulford you both might want to review this as well.
@davemulford think of this as an alternative image for must-gather (vs the one we manage) - excepte every script in it is run as a one off script.

I'd like to get the PR in today since the bug is marked against 4.8. The changes for renaming the repo from which the output gets downloaded versus avoiding specifying the "source-dir" can be done as a bug as well.
Let's get the feature of output downloading in first and we can fine tune it next week.

/lgtm

/lgtm
Once its all merged, I can test it out again..

/lgtm thanks!


rustfmt complains about long documentation lines after certain lifetime usages

A reduced test case is shown below. The first long documentation line is accepted, as it should be. The second line produces an error: line exceeded maximum width.
// This is a very long documentation line that does *not* bother rustfmt, because it does not come after the lifetime below.
fn foo() -> F<'a> {
    bar()
}
// This is a very long documentation line that *does* bother rustfmt, because it comes after the lifetime above.

This can occur in practice if the lifetime is 'static, or if the snippet is inside an impl<'a>.

This issue is for some reason hidden by some usages of a single quote after the problem lifetime statement.
// This is a very long documentation line that does *not* bother rustfmt, because it does not come after the lifetime below.
fn foo() -> F<'a> {
    bar()
}
fn hide_issue<'a>() {}
// This is a very long documentation line that now does *not* bother rustfmt, because the issue is hidden by hide_issue.

Another example, this time the quote inside a comment:
// This is a very long documentation line that does *not* bother rustfmt, because it does not come after the lifetime below.
fn foo() -> F<'a> {
    bar()
}
// ' <- single quote
// This is a very long documentation line that now does *not* bother rustfmt, because the issue is hidden by the commented single quote.

The implementation of Iterator for CharClasses assumes that the occurence of ' is the beginning of the literal char, which is wrong.


Add server health check to api endpoint

Add to the docker as well

@iomodo do we have some preferred way to do this? There is this library which does a similar thing. https://github.com/tavsec/gin-healthcheck.
Without the library I can do simple OK check

Also, a simple API call will also work in docker, right?


Getting the received data

Sorry guys for the noob question but maybe someone can help me.
I have a working esp_mqtt setup. I can publish data and subscribe to a topic.
My question is now:
How can I use the received data to turn a led on and off.
The received data gets processed in the void mqttDataCb() function.
The problem: I cant just do
if(data=="on") {
gpio_output_set(BIT0|BIT2, 0, BIT0|BIT2, 0);    //Set GPIO2 to HIGH / LED ON
}
What would be the right way?

Using strcmp function, lib string.h

This is how I use it now:
char key[] = "on";
if(strcmp(key, data)){
INFO("Turn SWITCH on!\r\n");
}
Do i have to use data oder dataBuf here?
With the version above the strcmp is always true.

hello eliabieri,
strcmp return 0 if both strings is equal, and make sure it end with '\0'

Ok this works now.
I do it like this:
if(strstr(data, wordToCheck)) {
INFO("matching string found'");
}


Disabling User Options

Hello, this isn't a bug, just a question. I am using CustomOreGen for a modpack and I want to not allow the user to have the ability (or need) to change the configuration file on the New World screen. What can I set in my XML files to not have them worry about that?
Thanks
Pap

Disregard. I found out that I can stop the imports of the .xml files using the CustomOreGen_Config_Default.xml file.


Support of mapping custom database types to custom application types

Hi,
we are currently evaluating ways to store measurements into a postgres database with entity framework core as the ORM that drives the backend. There is a postgres extension that implements a type that can handle various units (https://github.com/ChristophBerg/postgresql-unit).
Is it currently possible to implement npgsql type mappers and an entity framework plugin to map custom store types to a custom type that represents the same type in the application? Do you know any blockers that prevent this?

That's an interesting extension. It reminds me of the F# units of measure language feature.
Right now, I think the approach would be to write two plugins, one for the ADO.NET driver and one for the EF Core provider. Current examples of plugins include NodaTime, Json.NET, NetTopologySuite, and a few others.
Are you interested in submitting a couple of PRs for this?
/cc @roji @YohDeadfall

Npgsql 4.0 introduced a new plugin model that allows you to do exactly this - add support for additional types. We can still add support for extensions inside Npgsql itself (rather than a plugin).
However, your error seems to indicate that the extension does not implement binary transfer for the type(s) it introduces - ouch. PostgreSQL supports two encodings when transferring values - text and binary. Npgsql is a binary-only driver, and can only natively/easily support types which can be encoded in binary.
To make sure what the situation is, can you please execute the following against a database where your extension has been installed:
SELECT oid,typname,typreceive,typoutput FROM pg_type WHERE typname='unit'

If the typreceive column for your extension's type is null, that means there's no binary support. If that's the case I'd let whoever's maintaining the extension that you need it... It shouldn't be too hard to implement for this extension, I think.

Note: opened https://github.com/ChristophBerg/postgresql-unit/issues/21 to ask for binary transfer support, which would be the first step for 1st-class Npgsql support.

Note that even if full support for this extensions isn't there, EF Core has a proposed feature which would allow server-side conversions (https://github.com/aspnet/EntityFrameworkCore/issues/10861). This would allow you to simply cast the values to text server side before bringing them across, which would be an elegant way to map unit to .NET string.
Closing this issue as there's nothing immediate to be done. If the extension author adds binary support and Npgsql adds support at the ADO.NET level, we can reopen to add support at the EF Core level as well.


Implement ResourceCodeResolver.getLastModified

Hot reloading is currently unsupport for ResourceCodeResolver due to getLastModified always returning 0. By implementing the method on a best-effort basis, hot reloading can be supported when the resource URL points to a regular file outside a JAR file which is usually the case during development time. A project which uses jte can then be rebuilt which will cause the changes to be reflected.

I ran into this issue before. Looks like a reasonable fix. 👍🏻

@andreblanke thanks for the contribution! Right now, the ResourceCodeResolverTest fails, cause it isn't expecting this to work yet. Could you adjust the test and maybe add another test for the unhappy path, too?

Ah, sorry for that. I tested it locally but didn't actually check the tests. I will fix it.

Codecov Report
Base: 93.05% // Head: 93.00% // Decreases project coverage by -0.05% :warning:

Coverage data is based on head (a303b32) compared to base (3c4cfc5).
Patch coverage: 66.66% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #186      +/-   ##
============================================
- Coverage     93.05%   93.00%   -0.06%
- Complexity     1125     1126       +1
============================================
  Files            63       63
  Lines          2881     2886       +5
  Branches        464      465       +1
============================================
+ Hits           2681     2684       +3
- Misses          107      109       +2
  Partials         93       93

Impacted Files
Coverage Δ

...main/java/gg/jte/resolve/ResourceCodeResolver.java
83.33% <66.66%> (-6.15%)
:arrow_down:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


if index > 3

First of all congratulations for the excellent work. When I put more than 3 items, the following error occurs. Thanks for listening

thanks for your interest this issue is now fixed in v1.0.9


Create config that allows everything

I just started with cargo deny and I'm finding myself overwhelmed by the config. I was sort of hoping that if I just deleted everything it would deny nothing by default, but this doesn't seem to be the default behavior.
First off, I think this might be a nicer dev experience, allowing folks to opt in to checks over time. But barring that, I don't suppose there is an example config out there that means cargo deny check does nothing. I'd love to start with that and then add my own things to check.
Thanks a lot!

Most config options have a default lint level that matches the default config that gets generated if you don't have one already or you run init. You can either edit that default config to allow everything, or use -A denied to allow errors, or you can run individual checks like cargo deny check license sources bans advisories and fix each section as you go. IMO these are enough tools to gradually integrate a suitable config for your project, but I suppose I am biased in this regard.

I managed to get a config that allows everything by default like this, at least for my project:
[advisories]
vulnerability = "allow"
unmaintained = "allow"
notice = "allow"
unsound = "allow"
yanked = "allow"

[licenses]
unlicensed = "allow"
copyleft = "allow"
default = "allow"

[bans]
multiple-versions = "allow"
wildcards = "allow"
workspace-default-features = "allow"
external-default-features = "allow"
deny = []

[sources]
unknown-registry = "allow"
unknown-git = "allow"

I think it'd make sense for this to be the default, otherwise people new to cargo deny will be hit with a bunch of issues out of the gate. But I understand having an opinionated set of default checks too. Just my two cents!

we typically want to be somewhat opinionated with this, but maybe one could add your full-allow config to the docs? so new users that want to approach it that way can use that?

That works for me! Want me to open a PR? If you point me to the right place in the docs that'd be helpful.

One other thing I need to change for this default is maybe including something about only checking normal deps, not build and dev deps, but I can't figure that out.

To keep it clean I asked about it here: https://github.com/EmbarkStudios/cargo-deny/issues/563.