Add pdf output option

adding format: pdf: default to _quarto.yml resulted in pdf of only index.qmd on my first try. Cool that it adds "pdf" button to home page.
Noting attempt here.
format:
  html:
    theme:
      light: [cosmo, theme.scss] # from https://github.com/sta210-s22/website/blob/main/_quarto.yml
      dark: [cosmo, theme-dark.scss]
    mainfont: "Atkinson Hyperlegible"
    code-copy: true
    code-overflow: wrap
    toc: true
  pdf: default

Output
hyperlinks worked, but links to filename.qmd were linked to that filename, not the html

How to turn whole site to pdf? Searching
Typst available in Quarto 1.4:

Typst is a new open-source markup-based typesetting system that is designed to be as powerful as LaTeX while being much easier to learn and use. Typst creates beautiful PDF output with blazing fast render times.

Comments from posit/ quarto team in 2023  indicate we can pdf a book but not a website Is it possible to print your whole website to PDFs? #5782

This is great sleuthing Stef, thanks. Yes for the Reflections Booklet, we had to make the project a Book, not a Website. I'll ask  Andy about options for current website-to-pdf workarounds vs changing Series to a Book.

@jules32 the main barrier to converting the site to a book (which is the cleanest way to do this), is the three-level hierarchy (Core Lessons > GitHub Strategies > GitHub for Publishing). Books only allow two levels (Part > Chapter), so we would have to "flatten" that section to two levels...

Thanks @ateucher ! I've just merged and we can test to see how it looks

I've merged my branch with edits, so when @ateucher you're able to implement in Main, all my updates will be there. Thank you!

Closed by #54

