A cookie associated with a cross-site resource  without the SameSite attribute

While access reports site in chrome 80, the console is warning：
A cookie associated with a cross-site resource at https://yourReportSite/ was set without the SameSite attribute. It has been blocked, as Chrome now only delivers cookies with cross-site requests if they are set with SameSite=None and Secure. You can review cookies in developer tools under Application>Storage>Cookies and see more details at https://www.chromestatus.com/feature/5088147346030592 and https://www.chromestatus.com/feature/5633521622188032.
How can I fixed it? Thanks!

The same happens with the most current update to the Edge browser based on Chromium.
As a workaround you can set the browser flag "SameSite by default cookies" to disabled - this works on both Chrome and Edge.
Not a perfect solution to tell every user to use this workaround or use Firefox instead...
I don't know if this can actually be fixed (other than Microsoft updates PBI) as the Power BI website uses .NET 4.7 - but the samesite attribute needs .NET 4.7.2 or 4.8 and newer to work.
I'm also desperatly need a real sulution to this problem.

The same happens with the most current update to the Edge browser based on Chromium.
As a workaround you can set the browser flag "SameSite by default cookies" to disabled - this works on both Chrome and Edge.
Not a perfect solution to tell every user to use this workaround or use Firefox instead...
I don't know if this can actually be fixed (other than Microsoft updates PBI) as the Power BI website uses .NET 4.0 - but the samesite attribute needs .NET 4.7.2 or 4.8 and newer to work.
I'm also desperatly need a real solution to this problem.

Thanks!!

The same happens with the most current update to the Edge browser based on Chromium.
As a workaround you can set the browser flag "SameSite by default cookies" to disabled - this works on both Chrome and Edge.
Not a perfect solution to tell every user to use this workaround or use Firefox instead...
I don't know if this can actually be fixed (other than Microsoft updates PBI) as the Power BI website uses .NET 4.0 - but the samesite attribute needs .NET 4.7.2 or 4.8 and newer to work.
I'm also desperatly need a real solution to this problem.

Thanks a lot!!

Which version of the product are you using? You can set these two properties in your web.config file:
https://docs.microsoft.com/en-us/dotnet/api/system.web.security.formsauthentication.requiressl
https://docs.microsoft.com/en-us/dotnet/api/system.web.security.formsauthentication.cookiesamesite
Let me know if you still have problems after doing this.

I did set:
<forms ... cookieSameSite="None" requireSSL="true">
and

in the web.config of the reportserver ... and did the same for the portal site where I create the auth cookie.
Can you confirm that you actually see custom visuals in Power BI report server using custum authentication in your environment?
As this is the only visible problem seen by the enduser in Chrome or Edge (Chromium) ... (both browsers must be the most current versions)

Which version of the product are you using? You can set these two properties in your web.config file:
https://docs.microsoft.com/en-us/dotnet/api/system.web.security.formsauthentication.requiressl
https://docs.microsoft.com/en-us/dotnet/api/system.web.security.formsauthentication.cookiesamesite
Let me know if you still have problems after doing this.

Dear Gboreki, My PowerBi version is 1.8.7485.35104 (May 2020), I set the web.config like follow, but get the same warning.

Which version of the product are you using? You can set these two properties in your web.config file:
https://docs.microsoft.com/en-us/dotnet/api/system.web.security.formsauthentication.requiressl
https://docs.microsoft.com/en-us/dotnet/api/system.web.security.formsauthentication.cookiesamesite
Let me know if you still have problems after doing this.

Dear Gboreki, My PowerBi version is 1.8.7485.35104 (May 2020), I set the web.config like follow, but get the same warning.

I did set:
<forms ... cookieSameSite="None" requireSSL="true"></forms>
and
<httpCookies sameSite="None" requireSSL="true" />
in the web.config of the reportserver ... and did the same for the portal site where I create the auth cookie.
Can you confirm that you actually see custom visuals in Power BI report server using custum authentication in your environment?
As this is the only visible problem seen by the enduser in Chrome or Edge (Chromium) ... (both browsers must be the most current versions)
I'm using PBI Report Server Mai 2020 ...

hi lsteffens, I use powerbi May 2020, and set the custume authentication step by step as  https://github.com/microsoft/Reporting-Services/tree/master/CustomSecuritySample2016, and it works OK except get the cookieSameSite warning &  401 error.
GET https://mybisite/powerbi/libs/scripts/globalize.js net::ERR_ABORTED 401.

hi lsteffens, I use powerbi May 2020, and set the custume authentication step by step as https://github.com/microsoft/Reporting-Services/tree/master/CustomSecuritySample2016, and it works OK except get the cookieSameSite warning & 401 error.
GET https://mybisite/powerbi/libs/scripts/globalize.js net::ERR_ABORTED 401.

Hi yls087412. This is the behavior I see on my side ... but I get a bunch (~20) 401 errors including yours.
Are custom visuals (like Text Filter or Tag Cloud) visible for you?
(using the most current Chrome or Edge (Chromium!) browser....)
Everything worked perfectly until Chrome and Edge updated their browsers to use "SameSite by default cookies" flag.
Firefox still works ok - but with warnings about the SameSite attribute.

The October 2020 release of Power BI works with Samesite default settings again.
It took some time, but this bug was fixed at last.

The October 2020 release of Power BI works with Samesite default settings again.
It took some time, but this bug was fixed at last.

It is a good news! thank you very much!


Create and automate actions to compile data-analysis into runnable CLI executables

Is your feature request related to a problem? Please describe.
Currently, users have to manually install python and run from command line.
Describe the solution you'd like
As an alternative to the GUI solution, we should compile the existing superscript.py script into runnable executables for win10 and linux.
Additional context
Potential python to exe tools:

cx_Freeze for Windows, Linux, and Mac OS X (Python 2.7, 3.x)
pyinstaller for Windows, Linux, and Mac OS X (Python 2.7, 3.4-3.7)
bbfreeze for Windows and Linux (Python 2.4-2.7)
py2exe for Windows (Python 2.6, 2.7)
py2exe for Windows (Python 3.3-3.5)
pytoexe for Windows (Python 2, 3)
Freeze for Linux and maybe Mac OS X (Python 2.x)
py2app for Mac OS X (Python 2.x)

Reference: https://codeburst.io/building-beautiful-command-line-interfaces-with-python-26c7e1bb54df


Permission error in task k3s.orchestration.k3s_server : Copy kubeconfig to control node

Hi!
Creating a cluster drops the error:
TASK [k3s.orchestration.k3s_server : Copy kubeconfig to control node] *****************************************************************
fatal: [hypericum]: FAILED! => {"msg": "error while accessing the file /etc/rancher/k3s/k3s.yaml, error was: [Errno 13] Keine Berechtigung: b'/etc/rancher/k3s/k3s.yaml'"}

So the permission for the file /etc/rancher/k3s/k3s.yaml seem to be lacking.
ansible-playbook was called with -K so a become-password is available to the play.
System is a debian bullseye server.
Cheers,
Jörn

What does your inventory.yaml look like. I am unable to reproduce this issue.
TASK [k3s_server : Copy kubeconfig to control node] ****************************
changed: [server-0]


[Thinkit] Move where the DeviceId mock call happens in testing, Cleanup Switch Test Helper Tests, Fix ASAN error in test, Generalize setup helpers to make config push optional & Disable failing smoke_test while SAI fix is investigated.

Co-authored-by: Robert Halstead<EMAIL_ADDRESS>Jonathan Dilorenzo<EMAIL_ADDRESS>& Steffen Smolka<EMAIL_ADDRESS>


Update README URLs based on HTTP redirects

From #1102
Created with https://github.com/dkhamsing/frankenstein
GitHub Corrected URLs

Was
Now

https://github.com/czechboy0/Redbird
https://github.com/vapor/Redbird

https://github.com/czechboy0/Socks
https://github.com/vapor/Socks

          1 Warning

  :warning:
  Too many changes (when adding, please keep it to one project per Pull Request)

  Generated by :no_entry_sign: danger


Test Bob

We need to add tests to builder.go

This will have to wait until #21
In the meantime, some initial work was done with cd0180b0d9b7279d11d8c41bf0b5b5f3c8be512e, 06561f8e44697455314d1a43e224c51144b87d4d and 73b63e14baa017a69cd9042f76aaa8de40b99f4b


Noxious15

@noxious15

Ok

Thanku

Hi all
On Wed, Apr 27, 2022, 18:45 V1e2n3k4a5t6e7s8w9a10r11l12u13 <
@.***> wrote:

Thanku
—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/26#issuecomment-1110986872,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWBXKFC7L55ETS2MKCUFM73VHE4XFANCNFSM5UGMGFUQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Ok

Hi

这是来自QQ邮箱的假期自动回复邮件。 您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

Ok


Added subsection support to the FlowsheetDataGrid

Summary

What was changed:
Subsection support was added to the FlowsheetDataGrid component.
Why it was changed:
The change was made to satisfy a functional requirement of a consumer of the FlowsheetDataGrid component.
Testing

This change was tested using:

[X] WDIO
[X] Jest
[ ] Visual testing (please attach a screenshot or recording)
[ ] Other (please describe below)
[ ] No tests are needed

Reviews
In addition to engineering reviews, this PR needs:

[ ] UX review
[ ] Accessibility review
[X] Functional review

Additional Details

This PR resolves:
UXPLATFORM-10360

Thank you for contributing to Terra.
@cerner/terra

