feat(filter): custom title

allows a custom filter title:
window = {
    mappings = {
        ["/"] = { "fuzzy_finder", config = { title = "CUSTOM TITLE" } },

        ["o"] = { "show_help", config = { title = "Help", } }, -- NOTE: other popups do have this function
    }
}

Please add something about this in the doc at least as well. Remember that undocumented features will end up never being used ;)

Please add something about this in the doc at least as well. Remember that undocumented features will end up never being used ;)

done

Any reason this isn't also added to filters/init.lua?

Was not sure what keymap triggers that function? I've never seen those popups

I was talking about the vim helpdoc as there is a section specifically calling out "custom mappings" which is where this would be used.

IIRC common is what external sources will fall under. So any external source that doesn't implement the show_filter function will be provided that function by neo-tree on source load. By not including this option there, we are adding a "neo-tree only" feature that is inaccessible to external sources. Personal motivation aside, we should try to not do that :)

I was talking about the vim helpdoc as there is a section specifically calling out "custom mappings" which is where this would be used.

@miversen33 i've added the docs what do you think?

IIRC common is what external sources will fall under. So any external source that doesn't implement the show_filter function will be provided that function by neo-tree on source load. By not including this option there, we are adding a "neo-tree only" feature that is inaccessible to external sources. Personal motivation aside, we should try to not do that :)

Uhmm, im not sure what the best option is here. I think it's best if you make a decision on this, as aim fairly new to nvim/lua. Would you like me to adjust  filters/init.lua as well?

I was talking about the vim helpdoc as there is a section specifically calling out "custom mappings" which is where this would be used.

@miversen33 i've added the docs what do you think?

LGTM, Thanks!

IIRC common is what external sources will fall under. So any external source that doesn't implement the show_filter function will be provided that function by neo-tree on source load. By not including this option there, we are adding a "neo-tree only" feature that is inaccessible to external sources. Personal motivation aside, we should try to not do that :)

Uhmm, im not sure what the best option is here. I think it's best if you make a decision on this, as aim fairly new to nvim/lua. Would you like me to adjust filters/init.lua as well?

I am going to leave this up to @pysan3 to decide on. IMO it makes sense to expose this option to external sources as well, but I could also be completely wrong.


Sink / Target Naming Convention

Greetings, I'm a 20+ year SQL Server DBA and non-CS person. Maybe my vernacular is out of style and this is the new way but... I had to look up what "sink" meant. Would it be possible to keep or alias the word to the classic "Target" or "Destination" terminology? Every migration tool I've used uses one of the two terms and I can see having to add an extra slide defining sink when doing presentations on getting SQL Server data to Cosmos.
Thank you 🙏🏼

Thank you 🥳


Remove cookie banner

We don't actually use any cookies, except to track whether the user has
dismissed the cookie banner.
Additionally, there is no legal requirement to ask for consent (GDPR) or
provide an opt-out (CCPA) to use this type of cookie.

I think that was it at least

We used to have cloudflare analytics on the page, which is what the banner was for.

Hardly "functional cookies", but let's say bygones be bygones :)

Actually no, was because _cfuid: https://github.com/denoland/dotland/issues/1429. That has since been removed tough


[$500] IOU hold - Wrong error message is shown when trying to hold a paid IOU

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

Version Number: 1.4.50-2
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Issue found when executing PR https://github.com/Expensify/App/pull/37477
Action Performed:

[Employee] Create a manual request from workspace chat
[Employee] Go to request details page
[Employee] Click 3-dot menu > Hold request
[Admin] Pay the request
[Employee] After the background shows "paid", add a reason and submit it

Expected Result:
Employee should see a clear message as to why employee can't submit the hold reason
Actual Result:
Employee can't submit the hold message yet a wrong message is shown. "Fix the errors" message is shown instead
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/67d0214b-0b18-45fc-a339-0cd70cedcc52
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~018199dfef5419180f
Upwork Job ID:<PHONE_NUMBER>862301184
Last Price Increase: 2024-03-11

@iwiznia  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

Not a blocker https://expensify.slack.com/archives/C036QM0SLJK/p1710179174439839

@ShridharGoel I have tested your Proposal's option 1 and option 2, and output does not looking as expected, in both the scenario error message This request is being modified by another member. shows under the Reason input instead of the Form bottom, attaching both options videos

Option 1
https://github.com/Expensify/App/assets/35371050/45cedb07-1f49-4544-b0fa-66702e862fed

Option 2
https://github.com/Expensify/App/assets/35371050/196e31f8-f092-441d-961c-23c00a4edebc

I have checked the commit, where we have added the typeof errorMessage === 'string' check, and it looks like it was added for fixing the type warning, and in the latest code if we remove the typeof errorMessage === 'string' check ,we are not getting any type warning , i think we can remove this check

Type warning that we were trying to solve

ccing @kubabutkiewicz , as you were the author of the PR
@Krishna2323 's proposal of removing redundant check of the typeof errorMessage === 'string' looks good to me
🎀 👀 🎀 C+ reviewed

@GandalfGwaihir I have checked the issue that you've mentioned and it looks like there is no proposal yet selected or neither we have any update on the pending proposals yet, @iwiznia what's your thoughts on this, should we proceed with above proposal

Raised this for discussion internally as well.
My thought is that whatever GH makes it to the accepted proposal stage first gets the PR and the other is put on hold. There are many opportunities for bugs and we want to move forward with urgency when possible. This GH has a proposal further along and if the engineer agrees I say we move forward here.

Should have an update and able to take action tomorrow.

@Krishna2323 what is offending PR? This was marked as deploy blocker on Mar 11.
https://github.com/Expensify/App/pull/37426 is definitely not, as this was deployed to production on Mar 5.

We can put this on HOLD until https://github.com/Expensify/App/issues/37623 is completed. Doing so now.

On HOLD PR is still open.

Per this GH and this road-map doc, this on HOLD bug is best suited for this VIP.
@davidcardoza - If you disagree with this project, let me know. Or, as the curator if you believe this should be closed instead, feel free to do that.
Current status though is that this is held on #37623.
If we keep it open and assigned, I'll continue to monitor and retest once the hold GH is closed.

Oops, I had this GH open from yesterday and the comment from @jayeshmangwani didn't update until after I sent my message above. Reviewing that now and likely closing this.


Do we have benchmark result, triton version vs cutlass?

Hi,
I wonder know that whether we have benchmark result for triton version vs cutlass version?
Since both is implemented, it is interesting to know which is better.
Thx

The benchmark numbers are in the README.

The README benchmark result only list "flashatten triton", is it implentation for the V1?
What's the performance result for triton version for V2?

Idk, I might push some script to run the benchmarks soon. Feel free to run them.

I've just add a script to benchmark here. The best way to compare is to run the benchmark yourself.
The Triton implementation is evolving, but you can look at their performance data here. The last number in each line is the fraction of theoretical max performance, you can take that and multiply by 312 TFLOPs/s (for A100) and compare with the numbers we have for FlashAttention2 in the README. I think technically Triton's data is measured on A100 40GB and our data in the README is measured on A100 80GB, but the two settings should be pretty close.
I think right now Triton forward is on par with CUDA forward for headdim=64 but slower for headdim=128. The Triton backward is still the same as when I benchmarked it 2 weeks ago (shown in the README), so about 2x slower than the CUDA backward. This could change in the future as the Triton version is improved.

Thx, I would give it a try. ;)


#900 Added tiled benchmarking sample in sandbox

I added some of the maps I have been using to test performance and a sample class for testing them.
I included some maps with entirely flipped tiles which has terrible performance (as you can imagine) over ones with unflipped tiles. This could probably be improved, but in actual use I don't know if flipped tiles are common enough to warrant it.
I would include the 'closes' tag but I wanted to take a last look at any way I could improve loadView and looking at alternative buffers.
Can I ask what you were using to run your benchmarks in your comment a couple days ago?
Thanks!
Adam Bocco

Codecov Report

Merging #1013 (583c69e) into dev (58e7124) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                dev    #1013   +/-   ##
=========================================
  Coverage     59.82%   59.82%
  Complexity     2771     2771
=========================================
  Files           285      285
  Lines         13792    13792
  Branches       1372     1372
=========================================
  Hits           8251     8251
  Misses         5070     5070
  Partials        471      471

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 58e7124...583c69e. Read the comment docs.

The Tiled sample I ran is in sandbox/benchmark.

@adambocco
Given there is a UI and everything now, do you want to move my tmx there too and delete my sample?


Reduce E/SpannableStringBuilder﹕ SPAN_EXCLUSIVE_EXCLUSIVE spans cannot have a zero length

This error is active several times when starting the application. This reduces the usability of the debugger and leads to a decreased maintainability. Therefore this error shall be reduced. Namely it is active when either critiquing an item or viewing the items details. Furthermore in some cases it is also active when all items are displayed.

It seems that this cannot be fixed. Might be an error especially on Samsung devices?


Docker SSL configuration doesn't enable ssl port

Making /ssl volume available for the neo4j container doesn't seem to enable ssl. I followed the instructions on Neo4j Docker docs to enable ssl and also looked into docker-entrypoint.sh. I don't think that what is mentioned in the docs is working.
https://neo4j.com/docs/operations-manual/current/docker/security/
Steps to reproduce:
$ pwd
/opt/neo4j/certs/https
$ ls
neo4j.cert  neo4j.key  private.key  public.crt
$ docker run -d -p 7473:7473 -p 7474:7474 -p 7687:7687 --rm \
    -v /opt/neo4j/certs/https/:/ssl \
    neo4j:4.0

Expected behaviour
Https is enabled for browser with https scheme and https port (7473).
Actual behaviour
Https is not enabled. Remote interface is still only available at http scheme and http port (7474).
$ docker logs <container-id>
Starting Neo4j.
2020-01-25 22:20:58.078+0000 WARN  Use of deprecated setting dbms.directories.certificates. Legacy ssl policy is no longer supported.
2020-01-25 22:20:58.090+0000 INFO  ======== Neo4j 4.0.0 ========
2020-01-25 22:20:58.094+0000 INFO  Starting...
2020-01-25 22:21:02.275+0000 INFO  Bolt enabled on <IP_ADDRESS>:7687.
2020-01-25 22:21:02.276+0000 INFO  Started.
2020-01-25 22:21:03.468+0000 INFO  Remote interface available at http://<IP_ADDRESS>:7474/