Hi @ateucher ,
I noticed that the publish date was wrong so I fixed it to today's date and pushed, which updated the date on the html book. But I can't seem to update the PDF's date. I've tried via web browser (GitHub Action build) and also by running locally. When I tried running locally following the GitHub Action's code L29, I get this error:
JLos-MacBook-Pro:series jlo$ quarto::quarto_render("_make_pdf.qmd")
bash: syntax error near unexpected token `"_make_pdf.qmd"'

Do you mind having a look and updating the PDF? Thank you!

Following up @ateucher -
I realized I needed to also fix today's date in the _make_pdf.qmd file as well. The GitHub Action has now created a PDF with today's date.
Could you please still look into how to create the local build, and update the README with how to build the PDF locally? Thank you!

Nice, I didn't know about the last-modified quarto keyword (https://github.com/Openscapes/series/commit/ce9685d995f251daf0b6b64f87eb611444a1263f). I wonder if it should be today instead though? last-modified might not get updated if the constituent pages get updated but not _make_pdf.qmd itself?

Ah yes please update to today if that is the best current approach! Is there a way to delete it from the _make_pdf.qmd so that it only appears in one place, on the root index.qmd?

Not easily I don't think. But just saw this: https://github.com/quarto-dev/quarto-cli/issues/5967. Basically last-modified relies on the timestamp on your computer, so will always give today's date in GitHub Actions - basically the same behaviour as today :). I think the most straightforward is to just set today in both places...

Sounds like a good plan!

Continuing in #56


default text color to theme

default text color to theme, if not it will be always black (dark themes conflict for some items that are not in the if-else scheme).

One step further to close #415 . Thanks.


Add MGRS coordinates as an option to GRG Widget

Widget
GRG Widget
Version of widget
2.0
Enhancement
Add MGRS coordinates as an option for labeling cells in the grid.

@kgonzago can you confirm which part of the grid the MGRS value will be taken from - Center / Bottom Left?

@kAilshie We are looking into adding MGRS coordinates for the GRG widget. Since this enhancement came from your customer, can you provide some insight to @adgiles's question?

@kgonzago  I don't know that answer to that specific question but I reached out to my POC and I will relay any information I get back to you.

I would assume they mean like this:

So, I think it would probably be easier to create the GRG from the Grid Overlay widget, since an MGRS GRG goes counter to identifying your cell size, rotation, etc.
It would be like identifying the extent and the level of accuracy (4, 6, 8, 10 digits) and hitting OK.
If they just want to see what the coordinate is for the individual GRG grid square, they can use the Coordinate widget and select an MGRS output (red arrow above).

So, after some research, is been confirmed that customers want to use MGRS/USNG as an analysis mask, so not just as graphic overlay and not just for positioning/navigation. Part of that workflow is being able to create a GRG FROM the USNG/MGRS. For instance, if I was operating on the north side of Redlands, I'd want to choose the extent of that MGRS grid and create a feature for analytics - spatial joins, aggregation polygon, hot spots, etc. The extent of the AOI will help the user determine the size of the grid, large areas may use 1000km grids while small areas may want 100m grids. The benefit of MGRS/USNG is that it's not a temporary/arbitrary grid...it's a standard. Here's a poster that describes some of the use in more detail: https://www.dropbox.com/s/w8kesxwlal6y640/Shawnee USGIF Poster_v2.pdf?dl=0

This issue was moved to ArcGIS/solutions-grg-widget#3


feat(sink): add check license for snowflake, dynamodb opensearch sink

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
as title

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

cc: @xxhZs  could you resolve the conflicts? The license is expected to be enforced in the first candidate release of 2.0. Code freeze on 8/15


Replace GraphQL Playground with GraphiQL in templates

GraphQL Playground's features are being merged into GraphiQL and will only receive some maintenance and bug fix updates.
Our templates all use GraphQL Playground.
See https://github.com/prisma-labs/graphql-playground/issues/1143
Read the blog post announcing this: https://foundation.graphql.org/news/2020/04/03/web-based-graphql-ides-for-the-win-how-why-playground-graphiql-are-joining-forces/

We use Apollo Server Playground

Ah okay :)


[kubernetes-helm] curl docker compose URL is wrong

VSCode Version: 1.60.2
Local OS Version: Windows 10 21H1
Reproduces in: Remote - Containers
Name of Dev Container Definition with Issue: Kubernetes Helm

Steps to Reproduce:

Create a project from scratch
Build the devcontainer
Following errors appear during devcontainer build :

7 39.09 curl: (22) The requested URL returned error: 404
executor failed running [/bin/sh -c apt-get update && /bin/bash /tmp/library-scr
ipts/common-debian.sh "${INSTALL_ZSH}" "${USERNAME}" "${USER_UID}" "${USER_GID}"
 "${UPGRADE_PACKAGES}" "true" "true"     && /bin/bash /tmp/library-scripts/docke
r-debian.sh "true" "/var/run/docker-host.sock" "/var/run/docker.sock" "${USERNAM
E}" "${USE_MOBY}"     && /bin/bash /tmp/library-scripts/kubectl-helm-debian.sh "
latest" "latest" "none"     && /bin/bash /tmp/library-scripts/azcli-debian.sh
  && apt-get autoremove -y && apt-get clean -y && rm -rf /var/lib/apt/lists/* /t
mp/library-scripts/]: exit code: 22

Diagnostics and solution to fix it:
Here https://github.com/microsoft/vscode-dev-containers/blob/main/containers/kubernetes-helm/.devcontainer/library-scripts/docker-debian.sh#L211:
The request generated is wrong :
https://github.com/docker/compose/releases/download/v2.0.0/docker-compose-Linux-x86_64
In https://github.com/docker/compose/releases repo since 2.0 "Linux-x86_64" does not exist anymore. It needs to be change to https://github.com/docker/compose/releases/download/v2.0.0/docker-compose-linux-amd64

Dupe of https://github.com/microsoft/vscode-dev-containers/issues/1075.  Please update library-scripts/docker-debian.sh or library-scripts/docker-in-docker-debian.sh to the latest from the script-library folder in this repository.


Table: fix align attribute, Fixed #1337

Please makes sure these boxes are checked before submitting your PR, thank you!

[ ] Make sure you follow Element's Contributing Guide.
[ ] Make sure you are merging your commits to dev branch.
[ ] Rebase before creating a PR to keep commit history clear.
[ ] Add some descriptions and refer relative issues for you PR.

#1337

应该不是改这个地方，应该是生成 class 的地方。

https://github.com/ElemeFE/element/blob/dev/packages/table/src/table-column.js#L205
create之后是直接给columnConfig设置的。。

稍稍改了一下


Mica backdrop crashes WinUI 3 application

Describe the bug
Package versions
Microsoft.WindowsAppSDK: 1.3.230502000
Microsoft.Windows.SDK.BuildTools: 10.0.22621.756
CommunityToolkit.WinUI: 7.1.2
Microsoft.Extensions.DependencyInjection: 7.0.0
Crash details
Given this App.xaml.cs:
namespace Demo.WinUI;

public partial class App : Application
{
    private Window mainWindow;

    public App()
    {
        InitializeComponent();
    }

    protected override void OnLaunched(LaunchActivatedEventArgs args)
    {
        mainWindow = new MainWindow();
        mainWindow.SystemBackdrop = new MicaBackdrop() { Kind = MicaKind.Base };
        mainWindow.Activate();
    }
}

And this MainWindow.xaml:
<Window
    x:Class="Demo.WinUI.Views.MainWindow"
    xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
    xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
    xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
    mc:Ignorable="d">
</Window>

My WinUI 3 application crashes with:
Exception thrown: 'System.Runtime.InteropServices.COMException' in Microsoft.WinUI.dll
Exception thrown: 'System.TypeInitializationException' in Microsoft.WinUI.dll
Exception thrown: 'System.TypeInitializationException' in Microsoft.WinUI.dll
Exception thrown at 0x00007FF9AD5D536C (KernelBase.dll) in Demo.WinUI.exe: WinRT originate error - 0x80131534 : 'The type initializer for '_IMicaBackdropFactory' threw an exception.'.
onecore\com\combase\winrt\error\restrictederror.cpp(1017)\combase.dll!00007FF9AFDCC07A: (caller: 00007FF9AFDD15EF) ReturnHr(83) tid(cc5c) 8007007E The specified module could not be found.

When trying to use the XAML setter:
    <Window.SystemBackdrop>
        <MicaBackdrop Kind="Base" />
    </Window.SystemBackdrop>

The application crashes with a XAML parsing error:
Microsoft.UI.Xaml.Markup.XamlParseException
  HResult=0x802B000A
  Message=XAML parsing failed.
  Source=WinRT.Runtime
  StackTrace:
   at WinRT.ExceptionHelpers.<ThrowExceptionForHR>g__Throw|20_0(Int32 hr)

Without the MicaBackdrop the application works as expected.
Steps to reproduce the bug

Create a new project
Install Microsoft.WindowsAppSDK v1.3.230502000
Set MicaBackdrop to the Window
💥

Expected behavior
No crash, and a Mica backdrop.
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.3.1: 1.3.230502000
Windows version
Windows 11 (22H2): Build 22621
Additional context
No response

Hi, after following the repro steps above with a packaged WinUI3 app, I'm seeing the app build and run on 1.3.230502000, and it renders MicaBackdrop without crashing. Could you share a repro project or a crash dump? Also, was this a packaged or unpackaged app? Thanks!

Hi @christineyan4, thanks for the answer. I think I found the problem. It is a project with a WinUI project and an MSIX packaging project, I updated WindowsAppSDK in the WinUI dependency but did not notice the Packaging project still had a reference to an older WindowsAppSDK version. Once consolidated the project seems to build and run correctly, with a Mica backdrop.
I'm closing this issue.

Had the same issue with the same root cause. Thanks for posting this.


Cleanup linting system

There are a number of problems with the current linting system, most notably that lints could run without -Zcargo-lints being set. This PR fixes that issue and a few others that are low-hanging fruit.

@bors r+
As this is unstable, IM_A_TEAPOT is non-blocking.  So we dont' forget it, I'll note it in #12235

:pushpin: Commit 11d6013c1de69ada4dd009e1b3543afad5c608b3 has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit 11d6013c1de69ada4dd009e1b3543afad5c608b3 with merge 70fb49899402be512d09d0cb59eebe98eaad92d4...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing 70fb49899402be512d09d0cb59eebe98eaad92d4 to master...


Handle regular expressions

This should be fairly easy to get something working thanks to the fst crate.
It could be exposed in the query parse (not by default) with some nice syntax.
field:r"http://[\s]+" for instance ?

Now relatively easy given #299

I'll take a stab at this.


Code to extract overlaps from MPSs

This PR adds some utility functions to compute the overlaps from DMRG solutions (src/qb_gsee_benchmar/dmrg_utils.py) and shows a basic example of their usage (examples/get_overlap.ipynb). The branch also contains the latest scripts/overlaps.csv generated with examples/get_overlap.ipynb. To ensure reproducibility, it would be nice to make the MPSs accessible for internal use on the SFTP server (what do you think, @jp7745 ? If you agree, I can upload them and modify the jupyter notebook accordingly).

@aakunitsa:  yes, you can drop the MPS file on the SFTP server for posterity.  The MPS and overlaps are required for our approach to resource estimation, but not required for "the benchmark" on a general level, so I don't think anything more sophisticated than posting them on the SFTP server is necessary.  Thanks again!


Admin panel in portrait mode

Now, instead of all four admin photos on the same line in mobile view portrait mode, there will be two photos per line.  I changed ".admin {width: 50%;}" in the media query for max-width 960px.

Deploy preview processing.
Built with commit b84fec8b6df3d989cf08e13a10c8df6656d5be51
https://app.netlify.com/sites/fcccolumbus/deploys/59e67528a6188f5bc7b532bc

Coverage remained the same at 100.0% when pulling b84fec8b6df3d989cf08e13a10c8df6656d5be51 on amyngb:master into 1492fedfdb3878e2b071222a999d08817c924ab3 on FCCColumbus:dev.


Update Sidebar Styles Across Mobile and Desktop

Description

Consolidate all navigation links into the sidebar
Remove any user-related actions (settings, links) from the old sidebar
Show the user popover in the top right on mobile → this makes navigation between mobile and desktop consistent
Makes the styling consistent between the mobile and desktop sidebars

Changes since this post

[x] Close button is kind of just hanging there, there is space for something to the left of it. (logo or text?)
[x] User popover is not updated yet

Screenshots
Desktop Collapsed Sidebar Before & After:

Mobile Collapsed Sidebar Before & After:

Mobile Expanded Sidebar Before & After:

Desktop Sidebar (previously removed):

Steps to verify/test this change:

[ ] Verify changes work as expected on staging instance

Final Checks:

[ ] Verify successful deployment

Issues

Closes #

Updated the user popover and added the logo on the top left of the sidenav to fill the empty space. I think this looks better:

I think there may be a better way to implement the sidenav (would it be better as an overlay or something similar?), but what I have changes the existing the code the least (and I wasn't sure if things like overlays would be better for a full-height element) so I stuck with that.

I think the implementation is fine as is, don't think an overlay would be any better.

peterportal logo in the header has shrunk quite a bit and just looks odd to me lol. was that intentional?

I changed it from vw to a fixed size, so on most screens it is actually larger. This is intentional, but I can bump it up slightly or clamp it between two widths instead of locking it to one.

i think the link to the feedback form (and all the other links in the footer) should open in a new tab

Yes, thank you for reminding me! I did intend to do this.

there is an edge case where having just the week # in the header might be unclear

hmm that's interesting. I think we should come up with a better solution than putting it in the topnav though, so it's more of a temporary measure (and would hopefully be resolved before summer :wink:)

Updated/Fixed the header image size and the roadmap page.
Before:

After:

It is easier for me to think of things in px instead of rem. Additionally, I don't really find the argument for rem too convincing since the root element is almost always constant anyways (in PPC it's 14px), so it's just more conversion.

Well the main argument for em/rem is accessibility. By default 1rem is 16px (or I guess 14 here, idk why, that does make the math a little annoying), but someone who has trouble seeing might bump that up in a font size setting on their browser/device.

I changed it from vw to a fixed size, so on most screens it is actually larger. This is intentional, but I can bump it up slightly or clamp it between two widths instead of locking it to one.

I see, yeah that is much better on mobile. Thanks for fixing the usage of vw/vh, also seemed weird that everything would shrink in height/width based on screen. But yeah bumping it up so the desktop one is closer in size to what we had before would be nice.
edit: looks like you beat me to it

Well the main argument for em/rem is accessibility. By default 1rem is 16px (or I guess 14 here, idk why, that does make the math a little annoying), but someone who has trouble seeing might bump that up in a font size setting on their browser/device.

I see, that makes sense. I assumed that Zoom would fix that but I see how it can be more than just that. Is it safe to assume this is primarily for font sizes and not for other measurements then?

Yeah, I think that's a safe assumption.

Re-requesting review since I'm not sure if you got a chance to review the entire PR yet.
I don't think it's worth the extra effort in changing any additional font-size properties to rem since the root needs to change size later and there are 20 ish other instances of font-size being in px as well (it's not an odd one out).
Seems like it would be better to just leave it as is so it doesn't break when changing the root font size. Will just do all of the font size fixes in a future PR

When viewing on my laptop, there's a scrollbar now on the root.

Oh, did not realize this was not there before. I will take a look.


Correct interpretation of SystemVerilog C-style array dimensions

IEEE Std 1800™-2017 7.4.2 specifies that [size] is the same as [0:size-1].

This change looks good to me, but it's not clear to me whether this makes an observable difference for anything currently supported by yosys's frontend. Do you have an example that requires this fix? It should be merged in any case, so the implementation matches the specification, but it would be good to know whether I missed something obvious here.

I found this bug when comparing a pristine SystemVerilog design with the same design preprocessed by sv2v, using equiv_induct. At the very least, the issue causes noise if you wish to check for other issues in a similar way.
I don't think it's very likely that one would be find any real world example where this makes a difference, however it certainly can. I guess one could write a test e.g. based on the following statement from section 7.6 in the spec: "For example, if array A is declared as int A[7:0] and array B is declared as int B[1:8], the
assignment A = B; will assign element B[1] to element A[7], and so on." I believe that changing B to int B[7] in this example would reveal the issue. Should I try to add such a test to the PR?

Perhaps it's also possible to make a test based on 6.24.3 Bit-stream casting, even though IIRC support for type casting is quite limited in Yosys.

Assigning arrays is what I tried, but that doesn't seem to be supported. Neither is bit-stream casting. If you have any example you can share where you end up with different RTLIL before and after this change, even if not a minimized test case, that would be great. I'm not super familiar with the frontend, and having an example makes it easier to spot potential issues when reviewing changes.
I'm pretty sure the change in this PR could not break anything new, though, so if you can't reproduce it or when it would be too much work I'm fine with merging this as is.

I must have confused the use of equiv_induct with #3551, I found this issue using read_verilog -dump_vlog2. I am not able to find any construct in Yosys which can reveal the difference, and I guess that's why I am not able to show any difference in RTLIL either. The last point where I can see the difference is with -dump_vlog2. However this is the same with e.g. bit v[1:0] vs. bit v[0:1], as far as I can tell :man_shrugging:
I am a bit confused about the test you pasted above. Which PR is causing trouble with that - this PR, #3552, or both?

Ah, that's where the edit went 😅 No this test should have only gone to #3551. I edited the comment on #3551 to remove part of it when I realized I hadn't properly tested it and then after properly testing it wanted to undo that and must have switched to the wrong tab. Sorry for all that confusion.


Update Readme and AWS Blog example

Issue #304
Description of changes:

Moved the blog example sh to the branch.
Added the otel-collector-config with batch processor.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Readme changes are good 👍 . We can revisit docs when we start adding the scaling/tuning guidelines


Add shim for Python's gRPC core on Windows

cc @nathanielmanistaatgoogle @ctiller

Can one of the admins verify this patch?

Can one of the admins verify this patch?

 Also the binary dlls. If there's commits referencing these, the download size for the git clone will be forever affected, so please remove all the binary files and squash the commits.

^ addressed

Something broke for all languages in that PR. I'll look into that, possibly forking that PR.

#5064 replaces this one. Took me a while to realize what was broken - you moved the inclusion of stdint.h to the top of port_platform.h. Why did you move it ? :) It was down here for a reason.

Also in #5064 I am properly cleaning up the remaining binary files.


Meal analysis

One of the ideas of #102 is to use information already on the data as a first step on identifying unusual meal expenses. This PR, contains a Notebook that uses a robust oulier method to indentify some suspect cases.

Hi @talespaiva first of all, thank you so much for your contribuition, it's not a duplicate work, and it helps a lot. You did a great job at all! :)
I was looking at your code and notice some things, can you explain a little bit more of what you are trying to do in your code and the next-steps?

I think that the best way to continue this work is generating a dataset with the average prices that we found for all places, that is something that @talespaiva explored, what do you think about that?

Great. Will try to better explain the notebook. I added some comments in between, but apparently it didn't help a lot. In fact, re-reading the notebook now made me realize that it's a little mixed up.
The whole analysis is based on grouping only meal expenses by CNPJ/CPF. Up to cell [8], it's just data preparation. First, I've looked to the places that received most money (cell [9]). The first issue with the dataset is that some places have more than one CNPJ, like SENAC. The accumulated net_value does not say much about irregularities, so I've reordered the meals dataset to reveal the most visited places (cell [11]). From there, I combined the two previous dataframes to have an average value per visit (cell [13]), that has shown some strange values such as R$ 4200 spent on just one visits in a place that looks like a car selling store (according to its name). I have even contacted Lucio Big from OPS about this case and he said this congressperson had already been sued by him for things like that.
From this first part, I think that a future work could be identifying those places that are almost never visited and have high values spent on. There's not enough data to take an average value, but an overall price for the entire dataset may be useful to highlight these cases.
At cell [15], I've made a rather naïve assumption that the more a place is visited, the more money is spent in there. This is obviously not true since there are great differences in prices depending on which place is visited, but it turns out that a linear model wasn't too bad, but I think other models should be tested and compared later. From this part, the points that are too far below the line in the picture could be better investigated because they are either too expensive (they get more money with less visits).
From cell [16] on, I tried to make an individual analysis, more in the spirit of #102, by using only expenses in a single place (notice that there the two SENAC instances should be merged for a proper anaylis, but it wasn't done in this case). For that, I've used the Outlier Labeling Rule, that uses the median instead of the mean, a measure that is more robust to outliers. The authors of this methods recommends that any modified z-score greater than 3.5 should be considered as a potential outlier if the data is expected to come from a normal distribution. Basically, what follows is the application of this method for the first 10 most visited places, as it's not very helpful to make this to places less frequented.
I like the idea of creating a dataset with the average prices (may be with the median price too), however I think the number of visits should be included. Also, merging CNPJs must be addressed in this task (I don't know if it's only the case of SENAC) and I found that selecting a name to associate to a CNPJ/CPF let the analysis more pleasant to the eyes :-)
Another branch that can be created for the project is identifying names of places that don't have much to do with the subquota under it was paid, for example, the case of meal in a car selling store. I don't know if this is on your roadmap, just a random idea here.
I think I repeated some things already explained in the notebook. If it wasn't clear, please let me know if I can better explain some point.

@talespaiva that's awesome, thanks for the explanation, but don't you think that those comments about you code would fit better in your code? :)

I like the idea of creating a dataset with the average prices (may be with the median price too), however I think the number of visits should be included. Also, merging CNPJs must be addressed in this task (I don't know if it's only the case of SENAC) and I found that selecting a name to associate to a CNPJ/CPF let the analysis more pleasant to the eyes :-)

Couldn't agree more, I think that's a great idea, it will be nice to have a dataset that looks like this.

Another branch that can be created for the project is identifying names of places that don't have much to do with the subquota under it was paid, for example, the case of meal in a car selling store. I don't know if this is on your roadmap, just a random idea here.

Yes, it's in the roadmap, but it can be better investigated using machine learning algorithms

I think I repeated some things already explained in the notebook. If it wasn't clear, please let me know if I can better explain some point.

No, the notebook it's awesome, I was talking to @cuducos and I really wanted to do something like that. That's a great contribuition, I'm only thinking about doing a dataset, or what can we do more with this first step, for us to explore this data in a better way!

@anaschwendler, I have updated the notebook with some parts of my comment. I can help with creating the dataset, but I'm busy for the next few days.

@talespaiva that was exatcly what i was looking for, i will work based on what you had collected and try to create a dataset.
Thanks a lot for the contribuition, it saved a lot of time for us :heart:
Soon, i'll merge your pr to the master branch! :)

It's up to go :)
Merging it!


Architecture Thread

Architecture Thread
Summary
This is a compilation of thoughts on project architecture design. What are different ways that we can structure this? What would make future development the easiest? What would make the API the clearest?

What if we pursued a structure that looked something like this?
module Backtest

module Types

module InternalTypes
# encoding all non-API types
end # module

# encoding for all API types
end # module

module Internals
# code for all non-API functions
end # module

# code for all API functions

end # module

The major benefit is that it can very clearly separate the interface from the implementation.

It may be a good idea to get rid of StrategyOptions and instead move those arguments to run. I don't see a point in having the StrategyOptions layer of abstraction, since we can just perform behavior from within run that processes the StrategyOptions args.


Support power_state_topic updates in climate.mqtt

Proposed change
Currently climate.mqtt supports setting and updating the mode via mode_command_topic/mode_state_topic. Additionally, if power_command_topic is configured, it will be used to send an on/off command.
However, if the HVAC doesn't send an update on mode_state_topic when its power setting is changed, then the component is not updated.
Therefore I suggest the following change:
If power_state_topic is configured, then the state of the component will be derived first of all from the power state, and only if it's "on", from the mode.
Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New integration (thank you!)
[x] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Example entry for configuration.yaml:

# Example configuration.yaml
climate:
  - platform: mqtt
    name: AC
    modes:
      - "heat"
      - "cool"
      - "dry"
      - "off"
    fan_modes:
      - "high"
      - "med"
      - "low"
    power_command_topic: "devices/AC/power/state/set"
    power_state_topic: "devices/AC/power/state"
    payload_on: "on"
    payload_off: "off"
    mode_command_topic: "devices/AC/mode/state/set"
    mode_state_topic: "devices/AC/mode/state"
    temperature_command_topic: "devices/AC/temperature/state/set"
    temperature_state_topic: "devices/AC/temperature/state"
    fan_mode_command_topic: "devices/AC/fan/state/set"
    fan_mode_state_topic: "devices/AC/fan/state"

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[x] The code change is tested and works locally.
[x] Local tests pass. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist
[x] The code has been formatted using Black (black --fast homeassistant tests)
[ ] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated for www.home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly.
Updated and included derived files by running: python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt.
Updated by running python3 -m script.gen_requirements_all.
[ ] Untested files have been added to .coveragerc.

The integration reached or maintains the following Integration Quality Scale:

[ ] No score or internal
[ ] 🥈 Silver
[ ] 🥇 Gold
[ ] 🏆 Platinum

Hi @giladger,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

The climate integration does not have a notion of a power topic. Instead, HVAC_MODE_OFF is set as HVAC_MODE to indicate that a climate is turned off.
So instead of adding more functionality to power topics in MQTT Climate, we should remove the topic completely to align with the innerworkings of the climate integration.


Missing documentation for error messages

The following error/warning messages are missing in the documentation:
        /// <summary>
        /// Invalid package types
        /// </summary>
        NU1204 = 1204,

        /// <summary>
        /// project has an invalid dependency count
        /// </summary>
        NU1211 = 1211,

        /// <summary>
        /// Incompatible tools package/project combination
        /// </summary>
        NU1212 = 1212,

        /// <summary>
        /// Package contains unsafe zip entry.
        /// </summary>
        NU1402 = 1402,

        /// <summary>
        /// Package Signature is invalid
        /// </summary>
        NU1410 = 1410,

        /// <summary>
        /// Error_PackFailed
        /// </summary>
        NU5006 = 5006,

        /// <summary>
        /// Warning_SemanticVersion
        /// </summary>
        NU5113 = 5113,

What is the status here? I am looking for docs about error NU1211 and cannot find it on https://docs.microsoft.com/en-us/nuget/reference/errors-and-warnings


Fix file type for 66b49220e6802e0001eb4602

GCST90446475 should be gwas-ssf and GCST90446476 should be non-gwas ssf

GCST90446475 and GCST90446476 don't exist in the meta collection.

Added them to meta collection.

YAML files will be generated in 2 days.

@Santhi1901 please confirm these look ok

The file type is correct for the 2 studies


Fix typos

Thank you very much for this!

For some reason, Travis rejected the build request for this PR (https://travis-ci.com/coblox/bitcoinrpc_rust_client/requests). I already contacted Travis support about it!

Hm, it appears my account has been flagged for potential abuse :)
Not sure why.
I have also contacted Travis support.


Adding tools and Eula page to Resource Deployment

Resource Type Picker dialog:

First page of every deployment:

Changing options in deployment:

Currently changing wizard tiltes and pages does not work without closing and reopening the wizard. (it makes the changes to objects but visually everything remains the same).
Also, the done button listener gets broken when we close and reopen the dialog
For these reasons, I am creating a new wizard every time user changes the options.

Pull Request Test Coverage Report for Build 341466115

29 of 284 (10.21%)  changed or added relevant lines in 7 files are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.01%) to 39.555%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/resource-deployment/src/main.ts
0
1
0.0%

extensions/resource-deployment/src/services/resourceTypeService.ts
0
2
0.0%

extensions/resource-deployment/src/ui/resourceTypePickerDialog.ts
0
2
0.0%

extensions/resource-deployment/src/ui/resourceTypeWizard.ts
3
24
12.5%

extensions/resource-deployment/src/ui/pageLessDeploymentModel.ts
8
31
25.81%

extensions/resource-deployment/src/ui/toolsAndEulaSettingsPage.ts
9
215
4.19%

Files with Coverage Reduction
New Missed Lines
%

extensions/resource-deployment/src/ui/resourceTypePickerDialog.ts
2
0%

extensions/notebook/src/jupyter/serverInstance.ts
4
73.61%

Totals

Change from base Build 339271057:
-0.01%

Covered Lines:
20702

Relevant Lines:
48064

💛  - Coveralls

Rename toolsAndEulaSetingsPage.ts to toolsAndEulaPage.ts to match the class name.

With regards to this:
Currently changing wizard tiltes and pages does not work without closing and reopening the wizard. (it makes the changes to objects but visually everything remains the same).
Also, the done button listener gets broken when we close and reopen the dialog
For these reasons, I am creating a new wizard every time user changes the options.

Please add comments in appropriate code for the above.


Use field editorconfig when sanitising content

see https://github.com/silverstripe/silverstripe-framework/issues/10975 for the original issue
This changes to using the field configured HTMLEditorConfig instance when sanitising content for saving. It will fall back to the active config if none is defined on the field.
Issue

https://github.com/silverstripe/silverstripe-framework/issues/10975

Wasn't exactly sure which branch to target - currently 5 but maybe should b 4?

If this is a bug that affects CMS 4, please target the 4.13 branch.
For future reference: https://docs.silverstripe.org/en/5/contributing/code/#picking-the-right-version

thanks I'll keep that for reference as I often wonder which one :)


TimeoutError: socket hang up

When quickly putting more than 10-20 objects, I get a TimeoutError: socket hang up.
After this the flow has stopped and I am unable to catch the error with the Catch node.

How to reproduce:

Create a bucket on Scaleway
Get 100 objects +
Upload each as separate file

Also, as discovered later: if using node.send(msg) it overwrites the same file over and over (creating multiple versions).

Fixed in this commit.


Refactor sink helpers

Moved sink helpers to controller/sinks package since they aren't generic.
Removed superfluous logging to eliminate the context argument and keep logging out of library functions.
Get namespace from ObjectRef instead of separate parameter. The ContainerSource reconciler always forces this to be the same as the source namespace, same as before.
Create a new function for setting the sink argument since I find several early returns easier to follow than several nested ifs.
Change the condition reason from "NotFound" to "Missing" when the Source has no sink ref, since the user action required is different. This should really be validated by the webhook but we don't have a
webhook yet.
Also fixed a few comment typos caused by function renames.
/cc @n3wscott

@adamharwayne made a convincing argument that the namespace argument of GetSinkURI should remain because it's always required and burying it in the ObjectRef makes it easy to miss. I rolled that change back.

/lgtm
Thanks for the cleanup!!


Add throttle setting

For long running syncing, you might now want Dat to take over your whole bandwidth

+1 yeah good idea. this is a pretty standard feature in torrent clients

possible now in code, just need to expose through the UI


[Meta] Catch's performance

As well know, Catch's performance, at both runtime and compile time, could be better.
Some changes are on their way to master, but more could be always done. To that end, benchmarks (self-contained test suites) that show pathological usage patterns, representative usage patterns or just unexpected slowness, are welcome.
Obviously, so are PR's against dev-performance branch.

There is a request for special macros that only check for equality, without having expression decomposition, etc in #668.

After merging in dev-performance-round2, I did some benchmarking, comparing current master to last release (1.6.1).
Using 3 benchmarks

A long loop (2^20 iterations) with extremely simple REQUIRE condition.
A long loop (2^20 iterations) with simple CHECK condition, that fails.
Full benchmark suite of the Json for Modern C++ project (version 2.1.0, contains ~11M assertions).

I ended with these numbers

Benchmark
1.6.1
master

Passing tests
3.4s
0.7s

Failing tests
15.5s
15.1s

JSON for modern C++
6m 25s
4m 14s

Not bad! And nice use of a real world project

FYI, the complaint I was given by one build engineer last year was that the compile time was becoming too long. I suggested precompiled headers, but he wasn't sure how to do it right on Linux.

@GatoRat Yeah, there definitely has been some compile time regression (I added compile time information to the post above), as the framework gains more features there is more code that the compiler has to chug through. If you have some specific code that has gotten significantly worse, please share.
Otherwise, the compile times should get a bit better soon, Phil has some simplification in store.

The new compile time flag (CATCH_CONFIG_FAST_COMPILE) speeds up test compilation on linux by ~10%. Windows benchmark pending.

Another ~15% speedup for test compilation using CATCH_CONFIG_FAST_COMPILE was achieved in 7a8a0205b46a3770cc8f0d26a24a903a4c03e82a.


Custom entities: using unknown field doesn't create a corresponding Kubernetes event

Problem statement
When using KongCustomEntity with invalid fields, e.g. :
apiVersion: configuration.konghq.com/v1alpha1
kind: KongCustomEntity
metadata:
  namespace: default
  name: degraphql-route-example
  annotations:
    a: a
spec:
  type: degraphql_routes
  fields:
    uri: /contacts
    queryy: "query{ contacts { name } }" # Mark the misspelled field name
  controllerName: kong
  parentRef:
    group: configuration.konghq.com
    kind: KongPlugin
    name: degraphql-example

Kubernetes event is not created.
This is a consequence of the response body not having the appropriate tags. The response from Kong has the following body:
{
  "message": "declarative config is invalid: {}",
  "name": "invalid declarative configuration",
  "fields": {},
  "code": 14,
  "flattened_errors": [
    {
      "entity": {
        "uri": "/contacts",
        "service": {
          "id": "93e37996-f51b-5d53-8776-529c3bfd6421"
        },
        "queryy": "query{ contacts { name } }"
      },
      "entity_type": "degraphql_route",
      "errors": [
        {
          "type": "field",
          "message": "unknown field",
          "field": "queryy"
        },
        {
          "type": "field",
          "message": "required field missing",
          "field": "query"
        }
      ]
    }
  ]
}

Which KIC cannot use to produce an event because of lack of metadata to use to match the offending resource.
More information
Providing the tags does make Kong return them in the response but also returns an unknown field error:
{
  "message": "declarative config is invalid: {}",
  "name": "invalid declarative configuration",
  "fields": {},
  "code": 14,
  "flattened_errors": [
    {
      "entity": {
        "tags": [
          "k8s-name:c1",
          "k8s-namespace:default",
          "k8s-kind:KongConsumer",
          "k8s-uid:aebfbebc-6145-4428-9ac6-e408737f52b4",
          "k8s-group:configuration.konghq.com",
          "k8s-version:v1"
        ],
        "uri": "/contacts",
        "service": {
          "id": "93e37996-f51b-5d53-8776-529c3bfd6421"
        },
        "queryy": "query{ contacts { name } }"
      },
      "entity_type": "degraphql_route",
      "entity_tags": [
        "k8s-name:c1",
        "k8s-namespace:default",
        "k8s-kind:KongConsumer",
        "k8s-uid:aebfbebc-6145-4428-9ac6-e408737f52b4",
        "k8s-group:configuration.konghq.com",
        "k8s-version:v1"
      ],
      "errors": [
        {
          "type": "field",
          "message": "unknown field",
          "field": "tags"
        },
        {
          "type": "field",
          "message": "unknown field",
          "field": "queryy"
        },
        {
          "type": "field",
          "message": "required field missing",
          "field": "query"
        }
      ]
    }
  ]
}

Probable direction forward is to discuss with Gateway whether this can be made possible. If yes, then implement setting tags on configuration generation from KongCustomEntity. If not, then document that events are not going to be triggered for invalid fields in KongCustomEntity.
Related: #6629
Acceptance criteria

[ ] As a user I can expect that a Kubernetes event is created when I create KongCustomEntity with invalid fields

One option is to validate the Resource against the schema we pull from Kong.
We should also make sure there's an issue for having tags on degraphql_route and other entities created by admin apis.

Looked into it and I was able to solve both #6793 and #6794 with #6802.
Discussed with @randmonkey and he agreed with the approach so we might be able to put this in 3.4 if it merges this week.


[Consensus] BlockRetrieval types

Summary:
BlockRetrieval types used to be scattered across several network-related files.
This change is a noop from the business logic perspective: it just organizes the
BlockRetrieval related types logic in consensus types.
Testing:
unit tests
Issues:
ref #1491

@bors-libra r=zekun000

:sunny: Test successful - checks-circle_commit_workflow
Approved by: zekun000
Pushing 6b7b568b6f2d6c0c2f04ed43410b48e7ad1c3a1a to master...


Is it possible to map value(maybe string or int)  to int

JSON may be as follows:
{
    "id":123,
    "name":"XXX"
}

{
    "id":"123",
    "name":"XXX"
}

If one day server changes "id" from Number to String for some reason. But actually type of "id" is not determined. Once it has changed, I have to update my Model Class.
class Model: Mappable {
    var id: Int
    var name: String
    ......
}
update to:
class Model: Mappable {
    var id: String
    var name: String
    ......
}

Is there a way to get the Model correctly whatever the server sends me? I want "id" as Int even if it is a String in JSON, but without changing the implementation of Class?
Thanks!

You should be able to build a Transform that handles this. This example below converts a String to an NSURL.
public class URLTransform: TransformType {
	public typealias Object = NSURL
	public typealias JSON = String

	public init() {}

	public func transformFromJSON(value: AnyObject?) -> NSURL? {
		if let URLString = value as? String {
			return NSURL(string: URLString)
		}
		return nil
	}

	public func transformToJSON(value: NSURL?) -> String? {
		if let URL = value {
			return URL.absoluteString
		}
		return nil
	}
}

In the transformFromJSON function you could check the type of the input value and convert it to an Int appropriately depending on its type.

Thanks a lot


What does this PR try to resolve?
Fixes #14354
This approach chose to continue generating a .cargo_vcs_info.json for bare commit git repo.
How should we test and review this PR?
Compare the test changes before and after the two commits
Additional information

It hits the API limit

#2 [internal] load metadata for docker.io/library/alpine:3.20
#2 ERROR: failed to copy: httpReadSeeker: failed open: unexpected status code https://registry-1.docker.io/v2/library/alpine/manifests/sha256:eddacbc7e24bf8799a4ed3cdcfa50d4b88a323695ad80f317b6629883b2c2a78: 429 Too Many Requests - Server message: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit

@rfcbot fcp merge
This insta-stabilizes an edge case of .cargo_vcs_info.json:
When a Git repo has no commits, the file content would be
{
  "git": {
    # there is no "sha1" field because `HEAD` is missing
    "dirty": true
  },
  "path_in_vcs": ""
}

@weihanglo there are several solutions for this

Don't include the file
Do whatever non-git locations do
not have the git table

and likely more.
Is there a motivation for the proposed solution?  I didn't see it here or in the issue.

I am sorry that I just did a really quick FCP without thinking hard on it.
I don't have a good answer. Thinking backward from #13695, the requirement is to have commit hash as well as the path_in_vcs field for other tools if they want to post-process further. The current approach this PR takes, generating vcs info file even with no sha field, is still useful for preserving path_in_vcs.
However, I personally doubt it is useful in real life, since that happens on in an edge case that no commit exists. I am fine with whatever approach we choose. We just need to fix the regression.

@weihanglo Thanks you rapidly response.
Sorry for that, I had change my mind. I'm not sure if the absence of this field will cause problems in other ways,  so it's better to not include/generate the .cargo_vcs_info.json in this case.
Of course, this is only a reasonable change that I currently feel, which should be very rare for this scenario, and it may be better to maintain the consistency with the previous

Agree that not generating vcs info file is also a reasonable approach to take. It stays with the old behavior (not generating when --allow-dirty presents) without blocking cargo package.

Let's discuss different approaches in the issue then.

Thanks for the contribution. This should be the least controversial way to fix this. See the  discussion in #14354.
@bors r+

:pushpin: Commit 37cda2d9a0d5a68dfd1fae55934cc23958c9f084 has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 37cda2d9a0d5a68dfd1fae55934cc23958c9f084 with merge b66cad8038bd0a94164a4ecd1b2c12075a42610d...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing b66cad8038bd0a94164a4ecd1b2c12075a42610d to master...


Fixed and made Enhancements in Chatbot

Hey @sanjay-kv
I have

Implemented a smooth floating animation effect to make it appear more interactive and alive. (as the previous one was not working)
The animation is infinite, with a gentle up-and-down motion to draw user attention.

https://github.com/user-attachments/assets/8ffafc9b-588e-47a1-81d6-622994737dc5

Added a tooltip with a welcome message. (The tooltip is designed to provide a friendly greeting and brief introduction to the chatbot's purpose)
The tooltip contains the message:

"Welcome to RecodeHive Machine Learning Repos, How can I help you?"`