@cm9361 - I validated with Chrome, Edge, and JAWS with Edge. The KB navigation works as required. Screen reader navigation and announcements work as required. I observed one minor issue. If I am on a row in the body of the grid and press the End key (fn+right arrow on my MacBook), JAWS announces the labels of all the sections and subsections, regardless where the focus is located. For example, if focus is in column 6 row 4 in section 1, subsection 1, JAWS announces "End, Primary Contact, Test Section, Test Subsection, Test Subsection number 2, Test Section number 2, Quinzel, Harleen, Column 6"
Some thing similar happens if I navigate from a column header with the Down Arrow. For example when I navigate the last column to column 6 row 4. JAWS announces, "Fleck, Arthur, Primary Contact, Test Section, Test Subsection, Test Subsection number 2, Test Section number 2, Quinzel, Harleen, Column 6 row 4"
This happens for columns 2 - 5 for any cell that follows a section or subsection header.

@cm9361 - I validated with Chrome, Edge, and JAWS with Edge. The KB navigation works as required. Screen reader navigation and announcements work as required. I observed one minor issue. If I am on a row in the body of the grid and press the End key (fn+right arrow on my MacBook), JAWS announces the labels of all the sections and subsections, regardless where the focus is located. For example, if focus is in column 6 row 4 in section 1, subsection 1, JAWS announces "End, Primary Contact, Test Section, Test Subsection, Test Subsection number 2, Test Section number 2, Quinzel, Harleen, Column 6" Some thing similar happens if I navigate from a column header with the Down Arrow. For example when I navigate the last column to column 6 row 4. JAWS announces, "Fleck, Arthur, Primary Contact, Test Section, Test Subsection, Test Subsection number 2, Test Section number 2, Quinzel, Harleen, Column 6 row 4" This happens for columns 2 - 5 for any cell that follows a section or subsection header.

@mjpalazzo - Thank you for the feedback.  Isn't the issue you described a known pre-existing issue?  I thought we landed on the fact that the attributes were semantically correct in this scenario.  If it is the pre-existing issue and we want to address it somehow, do we need a JIRA for the work.  I thought we had one open previously.

@cm9361 - I validated with Chrome, Edge, and JAWS with Edge. The KB navigation works as required. Screen reader navigation and announcements work as required. I observed one minor issue. If I am on a row in the body of the grid and press the End key (fn+right arrow on my MacBook), JAWS announces the labels of all the sections and subsections, regardless where the focus is located. For example, if focus is in column 6 row 4 in section 1, subsection 1, JAWS announces "End, Primary Contact, Test Section, Test Subsection, Test Subsection number 2, Test Section number 2, Quinzel, Harleen, Column 6" Some thing similar happens if I navigate from a column header with the Down Arrow. For example when I navigate the last column to column 6 row 4. JAWS announces, "Fleck, Arthur, Primary Contact, Test Section, Test Subsection, Test Subsection number 2, Test Section number 2, Quinzel, Harleen, Column 6 row 4" This happens for columns 2 - 5 for any cell that follows a section or subsection header.

@mjpalazzo - Thank you for the feedback. Isn't the issue you described a known pre-existing issue? I thought we landed on the fact that the attributes were semantically correct in this scenario. If it is the pre-existing issue and we want to address it somehow, do we need a JIRA for the work. I thought we had one open previously.

@cm9361 - you are correct. I had forgotten that this is a pre-existing issue. We may already have a Jira for it. I will follow up with Chris. As I said it is minor and does not affect funtionality, so we will deal with it in the future. Your changes are working well so I approve.

@mjpalazzo @eawww When navigating to the Grid it announces the # of rows and columns, should it also announce the number of sections contained within the grid?   I was unable to find an ARIA example that had sections to test to see how they handled it.

@mjpalazzo @eawww When navigating to the Grid it announces the # of rows and columns, should it also announce the number of sections contained within the grid? I was unable to find an ARIA example that had sections to test to see how they handled it.

@chrismichalewicz - The table does not have a concept of sections.  That is functionality that we added.  The screen reader would not have a way to announce that.

@mjpalazzo @eawww When navigating to the Grid it announces the # of rows and columns, should it also announce the number of sections contained within the grid? I was unable to find an ARIA example that had sections to test to see how they handled it.

@chrismichalewicz - The table does not have a concept of sections. That is functionality that we added. The screen reader would not have a way to announce that.

Thanks @cm9361 for clarifying! It works well for me as well, great job! +1 I will also add the Functional Verified label.

@mjpalazzo @eawww When navigating to the Grid it announces the # of rows and columns, should it also announce the number of sections contained within the grid? I was unable to find an ARIA example that had sections to test to see how they handled it.

@chrismichalewicz - The table does not have a concept of sections. That is functionality that we added. The screen reader would not have a way to announce that.

Thanks @cm9361 for clarifying! It works well for me as well, great job! +1 I will also add the Functionally Reviewed label.

Agreed...WAI and MDN have notthing to say about tables with sections.


ZIPKIN_UI_BASEPATH doesn't seem to work with 3.1.0

We have upgraded to 3.1.0. We are using ZIPKIN_UI_BASEPATH to rewrite the base path for the UI, but it doesn't seem to work anymore. Tried running:
docker run -p 9411:9411 openzipkin/zipkin:3.1.0

and the UI shows nicely on http://localhost:9411 (redirects to http://localhost:9411/zipkin/). With
docker run -e ZIPKIN_UI_BASEPATH=/admin/zipkin -p 9411:9411 openzipkin/zipkin:3.1.0

it still redirects to http://localhost:9411/zipkin/ but shows blank page. If I try localhost:9411/admin/zipkin/ I get 404 Not Found.
Perhaps I am missing some configuration changes for the upgrade?

@ujo-trackunit so ZIPKIN_UI_BASEPATH only rewrites links so that when a proxy has zipkin under another path, the links reflects the expectations of the proxy. zipkin-server will still host everything under /zipkin
I'm not saying there isn't an issue with the link rewriting, but that's what the feature was for..
ProxyPass "/proxy/foo/myzipkin"  "http://localhost:9411/zipkin/"
ProxyPassReverse "/proxy/foo/myzipkin"  "http://localhost:9411/zipkin/"

https://github.com/openzipkin/zipkin/tree/master/zipkin-lens#running-behind-a-reverse-proxy

@ujo-trackunit can you mention the version you were moving from? I'd like to try to isolate what's going on..

We are running 3.0.6

What I am seeing is that stuff is not kept under /zipkin/ - I see the blank page instead.

ok I think maybe found it..
$ docker run --rm -e ZIPKIN_UI_BASEPATH=/admin/zipkin -p 9411:9411 openzipkin/zipkin:3.0.6
$ curl -s localhost:9411/zipkin/index.html |tidy -q -i --tidy-mark no --wrap 1024
line 1 column 22 - Warning: inserting missing 'title' element
<!DOCTYPE html>

  <base href="/admin/zipkin/">
  <meta charset="utf-8">
  <link rel="icon" href="./favicon.ico">

  <link href="./static/css/main.311b483b.css" rel="stylesheet">
  <title></title>

  You need to enable JavaScript to run this app.

Now, the href links are not relative (also we didn't strip comments but that's not a big deal)
$ docker run --rm -e ZIPKIN_UI_BASEPATH=/admin/zipkin -p 9412:9411 openzipkin/zipkin:3.1.0
$ curl -s localhost:9412/zipkin/index.html |tidy -q -i --tidy-mark no --wrap 1024
curl -s localhost:9412/zipkin/index.html |tidy -q -i --tidy-mark no --wrap 1024
line 18 column 3 - Warning: inserting missing 'title' element

<!DOCTYPE html>

  <base href="/admin/zipkin/">
  <meta charset="utf-8">
  <link rel="icon" href="/zipkin/favicon.ico">

  <link rel="stylesheet" href="/zipkin/static/css/index.a7aa86b4.css">
  <title></title>

  You need to enable JavaScript to run this app.

so I think the following change gets it working again, but it also breaks the zipkin-ui docker demo ;)
cc @openzipkin/ui as while I tend to pull a large share of troubleshooting, probably more healthy if someone else helps.
basically, we both want to have ZIPKIN_UI_BASEPATH working (as this issue suggests) and ZIPKIN_BASE_URL also working.
We don't have any integration tests for either, closest is that ZIPKIN_UI_BASEPATH is documented in zipkin-lens/README and  ZIPKIN_BASE_URL is used in the docker/examples for zipkin-ui
--- a/zipkin-lens/vite.config.ts
+++ b/zipkin-lens/vite.config.ts
@@ -17,9 +17,9 @@ import {defineConfig} from 'vite'
 import * as path from "path";

 // baseUrl is the default path to lookup assets.
-const baseUrl = process.env.BASE_URL || '/zipkin';
+const baseUrl = process.env.BASE_URL || './';
 // basePath is the default path to get dynamic resources from zipkin.