Try curl -k https://localhost:7473 returns ssl error however curl localhost:7474 does return a response from the server.
There is also the deprecation warning about dbms.directories.certificates in the logs, which makes me think that /ssl is not working as expected in the entrypoint.
Any help would be greatly appreciated!
Neo4j Docker tag: 4.0
Docker version: 18.03.1-ce
OS: Ubuntu 16.04.5 LTS

I'm having a similar issue. I used to mount /ssl, which is not possible anymore with Neo4j 4.0. After changing the docker configuration to
docker run -d \
--name neo4j \
--net=host \
--volume=/opt/neo4j/certificates/https:/var/lib/neo4j/certificates/https  \
--env NEO4J_dbms_ssl_policy_https_enabled=true \
--env NEO4J_dbms_ssl_policy_https_base__directory=certificates/https \
--env NEO4J_dbms_ssl_policy_https_private__key=private.key \
--env NEO4J_dbms_ssl_policy_https_public__certificate=public.crt \
neo4j:4.0.3

I'm getting the error
 ERROR Failed to start Neo4j: Starting Neo4j failed: Component 'org.neo4j.server.database.LifecycleManagingDatabaseService@4a335fa8' was successfully initialized, but failed to start. Please see the attached cause exception "/var/lib/neo4j/certificates/https/private.key (Permission denied)". Starting Neo4j failed: Component 'org.neo4j.server.database.LifecycleManagingDatabaseService@4a335fa8' was successfully initialized, but failed to start. Please see the attached cause exception "/var/lib/neo4j/certificates/https/private.key (Permission denied)".
org.neo4j.server.ServerStartupException: Starting Neo4j failed: Component 'org.neo4j.server.database.LifecycleManagingDatabaseService@4a335fa8' was successfully initialized, but failed to start. Please see the attached cause exception "/var/lib/neo4j/certificates/https/private.key (Permission denied)".
        at org.neo4j.server.exception.ServerStartupErrors.translateToServerStartupError(ServerStartupErrors.java:45)
        at org.neo4j.server.AbstractNeoServer.start(AbstractNeoServer.java:164)
        at org.neo4j.server.ServerBootstrapper.start(ServerBootstrapper.java:114)
        at org.neo4j.server.ServerBootstrapper.start(ServerBootstrapper.java:89)
        at org.neo4j.server.CommunityEntryPoint.main(CommunityEntryPoint.java:35)

I tried changing the permissions via
sudo chmod 777 /opt/neo4j/certificates/https/*
but permissions get changed from -rwxrwxrwx to -rwx------ after running the docker run command and I get the same error.
Running docker with --user="$(id -u):$(id -g)" or --user="0:0" didn't help either. Any ideas?

You're right, our SSL documentation for docker is horribly out of date. I apologise for that and I'm going to work on fixing it.
Here's what I did to get it working, and I suppose the new documentation will be some variation on this.
HTTPS Encryption in Neo4j Docker
These instructions assume you already have the necessary ssl key and certificate.

Create a certificates folder, with subfolders for each encryption policy you need.

This is documented at: https://neo4j.com/docs/operations-manual/4.0/security/ssl-framework/
The key and certificate must be called private.key and public.crt respectively. For example, this is the folder structure required for https encryption:
certificates/
  |  https/
       |  private.key
       |  public.crt
       |  revoked/
            |
       |  trusted/
            |  public.crt  (a copy of certificates/https/public.crt)

Start docker remembering to:

mount certificates to /var/lib/neo4j/certificates
run container as a user with at least READ access to the certificates folder and all its files.
publish port 7473
set the configurations enabling https, NEO4J_dbms_connector_https_enabled and NEO4J_dbms_ssl_policy_https_enabled, to true.

Here's an example:
docker run -it --rm \
    -p 7473:7473 -p 7474:7474 -p 7687:7687 \
    --user="$(id -u):$(id -g)" \
    -v $HOME/neo4j/certificates:/var/lib/neo4j/certificates  \
    -e NEO4J_dbms_connector_https_enabled=true \
    -e NEO4J_dbms_ssl_policy_https_enabled=true \
    neo4j:4.0.3

This should be enough to enable SSL encryption over HTTPS:
curl -k https://localhost:7473
{
  "bolt_routing" : "neo4j://localhost:7687",
  "transaction" : "https://localhost:7473/db/{databaseName}/tx",
  "bolt_direct" : "bolt://localhost:7687",
  "neo4j_version" : "4.0.3",
  "neo4j_edition" : "community"
}

@Schirmi136  Could you check who the owning user and group is on /opt/neo4j/certificates/https? I suspect it's not the same as the user you're running as. So either neo4j won't be able to read the files there (when running with --user="$(id -u):$(id -g)" OR it can't chown the files as the neo4j user because docker daemon hasn't got the necessary permissions.

I have roughly the same problem. I have a directory set up with different certificates for the bolt and https connectors:
/home/neo4j/certificates# find .
.
./bolt
./bolt/private.key
./bolt/public.crt
./https
./https/private.key
./https/public.crt

If I mount this certificates folder at the /ssl path in the container, I get an error that the configuration doesn't exist:
Directories in use:
  home:         /var/lib/neo4j
  config:       /var/lib/neo4j/conf
  logs:         /logs
  plugins:      /var/lib/neo4j/plugins
  import:       /var/lib/neo4j/import
  data:         /var/lib/neo4j/data
  certificates: /ssl
  run:          /var/lib/neo4j/run
Starting Neo4j.
2020-06-11 00:42:09.452+0000 WARN  Use of deprecated setting dbms.directories.certificates. Legacy ssl policy is no longer supported.
2020-06-11 00:42:09.462+0000 INFO  ======== Neo4j 4.0.4 ========
2020-06-11 00:42:09.466+0000 INFO  Starting...
2020-06-11 00:42:10.157+0000 ERROR Failed to start Neo4j: Starting Neo4j failed: Component 'org.neo4j.server.database.LifecycleManagingDatabaseService@17aad511' was successfully initialized, but failed to start. Please see the attached cause exception "Base directory '/var/lib/neo4j/certificates/bolt' for SSL policy with name 'bolt' does not exist.". Starting Neo4j failed: Component 'org.neo4j.server.database.LifecycleManagingDatabaseService@17aad511' was successfully initialized, but failed to start. Please see the attached cause exception "Base directory '/var/lib/neo4j/certificates/bolt' for SSL policy with name 'bolt' does not exist.".
org.neo4j.server.ServerStartupException: Starting Neo4j failed: Component 'org.neo4j.server.database.LifecycleManagingDatabaseService@17aad511' was successfully initialized, but failed to start. Please see the attached cause exception "Base directory '/var/lib/neo4j/certificates/bolt' for SSL policy with name 'bolt' does not exist.".
	at org.neo4j.server.exception.ServerStartupErrors.translateToServerStartupError(ServerStartupErrors.java:45)
	at org.neo4j.server.AbstractNeoServer.start(AbstractNeoServer.java:164)
	at org.neo4j.server.ServerBootstrapper.start(ServerBootstrapper.java:114)
	at org.neo4j.server.ServerBootstrapper.start(ServerBootstrapper.java:89)
	at org.neo4j.server.CommunityEntryPoint.main(CommunityEntryPoint.java:35)
Caused by: org.neo4j.kernel.lifecycle.LifecycleException: Component 'org.neo4j.server.database.LifecycleManagingDatabaseService@17aad511' was successfully initialized, but failed to start. Please see the attached cause exception "Base directory '/var/lib/neo4j/certificates/bolt' for SSL policy with name 'bolt' does not exist.".
	at org.neo4j.kernel.lifecycle.LifeSupport$LifecycleInstance.start(LifeSupport.java:465)
	at org.neo4j.kernel.lifecycle.LifeSupport.start(LifeSupport.java:111)
	at org.neo4j.server.AbstractNeoServer.start(AbstractNeoServer.java:157)
	... 3 more
Caused by: java.lang.IllegalArgumentException: Base directory '/var/lib/neo4j/certificates/bolt' for SSL policy with name 'bolt' does not exist.
	at org.neo4j.ssl.config.SslPolicyLoader.createSslPolicy(SslPolicyLoader.java:159)
	at org.neo4j.ssl.config.SslPolicyLoader.addPolicy(SslPolicyLoader.java:143)
	at java.base/java.util.HashMap$Values.forEach(HashMap.java:976)
	at org.neo4j.ssl.config.SslPolicyLoader.load(SslPolicyLoader.java:133)
	at org.neo4j.ssl.config.SslPolicyLoader.create(SslPolicyLoader.java:96)
	at org.neo4j.graphdb.factory.module.edition.CommunityEditionModule.<init>(CommunityEditionModule.java:110)
	at org.neo4j.graphdb.facade.DatabaseManagementServiceFactory.build(DatabaseManagementServiceFactory.java:118)
	at org.neo4j.server.database.CommunityGraphFactory.newDatabaseManagementService(CommunityGraphFactory.java:36)
	at org.neo4j.server.database.LifecycleManagingDatabaseService.start(LifecycleManagingDatabaseService.java:88)
	at org.neo4j.kernel.lifecycle.LifeSupport$LifecycleInstance.start(LifeSupport.java:444)
	... 5 more
2020-06-11 00:42:10.162+0000 INFO  Neo4j Server shutdown initiated by request

However, if I mount the same directory at /var/lib/neo4j/certificates, without changing anything else in the configuration, the system works.
Changed password for user 'neo4j'.
Directories in use:
  home:         /var/lib/neo4j
  config:       /var/lib/neo4j/conf
  logs:         /logs
  plugins:      /var/lib/neo4j/plugins
  import:       /var/lib/neo4j/import
  data:         /var/lib/neo4j/data
  certificates: /var/lib/neo4j/certificates
  run:          /var/lib/neo4j/run
Starting Neo4j.
2020-06-11 00:49:14.558+0000 INFO  ======== Neo4j 4.0.5 ========
2020-06-11 00:49:14.570+0000 INFO  Starting...
2020-06-11 00:49:20.469+0000 INFO  Bolt enabled on <IP_ADDRESS>:7687.
2020-06-11 00:49:20.470+0000 INFO  Started.
2020-06-11 00:49:22.632+0000 INFO  Remote interface available at https://localhost:7473/

The permissions are the same in both situations. In both situations, I am passing a uid and gid from the host system using the --user flag to docker. That is the user that owns the certificates and directories. They have 500 permissions for directories and 400 for files on the host:
/home/neo4j/certificates# ll
total 16
dr-x------ 4 neo4j neo4j 4096 Jun 11 00:49 ./
drwxr-xr-x 8 neo4j neo4j 4096 Jun 11 00:49 ../
dr-x------ 2 neo4j neo4j 4096 Jun 11 00:49 bolt/
dr-x------ 2 neo4j neo4j 4096 Jun 11 00:49 https/
/home/neo4j/certificates# cd bolt
/home/neo4j/certificates/bolt# ll
total 16
dr-x------ 2 neo4j neo4j 4096 Jun 11 00:49 ./
dr-x------ 4 neo4j neo4j 4096 Jun 11 00:49 ../
-r-------- 1 neo4j neo4j 1707 Jun 11 00:49 private.key
-r-------- 1 neo4j neo4j 1249 Jun 11 00:49 public.crt

I've finally updated the documentation and included examples of how to set up https encryption. See:
https://neo4j.com/docs/operations-manual/4.0/docker/security/
Mounting the certificates folder to /ssl doesn't work any more for 4.0.0 and onwards unfortunately. The legacy ssl system that used was deprecated.
I'm going to close this issue now.
There are still problems using self signed certificates, but that is tracked separately in issue #257 .

@davcamer the default directory Neo4j 4.0 looks for certificates is /var/lib/neo4j/certificates (see: https://neo4j.com/docs/operations-manual/4.0/security/ssl-framework/).
If you mount your certificates folder to /ssl then the certificates will be stored under /ssl instead, so if you don't also configure Neo4j to look there, then it won't be able to find the certificates. This is what your error message is complaining about. When you mount your certificates to /var/lib/neo4j/certificates, Neo4j is able to find them and it all works.
Check https://neo4j.com/docs/operations-manual/4.0/docker/security/ for more help, and maybe let me know if it didn't work for you!

@jennyowen In that case, should this section of the entrypoint script be removed?
https://github.com/neo4j/docker-neo4j/blob/master/docker-image-src/4.0/docker-entrypoint.sh#L315-L319

@davcamer ok you got me! In my defence, that bit of code is only in 4.0.5 and onwards.

Oops! My fault for looking at the wrong code then. 😞 And my /ssl mount would probably work on the 4.0.5 image. Thanks. 🙇♂


Asset rename

I can't select the text when i try to rename the asset

Select which text?

When you want to change the name of an art asset the selection is a little bit weird, you can't select the image name.
Date: Thu, 20 Aug 2015 08:21:18 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [RedWire] Asset rename (#502)
Select which text?
—
Reply to this email directly or view it on GitHub.

Ah, I see what you mean! It thinks that you're doing drag and drop.
2015-08-20 22:57 GMT+02:00 esther5576<EMAIL_ADDRESS>
When you want to change the name of an art asset the selection is a little
bit weird, you can't select the image name.
Date: Thu, 20 Aug 2015 08:21:18 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [RedWire] Asset rename (#502)
Select which text?
—
Reply to this email directly or view it on GitHub.
—
Reply to this email directly or view it on GitHub
https://github.com/CyberCRI/RedWire/issues/502#issuecomment-133168349.

Yes!
Date: Fri, 21 Aug 2015 01:08:01 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [RedWire] Asset rename (#502)
Ah, I see what you mean! It thinks that you're doing drag and drop.
2015-08-20 22:57 GMT+02:00 esther5576<EMAIL_ADDRESS>
When you want to change the name of an art asset the selection is a little

bit weird, you can't select the image name.

Date: Thu, 20 Aug 2015 08:21:18 -0700

From<EMAIL_ADDRESS>

To<EMAIL_ADDRESS>

CC<EMAIL_ADDRESS>

Subject: Re: [RedWire] Asset rename (#502)

Select which text?

—

Reply to this email directly or view it on GitHub.

—

Reply to this email directly or view it on GitHub

https://github.com/CyberCRI/RedWire/issues/502#issuecomment-133168349.

—
Reply to this email directly or view it on GitHub.


Added userful queries to filtersets

Pull Request
Related Issue

New Behavior
Added useful quick search functionality to

Access Lists,
Standard Rules,
Extended Rules,
Interface Assignments.

This also fixes a bug which would yield an error when searching for an access list in the interface assignment form (https://github.com/netbox-community/netbox-acls/issues/194).
Contrast to Current Behavior
Quick search mostly doesn't search anything useful.
Discussion: Benefits and Drawbacks
No known drawbacks.
Changes to the Documentation
No changes to the documentation necessary.
Proposed Release Note Entry
Enhanced quick search functionality.
Double Check

[X] I have explained my PR according to the information in the comments
or in a linked issue.
[X] My PR targets the dev branch.

There're a couple of minor questions I left in code.

Thanks for your comments! I looked at the code and though the standard and extended rule models do have descriptions in the model definitions, these fields aren't associated to the filtersets here
https://github.com/netbox-community/netbox-acls/blob/ec8aa7c6aca14f49309d3cd52690ee6be845e23e/netbox_acls/filtersets.py#L164
and here
https://github.com/netbox-community/netbox-acls/blob/ec8aa7c6aca14f49309d3cd52690ee6be845e23e/netbox_acls/filtersets.py#L189
Since I'm unaware of what the the original authors' reasoning behind this decision is I just worked with fields associated in the filtersets. I'd be happy to hear their thoughts on this and change the code accordingly if they approve.


Add Runlevels + docker labels

For CI, we're going to switch back to outputting a single image with multiple binaries (like in OSBS)
Also added the correct runlevel prefix (original docs were incorrect).

/approve

/lgtm

After this merged the installer is hanging:
E1009 01:37:54.959270       1 sync.go:52] error running apply for (/v1, Kind=ServiceAccount) openshift-operator-lifecycle-manager/olm-operator-serviceaccount: serviceaccounts "olm-operator-serviceaccount" is forbidden: caches not synchronized

I don't see a run level on the namespace

Namespace needs labels:
openshift.io/run-level=1

Fixed in https://github.com/operator-framework/operator-lifecycle-manager/pull/503


Websocket direct upgrade.

When using websockets the connection is not upgraded until the first request is made from the client.
It would be nice if the connection is directly upgraded so the server can begin pushing data to client.

This is a bug, since the code actually calls webSocketUpgrade in network establish.
@miyconst could you take a look?
The case is that you have pages like dashboards, chatrooms, etc, that should be available for WebSocket server push immediately after page load.

I just retested this and now it seems to work (strange)... i will try to make a deeper test.

That is true that, Puppet opens WebSocket after first HTTP request, but this request should be made on page load. It seems to be another reason to restructure the way we obtain URL for WebSocket connection.

@andwah is this still reproducible? Give me the steps to reproduce if so, please.

Chatter app works immediately after load.

Ping @andwah.

@miyconst I could not reproduce it.

@andwah thank you for testing. I close the issue as it is assigned to me.


Reopen #858: MultiGet does not respect source filtering fields

#858 appears to have been regressed, as early as version 1.0.2:
Just verified locally under the exact same conditions as originally. I was using version 1.0.2, and since upgraded to 1.4.0.
Here's the NEST query:
Client.MultiGet(s =>
	s.GetMany<Performer>(termItems.Select(x => x.Term))
	.SourceInclude<Performer>(x => x.Id, x => x.Name)

And the generated request:
http://qual-elastic002:9200/_mget?source_include=id%2Cname
{
  "docs": [
	{
	  "_index": "qual4-events014",
	  "_type": "performer",
	  "_id": "f6fc51e4-4198-4d32-a47a-4e0e17e025a3"
	},
	{
	  "_index": "qual4-events014",
	  "_type": "performer",
	  "_id": "98d2b8dd-0de8-4dfa-87f8-716a880fa6e0"
	}
	/* etc... */
  ]
}

