implement logic to compute node location information

Try to add location information on CstNodes

[x] implement basic logic
[x] make this optionnal/toggle-able
[x] add test(s)
[ ] performance analysis
[ ] updating documentation

Close #932

 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.

I just added a test, but some other should certainly be added.
I was not sure where it had to be added, and what exactly to test : please let me know what should be changed !

Awesome! I'd love to see some progress on this and would like to help out, when I find the time. Cheers!

I just added a test, but some other should certainly be added.

You should check that the position information is collected for in-lined rules too.

https://sap.github.io/chevrotain/docs/guide/concrete_syntax_tree.html#in-lined-rules

Have a look here for benchmarking:

https://github.com/SAP/chevrotain/blob/master/packages/chevrotain/benchmark_web/README.md

But make sure to rebase your changes with master first as there have been some performance fixes recently.

Regarding performance, I got these numbers, with the current implementation (with rebasing with master) :

I assume you enabled the CST creation in the benchmark options:

https://github.com/SAP/chevrotain/blob/master/packages/chevrotain/benchmark_web/parsers/options.js

What computer are you using? the overall performance seems pretty low, or perhaps you are not using Chrome to test?

I used firefox as it didn't seem working on Chrome (some permissions issue, I guess: I got the error Blocked a frame with origin "null" from accessing a cross-origin frame)

V8 performance is the most important and chrome duplicates this.
I am not familiar with the problem you are describing as I never encountered it on multiple machines and OS.
Perhaps you can disable some web security policies during local development in your chrome.

https://stackoverflow.com/questions/3102819/disable-same-origin-policy-in-chrome

Now I have these results (with outputCst enabled in dev and last) :

compare to these with the master version:

So that's not really good :sweat_smile:
I'll try to optimize with your suggestions

Really awesome to see so much progress already.
What is the current performance in "real numbers" (ms) for a typical use case, say for a typical file of around 1000 statements/expressions total? Any idea?
Needs to be less than 10ms for sure.

What  is the current performance in "real numbers" (ms) for a typical use  case, say for a typical code file of around ~1000 nodes in total? Any  idea? Needs to be less than 10ms for sure.

That is a very wide question.
The inputs for the JSON/CSS benchmark are ~1,000 lines files so for the JSON use case you are looking at 1.2Milion lines of code per second (divide by 100 for 10ms).
The JavaScript code sample is around ~3,000 LOC however which makes the math less straightforward.
But it also depends on the grammar, the more complex the grammar the slower the parsing would be.
Some extreme edge cases it may be possible to get as low as 20-30K LOC/second (Java Parser implemented in the JHipster org).  So we are looking at a very wide range of 1 million to several tens of thousands LOC per second in general.
The problem is that the current implementation seems to reduce performance by 30-40% which is quite significant.

@clement26695  why did you post two pictures? the "dev" variant is with your changes and the "latest" is the latest released version on npm which should be very close to master. basically the benchmark is meant to compare upcoming version vs previous version to detect regressions.

I didn't know if the performance fixes you mentioned were already released or not, so that's why I tested with the master branch (and as I saw the results, I saw some performance improvements)

But make sure to rebase your changes with master first as there have been some performance fixes recently.

In general, I'd recommend to always to avoid (at least complex) if statements or even multi line functions and use extract method to encapsulate every piece of distinct logic.
PS: On a personal note I never use any statements at this point, such as if, for, case, ... Expressions are always a better option (more composable etc)
You could simply do x() || y(a,b) || z(a) instead of using if/else or case.
Beautiful code :)

PS: On a personal note I never use any statements at this point, such as if, for, case, ... Expressions are always a better option (more composable etc)

This may be related but we are discussing a different kind of optimization:
let foo = function() {
   console.log("hello")
}

if (process.env["flag"]) {
   // noop function...
   foo = function() {}
}

function bar() {
  // if the flag is enabled foo === NOOP
  // So once bar would be optimized (in-lined) by V8, this call would completely disappear
  // and the optimized "bar" would only contain a single statement and thus be faster.
  foo()
  console.log("world")
}

I tried to implement it as you suggested.
I put
location: {
  startOffset: Number.MAX_VALUE,
  startLine: Number.MAX_VALUE,
  startColumn: Number.MAX_VALUE,
  endOffset: -1,
  endLine: -1,
  endColumn: -1
}

as default values, so that I don't have to check if the current values are NaN in the setNodeLocationXXX functions (=> check for NaN is commented right now, I'll remove it if your ok with that, ok put it again if you prefer using NaN as default values)

Regarding benchmark, I have to admit that I'm not completely sure about the figures I have, as they can change radically between two runs, without making any changes in the code. I got these results :

@bd82, did you had time to check this ? :smile:

did you had time to check this ? 😄

Sorry nope, pretty busy lately... 😢
but thanks for reminding me 👍
Hopefully I will have a look on the weekend.

O.k. finally got around to review this.
See for comments:

https://github.com/SAP/chevrotain/commit/114a3b70a4ac1e453f1c873bc92536b8a60e0d0a

I think:

In general this looks good.
We can still likely improve the performance a bit.
I am getting 20-30%% performance regression in the parser only test when doing full tracking.

This means 10-15% in the combined le.xer +parser flow.
It is not great but it may be o.k. considering:

Not everyone needs full location tracking on CST Nodes.
We can likely improve the performance a bit (See comments in review).
Perhaps this option should not be enabled by default.

Without knowing the details, I think for many typical use cases, the performance is ok at this point.
Let's get a solution that works "out in the open" and then we will see... Good job!
Cheers!

Without knowing the details, I think for many typical use cases, the performance is ok at this point.
Let's get a solution that works "out in the open" and then we will see... Good job!
Cheers!

The performance is o.k. for the general case, but Chevrotain is not so fast without continued obsession with performance 😄 .
Also @clement26695 already detected a possible bug here:

https://github.com/SAP/chevrotain/commit/114a3b70a4ac1e453f1c873bc92536b8a60e0d0a#r33667766

If you need this  feature now you can just add the relevant logic from this PR (with slight mods) to your parser by overriding   certain methods on your parser.
You can also add a post processing phase to your CST Nodes without  any changes to your parser.
But the official feature would have to be fully ready before it is merged.

Although it seems already 80% done to me. (Great work @clement26695 ).

I updated the logic with your suggestion and put a default value.
I also added some tests for the different cases (full, onlyStart, onlyOffset, none, and undefined)
I think I just didn't edit the documentation yet

I am merging this to continue work on master branch.
Thanks @clement26695 for this awesome work! 👍


Missing packages on apt run:

# dpkg --set-selections < /etc/packages.txt
dpkg: warning: package not in database at line 8: apt-file
dpkg: warning: package not in database at line 38: command-not-found
dpkg: warning: package not in database at line 56: dh-lua
dpkg: warning: package not in database at line 313: libncurses5-dev:amd64
dpkg: warning: package not in database at line 314: libncursesw5-dev:amd64
dpkg: warning: package not in database at line 378: libruby2.2:amd64
dpkg: warning: package not in database at line 456: libxml2-dev:amd64
dpkg: warning: package not in database at line 460: libxslt1-dev:amd64
dpkg: warning: package not in database at line 468: locales-all
dpkg: warning: package not in database at line 675: reptyr
dpkg: warning: package not in database at line 740: weechat-plugins
dpkg: warning: found unknown packages; this might mean the available database
is outdated, and needs to be updated through a frontend method

This was a sympton of packages.txt not being commited correctly.

No it wasn't.... I think. something weird is happening.
I think we need to run dselect update as part of the sync process.

We already do, see sync.yml.

We already do, see sync.yml.

update != upgrade.
But I added it: https://github.com/hashbang/admin-tools/blob/master/sync.yml#L15 :)

Oops, yes, misread.
We don't need

We don't need dselect update as far as I can see, since we already update dpkg available packages db.

Via what?
apt-get has a different package index/cache than dselect.


AI walls and windows

About the PR
Added structural assets for the Almayer's AI core.
Why / Balance
Apparently this is holding that section of the ship back from being mapped.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

not your problem. IntegrationTests fails as always.


Transparency on mobile

Hi,
I love the theme.
I use obsidian a lot on my mobile and there I find the following issue.
The bar on the left side of the screen is not hidden as marked in the screenshots.
It would be nice if the underlying bar is completely hidden by the layer above (files, search etc.)
Regards,
Wilco

Thank you for bringing this issue to my attention. I'm happy to let you know that the problem has been fixed. To resolve the issue you described, please ensure that you have the latest version of the theme.
I appreciate your feedback and hope that you continue to enjoy using the theme.


build:devserver and start:devserver fail to launch on windows after upgrading to 0.3.0

This is separate from trying to run production, which is still broken, but was before 0.3.0 as well: https://github.com/eddyystop/feathers-starter-react-redux-login-roles/issues/7#issuecomment-269819551
npm run build:devserver had been working before 0.3.0 (whatever was current yesterday morning worked.)  Now it gives this:
> NODE_ENV=devserver webpack-dev-server --config webpack.devserver.config.js --c
ontent-base public/dist/ | grep -v  "\[built\]"

'NODE_ENV' is not recognized as an internal or external command,
operable program or batch file.

npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\
node_modules\\npm\\bin\\npm-cli.js" "run" "build:devserver"
npm ERR! node v7.3.0
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>build:devserver: `NODE_E
NV=devserver webpack-dev-server --config webpack.devserver.config.js --content-b
ase public/dist/ | grep -v  "\[built\]"`
npm ERR! Exit status 255

I get similar results trying to run npm run start:devserver, which also had been working prior to 0.3.0.

npm run win:build:devserver should be used on Windows.
README and FAQ updated in 0.3.1.


Unrecognized option "disableIdentifierQuoting" under "propel.generator.objectModel"

Getting the following error in Propel 2 when propel.yaml has disableIdentifierQuoting set under objectModel.
Unrecognized option "disableIdentifierQuoting" under "propel.generator.objectModel"
Some of the column names in our mysql DB are reserved keywords so they need to be quoted.  With this option unavailable, the generated SQL statements have no quotes.
Is there a workaround for this?

I guess the documentation is outdated. "identifierQuoting" is now an option at <database> and <table>

Added it.

Thank you.  You've added the "propel.generator.objectModel.identifierQuoting=true|false" in propel.ext, right?  This would allow us to control quoting in the "propel database:reverse" command.

Please update the latest documentation and the references on the site ..
Newbies are struggling a lot by finding proper syntax ..
http://propelorm.org/documentation/
http://propelorm.org/documentation/reference/
http://propelorm.org/Propel/reference/
http://propelorm.org/Propel/documentation/