-const basePath = process.env.BASE_PATH || baseUrl;
+const basePath = process.env.BASE_PATH || '/zipkin';

 const zipkinProxyConfig = {
   target: process.env.API_BASE || 'http://localhost:9411',

unmarking myself from this as I'm pulling too many hours here and not enough on day job. 🤞 someone else can rescue this time!

@reta @anuraaga I'm actually inclined to roll back lens to react-scripts and re-raise vite progress here on another branch until it is fully tested and works on all scenarios we supported before. The only reason we changed to vite was because of dev dependency CVE, which is not a good reason to break production users. We can then take more time, and possibly we can all learn from this about merging change to major pieces. There were several PRs after the first commit to get things working and we still aren't done. This is a lot of human effort for a non-prod CVE, even if we re-merge it later. Thoughts?

@reta @anuraaga I'm actually inclined to roll back lens to react-scripts and re-raise vite progress here on another branch until it is fully tested and works on all scenarios we supported before. The only reason we changed to vite was because of dev dependency CVE, which is not a good reason to break production users. We can then take more time, and possibly we can all learn from this about merging change to major pieces without testing everything in and outside docker. There were several PRs after the first commit to get things working and we still aren't done. This is a lot of human effort for a non-prod CVE, even if we re-merge it later. Thoughts?

I'll give it a try. Sorry for your frustration over the frontend changes. Maybe I can come up with a solution this evening or I'll try it this weekend.

I'll give it a try. Sorry for your frustration over the frontend changes. Maybe I can come up with a solution this evening or I'll try it this weekend.

@SamTV12345 the main frustration is that I am not good at it, so for example, it takes as long as me to troubleshoot something simple on the JS side vs supporting a new version of cassandra or something. Also, each and any frontend related PR tends to need a lot of feedback to merge, which again is the same effort I would usually spend on a new feature. Sometimes over several days.. So there's displacement and a lot of time with each thing, often unplanned because I'm not aware there is a problem until someone notices..
Thanks for giving it a stab, please do run the docker/examples/README example for the UI after the fix to make sure this still works. I'll do that when you say it is working as well. Once whatever is merged that doesn't break the UI example, the ghcr.io/openzipkin/zipkin:master tag can be updated and @ujo-trackunit can verify.
If it turns out we can't sort this by early next week, I will raise a PR to revert vite and make a patch release that gets the UI working again, then raise a shared branch with vite again. This is surely not my preference, as it makes more work in the future to finish that. However, we can't leave this broken as it was a feature for many years.

@codefromthecrypt I've got a fix ready for review and testing. The paths are now correctly rewritten.

@SamTV12345 thanks. I'll leave comments but looks reasonable!

Hi - still seeing some issues - running with same config as for 3.0.6 - but seeing:

Missing some static resources and the "Run query" bar. This is a screen shot from 3.0.6:

ok before, the other assets were looked up relative like this. I'm not sure why now they are absolute, but perhaps at least when baseUrl is set, we can override it to relative. I'll raise a test PR we can revert if it doesn't fix things.. https://github.com/openzipkin/zipkin/issues/3742#issuecomment-1970958656

@ujo-trackunit can you try again please?

I think this never worked properly as it was never tested except ad-hoc. I have an httpd container in progress locally which needs a couple changes to fully work. will update

Testing with openzipkin/zipkin:latest@sha256:be6eedb41ecae348c1d61abe9b9f22e5af582604fec4ad12e4fa782e99746a6f I get blank page with 3.0.6 config.

Thanks @ujo-trackunit I know that https://github.com/openzipkin/zipkin/pull/3751 is part of the fix, just I noticed a glitch where if you click on something like Dependencies link, it will revert the URL back to /zipkin (even though it works, this is a problem for bookmarking). So, I'll ping you again once we fix the last dent.

@ujo-trackunit sorry this took so long. It has been on my mind every day, and 🤞 this is the last one.
I created an example that should have been added several years ago when that env variable was documented. That was my fault as it was on my watch back then. Anyway it is here now https://github.com/openzipkin/zipkin/tree/master/docker/examples#ui-proxy
I started the new example which defaults to /admin/zipkin
$ cd docker/examples
$ TAG=master docker-compose -f docker-compose.yml -f docker-compose-uiproxy.yml pull
$ TAG=master docker-compose -f docker-compose.yml -f docker-compose-uiproxy.yml up

Then, I checked the UI at http://localhost/admin/zipkin/ and it worked

and also curl looks sane, too
$ curl -s localhost:9411/zipkin/index.html |tidy -q -i --tidy-mark no --wrap 1024

<!DOCTYPE html>

  <base href="/admin/zipkin/">
  <meta charset="utf-8">
  <link rel="icon" href="./favicon.ico">
  <title>Zipkin</title>

  <link rel="stylesheet" href="./static/css/index.a7aa86b4.css">

  You need to enable JavaScript to run this app.

If you can try one more time pulling the master tag, let's see if it sorts you. If not, please paste the curl output you get.  Tested with:
:: version 3.1.1-SNAPSHOT :: commit 2bbc4bb ::

and it looks good - also did searches and filtering:

Thank you for looking into this.

3.1.1 going out now with this fixed! Thanks, all!


Grey out non-changed quota fields and only show changed ones (Jason L., Billy, Zhanna)

Grey out non-changed quota fields and only show changed ones (Jason L., Billy, Zhanna)
https://www.figma.com/file/jsrhMxnmN4awqlbU8LuyuK/Quota-Redesign?type=design&node-id=291-5659&mode=design&t=6sddqQu4TOrjUKX7-0

Example how it looked in the Previous Registry (the reason why Billy still uses old UI):

The Quota change is now more visible, like this:


Store joined to a intermediate store is not reflected in cube

Description
I have the below 3 stores:
netflix_store [key: "show_id"]
country_store [key: "show_id", "country"]
location_store [key: "country"]
neflix_store has a many-to-many relationship to country_store.
country_store has many-to-one relationship to location_store.
I am unable to see the hierarchy (country_code) and measures (longitude and latitude) from location_store after joining to country_store.

Reproduce
Please see attached zip
00_netflix.zip
Context

Atoti version: '<IP_ADDRESS>00323192733'
Python version: 3.7.6
Operating System: Windows 10 Pro

Logs (if relevant)

To reproduce:
import atoti as tt
import pandas as pd

session = tt.create_session()
dfa = pd.DataFrame({
    "a": ["a","b", "c"],
    "b": [10.0,20.0,30.0]
})
dfb = pd.DataFrame({
    "a": ["a","b", "c", "a", "b", "c"],
    "c": [1,1,1,2,2,2],
    "d": [100.0,200.0,300.0,400.0,500.0,600.0]
})
dfc = pd.DataFrame({
    "c": [1,2],
    "e": [1000.0,2000.0]
})
storeA = session.read_pandas(dfa, keys=["a"])
storeB = session.read_pandas(dfb, keys=["a", "c"])
storeC = session.read_pandas(dfc, keys=["c"])
storeA.join(storeB)
storeB.join(storeC)

cube = session.create_cube(storeA)
m = cube.measures
m["sumE"] = tt.agg.sum(storeC["e"])

Hello @HuifangYeo
This issue with joins will be fixed in the next release.


[Crash] Simple Copper Pipes 2.0.1 and Create Fabric 0.5.1-f-build.1417

Minecraft: 1.20.1
Create Fabric: 0.5.1-f-build.1417+mc1.20.1.jar
Fabric API: 0.92.2+1.20.11.jar
Simple Copper Pipes: SimpleCopperPipes-mc1.20.1-2.0.11.jar
The game wont start with the lastest SimpleCopperPipes-mc1.20.1-2.0.11.jar update
https://mclo.gs/vatuYFn
I made a new instance with the only required mods to make it easier

This has been fixed in release 2.0.2


Bumping a package version isn't reflected in the main site or CLI

I recently made a change to stableheapbtreemap and tried to bump the package version through mops (to mirror github tag)
Result:
% mops bump minor 0.3.3
Current version: 1.3.0
Updated version: 1.4.0

However, the main site still shows 1.3.0 as the current version, and the search command shows this too.
% mops search btree

NAME               | DESCRIPTION                                                                                | VERSION | UPDATED
────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
stableheapbtreemap | An imperative BTreeMap that can persist across upgrades via a stable variable in heap memory | 1.3.0   | 2023-05-29
rxmodb             | RxMoDb is a Motoko Database using Vector for its main storage and BTree for indexes          | 0.1.4   | 2023-07-08

Am I correctly bumping the package version, or is there a different way to sync this with mops?
Also, as an aside I'd like to be able to override the version automatically assigned by mops.

Thanks for making mops btw! It gets better and better as more packages and features are being added!

Am I correctly bumping the package version, or is there a different way to sync this with mops?

'mops bump' only changes version in mops.toml file, you also need to run 'mops publish' to publish it to mops.

Also, as an aside I'd like to be able to override the version automatically assigned by mops.

Version that is assigned on 'mops init'? Which version do you prefer?


Unable to use global host event handler

Hi,
I'm trying to use a global host event handler thru:
(showing only this attribute)
default_attributes(
:nagios => {
:conf => {
:global_host_event_handler => 'downscale_handler'
}
}
)
Which creates on nagios.conf this:
global_host_event_handler=downscale_handler
The problem is that the handler never gets called when a host is down...
Any ideas?
Thanks

My bad. it's properly working :)


Fix unix timestamps conversion #990

@Kartones mini-review, please (related to the previous one)
https://github.com/CartoDB/cartodb-postgresql/releases/tag/0.7.2

+1

Frontend tests were OK :+1: (details)


Slower than tf-faster-rcnn

This code is taking more time to train than the tf-faster-rcnn. When I am running at 1800 scale this code is taking 2.9 sec /iter as compared to 1.7sec/iter in tf-faster-rcnn.

Also going out of memory at 1800 input scale ( my gpu had 16gb of gpu memory) with this code, whereas tf-faster-rcnn isn't.

Are you referring to training times or during prediction?

Training time.


feat: add config settings to action

Description
Configuration and documentation added to the source code.
To-do list

[ ] ~I added appropriate tests for my changes.~ COMING SOON
[x] If my change impacts documentation, I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.

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


Popover doesnt close when navigating

When u have pop over on auto close when anchor is out of the screen, and u navigate to the prev page using the browser the pop over remains open

I think that it's up to you to close the popover if you want to.
@chrismcv Though?

it seems to work here http://material-ui.com/#/components/popover
make sure you are handling onRequestClose

This is a usage problem. I'm closing this issue. Feel free to reopen if this was a bug or the usage seems unclear.

@subjectix do you think we should change the PropType to isRequired ?


Make UNone extend WrappedNone, with a depth of 0.

This design removes one case in USome.apply().
However, it does require the public object UNone to extend the private[uoption] class WrappedNone, a design I'm not really fond of.

However, it does require the public object UNone to extend the private[uoption] class WrappedNone, a design I'm not really fond of.

How about:
diff --git src/main/scala/uoption/package.scala src/main/scala/uoption/package.scala
index a153621..02b5ccf 100644
--- src/main/scala/uoption/package.scala
+++ src/main/scala/uoption/package.scala
@@ -14,11 +14,12 @@ package object uoption {
     override def toString(): String = stringRepr
   }

-  object UNone extends WrappedNone(0, null) {
+  type UNone <: AnyRef
+  val UNone: UNone = new WrappedNone(0, null) {
     override def toString(): String = "UNone"
-  }
+  }.asInstanceOf[UNone]

-  type UOption[+A] >: UNone.type <: AnyRef
+  type UOption[+A] >: UNone <: AnyRef

   object UOption {
     @inline // only for Scala.js?

?

val UNone: UNone

Unfortunately that increases the amount of instructions necessary to load the instance. It's a module-load (i.e., a static field read) plus a field access, instead of just a module-load.
Moreover, it changes the type inferred for UNone, as it will be UNone instead of UNone.type. I'm afraid of the consequences of such a change.

I see what you mean.  UNone's .type is UNone, but the UNone type isn't a singleton.  But I'm not sure what consequence you're afraid of.

But I'm not sure what consequence you're afraid of.

Me neither, which is why I'm afraid of them. ;)


Doesn't  compile when using react-refresh-webpack-plugin

Describe the bug
Hi there!
I'm currently developing a react app with typescript, babel and webpack. I use react-refresh-webpack-plugin for fast refresh and it works well except in one case:
when a file in the dependency tree of a *.css.ts file contains a piece of code flagged as "dynamic state" executed during file evaluation.
In the linked reproduction repository, I have only one .css.ts file (src/global.css.ts). It imports another file (src/colors.ts). This file exports the variable Colors which is built when evaluating the file.
I did a little digging and here's what I found:

react-refresh generate code to register what could change between two refreshes.
In the repro, it add two lines at the end of the src/colors.ts file:__webpack_require__.$Refresh$.register(_c, "Colors$Object.entries(colors).reduce");
__webpack_require__.$Refresh$.register(_c2, "Colors");

__webpack_require__.$Refresh$ is undefined when running evalCode on the source in the processVanillaFile function, resulting in a TypeError

Unfortunately, my understanding on this stops here, but it doesn't seem right to have those lines in a source only used to generate css files.
Reproduction
https://github.com/elindorath/vanilla-extract-repro
System Info
System:
  OS: macOS 12.3
  CPU: (10) arm64 Apple M1 Pro
  Memory: 92.61 MB / 16.00 GB
  Shell: 5.9 - /opt/homebrew/opt/zsh/bin/zsh
Binaries:
  Node: 18.11.0 - ~/.config/asdf/installs/nodejs/18.11.0/bin/node
  Yarn: 1.22.19 - ~/.config/asdf/installs/yarn/1.22.19/bin/yarn
  npm: 8.19.2 - ~/.config/asdf/installs/nodejs/18.11.0/bin/npm
Browsers:
  Chrome: 106.0.5249.119
  Firefox: 102.0.1
  Safari: 15.4
npmPackages:
  @vanilla-extract/css: ^1.9.1 => 1.9.1
  @vanilla-extract/dynamic: ^2.0.2 => 2.0.2
  @vanilla-extract/sprinkles: ^1.5.0 => 1.5.0
  @vanilla-extract/webpack-plugin: ^2.2.0 => 2.2.0
  webpack: ^5.74.0 => 5.74.0

Used Package Manager
yarn
Logs
yarn run v1.22.19
$ NODE_ENV=development webpack serve --config webpack.config.ts --mode development
<i> [webpack-dev-server] Project is running at:
<i> [webpack-dev-server] Loopback: http://localhost:8080/
<i> [webpack-dev-server] On Your Network (IPv4): http://<IP_ADDRESS>:8080/
<i> [webpack-dev-server] On Your Network (IPv6): http://[fe80::1]:8080/
<i> [webpack-dev-server] Content not from webpack is served from '/Users/elindorath/dev/mine/vanilla-extract-repro/dist' directory
asset bundle.js 4.16 MiB [emitted] (name: main)
asset ./index.html 467 bytes [emitted]
runtime modules 32.2 KiB 16 modules
modules by path ./node_modules/core-js-pure/ 122 KiB 119 modules
modules by path ./node_modules/@pmmmwh/react-refresh-webpack-plugin/ 52.7 KiB 23 modules
modules by path ./node_modules/webpack-dev-server/client/ 55.8 KiB 12 modules
modules by path ./node_modules/webpack/hot/*.js 4.3 KiB 4 modules
modules by path ./node_modules/react/ 127 KiB 4 modules
modules by path ./node_modules/html-entities/lib/*.js 81.3 KiB 4 modules
modules by path ./node_modules/react-dom/ 1000 KiB 3 modules
modules by path ./src/ 1.29 KiB 2 modules
modules by path ./node_modules/react-refresh/ 20.2 KiB 2 modules
modules by path ./node_modules/scheduler/ 17.3 KiB 2 modules
+ 4 modules

ERROR in ./src/global.css.ts
Module build failed (from ./node_modules/@vanilla-extract/webpack-plugin/loader/dist/vanilla-extract-webpack-plugin-loader.cjs.js):
NonErrorEmittedError: (Emitted value instead of an instance of Error) TypeError: Cannot read properties of undefined (reading 'register')
    at processResult (/Users/elindorath/dev/mine/vanilla-extract-repro/node_modules/webpack/lib/NormalModule.js:755:12)
    at /Users/elindorath/dev/mine/vanilla-extract-repro/node_modules/webpack/lib/NormalModule.js:860:5
    at /Users/elindorath/dev/mine/vanilla-extract-repro/node_modules/loader-runner/lib/LoaderRunner.js:400:11
    at /Users/elindorath/dev/mine/vanilla-extract-repro/node_modules/loader-runner/lib/LoaderRunner.js:195:20
    at context.callback (/Users/elindorath/dev/mine/vanilla-extract-repro/node_modules/loader-runner/lib/LoaderRunner.js:124:13)
    at /Users/elindorath/dev/mine/vanilla-extract-repro/node_modules/@vanilla-extract/webpack-plugin/loader/dist/vanilla-extract-webpack-plugin-loader.cjs.dev.js:91:5
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
 @ ./src/main.tsx 4:0-22

webpack 5.74.0 compiled with 1 error in 937 ms

Validations

[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible example of the bug.

Similar issues were encountered in the Vite and Vitest environments
⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯ Failed Suites 1 ⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯

 FAIL  src/common/components/Toolbar/index.test.tsx [ src/common/components/Toolbar/index.test.tsx ]
TypeError: The argument 'path' must be a string or Uint8Array without null bytes. Received '\x00react/jsx-runtime'

I just hit a similar issue
In my case, the first change to a .css.ts/js file is fine, HMR works, and then, on the second change I get this:

Have you tried disabling react-refresh?

I see a similar issue using Storybook:

with the following config:
module.exports = {
  stories: ['../packages/**/*.stories.tsx'],
  addons: [
    '@storybook/addon-actions',
    '@storybook/addon-essentials',
    '@storybook/addon-knobs',
    '@storybook/addon-links',
    '@storybook/addon-storysource',
    '@storybook/preset-scss',
    'storybook-addon-swc',
  ],
  // https://storybook.js.org/docs/react/configure/typescript#mainjs-configuration
  typescript: {
    check: false,
  },
  core: {
    builder: 'webpack5',
    options: {
      lazyCompilation: true,
    },
  },
  framework: '@storybook/react',
  features: {
    storyStoreV7: true,
  },
};

@Elindorath Going to close this issue as it's a duplicate of #679 and #1043 (preferring 1043 as this issue has some other potentially unrelated comments). #679 contains a solution to the problem. In your case you can apply this fix by moving Colors to a separate file.
The issue arises from react refresh thinking that Colors is a component (more detailed explanation here). This issue will hopefully go away in the future as the webpack integration is migrated to the new VE compiler (currently still experimental).
@phil-lgr @okonet You may be able to fix your problems with the advice mentioned above, however if you believe your issues are caused by something else, please feel free to create a separate issue with a reproduction.


[DOC] Instruction for terms query with roaring bitmap needs to be fixed

What do you want to do?

[x] Request a change to existing documentation
[ ] Add new documentation
[x] Report a technical problem with the documentation
[ ] Other

Tell us about your request. Provide a summary of the request.
Instruction for terms query with roaring bitmap introduced since 2.17 does not work appropriately.  For more details of issue, please refer from https://forum.opensearch.org/t/bitmap-search-not-working/21631
I've confirmed roaring bitmap works well with fixed instruction. So I'd like submit fixed version to the repoisitory.
*Version: List the OpenSearch version to which this issue applies, e.g. 2.14, 2.12--2.14, or all.
2.17+
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.

https://github.com/opensearch-project/documentation-website/pull/8133
https://forum.opensearch.org/t/bitmap-search-not-working/21631

@tkykenmt: Thanks for submitting this issue! I'll submit a fix for this shortly.


Add a homepage

I'm not sure why npmjs.com isn't linking to the repo so this should help.

Thank you.

@f3l1x Oh I know why. It's because the metadata is based off of latest which is still outdated.
Perhaps you should publish to latest 😄


Platform-independent color formatter

By defining a macro TTTColor which evaluates to NSColor on macOS and UIColor on iOS/tvOS/watchOS, the TTTColorFormatter can be used regardless of the platform.
For backward compatibility the following methods are available for the corresponding platforms
- (NSString *)UIColorDeclarationFromColor:(UIColor *)color; // iOS,tvOS,watchOS
- (NSString *)NSColorDeclarationFromColor:(NSColor *)color; // macOS

Coverage decreased (-22.7%) to 15.023% when pulling c9cb591872819bd9398f193d64b91452e45d6eb0 on thomasguenzel:master into 92c662f360ac8de4b617633a9a3be31d5ffac2b3 on mattt:master.

Thanks for your PR, @thomasguenzel. Apologies for the delay in responding. I'm in the process of cutting a new release now.


Running QuickReturn sample

error: No resource identifier found for attribute 'cardCornerRadius' in package 'com.etiennelawlor.quickreturn'

I got the same error when importing the project to eclipse.

@feay1234 I have set up my gradle project in Android Studio.  I am not familiar with the exact configuration that is needed to make it run in Eclipse.

I have updated the README with simple setup instructions for Gradle or Maven.


There is no version 4.2.1 in the Helm repository.

In the Release page, milvus-4.2.1 is released.
But in the Helm repository, up to version 4.2.0 is released.

Hi @atuk721, thank you for pointing this out. I'll look on this.

I uploaded it manually. aritfacthub will be auto updated soon. Thank you again!

Hi @haorenfsa, I've confirmed Artifact HUB's version is updated.
Thank you for your quick response!


Issues with non-ascii folders on Windows

It is not possible to use configuration file which is located in a folder with non-ASCII characters on Windows. This is because simple char* is used to communicate filepath to a config file in Config::CreateFromFile().
In order to have full Unicode support it seems adding similar Config::CreateFromFile(wchar_t*) will be needed. Or at least will need to be done some ASCII->UTF conversion in the function itself.
In order to reproduce the issue it should be enough to simply call Config::CreateFromFile() with a filepath which has non-ascii characters, such as German umlauts for example.

Hi, would it be possible to request an update to this? This has major impact on loading from non-english environments on Windows. The fix doesn't seem to be too difficult either.

I am looking to get a pull request in for this hopefully later this week.

BTW, the way we solved this host-side was to call GetShortPathName Windows API to convert paths to something that OCIOv1 can eat.


refactor: cleanup code

coverage: 87.205%. remained the same
when pulling 66f0d5030bc984ab7ca41bd6f55df41e41987f17 on refactor_shared_type
into aa76faed49c8118837ed16bd0a63342ab806a7a2 on main.


USER_NOT_MUTUAL_CONTACT: any ways to add "unknown" user to a chat?

Hi,
If I trying to add user to supergroup using TdApi.AddChatMember method, I receive an error

code = 400, message = "USER_NOT_MUTUAL_CONTACT"

ok, I understand - spam protection etc.
But if I trying to add a user who is not in my contact list using official telegram client and my tg account - user adds to group without any troubles. What difference between official tg-client and tdLib client? I really, really can't add users who are not in my contact list to chats using tdLib?

USER_NOT_MUTUAL_CONTACT is returned for users left the group themselves. After that only mutual contact can invite them back. If the user is not a mutual contact, was previously a member of the group and left it, then the error USER_NOT_MUTUAL_CONTACT will be returned in any client.

where I can read a docs about statuses?

What statuses?

USER_NOT_MUTUAL_CONTACT, PEER_FLOOD etc

There is no description of all possible errors because they are subject to change often.
If an error is supposed to be handled in a special way by clients, you can find its description in source code of official clients. For example, you can find shown to user error messages in Android client in AlertsCreator.java.

ok, thanks @levlam


Showing "<a name=～"

ja-jp:"https://docs.microsoft.com/ja-jp/xamarin/ios/platform/multitasking"

en-us:"https://docs.microsoft.com/en-us/xamarin/ios/platform/multitasking"

#PingMSFTDocs

Doc issue description

Microsoft support for Xamarin will end on May 1, 2024 for all Xamarin SDKs including Xamarin.Forms. In preparation for this, all issues and PRs in this repository are being closed.
.NET MAUI is the replacement for Xamarin.Forms, and it's documentation can be found here.


#109 Restyle Comments fix

Fixed numerous warnings encountered when using Doxygen to document code.
Added new comments for easier code readability of some functions.
Note: There are still a few warnings that needs to be corrected by someone more familiar with the project's code. I have attached a file with the last warnings that needs correcting and where they can be found.
I hope you find my edits acceptable.
Documentation.leftover.txt

Hello there @bobbybrot and thank you for your contribution, it's very much appreciated!
We will look over this in the next couple days, suggest likewise improvements and fix the mentioned left over warnings.

Sure thing, I shall keep an eye out. :)

