Adding unnecessary files

Description
I got bored and had it on necessary files...
Issues Resolved
no issues
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

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

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

Spambot


README and docs refer to next-test-endpoint

The examples should refer to next-test-api-route-handler instead of next-test-endpoint. Small PR incoming with the fixes.
I also wanted to say thanks for building this 😃  I'd been moving toward building something similar after getting tired of mocking req/res. Linked to this module from a few related Next.js discussions:
https://github.com/vercel/next.js/discussions/15166
https://github.com/vercel/next.js/discussions/17528

Nice catch, thanks. And yeah, after copying and pasting this snippet for the nth time, it was about time to package it :smile:


Update Chronos package so framework-name and ID are the same property.

Previously, the Chronos package would use a separate ID and framework-name config parameter to set the app ID and package label appropriately. This PR creates a new package without the ID parameter, to be consistent with all of our other packages.
CC @mhausenblas.

please open a new pr targeted at version-3.x if this change should still be included


How to make multiple convesation during one model inference?

Hi, Thanks for your great work! In your model inference example, we use the  prompts like
[
'''The following is a conversation between a curious human and AI assistant. The assistant gives helpful, detailed, and polite answers to the user's questions.
Human: <image>
Human: Explain why this meme is funny.
AI: ''']
but the conversion can only take once. I wander if the model can handle multiple conversation during one model inference? What the prompts need to be? I think the prompt maybe looks like :
[
'''The following is a conversation between a curious human and AI assistant. The assistant gives helpful, detailed, and polite answers to the user's questions.
Human: <image>
Human: Explain why this meme is funny.
AI:
Human: xxxx(another question)
AI: ''']
But I cant get the expected results. Can you tell me the correct prompts?
Thank you !

Yes, you are right.

But it seems that the model only outputs the last question.
prompts = [
'''The following is a conversation between a curious human and AI assistant. The assistant gives helpful, detailed, and polite answers to the user's questions.
Human:
Human: Explain why this meme is funny.
AI:
Human: What's the color of the floor?
AI:''']
output: The floor in the image is blue.

Oh, I misunderstand your question. The prompt means the conversation history. So you need to add the history into prompt, and give a instruction to expect a output.


[BUG] Breadcrumbs are not visible

When breadcumbs is enabled in params.toml, they are not visible as they are overlapped by something else. What is just visible is the very bottom part of some of the characters in the breadcrumb.
To reproduce
Steps to reproduce the behavior:

Set breadcrumbs to true in params.toml.
Once the site rebuilds, some very small parts of the breadcrumbs are visible, below the search box.

Expected behavior
Obviously I expected the breadcrumbs to show :)
Screenshot

The screenshot shows, what is displayed when breadcrumbs are enabled and the 'Bootstrap elements' page is displayed. But, it is the same on every page that is supposed to show breadcrumbs. What is shown are some very small fragments of the characters in the breadcrumb like the very low part of the letter 'g' in 'Blog'.
Environment

Hinode version: v0.15.1, but actually using template v0.8.2 (the issue is there on both)
Host OS: Linux Mint Vera (21.1)
Node version: v18.16.1
Hugo version: hugo v0.114.0-9df2ec7988e5a217a14901cc76c0b7e76b2e9f02+extended linux/amd64 BuildDate=2023-06-19T17:01:43Z VendorInfo=gohugoio
Browser: Tested on Firefox 114.0.2 and on Brave version 1.52.129 (using Chromium 114.0.5735.198)

Additional context
I can fix the problem by editing the file: node_modules/bootstrap/scss/_variables.css and change the line $breadcrumb-padding-y: to 1rem !default. This is the first time I work with nodejs, so I suppose there probably is a neater way of fixing this besides editing bootstrap files, but I am unclear on where I should fix that.
Also this fix makes the breadcrumbs visible, but also increases the space between the breadcrumbs and the title, which looks to be a bit much.
I am using the Hinode template myself, but I checked if it is also in this repository by cloning it and enabling the breadcrumbs and it indeed shows the same issue.

Thanks for reporting this, I can confirm it is a bug. The breadcrumb is hidden when using a fixed navbar. I'll submit a fix.

Thanks for the quick fix. At first I thought the result was the same as before, but I forgot that I increased the logo size to 50 in navbar.html, so for me the multiplacation factor needs to be 0.5. So, it works now.
I do want to add that it is a very good theme. I checked a couple of other themes as well, but this one ticked all the (major) boxes. Even though I did have to make a few changes to the theme, to have it also tick some of the minor boxes, it is still a great theme. Thanks for making and sharing it.

Glad it works now. The multiplication factor was indeed intended to be customizable. The documentation is available on https://gethinode.com/docs/components/breadcrumb/#customization, but you already figured that out yourself.
Thanks for your kind words! Glad you like the theme. Feel free to share your suggestions for further improvement!


Input field in a popover within a modal is not focusable

I have a popover with an input field that is within a modal.  When the popover is displayed, it is not possible to type anything into the popover input field and the first field in the modal is highlighted instead of the popover input field.
I found a stack overflow post regarding this issue which fixed the problem by removing tabindex=-1 from the modal, which I did, and that appeared to fix the problem.
However, the stack overflow solution was using a beta version of Bootstrap 5, 5.0.0-beta1 and when I use my current version, 5.1.3, the problem returns.
I am on OSX 10.15.7 Bootstrap version 5.1.3 and using Chrome  version 103.0.5060.53.
My code is in the attached zip file
modal_popover.html.zip