Add [Stellargame] #airdrop GAFNTJ…K4NSJE

Address: GAFNTJCPNSJIJJKGVKVLD3XYZ4RRHNMOURY7NZJ57UO3JXXZL7K4NSJE
Tags: airdrop
Title: Stellargame
Requested by @dartersanter

SELF DECLARED PONZI SCHEME
"Ocher Distrubutor
Is a distributor of highly profitable Ponzi tokens - high risks along with high returns! Every day we send half of the XLM we earn from the sale of tokens, the more of our tokens you have - the greater the % of the payment from the total amount of sales you receive every day.
All operations with tokens are at your own risk!"


bug: Canceling an infinite-looping protocol leaves the OT-2 unconnectable

Overview
If you're running a protocol on the OT-2, and that protocol gets stuck in an infinite loop, trying to cancel the protocol will leave the whole OT-2 in a weird unconnectable state.
Steps to reproduce
Run this protocol on an OT-2 and try to cancel it.
metadata = {
    'apiLevel': '2.9'
}

def run(protocol):
    tip_rack = protocol.load_labware('opentrons_96_tiprack_300ul', 8)
    pipette = protocol.load_instrument('p300_multi_gen2', 'left')

    if not protocol.is_simulating():
        while True:
            pass

    pipette.pick_up_tip(tip_rack.wells()[0])
    pipette.drop_tip(tip_rack.wells()[0])

Current behavior

Canceling will give you an infinite spinner on the Run tab in the Opentrons App.
If you close and reopen the Opentrons App, the "Connect" button and the toggle next to the robot's name won't do anything. The "Connect" button will show an infinite spinner.
Most robot controls in the Opentrons App will be disabled.
Eventually, you'll get a "Could not connect to robot" dialog.

Expected behavior
Canceling an infinite-looping protocol should behave as if you're canceling a well-behaved protocol. The cancelation should be reflected promptly in the Opentrons App. The robot should be usable after the cancelation, as normal.
See also

#6126, which is about the lack of timeout/cancel when infinite-looping protocols are simulated.
#7742, which is arguably a problem caused by the same underlying architecture problem as this issue: we don't run protocols in their own processes, so they're hard to clean up, in nontrivial cases.

There's one more ticket on infinite protocols that we should find and link this to.

I haven't looked at the code, but here's my technical understanding of the problem, gathered from internal chats:

We implement protocol cancelation by having opentrons-controlled code check an "is canceled" flag. If the protocol has been canceled, the opentrons-controlled code raises a special exception.
The exception bubbles up through the user-written protocol code to terminate it.
Protocols normally call into opentrons-controlled code frequently, so this works.
On the other hand, infinite-looping protocols (and probably other kinds of "weird" protocols) will rarely, or never, call into opentrons-controlled code. So the exception will never be raised.
The robot is left in a weird unconnectable state because the ongoing thread occupies one of the RPC thread slots? (I don't really get this part.)

Fixed in v5


[Feature]: Make it easier to work with opentelemetry-rust's main branch

Related Problems?
The opentelemetry-contrib crate was moved out of the opentelemetry-rust repository, and unfortunately, this has made it harder for us to keep up to date with non-release builds of OpenTelemetry since we use the opentelemetry-contrib crate.
Previously, we could point our cargo.toml version for all opentelemetry-rust crates to a single git revision to build against the main branch.  This is no longer possible now that contrib was moved out of that repository and is pointing towards a versioned release.
The problem is that opentelemetry-contrib points to opentelemetry-rust 0.21, so if we are using opentelemetry-rust with the git revision set to the latest main commit, we now have two different versions of opentelemetry-rust being used, which causes compile errors.
The main way I've been working around this is

fork this repository and create a branch
Update opentelemetry-contrib's cargo.toml to point to the current opentelemetry-rust main commit
Push the branch
Set my application to use my branch's commit of opentelemetry-contrib and the associated main commit of opentelemetry-rust repository
Make sure these are kept up to date with each other as the opentelemetry-rust repository is updated.

Likewise, I see this becoming an issue if we need to have breaking changes in the API, as it will be harder to test contrib changes against the latest opentelemetry-rust crate.
What component are you working with?
opentelemetry-contrib
Describe the solution you'd like:
There are several solutions and I don't have a big opinion on which way we go (or other way).  I'm just looking for this process to be easier for us.
Possible solutions are:

Move contrib crate back into opentelemetry-rust repo.  It's not totally clear why these are considered separate, since contrib just adds additional APIs and ergonomics
Have a CI pipeline that repoints the cargo.toml to the opentelemetry-rust main commit revision (either on demand or on a cadance)

Considered Alternatives
Open to other suggestions.
Additional Context
No response

Well, after some fiddling and research it turns out the solution to my immediate problem can be solved with cargo patching.  E.g.
[workspace.dependencies]
opentelemetry = { git = "https://github.com/open-telemetry/opentelemetry-rust", rev = "d52ff83f300f46ebbd7d522a96eac4e7fc720eac"}
opentelemetry_sdk = { git = "https://github.com/open-telemetry/opentelemetry-rust", rev = "d52ff83f300f46ebbd7d522a96eac4e7fc720eac"}
opentelemetry-contrib = "0.13"
opentelemetry-otlp = { git = "https://github.com/open-telemetry/opentelemetry-rust", rev = "d52ff83f300f46ebbd7d522a96eac4e7fc720eac"}
opentelemetry-semantic-conventions = "0.13"

[patch.crates-io]
opentelemetry = { git = "https://github.com/open-telemetry/opentelemetry-rust", rev = "d52ff83f300f46ebbd7d522a96eac4e7fc720eac"}
opentelemetry_sdk = { git = "https://github.com/open-telemetry/opentelemetry-rust", rev = "d52ff83f300f46ebbd7d522a96eac4e7fc720eac"}
opentelemetry-proto = { git = "https://github.com/open-telemetry/opentelemetry-rust", rev = "d52ff83f300f46ebbd7d522a96eac4e7fc720eac"}


Error: postgres: column "oid" was not found for index "pg_subscription_oid_index"

Hello,
while using altas against  GitLab Omnibus PostgreSQL service which is custom , I am getting this issue.
atlas schema inspect -u "postgres://gitlab:xxxx@<IP_ADDRESS>:5455/gitlabhq_production?sslmode=disable" --format '{{ sql . }}' > dev.sql
Error: referenced schema "pg_catalog" was not found in realm

Then I have added pg_catalog too in search path ,
╰─ $ atlas schema inspect -u "postgres://gitlab:xxxx@<IP_ADDRESS>:5455/gitlabhq_production?sslmode=disable" -s pg_catalog,public --format '{{ sql . }}' > dev.sql
Error: postgres: column "oid" was not found for index "pg_subscription_oid_index"

any thoughts,
Thank you.

I'm a new user just starting out with atlas and have encountered the same problem:
$ atlas schema inspect -u postgres://xxx@xxx:5305/foo_dev
Error: referenced schema "pg_catalog" was not found in realm

I can access items in the pg_catalog schema when I connect with the same credentials.
The postgres server version is 13.6.
I installed atlas via homebrew: atlas version v0.17.1-75ae281-canary
I have no atlas config file.
If I change the db name in the url from /foo_dev to /postgres then the command produces output for some table in that db. But that's not the db I want to inspect.

Thanks for reporting this, @timbunce and @rinshadka.
To debug this, can you share what PostgreSQL server are you using? If it's a Docker container, a reference to the Docker image will help.

The postgres server version is 13.6 in my case.

Is there a way to enable extra debug/logging/tracing?

I've reproduced it using the official postgres:13.6-alpine docker image.
I enabled statement logging and last statement executed before the error is:
2024-01-04 13:30:15.626 UTC [4897] LOG:  execute <unnamed>:
	SELECT DISTINCT
		parent_ns.nspname AS view_schema,
		parent_view.relname AS view_name,
		parent_view.relkind,
		child_ns.nspname AS referenced_schema,
		child_view.relname AS referenced_name,
		child_view.relkind
	FROM
		pg_catalog.pg_depend
		JOIN pg_catalog.pg_rewrite ON pg_rewrite.oid = pg_depend.objid
		JOIN pg_catalog.pg_class parent_view ON parent_view.oid = pg_rewrite.ev_class
		LEFT JOIN pg_catalog.pg_depend parent_dep ON parent_dep.objid = parent_view.oid AND parent_dep.deptype = 'e'
		JOIN pg_catalog.pg_class child_view ON child_view.oid = pg_depend.refobjid
		JOIN pg_catalog.pg_namespace parent_ns ON parent_ns.oid = parent_view.relnamespace
		JOIN pg_catalog.pg_namespace child_ns ON child_ns.oid = child_view.relnamespace
	WHERE
		parent_view.relkind IN ('v', 'm')
		AND parent_view.oid != child_view.oid
		AND parent_ns.nspname IN ($1)
		AND parent_dep.objid IS NULL

2024-01-04 13:30:15.626 UTC [4897] DETAIL:  parameters: $1 = 'public'

If I run that command manually with public as $1 I get this:
 view_schema |    view_name    | relkind | referenced_schema | referenced_name  | relkind
-------------+-----------------+---------+-------------------+------------------+---------
 public      | v_activity      | v       | pg_catalog        | pg_stat_activity | v
 public      | v_blocked_locks | v       | pg_catalog        | pg_locks         | v
 public      | v_blocked_locks | v       | pg_catalog        | pg_stat_activity | v
 public      | v_config        | v       | public            | bucket           | r
 public      | v_config        | v       | public            | config           | r
 public      | v_config        | v       | public            | v_property       | v
 public      | v_property      | v       | public            | bucket           | r
 public      | v_property      | v       | public            | config           | r
 public      | v_property      | v       | public            | property         | r

I suspected the trigger is having a view that refers to an entity in pg_catalog.
So I deleted the v_activity and v_blocked_locks views and now the atlas schema inspect works.

Thanks for investigating this, @timbunce. This was already fixed in the "new way" we use for inspecting dependencies (which should be the default at some point). You can get access to it using the atlas login - let me know if this solves the issue.


Questions
1 Not all have unit test

randperm

2 Why randperm_out instead of randperm?
3 Wh randint_like instead of randint?

Not all have unit test.

See line 143 in TestTorchSharp.cs

Why randperm_out instead of randperm?

That's a mistake based on the libtorch native API, which has a separate one. I'll correct that in the next release. There's also a randperm()

Why randint_like instead of randint?
https://pytorch.org/docs/stable/generated/torch.randint_like.html?highlight=randint#torch.randint_like


cannot resolve directory<EMAIL_ADDRESS>

@import<EMAIL_ADDRESS>@import<EMAIL_ADDRESS>@import<EMAIL_ADDRESS>

I've the same problem with angular 2.4.4 and ngx-datatable 6.0.2: I created two demos for this. http://encircle360share.s3-eu-central-1.amazonaws.com/css-not-working.zip

Are you using Angular CLI? I had to use relative paths to properly import the files into my scss.

I'm using webpack and npm directly, no angular cli involved. Relative paths are also not working. Using an older version of angular (2.1) with the older ng2-datatable worked fine.
In the demo posted above i used angular-cli with absolute and also relative paths. There it also didn't work.

The documented paths aren't correct, add components/ to your path.
<EMAIL_ADDRESS>

@arlowhite This also doesn't help. I already evaluated the paths and fixed them. Also the IDE is able to resolve the path. It seems that these files don't get served by the internal webserver.

This is a packaging issue I'm working through right now. I can't find a good fix :(

I use ionic2.0.1 with angular 2.2.1 and I'm able to include the css from the angular component doing like this
styleUrls: [<EMAIL_ADDRESS>]
or like this
import indexStyle  from<EMAIL_ADDRESS>styleUrls: [ indexStyle ]

On load the css I get an error
Error: Module parse failed<EMAIL_ADDRESS>Unexpected token (6:0)
You may need an appropriate loader to handle this file type.
|  * Licensed under MIT
|  */
| .ngx-datatable {
|   display: block;
|   overflow: hidden;
    at Object.<anonymous> (http://localhost:8100/build/main.js:129939:7)
    at __webpack_require__ (http://localhost:8100/build/main.js:20:30)
    at http://localhost:8100/build/main.js:63890:25
    at Object.<anonymous> (http://localhost:8100/build/main.js:63896:2)
    at __webpack_require__ (http://localhost:8100/build/main.js:20:30)
    at Object.<anonymous> (http://localhost:8100/build/main.js:93695:105)
    at __webpack_require__ (http://localhost:8100/build/main.js:20:30)
    at Object.<anonymous> (http://localhost:8100/build/main.js:80311:70)
    at __webpack_require__ (http://localhost:8100/build/main.js:20:30)
    at Object.<anonymous> (http://localhost:8100/build/main.js:129929:70)

So finally I'm able to load/find the file from node_module folder but get an error on parse the file.
If I copy the css and all fonts and the other files under the project assets folder I can import the css from the scss file, and it loads right and works. But there is another mistake
importing the icons.css file in my scss file I override the default icons, so after that in my app there strange icons, so I'm not able to encapsulate the icons for my component.
Some ideas?

This should be solved now :)

How is it solved?

I have still the problem

You no longer need to import the css.

Confirm also on ionic2 we do not have to import anymore the index.css and the material.css files.

You should only have to do the theme css, the base styles are auto.

Do you have a .gitignore file ?


Theme wont apply?

I need some help, So I open the config on Spicetify, to direct it to Tidal... after I do Spicetify backup apply, and then Spicetify auto, It doesn't apply a theme, it just open spotify with the basic regular theme... Any idea why?

I guess it's not updated to V2 standards? Or idk.. Theme just stopped working.. i've tried everything.. :/

It seems to be applying for me. Can you send your config file?

There's the config..
and this is what my Spotify looks life when I tried applying it x.x

config.zip

Join the Spicetify Discord and I can help you with it there. https://discord.gg/PHPz4yw3Yp


Homework/Assignment/Assessment 4

Drift video link:
https://video.drift.com/v/ab3qpb36jnvvME8PBJn1yN3BORE8C6uTxTMag1G0pqI/

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
3

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


CEC module library is behind current version in SAM

As discussed in https://github.com/pvlib/pvlib-python/discussions/1344, the CEC module table shipped with pvlib (SAM 2018.11.11 r2, 1/3/2019) is behind the latest version shipped with SAM (SAM 2020.2.29 r3, 11/25/2020).
@mikofski in https://github.com/pvlib/pvlib-python/issues/440 you brought up pvfree -- any thoughts there this time around?
Relevant:

https://github.com/NREL/SAM/issues/17
https://github.com/pvlib/pvlib-python/issues/769
https://github.com/pvlib/pvlib-python/issues/761

It's possible to update pvfree by clicking the admin pulldown, selecting upload, and uploading the desired csv. IMO some advantages of using pvfree or something similar:

library updates don't require a pvlib re-release if they two are decoupled
pvfree db doesn't allow integrity or validation errors, meaning, you can't upload the same file twice or put a string in a number field
human readable, searchable, filterable
API has built in filters that make it easier to locate desired module(s) or inverter(s)
uses off the shelf, vanilla packages (django, tastypie) and 3rd party zero-admin webhosting and deployment (github, heroku) and database tools (elephantSQL). This means pvfree has built in convenience tools like admin page, filtering, security updates, etc. which lowers maintainer burden.

see https://github.com/BreakingBytes/pvfree/issues/34 might need to update this table in the source, easy pr

library updates don't require a pvlib re-release if the two are decoupled

+1. But that feels like a breaking change even if it's not a real API issue, so I'll make the same suggestion for a short term solution as in #440: make a minor pvlib release with the new files AND update pvfree.
https://github.com/pvlib/pvmodules is also relevant. I don't recall if there was a block other than lack of time. @cwhanse?

I don't recall if there was a block other than lack of time.

That's mostly it. NREL also wants to open source the production of the CEC module file. @janinefreeman

library updates don't require a pvlib re-release if the two are decoupled

+1. But that feels like a breaking change

Perhaps that concern could be addressed by pvfree exposing multiple versions of the CEC table via a new API parameter, similar to how the PSM3 API has tmy-2018 and such.  Then code that specifies cecdb='2018.11.11' (or whatever) would behave the same even after a new version of the table is added.

As an interim, we could update the file distributed with pvlib-python to the current SAM release.

SAM 2022.11.21 is released with updated module and inverter libraries: https://github.com/NREL/SAM/tree/patch/deploy/libraries


docs: mark node-to-node IPSec encryption as beta

Keep node-to-node encryption in beta mode, to indicate that some issues might remain to be fixed.
(Note: The whole IPsec page was marked as stable/beta for 1.9).

I'm OK with this, but would consider adding workflows for node to node case and then flipping it out of beta. But, without workflows for us to verify its working in all our supported environments its easy to have a regression in one of the corner cases. So, marking BETA makes some sense until this happens.

(Note: The whole IPsec page was marked as stable/beta for 1.9).

Let's backport anyway because otherwise in v1.9 it's not super clear that node-to-node encryption is beta.


Add a field for a verbatim qualifying clause

We should record a verbatim qualifying clause in a separate field. Eventually, we also need to figure out how to support qualifying clauses in OWL, but this will allow us to identify phyloreferences that have qualifying clauses.

Qualifying clauses will be included in the verbatim definition in any case, so will be visible to users in Klados. Since we haven't implemented qualifying clauses in the OWL representation, I don't think we need to represent them separately in Klados. Closing.

You may end up being right with this, but it seems premature to make that conclusion now.


Add brackets to boxed variable

My bad, wrong change from the previous PR.
Should fix #413

Wait for the ci

Looks like clippy is complaining when there is only 1 boxed value in the function parameter: https://github.com/fzyzcjy/flutter_rust_bridge/runs/6139942673?check_suite_focus=true

Just add ignores. We have already added tons of ignores for the generated rs file. Just follow it

@all-contributors please add @syndim for code

There are some cases where this causes an error at least in my private project, that's why I have some dirty hack lines in gen part of just file:
sed -i -e 's/mirror_SomeStruct((\*v))/mirror_SomeStruct(v)/g' native/src/bridge_generated.rs

Will try to create test case later today.


fix: casing in set params

Minor case fix as new Terraform provider is case sensitive about these things.
Issue was that the parameter name did not exactly match the referenced built-in name.

Hi @Springstone this is a ver minor one that won't affect ARM at all, but will make TF happy - can you take a quick look please?
ta


⚠ Samsung has degraded performance

In 9cc0df4, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2203 ms

Resolved: Samsung performance has improved in cbd2482.


Fix #450 - fast and feature-complete csv parser in go

This is pretty good.  But a lot of what we saved on allocation is now being spent in runtime.slicebytetostring, looks like (needs confirmation) as a result of tokenizer.BufSubstring, which (needs confirmation) also allocates a lot.  Possibly this is a requirement since strings are immutable.  That is unfortunate, virtually all of those strings are discarded almost immediately.  What we need here is the ability to parse floats and ints (especially) directly on []byte, and for there to be match() variations that don't allocate strings.
At the moment I've been looking at data from Saga which of course are untagged and so BufSubstring is very hot because every field is allocated as a string.  But it's going to be the same for newer data because match() will allocate one string when it matches, and all fields match.

This is code from the sandbox, with all optimizations applied.  It needs a few things:

[ ] Thorough review and some cleanup (ongoing)
[ ] Testing against all the existing data with some way of ensuring that we read the same values as the Rust code
[ ] More test cases

To test relative to the Rust code it's probably easiest to rebase the sandbox code on top of this code and evolve it so that it is feature-compatible with sonalyze parse, and then just run the two side-by-side on the same input and compare the output.

The profile looks good, mostly.  On data from the ML nodes, runtime.mallocgc is self+children about 7% (almost 4 months of data, 3.9e6 records in 863 files).  On data from Saga, runtime.mallocgc is self+children about 18% - partly because there are many records (31.6e6 records in 31127 files) and probably because the density of string-allocated data is higher, with records being much shorter and timestamps being 1/8 of fields.  The bulk of allocations for the latter case comes from runtime.newobject which is probably mostly for data records.  We could allocate these inside large arrays, for sure.

Going to merge this and file followup bugs for the allocation optimizations.

Sandbox code saved on larstha-451-sonalyze.


feat(leaves): Add module  for defining user customizable keybindings

This method currently has the downside of not having any way to require that all bindings exist for the given variant type.

Hi @charleslambert, thanks for opening the PR! ✨
Could we refactor this to not use a functor? I'd like the API to be as small/simple as possible.
I was thinking maybe we could have a type like (Event.key * model -> model) list so creating bindings becomes rather trivial. Something like this:
module Key_bindings : sig
  type t
  type 'model binding
= struct
 type t = binding list
 type 'model binding = key * 'model -> 'model
 let on key fn = (key, fn)
 let make bs = bs

 let update key model t =
   match event with
   | KeyDown k ->
      let fns = find_fns k t in
      run_fns fns model
   | _ -> model
end

let bindings = Key_bindings.make [
 on Key.Enter enter_fullscreen;
 on Key.Down move_left
]

What do you think?

I agree removing the functor is probably a good idea. I would still argue to keep the idea of a msg variant rather than directly binding to functions. It makes it easier for a component author to make changes without breaking a consumer of that component. It also gives direct documentation of the possible operations on a component. I will refactor to remove the functor and add a few convenience functions to make the api easier to consume.

Hey @leostera, I removed the functor and updated the api to hopefully be a little better.
What do you think?

Oh i see what you mean now! Love the update looking super clean 🤩
We also have the ability to send custom events which could work even more uniformly since you'd just have to add:
| Event.KeyDown _ as ev -> KeyMap.update m.keys ev

and this would be enough for they KeyMap to update itself and send a custom event to your app that you can match in your update loop:
| Event.Custom My_event -> (* do thing *)

Custom events are just typed process messages so you defined them with type Messgae.t += | My_event.
i'll have another look in a minute when i get to a computer but overall is looking like we could merge this.
thanks again for contributing ✨

After giving this some more thought I'm not sure what the custom events buy us that the function approach wouldn't. 🤔
It seems to me that not knowing what the model is makes it hard to map from Minttea events to actual behaviors in an app. Maybe I'm missing something here tho!
For example, say you're building a vim-style mini editor. You have a rough keymap of hjkl that likely works the same everywhere all the time, but if you are in "insert mode", then hjkl mean something different.
With the function keymap I can write this:
type model = {
  document : string;
  cursor : Cursor.t;
  mode : [ `insert | `normal ];
  keys : Key_map.t;
}

let exit_mode key m =
  match (key, m.mode) with
  | Event.Escape, `insert -> ({ m with mode = `normal }, Command.Noop)
  | _ -> (m, Command.Noop)

let handle_input key m =
  match (m.mode, key) with
  | `insert, Event.Key char -> ({ m with document = document ^ char }, Command.Noop)
  | `normal, Event.Key ("j" | "k" | "l" | "h") -> move_cursor key m

let keymap =
  Key_map.
    [
      on [ Escape ] exit_mode;
      on [ Up; Down; Left; Right ] move_cursor;
      on [ AnyKey ] handle_input;
    ]

and then my update function becomes a tidy call to Key_map.update
let update ev m =
  match ev with
  | Event.KeyDown _ -> Key_map.update m.keys ev m
  | _ -> (* ... handle other events in my app *)

This approach makes one (i think) reasonable assumption that your keymaps will be dependant on the model – it gives you a lot of flexibility and its rather easy to think about: you associate a little update function to a set of keys.
When I try to do the same thing with the custom event approach it looked like this:
(* same model and other functions from the last example *(

type key_ev = | Exit_mode | Move_cursor | Handle_input

let keymap =
  Key_map.
    [
      on [ Escape ] Exit_mode;
      on [ Up; Down; Left; Right ] Move_cursor;
      on [ AnyKey ] Handle_input;
    ]

let update ev m =
  match ev with
  | Event.KeyDown k ->
     (match Key_map.find_match m.keys k with
      | Some Exit_mode -> exit_mode k m
      | Some Move_cursor -> move_cursor k m
      | Some Handle_input -> handle_input k m
      | None -> m)
  | _ -> (* ... handle other events in my app *)

Which seems to be repeating the dispatching code between the custom event (Exit_mode, Move_cursor, etc) and the functions that I want to trigger from them.
Maybe I'm missing something, so feel free to let me know if there's something about the custom events approach that I can't see here.
Do you see any drawbacks to the function approach? 🤔

Hi, just so you know, I am playing around with a toggleable help bar leave, and it requires some representation of key binding, so I am waiting on this to be completed before I submit the leave.

Hey  @leostera, sorry for dropping off the earth. I am getting married in about a month or so and it has eaten most of my time in last couple of months.
I think I see the misunderstanding we are hitting here. The way I see it there are two types of keybindings, there are bindings you use in your application code and bindings made by components you use (3rd party). In the case of your application code it totally makes sense to use a function based style. The solution I have been working is more useful for components that don't want to expose their internals to the application layer and don't want to handle key presses directly.
For example:
let's say I have a list component that can move up and down. It can export a variant that describes all the messages that can be passed to it. The keymap feature takes that variant and allows the user of that component to define what keys should trigger actions in that component. The result is that we can pass the main minttea message (which is a keypress) to the component and it will have a keymap to translate into other actions.
Here is a code example of how I thought this would be used
`ocaml
module ExampleComponent = struct
type msg = Upcase | Reset
type model = { keymap : msg Key_map.t; value : string }
let update event model =
match event with
| Event.KeyDown (key, _mod) -> (
match Key_map.find_match key model.keymap with
| Some Upcase ->
( { model with value = String.uppercase_ascii model.value },
Command.Noop )
| Some Reset -> ({ model with value = "hello world" }, Command.Noop)
| _ -> (model, Command.Noop))
| _ -> (model, Command.Noop)
let view model = model.value
end
type model = { example_component_model : ExampleComponent.model }
let initial_model =
({
example_component_model =
{
value = "hello world";
keymap =
(let open Key_map in
[
on
~help:{ key = "u"; desc = "upcase" }
[ Minttea.Event.Key "u" ] ExampleComponent.Upcase;
on
~help:{ key = "r"; desc = "reset text" }
[ Minttea.Event.Key "r" ] ExampleComponent.Reset;
]);
};
}
: model)
let init _ = Command.Noop
let update event model =
let ec_model, _ec_cmd =
ExampleComponent.update event model.example_component_model
in
({ example_component_model = ec_model }, Command.Noop)
let view model = ExampleComponent.view model.example_component_model
let app = Minttea.app ~init ~update ~view ()
let () = Minttea.start app ~initial_model
`
Does that make more sense?

Hi, @charleslambert I think you have a good point: having written a small component myself, I see the value of defining basic actions, but letting the user define the effective key presses.
I think if we don't have that, we will end up with ad hoc configuration ways for each component.
On the other hand, I agree with @leostera that the double abstraction may feel clunky on the user side.
Here is my two cent proposal.
Maybe we could devise instead two modules:

Action: designed for component interface, it would map custom events and help messages to internal functions.
Key_map: the user facing keymap handling, would just map key events to functions. It would be used by the user to make their own mappings directly, as well as triggering component actions.

The usage for the user could look something like that:
let keymap =
  Key_map.
    [
      on [ Escape ] exit_mode;  (* exit_mode is a callback *)
      on [ Left; Right ] move_cursor;
      on [ Up ] (action My_component.up);  (* action is a helper function in Key_map that emit the custom event *)
      on [ Down ] (action My_component.down);
      on [ AnyKey ] handle_input;
    ]

Compared to @leostera proposition, this one gives a straightforward way to configure component keymaps.
Compared to @charleslambert proposition, this one increases the level of indirection in the case of components, but it simplifies the user side.

Hey @Lattay, thank you for the input 😃. The only problem I see with that approach is that we would have to have all the mapped functions be imperative callbacks, as I don't think there would any way to generalize over their combined return types.

I guess you would have the functions be model -> model or 'something -> unit.

So the current implementation of bindings looks like:
type binding = {
  keys : Minttea.Event.key  list;
  help : help option;
  disabled : bool;
}

type 'a t = ('a * binding) list

We could add a variant type to account for the different types of (actions, functions)
type ('model, 'msg) binding_action =
  | Fn of ('model -> 'model)
  | Action of ('msg -> unit)

The problem with this type though is that it can't be generic for multiple msg variants. In other words you could not bind to msgs for more than one component in single keymap. It also requires that Action types be callbacks, which I don't think is ideal.


Binary arithmetic gates

Summary
Part of #13046: gate representations & plugins for the binary arithmetic operations (adder and multiplier).

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

282 of 296 (95.27%)  changed or added relevant lines in 17 files are covered.
1062 unchanged lines in 47 files lost coverage.
Overall coverage decreased (-0.7%) to 87.934%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/synthesis/arithmetic/adders/draper_qft_adder.py
25
26
96.15%

qiskit/synthesis/arithmetic/multipliers/hrs_cumulative_multiplier.py
24
26
92.31%

qiskit/synthesis/arithmetic/multipliers/rg_qft_multiplier.py
22
24
91.67%

qiskit/circuit/library/arithmetic/multipliers/multiplier.py
18
21
85.71%

qiskit/transpiler/passes/synthesis/hls_plugins.py
48
54
88.89%

Files with Coverage Reduction
New Missed Lines
%

qiskit/transpiler/passes/scheduling/alignments/pulse_gate_validation.py
1
96.3%

qiskit/transpiler/passes/calibration/pulse_gate.py
1
95.45%

qiskit/pulse/instructions/reference.py
1
93.55%

qiskit/pulse/instructions/directives.py
1
97.14%

crates/accelerate/src/unitary_synthesis.rs
1
92.24%

qiskit/pulse/instructions/play.py
1
97.5%

qiskit/providers/fake_provider/fake_pulse_backend.py
1
95.45%

qiskit/qpy/binary_io/circuits.py
1
91.74%

qiskit/pulse/library/pulse.py
2
83.33%

qiskit/pulse/calibration_entries.py
2
91.19%

Totals

Change from base Build<PHONE_NUMBER>2:
-0.7%

Covered Lines:
49097

Relevant Lines:
55834

💛  - Coveralls


Create Convays's life game.py

Please add this under Hactoberfest2024

Thanks for your contribution!

PR has been merged under Hacktoberfest!

Happy contribution!


Missing media player in Home Assistant

Hey guys,
Glad to see that issue 17 was closed and support for gen 2 products have been added. However, I just updated to the latest version and now the media player is missing for both my hatch devices (Hatch Rest Mini & Hatch Rest+ Gen 2) in Home Assistant. I'm on the current stable version of Home Assistant (2022.12.8)
Here's the error I found from the logs:
Logger: homeassistant.components.media_player
Source: custom_components/ha_hatch/media_player.py:32
Integration: Media player (documentation, issues)
First occurred: 12:17:07 PM (1 occurrences)
Last logged: 12:17:07 PM
Error while setting up ha_hatch platform for media_player
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 281, in _async_setup_platform
await asyncio.shield(task)
File "/config/custom_components/ha_hatch/media_player.py", line 32, in async_setup_entry
media_player_entities = list(map(lambda rest_device: RiotMediaEntity(rest_device) if isinstance(rest_device, RestIot) else RestMediaEntity(rest_device), rest_devices))
File "/config/custom_components/ha_hatch/media_player.py", line 32, in
media_player_entities = list(map(lambda rest_device: RiotMediaEntity(rest_device) if isinstance(rest_device, RestIot) else RestMediaEntity(rest_device), rest_devices))
TypeError: RestMediaEntity.init() missing 1 required positional argument: 'config_turn_on_media'

i have 3 % left on my battery, i pushed a fix that should fix the missing media player for rest plus first gen but will have to follow up when my battery is chargable

looks like it fixed my missing media player, thanks for reporting this so fast and if it is fixed for you too, i'll close this

Works for me now too. Thanks for the quick fix! You can close this.


Add virtiofsd log and fix qemu hang due to virtiofsd vq setup failure

When hugepage is not enabled, virtiofsd fails to setup vq with qemu and it causes qemu process to hang forever. waitSandbox only checks qmp so cannot detect such failure. Add virtiofsd debug log and stop qemu if virtiofsd quits.

/test

/test


[FEATURE]: Store user's coding language

Feature Summary
We need to store the user's coding language after they select one so that they can avoid having to take an extra step to select a coding language every time they'd like to play. Each time they change the language, the db will reflect that.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Will work on this

@mmcbride2929 Still working on this? Otherwise i'd be happy to pick it up.

@Littletonconnor shes all yours

This feature has been implemented. This issue can be marked as closed.


simplify deployment / release-management of echo

Fixes #35.
Background
Deploying echo is unnecessarily complex due to our custom Atmosphere packages.
What's in this PR?
To simplify our release management and deployment, we are consolidating all of our Rocket.Chat related code into our echo-chat repository. Our custom packages now live in the packages directory of our special lg-master branch.
This deprecates the rocketchat-lg-XXX repositories. There will be some follow-up work to:

update the READMEs of those repositories to mark them as deprecated
unpublish the Atmosphere packages
migrate the issues from those repos to this repository

Notes

no actual code was changed relative the master branches of each of the rocketchat-lg-XXX repositories
this obviates the need for the blasted HISTORY.md files (yay!)

I can't check this out and try it right now (kids owatching a show on my laptop) but I approve of the approach and am ok if you want to merge now. Especially since it will be off hours. If you'd like to wait for me to test it locally I can probably get to that later tonight.

trying locally now

LGTM. Works on my machine


Adjust header and footer for text resizing

Feature Request
Description
Adjust header so that text can be resized to 200% without losing functionality.
implementation details
some ideas for how this can be done:

reduce overall header size (a height of 5.25rem might be enough?) and give it a max-height to give more space to the main page.
give the Spy Logic logo a max-width
reduce some of the gaps or padding
allow the buttons to wrap their text (and removed 100% width of the left/middle/right containers so the buttons don't have to wrap if there is enough space)

Screenshots
Roughly expected outcome at 100% (no zoom)

Roughly expected outcome at 200%

Additional context
To fulfil WCAG 1.4.4 Resize Text.  https://www.w3.org/WAI/WCAG21/Understanding/resize-text.html
Acceptance criteria
GIVEN the user uses the app in the full screen browser on a 13" laptop
WHEN the user zooms to 200%
THEN all of the buttons and links of the header and footer are still visible.

Hi @dhinrichs-scottlogic, I believe I can help with this issue. Would you mind assigning it to me?

Hi @dhinrichs-scottlogic, I believe I can help with this issue. Would you mind assigning it to me?

Yes, go ahead, I've assigned this issue to you. Thank you so much for contributing!

@dhinrichs-scottlogic I think I may be missing something here. When I edit any files when running the application, the changes are not shown at all. This goes for the css files as well. The two files in question are the 'MainHeader.tsx' and its CSS counterpart, 'MainHeader.css'.
 Any assistance would be helpful!

@dhinrichs-scottlogic I think I may be missing something here. When I edit any files when running the application, the changes are not shown at all. This goes for the css files as well. The two files in question are the 'MainHeader.tsx' and its CSS counterpart, 'MainHeader.css'.
 Any assistance would be helpful!

Hi Kenny, hmm that's a tricky one 🤔
How are you running the application at the moment? Are you using the VS Code launch file or just through a terminal?
Are you able to see any changes if you completely restart the application?
One thing that's caught me out in the past is that the backend and frontend are run separately, so I'd make sure that the frontend is running. Backend can be running too but it shouldn't be required for this ticket.

I am starting them up separately, the backend and then the frontend.
When you are working on it, do the changes occur immediately? How are you running it so that it works when you make changes?

I am starting them up separately, the backend and then the frontend.
When you are working on it, do the changes occur immediately? How are you running it so that it works when you make changes?

Yes, changes are showing after I save them.
I'm personally running it through VS Code, but I've also just checked and it still works if the backend and frontend are started from a terminal.
Are you seeing any errors at all?

I am starting them up separately, the backend and then the frontend.
When you are working on it, do the changes occur immediately? How are you running it so that it works when you make changes?

Yes, changes are showing after I save them. I'm personally running it through VS Code, but I've also just checked and it still works if the backend and frontend are started from a terminal. Are you seeing any errors at all?

Ahh I got it working. Here's what I did differently:
I ran only the frontend using 'npm run dev' instead of 'npm run start:ui'. I did not start the backend (since I am only working with css). Now this allows live changes to be shown. Thanks for the help!

I am starting them up separately, the backend and then the frontend.
When you are working on it, do the changes occur immediately? How are you running it so that it works when you make changes?

Yes, changes are showing after I save them. I'm personally running it through VS Code, but I've also just checked and it still works if the backend and frontend are started from a terminal. Are you seeing any errors at all?

Ahh I got it working. Here's what I did differently:
I ran only the frontend using 'npm run dev' instead of 'npm run start:ui'. I did not start the backend (since I am only working with css). Now this allows live changes to be shown. Thanks for the help!

Great to hear!

@dhinrichs-scottlogic We are likely to run into problems with the header when zooming, as there are a lot of components in there. At some level we might want to move the function buttons (reset, getting started, handbook) into a dropdown menu, remove the button icons, etc, using media queries as @Kenny4297 suggests.
I think this requires some more thought before implementing anything.

I had a quick go locally to see what css rules we'd need to change. Below is what I changed, which are prime candidates for a media breakpoint. Note that I've only downsized two fonts and then not by much; most changes are adjustments to spacing, while not making things too compacted.
.themed-button > img {
  display: none;
}

.main-header {
  height: 3.5rem;
  padding: 0 0.5rem;
  gap: 0.5rem;
}
.main-header .main-header-middle {
  gap: 0.5rem;
}
.main-header .main-header-middle .main-header-level {
  font-size: 1.5rem;
}
.main-header .themed-button.level-button {
  height: 2.5rem;
  padding: 0.5rem;
}
.main-header .main-header-right {
  gap: 0.5rem;
}

button.level-mission-info-banner {
  padding: 0 0.5rem;
  border-width: 0.5rem 0.25rem;
}
span.level-title-area {
  font-size: 1.25rem;
  white-space: nowrap;
  min-width: unset;
}

.chat-box .chat-box-feed {
  justify-content: stretch;
  /* REMOVE height: fit-content */
}
.chat-box .footer {
  padding: 1rem;
  gap: 0.5rem;
  /* REMOVE font-size: 0.75rem */
}
.chat-box .footer .control-buttons {
  gap: 0.5rem;
}
.chat-box .footer .messages textarea {
  padding: 0.5rem;
}

.main-footer {
  height: 2rem;
}
.main-footer a svg {
  height: 1.75rem;
}
.main-footer a svg,
.main-footer .links,
.main-footer .links a:not(:last-child)::after {
  padding: 0 0.5rem;
}

Result:

I had a quick go locally to see what css rules we'd need to change. Below is what I changed, which are prime candidates for a media breakpoint. Note that I've only downsized two fonts and then not by much; most changes are adjustments to spacing, while not making things too compacted.
.themed-button > img {
  display: none;
}

.main-header {
  height: 3.5rem;
  padding: 0 0.5rem;
  gap: 0.5rem;
}
.main-header .main-header-middle {
  gap: 0.5rem;
}
.main-header .main-header-middle .main-header-level {
  font-size: 1.5rem;
}
.main-header .themed-button.level-button {
  height: 2.5rem;
  padding: 0.5rem;
}
.main-header .main-header-right {
  gap: 0.5rem;
}

button.level-mission-info-banner {
  padding: 0 0.5rem;
  border-width: 0.5rem 0.25rem;
}
span.level-title-area {
  font-size: 1.25rem;
  white-space: nowrap;
  min-width: unset;
}

.chat-box .chat-box-feed {
  justify-content: stretch;
  /* REMOVE height: fit-content */
}
.chat-box .footer {
  padding: 1rem;
  gap: 0.5rem;
  /* REMOVE font-size: 0.75rem */
}
.chat-box .footer .control-buttons {
  gap: 0.5rem;
}
.chat-box .footer .messages textarea {
  padding: 0.5rem;
}

.main-footer {
  height: 2rem;
}
.main-footer a svg {
  height: 1.75rem;
}
.main-footer a svg,
.main-footer .links,
.main-footer .links a:not(:last-child)::after {
  padding: 0 0.5rem;
}

Result:


regenerate BPF binaries after PR #1294

Regenerates the BPF compiled binaries after this contribution: https://github.com/grafana/beyla/pull/1294

Merging without waiting for an approval, as some users started to report that beyla is failing in main image tag.


sim pass: support for memories

Included changes are here to support two different ways of representing memory locations in VCD/FST file, one is memory_cell<hex_address> and other memory_cell[decimal_addres], in both cases we create special memory structure with mappings of specific addresses to data inputs in FST (so mem cells can use that directly) and also add mapping to memory_cell[decimal_addres] as wire, since memory pass can convert mem cells to registers.
Also including change in btor writer to put aliases for main clock signal in info file, that then generates proper VCD output created by btorsim.

@nakengelhardt Can you please confirm if btor info file should contain aliased clock lines, or that should be fixed on btorsim side.


Whiltelisting logic should be fixed.

Ref. : https://medium.com/@ItsCuzzo/using-merkle-trees-for-nft-whitelists-523b58ada3f9
Don't use for loop for whitelisting.
It is not efficient, and when the number of whilelist increases, the for loop must be reverted.

The answer is using merkletree for whiltelisting.

Yes, I agree that a merkle tree is the most efficient solution, but that's a relatively more complex implementation.

Not that complex. Let me implement it in 20 minutes.

Anyone can create a collection, so if Merkle data can be stored and served from somewhere, it seems a good option.
The key is where to put the off-chain data.

MerkleRootHash can be a on-chain data.

I mean, the branch data that makes up the rootHash. Normally, only root hash doesn't work.

https://github.com/Steemhunt/dixel-v2-contract/pull/5/commits/cd118ecea25afebb16a3b72858af022b17380619

컨트랙은 이거 쓰면 되고, 외부에서 MerkleHashRoot 만들어서 넣으면 됩니다. 저는 여기까지.......

Related to #2 #6 #8
Oh using Merkle Tree could be the most efficient way of checking whitelist!
However, we probably have to think about where to store actual address data because we want to give the owner a nice interface so they can

Check all whitelisted addresses
Add / remove addresses from the list

We'd like to avoid maintaining an external db or indices, so I guess I'll have to think about it more tomorrow.
Thank you so much guys for all thoughts and feedbacks here.

You don't have to store the white list.
You don't have to maintain an external db or indices.
You can throw away the rest if you just get Merkle Root Hash.

You don't have to store the white list.
You don't have to maintain an external db or indices.
You can throw away the rest if you just get Merkle Root Hash.

Apart from verifying whether the address is in whitelist or not, can we restore the entire whitelist from Merkle root hash?
Because otherwise the owner would not be able to add or remove from the list it if they don't keep the whole list themselves.
Also, to verify if the whitelisted address already claimed the nft or not, each claim process need to save an extra claimedAddress data on the contract - isn't it?


Add hand raise, pin presentation

Add support for raising your hand and pinning a presentation.
I noticed the toggling logic is tied to muting, so I ended up continuing that metaphor to muting any arbitrary aria-label.

Hey @gototen, I'm not sure what happened, but this PR looks to be only changes from willnorris's fork that have already been squash-merged into this repo (https://github.com/ChrisRegado/streamdeck-googlemeet/pull/1). I don't see the code for hand raising or pinning in here.

Yeah something got messed up.  I will create a new fork and new PR.


fix: Restore doesn't work when backup file is renamed

When backups are renamed from their original generated name, restore operation can't process that backup anymore. This PR should fix that.
Automation
/test sanity
:mag: Cypress test results

[!WARNING]
Tests have not run on the HEAD 61c893aadd46b38dd4b90626ec18ea8e73a4900f yet
Mon, 25 Nov 2024 05:14:57 UTC

Communication
Should the DevRel and Marketing teams inform users about this change?

[ ] Yes
[x] No

/build-deploy-preview skip-tests=true

@coderabbitai review


add flush api to default stdout/stderr logger

motivation: default logger may not flush the streams on crush, so need to make flushing moe explicit
changes:

add a constructor argument to control flushing behaviour, default to "everytime"
add a "flush" method to allow user defined flush timing. this api is now internal but we could expose it if there is a need
add tests

solved #86

@weissi addressed


Corrected typo.

Corrected README. Environment was spelled incorrectly.

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! :smile:

@naunga I know it's a minor typo, but can you please follow the CLA link above and accept the term. Thanks.

I already did. 🙂
On Tue, Nov 21, 2017 at 12:22 PM Henry Avetisyan<EMAIL_ADDRESS>wrote:

@naunga https://github.com/naunga I know it's a minor typo, but can you
please follow the CLA link above and accept the term. Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yahoo/athenz/pull/297#issuecomment-346098997, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACOeMmDKYkYxMU9dDY6gNpMRiHlfpA9_ks5s4wbXgaJpZM4QGEDS
.

👍


Add a high resolution __time on the global object

Calling
var n = __time();

Will get the current time in milliseconds but it is accuracy goes beyond millis. About 3 digit micro secs. are stored as fraction. Date.now() truncates to milis and is about 6 times slower.

run ci

run ci

run-ci

run ci


Update Hello World.cpp

Hello World program in c++.

See https://github.com/leachim6/hello-world/pull/774 and https://stackoverflow.com/questions/1452721/why-is-using-namespace-std-considered-bad-practice


Updated for Prooph/Common v4

Checks fail b/c prooph common v4 supports only php 7.1

thx @mkherlakian
Can you change travis configl to use that matrix instead. This should solve the problem with PHP version.

Hey @codeliner I updated the Travis config with your suggested matrix, but also had to add react/promise as a dependency to make tests pass - otherwise one of the tests threw a TypeError as it uses a deferred promise. Should be good to go.
I had refrained from updating as I wasn't sure that you wanted to drop support for php 5.X, but I guess since prooph/common v4 uses php7 only features, it makes sense. Cheers!

thx @mkherlakian awesome work!


cpu affinity - isolcpus

First of all high respect and thank you for Jens Axboe for his exceptional input in the linux kernel.
We have isolated most cpus for application usage with isolcpus kernel boot option
but io_uring does not respect this kernel option - io_uring workload spreads over all cpus.
Is it possible to get io_uring to respect the isolcpus kernel boot option or set cpu affinity for the worker pool?
Based current understanding io_uring_params.sq_thread_cpu only affects the poll thread,
but has no effect on all other workers. Is this right?

Moved to discussions


Update ubi-minimal base image to ubi-minimal:8.5-243

Change Overview
Update ubi-minimal base image to ubi-minimal:8.5-243

Looks like the automation opened one too. We can merge that 👍🏼


fix: add error handling to jsonReviews

This commit enhances error handling in the fetchReviews function in fetch-data.ts. It now catches errors and logs them to the console. It also returns a Promise even when reviewsJson[id] is available.

Preview environment images deployed!
The preview environment has been updated:
client image tag: preview-1019-c62a7298421496ae77854789fd9ad4e1e05f0660-20230810.0809
storybook image tag: preview-1019-c62a7298421496ae77854789fd9ad4e1e05f0660-20230810.0809
Client URL: https://pr-1019--epp.elifesciences.org
Storybook URL: https://pr-1019--epp-storybook.elifesciences.org
Note: Updated images can take a few minutes to roll out.

Preview environment created!
The preview environment has been deployed to the cluster.
Client URL: https://pr-1019--epp.elifesciences.org
Storybook URL: https://pr-1019--epp-storybook.elifesciences.org
Note: If this is a new environment, it can take a few minutes for DNS to update.


Scrollbar in grid layout is not correct

I found the easiest to reproduce is with this integration test:
test.only('editor, assign and check view columns', async () => {
	const doc = await workspace.openTextDocument(join(workspace.rootPath || '', './far.js'));
	const p1 = window.showTextDocument(doc, ViewColumn.Nine).then(editor => {
		assert.strictEqual(editor.viewColumn, ViewColumn.Nine);
	});

	await new Promise(resolve => { });

	await p1;
});

This correctly opens a group at the ninth column but the scrollbar is still showing as if it was not scrolled even though it is.

Very likely an issue in vs/base/common/scrollable et al.

@joaomoreno Just submitted a PR.

I see the scrollbar on the right, but it also looks like there is a second shorter scrollbar behind it


[Feature Request] Pi-Hole version

Hi
This Project looks amazing, however im running pi-hole as my main Adblocker, any way to get a pihole compatible version?

+1

Currently, no there isn't a Pi Hole version. I will consider making one, but I'm not sure of a time frame.
But in the meantime Pi-Hole does ship with their own alternative: pi-hole/PADD


Toggling some options causes display driver to crash in Chess Titans

Describe the bug

Toggling some options -that generally have significant impact on gpu- causes the game to crash.
bridge-remix: 06d10f5 and dxvk-remix: 49711d9
RTX 2060 Desktop
536.67
Chess Titans, bundled with Windows 7 (patched)

Attach logs!
NvRemixBridge.log
d3d9.log
chess_d3d9.log
To Reproduce
Steps to reproduce the behavior:

Launch the game
Enable enhanced assets on a scene where majority of materials will be replaced. (This always causes the issue in my testing)
or
Toggle Reflex or DLSS on and off (This doesn't always cause the issue, rather majority of the time. I think it has a higher chance when performance profiler enabled)

Expected behavior
The game will continue to function normally
Actual behavior
The display driver crashes. Event viewer log:
\Device\Video3
Error occurred on GPUID: 900

source: nvlddmkm
Screenshots
Video showcase:
https://github.com/NVIDIAGameWorks/rtx-remix/assets/84380947/da3f4784-d720-43f3-9365-82e0fa67425e
During display driver crash (the performance graph glitched):

IMGUI briefly shows this during transitions

This initially happened on my system, which is the following:
RTX 3090
537.42
The bug can be consistently reproduced across these two systems. Also, upon launching the game, it can crash the Nvidia driver. This happens (on my system) 2 out of 3 times. Looking in event viewer makes the crash evident, although the screen does not flicker black as would be typical with a driver crash.

https://file.io/N3HkiPKwZiAd
Here are the apitraces of the game with and without dxwrapper (dxwrapper was used for borderless fullscreen, not for api convertion)

REMIX-2158 for tracking.

Resolved as of https://github.com/NVIDIAGameWorks/dxvk-remix/commit/7fcb5e3e5229316ac5539b9b792d718cbe6aa3b1 in response to the issue https://github.com/NVIDIAGameWorks/rtx-remix/issues/261


Error 0x8008005 Server Execution Failed

My WSL2 was working fine.. I hadn't opened my laptop couple of days, Fired up my kali and This error started..
In PowerShell "WSL -l -v" takes forever to output a reply and manually starting kali.exe or even the app icon of Kali-Linux gives a response saying Installing and then the error is printed.

In some closed thread I read they were unable to install. This is a large image of Kali-Linux with KEX for graphical interface whioch was working smooth till 3 days ago.

After 1 Week I came back, I was out of my house for a week and now I launched Kali and It launched normally, I think windows update fixed this issue automatically, Today I launched Kali-Linux and I was greeted by my default Terminal screen

Sorry I couldn't contribute much to the solution guys


Fix lmdb test dir cleanup

This fixes the issue that the TempDir object in the test is destroyed early, which leads to lack of cleanup.

Should check whether sqlite backend has the same issue

Should check whether sqlite backend has the same issue

into_path() only exists in this file.


⚠ GTru has degraded performance

In 0505368, GTru ($GTRU) experienced degraded performance:

HTTP code: 521
Response time: 1471 ms

Resolved: GTru performance has improved in b601261 after 26 minutes.


Modifier, Prop, and HtmlAttr are Contravariant

Following from https://github.com/armanbilge/calico/pull/168#issuecomment-1405597149. In this case it was more natural because the types already has the appropriate type parameters.
I finally had to kill Codec since codecs are not contravariant.

this makes clear we currently don't even try to understand html not produced by calico

is that a good thing? 😅 I think we should open an issue/discussion about this topic, since it keeps coming up :)


initial 5.0 commit

Version.props changes.
updating most packages 5.0.0-preview2 currently.
minor changes to upgrade obsolete EFCore code.

commits are wrong. Will do a rebase and open.


How create an ISO from a URL?

Hi friend, I'm using the project to handle my Vultr account and it's going pretty well, however, how could I create an ISO image from a URL? That is, how can I use the "/v1/iso/create_from_url" API?

Hmmm... Incredible that I missed that API call!
It's not implemented in this client so I'm going to add it ASAP and will let you know when it's ready!

Added support for this call. If you could confirm it works, I'll add a new release tag...

This looks amazing and works great.


feat: updated the study group and servicemap api schemas

Updated the schema with minimal changes: Unit Id and new unit information should be used in UI.
PT-1264 PT-1291.

Codecov Report

Merging #169 (b9df7f9) into master (91878de) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #169   +/-   ##
=======================================
  Coverage   85.48%   85.48%
=======================================
  Files         103      103
  Lines        2700     2700
  Branches      582      582
=======================================
  Hits         2308     2308
  Misses        387      387
  Partials        5        5

Impacted Files
Coverage Δ

...c/domain/enrolment/enrolmentForm/EnrolmentForm.tsx
100.00% <ø> (ø)

...omain/enrolment/enrolmentForm/ValidationSchema.tsx
100.00% <ø> (ø)

Continue to review full report at Codecov.

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

Hmm. it seems like not all the changes were included in this PR. If the codegen did not bring all the schema updates, they sure are included in this complementary pull request #171 .

Closed because didn't include all the changes. The content of this PR is included in #171 anyway


Fix build docker image script issues

The build script does not parse positional arguments and have redudant variable declarations. This PR fix this problems.

Besides that, the first positional argument still conflicts with the parameter --device. We must decide which one we want to keep.

LGTM


SELECT foo PARTITION (p1) syntax

Background: https://airtable.com/tblD3oZPLJgGhCmch/viwINAgXA5on9pr2T/recQCnBLAS5iGUoy3
PRs:

Note from @benesch:

users will be able to SELECT ... FROM foo PARTITION bar to limit their query to a partition of a table. The exact details are still being worked out—i.e., you will probably be able to do the same thing on DELETE, UPDATE, and maybe even INSERT.

@Amruta-Ranade, looks like this has been moved to 2.1.


Handle create alias error

This PR handle create alias error when deployment is not ready.

Codecov Report

Merging #2514 into canary will increase coverage by <.01%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           canary    #2514      +/-   ##
==========================================
+ Coverage   11.96%   11.96%   +<.01%
==========================================
  Files         257      257
  Lines        9310     9318       +8
  Branches     1047     1049       +2
==========================================
+ Hits         1114     1115       +1
- Misses       8076     8083       +7
  Partials      120      120

Impacted Files
Coverage Δ

src/commands/alias/set.ts
0% <ø> (ø)
:arrow_up:

src/util/alias/create-alias.ts
0% <ø> (ø)
:arrow_up:

src/util/errors-ts.ts
45.83% <33.33%> (-0.23%)
:arrow_down:

Continue to review full report at Codecov.

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

@sophearak we need a link associated with that deprecation message


What design pattern is used by phy? ECS mode? Component mode?

What design patterns were used?

Is it decoupled from three. js? Can other rendering engines be used?

hi each physics engine use they own pattern so is possible to use other rendering engine for phy side


internal/function: implement ref_name functions

Closes #9
Closes #8
Adds the is_tag and is_remote functions along with a function to register them on a catalog.
Perhaps on the future we should move the logic of initializing the sqle engine to the gitquery package where you just pass a path to  a New function and it returns everything setup.

rebase please


Easier way to translate?

I love translating to my native language and have translated many applications that I use often. And Lubelogger is the most painful so far to translate. I need to sync repo, prettify english JSON, sort by alphanumeric and then compare with my old Slovenian translation.
And that is exactly what I have done just now, but I can see many strings that were before in en_GB.json and are no longer there and it is just a big mess.
Can I please ask for you to move translations to web pages like crowdin or weblate? That integrate with github to pull translated files and I am sure json as a translation file is fully supported. That would make things much much easier.
Thank you,

You're going to like the next release,
In the next release, root users will have access to a built in translation editor, you can modify the text and then save the translation without exiting the app.
To contribute to the git repo, you can export the translation file
See it in action:

This looks great. Will wait for next release then. Thank you.

This is now released, if you encounter an error, it's because of a very small bug detailed with workaround here
But shouldn't be an issue since you'd already have translations uploaded previously.
There is also a feature not shown in my original gif that I uploaded, where it will allow you to download translations.

This is now released, if you encounter an error, it's because of a very small bug detailed with workaround here
But shouldn't be an issue since you'd already have translations uploaded previously.
There is also a feature not shown in my original gif that I uploaded, where it will allow you to download translations.

Thank you very much! Already made a PR with updated translation.


Preserve non alphanumerical characters

I love this package, but when I updated from 3.1.0 to 4.1.0, I realized the behavior. It's to remove non alphanumerical characters like the following.
changeCase.camelCase("hello hello"); // "helloHello"
changeCase.camelCase("hello hello!"); // "helloHello"
changeCase.camelCase("!!!"); // ""

As release notes said, the latest version does not support unicode conversion so I think it's expected, but if our users use English in our projects which handle i18n, it should be converted any cases such as title case. Do you have a plan to re-implement to preserve them such as using option?

@jagaapple This hasn't changed since 3.1.0 and all your examples work the same on 3.1.0, I can't replicate any difference.

@blakeembrey Ahh, I just realized it's multibyte characters correctly like the following.
// 3.1.0
changeCase.camelCase("あああ"); // "あああ"

// 4.1.0
changeCase.camelCase("あああ"); // ""

And do you have a plan to implement to preserve non-alphanumerical and multibyte characters when changing?

@jagaapple Would adding it as a configuration option work for you? The reason it isn't supported by default anymore is because it added something like 26KB to browsers. See<EMAIL_ADDRESS>for the changes.

@blakeembrey hmm... if I can get solutions, I'll create pull request! Thank you😃

@jagaapple Probably re-exposing a RegExp built from XRegExp - that's what the previous versions did. We could put it in a separate package by default and you'd just need to import and add to the options.


October 22nd, 2024 - Regulation Innovation SIG Meeting Minutes

Regulation Innovation SIG
Date
October 22nd, 2024 - 12pm EST / 5pm BST
Untracked attendees

Fullname, Affiliation, (optional) GitHub username
...

Meeting notices

FINOS Project leads are responsible for observing the FINOS guidelines for running project meetings. Project maintainers can find additional resources in the FINOS Maintainers Cheatsheet.

All participants in FINOS project meetings are subject to the LF Antitrust Policy, the FINOS Community Code of Conduct and all other FINOS policies.

FINOS meetings involve participation by industry competitors, and it is the intention of FINOS and the Linux Foundation to conduct all of its activities in accordance with applicable antitrust and competition laws. It is therefore extremely important that attendees adhere to meeting agendas, and be aware of, and not participate in, any activities that are prohibited under applicable US state, federal or foreign antitrust and competition laws. Please contact<EMAIL_ADDRESS>with any questions.

FINOS project meetings may be recorded for use solely by the FINOS team for administration purposes. In very limited instances, and with explicit approval, recordings may be made more widely available.

Agenda

[x] Convene & roll call (5mins)
[x] Display FINOS Antitrust Policy summary slide
[x] Review Meeting Notices (see above)
[ ] Approve past meeting minutes

[ ] https://github.com/finos/open-regtech-sig/issues/74

[ ] Update on Proposed Rule: Public Comments, Compliance, and Data Modernization - Ashley Nelle-Davis, Data Foundation
[ ] Revisiting the topic of a digital solution for SSI exchange - Ffion Acland, Global Co-Lead Of Data Models & Governance, Goldman Sachs
[ ] General Discussion: Next steps with ORR - Ian Sloyan, Managing Director South Cardinal
[ ] AOB, Q&A & Adjourn (5mins)

Meeting Minutes
Action Items
Join Zoom Meeting
https://zoom.us/j/95819415471?pwd=VmxmeExMcXJFZWs4OGtnTXBiWnVHdz09

Meeting ID: 958 1941 5471
Passcode: 915672
Find your local number: https://zoom.us/u/acoFmoLAFE

Eteri / FINOS

Ashley / Data Foundation

PJ Di Giammarino/ JWG

Jane / FINOS

Stephen Goldbaum @ Morgan Stanley

Ian Sloyan - FINOS/South Cardinal advisors

Tom Schmertz - Synechron

Stephen Sui - RBC


add bsize keyword tests v2

This adds signature tests for the various bsize keyword values.
Signed-off-by: jason taylor<EMAIL_ADDRESS>

cc @jlucovsky as you work on bsize in some PRs, right ?

cf #238

continued in #922


Replace google analytics

#691

coverage: 33.231% (-0.02%) from 33.246% when pulling 6f15ce83fbb399993eb0ba051f62bc474c7fc87c on ginger/analytics into ba9ec4cd731f806e69f1bd865096c388180b2364 on dev.


Docker images not created for "<IP_ADDRESS>363"

Hi, thank you a lot for the NXWitness Docker images. I noted that for the latest stable release "<IP_ADDRESS>363" of 11 March 2024 no Docker images are created. Again thanks for your work.

Hi, thank you a lot for the NXWitness Docker images. I noted that for the latest stable release "<IP_ADDRESS>363" of 11 March 2024 no Docker images are created. Again thanks for your work on these.

5.1.3 has not been released, release_date is null and release_delivery_days is null.

I saw that the release_date was missing, guess that was the reason for the build not to happen. Must be an oversight of NXW, version is released however as can be seen at [https://updates.networkoptix.com/default/#note_<IP_ADDRESS>363].


Fix too many redirect

Fixes #1429
/cc @argeiger
Signed-off-by: Emile Vauge<EMAIL_ADDRESS>

:shipit:
LGTM


Make "Add property" button optional

It would be great to have an option to make displaying the Add property button optional. In our application we want to control all user inputs via the Profile and don't want to support adding random properties via "Add".
I imagine something like this:
        <describo-crate-builder
            :enable-add-property="..."
        />

I can provide a PR for this if you agree.

If you set definition to "override" in each of your defined classes then that control gets disabled for that class.
The reason for not making it a component wide configuration param is so that any classes not defined in the profile at least get the base properties from Thing (https://schema.org/Thing) like name and description. If that control is disabled and there is no definition for a class in the crate (that may have come in from somewhere else) then even name and description won't be available.

Thanks, Marco!
I already have "definition": "override" for my classes in the profiles,, which makes the button disabled all the time.
This, however, causes confusion for our users why that button is disabled and when gets it enabled and usable. That's why it would be great to be able to allow not do display this button at all. As in our application we are full control of the profile and the type of data our users can enter via our profiles they won't need/be able to add any properties in an ad-hoc manner.
So I think it would be safe in this case to not have the "Add property" button at all.

Ahh ok - make sense. Let's discuss this next week as well (implementation) but it sounds good to me!

This is dealt with by #66 in the AROMA case where they will actually disable the whole controls bar. If this becomes an user for users that want to keep controls but just disabled the add button then we can revisit.


[hmac/rtl] Save and Restore digest mismatch when SHA digest update is too close to a block boundary

Description
While testing the Save and Restore, I fell on the case described in this temporary PR.
hash_done is triggered but the last SHA digest update is because of the round_q threshold has been reached just before and not because the block has been processed (24 bits are missing to reach 1024).
I have done a fix, this has been reviewed with @andreaskurth .
FYI : @gdessouky

Hi @martin-velay,
Could you specify the exact sizes for the test vector, so that I can also try to run it through the driver/top-level. What I gathered from the referred waveform is that you are running SHA256 operation with message_len = 1000 bits, is that correct?

Hi @ballifatih,
message_len = 1024 bits
digest_size = SHA2_256
key_length = 256


When I request an OTP, he calls me by phone and does not send the code

When I request an OTP, he calls me by phone and does not send the code

What you are saying ?
You are not able to recieve an otp

yeah,
i recivecd call not msg

What is saying in the call ?
Which  country you belong to ?
Which operating system are you using ?

nothing
egypt
windows 11

Will you try again one more time by installing again
~$ npm install -g truecallerjs

Here is my new update<EMAIL_ADDRESS>Now you can login with json file.

On Truecaller App, tap on the 3 line menu on top left.
Tap on Privacy Center and then click on Download my data.
Now a json file is downloaded.
Save the json file on your PC at any location e.g /home/HP/1234567890-99123456789.json
On you terminal or command prompt enter below command.

~$ truecallerjs login /home/HP/1234567890-99123456789.json

Now you are successfully logged in.


Add future::delay

close: https://github.com/async-rs/async-std/issues/341

I fixed it and rebase master branch.
Can you review again?

Thanks!


add "fill_missing_values" process

What

adds a wrapper for gdalcubes fill_time() function to openeocubes
this also implements the new code layout suggested in #91
this function fills missing values in a datacube by various interpolation methods provided by gdalcubes fill_time() function

Why

it extends the coverage of gdalcubes functions, that are wrapped by openeocubes
filling missing values can be useful in the context of machine-learning, as it can increase the quality of the data contained in a datacube.

I have modified this with #98  hence closing it.


[Bug] Offline Support For Realtime DB does not work with Unity Android SDK

Description
ISSUE
I am trying to enable offline capabilities for Realtime DB on Android, but it does not seem to work. The offline capability works fine on Editor and even though Android seems to have the feature, it doesn't work on it.
Code Used
FirebaseApp firebaseApp = Firebase.FirebaseApp.Create(options, Guid.NewGuid().ToString());
_realtimeDbInstance = FirebaseDatabase.GetInstance(firebaseApp);
_realtimeDbInstance.SetPersistenceEnabled(true);

Reproducing the issue
On Editor:
Launch the game, turn off internet, change player configs, close the game, relaunch after enabling internet. The updated configs show up
On Android:
Repeat the same steps, but after relaunch, the old configs show up , since they were never written to the firebase realtime db
Firebase Unity SDK Version
12.0.0
Unity editor version
2022.3.22f1
Installation Method
.unitypackage
Problematic Firebase Component(s)
Database
Other Firebase Component(s) in use
Analytics, Authentication, Crashlytics, Firestore, Remote Config
Additional SDKs you are using
UnityIAP, Mirror, PlayGames
Targeted Platform(s)
Apple Platforms, Android
Unity editor platform
Mac, Windows
Scripting Runtime
IL2CPP
Release Distribution Type
Pre-built SDK from https://firebase.google.com/download/unity
Relevant Log Output
No response
If using CocoaPods for Apple platforms, the project's Podfile.lock
No response

Thanks for reporting, @saifullah73. I was able to reproduce the behavior. It looks like the Unity Editor, and the native Firebase Android Quickstart works properly. I'll go ahead and inform our engineers. Thanks!


Make more ducumentation how to use cdm_reader_mapper

@lizkent: After reading your last e-mail, I think I have understood your problem. First of all, thank you for your feedback. For me, the code is understandable as I am working on it very intensively. Therefore, feedback from a user helps me a lot to improve the module. An outside perspective can sometimes be very helpful :wink:.
You will find the badge docs In the table at the beginning of the README. Just click on it to get to the documentation.
I have changed your suggestions in the documentation.

dding the new NOC logo
adding a link to the documentation in the README
update the link to the GitHub repository

You can find the new documentation in my PR branch.
Do you think the usage of cdm_reader_mapper is easier to understand now?
I look forward to any feedback. Please feel free to contact me. Do you have  any other suggestions/comments/questions?

Hi @lizkent, until end of this week I want to merge this PR. Do you have any further suggestions for improvement?


Node status showing blank on operators rewards tab

Describe the bug
Node (EPCAP7UsgEF8mxuzPWMt2TvC8B8CR4XaYrobwMuP1bjk) is running since start of testnet with > 93% uptime and is currently running now although with timeout sync error.
In the node operator rewards tab the status is showing blank which is probably not expected since its tagged to my discord, and the wallet has staked 500+ shdw with a healthy uptime.

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
With the testnet currently facing downtime, and given the node is still running, ideally the status should be waiting. Not sure what a blank status means.
Screenshots
Added screenshot
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Thanks for reporting this! This is simply a frontend UI bug. We'll get on resolving this.


Track progress isn't in sync when using Player Groups (Slimproto)

What version of Music Assistant has the issue?
2.3.0b33
What version of the Home Assistant Integration have you got installed?
2024.10.0
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
Using the newest beta with the refactored player groups, when playing a song to a group that contains my Slimproto devices, the track progress isn't in sync. You need to pause and play the song for the track progress to correct itself. In some cases the track progress is way past the duration of the song, almost as if the counter continues even when music is paused / off.
How to reproduce

Have a Player Group of Slimproto devices (I do not have any other devices types to try to see if this is also problematic there)
Play a song
Play the next track
Observe the progress continues to count instead of resetting
Pause the song, see the track progress corrects itself

https://github.com/user-attachments/assets/ac3641d1-ab3a-4308-9b73-469479785539
Music Providers
Spotify (don't use any others)
Player Providers
Player Group - Slimproto Devices (don't use any other types)
Full log output
music-assistant.log
Additional information
No response
What version of Home Assistant Core are your running
2024.10.3
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

Was testing the theory that the track progress continues even if the song is paused and it seems possible. I had a song paused, waited a bit, pressed play and MA thinks the next track is playing but it's the same song that was paused (playing from the position it was paused at).

Is this the same as #2852

It's similar but not the same, the previous issue was that there was no progress tracking at all. It does now sort of work, just the wrong progress being tracked.

Unfortunately I can't seem to get your video to play and I have just joined two picoreplayers together using the manual sync method and it is working fine. How are you grouping the players?

These were existing grouped players. I've just tried creating a new group player (Settings -> Players -> Add Group Player -> Slimproto as Group Type) with two picore players (that's what I use too) and it's the same issue. That being said, I've done what I think you're doing which is ticking the box next to another player when playing on one player and that seems to work fine - the progress is in sync so it seems when using an actual Group Player that's where the sync issue comes in

Yes I can reproduce this. It will get looked at soon

Please check this with 2.3.0rc2

can you retry with rc3 ?

We will close this soon due to no follow up

Sorry for the delay, I think this is good to close as it works on rc3 much better.


Discord - notification icon does not change

Describe the bug
Franz and discord icon does not change when I get new message or other update
I get classic notification, but changes on icons
other services work correct. I have classic notification and icons badge update
Screenshots

Desktop (please complete the following information):

OS: MacOS 10.14.4
Browser Safari 12.1

I have had this issue for months too.
Apparently Discord changed how it worked on their end because it is a bug that propagated across all programs like Franz.  Such as Rambox.
In Rambox they managed to fix it with a edit. Not sure if we can put the same fix in Franz?
Here is a link to the fix
https://github.com/ramboxapp/community-edition/pull/2124

I got 0 notifications from Discord via Franz!!!
It's a big problem for me, I literally missed a meeting because they were messaging me, and I got no notifications lol. 😂

https://gitlab.com/thanhpt.bkgm/franz-discord-fix
I make some fix for the discord recipe, you can add it to the recipes/dev directory.

Does not wor for me eiter. On latest Linux version

It's still not working, hmm

I had to stop using Discord on Franz because of this... Discord only has working notifications on their own app.
A real shame, 'cause I liked Franz. Now I have mixed feelings. :'(

Is there any way to fix this? It's very frustrating to not have it work and missing important notifications. :c I have nooo idea how to even begin to fix it or I would try to trouble shoot myself.

Is there any way to fix this? It's very frustrating to not have it work and missing important notifications. :c I have nooo idea how to even begin to fix it or I would try to trouble shoot myself.

https://github.com/meetfranz/recipe-discord/blob/master/webview.js#L4 <- this function is seem to be broken, because the Discord website changed and nobody cared to fix it. All it needs tested fix + PR.

Are there any improvements coming for this? I'm having the same issue... I'm surprised this has been going on since Sep 2019 😄

Bumping. Would really appreciate a fix for this. 🙏

Any chance this is going to be fixed anytime soon?


Provide a full docker build and run example

When running mongodb and the application to connect to it, my mongodb application would not connect successfully to the database: I needed a few extra steps I couldn't see in the readme.
The following works OK so I suggest we mention it (I can do the PR):
docker build -t mernapp:v1.0.0 .
docker run -d -p 27017:27017 mongo
docker inspect <the mongo container name> | grep IPAddress
docker run -p 3000:3000 -e MONGO_URL=<IP_ADDRESS> (this is the IP address determined from above, localhost doesn't work) mernapp:v1.0.0

Thanks @a-roberts , that'd be a good addition.

Docker compose should take care of this, no? You don't need to run the individual containers.


tabular data/ noisy instances/ new datasets

Hi,
thanks for sharing your implementation. I have some questions about it:

Does it also work on tabular data?
Is the code tailored to the datasets used in the paper or can one apply it to any data?
Is it possible to identify the noisy instances (return the noisy IDs or the clean set)?

Thanks!

1&2: Yes, it works for any tabular data. Please refer to the notebook Demo_**.ipynb to see the input dataset format.
3: the noise degree is represented by predicted_loss_diffs in Demo_***.ipynb. To get the cleaned subsets, please refer to the sampling process executed in the notebook.

many thanks!


Nested while post

@yegor256 please merge

@yegor256 reminder

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 1min)


Fix broken link in get-set-up.md

Fix broken link based on issue #2858 and #2885

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

Verified the links (they work in preview). Thanks @TeeVenDick