The expected request:
http://qual-elastic002:9200/_mget
{
  "docs": [
	{
	  "_index": "qual4-events014",
	  "_type": "performer",
	  "_id": "f6fc51e4-4198-4d32-a47a-4e0e17e025a3",
	  "_source": [ "id", "name" ]
	},
	{
	  "_index": "qual4-events014",
	  "_type": "performer",
	  "_id": "98d2b8dd-0de8-4dfa-87f8-716a880fa6e0",
	  "_source": [ "id", "name" ]
	}
	/* etc... */
  ]
}

@lakario You're applying source at the query param level, hence the request url: http://qual-elastic002:9200/_mget?source_include=id%2Cname which applies source filtering for the entire request.
Instead, you want to set source filtering at the document level.  The syntax is a bit different:
Client.MultiGet(s =>s
    .GetMany<Performer>(termItems.Select(x => x.Term), gm => gm
        .Source(src => src
            .Include("id", "name")
        )
    )
)

@gmarz The above signature doesn't seem to exist for me.
public MultiGetDescriptor GetMany<T>(IEnumerable<string> ids, Func<MultiGetOperationDescriptor<T>, string, MultiGetOperationDescriptor<T>> getSelector = null) where T : class;
public MultiGetDescriptor GetMany<T>(IEnumerable<long> ids, Func<MultiGetOperationDescriptor<T>, long, MultiGetOperationDescriptor<T>> getSelector = null) where T : class;

Also, your example does not take advantage of the type-safe expression syntax, which is what I am after.

@gmarz The above signature doesn't seem to exist for me.
public MultiGetDescriptor GetMany<T>(IEnumerable<long> ids, Func<MultiGetOperationDescriptor<T>, long, MultiGetOperationDescriptor<T>> getSelector = null) where T : class;
public MultiGetDescriptor GetMany<T>(IEnumerable<string> ids, Func<MultiGetOperationDescriptor<T>, string, MultiGetOperationDescriptor<T>> getSelector = null) where T : class;

Also, your example does not take advantage of the type-safe expression syntax, which is what I am after.

Oops sorry, I left out the long argument in the lambda.  It should be:
Client.MultiGet(s =>s
    .GetMany<Performer>(termItems.Select(x => x.Term), (gm, i) => gm
        .Source(src => src
            .Include(x => x.Id, x.Name)
        )
    )
)

Notice above, you can also use the expression form for your properties.

@gmarz Sure enough, that's what I was looking for. Thanks.


Cannot be inferred using the last model provided？