Light Mode

Dark Mode

Additionally, a media query was implemented to ensure the tooltip did not appear on touch devices or screens that were narrower than 768px. This prevents any interference with the user experience on mobile devices

Damn that's a great work @MastanSayyad

awesome, this same thing we will implement on gssoc as well later.

@sanjay-kv Would love to implement there as well!


Address returned by read_address() doesn't match with the one in objdump of object file

Hi,
I am reading a relocatable object file, to get a location list at a particular offset and I'm using this ReaderRelocate implementation: https://github.com/philipc/rewrite/blob/master/src/dwarf.rs#L363. I'm trying to read existing location list, add a new entry there and then update the object file with the newly created location list.
I see that ReaderRelocate::read_address() uses a Vec<write::Address> and returns an index into this Vec, instead of the actual address value.
But, somehow, when I am reading begin and end addresses in a location list at a particular offset, it's showing a different address, as compared to one shown by objdump of the object file.
Could you please suggest something?
Thanks!

Can you give me steps for how to reproduce the issue?

Sure.
I am working with this object file - bzip2.o and I am looking at variable zz inside the UPHEAP function.
First, I am traversing the DIEs, to reach at function UPHEAP and when I see the variable zz, I am printing the address ranges inside its location list. It is showing the output as follows:
output1:-
Entry tag: Some("DW_TAG_variable")
Index : 0
location lists offset: LocationListsOffset(252)
loc range: Range { begin: 130, end: 131 }
loc range: Range { begin: 132, end: 133 }
loc range: Range { begin: 134, end: 135 }
Name: zz

