Override comment colour

Describe the solution you'd like
I'd like to be able to override comment foreground colour by doing something like this in my user settings:
"editor.tokenColorCustomizations": {
  "comments": {
    "foreground": "#FFFFFF",
  }
}

or
"editor.tokenColorCustomizations": {
  "[Night Owl]": {
    "comments": {
      "foreground": "#FFFFFF",
    }
  }
}

Is your feature request related to a problem? Please describe.
The above override doesn't work for Night Owl, while it works for other themes. I have tried this with double slash comments and /* .. */ style comments, for Go specifically.
Thank you!

Try this and let me know if it worked:
"editor.tokenColorCustomizations": {
  "[Night Owl]": {
    "comments": {
      "foreground": "#FFFFFF",
    },
   "comment.line.double-slash": {
      "foreground": "#FFFFFF",
    }
  }
}

@sdras Thanks for the suggestion. I tried and it didn't work unfortunately. I also tried this which didn't work either:
"editor.tokenColorCustomizations": {
    "comments": {
      "foreground": "#FFFFFF",
    },
   "comment.line.double-slash": {
      "foreground": "#FFFFFF",
    }
}

Try this:
"workbench.colorCustomizations": {
    "comments": {
      "foreground": "#FFFFFF",
    },
   "comment.line.double-slash": {
      "foreground": "#FFFFFF",
    }
}

Didn't work either 😞 I think it may be an issue with VSCode itself. Thanks for the suggestions (and for the themes!), @sdras.

Ok! I'll close it out then, sorry it's not working. Let me know if you think there's something to be done.

I finally got this to work:
  "editor.tokenColorCustomizations": {
    "[Night Owl]": {
      "comments": {
        "foreground": "#ffffff"
      },
      "textMateRules": [
        {
          "scope": "comment.line.double-slash",
          "settings": {
            "foreground": "#ffffff"
          }
        }
      ]
    }
}

None of these work for me😥😥😥

Thanks Ronny! I was searching for the exact same issue!

None of these work for me😥😥😥

The Last script posted by Ronny worked for me. Make sure your theme name is same. My theme name was "Night Owl (No Italics)" but the theme name in script is [Night Owl].


Updated for Jetbrains 2022.3

I wanted to use the plugin and JB wouldn't install so I figured I would review your previous changes and it appeared only to require version updates in the gradle.properties

Thank you @tlcoffee for submitting this. I am always falling behind on their release schedule.

@will7200 will do. Thanks!


4.x refactor app test prophecy, improvements

Changes already applied to original PR as per our private discussion.


fake commit

Description

Pull request checklist

[ ] add tests
documentation

[ ] add docstrings
[ ] if a module is new: add API reference file for Sphinx in docs/source/api-reference
[ ] check API doc of Sphinx - build and show with make sphinx && make open-sphinx

[ ] fix checks

start locale checks with make check
fix formatting with make format

Python type info

[ ] check and improve Python type info
[ ] double check if return types are specified - inclusing -> None

[ ] check TODO comments in code
[ ] check FIXME comments in code
[ ] check if the copyright header needs an update

just for testing


Administrator password reset vulnerability exists in RapidCMS Dev.1.3.1

RapidCMS Dev.1.3.1 was discovered to contain Administrator password reset vulnerability in admin/run-movepass.php.
For more details, see:
RapidCMS-PasswordReset.pdf

This issue has been solved in the latest release.
https://github.com/OpenRapid/rapidcms/commit/4dff387283060961c362d50105ff8da8ea40bcbe#diff-fc57d4c69cf5912c6edb5233c6df069a91106ebd481c115faf1ea124478b26d0


[e2e]: go tag main config-ldflags-assets slsa3

Repo: https://github.com/slsa-framework/example-package/tree/v13.0.869
Run: https://github.com/slsa-framework/example-package/actions/runs/11510731973
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.tag.main.config-ldflags-assets.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.tag.main.config-ldflags-assets.slsa3.yml
Trigger: push
Branch: v13.0.869
Date: Fri Oct 25 02:29:47 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/v13.0.870
Run: https://github.com/slsa-framework/example-package/actions/runs/11528093381
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.tag.main.config-ldflags-assets.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.tag.main.config-ldflags-assets.slsa3.yml
Trigger: push
Branch: v13.0.870
Date: Sat Oct 26 02:27:26 UTC 2024
Tests are passing now. Closing this issue.


Add archives, add jekyll-menus support, fix RSS

Closes: https://github.com/thelehhman/plainwhite-jekyll/issues/8

@thelehhman This PR catches up and rebases my changes that you had requested in https://github.com/thelehhman/plainwhite-jekyll/pull/28

Thanks @benniemosher.
There's a problem with this PR though , i.e. we can't use jekyll-menus because github pages doesn't support it. Github pages compatibility is highly desired.

@thelehhman suggested course here? I need jekyll-menus to support forestry.io, but would prefer to not verge from your repository here. Suggestions?


Variable height on vertical carousel?

Hey,
I've been using Slick for a year or so on various projects, and it is truly amazing! However I've hit a bit of a problem on a vertical carousel.
It's a leaderboard showing scores - I need to show six at a time out of about 30, hence vertical is the way I need to go, although it's the first time I've used this setting.
The problem I've got is responsive - on large desktop each item starts out as 60px height each, dictated by a square image, but with text on one line, so the overall height of the container is 360px (6x60). The elements don't have a fixed height, it's the image the makes it 60px.
But when I scale down to mobile, and the text starts to wrap onto two lines making each item taller than 60px, the carousel doesn't resize - it stays at 360px, with that height measurement being applied by the plugin.
It's as if there needs to be a 'variableHeight' setting, similar to the variableWidth setting for horizontal carousels.
Is there a way around this currently?

+1

Can you please create a jsfiddle showing the behavior?

I'm highly interested myself, since we need this functionality for a client. Will provide a fiddle next week.

Yeah, seems to be same issue. I've already provided a fiddle here: https://jsfiddle.net/yypesh3d/7/

1 for this feature

Currently having this workaround:
// Calculate the heighest slide and set a top/bottom margin for other children.
// As variableHeight is not supported yet: https://github.com/kenwheeler/slick/issues/1803
var maxHeight = -1;
$('.slick-slide').each(function() {
  if ($(this).height() > maxHeight) {
    maxHeight = $(this).height();
  }
});
$('.slick-slide').each(function() {
  if ($(this).height() < maxHeight) { console.log('');
    $(this).css('margin', Math.ceil((maxHeight-$(this).height())/2) + 'px 0');
  }
});

@iampuma That workaround fixed my problem perfectly. Thanks! Hopefully something along those lines gets worked into the official branch at some point.

Figured out how to let slick slider calculate the proper offset: currently slick uses the outer height of the first slider item and multiplies it with the slider index on each slide (getLeft function).
We've extended the getLeft function letting it count the sum of the outer heights of all previous slides:
if (_.options.vertical === false) {
    targetLeft = ((slideIndex * _.slideWidth) * -1) + _.slideOffset;
} else {
     //targetLeft = ((slideIndex * verticalHeight) * -1) + verticalOffset;
     var curIndex = _.currentSlide;
     var sum = 0;
     for (var i = 0; i < curIndex; i++) {
         sum = sum + _.$slides.eq(i).outerHeight(true);
    }
    targetLeft = (sum * -1) + verticalOffset;
}

@IMPLATRIX I tested your code on a simple vertical slider where some slides have different height and didn't worked, while also breaking the slide change animation (i'm using centerMode and focusOnSelect)

@fleps just tested it and it's also working with centerMode and focusOnSelect. You can see our workaround in action here: http://dev.riwa-architekten.de/innenarchitektur/einfamilienhaus-in-duesseldorf-urdenbach.html
Our animation also got broken and we fixed it with adding an extra CSS transition to our slider's .slick-track:
.vertical-image-slider .slick-track {         transition: .5s linear;     }

@IMPLATRIX ah now I see. My case is different than yours, I have a vertical slider that is almost 100% height but my slides are not using the entier height, they little boxes and I show 3 at time clipped in the center, it's a totally different scenario =)

@iampuma is it possible to make your workaround responsive? On smaller viewports the margin becomes too big for contents with smaller heights.

if (_.options.vertical === false) {             targetLeft = ((slideIndex * _.slideWidth) * -1) + _.slideOffset;         } else {           verticalOffset = 0;           for (var i = 0; i < slideIndex; i++) {             verticalOffset += _.$slides.eq(i).outerHeight(true);           }           targetLeft = (verticalOffset * -1);         }

@lucasskywalker I try your code
        if (_.options.vertical === false) {
            targetLeft = ((slideIndex * _.slideWidth) * -1) + _.slideOffset;
        } else {
-            targetLeft = ((slideIndex * verticalHeight) * -1) + verticalOffset;
+            verticalOffset = 0;
+            for (var i = 0; i < slideIndex; i++) {
+                verticalOffset += _.$slides.eq(i).outerHeight(true);
+            }
+            console.log((verticalOffset * -1));
+            targetLeft = (verticalOffset * -1);
        }

but doesn´t work - I have different gaps at top of slider

At another page https://www.gut-cert.de/home.html I use the carouFredSel (https://github.com/Codeinwp/carouFredSel-jQuery)  - works... but I´ll change all sliders to slick
with the slick slider the problem is known since 2015 - a fix would be great

Hi! Can you send me the code you use in your project<EMAIL_ADDRESS>

fixed for slick options => infinite = true

@lucasskywalker thanks for your code - are you doing PR, or should I do PR?
btw: is the project still supported by @kenwheeler ? over 700 issues are open and there are almost 150 PRs?
/*
        if (_.options.vertical === false) {
            targetLeft = ((slideIndex * _.slideWidth) * -1) + _.slideOffset;
        } else {
            targetLeft = ((slideIndex * verticalHeight) * -1) + verticalOffset;
        }
*/
// NEW
        if (_.options.vertical === false) {
              targetLeft = ((slideIndex * _.slideWidth) * -1) + _.slideOffset;
            } else {
              verticalOffset = 0;

              for (let i = 0; i < slideIndex; i++) {
                verticalOffset += _.$slides.eq(i).outerHeight(true);
              }

              if (_.options.infinite === true) {
                Array.from(_.$slideTrack[0].children).forEach((slide) => {
                  if (slide.dataset.slickIndex < 0) {
                    verticalOffset += $(slide).outerHeight(true);
                  }
                });
              }

              targetLeft = verticalOffset * -1;
            }
//

@zonky2 using your solution, thanks!
One issue discovered  - the animation is broken whenever I try going "prev" over a slide with more height than the rest

@zonky2 Your code throws syntax error in IE11.

@zonky2 Your code throws syntax error in IE11.

