Add cookie-based-affinity-distinct-name annotation

Checklist

[X] The title of the PR is clear and informative
[X] If applicable, the changes made in the PR have proper test coverage
[X] Issues addressed by the PR are mentioned in the description followed by Fixes.

Description
Generate a unique affinity cookie name when the annotation cookie-based-affinity-distinct-name is set to true in an ingress. This maintains affinity when using path based routing for different backends..
Adding the following annotations will ensure every backend has a different affinity cookie:
    appgw.ingress.kubernetes.io/cookie-based-affinity: "true"
    appgw.ingress.kubernetes.io/cookie-based-affinity-distinct-name: "true"

Fixes
#772 Enhance annotation list of AGIC

@rbickel Thanks for the PR.
How is the application using the the cookie at the backend ? Would backend application need to know the cookie name beforehand ?

The backend doesn't use the cookie. It is only for application gateway to route the request to the proper backend. If you have 2 different routing path on your ingress, when moving from a route to another, it overwrites the cookie with the affinity of the other path. You need different cookie name to ensure when you navigate back you still hit the same backend.
Affinity cookies in Application Gateway are just a hash of the pod IP, so there isn't any usage of this on the backend
e.g. with below ingress. When navigating to one path to the other, the default cookie ApplicationGatewayAffinity is overridden. When navigating back to the other path, that may have multiple pods as replicas, the affinity is lost :
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: go-server-ingress-affinity
  namespace: test-ag
  annotations:
    kubernetes.io/ingress.class: azure/application-gateway
    appgw.ingress.kubernetes.io/cookie-based-affinity: "true"
    appgw.ingress.kubernetes.io/cookie-based-affinity-distinct-name: "true"
spec:
  rules:
  - http:
      paths:
      - path: /affinity1/
        backend:
          serviceName: affinity-service
          servicePort: 80
      - path: /affinity2/
        backend:
          serviceName: other-affinity-service
          servicePort: 80

@akshaysngupta  We could prefix the cookie name with appgw-affinity- and then the backend hash, so we provide some hints to the browser what is the cookie for ? However I didn't want to leak appgw backend information back to the browser, reason of why I am using a hash.


Exam: Colums for complaint responses and average points swapped assessment-dashboard

Describe the bug
In the assessment-dashboard for an exam the columns for average points and answered complaints are swapped in the statistics-table.
This is not the case for regular programming exercises as well as the exam-dashboard.
(See screenshots below)
To Reproduce

Create an exam
Create submissions
Start grading them with TAs
See different values in both tables

Expected behavior
The columns should show the data the tables heading claims to be.
Screenshots

Which version of Artemis are you seeing the problem on?
6.1.2
What browsers are you seeing the problem on?
Chrome, Firefox
Additional context
No response
Relevant log output
No response

FYI @dfuchss


E2E test now waits for webhook to complete setup

What type of PR is this?
/kind bug
NONE

/ok-to-test

/ok-to-test

how did this merge and the e2e tests didn't pass?

I merged it, thought that the e2e test will not run with the latest changes, my bad.

@Bslabe123 is working on fix it right now


Any update to Vue3?

Any update to Vue3?

+1

+1

While I'd like to eventually update this project to Vue 3, Vue 2 is still the default version that Vue installs when you run npm install vue, so until the team has decided that Vue 3 is 100% ready and has replaced Vue 2 as the recommended version, I’m not going to prioritize this.


Question about the error development

Hi all,
I have a question about the development of the error in this figure:
https://cloud.githubusercontent.com/assets/11435359/13046277/e904c04c-d412-11e5-9260-efc5b8301e2f.jpg
What is happening after 30 and 60 epochs that there is a huge drop in the error? I've seen this drop in the error in multiple papers now, but I never had this with my own models.
If this is not the correct place to discuss this, please let me know.
Cheers,
Maurits

Ok,
I figured out that it is the learning rate decay!


Roll engine cbdc7bde224c..3851981b863a (1 commits)

https://github.com/flutter/engine/compare/cbdc7bde224c...3851981b863a
git log cbdc7bde224c..3851981b863a --first-parent --oneline
2020-01-04<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from Vk6uT... to Kom_K... (flutter/engine#15142)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

Auto-roller completed checks. About to merge.


Modify custom drag layer if it's over DropTarget

Hi all,
Is there a way to check if custom DragLayer is over DropTarget?
Thanks

See here: https://github.com/gaearon/react-dnd/issues/448
It's not the best solution, but gets the job done for now.

That will help me too.
I have an item which should have different previews based on which DropTarget is over.
When I am over a DeleteDropTarget I want the preview to scale(.5).

@mindtricksdev hi mindtricksdev，any updates? im also looking for the solution

I worked around that. I made the drop target react to custom drag layer hover. This was acceptable as UX for my DeleteDropTarget as I changed the drop background to red instead of scaling down the dragged item.


Hydrator is trying to instantiate the Abstract class

In current implementation, we have a check for interfaces but not for abstract classes. Therefore, if I have an abstract class type field then Hydrator tries to instantiate it and resulted in an exception.

I fixed the issue under: https://github.com/mayankgoyel/ObjectHydrator/commit/0312bad51026cef7c5825156e4603642d07c90be
Will pull it once my last pull request is merged.


CI

This pull request fixed some CI warning due to googletest compilation and Micro-XRCE-DDS-Client third-party.

Build status:

Linux
Windows

Build status:

Linux
Windows

Build status:

Linux
Windows

Build status:

Linux
Windows

Build status:

Linux
Windows


Markdown writer adds an extra newline after each body written to file

The markdown writer (for @auto nodes) adds an extra newline after each body content when writing files.
** @edreamleo  Note: Leo does it but only on nodes with are non-terminal nodes.** (is it a bug also?)
Easily reproduce with this command to clone a minimal reproducible example repo:
git clone https://github.com/boltex/test-markdown.git
And then reproduce with ctrl+shift+w to re-write the markdown file and see the diff it creates.

On Mon, Dec 11, 2023 at 11:13 PM Felix @.***> wrote:

The markdown writer (for @auto https://github.com/auto nodes) adds an
extra newline after each body content when writing files.
** @edreamleo https://github.com/edreamleo Note: Leo does it but only
on nodes with are non-terminal nodes.** (is it a bug also?)
Easily reproduce with this command to clone a minimal reproducible example
repo:
git clone https://github.com/boltex/test-markdown.git
And then reproduce with ctrl+shift+w to re-write the markdown file and see
the diff it creates.

Iirc, this behavior is on purpose. Markdown requires a blank line after
section names.
Perhaps the writer should only write extra blank lines if necessary, but I
wouldn't assume that doing so would be an improvement.
Edward


Validate format option values using in attribute

Description

As we have supported :summary, :text, :json, :yaml and :pp format values but other than these values it raises error eg. ERROR: ArgumentError: Unknown output format raw.
After this fix it also does not allows any string starts with s, t, y, j and p
As per the code
https://github.com/chef/chef/blob/340ceac008bee94c053fd791175534a0b2a87ec2/lib/chef/knife/core/generic_presenter.rb#L112-L127
so previously it also taking following values as a valid format
=> knife cookbook list -F jjjjjjjjjjjjjjjjjj

=> knife cookbook list -F text_format_invalid

=> knife cookbook list -F p_but_not_exit

But after fixes, these raise an error?
Related Issue

Fixes https://github.com/chef/chef/issues/2979
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] All commits have been signed-off for the Developer Certificate of Origin.
Signed-off-by: Vivek Singh<EMAIL_ADDRESS>

Nice catch

Thanks


Fix bug where it was impossible to create hexagons

Currently, it is not possible to create hexagons.
Interestingly, the author documented the bug but did not fix it until now.
This PR solves this bug and updates the documentation accordingly. ;)

Thank you for your contribution. It’s interesting that a hexagon is technically feasible, but after careful consideration, I do not see the use case for such a feature.


IS 5.3.0 to 5.4.0 migration script for identity fails

The migration script for IS 5.3.0 to 5.4.0 does not run when we execute with the command [1] as per the document[2]. However, it was noted that once the script was run manually the first two alter  steps of the script was applied
after running the tool

manually applying the changes

[1]sh wso2server.sh -Dmigrate -Dcomponent=identity
[2]https://docs.wso2.com/display/IS540/Upgrading+from+a+Previous+Release

sp_rename 'IDN_OAUTH2_SCOPE.NAME', 'DISPLAY_NAME', 'COLUMN';
sp_rename 'IDN_OAUTH2_SCOPE.SCOPE_KEY', 'NAME', 'COLUMN';
ALTER TABLE IDN_OAUTH2_SCOPE ALTER COLUMN NAME VARCHAR(255) NOT NULL;
ALTER TABLE IDN_OAUTH2_SCOPE ALTER COLUMN DISPLAY_NAME VARCHAR(255) NOT NULL;
ALTER TABLE IDN_OAUTH2_SCOPE DROP COLUMN ROLES;
UPDATE IDN_OAUTH2_SCOPE SET TENANT_ID = -1 WHERE TENANT_ID = 0;
CREATE UNIQUE INDEX SCOPE_INDEX ON IDN_OAUTH2_SCOPE (NAME, TENANT_ID);

DECLARE @Command NVARCHAR(max), @ConstaintName NVARCHAR(max), @TableName NVARCHAR(max),@ColumnName NVARCHAR(max)
SET @TableName = 'IDN_OAUTH2_SCOPE'
SET @ColumnName ='TENANT_ID'
SELECT @ConstaintName = name
    FROM sys.default_constraints
    WHERE parent_object_id = object_id(@TableName)

SELECT @Command = 'ALTER TABLE '+@TableName+' drop constraint '+ @ConstaintName

IF @Command IS NOT NULL
BEGIN
    EXECUTE sp_executeSQL @Command
    SELECT @Command = 'ALTER TABLE '+@TableName+' ADD CONSTRAINT '+@ConstaintName+' DEFAULT -1 FOR ' + @ColumnName
    EXECUTE sp_executeSQL @Command
END

DECLARE @COMMAND1 NVARCHAR(200);SELECT TOP 1 @COMMAND1= 'ALTER TABLE IDN_OAUTH2_RESOURCE_SCOPE DROP CONSTRAINT ' + RC.CONSTRAINT_NAME + ';' FROM INFORMATION_SCHEMA.REFERENTIAL_CONSTRAINTS RC JOIN INFORMATION_SCHEMA.KEY_COLUMN_USAGE KF ON RC.CONSTRAINT_NAME = KF.CONSTRAINT_NAME JOIN INFORMATION_SCHEMA.KEY_COLUMN_USAGE KP ON RC.UNIQUE_CONSTRAINT_NAME = KP.CONSTRAINT_NAME WHERE KF.TABLE_NAME = 'IDN_OAUTH2_RESOURCE_SCOPE' AND KP.TABLE_NAME='IDN_OAUTH2_SCOPE';EXEC (@COMMAND1);