But, when I look into the objdump of the same object file, I get the following location list:
output2:-
    000000fc 000004b6 000004bd (DW_OP_breg4 (esp): 12)
    00000108 000004bd 000004cd (DW_OP_reg2 (edx))
    00000113 000004e9 000004eb (DW_OP_reg2 (edx))

We can see that the LocationListsOffset(252) from output1 is indeed the one present in the location list from output2 - 0xfc.
And as we are using ReadAddressMap to store addresses, I expect that the locations 130..135 in output1 should point to the addresses 4b6..4eb in output2.
But, they seem to be pointing to the addresses 43e..490, when I print the ReadAddressMap.
Also, if I try to get the mapping of the key 130 in ReadAddressMap, while printing the ranges in output1 above, it shows an error - indicating that the map only has around 68 entries in it.
Could you please have a look at it?
Thanks!

Sure, might be a few days though. Let me know if you fix the problem before then.

Using https://github.com/philipc/rewrite/commit/9d3e9481fe3bb98e1cf8c84870f04763242cf851, I get
input:
    000000fc 000004b6 000004bd (DW_OP_breg4 (esp): 12)
    00000108 000004bd 000004cd (DW_OP_reg2 (edx))
    00000113 000004e9 000004eb (DW_OP_reg2 (edx))
    0000011e <End of list>