Task003_tumor Cannot be inferred using the last model provided？
Traceback (most recent call last):
File "/home/lcl/unetr_plus_plus/unetr_pp/training/model_restore.py", line 168, in
trainer = restore_model(pkl, checkpoint, train)
File "/home/lcl/unetr_plus_plus/unetr_pp/training/model_restore.py", line 58, in restore_model
info = load_pickle(pkl_file)
File "/home/lcl/anaconda3/envs/unetr_pp/lib/python3.8/site-packages/batchgenerators/utilities/file_and_folder_operations.py", line 50, in load_pickle
a = pickle.load(f)
_pickle.UnpicklingError: A load persistent id instruction was encountered,
but no persistent_load function was specified.

Could you please share the command you using for evaluation?

@Amshaker
I encounter the same problem, and find that the evaluation code of BraTS is different from the other datasets.
The run_evaluation_turom.sh uses the unetr_pp/inference/predict_simple.py to generate the prediction, while other datasets using the run_training.py to generate segment masks.
The predict_simple.py calls the function restore_model from unetr_plus_plus/unetr_pp/training/model_restore.py to load the pre-trained weights.
The first line of restore_model loads the pre-trained weights in the .pkl format, but the pre-trained weights are  in the .model format, which causes  the problem above.
You can repeat the above error by running the evaluation_scripts/run_evaluation_turmor.sh directly.
Could you provide the pre-trained weights in .pkl format, as the mix of training and inference code together is hard to use.
I think inference the segment mask of raw input using the predict_simple.py is more reasonable than using the unetr_pp/run/run_training.py.


Apply patch Feature: Render geometry & output as SVG

Kudos to Marc Palmans!
Feature: Render geometry & output as SVG

I've added code to render to a Geometry and then convert it to an SVG snippet which then can be inserted in an SVG document. At the moment it's hard coded to black with no stroke as I don't need anything else. It should be very easy to add those features if you need them.

Example:
render in Inkscape

svg-example.zip
#3

Thanks!


MacOS Installation script release 1.4.0

What does this PR do?
Preparation for the release of the macOS install script 1.4.0

Changelog :

Fix macOS headless installation for macOS 14+ #22227
Notify user how to upgrade / install rosetta if missing/incompatible #23331

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Add border back in for input

This PR closes issue #231 ... by adding back in the border to the input.

Deploy preview for project-koku ready!
Built with commit 7f077ca4706c13f815c89a951ebd425715d3413a
https://deploy-preview-256--project-koku.netlify.com

Codecov Report

Merging #256 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #256   +/-   ##
=======================================
  Coverage   91.92%   91.92%
=======================================
  Files         120      120
  Lines        1374     1374
  Branches      160      160
=======================================
  Hits         1263     1263
  Misses         99       99
  Partials       12       12

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 810107c...7f077ca. Read the comment docs.


clean up convert-based fallbacks

fix https://github.com/SciML/LinearSolve.jl/actions/runs/5417914034/jobs/9849418672?pr=337#step:6:13263

with this branch,
julia> using SciMLOperators, LinearAlgebra

julia> a = MatrixOperator(rand(4,4))

julia> b = a + a * a \ a

julia> issymmetric(b)
┌ Warning: using convert-based fallback in issymmetric.
└ @ SciMLOperators ~/.julia/dev/SciMLOperators/src/interface.jl:387
false

julia> sum(Base.Fix2(*, 2), (a,a); init = a)
(MatrixOperator(4 × 4) + ScalarOperator(2) * MatrixOperator(4 × 4) + ScalarOperator(2) * MatrixOperator(4 × 4))

julia> prod(Base.Fix2(*, 2), (a,a); init = a)
(MatrixOperator(4 × 4) * ScalarOperator(2) * MatrixOperator(4 × 4) * ScalarOperator(2) * MatrixOperator(4 × 4))

The downstream tests are still failing on this merge? https://github.com/SciML/SciMLOperators.jl/actions/runs/5437055376/jobs/9887334855#step:6:966

Thanks for pointing this out. I was wrong to remove fallback methods for sum, prod. Let me fix that.

I made the mistake of confusing the methods
sum(f, (A, B); init) # returns lazy AddedOperator

with
sum(A::AbstractArray; dims) # reduces with sum across dims

I added back the removed methods in #209 with an example.


What if I don't find my LSP in MASON

Hey, I'm getting started with Neovim and all of its capabilities. I came across this package because ThePrimagen recommended it in one of his videos.
My issue is that I cannot find my LSP in Mason (sourcekit-lsp), I looked at their docs and it's using lspconfig in order to set it up, I'm wondering how to add the same keybindings from lsp-zero to lspconfig or where should I put the configuration.
Thanks in advance!

Is it 0 to LSP tutorial? That video uses an old version of lsp-zero. Make sure you are using lsp-zero version 3 (use the branch v3.x). You can use the command :help lsp-zero-version to check which version you are using.
If a language server is not available in mason, you have to install it manually. After that, if the language server is supported by lspconfig then just call the setup function for it. The list of supported servers is here: server_configurations.md.
Since sourcekit is already supported by lspconfig this should work.
local lsp_zero = require('lsp-zero')

lsp_zero.on_attach(function(client, bufnr)
  lsp_zero.default_keymaps({buffer = bufnr})
end)

require('lspconfig').sourcekit.setup({})

If the language server is not supported by lspconfig you can still add a configuration to make it work. There is a section in the documentation that describes how to do it: Custom servers.

That was helpful, thanks!
Now I'm presenting this issue:
Error detected while processing /Users/user-name/.config/nvim/after/plugin/lsp.lua:
E5113: Error while calling lua chunk: /Users/user-name/.config/nvim/after/plugin/lsp.lua:86: attempt to call field 'cmp_format' (a nil value)

It looks like when I try to set up cmp with the formatting value, lsp-zero.cmp_format() is giving a nil value.

Here's the code snippet:
Error detected while processing /Users/santiagotorres/.config/nvim/after/plugin/lsp.lua:
E5113: Error while calling lua chunk: /Users/santiagotorres/.config/nvim/after/plugin/lsp.lua:86: attempt to call field 'cmp_format' (a nil value)

Make sure you are using the branch v3.x of lsp-zero.


⚠ Bofrost.at has degraded performance

In 9a2f011, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 946 ms

Resolved: Bofrost.at performance has improved in c9b9084.


Remove jupyter.useNotebookEditor setting

Validated. Setting is gone now.

Is there an alternative to this? I can't seem to find a way to automatically open '.ipynb' as JSON anymore. Sometimes I just want to see the raw file, and not start a whole notebook editing instance.

@joyceerhl @IanMatthewHuff Could one of you clarify? I'm not clear on where justifications for planned changes are documented (this issue doesn't have a description?) and I would really like to know of any alternatives.
It's quite frustrating for .ipynb files to automatically open up in notebook form without an option to view them in json form, especially for diff in source control.  And sometimes when the .ipynb file is broken for whatever reason (e.g. because source control did a bad merge because we can't view the file in json form), it can't be fixed in vscode since the jupyter viewer just crashes (I renamed the file to .json, edited, then renamed it back, then repeated until all the mistakes were fixed).
Combined with the fact that I can't seem to uninstall the Jupyter extension without uninstalling the entire Python extension, this is kind of a big inconvenience and I bet a pretty decent chunk of the python coding world might have the same issue?

#633 just linking in this related issue


fix(linter/no_empty_file): point to start of file instead of the entire file

Since it may contain lots of comments, which gives us a screenful of comments

Current dependencies on/for this PR:

main

PR #1104

PR #1105   👈

This comment was auto-generated by Graphite.

Merge activity

Oct 30, 4:11 AM: @Boshen merged this pull request with Graphite.


Clarification Regarding Using WFST decoding

Hey,
Much thanks for this very useful project. After following this lecture slide, I was able to generate final decoding graph TLG.fst. For my project requirement, I want to use the WFST decoding with CTC (char label) generated by external software such as DeepSpeech. The scripts given in examples for such task also uses the Essen Acoustic Model in loop (to the best of my knowledge), which I dont require.  Can you please help me to look into the right direction for writing a script for my task?
Thanks,
Ram

Found it. https://github.com/srvk/eesen/issues/148.
Thanks.


Play 2.4 and Silhouette 3 changes

Updated configuration and source files to Play 1.4 and Silhouette 3.0.0

@anotherhale Thanks for your work on this! I really appreciate that!
The angular seed template doesn't work with OpenID or Clef because the client side auth framework Satellizer isn't compatible with this methods. It has also problems with Xing. So we can only use the providers as implemented in the last version of the template.
There exists also a lot of additional configuration/logging settings that are not really needed by the template. The database/jdbc stuff as example.

@anotherhale Hi Andy, do you find the time to polish this PR? Otherwise I would polish the PR and send you a pull request to your repository. Then you can merge it into your repository and this pull request gets automatically updated.

I have updated the fork to 3.0.0-RC2 and removed the extra code/configurations. I have not tested this as the sbt-yeoman bug is not fixed yet.

Looks good, expect the small changes I've commented. If you have made the changes, please could you squash the single commits into one with a meaningful commit message. Here you can find a tutorial: https://docs.openshift.org/origin-m4/oo_contributors_guide.html#combining-multiple-commits

I am working on this now.

Sorry, I can squash the commits tonight.

It would be nice if you could squash the commits. Then I can merge the pull request.

@anotherhale Any progress on this issue? I currently release 3.0 and this is the only missing part.

I thought that I squashed the commits, but I had not done it before. I followed the instructions from the github docs.  Let me know if you still need me to do more.

I saw that play-yeoman incorporated the fix with 0.8.1. I will try to test with this new release.

I think you've hash squashed the last change with the previous. But now there are still two commits. These both commits should be squashed into one.

I will take a look at it again and get back to you.

I am still troubleshooting this and will squash all commits when I have it totally working.

What doesn't work?

@anotherhale Version 3.0.0 is out. No it's up to you when the release info goes out.
If you do not have the time to polish the pull request, please squash it. That's not a problem. I'll do the rest.

@anotherhale Sorry Andy, but I'll not wait anymore! 3.0.0 is out and this is the only missing part. If you do not squash the commits with a meaningful commit message until tomorrow, I'll change the template by myself. I hope you can understand this.

Sorry I have been busy with work. I have squashed the commits. Let me know if there is anything else you need.

@anotherhale Thanks Andy! Please could you rename the commit message. The changes are for Play 2.4 and not Play 1.4.
You can use:
git commit --amend -m "Play 2.4 and Silhouette 3 changes"
git push --force

Thanks Andy!

@anotherhale Facebook and Google authentication does now work. The only problem is the Twitter authentication. Do you have any idea what's wrong here?

Ah sorry for the typo.    I will take a look at the Twitter Auth too.