Because i use forEach ("forEach" doesn't work in IE11 => rewrite to "for")

In case anyone looking for IE11 support. This is the updated code
`if (_.options.vertical === false) {
targetLeft = ((slideIndex * _.slideWidth) * -1) + _.slideOffset;
} else {
verticalOffset = 0;
for (let i = 0; i < slideIndex; i++) {
verticalOffset += _.$slides.eq(i).outerHeight(true);
}
if (_.options.infinite === true) {
var $slideCounts = Array.prototype.slice.call( _.$slideTrack[0].children );
for (var i = 0; i < $slideCounts.length; i++) {
if ($slideCounts[i].dataset.slickIndex < 0) {
verticalOffset += $($slideCounts[i]).outerHeight(true);
}
}
}
targetLeft = verticalOffset * -1;
}`

I have a vertical slider which gets taller if slide(s) get taller from text wrapping.
I'd like to keep the overall height fixed, even if the slides inside change height randomly on resize. I'm using center mode and infinite and it just doesn't seem do all the dynamic math for heights -- i.e. keeping the center slide centered even if the slide below or above got taller from text wrapping.
My only work around is to use fixed heights for all slides and ellipsis the text so nothing ever wraps.
Also, I don't think @kenwheeler maintains this anymore, though I am grateful for this wonderful, free tool.

/ Calculate the heighest slide and set a top/bottom margin for other children.
// As variableHeight is not supported yet: https://github.com/kenwheeler/slick/issues/1803
var maxHeight = -1;
$('.slick-slide').each(function() {
  if ($(this).height() > maxHeight) {
    maxHeight = $(this).height();
  }
});
$('.slick-slide').each(function() {
  if ($(this).height() < maxHeight) {
    $(this).css('margin', Math.ceil((maxHeight-$(this).height())/2) + 'px 0');
  }
});

Thanks man! this work!

These approaches above didn't work for me to have variable height slides, but I was able to get it to determine height correctly in my scenario by changing the following sections of code:
if (_.options.vertical === false) {
  if (_.options.centerMode === true) {
      _.$list.css({
          padding: ('0px ' + _.options.centerPadding)
      });
  }
} else {
  // OLD
  // _.$list.height(_.$slides.first().outerHeight(true) * _.options.slidesToShow);
  // NEW
  let $verticalHeight = 0;
  _.$slides.each(function() {
    $verticalHeight = $verticalHeight + $(this).outerHeight(true);
  });
  _.$list.height(Math.ceil($verticalHeight));
  // END NEW
  if (_.options.centerMode === true) {
      _.$list.css({
          padding: (_.options.centerPadding + ' 0px')
      });
  }
}

if (_.options.vertical === false && _.options.variableWidth === false) {
  _.slideWidth = Math.ceil(_.listWidth / _.options.slidesToShow);
  _.$slideTrack.width(Math.ceil((_.slideWidth * _.$slideTrack.children('.slick-slide').length)));

} else if (_.options.variableWidth === true) {
  _.$slideTrack.width(5000 * _.slideCount);
} else {
  _.slideWidth = Math.ceil(_.listWidth);
  // OLD
  // _.$slideTrack.height(Math.ceil($verticalHeight));
  // NEW
  let $verticalHeight = 0;
  _.$slides.each(function() {
    $verticalHeight = $verticalHeight + $(this).outerHeight(true);
  });
  _.$slideTrack.height(Math.ceil($verticalHeight));
}

None of your solutions work fine, if you want to have true adaptive height with a vertical:true you have to add this code in "setDimensions" method :
if(_.options.adaptiveHeight === true){                 _.$list.height(_.$slides.eq(_.currentSlide).outerHeight(true));             } else {                 _.$list.height(_.$slides.first().outerHeight(true) * _.options.slidesToShow);             }
also add adaptiveHeight:true with vertical:true.

@Chousse would you mind share your solution into JSFiddle or Codepen playground? :)

Hi. I seem to have found a solution for slides of different heights. You need to add the following code to slick.js in getLeft function after line 1155:
if (_.options.vertical === true && _.options.variableHeight === true) {
    targetSlide = _.$slideTrack.children('.slick-slide').eq(slideIndex + _.options.slidesToShow);
    targetLeft = targetSlide[0] ? targetSlide[0].offsetTop * -1 : 0;
}

And add the parameter to the carousel settings:
variableHeight: true

Currently having this workaround:
// Calculate the heighest slide and set a top/bottom margin for other children.
// As variableHeight is not supported yet: https://github.com/kenwheeler/slick/issues/1803
var maxHeight = -1;
$('.slick-slide').each(function() {
  if ($(this).height() > maxHeight) {
    maxHeight = $(this).height();
  }
});
$('.slick-slide').each(function() {
  if ($(this).height() < maxHeight) {
    $(this).css('margin', Math.ceil((maxHeight-$(this).height())/2) + 'px 0');
  }
});

I had vertical slider.
For me in was need to add equal margin bottom for every slide item I need. So I adde it by css.
.slider-item {        margin-bottom: 40px; }
`var maxHeight = -1;
$('.your-slider .slick-slide').each(function() {
if ($(this).height() > maxHeight) {
maxHeight += $(this).height()+40;
}
});
$('.your-slider .slick-list').css('min-height', maxHeight-40 + 'px');`

Sadly that we stiil don't have the options for this...


Automated cherry pick of #10856

Cherry pick of #10856 on release-7.2.
/cc  @michelengelen
NONE

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: f5e95809587ff2aa1deb9751c388edbdac6feb9e


Copy Commit Message does not work anymore

GitLens Version: 10.2.1

VSCode Version: 1.42
OS Version: Windows 10 64-bit with latest updates

Steps to Reproduce:

Click in GitLens icon in Sidebar
Mouse right clicking in any commit, choose Copy Commit Message

@phamquocduy @bcanzanella I can't seem to reproduce this -- are you seeing this with 10.2.1? Can you please enable output channel logging by setting "gitlens.outputLevel": "debug" in your settings. This will enable logging to the GitLens channel in the Output pane. Once enabled, please reproduce the issue and attach the log from the GitLens channel.

@eamodio This is working for me now with 10.2.1 in VSC 1.42.1 and 1.43.0-insider

@eamodio I confirm this is working for me right now with 10.2.1 and VS Code 1.43 (switched back to GitLens, before that I have it disabled and used Git History because of this bug)


Error with add CustomizedProductSerializer

in doc:
from shop.serializers.bases import ProductSerializer

class CustomizedProductSerializer(ProductSerializer):
        class Meta:
            model = Product
            fields = [all-the-fields-required-for-the-list-view]

and add in url.py
    from django.conf.urls import url
    from shop.views.catalog import ProductListView
    from myshop.serializers import CustomizedProductSerializer

    urlpatterns = [
        url(ProductListView.as_view(
            serializer_class=CustomizedProductSerializer,
        )),
        # other patterns
    ]

I get an error
File "/web/garantum/urls.py", line 31, in
webapp_1_d116d9535b47 |     serializer_class=CustomizedProductSerializer,
webapp_1_d116d9535b47 | TypeError: url() missing 1 required positional argument: 'view'
my code in garantum/serializers.py:
class CustomizedProductSerializer(ProductSerializer):
        class Meta:
            model = Programm1c
             fields = ['id', 'product_name', 'product_code', 'product_url',
                  'product_type', 'product_model', 'price']

url.py
urlpatterns = [

     url(ProductListView.as_view(
                serializer_class=CustomizedProductSerializer,
        )),
      url(r'^robots\.txt$', render_robots),
      url(r'^sitemap\.xml$', sitemap, {'sitemaps': sitemaps}, name='sitemap'),
       url(r'^shop/', include('shop.urls', namespace='shop')),
       url(r'^', include('myapp.urls')),
    ]

Could not add the product_code field to the product list

Q1:

TypeError: url() missing 1 required positional argument: 'view'

which version of Django are you using?

Q2:
    url(ProductListView.as_view(
        serializer_class=CustomizedProductSerializer,
    )),
    ...

are there any special reasons, you want to hard code the Catalog's List view into your routes, I usually do it through my CMS pages.

Q1:

TypeError: url() missing 1 required positional argument: 'view'

which version of Django are you using?
Q2:
    url(ProductListView.as_view(
        serializer_class=CustomizedProductSerializer,
    )),
    ...

are there any special reasons, you want to hard code the Catalog's List view into your routes, I usually do it through my CMS pages.

Django version: 1.11.20 from https://github.com/awesto/cookiecutter-django-shop
There's no reason to do it that way. I need to display the article field in Catalog List and I do the documentation. Sorry for my English )

can I close this?


urllib3.exceptions.ProtocolError is not mapped

For example, a "connection reset by peer" error should ideally be mapped to the corresponding socket error.

@stevedalton thanks for reporting. I'm happy to review a PR if you want to add a mapping for it. :)


Update arrayvec to 0.5.1

This will also cancel the dependency of nodrop, which is deprecated there

Unfortunately, it also increases the minimum Rust release to 1.36; currently, this crate works on at least 1.20 (tested by travis-ci), and might even work on earlier releases. So this change can only be merged on the next incompatible release. Since the latest 0.4.x versions of arrayvec seem to have all the relevant fixes (through a build.rs which autodetects Rust 1.36 or later), and the only user of arrayvec is not part of this crate's API, I don't see any urgency.

@cesarb relying on very old crates make packaging for linux distro harder. arrayvec is used by other crates with the more recent version, blake2-rfc requires an old version which isn't compatible with the new version.
It forces us to either patch a crate, or to package two versions of arrayvec


OEL-3382: Styles css prefixer flexibility.

[skip chromatic]

https://github.com/openeuropa/bootstrap-component-library/pull/592/files
It's cloned after this pr. It was redone because of the failing tests, Approved on my side.


Be smart about scanning items for meta data

Right now, the LRU scans all items for meta data at instantiation.  This can be lazy operation.  Also, maybe allow configuring number of records to scan for performance optimization.

Closing this as the support has been added in release 0.1.0.


Fulltext search in docs

Comment from @gskyarts
Search :: The current search seems to look through navigation rather than document content, which isn't very helpful for me.

Idea: Use default shortcut to open/focus fulltext search.


Tapeline Visible In Map View

Is your feature request related to a problem? Please describe.
I would like to use tapeline to measure large distances, such as 5.5km away.
Describe the solution you'd like
When using the tapeline tool in the map view (default simple ruler), a line and a human readable number will appear when left-click dragging across the map.
Additional context
Current functionality is the tapeline is only visible in the normal view. Yes I acknowledge that in most cases, only people that are playing with fission and fusion weapons would ever need to know large distances such as 5.5km.

I would absolutely love this, but unfortunately it is impossible. Mods can't draw arbitrary things on the chart.


defadvice for shell-command-on-region breaks with C-u

I often sort lines of text by selecting a region, and using C-u M-|, for replacing the text in the region with the output of shell-command-on-region.
The defadvice clause for shell-command-on-region in init-compile.el breaks this.  It gives "wrong number of arguments".  I don't know how to fix this without simply deleting the defadvice clause, which makes it work again.

Thanks for reporting that — fixed!


Plumbs serialized particle expression through data structures

Assumption: We will pass expressions in a text format for which we have a Parser on the Kotlin side.
Plumbs expression as an optional field on the HandleConnectionSpec objects through all the data structures.

Manifest AST nodes.
Recipe data structure in TS.
Protocol Buffer
Recipe data structure in Kotlin.
Plan data structure in Kotlin.
Parcelable
Arc Host serialized state

(whoa!)
We don't yet have TypeScript serializer for expression, so this is plumbing a AST node kind field as a stop-gap.

Open question: Do we really want the field to be called expression ? I cannot come up with anything that's less vague but not very long :( maybe it will become clear what it is once the feature become well known ?

Oh, sorry, somehow I didn't see your comments! Will address in a follow up PR!


pysteps 2.0

During a meeting with @dnerini, @loforest and @wdewettin, the suggestion was made to start working towards a new version of pysteps.