As a question, what will happen with the leftover functions that need documenting?

I would suggest to fill a follow up issue and merge this pull request now. Since it will go first intos @a-teammate's branch he needs to decide.

Alright thanks for letting me know. I will wait it out and see what happens.

(Sorry wrong button :D)

contents of documentation.leftover.txt:
yo

inexor/texture/slot.hpp line:190, 'root' is not documented

inexor/texture/slot.hpp line:158, 'forceload' is not documented

inexor/server/gamemode/gamemode_server.hpp line:60, 'type' and 'p' is not documented

inexor/engine/worldio.cpp line:924, 'changed' is  not documented

inexor/engine/worldio.cpp line:454, 'f' 'co' 'size' and 'failed' are not documented

inexor/engine/worldio.cpp line:468, all parameters are missing documentation information

inexor/engine/world.cpp lines:552-555, all functions are missing parameter documentation

inexor/util/Subsystem.hpp line:125, Not sure how to deal with not declared or defined warning for 'void tick()' function

void tick() gets executed every frame (we have a main loop).
this function is actually virtual meaning any class deriving from this class, will actually want to implement this function (which then gets executed every frame)

inexor/util/textedit.hpp line:767, same as issue above

inexor/engine/sound.cpp line:601, include file 'unistd.h' not found, perhaps you forgot to add its directory to INCLUDE_PATH?