@anotherhale No worry! This is already merged and I've renamed the commit message already.
I think the Twitter issue has to do with sahat/satellizer#287 or sahat/satellizer#417. I try to cache the second request. Hope this helps as a workaround until Satellizer is fixed.

Awesome.  Thanks for being patient with me.  I started this project to really force myself to learn Scala and Play.  I had zero experience with Scala or Play 2 months ago.  I learn much better manipulating code rather than reading books.

:+1:


Nanoc watch and autocompile don't pick up changes

On OSX lion, they don't pick up changes, ever. It missed things occasionally in snow leopard, now in lion it never works.

Is this issue still open? I haven’t heard back from the original reporter in a few months so I assume this is no longer the case. Feel free to reopen the issue if the issue persists!


Team update - Nov 23, 2022

This is a @Quansight-Labs/czi-a11y-grant team sync 🎉🎉🎉! This is a way for the Team Members to provide status reports on what they've been up to this week and request help and attention for things they are working on. This issue will be closed at the end of the day.
Copy and paste the template below, and answer questions as you wish!

Response Template
**Thanks I'd like to give 🙌**
- So-and-so helped me out a lot with foo...

**Updates from last week :heavy_check_mark:**
- I worked towards deliverable: <link-to-deliverable>

**Challenges I faced and things I'd like assistance with 🙏**
- I had a hard time figuring out ...

**My availability for next week**
- I'll be off on foo day...

**Important items for discussion 💬**
- I have a question about goal <link-to-goal>
- Can @foo give a comment on issue #NN ?
- I opened #NN for discussion, please chime in
- I need to discuss foo

:mag: Needs Triage
The Needs Triage issues require an initial assessment and labeling.

Tab trab in JupyterLab terminal

:dart: Project boards
Please make sure the boards reflect the current status of the project.
Jupyter a11y grant project

Updates from last week :heavy_check_mark:

I continue working in the hamburger menu - after a talk with Mike he gave me a couple of ideas on how to make the change in Lumino, so now I'm trying that one.

Challenges I faced and things I'd like assistance with 🙏

I'm sick so my brain power for the week has been super low

My availability for next week

I'll be here!

Important items for discussion 💬

Do I need to do something for the jupytercon proposals?

My update for Nov 17-23 is just that I've been heads down working on fixing keyboard navigation in the JupyterLab file browser.


mv: cannot remove '/etc/bind': Directory not empty

pulled latest version using latest version of docker for centos7
run by:
docker run --name bind -d
--publish 53:53/tcp --publish 53:53/udp --publish 10000:10000/tcp
--volume /srv/docker/bind:/data
sameersbn/bind
got this error when starting the container:
mv: cannot remove '/etc/bind': Directory not empty

+1

https://github.com/docker/docker/issues/27214

@littlewitchanita @lvthillo sorry for the lateness of my response. I believe the issue is because you're using  the overlayfs storage driver for the docker daemon along with XFS.
OverlayFS remains a Technology Preview in Red Hat Enterprise Linux 7.2 under most circumstances. Please refer to https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/7/html/7.2_Release_Notes/technology-preview-file_systems.html for additional details and recommended actions to take for getting XFS and OverlayFS to play nicely on these distros.
Closing the issue, please let us know if this helped.


Error : Async callback was not invoked within timeout specified by jasmine.DEFAULT_TIMEOUT_INTERVAL.

I'm using Jest to do unit test based on Jasmine 2
When I was testing my async action store having Parse query
It throws this error :
''Async callback was not invoked within timeout specified by jasmine.DEFAULT_TIMEOUT_INTERVL."
Here is my code from _tests/actions/auth-test.js
it('should creates SIGNUP_REQUEST and SIGNUP_SUCCESS when signup has been done successfully', (done) => {

		/* If this account has already exists in Parse database, it will get SIGNUP_ERROR */

		const expectedActions = [
			{ type: 'SIGNUP_REQUEST', payload: { username, password } },
			{ type: 'SIGNUP_SUCCESS', payload: { user: { /* whatever. Anyway, it needs to be defined */ } } },
		];

		const store = mockStore({}, expectedActions, done);

    	store.dispatch(Auth.signup(username, password));
	})

Finally, I revised some code from yours and tested successfully.
It seems that the main problem is Jasmine done() doesn't be invoked, so it causes this error.
Here is my revised code :
function configureMockStore(middlewares = []) {

	return function mockStore(getState, expectedActions, done) {

		if (!Array.isArray(expectedActions)) {
			throw new Error('expectedActions should be an array of expected actions.');
		}

		if (typeof done !== 'undefined' && typeof done !== 'function') {
			throw new Error('done should either be undefined or function.');
		}

		function mockStoreWithoutMiddleware() {
			return {
				getState() {
					return typeof getState === 'function' ? getState() : getState;
				},

				dispatch(action) {

					const expectedAction = expectedActions.shift();

					/* Customize for auth action test */
					switch(action.type) {

					case 'SIGNUP_REQUEST':
					case 'LOGIN_REQUEST':
						expect(action).toEqual(expectedAction);
						break;
					case 'SIGNUP_SUCCESS':
					case 'LOGIN_SUCCESS':
						expect(action.type).toEqual(expectedAction.type);
						expect(action.payload.user).toBeDefined();
						break;
					case 'SIGNUP_ERROR':
					case 'LOGIN_ERROR':
						expect(action.type).toEqual(expectedAction.type);
						expect(action.payload.error).toBeDefined();
						break;
					}

					if (done && !expectedActions.length) {
						done();
					}

					return action;
				}
			}
		}

		const mockStoreWithMiddleware = applyMiddleware(
			...middlewares
		)(mockStoreWithoutMiddleware);

		return mockStoreWithMiddleware();
	}
}

+1

I have a hard time seeing the difference in the code, could you publish a diff?

There's a new version out, please give it a try :facepunch:


📈 Instrument with fh-component-metrics

@paolobueno Should we merge this back to master, or just use it from the branch, like we do for the demo cloud app? I think this one could be merged more easily, since it doesn't contain the fork of fh-mbaas-api. What do you think?

@grdryn my :+1: on merging this to master, though I think it would require a readme update.
I can see that fh-component-metrics has some relevant info which could be linked to, but atm it only mentions 'RHMAP components' as opposed to a cloud app's context and seems a bit hard for customers to consume.
Do SaaS and on-prem deployments include influx+grafana? If so we could add their addresses\FH env-vars as extra defaults and mention them on the readme for this or fh-component-metrics'.
I'd also like @nialldonnellyfh to take a quick look if possible.

Do SaaS and on-prem deployments include influx+grafana? If so we could add their addresses\FH env-vars as extra defaults and mention them on the readme for this or fh-component-metrics'.

@paolobueno they don't include influx+grafana. @wei-lee is actively working on a statsd backend for fh-component-metrics though, here: https://github.com/feedhenry/fh-component-metrics/pull/6

Rebased on latest master


[Bug] script resultset download 404

Search before asking

[X] I searched the issues and found no similar issues.

Linkis Component
web
What happened + What you expected to happen
After script running JDBC engine, click result set download, and the page prompts 404
http://dssdev01:8089/undefined/#/sheet/add?simpleMode=true&showBottomBar=false&importConfig={"dataSourceConfig":{"dataSourceType":"linkis","dataSourceOptions":{"taskID":138}},%22config%22:{%22myConfig%22:{%22resultSetPath%22:[%22hdfs:///tmp/linkis/hadoop/linkis/20220602_095408/IDE/138/_0.dolphin%22]},%22importConfig%22:{%22mergeTables%22:true,%22limitRows%22:5000,%22pivotTable%22:false,%22tableHeaderRows%22:1}}}&noGoingBack=true
Relevent platform
centos 7.4
apache-linkis-1.0.3
wedatasphere-DataSphereStudio-1.0.1
Reproduction script
show tables
Anything else
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

How does undinfined appear in the url?Looks like a DSS issue


Subpop pref

Update to #230 ahead of base notebook revision.
Creates a rough formulation that measures the interest our sub-population of interest has in repos w.r.t the interest of the general population in the same repos.
Attempting to create a metric that is higher when our sub-population is disproportionately interested in a repo, and the interest is strong (many events) and lower both when the sub-population's interest is reasonable compared to the general popularity of the repo, and when the interest is relatively weak (few events.)

Is identical to #230 until the final markdown cell.
Since #230 hasn't been reviewed/merged, new branch includes more changes to notebook that can be reviewed independently from analysis done in #230.

I'm closing this PR because its contents are in an overleaf document and the notebook has diverged in PR #230 .


use safe class_exists call to prevent issue with magentos classloader

Q
A

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Related tickets
fixes #221

Documentation
-

License
MIT

What's in this PR?
We need to catch exeptions during class_exists, because some classloaders (like magento's) throw an exception if the class does not exist.
Checklist

[x] Updated CHANGELOG.md to describe BC breaks / deprecations | new feature | bugfix

can't get the roave check to work unfortunately. lets hope https://github.com/Nyholm/roave-bc-check-docker/issues/32 eventually gets fixed


Change orbital dumper to use xxhash instead of MD5

Fixed issues with previous pull request.

Thank you for the pull request. I'm really sorry that I didn't give a reply earlier.
Unfortunately, the dumper and its blobs are now deprecated. For the sake of interoperability/consistency with legacy versions I would prefer that the hash function wasn't changed.


circt: fix build with non clang stdenv

lib.optionalString creates an empty string then the condition is false And the circt test framework interprets this as disabling all tests
failing the build with error: filter did not match any tests
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

@ofborg build circt


redundant pages for genotypes, & multiple extensions on gene pages

I've just noticed that there are multiple genotype pages for quite a few genotypes. From the link URLs on reference pages and FYPO terms pages, I gather that this happens when one genotype appears in more than one Canto session. E.g. d2288b20e67eb62d-genotype-4 and d378014af8ea8779-genotype-5 are both rad3delta.
It would be a lot more useful to lump these identical genotypes into one page than to keep them separate - if they're lumped, people can see which phenotypes have been curated from which papers for the genotype without going to multiple pages (and therefore knowing they'd have to look at multiple pages).
I think you should be able to use the genotype description (i.e. allele descriptions + expression levels) to tell when two genotypes from different sessions are really the same genotype.

I think this is because of: pombase/pombase-chado#577