The initial motivation is the implementation of the blending with NWP (#212) which is likely to require some API-breaking changes.
Secondly, a move towards a new data model based on xarray would simplify the existing code as well as the future implementation of the blending. This was suggested before, see #12 and this gist by @kmuehlbauer. This move might also simplify #215 and #5 since the self-describing structure of xarrays is close to the netCDF format.
Implementation of @dnerini's Kalman Filter would be nice to have.
The interface should be generic enough to allow for machine learning nowcasting and post-processing methods, which require multiple predictors. Maybe OO or duck typing can be useful here. The interface to perform the nowcasts could have a generic fit/predict/update structure.
Logging can also be improved: #140
Achieve 90% test coverage (#32)

At this point, it would be interesting to hear the opinion of the other core contributors @pulkkins @aperezhortal @cvelascof @RubenImhoff and in general the whole pysteps community.
Also, the list above is by no means exhaustive nor fixed. Version 2 will be the opportunity to introduce substantial changes based on our experience so far: we need your inputs! For this purpose, I created a new project where to list and track issues and PRs related to this effort.
In terms of roadmap, I'd suggest the following:
Phase 1: up to end of October 2021(*):

master contains pysteps-v1
version 1.5 is released, after which no more PRs into v1 except for bug fixes
setup new development branch for pysteps-v2 (alpha), features branches are forked and merged into this v2 branch

Goal: pysteps-v2 is published as a pre-release (beta)
Phase 2: up to September 2022 (ERAD):

previous master with v1 is renamed to pysteps-v1
development branch pysteps-v2 is renamed master and becomes default branch
submit abstract to ERAD2022

Goal: consolidate v2, which becomes default version.
Phase 3:

pysteps v2 is stable
pysteps v1 is discontinued (no more support)
paper on v2

Goal: close project, publish results.
(*) This deadline is given by time constrains on the RMI side, which has the lead on this initial phase of the project (@ladc )

Hi everyone,
Unfortunately I couldn't join the last meeting due to a week of vacation, but this looks really good! It is great to see how our initial idea to pick up blending again is a reason to move forward to a v2 of pysteps!
The present ideas are what I was thinking of as well, so not much to add at this point. I think that during my stay at RMI in August, we will come up with some (additional) ideas. The new importer that allows for importing radar and NWP data, hopefully allowing for some sort of up-/downscaling in space and time to match the resolution of the radar and NWP data, will be one of the challenges here. I think if we can make that as generic as possible, we already have a super useful tool (let alone for the blending and other functionalities that we will add). So, a great move forward!
In a week from now, I can devote most of my time to the project, so we will keep in touch!

Hi everyone,
Version 2 sounds great :muscle:
This new version maybe is a good opportunity to change the pystepsrc parameter file format. Although the JSON configuration file does the job, it has a few disadvantages. A better, and more readable, alternative is the YAML format that I personally like a lot.
Also, this is a good opportunity to revisit the interface as @ladc mentioned in point 4. For example, using an interface like Scikit-learn allows a better integration of PySTEPS with other ML libraries (pytorch, keras, scikit-learn, etc.).

Hi everyone,
I was also on a vacation, so I could not join the last meeting.
The points listed by @dnerini are very good and clearly show the limitations of the current version. It should be noted that its main goal was to implement a Python version of STEPS. As a result, the codebase is not generic enough to contain all the planned features (or even the currently implemented ones). The design was also a bit rushed since we needed a minimal working version for ERAD 2018. For implementing more advanced features like blending (item 1), a new data model (item 2) or a more generic interface compatible with machine learning methods (item 4), a major redesign makes a lot of sense.

Hi everyone,
I don't know if the topic has already come up but I would suggest splitting the library into multiple components (eg: core, visualization, etc.) so that you can only install what you need (eg: downscaling) and avoid what you don't need (eg: eg: visualizattion) and all related dependencies (eg: matplotlib).
I say this for all those like me who find themselves installing libraries in AWS Lambda (and similar) where space is precious and therefore it is very convenient to be able to install only what you need without having to manually delete files and folders after installation.

Hi @fox91, thanks for the suggestion. I like the idea.
I did a quick search and it seems that we can do that using Namespace packages without much effort.

Hi @fox91, thanks for the suggestion. I like the idea.
I did a quick search and it seems that maybe we can do that using something like Namespace packages.

I don't know how is implemented but I like the way I can install submodules in boto3-stubs.

Hi Fox
In a way, we already partially support the same idea by using optional dependencies.
What is still missing is the option to install a particular set of dependencies depending on the needs.
For example, pip install pysteps should only install the core dependencies (and I agree that matplotlib should not be part of it), while pip install pysteps[visualization] could come with matplotlib and pip install pysteps[develop] would come with black, pytest and other.
This should be easily achievable with the extra_requires option in the setup.cfg file.
Or do you have something different in mind?
(ps: I suggest we open a dedicated issue and continue the discussion there?)

I love the idea of using xarray for pySTEPSv2.
However wondering if we can agree the process we will follow with the conversion of original pySTEPS rutines into xarray.
The possible options that I can see are:

Modify the contents of each routine to work with both xarray and legacy and keep the name of the routine.
To create new versions of each routine solely using xarray and use different names for the new routines.
To create wraps of the original routines using xarray as it was proposed in this gist  by @kmuehlbauer.
A combination of all above.

What are your thoughts?

All very relevant points, thanks @cvelascof for raising them. This is my personal opinion:
I see this as an iterative process, where version 2.0.0 should include the changes necessary to provide support to xarray across the whole library. This means that all methods need to be able to use and return xarray objects instead of numpy arrays as it is currently done, meaning

no more ndarray, metadata dichotomy
all methods that apply to precipitation fields ingest and output a xr.DataArray object that includes x and y coordinates as well as timestamps when needed
deprecate utility methods that are already part of the xarray library (for example the utils.dimension module)

How this changes will be implemented in practice will depend a lot on the individual methods. My impressions is that most methods would benefit from a more in-depth refactoring leveraging the advanced functionalities of xarray, but I'm also conscious that this would represent a considerable effort. As a compromise, we could adapt the most complex methods (as for example the existing nowcasting methods) so that the input xarray object is simply converted into a numpy array at the beginning of the function, and back to an xarray at the very end (I guess this would correspond more or less to the wrapping option that you mention above).
Because the switch to v2, we can introduce breaking changes, including renaming methods and modules. So I wouldn't worry too much in terms of back-compatibility, although it makes sense to try and keep the general structure as close as possible to v1.
There will be plenty of details to sort out, but I think we can discuss and solve them within the relevant PRs as we proceed with the refactoring (see #223 for a practical example).


Report embedding broken -- GROUP_REPORT_EMBED_URL no longer supports accessLevel view

The getReportEmbedToken function, by default, uses accessLevel 'view'.  It seems that MS has made a restriction in what access levels that the GROUP_REPORT_EMBED_URL accepts -- 'view' is no longer supported and returns this error message:
Resource access type View unsupported for Reports
A Microsoft employee[1] has told me that we must use this URL instead:
https://api.powerbi.com/v1.0/myorg/GenerateToken
That endpoint takes different parameters, so it doesn't appear to be just a simple replacement of the existing GROUP_REPORT_EMBED_URL
[1] https://community.powerbi.com/t5/Developer/InvalidRequest-Resource-access-type-View-unsupported-for-Reports/m-p/2680597

Despite what the Microsoft employee said, this turned out to be something completely unrelated, and the currently used API call appears to be fine.


OLS Slides Redesigned

Thank you for the contribution made here, Olakid! 🙌
This will not be merged because OLS has a new brand identity. However, your work is worthy of acknowledgement!
Have fun creating more masterpieces in the future. 🚀


OpenBazaar

OpenBazaar 2.0 Server Daemon in Go and  use ipfs node.
OpenBazaar is an Online Marketplace | Peer-to-Peer Ecommerce
https://github.com/OpenBazaar/openbazaar-go

has one already...


Fixed #25085 -- Overrode Select Widget's deepcopy

buildbot, add to whitelist.

merged in 8ee6a3f1a855bf983639a14fc2393baa8ead741f, thanks!


Chessboard does not always render position updates

I've been working on building an app with Chessboardjsx that allows me to navigate through a list of board positions. I've really appreciated this package so far, but I believe I've encountered a problem. Given the following code, the Chessboard does not always update to the new position on button click. Sometimes it takes two increments/decrements for the board to catch up and update.
One example of the issue can be seen by clicking 1. Next, 2. Next, 3. Previous, 4. Previous. If you try for yourself, you'll notice that the board only updates on clicks 1, 2, and 4, but not 3. The issue seems to happen any time you switch direction from Next to Previous or vice versa.
Thank you very much for your help. Please let me know if there's any more information I can provide about this issue.
I've also forked your demo board so that you can see the issue in this sandbox: https://codesandbox.io/s/chessboardjsx-withmovevalidation-v2-forked-yxqgo
const fens = [
    'rnbqkbnr/pppppppp/8/8/8/8/PPPPPPPP/RNBQKBNR w KQkq - 0 1',
    'rnbqkbnr/pppppppp/8/8/3P4/8/PPP1PPPP/RNBQKBNR b KQkq d3 0 1',
    'rnbqkbnr/ppp1pppp/8/3p4/3P4/8/PPP1PPPP/RNBQKBNR w KQkq d6 0 2',
    'rnbqkbnr/ppp1pppp/8/3p4/3P1B2/8/PPP1PPPP/RN1QKBNR b KQkq - 1 2',
    'rnbqkbnr/pp2pppp/2p5/3p4/3P1B2/8/PPP1PPPP/RN1QKBNR w KQkq - 0 3',
    'rnbqkbnr/pp2pppp/2p5/3p4/3P1B2/5N2/PPP1PPPP/RN1QKB1R b KQkq - 1 3',
    'rnbqkbnr/pp3ppp/2p1p3/3p4/3P1B2/5N2/PPP1PPPP/RN1QKB1R w KQkq - 0 4',
    'rnbqkbnr/pp3ppp/2p1p3/3p4/3P1B2/4PN2/PPP2PPP/RN1QKB1R b KQkq - 0 4',
    'r1bqkbnr/pp1n1ppp/2p1p3/3p4/3P1B2/4PN2/PPP2PPP/RN1QKB1R w KQkq - 1 5'
];
const [position, setPosition] = useState(fens[0]);
const [index, setIndex] = useState(0);

const prevPos = () => {
    const newIndex = index-1;
    setPosition(fens[newIndex]);
    setIndex(newIndex);
    console.log('prev');
}
const nextPos = () => {
    const newIndex = index+1;
    setPosition(fens[newIndex]);
    setIndex(newIndex);
    console.log('next');
}

return (
        <div>
            <Chessboard
                position={position}
            />
            <button onClick={prevPos}>Prev</button>
            <button onClick={nextPos}>Next</button>
        </div>
    );

Hi @hwkerr, I see your point.  I am currently not maintaining the project because it's in a need of a rewrite which I do not have time for.  I hope you understand and if you would like to make a pr I can take a look and merge it in.

Thanks for your response @willb335, that's understandable. I'll look into it myself and make a pr if I come up with a solution.

@hwkerr on npm if you choose not to use your fork.  Thanks!

Fantastic! Thanks for making that happen


Contributor Issue

Even though I had made 31 commis in an organisation's repository, but i still doesn't count as one of the contributor...Why??
Ps: I didn't set my email as private and I also use my email to pull and push on the repository.

This is not GitHub support.


Unable to filter Date using Range

Hi!
I was trying to filter record based on Date using the Range class, But I met with an exception of
Expected a numeric. Got: string {"exception":"[object] (Webmozart\\Assert\\InvalidArgumentException(code: 0): Expected a numeric. Got: string at /var/www/html/standards-api/vendor/webmozart/assert/src/Assert.php:2074)
let me know if there is a work around ?

I would need more context on what you did.


WIP: Updated JSON-API to match 1.0 format

Changes:

"links" is not separated into a "relationships" object and a "links" object.
"linkage" has been changed to "data".

Awesome work, by the way, we this PR almost all of JSON API spec should be implemented.
There is, however, one case I would like to mention, since you your PR suggests that now Yaks will fully conform to the JSON-API spec. I think null resources aren't handled properly. If a has_one relationship is included in the resource, and it's null, as far as I remember from the JSON-API spec, the JSON should be
{
  "type": "post",
  "relationships": {
    "author": null
  }
}

Currently Yaks just ignores keys of missing associations. For has_many I think it should be an empty array.

One other thing, in the JSON API specification it says that the type key can contain both the plural and singular name of the resource (people or person), and that it's the choice for the implementer which one to pick, it just needs to be consistent.
I'm wondering if there is a way to allow the JSON-API formatter to be configured, so that users can choose for themselves, with plural being the default. For example, I think I want to switch to singular :)

I did not noticed about the null/empty linkage. I'll try to work on it and see how it goes.
For the type, I already thought about it a few times but I did not come up with a solution yet. If you have any suggestions, let me know.

@plexus @janko-m I need your feeback on this. In order to build self and related urls, should we add some new options to the associations/links or should we use something like this:
class PostMapper < Yaks::Mapper
  link "relationships.user.self", template: "whatever"
  has_one :user
end

Sorry for the slow reply, days are pretty full. I'll get back to you probably tomorrow.

@plexus No problem :wink:

Yeah, I don't really use links, so I don't have the feeling for what option is better. I'm currently on a long programless holiday, I won't really be available to review PRs for some time now, so don't wait for me ;). I wish you luck with this PR, it's great that you're doing this.

@janko-m Enjoy your holidays then! :tropical_drink:

@plexus @janko-m I need your feeback on this. In order to build self and related urls, should we add some new options to the associations/links or should we use something like this:
class PostMapper < Yaks::Mapper
  link "relationships.user.self", template: "whatever"
  has_one :user
end

My first reaction would be that the user links would be part of rendering the user.
class PostMapper < Yaks::Mapper
  has_one :user
end
class UserMapper < Yaks::Mapper
  link :self, '...'
end

It seems JSON-API expects the self link to change depending on what top-level object you're including it from. I'm not sure I see the use case for that, but I can think of a couple ways to do that, for example.
class PostMapper < Yaks::Mapper
  has_one :user, mapper: Class.new(UserMapper) { link :self, ... }
end

Looks good :ok_hand: . I'm gonna merge it, we can take the discussion about the related object links to a separate issue.


Add CodeOwners

We should have a CodeOwners and require approval from them before merging.

Closed by PR #13


Why audio permission is always required

I found that for the library to correctly work the app must have the audio permission granted.
It would be nice that the library only use audio related features in the case that the streaming is going to have audio input.

But it's very strange case when you're streaming without any sound О_о

I think that there are a few use cases in which is not strange at all streaming without sound, imagine for instance that you want to do a presentation of your new android app and all you want to show is the screen of your device. No sound involved, hence, I think that the sound permission should only be needed when the streaming has audio enabled.

Strange. I have upgraded to new release that should work with no audio permission, but now  RTMP does not work without prepareAudio() at all.
    java.lang.NullPointerException: Attempt to invoke virtual method 'void android.media.MediaCodec.start()' on a null object reference
        at com.pedro.encoder.audio.AudioEncoder.start(AudioEncoder.java:79)
        at com.pedro.encoder.BaseEncoder.start(BaseEncoder.java:25)
        at com.pedro.rtplibrary.base.Camera1Base.startEncoders(Camera1Base.java:490)
        at com.pedro.rtplibrary.base.Camera1Base.startStream(Camera1Base.java:480)

1.8.5.

This change was only for display mode. In camera1 and camera2 audio is necessary. I basically merged PR that only included display mode.
I will implement it in the next version if you need it.


How to plot two histograms in the same window ?

Hi,
I need to compare two histograms together, I would like to plot them in the same window. When I pass legend option Visdom crashes.

Visdom histograms currently do not support more than one entry at a time. You can directly display plotly figures if you use the feature introduced in #402, which will let you display any kind of histogram offered by the Plot.ly Python Library

I see. Thank you.

Of course, it'd be possible to more natively support this feature in the same way we do for line and scatter plots. Feel free to open a feature request if you find that the Plot.ly version doesn't suit your use case.


Expand pip commands into separate entries

My serverless.yml file looks like:
pythonRequirements:
    pythonBin: ".venv/bin/python3"
    invalidateCaches: true
    pipCmdExtraArgs:
      - "--trusted-host pypi.python.org"
      - "--trusted-host pypi.org"
      - "--trusted-host files.pythonhosted.org"

Before this change, the generated pipCmds looked like:
pipCmds: [
  [
    '.venv/bin/python3',
    '-m',
    'pip',
    'install',
    '--trusted-host pypi.python.org',
    '--trusted-host pypi.org',
    '--trusted-host files.pythonhosted.org',
    '-t',
    '~/Library/Caches/serverless-python-requirements/01dde858536f71d67120395d14566ab6f37cce626069dc5d99d750fcd314e5c2_slspyc',
    '-r',
    '~/Library/Caches/serverless-python-requirements/01dde858536f71d67120395d14566ab6f37cce626069dc5d99d750fcd314e5c2_slspyc/requirements.txt',
    '--cache-dir',
    '~/Library/Caches/serverless-python-requirements/downloadCacheslspyc'
  ]
]

And resulted in this error:
Error --------------------------------------------------

  Error: STDOUT:

  STDERR:
  Usage:
    ~/envs/git/myproject/api/.venv/bin/python3 -m pip install [options] <requirement specifier> [package-index-options] ...
    ~/envs/git/myproject/api/.venv/bin/python3 -m pip install [options] -r <requirements file> [package-index-options] ...
    ~/envs/git/myproject/api/.venv/bin/python3 -m pip install [options] [-e] <vcs project url> ...
    ~/envs/git/myproject/api/.venv/bin/python3 -m pip install [options] [-e] <local project path> ...
    ~/envs/git/myproject/api/.venv/bin/python3 -m pip install [options] <archive url/path> ...

  no such option: --trusted-host pypi.python.org

      at ~/envs/git/myproject/api/node_modules/serverless-python-requirements/lib/pip.js:324:13
      at Array.forEach (<anonymous>)
      at installRequirements (~/envs/git/myproject/api/node_modules/serverless-python-requirements/lib/pip.js:311:28)

After this change, the pipCmds looks like which seems to pass successfully to pip:
pipCmds: [
  [
    '.venv/bin/python3',
    '-m',
    'pip',
    'install',
    '--trusted-host',
    'pypi.python.org',
    '--trusted-host',
    'pypi.org',
    '--trusted-host',
    'files.pythonhosted.org',
    '-t',
    '~/Library/Caches/serverless-python-requirements/01dde858536f71d67120395d14566ab6f37cce626069dc5d99d750fcd314e5c2_slspyc',
    '-r',
    '~/Library/Caches/serverless-python-requirements/01dde858536f71d67120395d14566ab6f37cce626069dc5d99d750fcd314e5c2_slspyc/requirements.txt',
    '--cache-dir',
    '~/Library/Caches/serverless-python-requirements/downloadCacheslspyc'
  ]
]

The build failures don't seem related to this change.

@dschep any thoughts on this one?

@dschep I was wondering when the next release is going to be released that includes this change?  Thanks.


Added regular roulette

So uh.. yeah, regular casino roulette!
I haven't set up an environment to run Commando so I only tested the roulette mechanics, not quite sure how it's going to work with currency and discord yet, which means the code needs good reviewing and some testing.

I'll let Grey review over the rest, since some of the things modify code he wrote 👌
No other changes requested from my side so far.

Sure thing


Create a dockerfile to try out the Jaxa bridge

Name is descriptive enough.

Created a docker in this branch: https://github.com/traclabs/brash_docker/tree/feature/jaxa_bridge . We do not merge and leave it as a branch as we are using the jaxa bridge purely for testing purposes.

Instructions on how to run are in the README  file


Moved exit-1 to the last step so push can happen before that

Current dependencies on/for this PR:

main

PR #172   👈

This comment was auto-generated by Graphite.


[Question] Could materialized view work for historical data?

Hi, I have some doubts after I read the design document.

Could materialized view work for historical data?
If it could, so it relies on the sourcer(kafka,mq) to reset the offset to the earliest?

Yes, if the offset of source can be subscribed.
Another option is creating a materialized source, sorry for that it’s not well documented, but in fact it’s an materialized view behind a source. Then while you create a new mview based on the materialized source, the historical data can be recovered, see https://github.com/singularity-data/risingwave/blob/main/docs/mv-on-mv.md for more information.


Update documentation with the steps to hexify x5t claim in JWT header

We have introduced a new system property to be enabled to hexify the x5t claim in JWT header in WUM update for API-M 2.2.0.
If the x5t claim needs to be hexified, the system property enableThumbprintHexify has to be set as true, as mentioned below:
./wso2server.sh -DenableThumbprintHexify=true
The documentation of "Passing end-user attributes to back-end using JWT" - [1] needs to be updated with the above information.
[1] https://docs.wso2.com/display/AM220/Passing+Enduser+Attributes+to+the+Backend+Using+JWT

Closing Since the issue has been stale for a long time.


Iterative pogs memory leak in MATLAB 2018

This is a really great method! However, I have embedded pogs in a loop that iterates many many times and there is a memory leak resulting in an inevitable crash. Any ideas/suggestions? I'm assuming somewhere an array needs destroying.
Thanks

Ah, may be worth mentioning I'm running this on Centos7.

Same problem on my end, ran valgrind on it, the leak is here:
matrix_dense.cpp +83
  // Copy Matrix to GPU.
  _data = new T[this->_m * this->_n];

Please Pogs project team, fix this :)

This response is way overdue. Thanks for reporting that @CPLambert and for figuring out the issue @eloi-loomai. Definitely a bit embarrassing to forget to delete the data from a matrix class. I've pushed a fix.
@bungun thanks for the notes on fixes. I'll try and diff to see what else might be missing.


Using ClockCycles and RisingEdge concurrently breaks coroutines

When a forked coroutine tries to yield a RisingEdge, while their parent is waiting for a yielded ClockCycles (to wait for N cycles without using a for loop), the RisingEdge does not register and the coroutine just hangs.
In the log, a message like the following appears:
Attempt to prime an already primed trigger for cbValueChange!
We seem to already be registered, deregistering cbValueChange!

A workaround is to use a for loop in the parent, but I guess this should either be fixed or documented in any case. This happens for both GHDL (VHDL) and Icarus Iverilog (Verilog).
I prepared two simple test cases to reproduce the issue:

VHDL: clockcycles_testcase.zip
Verilog: clockcycles_testcase_verilog.zip

I realize this was already flagged in #520 and was recently fixed on current master. Looking forward to the next release then!


Notification issue as well

like #2 i am having notification issues on windows 10. firewall port 5115 is open on tcp

added the notification entity in config.yaml and it shows in automations
notify:
  name: "reepc"
  platform: hass_agent_notifier
  resource: http://<IP_ADDRESS>:5115/notify

running curl from with in HA shows
curl -i http://<IP_ADDRESS>:5115
HTTP/1.1 200 OK
Content-Length: 17
Server: Grapevine/<IP_ADDRESS> (Microsoft Windows 10.0.19043 ) Microsoft-HTTPAPI/2.0
Date: Sat, 05 Feb 2022 02:09:39 GMT

HASS.Agent Active#

and it is installed in ha via hacs

Hi @noideaman, I've had a few reports of Windows not showing notifications.
Can you please check your windows (new) configuration screen, and check the notification settings to reflect these:

Also, is the HASS.Agent icon visible in the system tray (next to your clock)?
Lastly, have you used any 'telemetry disabling tool' like o&o-shutup or decrapify?

Closing this as stale, but let me know if you still need help


MWPW-141104: Consume Milo SWC in the unit tests

Consume Spectrum Web Components from Milo in CC unit tests
Resolves: MWPW-141104
Test URLs:

merging as commercev3


Feat: Convert the project into Next js 13 with Typescript 🚨

What feature?
The new feature is requested to convert this whole project into the Next js 13 with app routing and typescript. The base css library will be tailwind along with flowbite-react for extra component library support.
Need developers as soon as possible
Add screenshots
N/A
Record

[X] I agree to follow this project's Code of Conduct
[X] I want to work on this issue

This has been completed


Invalid predecessor DB.SCHEMA."[]" was specified.

Provider Version
snowflake-labs/snowflake v0.37.0
Terraform Version
Terraform v1.2.2
Describe the bug
terraform apply first time using 0.37.0 got to
 Error: error removing old after dependency from task DATABASE|SCHEMA|TASK: 091085 (42601): Invalid predecessor DATABASE.SCHEMA."[]" was specified.

   with module.snowflake.snowflake_task.task,
   on ../../../modules/snowflake/tasks.tf line 1, in resource "snowflake_task" "task":
    1: resource "snowflake_task" "task {

Expected behavior
Should apply the changes correctly.
Code samples and commands
resource "snowflake_task" "task" {
  database             = "DATABASE"
  name                 = "TASK"
  schema               = "SCHEMA"
  warehouse            = "WAREHOUSE"
  schedule             = "60 MINUTE"
  sql_statement        = "call merge()"
  enabled              = true
  user_task_timeout_ms = 86400000
}

Additional context
We experienced the issue described on https://github.com/Snowflake-Labs/terraform-provider-snowflake/issues/1071 before the current problem.  Upgrading to 0.37.0 seemed to solve it, yet we're facing this new one.
UPDATE: It only showed once after the first apply.  Following plans seemed to work as expected. Will expand shortly.

@ezeql We're also now seeing this problem. Hopefully gets sorted quickly. I'm sure there's some hacky way to get around it - but I've not quite figured it out yet!

hi @jodybuz, this issue only showed up once, the first apply after using 0.37.0.
terraform plan worked correctly afterward.

I can confirm with @ezeql that this issue only occurred once after upgrading to 0.37.0. Stranger even that this issue only occured on 1 task out of 8 possible ones. Whatever changes it tried to make caused the task to become suspended.

Hey @ezeql
Since 0.37.0 is a very old version, I have to close the issue. In the provider's v0.99.0 version, the tasks will be reworked and should not contain any of the reported issues and should be feature-complete (except for newly added parameters that appeared in the documentation a few days ago). In case of any problems, please create another issue (we are currently more responsive and try to fix or categorize the issues for future improvements as soon as possible)


Demo showing the usage of blur effect with this custom transition

In response of the issue "Trouble when ModalView has a clear background #4", I made a demo using the same music player transition, but adding blur effect to the modal view.

Thanks!
I will try.

Problem appears to be resolved.
Thanks!


Deprecate repository

Updates everything to point at urbit/urbit.

presses F

requests changes

Are we going to unpin it from github.com/urbit?


nodeCheck.waitForKuberhealthy doesn't actually make a request to kuberhealthy

Describe the bug
The comment above nodeCheck.waitForKuberhealthy says waitForKuberhealthyEndpointReady hits the kuberhealthy endpoint every 3 seconds to see if the node is ready to reach the endpoint. but in fact that function only builds the request, it doesn't then use it with http.Client.Do(..).
This may be deliberate, but then the description doesn't really make sense to me.
Steps To Reproduce
Doesn't really need reproduction, it can be seen in the code.
Expected behavior
I would expect it to create the http.Client on line 80, perhaps with a 1 second timeout, and then, after creating the request, add a call to execute the request
_, err = client.Do(req)

and then just check for the error, if there is no error then return success/push to the doneChan.
I'm happy to PR a fix for this, if you agree this is a bug.
Screenshots
If applicable, add screenshots to help explain your problem.  You can paste images into github and they will upload automatically.
Versions

Cluster OS: [e.g. CoreOS 1800.1.0]
Kubernetes Version: 1.9.7
Kuberhealthy Release or build [e.g. 0.1.5 or 235]

Additional context
Add any other context about the problem here.

Doing some quick testing in a fork  by changing the call to a plain http.Get I was able to get this working very quickly.
I do see the same issue @martinsmatthews posted, since just an http.Request is being created an http request is never made to the kuberhealthy endpoint. So in reality all that check is doing is testing that net/http can created a value http.Request object with the given parameters.


chore: add release scripts and .gitignore file

This pull request is intended to:

Add release scripts in each package inside the /shared directory.
Add release scripts in the root package.json file.
Add a .gitignore file.

bumpp-version@latest - Used to bumpp the version of a package interactively.

npx changelogen@latest --release - This command is used to automatically generate a changelog based on the commits made. The --release flag automatically bumpps the package version.

The .gitignore file's content was automatically generated at: gitignore.io

Thanks for the contribution bro 🙏🏿


[4628] Update logic for setting start_dates from HESA

Context
https://trello.com/c/vEFfu1V8/4632-s-rename-commencementdate-as-traineestartdate-in-the-database-and-in-register
Changes proposed in this pull request

Set itt_start_date and commencement_date to ITTCOMDATE if present
otherwise use the required field COMDATE

Guidance to review
Important business

[ ] ~Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?~
[ ] ~Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?~

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 [M] Re-map HESA ITTCOMDATE & COMDATE


Caregiver Information and Onboarding

There is a lot of information from the caregiver that we need to gather, in order for the nonprofit to function properly. This was historically done with inconvenient methods, and so this is one of the main components of the app right now.
The Task
Your task is to write the caregiver form to fill out information about them. Don't put every field on the same page, but separate them out by topic (for example, name/address/phone/email should probably be on the same page). It should also publish this data to our firebase schema.
You need to include the following fields:

Caregiver Name
Address
Cell Phone
Email address
Number of people in household
Number and age of other children in household
Name of Mother
Estimated due date of Infant or date of birth (if the baby has already been born)

Make these conditional fields with a question before: Has the baby already been born?

Do you prefer text, phone or email communication?

Each field needs to have the proper validation, and let the first four be required.
Important Information

Don't worry about styling just yet.
Make sure that once each page is passed,
You won't have authentication finalized yet, but do your best to work without it

Right now, firebase is set up to allow anyone to read or write to the database. You can give the caregiver a fake caregiver id, and feel free to name the fields in the schema whatever you want.

Never hesitate to reach out for help!

@saisahitib Update: leave out "name of mother" and "estimated due date of infant"!


Add a pivot to the tab color picker

Summary of the Pull Request
The "Custom" button on the color picker is a bit strange (at least with regards to accessibility). It's a magical toggle button that reveals/hides content but doesn't interact with the formerly existing content. Instead, I've added a Pivot control to introduce two "modes": "Standard" and "Custom". This aligns with the experience on MS Word's color selection for text.
With regards to accessibility, the role of the "Custom button" won't matter anymore because it won't exist.
While I was here, I also added did a little bit of code cleanup by sprinkling a few const around and removing some unnecessary XAML code.
References
Will resolve the following bug upon validation from a11y team: #12045
Validation Steps Performed

color picker works
acceptable keyboard navigation on the color picker

Ah, I should fix https://github.com/microsoft/terminal/issues/12044 and https://github.com/microsoft/terminal/issues/12043 too

Does it automatically focus Pivot when we determine that the color was already Custom?

We can now close accessibility bugs. The accessibility team will verify them and mark them with another validation label after they've been closed.

Does it automatically focus Pivot when we determine that the color was already Custom?

No. But we could do that pretty easily (and in this PR too). Just a matter of...

get the color of the tab
if it's one of the standard colors --> pivot.selectedItem = 0
else --> pivot.selectedItem = 1

#12044: renamed the "Reset" button
#12043: closed as "by design". We could go through and replace all the color buttons with toggle buttons, but then we need to collect the current color from the tab, compare it, ensure the source is from runtime color picker, and toggle the color that's selected. But I worry that'll cause more confusion as this is a one-directional scenario where you "set" the color regardless of the existing one. If you disagree, please submit discussion in that thread/issue.

One idea I had for the color picker was to have the ability to choose custom colors that can be reused, just like you can in Paint. Would this design work with that?

One idea I had for the color picker was to have the ability to choose custom colors that can be reused, just like you can in Paint. Would this design work with that?

Oooh! I like that! Yeah we could add those to the "custom" page where we have a "Recently used" section at the top?
Alternatively, we could put them under the "Standard" tab, but I personally don't like how Paint creates "white" tiles that then get updated with the recently used one.

One idea I had for the color picker was to have the ability to choose custom colors that can be reused, just like you can in Paint. Would this design work with that?

Oooh! I like that! Yeah we could add those to the "custom" page where we have a "Recently used" section at the top? Alternatively, we could put them under the "Standard" tab, but I personally don't like how Paint creates "white" tiles that then get updated with the recently used one.

We should for sure write this one up for the backlog!

I think this is mostly fine, but I'm still really concerned about the hardcoded widths... Basically: Will zis break when "Custom" gets translated as "BENUTZERDEFINIERT" (6 vs 17 chars)?

Yeah, text is definitely gonna get cropped when the strings get localized into longer text. I don't suggest merging this to main until that gets fixed.
Sent an email to Xaml Discussions to see if I can get some help. I've tried a few things (as mentioned here) and no luck (yet)

Drafted it so we don't accidentally merge it

After quite a bit of trying, I just couldn't get Pivot to work right. There's something funky going on with the width and Pivot just isn't having it.
Since this would just close an a11y sev3, there isn't a major urge to get this done right now. I'll leave the branch up in case anybody wants to pick up where I left off.


Fixes: TypeError: Cannot read property 'getChildCount' of undefined

This happens when i open a {

Please merge this PR, the current behaviour makes block-spacing pretty useless in this state. :(


Object of class Illuminate\Routing\UrlGenerator could not be converted to string laravel 5.2 Error

I install new laravel 5.2 project and i m facing this error on URL() code. Please suggest any thing.

use {{ url('/')}}


Update convert to properly handle complex Jinja expressions

Resolves https://github.com/conda-incubator/conda-recipe-manager/issues/166

If you compare with the last integration test with main, this fixes compatibility with 6 recipes. Nice!


Bug Report, settings not showing

Environment
Windows build number: 1903
Windows Terminal version (if applicable):

Any other software?

Steps to reproduce

load up windows terminal (dev build) after clean install
Expected behavior

I'm expecting to see a settings drop down where i can select to use, powershell, windows etc.
Actual behavior

When i open the console i can only see powershell (see attatched)

Do you try pressing Ctrl + t ?

Please read the README before filing new issues.


Documentation updates for Worker API changes

We've made some changes to the Worker API with 5.6 and we'll want to update the docs, guides, and release notes to reflect those changes.

Merged with #10078


Possibility of passing callback function tu be run after render/hydrate

ReactDOM.render and ReactDOM.hydrate receives an optional callback as third param, which is called after the render/hydrate has finished. There is a possibility to do that with this package? Because in the docs is not specified.
Link to React's doc: https://reactjs.org/docs/react-dom.html#hydrate & https://reactjs.org/docs/react-dom.html#render
Thanks!

Looks like we don't currently support it, but we actually use it ourselves:
https://github.com/single-spa/single-spa-react/blob/master/src/single-spa-react.js#L192
The whenFinished function is just a custom function defined by us
https://github.com/single-spa/single-spa-react/blob/master/src/single-spa-react.js#L91-L93
so I think it would just take passing in a new function and adding it to the body of whenFinished so it's called before resolve(this).
Would probably be a good candidate for a first PR :)

This is already possible and supported - no changes needed:
const lifecycles = singleSpaReact({...})

export const mount = [
  lifecycles.mount,
  async () => {
    console.log('ReactDOM.render / ReactDOM/hydrate finished!')
  }
]

See more in these docs about setting lifecycle functions like mount to an array.


Implement native Video API support

Once we have libtheora, libwebm or another native video library enabled, we should implement initial video API support
Blocked by: #774

Bump.
Also, if WebM was to get support, it would be nice to get WebP support as well.


Edit Occurrence from calendar broken

Looks like the links on the calendar for Edit Occurrence (in the tooltip for a recurring event) are no longer formatted correctly - they point to a view instead of an edit page, e.g.
http://events.olin.build/view/59774841e44d46221dfd0fa8/1501632000000
vs.
http://events.olin.build/edit/59774841e44d46221dfd0fa8/1501632000000
The process of editing still goes fine if you can get to the right page, the link is just wrong.

I take it back the process of editing does not go fine, it's all broken and idk if it's us or ABE.

@audreywl I can't determine if this is on our end or on the backend. The logs for the backend are revealing nothing, so I have no insight into what's causing the problem.
@Elepert or @newsch, should the URL for a PUT request for a recurring event be in the form of /events/<sid> or /events/<sid>/<rec_id>? Right now we're sending it as the former, but it looks like we've been doing that for a while...

Scratch that, I was looking at the wrong log. This is what it says:
File "/app/abe/resource_models/event_resources.py", line 138, in put
|     received_data['rec_id'] = dateutil.parser.parse(str(received_data['rec_id']))
| NameError: name 'dateutil' is not defined

Resolved in olinlibrary/ABE#bbb60d1


changed reference for color module. now resolves

I think this should already be fixed in https://github.com/ExpediaDotCom/alexa-conversation/pull/5
Let me know if that's not the case.
Thanks!


New release?

Latest release of gem was at 2016
There are sevral improvements since:
https://github.com/leonid-shevtsov/headless/compare/v2.3.1...master

@leonid-shevtsov any news?

Another year goes on :(
@leonid-shevtsov any news?


🧰 Feature: replace remappings.txt with foundry.source

Component
Forge, Cast, Other (please describe)
Describe the feature you would like
foundry.source.json - replacement for remappings.txt
TLDR: remappings.txt is encumbered by its own inadequacies. Such great rust-based tooling must be worthy of superior configuration format than the currently used one.

🧰 This is a working draft, your questions and critiques are most welcomed.

[TOC]

Motivation

current format: remappings.txt

proposed format: foundry.source.json

Import Maps

Import Maps

Generate the import mappings

Inline Imports for contract testing

💡 Motivation
Replace remappings.txt with a new file, foundry.source.json

Remappings can be auto-generated and declared
Support for inline imports via remapping
Support for parallelizable test running via @Scoping
Support for Resolutions for changing a contract dependency with a user replacement
Support for nested directory structures for monorepos
Enable deployment a more declarative deployment process (potentially)

Here are some issues that this change can potentially solve / eliminate
https://github.com/gakonst/foundry/issues/768 - resolutions
https://github.com/gakonst/foundry/issues/744 - inline imports
https://github.com/gakonst/foundry/issues/768 - scopes
🚨 current format: remappings.txt
ds-test/=lib/ds-test/src/
@openzeppelin/=node_modules/@openzeppelin/
@rari-capital/=node_modules/@rari-capital/
@manifoldxyz/=node_modules/@manifoldxyz/

🛠 proposed format: foundry.source.json
Foundry Source file format could be .jsonc much like tsconfig.json for typescript (to accommodate comments)
{

	"#imports": {
		"@openzeppelin/contracts/": "/node_modules/@openzeppelin/contracts/", // validate against: ^[a-z][-a-z0-9]{0,255}$
		 "@uniswap/v2-core/": "/node_modules/@uniswap/v2-core/",
		 "@uniswap/v3-periphery/": "/node_modules/@uniswap/v3-periphery/"
		 "@boringcrypto/boring-solidity": "git+https://github.com/boringcrypto/BoringSolidity#ccb743d4" // example using git repo
	},
	"#inline": {
<EMAIL_ADDRESS>["@scope1", "BaseStrategy"], // see Inline Imports section for more
	},
	"#resolutions": {
<EMAIL_ADDRESS>"src/updates/IUniswapV2Pair02.sol",
    "forge-stdlib": {  // replacing hardhat's console with a forge-stdlib version
      "stdlib/console.sol": "hardhat/console.sol"
    },
	},
  "#scopes":  {
  "tbd": "l8r"
}

Import Maps

ex. import_map rust crate used by Deno

Note to use with esm / URL based imports

 ?pin=BUILD_VERSON

Import Maps
Read the specification here: https://github.com/WICG/import-maps
{
 "dependencies": {
      "@openzeppelin/contracts": "^4.5.0",
      "@uniswap/v2-core": "^1.0.1",
      "@uniswap/v3-periphery": "^1.4.0"
    }
}

Generate the import mappings
Generating a useful import map for existing nodejs projects is obtained by running the following command
$ npx importly < package.json > forge-importmap.json
$ cat forge-importmap.json

Which gives us:
{

"imports": {
	"@openzeppelin/contracts/": "/node_modules/@openzeppelin/contracts/",
	"@uniswap/v2-core/": "/node_modules/@uniswap/v2-core/",
	"@uniswap/v3-periphery/": "/node_modules/@uniswap/v3-periphery/"
	},
"scopes": {}
}

Inline Imports for contract testing
Create a template solidity file and have forge compile however many times you define an import for that file
pragma solidity *;

// We are actually importing
// @yearn/sdk/contracts/BaseStrategyV1.sol, as defined in the forge-source file
import '<%= scope1 %>';

contract <%= scope1 %>Factory {

    function newInstance() public returns(address) {
     //  contractName = BaseStrategy, taken from @title in natspec
        return new <%= contractName %>Data();
    }
}

Summary

Additional context
No response

Actually, running
forge config --basic > foundry.toml

creates a remappings for you as well, so this part at least should be aliased possibly

Updated spec to include example of using a URL in which a colon :, is prefixed to the URL. This is  necessary when the remapping context is empty. Otherwise the https: part would be interpreted by the compiler as the context.
{@link https://docs.soliditylang.org/en/develop/path-resolution.html#using-urls-in-imports }

the newest forge update includes a command forge tree which is similar to the example I provided above
$ forge tree
src/TxManager.sol 0.6.7
├── lib/ds-auth/src/auth.sol >=0.6.7
├── lib/ds-token/lib/ds-math/src/math.sol >0.4.13
├── lib/ds-token/lib/ds-stop/lib/ds-note/src/note.sol >=0.4.23
└── lib/ds-token/lib/erc20/src/erc20.sol >0.4.20
src/TxManager.t.sol 0.6.7
├── lib/ds-test/src/test.sol >=0.4.23
├── lib/ds-token/src/base.sol >=0.4.23
│   ├── lib/ds-token/lib/erc20/src/erc20.sol >0.4.20
│   └── lib/ds-token/lib/ds-math/src/math.sol >0.4.13
└── src/TxManager.sol 0.6.7 (*)

This is partially implemented here: https://github.com/sambacha/forge-nix

Closing this as out of scope for a couple of reasons:

I don't think this will see much usage, mostly because it isn't very explicit in the source code itself, which makes reasoning about imports harder, which makes auditing harder
This augments Solidity, which means we need to add some sort of preprocessing step which is itself out of scope for now. Augmenting Solidity also breaks compatibility with every other tool people might use (slither, echidna etc.)


Scoping not working under Rails 5

Hi,
The docs for scoping say to use referenced_in and references_many but these options appear to have been deprecated in favour of more ActiveRecord-style belongs_to and has_many (I think--??).
Under Mongoid 6.0.2 and mongoid-slug 5.3.1 (master branch), the scope option doesn't work (at least when set up with belongs_to and has_many -- using referenced_in and references_many, as specified in the docs, throws an exception, so what am I doing wrong?
Saving a record with a duplicate slugged attribute (but a different scope) throws the dreaded 'E11000 duplicate key error index' error. I've tried explicitly defining the _id association as well, as suggested by the docs.
If scoping is still possible, could the README.md please be updated to indicate how to do this?
Thanks!

Do you think you can write a spec for this?

No, don't really know how to do that. Could the documentation just be corrected? Is there something beyond changing reference_in to belongs_to and references_many to has_many?

Maybe. Make a pull request and we can talk about it. I didn't dig any deeper into the issue, but I can try.

I have the same problem,
Example:
A channel has several episodes:
Channel
class Channel
  include Mongoid::Document
  include Mongoid::Slug

  field :title, type: String

  slug :title

  validates :title, presence: true

  has_many :episodes
end

Episode
class Episode
  include Mongoid::Document
  include Mongoid::Timestamps

  field :title, type: String
  field :description, type: String
  field :url, type: String
  field :published_at, type: DateTime

  slug :title, scope: :channel

  validates :title, presence: true
  validates :url, presence: true
  validates :published_at, presence: true

  belongs_to :channel

end

I have tried to scope with :channel, :channel_id, both belongs_to and referenced_ in.
Always the same error:
Mongoid::Errors::UnknownAttribute:
       message:
         Attempted to set a value for 'channel_id' which is not allowed on the model Episode.
       summary:
         Without including Mongoid::Attributes::Dynamic in your model and the attribute does not already exist in the attributes hash, attempting to call Episode#channel_id= for it is not allowed. This is also triggered by passing the attribute to any method that accepts an attributes hash, and is raised instead of g
etting a NoMethodError.
       resolution:
         You can include Mongoid::Attributes::Dynamic if you expect to be writing values for undefined fields often.

@cenotaph have you found any solution?


Home page not displaying as expected after installation

@acekyd after following the instructions for installation my homepage is not displaying as expected. i am running on a windows 7 machine

Run npm install -s node. This happens when the node version you have is different from the initial one when setting up the project I believe.


CRUD: users

Create, read, update, delete operations for User entity.

Read, Update, Delete left.

Done.


find out why ci doesn't pass for a phoenix project that uses testcontainers elixir

have created a bare bone test app here
https://github.com/jarlah/hello_testcontainers
It works locally, so it has to be some version issue or something I have missed. Or maybe the macro is checking something that shouldn't be checked

Actually why not just let it crash if the supplied app atom is not a loaded app

fixed by https://github.com/jarlah/testcontainers-elixir/commit/56295e5d9b95c878c7c64d167d9cae261ba5dd63


Installation / Login Error (Status Code 400 when fetching data)

Hello,
I have tried to install the latest version of this integration (0.8.2) on my home assistant:
Core 2024.6.3
Supervisor 2024.06.0
Operating System 12.3
Frontend 20240610.1,
after some time (I guess after a timeout occurs) the installation finishes, but on the integrations page I only see the integration (no entities). The logfiles shows the following entries:
2024-06-16 17:14:36.106 DEBUG (SyncWorker_9) [weconnect_cupra] Retrieved data from url: https://ola.prod.code.seat.cloud.vwgroup.com/vehicles/XXXX/charging/settings 2024-06-16 17:14:36.106 DEBUG (SyncWorker_9) [weconnect_cupra] {"settings": {"maxChargeCurrentAC": "maximum", "carCapturedTimestamp": "2024-06-16T12:37:36Z", "autoUnlockPlugWhenCharged": "off", "targetSoc_pct": 80, "batteryCareModeEnabled": true, "batteryCareTargetSocPercentage": 80}} 2024-06-16 17:14:36.327 DEBUG (SyncWorker_9) [weconnect_cupra] Retrieved data from url: https://ola.prod.code.seat.cloud.vwgroup.com/vehicles/XXXX/charging/status 2024-06-16 17:14:36.327 DEBUG (SyncWorker_9) [weconnect_cupra] {"status": {"battery": {"carCapturedTimestamp": "2024-06-16T12:37:38Z", "currentSOC_pct": 67, "cruisingRangeElectric_km": 366}, "charging": {"carCapturedTimestamp": "2024-06-16T12:37:38Z", "chargingState": "notReadyForCharging", "chargeType": "invalid", "chargeMode": "off", "chargingSettings": "default", "remainingChargingTimeToComplete_min": null, "chargePower_kW": 0.0, "chargeRate_kmph": 0.0}, "plug": {"carCapturedTimestamp": "2024-06-16T12:37:41Z", "plugConnectionState": "connected", "plugLockState": "locked", "externalPower": "unavailable"}}} 2024-06-16 17:14:36.499 DEBUG (SyncWorker_5) [weconnect_cupra] Retrieved data from url: https://ola.prod.code.seat.cloud.vwgroup.com/vehicles/XXXX/charging/settings 2024-06-16 17:14:36.499 DEBUG (SyncWorker_5) [weconnect_cupra] {"settings": {"maxChargeCurrentAC": "maximum", "carCapturedTimestamp": "2024-06-16T12:37:36Z", "autoUnlockPlugWhenCharged": "off", "targetSoc_pct": 80, "batteryCareModeEnabled": true, "batteryCareTargetSocPercentage": 80}} 2024-06-16 17:14:36.565 ERROR (SyncWorker_9) [weconnect_cupra] Failed to retrieve data for VIN XXXX: Could not fetch data. Status Code was: 400 2024-06-16 17:14:36.565 ERROR (SyncWorker_9) [weconnect_cupra] Could not fetch data. Status Code was: 400 2024-06-16 17:14:36.566 DEBUG (MainThread) [custom_components.cupra_we_connect] Finished fetching cupra_we_connect data in 1.048 seconds (success: True) 2024-06-16 17:14:36.671 DEBUG (SyncWorker_5) [weconnect_cupra] Retrieved data from url: https://ola.prod.code.seat.cloud.vwgroup.com/vehicles/XXXX/charging/status 2024-06-16 17:14:36.671 DEBUG (SyncWorker_5) [weconnect_cupra] {"status": {"battery": {"carCapturedTimestamp": "2024-06-16T12:37:38Z", "currentSOC_pct": 67, "cruisingRangeElectric_km": 366}, "charging": {"carCapturedTimestamp": "2024-06-16T12:37:38Z", "chargingState": "notReadyForCharging", "chargeType": "invalid", "chargeMode": "off", "chargingSettings": "default", "remainingChargingTimeToComplete_min": null, "chargePower_kW": 0.0, "chargeRate_kmph": 0.0}, "plug": {"carCapturedTimestamp": "2024-06-16T12:37:41Z", "plugConnectionState": "connected", "plugLockState": "locked", "externalPower": "unavailable"}}} 2024-06-16 17:14:36.748 DEBUG (SyncWorker_1) [weconnect_cupra] Retrieved data from url: https://ola.prod.code.seat.cloud.vwgroup.com/vehicles/XXXX/charging/settings 2024-06-16 17:14:36.748 DEBUG (SyncWorker_1) [weconnect_cupra] {"settings": {"maxChargeCurrentAC": "maximum", "carCapturedTimestamp": "2024-06-16T12:37:36Z", "autoUnlockPlugWhenCharged": "off", "targetSoc_pct": 80, "batteryCareModeEnabled": true, "batteryCareTargetSocPercentage": 80}} 2024-06-16 17:14:36.874 DEBUG (SyncWorker_3) [weconnect_cupra] Retrieved data from url: https://ola.prod.code.seat.cloud.vwgroup.com/vehicles/XXXX/charging/settings 2024-06-16 17:14:36.874 DEBUG (SyncWorker_3) [weconnect_cupra] {"settings": {"maxChargeCurrentAC": "maximum", "carCapturedTimestamp": "2024-06-16T12:37:36Z", "autoUnlockPlugWhenCharged": "off", "targetSoc_pct": 80, "batteryCareModeEnabled": true, "batteryCareTargetSocPercentage": 80}} 2024-06-16 17:14:36.977 DEBUG (SyncWorker_1) [weconnect_cupra] Retrieved data from url: https://ola.prod.code.seat.cloud.vwgroup.com/vehicles/XXXX/charging/status 2024-06-16 17:14:36.977 DEBUG (SyncWorker_1) [weconnect_cupra] {"status": {"battery": {"carCapturedTimestamp": "2024-06-16T12:37:38Z", "currentSOC_pct": 67, "cruisingRangeElectric_km": 366}, "charging": {"carCapturedTimestamp": "2024-06-16T12:37:38Z", "chargingState": "notReadyForCharging", "chargeType": "invalid", "chargeMode": "off", "chargingSettings": "default", "remainingChargingTimeToComplete_min": null, "chargePower_kW": 0.0, "chargeRate_kmph": 0.0}, "plug": {"carCapturedTimestamp": "2024-06-16T12:37:41Z", "plugConnectionState": "connected", "plugLockState": "locked", "externalPower": "unavailable"}}} 2024-06-16 17:14:37.115 DEBUG (SyncWorker_3) [weconnect_cupra] Retrieved data from url: https://ola.prod.code.seat.cloud.vwgroup.com/vehicles/XXXX/charging/status 2024-06-16 17:14:37.115 DEBUG (SyncWorker_3) [weconnect_cupra] {"status": {"battery": {"carCapturedTimestamp": "2024-06-16T12:37:38Z", "currentSOC_pct": 67, "cruisingRangeElectric_km": 366}, "charging": {"carCapturedTimestamp": "2024-06-16T12:37:38Z", "chargingState": "notReadyForCharging", "chargeType": "invalid", "chargeMode": "off", "chargingSettings": "default", "remainingChargingTimeToComplete_min": null, "chargePower_kW": 0.0, "chargeRate_kmph": 0.0}, "plug": {"carCapturedTimestamp": "2024-06-16T12:37:41Z", "plugConnectionState": "connected", "plugLockState": "locked", "externalPower": "unavailable"}}} 2024-06-16 17:14:37.155 ERROR (SyncWorker_1) [weconnect_cupra] Failed to retrieve data for VIN XXXX: Could not fetch data. Status Code was: 400 2024-06-16 17:14:37.155 ERROR (SyncWorker_1) [weconnect_cupra] Could not fetch data. Status Code was: 400 2024-06-16 17:14:37.156 DEBUG (MainThread) [custom_components.cupra_we_connect] Finished fetching cupra_we_connect data in 1.636 seconds (success: True) 2024-06-16 17:14:37.165 ERROR (SyncWorker_5) [weconnect_cupra] Failed to retrieve data for VIN XXXX: Could not fetch data. Status Code was: 400 2024-06-16 17:14:37.165 ERROR (SyncWorker_5) [weconnect_cupra] Could not fetch data. Status Code was: 400 2024-06-16 17:14:37.166 DEBUG (MainThread) [custom_components.cupra_we_connect] Finished fetching cupra_we_connect data in 1.647 seconds (success: True) 2024-06-16 17:14:37.320 ERROR (SyncWorker_3) [weconnect_cupra] Failed to retrieve data for VIN XXXX: Could not fetch data. Status Code was: 400 2024-06-16 17:14:37.320 ERROR (SyncWorker_3) [weconnect_cupra] Could not fetch data. Status Code was: 400 2024-06-16 17:14:37.321 DEBUG (MainThread) [custom_components.cupra_we_connect] Finished fetching cupra_we_connect data in 1.802 seconds (success: True)

Did you find a solution?
My cupra was working fine for months and broke about a week ago.
I used HACS to upgrade to the lastest version, and reconnected to weconnect, but I get a 400 too:
2024-08-11 22:20:16.876 ERROR (SyncWorker_4) [weconnect_cupra] Failed to retrieve data for VIN XXXX: Could not fetch data. Status Code was: 400

I am afraid not really. There have been some changes in the Cupra API
obviously (my evcc addon, which also accesses it, broke two times over the
last month and was fortunately repaired in updates), and with the current
version this integration started working for the first time for me.
But what I did: I uninstalled the integration, logged out of the cupra app
on my phone, uninstalled the app there, reinsalled and logged into the app
there and the tried the integration in HA again. I have got no idea if this
had anythind to do with it.
On Sun, Aug 11, 2024 at 10:35 PM Sean Boran @.***>
wrote:

Did you find a solution?
My cupra was working fine for months and broke about a week ago.
I used HACS to upgrade to the lastest version, and reconnected to
weconnect, but I get a 400 too:
2024-08-11 22:20:16.876 ERROR (SyncWorker_4) [weconnect_cupra] Failed to
retrieve data for VIN XXXX: Could not fetch data. Status Code was: 400
—
Reply to this email directly, view it on GitHub
https://github.com/daernsinstantfortress/cupra_we_connect/issues/59#issuecomment-2282881455,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AELUH7UUQVTOFLTRQ4ABX6LZQ7DHLAVCNFSM6AAAAABJMV2AOWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBSHA4DCNBVGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

"I uninstalled the integration, logged out of the cupra app
on my phone, uninstalled the app there, reinsalled and logged into the app
there and the tried the integration in HA again."
Hmm, tried all that. All my entities are gone now :-). Also upgraded  HA .
Logs as before:
Logger: weconnect_cupra
Source: /usr/local/lib/python3.12/site-packages/weconnect_cupra/api/cupra/api.py:73
Last logged: 10:50:40 PM
Failed to retrieve data for VIN XXXX: Could not fetch data. Status Code was: 400
Core
2024.8.1
Supervisor
2024.08.0
Operating System
13.0
Frontend
20240809.0

I have the exact same issue with my Formentor. As this package depends on the WeConnect-Cupra-python library, isn't this something that has to be reported there?

I have the exact same issue with my Formentor. As this package depends on the WeConnect-Cupra-python library, isn't this something that has to be reported there?

I was under the impression that this integration only supported the Born or at least had only been tested on the Born as I know the dev pulled this from the ID.3 Connect integration.


sfu-ws subscriber example doesn't work on mobile browsers: safari and google chrome

sfu-ws subscriber example doesn't work on mobile browsers: safari and google chrome

The issue also exists with audio-only broadcasting. It works fine on all my windows and macbook desktop browsers: Chrome/Firefox/Safari have no issue in any combination of publish/subscribe.
But mobile browsers are the opposite. I've been trying for days now and I cannot get any IOS or Android phone, iPad Pro or Samsung Android tablet to publish or subscribe Opus audio via HTTP (localhost) or HTTPS on LAN or Internet. On HTTP/LAN the mobile browsers simply do nothing when buttons clicked.  On HTTPS/Internet (AWS server), I get the following from all IOS/Android mobile browsers including Firefox, Chrome and Safari (latest versions):
http: TLS handshake error from xxx.xxx.xxx.xxx:65291: remote error: tls: unknown certificate
My SSL certificate is a purchased wildcard RapidSSL. My code is simply the Pion SFU example but adjusted for Opus audio. The frustraing thing is all desktop browsers work fine as both publisher or subscriber. Is there a Pion opus based audio broadcasting example anyone knows of? I'm now going to try datachannels on mobile browsers to see if that works.

Subscribe doesn't work on Safari MacBook Pro but publish works fine.
My Safari Version: 14.0 (156<IP_ADDRESS>.9, 15610)

This is fixed!
In pion/webrtc@v2 we expected users to handle PayloadTypes. Now that I have upgraded to /v3 SSRC/PayloadTypes are chosen at runtime and things just work.
Sorry for the bugs, and thanks for using Pion!


Added/updated API docs (WIP)

Start of work to add API documentation, starting with property-accessors and property-effects.  Some minor API refactoring was done along the way.

:tada:


add once flag to exit loop after first sync

--once terminates the program after first sync.
@ideahitme I'm currently writing a simple integration test where I need this functionality. However, the busy loop at the end of the program still seems odd to me.

IMO instead of closing the stopChan it should rather trigger RunOnce from controller. This way it relies on the internal implementation of the controller, which is not the best solution IMO

That does make a lot of sense, indeed ^^

@ideahitme updated


[1.10.2] Server crash when changing dimensiosn (Unsure if it is advancedrocketry)

Version of Advanced Rocketry
1.4.0-4
Have you verified this is an issue in the latest unstable build
Y
Version of LibVulpes
0.2.8-8
Version of Minecraft
1.10.2
Crash report or log (if applicable)
https://pastebin.com/a2ynz5g9
Description of the problem
When i change dimensions sometimes the server crashes. I dont know if it is AdvancedRocketry or any other mod. Even changing from Overworld to nether crashes sometimes.

Is this still a issue? Please comment if it is and i will reopen if if you are unable? We are closing old issues to see what is still relevant and needs to be fixed.


[Bug] Binary was created using {relax.Executable} but a loader of that name is not registered.

🐛 Bug
I run an iOS application. And have this error log:
libc++abi: terminating due to uncaught exception of type tvm::runtime::InternalError: [19:32:52] /Users/dmitrykuklin/Documents/Projects/Samsung/mlc-llm/3rdparty/tvm/src/runtime/library_module.cc:122: Binary was created using {relax.Executable} but a loader of that name is not registered. Available loaders are . Perhaps you need to recompile with this runtime enabled. Stack trace:   [bt] (0) 1   HealthWise.debug.dylib              0x0000000108a38c60 tvm::runtime::detail::LogFatal::Entry::Finalize() + 100   [bt] (1) 2   HealthWise.debug.dylib              0x0000000108a38bfc tvm::runtime::detail::LogFatal::Entry::Finalize() + 0   [bt] (2) 3   HealthWise.debug.dylib              0x0000000108a3879c std::__1::basic_ostream<char, std::__1::char_traits<char>>& std::__1::__put_character_sequence[abi:ne180100]<char, std::__1::char_traits<char>>(std::__1::basic_ostream<char, std::__1::char_traits<char>>&, char const*, unsigned long) + 0   [bt] (3) 4   HealthWise.debug.dylib              0x0000000108c296e4 tvm::runtime::LoadModuleFromBinary(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>> const&, dmlc::Stream*) + 1308   [bt] (4) 5   HealthWise.debug.dylib              0x0000000108c29e30 tvm::runtime::ProcessModuleBlob(char const*, tvm::runtime::ObjectPtr<tvm::runtime::Library>, std::__1::function<tvm::runtime::PackedFunc (int (*)(TVMValue*, int*, int, TVMValue*, int*, void*), tvm::runtime::ObjectPtr<tvm::runtime::Object> const&)>, tvm::runtime::Module*, tvm::runtime::ModuleNode**) + 864   [bt] (5) 6   HealthWise.debug.dylib              0x0000000108c2abb8 tvm::runtime::CreateModuleFromLibrary(tvm::runtime::ObjectPtr<tvm::runtime::Library>, std::__1::function<tvm::runtime::PackedFunc (int (*)(TVMValue*, int*, int, TVMValue*, int*, void*), tvm::runtime::ObjectPtr<tvm::runtime::Object> const&)>) + 624   [bt] (6) 7   HealthWise.debug.dylib              0x0000000108c584c8 tvm::runtime::SystemLibModuleRegistry::GetOrCreateModule(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>>) + 236   [bt] (7) 8   HealthWise.debug.dylib              0x0000000108c581a0 tvm::runtime::PackedFuncObj::Extractor<tvm::runtime::PackedFuncSubObj<tvm::runtime::$_0>>::Call(tvm::runtime::PackedFuncObj const*, tvm::runtime::TVMArgs, tvm::runtime::TVMRetValue*) + 228   [bt] (8) 9   HealthWise.debug.dylib              0x0000000108b19440 mlc::llm::serve::FunctionTable::Init(tvm::runtime::String, DLDevice, picojson::object_with_ordered_keys, tvm::runtime::Optional<tvm::runtime::Session>, int, int) + 3096
I know there was the similar issue
But I didn't find a way to solve it
Environment

Platform (IOS):
Operating system (ios 18):
Device (iPad Pro 2020)
How you installed MLC-LLM (conda):
How you installed TVM-Unity (pip):
Python version (e.g. 3.12.2):

Hi @LLIo6oH thanks for bringing this up. Just want to confirm first, are you following the instructions in our documentation https://llm.mlc.ai/docs/deploy/ios.html? And I am curious about at which step you encounter this issue. Thanks.

Yes, sure, I'm following the instruction

I installed MLC LLM Python Package (Pre build)
`
(base) d**n@Ds-MacBook-Air ios % conda info --envs

conda environments:

base                 * /Users/d******n/miniconda3
`

Verifying
(base) d******n@D****s-MacBook-Air ios % python -c "import mlc_llm; print(mlc_llm)" <module 'mlc_llm' from '/Users/d*******n/miniconda3/lib/python3.12/site-packages/mlc_llm/__init__.py'>
Than I follow this instruction
In my swift code I have:
func reload(modelPath: String, modelLib: String) {      Task {          await mlcEngine.reload(modelPath: modelPath, modelLib: modelLib)      }  }

And here I have a crash that I provided above

I see. The error didn't occur the last time we build the app. We will try to see if we can reproduce it.


Action workflows using quickstart openshift / frontend initial commit

configured action workflows that follows the quickstart openshift project practices
changed the exiting folder names to match other openshift projects
frontend initial commit included configuration for user login authentication

This seems to have broken the CI build.


[BUG] Devtools cannot be used outside of localhost

Describe the bug
I would like use Devtools in my Refine project. I'm running my development in Docker containers and since Devtools bind exclusively on localhost, Devtools client cannot reach the server.
Steps To Reproduce

Start a new Devtools project
Observe it's binding on localhost and no other network interface
Look for a configuration option for defining binding IP.
Fail to find anything and cry a little in despair :)

Expected behavior
Binding interface (and port) should be configurable.
Screenshot
No response
Desktop
No response
Mobile
No response
Additional Context
There has been some attempts to implement port configuration, which have been reverted in https://github.com/refinedev/refine/pull/5056, with a promise to provide a solution in the future.

Hello @rassie thanks for the issue, and sorry for the despair :)
We need to clarify some things:

Could you share your dockerfile (minimal version)?
Are you running devtools and your app separately / in different containers?
Is your app running at <IP_ADDRESS>?
Maybe you can put all of them into the same network
If your app/devtools not exposed to your machine, could you try --network=host option? https://docs.docker.com/engine/reference/run/#network-host

It's not stale.
@BatuhanW could you elaborate what's the difficult part of this issue? I wanted to take a stab at it myself and haven't seen anything inherently difficult apart from the "working with unknown code" part for me.

Can you make the origin overridable? I understand that the "normal" use case is to develop on localhost, however remote development instances are not that  unusual anymore and those won't be called localhost in most cases.

Allowed origin configuration is dictated on the backend with white-list approach. We are using ORY Kratos for authorization server and kratos doesn't even allow using localhost or <IP_ADDRESS> as origin: https://www.ory.sh/docs/guides/cors
So we have our forked version of self-hosted ory kratos, behind Ory's Oathkeeper reverse proxy.
Since we can't have a growing list of allowed origins there, only solution seems to have wildcard for allowed origins, but I need to check with the team if we can do that if that causes any security issue. cc @omeraplak

Hello @rassie, a quick update. We discussed with @omeraplak and agreed to allow all localhost and <IP_ADDRESS> origins. We can also allow certain origins like GitHub codespaces if needed.
I tried to use wildcard for port as defined in OAuth spec and Google's match patterns doc, but it didn't work.
I'll take a look at it next week.

Hi! I'm having the same error and it would definitely help to allow GitHub codespaces as an origin.

@BatuhanW I still would like to have this, thank you!

I'm sending some love for this issue to be solved 💜🙏🏽

This issue is not stale - I'm also looking at running refine dev in a container next to my APIs all in the same compose stack. This is preventing the devtools from working.
Regards,
Doug

Bumping this. My current development environment does not allow the project to be ran locally, options to expose the devtools like the --host option from vite would be nice

We've released an update to the related Devtools packages and @refinedev/cli to allow Devtools to be used outside of localhost, with a custom port or a domain. 🙏


Fix build

After https://github.com/phpstan/phpstan-src/pull/2953 landed we don't need this whitelist of constants anymore.

I think the build will be green on PHP 7.2 after https://github.com/ondrejmirtes/BetterReflection/pull/36 landed

Looks like the tests still fail on 7.2: https://github.com/phpstan/phpstan-deprecation-rules/actions/runs/8261289348/job/22598303664
You don't need to care about PHPStan failing, that will take care of itself when 1.10.61 is released.

I will have a look

I think the remaining problem is, that in jetbrains stubs constants are defined via define('X', 12) while the PhpStormStubsSourceStubber currently expects Const_ nodes. I will dig deeper

Yeah, PHPDoc above define() needs to be supported as well. I thought it already works.

fix in https://github.com/phpstan/phpstan-src/pull/2967