Thanks for reporting this issue @lcsqlpete.
Could you please try to embed your example in a CodePen or [StackBlitz)(https://stackblitz.com/)?
For security reasons, on my side, I won't download a zip file.

OK, will try to do that today.  I tried to paste the code into the issue report but for some reason, it didn't interpet all of it as html code and displayed it as a web page.

The codepen is here

Your use case seems to be related to what's described in the 2nd and 3rd paragraphs of Making popovers work for keyboard and assistive technology users isn't it?

I confirm that it is working in your CodePen when I change the reference to use the 5.0.0-beta1.
But it appears that you have removed the tabindex="-1" isn't it? If I put it back, it doesn't work anymore even with the 5.0.0-beta1. IMHO tabindex="-1" shouldn't be removed, otherwise the accessibility for the modal will be broken.
Keeping that in mind, I'll try to find what's changed between 5.0.0-beta1 and 5.1.3.

modal dialogs need to contain their focus. once a modal is open, focus is not allowed to go out of it (e.g. into the underlying page behind the semi-transparent overlay) to conform with https://www.w3.org/WAI/ARIA/apg/patterns/dialogmodal/.
newer versions of bootstrap strictly enforce this. that can cause issues with other widgets that dynamically generate things like popovers etc if these live outside of the modal dialog.
try providing a container inside the modal, and explicitly pointing the popover to use that https://getbootstrap.com/docs/5.2/components/popovers/#custom-container - and in light of https://getbootstrap.com/docs/5.2/components/popovers/#making-popovers-work-for-keyboard-and-assistive-technology-users make sure that container follow immediately after the "Add" button.
as an aside, note that your "Add" button can't be activated using the keyboard, it seems.
and the tabindex="-1" on the modal is needed to make sure that it receives focus once it's opened, and for the correct focus trapping.

Thanks for the suggestions, I will try them.  I'm closing this now since if the suggestions don;t work, I will go with a hidden div as mentioned in an earlier post.

@lcsqlpete i had a bit of spare time to see if i could get my suggestion above to work ... and indeed, forked your original codepen and just by changing the content: ... parameter for the popover in the JS to use .modal-body instead of body I think I got it to work just fine, without the need for any hacks like removing tabindex="-1" etc.
https://codepen.io/patrickhlauke/pen/GRxyjoG
as said, this is required because modals now enforce focus to stay inside the modal (as it should), and if the popover is generated/added to the end of the  element by default it's going to be outside of the modal. with this tweak, it's generated/appended to the content of .modal-body (still not ideal, you'd ideally want to provide an extra container that comes directly after the "Add" button ... but as a first stab this will do).

Slightly modified version, where I use an actual separate

 that comes straight after the "Add" control (now turned into a proper <button>) ... though this seems to mess with the placement a bit, but can likely be tweaked further.

Thank you Patrick, both of your solutions appear to work.

got a small mention of this into the docs https://github.com/twbs/bootstrap/pull/36867

Thanks, I'm sure that will be helpful for other folks who run into this.
On Wed, Aug 3, 2022 at 11:32 AM Patrick H. Lauke @.***>
wrote:

got a small mention of this into the docs #36867
https://github.com/twbs/bootstrap/pull/36867
—
Reply to this email directly, view it on GitHub
https://github.com/twbs/bootstrap/issues/36692#issuecomment-1204330995,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACA2CFF3PGFAX2APTQO4I5DVXK3KPANCNFSM523OPQ5Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

I was stuck in same problem while upgrading my old theme along with plugins.
The solution is
when you initialize the popover that is inside a popover in a modal.
you have to specify the container of the popover
 $('#my-popover').popover{ container: 'modal' // <== Just add this to make it focusable. }


Error for shoppingcart in admin

I am testing shopping cart on http://feelgood.prinla.com/shop and it seems to work very fine, BUT when I want to enter in the admin to the plugin it throws out an error. I almost re-installed the plugin but still the same. Though it works fine on the website.
Any suggestion?
Thanks
Oliver

Same here.  In my local xampp environment the error does not exists, but on my server!

I have this problem with PHP 7.1 but there isn't any problem with PHP 7.2. For fix at PHP < 7.2, you can update dependencies for gateway plugins by running this command at each gateway plugin's folder.
composer update


Make dependency charts optional, fixes #26

All subcharts are now optional, and we are following
https://docs.helm.sh/chart_best_practices/#conditions-and-tags

LGTM


chore: Upgrade controller-runtime to version 0.7.2

Release Note
chore: Upgrade controller-runtime to version 0.7.2

Let's merge this. OpenShift workflow already fails in master branch.


Mouse Wheel Scrolling by Item does not always work

Describe your issue
I have set VirtualizingStackPanel.ScrollUnit="Item"
However scrolling with mouse wheel does not scroll by Item when ItemSize is set.
Scrolling by clicking on the arrows of the scroll bar works as expected.
It's reproducable in Tab2 of your examples.

Version Info
Package Version 1.5.3
.NET Version: 4.8
OS Version: Windows 10 1903 build 18362.1139

Hi,
that's the expected behaviour. When using the mouse wheel the VirtualizingWrapPanel scrolls 3 items by default. But it never scrolls more than the viewport, therefore it may looks like it does not scroll itemwise.
You can change the number of items scrolled by mouse wheel by setting the the property MouseWheelDeltaItem to an integer value.

If that's so why does it scroll by a whole item when you click on the arrows.

That's because the implementation handling the scrollbar arrows does not use the property and scrolls fixed one item.

This difference also exists when scrolling by pixel. The mouse whell scrolls 48 pixels and arrows 16 pixels. This are the default values also used by other WPF controls.

I shall try setting MouseWheelDeltaItem to 1 to see if it fixes our issue

Right, let me know if that solves your issue.

Thanks, setting MouseWheelDeltaItem to 1 did the trick.


Deliver nightly release candidates 24Q3

Epic: Deliver nightly release 24Q3
Scope
As Garden Linux developer, I want to have an early alert system in case there is significant change in package relevant for Garden Linux.
As Garden Linux advanced user, I want to have regular access to latest changes in order to test them out and provide early feedback, before a release is published to broader group of users.


refactoring for probabilistic forecasts in reports

We need to extend/refactor a handful of things for probabilistic forecasts. Let's keep things simple and restrict reports to only probabilistic or only deterministic.
A to do list that is almost certainly incomplete:

[ ] Update report json metadata schema and api.process_report_dict parsing to allow for probabilistic forecasts.
[ ] ForecastObservation and ForecastAggregate need to be modified or copied for probabilistic forecasts. They currently have an attribute forecast: Forecast. Could be forecast: Union[Forecast, ProbabilisticForecast, ProbabilisticForecastConstantValue].
[ ] reports.main.get_data_for_report assumes deterministic forecast.
[ ] api.get_values allows for ProbabilisticForecastConstantValues but not ProbabilisticForecast. Consider modifying. In fact, there is no convenience method to get all forecast values for all ProbabilisticForecastConstantValues within a ProbabilisticForecast. Probably will want that in api.
[ ] solarforecastarbiter/metrics/calculator.py (almost complete in #224) needs to support probabilistic forecasts
[ ] simple plot with lines for each ProbabilisticForecastConstantValues within a ProbabilisticForecast

related but address in separate issues:

[ ] plot with progressively shaded areas for matched probability bands (e.g. 0/100, 10/90, 20/80, etc. percentiles)
[ ] reliability diagram
[ ] rank histogram

let me know what I'm missing.
@alorenzo175 any opinions before I get started?

I agree with most of the to do list and will have to think on it more. I do want to refactor reports at some point, but shouldn't wait for that now. For the report metadata, lets add a  type key or similar that can be probabilistic or deterministic and then metric validation etc can use that.

Do we also need a probabilistic_constant_value type? Assume that in the database a probabilistic forecast contains multiple percentiles and consider some cases:

user wants to know the Brier score for a single percentile.
user wants to know the Brier score for each percentile.
user wants to know the CRPS for the full forecast.

1 and 2 might be best specified by indicating
"forecast_type": "probabilistic_constant_value",
"object_pairs: [{"forecast": uuid_of_constant_value, "observation"...}, ...]

and 3 by
"forecast_type": "probabilistic",
"object_pairs: [{"forecast": uuid_of_full_probabilistic_forecast, "observation"...}, ...]

@awig @dplarson

Summary of the metrics i/o requirements:

Metric
Obs
Ref (fx,fx_prob)
norm
fx
fx_prob
Total # metrics

BS
yes

yes
yes
N_i

REL
yes

yes
yes
N_i

RES
yes

yes
yes
N_i

UNC
yes

yes
yes
N_i

BD
yes

yes
yes
N_i

CRPS
yes

yes
yes
N_i

BSS
yes
yes

yes
yes
N_i

SH

2

1

As you can see there are only a few special cases. Just to be sure we are on the same page:

Should Sharpness (SH) be calculated only for the furthest pair of probabilistic forecasts or should it be calculated for each symmetric pair of forecasts (10/90, 20/80, 30/70, ...)? Other ideas?
As discussed before, we agree that all probabilistic forecasts in a report have the same intervals and axis
As discussed before, we agree to drop any nan timestamps for all probabilistic forecasts
As discussed before, we agree that the reference probabilistic forecasts must have the same probabilistic intervals and axis as the non-reference probabilistic forecasts

N_i = Number of ProbabilisticForecastConstantValue in a ProbabilisticForecast? If so, I think there should be only 1 CRPS metric.
What's BD?
Users might want to know the Brier score for every ProbabilisticForecastConstantValue in a ProbabilisticForecast.
Users might want to know the Brier score for a similar ProbabilisticForecastConstantValue from many  ProbabilisticForecast.
I don't think it's likely that people are going to want to know the score for every ProbabilisticForecastConstantValue from many  ProbabilisticForecast. The multiplicity of that scenario is horrendous so it may be that the dashboard needs to make that evaluation tedious to request.

@wholmgren
Yeah, I believe you are correct for the CRPS.
BD is the Briar Decomposition which just makes up the REL, RES, UNC that makes up the Briar Score. I'll remove it from the list since it isn't really a metric itself.
Regarding the comments about the BS, I'm not using the correct term as it can be $N_i$ or $n$ depending on what axis is given for the report. Does that make sense?  (Noting that the axis can't be both in the report that is your last point.  Am I understanding correctly?
@dplarson  let me know if you agree, please

The notation across probabilistic metrics isn't the best in general, so I'll avoid that in my replies:
I think you've got the main parts correct:

CRPS is computed based on all intervals at once (so one CRPS value per ProbabilisticForecast)
Brier Score, REL, RES, and UNC is computed per interval (e.g. 5 intervals ==> 5 Brier Score values)
Sharpness is computed between intervals, but the interval pairs do not need to be symmetric

Regarding Sharpness (SH): while computing this for each symmetric interval pair makes sense, there is currently no guarantee that the intervals provided will have symmetric pairs (e.g. the intervals could be 10, 25, 43, 91, 94, 95, 99). Therefore, one approach would be to have the user select the interval range they want SH computed over and only allowing one interval pair to be selected.
For Brier Score, REL, RES, and UNC, it should go something like this:
results = []
for each ProbabilisticForecastConstantValue in ProbabilisticForecast:
    obs, fx, fx_prob = transform(ProbabilisticForecastConstantValue)
    bs = brier_score(obs, fx, fx_prob)
    rel, res, unc = brier_decomposition(obs, fx, fx_prob)
    interval_name = ...   # how we're keeping track of the intervals
    results.append({"interval_name": interval_name, "bs": bs, "rel": rel, ...})

In this setup, if there are 10 intervals, then there should be 10 Brier Score values, 10 REL values, etc. And the report should then have figures show the 10 values per metric (e.g. a bar chart with one bar per metric value/interval).
And I agree with Will's point about challenges with multiple ProbabilisticForecast. If we're having the user set the interval pair for the Sharpness metric, maybe we can do the same for Brier Score, REL, etc. for multiple ProbabilisticForecast (i.e. if comparing multiple ProbabilisticForecast, then the user has to select one interval for comparing Brier Score, REL, etc.).

Thanks, David.  I think this gets to heart of the trouble I've having determining how to implement. This discussion on cleared up a few things about what is capable, but not about what is practical and what the user would intend to be implemented.
So, a couple of points are still bothering me:

What are the combinations of the ProbabilisticForecast and ProbabilisticForecastConstantValue that are possible.
Does the axis parameter (as I thought but now I'm a bit confuse) determine how the many-one or one-many relationship how to perform the calculations based with regards to theProbabilisticForecast and ProbabilisticForecastConstantValue ? So, with regards to the last question, I thought the axis parameter was to address David's last point about multiple ProbabilisticForecast and we don't provide any many-many relationship.
IMHO I think the sharpness index should be forced to be calculated on symmetric percentages only, otherwise I think we are asking the user to provide to much input and I can't really see any value of applying it to non-symmetric %s.

Does the axis parameter (as I thought but now I'm a bit confuse) determine how the many-one or one-many relationship how to perform the calculations based with regards to theProbabilisticForecast and ProbabilisticForecastConstantValue ? So, with regards to the last question, I thought the axis parameter was to address David's last point about multiple ProbabilisticForecast and we don't provide any many-many relationship.

My understand has been that the axis parameter tells you whether the forecast values in the ProbabilisticForecastConstantValue are a) the probabilities [%] for some event (e.g. probability that the power is less than or equal to 10 MW: prob(obs(t) <= 10 MW)) or b) the physical values that have a set probability (e.g. forecast the X MW at time t that has a probability of 40%). It does not refer to the index direction of a 2D array (i.e. axis="x" does not mean compute metrics down the rows and axis="y" does not mean computer metrics down the columns). But either way, the forecast in each ProbabilisticForecastConstantValue is still a time-series of timestamp,value (where the value is either a probability or a physical quantity).
Examples:
# forecast the probability for <= 10 MW;
# values=probabilities
timestamp,value
2020-01-01 00:00:00,10  probability of 10%
2020-01-01 01:00:00,15  # probability of 15%

# forecast the physical value corresponding to
# a probability of 35%; values=physical values
timestamp,value
2020-01-01 00:00:00,732   # 732 W/m^2
2020-01-01 01:00:00,642  # 642 W/m^2

Thanks, but there is obviously some confusion still here. I did not at all mean that axis determines how to compute the index direction. I meant it was to determine the relationship between mapping ProbabilisticForecast to ProbabilisticForecastConstantValue. What I mean is that the relationship manyProbabilisticForecast to individual ProbabilisticForecastConstantValue or many ProbabilisticForecastConstantValue to inidvidual ProbabilisticForecast. But, we don't support many  ProbabilisticForecast to many ProbabilisticForecastConstantValue from a report.  I do understand how fx and fx_prob should map.

Yeah, I agree I seem to have misunderstood your questions. Maybe this will be easier to work through during the call today.

Ugh, I think I'm realizing my confusion is really the relationship of the ProbabilisticForecast to ProbabilsiticForecastConstantValue and how that relates with axis. I think I'm clear when there is only one value for the constant value, but not when there are many. I'll probably have to writeup a few pseudocode examples and have you guys review.

OKAY
After having it beat into my skull, I understand now. Each ProbabilisticForecast will still have its timeseries of forecast_values once the data is pulled and to which it can have one or more ProbabilisticForecastConstantValue indicating its specific unit/% with the axis parameter. The report can still support multiple ProbabilisticForecasts of course.
Typical function:

Iterate through the ProbabilisticForecasts
Iterate through the ProbabilsiticForecastConstantValues
Compute score (except CPRS) using forecast_values and constant_values (transforming constant_values to timeseries as necessary) with mapping assigned by axis to actual compute function.

Also discussed,

all ProbabilsiticForecastConstantValue must have the same axis
Do not support sharpness, SH at this time
CRPS requires stacked 2-d inputs of both
RES requires getting unique forecast probabilities using _unique_forecasts via binning, may need some tuning

@wholmgren @alorenzo175
Just to be sure, does a ProcessedForecastObservation with the timeseries forecast_value have to be associated with a unique constant_value in a ForecastObservation? Assuming I'm correct then. How do I know which constant_value to use? Right now I'm just assuming its in order but if there is a better way please let me know.

Yes, each contant_value will have an associated timeseries. I don't think we've written the code to get all the constant values timeseries yet, but I think something like a dict of constant_value (decimal): timeseries is reasonable.

@awig I'm going to nitpick your comment above so that we can be sure we're on the same page:

Each ProbabilisticForecast will still have its timeseries of forecast_values once the data is pulled

The "timeseries" data of a ProbabilisticForecast is two dimensional. @alorenzo175 suggested a dict. I think @dplarson might have done some prototyping in the fall with DataFrames. The metrics.probabilistic.crps function requires a 2D array.

and to which it can have one or more ProbabilisticForecastConstantValue

can --> must

indicating its specific unit/% with the axis parameter

The ProbabilisticForecast also has a variable and axis attribute. All ProbabilisticForecastConstantValue within a ProbabilisticForecast must have the same variable and axis attributes.

(transforming constant_values to timeseries as necessary)

I don't understand this. The constant_value (e.g. 10% or 10 MW) is not a time series.
Sorry for all the confusion. We should have written a doc page with examples for these objects. I can work on that later today if it would be helpful.

Perfect thanks. I'll use this for my testing and for now will modify the code so that it creates this dict and can be substituted once that has been implemented.

@wholmgren Glad to have it nitpicked. Important to avoid more confusion. Allow me to nitpick the nitpick.

The "timeseries" data of a ProbabilisticForecast is two dimensional. @alorenzo175 suggested a dict.

In my mind, ProbabilisticForecast does not have any (instead of "timeseries" ill use "series" of data). In the datamodel only ProcessedForecastObservationhas series. In particular forecast_values is listed as a pandas.Series. I don't see how it could be 2-d. What am I missing?
class ProcessedForecastObservation(BaseModel):
   ...
    forecast_values: Union[pd.Series, str, None]

I could change it to be pandas.DataFrame or a Tuple[Union[pd.Series, str, None]] but I don't enough to know that would work with the other models and api.

I think @dplarson might have done some prototyping in the fall with DataFrames. The metrics.probabilistic.crps function requires a 2D array

I've been transforming the data into a np.array(n,d) before sending to crps by gathering all ProcessedForecastObservation forecast_values where the original.forecast.id match. This shouldn't be an issue as long as I can plan for how the arrays come in.

can --> must

I'll add a check for >1, is 2 sufficient?

The ProbabilisticForecast also has a variable and axis attribute. All ProbabilisticForecastConstantValue within a ProbabilisticForecast must have the same variable and axis attributes.

Understood regarding the axis and variable. I had axis but not variable checks, so I'll add checks for both that all values for all ProbabilisticForecast have the same values for all ProbabilisticForecastConstantValues.

I don't understand this. The constant_value (e.g. 10% or 10 MW) is not a time series.

In order to perform the calculations it requires arrays of the same size as the forecast_values. E.g., if my forecast values are a series of [< 3.2MW, < 4.1MW, < 4.6MW] for a constant value of 10%, I need to create a matching series [10%, 10%, 10%] in order to pass to the probabilistic metric functions. @dplarson  can correct if I'm mistaken.

Sorry for all the confusion. We should have written a doc page with examples for these objects. I can work on that later today if it would be helpful.

I say don't worry about it as I've already got most of the functionality in place I think. If there are still confusion after a get the PR, then maybe that would be a good time. The main issue I've struggled with is how to "extract" the right data from the appropriate datamodels to pass the calculations.

Ok, so ProcessedForecastObservation will only reference ProbabilisticForecastConstantValue objects?

I've been transforming the data into a np.array(n,d) before sending to crps by gathering all ProcessedForecastObservation forecast_values where the original.forecast.id match. This shouldn't be an issue as long as I can plan for how the arrays come in.

Sounds reasonable to me. @alorenzo175?

can --> must
I'll add a check for >1, is 2 sufficient?

I was trying to say >= 1 instead of >= 0. But it turns out ProbabilisticForecast can be constructed without any constant_values so 0 is actually an option. Not sure if that's a bug.

Understood regarding the axis and variable. I had axis but not variable checks, so I'll add checks for both that all values for all ProbabilisticForecast have the same values for all ProbabilisticForecastConstantValues.

The ProbabilisticForecast constructor checks axis but not variable. Maybe better to add the check there?

Sounds reasonable to me. @alorenzo175?
Yes, transforming is reasonable.

Ok, so ProcessedForecastObservation will only reference ProbabilisticForecastConstantValue objects?

Will we be supporting comparing each ProbabilisticForecastConstantValue with a different observation? If not, it make make sense to make a separate ProcessedProbabilisticForecastObservation (with forecast_values either a dataframe or another model with the constant value and series) and avoid duplication

Will we be supporting comparing each ProbabilisticForecastConstantValue with a different observation?

I think this is required if we want to support probabilistic forecast comparisons across sites. So I'd say yes.

Ok, so ProcessedForecastObservation will only reference ProbabilisticForecastConstantValue objects?

Not directly, but through the ProbabilisticForecast.

I was trying to say >= 1 instead of >= 0. But it turns out ProbabilisticForecast can be constructed without any constant_values so 0 is actually an option. Not sure if that's a bug.

Okay so maybe I'll forego adding tests for those checks for now

Not directly, but through the ProbabilisticForecast.

Then I'm confused again. Here's a link a ProbabilisticForecast. The data associated with it is 2-D: one dimension is the constant value, the other is the time series for each constant value. So if a ProcessedForecastObservation references a ProbabilisticForecast then it is unclear what its forecast_values attribute actually refers to.

Then I'm confused again. Here's a link a ProbabilisticForecast. The data associated with it is 2-D: one dimension is the constant value, the other is the time series for each constant value. So if a ProcessedForecastObservation references a ProbabilisticForecast then it is unclear what its forecast_values attribute actually refers to.

So it makes sense to me that each ProbabilisticForecastConstantValue woould have an associated series with it. But, I do see any methods for pulling it so I believe you need to have ProcessedForecastObservation to realize the series based on the dates resolved from it. If there are helper functions to realize that, please let me know.  Right now I don't see any probabilistic forecast in the reports that would give me an example of how this works, but if there is somewhere that would be great.

The API currently only supports fetching the values for a ProbabilisticForecastConstantValue https://github.com/SolarArbiter/solarforecastarbiter-core/blob/e193f5d88e5783b2e8ace381c7ccc67996bf58e8/solarforecastarbiter/io/api.py#L533-L565

Right now I don't see any probabilistic forecast in the reports that would give me an example of how this works, but if there is somewhere that would be great.

We need your code to make this possible.

Fair enough.
After looking at api.get_values() I see that that above function is being called if the value is a ProbabilisticForecastConstantValue, so now I understand the "data" returned from main.get_data_for_report() will be populated with the Series from that function. Not sure how I'll handle all that yet. But, I will play around to see in what exact form the data exists so I can wrap my head around it, should have just done that at the beginning.


feat: support http cookies

I found the latest tag is in the main branch, so I submitted the code to the main branch. If you need to adjust the merged branch, please feel free to do it directly.

@nuomi1  thanks so much for your contribution!

@nuomi1  we have a lot of conlficts coming :( Could you can recreate the PR from the develop?

Change the base to develop

Done.


Unable to resume training

Hey,
So I started training a model, but seeing how long it was going to take I wanted to double check I could successfully resume training.
I ran:
python3 main.py train --epochs 4 --style-folder images/xmas-styles/ --save-model-dir trained_models/
until it generated the first checkpoint, then I ran
python3 main.py train --epochs 4 --style-folder images/xmas-styles/ --save-model-dir trained_models/ --resume trained_models/Epoch_0iters_8000_Sat_Dec__9_18\:10\:43_2017_1.0_5.0.model
and waiting for the first feedback report, which was
Sat Dec  9 18:17:09 2017	Epoch 1:	[2000/123287]	content:<PHONE_NUMBER>59	style: 1666218.549250	total: 1920239.380609
so it appeared to not have resumed at all.
Also slight side question... Say I train with --epochs 4 til I get final model... If I were to use the last checkpoint before final to resume, but set --epochs 5 or higher, would that work correctly and just keep going through to 5 epochs before generating another final, and have no issues etc?

--resume path/to/xxx.pth

Sorry, not sure what that means? I added path to the .model checkpoint file created on the first run, it there a .pth file somewhere I should be referring to instead? If so, where would this be? The directory I gave for --save-model-dir only contains a .model file Epoch_0iters_8000_Sat_Dec__9_18:10:43_2017_1.0_5.0.model

After resuming, it starts at Epoch 1.

its progress is at 0 iterations into the first epoch however, whereas when the checkpoint was saved it was 8000 iterations into the total of 123287. When it resumes, it doesn't resume from the number of iterations it was previously at?

That's correct. It doesn't care about epochs and iters, only load weights.

Ohh OK, so I could just keep coming back day after day, and training a bit here or there etc and it would only really ever get better etc? Will start doing that once I have a bunch of styles set in stone that I want to use.
Thanks for this BTW! Not really sure if its of use, but im writing a shell script that allows for running multiple of these. Its kinda in progress still, but heres what I'm using: https://pastebin.com/Gi7cr25C
It does eval or optim, and has 3 modes: 121, which acts same as normal. 12Many, which applies many styles to 1 original image, and Many2Many which applies many styles to many original images.
It also Attempts at 2048 content size, and keeps trying until it finds a content-size we have enough memory for.


When sending bitcoin don't use an unconfirmed transaction as input

Make it so that when sending bitcoin it uses transactions that have already confirmed as inputs if possible.
Its annoying when creating a transaction with low fee and then another transaction with higher fee
just to find out that i will have to wait for the first transaction to confirm before the second can confirm.

Do you have an example, because this is actually what it does. We are consuming the most confirmed inputs first.

It created transaction 9c42eb15295800d772534526bd8405f8507f830af09bc83bcfd7cfa61c8f8be0 when dfa06c70398b0db1da080541a8ff79e85ac9307cf4ce2b5e5bca2f11466d134a was not confirmed.

Yes but do you have the list of your UTXO at this moment. Because if you don't have any confirmed UTXO, the application will use an unconfirmed one.

It could have skipped the unconfirmed input and still have enough.

Here is an example from today
https://blockchain.info/tx/3c11559cebb72a93ea5f8bdb3998e2def2cc197119fbd237dc52da40a3518e1c
outputs to 1N5DsWTtPvWt32LWXMTa6cNN3eskNQAuLX and that was used as an input to
https://blockchain.info/tx/837943a19f027b770123a4c318638dd82486988cc926b84667e7655b1b143489


plantuml.jar is overwritten if a docker volume is mounted on /home

The plantuml.jar is overwritten if a docker volume is mounted on /home. If you then try to compile a document which uses the plantuml CTAN package plantuml.jar is not available.
A quick workaround:  docker run --rm -it -v /path/to/project:/home/work danteev/texlive bash -c 'cd work && latexmk -synctex=1 -interaction=nonstopmode -file-line-error -shell-escape -pdf -pdflatex=lualatex document.tex'
To fix this issue I'd suggest to either mount the plantuml.jar in another place or add a note to the readme.

I totally overlooked that thing. /home is now used for plantuml - and the build dir is now /workdir.
See https://github.com/dante-ev/docker-texlive/commit/f997b25822904726f33fc0025c604c170f401433#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5 for the diff.


Exporting data with , changes value

I am trying to export a date as follows and the export is changing the value.

"2-8" -> 2-Feb
"April 15, 2021" -> 15-Apr then the next cell will have the 2021 in it

Also I have full sentences in a cell, however the excel sheet is not adjusting the column width to fit the label inside. Is there a way to fix this?

Is there a way to change a specific cell's background color and text color?

What can I do to get this fixed?


feat: add prettier as a git hook

install prettier as a dev dependency
install husky as a dev dependency
add a pre-commit hook to format w/ prettier

prettier is added in PR #10


Fix TerrainLayer support for single mesh

For #3591

Change List

Change the embedded example in TerrainLayer docs to use single mesh (easier for users to click on images and see what they are). We already have a tiled example in the Examples tab.
Forward the material prop to SimpleMeshLayer
SimpleMeshLayer: added a hack to allow mesh to use world coordinates instead of meter-offsets @xintongxia

Coverage decreased (-2.5%) to 80.644% when pulling eacfa43b8c7ac89c2551fa7a59e8da10c9b830a6 on x/terrain-single into 06f37269db77d504ba5f558ce32a0afd1ff693eb on x/api-audit-2.


Fixed rendering of dates over 100 years in the future or past

If a date is not in 20xx, ie. 2150 then the date was not rendered correctly. This commit fixes this issue.

@jboesl do you have time to have a look at this PR? It's open for quite some time now, and unfortunately I can't check it today.

There are two issues with that merge request.

I don't know why I should include package-lock.json.
Both the old implementation and the one from the request are error-prone. I don't want to replace a bad implementation with a bad one.

You don't have to include it, however it is wise - from the docs:

package-lock.json is automatically generated for any operations where npm modifies either the node_modules tree, or package.json. It describes the exact tree that was generated, such that subsequent installs are able to generate identical trees, regardless of intermediate dependency updates.

How is it error prone? I only see one use case where it would fail, and that is is the index.txt file itself becomes corrupted - in which case you have bigger problems.
OpenSSL provides time stamps in one of 2 formats; YYMMDDHHMMSSZ or YYYYMMDDHHMMSSZ; hence testing the length and slicing is as good as a method as any.

If you are going to label something as 'bad' then explain why it is 'bad' so that it can be discussed or rectified.

Okay, I didn't know about package-lock.json. Including it sounds reasonable but I'd still prefer including it in a separate commit.

Just checking for 13 digits and assuming it got 15 otherwise indeed does look pretty error prone to me. Afaics OpenSSL uses rfc3161 for timestamps and thus might have more digits. It's well possible your code is  more compatible than the code before but still it smells.
Isn't there some battle tested npm that handles timestamps? That would be the way to go imho.

Perfectly understandable.
I did some digging - RFC3161 actually refers to the protocol and references the timestamp as an ASN.1 GeneralizedTime. A quick search of this yielded a library for handling ASN.1 - see the implementation here: https://github.com/PeculiarVentures/ASN1.js/blob/master/src/asn1.js#L3984
Which in my opinion, based on your previous argument still smells.
From what I have dug up I have not found an implementation that uses a string that represents a time in any greater accuracy than seconds.
The format is actually defined as:
Local time only. YYYYMMDDHH[MM[SS[.fff]]]'', where the optional fff is accurate to three decimal places. Universal time (UTC time) only. YYYYMMDDHH[MM[SS[.fff]]]Z''.
Difference between local and UTC times. ``YYYYMMDDHH[MM[SS[.fff]]]+-HHMM''.
Which imho is actually pretty sucky - especially considering openssl explicitly uses the YYMMDDHHMMSSZ format - how does one accurately determine that the extra 2 digits are in fact part of the year rather than an extension to the seconds?
Open to suggestions.

Okay. I haven't checked external libs so far but I believe you checked it.
So just put your code in a separate function with proper naming and extend it to check the length in front of the 'Z' or the dot.
I see you really want to improve nodepki and when you do these adjustments I'll be happy to merge your request.


added .gitignore

nice
On Fri, Dec 10, 2021 at 12:02 AM SharkAce @.***> wrote:

Merged #3 https://github.com/SharkAce/auto-theme/pull/3 into main.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SharkAce/auto-theme/pull/3#event-5745421664, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AWZ4PK4DXNYXXDR6I6M65XLUQGCW5ANCNFSM5JYHWGDA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Partial revert of eaaa55ee8d290a39c2204e42f2f3ad73a6bff925 to fix YoutubeExplode version.

Note: I think version 1.9.24 setup/release is broken because it comes with an older version (6.1.14) of YoutubeExplode instead of the last one (6.3.13).

Oh dont worry, I build youtube explode manually because I remove some of the dependencies so the program will work everywhere (the author of youtube explode blocks usage in Russia and some other regions)
It was a mistake on my part to name the version 6.1.14 instead of 6.3.13, but yeah it should be 6.3.13 on the project file

That would explain why the size of YoutubeExplode.dll from the installer wouldn't mach any DLL from the NuGet.


Fix shader crash when users miss the return statement

This will fix the shader crash by forcing a user to define return statement inside non-void functions

So now the function will recursively search for at least one return statement - if its not found then error emitted. Non-void function must provides at least one return statement to prevent crash.


Use convert_calendar in ensemble alignment

Pull Request Checklist:

[ ] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #xyz

[ ] Tests for the changes have been added (for bug fixes / features)
[x] Documentation has been added / updated (for bug fixes / features)
[ ] HISTORY.rst has been updated (with summary of main changes)
[ ] bumpversion (minor / major / patch) has been called on this branch
[ ] Tags have been pushed (git push --tags)

What kind of change does this PR introduce?
Instead of implicitly doing the calendar conversion, ensembles.create_ensemble now has an explicit option to control the output calendar. Control is minimal: only the calendar name can be modified and the default options of  convert_calendar are used.

In fact, I broke create_ensemble in the last PR that modified the alignment process, but didn't realize until now. '360_day' calendar were broken because of the 30th of Feb. This fix has the same default behaviour than the old code and adds some control.

Does this PR introduce a breaking change?
No, the default for the calendar argument is default.

Other information:

@aulemahal If you are working on ensembles.py I recently had the idea of implementing a add_perc_as_dim  flag.   Default would be False and would run as currently implemented creating a new variable for each percentile.  If True the output would be a single dataarray with added 'percentile' dim ( coords == input percntiles).  It should be somewhat 'cleaner' if users want a larger number of percentiles.   I think we could calculate all percentiles at once even instead of looping over values as we do currently?
We could make this a separate issue I suppose
@huard opinion on a 'add_perc_as_dim' option?

I think it's a good idea! I would also suggest to have this in a separate PR/issue. Shouldn't be difficult to implement. Also, while doing this, we should make ensemble_percentiles work on DataArrays too, the current code is unneedingly dataset-only.

I think it's a good idea! I would also suggest to have this in a separate PR/issue. Shouldn't be difficult to implement. Also, while doing this, we should make ensemble_percentiles work on DataArrays too, the current code is unneedingly dataset-only.

Sounds good to me .. can you open an new issue?  I will try to take a look at this PR in the meantime.  Seems fairly straightforward

I would suggest reviewing the API for this. It seems to me the natural xarray approach would be to always have this new percentile dimension. Then, when writing to file, you could have a split method to create individual files for each percentile value.

Pull Request Test Coverage Report for Build 1933

6 of 6 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 77.348%

Totals

Change from base Build 1926:
0.05%

Covered Lines:
2677

Relevant Lines:
3461

💛  - Coveralls


Self-supervised pretrained model link.

@bryanyzhu Hi, thank you for your excellent work! Is there a link to the Self-supervised pretrained model?
Looking forward to your reply!

Hi, you mean the self-supervised pretrained model on Kinetics400 training set? It is in the linear evaluation section of README. I attach its link here as well, https://haofeik-data.s3.amazonaws.com/VCLR/pretrained/vclr_k400.pth


avm-res-compute-availabilitysets

Check for previous/existing GitHub issues/module proposals

[X] I have checked for previous/existing GitHub issues/module proposals.

Check this module doesn't already exist in the module indexes

[X] I have checked for that this module doesn't already exist in the module indexes.

Bicep or Terraform?
Terraform
Module Classification?
Resource Module
Module Name
avm-res-compute-availabilitysets
Module Details
It helps improve the resiliency of our applications by minimizing the impact of potential failures like hardware or software issues within Azure data centers.
Do you want to be the owner of this module?
No
Module Owner's GitHub Username (handle)
No response
(Optional) Secondary Module Owner's GitHub Username (handle)
No response

@prjelesi - this shouldn't be a module. We are recommending Availability Zones and if zones are not available we are suggesting they use VMMS with flexible orchestration mode https://learn.microsoft.com/en-us/azure/virtual-machines/availability-set-overview

Closing this as not needed


List of types inheriting from Cool out of date in Cool documentation ?

Problem or new feature
the list of types inheriting from Cool, a listed at in https://github.com/Raku/doc/blob/90efdf1f9e9a2037d244f2335c4ea2405c44a5b9/doc/Type/Cool.pod6#L25 seems slightly out of date.
Suggestions
Drop this list in favor of the type graph generated from type-graph.txt and let the reader follow the inheritance tree via the type graphs in types' documentation?

Great idea, finally got to it. Thanks!


Update java section

Update README.md file adding a new kotlin library: https://github.com/ThomasConstantinBity/ICP-Kotlin-Kit

Dear @0xTommy,
In order to potentially merge your code in this open-source repository and therefore proceed with your contribution, we need to have your approval on DFINITY's CLA.
If you decide to agree with it, please visit this issue and read the instructions there. Once you have signed it, re-trigger the workflow on this PR to see if your code can be merged.
— The DFINITY Foundation

@0xTommy Please agree to the CLA as mentioned by the bot above.
Would you mind also adding https://github.com/kosta-bity/IcpKit ?


package.json command icon hover resolves icon reference

It's not clear from the hover that you're meant to use $(id) because the icon gets resolved:

Making theme icon reference a link pointing to API docs would be helpful too.

@aeschli I think this is a regression?

Yes, I probably cased that some days ago.

I'm not able to reproduce in insiders:
(Version: 1.54.0-insider Commit: 93f705ab40b37aade9d3b5165ed09114a8c87ac9 Date: 2021-02-11T05:44:44.032Z)

I can:

Version: 1.54.0-insider (user setup)
Commit: 6ac9a3ecb3698e82bf901f11bbb5940f6bc3c197
Date: 2021-02-12T05:13:16.111Z
Electron: 11.2.3
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042

Hmm, strange, I tested also on Window on the same commit (6ac9a3ecb3698e82bf901f11bbb5940f6bc3c197)
Can you turn on "json.trace.server": "verbose", and check out  the JSON Language Server output view. It should be:

Then setting a breakpoint in markdownRenderer.ts - renderMarkdown: https://github.com/microsoft/vscode/blob/6a698d7d435466f13dff1b9852e95954a91c87f4/src/vs/base/browser/markdownRenderer.ts#L48

[Trace - 6:54:50 am] Received response 'textDocument/hover - (59)' in 9ms.
Result: {
    "contents": [
        "\\(Optional\\) Icon which is used to represent the command in the UI\\. Either a file path, an object with file paths for dark and light themes, or a theme icon references, like \\`$\\(zap\\)\\`"
    ],
    "range": {
        "start": {
            "line": 51,
            "character": 8
        },
        "end": {
            "line": 51,
            "character": 14
        }
    }
}

I disabled the en-gb language pack and it still seems to happen.
I tried to use Code - OSS and set a breakpoint but it works fine there 🤷. Source maps don't want to work for me in insiders and I couldn't find the minified part to debug into.

I can reproduce with the English language package.
It's missing the \ before the $(zap):
https://github.com/microsoft/vscode-loc/blob/8414365cca49966a7e50894ecc6106db6a6def64/i18n/vscode-language-pack-en-GB/translations/main.i18n.json#L2165

I can see the \ on the suggest widget but not on hover (today's Insiders on macOS):

Oh I guess I didn't reload properly to apply the uninstalled language pack, thanks 👍

@aeschli shouldn't \\$(zap) in the string be {0} so we don't hit translation problems like this?

I added a message to the NLS key to let translators know.

Note: All translations have been reviewed and I have ensured that they are all set with...
\$(zap)
any changes will be reflected in the build in a few days

relevant culture fixes would be in the build by now... closing


Non-text content: textual alternatives (WCAG 2.2 Level A - Accessibility)

Background
The eQ service identified this issue within the ONS Design System following their DAC audit report (18/01/2024).
Issue
Non-text content did not have sufficient textual alternatives for screen reader users.

The tick icons adjacent to the ‘Big Shop’ and ‘Small Shop’ have no textual alternative, meaning that the information conveyed through the tick icons will not be made available to screen reader users. As a result, these users will be unaware of the completed status provided once details have been filled in for each company.
Screen reader user comments
On returning to the ‘Companies’ page I was not sure of which company details had been completed. This confused me as it was not clear that other details may be needed to be entered as the page did not change.
Adding some text next to the company whose details have been completed will make it clear what still needs to be completed.
Suggested solution
Ensure that non-text content has sufficient textual alternatives where it represents nondecorative content. In this instance, the text ‘Completed’ could be placed adjacent to the company name or within parentheses, similarly to other content within the service.
Tasks

Investigate whether the change can be made and any impacts
Decide the best solution
Implement the solution
Update eQ with progress

Info

eQ DAC reports/docs
Issue ID: DAC_Non_Text_Content_01
DAC report pages: 29 to 31
WCAG Reference: 1.1.1 Non-text Content (Level A):

Understanding Non-text Content
How to Meet Non-text Content

@rmccar checked with eQ to find the pattern being used in test_list_collector_content_page test schema. The pattern is not currently being used in any live surveys. It's currently not an issue that needs fixing and it may also be an issue by way of implementation on eQ.

Just to clarify, I'm still investigating. It isn't currently being used in any live surveys, but this pattern will be used in prepop surveys very soon so it is something that will need fixing. Depending on the solution it might be that we can fix it on the eQ, DS side or both but I'm still working out the best way of dealing with it. So I've suggested we put this one on hold for the time being while I finish looking into. Need to have a chat with Joe and Dina about it so we can work out a solution from a design point of view.

@rmccar Do you have any update on this? Would it be better to close this ticket and then open a ticket based on the solution found from your investigation?

@rmccar Do you have any update on this? Would it be better to close this ticket and then open a ticket based on the solution found from your investigation? @daniellecorke

No update at the moment because Ive been working on other things and have been on leave. We could add another card to make the change for the solution I come up with if you want but that would still be looking to address this DAC issue so I think it makes sense to keep this ticket open because this is still likely to be work on the DS side and not eQ and the DAC issue hasn't been addressed yet.

Done some more research on this and have spoken to DAC about some ideas on how to fix this. Out of this the best solution seems to be that we add the ability for users to set some context on the icon that can be used in summaries. This would be through adding a <span> with the ons-u-vh class that contains the text that the user can set using a new param that would give the context for the tick icon. We have used this solution in other places such as the row actions in summaries so looking at that will give you an idea on the solution.


Ensure we're mangling properties consistently across files

We need to add the nameCache minify option to our terser config to ensure properties and variables are renamed consistently across files.
I don't think we're currently affected by any issues resulting from not doing this, but we almost certainly would be if we ever start extending classes from other packages.

Is this still worth tracking, now that we're shifting focus away from our pre-built bundles and towards developers using their own custom bundles?

I think so. As long as we're publishing separate files to the CDN, it's potentially a problem. But IIRC it'll only be a problem if we're extending a class from one package in another package. In that case we need to ensure that the property names are always mangled to the same final symbol.
Since we don't do that (AFAIK), it's not an issue at the moment.

Now that we're exposing a Strategy base class for other packages to potentially extend, we need to ensure this is added before v6 is released.


Update Gradle logo

Context

Update Readme logo to latest Gradle logo.
Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[x] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[x] Provide unit tests (under <subproject>/src/test) to verify logic
[x] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[x] Ensure that tests pass locally: ./gradlew <changed-subproject>:check

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

To avoid turnaround because of DCO, I just pushed the new logo for you in master: https://github.com/gradle/gradle/commit/40fd6f2af206c2fe58ba9557bd504bc89674552c


Define top three features for the MVP

AC: Have a list of the three top features ready for wireframe work

Example:

Explore page
Full text search
Services detail page (who, what, where)


Update Links

See https://github.com/Senzing/template-repository/issues/40
docker-apt/README.md:    1. [docker](https://github.com/Senzing/knowledge-base/blob/main/HOWTO/install-docker.md)
docker-apt/README.md:    1. [git](https://github.com/Senzing/knowledge-base/blob/main/HOWTO/install-git.md)
docker-apt/README.md:    1. [make](https://github.com/Senzing/knowledge-base/blob/main/HOWTO/install-make.md)

Pull requests:

https://github.com/Senzing/docker-apt/pull/61


Possible to implement with bounded while loop?

Is it possible to add a function that integrates with a

bounded while loop to allow for early stopping
and is also vmap-able to allow for multiple progress bars?

Currently, I use a scan with an early stopping condition. If the condition is met, I skip the computationally expensive updates and simply iterate the remaining steps of the progress bar until completion. I've gotten my code to work with vmap and display multiple progress bars (yay!), but the issue is that vmap-of-cond executes both branches, so I'm forced to execute the expensive branch regardless of if I've met my convergence threshold (boo!). Since it seems like we won't be seeing a scan with early stopping any time soon (see this PR), I'll have to settle with a bounded while loop. Do you have any thoughts or insights into whether or not this is possible? Thanks.

Definitely +1 for vmap, I recently ran into a use-case for this and was going to look into a solution, would be interested to see how you got this working.

Are you suggesting we add a wrapper around an existing bounded while loop implementation?

Yes, a wrapper that works with a bounded while loop is what I am looking to implement!
Here's a simplified example that demonstrates my use case. Admittedly I am new to JAX and not too confident in this implementation. When I use this scheme to fit simple HMMs, it appears to work (log likelihoods are monotonically increasing for each iter). However, when I shift to more expensive/complex models, the likelihoods jump around erratically and do not monotonically increase when trying to vmap over more than a single model fit. Not sure if this is a problem with my code or the package I am using (dynamax).
Interestingly, in my linked example the early stopping scan seems to work, yet doesn't in my actual implementation. Open to any thoughts or critiques.

I see what you are doing with the vmap, would be interested in adding similar functionality. May need to think about how to implement it as part of the API (i.e. would it still be a wrapper around the scan inner function, or would it be better mapping the vmap somehow?).
For the bounded while loop, it's like the for loop, but it should initialise the progress bar with the upper bound and then stop if the while looper terminates?
Looking at the code this seems reasonable, I guess you'd initialise and update the progress in the bar but have to check for termination, I wonder if there would be some issue with step counting? The JAX while loop only carries a single value trough, it doesn't pass in an step number or mapped value (as we use in the scan).

@andrewlesak I just released an update with functionality to hand multiple progress bars inside a vmapped function. Will try look at the bounded while loop at some point.

Thanks for the updates! I was able to adapt the new code to get a wrapper for the bounded while loop working so I can put together a draft PR soon for you to check out


graphiqlExpress setup with endpoint that requires authentication and CORS?

Is there a way to configure graphql-express to use the graphiqlExpress configured with custom headers for authentication and handling CORS?

@Magneticmagnum I think there's a way to essentially put a string in the header. It's a bit hacky, but it was used for the meteor integration. See passHeader here: https://github.com/apollographql/graphql-server/blob/79458e2ebb687a8dc76ee986ca1e5dd6dd3397eb/packages/graphql-server-module-graphiql/src/renderGraphiQL.ts#L17
If that doesn't do the trick for you, a clearer description of your use-case, and how important it is to you would be great!

Ah, yes, thanks!
It definitely does feel hacky, as you'd have to do something like passHeader: '"token": "xxxxxx"', \n '"token2": "xxxxxxx"'
But works for me for now!  Thanks @helfer

Yeah most people eventually end up setting up a custom GraphiQL, or using the one in the Apollo dev tools.


connect() method can use NEOVIM_LISTEN_ADDRESS

Hi @tarruda
Thanks for this library, i just improve the possibility to use the NEOVIM_LISTEN_ADDRESS in connect() method if address is empty, with this improvement we can directly execute python script in neovim like (neovim set NEOVIM_LISTEN_ADDRESS if not exists)
:! python myscript.py

Regards,

Not really sure this is a feature we want, and the PR hasn't been updated. So, closing.


Add endpoint id to trace query.

Add a new condition for trace query. FYI @peng-yongsheng

Coverage decreased (-0.006%) to 14.279% when pulling fb4ff004a537fbbf81f76188eb5ac3e11c3e42c5 on support-endpointid-query-trace into 750bca2241177cb3450d3677be8ece7e71808507 on master.


Scrolling shouldn't drop a duplicated block

Expected Behavior
Sometimes once you've duplicated a block, you need to scroll a pretty far distance before you get to where you want to place it. (This is especially frequent if you use Scratch on a small screen or are zoomed in to make the blocks bigger.) Duplicating a block attaches the block to the cursor, so that it follows your mouse even though you aren't dragging. This block should stay attached to your cursor until you click the place where you want to drop it.
Actual Behavior
If you scroll (via scrollwheel) while a block is attached to your cursor, that block will get "dropped", being placed onto the workspace wherever your mouse was when you started dragging. This is impractical because it means you have to click and drag your block back up to get it where you want it.
Steps to Reproduce
Duplicate a block so that it is attached to your cursor, then scroll the workspace via the scrollwheel (or a trackpad, etc).
Operating System and Browser
Firefox 65.0b7, Linux (Debian).

I noticed this is even a problem if you actually are dragging a block with your mouse pressed down -- scrolling will cancel that and drop the block wherever your mouse was. No good if you start dragging a block before realizing that the place where you want to put it isn't on screen yet!

This is a surprisingly gigantic task, from what I've tried.
Here is my WIP code on this feature: https://github.com/LLK/scratch-blocks/commit/f791e2f451255c85b963f497bc5c535b619369b3 - unfortunately, it only mostly works. Insertion markers are a frustratingly prominent issue-causer. The basic code all does what it's supposed to, so I'm stumped as to why insertion markers are still causing problems.
For some context, virtually all issues arise when the workspace attempts to resize itself to fit all the content - while your block is being dragged. This causes a series of issues my code tries to fix, but it doesn't catch them all.


Manage Realms Issues/Suggestions

[ ] Harvesting food - I've harvested food and resources successfully on my realm 122. But for the past few hours it has shown food harvests are available, I sign the TX and it completes successfully, but no food is added and the UI still shows harvests available. Harvest TX: https://testnet.starkscan.co/tx/0x231a8245a211d4f8f7ed10679b49c91df42b1ca7a6f58f745ba0e56c9c672ea (the screenshot above is several minutes after the TX above was accepted to L2)

[x] Construct farm - When clearing the default max build of farms/fishing villages, once you backspace/clear it, it takes you back to the landing page

[x] Storehouse - After adding food to my storehouse, the building sqm went to 15 significant digits. Perhaps should limit to 2 decimals

im going to take this

all resolved in #308


Update Noxhil_V-Sek.pl

I'm altering the apostrophe for a tilda here.  I think the dark elf naming convention uses the tilda ` instead of the apostrophe, so this would be consistent with the naming throughout.  I suspect the write up just typed the apostrophe for ease.

Tested tilda and doesn't work, so that explains it.


gtkmm depends on missing stub libraries

Describe the bug
A clear and concise description of what the bug is.
Environment

OS: Windows 10 21H2
Compiler: Microsoft(R) Optimizing Compiler version 19.32.31328 for x64
In KVM under virt-manager

To Reproduce
Steps to reproduce the behavior:

./vcpkg install gtkmm:x64-windows
git clone this project (jgcodes2020/tas-input-v2)
Download Dependencies. Once extracted to a folder, .\dependencies -modules <gtkmm dll path>.
The output shows a bunch of missing DLLs.

[NOT_FOUND] ext-ms-win-security-chambers-l1-1-0.dll :
[NOT_FOUND] ext-ms-win-oobe-query-l1-1-0.dll :
[NOT_FOUND] ext-ms-win-core-winrt-remote-l1-1-0.dll :
[NOT_FOUND] ext-ms-win-com-apartmentrestriction-l1-1-0.dll :
[NOT_FOUND] ext-ms-win-com-suspendresiliency-l1-1-0.dll :
[NOT_FOUND] ext-ms-win32-subsystem-query-l1-1-0.dll :
[NOT_FOUND] ext-ms-win-shell-knownfolderext-l1-1-0.dll :
[NOT_FOUND] ext-ms-onecore-defaultdiscovery-l1-1-0.dll :
[NOT_FOUND] ext-ms-win-containers-policymanagercli-l1-1-0.dll :
[NOT_FOUND] HvsiFileTrust.dll :
[NOT_FOUND] ext-ms-win-appmodel-deployment-l1-1-0.dll :
[NOT_FOUND] ext-ms-win-rtcore-minuser-input-l1-1-1.dll :
[NOT_FOUND] ext-ms-win-rtcore-minuser-private-ext-l1-1-1.dll :
[NOT_FOUND] ext-ms-mf-pal-l2-1-0.dll :
[NOT_FOUND] api-ms-win-coreui-secruntime-l1-1-0.dll :
[NOT_FOUND] ext-ms-onecore-orientation-l1-1-0.dll :
[NOT_FOUND] ext-ms-win-rtcore-minuser-private-ext-l1-1-0.dll :
[NOT_FOUND] ext-ms-win-rtcore-minuser-internal-l1-1-0.dll :
[NOT_FOUND] ext-ms-onecore-shellchromeapi-l1-1-0.dll :
[NOT_FOUND] ext-ms-win-appmodel-viewscalefactor-l1-1-0.dll :
[NOT_FOUND] ext-ms-win-audiocore-pal-l1-2-0.dll :
[NOT_FOUND] ext-ms-win-audiocore-spatial-l1-1-0.dll :
[NOT_FOUND] ext-ms-win-appmodel-usercontext-l1-1-0.dll :
[NOT_FOUND] ext-ms-win-ui-viewmanagement-l1-1-0.dll :
[NOT_FOUND] UpdateAPI.dll :
[NOT_FOUND] ext-ms-win-networking-wlanstorage-l1-1-0.dll :
[NOT_FOUND] ext-ms-win-core-app-package-registration-l1-1-0.dll :
[NOT_FOUND] ext-ms-windowscore-deviceinfo-l1-1-0.dll :
[NOT_FOUND] ext-ms-win-wer-xbox-l1-1-0.dll :

Expected behavior
All dependencies should be found.

I verify this issue locally by tool depends.exe. I will double confirm this is a port bug on VCPKG or upstream.

I found the similar issue: https://github.com/lucasg/Dependencies/issues/119, according this comment, this issue should be a upstream issue, could you submit an issue to report this on upstream?

Ok I found the root cause of the bug : apiset lookups was done case sensitive-wise, where Windows usually accepts both cases.

cc @Neumann-A

I would say this is not related to the port and is such not a vcpkg issue. The "missing" dlls are all windows dlls...

Should have closed this earlier.
The correct CMake command was:
cmake -DCMAKE_TOOLCHAIN_FILE=C:\path\to\vcpkg\scripts\buildsystems\vcpkg.cmake

And I had done:
cmake -G "Visual Studio 17 2022" -A x64 -DCMAKE_TOOLCHAIN_FILE=C:\path\to\vcpkg\scripts\buildsystems\vcpkg.cmake


chore: Add license file

The project has always stated in the readme that it's AGPL 3. This just adds the license file to the repo.
Closes #230

Marking as draft while Axim has a quick internal discussion to confirm this is the right license.

@bmtcril any updates on this?

@Ian2012 waiting on legal, the rest of Axim is on an outing today, but I should have an answer tomorrow.


HADOOP-18633. fix test AbstractContractDistCpTest#testDistCpUpdateCheckFileSkip

Description of PR
Fixing test flakiness caused by race condition.
How was this patch tested?
Tested (ap-south-1)S3a,(us-west-2) ABFS, HDFS and LocalFS implementation of the test.
For code changes:

[ ] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 43s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
45m 25s

trunk passed

+1 :green_heart:
compile
0m 31s

trunk passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
compile
0m 35s

trunk passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
checkstyle
0m 30s

trunk passed

+1 :green_heart:
mvnsite
0m 34s

trunk passed

+1 :green_heart:
javadoc
0m 33s

trunk passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
javadoc
0m 27s

trunk passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
spotbugs
1m  1s

trunk passed

+1 :green_heart:
shadedclient
23m  2s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 34s

the patch passed

+1 :green_heart:
compile
0m 28s

the patch passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
javac
0m 28s

the patch passed

+1 :green_heart:
compile
0m 26s

the patch passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
javac
0m 26s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 14s

the patch passed

+1 :green_heart:
mvnsite
0m 22s

the patch passed

+1 :green_heart:
javadoc
0m 18s

the patch passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
javadoc
0m 31s

the patch passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
spotbugs
0m 53s

the patch passed

+1 :green_heart:
shadedclient
24m 38s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
14m 39s

hadoop-distcp in the patch passed.

+1 :green_heart:
asflicense
0m 35s

The patch does not generate ASF License warnings.

119m 11s

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5401/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5401

Optional Tests
dupname asflicense mvnsite codespell detsecrets compile javac javadoc mvninstall unit shadedclient spotbugs checkstyle

uname
Linux 36e92caefe45 4.15.0-200-generic #211-Ubuntu SMP Thu Nov 24 18:16:04 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / b6d94e769062252d97008e7813bc5506edc33a7f

Default Java
Private Build-1.8.0_352-8u352-ga-1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_352-8u352-ga-1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5401/1/testReport/

Max. process+thread count
721 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-distcp U: hadoop-tools/hadoop-distcp

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5401/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.


Issue

https://github.com/microsoft/vscode-remote-repositories-github/issues/464

Closing the issue as not actionable. Please add a description with steps on how to reproduce the problem and I will re-open the issue.


Small mistake in example configuration file on docs for "creating project templates"

Describe the issue or suggestion
Hi,
doc reference: https://learn.microsoft.com/en-us/dotnet/core/tutorials/cli-templates-create-project-template
doc section: create-the-template-config
While executing a tutorial on creating project templates we ran into the issue that the example was not working. It seems that in the template.json definition a small mistake has occurred. We noticed that the sourceName has been set to "ExampleTemplate.AsyncProject". This causes that the command dotnet new consoleasync -n <somename> will not cause to rename the project.
To make this work the sourceName tag should be updated on the page from value
"sourceName":"ExampleTemplate.AsyncProject",

To value
"sourceName":"consoleasync",

@adegeo


Upgrade to Dreem syntax highlighter for sublime. Coffeescript support is...

... better, but not perfect. Improved autocompletion for handler/method/setter.

:+1:


[SELFTEST - EXPENSE] Display receipt automatically after donereceipt command

maybe same with paid command, or additem command


Update pinned formatter version

The new relation is giving linting error because the pinned version is quite old.

This is probably taken care of via https://github.com/prisma/vscode/tree/master/scripts on publish. Agree?

Ho maybe, I was not able to able a formatter with the latest alpha syntax so thats why I opened this issue.

Fixed with switching to beta


Documentation how to install with wifi

+1

https://github.com/alexreinert/piVCCU/blob/master/docs/setup/wlan.md

Please: https://github.com/alexreinert/piVCCU/pull/34


README: update default value for disable_filetype option

Looking at the source code, it seems that the default value for the disable_filetype option is { "TelescopePrompt", "spectre_panel" }.
I updated the README.md with this correct value.

thank


DCAP quote verification -- discussion and todo list

Here is a proposal for an attestation flow for PDO, which closely follows what FPC does.
The attestation library is available under the attestation-dev branch.
The library is divided in 3 main components:

attestation generation, inside the enclave
attestation conversion to evidence, outside of the enclave
evidence verification, inside or outside of the enclave

The figure depicts the flows for attestation generation, conversion and verification -- for additional technical details look at the test/attested_evidence_test.sh script in the library.


Revert "dts: adsp: ace: Changed used watchdog device"

This reverts commit c558fd532317408855400b23d510567287367a6d.
This change results in boot failures on ace15 platform.
Link: https://github.com/thesofproject/sof/issues/7433

This blocks multiple PRs like https://github.com/thesofproject/sof/pull/6480 on SOF side as we cannot use latest Zephyr main currently.


Conduit Wires

Based on PJB design idea. A wire occupies a specific "layer," and can connect to other adjacent wires on the same layer. You cannot place multiple wires onto the same layer. Easily extendable to pipes when we add them.
To get this to work, the serialization of NodeContainerComponent was changed. Instead of getting names and requesting the associated implementation from a factory, the Nodes implement IExposeData and the type is specified directly in Yaml (- !type:ClassName).
There's no graphical indication of what layer a wire is on - we will need to make a visualizer or something for that.

I still don't see the point of wire layers.

Conduit changes not complete enough to warrant merging, but I want to keep the serialization changes which I'll put in a different pr.


[Bug report] VuePress failed to resolve images without ./

Description
VuePress can't resolve images without ./

package.json
{
  "devDependencies": {
    "@vuepress/plugin-medium-zoom": "2.0.0-beta.48",
    "@vuepress/plugin-nprogress": "2.0.0-beta.48",
    "@vuepress/plugin-pwa": "2.0.0-beta.48",
    "vuepress": "2.0.0-beta.48"
  },
  "scripts": {
    "docs:dev": "vuepress dev docs",
    "docs:build": "vuepress build docs"
  }
}

.vuepress/config.js
module.exports = {
  "title": "mirai",
  "description": "Mirai Project",
  "markdown": {
    "lineNumbers": true
  },
  "dest": "E:\\IDEAProjects\\project-mirai_docs\\vuepress-dist",
}

Reproduction
null
Used Package Manager
yarn
System Info
System:
    OS: Windows 10 10.0.19044
    CPU: (24) x64 12th Gen Intel(R) Core(TM) i9-12900K
    Memory: 45.02 GB / 95.81 GB
  Binaries:
    Node: 14.17.4 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.19 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 8.6.0 - C:\Program Files\nodejs\npm.CMD
  Utilities:
    Git: 2.35.3. - /cmd/git
  Browsers:
    Chrome: 102.0.5005.115
    Edge: Spartan (44.19041.1266.0), Chromium (102.0.1245.41), ChromiumDev (Yes it loads)
  npmPackages:
    @vuepress/bundler-vite:  2.0.0-beta.48
    @vuepress/bundler-webpack: Not Found
    @vuepress/cli:  2.0.0-beta.48
    @vuepress/client:  2.0.0-beta.48
    @vuepress/core:  2.0.0-beta.48
    @vuepress/markdown:  2.0.0-beta.48
    @vuepress/plugin-active-header-links:  2.0.0-beta.48
    @vuepress/plugin-back-to-top:  2.0.0-beta.48
    @vuepress/plugin-container:  2.0.0-beta.48
    @vuepress/plugin-docsearch: Not Found
    @vuepress/plugin-external-link-icon:  2.0.0-beta.48
    @vuepress/plugin-git:  2.0.0-beta.48
    @vuepress/plugin-google-analytics: Not Found
    @vuepress/plugin-medium-zoom: 2.0.0-beta.48 => 2.0.0-beta.48
    @vuepress/plugin-nprogress: 2.0.0-beta.48 => 2.0.0-beta.48
    @vuepress/plugin-palette:  2.0.0-beta.48
    @vuepress/plugin-prismjs:  2.0.0-beta.48
    @vuepress/plugin-pwa: 2.0.0-beta.48 => 2.0.0-beta.48
    @vuepress/plugin-pwa-popup: Not Found
    @vuepress/plugin-register-components: Not Found
    @vuepress/plugin-search: Not Found
    @vuepress/plugin-shiki: Not Found
    @vuepress/plugin-theme-data:  2.0.0-beta.48
    @vuepress/plugin-toc: Not Found
    @vuepress/shared:  2.0.0-beta.48
    @vuepress/theme-default:  2.0.0-beta.48
    @vuepress/utils:  2.0.0-beta.48
    vue:  3.2.37
    vue-loader: Not Found
    vue-router:  4.0.16
    vuepress: 2.0.0-beta.48 => 2.0.0-beta.48
    vuepress-vite:  2.0.0-beta.48
    vuepress-webpack: Not Found

Expected. See https://v2.vuepress.vuejs.org/guide/assets.html#relative-urls

VitePress implement a custom markdown-it plugin to resolve this issue:

https://github.com/vuejs/vitepress/issues/450
https://github.com/vuejs/vitepress/pull/514

However, it's not an ideal solution. It made the alias fail to work.

I think it does be better to make:

![img](foo/bar.png) be ./foo/bar.png
<img src="foo/bar.png" /> be bar.png in package foo

Just like the absolute markdown link.

Markdown syntax can be special handled to align user's intuition when using other markdown tools, and advanced operations can be preserved when using html tags. @meteorlxy

Or in another word, I think it's do strange for people to use markdown syntax ![img](foo/bar.png) to refrence an alia.  Writing <img src"xxx" /> to do that fits people's intuition  better.


Update tracker to include proxy

Coverage decreased (-1.7%) to 97.328% when pulling 0137aaf029ab73c67257a7afcb0beef259ffb757 on vdriet:patch-1 into 447b4aacdea6ff6148f88fbf33b81f5f69bb9cf2 on piwik:master.

Coverage increased (+0.02%) to 99.046% when pulling 54987a0bf74d76a6b2b604c1f881db5f91d3f470 on vdriet:patch-1 into 447b4aacdea6ff6148f88fbf33b81f5f69bb9cf2 on piwik:master.

Thanks for the addition @vdriet!

Hello Brett Csorba,
Any plans to release this, would be nice for us.
Peter

Released in 1.1


Update steps-checkout.md

Modified details about the clean property and the default value in the UI.

Learn Build status updates of commit 51b0174:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

content/steps-checkout.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

#sign-off


Destination and ETA

Hi, love the app!
A feature that would be really useful: tracking towards a destination with different routing profiles (car, foot, bike etc) that is calculated when shared link is opened (sharer only provides location, destination and profile to destination).
Glympse has such a feature and I always use it.
Lots of kudos!

That's more like another app in itself say OrganicMaps or OSMAnd

OrganicMaps and OSMAnd only support sharing current location (without destination and ETA).
As I said, Glympse provides this functionality, but it's not self-hosted.

I meant as, use those other apps to tell you how long the trip (might) take.
How does the other app do it? It need to calculate the route? You want Hauk (the app) to do this for you? The PHP frontend on my Raspberry Pi1? C'mon

I meant as, use those other apps to tell you how long the trip (might) take.
That wasn't my original intent. Any tracking app can do this, no big deal. My intent was to SHARE the ETA and position/progress on a pre-defined track/routing.
How does the other app do it? It needs to calculate the route?
Of course, it needs to calculate and update the calculation if route is diverted
You want Hauk (the app) to do this for you?
Yes.
The PHP frontend on my Raspberry Pi1?
I don't know, I use docker on my NAS. And route calculation can be done offline even on my android using BRouter or any open maps app that supports offline (street) maps, so if an android phone can do it, an rpi is probably not so much worse?
Why do this when those 2 apps already do this, and better.
Because they're not self-hosted, and only one app does it: Glympse.

The ETA would be harder to impliment, per the discussion here.  But, the destination would be pretty easy.  Just have the map render a point at the specified address/coordinates in addition to the indicator arrow.  I'd personally like to see this feature.  This is a great app, thanks so much!


feat: recent unpaid bounties selection

Description
Added an option to show only recent unpaid bounties. Fixes #1586
Screenshots
The current bounties selection shows both paid and unpaid bounties:

The introduced unpaid bounties option filters out paid bounties and show only available ones:

Additional Context
I have some concerns, mainly due to to my lack of experience in the project:

This is the first selection option that includes a space inside. May it be a source of problems in the future?
The additional selection has been implemented as a separate option in the dropdown menu. Is there a way to add it as an additional option, such as a checkbox, while showing the "bounties" results? That would be interesting but as of now the query is merely done in terms of the route.
The option has been added to the ITEM_TYPES constant. Should it be added to the ITEM_TYPES_USER as well? Where is this other constant used?

Checklist
Are your changes backwards compatible? Please answer below: Yes
On a scale of 1-10 how well and how have you QA'd this change and any features it might affect? Please answer below: 8, I have tested it manually but were unable to add automated tests.
For frontend changes: Tested on mobile? Please answer below: Yes, everything works.
Did you introduce any new environment variables? If so, call them out explicitly here: No

The additional selection has been implemented as a separate option in the dropdown menu. Is there a way to add it as an additional option, such as a checkbox, while showing the "bounties" results? That would be interesting but as of now the query is merely done in terms of the route.

Yes, this should be possible and I would prefer this a lot more than having another dedicated option:

We don't use dynamic checkboxes anywhere but we have dynamic filters depending on what you selected in top. Maybe the code for them is inspiring.

I have modified the widget to use a checkbox instead of a separate select option:

The dropdown is still redirected to "bounties", I believe the UX is fluid enough. Let me know what you think of this reworked version :)

Thank you, I will take a look tomorrow

Sorry, forgot about this yesterday. Will review now

Hello, I totally agree with your review. I should've used a query param since the beginning, that was the most straightforward solution, sorry for that. I will correct the code and update the PR as soon as I can.

Thanks for your suggestion, I have applied them to my branch and tested that everything keeps working. I have decided to refactor variablesFunc adding explicit conditionals to avoid the nested ternary operators, but let me know if you prefer to keep with the previous version.

@ekzyis Thanks for your feedback. I am curious, why would prefer to use replace instead of push? Anyway, once you take the decision I will update the PR accordingly.

Hello, is there anything missing here and in my other pull requests? I am willing to correct them if necessary once I've got your feedback.

@aegroto we've just been swamped with higher priority things right now. I'm sorry for the delay.
I will get to your PRs before the end of next week at the latest. Feel free to pickup more issues in the meantime.


Please confirm ROCm works on Ubuntu 18.10

Does ROCm work on Ubuntu 18.10, which was released on October 18
https://wiki.ubuntu.com/CosmicCuttlefish/ReleaseSchedule

I ran it on 18.10 beta (downloaded 2018-10-05) and it worked fine. Installed without dkms.
There hasn’t yet been an amdgpu-pro release for 18.10, however. Just in case you want that too.

I can confirm what @949f45ac says. I've run ROCm 1.9.1 user-land code on top of the default Ubuntu 18.10 kernel and haven't run into any major problems.
That said, Ubuntu 18.10 is not an officially supported platform, so we have not rigorously tested all layers of the ROCm stack against it. It should work as far as we know, but it does not go through the same level of pre-released testing as our officially supported distros.

Installing and using GCC 7.3 over the default 8.2 may help with any compiler issues.

Thanks, I confirm I can use ROCm 1.9.1 OpenCL correctly on Ubuntu 18.10 with the default Linux kernel 4.18.0.
As a side note, it would be nice to mention somewhere on ROCm install instructions the udev setup that is needed when installing without dkms:
echo 'SUBSYSTEM=="kfd", KERNEL=="kfd", TAG+="uaccess", GROUP="video"' | sudo tee /etc/udev/rules.d/70-kfd.rules

I believe we do mention that specifically when discussing using upstream kernels with ROCm.

@jlgreathouse: yes, indeed that is documented, my bad.

Just to add for the record - on ubuntu 18.10, ROCM-based OpenCL acceleration for darktable works flawlessly (and a tad faster than with the legacy amdgpu-pro drivers).
The only package required is rocm-opencl; that, the udev rule above and adding the user to the 'video' group. Thanks @amd folks for making it simple

I'm able to install rocm-dev without error but the udev command doesn't work.
aadibajpai@ubuntu_hydralatest:~$ echo 'SUBSYSTEM=="kfd", KERNEL=="kfd", TAG+="uaccess", GROUP="video"' | sudo tee /etc/udev/rules.d/70-kfd.rules
tee: /etc/udev/rules.d/70-kfd.rules: No such file or directory
SUBSYSTEM=="kfd", KERNEL=="kfd", TAG+="uaccess", GROUP="video"

I believe we do mention that specifically when discussing using upstream kernels with ROCm.

The command you mention there is wrong because the character | is missing. The correct one is:
echo 'SUBSYSTEM=="kfd", KERNEL=="kfd", TAG+="uaccess", GROUP="video"' | sudo tee /etc/udev/rules.d/70-kfd.rules
So please, fix it.


Empty State Component

Visuals want to look at this to refine that visual approach.
The ixd/behavior will remain as seen here: https://www.patternfly.org/pattern-library/communication/empty-state/

Screens shared at demo 9/18/18
Empty State Demo.pdf

Per discussion in 9/20 PF Design Sync, want to add an example that shows what to do if there are multiple secondary actions or links.

Per discussion with @kybaker I am marking this as 'design approved' and ready for implementation. In HTML/CSS creating examples, however, we should allow for multiple secondary actions using the link-button style defined in the visual mockups.

@mcarrano re:

we should allow for multiple secondary actions using the link-button style defined in the visual mockups.

would those additional actions stack vertically so you might have primary on top, secondary under primary, tertiary under secondary?
@mceledonia it looks like the empty state is always vertically centered in its parent? Assuming so, can you explain what this variation represents where it's vertically aligned to the top of its parent?

@mceledonia

how big should the icon be? Is this size icon something we'll re-use? As in, should I create a variable for it (or even create sm, md, lg, etc sized vars for creating different sized icons)? Or is it a one-off?
what should the max-width be?
can you confirm the padding to use around the whole component (top/bottom/left/right), between each section, then between buttons?

@mcoker Great questions!

The icon size should be 54px. I suspect this is a one-off, as any other icons will be significantly smaller.
In my mockups I am using a max-width of 400px, let's try that and see how it works.
Padding

Between icon and header: 32px
Between header and description: 16px
Between description and button: 32px
Between button and secondary button: 16px
Padding around entire component, all sides: 32px
Note about padding:

In my mockups, in addition to the padding, I have the entire component centered in whatever it is contained in, both horizontally and vertically. Often times the component is in a space that is larger than the padding anyways which is why I wanted to note this.

Also from @mceledonia

the title is the only required element in empty state
action buttons should vertically stack on top of one another
empty state should always be vertically centered (even in responsive view)

@mceladonia is it true that the secondary actions will always stack even it there are multiple secondary actions? That's not necessarily what I expect.

@mceledonia ^^

we also updated the spacing so there is

24px between icon and title
16px between title and body
32px between body (or title, if body not present) and actions
8px between buttons


Make the fv3run functions lazy

The 3 fv3run functions each have different behaviors when it comes to waiting for the job to complete.

run_native, run_docker will block until execution is complete
run_k8s will not block

Behavior #1 is not the most convenient use for interactive use e.g. performing multiple run_docker runs within a single jupyter notebook. In this context, it would be more flexibile to return a subprcesss.Popen object and allow the user to wait for it complete or not.
Behavior #2 is great for one-off jobs, but it would be nice to be able to wait for completion e.g. with the one_step jobs.
To resolve this difference in behavior, I think all the run_ functions should immediately return some sort  of "promise" object that can be polled for completion, or be used to interact with the job in some simple way. There are many implementation strategies I can think of

make run_ be generators. For instance:

def run_native(...):
    yield subprocess.check_call(...)

generator = run_native(config) # returns immediately
next(generator) # blocks until check_call completes.

use some kind of "promise" object. e.g. dask.delayed, or python Future
For run_native and run_docker we could simply return the subprocess.Popen object. Then for run_kuberentes we could implement an object with the same interace as subprocess.Popen. This interface would have methods for waiting and polling for completion, and possible interacting with the logging streams.

@brianhenn and @frodre might be interested in this for the orchestration.

Overall lazy submission is a good idea. Two main thoughts on this:
This is a change that would break any workflow currently calling run_native or run_docker. The ways around that are either to update all of those calls, or to implement this in a backwards-compatible way. For example, we could add submit_native and submit_docker routines with this behavior, rename run_kubernetes to submit_kubernetes, and have run_kubernetes call submit_kubernetes with a deprecation warning. We'd remove run_kubernetes at a later time. It may be worth keeping the other run commands around for convenience.
I don't know how you'll implement this for kubernetes. I looked at it when I was writing run_kubernetes, and at the time decided job monitoring was better left to the Google Cloud Console because the python tools for it seemed either hard to use or not there. It could be worth another look.
It has no bearing on whether to do these python interface changes, but I wouldn't change the fv3run command line behavior.

Even if we plan to fully remove the run_ routines and move over to this new behavior, I'd still suggest moving over to new names and having a transition period where calling the old routines gives a deprecation warning.

I like the idea of changing the names.
As far as implementation goes, Stephen daspit implemented a loop and check for completion approach in his original prototyping of the one_step pipeline: https://github.com/VulcanClimateModeling/fv3net/blob/50548c69ece531d917c9420aa8cec4852cea0574/workflows/rerun-fv3/scale.sh#L18

I guess the approach then could be to submit the job as before, and then return a subprocess object that has called a bash script waiting to return based on this kubectl command? That could work. It might need some modification to be able to give an error if the job fails. The issue I ran into was there were no inside-python tools for this, I hadn't thought of calling kubectl.

Huh, I am surprised there is no kubernetes API object for this. Since the python k8s api simply resolves everything to HTTP requests to the k8s server, I think we could use requests in python to query for completion. As usual, the main problem is authentication, but hopefully we can leverage the k8s python library for that.

I was also surprised I didn't find something in the kubernetes package for this. There might be something I missed. I think it's somewhat telling though that e.g. this kubernetes monitoring app written in flask/python uses kubectl calls instead of the kubernetes package.

Okay. I figured out how to do this with the k8s python api. The batch_v1 api has a read_namespaced_job method that can get the run status. I wrote an exmaple here: https://gist.github.com/nbren12/7196d56f782947d454cbdd676f1ea8de

Nice!

We discussed removing this functionality altogether, so I will close the issue.


[iOS] Add Timetable List UI

Issue

close #378

Overview (Required)

Create Timetable List UI
Added singleLineFont modifier in TimetableFeature
Added Component folder in TimetableFeature

for reviewer

I wasn't able to fetch any speakers' info. It's a known issue of KMM library, right? → Fixed it by #388

And there aren't any design of multiple speakers, so I implemented it arbitrarily.

I think some sessions have same title and same durations, and different room. Can such sessions be displayed separately?
I try to implement setFavorite action, and we can see its working. But I'm not familiar with TCA, so I'm not sure if the implementation is correct.

Current implementation refresh all timetable when we toggle favorite :(

I cannot use preview feature of Xcode. Maybe the reason is these permission errors while executing build plugins. I was faced with similar problem in my other projects. Anyone know how to solve this?

Links

singleLineFont reference

Screenshot

Figma
After

I cannot use preview feature of Xcode. Maybe the reason is these permission errors while executing build plugins. I was faced with similar problem in my other projects. Anyone know how to solve this?

I will check this tomorrow. 🙏🏼

I will check this tomorrow. 🙏🏼

Thank you! But I can run without these error today after restarting Xcode, so it's okay if you don't have to check.