unistd.h is for unix only. try https://stackoverflow.com/a/22445280

inexor-core/readme.md line:3, "Unexpected HTML tag <img> found within <a href=...> context"
Can this warning be ignored and if not, how do you fix this?
yeah just ignore it or maybe google got a solution

dunno about the rest atm :D try to add the documentation or if not possible reformat the info in the doc comment to silence the warning :)

There we go, that should bring us closer to closing issue #109

Very nice! Thank you, I will put this into server_standalone as soon as I can resume working on it (currently in holidays)
This will make it into 0.9 definitely :)

merged with remove_standalone2 in 3760f2dc25efe0e88aa79ab36539e7c61a155649 ! :)
As i can finally resume working on remove_standalone2, it will come into master soon


Added a no-this-reassignment rule

PR checklist

[x] Addresses an existing issue: #2861
[x] New feature, bugfix, or enhancement

[x] Includes tests

[x] Documentation update

Overview of change:
Bans phrases like "var self = this;". Port of the tslint-microsoft-contrib no-var-self rule.
CHANGELOG.md entry:
[new-rule] no-this-reassignment

no-this-reassignment

Will do.

const { props, state } = this;

Seems reasonable. I'll add it as an on-by-default option.

Oh almost forgot! This was made with help from @sethob; he should be credited in any contributors mention.

Oh, almost forgot! This was made with help from @sethbrenith; he should be mentioned in any contributors list.

Thanks for the shout-out, but I think this is all you. The only thing I wrote is a fixer for it, which doesn't seem to be included here.


Add media support

Please add support for media fields.
Image Response:
{
    "data": {
        "attachments": {
            "media_keys": [
                "3_1541785060919185408"
            ]
        },
        "id": "1541785063028842498",
        "text": "This player just won't let go of his old boots... 😆 \n\nWho is it? 🤷♂\n\n#UCL https://t.co/4dETe8vjyL"
    },
    "includes": {
        "media": [
            {
                "type": "photo",
                "height": 712,
                "media_key": "3_1541785060919185408",
                "url": "https://pbs.twimg.com/media/FWWFWBwXkAApnXt.jpg",
                "width": 632
            }
        ]
    }
}

GIF Response:
{
    "data": {
        "id": "1323351722119495681",
        "text": "https://t.co/aLA9mVBAfB",
        "attachments": {
            "media_keys": [
                "16_1323351712476717057"
            ]
        }
    },
    "includes": {
        "media": [
            {
                "preview_image_url": "https://pbs.twimg.com/tweet_video_thumb/El19Xv8UYAEWxVd.jpg",
                "width": 498,
                "height": 278,
                "type": "animated_gif",
                "media_key": "16_1323351712476717057"
            }
        ]
    }
}

Video Response:
{
    "data": {
        "attachments": {
            "media_keys": [
                "13_1335947635014905857"
            ]
        },
        "text": "All types of goals from all situations 🔥\n\nCristiano Ronaldo's best Champions League goals are something else! 🍿\n\nhttps://t.co/lSzee8d61m",
        "id": "1543286684846104576"
    },
    "includes": {
        "media": [
            {
                "media_key": "13_1335947635014905857",
                "duration_ms": 256240,
                "type": "video",
                "width": 1920,
                "preview_image_url": "https://pbs.twimg.com/media/Eoo9YilXcAI6tNm.jpg",
                "public_metrics": {
                    "view_count": 231570
                },
                "height": 1080
            }
        ]
    }
}

from: https://twittercommunity.com/t/v2-endpoints-not-returning-images-for-most-tweets/173333

Understood. I will implement it in the near future.

I added some fields to support media.
example(test) is here:
https://github.com/takke/twitter4j-v2/blob/31b5e9531d33fb84651f865e277cb1c3e7ce784b/twitter4j-v2-support/src/test/kotlin/twitter4j/GetTweetsTest.kt#L256-L523
Please try.

That's perfect. Thank you!

Released with v1.3.0.
Thanks.


Handle empty input files when saving embedding plots

This was a bit trickier than I thought because I think the pipeline will complain if there is no output. What I have done is:

Check if the input .h5ad file is empty, if yes:
Delete any existing embedding output files
Create empty output files

I think it works but I have only tested it very quickly so probably good if @danielStrobl can try running it before we merge.

Why delete the existing embedding output files? What should there be if there is an empty h5ad file?

I assumed an empty .h5ad meant a failed run and that any existing embedding output was from some previous run so should be deleted. They will just get overwritten will blank files otherwise which is pretty much equivalent. Unless there is some other reason for an empty .h5ad. Easy enough to change if we want.