ALTER TABLE IDN_OAUTH2_RESOURCE_SCOPE ALTER COLUMN SCOPE_ID INTEGER NOT NULL;
ALTER TABLE IDN_OAUTH2_RESOURCE_SCOPE ADD FOREIGN KEY (SCOPE_ID) REFERENCES IDN_OAUTH2_SCOPE(SCOPE_ID) ON DELETE CASCADE;

~

This issue is occurring when duplicate entries for the key NAME,TENANT_ID are exist in the IDN_OAUTH2_SCOPE table. There should be no possibility for adding duplicate data for this key. When analyzing it was identified that duplicate records are added to the IDN_OAUTH2_SCOPE table when we design an API. This causes the migration scripts "step2" for (ISKM 5.4.0)  to fail. So raised [1] in APIM to track this.
[1] https://github.com/wso2/product-apim/issues/3190

This issue should be ideally fixed from APIM side where they prevent multiple records from being inserted into the table

Doc Impact :N/A


Test testCommissioning fails by chance

Test testCommissioning in test_commissioning.py fails by chance.
Seems some times the Discovery Response is not received by the Joiner (no ACK found).

This issue is likely fixed by OTNS2 (https://github.com/EskoDijk/ot-ns) which contains updates for all unit tests, stress tests, etc. It has a feature to run simulations fully deterministically (pseudo-random) which can avoid test failures due to rare random coincidences. Leaving this issue open for until OTNS2 code is merged; and then if resolved, this can be closed I think!


Version on github does not match crates.io

The version on github is 0.4.4, but crates.io has 0.4.5 (published Mar 26, 2019).  I am a bit confused.
It also seems to be missing a change that is in the 0.4.5 version:
diff -r rustfix-0.4.5/src/diagnostics.rs /Users/eric/Proj/rust/rustfix/src/diagnostics.rs
4a5,6
> use serde::Deserialize;
>
diff -r rustfix-0.4.5/src/lib.rs /Users/eric/Proj/rust/rustfix/src/lib.rs
0a1,2
> #![warn(rust_2018_idioms)]
>
8,10c10
< #[macro_use]
< extern crate serde_derive;
< extern crate serde_json;
---
> use serde_json;
18c18
< use diagnostics::{Diagnostic, DiagnosticSpan};
---
> use crate::diagnostics::{Diagnostic, DiagnosticSpan};
47c47
<     fn fmt(&self, f: &mut std::fmt::Formatter) -> std::fmt::Result {
---
>     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
59c59
<     fn fmt(&self, f: &mut std::fmt::Formatter) -> std::fmt::Result {
---
>     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
181,182c181,182
<                     use Filter::*;
<                     use diagnostics::Applicability::*;
---
>                     use crate::Filter::*;
>                     use crate::diagnostics::Applicability::*;

Oh, that seems to be my fault! I just checked my local repo and it seems that I messed up my git config so it didn't have the correct URL for this repo set (it was moved from my account, and I later forked it).
The 0.4.5 tag was fa8a2f04f12b189854e8a4607d0786580be2dbd5 with the version changed. I've pushed it now but the precise commit it is unfortunately not on master. I'll also push the rebased version bump just to make this consistent again!

#171

Closing since 0.4.6 was released.


Be able to connect to Opennode or Strike using Apis.

Hi,
As I can see https://www.opennode.co/ and https://strike.acinq.co/ services have Apis. I know this service allow to connect to them: https://tallyco.in/
I think Joule could allow users to connect to Strike or Opennode. So I could use my funds on Opennode to pay or request bitcoin.
Thanks.

Hi,
If a service has an API, I'm sure they want to be integrated on any other service, including Joule. This hardware devices are good and ideal, but for people on countries underdeveloped, it's yet expensive to buy it. I think you should give people as many options as possible. So my suggestion is to add services like Strike or Opennode to Joule.

These third-party services provide API's to receive payments, not send them. The need they fill is to allow businesses to accept lightning payments by integrating the merchant's shopping cart or POS system with OpenNode/Strike. Then users can purchase their products/services with btc/lightning.
Joule is a wallet for users to store and send payments. Joule is on the opposite side of the transaction and therefore there is no integration to be done. Lightning is the language both sides speak to fill the gap. The best solution you could have is for those payment processors to add WebLN to their plugins to streamline the payment flow for users.


ScalarType nullability is lost in JSON serialization round-trip

Ran into this while testing schema serialization/deserialization. Here's a minimal failing test case:
import ml.combust.mleap.core.types.{DataType, ScalarType}
import org.scalatest.Assertions._
import ml.combust.mleap.json.JsonSupport._
import spray.json._

val before = ScalarType.String.nonNullable
val after = before.toJson.compactPrint.parseJson.convertTo[DataType]
assert(!after.isNullable)

I think the fix is just adding a .setNullable(false) to this line - https://github.com/combust/mleap/blob/185cdf4d7ddc1efbefc43ac6d9b7ce358fbf7574/mleap-runtime/src/main/scala/ml/combust/mleap/json/JsonSupport.scala#L141

@pidge https://github.com/combust/mleap/pull/271
Fixed this issue as well as other nullability issues with serializing/deserializing. Thank you for the find! Will release a 0.8.1 version with this fix soon.

Merged the PR for this, closing.


[FEATURE] Oracle driver

I would like to have support for oracle databases go_ora maybe?

Yeah, why not. We recently added support for sqlite. But first, we need to do a massive refactor, to add more drivers, in fact.

Greate @danvergara when will the release with oracle support be available?

@sschneider-ihre-pvs v0.24.0 is already available!!

great

:(
is there a db version restriction?

or windows

@sschneider-ihre-pvs I think it has to do with the syntax of your query. Is your query using JOINs or ending with semicolons. I got the same error because I wasn't aware of the changes of the Oracle syntax.

I tried to select a table to see the data
so ctrl+h navigate to table and hitting enter

This is the container image I use for testing:

Ah yes, I have an older db that does not support things like TAKE FETCH NEXT OFFSET and so on, so that would explain it.
So I guess you need at least v12c


pygw starter code

Starter code for the pygw project.
Includes:

Base wrapper models for Python
Working implementations of RocksDB & Redis datastores and SpatialIndex and SpatialTemporal Index
A working prototype pipeline for the Simple Feature Ingest Example

See README in python/src/main/python for more info.

Coverage increased (+0.006%) to 42.213% when pulling 46e1d2628d08ad5e8eaf68a81bebc7f05ab6cf7f on charlesyu108:python-bindings into a40297ddc9e6287ba11b04ef21ca95dbf7558358 on locationtech:python-bindings.


The question about DF2K dataset

I have checked your training log and the number of images mentioned in the log is 3550, but in Readme it is written as 3450. Most models are trained on 3450 images.

Hi, in the classic SR training, we mainly follow the previous work ART which uses DF2K containing 3550 images for training. We do not find significant performance difference between using 800 and 900 images from DIV2K, since the training iteration is sufficient.


Use isScratcher for scratcher check

Will respond with up-to-date data and save a few bytes on the user agent header. Closes #52 (Fun fact: PRs can close PRs. I learned that while experimenting)

I'll work on it myself and find a way to make it work without requesting any API if possible

I'll work on it myself and find a way to make it work without requesting any non-scratch API if possible

I'll make my code work as a module.

It's ok! I'll reopen this PR if I have to.

It's ok! I'll reopen this PR if I have to.

I'm only doing this because I want to use your website, you know.

@webdev03 Is it weird that I have netcat on my isScratcher server?

Done, a real parser has been sent to the server

Done, a real parser has been sent to the server

What? I just finished my API!

Wow cool! I implemented a new getStatus thing can you see if it works?

Wow cool! I implemented a new getStatus thing can you see if it works? You could make an AT thread and eventually get enough info?

Let me see.

Watch your logs!

Doesn't work!

UGH!

It's saying it's invalid!

Try again, tell me when you're on and then I'll check logs

Try again, tell me when you're on and then I'll check logs

Trying right now.

Started authentication, posting comment now

Posted comment

Tried to authenticate, result was INVALID with HTTP response code 500.
I'll try again and get data regarding the headers.

interesting..

 interesting..

This?
https://github.com/webdev03/scratchinfo/blob/9e8f2c7783fc614fe58d5f76002ef1545040092c/src/lib/getStatus.ts#L1

Yes! I don't know why.

Yes, almost, but it says it isn't a function! I'm going to try to get the type of it I don't know why.

Pretty sure it'd be this.
undefined

Got an idea, wait a moment

I've set up logging, once it rebuilds can you try again?

I've set up logging, once it rebuilds can you try again?

Could you maybe fork https://github.com/hello-smile6/isScratcher ? I created a new branch, and I could make a PR that uses your fork for verification.

Can you try logging in now

Can you try logging in now

Trying yet again

Done. Same result.

parse is undefined.... what?!?!?

parse is undefined.... what?!?!?

https://github.com/webdev03/scratchinfo/blob/9e8f2c7783fc614fe58d5f76002ef1545040092c/src/lib/getStatus.ts#L1
Just use isScratcher! If you fork it, I can make a PR that uses a version that you control.

I think that I can try one more thing please wait

In case this matters, it shows an error when I try to use scratchinfo with replit.
0 info it worked if it ends with ok
1 verbose cli [ '/usr/local/bin/node', '/usr/local/bin/npm', 'install', '$lib' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose npm-session 65e2d24b230fed5e
5 silly install loadCurrentTree
6 silly install readLocalPackageData
7 timing stage:rollbackFailedOptional Completed in 1ms
8 timing stage:runTopLevelLifecycles Completed in 106ms
9 verbose stack Error: Invalid tag name "$lib": Tags may not have any characters that encodeURIComponent encodes.
9 verbose stack     at invalidTagName (/usr/local/lib/node_modules/npm/node_modules/npm-package-arg/npa.js:91:15)
9 verbose stack     at fromRegistry (/usr/local/lib/node_modules/npm/node_modules/npm-package-arg/npa.js:296:13)
9 verbose stack     at resolve (/usr/local/lib/node_modules/npm/node_modules/npm-package-arg/npa.js:81:12)
9 verbose stack     at npa (/usr/local/lib/node_modules/npm/node_modules/npm-package-arg/npa.js:52:10)
9 verbose stack     at /usr/local/lib/node_modules/npm/lib/install/deps.js:231:14
9 verbose stack     at /usr/local/lib/node_modules/npm/node_modules/slide/lib/async-map.js:52:35
9 verbose stack     at Array.forEach (<anonymous>)
9 verbose stack     at /usr/local/lib/node_modules/npm/node_modules/slide/lib/async-map.js:52:11
9 verbose stack     at Array.forEach (<anonymous>)
9 verbose stack     at asyncMap (/usr/local/lib/node_modules/npm/node_modules/slide/lib/async-map.js:51:8)
9 verbose stack     at exports.getAllMetadata (/usr/local/lib/node_modules/npm/lib/install/deps.js:228:3)
9 verbose stack     at Installer.loadArgMetadata (/usr/local/lib/node_modules/npm/lib/install.js:366:3)
9 verbose stack     at /usr/local/lib/node_modules/npm/lib/install.js:704:16
9 verbose stack     at /usr/local/lib/node_modules/npm/lib/install/read-shrinkwrap.js:33:16
9 verbose stack     at tryCatcher (/usr/local/lib/node_modules/npm/node_modules/bluebird/js/release/util.js:16:23)
9 verbose stack     at Promise._settlePromiseFromHandler (/usr/local/lib/node_modules/npm/node_modules/bluebird/js/release/promise.js:517:31)
10 verbose cwd /home/runner/scratchinfo
11 verbose Linux 5.11.0-1029-gcp
12 verbose argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "$lib"
13 verbose node v12.22.6
14 verbose npm  v6.14.15
15 error code EINVALIDTAGNAME
16 error Invalid tag name "$lib": Tags may not have any characters that encodeURIComponent encodes.
17 verbose exit [ 1, true ]

Figured out the cause of the Replit problem, I think

Well, it works now! Try it, trust me!

Well, it works now! Try it, trust me!

Okay

Worked.

Found a UI issue, I'll see if I can figure out how it could be fixed

Well, https://scratchinfo.vercel.app/collabs/create number picker background color and text color are the same.
For reference:
appearance auto
background-color rgb(255, 255, 255)
border-bottom-color rgb(229, 231, 235)
border-bottom-style solid
border-bottom-width 0px
border-image-outset 0
border-image-repeat stretch
border-image-slice 100%
border-image-source none
border-image-width 1
border-left-color rgb(229, 231, 235)
border-left-style solid
border-left-width 0px
border-right-color rgb(229, 231, 235)
border-right-style solid
border-right-width 0px
border-top-color rgb(229, 231, 235)
border-top-style solid
border-top-width 0px
box-sizing border-box
color rgb(255, 255, 255)
cursor text
display inline-block
font-family ui-sans-serif, system-ui, -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, "Helvetica Neue", Arial, "Noto Sans", sans-serif, "Apple Color Emoji", "Segoe UI Emoji", "Segoe UI Symbol", "Noto Color Emoji"
font-size 16px
font-stretch 100%
font-style normal
font-variant-caps normal
font-variant-east-asian normal
font-variant-ligatures normal
font-variant-numeric normal
font-weight 400
height 24px
letter-spacing normal
line-height 24px
margin-bottom 0px
margin-left 0px
margin-right 0px
margin-top 0px
padding-bottom 0px
padding-left 0px
padding-right 0px
padding-top 0px
tab-size 4
text-align start
text-indent 0px
text-rendering auto
text-shadow none
text-size-adjust 100%
text-transform none
width 181px
word-spacing 0px
writing-mode horizontal-tb
-webkit-rtl-ordering logical
-webkit-border-image none

#54

Wait, corepack actually isn't preinstalled with node! Guess it's because I always use the .zip option since I don't have admin.

Corepack is preinstalled - maybe only in new versions of node.

Corepack Loader is preinstalled (not pnpm or yarn though) - maybe only in new versions of node.

Oh. Okay!

@webdev03 Do you like the outlines that show up on inputs? If not, you can turn that off.

It's OK

It's OK

Okay.


Fix several response problems

This PR attempts to address several problems discovered in #76.

The static internal instances of DioAdapter and DioInterceptor exist to allow building chains of requests. This is a great feature but in its current design breaks when running parallel tests. To mitigate this problem I changed the implementation to use Dart builder/chaining (adapter..onGet()..onGet()) and a response handler syntax.

When sending 2 or more requests that match the same RequestMatcher, then the response can not be reused because the stream has already been closed when sending the first response. To avoid this the RequestHandler now has a map of statusCode to a function that returns a Responsable - this allows to construct a new response every time.

This seems to work well in all kinds of scenarios that previously failed.

CC @LukaGiorgadze @GiorgiBeriashvili @campfire5

Yea, would be great if this can be released as a new minor and afterwards there could be a 1.0.0-nullsafety version.

@kuhnroyal Very beneficial changes, thank you very much!
I have a question regarding handler, - is there a reason why it's an optional parameter? Perhaps it could be a positional parameter instead since it's always provided?

this would be great to be introduced soon, we are facing the same problem.
How long till you can push this or do you have some examples for a workaround in the interim?

Thank you for your patience. @kuhnroyal solved this wonderfully as I see it, I advocate that this should be merged into develop branch quite soon.

Yea, would be great if this can be released as a new minor and afterwards there could be a 1.0.0-nullsafety version.

I was thinking on v0.2.0 after this (and maybe some other minor changes later) and then the nullsafe version as you pointed out.

Yes, before the changes it actually was optional in onRoute but that has changes. I renamed the parameter to callback and defaulted to naming the callback parameter request. That reflects the intent a little better. Basically you call reply on the request to create a Response.

My last push broke the OpenAPI generator build where I started using this great library to test the Dart generator 😔 https://github.com/OpenAPITools/openapi-generator/pull/8870

My last push broke the OpenAPI generator build where I started using this great library to test the Dart generator pensive OpenAPITools/openapi-generator#8870

The commit where you changed optional parameter to positional parameter? Seems weird if that's the case, as the functionality should remain the same.
Is there any way I can help?

Nah it's all good, the PR is up to freeze this to my latest commit until a version is released.
Well the method signature changed and now has a required and non-named parameter, so that's always a compile error :)


Support smallest_time_grain and inf time range

Closes #1766

Closes https://github.com/rilldata/rill-developer/issues/1766

Tagging @djbarnwal for review so he can assess if this works as expected. Thanks for the PR @begelundmuller


require is not defined for all test files

when i run npm test I get the error "ReferenceError: require is not defined" for each test file I have.  A short while ago my tests were running but I am not sure what is wrong anymore.
Karma.config.js

.babelrc

A test file

Error in console

devdependencies

I would appreciate all the help and will provide any information needed

babel-plugin-transform-es2015-modules-commonjs

This plugin transforms ES2015 modules to CommonJS.

so this plugin generates commonjs code which does not natively run in the browser.
so probably best to remove it?

Still getting the same error

I figured it out. Class properties in js weren't appreciated when testing and were causing the error. I configured typescript as of now and have a separate issue where I can't seem to directly test spec.ts files despite following the steps outlined on https://open-wc.org/testing/karma-esm.html#usage. Any help from @LarsDenBakker or @daKmoR  would be greatly appreciated.

@lakshay35 I'm glad your issue with commonjs was resolved.  😁
If your problem with .spec.ts files is still occurring, please open a new issue for it.


Link to meeting recordings?

We recorded the last meeting. Presumably we intend to post the recording somewhere public? Update the README with a link to the meeting recordings.

We just got our youtube channel here: https://www.youtube.com/channel/UCUdhiXNEBEayowJXY_v7AXQ/
I'm uploading the video today.

Ref #12


Small documentation fixes

fix some issues discovered while working on the website (incorrect
links in OVS pipeline documentation)
fix some Markdown formatting to simplify generation of the website

Codecov Report

Merging #1491 into master will decrease coverage by 14.09%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #1491       +/-   ##
===========================================
- Coverage   56.36%   42.27%   -14.10%
===========================================
  Files         126       91       -35
  Lines       11651    10704      -947
===========================================
- Hits         6567     4525     -2042
- Misses       4456     5777     +1321
+ Partials      628      402      -226

Flag
Coverage Δ

integration-tests
?

unit-tests
42.27% <ø> (+0.03%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/ovs/openflow/default.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/config/node_config.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/apis/controlplane/v1beta1/register.go
0.00% <0.00%> (-92.86%)
:arrow_down:

pkg/ovs/openflow/logs.go
9.52% <0.00%> (-90.48%)
:arrow_down:

pkg/ovs/openflow/ofctrl_group.go
0.00% <0.00%> (-84.38%)
:arrow_down:

pkg/ovs/openflow/ofctrl_action.go
17.07% <0.00%> (-74.15%)
:arrow_down:

pkg/ovs/openflow/ofctrl_bridge.go
1.97% <0.00%> (-66.80%)
:arrow_down:

pkg/ovs/ovsconfig/ovs_client.go
1.83% <0.00%> (-62.25%)
:arrow_down:

pkg/ovs/openflow/ofctrl_builder.go
24.25% <0.00%> (-53.20%)
:arrow_down:

pkg/agent/openflow/pipeline.go
22.80% <0.00%> (-43.22%)
:arrow_down:

... and 53 more

/skip-all


Is it possible to trigger Snippets in milkdown editor?

Generally, you can insert snippets into markdown file in VS Code. However it seems snippets can not be triggered when using milkdown editor. Is it possible to insert snippets in milkdown editor? I think it's useful for math edit. 😃

I have no idea, it needs some investigation.


Error message displayed for split distance created in Korean Won currency

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.59-3
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @jacobkim9881
Slack conversation (hyperlinked to channel name): ts_external_expensify_open_source
Action Performed:
Prerequisite: Default currency in Korean Won

Create a group chat
Click the + in composer and select split expense
Select Distance and enter address for Start and Stop field
Click Next

Expected Result:
Amount equally split and user able to click split button to submit the expense
Actual Result:
The sum of splits must equal the total amount error message displayed and split button non functional
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/19ca9061-6b3d-4908-97c3-6bcb33486d57

View all open jobs on GitHub

I found there is a difference between total amount and split shares:
transaction:
Object
amount: 519357        /////// not rounded
attendees: [Object] (1)
billable: false
category: ""
comment: {waypoints: Object, customUnit: {customUnitRateID: "_FAKE_P2P_ID_", quantity: 4281.895}, isLoading: false}
created: "2024-11-12"
currency: "KRW"
errorFields: {}
iouRequestType: "distance"
isFromGlobalCreate: false
merchant: "4.28 km @ ₩1213.45 / km"
participants: [{accountID: 18602521, selected: true}] (1)
participantsAutoAssigned: true
pendingFields: {}
reportID: "4715962770440927"
routes: {route0: Object}
shouldShowOriginalAmount: false
splitPayerAccountIDs: [18608989] (1)
splitShares: Object
/////////////// rounded here below
18602521: {amount: 259700, isModified: false}
18608989: {amount: 259700, isModified: false}
Object 견본
transactionID: "1"

https://github.com/Expensify/App/blob/64cb4ee92b1f4c66e5deb96e23caf47e25423c00/src/pages/iou/request/step/IOURequestStepDistance.tsx#L144-L153
There is code for decision how the total is same with split shares. I tried to round total amount from 519357 to 519400. For this calculation, round method is needed but needed parameter currency above always be undefined. Why mileageRate?.currency, policyCurrency are undefined? I am tracing the reason.

Proposal
Please re-state the problem that we are trying to solve in this issue.
The sum of splits must equal the total amount error message displayed and split button non functional
What is the root cause of that problem?
The total amount is calculated by calculateAmount function
https://github.com/Expensify/App/blob/64cb4ee92b1f4c66e5deb96e23caf47e25423c00/src/libs/actions/IOU.ts#L7661
The total amount of the split bill is rounded before we calculate the split amount
https://github.com/Expensify/App/blob/64cb4ee92b1f4c66e5deb96e23caf47e25423c00/src/libs/IOUUtils.ts#L51
then if the rounded total amount is different from the current amount, the error appears here
https://github.com/Expensify/App/blob/64cb4ee92b1f4c66e5deb96e23caf47e25423c00/src/components/MoneyRequestConfirmationList.tsx#L452-L454
What changes do you think we should make in order to solve the problem?
When calculating the split amount, we should remove the Math.round here so that we do not round the total amount but only the split amount.
const totalInCurrencySubunit = (total / 100) * currencyUnit;

https://github.com/Expensify/App/blob/64cb4ee92b1f4c66e5deb96e23caf47e25423c00/src/libs/IOUUtils.ts#L51
What alternative solutions did you explore? (Optional)

Can confirm

But this only happens for this currency - I tried two other currencies and did not get the same response. So not sure how high priority this will be. It's likely uncommon to split a mileage expense, and this would only affect a subset of users

Added to Quality and posted in Slack for prioritization

Does this only affect distance expenses, or does it happen to any split?

This only affects distance expense, the reason is the rate is odd.

Will test this a bit more first thing tomorrow morning - had a managed customer issue that threw my day off today 😓

Can confirm it does not affect regular splits - just distance. But for the life of me I cannot remember how to change the currency on a distance expense in ND 🤦♀ So working on that now 😅

Figured the currency out - so ignore me lol. Testing them now

@mkzie2 I couldn't reproduce with VND - Or other currencies with 'decimals' => 0, I tested about 20 currencies all together🤔

@OfstadC Yes I can, an example for VND.
https://github.com/user-attachments/assets/10a246af-df19-4a8d-b13c-6a3426550456

Thanks @mkzie2 - i'm going to have to try this again 🤔

I'm still having issues 🤔 .

Ah - so this does not happen in a workspace chat - only a group DM

Tested another handful in the group chat - not workspace chat 😅 :
Error occurs on these currencies:
ALL, AFN, BIF, CLP
Does not occur for:
AMD, ANG, CAD, CUP
So I believe the theory that it occurs for Currencies where 'decimals' => 0, is listed - is accurate.

@rushatgabhane My proposal is here.

For example, the current amount 16410076,54

totalInCurrencySubunit will be rounded to 16410077 here.
https://github.com/Expensify/App/blob/64cb4ee92b1f4c66e5deb96e23caf47e25423c00/src/libs/IOUUtils.ts#L51
Then we calculate the split amount based on this rounded amount 16410077.
So the sumOfShares is 16410077 which is different from 16410076,54
https://github.com/Expensify/App/blob/64cb4ee92b1f4c66e5deb96e23caf47e25423c00/src/components/MoneyRequestConfirmationList.tsx#L452-L454

@mkzie2's proposal LGTM 🎀👀🎀
https://github.com/Expensify/App/issues/52310#issuecomment-2469613318

@rushatgabhane The PR is ready.


[BUG] DeepSpeed allocates GPU memory in an unbalanced way.

Describe the bug
Memory usage on every GPU in single-node multi-gpu is unbalanced. In particular, GPU 0 is used by all processes in distributed to store an additional small amount of memory. With DDP, everything is slower but balanced.
To Reproduce
Steps to reproduce the behavior:
Run a simple classification experiment with a Transformer model in pytorch-lightning and set strategy=deepspeed_stage_2.
Expected behavior
Every process should allocate GPU memory only on its GPU.
ds_report output
--------------------------------------------------
DeepSpeed C++/CUDA extension op report
--------------------------------------------------
NOTE: Ops not installed will be just-in-time (JIT) compiled at
      runtime if needed. Op compatibility means that your system
      meet the required dependencies to JIT install the op.
--------------------------------------------------
JIT compiled ops requires ninja
ninja .................. [OKAY]
--------------------------------------------------
op name ................ installed .. compatible
--------------------------------------------------
async_io ............... [NO] ....... [OKAY]
cpu_adagrad ............ [NO] ....... [OKAY]
cpu_adam ............... [NO] ....... [OKAY]
fused_adam ............. [NO] ....... [OKAY]
fused_lamb ............. [NO] ....... [OKAY]
quantizer .............. [NO] ....... [OKAY]
random_ltd ............. [NO] ....... [OKAY]
 [WARNING]  sparse_attn requires a torch version >= 1.5 and < 2.0 but detected 2.0
 [WARNING]  using untested triton version (2.0.0), only 1.0.0 is known to be compatible
sparse_attn ............ [NO] ....... [NO]
spatial_inference ...... [NO] ....... [OKAY]
transformer ............ [NO] ....... [OKAY]
stochastic_transformer . [NO] ....... [OKAY]
transformer_inference .. [NO] ....... [OKAY]
utils .................. [NO] ....... [OKAY]
--------------------------------------------------
DeepSpeed general environment info:
torch install path ............... ['/home/lucadiliello/anaconda3/envs/nlp/lib/python3.10/site-packages/torch']
torch version .................... 2.0.1+cu117
deepspeed install path ........... ['/home/lucadiliello/anaconda3/envs/nlp/lib/python3.10/site-packages/deepspeed']
deepspeed info ................... 0.9.2, unknown, unknown
torch cuda version ............... 11.7
torch hip version ................ None
nvcc version ..................... 11.7
deepspeed wheel compiled w. ...... torch 2.0, cuda 11.7

Screenshots

System info (please complete the following information):

OS: [e.g. Ubuntu 20.04.6 LTS]
GPU count and types: 1 machine with 8x A100 GPUs
Python version: 3.10.9
Lightning version: 2.0.2

Launcher context
Launching experiments with pytorch-lightning and by setting strategy=deepspeed_stage_2. When running with strategy=ddp or strategy=fsdp, memory is allocated correctly. Thus, I suspect this is a bug with DeepSpeed.

Hello @lucadiliello. Thank you for reporting this issue to us. Could you share a script or commandline for us to reproduce this issue?

 Many thanks!

The same problem. I am using zero3 to train a transformer with multi-nodes. On each node, deepspeed allocates much larger memory to the GPU with local_rank=0.

I do not see this issue anymore after upgrading to lightning>=2.2.0 and deepspeed>=0.13.1


"Perhaps you want to add _ to the import list" quickfix

This deserves a quickfix:
[ghcmod]
Not in scope: type constructor or class `LspId'
Perhaps you want to add `LspId' to the import list in the import of
`Language.Haskell.LSP.Types'
(C:\Users\Gurkenglas\AppData\Local\Temp\ghc-mod15709\Plu9C37.hs:(97,1)-(114,59)).

Afaik, this has now a quickfix!

Some imports are incorrect, e.g. importing constructors does something wrong, will take care of that!

With the latest update of #1284 this is now possible!


[v2] build does not work on a fresh install

🐛 Bug Report
After creating a fresh docusaurus page and executing yarn run build, it build Client successfully, but throws an error when building Server starting with Error: Cannot find module '@theme/NotFound' (see below for the full log).
Have you read the Contributing Guidelines on issues?
Yes.
To Reproduce

Create new Docusaurus v2 page: npx @docusaurus/init@next init test classic
Enter the directory: cd test
Run build: yarn run build

Expected behavior
Static page is built.
Actual Behavior
The following error message is displayed:
yarn run v1.22.4
$ docusaurus build
Creating an optimized production build...

✔ Client
  Compiled successfully in 8.91s

✖ Server
  Compiled with some errors in 9.94s

Error: Cannot find module '@theme/NotFound'
    at async Promise.all (index 0)Error: Cannot find module<EMAIL_ADDRESS>Cannot find module '@theme/BlogListPage'Error: Cannot find module '@theme/BlogPostPage'Error: Cannot find module '@theme/BlogPostPage'Error: Cannot find module '@theme/BlogTagsListPage'Error: Cannot find module '@theme/BlogTagsPostsPage'Error: Cannot find module '@theme/BlogTagsPostsPage'Error: Cannot find module '@theme/BlogTagsPostsPage'Error: Cannot find module '@theme/BlogTagsPostsPage'Error: Cannot find module '@theme/BlogPostPage'Error: Cannot find module '@theme/DocPage'Error: Cannot find module '@theme/DocPage'Error: Cannot find module '@theme/DocPage'Error: Cannot find module '@theme/DocPage'
(undefined) Error: Cannot find module '@theme/NotFound'
    at webpackEmptyContext (main:30401:10)
    at main:31155:171
    at async Promise.all (index 0)
    at async render (main:31262:137)
(undefined) Error: Cannot find module<EMAIL_ADDRESS>    at webpackEmptyContext (main:152:10)
    at main:31140:3411
(undefined) Error: Cannot find module '@theme/BlogListPage'
    at webpackEmptyContext (main:152:10)
    at main:31140:2517
(undefined) Error: Cannot find module '@theme/BlogPostPage'
    at webpackEmptyContext (main:152:10)
    at main:31140:3579
(undefined) Error: Cannot find module '@theme/BlogPostPage'
    at webpackEmptyContext (main:152:10)
    at main:31140:3579
(undefined) Error: Cannot find module '@theme/BlogTagsListPage'
    at webpackEmptyContext (main:152:10)
    at main:31140:160
(undefined) Error: Cannot find module '@theme/BlogTagsPostsPage'
    at webpackEmptyContext (main:152:10)
    at main:31140:1758
(undefined) Error: Cannot find module '@theme/BlogTagsPostsPage'
    at webpackEmptyContext (main:152:10)
    at main:31140:1758
(undefined) Error: Cannot find module '@theme/BlogTagsPostsPage'
    at webpackEmptyContext (main:152:10)
    at main:31140:1758
(undefined) Error: Cannot find module '@theme/BlogTagsPostsPage'
    at webpackEmptyContext (main:152:10)
    at main:31140:1758
(undefined) Error: Cannot find module '@theme/BlogPostPage'
    at webpackEmptyContext (main:152:10)
    at main:31140:3579
(undefined) Error: Cannot find module '@theme/DocPage'
    at webpackEmptyContext (main:152:10)
    at main:31140:475
(undefined) Error: Cannot find module '@theme/DocPage'
    at webpackEmptyContext (main:152:10)
    at main:31140:475
(undefined) Error: Cannot find module '@theme/DocPage'
    at webpackEmptyContext (main:152:10)
    at main:31140:475
(undefined) Error: Cannot find module '@theme/DocPage'
    at webpackEmptyContext (main:152:10)
    at main:31140:475
(undefined<EMAIL_ADDRESS>7:126-136
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:231-241
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:400-410
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:551-561
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:713-723
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:904-914
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:1106-1116
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:1324-1334
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:1565-1575
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:1752-1762
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:1921-1931
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:2113-2123
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:2321-2331
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:2532-2542
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:2748-2758
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:2942-2952
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:3163-3173
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:3327-3337
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:3503-3513
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:3694-3704
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:3870-3880
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:4034-4044
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:4213-4223
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:4412-4422
Critical dependency: the request of a dependency is an expression
(undefined) ./.docusaurus/registry.js 1:4590-4600
Critical dependency: the request of a dependency is an expression
Error: Failed to compile with errors.
    at /home/m4tx/husarion/test/node_modules/@docusaurus/core/lib/commands/build.js:37:24
    at finalCallback (/home/m4tx/husarion/test/node_modules/webpack/lib/MultiCompiler.js:254:12)
    at /home/m4tx/husarion/test/node_modules/webpack/lib/MultiCompiler.js:277:6
    at done (/home/m4tx/husarion/test/node_modules/neo-async/async.js:2931:13)
    at runCompilers (/home/m4tx/husarion/test/node_modules/webpack/lib/MultiCompiler.js:181:48)
    at /home/m4tx/husarion/test/node_modules/webpack/lib/MultiCompiler.js:188:7
    at /home/m4tx/husarion/test/node_modules/webpack/lib/MultiCompiler.js:270:7
    at finalCallback (/home/m4tx/husarion/test/node_modules/webpack/lib/Compiler.js:257:39)
    at /home/m4tx/husarion/test/node_modules/webpack/lib/Compiler.js:273:13
    at AsyncSeriesHook.eval [as callAsync] (eval at create (/home/m4tx/husarion/test/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:40:1)
    at AsyncSeriesHook.lazyCompileHook (/home/m4tx/husarion/test/node_modules/tapable/lib/Hook.js:154:20)
    at onCompiled (/home/m4tx/husarion/test/node_modules/webpack/lib/Compiler.js:271:21)
    at /home/m4tx/husarion/test/node_modules/webpack/lib/Compiler.js:681:15
    at AsyncSeriesHook.eval [as callAsync] (eval at create (/home/m4tx/husarion/test/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:4:1)
    at AsyncSeriesHook.lazyCompileHook (/home/m4tx/husarion/test/node_modules/tapable/lib/Hook.js:154:20)
    at /home/m4tx/husarion/test/node_modules/webpack/lib/Compiler.js:678:31
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Your Environment

Docusaurus version used: 2.0.0-alpha.50
Environment name and version (e.g. Chrome 78.0.3904.108, Node.js 10.17.0): Node v13.13.0, npm 6.14.4, yarn 1.22.4
Operating system and version (desktop or mobile): Arch Linux, rolling

Reproducible Demo
https://github.com/m4tx/docusaurus-test

It seems that this bug is still relevant for v2.0.0-alpha.49 :confused:

I am getting this for all the versions I tried even with v2.0.0-alpha.50

Any workaround for this. Appreciated.

Exactly the same issue for me, was unable to finish production build after upgrading to alpha-50. It has something to do with theme past alpha-43. Fixed it by downgrading to pre-update yarn.lock where alpha-43 theme links were saved.

Possibly related to #2244. Not swizzling any components might fix the issue.

Possibly related to #2244. Not swizzling any components might be a workaround for the issue.

In my case no components were implemented or used, just pure md files.

Seeing the same on version 2.0.0-alpha.43 with preset-classic theme

Seeing the same on version 2.0.0-alpha.43 with preset-classic theme

Did you upgrade to alpha-50 and then downgraded back to alpha-43. If that is the case check your yarn.lock, it actually still has alpha-50 theme links as deps. That's what happened to me.

@luxplanjay I have not, but will try that. Thanks.

Did you upgrade to alpha.50 and then downgraded back to alpha.43. If that is the case check your yarn.lock, it actually still has alpha.50 theme links as deps. That's what happened to me.

So before today I was on alpha.43 and had not upgraded yet to alpha.50 and I was getting these errors.
It looks like when installing any version of docusaurus the yarn.lock resolves to alpha.50 for several of the plugin packages. This was with a fresh install. Also, package.json is locked to alpha.40
package.json
{
    "@docusaurus/core": "2.0.0-alpha.40",
    "@docusaurus/preset-classic": "2.0.0-alpha.40",
}

yarn.lock
 "<EMAIL_ADDRESS>  version "2.0.0-alpha.50"
  resolved<EMAIL_ADDRESS>

@coreybrown89 I've encountered a similar issue in the past, especially when using npm for package management. By default, npm doesn't honor the lock file for package installs, so we ended up adding npm ci right before our build command so it will always base package installs on the lock file. Anyway, yarn honors the lock file for installs by default.
Concerning the issue you pointed out, I suspect it has something to do with how dependencies are pinned in D2 - using the ^ symbol. To work around this, I ended up exhaustively pinning all package dependencies without the ^ symbol, which has worked much better so far (for npm).
Example:
    "@docusaurus/core": "2.0.0-alpha.48",
    "@docusaurus/mdx-loader": "2.0.0-alpha.48",
    "@docusaurus/plugin-content-blog": "2.0.0-alpha.48",
    "@docusaurus/plugin-content-docs": "2.0.0-alpha.48",
    "@docusaurus/plugin-content-pages": "2.0.0-alpha.48",
    "@docusaurus/plugin-google-analytics": "2.0.0-alpha.48",
    "@docusaurus/plugin-google-gtag": "2.0.0-alpha.48",
    "@docusaurus/plugin-sitemap": "2.0.0-alpha.40",
    "@docusaurus/preset-classic": "2.0.0-alpha.48",
    "@docusaurus/theme-classic": "2.0.0-alpha.48",
    "@docusaurus/theme-live-codeblock": "2.0.0-alpha.48",
    "@docusaurus/theme-search-algolia": "2.0.0-alpha.48",
    "@docusaurus/types": "2.0.0-alpha.48",
    "@docusaurus/utils": "2.0.0-alpha.48",

Please add the code below to your package.json file and reinstall your deps.
We will pin the version of this dependency (babel-plugin-dynamic-import-node) in the next release and you (and our new users) will not need to do any manual work, so think about this as a workaround  then!
  "resolutions": {
    "babel-plugin-dynamic-import-node": "2.3.0"
  }

We will consider a solution to use the exact versions of the dependencies used (without caret (^) char) to avoid similar problems in the future!
Sorry about that!

Manual fix is no longer required, the building website should work as expected.
https://github.com/airbnb/babel-plugin-dynamic-import-node/commit/5fdba6e43d1cf8ec768e2cba251691c3ea67e78e

Thanks a lot. Things have started to work without any changes!!

Thank you for the quick update! Much appreciated


[Feature]: Support for newer flash-attention versions (e.g. ≥2.1.0)

Suggestion Description
When we need to do long context inference (using LongLoRA), sometimes errors occur: need flash-attn version ≥2.1.0. So wondering if a newer version will follow.
Operating System
SUSE
GPU
MI250X
ROCm Component
ROCm 5.4.3

I don't know what another +1 is worth, but catching up with specifically lower-right causal masking and paged attention would make a world of difference for ROCm users.


nthread inside xgboost() function not working in R

The nthread parameter is evidently being ignored in R. When setting nthread = 1 xgboost appears to be defaulting to the maximum number of threads on my machine because my cpu is instantly maxing out. I've included a reprex and my session info below:
Reprex:
require(dplyr)
require(xgboost)

df1 <- sample_frac(iris, size = 500, replace = T) %>%
  mutate(species_num = dense_rank(Species))

xgb2 <- xgboost(data = data.matrix(select(df1, -Species, -Sepal.Length)),
                  label = data.matrix(select(df1, Sepal.Length)),
                  colsample_bytree = 0.5,
                  nrounds = 25000,
                  min_child_weight = 50,
                  max_depth = 14,
                  eta = 0.1,
                  gamma = 0,
                  subsample = 0.8,
                  nthread = 1L,
                  early_stopping_rounds = 24000,
                  verbose = 0
                  )

Session Info:
> sessionInfo()
R version 4.0.4 (2021-02-15)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 10 x64 (build 19043)

Matrix products: default

locale:
[1] LC_COLLATE=English_United States.1252  LC_CTYPE=English_United States.1252    LC_MONETARY=English_United States.1252
[4] LC_NUMERIC=C                           LC_TIME=English_United States.1252

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] xgboost_<IP_ADDRESS> dplyr_1.0.7

loaded via a namespace (and not attached):
 [1] lattice_0.20-41   fansi_0.4.2       assertthat_0.2.1  utf8_1.1.4        crayon_1.4.1      grid_4.0.4        R6_2.5.0
 [8] jsonlite_1.7.2    DBI_1.1.1         lifecycle_1.0.0   magrittr_2.0.1    pillar_1.6.1      rlang_0.4.10      rstudioapi_0.13
[15] data.table_1.14.0 Matrix_1.3-2      vctrs_0.3.8       generics_0.1.0    ellipsis_0.3.2    tools_4.0.4       glue_1.4.2
[22] purrr_0.3.4       compiler_4.0.4    pkgconfig_2.0.3   tidyselect_1.1.1  tibble_3.1.2

Hmm, that happens during DMatrix construction, the nthread parameter is not exposed in R.


Fix "version is required" error, update for latest API usage

Hello everyone!
In latest VK API "version" parameter has required. Without this parameter VK return error when user info requested (Invalid request: v (version) is required). More info here: https://vk.com/dev/implicit_flow_user.
Also in version upper 5.0 has changed some names like uid -> id and so on (https://vk.com/dev/version5).
In this commit I have added new option "ApiVersion". By default api version is 5.73.

@diseks please see #230

Here is hardcode to change uid to id. This is potential error.
(https://github.com/TerribleDev/OwinOAuthProviders/pull/228/commits/c55c3eefa3b0adc867d45c512e9d462e37a4850c#diff-8d62e39fec7f900f4ffea94b590dac3d)

Id = TryGetValue(user, "uid");
Id = TryGetValue(user, "id");

This is not working if user set api version less then 5.0 over "ApiVersion" ptoperty. In my commit I processed this situation.


What is the general limits of the actions ?

Hello I made an app for mobile devices, and it's checking user stories for every 15-20 mins. but I have no idea if it's gonna cause an feedback_required or challenge_required in future. My other problem is when I unfollow 15/20 people from  who don't follow me back; Instagram blocks the requests and gives  feedback_required to me but in actual instagram app I have no problem... I have added a waiting for 1-1.5 seconds after unfollowing someone. but it still occur How can I solve these problems and does anyone know the limits ?

I have the same issue.
In one device gets feedback_required and on mobile app no restrictions.

I have fixed my checking user story problem but I still can't solve the challenge_required error while unfollowing/following a person.
My app always wait randomly ±1 second after unfollowing or following a person. But on some accounts some users immediately get challenge reguired error after their 5-10th follow/unfollow action.
I think I have issue about request headers and bodies. I'm making an signed body for follow and unfollow actions but I'm not using all of the headers that official Instagram app sends

After deleting cookies, I was able to unfollow 25 users and stopped to avoid another restriction. I have random 10-15s between requests.
Probably IG is blocking actions based on device. So, have to change something in constants to workaround.

Don't forget to write back here in the future when you solve your problem ☺

I am also quite annoyed by this problem, so it will be helpful.
Does changing the device mean logging in again on a different device?
Sure, changing devices tends to bring back actions, but isn't it more likely to be blocked at login?
Also, what exactly is the process of deleting cookies?
If possible, it would be helpful if you could provide sample code for a series of actions.
like
ig.state.generateDevice(new_devise_name);

Does changing the device mean logging in again on a different device?

No. I mean changing information on constants file. In my case I have Dilame's code in Python, so it is constants.py. But I am sure you can find constants within this repository.

Sure, changing devices tends to bring back actions, but isn't it more likely to be blocked at login?

So when you change this, I believe IG antispam presumes you are on another device and it is account owner acting as a real person (unless proved other way). Makes sense for me. If I was IG developer, I would find logins with similar devices. And, in fact, everybody that use this repository has same login information.

Also, what exactly is the process of deleting cookies?

To avoid multiples logins, it is recommended to save cookies. In my case, it is credentials.json. Every time I delete it, I get a new login. I strong believe this login info is also used to identify bots in some way. When blocked, deleting it brings back actions not allowed before.

If possible, it would be helpful if you could provide sample code for a series of actions.

My last attempt is to shuffle actions. My bot follow/unfollow/watch reels/like feed; and all these actions were always on the same sequence. Very easy for an AI system to identify the pattern. I believe shuffling will reduce chances of being blocked again.
Consider that IG people might check this forum. We are like Tom & Jerry.

Does changing the device mean logging in again on a different device?

No. I mean changing information on constants file. In my case I have Dilame's code in Python, so it is constants.py.
 ANDROID_VERSION = 30 ANDROID_RELEASE = '11.0' PHONE_MANUFACTURER = 'samsung' PHONE_DEVICE = 'SM-G930F' PHONE_MODEL = 'herolte' PHONE_DPI = '640dpi' PHONE_RESOLUTION = '1440x2560' PHONE_CHIPSET = 'samsungexynos8890' VERSION_CODE = '185203708'
Maybe I am wrong, but after changing some info regarding device and constants, my bot came back to life.

Also, what exactly is the process of deleting cookies?

To avoid multiples logins, it is recommended to save cookies. In my case, it is credentials.json. Every time I delete it, I get a new login. I strong believe this login info is also used to identify bots in some way. When blocked, deleting it brings back actions not allowed before.

If possible, it would be helpful if you could provide sample code for a series of actions.

My last attempt is to shuffle actions. My bot follow/unfollow/watch reels/like feed; and all these actions were always on the same sequence. Very easy for an AI system to identify the pattern. I believe shuffling will reduce chances of being blocked again.
Consider that IG people might check this forum. We are like Tom & Jerry.

Thank you so much for your message.
I noticed something about the challenge_required error today. My app has been used by over 150 people worldwide in the last 7 days. I noticed that out of 150 users, only 6 users from Turkey got the challenge_required error. However, Turkish users are the lowest audience of my app. I don't understand why this only happens in Turkey. Any idea about this?
Screenshot


Support custom user model

This PR refactors the usage of the build in User model to support custom auth models

[x] refactors all uses of User
[ ] unit tests to ensure custom user models work

Does this also break related model fields in plugins that use the auth.user model?

@wolflu05 yes, that is the main "breaking" thing

This is targeted for 0.16.0 now and will be revisited once 0.15.0 is out.
I think it is already advisable for plugin deps to either target Owners or the django.cong.settings auth model - a list of best practices is on my personal roadmap for 0.16.0

@matmair LGTM

Any documentation needed around this?
What happens to the migrations if  a custom user model is actually used? Do we need to edit existing migrations?

Needs docs

What happens to the migrations if  a custom user model is actually used? Do we need to edit existing migrations?

I have tested that it works but I will try to add a CI test for that too

@matmair do you want to target 0.16.x or bump to 0.17.x?

@SchrodingersGat let's move this, I am focusing on getting user custom states in for 0.16.x

Closing this in favour of an upcoming PR that adds a custom user model


[+] support harmony

 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.


Version update

Increment to EMET 5.51

Modified use of node attributes to handle MS's discrepancy between the packages name and its install directory

Thanks @andrewunderwood-wf LGTM!


gpsprune: 19.1 -> 19.2

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/gpsprune/versions.
meta.description for gpsprune is: '"Application for viewing, editing and converting GPS coordinate data"'.

Checks done (click to expand)

built on NixOS
Warning: no invocation of /nix/store/8dj9s7cdr9r03as76bf730hd6j0g07fa-gpsprune-19.2/bin/gpsprune had a zero exit code or showed the expected version
0 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 19.2 with grep in /nix/store/8dj9s7cdr9r03as76bf730hd6j0g07fa-gpsprune-19.2
directory tree listing: https://gist.github.com/47a18548648046664216a0b96a30a8d1
du listing: https://gist.github.com/df19c61a7dfad5c00d017c110158899e

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)
1 package rebuild(s)
1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
gpsprune

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8dj9s7cdr9r03as76bf730hd6j0g07fa-gpsprune-19.2 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A gpsprune https://github.com/r-ryantm/nixpkgs/archive/5929ef972fa10b665593aff8cef0fc3c8f0be9af.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8dj9s7cdr9r03as76bf730hd6j0g07fa-gpsprune-19.2
ls -la /nix/store/8dj9s7cdr9r03as76bf730hd6j0g07fa-gpsprune-19.2/bin

cc @rycee for testing.

Rebased to master in 4c10970b1a1574f5130de018a12e27b1ca1e5e6a.


feat(net): Cache a list of useful peers on disk

Motivation
Zebra currently uses DNS seeders as its only source of peers. To make it more secure and reliable, we store a list of recent useful peers on disk, and read it at startup.
Close #1956
Specifications
Peer behaviour is not specified, but zcashd does something like this. (In fact it tries to rely on DNS even less.)
Complex Code or Requirements
This PR introduces a new async task, and new filesystem code that must be run in an async context.
Solution

Add a new zebra-network config for the peer cache directory
Regularly write recent useful peers to a file

Do it atomically so the file isn't corrupted if Zebra shuts down or panics while it is being written
If the peer cache would become empty, keep the previous cache file
Limit the number of peers in the peer cache
Add a new zebra-network peer set task to run this code

At startup, load the peers from a file, and use them in addition to DNS
Add metrics and logging for the peer cache

Related changes:

Update state cache docs to fix outdated info
Add some zebra-chain and zebra-network convenience functions
Panic if the configured peer addresses in zebrad.toml are invalid, rather than continuing

Testing
Config:

Add 3 new zebrad.toml config test files containing the variants of the new config field

Unit:

make sure the peer cache is written by zebra_network::initialize and can be read back manually
make sure the peer cache is written by zebra_network::initialize and is automatically loaded by the next instance

Acceptance:

make sure the peer cache file path is written by zebrad in the existing persistent cache test
make sure that the peer cache is not written when the cache is disabled (existing tests)
make sure that a missing cache_dir config gets the default path, but a disabled config stays disabled when written to disk (existing tests)

Test Fixes:

Update some existing tests to disable or configure the peer cache as needed
Update the config test failure message for the new toml format using standard quotes

Review
This is a routine audit fix.
Design questions:

should cached peers replace DNS peers, or be used in combination with DNS peers?
the peer cache is shared by all Zebra instances run by a single user, is this ok?

Reviewer Checklist

[ ] Will the PR name make sense to users?

[ ] Does it need extra CHANGELOG info? (new features, breaking changes, large changes)

[ ] Are the PR labels correct?
[ ] Does the code do what the ticket and PR says?

[ ] Does it change concurrent code, unsafe code, or consensus rules?

[ ] How do you know it works? Does it have tests?

Follow Up Work
We might also want to add hard-coded IP addresses to the default list of peers.

@arya2 anything I can do to help get this reviewed and merged?

@mergifyio refresh

@arya2 this is ready for re-approval. (Or anyone can do it, it's just a dependency merge.)


Receive Popup Activity

Add receive pop up activity, removing it from the bottom navigation, and its associated fragment files
Getting the address from LocalStore public key that was saved on key generation
QR code works as tested with @JonLister

@Daniel-Wang LGTM. Feel free to merge it in.


feat: Support Python 3.11

Closes #6922

Pleasantly surprised that adding it was so smooth 😁

Pleasantly surprised that adding it was so smooth :grin:

Same


Prepare basic makefile

Make has been chosen for build system for this project. Therefore basic makefile should be made. It should allow for basic compiling examples and unit tests.

Decision has been made to move all build-related scripts to separate repository. Therefore this issue is no longer valid and will be closed.


applyTransform() fails on MultiLineString

Describe the bug
It seems to me that applyTransform() fails on MultiLineString geometries.
To Reproduce
In the API manual is written that applyTransform() applies a transform function to each coordinate of a geometry. Therefore in the following javascript code both variants "Snippet 1" and "Snippet 2" should produce identical output:

var trk, mls, map = new ol.Map({
target: 'map',
layers: [new ol.layer.Tile({source: new ol.source.OSM()})],
view: new ol.View({center: ol.proj.fromLonLat([7,50]), zoom: 4})
});
if (flg) { // Snippet 1
trk = [ [[0,51],[2,48],[4,50]] , [[4,52],[13,52]] ];
mls = new ol.geom.MultiLineString(trk);
} else { // Snippet 2
trk = [ [[51,0],[48,2],[50,4]] , [[52,4],[52,13]] ];
mls = new ol.geom.MultiLineString(trk);
mls.applyTransform(function(coordinate) { return coordinate.reverse(); });
}
var ftr = new ol.Feature({geometry: mls.transform('EPSG:4326', 'EPSG:3857')});
var src = new ol.source.Vector({features: [ftr]});
map.addLayer(new ol.layer.Vector({source: src}));

But the track segments displayed on the map differ:

Tested with
<EMAIL_ADDRESS><EMAIL_ADDRESS>

Simply reversing the corrdinates will not work with most geometries.  It only works with a simple linestring (where the coordinates do not include elevation as with GPX) but it also reverses the direction of the string.  A correctly specified transform function https://openlayers.org/en/latest/apidoc/module-ol_proj.html#~TransformFunction would look like
mls.applyTransform(function (input, opt_output, opt_dimension) {
    var length = input.length;
    var dimension = opt_dimension !== undefined ? opt_dimension : 2;
    var output = opt_output !== undefined ? opt_output : new Array(length);
    for (i = 0; i < length; i += dimension) {
        var x = input[i];
        var y = input[i + 1];
        output[i] = y;
        output[i + 1] = x;
    }
    return output;
});

Don't confuse applyTransform with the forward and reverse transform functions for https://openlayers.org/en/latest/apidoc/module-ol_proj.html#.addCoordinateTransforms which take a single coordinate pair
trk = [ [[51,0],[48,2],[50,4]] , [[52,4],[52,13]] ];
mls = new ol.geom.MultiLineString(trk);

var projLatLon = new ol.proj.Projection({ code: 'LatLon' });
ol.proj.addProjection(projLatLon);
ol.proj.addCoordinateTransforms(
    'EPSG:4326',
    projLatLon,
    function(coordinate) { return coordinate.reverse(); },
    function(coordinate) { return coordinate.reverse(); }
);
mls.transform('LatLon', 'EPSG:4326')

var ftr = new ol.Feature({geometry: mls.transform('EPSG:4326', 'EPSG:3857')});

I agree that the reverse() function does not work on 3-dimensional coordinates because [latitude, longitude, elevation] is converted to [elevation, longitude, latitude] which does not make sense in most cases. But for a value pair [latitude, longitude] the reserve() function returns [longitude, latitude], and that is OK.
The real problem is, that applyTransform() does not work as described in the API manual. The sentence "Apply a transform function to each coordinate of the geometry" implies that the transform function is called 5 times for a multiline string containing 5 coordinates, once for each value pair [latitude, longitude]. In reality [[[0,51],[2,48],[4,50]], [[4,52],[13,52]]] is converted to [0,51,2,48,4,50,4,52,13,52]. Afterwards the transform function is executed on this 1-dimensional array of 10 elements that results in [52,13,52,4,50,4,48,2,51,0]. Finally that is interpreted as [[[52,13],[52,4],[50,4]], [[48,2],[51,0]]].
The description in the API manual should be revised.

"Apply a transform function to the coordinates of the geometry" might be more appropriate description when "A transform function accepts an array of input coordinate values".  There is also another OL6 API issue: applyTransform() is not appearing in the subclasses of SimpleGeometry as it did in previous versions (or maybe flat coordinate methods should not be in the public API when they can be achieved using projection transforms?)


Battery charge sensor has invalid value in 3.0.0 beta 1

Just got this error after upgrading to v3.0.0 beta 1:
2024-02-02 04:00:26.981 INFO (MainThread) [custom_components.jlrincontrol] Unloading JLR InControl Component
2024-02-02 04:00:26.996 WARNING (MainThread) [homeassistant.helpers.template] Template variable warning: 'None' has no attribute 'batteryVoltage' when rendering '{{ state_attr('sensor.discovery_sport_all_info','core status').batteryVoltage }}'
2024-02-02 04:00:26.996 ERROR (MainThread) [homeassistant.helpers.event] Error while dispatching event for sensor.discovery_sport_all_info to <Job track state_changed event {'sensor.discovery_sport_all_info'} HassJobType.Callback <bound method TrackTemplateResultInfo._refresh of <TrackTemplateResultInfo {Template<template=({{ state_attr('sensor.discovery_sport_all_info','core status').batteryVoltage }}) renders=8>: <RenderInfo Template<template=({{ state_attr('sensor.discovery_sport_all_info','core status').batteryVoltage }}) renders=8> all_states=False all_states_lifecycle=False domains=frozenset() domains_lifecycle=frozenset() entities=frozenset({'sensor.discovery_sport_all_info'}) rate_limit=None has_time=False exception=None is_static=False>}>>>
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/sensor/__init__.py", line 639, in state
    numerical_value = int(value)
                      ^^^^^^^^^^
ValueError: invalid literal for int() with base 10: ''

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/sensor/__init__.py", line 642, in state
    numerical_value = float(value)
                      ^^^^^^^^^^^^
ValueError: could not convert string to float: ''

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/event.py", line 296, in _async_dispatch_entity_id_event
    hass.async_run_hass_job(job, event)
  File "/usr/src/homeassistant/homeassistant/core.py", line 691, in async_run_hass_job
    hassjob.target(*args)
  File "/usr/src/homeassistant/homeassistant/helpers/event.py", line 1215, in _refresh
    self.hass.async_run_hass_job(self._job, event, updates)
  File "/usr/src/homeassistant/homeassistant/core.py", line 691, in async_run_hass_job
    hassjob.target(*args)
  File "/usr/src/homeassistant/homeassistant/components/template/template_entity.py", line 429, in _handle_results
    self.async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 945, in async_write_ha_state
    self._async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1066, in _async_write_ha_state
    state, attr, capabilities, shadowed_attr = self.__async_calculate_state()
                                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1003, in __async_calculate_state
    state = self._stringify_state(available)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 951, in _stringify_state
    if (state := self.state) is None:
                 ^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/sensor/__init__.py", line 646, in state
    raise ValueError(
ValueError: Sensor sensor.discovery_sport_battery_voltage has device class 'voltage', state class 'measurement' unit 'V' and suggested precision 'None' thus indicating it has a numeric value; however, it has the non-numeric value: '' (<class 'str'>)
2024-02-02 04:00:27.013 WARNING (MainThread) [homeassistant.helpers.template] Template variable warning: 'None' has no attribute 'tuStatusPrimaryChargePercent' when rendering '{{ state_attr('sensor.discovery_sport_all_info','core status').tuStatusPrimaryChargePercent }}'
2024-02-02 04:00:27.013 ERROR (MainThread) [homeassistant.helpers.event] Error while dispatching event for sensor.discovery_sport_all_info to <Job track state_changed event {'sensor.discovery_sport_all_info'} HassJobType.Callback <bound method TrackTemplateResultInfo._refresh of <TrackTemplateResultInfo {Template<template=({{ state_attr('sensor.discovery_sport_all_info','core status').tuStatusPrimaryChargePercent }}) renders=8>: <RenderInfo Template<template=({{ state_attr('sensor.discovery_sport_all_info','core status').tuStatusPrimaryChargePercent }}) renders=8> all_states=False all_states_lifecycle=False domains=frozenset() domains_lifecycle=frozenset() entities=frozenset({'sensor.discovery_sport_all_info'}) rate_limit=None has_time=False exception=None is_static=False>}>>>
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/sensor/__init__.py", line 639, in state
    numerical_value = int(value)
                      ^^^^^^^^^^
ValueError: invalid literal for int() with base 10: ''

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/sensor/__init__.py", line 642, in state
    numerical_value = float(value)
                      ^^^^^^^^^^^^
ValueError: could not convert string to float: ''

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/event.py", line 296, in _async_dispatch_entity_id_event
    hass.async_run_hass_job(job, event)
  File "/usr/src/homeassistant/homeassistant/core.py", line 691, in async_run_hass_job
    hassjob.target(*args)
  File "/usr/src/homeassistant/homeassistant/helpers/event.py", line 1215, in _refresh
    self.hass.async_run_hass_job(self._job, event, updates)
  File "/usr/src/homeassistant/homeassistant/core.py", line 691, in async_run_hass_job
    hassjob.target(*args)
  File "/usr/src/homeassistant/homeassistant/components/template/template_entity.py", line 429, in _handle_results
    self.async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 945, in async_write_ha_state
    self._async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1066, in _async_write_ha_state
    state, attr, capabilities, shadowed_attr = self.__async_calculate_state()
                                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1003, in __async_calculate_state
    state = self._stringify_state(available)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 951, in _stringify_state
    if (state := self.state) is None:
                 ^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/sensor/__init__.py", line 646, in state
    raise ValueError(
ValueError: Sensor sensor.discovery_sport_battery_charge has device class 'battery', state class 'measurement' unit '%' and suggested precision 'None' thus indicating it has a numeric value; however, it has the non-numeric value: '' (<class 'str'>)****

Can you provide diagnostic download. Thx

Sure. Here you go.
config_entry-jlrincontrol-f9956c9594093151c8e7d18c2f427721.json.txt.json

Great thx.  So I think this is a template sensor you have created that errored when the integration was unloaded.  Does this template sensor work with v3.0.0beta1 after you installed it?  If not, what is the yaml of the template sensor?

I was about to reply that I don't have any template sensors but thought I should double-check just to be sure. Turns out I had two sensors configured via the Helpers screen and I don't even remember creating them. Deleting them has resolved the issue.


Shared Activation Issue

Is this document updated to reflect the steps required eluded to in the below issue?
I do not see any additional steps described beyond setting the key which is what caused our issues as well.
https://github.com/MicrosoftDocs/OfficeDocs-DeployOffice/issues/142
For those searching for a resolution below is the script we performed that seems to be resolving the issue.
Script is being executed as a part of PSADT toolkit
	#Get Office bits
	If (Test-Path "$envProgramfiles\Microsoft Office\Office16\ospp.vbs"){
		$officebits = "$envProgramfiles\Microsoft Office\Office16"
	} Else{
		$officebits = "$envProgramFilesX86\Microsoft Office\Office16"
	}

	$officeitems = Get-WmiObject -Class SoftwareLicensingProduct -Property "ID, ApplicationId, PartialProductKey, Description, Name, ProductKeyID" -Filter "ApplicationID='0ff1ce15-a989-479d-af46-f275c6370663' and PartialProductKey <> NULL"

    Foreach ($office in $officeitems){
        $key = $office.PartialProductKey
		Execute-Process "cscript.exe" -Parameters "`"$officebits\ospp.vbs`" /unpkey:`"$key`""
	}

Question: Would users that activated before fixing the activation continue to have issues until their office credentials are cleared?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8ab31873-7ad3-afde-3c72-8a7fb4f2e551
Version Independent ID: 19f7f483-9f53-f577-1cd4-d360c0b234fc
Content: Overview of shared computer activation for Office 365 ProPlus
Content Source: DeployOffice/overview-of-shared-computer-activation-for-office-365-proplus.md
Service: o365-proplus-itpro
GitHub Login: @DHB-MSFT
Microsoft Alias: danbrown

Hi LionZum,
Looks like someone has mentioned it here:
https://docs.microsoft.com/en-us/deployoffice/troubleshoot-issues-with-shared-computer-activation-for-office-365-proplus
Under the closed comment, there is an issue with applying the GPO after activation has taken place...

We have users encountered the same issue. When the computer has activated Office with the normal method before, there will be a local license in the machine. SCA cannot create a license token if there are local licenses and Office shows as Unlicensed. The license needs to removed first with OSPP tool. When will this KB be updated and include this?

I've submitted pull request #736 to update the documentation on this issue. It is linking off to a description on how to reset activation. Thanks for bringing this up.
@DHB-MSFT Please label and close


CTest to run detected interpreter

This allows to test with another interpreter (eg python3) than the default one used in the shebangs.

@Achimh3011 hi, could you review this one too ?


groupInt on negative numbers

sformat (groupInt 3 ',') -100 gives "-,100"

Added regression tests for this.


libnvme, nvme-cli: 1.4 -> 1.6, 2.4 -> 2.6

Description of changes
libnvme release: https://github.com/linux-nvme/libnvme/releases/tag/v1.6
libnvme commits: https://github.com/linux-nvme/libnvme/compare/v1.4...v1.6
nvme-cli release: https://github.com/linux-nvme/nvme-cli/releases/tag/v2.6
nvme-cli commits: https://github.com/linux-nvme/nvme-cli/compare/v2.4...v2.6
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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.

@ofborg build pkgsStatic.nvme-cli pkgsMusl.nvme-cli

FAILED: test/ioctl/libmock-ioctl.so.p/mock.c.o
gcc -Itest/ioctl/libmock-ioctl.so.p -Itest/ioctl -I../test/ioctl -fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -std=gnu99 -fomit-frame-pointer -D_GNU_SOURCE -include internal/config.h -fPIC -MD -MQ test/ioctl/libmock-ioctl.so.p/mock.c.o -MF test/ioctl/libmock-ioctl.so.p/mock.c.o.d -o test/ioctl/libmock-ioctl.so.p/mock.c.o -c ../test/ioctl/mock.c
../test/ioctl/mock.c:117:5: error: conflicting types for ‘ioctl’; have ‘int(int,  long unsigned int, ...)’
  117 | int ioctl(int fd, unsigned long request, ...)
      |     ^~~~~
In file included from ../test/ioctl/mock.c:9:
/nix/store/qnzi64i0hccghn024klcax1nd31s578f-musl-1.2.3-dev/include/sys/ioctl.h:115:5: note: previous declaration of ‘ioctl’ with type ‘int(int,  int, ...)’
  115 | int ioctl (int, int, ...);
      |     ^~~~~

it appears that the mock might need a musl-specific patch...

Thanks @Mic92!

We can't evaluate now, as a channel-critical test is broken.  Test:
$ nix eval -f nixos/release-combined.nix nixos.tests.boot.uefiUsb.aarch64-linux.outPath
error: Package ‘libhugetlbfs-2.23’ in pkgs/development/libraries/libhugetlbfs/default.nix:44 is marked as broken, refusing to evaluate.

OfBorg did build the stuff, so maybe the broken mark shouldn't really be there anymore?

Actually libhugetblfs builds just fine on aarch64: https://github.com/NixOS/nixpkgs/pull/264439


Add mdc to update command (DAT-12602)

Impact

[ ] Bug fix (non-breaking change which fixes expected existing functionality)
[x] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
Adds mdc values for the update command.
Things to be aware of
Opted to create a MdcKey class to use to keep track of all relevant keys, as well as an MdcValue class for any custom values that currently didn't map to something.
Things to worry about
This technically reaches beyond just the update command, but there's not much I can do about that without refactoring a good amount of code. Not necessary for a POC imo.
Additional Context
N/A

Nice work here, this looks great!

@abrackx Can we add some tests?


Unable to read books after upgrade to 0.25.1 from 0.24.1

Describe the bug
After upgrading from 0.24.1 to 0.25.1 I can create a new book (it shows in the books list), but I can't open the book or any old books and pages, just getting a "An unknown error occurred" message. The new book doesn't show in the recent activity stream on the homepage.
The log file has this is for each attempt to view
[2019-02-06 09:41:44] production.ERROR: Type error: Argument 1 passed to BookStack\Services\ViewService::add() must be an instance of BookStack\Entity, instance of BookStack\Entities\Book given, called in /opt/bookstack/BookStack/vendor/laravel/framework/src/Illuminate/Support/Facades/Facade.php on line 221 {"userId":1,"email":"removed","exception":"[object] (Symfony\Component\Debug\Exception\FatalThrowableError(code: 0): Type error: Argument 1 passed to BookStack\Services\ViewService::add() must be an instance of BookStack\Entity, instance of BookStack\Entities\Book given, called in /opt/bookstack/BookStack/vendor/laravel/framework/src/Illuminate/Support/Facades/Facade.php on line 221 at /opt/bookstack/BookStack/app/Services/ViewService.php:27)
Full stacktrace here: https://pastebin.com/56W4GnB8
Steps To Reproduce
Steps to reproduce the behavior:
Not been able to test this on a clean 0.24.1 install unfortunately.

Update to 0.25.1 from 0.24.1
Try and view a book or page
See error

Expected behavior
Book or page is shown.
Your Configuration (please complete the following information):

Exact BookStack Version (Found in settings): 0.25.1
PHP Version: PHP 7.1.23-2+0~20181017082658.9+jessie~1.gbpab65a0
Hosting Method (Nginx/Apache/Docker): Nginx

Additional context
I can see no issues during the update, log here: https://pastebin.com/8hLFCvfN

Ok this seems to have been down to the PHP version or something broken with my PHP install that was fixed by updating to the latest version on my system which seems to be PHP 7.1.26-1+0~20190113101856.12+jessie~1.gbp7077bb.


renovate: configuration

Description of your changes

Configure renovate, configuration similar to crossplane's one. Not targeting release branches on purpose.
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
Configured on my fork.

@phisco looks like we also have dependabot configured: https://github.com/crossplane/crossplane-runtime/pulls/app%2Fdependabot
AFAIR, we have agreed on converging on renovate, should we disable dependabot in this case?

Take care not to disable dependabot security alerts though.

Ok, disabled Dependabot security updates and Dependabot version updates was already disabled since both will be covered by renovate. Can you confirm?

Perfect, thanks! 👍


Link to class and attribute documentation in docs

In the docs, it would be nice to consistently link to referenced class and attribute names. In the RST files, this can be done using, e.g., :attr:.
However, in the docstrings is there a way to do this without making the strings ugly to read in their raw form? I'd like to preserve readability via help() and while looking at the source.

I'm going to close this since it seems we silently bit the bullet and proceeded with the "ugly" option, linking via :attr:, :class:, etc, where applicable.


Add Firefox platform feature statuses

https://mozilla.github.io/platatus/#canvas_media_capture
https://mozilla.github.io/platatus/status.json
https://github.com/mozilla/platatus

Awesome, been a bit out of the loop lately so thanks for pointing this out. Hopefully can get it added soon.

Done! I've made a pass through their current listed and added IDs, though should probably also remove links already covered.

X-Ref https://github.com/mozilla/platatus/issues/36

Thanks a lot @cvrebert, great to see everything becoming interconnected even before we hit platatus' v1 milestone! There should be an official update on platatus soon and moving to an official domain and repo.


Bug: TypeError: Cannot read property 'from' of undefined

Are you reporting a bug?
Yes.

Fresh install of react-static with npm install -g react-static
Calling react-static create fails. See below.

Environment

react-static -V:
6.0.0

node -v:
v8.11.4
v10.14.2 (same error)

npm -v:
5.6.0

Operating system:
Windows 10

Steps to Reproduce the problem
Base your steps off of any freshly installed react-static template!
d:\myTemp>react-static create
C:\Users\karland\AppData\Roaming\npm\node_modules\react-static\node_modules\convert-source-map\index.js:61
  return SafeBuffer.Buffer.from(json, 'utf8').toString('base64');
                           ^

TypeError: Cannot read property 'from' of undefined
...

works for me:
install as dependency, call
node ./node_modules/react-static/bin/react-static-create
but the error with time reappears and helps reinstalling

node -v
v10.14.2

npm -v
6.4.1

Operating system:
Windows 10

I just released a fix for the create command. Can you please try again?

I tried to install, but after the install of
npm install -g react-static

I get
react-static -v
4.3.2

I checked. 6.0.2 is already on npm.
Thereafter some errors occur, but I suppose they are irrelvant since the versions got mixed up?

It appears this has been resolved. Reopen if more information becomes available. Thanks!


Remove the OverlayManager.unregister method since it's completely unused

This method was added only for consistency with the register-method, however it's never actually been used. To avoid including dead code in the builds, let's just remove the unregister-method for now.
Please note: If this method ever becomes useful, it'll be trivial to revert this commit.

/botio-linux preview


fix: add missing main entry to the packages

This allows the tools to work properly. For example, eslint-plugin-import, TypeScript LSP hyperclick, and many other tools rely on main.
What I did

add missing main entry to the packages

The changeset doesn't work for me. That's why I usually find describing the changes in the commit message easier than duplicating them in a third place.
❯ pnpx changeset
Packages: +3
+++
Packages are hard linked from the content-addressable store to the virtual store.
  Content-addressable store is at: C:\Users\aminy\.pnpm-store\v3
  Virtual store is at:             node_modules/.pnpm
Progress: resolved 3, reused 3, downloaded 0, added 3, done
 WARN  changeset has no binaries

C:\Users\aminy\AppData\Local\npm-cache\_npx\10764\pnpm-global\4:
+ changeset 0.2.6
EISDIR: illegal operation on a directory, read
⨯ aminy@LAPTOP-DHBEBJRL  ~\Documents\GitHub\JavaScript\rocket   fix-missing-main ≣                         [19:11]

❯ npx changeset
npm ERR! could not determine executable to run

npm ERR!      C:\Users\aminy\AppData\Local\npm-cache\_logs\2021-03-06T01_11_27_743Z-debug.log

this project requires yarn 1.x as it uses yarn workspaces... then a yarn changeset should work.
But no worries a changeset is just a file - which I added now 🤗
thank you for your help 🙇♂


NullReferenceException on get_SharedDomainAddress onlinux

using the gcroot sample by
https://github.com/Microsoft/dotnet-samples/blob/master/Microsoft.Diagnostics.Runtime/CLRMD/GCRoot

my only change is change the LoadCrashDump to LoadCoreDump
System.NullReferenceException: Object reference not set to an instance of an object.
   at Microsoft.Diagnostics.Runtime.Desktop.DesktopRuntimeBase.get_SharedDomainAddress()
   at Microsoft.Diagnostics.Runtime.Desktop.DesktopGCHeap.LoadAllTypes()
   at Microsoft.Diagnostics.Runtime.Desktop.DesktopGCHeap.EnumerateTypes()+MoveNext()
   at Microsoft.Diagnostics.Runtime.Desktop.DesktopGCHeap.EnumerateRoots(Boolean enumerateStatics)+MoveNext()
   at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
   at EEHeap.Program.FillRootDictionary(ClrHeap heap) in /coretrace/gcroot/Program.cs:line 175
   at EEHeap.Program.Main(String[] args) in /coretrace/gcroot/Program.cs:line 50

i try this on both 2.2 and 3.0 privew container

It is probably due to the fact that the shared domain was removed from .NET core few months ago and daccess now reports the sharedDomain as being null.

This was fixed in cd072107.


Two AUX ephemeris files V01 and V01U for some days

On the server, there are sometimes two aux-ephemeris files, for example:
| solo_L2_stix-aux-ephemeris_20231004_V01.fits | 2023-10-25 15:14 | 383K |
-- | -- | -- | -- | --
  | solo_L2_stix-aux-ephemeris_20231004_V01U.fits | 2023-10-27 22:06 | 383K |
This seems to be the case for many days in October 2023. There are also days in September 2023 where only a V01U file is available, although the data should be complete by now...

Can this be close @nicHoch / @FredSchuller


question

Hi Most of the wrappers not working with Material-UI v1.. any  plan to release for this.

@viveknaragude  Make sure you're using next version.

Yes i using next version (material-ui/core": "^1.4.2",)


PHP 8.1 support

Adding PHP 8.1 support (issue #93)

Updating dependencies (composer.json)
Updating tests
Updating PHP requirements (PHP >= 7.3)

Thanks @msztorc
I thought I had done this, but apparently not!


Directory listing changes in Pelican client

This is a different approach to the change of directory listings. Now, for recursive downloads, we will query the director for a PROPFIND to get the location of our "dirlisthost" (or origin). Then procede with the normal recursive download code. If we receive a 405 from the director doing this PROPFIND, we will revert back to the old way of doing things

Note: this still needs a unit test to test multiple origin support so hold off on a review for now.

@bbockelm see if you like these changes better, I think I hit all of the suggestions you gave. Also, we want listings to be true by default correct?


Rewording resume option note

Rewording resume option note

@PPacentMSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks fine, thanks. #sign-off