python3Packages.mindsdb-evaluator: init at 0.0.11

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 258630 run on x86_64-linux 1

4 packages built:

python310Packages.mindsdb-evaluator
python310Packages.mindsdb-evaluator.dist
python311Packages.mindsdb-evaluator
python311Packages.mindsdb-evaluator.dist

https://github.com/NixOS/nixpkgs/issues/247089


Fan speed percentage value fluctuating

Hey there,
first of all thank you for this awesome addition to HA I LOVE it!
One thing I have recognized with a 69 Pro is that the fan speed variable is fluctuating. I have set the thing to 1 speed value per degree so that it automatically switches between speeds on the controller. HA should just record these settings. The controller itself is always only on 10%-20% at these time but strangtely jumps on the speed percentage reading up to 70% and then back to the "real" value.
On the controller display the fan speed is not going up to 7 or 70%, very strange:

I have made a workaround template, only when it got to 70 and the temperature is on 30 °C the 70% will be displayed  (as set up in my controller settings) otherwise it will not read the value. But would be really awesome to find out what or why it got the wrong values in first place...
{% if state_attr('fan.controller_69_pro_fan','percentage')|int(0) == 70 %}
  {% if states('sensor.controller_69_pro_temperature')|int(0) == 30 %}
    {{ state_attr('fan.controller_69_pro_fan','percentage')|int(0) }}
  {% else %}
    {{ none }}
  {% endif %}
{% else %}
  {{ state_attr('fan.controller_69_pro_fan','percentage')|int(0) }}
{% endif %}

Have written with AC Infinity support because the values are also fluctuating on the display itself. They told me to send them a screenshot. After I send them they say simply that fluctuations will not be harmful and that was it. Would be really awesome if someone could crack this device open for an opensource firmware alternative, because the stock firmware it is absolutely utter sh...


Fix bad chunking for parametric quantile

Pull Request Checklist:

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

This PR fixes #xyz

[ ] (If applicable) Documentation has been added / updated (for bug fixes / features).
[x] (If applicable) Tests have been added.
[ ] CHANGELOG.rst has been updated (with summary of main changes).

[ ] Link to issue (:issue:number) and pull request (:pull:number) has been added.

What kind of change does this PR introduce?

Either apply_ufunc has become more picky, or something changed in the behavior of the output of fit. Multiple functions failed because there were chunks on the dparams dimension.

Does this PR introduce a breaking change?

No.

Other information:

@RondeauG The latest dask is causing issues with chunking for some reason. We may need to pin to the previous version if we can't figure it out.


RPM has conflicts on Fedora 34

Describe the bug
Fedora made a change in Fedora 27 to allow for multiple debug info packages to be installed simultaneously.  The /usr/lib/.build-id directory contains this data.  If a package includes binaries, like Lens, the build-id files will be generated for those binaries.  I also have Bitwarden installed on my system, and like Lens, it installs some of the same binaries as Lens.
% sudo rpm -qlp /tmp/Lens-5.4.5-latest.20220405.1.x86_64.rpm | grep '\.so'
/opt/Lens/libEGL.so
/opt/Lens/libGLESv2.so
/opt/Lens/libffmpeg.so
/opt/Lens/libvk_swiftshader.so
/opt/Lens/libvulkan.so.1
/opt/Lens/swiftshader/libEGL.so
/opt/Lens/swiftshader/libGLESv2.so
% sudo rpm -ql bitwarden | grep '\.so'
/opt/Bitwarden/libEGL.so
/opt/Bitwarden/libGLESv2.so
/opt/Bitwarden/libffmpeg.so
/opt/Bitwarden/libvk_swiftshader.so
/opt/Bitwarden/libvulkan.so.1
/opt/Bitwarden/swiftshader/libEGL.so
/opt/Bitwarden/swiftshader/libGLESv2.so

The problem is that Bitwarden and Lens can't be installed simultaneously as each one believes it should be the owner of the associated build-id files.
Running transaction check
Transaction check succeeded.
Running transaction test
Error: Transaction test error:
  file /usr/lib/.build-id/0e/1c79d2f40b01d3060f45cc9b305c6964c474dd from install of lens-5.4.5_latest.20220405.1-20220405.1.x86_64 conflicts with file from package bitwarden-1.30.0-1.x86_64
  file /usr/lib/.build-id/2c/0a7b75d5cded85430afc78e219f4a989536e17 from install of lens-5.4.5_latest.20220405.1-20220405.1.x86_64 conflicts with file from package bitwarden-1.30.0-1.x86_64
  file /usr/lib/.build-id/42/905812537dc2bcadee5dd60a163f2e4d6594da from install of lens-5.4.5_latest.20220405.1-20220405.1.x86_64 conflicts with file from package bitwarden-1.30.0-1.x86_64
  file /usr/lib/.build-id/bd/3fe8596784eb351e692f2398dd9592066728df from install of lens-5.4.5_latest.20220405.1-20220405.1.x86_64 conflicts with file from package bitwarden-1.30.0-1.x86_64
  file /usr/lib/.build-id/cb/97168ca6fd1390463b6ca6ac35f49f3f240852 from install of lens-5.4.5_latest.20220405.1-20220405.1.x86_64 conflicts with file from package bitwarden-1.30.0-1.x86_64
  file /usr/lib/.build-id/f8/c30596c6161d7ed17ce639e5bcf4918c93bf9a from install of lens-5.4.5_latest.20220405.1-20220405.1.x86_64 conflicts with file from package bitwarden-1.30.0-1.x86_64
  file /usr/lib/.build-id/fc/fecbfa9b4a318c9a1d6000357b92f3832de7a6 from install of lens-5.4.5_latest.20220405.1-20220405.1.x86_64 conflicts with file from package bitwarden-1.30.0-1.x86_64