output:
    00000123 000004b6 000004bd (DW_OP_breg4 (esp): 12)
    0000012f 000004bd 000004cd (DW_OP_reg2 (edx))
    0000013a 000004e9 000004eb (DW_OP_reg2 (edx))
    00000145 <End of list>

This looks correct to me, so you need to give me more information on how to reproduce this problem.

Oh, I see. Thanks!
I was getting different value, when I print it while iterating over the location lists, before converting read::Dwarf to write::Dwarf, like below:
let mut loclist_iter = dwarf.locations(&unit, location_lists_offset).unwrap();
while let Some(loclist_entry) = loclist_iter.next().unwrap() {
    eprintln!("loc range: {:?}", loclist_entry.range);
}

Looking at your output, I thought there might be something that is being added to the original value of the range.
So I debugged through the calls to loclist_iter.next() inside gimli and I found that a base address is being added to the range of an location entry, if it was AddressOrOffsetPair:-
RawLocListEntry::AddressOrOffsetPair { begin, end, data }
| RawLocListEntry::OffsetPair { begin, end, data } => {
    let mut range = Range { begin, end };
    range.add_base_address(self.base_address, self.raw.encoding.address_size);
    (range, data)
}

I found that this base_address is coming from unit.low_pc, whose value was non-zero, as it was representing an index into the ReadAddressMap.
Now, for using the range of a location list entry, I will have to subtract this base_address to get the correct index into the ReadAddressMap.
The issue is resolved now. Thanks for your help! I couldn't figure it out earlier.