That does look relevant, tho the page issue isn't restricted to deletions. For example, http://pombase2.bioinformatics.nz/genotype/c1136816cdaa2d17-genotype-1 and http://pombase2.bioinformatics.nz/genotype/6998329e8f9ad0bc-genotype-1 are both cdc19-P1 (an allele of mcm2) at the same expression level.

That does look relevant, tho the page issue isn't restricted to deletions.

Thanks for the mcm2 example.  I've updated pombase/pombase-chado#577.

I'll close this since I think it's the same problem as #245.


ECHO queries may return stale data

Likely caused by the root doc not being updated in time.

Dima: It seems like this issue rarely comes up in practice -- only when you have just replicated some data, and then immediately do a query, it's generally fine if the data is stale after replication, ECHO is eventually consistent after all. But that being said, I want to give users more control over the state of the database and data version


[Teams]How can someone in my team run terraform apply from his mac?

[V] ✋ I have searched the open/closed issues and my issue is not listed.

I'm using Teams module to manage our users in EKS. It works fine when accessing the cluster but when someone from my team wants to add a new module to our TF code - it's not possible because of a lack of permissions. I need to run 'terraform apply' instead of them because I created the cluster and their user is inside a role with least privilege access.

The problem occurs with the platform_teams (admin access)
When using Teams module, it creates a new role with Trust relationships to users.
When you create the cluster, your user is the only one that can view/change the cluster without using the administrator role.
If someone else wants to view/change via terraform the cluster - they need to use the role.
But because the role has only access to the EKS cluster - you will get an error when trying to perform 'terraform apply' because you don't have access to KMS (for example).

Provide a link to the example/module related to the question
Teams

Additional context

Hey! The original design for teams was to help easily configuring cluster access to users/teams and not for terraform users who seek to apply changes to the infrastructure outside the cluster.
I see this as a feature requests where we should allow users to pass existing policies to the created platform role, or allow to pass existing platform role as you may end using the same role for creating clusters resources and AWS infrastructure resources  as-well.
We're working on the next design iteration for Teams (see https://github.com/aws-ia/terraform-aws-eks-blueprints/issues/842), I recommend you to share any other feature requests you'd like to have there as-well.
As of now, one of the workaround that you can do is to add the missing policies that you need to the existing role.

Depends on https://github.com/aws/containers-roadmap/issues/185

Closing this out for now. As linked above, EKS is working on improved support for vending access into EKS clusters, and the new teams module is now available here https://github.com/aws-ia/terraform-aws-eks-blueprints-teams - we will continue to support the current implementation here and will provide an upgrade/migration guide before removing it from the project


gst_all_1.gst-plugins-rs: 0.10.8 -> 0.10.9

Description of changes
Minor bump: https://gitlab.freedesktop.org/gstreamer/gst-plugins-rs/-/blob/0.10.9/CHANGELOG.md#0109-2023-06-19
I also cleaned up #238411 to use the list to disable plugins instead of just removing the test (since the plugin likely will crash at runtime as well)
Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin
[x] x86_64-linux -> aarch64-linux (cross)

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

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

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

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

[x] Fits CONTRIBUTING.md.

getting:
         specified: sha256-5YzLEuqVBS16Ifrf63v7zcphVtiHm1iKc/IxplTcps4=
            got:    sha256-FHU1ispIw9gCl1dKnLSm5iF+PfpuNUJeLePpXMOE4tk=

getting:
         specified: sha256-5YzLEuqVBS16Ifrf63v7zcphVtiHm1iKc/IxplTcps4=
            got:    sha256-FHU1ispIw9gCl1dKnLSm5iF+PfpuNUJeLePpXMOE4tk=

Ha, the serialization from tomlq is not stable across versions 🫠. The version bump in #238945 changed the output to start adding needless commas that it did not before
I'll come up with something more stable that modifying the lockfile with tomlq then and push that (probably some basic sed or something)

Alright so that's some ugly sed, but it makes it work with hash stability on all four platform triples I tried ([aarch64, x86_64] x [darwin, linux])
I guess I should bug gst-plugins-rs upstream sooner rather than later to move off the color-name dependency, since the color-name maintainer never replied to my PR that fixes their problem. That's not something that needs to be blocking this bump though

That was push was a rebase for https://github.com/NixOS/nixpkgs/commit/109633bef055415528456848bbbe93fdfdfb73ba and I made the patch from #239681 a revert patch of the commit that broke it (which was itself a revert, instead of the original commit that got reverted), to make it clearer what is going on with that


AttributeError: 'CompletedSubTask' object has no attribute 'aspect_representation'

Hi, I have a following problem: when I start the example you provided, I manage to obtain slack and price sentiments but the problem occurs when I try to create html (html = absa.probing.explain(slack)). This command is followed by the error:
AttributeError                            Traceback (most recent call last)
 in
----> 1 html = absa.probing.explain(slack)
~/anaconda3/envs/torch_tf2/lib/python3.7/site-packages/aspect_based_sentiment_analysis/probing/plots.py in explain(example)
47
48 def explain(example: PredictedExample):
---> 49     aspect = example.aspect_representation
50     texts = [f'Words connected with the "{example.aspect}" aspect: ']
51     texts.extend(highlight_sequence(aspect.tokens, aspect.look_at))
AttributeError: 'CompletedSubTask' object has no attribute 'aspect_representation'

See examples/pattens.ipynb.
Minimum viable example:
import aspect_based_sentiment_analysis as absa
import numpy as np

recognizer = absa.probing.AttentionGradientProduct()
nlp = absa.load('absa/classifier-rest-0.1', pattern_recognizer=recognizer)

text = ("We are great fans of Slack, but we wish the subscriptions "
        "were more accessible to small startups.")

task = nlp(text, aspects=["slack", "price"])

slack_example, price_example = task.batch
slack_scores = np.round(slack_example.scores, decimals=3)
price_scores = np.round(price_example.scores, decimals=3)

print(f'Sentiment for "slack": {repr(slack_example.sentiment)}')
print(f'Scores (neutral/negative/positive): {slack_scores}')

print(f'Sentiment for "slack": {repr(price_example.sentiment)}')
print(f'Scores (neutral/negative/positive): {price_scores}')

html = absa.probing.explain(slack_example)
display(html)

html = absa.probing.explain(price_example)
display(html)

Hey folks! Thank you @visini for your reply.
I have a few things in progress but after that I will update the readme.


Add opportunity to force user of function to handle exception

Something similar to throwable from Java maybe?

I think that's #984.

required onError callback as an alternative, right?

I don't think people manually calling Future.then is going to be the main issue. Most people use await instead. If we never make it work for direct use of futures, just normal functions and awaited async functions, it'll probably be adequate.
And just making it work for synchronous functions is hard enough.


Fix flaky 'should_fetch_from_multiple_peers' test

This PR updates the 'should_fetch_from_multiple_peers' which was failing with CL_TEST_SEED=3a110c8a84699e489929a89b4193ff92 due to too many transactions of a given category being created.
The solution was to modify chainspec for this test to bump the limits.

bors r+

bors r+


textfield

A project requires a variety of textfiled styles

@herowws A new version v3..0.1 has been released with these features implemented, Kindly check


Run CI build and tests on macOs execution environment

Reasoning
There are some differences between running IDE with the plugin on Linux vs running on macOs. Therefore, adding proper tests that would run on macOs in the CI jobs could help us detect potential issues that could appear only for macOs.
Acceptance criteria
In addition to current CI jobs, run build- and test- related jobs on macOs execution environment. Probably, we could use this: https://circleci.com/docs/using-macos
To be decided
Before completing the task, we would have to decide, which jobs/gradle tasks in particular should run on macOs and which not.

Might be hard to run UI tests (which would have the highest value) from Mac machines in the CI... see https://serverfault.com/questions/239075/xvfb-alternative-for-mac-server (xQuartz?)


Restore the 'dev12 tests' trigger. The requirement of the word tests serves as a disambiguator and calling out 'dev12' is more natural than 'legacy'.

This change is

@mmitche FYI


Add API helpers in SDK to deprecate bot-helpers in core

The current bot-helpers package in Rocket.Chat core was an experiment in accessing user collections for bot scripts. It hasn't been widely implemented but had some good concepts, like helper methods to get all onlineUsers and other ways of querying subsets of users. The admin setting to control what user fields bot users have access to is also good for security and privacy.
However, it ultimately feels like a double-entry to the more complete and maintained features of the standard rest API. It makes more sense to me to just let bots call the API. If extra requests are needed, they should be addressed as new endpoints, instead of having competing approaches for requesting data.
This issue would propose adding a module to the SDK that provides bot adapters easy access to calling API endpoints using the bot user's credentials. It would handle the login and auth for requests, with a simple interface for either .get or .post requests and helpers for some common endpoints and filtering results however is most frequently needed by bot scripts (e.g. to replicate the onlineUsers request).
The difficult part would be replacing the existing field level access for API results. I don't know if the API can be modified to filter results based on the requesting user having a bot role, but that would be nice to have.

I've now added some of the bot-helpers methods to be accessed from api.users, see the demo script here: https://github.com/RocketChat/Rocket.Chat.js.SDK/blob/master/src/utils/users.ts
Run the above via ts-node src/utils/users.ts to get the output of all user queries.
However, I found that Rocket.Chat does not return the user's role from the bot's API queries.
See https://github.com/RocketChat/Rocket.Chat/issues/10688
So some of the functionality from bot-helpers cannot be provided in the SDK alone at this time.
I don't want to encourage calling database queries directly via DDP methods and would like the SDK api helpers that I've started with this commit to eventually replace the need for bot-helpers, but it will take some work to replicate them all and will probably involve PRs to Rocket.Chat core.
See here for the full set of functionality (to eventually be ported into the SDK as API helpers):
https://github.com/RocketChat/Rocket.Chat/blob/develop/packages/rocketchat-bot-helpers/server/index.js


BUG on TOOL run (Boundaries)

The tool should show the number of admin regions upon processing, not sure what this huge number is. See below original tool. \
(Also, the entire country is a result, the spatial filtering doesnt work anymore.)

It seems that the  selection does not apply to the Zone Score, as it should - but only to the suitable areas;
The zone score is computed in 25km2 grid - please correct and make sure this is reflected properly also in the exported outputs

I fixed this and it the app is working on staging without this bug

Perhaps this image is better suited here; it seems there is still a bug with this feature, please see below.
I was testing another ticket for Aruba, and the below happened: I got an error message of NO Zones even if I have suitable areas in the country.
If you want to try to replicate the error:
select Aruba, Solar and country borders -> run tool -> suitable areas and 1 zone appears
change selection to 25km2 grid and re-run tool -> this happens