The issue can be fixed by placing the directive %define _build_id_links none to the specfile (according to https://github.com/pavlobu/deskreen/issues/129).
To Reproduce
Steps to reproduce the behavior:

Install Bitwarden 1.30.0 on a Fedora system
Attempt to install Lens 5.4.5

Expected behavior
Both packages install.
Environment (please complete the following information):

Lens Version: 5.4.5
OS: Linux - Fedora 34
Installation method (e.g. snap or AppImage in Linux): RPM

Interesting, we have already removed the build_id metadata in 5.4.0 https://github.com/lensapp/lens/pull/4464
I assume you downloaded 5.4.5 from the website.

Yes.  I just retried with Lens-5.4.6-latest.20220421.1.x86_64.rpm downloaded directly from https://k8slens.dev/ with what appear to be identical results from before
Error: Transaction test error:
  file /usr/lib/.build-id/0e/1c79d2f40b01d3060f45cc9b305c6964c474dd from install of lens-5.4.6_latest.20220421.1-20220421.1.x86_64 conflicts with file from package bitwarden-1.30.0-1.x86_64
  file /usr/lib/.build-id/2c/0a7b75d5cded85430afc78e219f4a989536e17 from install of lens-5.4.6_latest.20220421.1-20220421.1.x86_64 conflicts with file from package bitwarden-1.30.0-1.x86_64
  file /usr/lib/.build-id/42/905812537dc2bcadee5dd60a163f2e4d6594da from install of lens-5.4.6_latest.20220421.1-20220421.1.x86_64 conflicts with file from package bitwarden-1.30.0-1.x86_64
  file /usr/lib/.build-id/bd/3fe8596784eb351e692f2398dd9592066728df from install of lens-5.4.6_latest.20220421.1-20220421.1.x86_64 conflicts with file from package bitwarden-1.30.0-1.x86_64
  file /usr/lib/.build-id/cb/97168ca6fd1390463b6ca6ac35f49f3f240852 from install of lens-5.4.6_latest.20220421.1-20220421.1.x86_64 conflicts with file from package bitwarden-1.30.0-1.x86_64
  file /usr/lib/.build-id/f8/c30596c6161d7ed17ce639e5bcf4918c93bf9a from install of lens-5.4.6_latest.20220421.1-20220421.1.x86_64 conflicts with file from package bitwarden-1.30.0-1.x86_64
  file /usr/lib/.build-id/fc/fecbfa9b4a318c9a1d6000357b92f3832de7a6 from install of lens-5.4.6_latest.20220421.1-20220421.1.x86_64 conflicts with file from package bitwarden-1.30.0-1.x86_64

As of Fedora 35, lens-5.5.4_latest.20220609.2-20220609.2.x86_64, and bitwarden-1.33.0-1.x86_64 there is no longer a conflict.


Is there existing support to add a custom field to the alert added in elastalert_status index

The alert in elastalert_status index (_type=elastalert) looks like this:

"_source": {
"alert_info": {
"type": "jira"
},
"@timestamp": "2017-09-20T09:26:47.121032Z",
"alert_sent": true,
"match_body": {
"_type": "test",
"code": 1,
"num_hits": 477,
"@timestamp": "2017-09-20T09:20:01.041Z",
"_index": "test_index",
"num_matches": 477,
"_id": "AV6elLvSHurQbYZERFRv"
},
"rule_name": "Rule 1",
"alert_time": "2017-09-20T09:26:11.462653Z"
}

Is there a way to add another field (e.g app_name) to the source?

"_source": {
"alert_info": {
"type": "jira"
},
"@timestamp": "2017-09-20T09:26:47.121032Z",
"alert_sent": true,
"match_body": {
"_type": "test",
"code": 1,
"num_hits": 477,
"@timestamp": "2017-09-20T09:20:01.041Z",
"_index": "test_index",
"num_matches": 477,
"_id": "AV6elLvSHurQbYZERFRv"
},
"rule_name": "Rule 1",
"alert_time": "2017-09-20T09:26:11.462653Z",
"app_name":"My App"
}

@wittyameta @Qmando  I am looking for same use-case. Can you please help me on this?

@mukeshchouhan You can try and update the elastalert.py file. Look at the get_alert_body method and update it to add any custom field.

Thanks @wittyameta  for your quick response.
Do i need to recreate index if i modify elastalert.py?

@wittyameta As you suggested, I am able to add the custom field through 'get_alert_body' and assign its constant value from the Rule file.
But I want to get its value from one of the key of matched record (which i will specify in Rule file). Can you help me which function do i need to modify for that?

I'm trying to create a custom jira field called status that will alert if a server goes down or back up. I created a custom field in a jira issue called status. Can you provide an example of where and how the syntax will need to be to setup a rule to be able to update that. Would I just enter the status variable into the get_alert_body method and then set it in the rule? I've been trying to do something like that, but the field never updates whereas I can update the jira workflow (in progress, closed, etc.).

https://elastalert.readthedocs.io/en/latest/ruletypes.html#jira
See section on Arbitrary Jira fields

At some point it looks like jira-python moved updating to a singular .update call.
It doesn't look like there are any references in the a alert.py file for updating to Jira as there are no ".update" calls that I could find.
Am I missing something or will I need to do development work to add updating functionality.

Oh, you want to be able to transition issues. Sorry, I misread!
Try jira_transition_to

When you say "whereas I can update the jira workflow (in progress, closed, etc.)do you mean using elastalert? Elastalert supports transitioning issues viajira_transition_to. I've always know that as THE status` field. If that's not what you need, you would probably have to add the functionality yourself.


use standard script names

use build, test, start, watch as standard names for tasks

Opened the issues page to say this. :+1:


Feature: Mapsui for WindowsForms

Adds Windows Forms support (Exploratory pull request to see if the Build works).
Known Bugs

[x] There is an error when the app is started.
[x] If you rotate the map via the Navigator, then the rotation track bar isn't updated.

@pauldendulk can you review the changes?

Things done:

Added Getting startet.
Added nuget to Readme

Thanks @inforithmics and @charlenni

@pauldendulk Will there be a nuget package available for this?

Yes, with the release of 5.0.0-beta.3.


File browser only showing image files

Hi,
I'm having problems opening html and javascript files in the otherwise excellent paid version of acode. Basically, the file browser is only showing the image files, so I can't select and load the code files. Unfortunately, that means I can't use acode now as the file browser seems the only way to open files. I've attached screenshots of the folder list in acode and one of the folders in CX File Explorer so you can see that files other than images are there.
I have this problem on an S6 Lite tablet and S20 Ultra smartphone. I just had acode update and still have the problem. In case it helps, both devices are Android 11 and I don't remember having any issues when running Android 10 (although that could be a red herring). However, possibly the new Android 11 file access permissions for files outside the app's folder are the problem.
I hope you can solve this so I can get using acode again.
Thanks,
Andrew.

Make sure you are using the latest version of acode b'cz it is working fine on my device. Try clearing data and restart. If the problem persist the comment here with with device model android version and acode version. And also try changing theme.

Hi,
Thanks for your reply. Yes, as previously mentioned I was using the latest version. Your idea of clearing the data and cache, albeit rather drastic, worked. However it seems only because it made acode ask for file permissions again, which resulted in an error that acode couldn't run, but then it worked after I'd closed and restarted the app again.
So when I then went into the file browser I could not open the internal storage until I clicked the link and gave acode 'all file' permissions: and that link only appeared after clearing the cache/data (see screenshots).
So the issue seems to be that acode ends up with some sort of legacy permissions when installed on a device that upgrades to Android 11, and doesn't show the link asking for all file permissions until data/cache are cleared. I assume Android 11 then only allows it access to safe files like images and so the file browser can't show 'unsafe' files like html, css and js.
I hope that helps with this issue as it could be very confusing if others have to clear their data and cache (and so all the coding, etc, preferences) when their device moves to v11. Or, maybe even they will give up on acode because they can't open code files and don't know they have to clear the data and lose all their settings and preferences.
Thanks,
Andrew.

You said right. It may cause bad user experience. Android is increasing its security day by day. People who upgraded their phone to android 11 facing a lot of problem not only in this app but also in other apps. If you think, your problem is solved you may close this issues.


SSR & Supabase Cache Helpers

Is your feature request related to a problem? Please describe.
I am working with NextJS and would like to use SSR to provide fallback data to this library and as such render the page with data already present.
Describe the solution you'd like
I would like to be able to wrap supabase cache helpers in a server component. This would take a query as its argument and use this to request server side data. It would then pass the query builder and an encoded cache key down to the client component as props, which would be wrapped in an SWRConfig provider to provide fallback data. The query could then be used in useQuery on the client side and the keys would match.
Have you had any initial thoughts in this direction? Your library is incredible useful so far and this feels like a missing piece.
Describe alternatives you've considered
I have tried to import PostgrestParser & encode @supabase-cache-helpers/postgrest-swr and then parse my supabase query in a server component and generate a cache key. However, because of the use of state in PostgrestParser it is not possible to do this on the server. Ideally I'd then like to use the PostgrestBuilder in useQuery with my browser supabase client rather than the server supabase client.
I have tried writing a separate wrapper for each useQuery but the number of different components multiplies and it doesn't feel like a maintainable pattern. Having the same query in two places is a pain when updating one or the other and it seems simpler to me to find a way to serialise the key and somehow the builder and pass it from server to client.
Then I can have one generic wrapper I reuse and only the query and config will differ.
Additional context
N/a

Sorry for the late reply, and thanks for looking into this! Could you share a bit of code? Would be interesting to see.
At the moment, I think that this has to be solved on framework level. React query for example recently published a new experimental package. This would enable us to add a adapter without the hustle of manually passing props around. I am also eagerly awaiting a few api improvements to the NextJS mutations api that would enable a proper nextjs adapter.
Of course, the fallback data bug is unrelated to this. I will fix it next week.

what do you think of leveraging fallback data or initial data instead?

define a common buildQuery function that builds the query and returns the supabase client
Make query server side using the returned supabase client, and pass data to client component
in client, build the supabase client using the same function, and set initial data from props

This should have the desired outcome without deserialising the key.

I'm very interested in this. Figuring out composable and elegant data fetching for server components has been a nightmare for me.
(this may be irrelevant) Right now, what I have settled on is using a client component with props matching a global store's type which populates the store, fetching data using supabase in a server component and passing it as props to this client component, then all client components (that need optimistic data) under the server component use the data through the store. When mutations are made, I run an optimistic update on the store and call a server action that calls supabase and revalidatesCache.
It would be great to not have to write both a supabase mutation call and logic for updating the store. This makes this package quite appealing. However, SSR (which is important to me) still seems quite complex.
Is there any hope for this in this package? Could the  (Also why is this issue closed?)

Is there any hope for this in this package? Could the (Also why is this issue closed?)

because the original question was about fallbackData, which is fixed. lets reopen.
to answer your first question:

At the moment, I think that this has to be solved on framework level. React query for example recently published a new experimental package. This would enable us to add a adapter without the hustle of manually passing props around. I am also https://github.com/vercel/next.js/discussions/50043 a few api improvements to the NextJS mutations api that would enable a proper nextjs adapter.

I will implement a proper adapter as soon as any of these become available.

hey everyone, since it will most likely take a while before a nextjs adapter becomes possible, I was thinking about providing helpers that would make this easier for React Query and SWR.
The relevant pieces of documentation are here:

react query: https://tanstack.com/query/v5/docs/react/guides/advanced-ssr
swr: https://swr.vercel.app/docs/with-nextjs

Since I do not ssr and react query in production, I would like to get some feedback on the api design.
React Query
As of now I think we are just missing a wrapper around prefetchQuery that could look like this:
import { prefetchQuery } from '@supabase-cache-helpers/react-query'

...

export async function getStaticProps() {
  const queryClient = new QueryClient()

  await prefetchQuery(queryClient, supabase.from('table').select(...));

  return {
    props: {
      dehydratedState: dehydrate(queryClient),
    },
  }
}

SWR
For swr, we could provide a helper that returns the fallback data in an object with key and value, so that you could do
 export async function getStaticProps () {
  // `getStaticProps` is executed on the server side.
  const article = await getArticleFromAPI()
  return {
    props: {
      fallback: {
        ...(await fetchFallbackData(supabase.from('table').select(...)))
      }
    }
  }
}

or
 export async function getStaticProps () {
  // `getStaticProps` is executed on the server side.
  const article = await getArticleFromAPI()
  return {
    props: {
      fallback: await fetchFallbackData([supabase.from('tableA').select(...), supabase.from('tableB').select(...)])
    }
  }
}

wdyt?

For me the key pain-point is managing the way the new nextjs app router splits server and client components and the move away from the getStaticProps examples above.

For app router, the example would look similar: put the fetchFallbackData in a Server Component, and pass it as a prop the the client component. You can find an example in the docs of Tanstack Query:
export default async function PostsPage() {
  const queryClient = new QueryClient()

  await queryClient.prefetchQuery({
    queryKey: ['posts'],
    queryFn: getPosts,
  })

  return (
    // Neat! Serialization is now as easy as passing props.
    // HydrationBoundary is a Client Component, so hydration will happen there.
    <HydrationBoundary state={dehydrate(queryClient)}>
      <Posts />
    </HydrationBoundary>
  )
}

But this also means that

ensuring the fallback query always equals the client side query in every component in two files

is not really something where we can improve dx I fear. You will always have to define two queries, especially since client-side, you might be adding dynamic filters etc to it.
But I agree, I think a flexible api like fetchFallbackData(supabase.from()): [string, Data] would be preferable.

I believe we can use server actions to use the same fetching function! This is because actions can return serializable data. Create the function in a different file with a "use server" below the header and use the function in the server and client component.

I believe we can use server actions to use the same fetching function! This is because actions can return serializable data. Create the function in a different file with a "use server" below the header and use the function in the server and client component.

that's interesting. can you provide a code snippet?

Yes I believe they would work the same because the server action always runs on the sever no matter the calling context.
As for the example, it would be like this
// actions.ts
async function databasething(filter: string) {
    "use server"
   // what ever this syntax is idk
    const supabase = createServerActionClient({cookies})

    return supabase.select ... with filter
}

// page.tsx
export default async function() {

    // query with the server action

    return clientcomponent and react query wrapper

}

// clientcomponent.tsx
import {databasething} from "./actions"
export default function () {
    react query with the databasething

    ...
}

boom no code duplication.

docs:
SWR: https://supabase-cache-helpers.vercel.app/postgrest/ssr/swr
React Query: https://supabase-cache-helpers.vercel.app/postgrest/ssr/react-query


Marking payments in different currencies as completed fails

Describe the bug
Marking bank transfers in a currency different from the fiscal host fails.
To Reproduce
Fiscal host Flarum Foundation
Payment I'd 173194
Screenshots

@kewitz added a setting to allow this in https://github.com/opencollective/opencollective-api/pull/4548

Reported from mobile. Some other thoughts.
As we already have a discrepancy between funds in the bank account versus what is mentioned on our open collective because we work with different currencies, I don't mind any differences in what is donated vs what is transferred. However.. the foundation received a bank transfer of 5,50 euro instead of usd, this probably is a mistake by the user (even not intentionally); it would however be nice to correct the donated amount based on the actual transfer.
In addition I don't understand why it mentions the platform tip, when the payment is done how is the tip processed?
Sorry for the spam 🙈

@luceos If there is an error in the amount/currency, we can change it so you can record the proper amount.
For the feature flag, I would say it's better to not opt in if there is no need. But I can do it if you understand the problems of messed up currencies as Fiscal Host.
For the platform tip, you're expected to give it back to Open Collective. On credit card transactions, we're able to collect it in real time but for bank transfers that's not possible. We're normally reconciliating this in a monthly Expense but it will not be triggered for a low amount like $0.50.

For the feature flag, I would say it's better to not opt in if there is no need. But I can do it if you understand the problems of messed up currencies as Fiscal Host.

I guess being able to mark those donations as paid is more beneficial to whatever drawbacks the feature has?

I edited this Order to make it 5.50 EUR
I changed your setting to be able to accept such Orders in the future


Issue #SC-1061 feat: System update API changes

Codecov Report

Merging #488 into release-2.2.0 will increase coverage by 0.04%.
The diff coverage is 57.89%.

@@                 Coverage Diff                 @@
##             release-2.2.0     #488      +/-   ##
===================================================
+ Coverage            31.99%   32.04%   +0.04%
- Complexity            1210     1215       +5
===================================================
  Files                  172      172
  Lines                15116    15146      +30
  Branches              2396     2403       +7
===================================================
+ Hits                  4836     4853      +17
- Misses                9557     9567      +10
- Partials               723      726       +3

Impacted Files
Coverage Δ
Complexity Δ

...g/ekstep/taxonomy/mgr/impl/BaseContentManager.java
56.88% <57.89%> (+0.54%)
75 <9> (+5)
:arrow_up:

...org/ekstep/graph/engine/router/ActorBootstrap.java
56.29% <0%> (-1.33%)
16% <0%> (ø)

Continue to review full report at Codecov.

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


Update zio, zio-test to 2.0.20

About this PR
📦 Updates

dev.zio:zio
dev.zio:zio-test

from 2.0.19 to 2.0.20
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #37.


attach recharge form to invoice

the mailed one
and the printable one

maybe add image to printable web page one


fix: add internal semantic-release-action and pypi-publish-action (#350)

chore: add internal semantic and pypi-publish actions

chore: add wget issues handling to ci

chore: add authentication to curl

chore: introduce cache for ta example

chore: fix download Splunk_TA example step

chore: unpack Splunk_TA example

chore: refactor ui setup testing

chore: fix ui setup for testing

chore: fix SPLUNK_APP_PACKAGE path

chore: take prerequisites-ui-tests job version output

:tada: This PR is included in version 2.3.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Only write EULA when it's set to TRUE

Issue
When you for example set the EULA environment variable to true (instead of uppercase TRUE) the eula.txt is written with a wrong value and the server won't start. After that, it is not possible to change the value using the EULA environment variable and you would have to change the file manually, which might be a little complicated for some users when they use docker volumes instead of bind mounts.
Fix
This pull request fixes this by checking the value to be TRUE (uppercase) before writing the eula.txt file. For any other value, the request to set the value using -e EULA=TRUE is displayed.

This is now available on Docker Hub


make it easier to contact me / pull request

Fork me on github banner?

Wouldn't it be neat if each page had a customized button that took you to the github editing screen for that page?
That'd be helpful for me most of all!

Closed in favor of https://github.com/stevekrouse/unbreakable-links


Added a new sensor to read data from the efergy energy monitor

This PR adds an efergy energy monitor sensor.

Awesome, looks good!


[Low Fantasy Gaming] Sheet Overhaul

Expanded fields and features on the sheet to improve experience and compatibility with Low Fantasy Gaming system. Will convert from old sheet.
Changes
Utilized a lot of the 5E Sheet that had been commented out to extend the sheet, and made it compatible with LFG where needed.
Added fields and features unique to LFG, and roll buttons where appropriate, extending the existing roll template.
Fixed several errors in attribute naming, but added version checking for bringing the old attribute data forward.
Additional comments
Blessing received from old sheet creator (also creator of LFG), via MeWe LFG discussion page.

Oh, bummer. I just use the editor on my roll20 campaign settings page, and then copy and paste it to github! Well, I think I fixed it. Just the html, right? Only changing the line feeds was giving me an error on commit, so I also just cleaned up an old comment, that seemed to do the trick.

Interesting. Let me check it when I merge it in. It may be fine and just look the way it does since you copy/paste it from the Campaign Settings page. Typically when it looks like a brand new .html file its because the line endings may be wrong but that is not likely the issue now that I know your process.


SHOW CREATE TABLE SqlException with TIMESTAMP WITHOUT TIME ZONE

CrateDB version: 4.0.4
Environment description:
Crate DB on single node Docker (vs. 18.06.1-ce) running on Ubuntu 18.04 VM
Problem description:
SHOW CREATE TABLE returns the following error, when used on a  table, that has a TIMESTAMP WITHOUT TIME ZONE field:
SQLActionException[SQLParseException: line 1:14: mismatched input 'without' expecting {<EOF>, '[', 'OR', 'AND', 'IN', 'NOT', 'BETWEEN', 'LIKE', 'IS', '=', NEQ, '<', '<=', '>', '>=', '~', '!~', '~*', '!~*', '+', '-', '*', '/', '%', '||', '::'}]
The created table can be used normally, I can insert, query, update.
Steps to reproduce:
Create table like so:
CREATE TABLE create_test2 (     ts TIMESTAMP WITHOUT TIME ZONE,     ts_hour TIMESTAMP WITHOUT TIME ZONE DEFAULT date_trunc('hour', CURRENT_TIMESTAMP),     id STRING,     vst INT,     dif STRING,     topic STRING,     data_obj OBJECT ) PARTITIONED BY (ts_hour);
then
SHOW CREATE TABLE create_test2
gives the error.
If I create the table with TIME ZONE, the SHOW CREATE TABLE query works, but this is not what I want.
edit:
actually the line:
ts_hour TIMESTAMP WITHOUT TIME ZONE DEFAULT date_trunc('hour', CURRENT_TIMESTAMP)
is what seems to cause the error

@heyoka Thanks for reporting, we'll have a look into this. Indeed it seems to be a bug related to using TIMESTAMP WITHOUT TIME ZONE in conjunction with a DEFAULT expression.

@heyoka We've solved this, the fix will be part of the next 4.0 hotfix release.
Thanks again for reporting.


Update pipeline.user.windows.yml

Summary of the Pull Request
We removed wox.core.  Need to do a full sign build to validate change
PR Checklist

[ ] Applies to #7882
[ ] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request
What does this include?
Validation Steps Performed
How does someone test & validate?

https://github-private.visualstudio.com/microsoft/_build/results?buildId=15321&view=results is the run to validate


do not highlight white spaces

There is a feature in VS code which would highlight the word you selected. It is nice and convenient but it also highlights white spaces, like below

This does not seem necessary, at least to some people. It would be nice if we can have the feature "do not highlight white spaces".
Thanks!

Code doesn't do this by default. Please check your "editor.renderWhitespace" setting and try to disable all extensions (by running the "Developer: Run With Extensions Disabled" command).

@zhyiyu  see comment above. This is not default behavior. Please check your extensions. Thank you!

/needsMoreInfo

Yes, you are right, it is highlight icemode that does, thanks!


Offload bn.js

Description
This PR removes bn.js dependency from stacks.js packages.
bn.js minified size is around 43kb. This will help to reduce the bundle size. In the subsequent pr, further dependencies can be removed to reduce the bundle size further.
For details refer to issue #1120
Type of Change

[ ] New feature
[X] Bug fix
[ ] API reference/documentation update
[ ] Other

Does this introduce a breaking change?
No
Are documentation updates required?
No
Testing information

npm run test

Checklist

[X] Code is commented where needed
[X] Unit test coverage for new or modified code paths
[X] npm run test passes
[ ] Changelog is updated
[X] Tag 1 of @yknl or @zone117x for review

@ahsan-javaid amazing to see some work to improve the bundle size, much needed 🙏

@zone117x Share your thoughts on this pr as well. Thanks 🙏

Codecov Report

Merging #1150 (d02c945) into master (534f1b8) will increase coverage by 0.25%.
The diff coverage is 70.66%.

@@            Coverage Diff             @@
##           master    #1150      +/-   ##
==========================================
+ Coverage   64.56%   64.81%   +0.25%
==========================================
  Files         122      122
  Lines        8359     8373      +14
  Branches     1539     1546       +7
==========================================
+ Hits         5397     5427      +30
+ Misses       2853     2840      -13
+ Partials      109      106       -3

Impacted Files
Coverage Δ

packages/cli/src/network.ts
29.49% <ø> (-0.51%)
:arrow_down:

packages/keychain/src/wallet/signer.ts
37.50% <ø> (-2.50%)
:arrow_down:

packages/stacking/src/utils.ts
54.78% <0.00%> (-0.39%)
:arrow_down:

packages/cli/src/cli.ts
14.16% <11.76%> (+0.05%)
:arrow_up:

packages/common/src/utils.ts
41.48% <84.21%> (+13.81%)
:arrow_up:

packages/bns/src/utils.ts
73.33% <100.00%> (ø)

packages/encryption/src/ec.ts
76.87% <100.00%> (+1.25%)
:arrow_up:

packages/stacking/src/index.ts
88.13% <100.00%> (-0.07%)
:arrow_down:

packages/transactions/src/clarity/serialize.ts
97.97% <100.00%> (-0.03%)
:arrow_down:

... and 1 more

Continue to review full report at Codecov.

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

@zone117x Requesting the review on this pr 🙏.

@zone117x Any further thoughts on this pr 🙏

What's the latest on this PR? The switch to BigInt from bn.js prevents stacks.js from being used in environments like React Native due to the lack of BigInt support in the JavascriptCore version used by RN.
As a result we're currently unable to update to the latest version of this library.
I propose switching back to bn.js for all packages in stacks.js if the only concern is bundle size.

@yknl good point! Do you know if solutions like https://github.com/facebook/react-native/issues/28492#issuecomment-824698934 would work for you in this case?

@yknl good point! Do you know if solutions like facebook/react-native#28492 (comment) would work for you in this case?

Unfortunately the suggested polyfill package does not work for stacks.js. There are differences between the polyfill and the BigInt type. For example, the polyfill cannot instantiate from a 0x prefixed hex string which something inside stacks.js uses. This causes a crash.

@yknl Got it.
Have you tried this https://github.com/react-native-community/jsc-android-buildscripts by any chance? From what I'm reading, jsc is the problem (which is up-to-date on iOS, as it uses the built-in jsc), but android uses a crazy old bundled jsc. This buildscript allows shipping with an up-to-date jsc on android.

Note that this specific PR is a relatively minor move towards native bigint compared to many changes that have already been pushed. For example, the new and much improved cryptography dependencies are heavily dependent on native bigint, see https://github.com/paulmillr/noble-secp256k1/blob/main/index.ts (totally dependent on bigint: exponentiation operator **, bigint literals 123n, etc).
We should not hold back from merging this PR due to react native support, as that's already been thrown out the window.
Facebook's react native js runtimes are embarrassingly outdated, but looks like they are making progress towards bigint support at least https://github.com/facebook/hermes/issues/510.
Alternatively, @aulneau and I have had discussions around this topic, and found offloading certain operations to native webviews as a potential viable solution, e.g. https://github.com/webview-crypto/react-native-webview-crypto.
@aulneau do you have any updates around this issue?

Thanks for the insight and research @aulneau!
We should consider a similar approach in stacks.js if we prioritize react-native support. Keep in mind one of their maintainers mentioned upcoming support for bigints just a week or so ago.


ffmpeg in dockerfile

Unless the segmenter wrapper does file-slicing at the end of annotate, I don't think we need to install ffmpeg in the dockerized version.

fixed in 4a4392b5bbeb8a0a5f603cc978a738269844a62a


Add time to message header

done https://github.com/johnshew/GraphDashboard/commit/8d7817a449d4dd9cd9e3c07a31f33e3a70e98bce


[Performance]: The performance of version 0.6.3 is weaker than that of version 0.6.2 in stress testing.

Proposal to improve performance
The performance of version 0.6.3 is weaker than that of version 0.6.2 in stress testing.
Scenario: Agent
Stress Testing Data: Input 50 tokens, output 20 tokens.
Version 0.6.3: 22 QPS, with a significant drop after reaching the maximum value.

Version 0.6.2: 24 QPS, with available performance reaching over 90% at 36 QPS. The decline after reaching the maximum value is gradual.

Under the same conditions, comparing version 0.6.3 with 0.6.2, it was found that the prefill time for version 0.6.3 is about 13ms longer per instance than version 0.6.2. The main reason is a pause of 200-300 microseconds between two blocks.
Conditions for obtaining the data: Using batch_size=20, offline use of LLMEngine with 50 iterations to get the data.

https://github.com/skylee-01/experimental_data/blob/main/nsys_vllm_062.nsys-rep

https://github.com/skylee-01/experimental_data/blob/main/nsys_vllm_063.nsys-rep
Report of performance regression
No response
Misc discussion on performance
No response
Your current environment (if you think it is necessary)
The output of `python collect_env.py`

Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

you can follow https://docs.vllm.ai/en/latest/getting_started/installation.html#install-the-latest-code to biset all the commits, and find the commit to blame.
and if you can come up with a fix, without affecting the performance of irrelevant usecases, that would be better.

Is this related to #9764

Is this related to #9764这与#9764 相关吗？

Thank you for your reply, I will verify it later.


POSIX: sys_mutex should be recursive, fix #103.

Thanks for your contribution.
Would you mind please to correct the code being compliant with the rest of the library?
I see problems with missing brackets for if statement and no space after if keyword. That's at minimum.


Charts overlapping with UI elements for gallery

Found by: Olivieri,Celina (elle, la | she, her)

Product: Academic Challenge: NPRI Academic Challenge - Canada.ca (ec.gc.ca)
Language: Both
Issue Description: Table data expands over graph switch
Steps to Reproduce:
a.	Scroll to 'University of Québec in Abitibi-Témiscamingue' chapter
b.	Click hamburger menu then Show Table Data
c.	Expected: I expect the displayed table data to be fully visible and not be covered
d.	Actual:
e.	^ The last row of the table is covered by choosing different tables
Also: columns don't seem to have any padding.

Feng,Yi Lei
Looks like on mobile the datatable will be completely cut off, so we will need to adjust the chart size or just turn off this option on mobile resolutions.

Related #336


Firmware Update detection issue

So the way I personally use the Pocket Updater is to keep it in my PC Hard Drive and update to "Current Directory" first (as a backup & test) and then I run the update again against my SD card (Removable Storage).
When I updated just now (current directory on HDD) it found the new Pocket firmware update and downloaded it to my HDD. I then put my SD card in my PC and ran the updater again on "Removable Storage" and it says there were no updates found. I quit and relaunched the app and it still didn't find any updates. This time I paid close attention and it said my firmware was already up to date.
Somehow if the firmware update is already in the "current directory" (my PC hdd) it decides my SD card already has it as well, even if it doesn't.
I did not experience this in a previous version of the app and I have no idea if that is because the older versions didn't have the issue or just the timing of firmware releases prevented it from presenting itself.

Hi - Can you see if v1.4.4 resolves this issue when possible? I believe it should.

Hi - Can you see if v1.4.4 resolves this issue when possible? I believe it should.
I would say it's fixed in v1.4.4 if the following steps are considered a valid test:

Removed current FW file from SD card
update "current directory" on local HDD (newest FW is now present there)
update SD card

In that scenario it did find and add the current FW to the SD card instead of saying it was already downloaded.
If the above is not a valid test, I will be sure to test in the same order when the next Pocket FW releases.
Thank you!


Error on odd tank size

Setting desired tank sizes to an odd number of wagon lengths (in this case 5) gives variations on the following error on startup:

Error while loading entity prototype "cust-storage-tank-034" (storage-tank): invalid pipe connections specification for offset {-17.000, 1.5000}. The offset must be outside the entity bounding-box.

Hi, is the problem solved with the crash when inputting an odd number of tanks, aka "3" ?


About speed comparison?

Thanks for your excellent work!
I noticed convcrf is run on gpu, while fullcrf is run on cpu. Is it unfair to compare two algorithms on different platforms?

The reason for this comparison is that there is simply no working fullcrf implementation on gpu available. CRFasRNN, DeepLab and other segmentation systems utilizing CRFs use the very same CPU implementation. So it is a fair comparison since it improves the state of the art.

Yes, densecrf does bilinear downsampling internally.

Thanks for the prompt reply!
I understand that there is some interpolation happening when mapping the pixels onto a permutohedral lattice. Are you referring to that? There downsampling is done in the high-dimensional lattice space. In ConvCRF, the downsampling is done on image space (XY) instead.

Hi, Marvin:
I notice that in your paper, you post the speed of ConvCRF with different receptive field size from 3 to 13. Here is my question, why there's no more larger conv-size than size-13 ? And is the speed posted in table.1 indicate that the time cost for just one iteration?

Hi Alex,
the performance does not improve past 13. Also, if you go bigger then ~21 I ran into GPU memory issues (@ 11GB). The memory consumption also increases quadratically with filter size. So there is no reason why you want to go past 11.


BUG: Website Title in krishiconnect/login.html

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Add krishiconnect Website Title in krishiconnect/login.html

Could you pls assign this issue to me under GSSOC'24?
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Fixed in #146


QoS: Change how provisioning blocking works

Three things:

Fixes AZURE_CONFIG_DIR that has been sorta broken since I added it probably. Only noticeable in some rare cases.

Moves us to the 2.0 version of CustomScript for Linux to avoid this issue: https://github.com/Azure/azure-linux-extensions/issues/367

Fixes the QoS impacting issue so that SPN failures will propagate up as deployment failures:
i. makes the master CSE run in blocking mode so that failure is propagated up
ii. makes it so that Windows agent provision in the background via use of Start-Job with PowerShell

Only wait 5 minutes for apiserver instead of 10.

This change is

@colemickens,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@acs-bot test this please

This was tested with a bad SP. The deployment fails fast, meaning that the windows agents didn't block the deployment, and it shows that the master CSE failed properly as a result of apiserver not coming up. These were the stated objectives for this particular QoS bug.

cc: @anhowe

The deployment failure manifests like this. There doesn't appear to be a way to get CSE to propagate a proper error message all the way up out to the CLI, unfortunately, so this is the best we can do from a UX perspective.
+ az group deployment create --name colemick-58e830a2 --resource-group colemick-58e830a2 --template-file /mnt/persistentdisk/home/cole/code/colemickens/acs-engine_gopath/src/github.com/Azure/acs-engine/test/../_output/colemick-58e830a2/azuredeploy.json --parameters @/mnt/persistentdisk/home/cole/code/colemickens/acs-engine_gopath/src/github.com/Azure/acs-engine/test/../_output/colemick-58e830a2/azuredeploy.parameters.json
At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-debug for usage details. {
  "status": "Failed",
  "error": {
    "code": "ResourceDeploymentFailure",
    "message": "The resource operation completed with terminal provisioning state 'Failed'.",
    "details": [
      {
        "code": "VMExtensionProvisioningError",
        "message": "VM has reported a failure when processing extension 'cse0'. Error message: \"Enable failed: failed to execute command: command terminated with exit status=1\"."
      }
    ]
  }
}  Correlation ID: 15445a1b-3eed-4dbe-91ed-28e5d49bec66

@acs-bot test this please

cc: @dmitsh
So, when the deploy script was cut up, the AZURE_CONFIG_DIR stopped actually working. When I fix it here, it ends up breaking Jenkins... I suspect that Jenkins is calling these scripts in such a way that the AZURE_CONFIG_DIR is set for the login, but not set when running the actual deployment.
I can't really trace through all of the steps that are going on in this job anymore. There's a shell script in Jenkins which runs Docker which runs a shell script, which runs Go code, which later runs the bash scripts in test/. Can you take a look and see where the export AZURE_CONFIG_DIR... should go such that it will be in effect for the login and the actual deployment step(s)?
Thanks.

@acs-bot test this please

Okay, well, I just moved it so that AZURE_CONFIG_DIR is only set/used for the all-in-one deploy.sh script. Jenkins is proceeding now at least...

@acs-bot test this please

@acs-bot test this please

@acs-bot test this please

@acs-bot test this please


[LIBP2P] - Research the Stream Behaviour

What is this task and why do we need to work on it?
Rust libp2p offers a stream Behaviour that seems pretty lightweight easy to integrate into our application.  See the exampes/stream in rust-libp2p.
What work will need to be done to complete this task?
No response
Are there any other details to include?
No response
What are the acceptance criteria to close this issue?
Make an informed decision on whether streams is better for us than RequestResponse
Branch work will be merged to (if not the default branch)
No response

One initial issue: stream is so recent that there's been no libp2p release containing stream. The closest I see in 0.53 is this, which is not the same thing. To add stream, we would either (1) need to wait until a new release is stabilized or (2) point libp2p at a recent commit.

I've been looking this stream example in main and didn't even notice it's not in the current release. I'll put this in the backlog


DEV:FsR:Build Issue for WitsmlObjectsLibrary HIGH PRIORITY

UPDATE: Prem put his instructions for generating another WitsmlObjectsLibrary. But, as I mentioned to Prem, the delays I was experiencing do not match what he is seeing. I learned that the Intellij IDE sometimes needs to be told to explicitly load the new jar file for this library. All developers have been told, and now we have a successful recourse for getting this library in a timely manner.


remove support for codu style pids

done we're not using pids

we are persisting old pids in handles

just talked to @jackflaps, they're going to persist old fedora pids in archivesspace as a user defined field for preservation purposes, we are not going to worry about that, whatever metadata comes from archivesspace if that includes fedora pids is OK because backend can support new json fields/mappings


Chat does not work

Followed the instructions and added the chat service SID.
getting this error

Same here.

Although, the TwilioChat-Node project did work for me. So that at least proves that my account is working well. https://github.com/TwilioDevEd/twiliochat-node

This works for me no problem. Make sure you uncomment the line with your chat ID from:
# Required for Chat
# TWILIO_CHAT_SERVICE_SID=ISXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX

...to...
# Required for Chat
TWILIO_CHAT_SERVICE_SID=ISXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX


Make tabs sortable

I keep trying to do this and it doesn't work, so I thought I would implement it 🙂

Two questions about the implementation details:

In chrome the new tab button disappears when you are dragging tabs, so that's how I implemented it here.  However the playground tab is a little more visible, so I was thinking about maybe leaving it visible and making it the right border of the draggable area.  Anyone have thoughts on what looks best?
In chrome dragging a tab also makes it the active tab.  I was having trouble getting the dragged tab to look "active".  I think I know how I could do it, but I'm not sure how much people expect this behavior, so I thought I would ask first if it should be implemented.

Also, the snapshot tests are failing.  Should I go ahead and update them in the PR?

 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.

Wow! Nice work, I really like it. I tested and its working nicely. The whole topbar needs little bit change, because currently there is a problem with tabs when they overflow on the right side. It would require to rethink it more. In my opinion, it should change to active tab because you want to "do" something with that tab.

Added two fixes:

Dragging a tab will cause it to be active
If you have enough tabs to make the tab bar scrollable, it will scroll if you drag a tab towards the edge

I think this is good to go, but let me know if you think of anything else that should be changed.

Thanks for your first contribution! It will be in next release 🚀


(PUP-7425) Refactor Gemfile / .gemspec for Bundler workflow

Without platform-specific gem dependencies marked as such inside the
.gemspec, this can cause issues with a bundler based workflow that
uses bundle install --system --binstubs c:\ruby\bin on Windows
machines even if the given Ruby version already satisfies all of
the Puppet platform-specific dependencies. This is due to how a
Gem is installed by Bundler when it refers to a git repo and its
.gemspec is written.

Note that all runtime dependencies are now present in the .gemspec
and that any development dependencies are still defined in the
Gemfile itself.  The .gemspec is included in a bundle install by
calling the gemspec method inside the Gemfile.  This is the
intended inclusion mechanism for Bundler.
The Gemfile continues to allow a few gem overrides with the
environment variables:
FACTER_LOCATION
HIERA_LOCATION
These overrides are only activated when the relevant environment
variables are defined.

Note that the environment variable BUNDLE_SPECIFIC_PLATFORM also
had to be set to properly use platform-specific versions of any
gems that have been referenced.  This will be the default behavior
in Bundler 2, and given platform-specific versions of Facter are
shipped - should always be the case for Puppet. Without setting
this Bundler will raise a warning during installation.

dbebd0d seems to be causing some weird behavior on OSX when using FACTER_LOCATION, and should probably be moved to its own PR / removed altogether.  I thought that the behavior I was fixing was a new problem caused by this refactor, but it turns out that it's always been a problem.

CLA signed by all contributors.

One thing to potentially look into in the future - ext/mkrf_conf.rb can supposedly be used to include platform specific gems at runtime as long as gemspec.extensions = 'ext/mkrf_conf.rb' is added to the gemspec.  This could make it so that Puppet does not need to have platform specific gems (and could remove the need for a separate gem building rake task that uses project_data.yaml).  Probably worth another ticket / investigation for a later date.

Interestingly I moved rake to a dev dependency, but now this has caused a failure in AppVeyor:
bundle exec rake parallel:spec[2]
C:/Ruby23-x64/lib/ruby/gems/2.3.0/gems/bundler-1.14.5/lib/bundler/rubygems_integration.rb:408:in `block in replace_bin_path': can't find executable rake (Gem::Exception)
	from C:/Ruby23-x64/lib/ruby/gems/2.3.0/gems/bundler-1.14.5/lib/bundler/rubygems_integration.rb:434:in `block in replace_bin_path'
	from C:/Ruby23-x64/bin/rake:22:in `<main>'
Command exited with code 1

The AppVeyor invocation uses the following Bundler invocation which doesn't jive:
bundle install --jobs 4 --retry 2 --without development extra

Interestingly TravisCI appears to do the same via bundler_args: --without development at --https://github.com/puppetlabs/puppet/blob/master/.travis.yml#L6 ... but manages to install rake!
Installing rake 10.1.1

Hmmmmmm

See https://tickets.puppetlabs.com/browse/PUP-7433 for the next phase, which will require a bit more coordination.

Note that https://github.com/puppetlabs/puppet/pull/5781 which builds on this commit has cleared Travis and AppVeyor at https://github.com/puppetlabs/puppet/pull/5781 which is validation that this refactor is good in both of those environments.


Smash token

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

[x] Filename in the lowercase format
[x] Image format: png
[x] Image size: 256 by 256 px

Name : Smash
Total supply : 1,000,000
Etherscan : https://etherscan.io/token/0x12bac8c392f5a2ec82f3c2799289a1a7515d8f02


train_on_batch causes really slow process

Hi, I've a ResNet that runs fast on Keras when I use the normal fit method.
However as I require to use train_on_batch, it seems that it's much slower when I do that.
I've an external call, that calls train_on_batch for every iteration.

Perhaps this isn't all that surprising considering train_on_batch does some additional checks (_standardize_user_data and _make_train_function in particular, I guess) that you'll be doing on every batch instead of just once for all batches.
See train_on_batch vs fit.
You could run a profiler such as yappi to see what extra function calls are costing you the noticeable performance difference. What do you see?

_make_train_function is "lazy generator" that is called in fit as well

I realised my issue was that I did some calculations on


Reconnect to shim event stream after containerd restart

There are three aspects which need to be covered:

the runtime needs to restart its event pump when it reconnects (in
loadContainer).
on the server side shim needs to monitor the stream context so it knows when
the connection goes away.
if the shim's stream.Send() fails (because the stream died between taking
the event off the channel and calling stream.Send()) then to avoid losing
that event the shim should remember it and send it out first on the next
stream.

The shim's event production machinery only handles producing a single event
stream, so add an interlock to ensure there is only one reader of the
s.events channel at a time. Subsequent attempts to use Events will block
until the existing owner is done.
Fixes #921.
Signed-off-by: Ian Campbell<EMAIL_ADDRESS>

Subsequent attempts to use Events will block until the existing owner is done.

Perhaps they should return some sort of Busy indication instead? However I notice there is no TryLock on sync.Mutex (https://github.com/golang/go/issues/6123 suggests its not going to happen), I suppose I'd need to fiddle around with a channel based ownership mechanism to implement that.

Thinking about it over lunch given the usage scenario the chances of there being a second simultaneous caller of this seems pretty slim, so the simple lock based method is probably fine.

@ijc25 Can we really avoid losing event across containerd restart?
What if shim successfully sends the event to containerd, but containerd got restart before it sent out the event to client? The event could still lose unless we checkpoint it, right?
If so, maybe we should not assume containerd doesn't lose event across restart on the client side, and always relist when reconnecting to containerd?

@Random-Liu we can avoid losing one in this particular way, you are correct that there are other ways one could lose an event.
What do you mean by "relist"? AFAIK nothing is keeping track of historical events which would allow a replay.

@ijc25 I mean List and Inspect all images/containers and recover system state. This may take a while, but only happens during initialization.

@Random-Liu I see, yes, clients should indeed do something like that.
Shall I drop that bit of this PR then?

@ijc25 I'm not sure. The lock stuff looks a little bit scary. :p

Thinking about it over lunch given the usage scenario the chances of there being a second simultaneous caller of this seems pretty slim, so the simple lock based method is probably fine.

I think we have namespaces just for multiple consumers?

@Random-Liu @ijc25

What if shim successfully sends the event to containerd, but containerd got restarted before it sent out the event to the client? The event could still lose unless we checkpoint it, right?

In this case it's unavoidable that containerd will lost that event, and event lost can be very painful in production environment, so we fixed it in another way. In our internal version of 0.2.x branch, we implemented transaction for important actions whose events can be critical, like delete, exit, update, etc.. and transaction is closed only when containerd saved that event, if containerd is restarted with some unfinished transactions, actions will be replayed and events will be re-sent.

If so, maybe on the client side, we should not assume containerd doesn't lose event across restart, and always relist when reconnecting to containerd?

You don't have to, you can always trust containerd will sent the right events with the solution I said above. Instead of concerning about containerd restarting, another scenario is more common, which is client got disconnected, so we should make sure client got all events during that time when it connected with containerd again.
To implement this, we should make containerd itself as a consumer of shim events as we did in 0.2.x branch, and save all events it got into a buffer, so client can get old events from that buffer with a certain timestamp. Which is why I added timestamp to events args in https://github.com/containerd/containerd/pull/891 .

Anyway this PR looks good to me.
LGTM

@Random-Liu

@ijc25 I'm not sure. The lock stuff looks a little bit scary. :p

The lock stuff is not specifically for the event loss (although it does synchronise access to s.deferredEvent), it's main purpose is to ensure only a single event stream is pulling things off <-s.events at a time. I'm not sure what the actual semantics of having multiple readers on a channel are, but I am pretty sure they would not be what the callers of this stream would be wanting (I expect the events would get delivered to one or the other channel at random).
So my question is only whether I should remove the uses of s.deferredEvent, the locking needs to stay.

I think we have namespaces just for multiple consumers?

I think that would be at the client<->containerd boundary, the interface here is on the containerd<->containerd-shim boundary which I think is always going to be 1:1 by design.
For the N:1 client<->containerd I'd expect the fanout to happen in containerd rather than the shim.

@hqhq somewhat off topic for this PR but does your solution not require keeping a log of all events ever, since you don't know if or when a disconnected client might come back, or a completely new client wants to join in.

@ijc25 There's a retaincount to specify the buffer size for event logs retained by containerd, see: https://github.com/containerd/containerd/blob/v0.2.8/supervisor/supervisor.go#L80-L108 , default value of retaincount is 500 (the actual buffer size would be 2*500-1 IIRC). So we don't keep a log of all events forever, and this demands the clients should not be disconnected for a long time, there's healthcheck api from grpc so it won't be hard for clients to archive that demand.
For new clients, we should have options for them to ask for new events or events since a certain timestamp or events for certain containers.

I think that would be at the client<->containerd boundary, the interface here is on the containerd<->containerd-shim boundary which I think is always going to be 1:1 by design.

I see. Thanks for explaining.

There's a retaincount to specify the buffer size for event logs retained by containerd, see: https://github.com/containerd/containerd/blob/v0.2.8/supervisor/supervisor.go#L80-L108 , default value of retaincount is 500 (the actual buffer size would be 2*500-1 IIRC). So we don't keep a log of all events forever, and this demands the clients should not be disconnected for a long time, there's healthcheck api from grpc so it won't be hard for clients to archive that demand.

@hqhq Thanks for your suggestion! Does containerd event loss happen frequently in the production environment? Will containerd lose event even when it seems working properly (Recv doesn't return error and healthcheck returns ok)?
If containerd loses event without any portent, we should fix it;
If containerd only loses event when containerd is down, or grpc connection is disconnected for some reason, I think list and recover the state on the client side seems to be acceptable and simpler solution.

LGTM

@Random-Liu

Does containerd event loss happen frequently in the production environment?

Not that frequently in normal production environment, but it's easy to reproduce in our test environment, for example, when use docker, you keep pausing and unpausing a container and in the same time keep restarting containerd, it's possible that after containerd-shim called runc.Pause, then got restarted before it got the event, on docker side, you'll see the container is running but actually it's paused.

Will containerd lose event even when it seems working properly (Recv doesn't return error and healthcheck returns ok)?

When losing event happens, containerd all seems working fine, the issue is on client side, it got inconsistent states.

If containerd loses event without any portent, we should fix it;
If containerd only loses event when containerd is down, or grpc connection is disconnected for some reason, I think list and recover the state on the client side seems to be acceptable and simpler solution.

AFAIK it only happens when containerd is down, it's acceptable that you can do the recovering on client side, like do something in https://github.com/moby/moby/blob/v17.05.0-ce/libcontainerd/remote_unix.go#L138-L177 for moby, but I doubt it'll be simpler, you don't know what event you might be missing, so you'll have to check all states for all containers, efficiency is one thing, what's more, not all events are about state which you can get by a simple "list", such as oom event.


Indicate unread posts in tags owed

Via Moriwen:
I would find it very helpful to have something on the Tags Owed page marking which threads I have unread posts in and which I don't. I often lose track of whether I haven't gotten around to replying to something, or whether I have and the other person has already replied back; this results in me sometimes not seeing a new reply for a long time.
http://alicorn.elcenia.com/board/viewtopic.php?f=12&t=459&start=1630#p30737

Bold could work.

Changing the color of the unread icon would be better, apparently. Maybe we should make it slightly gray-er if there are no unread replies, so it's consistent with other pages when you have unreads but the tags owed page still has the appropriate button.


x/pkgsite: add "Package files" section for commands

What is the URL of the page with the issue?
https://pkg.go.dev/github.com/google/gopacket/examples/httpassembly?tab=doc
What is your user agent?
Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:80.0) Gecko/20100101 Firefox/80.0
Screenshot
not relevant
What did you do?
Just went to the doc page for this package
What did you expect to see?
A Package files section, with links to the underlying code files, as the former godoc.org site would offer :
https://godoc.org/github.com/google/gopacket/examples/httpassembly?tab=doc
has a main.go link, which leads to :
https://github.com/google/gopacket/blob/master/examples/httpassembly/main.go
What did you see instead?
I found no such link, in none of the tabs of the pkg.go.dev version for this url.

In this specific case, there is no public symbol to "document", and the documentation clearly implies "go read the code of the main function", so the lack of Package files section is particularly emphasized here.

We'll be adding a Package Files section soon.

This feature is now available on beta.pkg.go.dev. See https://github.com/golang/go/issues/41585#issuecomment-718143563 for details, and https://beta.pkg.go.dev/github.com/google/gopacket/examples/httpassembly#section-sourcefiles as an example.
Closing this issue for https://github.com/golang/go/issues/41586.


[otp_ctrl] Disable OTP consistency check timeout during prog ops

This is done to avoid spurious consistency check timeouts. Programming ops are much slower than readout operations in the OTP macro, and can hence interfere with the consistency check timeout mechanism.
Note that this does not disable the consistency checks, and it also does not affect integrity checks.
Signed-off-by: Michael Schaffner<EMAIL_ADDRESS>

@msfschaffner one thing that popped into my head today... if someone updates the flash / sram scrambling seed in otp.. does this also mean there's a chance of consistency checks failing, assuming it's been enabled?
i think the right answer is that if software knows it'll make updates it should be enabling consistency checks, but wanted to confirm with you.

@msfschaffner one thing that popped into my head today... if someone updates the flash / sram scrambling seed in otp.. does this also mean there's a chance of consistency checks failing, assuming it's been enabled?
i think the right answer is that if software knows it'll make updates it should be enabling consistency checks, but wanted to confirm with you.

Ah I think that case should not be of concern, since the consistency checks only kick in once the digest has been programmed. So that basically means that only partitions without a digest, but with a consistency check are prone to this failure mode, which essentially only applies to the life cycle partition.

Ah I think that case should not be of concern, since the consistency checks only kick in once the digest has been programmed. So that basically means that only partitions without a digest, but with a consistency check are prone to this failure mode, which essentially only applies to the life cycle partition.

ah yes thanks for the reminder.
On Tue, Jan 5, 2021 at 11:57 AM Michael Schaffner<EMAIL_ADDRESS>wrote:

Ah I think that case should not be of concern, since the consistency
checks only kick in once the digest has been programmed. So that basically
means that only partitions without a digest, but with a consistency check
are prone to this failure mode, which essentially only applies to the life
cycle partition.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/4705#issuecomment-754863661,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSUTQVX7ZEUJMOYOHK3SYNVJ5ANCNFSM4VO23ICA
.

ah yes thanks for the reminder.
On Tue, Jan 5, 2021 at 11:57 AM Michael Schaffner<EMAIL_ADDRESS>wrote:

Ah I think that case should not be of concern, since the consistency
checks only kick in once the digest has been programmed. So that basically
means that only partitions without a digest, but with a consistency check
are prone to this failure mode, which essentially only applies to the life
cycle partition.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/4705#issuecomment-754863661,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSUTQVX7ZEUJMOYOHK3SYNVJ5ANCNFSM4VO23ICA
.


Remove chat-pic

This PR removes the chat pic in favor of adding a similar component in bepstore-ui-chat

@bartist @WouterKvG Please review!


False posiive with Malwarebytes

We have an install using Newtonsoft.Json dll - all was great until the latest update.
In version 13.01 MalwareBytes is reporting the dll as a virus and then deleteing it.
What can be done?

Stop using MalwareBytes?

This is a useless comment. We have little control over the software our clients use. But you do have control over the code that you generate and if it causes false positives in anti virus software. In the end we had to notify MalwareBytes and notify them that your dll was generating false positives. Thank you, your welcome.


Fix typos in NetworkProxy CSP detail page

@manikadhiman This PR that has been contributed by the community is available for review. Thanks.


Adding more cases to cases.txt issue

Very small bug, maybe you know of it, but if you add another case to the end of the file and you don't leave an empty line below it, then the case result will be cut off when evaluating.
Expected:

Actual:

Fix was just to leave an empty line in the cases.txt as such:

I figured it out pretty quickly luckily enough (mostly because I've seen such cases before, especially with compilers :P) and I just wanted to mention it.

Is this a problem with the current build script? If you run ./bin/build.py will it create output it can't read?
Anyway I guess this would be a easy fix in the case reader.

Is this a problem with the current build script? If you run ./bin/build.py will it create output it can't read?
Anyway I guess this would be a easy fix in the case reader.

I was actually not using the build.py, didn't know exactly how to use it. I just manually compiled, decompiled the classes/new methods and then added the case at the bottom of the cases.txt. So that may have been my bad if there was a better way to do it!


Use CSV for places.

Use CSV to set place kinds and min zooms, except for WOF.
Connects to #646.
@rmarianski could you review, please?

:+1:


Top k Collector with Linear time selection

From https://github.com/tantivy-search/tantivy/issues/283#issuecomment-387333453

About the time complexity, using linear time selection for each segment has slightly better time complexity in theory.
Linear time selection in each segment works like this. Instead of the heap of size k, create an array of capacity 2k. For each hit, push back in the array. When the count of docs in the array reaches 2k, perform linear selection to find the top k docs. Drop the rest. Continue this. The final result is the top k docs in random order.
In the merge phase, use a heap to get docs in sorted order. O(k log (kS)) heap insertions.

Let's experiment the idea. The algorithm might work better for large k.
Time Complexity
If capacity = 2 k, collect() runs in amortized O(1). score_docs() runs in O(k log k).
Of course we can try different capacity, but it is not that interesting. If capacity = (1+α)k,
→ linear time selection happens at most (n - (1+α)k) / (αk) times,
→ collect() runs in amortized O(1+α ⁻¹). score_docs() runs in O(k log k).
About the Current topcollector
binary heap is used. For each insertion to the heap, it takes O(log k). From https://github.com/tantivy-search/tantivy/issues/283#issuecomment-386069648 "If hits are shuffled ... The overall number of inserts is asymptotically equivalent to k log(n)".
Linear time selection algorithm
There is order-stat crate, but maybe we need better Floyd Rivest implementation which handles duplicate elements efficiently.

the probability of having to insert doc n in the heap is k/n.
From #283 (comment)

I think some of the hits will replace the old one in the heap right? The insertion number can be significantly larger than that.
Insertion is the dominant operation, if we just look at the complexity assuming all top-k hits would need to be inserted, as @kcm1700 stated, the insertion is O(log k) vs O(1). (in minheap case, the complexity given k insertion is k log(k), and in the stated linear selection approach, complexity is (k) + (k + k/2 + k/4 + ...) = k + sum_{d=0}^{log k} (k/(2^d)) = O(k) )
But I suspect the mainstream search engine still uses binary heap is for cache efficiency because heap has better memory spatial locality.

Right, the insertion number can be significantly larger. The comment assumes documents are shuffled randomly, which is not necessarily true. But if we assume that, the probability of having to insert the i-th document in the heap is k / i. Then, E(# of insertions) = sum_{i=k}^{n} k / i = O(k log n) Therefore, on average, time complexity is O(k log n log k). In the worst case, time complexity is O(n log k).
I think linear selection algorithms are usually cache friendly, so I guess it shouldn't really matter.
I guess the most tools are using binary heap because the performance of this part is not important.

My major concern with linear selection is when the array gets reallocated longer and longer (that is the case where it performs better in theory), the entire array just does not fit into cache, and in minheap, when you insert, you rotated up level by level with better locality. But this is purely based on my perception, it would be interesting to see both running and compare them.

You misunderstood @kcm1700 solution. He suggests to work off a Vec of capacity 2k.
You add elements to it (probably if they are below the threshold). Once the array reach its capacity apply linear selection to bring it back to a len of k. (if a threshold is used compute the max as well).

It has been implemented already.


Feature Request: allow additional settings from package.json config

Do you want to request a feature or report a bug?
feature request
What is the current behavior?
The library loads settings like this:

One path and one maxSize from CLI options, OR
An array of path/maxSize settings from package.json at key bundlesize
An API token via ci-env and process.env settings
(other sources in elsewhere in code?)

If this is a feature request, what is motivation or use case for changing the behavior?
Two-part request:

permit more settings to come from package.json
collate all the config setting sources together in src/config.js, and use those where needed in code.

I have a specific setting I'd like to request, but in deference to the contribution guide, here's our discussion issue first.  :)
Also my feature is semi-trivial, but the existing code doesn't easily allow extra settings.  So to fix that, I have already done a proof of concept over here.  It includes all the changes I guessed were necessary to allow more configuration.  I also added unit tests via jest in addition to self-exercise test script that was inline in the current package.json file.
I picked jest rather than a more minimal testing framework for its ease of mocking require statements.   Because the library uses both package.json parsing and commander argv parsing, I thought it was worthwhile.
Please mention other relevant information.

node version 8.4
npm version 5.4
Operating system Linux/ WSL-windows
bundlesize version 0.15.3
CI you are using:  Circle

Hey @sethbattin, thanks for opening a detailed issue.
Makes sense! In fact, this is something that we know we will need, were just waiting for the opportunity.
https://github.com/siddharthkp/bundlesize/pull/171 and https://github.com/siddharthkp/bundlesize/pull/172 have opened that up for us.

Great.  Looks like #172 expands the definition of a single file, and #171 adds an extra source.  I think I could adapt my patches to use both of those; do you know when those PRs might be merged?
Also, do you want the unit test addition?  If it helps, I could revert https://github.com/sethbattin/bundlesize/commit/40f2536aef1331012f26b8f6827faf56f21a30aa and https://github.com/sethbattin/bundlesize/commit/2f10a06baee4f417cf190415956da3da1a9a1fac so that the merge is easier, then make a separate PR for them.

do you know when those PRs might be merged?

Can't say for sure 😅

do you want the unit test addition? If it helps, I could revert ...

Don't worry about it, i'll take of it 😉

Sorry for my confusion here, but I don't know what you are intending.  Should I submit a PR, or wait for those other tickets, or what?

Implemented in https://github.com/siddharthkp/bundlesize/pull/316 and published in 0.18.0 🎉
Check out README for details


Add configuration, configurable routes

This adds possibility to use customer URLs for batch_sequential and batch_parallel endpoints.
For achiving that, Configuration class has been introduced.
Also, method BatchRequestApi.configure is available for setting the configuration values. When not specifying custom configuration, the default values are used. Because of that, this pull request is backward compatible.

This gem definitely needs this feature, but I think it would be better to provide a call for the user to put in their config/routes.rb, similar to devise_for, ActiveAdmin.routes or mount_ember_app (from ember-cli-rails). Then they can just be wrapped in scopes for api/v1 if that's what you want, and no routes get unexpected added to your app like what currently happens.

We plan to review and merge this soon.. Thank you for the pull request

Thank you for the feedback!
@DuBistKomisch, I agree with you, this would be more transparent approach. However, in this PR my goal was to make the routes customizable while maintaining current functionality without requiring changes from gem updaters. In short, adding another option to the current behaviour.
I suppose the approach described in your comment would be definitely step forward, so I'll leave it for another pull request (either yours, mine or somebody elses). Then the possible alternatives can be analysed thoroughly and we can be sure that a proper solution gets implemented.


Download Uncen version only for some anime series

Describe the feature here
I wonder if there's a way to download Uncen version only in a single command instead of download each one of them individually.

I'm sorry. I don't understand. What's uncen?

Some series has both censored and uncensored version. I want to know if it is possible to download each version independently.

Can you give an example?

Like this one:
16: High School DxD Hero (Dub)                  DUB | Ep 12-Uncen/12
It has 12 episodes on both version but it say 25 when view the series.
Title: High School DxD Hero (Dub)
episodes_done: 0
Length: 25
Provider: 9anime

Hmm. Ok. I'll see what I can do about it.

The episodes are divided so that the odd numbered episodes are the censored ones and the even numbered episodes are uncensored. I don't know if that makes it easier for you.

@FarlsBarkley Thanks for the info. I was thinking of a more generic solution. Maybe one based on regular expressions. Maybe a new argument --filter-episodes which take a regex and filters on basis of it.

#145


fix(engine): kill docker when timeout

Hello, I found something was not expected when docker run timed out.

Ok. Does not reproduce on my linux machine, but it happens in my mac and wsl2...
I will close it.
Thanks.


Focus on already opened editor instead of default editor provided by extension

Sometimes I open a file with the text editor instead of a customEditor. In this case, when this editor is still open and that I click in Explorer view on the file, I would like to have the focus on the text editor as there is already an opened editor for it.
Currently it is opening another editor for the same file:

I'm wondering if it would make sense to even remember the last type of editor used for each file so that it will reopen will last editor used even if editor is still opened or not.

I'm not familiar with VS Code codebase. I think the code to adapt will be in https://github.com/microsoft/vscode/blob/dbed937b8033765edb75c10f7978bcf2b1fca6ba/src/vs/workbench/services/editor/browser/editorResolverService.ts#L387
but hard to investigate more as I'm currently unable to build VS Code locally https://github.com/microsoft/vscode-discussions/discussions/632

I have implemented a solution to prefer the open editor instead, if this is not the desired behavior please update the ticket 🙏
https://github.com/microsoft/vscode/pull/189733


Add information for backup process

Is your feature request related to a problem? Please describe.
Downloading a backup says in german:
"Das System-Backup wird jetzt erstellt. Dieser Vorgang kann einige Minuten dauern."
Below we have an OK button. I am always confused if I should press OK, or just wait.
Describe the solution you'd like
Add to the text:
"Das System-Backup wird jetzt erstellt. Dieser Vorgang kann einige Minuten dauern. Bitte warten. Der Download startet automatisch."
I would have made an PR, but have no idea where to change the strings:
https://github.com/jens-maus/RaspberryMatic/blob/master/buildroot-external/patches/occu/0019-WebUI-EqualThanCondition/occu/WebUI/www/webui/js/lang/de/translate.lang.js#L535
or
https://github.com/jens-maus/RaspberryMatic/blob/master/buildroot-external/patches/occu/0001-RaspberryMatic/occu/WebUI/www/webui/js/lang/de/translate.lang.js.orig#L540

I am really ready to help here with this small improvement.

I am really ready to help here with this small improvement.

Then go ahead and add a new WebUI patch which modifies these strings to your likings and then submit it as a PR so that I can review your proposal and potentially integrate it.

Then go ahead and add a new WebUI patch which modifies these strings to your likings and then submit it as a PR so that I can review your proposal and potentially integrate it.

I wrote in the first comment that I have no idea which is the correct place:

I would have made an PR, but have no idea where to change the strings

I wrote in the first comment that I have no idea which is the correct place:

I would have made an PR, but have no idea where to change the strings

The correct place would be to submit a PR with a patch against the WebUI/OCCU sources. See here for the directory where we collect hundreds of patches against WebUI/OCCU which change different aspects in the WebUI:
https://github.com/jens-maus/RaspberryMatic/tree/master/buildroot-external/patches/occu
All you have to do is, fork this repository, create a dedicated branch and then add a similar WebUI patch in this directory and then execute create_patches.sh to generate the *.patch file automatically. And if this works and the patch looks reasonable feel free to submit your branch as a PR here on GitHub.

The correct place would be to submit a PR with a patch against the WebUI/OCCU sources. See here for the directory where we collect hundreds of patches against WebUI/OCCU which change different aspects in the WebUI:
https://github.com/jens-maus/RaspberryMatic/tree/master/buildroot-external/patches/occu
All you have to do is, fork this repository, create a dedicated branch and then add a similar WebUI patch in this directory and then execute create_patches.sh to generate the *.patch file automatically. And if this works and the patch looks reasonable feel free to submit your branch as a PR here on GitHub.


Add code sample for post dropdown menu action

In response to a customer escalation

@mickmister applied the change


Merge to live!

Docs Build status updates of commit e346a42:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/develop/authorize-to-microsoft-graph.md
:white_check_mark:Succeeded

docs/develop/create-sso-office-add-ins-aspnet.md
:white_check_mark:Succeeded

docs/develop/create-sso-office-add-ins-nodejs.md
:white_check_mark:Succeeded

docs/develop/overview-authn-authz.md
:white_check_mark:Succeeded

docs/develop/sso-in-office-add-ins.md
:white_check_mark:Succeeded

docs/develop/troubleshoot-sso-in-office-add-ins.md
:white_check_mark:Succeeded

docs/develop/use-the-oauth-authorization-framework-in-an-office-add-in.md
:white_check_mark:Succeeded

docs/excel/custom-functions-debugging.md
:white_check_mark:Succeeded

docs/excel/custom-functions-errors.md
:white_check_mark:Succeeded

docs/images/auth-decisions-flowchart.gif
:white_check_mark:Succeeded

docs/images/auth-decisions-flowchart.vsdx
:white_check_mark:Succeeded

docs/images/yo-office-excel-project.png
:white_check_mark:Succeeded

docs/includes/register-sso-add-in-aad-v2-include.md
:white_check_mark:Succeeded

docs/reference/javascript-api-for-office-error-codes.md
:white_check_mark:Succeeded

docs/reference/objectmodel/preview-requirement-set/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/preview-requirement-set/outlook-requirement-set-preview.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.1/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.2/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.3/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.4/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.5/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.6/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.7/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.8/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/requirement-sets/office-add-in-requirement-sets.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

docs/tutorials/share-data-and-events-between-custom-functions-and-the-task-pane-tutorial.md
:white_check_mark:Succeeded

docs/word/word-add-ins-programming-overview.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[dashboard-request] Dasboard-Vaccinations.js

name of dashboard

Dasboard-Vaccinations.js

description of the dashboard

A dashboard showing vaccination progress across a country, it's NHS-boards, and its councils.

dashboard JS filename

dashboard-vaccination.js

specific datastreams to include (which file, which URL, you can link to your spreadsheet if you want)

https://www.opendata.nhs.scot/dataset/6dbdd466-45e3-4348-9ee3-1eac72b5a592/resource/42f17a3c-a4db-4965-ba68-3dffe6bca13a/download/daily_vacc_scot_20210514.csv
https://www.opendata.nhs.scot/dataset/6dbdd466-45e3-4348-9ee3-1eac72b5a592/resource/9b99e278-b8d8-47df-8d7a-a8cf98519ac1/download/daily_vacc_age_sex_20210514.csv
https://www.opendata.nhs.scot/dataset/6dbdd466-45e3-4348-9ee3-1eac72b5a592/resource/d5ffffc0-f6f3-4b76-8f38-71ccfd7747a4/download/daily_vacc_la_20210514.csv
https://www.opendata.nhs.scot/dataset/6dbdd466-45e3-4348-9ee3-1eac72b5a592/resource/758f72d6-7371-4eee-9e6b-0b0798470d7e/download/daily_vacc_hb_20210514.csv

You are right. I already have that.


A few final logging touch-ups

Pick up from things I noticed in #4293. These are tiny non-functional / cosmetic changes only:

Remove two unused events (they were removed in #4104)
More structured ConcurrencyLine
Instead of starting or ending messages with \n, precede/follow them with EmptyLine() events. Note: The legacy logger used to wrap these in with TextOnly(), in order to avoid writing them to JSON-formatted logs. I don't care too much about the legacy logger, but I bet we could do something very simple to achieve the same behavior with the new event logger, that is, avoid firing EmptyLine() events when JSON formatting is enabled.

Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
~This PR includes tests, or tests are not required/relevant for this PR~
~I have updated the CHANGELOG.md and added information about my change~ — centralizing in #4389

... avoid firing EmptyLine() events when JSON formatting is enabled.
yup. we really should do this.


Added Contact Page and did routing for the same

closes issue #18

Added padding
Mobile View

Laptop and Tab view

I have changed the route to /user/contact

change the route path to /contact only

Done!


Widen "Uses a testing framework" to "Uses testing"

Not sure how feasible this is without shallow cloning from https://github.com/cran but the idea is that you only check for a non-empty tests/ directory. I'm unsure if the GitHub api allows you to inspect the directory structures and avoid a shallow clone :man_shrugging:

I'm unsure if the GitHub api allows you to inspect the directory structures and avoid a shallow clone

Yes, this is technically possible

We had a related discussion yesterday. In particular with the example of data.table which doesn't use a testing framework.
I think it then potentially opens the question of "is it still best practice?" :thinking:. I chatted with data.table contributors recently who shared how the unusual infrastructure made it more difficult for them to contribute.
My view is probably that:
no tests < tests with no testing framework < testing framework


libraw: 0.18.9 -> 0.18.10

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/libraw/versions.
These checks were done:

built on NixOS
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/raw-identify -h’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/raw-identify --help’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/raw-identify help’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/unprocessed_raw -h’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/unprocessed_raw --help’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/unprocessed_raw help’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/4channels -h’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/4channels --help’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/4channels help’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/simple_dcraw -h’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/simple_dcraw --help’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/simple_dcraw help’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/mem_image -h’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/mem_image --help’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/mem_image help’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/dcraw_half -h’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/dcraw_half --help’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/dcraw_half help’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/half_mt -h’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/half_mt --help’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/half_mt help’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/multirender_test -h’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/multirender_test --help’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/multirender_test help’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/postprocessing_benchmark -h’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/postprocessing_benchmark help’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/dcraw_emu -h’ got 0 exit code
ran ‘/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10/bin/dcraw_emu help’ got 0 exit code
found 0.18.10 with grep in /nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10
directory tree listing: https://gist.github.com/8de105fd367301af8971e9bc3012a0d3

Success on x86_64-linux (full log)
Attempted: libraw
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/l6qq4p0al8m5x5jdby51bzab9lq0da5i-libraw-0.18.10-dev
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/l6qq4p0al8m5x5jdby51bzab9lq0da5i-libraw-0.18.10-dev/lib
patching script interpreter paths in /nix/store/l6qq4p0al8m5x5jdby51bzab9lq0da5i-libraw-0.18.10-dev
checking for references to /build in /nix/store/l6qq4p0al8m5x5jdby51bzab9lq0da5i-libraw-0.18.10-dev...
shrinking RPATHs of ELF executables and libraries in /nix/store/sdjm7j0qjss0cxs4vcg8j4kbyzs46s35-libraw-0.18.10-doc
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/sdjm7j0qjss0cxs4vcg8j4kbyzs46s35-libraw-0.18.10-doc
checking for references to /build in /nix/store/sdjm7j0qjss0cxs4vcg8j4kbyzs46s35-libraw-0.18.10-doc...
/nix/store/2x920hvy7c1w6m9jv3w592ksj4vmnpyj-libraw-0.18.10

Success on aarch64-linux (full log)
Attempted: libraw
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/i8q3ihv4png254h41vd9vahx2xg3v3dj-libraw-0.18.10-dev
strip is /nix/store/gp7fylxwn18b7pl2c18ks89hsiaxyfvf-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/i8q3ihv4png254h41vd9vahx2xg3v3dj-libraw-0.18.10-dev/lib
patching script interpreter paths in /nix/store/i8q3ihv4png254h41vd9vahx2xg3v3dj-libraw-0.18.10-dev
checking for references to /build in /nix/store/i8q3ihv4png254h41vd9vahx2xg3v3dj-libraw-0.18.10-dev...
shrinking RPATHs of ELF executables and libraries in /nix/store/hach2d246k3894kvix2kq6907mwijj7r-libraw-0.18.10-doc
strip is /nix/store/gp7fylxwn18b7pl2c18ks89hsiaxyfvf-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/hach2d246k3894kvix2kq6907mwijj7r-libraw-0.18.10-doc
checking for references to /build in /nix/store/hach2d246k3894kvix2kq6907mwijj7r-libraw-0.18.10-doc...
/nix/store/imacfs7yzygncrpqj8rad5l2573j8j5a-libraw-0.18.10

Success on x86_64-darwin (full log)
Attempted: libraw
Partial log (click to expand)
patching script interpreter paths in /nix/store/jqra3xcfnkmikfdv6vsj34dz3nsd4q6l-libraw-0.18.10
strip is /nix/store/kdff2gim6417493yha769kh00n63lnrw-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/z7zdw0fsc6xvx6vxh8ir75ijx4ssrdxd-libraw-0.18.10-lib/lib
patching script interpreter paths in /nix/store/z7zdw0fsc6xvx6vxh8ir75ijx4ssrdxd-libraw-0.18.10-lib
strip is /nix/store/kdff2gim6417493yha769kh00n63lnrw-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/ww9ffrgv0n9kc592ak15w21cqi2ic0fc-libraw-0.18.10-dev/lib
patching script interpreter paths in /nix/store/ww9ffrgv0n9kc592ak15w21cqi2ic0fc-libraw-0.18.10-dev
strip is /nix/store/kdff2gim6417493yha769kh00n63lnrw-cctools-binutils-darwin/bin/strip
patching script interpreter paths in /nix/store/bp6f7y3zv9fxsh831xpmj6ab2rhv3b5b-libraw-0.18.10-doc
/nix/store/jqra3xcfnkmikfdv6vsj34dz3nsd4q6l-libraw-0.18.10