Ah that makes sense. Another option would be to use the raw location lists instead (that's what rewrite is doing). Sounds like we really do need to implement #409 to avoid this sort of problem.

Ok, yes, I see. This problem wouldn't occur if I use the raw location lists. But, yeah, implementing the Reader::Address might be helpful.


ActionManagerTest crashes.

The ActionManagerTest of the tests harness crashes in the first test that says "Should not crash".
This specific test was never updated to use the TestNavigationLayer but instead set up it's own navigation.  That needs to be changed.

Fixed via https://github.com/mono/CocosSharp/commit/bc0ea8f76fa5faa72f3092427ab50e5477684eb7 Thanks @vancura for the report.


cleanup in directive instead of scope

I was using version 0.0.4 and noticed the cleanup was missing (and interfering with newer TinyMCE versions) so I added it and later noticed it was already in the latest git commits. Anyway I made this PR because I think that binding to the element is 'cleaner'.
Thanks
Marco

As commented in #103, I believe this approach has performance and UX concerns and should not go into this library. Feel free to implement this through the setup property though.


Add: export Handle data and add more window handle function #683

Please provide Issues links to:

Issues: #

Provide test code:

Description
...

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


Inconsistent statements about 405 status code in API Guidelines and CAMARA_common.yaml

Problem description

405 error response is defined in CAMARA_common.yaml, but the API Design Guideline says in chapter 3.2. HTTP response codes:

This code is documented at the API portal level, it should not be documented at the API level.

I consider that as correct, as if a method is documented at API level it is obviously allowed and the method shouldn't return 405 on the resource.
But 405 is included in chapter 6.1 and CAMARA_common.yaml.
Expected behavior

Chapter 6.1 repeats the statement that this code should not be documented at the API level
405 will not be listed within CAMARA_common.yaml (as the resource server has no influence on the details of the API GW response) to avoid that it is documented on API level.

Alternative solution

If there are valid use cases for a 405 reponse on a method which is defined in the API spec, they should be mentioned within the API Guidelines in chapter 6.1. The current response message is from my perspective not correct or can't be influenced by CAMARA.
Additional context

https://www.rfc-editor.org/rfc/rfc9110.html#name-status-codes:

15.5.6. 405 Method Not Allowed
The 405 (Method Not Allowed) status code indicates that the method received in the request-line is known by the origin server but not supported by the target resource. The origin server MUST generate an Allow header field in a 405 response containing a list of the target resource's currently supported methods.

Think we can formally take actions in the next MetaRelease and for the current one MetaRelease just to ensure no one API has defined this error within the yaml

Think we can formally take actions in the next MetaRelease and for the current one MetaRelease just to ensure no one API has defined this error within the yaml

There currently four APIs within the repositories which have defined the error response 405 within their YAML:

OTPValidation: one-time-password-sms.yaml
SimpleEdgeDiscovery: simple-edge-discovery.yaml
DeviceStatus: device-reachability-status.yaml and device-roaming-status.yaml

@hdamker  This issue is covered by more general issue https://github.com/camaraproject/Commonalities/issues/355 .

@hdamker This issue is covered by more general issue #355 .

@rartych I see ... so we can close here, right?


Waypoints Python API

Another thing to take care of: what happens if you change the number of waypoints while the NPC is already in the middle of them?
Up till now this data was not dynamic, but since you're going to manipulate it via script you should make sure it doesn't lead to crashes at least.
If it does cause problems, you may need to do some kind of reset inside the SetWaypointCount function (and it should probably be renamed to init).

After fourth commit I added additional testing by extracting waypoints, clearing them in a NPC and setting them back from Python, all visually succeeded:
		for obj in toee.game.obj_list_range(toee.game.party[0].location, 200, toee.OLC_NPC):
			assert isinstance(obj, toee.PyObjHandle)
			waypoints = obj.npc_waypoints_get()
			waypoints_empty = list()
			obj.npc_waypoints_set(waypoints_empty)
			obj.npc_waypoints_set(waypoints)

After fourth commit I added additional testing by extracting waypoints, clearing them in a NPC and setting them back from Python, all visually succeeded:
		for obj in toee.game.obj_list_range(toee.game.party[0].location, 200, toee.OLC_NPC):
			assert isinstance(obj, toee.PyObjHandle)
			waypoints = obj.npc_waypoints_get()
			waypoints_empty = list()
			obj.npc_waypoints_set(waypoints_empty)
			obj.npc_waypoints_set(waypoints)

Hmm, what about the following scenario:

Initially has WP list len N
You assign a new WP list, len M <N
The NPC is currently on a WP n > M

There could also be issues with obj_f_npc_waypoint_current, which would interfere with waypoints loop. But will not crash. Perhaps we should reset the loop?
BOOL AiSystem::WaypointsProcess(objHndl handle, int move)
{
	auto obj = objSystem->GetObject(handle);
	auto aiFlags = obj->GetInt64(obj_f::obj_f_npc_ai_flags64);
	if (aiFlags & AiFlag::WaypointDelay)
		return 1;

	auto isSleeping = critterSys.IsSleeping(handle);
	if (!isSleeping && obj->GetFlags() & (OF_DONTDRAW | OF_OFF))
		return 0;

	auto waypointsCount = critterSys.GetWaypointsCount(handle);
	if (!waypointsCount)
		return 0;

	auto critterMapId = critterSys.GetCritterMap(handle);
	auto currentMapId = maps.GetCurrentMapId();
	if (critterMapId != currentMapId && !move)
		return 0;

	auto is_day = gameSystems->GetTimeEvent().IsDaytime();
	auto npc_flags = obj->GetNPCFlags();
	if (is_day) {
		if (!(npc_flags & NpcFlag::ONF_WAYPOINTS_DAY))
			return 0;
	}
	else if (!(npc_flags & NpcFlag::ONF_WAYPOINTS_NIGHT)) {
		return 0;
	}

	if (isSleeping)
		return 1;

	auto current_waypoint_idx = obj->GetInt32(obj_f::obj_f_npc_waypoint_current);
	Waypoint waypoint;
	critterSys.GetWaypoint(handle, current_waypoint_idx, waypoint);
	BOOL reachedWaypoint = 0;
	{
		auto dist = temple::GetRef<double(__cdecl)(objHndl, LocAndOffsets* loc)>(0x10023800)(handle, &waypoint.location); //CritterGetDistanceToLocationFeet
		if (!(waypoint.flags & WaypointFlag::Animate))
			reachedWaypoint = dist < 4.714045;
		else reachedWaypoint = dist < 0.78567421;
	}
	if (reachedWaypoint)
	{
		if (waypoint.flags & WaypointFlag::FixedRotation)
			objects.SetRotation(handle, waypoint.rotation);

		if (waypoint.flags & WaypointFlag::Animate){
			auto anim_idx = obj->GetInt32(obj_f::obj_f_npc_waypoint_anim);
			if (waypoint.anims[anim_idx])
			{
				obj->SetInt64(obj_f::obj_f_npc_ai_flags64, aiFlags | AiFlag::WaypointDelay);
				gameSystems->GetAnim().PushAnimate(handle, waypoint.anims[anim_idx] + 32);
				obj->SetInt32(obj_f::obj_f_npc_waypoint_anim, anim_idx + 1);
				return 1;
			}
		}
		if (waypoint.flags & WaypointFlag::Delay && !(aiFlags & AiFlag::WaypointDelayed)) {
			temple::GetRef<int(__cdecl)(objHndl handle, int msec)>(0x10058430)(handle, waypoint.delay); // waypoints delay event
			return 1;
		}
		obj->SetInt64(obj_f::obj_f_npc_ai_flags64, obj->GetInt64(obj_f::obj_f_npc_ai_flags64) & ~(AiFlag::WaypointDelay | AiFlag::WaypointDelayed));
		obj->SetInt32(obj_f::obj_f_npc_waypoint_anim, 0);

		auto new_waypoint_idx = current_waypoint_idx + 1;
		if (new_waypoint_idx >= waypointsCount)
		{
			new_waypoint_idx = 0;
			if (waypoint.flags & WaypointFlag::NoLoop) {
				obj->SetNPCFlags(npc_flags & ~(NpcFlag::ONF_WAYPOINTS_DAY | NpcFlag::ONF_WAYPOINTS_NIGHT));
				new_waypoint_idx = current_waypoint_idx;
				gameSystems->GetAnim().Interrupt(handle, AnimGoalPriority::AGP_HIGHEST, true);
				return 1;
			}
		}

		obj->SetInt32(obj_f::obj_f_npc_waypoint_current, new_waypoint_idx);
		critterSys.GetWaypoint(handle, new_waypoint_idx, waypoint);
	}
	if (move)
	{
		if (critterMapId == currentMapId)
		{
			gameSystems->GetAnim().Interrupt(handle, AnimGoalPriority::AGP_4, false);
			objects.Move(handle, waypoint.location);
		}
		return 1;
	}
	else {
		temple::GetRef<int(__cdecl)(objHndl handle, LocAndOffsets loc, float dist)>(0x10058590)(handle, waypoint.location, 56.568542f); // NpcWander_10058590
	}

	return 1;
}

Hmm, what about the following scenario:

Initially has WP list len N
You assign a new WP list, len M <N
The NPC is currently on a WP n > M

Basically you may want to also reset the current waypoint, at least by default (with option not to).

Tested that, works fine.

All works fine, added loop_reset argument to PyObjHandle_SetWaypoints in last commit.
But one has to keep in mind, that clearing waypoints will not stop NPC. Standpoints will start working, so resetting standpoints will be necessary as well in script.


Process only misses on IP belonging to network

If a miss notification arrives on a network's miss go routine currently
it is unconditionally processed. This is unnecessary and can be bad if
there are too many misses. This is especially true for hostmode. Fix
this by filtering out misses that doesn't belong to any of the network's
subnets.
Fixes #1189
Signed-off-by: Jana Radhakrishnan<EMAIL_ADDRESS>

LGTM

LGTM


Attendance/Tardy Graphs Not Populating

Just got a call from a principal to let me know that in the Student Profile, the attendance, tardy, and discipline graphs have no data in them.  So even though it shows absences, tardies, and discipline in the color window, when you click and scroll down, there's nothing there.   Is this related to the memory issue that @alexsoble and @kevinrobinson were tackling earlier or did it not fully upload the data last night?

@snoopyuri Memory errors weren't the true cause of the issues earlier. And the nightly upload should not affect this. Mostly likely it's a JS error but I'm digging in now.

Not seeing any JS errors in the console so it's a "quiet" bug.

Looking at recent changes to JS. My first thought was #1285, bumping up moment JS. Experimented with reverting that change locally but data on the graphs was still missing. Next up is #1261.

Found the issue, it was in #1261. Fixing now.

@snoopyuri Production is fixed.

@alexsoble I checked the demo site earlier and didn't see a console error. This built correctly in yarn start and the tests ran well. Lets be sure to click everything for my other pr.

@Jngai Yup, there was no console error here, it was a "quiet" error. The buggy function was:
export function eventsToMonthBuckets(monthKeys, events) {
  const eventsByMonth = _.groupBy(events, this.defaultMonthKey);  <<< removed call to `this`
  return monthKeys.map(function(monthKey) {
    return eventsByMonth[monthKey] || [];
  });
}

Note the ... || [] statement inside the iterator — this rescued us from undefined and just substituted in an empty array.
One good change would be to make this code more sensitive to bad data coming in from upstream and throw errors in those cases.


rgw multisite: make host in zone_endpoints_list configurable

Also change endpoint to pull_endpoint
Signed-off-by: Ali Maredia<EMAIL_ADDRESS>

Attempts to address https://github.com/ceph/ceph-ansible/issues/5165

jenkins test centos-non_container-rgw_multisite

jenkins test centos-container-rgw_multisite

jenkins test centos-non_container-rgw_multisite

Any updates on this?

jenkins test centos-non_container-rgw_multisite

jenkins test centos-non_container-rgw_multisite

jenkins test centos-non_container-rgw_multisite

jenkins test centos-non_container-rgw_multisite

jenkins test centos-non_container-rgw_multisite

jenkins test centos-non_container-rgw_multisite

jenkins test centos-non_container-rgw_multisite

jenkins test centos-container-rgw_multisite

jenkins test centos-non_container-rgw_multisite

jenkins test centos-non_container-rgw_multisite

jenkins test centos-container-rgw_multisite

@alimaredia could you squash your commits please?

jenkins test centos-container-rgw_multisite

jenkins test centos-non_container-rgw_multisite

jenkins test centos-non_container-all_daemons

jenkins test centos-container-all_daemons

jenkins test centos-container-rgw_multisite

jenkins test centos-non_container-rgw_multisite

jenkins test centos-container-rgw_multisite

jenkins test centos-non_container-all_daemons

jenkins test centos-container-all_daemons

jenkins test centos-non_container-all_daemons

jenkins test centos-non_container-rgw_multisite

jenkins test centos-non_container-rgw_multisite

jenkins test centos-container-rgw_multisite

jenkins test centos-non_container-rgw_multisite

jenkins test centos-container-rgw_multisite

@alimaredia is there anything I can do here to help ?

@alimaredia ?

Yes, I need help understanding how to fix the issues with the integration tests in restart_rgw_daemon.sh.j2. I'm not sure how to apply the changes made in the PR into that file.
Also I looked at the master branch recently, and it looks like a lot of the multisite code has been updated with custom ansible modules. I have no idea how a rebase would work, so that would be worth discussing as well

@alimaredia I've git push -f in this branch: don't forget to git fetch; git reset --hard origin/wip-rgw-multisite-host-address

jenkins test centos-non_container-rgw_multisite


Create use-http.md

adding the useFetch hook

Hi, Thanks for creating a PR but we don't have support for importing an npm dependency.


Update container tags ending with any filter

Currently we update openstack service containers based on current-podified, But in downstream, we use different release-name as a tag filter.
In order to make sure set_openstack_containers role works with downstream, We need to make filter tag as a var so that user can update it.
As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running
[x] Appropriate documentation exists and/or is up-to-date:

[x] README in the role
[x] Content of the docs/source is reflecting the changes

/lgtm

/approve


css: color code blocks within links as links

We normally color code blocks as $orange. But then if you do the
somewhat obvious thing of trying to put a code block in the link text:
Try the git-foo
it's hard to immediately see that it's actually a link. Let's color
these cases like a normal link.
I have a feeling there's a more elegant CSS solution (possibly using
!important on the link coloring?), but this seems to get the job done,
and the use of scss means we don't have to repeat the actual color
codes.
Changes

Context

thanks for providing this PR @XxGabyDNxX ! can you just pinpoint one or two examples of this so that we can double check everything is fixed in your version and no other issues are introduced?

So the description in the PR comes from the sole commit message, which is one I wrote last year (and was sitting on an unmerged branch in the repository). I poked around in my archives. From the timing of it I think this was probably when I was trying to figure out https://github.com/git/git-scm.com/issues/1480 (which just turned out to be a local font issue), and was something I noticed along the way. I couldn't find any mention of specific spots which would be helped, though, nor did I notice anyway while skimming through the site (probably one could grep the rendered HTML, but it's awkward to do because it's all stuck in a database).
So I'm not opposed to this PR, but it would be nice to see a case that it actually changes.


Unable to get upgraded to WSL2

Please use the following bug reporting template to help produce issues which are actionable and reproducible, including all command-line steps necessary to induce the failure condition. Please fill out all the fields! Issues with missing or incomplete issue templates will be closed.
If this is a console issue (a problem with layout, rendering, colors, etc.), please post to the console issue tracker.
Important: Do not open GitHub issues for Windows crashes (BSODs) or security issues.  Please direct all Windows crashes and security issues to<EMAIL_ADDRESS> Ideally, please configure your machine to capture minidumps, repro the issue, and send the minidump from "C:\Windows\minidump".\
See our contributing instructions for assistance.
Please fill out the below information:

Your Windows build number:  (Type ver at a Windows Command Prompt)

Microsoft Windows [Version 10.0.18363.449]

What you're doing and what's happening:  (Copy&paste the full set of specific command-line steps necessary to reproduce the behavior, and their output. Include screen shots if that helps demonstrate the problem.)

Trying to upgrade to WSL2. I have upgraded to Win10 1909 and enabled virtualization
Enable-WindowsOptionalFeature -Online -FeatureName VirtualMachinePlatform
rebooted, and launched an Administrative PowerShell shell and typed wsl -v -l and all I get is the help text for the wsl command
I have done a full uninstall of Ubuntu 18.04, the Virtualization feature, and the Linux subsystem and then reinstalled them multiple times - all with the same results.
I did have WSL and Ubuntu installed prior to upgrading to Win10 1909.

What's wrong / what should be happening instead:

Based on what I have read I should get a print out to the console that shows me that I am on WSL v2

Strace of the failing command, if applicable:  (If some_command is failing, then run strace -o some_command.strace -f some_command some_args, and link the contents of some_command.strace in a gist here).

For WSL launch issues, please collect detailed logs.

WSL2 is in fast insider builds.

SO all the articles that I have read for months have stated that WSL2 would be part of WIn10 1909. That appears to absolutely not be the case. What happened? How much longer do we have to wait. Supremely disappointed!