Hi @ClaraIV and @NEDJIMAbelgacem
I checked this and found that there is no data in case of ARUBA country, I checked for AFGHANISTAN with the same paraemters.(You can check the data using the below links.)
https://gre.staging.derilinx.com/public/zones/country/AFG.topojson
https://gre.staging.derilinx.com/public/zones/country/ABW.topojson

Hi @ClaraIV, what do you think about this ticket? I think it's more of a data issue rather than a bug.

hi @dimasciput , why do you think it is a data issue?
I checked and saw that all the layers have data for Aruba, you can double-check that in the tool itself.
It is surely a bug, regardless of data/no-data because if there would be no data, then there would be no resulting zones; the tool returns both results (the pink areas) and the warning sign that there is no data, therefore this needs to be investigated and fixed.

Hi @ClaraIV , thank you for the input, we will investigate this further.

The issue of no zone being generated is under #70. The bug on tool run issue is resolved and the count that happens when the tool is ran reflects the admin regions count


Ignoring formatting in (comment) forms

The formatting (especially spacing between top level forms) have meaning to me in a (comment) form. zprint right now smushes all the forms together.
Rest of the file I don't care. I would like to keep this manual formatting:
(comment

  (def burst-segment {:body " 0,PT2,17.61,18.3000"})

  (make-sensordata-from-burst-csv burst-segment)
  (make-sensordata-from-burst-string (make-sensordata-from-burst-csv burst-segment))

  (start-csv-dump-of-burst!)
  (start-csv-dump-of-interval!)

  ;; fin
)

I can add ;!zprint {:format :off} to every comment form, and I would like to be able to globally do it

Thanks for the clear explanation.  I understand what you want and why you want it.  I will work on making that possible.  I have an idea of how it might work out, but I need to do a little research to see how well that idea will go together.  But it seems like a good idea, and if people can turn off formatting for comment, they could turn off formatting for other things too, and that would probably be useful well beyond your particular use case -- which is always a nice plus.

I've been thinking about how to do this.  Some of the ideas:

simply not format anything in comment forms.
respect newlines at all levels in comment forms, but otherwise format things correctly (which would mean indenting them correctly, but not shifting lines)
respect newlines at the top level of comment forms, but do regular formatting of anything below the first level

Do any of these sound better or worse to you?  I'm not sure which ones would work out in a reasonable way.  I'm leaning toward trying to do 2, and then see if it is straightforward to add on 3.  But I'm interested in your opinion, since this was your idea.  Thanks.

2 is interesting to me. And 2 in other places will be interesting to me too.
Sometimes I do like to create space around forms in a defn, for example, to "punctuate" the flow of thoughts.
(it's usually an indication that the defn is too long, but not always)

You can now get exactly what you want in version 0.5.0.  See the section: "Altering the formatting inside of certain functions" in the README, and in that section you will see an example which will do precisely what you want -- keep newlines at the top level of a comment function, but format everything down one level from that normally.


Check if the asterisk on the path exists

Situation
Check that if users provide the storage path without asterisk still return files, or throw an exception with descriptive message.
Acceptance Criteria

Refactored metadata reader to check that returned files have proper extensions
Added integration tests

We decided not to throw an exception if a user does not provide an asterisk at the end of the bucket path. Instead, we try to list the files from the bucket path.
An exception will be thrown if bucket path itself does not exist.


fix start lower-bound for transaction filtering

Issue Number

Overview

[ ] I have adjusted the start time to make sure that we don't yield transactions that are after the given time because of the conversion between UTCTime and slot times.

Comments

Note that this makes the start range completely exclusive. We could make it inclusive by adding only slotLength - 1 I believe :thinking:

Note that this makes the start range completely exclusive. We could make it inclusive by adding only slotLength - 1 I believe thinking

After slack discussion it's been made inclusive.
I have tested it on byron testnet with http-bridge.
for tx inserted at time = 2019-08-06T09:36:16Z
--start 2019-08-06T09:36:15Z - is showing tx
--start 2019-08-06T09:36:16Z - is showing tx
--start 2019-08-06T09:36:17Z - is NOT showing tx


udev rules are not correct

On boot, I see the following errors in the logs:
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/10-switch.rules:1 Unknown group 'nintendo_switch', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /etc/udev/rules.d/51-android.rules:206 Invalid key/value pair, ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /etc/udev/rules.d/51-android.rules:207 Invalid key/value pair, ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /etc/udev/rules.d/51-android.rules:517 Invalid key/value pair, ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /etc/udev/rules.d/51-android.rules:518 Invalid key/value pair, ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /etc/udev/rules.d/51-android.rules:521 Invalid key/value pair, ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /etc/udev/rules.d/51-android.rules:157 GOTO="go_adb" has no matching label, ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /etc/udev/rules.d/51-android.rules:158 GOTO="go_adbrndis" has no matching label, ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /etc/udev/rules.d/51-android.rules:159 GOTO="go_adbmtp" has no matching label, ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /etc/udev/rules.d/51-android.rules:160 GOTO="go_adbptp" has no matching label, ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /etc/udev/rules.d/51-android.rules:161 GOTO="go_adbmidi" has no matching label, ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /etc/udev/rules.d/51-android.rules:162 GOTO="go_adb" has no matching label, ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:20 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:23 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:26 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:29 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:32 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:35 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:38 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:41 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:44 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:47 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:50 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:53 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:56 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:59 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:62 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:63 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:66 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:69 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:72 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:75 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:78 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:81 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:84 Unknown group 'plugdev', ignoring.
Oct 20 10:27:01 systemd-udevd[910]: /usr/lib/udev/rules.d/70-u2f.rules:87 Unknown group 'plugdev', ignoring.

As far as I can tell there are 2 issues here.

It looks like other ublue images have/are creating a plugdev group.
Based on this discussion it seems the android logs should have already been fixed upstream a few months back, but perhaps wayblue is pulling the incorrect package?

For some reason, older udev rule is still pulled from Ublue, so I use this workaround to fix it.
https://github.com/fiftydinar/gidro-os/blob/main/files/scripts/android-udev-rule-workaround.sh

@fiftydinar where is the newer package? I'm pulling from the ublue repo

@fiftydinar where is the newer package? I'm pulling from the ublue repo

There is no newer package. The issue is that Ublue's rebuild in COPR somehow didn't include the latest udev rule.
Original spec file is here: https://github.com/M0Rf30/android-udev-rules/blob/main/rpm/android-udev-rules.spec
Ublue just forked this spec without changes to rebuild in staging COPR repo

oh I see. damn
secureblue needs a fix for this too then

Thanks, @sgallag-insta do you mind also reporting this to ublue?

It seems like the ublue/staging repo doesn't really have a ton of issues filed against it, so I'm not sure the best place to bring it up. However, I've posted in their discord about it

After some back and forth with the ublue maintainers I can confirm that their latest build does include the fix for this issue.
I think we must be calling the wrong URL or something.

@sgallag-insta the old url looks correct:
curl -Lo /etc/yum.repos.d/_copr_ublue-os_staging.repo https://copr.fedorainfracloud.org/coprs/ublue-os/staging/repo/fedora-"${OS_VERSION}"/ublue-os-staging-fedora-"${OS_VERSION}".repo

this matches:
https://copr.fedorainfracloud.org/coprs/ublue-os/staging/repo/fedora-40/ublue-os-staging-fedora-40.repo
which i directly copied from the copr link

@qoijjj Yes, I've spend some more time looking at this but I don't really understand what the problem is.

The newest build has the correct file.
The staging repo seemingly has the latest build (although it does have older builds too.)
As you've said, the URL seems correct.

For some reason we are seemingly not getting the latest build from the repo. I've also taken a quick look at what bluefin is doing and it doesn't seem significantly different as far as I can tell :confused:

@sgallag-insta
ublue-main logs:
android-udev-rules-20230310-1.fc41.noarch (copr:copr.fedorainfracloud.org:ublue-os:staging)

secureblue logs:
android-udev-rules-20230310-1.fc41.noarch (copr:copr.fedorainfracloud.org:ublue-os:staging)

what the hell? :|

@sgallag-insta I have a theory.
I think it has something to do with the fact that they're not versioning the spec whatsoever, which means the rpm that gets pulled is arbitrary between one of these four iterations?

https://github.com/M0Rf30/android-udev-rules/pull/333

@qoijjj Yeah, I'm also sus of the whole version number not changing thing. Wouldn't surprise me if it was random (or  always earliest or something)

I submitted that PR upstream of upstream, once it's merged we can go back to the rpm

We'll also need to make sure it makes it's way into ublue's fork
https://github.com/ublue-os/android-udev-rules

https://github.com/ublue-os/android-udev-rules/pull/9

merged upstream, reverting the direct pull
https://github.com/wayblueorg/wayblue/commit/5eeef630024ebf9ffb0bcc8f1d046a2ffbb0ce5f


How to use configd.py or the backend registry/database to add elements using a python script and without writing xml

Hi,
so i wanted to recreate the module prototypes using python classes and the provided models in /src/opnsense/mvc/app/models/OPNsense/. so i wrote myself a parser that creates classes from the prototypes.
But then i notices that elements dont get placed in the same field in /conf/config.xml
For example i cant add Firewall lan rules like i add snat rules, but the prototypes are both in the same xml file.
Then i thought: ok never mind, i just can search the parent element, but this wont work either cause the structure of the containers also differs.
Example Filter.xml
this is the model  file
<model>
    <mount>//OPNsense/Firewall/Filter</mount>
    <version>1.0.4</version>
    <migration_prefix>MFP</migration_prefix>
    <description>OPNsense firewall filter rules</description>
    <items>
        <rules>
            <rule type=".\FilterRuleField">
 ...

theres rules, snatrules and a bunch of others

in the config.xml the filter rules are defined as this:

  <filter>
   <rule>
     <type>pass</type>
     <ipprotocol>inet</ipprotocol>
     ...

but snatrules are under:
 <OPNsense>
    <Firewall>
      <Filter version="1.0.4">
        <rules />
        <snatrules />

in the config.xml i can only apply snat rules via OPNsense.Firewall.Filter and rules only under filter.rules.
Is there any other way i can find all the attributes for the wrapper-classes i want to create, other than scanning the php-files, or adding every module-elemt at least a single time via the web gui?

I am not really sure i f i should use the php scripts rather than the daemon or something else.
I just dont know if configd is having  a in memory representation of the config.xml.
so it wouldbe cool if i could add the objects directly to the registry, rather than dumping xmls. for example like this:
daemon= configd_daemoin
config=daemon.get_config()
config["Fiter"].append(New Filter_Rule())
daemon.apply(config)
daemon.verify()
daemon.save()

Mhh, I don't think we should support writing confg.xml directly from configd -- it has read access for templating, but the rest is better done via PHP as it always has been.
Cheers,
Franco

Mhh, I don't think we should support writing confg.xml directly from configd -- it has read access for templating, but the rest is better done via PHP as it always has been.
Cheers, Franco

hey Franco,
so i think i know now how to approach this.
So i guess i can create a new Filter like this:
$loader->register();
use OPNsense\Firewall\Filter;

$filter = new Filter();
$filter->rules = new Rules();
$filter->snatrules = new SnatRules();
$filter->npt = new Npt();
$filter->onetoone = new OneToOne();

and then maybe use ApiMutableModelControllerBase.php to to validate and save.
Will see if that works.
Thanks for your reply and greetings:
ji

Can you start from the top: what do you want do do besides saving firewall rules that the API can already handle?

Can you start from the top: what do you want do do besides saving firewall rules that the API can already handle?

sure, i want to do some stuff that i could not do with the api, like setting physical interfaces.
I already managed to do this using the  config.xml and the script that reconfigures interfaces.
The reason why i needed this was that i wrote a ansible collection that creates ovs-bridges, configures libvirt and sets up docker networks  to isolate vm and container traffic.
Next i wanted to automate the firewall and routing for those networks using ansible.
I created wrapper classes for all the opnsense scripts you can call and provided  autodoc api docs for them. i recreated the argParse objects for those scripts to fetch all the defaults and options, etc.
Next i thought: why dont i just look for a way to create those python wrapper classes generically.
And thats where i am now. I created a parser for the xmls in  opnsense/mvc/app/models, to create python classes for this using the defaults and datatypes.
Now i just need a way to pass the data from the python objects to the php data store.
The cool thing is that i can create autodocs for all the required arguments and methods and you have the descriptions directly in the ide .

Do you mean you need an API to interface settings and assignments? That's one large chunk of work that still needs to be done eventually, but it's not in the current roadmap scope. You are better off templating XML snippets for your needs.

If you need to update legacy sections, you can use the config object, for example to update an existing interface:
require_once("script/load_phalcon.php");
$cfg = OPNsense\Core\Config::getInstance()->object();
$cfg->interfaces->wan->if = 'vnet0';
OPNsense\Core\Config::getInstance()->save(
        [
            'username' => 'root',
            'time' => microtime(true),
            'description' => 'my custom update'
        ]
    );

object() returns a SimpleXML object (https://www.php.net/manual/en/book.simplexml.php)

ok thank you both for your feedback!!
Ill close this now and check out the provided snippet!
Love from germany!


Update UDP docs point splitting

Maybe just start linking right to the UDP readme like we do with the other service plugins - also make it so that the sidebar links right to GitHub instead of having to click the sidebar link, read the short blurb about having to go to GitHub, and then click the GitHub link.

I like this idea. Let's make UDP just like the others, and let the docs in the repo be the source of truth.


graphql-transport-ws: Do not send "complete" after "error".

Description
The graphql-transport-ws protocol specifies that an error message should be the final
message for an id coming from the server.  There were, however cases in strawberry where an
extra "complete" would be sent in violation of the protocol.
This PR cretates an Operation object which represents the state of an individual operation and ensures that
a "complete" or "error" message are the final messages for an active operation.

Types of Changes

[x] Core
[x] Bugfix
[ ] New feature
[ ] Enhancement/optimization
[ ] Documentation

Issues Fixed or Closed by This PR

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

Hi, thanks for contributing to Strawberry 🍓!
We noticed that this PR is missing a RELEASE.md file. We use that to automatically do releases here on GitHub and, most importantly, to PyPI!
So as soon as this PR is merged, a release will be made 🚀.
Here's an example of RELEASE.md:
Release type: patch

Description of the changes, ideally with some examples, if adding a new feature.

Release type can be one of patch, minor or major. We use semver, so make sure to pick the appropriate type. If in doubt feel free to ask :)
Here's the tweet text:
🆕 Release (next) is out! Thanks to @kristjanvalur for the PR 👏

Get it here 👉 https://github.com/strawberry-graphql/strawberry/releases/tag/(next)

I intentionally didn't tack on some more refactoring which I have in mind for this code, to keep this PR as limited as possible.  But I have in preparation a few PRs which address subtle problems in the protocol, which will benefit from further refactoring.

Also, please see my discussion #2610
A lot of the unittests for the websocket protocol have nothing to do with the integration, and are just looking at
protocol behaviour, i.e. the handler after a websocket has been established.  For those cases, I would prefer if we just added them to the aiohttp integration folder, rather than duplicate them to all three.


Make Node#ping work for all lavalink versions

Use the original lavalink ws message "Unknown op" to calculate websocket latency, making Node#ping method work for all lavalink versions.

Original lavalink version does not send any ws message on unknown op code.


It's a platformer and maze following a little blob.

How do you play your game?
WASD to move, j to restart, i to skip a level. Reach the very right side of the screen to progress to the next level.

Code
Check off the items that are true.

[x] The game was made using the Sprig editor.
[x] The game is placed in the in the /games directory.
[x] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[x] The game runs without errors.
-There is an error when you beat or lose the game, but it does not affect playability
[X] The name of the file/game contains only alphanumeric characters, -s, or _s.
[X] The game name is not the same as the others from gallery

Image (If an image is used)

[ ] The image is in the /games/img directory.
[ ] The name of the image matches the name of your file.

Thanks for your PR!
I triple-dog dare the reviewer to beat the game in under 10 minutes

I first heard about it at the Hack OC hackathon, and later me and my friend started planning a hack club for our school, I learned more about it

not really, just inspired by other platformers i guess

Right:) What about increasing gravity delay?