I might mean we were not careful in how we save our data if there is sth to delete. Did you come across this case? I would maybe throw an error if this happens.. what do you think? Typical Snakemake procedure would be to overwrite though.

Ah, I remember why I did this now. The Path().touch() command only changes the timestamp if the file exists so I had to delete it so out of date stuff was left there. I can probably come up with another way to overwrite with a empty file though.
Does anyone know what we do with the metrics if the .h5ad is empty? Guess we should probably do the same thing here.

In the metrics case we just output a file that has no value for all metric outputs (I think it was this, and not NAs). That is then captured in the metrics aggregation script to say the method wasn't run. @mumichae might be able to confirm this.
I think the safest approach here would be to throw an error if the file exists. Then we can check manually and just run the snakemake command again if needed.

In case the adata is empty, we would want empty output. If we had a non-empty output on the previous run, but for some reason the current run produces an empty adata, we wouldn't want to keep the previous embedding, so I'd overwrite any file that is output from the script but not captured by snakemake. But seeing as we provide all output filenames to the snakemake rule, snakemake should remove these files if they exist before a rerun. So all that you'll need to do is fix your script to output empty files.

So to clarify, when we have failed runs, they result in empty .h5ad files. That means an output file will be created. Snakemake doesn't execute a rule if there is no input for it. If you require snakemake to create an output file that it can't create, it'll throw a missing input error.

Ok, I changed how empty output files are created. I think this is more what we want but if not I'm probably a bit confused about what to do.

Yes, this looks fine


Browser support Firefox

Hello,
Any idea of supporting Firefox?
Regards,
Lino

Sorry, it's not possible.

I tested this in Firefox, which supports a lot of the same APIs that Chrome does, and it worked.
I just had to fix a small thing in the manifest: "options_page": "options.html".
It looks like Firefox does not recognize this configuration at all, and that prevents the addon from loading.
But other than this, it appears to work fine, as shown below on cnn.com:

This extension is really useful for people who just want to know if something's wrong quickly without opening devtools first.
It would be really good to make it Firefox-compatible (especially because it looks like it can be done very easily) and have it on addons.mozilla.org for Firefox users to use too.

You can specify an options page by using the following syntax:
"options_ui": {
  "page": "options.html",
  "browser_style": true
},

It is supported by both Chrome and Firefox :

https://developer.mozilla.org/en-US/Add-ons/WebExtensions/user_interface/Options_pages
https://developer.chrome.com/extensions/optionsV2

@barbushin It really would be awesome if you could make the small change and make it available in both the Firefox and Chrome stores!

Here we go https://addons.mozilla.org/en-US/firefox/addon/javascript-errors/

Guys, please rate it in FF store. Thanks!


why should flip the right eye image horizontally?

Hello, i can not very clearly understand why should flip the right eye image horizontally as mentioned in the paper and source code.
https://github.com/HzDmS/gaze_redirection/blob/549b39930e1fe9bb02737cdd7ebb70c6583d14ba/src/data_loader.py#L122
And the meaning of the file name suffix('L' or 'R') in the dataset, what's the corresponding relation?
Thx~

Hi @linccnu, I'm not the author but I've played with the repo a bit. L and R correspond to Left and Right eyes respectively. My guess on why the R eye needs to be flipped is to have a consistent dataset for training. If you're familiar with stylegan, we find that centering the face helps training ... a lot! Flipping the eyes so that the cornea is on the same side is similar to this.


Solve struct ellipsoid caused the model to dark

Solved the problem that struct ellipsoid caused the model to darken on Qualcomm platform

Thanks for the pull request @verybigzhouhai!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

Fixed #7651

@OmarShehata Hi，Now I am modifying the relevant code of the ellipsoid struct. Do you think it is acceptable?

This is ready!

Just to close the loop, this is the same fix as implemented in https://github.com/AnalyticalGraphicsInc/cesium/pull/7911 right?

@OmarShehata yes, it's exactly the same.

@OmarShehata here are two test failed, it doesn't seem to have anything to do with my code changes.Can you help me confirm it?

@OmarShehata Will you review this?

I can confirm this fixes the issue at https://github.com/AnalyticalGraphicsInc/cesium/issues/6622. I believe this also fixes https://github.com/AnalyticalGraphicsInc/cesium/issues/7651 and https://github.com/AnalyticalGraphicsInc/cesium/issues/7403.
Testing on the physically based Sandcastle, in the hosted Sandcastle on Android:

In this branch:

@verybigzhouhai my main notes here are:

I would avoid copying the value manually everywhere it's used in the code. Consider adding something in the shader constants that can be re-used like https://github.com/AnalyticalGraphicsInc/cesium/blob/master/Source/Shaders/Builtin/Constants/degreesPerRadian.glsl
I think this can be removed if it's not being used anywhere https://github.com/AnalyticalGraphicsInc/cesium/blob/master/Source/Shaders/Builtin/Structs/ellipsoid.glsl

@OmarShehata I am very happy that the function is normal in your testing.
I did use some constants to replace the structure at the beginning, but it caused a lot of unit tests to fail. I don't know how to avoid or need to modify a lot of unit test code, so I used the current modification method.
Even if we use repeatable constants, I don't think this is the best way to modify them, because the structure itself is hoping to make the program simpler, so I think the best way to do this is to dynamically before compiling the shader. Replace the structure, but this approach may not be that simple, I have raised a similar issue here,  https://stackoverflow.com/questions/56479172/about-remove-replace-struct  , if we can do this, could make the code Keep the original elegance.
What do you think about the above, if nothing, I will modify it again according to what you mentioned.

@verybigzhouhai I'm curious about the unit tests that fail using that. Can you list a few of them and the errors you get?

@OmarShehata I found some errors of my own, which will not lead to unit test failure now. Now I add two constants and delete files that do not need to be used. Updates have been submitted，you can check whether they meet the requirements.

@verybigzhouhai one more thing: Can you update CHANGES.md to add a bullet point under "fixes"? It should say something like:

Fixed a bug that caused 3D models to appear darker on Android devices. #7944

Thank you very much for your guidance, @OmarShehata  now it should be ready to be merged.

Great - I'm confident merging this since it's largely just rewriting structs to use builtin constants. It works in Chrome, Firefox, Edge, and IE11. It works correctly in Chrome on Android now.
@lilleyse if you want to skim the changes later when you get a chance in case I missed any shader styling thing I wasn't aware of, that would be helpful!

If this is a first-time contribution, we'd like to share it. @verybigzhouhai do you have a Twitter handle?

@slchow twitter handle @zhouhai19931228


Crash reporting - IOS Objective C

Users/testeuphony/Library/Developer/Xcode/DerivedData/innerVoice-fkokqiykrziydyhhujludfggkytb/Build/Intermediates/innerVoice.build/Debug-iphonesimulator/InnerVoice.build/Script-0EB62B041D13493D0096F953.sh: line 12: /Users/testeuphony/Desktop/innerVoice_Git/InnerVoice/iOS/Pods/FirebaseCrash/upload-sym: No such file or directory
This error popped when tried to integrate the crash reporting.

same for me

The upload sym file should be present in all recent builds - will close this but please reopen if you still arent seeing it!


Remove elbow known issue

Now that XR_ULTRALEAP_hand_tracking_elbow is supported the known issue is not accurate anymore and should be removed.

Blocked until migration to GitHub is complete. Push mirroring would clobber it if I merged it now. Will action this afterwards.

No longer blocked :)


Fix cli opts for level/part

The readme and nog -h mention --level to specify the part to submit but it doesn't work. So, I added --level=<> as a cli opt to adhere to readme. Kept --part=<> for backwards compatibility.

Cool, thanks. I didn't realize they were different


Feat/embed

Context
Link to Notion ticket
and for the notification banner - Link to Notion ticket
Additional information
The embed will not work in live editing if it has a script (hence why I developed the note here)
Have tested with:

Lottie File
PDF
Google docs
Hubspot form

Testing (tester)
The reviewer should check on this branch:

The code
The component in Bookshop browser
The site in CloudCannon (link here)

Before merge (PR owner)

[ ] Delete the test site in CloudCannon
[ ] For "generic" components only: remove - content_blocks from structures in the YAML file (this was added for testing).

What should this component support? Tried with YouTube embed - got a lot of odd behaviour...

if this should be a generic 'everything' embed, then needs more work/testing on different types etc...
if it is specifically not for things like YouTube videos etc... then it probably needs some text to that effect, maybe a list of supported providers etc...
which then begs the question, should there be a 'media' embed component for embedding things like YouTube videos etc?

Also - tested with a Google Doc - did not seem to resize correctly using the ratio option thingee... could see the container resizing, but the actual embedded content remains small... again, not sure what the intended behaviour is? If this is something that can't be controlled, should the option be there?

Question: should the custom embed be on the component by default? I realize that they will need to add it but what is the point of the embed component without it, you know?

also: does the embed code block need a description or comment? I am used to this look as a developer, but this is for editors... perhaps they will not realize this is a code block to paste their code into? Maybe even some basic instructions along the lines of "Go to the service you want to embed, copy the code as per their instructions, and paste the code in here"... idk...

There seems to be a weird caching bug (at least on Chrome) with Google docs - this could be a browser thing, a GD thing, or a CloudCannon thing... do we care?