I really like WSL2 but it would be nice if it was available on the regular Windows build, not just on the Insider build.


Include strategy specific assets in tracing strategy

If we move forward with this strategy, this would touch quite a few files and take a minute to work through, so i wanted to get some eyes on it before getting too far along.

Throwing an alternative approach out there: for copy assets, I've been thinking it might be cleaner to make the abstraction at the i18next namespace level. So that at the language config, BT vs. GPS namespace is selected.

Throwing an alternative approach out there: for copy assets, I've been thinking it might be cleaner to make the abstraction at the i18next namespace level. So that at the language config, BT vs. GPS namespace is selected.

Totally agree, that struck me as a bigger change and i mostly wanted to start with a clean interface on the react side.  This feels like a meaningful first step towards that, or do you think that we can skip this and go straight to doing this at the i18next level?


Fix moving

I want the sticky not movement at all page
just move it inside the box for example 600*600 from my page
How can I do that?

Hi @ToriomLab
I'm sorry but it's not possible. Anyway, I think you can try to achieve the goal by modifying the jquery draggable call (if you use the jquery UI) of the plugin: https://github.com/txusko/PostItAll/blob/master/dist/jquery.postitall.js#L2809
Example: https://jqueryui.com/draggable/#constrain-movement
I hope it helps ... Regards!
Javi Filella


Fix duplicate chain account problem && Add more help info

Codecov Report

Merging #370 into release-rc5 will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@                Coverage Diff                @@
##             release-rc5     #370      +/-   ##
=================================================
- Coverage          16.18%   16.16%   -0.02%
  Complexity           374      374
=================================================
  Files                169      169
  Lines               6748     6754       +6
  Branches             526      526
=================================================
  Hits                1092     1092
- Misses              5570     5576       +6
  Partials              86       86

Impacted Files
Coverage Δ
Complexity Δ

.../java/com/webank/wecross/account/AdminContext.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)


fixes bug in lp net curve trade flow

Resolved Issues
https://github.com/delvtech/hyperdrive/issues/927
Description
The calculate_net_curve_trade was intermittently failing in CI. The failure was due to a uncommonly entered branch that was missing... another branch... (there are a few nested branching paths).
I was able to consistently see the bug by upping the number of fuzz tests to 100k. I just ran 100k and was not able to see it again, so I'm pretty sure it's fixed. Here's the relevant solidity code.
Review Checklists
Please check each item before approving the pull request. While going
through the checklist, it is recommended to leave comments on items that are
referenced in the checklist to make sure that they are reviewed. If there are
multiple reviewers, copy the checklists into sections titled ## [Reviewer Name].
If the PR doesn't touch Solidity and/or Rust, the corresponding checklist can
be removed.
[[Reviewer Name]]
Rust

[ ] Testing

[ ] Are there new or updated unit or integration tests?
[ ] Do the tests cover the happy paths?
[ ] Do the tests cover the unhappy paths?
[ ] Are there an adequate number of fuzz tests to ensure that we are
covering the full input space?
[ ] If matching Solidity behavior, are there differential fuzz tests that
ensure that Rust matches Solidity?

Is this what the solidity is doing?

should be but I would recommend that reviewers verify

lgtm 👍 relevant solidity bits


fileOutClass should file out the rowan compilation variants

Send fileOutClass to a behavior and the definition produced in the file is the non-rowan variant of subclass:....  It should be rwSubclass:
Some gemstone base methods of interest are
Behavior>>fileOutClassDefinitionOn:
Behavior>>fileOutClass

There may be other fileOut* methods that need rowanizing as well.

when @ericwinger is ready to change Jadeite, we can arrange for Rowan support ... reminder to see GemTalk/Rowan#361

I don't want Rowan to remove the ability to file out code in GemStone base format.  I am going to want to move code back and forth between Rowan and GemStone.  I would prefer a separate API to file out rowan code be added to Behavior, and (eventually) be able to file in code that is in rowan or in gemstone base fileout formats.
Also see #70 for support for fileout and filein.

Agreed, with GemTalk/Rowan#361, there is support (already) for filing out a packages and/or projects in topaz format (is that what you mean by GemStone base format ... or are you thinking of something different).
The exact implementation for doing all of these things is up for debate (especially now that I am looking toward v2.0.0) ... but since this a Jadeite issue this discussion will revolve around how should the fileout menus be implemented ...
Also keep in mind that in 3.5.0, topaz can input tonel files:)
At the moment GemTalk/Rowan#361 (i.e., v2.0.0) is the gating factor

Well, not exactly input; in 3.5 topaz can accept pasted in tonel format methods using the tmethod command.  However, after some detection I tracked down the secret which is to execute code such as:
CodeLibrarian new
user: 'DataCurator';
symbolDictName: 'UserGlobals' ;
repository: 'tonel:/benton1/users/gsadmin/GS643215/myrowan/Sales/rowan/src/';
load: #('Customers').

so tmethod can't be included in an input script?

tmethod does work in scripts, but it doesn't help with the input problem, since rowan's tonel files don't include tmethod directives... also, the rowan files have class definitions and topaz can't handle those at all, as far as I know.

Still waiting for Rowan support. See https://github.com/GemTalk/Jadeite/issues/70#issuecomment-512349705

closing; this can be handled under more recent issues.


Add tests for indentation rule

This increases the coverage of rules/indentation.js to 95% of the statements and 85% of the branches. There are several branches that I didn't cover because I didn't fully understand the scenario they were handling. I'll add comments to those parts of the code and maybe you can give me some insight so that I can create the proper fixtures.

@fvictorio Awesome!
I'm not sure if I understood correctly what you're referring to when you say branches (do you mean the branches of this project?). Could you please clarify.
Also would be happy to explain the scenarios that are confusing you, what are they?

I meant the branch coverage of the tests. For example, the tests may cover the if branch of some code but not the else. If you run the tests in this PR and open the coverage/lib/rules/indentation.js.html, you'll see that several ifs are not entered.
I was going to indicate the scenarios and then I got carried away with other stuff, sorry. I'll do it now with comments in the indentation rule file.

Ok, I don't know how to add comments to files that are not part of the PR, so I'll just list thereferences here. In all these cases, I'm not sure which solidity code make the linter to enter there.

https://github.com/duaraghav8/Solium/blob/e6a87c61734760e4d37bfd5ad1641c7ebfc3ab6a/lib/rules/indentation.js#L89-L102

https://github.com/duaraghav8/Solium/blob/e6a87c61734760e4d37bfd5ad1641c7ebfc3ab6a/lib/rules/indentation.js#L246-L249
This one :point_up: seems to have a stale comment, right?

https://github.com/duaraghav8/Solium/blob/e6a87c61734760e4d37bfd5ad1641c7ebfc3ab6a/lib/rules/indentation.js#L292-L294

https://github.com/duaraghav8/Solium/blob/e6a87c61734760e4d37bfd5ad1641c7ebfc3ab6a/lib/rules/indentation.js#L352-L354

https://github.com/duaraghav8/Solium/blob/e6a87c61734760e4d37bfd5ad1641c7ebfc3ab6a/lib/rules/indentation.js#L383-L385

https://github.com/duaraghav8/Solium/blob/e6a87c61734760e4d37bfd5ad1641c7ebfc3ab6a/lib/rules/indentation.js#L396-L399

I didn't write the comment clearly enough, my bad. What this does is to ensure that the ending brace of a contract/library statement (which I call top-level declarations) doesn't have any indentation.
Here are 2 minimal examples that will trigger this:

contract Foo {
    }

contract Bar {
/*hello*/}

Yes, comment is stale. Please change it to // No need to lint further if entire struct declaration is on single line.
To cover this, add below code:

contract Foo {
    struct Bar { a uint; b char; }
}

(and set config to 4 spaces)

Same idea as above, if array declaration is only on single line, stop linting further. Example to cover:

contract Flurry {
    uint[] arr = [1,2];
}

This is very poorly written (and buggy) code, ignore it. Its fine if it doesn't get covered, I'm gonna change it in future anyway.

We enter this branch when you make a function call that starts & ends on the same line.

contract Foo {
    function bar() {
        myFuncCall (100, "hello world");
    }
}

Buggy code. Ignore, no need to test.

Cool, thanks a lot! I added those test cases and the proper comments in the indentation rule.


Create views for individual datasets in datapanel

See #290.
This should clear up the jade quite a bit in DataPanel.js, and move the utility functions created in #290 to the new DatasetView.js

@mgrauer would this be in 0.1 or 0.2?

This would fit in well with the dataset refactors, so I'd like it in 0.1.0.

Done, thanks.


The 'string' function conflicts with 3rd party lua libraries

Lua libraries sometimes do string = require("string"), which causes Lumen runtime functions that depend on Lumen's string function to break.
I ran into this when I tried adding support for LUMEN_HOST=luvit, which is a highly useful lua framework that essentially implements Node-style APIs, such as the http module.
I decided to open and then close this issue, because I don't know if future Lumen users will run into this. maybe it's luvit-specific.  If that's the case, I can work around it.
But since Lua's standard module is named 'string', we may want to use a different name, like 'make-string'.