I think some levels might become easier than I would like if I increased the gravity delay...

Hmmm I understand but my fear is that it is impossible on the console, but if you really want to keep, lmk

lets keep it for now. If it gets to difficult on the console, i can change it

Did you manage to win the game?

i've won twice, took about 30 minutes each time

My apologies for the delay, could you wait until tonight? I'm travelling, but I'm back this night and want to play your game with more time:)

sure!

just beat it with a new record of 8 minutes and 55 seconds!

Anyway to play my game on mobile?

I'm kinda in new york and don't have my laptop

yes, just open this link https://sprig.hackclub.com/share/GQYbBKmcgTue2ZSWNJ2w :)

I can't seem to get to the win stage on mobile. It was working on my computer though

I can wait until you get your computer:)

I think I found the problem. I think I might  have uploaded the wrong version of the game. I added a commit that should fix it

Can you send the updated share link? Or is it thr same one

Here is: https://sprig.hackclub.com/share/grAWI0fWxvrAKkN3bfzQ! btw - merry christmas!

sorry for the late response.  Here are some screenshots of the end.

If you think it's still to difficult, I can make some changes to the map to make it easier.

Tysm!


Finger algorithm notes

Link to papers: https://docs.google.com/document/d/1izom-f-79Li8H_0Hno6znYwcRqMsLadul3jDyxz0ac4/edit#
Essentially, we have two options: build a genetic algorithm or build a graph traversal algorithm (explained below)
Graph traversal algorithm
Input: A trellis (basically a map of nodes arranged in layers by time)

Each layer represents one note on or note off event
Simultaneous patterns of notes can be represented as chords
All the possible fingerings are represented and connected in each layer to the next fingerings in the next layer

Cost function: basically a function that assigns a cost based on how easy each fingering is to play

low cost if normal amount of finger spread
assign penalties for crossover

Goal: find the minimum possible cost path between the layers of the trellis

Multiple algorithms to do this, ex. Viterbi algorithm

New algorithm outline: https://docs.google.com/document/d/1lD8waarweIsPl4iwtFIJlLRSgWqLsiSdiGxPiHmR4F8/edit

https://github.com/TPNxl/viterbiTest

https://arxiv.org/abs/1607.04229 - speeding up Viterbi is hard


Overflow of navigation bar

Hello, it's me again ;)
The navbar in the header overflows on mobile devices, instead of getting hidden. I don't think that's the desired behaviour.
This also moves the hamburger menu out of view, so you are entirely unable to navigate.

Hi there, I will take a look at it asap!

Finally fixed in the latest version ...
🙂


"Application not properly entitled for push notifications" error on APN token register

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project uses.
[X] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Electron Forge version
6.0.4
Electron version
21.1.1
Operating system
macOS 12.6
Last known working Electron Forge version
6.0.0-beta.66
Expected behavior
Apple Push Notification feature should work after the app is built
Actual behavior
Receiving this error when the app try to register push notifications token:
(Error: 13 NSOSStatusErrorDomain {\n    NSDebugDescription = \"Application not properly entitled for push notifications.\";\n})."}

Steps to reproduce
  const { pushNotifications } = require('electron')
  pushNotifications.registerForAPNSNotifications()

Additional information
This is the package.json working:
    ...
    "@electron-forge/cli": "6.0.0-beta.66",
    "@electron-forge/maker-deb": "6.0.0-beta.66",
    "@electron-forge/maker-dmg": "6.0.0-beta.67",
    "@electron-forge/maker-rpm": "6.0.0-beta.65",
    "@electron-forge/maker-squirrel": "6.0.0-beta.65",
    "@electron-forge/maker-zip": "6.0.0-beta.66",
    ...

@hectorneyra what did your entitlements look like for the working release, and what do they look like now?

Also as an aside: all your Forge package versions should match with the same number, as we publish all packages simultaneously.
For your non-working version, are all packages at 6.0.4?

Our entitlements look the same as they were before the forge version upgrade, we only updated the forge version and them APN stop working.

For your non-working version, are all packages at 6.0.4?

yes, they are all 6.0.4

For your non-working version, are all packages at 6.0.4?

Yes, they are all 6.0.4

@hectorneyra https://github.com/electron/forge/releases/tag/v6.0.0-beta.68 included a breaking change to the package for macOS code signing.
The osxSign parameters now work pretty differently and come with a lot more things by default. Can you see the MIGRATION.md to see if the 1:1 migration pattern works for you?

That'd makes sense,
We'll tweak our config according to the migration document and will come back here to share how it turned out. Thanks for sharing it.

@erickzhao it works now, thanks you!