Updating the Google Doc does not update in the published CC site (or live editor)... hard refresh with cache clearing seems to fix? (I don't know what default GD behaviour for embeds is meant to be - it sounds like it should update automatically? Investigate, or don't care?

Can't seem to remove the custom width px after being set - deleting the number then just defaults it to 0, and hides the embedded content.

setting to -1 makes the default 100% width work again... but I assume this is just because I've broken the px and it's defaulting back to 100% - this could be a hacky workaround to leave in there for the user, but then there needs to be some text to this effect (put -1 to make it default back to 100%.... that sort of instruction?)

It's great! Alignment works well... I have:

Added the same conditional check for heading OR description - if neither, hide component... same as in centred asset block (should probably do a pass at the end for all components?

Only other comment would be maybe we want different orientations for A4 and video in the dropdowns? But that's a thing we can always add to later... I reckon merge this in to get it off the tickets and we can do a 'nice to have' run at the end if we have time?


Don't reset logs when JS tests restart the server

When a JS test requested a restart server we would wip the current log
file. This makes it hard to debug failing tests occasionally when they
happen just after a restart. This change prevents just opens log files
in read/write mode specifically when a test requests a server restart.
The current behavior for interactive use of dev/run will continue to
truncate log files on startup.

nice, +1


Increase minimum limit for fontSize and lineHeight

VSCode Version: 1.10.0-insider
OS Version: Windows 10 x64

Steps to Reproduce:

Open settings.json.
Enter without saving: "editor.fontSize": 1 and wait.
Small font size is applied.

I would expect to have a font size lower bound when font size is applied dynamically without saving, or asking for confirmation on small font size. The user can enter number "1" and then have a think for another several seconds before deciding on second digit for the size.
UPDATE:
"files.autoSave": "afterDelay" has to be set. Thanks to @joaomoreno comment.

It's not that you didn't save, but you have auto save enabled after a certain timeout. Hence, the wait part.

The limit is today 0 or something like that. Perhaps it should be 5 or something.

Hi, I believe this issue has been solved already, with the current minimum fontSize and lineHeight limit set to 8, on fontInfo .
Do you mind if I set a constant for both minimum fontSize and lineHeight for better code understanding?
It would take a big part in a school assignment.


fix fix SdkClientException: Multiple HTTP implementations were found on the classpath #17136

Proposed changes
cherry-pick from #17136

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run feut


Generamba crashes after Xcode update to 8.2.1

To fix that crash, please update xcodeproj gem dependency to 1.4.2 - i've tested locally and it works with that version without crash

@alexgarbarev thanks for your report, I'll release a new version this week.


Added difference in object attributes between microflows and nanoflows

When retrieving an empty attribute of an object, an empty string is returned instead inside nanoflows.

Needs technical review first.

@Robinfr Pls let me know if this is ready for tech writer review.
I'll specify that this is for nanoflows. I think the following is easier to understand:

In nanoflows, when retrieving an empty attribute of an object, an empty string ('') is returned.

@Yiyun333 Yes this can be picked up by tech writers :) Yes good point to clarify we are talking about nanoflows!

@Robinfr I've rephrased the sentence a bit. Pls let me know if I can go ahead and merge the PR. Thanks!

@Robinfr I've rephrased the sentence a bit. Pls let me know if I can go ahead and merge the PR. Thanks!

Agreed via Slack.


Change the default style for embedded regex

As shown in the picture, the default style is markup.italic, however, it makes distinguish | and / hard (especially if they're not adjacent)
How about change the style to bold or simply remove the italic?

I've simply just removed the italic
as I want character class literals to look ever so slightly different to normal literals (which are bold)

Fine, it works.


DNN-7569: Remove the FeedBrowser.

@Ben - can you tweak this to use the incremental version <IP_ADDRESS> - rather than <IP_ADDRESS> - you will need to add a new cleanup file (<IP_ADDRESS>.12.txt) and a new private method in Upgrade.cs.  Otherwise people who are using CTPs won't get the change correctly.
Thanks

done.

sorry folks, <IP_ADDRESS> is being used already, it'll need moved again

:shipit:


Dl3 reader for LST data

DL3 reader; first for LST data only; to add a DL3 reader also for MAGIC data the filetype check needs a rework, which will go into a separate PR.

In general PR looks okey save the inserted docstring placeholders (useless unless replaced with actual docs). I would encourage in the future not to mix topical PRs like this with bug and style fixes in the unrelated classes and modules.

The latest changes allow the DL3 reader to not only read the LST files, but DL3 files in general. I tested it with LST, H.E.S.S. (public data release) and MAGIC DL3 files.
Furthermore added the suggestions from Lukas regarding the interpolator for the coordinate transform and the changes regarding the Obs ID retrieval; changed the compatibility check to not reject compressed files as well.

Hi,
I got a notification here and had a cursory glance, but what I saw made me very nervous about timing information.
You add and subtract raw mjd and unix values at different points in the code here instead of using astropy Time and TimeDelta objects.
This will result in timestamps that are off by the number of leap seconds between those two times. Never do that.

@mstrzys I think something went seriously wrong in your last merge commit, all the changes related to reading LST DL3 data were removed


Merging Dev into Master

Landing page and Cloudant integration completed

Merged into master

IBM Cloud toolchain: Delivery Pipeline deployed LRP-web-app to test, including commit b263a960cc44bc824c213307254a559d51ca911d

IBM Cloud toolchain: Delivery Pipeline deployed LRP-web-app to test, including commit b263a960cc44bc824c213307254a559d51ca911d

IBM Cloud toolchain: Delivery Pipeline deployed LRP-web-app to prod, including commit b263a960cc44bc824c213307254a559d51ca911d


Automatic error report

Problem Description
What steps will reproduce the problem?

I tried to open an existing file by doing {Command + O}. That caused this error.
Traceback
  File "/anaconda3/lib/python3.6/site-packages/spyder/plugins/editor.py", line 1860, in load
    self.edit_filetypes = get_edit_filetypes()
  File "/anaconda3/lib/python3.6/site-packages/spyder/config/utils.py", line 124, in get_edit_filetypes
    supported_exts = _get_pygments_extensions()
  File "/anaconda3/lib/python3.6/site-packages/spyder/config/utils.py", line 86, in _get_pygments_extensions
    for lx in lexers.get_all_lexers():
  File "/anaconda3/lib/python3.6/site-packages/pygments/lexers/__init__.py", line 53, in get_all_lexers
    for lexer in find_plugin_lexers():
  File "/anaconda3/lib/python3.6/site-packages/pygments/plugin.py", line 53, in find_plugin_lexers
    yield entrypoint.load()
  File "/anaconda3/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2323, in load
    self.require(*args, **kwargs)
  File "/anaconda3/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2346, in require
    items = working_set.resolve(reqs, env, installer, extras=self.extras)
  File "/anaconda3/lib/python3.6/site-packages/pkg_resources/__init__.py", line 783, in resolve
    raise VersionConflict(dist, req).with_context(dependent_req)
pkg_resources.ContextualVersionConflict: (parso 0.1.1 (/anaconda3/lib/python3.6/site-packages), Requirement.parse('parso>=0.2.0'), {'jedi'})

Package Versions

Spyder version: 3.2.8
Python version: 3.6.5
Qt version: 5.6.2
PyQt5 version: 5.6
Operating system: Darwin 16.5.0

Dependencies
pyflakes >=0.6.0 :  1.6.0 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pygments >=2.0   :  2.2.0 (OK)
pandas >=0.13.1  :  None (NOK)
numpy >=1.7      :  None (NOK)
sphinx >=0.6.6   :  1.7.3 (OK)
rope >=0.9.4     :  0.10.7 (OK)
jedi >=0.9.0     :  0.12.0 (OK)
psutil >=0.3     :  5.4.3 (OK)
nbconvert >=4.0  :  5.3.1 (OK)
sympy >=0.7.3    :  None (NOK)
cython >=0.21    :  0.27.3 (OK)
qtconsole >=4.2.0:  4.3.1 (OK)
IPython >=4.0    :  6.3.1 (OK)
pylint >=0.25    :  1.8.4 (OK)

You need to update the parso package. For that you need to open Terminal.app and run there
conda update parso

or (if you unfortunately are using pip to install conda packages, which is a bad idea)
pip install -U parso

@ccordoba12 We're also getting the same error on our Py3 + conda builds on Travis when trying to run completion-related tests; it looks like jedi is pulling the wrong parso dependency again or something along those lines. Should we pin our CIs to the older jedi, or conversely require 0.2.0 (which looks to be available on conda)?


Style: Uncrustify FreeRTOS-Plus files

Uncrustify FreeRTOS-Plus files
Description
Uncrustify FreeRTOS-Plus files and add a configuration file for uncrustify
The difference between this file and the one from Amazon FreeRTOS repository, is that it aligns the equal sign on assignment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Difference with uncrustify.cfg of amazon-freertos is the addition of 4 configurations in this PR:


Appstudio update test-component-pac-huws

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-huws-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
14 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
35 seconds
sast-snyk-check

✅ Succeeded
25 seconds
deprecated-base-image-check

✅ Succeeded
25 seconds
inspect-image

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
36 seconds
clamav-scan

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
1 minute
label-check

✅ Succeeded
6 seconds
show-sbom

✅ Succeeded
7 seconds
show-summary

Pipelines as Code CI/test-component-pac-huws-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
14 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
35 seconds
sast-snyk-check

✅ Succeeded
25 seconds
deprecated-base-image-check

✅ Succeeded
25 seconds
inspect-image

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
36 seconds
clamav-scan

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
1 minute
label-check

✅ Succeeded
6 seconds
show-sbom

✅ Succeeded
7 seconds
show-summary

Pipelines as Code CI/test-component-pac-huws-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
18 seconds
clone-repository

✅ Succeeded
18 seconds
sast-snyk-check

✅ Succeeded
38 seconds
build-container

✅ Succeeded
8 seconds
inspect-image

✅ Succeeded
10 seconds
deprecated-base-image-check

✅ Succeeded
35 seconds
clamav-scan

✅ Succeeded
9 seconds
sbom-json-check

✅ Succeeded
11 seconds
clair-scan

✅ Succeeded
7 seconds
label-check

✅ Succeeded
6 seconds
show-sbom

✅ Succeeded
6 seconds
show-summary


@types Support

TypeScript doesn't pickup these files based on our incremental file expansion so we need to do this ourselves as a part of the compilation context expansion.
The algorithm is covered here : https://www.typescriptlang.org/docs/handbook/tsconfig-json.html
Test
package.json
{
  "name": "nodetest",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "devDependencies": {
    "@types/node": "^6.0.45"
  }
}

{
    "compilerOptions": {
        "module": "commonjs",
        "target": "es5",
        "noImplicitAny": false,
        "sourceMap": false,
        "types": [
          "node"
        ]
    }
}

import * as fs from 'fs';

See https://github.com/TypeStrong/tsify/issues/203#issuecomment-252438669 :rose:
nodetest.zip

Released with 2.2.3 :rose:


EICAR test

This is to prove that the Azure ML image is scanned for viruses and fails when one is found. Don't merge this; I'll delete it after we prove the build fails.

Closing since the test was successful


Can't use because I used Google Single Sign On as auth for OpenAI

Get this after trying to login with Google.
The issue is that in OpenAI web page, I can't see how I can make it work with email/password if I chose Google sign on once.
Is there any way I can perhaps make it work?

I opened an issue with OpenAI to support adding email/pass combo to existing accounts that chose Google sign on. Hopefully they reply soon and resolve it.

But I do wonder if there is any way to hack around this in the time I wait for response from them.

2022/12/04 16:36:17 Please log in to OpenAI Chat

Perhaps I can pass the key manually? And do auth in my normal browser or something.

Signing using email + pass is working fine.

I made new OpenAI account.
But now it breaks, when I press login, the Playwright chrome window just goes away. Don't know why.

Just see this in terminal.

Hmm, sign in with Google works fine for me.

sign in with Google works fine for me

Any reason why it can break? 🤔

btw can I reinstall Playwright?

I also can't log in with Google in chromium. Either get the same error message from google or it just crashes.  Would it be possible to instead use the openAI API key?

Or maybe use https://github.com/acheong08/ChatGPT directly

Or https://github.com/transitive-bullshit/chatgpt-api

I will try look at the code, not sure fully how it works but I imagine you can cut away from using chromium to proxy the calls.

Ok thanks to @m1guelpf I got it working.
For anyone else hitting this issue. Replace Chromium with Safari and do authentication.
In playwright.RunOptions do Browsers: []string{"webkit"},
And do browser, err := pw.WebKit. in launchBrowser function.


Allow creation of a snapshot of an empty repository

Hi!
When trying to snapshot an empty repository, we get:
ERROR: unable to create snapshot: local repo doesn't have packages

This is easy to circumvent by creating an empty snapshot instead but it would be more convenient if aptly just created an empty snapshot in this case.

Hello,
I agree, there should be an option to allow creation of empty snapshots.
So one can create multi-component publish eg. with aptly-publisher even when there are no packages in some of the components.

+1, this should be easy to fix, this is just check in the code which should be removed

Cool, thank you for fixing this :-)

Should be available now in aptly_1.0.0+30+ga245b72 nightly


600ms addtional recording on online videos

As youtube auto-generated subtitles often cut one sentence into many pieces, while mining on the youtube, I often need to adjust the audio cutting point manually.
Cause there is no reference to determine if a certain word is included, I need to adjust the starting and ending point and hit the white recording dot on the right of the audio field over and over again. It's indeed something I have to do. However, the recording timeline is a little bit tricky. After clicking the dot, I can hear the full recording based on the period I chose. But after exporting to anki and cheking the card on anki, the audio on anki is about 600ms shorter than the recording I hear; the last 600ms is omitted.
I guess the recording I hear whist mining is actually 600ms longer than the preset time, the recording did not stop at the right place. The result is that every time during mining, I need to add 0.6s at the ending point after I hear the right audio and click the white dot once again to record what I want (at this time I will also hear an additional 600ms audio that will not be included on anki card).
I tried to adjust Audio Padding, but it does not help but add the padding to the timeline, which does not change the recording anyway.
Also, for local videos, the cutting point is as precise as I hear during mining. It looks like just a glitch of recording.
Thank you for reading my post and hopefully I stated my question clearly. This is really a fantastic extention for second language learners!!!

Hi, I believe I understand the issue. asbplayer is mp3-encoding the audio immediately after recording. This should be a relatively simple fix. In the mean time you can work around the issue by turning off the "re-encode audio in mp3" option under the MINING section of the settings. Unfortunately this means that your audio will be in webm if you're using Chrome and not mp3, but this isn't a problem if you only review Anki cards on a desktop computer. Thanks for reporting!


Simplify configuration

On Irma they're using a sarek function, which is like:
sarek ()
{
    nextflow run /path/to/sarek/$1 -c /path/to/config/sarek_specific.config -profile slurm ${@:2};
}

and in the config file there is:
params.project = 'PROJECTID'
params.containerPath = '/Path/To/Containers'
params.tag = '2.0.0'

We had some trouble with probably some configuration priorities, but that's a separate issue (probably a Nextflow one).
But at least, we should try to explore and simplify this process.

Done on nf-core/sarek


RenderTemplate how to pass vars to view?

Well how to use:
return a.RenderTemplate("Admin/Index.html")
and pass some var user string for example?

I got it:

c.RenderArgs["routes"] = routes
c.RenderArgs["categories"] = categories
return a.RenderTemplate("Admin/Index.html")

:+1: thanks for sharing your fix!


Added top mergers page

Adds a page that lists people who've merged a pull request that was opened as part of 24pullrequests, ordered by most merged, related to https://github.com/24pullrequests/24pullrequests/issues/1611

Also works with 2015 data:

Looks good, I see you're still adding stuff so I'll let you go ahead and merge this one when you want. 👍

Have added a randomized maintainers box to the homepage as well:

Ready to merge now @xadamy if you'd like to review?

Looks good! Also TIL about number_with_delimiter! 🙌

@xadamy http://api.rubyonrails.org/classes/ActionView/Helpers/NumberHelper.html#method-i-number_to_human is also neat


bug: Heading falls behind Navbar in Payout-methods route

Describe the bug
The Heading of the Payout method route falls behind the navbar
To Reproduce
Steps to reproduce the behavior:

Go to https://app.100xdevs.com/payout-methods
See error

Expected behavior
The Heading Should be below the navbar
Screenshots or GIFs
https://github.com/user-attachments/assets/bd56f9ba-4cc4-4d97-abfd-a742b75ff684
Info (please complete the following information):

Browser [Safari]
Version [17.5]

I would like to work on it

resolves #1169


openboard: init at 1.6.1

Motivation for this change
Add OpenBoard to Nixpkgs. The derivation was written by @OPNA2608, who also gave me permission to use it in this PR.
Darwin support is coming soon™.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

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

1 package built:

openboard

Oh no, i missed the commits.
Please squash them when you accept changes from reviews in the future.
This added a mess to master.

Ah dammit, I thought GitHub's squash commits was set as default for merging PRs in nixpkgs.

You can't remove commits in a branch that others use without causing a huge mess. Everyone would have to rebase. You can rewrite the history in a pull request, but never do that in master.


[Backup Schedule] Move CronAnything from oracle to common.

As described in go/anthos-postgres-backup-schedule-dd, we should have a general
framework for Backup shedule. Oracle adopts CronAnything to do the cron
backup job. Our current design is to reuse the work in oracle, so the first step is to move the CronAnything from oracle/ to
common/.
Bug: b/193256355

/test operator-oss-presubmit

/ok-to-test

Does this change pass all the integration tests?


"Workspace" is treated as an undefined global

Meanwhile it's another way to reference the "workspace".

Workspace isn't a global it's a service so this error is correct, workspace is just a shortcut for accessing it without having to game:GetService("Workspace"):
do -- service access
    local Workspace = game:GetService("Workspace")
    print(Workspace) -- no error
end

do -- shortcut global access
    print(workspace)
end

-- these two examples are the same, the first is just longer

Ah, I didn't run the code as I was on my phone, I genuinely believed it was service only and chose to reply anyway, forgot there was a global since it's been deprecated forever so I've never used it.

I retract what I said then, I apologize for my baseless argument.


WIP: Notifications

Add catch-all updates feature

Plug the notifications to updates

To make this work well we also need #223

Added notify about new features

Added notify about new features

This will need some design love. Maybe we can put it in the footer for now until we find the best spot?

Fine in Nightly, but in Firefox main version
TypeError: ServiceWorker script at http://localhost:4000/offline-worker.js for scope http://localhost:4000/ encountered an error during installation.

Error in payload in Chrome
GET http://localhost:4000/get_payload/3976a6a2af6cafc17cbb 404 (Not Found) undefined:1 Uncaught (in promise) SyntaxError: Unexpected token C

Not related to the error, but with the new version of the web-push library and Chrome 50, we won't need the get_payload method anymore.


I haven't thanked you enough.

log4cl is wonderful, THANK YOU!!

I can't thank you enough, but I'll close this anyway to keep your issues list free of clutter.


Native module cannot be null on import of NfcManager in RN 0.59.2

import NfcManager, { Ndef } from 'react-native-nfc-manager';
is resulting in a 'Native module cannot be null.' error on ios using react-native-nfc-manager 1.2.1.
I have verified that it's linked and entitlements are correct. I tried unlinking removing, adding an linking again.
I was previously successfully using RN 0.57.1 and 0.6.0. In that version I had libNfcManager.a in Link Binary with Libraries but it doesn't exist on the new version. Along with that there isn't an NfcManager.xcodeproj installed in the Libraries in Xcode on the new version.

Hi @robseaman , I just do some quick test regarding this, however it's working for me.
My steps are:
react-native init NfcTest
cd NfcTest
npm install --save react-native-nfc-manager
react-native link react-native-nfc-manager

Both the xcodeproj and the .a are placed as they're expected.
If the issue persists, maybe you can perform some manual link for now?

Thanks @whitedogg13 for getting on this so quickly. It does create everything correctly when I start a new project from scratch and install it first. It's not hard for me to recreate my project so that will be good enough. I don't have any idea how to recreate the situation I was in so I'll close this.