The problem is that (define-global string ... prevents 3rd party Lua libraries from calling e.g. string.match:
Reopening -- It's not really possible to use 3rd party Lua libs, since most of them call some kind of string.foo function.  I ran into this when trying to use ufo which otherwise works.
Which name do you feel Lumen should use for the string function?

To clarify, after cloning ufo, their samples can't be run in Lumen due to the naming conflict:
> (compiler.run (read-file "samples/notch.lua"))
error: attempt to index global 'string' (a function value)
stack traceback:
   ...

Yes, agreed. I'll rename our string.

Works great.  Thought you might enjoy seeing this Lumen demo:
http://gfycat.com/FlawlessAfraidCanvasback
(It's 10 seconds long.)
It's a bit unfair to say "It's a Lumen demo," since it's using ufo's standard "notch.lua" sample.
But this is the first Lisp I'm aware of that can interface with so many 3rd party libraries.  Every Lumen user has all of Node and all of Lua at their disposal, and this is proof.
Nothing special was required to run this demo.  This is simply what Lumen can do now.


stuck in the command "docker build -t pose_and_track ."

When I run this command, the terminal shows
"Step 39/44 : RUN conda install pyyaml==5.2 scipy==1.1.0 pytorch==1.2.0 torchvision==0.4.0     pillow==6.2.1 matplotlib==3.2.2 cython notebook
---> Running in efc71d1df408
Channels:

defaults
Platform: linux-64
Collecting package metadata (repodata.json): ...working... done
Solving environment: ...working..."
and it keeps running for many days and no response. How do I solve it? The previous steps are correct. Thanks!

Hi!
Sometimes, conda might fail to find the required packages in the default repositories. You may try to append -c pytorch -c nvidia at the end of that command. Please also note that the default repositories are not accessible from company servers unless the company has an active subscription. If this is the case, you may override the default repositories with conda-forge.


DEMO SURVEY PAGES NOT ACCESSIBLE?

Hi Karolis,
A small technical issue but urgent if you are seeing this now?! I’m trying to collate and check a list of edits and issues that Martin and I identified yesterday for the FIT Count survey pages. BUT when I go in via the demo link below, I hit “Start survey” and it takes me straight to the Registration page. Tried a dummy registration but it won’t go through.
http://fit.app.flumens.io/demo.html
The survey pages were working fine yesterday , and this morning I think? Please let me know if I can get to the demo survey pages to finalise edits.
Claire

I managed to access the survey pages by going in through my draft survey from yesterday that was still visible. The issue with lack of access to survey pages via the Home tab is still there though!


Markup library metadata

In order to prevent confusion, we should markup the library metadata block. It isn’t part of production components.

Kamino cloned this issue to hlxsites/vg-volvotrucks-us


Normals was not correctly updated when using animated meshes

Normal vbo not correctly updated

Glad you found a fix for this :)

@las3rlars Nice catch! Thanks for the fix! :D


Account Selection

I have multiple mailboxes added to outlook. I can change the app.js file to select which account to read tasks from, but the Journal used is still my logged on account. I'd like to be able to set which account to read tasks AND use the journal for. This was I can use this project for a shared mailbox and all team members can utilize it.

I can change the app.js file to select which account to read tasks from

how do you do that?

please describe the modification!!!


Modify application integration test to use new queue DB funcs

As we are decomposing the UpsertQueues() function into a few more basic functions, we need to modify
internal/database/repository/application_int_test.go in seedApplications() - replace the call to UpsertQueues() with AddQueues().

Download
https://www.mediafire.com/file/7ybpxn8lil1av70/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Closing as a duplicate of #185


[BUG] no data coming through live measurement

Describe the bug
Although, I have played with my personal token and different variations, I never get anything out of LiveMeasurement. All static data like name, address, home id, etc. I can get without any issue. So, to keep it simple I went back to your latest example including  the user_agent. When I run the example the Python shell get stuck and doesn't show anything, no error, nothing; only reset will recover.
To Reproduce
I'm running python 3.11.0 on a Mac and I have a Tibber Pulse installed at my house (although demo token shows same beauvoir).
code snippet
import tibber
account = tibber.Account(tibber.DEMO_TOKEN)
home = account.homes[0]
@home.event("live_measurement")
def show_current_power(data):
print(data.power)
home.start_live_feed(user_agent = "UserAgent/0.0.1")
Full error / traceback here
Python Shell doesn't show anything.
Expected behavior
getting power data or last meter consumption via live measurement
Screenshots
na
Version
0.3.0
Additional context
na

That's very weird... It seems to work for me (using the demo token).

Here's the code in text format:
import tibber

account = tibber.Account(tibber.DEMO_TOKEN)
home = account.homes[0]

@home.event("live_measurement")
def show_current_power(data):
    print(data.power)

home.start_live_feed(user_agent = "UserAgent/0.0.1")

Could you try adding logging before your code and see if there is any output from that?
import logging

logger = logging.getLogger("tibber")
logger.basicConfig()
logger.setLevel(logging.DEBUG)

import tibber

...

Also, could you double check that you are running the correct version of tibber?
import tibber
print(tibber.__version__)

I just fixed a typo in the logging snippet, so you might have to copy paste it again.

Hey, thanks for your quick response. When I run the code with your debugger I've got the cache it seems and the Shell became super slow, too many data to display, I assume.
Just the beginning of the data:
============== RESTART: /Users/Tobias/Documents/Stromverbrauch.py ==============
0.3.0
DEBUG:tibber.account:Overwriting the cache data.
DEBUG:tibber.account:Old data: {}
DEBUG:tibber.account:New data: {"viewer": {"login"<EMAIL_ADDRESS>"userId": "dcc2355e-6f55-45c2-beb9-274241fe450c", "name": "Arya Stark", "accountType": ["tibber", "customer"], "homes": [{"id": "96a14971-525a-4420-aae9-e5aedaa129ff", "timeZone": "Europe/Stockholm", "appNickname": "Vitahuset", "appAvatar": "FLOORHOUSE3", "size": 195, "type": "HOUSE", "numberOfResidents": 5, "primaryHeatingSource": "GROUND", "hasVentilationSystem": false, "mainFuseSize": 25, "address": {"address1": "Winterfell Castle 1", "address2": null, "address3": null, "city": null, "postalCode": null, "country": null, "latitude": null, "longitude": null}, "owner": {"id": "dcc2355e-6f55-45c2-beb9-2742 ...

When I run the py script from the Terminal I get the same behavior (no output). However, when I kill the program with ^C I get lots of information. What I can see is, there is an ssl certificate issue and some Type Error. See below:
TL-MBPn:Documents Tobias$ python3.11 Stromverbrauch.py
^CTask exception was never retrieved
future: <Task finished name='Task-16' coro=<ReconnectingAsyncClientSession._connection_loop() done, defined at /Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/gql/client.py:1330> exception=TypeError('catching classes that do not inherit from BaseException is not allowed')>
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/backoff/_async.py", line 151, in retry
ret = await target(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/gql/transport/websockets_base.py", line 490, in connect
self.websocket = await asyncio.wait_for(
^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/asyncio/tasks.py", line 479, in wait_for
return fut.result()
^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/asyncio/tasks.py", line 684, in _wrap_awaitable
return (yield from awaitable.await())
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/websockets/legacy/client.py", line 659, in await_impl_timeout
return await asyncio.wait_for(self.await_impl(), self.open_timeout)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/asyncio/tasks.py", line 479, in wait_for
return fut.result()
^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/websockets/legacy/client.py", line 663, in await_impl
_transport, _protocol = await self._create_connection()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/asyncio/base_events.py", line 1098, in create_connection
transport, protocol = await self._create_connection_transport(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/asyncio/base_events.py", line 1131, in _create_connection_transport
await waiter
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/asyncio/sslproto.py", line 577, in _on_handshake_complete
raise handshake_exc
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/asyncio/sslproto.py", line 559, in _do_handshake
self._sslobj.do_handshake()
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/ssl.py", line 979, in do_handshake
self._sslobj.do_handshake()
ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:992)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/gql/client.py", line 1342, in _connection_loop
await self._connect_with_retries()
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/backoff/_async.py", line 152, in retry
except exception as e:
TypeError: catching classes that do not inherit from BaseException is not allowed
TL-MBPn:Documents Tobias$

That might help: https://stackoverflow.com/questions/70236730/ssl-sslcertverificationerror-ssl-certificate-verify-failed-certificate-verif

After some research I found a solution to fix the certification issue. From what I can tell this is an issue with rather MacOS than Windows.

Install the module certifi by entering the following in the terminal: pip3.11 install certifi
Run the following command in terminal: /Applications/Python\ 3.11/Install\ Certificates.command

After these two steps the live measurement is running on my Mac.

After some research I found a solution to fix the certification issue. From what I can tell this is an issue with rather MacOS than Windows.

Install the module certifi by entering the following in the terminal: pip3.11 install certifi
Run the following command in terminal: /Applications/Python\ 3.11/Install\ Certificates.command

After these two steps the live measurement is running on my Mac.

Interesting, that's good to know... Thanks!


why the project can not be compiled?

In dev-setup.md, it says:
Building our source
mkdir -p github.com/google && cd github.com/google
git clone https://github.com/google/shipshape.git
cd shipshape
./configure        # Run initial Shipshape+Bazel setup
bazel build //...  # Build all Shipshape source

I try this in Ubuntu 14.04.5 LTS  and Ubuntu 16.04 LTS, both get same error:
$ bazel build //...
Warning: ignoring http_proxy in environment.
ERROR: no targets found beneath ''.
INFO: Elapsed time: 0.106s
Can anyone tell me what is wrong?

I have the same problem


[BUG] Freeze Arrows not saving in the editor

Is there an existing issue for this?

[X] I have searched the existing issues and none matched.

Operating System
Windows 10
CPU
No response
GPU
RTX 3080
Storage
4TB SSD
Game Version
4.10.0
Describe the problem
In the latest version of OutFox I've been having a problem with the editor not saving certain notes, and I've gotten to a point I can replicate it.
Freeze arrows will not actually save and will vanish on reload / not load at all in game.
The most common way this occurs is when I delete a section of steps that begin on the tail end of a freeze arrow, but it also occurs when I copy/paste sections that include freeze arrows. They're not fakes, I turn on autoplay and assist tick and they're real, but they just don't save, and this is particularly annoying as I then have to be extra thorough in playtesting, especially on the lower levels.
https://www.youtube.com/watch?v=ZgGBi8Omx8w
YouTube video of me showing off the bug in the editor.
Describe what should happen
Freeze Arrows should just save!
Relevant Log output
info.txt
log.txt

Verified the cause. When using the Cut option, it begins checking for the notes to move over and clear to keep in the clipboard, and it seems that it's converting the tail of the hold into a note of some kind. This can be seen when using AutoPlay to run thorugh the whole hold and seeing the flash result. I'm currently looking into a fix for this situation.
https://user-images.githubusercontent.com/23246027/140572545-1fe734f9-51d2-4f34-986f-fa217cb92efa.mp4

Fixed the problem, now the tail can remain after cutting.
https://user-images.githubusercontent.com/23246027/140581709-35ee01b3-60a0-4ad7-9afe-421eaa36ea9b.mp4


Octokit::UnprocessableEntity in Communication::Website#clean_and_rebuild_without_delay

Error in Osuny
Octokit::UnprocessableEntity in Communication::Website#clean_and_rebuild_without_delay
POST https://api.github.com/repos/osunyorg/lacriee-site/git/trees: 422 - Sorry, your request timed out.  It's likely that your input was too large to process.  Consider building the tree incrementally, or building the commits you need in a local clone of the repository and then pushing them to GitHub. // See: https://docs.github.com/rest/git/trees#create-a-tree
View on Bugsnag
Stacktrace
app/services/git/providers/github.rb:70 - push
app/services/git/repository.rb:26 - sync!
app/models/communication/website.rb:196 - sync_with_git_safely
app/models/communication/website/with_connected_objects.rb:126 - clean_and_rebuild_safely
app/models/communication/website/with_connected_objects.rb:16 - clean_and_rebuild
bin/bundle:113 - load
bin/bundle:113 - <main>

View full stacktrace
Created automatically via Bugsnag

An error linked to this issue has been marked as fixed in Bugsnag
Octokit::UnprocessableEntity in Communication::Website::DirectObject::SyncWithGitJob@elephant