Fix GUI bugs and enhance GUI

Enhanced summary UI
Enhanced result display UI
Fixed text overflow issue
Implemented tag length restriction (max 50 characters allowed)

Closes https://github.com/AY2122S1-CS2103-T14-1/tp/issues/150
Closes https://github.com/AY2122S1-CS2103-T14-1/tp/issues/159
Closes https://github.com/AY2122S1-CS2103-T14-1/tp/issues/165
Closes https://github.com/AY2122S1-CS2103-T14-1/tp/issues/111

Codecov Report

Merging #194 (879e874) into master (05d3095) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #194      +/-   ##
============================================
- Coverage     70.73%   70.68%   -0.06%
- Complexity      628      629       +1
============================================
  Files            90       90
  Lines          1958     1958
  Branches        235      235
============================================
- Hits           1385     1384       -1
  Misses          482      482
- Partials         91       92       +1

Impacted Files
Coverage Δ

...address/model/healthcondition/HealthCondition.java
70.00% <0.00%> (-10.00%)
:arrow_down:

Continue to review full report at Codecov.

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


Typo in no-static-element-interactions doc

thanks!
copy and paste 😢

Coverage remained the same at 99.322% when pulling 5cb33472b5b510487e4c0f05cfef99b51d11ad9a on EvNaverniouk:patch-1 into 73be7a8bcb024c81da8924f4b9bc0a66c6ee4131 on evcohen:master.


Scroll Inertia Broken by MaterialTheme

Usage Information
v3.4.3 / 11.2.999-cibuild0046730-alpha / iOS 17.4.1 (probably Android as well)
Description
When using the MaterialTheme in my Avalonia app, it breaks scroll inertia for all ScrollViewers (including things like ListBox)
If I use the FluentTheme however, it works fine
Example app:
ScrollViewerInertiaTest.zip
Reproduction Steps
Open the included test app and swipe/flick up and down
Expected Behavior
Scrolling should have inertia when I release touch
Actual Behavior
When using MaterialTheme, there is no inertia. The scroll stops dead upon release of touch. Switching out the themes in App.axaml resolves the issue
Regression?
No response
Known Workarounds
No response
Anything else?
No response
Could you help with a pull-request?
No
Is there an existing issue for this?

[X] I have searched the existing issues

I'll take a look


fix: add escape command

so its easy to take any YAML which includes {{ or }} (typically in sample ConfigMap data) and convert them so they can be packaged as helm charts

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign jstrachan
You can assign the PR to them by writing /assign @jstrachan in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jstrachan
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jstrachan]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


improve package gredis

Codecov Report
Base: 75.96% // Head: 75.80% // Decreases project coverage by -0.15% :warning:

Coverage data is based on head (6439ee1) compared to base (d5d9896).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@                    Coverage Diff                    @@
##           feature/v2.2.0-gredis    #2162      +/-   ##
=========================================================
- Coverage                  75.96%   75.80%   -0.16%
=========================================================
  Files                        588      585       -3
  Lines                      48377    48383       +6
=========================================================
- Hits                       36749    36677      -72
- Misses                      9625     9706      +81
+ Partials                    2003     2000       -3

Flag
Coverage Δ

go-1.15-386
?

go-1.15-amd64
?

go-1.16-386
?

go-1.16-amd64
?

go-1.17-386
?

go-1.17-amd64
75.80% <0.00%> (-0.15%)
:arrow_down:

go-1.18-386
?

go-1.18-amd64
?

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

 Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Pass the entire controller to its internal processor for easier access of non-serializable states

we decide not to pass the entire controller.


I can't connect images

Hello, I ran into a problem,I connect the image in this way <li style= "background-image: url( {{ record.imagefon|media }});" > but still gives an error 404 file not found although I go to the path that is specified it is there.I'm new to using October so I'm asking for help.

GitHub issues really isn't the place for questions like this. Please use one of the many other avenues available instead of submitting new issues for your questions.
Forum: https://octobercms.com/forum
StackOverflow: https://stackoverflow.com/questions/tagged/octobercms
IRC: #october on Freenode or https://octobercms.com/chat
Slack: https://octobercms-slack.herokuapp.com/
Discord: https://discord.gg/6S777kh

GitHub issues really isn't the place for questions like this. Please use one of the many other avenues available instead of submitting new issues for your questions.
Forum: https://octobercms.com/forum
StackOverflow: https://stackoverflow.com/questions/tagged/octobercms
IRC: #october on Freenode or https://octobercms.com/chat
Slack: https://octobercms-slack.herokuapp.com/
Discord: https://discord.gg/6S777kh


Help

Added a help function.

Thanks for this. But I think #38 will make this PR obsolete, because all commands will be self-documenting.
Feel free to participate in improving the usage examples, etc. in the CLI.


Export SecretStoreEntry class

Fixes #11 This PR exports SecretStoreEntry class which corresponds to the actual js-compute implementation.

This is just a little change so I will merge this


Our Architecture Diagrams are not being displayed

Our NPM registry entries include an architecture diagram like other constructs in the hub.
But our Construct Hub entries don't display the diagram.
Is there some setting we are missing that will cause our diagrams to be imported?
Thanks

Looks like an issue with the conversion to absolute urls https://github.com/cdklabs/construct-hub-webapp/blob/main/src/components/Markdown/Markdown.tsx#L96
Will check this out.

Keep


fix(TYP-2386): Pass hidden fields via fragment

Report: TYP-2386
Bug introduced in DIST-241 by changing passing of hidden fields from query param to fragment.

[BOT] Preview available: https://embed.typeform.com/5c3b9aa9ee623fc101b5c3cad7c0c4ceeac74f77/embed.js

:tada: This PR is included in version 0.22.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Can you add a parameter to determine whether gitlab releases are created or not

Such as github changesets actions https://github.com/changesets/action , It has a parameter like createGithubReleases - A boolean value to indicate whether to create Github releases after publish or not. Default to true
Thanks!

PR welcome.


test(@toss/react): add convertNewlineToJSX test code

Overview

03e62e9 add convertNewlineToJSX test code

PR Checklist

[x] I read and included theses actions below

I have read the Contributing Guide
I have written documents and tests, if needed.

The commit history is too dirty, so I created a new pr


Support hash/comparison operations on var_list and fixed_list

Hash/comparison operations are currently not supported on var_list and fixed_list datatype, which means we are not able to group by/compare a var_list and fixed_list.
For example, we should support such query (group by var_list which computes hash on var_list):
match (p:person) return p.workedHours, count(p.ID)

Similarly we need to support hashing over nested data types. This issue, which asks for a better error message should be resolved automatically once we support hashing over nested data types: https://github.com/kuzudb/kuzu/issues/2637

Should be done in #3174


ActiveSupport::Concern coverage

Hello,
    require 'simplecov'
    SimpleCov.use_merging false
    SimpleCov.start 'rails'

I have this structure:
app/models/concerns/api/account.rb
Added via application.rb:
config.autoload_paths += ["#{config.root}/app/models/concerns", "#{config.root}/app/sweepers)"]

My model app/models/account.rb
class Account < ActiveRecord::Base
  include Api::Account
end

The coverage report does not include app/models/concerns/api/account.rb at all.
I tried to add_group, no dice.

I can confirm I am experiencing this as well, in 0.10.0
My concerns are not tested for coverage, even if its test is in models/concerns
(I cant work with any of the 0.11.x versions, as it generates reduced coverage with the blank line issues and such).

@dankohn please open a new issue describing your version, your test suite, how your running your tests, your gem statement in your gemfile, and where you require/configure simplecov (and reference this issue). Thanks!


Update simulate-parachains.md

Update for PBA!

@lisa-parity this is bumping the version to 0.9.37 and has minor updates to the zombienet config.

@lisa-parity good time to update all parachain content, as the present docs are quite out of date. Should only be minor things to tweak throughout.

Sorry if it was not clear, i think this info update does not work with the other parachain content using 0.9.32. as the versions are too old. Is that the case @brunopgalvao ?
If so this pr should be reverted imho, because its a footgun for users to fight with version mismatches and usually they don't find that is the problem without substantial headache first.


How does it work in Gridsome?

This is the tracking code script:
<script async defer data-website-id="xxx-yyy-zzz" src="http://localhost:3000/umami.js" />
This is what I've tried in my 'Index.vue' file in Gridsome:
<script>
export default {
  metaInfo: {
    title: "Home",
    script: [{ async: true, defer: true, id: 'xxx-yyy-zzz', src: "http://localhost:3000/umami.js" }]
  },
};
</script>
It does NOT work.
How do we change attribute 'id' to 'data-website-id'?
Is this even the right way?

Have you tried 'data-website-id': 'xxx-yyy-zzz' with the quotes?

That's it. It works!


share_util: Remove usages of deprecated API

Do you plan to

Please, continue

Please, continue

No.


error creating an instance of haxilium without player attribute

if you don't specify the player attribute in the haxilium it throws following error:
TypeError: config.player is undefined
uncaught exception: null

Thanks for reporting. It is fixed now. Also, I have changed config format too.
Old format:
const room = haxilium({
  roomName: 'Room',
  playerName: 'Bot',
  maxPlayers: 12,
  player: {
    fields: {
      // Player's properties goes here
      afk: false
    },
    rights(player) { return [0] },
    filter(player, opts) { return true }
  }
})

New format:
const room = haxilium({
  roomName: 'Room',
  playerName: 'Bot',
  maxPlayers: 12,
  player: {
    // Player's properties goes here
    afk: false
  },
  playerRights(player) { return [0] },
  playerFilter(player, opts) { return true }
})


Disconnect when returning to the foreground

Entering the background and returning to the foreground will disconnect, but it will not disconnect in version 3.1.1. Is there any way to stay connected?
Receive failed with error "Socket is not connected"
nw_read_request_report [C1] Receive failed with error "Socket is not connected"
websocket encountered an error: The operation couldn’t be completed. (Network.NWError error 0.)
Received data: cancelled

facing the same Error, downgrading to 3.1.1 resolved the problem for me too

face the same issue
Starscream version: 4.0.4
iOS 14.2
Xcode 12.2

same issue here
Starscream version: 4.0.4
iOS 14.4
Xcode 12.4

same issue
Starscream version: 4.0.4
iOS 14.4
Xcode 12.4

Facing the same issue... please let us know if this is fixed in newer version. Thanks

Same issue. @xjimi Could you tell me how to fix this issue?


QTimer may cause potential crash

Confirm if created and use on different thread.

The QTimer seems fine...


Panic when uploading to BackBlaze B2

Output of restic version
restic 0.8.0
compiled with go1.9.2 on linux/amd64

How did you run restic exactly?
First config:
#!/bin/sh

export B2_ACCOUNT_ID=<snip>
export B2_ACCOUNT_KEY=<snip>
export RESTIC_REPOSITORY=b2:<snip>:/
export RESTIC_PASSWORD=<snip>

restic backup -x /data

Second config:
#!/bin/sh

export B2_ACCOUNT_ID=<snip>
export B2_ACCOUNT_KEY=<snip>
export RESTIC_REPOSITORY=b2:<snip>:/
export RESTIC_PASSWORD=<snip>

restic backup -x /

What backend/server/service did you use to store the repository?
BackBlaze B2
Expected behavior
The backup is executed correctly, with no errors.
Actual behavior
Crash related to first config:
unable to open cache: unable to locate cache directory (XDG_CACHE_HOME and HOME unset)
scan [/data]
[0:19] 23530 directories, 71215 files, 104.304 GiB
scanned 23530 directories, 71215 files in 0:19
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x3a pc=0x429276]
goroutine 77028 [running]:
internal/poll.runtime_pollWait(0x7f55d654c430, 0x72, 0x0)
        /usr/lib64/go/src/runtime/netpoll.go:173 +0x57
internal/poll.(*pollDesc).wait(0xc4222a9018, 0x72, 0xffffffffffffff00, 0xe48900, 0xe436c0)
        /usr/lib64/go/src/internal/poll/fd_poll_runtime.go:85 +0xae
internal/poll.(*pollDesc).waitRead(0xc4222a9018, 0xc422b09600, 0x200, 0x200)
        /usr/lib64/go/src/internal/poll/fd_poll_runtime.go:90 +0x3d
internal/poll.(*FD).Read(0xc4222a9000, 0xc422b09600, 0x200, 0x200, 0x0, 0x0, 0x0)
        /usr/lib64/go/src/internal/poll/fd_unix.go:126 +0x18a
net.(*netFD).Read(0xc4222a9000, 0xc422b09600, 0x200, 0x200, 0x2f, 0xc420168bf0, 0x4438e7)
        /usr/lib64/go/src/net/fd_unix.go:202 +0x52
net.(*conn).Read(0xc42000e1f0, 0xc422b09600, 0x200, 0x200, 0x0, 0x0, 0x0)
        /usr/lib64/go/src/net/net.go:176 +0x6d
net.(*dnsPacketConn).dnsRoundTrip(0xc420337860, 0xc4222a8f00, 0xe8a460, 0xe8a460, 0x0)
        /usr/lib64/go/src/net/dnsclient_unix.go:57 +0xde
net.(*Resolver).exchange(0xe88f70, 0xe4dac0, 0xc420c23260, 0xc4201b4470, 0xb, 0xc4205158e0, 0x1e, 0xc42020001c, 0x12a05f200, 0x0, ...)
        /usr/lib64/go/src/net/dnsclient_unix.go:138 +0x48e
net.(*Resolver).tryOneName(0xe88f70, 0xe4dac0, 0xc420c23260, 0xc4201dd5e0, 0xc4205158e0, 0x1e, 0xc42010001c, 0xc420207fa8, 0xc425029650, 0xc420c22fc0, ...)
        /usr/lib64/go/src/net/dnsclient_unix.go:161 +0x148
net.(*Resolver).goLookupIPCNAMEOrder.func1(0xe88f70, 0xe4dac0, 0xc420c23260, 0xc4201dd5e0, 0xc420337790, 0xc420c23320, 0xc4201d001c)
        /usr/lib64/go/src/net/dnsclient_unix.go:483 +0x8d
created by net.(*Resolver).goLookupIPCNAMEOrder
        /usr/lib64/go/src/net/dnsclient_unix.go:482 +0x21c

Crash related to second config:
Click me!
In the log you can see how I restarted the backup and everything went ok the second time around.
Steps to reproduce the behavior
Just run the backup again.
Do you have any idea what may have caused this?
Unfortunately, no.
Do you have an idea how to solve the issue?
See above.
Did restic help you or made you happy in any way?
It's fantastic! :-D
I have just switched to restic, it looks very promising! Keep up the good work!

Huh, that's odd, it looks to me like a system failure (memory maybe?). If you can reproduce the issue, can you try with GOTRACEBACK=system set? We may need to escalate this to the Go developers...

Started another run with GOBACKTRACE=system set.
I found nothing in dmesg and I will run memtest86 as soon as possible.

Two other panics, unfortunately.
Here are the logs:
Click

It may be a DNS resolution problem, no DNS response packet should crash the Go runtime (in the stdlib) in this way. I'll ask the Go developers what we can do here.

I confirm I had a very short power outage this night. This lead to the
(possible) DNS resolution error.
2017-12-17 11:21 GMT+01:00 Alexander Neumann<EMAIL_ADDRESS>
It may be a DNS resolution problem, no DNS response packet should crash
the Go runtime (in the stdlib) in this way. I'll ask the Go developers what
we can do here.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/restic/restic/issues/1486#issuecomment-352245367, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFwP67s1tLyZCpkaMoXkIwRUCYFzgObcks5tBOslgaJpZM4RER-o
.

--
- Giulio

Maybe, but that cannot explain the crash in the garbage collection function. Would you mind running memtest86 for a while?

@fd0, done a few memtest's rounds and I got 758k errors. 😭
I removed the defective module and I'm stress testing my server with 3 concurrent backups (2 remote, 1 local).
I will let you know as soon as they finish / fail!

Nice, thanks for the feedback! Try to see the positive side: You found the bad memory before it caused any serious issue.
And you're not alone, that's about the fifth case of bad hardware exposed by restic.

Sad news: another panic.
Full log (including last crashes): here
Ideas?

Have you run memtest again to verify that all of your current memory is 100% good? If not, that might be a good idea.

Done that, that's why I restarted my backups. :(
-- Giulio De Pasquale
On Dec 19, 2017 11:48 PM, "rawtaz"<EMAIL_ADDRESS>wrote:

Have you run memtest again to verify that all of your current memory is
100% good? If not, that might be a good idea.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/restic/restic/issues/1486#issuecomment-352910513, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFwP629rfJyw6HVmffEP5YPqrFm-hRFoks5tCD1JgaJpZM4RER-o
.

Okay, it wasn't clear to me that you had run memtest after removing/replacing faulty memory, to make sure that the current (possibly newly added) memory is good. I assume you have, though.

Yes, I am sorry I was unclear about that.
-- Giulio De Pasquale
On Dec 20, 2017 12:24 AM, "rawtaz"<EMAIL_ADDRESS>wrote:

Okay, it wasn't clear to me that you had run memtest after
removing/replacing faulty memory, to make sure that the current (possibly
newly added) memory is good. I assume you have, though.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/restic/restic/issues/1486#issuecomment-352917451, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFwP67gIRPYmg-IJy8JHZMYWiuUOgY01ks5tCEXBgaJpZM4RER-o
.

I'm really sorry that you're having so much trouble running restic. It still smells like a hardware issue, both crashes are in the Go standard library at different places. In both cases, it's a segmentation fault with strange addresses:
dic 16 00:52:34 backup-remote-data[30571]: [signal SIGSEGV: segmentation violation code=0x1 addr=0x3a pc=0x429276]
dic 16 00:52:34 backup-remote-data[30571]: internal/poll.runtime_pollWait(0x7f55d654c430, 0x72, 0x0)
dic 16 00:52:34 backup-remote-data[30571]:         /usr/lib64/go/src/runtime/netpoll.go:173 +0x57
dic 16 00:52:34 backup-remote-data[30571]: internal/poll.(*pollDesc).wait(0xc4222a9018, 0x72, 0xffffffffffffff00, 0xe48900, 0xe436c0)
dic 16 00:52:34 backup-remote-data[30571]:         /usr/lib64/go/src/internal/poll/fd_poll_runtime.go:85 +0xae

and
dic 16 23:16:13 call_restic[28365]: unexpected fault address 0xf9aa0670efe
dic 16 23:16:13 call_restic[28365]: fatal error: fault
dic 16 23:16:13 call_restic[28365]: [signal SIGSEGV: segmentation violation code=0x1 addr=0xf9aa0670efe pc=0x55a14b]
dic 16 23:16:13 call_restic[28365]: goroutine 149767 [running]:
dic 16 23:16:13 call_restic[28365]: runtime.throw(0xaf6921, 0x5)
dic 16 23:16:13 call_restic[28365]:         /usr/lib64/go/src/runtime/panic.go:605 +0x95 fp=0xc421b1e8e0 sp=0xc421b1e8c0 pc=0x42c645
dic 16 23:16:13 call_restic[28365]: runtime.sigpanic()
dic 16 23:16:13 call_restic[28365]:         /usr/lib64/go/src/runtime/signal_unix.go:374 +0x227 fp=0xc421b1e930 sp=0xc421b1e8e0 pc=0x442857
dic 16 23:16:13 call_restic[28365]: context.(*cancelCtx).Value(0xf9aa0670efe, 0xa39dc0, 0xeaa458, 0x0, 0x0)
dic 16 23:16:13 call_restic[28365]:         <autogenerated>:1 +0x2b fp=0xc421b1e968 sp=0xc421b1e930 pc=0x55a14b
dic 16 23:16:13 call_restic[28365]: context.(*cancelCtx).Value(0xc420365b00, 0xa39dc0, 0xeaa458, 0x0, 0x0)

I don't know what we can do here, and apart from running memtest (which you already did) I don't have more ideas. Sorry for that.

@peperunas What are the system specifications? Is it in a basic configuration or have you tuned anything, overclocked, etc? Is there any part of the system that gets very hot during these backup runs?

@fd0, no worries!
@rawtaz, I am using a Lenovo T530. No overclock or any other hardware
modification. I haven't noticed any surface that gets very hot. I will try
to run the same backup (same files, same configuration) on another PC.
Il 20 dic 2017 15:28, "rawtaz"<EMAIL_ADDRESS>ha scritto:
@peperunas https://github.com/peperunas What are the system
specifications? Is it in a basic configuration or have you tuned anything,
overclocked, etc? Is there any part of the system that gets very hot during
these backup runs?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/restic/restic/issues/1486#issuecomment-353077062, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFwP63ar5W06Xw01yV3zF8uqR6YW7YKfks5tCRmHgaJpZM4RER-o
.


Update broccoli-persistent-filter

If you update broccoli-persistent-filter to the latest version 1.2.13, this will include a fix for a very old bug: https://github.com/kimroen/ember-cli-coffeescript/issues/112

Thanks. I bumped the dependency in version 1.0.1.


Project Delete Performance Testing

User Story
We currently have some questions about how long it will take for a project to be fully deleted with the addition of deleting the project from ingested nodes.
Below are the tests we want to run for the average user and high load user:

How long would it take to delete one project that is on all the resources.
How long would it take to delete one project that is on half the resources.
How long would it take to delete one project that is on 1/4 of the resources.
How does running multiple project deletes at the same time affect the delete times.

A high load user would be 90 days with 24 compliance reports per day. How many nodes?
An average user would be 30 days with 6 compliance reports per day. How many nodes?
Definition of Done
A writeup showing the high load and average users project delete times.
Open Questions

do we need a special case for if a project has zero ingest rules and is up-to-date?
Does Elasticsearch filtering for only resources that have the project that is going to be deleted speed up deletion?

@susanev We do not need this issue any longer with the new path of project delete.

i dont think so, @sdelano what do you think?
do we have a parallel card for project update? if not, prolly good to at least create that..

closing based on convo here, can reopen if needed.


Fix incorrect maintainer

I accidentally mistyped the username for the maintainer 😅

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

No need to wait for CI


Click image keyword should also work for images inside form

Originally submitted to Google Code by m.marag... on 20 Nov 2008

What steps will reproduce the problem?

click image in a page with source similar to the following

<FORM ACTION='/opt/cgi-bin/example.cgi' METHOD='POST'>

Example

A List
<SELECT NAME='1' SIZE=1><OPTION VALUE='(empty)'>No selection</OPTION>
<OPTION SELECTED VALUE='hello'>hellot</OPTION>
</SELECT>

Example...	<INPUT NAME='10' TYPE=TEXT SIZE=10
VALUE='14' MAXLENGTH=10>

<INPUT TYPE=IMAGE SRC='/images/savebutton.gif' BORDER=0></FORM>
What is the expected output? What do you see instead?
ERROR: Element<EMAIL_ADDRESS>or
@src="/images/savebutton.gif" or @alt="/images/savebutton.gif"] not found
What version of the product are you using? On what operating system?
Latest Selenium 2.0.2
Please provide any additional information below.

Originally submitted to Google Code by @yanne on 20 Nov 2008

Thanks for reporting, I'll take a look at this next week

Originally submitted to Google Code by @yanne on 28 Nov 2008

Click Image normally works on images that are inside  tag, ie. links.
In this case, the image is inside a form, and in an input tag.
You can already submit the form with Submit Form keyword, but I will add a support
for clicking images directly inside a form to next version of SeleniumLibrary

Originally submitted to Google Code by @yanne on 28 Nov 2008

The fix that makes the original case to pass is implemented in r86.

Originally submitted to Google Code by @pekkaklarck on 5 Dec 2008

This was included in 2.1, right?


AlignHash with separator causing problems

beyond getting this
An error occurred while Style/AlignHash cop was inspecting <file name>
Errors are usually caused by RuboCop bugs.
Please, report your problems to RuboCop's issue tracker.
Mention the following information in the issue report:
0.35.1 (using Parser <IP_ADDRESS>, running on ruby 2.1.2 x86_64-darwin13.0)

it also sometimes messes the hash up, the key names etc.
specifically this happened in a code that looks like this, in a spec file
let(:params) do
    {
      key1:                    val,
      long_key: looooooong_val,
      some_parm:              {
        key_a: val_a,
        key:       value
      }
    }
  end

with the settings
Style/AlignHash:
  EnforcedHashRocketStyle: table
  EnforcedColonStyle: seperator

That should be separator, not seperator. Please try with the correct spelling in the config file.

yea, that was a typo when i created the issue, I typed it by hand.
It still happens, I'll try to create a demo hash where it happens

Tried the sample code and config which you show above. It doesn't produce an error (at least on master).

try this sample, which causes a mashed-up hash as a result and also causing the file to have syntax errors
let(:params) do
    {
        sjtjo: sjtjo,
        too_ono_ilitjion_tofotono_o: too_ono_ilitjion_tofotono_o,
        o_lionj_tofotono_o: o_lionj_tofotono_o,
        gtjowty_tofotono_o: gtjowty_tofotono_o,
        o_onnoo_jot_tofotono_o: o_onnoo_jot_tofotono_o,
        o_utono_y: "QWE",
        tmoj: 123,
        o_onnoo_jot: "pt55yon",
        jimosjtmp: jimosjtmp,
        soson_o_lionj_dtjt: soson_o_lionj_dtjt.jo_json,
        o_onnoo_jot_o_lionj_dtjt: { o_onjto_j_id: 'Pt4yon-Vtlijot-o_ttd' }.jo_json,
        o_todij_o_ttd: {
            o_ttd_nbot: o_ttd_nbot,
            btnd: btnd,
            ox_ott: ox_ott,
            op_onjh: op_onjh,
            holot: holot,
            joaon: joaon,
            jypo: o_ttd_jypo
        }
    }
  end

OK, one discovery: there is no problem when auto-correct is not used.
Autocorrection is breaking syntax, causing a syntax error.

Autocorrection is also eating up some of the key/value pairs. They are completely gone.

The problem still occurs if the EnforcedHashRocketStyle parameter is removed. It appears that it should be related to EnforcedColonStyle: separator.

OK, the problem is happening on line 245 of align_hash.rb. If a key/value/etc is too far to the right, the cop deletes some characters on its left side. But it is deleting so many characters, that it takes out part of the token on its left.

OK, it's getting clearer.
Do you see the key which is soson_o_lionj_dtjt? It wants to move that one to the left, so its colon will align with the colon on the previous line.
But this is impossible. The key is so long, and the colon is so much further to the right than the above one, that even if all the whitespace on its left was deleted, it still wouldn't align. But in an attempt to align, the cop is deleting MORE than all the whitespace on its left.

I have a fix for this, will push soon.

hmmm. I thought it will take the rightest possible colon without deleting and aligning other keys by it

Pushed a fix to my open PR. The cop could still align the code better, but at least it doesn't break it now.


Setup integrations on discord

GitHub
Meetups Group activity
Google Alerts / Twitter Search alerts -> monitor people talking about us

GitHub and twitter done


fix the documents, and add an appendix for the MyEF sample project

Hi team!
As I was working through the MyEF project demo, I had to do some troubleshooting.
Therefore, I think that I need to add troubleshooting tips!
And many people will want to deploy Azure and check its operation.
Therefore, I have included tips on creating Azure resources in the appendix! :-)

Thank you for the approval, @chullybun .
Oh! I didn't realize the feature had been added yet.
I would like to try it right away!


run(params=...) removes variables from doc()

It seems that changing a parameter or variable removes that variables from the doc() output.
e.g.
print( model.doc() ) # output has 'Pi' in model.doc()['Real Name']
model.run( ... , params = {'Pi':3.14}, ...)
print( model.doc() ) # output does not have 'Pi'
I was doing checks between simulation runs that the params-dictionary doens't have wrong names in there by comparing to model.doc()['Real Name'].

Hi @jukkaranta
I have never thought about that but you are totally right. The issue is that the model.doc method uses the docstring of each model variable defined as a function in the model file to generate the output dataframe. When you change the value of a parameter a new function is defined without docstring. I can manage to change that and keep the old function docstring in a future release.
Thanks for opening this issue :)


Floss not working after running cup all

Floss was working in my initial installation of the Flare-VM on Windows 10 VM.
After running cup all
C:>floss
Traceback (most recent call last):
  File "C:\Python36\Scripts\floss-script.py", line 11, in <module>
    load_entry_point('floss==1.5.1', 'console_scripts', 'floss')()
  File "c:\python36\lib\site-packages\pkg_resources\__init__.py", line 565, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "c:\python36\lib\site-packages\pkg_resources\__init__.py", line 2631, in load_entry_point
    return ep.load()
  File "c:\python36\lib\site-packages\pkg_resources\__init__.py", line 2291, in load
    return self.resolve()
  File "c:\python36\lib\site-packages\pkg_resources\__init__.py", line 2297, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "c:\python36\lib\site-packages\floss\__init__.py", line 3, in <module>
    from utils import ONE_MB
ModuleNotFoundError: No module named 'utils'

Please let me know if further information is required or I should move this to the floss project.

Look like the Python interpreter is 3.6. FLOSS does not support Python3.
I am not sure if cup all on current system automatically update python interpreter as well, which may cause this issue. However, update feature has always been "not fully supported" before version 2.0. If you can, please reinstall FlareVM. We recently pushed a new version that has proper support for update.


Fix not detecting unambiguous scores properly

Took a while, but here it is :-)
Hopefully fixes #4.

Thanks a lot! I had been too busy to spend time on this.
Clean solution, and the tests prove that it works. I'll release this in 3.2.1 as a bugfix.


Missing Powershell cmdlets for Teams Trial Licensing

Hi, I'd say it should be noted that the SKU appears only after the first trial has been activated, otherwise you can't assign them from admin portal
The second thing: what's this SKU powershell cmdlet to enable mass trial activation? Thanks

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

ID: 28550025-fde2-9a83-a826-f7e3ccb313db
Version Independent ID: 2a304432-b10c-220f-3453-e19e7a155c67
Content: Manage the Microsoft Teams Commercial Cloud Trial offer
Content Source: Teams/iw-trial-teams.md
Service: msteams
GitHub Login: @chuckedmonson
Microsoft Alias: chucked

@AlexHill90  - Thank you for submitting this feedback.
I was able to get this into the backlog for the Teams team. The team will get this prioritized into a sprint plan. Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

Hi @AlexHill90 Thanks for your feedback, we really appreciate it.
I tested in lab and the "Microsoft Teams Trial" toggle appeared once the first user signed up for it.
So it's true that from that point, you will be able to assign the trial to other users through Admin Portal, I'll create a Pull Request to clarify this.
About PowerShell, you would have to use Office 365 PowerShell cmdlet Set-MsolUserLicense to assign licenses.
Hope this information has helped you!


Topic/get optional

Mostly usable, but there's a bug in the constraints checker (see repl tests).

:meat_on_bone:
yeah just merge regardless of the bad constraint

I also wonder whether we should modify the tests to use a special test-prelude, so we don't have to change the "loaded 21 functions" every time... eh or just don't print that


new: Add query sharing docs and update ENV variables for Lab

Description
Add all new environment variables for the Lab. Add query-sharing feature page.
For the Lab release: Lab 2.15.
Pull request type
Please check what kind of PR this is:

[x] Fix or improvement of an existing page
[x] New documentation page, release related

Related PRs and issues

Checklist:

[x] Check all content with Grammarly
[x] Perform a self-review of my code
[x] Make corresponding changes to the rest of the documentation (consult with the DX team)
[x] The build passes locally
[x] My changes generate no new warnings or errors
[x] Add a corresponding label
[ ] If release-related, add a product and version label
[ ] If release-related, add release note on product PR

I've updated Query sharing docs and added all the reported fixes.
Note: In the query sharing docs I've added TODO parts where images and/or video should be added, some are sketches, and some are screenshots. Please consult with Ana and the team for them to have them ready by Wed.


The self hosted avwx-api returns more than one METAR or TAF at once.

Example:
curl 'http://<IP_ADDRESS>:8000/api/metar/KLAX?filter=raw'
{
  "meta": {
    "timestamp": "2023-10-23T08:14:09.900113Z",
    "stations_updated": "2023-10-01"
  },
  "raw": "KLAX 230753Z 17006KT 10SM FEW012 BKN016 BKN028 17/16 A2976 RMK AO2 SLP076 T01720156 402280139 $ KLAX 230653Z 24007KT 10SM SCT012 BKN015 BKN075 18/16 A2977 RMK AO2 SLP079 T01780156 $"
}

Note there are two METARs combined at one single line in the returned result, which is unexpected.

Fixed after updating avwx-engine 1.8.24


Add overcommit and pre-commit

Two projects with similar goals and vastly different implementations. Overcommit is written in Ruby, and provides a framework for managing a variety of git hooks with a particular focus on pre-commit hooks. Pre-commit is written in Python, and provides a similar framework but solely for pre-commit hooks.

I figured these would be better placed in an awesome-git somewhere, but failing to find that, I guess I'll add them later.

I thought the same, but I couldn't find one, strangely enough. There is an awesome-github, though.


What is network_control.energies.minimum_energy_fast doing?

I've implemented a function called state_to_state_transition in our nict repo and would like to make sure that I don't implement functions that have already been implemented by you. The purpose of this function is to either calculate optimal control energy or minimum control energy for all combinations of input brain states. The function outputs a TxNxM 3D-array where

where T is the number of time points, N is the number of nodes and M denotes each combination of states.

In other words, it's just a 3D array that stores multiple outputs from either minimum_input or optimal_input.
Since we are both interested in combinations (e.g. only AB but not BA) and permutations (e.g. AB but also BA), I've implemented this logic in nict._get_layout. (so the user can choose between 'combinations' and 'permutations'
Now I've discovered minimum_energy_fast but I can't really figure out what this function does? E.g what do you mean with numpy array (N x n_transitions) array? And what is meant by:

Returns:
  E: numpy array (N x n_transitions)
        Regional energy for all state transition pairs.
        Notes,
            np.sum(E, axis=0)
                collapse over regions to yield energy associated with all transitions.
            np.sum(E, axis=0).reshape(n_states, n_states)
                collapse over regions and reshape into a state by state transition matrix.

P.S.: Especially pinging @jastiso here because I promised to share the link to this repo.

Hi Johannes,
The minimum_energy_fast function simply estimates energy associated with all possible state transitions simultaneously, eschewing the need to process in serial. It does so by taking advantage of the fact that a few steps in the process need only be estimated once (e.g., computing and inverting the Gramian; see section titled Evaluating Minimum Control Energy  at the bottom of this document).
The output E has regions on the rows and state transitions on the columns, so if you sum over regions and then reshape to a square matrix, you end up with a matrix of transition energies. Note, this is only true if you use our utils.expand_states function to define x0_mat and xf_mat (sorry, that's not so clear from the code!). See this notebook for an example.
Thanks,
Linden

Hi Linden, I never thanked you for the answer which I would like to do now (also thanks for the detailed explanation)! Over the holidays I reworked the nict package. It now contains a better separation between computing control energy for one subject vs. multiple subjects (nict.single_subject, nict.multi_subjects).
I have a few follow-up questions because I think one could speed up computations in the nict package based on what you have said? In my package, I have this state_to_state_transition function. It allows to compute all possible state-to-state-transitions based on an input A and a matrix of states X where the rows denote each state and the columns denote the nodes.
As you can see the function just uses a for-loop to iterate through all combinations of possible source and target states to either compute minimum or optimal control energy.
Couldn't one also use the same logic here as in minimum_energy_fast? And is there also an analog for computing optimal control energy (or is this form of mathematical speed-up only possible when computing minimum control energy)? Lastly minimum_energy_fast is constrained to binary states, but here in Mannheim we are working with with activation maps (beta-images / T-images, etc.).  Would such a speed up also be working for dimensional states?

Hi Johannes,
No problem, we're glad you're finding the package useful!
Yes, one of the reasons we wanted to implement minimum_energy_fast was to achieve exactly what you're doing in your own state_to_state_transition function but faster! So I would definitely suggest swapping in our minimum_energy_fast to speed things up. Something like the following in place of your for-loop should work:
E = np.zeros((n_states, n_states))
e = minimum_energy_fast(A_norm, T, B, x0_mat, xf_mat)
E[:] = np.sum(e, axis=0).reshape(n_states, n_states)

There are two considerations:

You will only get e out, not x or u.
You need to define the matrix of state transitions ahead of time (x0_mat, xf_mat). This should work with non-binary states, but I haven't tried it myself. In any case, I think you'll have to make a few changes to your X matrix. We define x0_mat and xf_mat with nodes on rows and transitions on columns. Note, every single transition is stored in its own column, which means the actual states themselves are duplicated many times in each of x0_mat and xf_mat. For example, if the first 3 transitions are from state_0 to state_1, state_0 to state_2, and state_0 to state_3, then the first 3 columns of x0_mat are identical (all representing the node activity levels associated with state_0). So you'll have to duplicate your activation map-derived states.

Sorry, we won't have a fast version of optimal control at the moment.
Hope this helps!

Hi Linden, thanks for the explanation, i've written the following test script to ensure that both minimum_input , nict.single_subject.state_to_state_transition and minimum_energy_fast produce the same error. Computing the error for one state-to-state transition using nict gives the same result as manually computing minimum_input using the same source and target state. (which makes sense because nict is simply using minimum_input under the hood). However, minimum_energy_fast does not give the same error (and they are surprisingly large?). Did I do something wrong or is this a bug?:
# -*- coding: utf-8 -*-
"""
Compare minimum_energy_fast from network control with manually computed
error using nict package

@author: Johannes.Wiesner
"""
import sys
sys.path.append('./nict')

from nict.single_subject import state_to_state_transition
from nict.single_subject import _set_transition_order
from network_control.utils import matrix_normalization
from network_control.energies import minimum_energy_fast
from network_control.energies import minimum_input
import numpy as np

# create artifical data
np.random.seed(28)
A = np.random.rand(10,10)
A_norm = matrix_normalization(A,c=1,version='continuous')
X = np.random.rand(4,10) # 4 (dimensional) sample states and 10 nodes
# X = np.random.choice(a=[0,1],size=(4,10),p=[0.5,0.5]) # 4 (boolean) sample states and 10 nodes
B = np.eye(10)
S = np.eye(10)
T = 10 # make T large enough to get big errors
version='continous'
n_states,n_nodes = X.shape

# 1.) first compute all state-to-state-transitions using johannes nict package
X_out,U_out,E_out = state_to_state_transition(A=A_norm,T=T,B=B,X=X,rho=None,S=None,order='permutations')

# 2.) Now do sanity check by simply recomputing one randomly chosen state-to-state-transition
# and compare that with nict package
x,u,e = minimum_input(A=A_norm,T=T,B=B,x0=X[0,:],xf=X[1,:])
print(np.equal(E_out[0],e))

# 3.) now run minimum_energy_fast from network_control package and check if the
# computational error is the same
# create x0_mat and xf_mat using nict package (we could probably also use
# expand states here but the outcome is the same)
n_transitions,transition_idxs = _set_transition_order(n_states=n_states,order='permutations')
x0_mat = np.zeros((n_nodes,n_transitions))
xf_mat = np.zeros((n_nodes,n_transitions))

for idx,src,tgt in transition_idxs:
    x0_mat[:,idx] = X[src,:]
    xf_mat[:,idx] = X[tgt,:]

# run minimum energy fast and sum error over nodes to get one value for each
# transition
e_fast = minimum_energy_fast(A=A_norm,T=T,B=B,x0_mat=x0_mat,xf_mat=xf_mat)
e_fast = np.sum(e_fast,axis=0)
print(np.equal(E_out[0],e_fast[0]))

Follow up question regarding:

You will only get e out, not x or u.

Is this something that could be implemented in the future or simply not possible? If it could be implemented I could open a feature request for this? :) Receiving u would definitely be something that I would need for my project

Sorry, we won't have a fast version of optimal control at the moment.

No need to say sorry! I am the one that should come up with improvements (I am just really sure that I can't...) Same as above, I would then open a feature request if it would be theoretically possible to create optimal_energy_fast?


Generating permissions into permissions instead of clusterPermissions

Type of question
Open question
Question
The permissions for the go based operator are generated from kubebuilder annotations in controllers code.  By default, the permissions are generated into (csv).spec.install.spec.clusterPermissions
What did you expect to see?
I would like to have the permissions listed in (csv).spec.install.spec.permissions
Is there a way how to modify the kubebuilder annotations to do so?
Environment
Operator type:

Kubernetes cluster type:
OpenShift
$ operator-sdk version
operator-sdk version: "v1.23.0", commit: "1eaeb5adb56be05fe8cc6dd70517e441696846a4", kubernetes version: "1.24.2", go version: "go1.18.5", GOOS: "linux", GOARCH: "amd64"
$ go version (if language is Go)
go version go1.18.7 linux/amd64
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.0", GitCommit:"04ad1b56880418de7bd6feb9ff37a8518fbc1a0e", GitTreeState:"clean", BuildDate:"2022-05-12T09:52:02Z", GoVersion:"go1.17.5", Compiler:"gc", Platform:"linux/amd64"}

I can look into this

hi @stanislavulrych You can achieve what you want by setting an arbitrary namespace in the kubebuilder rbac annotations. For example, if you want an operator to be able to create statefulsets and pods at the namespace level, it would look like shown below:
//+kubebuilder:rbac:groups=apps,resources=statefulset,namespace=placeholder,verbs=get;list;watch;create;update;patch;delete
//+kubebuilder:rbac:groups=core,resources=pods,namespace=placeholder,verbs=get;list;watch;create;update;patch;delete

Notice I have namespace set to namespace=placeholder.
While doing this will allow the cluster roles to be made to be roles, they would not be added to .spec.install.spec.permissions in the Cluster Service Version like you would expect it to. However, the role would be added as a separate file in the bundle/manifests directory.
Full path should take the pattern, bundle/manifests/[operator-name]-manager-role.rbac.authorization.k8s.io_v1_role.yaml

Hi @OchiengEd, thank you for the answer.
I have tried that and it really generates bundle/manifests/[operator-name]-manager-role.rbac.authorization.k8s.io_v1_role.yaml.
I just do not understand, why these permissions are not added to .spec.install.spec.permissions in the Cluster Service Version. Is there any special reason for it not being added to it?

@stanislavulrych  Not sure of the reason for this bwhavior. At this time, the issue has been marked as a bug and I believe Jesus will be looking into why the decision was made to have it as a separate file before any action can be taken.

hey @stanislavulrych, are you completely blocked by this issue?  If so, will it be possible for you to manually edit the .spec.install.spec.permissions in ClusterServiceVersion as a workaround for unblocking?

@tlwu2013 so far, I am building the operators without any manual steps, which is the reason why I opened this issue - it is possible to edit the CSV manually, however, you might understand that it somehow complicates the automated pipelines for the build.

/unassign jmrodri

Hi Everyone!
There's any idea on continue the work on this issue?
I'm currently facing the same issue and I was having an idea that it's adding an option like --cluster-permission-to-permission (clearly not that long option) that switch the permission to the permission section.
The other idea is to add a --exclude-cluster-permission which basically exclude a set of permission resources of being moved into the permission section and being kept in the clusterPermission section, why? because some permissions like getting the nodes or namespace shouldn't be namespaced.
Any other idea on this?

Operator SDK should already handle this automatically if you have a RoleBinding in your manifests that binds the operator's service account to the Role.

Hi everyone and @joelanford , in the later versions of operator-sdk, the documentation was greatly improved (thanks everyone involved for that!). By properly following the instructions in
https://sdk.operatorframework.io/docs/building-operators/golang/operator-scope/#changing-the-permissions-to-namespaced
and
https://sdk.operatorframework.io/docs/building-operators/golang/crds-scope/
I solved the problem of generating permissions instead of clusterpermissions. There is now really nothing to be added to those steps and the description is very informative. As described in the above, there is one manual step needed:

Use RoleBindings instead of ClusterRoleBindings. The config/rbac/role_binding.yaml needs to be manually updated

However, since role_binding.yaml is not regenerated during the operator build, it is not a problem even for automated build.
Thus,I am closing this.


Build system change auto -> cmake + Full windows support

The goal of these commits is to enable windows support. On the way there I decided to switch out the build system, rather than just (for example) adding a Visual studio project in a subdirectory.
The new jansson.dll works perfectly and passes all 121 tests on windows. I have added binaries to my download area: https://github.com/downloads/DanielT/jansson/jansson_windows_post2.2.1.zip
Infos about building, running tests etc is in the commit messages.

I'm sorry, but I'm not willing to maintain something I don't have a slightest clue about. Some of your changes also seem quite Windows-specific (removal of conditional inline and localeconv tests, for example).
I'd like to have better Windows support, but this is not the right way to go.

Well, thats a shame.
I am curious about the localeconv and inline tests. Why are they there?
My thinking was this:

localeconv is in C89. Every C compiler in 2012 supports C89. Therefore every compiler supports localeconv.
inline is a keyword in C99. Every compiler I know of apart from Microsoft's supports C99. MSC has__inline instead.
I think you could safely take those changes even if you don't want the build system change.

Old proprietary unix-like systems use old compilers, for example. inline was conditinalized because of bug reports, so there's an actual need to build without inline. Conditional use of localeconv was just a safety measure, and it was there from the beginning.


Rewrite doc

What problem does this PR solve?
This pr improves:

README: decrease the TOC, most of the links are moved to user guide
user guide

Rewrite current user guide to userguide_2.0 for TiSpark >= 3.0
Add userguide_2.0 for TiSpark 2.4
TiSpark 2.5 does not have user guide

dev guide: rewrite some docs
wiki
other

some docs are deleted for they are put into the user guide directly like configuration.md
add some docs like push down doc
reorg docs

I have fixed the problems by the pre review and polished the user guide again. This pr is ready to be reviewed again

/merge

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests


python312Packages.fontpens: normalize pname

Description of changes
part of #245383
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

Add a :+1: reaction to pull requests you find important.

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

21 packages failed to build:

fira-math
fontbakery (python311Packages.fontbakery)
fontbakery.dist (python311Packages.fontbakery.dist)
python311Packages.fontmake
python311Packages.fontmake.dist
python312Packages.babelfont
python312Packages.babelfont.dist
python312Packages.collidoscope
python312Packages.collidoscope.dist
python312Packages.fontbakery
python312Packages.fontbakery.dist
python312Packages.fontfeatures
python312Packages.fontfeatures.dist
python312Packages.fontmake
python312Packages.fontmake.dist
python312Packages.glyphsets
python312Packages.glyphsets.dist
python312Packages.glyphslib
python312Packages.glyphslib.dist
python312Packages.glyphtools
python312Packages.glyphtools.dist

55 packages built:

deltachat-desktop
nanoemoji
nanoemoji.dist
nomnatong
noto-fonts-color-emoji
python311Packages.afdko
python311Packages.afdko.dist
python311Packages.babelfont
python311Packages.babelfont.dist
python311Packages.booleanoperations
python311Packages.booleanoperations.dist
python311Packages.collidoscope
python311Packages.collidoscope.dist
python311Packages.fontfeatures
python311Packages.fontfeatures.dist
python311Packages.fontparts
python311Packages.fontparts.dist
python311Packages.fontpens
python311Packages.fontpens.dist
python311Packages.glyphsets
python311Packages.glyphsets.dist
python311Packages.glyphslib
python311Packages.glyphslib.dist
python311Packages.glyphtools
python311Packages.glyphtools.dist
python311Packages.nototools
python311Packages.nototools.dist
shaperglot (python311Packages.shaperglot)
shaperglot.dist (python311Packages.shaperglot.dist)
python311Packages.statmake
python311Packages.statmake.dist
python311Packages.ufo2ft
python311Packages.ufo2ft.dist
python311Packages.ufoprocessor
python311Packages.ufoprocessor.dist
python312Packages.afdko
python312Packages.afdko.dist
python312Packages.booleanoperations
python312Packages.booleanoperations.dist
python312Packages.fontparts
python312Packages.fontparts.dist
python312Packages.fontpens
python312Packages.fontpens.dist
python312Packages.nototools
python312Packages.nototools.dist
python312Packages.shaperglot
python312Packages.shaperglot.dist
python312Packages.statmake
python312Packages.statmake.dist
python312Packages.ufo2ft
python312Packages.ufo2ft.dist
python312Packages.ufoprocessor
python312Packages.ufoprocessor.dist
twitter-color-emoji
whatsapp-emoji-font


Add Instagram Provider

Changes:

add instagram to README
add instagram to external.go
add instagram to settings.go
add instagram to settings_test.go
add instagram to configuration.go
add external_instagram_test.go
add provider/instagram.go
add instagram to .env.example

hey @aaronsmulktis, thanks for contributing to this project, at the moment, we don't really have the bandwidth to support adding additional OAuth providers and we intend to support this with a more generic implementation - https://github.com/supabase/auth/issues/451#issuecomment-1641660848
if there's an urgent need to use instagram as an oauth provider - we strongly recommend forking the project and self-hosting for the time being

hey @aaronsmulktis, thanks for contributing to this project, at the moment, we don't really have the bandwidth to support adding additional OAuth providers and we intend to support this with a more generic implementation - #451 (comment)
if there's an urgent need to use instagram as an oauth provider - we strongly recommend forking the project and self-hosting for the time being

All good.. Thanks for the quick reply Kang!


Call login() from outside template

Is there a way to call the directive's login() and logout() functions from outside the directive?

Not at the moment. One possible solution could be to add some events listeners into the directive.

Thanks for confirming. I like that idea about triggering an event.
The use-cases I'm looking for are:

add links to the bottom of a sign up page, eg already signed up?

non user initiated logout

I can call your services directly but it feels brittle.
On 16 Jul 2015 1:30 am, "Andrea Reginato"<EMAIL_ADDRESS>wrote:

Not at the moment. One possible solution could be to add some events
listeners into the directive.
—
Reply to this email directly or view it on GitHub
https://github.com/andreareginato/oauth-ng/issues/88#issuecomment-121876844
.

Hi @GregTurner. For the use case 1 you could also duplicate the directive code and assign a differente template. It should solve your problem. I didn't get the point 2.
In any case, events could be useful also in the future.

That'd work!  Great suggestion.
Re: UC#2: not important at this time


add docs for brew services

One of my colleagues was setting up mailpit at my recommendation but missed this step, so I figured it might be useful to others too.

I can't add this where you have inserted it as these instructions are specific to brew installations alone, not all platforms. If you can update your PR to add the sentence directly after "Install Mailpit with brew install mailpit. To run automatically run ...."  (ie: same paragraph) then I'll merge this in, thanks.

@axllent Sure, that makes sense. I added it by the brew (mac) section, and tweaked the usage docs to make it more clear. However, I can remove that if you want.
(The other when troubleshooting with my colleague, I skipped over the installation section and went straight to usage, so I figured it might be good to at least have something there 🤷🏻♂)


McapHandler improvements

After:

https://github.com/eProsima/DDS-Record-Replay/pull/58

This PR tackles some inconsistencies introduced when adding the only-with-type configuration option, and extends the accepted values for max-pending-samples, now being possible to have none, infinite or a finite number of pending samples.

Please, update Forthcoming Version.

Codecov Report
Patch coverage: 18.45% and project coverage change: -0.57 :warning:

Comparison is base (73c3b04) 19.84% compared to head (e0f7bcf) 19.28%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #63      +/-   ##
==========================================
- Coverage   19.84%   19.28%   -0.57%
==========================================
  Files          23       23
  Lines        2036     2085      +49
  Branches     1077     1111      +34
==========================================
- Hits          404      402       -2
- Misses       1266     1304      +38
- Partials      366      379      +13

Impacted Files
Coverage Δ

...ecorder_participants/recorder/mcap/McapHandler.hpp
100.00% <ø> (ø)

ddsreplayer/src/cpp/tool/DdsReplayer.cpp
41.78% <0.00%> (+0.30%)
:arrow_up:

..._yaml/src/cpp/recorder/YamlReaderConfiguration.cpp
9.48% <5.26%> (-0.34%)
:arrow_down:

..._yaml/src/cpp/replayer/YamlReaderConfiguration.cpp
22.58% <9.09%> (ø)

...participants/src/cpp/recorder/mcap/McapHandler.cpp
42.16% <21.16%> (-5.63%)
:arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


It is not connecting properly with the domain

Before opening, please confirm:

[X] I have checked the custom domains troubleshooting guide.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
m-fa01ece1955f400bbf8affd68dac2146
Region
eu-west-1
Step
Domain activation
Status
Failed
Issue or question
This app was working and suddenly stop working.
I have this domain in route 53 and it works for amazon workmail.
But in the case of amplify is not working.
I have reviewed the domain but I don't know what else to do.
Error message
Error message: One or more of the CNAMEs you provided are already associated with a different resource.
Additional information
No response

Is this domain linked with any other resource like CloudFront or API Gateway Custom domain in your account?
Also refer below Knowledge Center article:
https://aws.amazon.com/premiumsupport/knowledge-center/resolve-cnamealreadyexists-error/

I linked it automatically with the tools that the Amazon workmail provides.
They do it automatically so I don't know how to see it.
When I go inside CloudFront there is nothing there.
El mié, 11 ene 2023 a las 5:58, anurag128 @.***>)
escribió:

Is this domain linked with any other resource like CloudFront or API
Gateway Custom domain in your account?
Also refer below Knowledge Center article:
https://aws.amazon.com/premiumsupport/knowledge-center/resolve-cnamealreadyexists-error/
—
Reply to this email directly, view it on GitHub
https://github.com/aws-amplify/amplify-hosting/issues/3222#issuecomment-1378240913,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACGSVQO32W5SRA7RYW3FOXLWRY4YDANCNFSM6AAAAAATO5SOUU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi @Elektroi can you provide your app ID or the custom domain you're trying to add so we can see what is causing the error?

The domain is abaraskaetxea.eus and this is the arn of the application
arn:aws:amplify:eu-west-1:719270881588:apps/db3kv6rw5o8ey
El mié, 1 feb 2023 a las 18:12, hloriana @.***>)
escribió:

Hi @Elektroi https://github.com/Elektroi can you provide your app ID or
the custom domain you're trying to add so we can see what is causing the
error?
—
Reply to this email directly, view it on GitHub
https://github.com/aws-amplify/amplify-hosting/issues/3222#issuecomment-1412418355,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACGSVQJPVV3XW4EWJ7TGPODWVKKRPANCNFSM6AAAAAATO5SOUU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Elektroi 👋 , thanks for providing us that information. The domain abaraskaetxea.eus is already connected to an Amplify app d2m33ds2ik6p0x which resides in the same AWS Account as Amplify app db3kv6rw5o8ey.
If you would like to connect this domain to the Amplify app db3kv6rw5o8ey, you will then need to first delete the domain association from the Amplify app d2m33ds2ik6p0x and then retry the domain activation process in the Amplify Console.
Just a side note, I went ahead and have redacted the AWS Account ID from your previous response due to security concerns.

I removed and created again.
When it arrives to domain activation appears:
Error message: One or more of the CNAMEs you provided are already
associated with a different resource.
Yes that domain is associated with the Amazon Workmail but I suppose that
they are not in conflict...
El sáb, 18 feb 2023 a las 7:35, Jay Raval @.***>)
escribió:

@Elektroi https://github.com/Elektroi 👋 , thanks for providing us that
information. The domain abaraskaetxea.eus is already connected to an
Amplify app d2m33ds2ik6p0x which resides in the same AWS Account as
Amplify app db3kv6rw5o8ey.
If you would like to connect this domain to the Amplify app db3kv6rw5o8ey,
you will then need to first delete the domain association from the Amplify
app d2m33ds2ik6p0x and then retry the domain activation process in the
Amplify Console.
Just a side note, I went ahead and have redacted the AWS Account ID from
your previous response due to security concerns.
—
Reply to this email directly, view it on GitHub
https://github.com/aws-amplify/amplify-hosting/issues/3222#issuecomment-1435499588,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACGSVQPB625NOKDY6TGOK6TWYBUSBANCNFSM6AAAAAATO5SOUU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hello! I hope you don't mind me piggybacking on this issue - I'm having a similar issue as well.
Jay, if you don't mind me asking, how do I check if a domain is already connected to another Amplify app or Cloudfront distribution? I went into the Cloudfront page, but it says no distributions currently exist - I'm not sure how to delete whatever it is that's conflicting with my current domain.

So I ended up making a ticket and it turns out there was an identical Amplify distribution in a different region (North America).
I have absolutely no memory of doing this - I'm in the APAC region, but I must have done so at some point.
You might have to go through Amplify manually in each region to check for other apps with the same name thay may have been deployed there

@Elektroi I tested a scenario with using the same domain for an Amplify app and Amazon Workmail organization and there was no conflict with CNAME records. In your case, there is another Amplify app already associated with abaraskaetxea.eus. Please make sure you check your other Amplify applications including any apps in other regions to ensure you properly remove the current domain association.


User adjusted opacity

User can adjust the opacity of a layer

i would like to work on it @GMMULLER please assign this to me

Thank you! Just a reminder, create a new branch from master attached to this issue.


drivers: gnss: gnss_u_blox_m10: fix compiler warning

Fix warning on implicit declaration of function 'malloc' when compiling
gnss_u_blox_m10.c.

Need to check how to set up Signed-off-by properly


Property 'size' does not exist on type 'MMKV'

// get size of MMKV storage in bytes
const size = storage.size
if (size >= 4096) {
// clean unused keys and clear memory cache
storage.trim()
}
in here storage.size is undefined...

size is part of V3, which requires new arch


🐞 [Bug]: can not access funkwhale vm while the solution is already working

Is there an existing issue for this?

[X] I have searched the existing issues

which package/s did you face the problem with?
Dashboard
What happened?
deployed a funkwhale vm but could not access it using mycelium or yggdrasil ip
Steps To Reproduce
No response
which network/s did you face the problem on?
Dev, Main
version
latest on each network
Twin ID/s
No response
Node ID/s
No response
Farm ID/s
No response
Contract ID/s
No response
Relevant screenshots/screen records

it shows the trial of both ips (mycelium binary is running in the background)

Relevant log output
na

Blocked on https://github.com/threefoldtech/tf_operations/issues/2473


IMN-489 - Refactoring auth middlewares

Closes IMN-489
WIP

Closing, old and stale, will be done from scratch when working on https://pagopa.atlassian.net/browse/IMN-489


[Phrase] function getByCommandAndName should be accepte Command type not CommandContext

Hi
I think this function should be accepte type of command not CommandContext because when you make a new command you pass command in the me function (in readme.md) example that is not CommandContext you pass it's command type.
https://github.com/dawalters1/wolf.js/blob/9cc8dedd6f0b2e3816f7b5120ce064ed5273a7b7/src/helper/phrase/Phrase.js#L122-L132

command prams here it's Command type not CommandContext
https://github.com/dawalters1/wolf.js/blob/9cc8dedd6f0b2e3816f7b5120ce064ed5273a7b7/README.md?plain=1#L158-L179

In WJS 2 Command is referred to as Command Context in this case


First draft of Type Coercion CIP

It might be nice if this document summarised the existing coercions (i.e. what the Neo4j implementation already does) and then also listed a set of alternatives for modifying that, where each alternative may add and/or remove coercions from the specification.

@cleishm It does summarise the existing coercions. It's not intended to modify anything, but to clearly document the current language behaviour, as a resource to refer to when considering implementation.

@Mats-SX Ah, right. It's under "proposal".
It might be worth using a different format for this, since there is no proposal - it's capturing what the existing language semantic is. That'll make it more obvious too, since I totally missed that one line (which is actually the point of the whole document!).

I'll leave that to the CLG to decide. This CIP is done at their request.


I have something for embedly, Idk how to contact u, so I am typing here

Whatever it is named, It is an embed code generator for scratch projects. U have 2 options.. a compiled Turbowrap embed and a normal scratch embed
See this: https://sparkscratch-p.github.io/AI-HTMLifier/
Source Code: https://turbowarp.org/448141362/editor
Get it on TurboWrap:
https://turbowarp.org/embed.html#448141362
Released on Forkphorus:
https://forkphorus.github.io/app.html?id=448887779&controls
Forum Post: https://scratch.mit.edu/discuss/topic/457086/?page=1#post-4597366
Scratch Wiki Page:
https://en.scratch-wiki.info/wiki/Porting_Scratch_Projects#Porting_SB3

If possible, can I join ur organization?


TFProf is unable to parse the profile generated by profileContext.

TFProf is unable to parse the profile generated by profileContext.
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution : Linux Ubuntu 18.04:
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): 1.11
Python version: 3.6
Bazel version (if compiling from source): 0.16.1
GCC/Compiler version (if compiling from source): 7.3
CUDA/cuDNN version: 10.0
GPU model and memory: V100, 16GB

Describe the current behavior
TFProf unable to parse the profile generated by profileContext.
Describe the expected behavior
When I generate the profile file by using the low level approach of runOptions, runMetadata, and then writing the string generated by profiler.serialize_to_string(), I can load it with tfprof and analyze it just fine. This should happen even when using ProfileContext.
Code to reproduce the issue
with tf.contrib.tfprof.ProfileContext('./profiles', trace_steps=range(50,100), dump_steps=[100]) as pctx:

 ... Regular session.run

I tried this twice, one where I killed the program after the trace was dumped. Other where I made the training loop exit cleanly.
/home/ubuntu/tensorflow/bazel-bin/tensorflow/core/profiler/profiler
--profile_path=profiles/profile_100
Reading Files...
Failed to parse profile
tfprof>
Cannot start interative shell, use 'bazel-bin' instead of 'bazel run'.
(tf_p36_cu10) ubuntu@ip-172-31-43-37:~/OpenSeq2Seq$

(tf_p36_cu10) ubuntu@ip-172-31-43-37:~/OpenSeq2Seq$ tfprof --profile_path=profiles/profile_70
/home/ubuntu/tensorflow/bazel-bin/tensorflow/core/profiler/profiler
--profile_path=profiles/profile_70
Reading Files...
[libprotobuf ERROR external/protobuf_archive/src/google/protobuf/wire_format_lite.cc:626] String field 'tensorflow.tfprof.ProfileProto.IdToStringEntry.value' contains invalid UTF-8 data when parsing a protocol buffer. Use the 'bytes' type if you intend to send raw bytes.
Failed to parse profile
tfprof> op -select micros,bytes,occurrence -order_by micro
Cannot start interative shell, use 'bazel-bin' instead of 'bazel run'.
(tf_p36_cu10) ubuntu@ip-172-31-43-37:~/OpenSeq2Seq$ tfprof --profile_path=profiles/profile_100

Wish I could help, maybe another TensorFlow contributor is more familiar with the C++ behind TFProf?

Wish I could help but I do not know much about the C++ interface.

@shlens @ChrisAntaki   -  Could you please suggest a right person to assign this issue ?

Nagging TensorFlower: It has been 14 days with no activity and the awaiting tensorflower label was assigned. Please update the label and/or status accordingly.

Hi @rahul003 !We see that you are using old version of Tensorflow which is officially considered as end of life, We recommend that you upgrade to 2.4 or later version and let us know if the issue still persists in newer versions .Please open a new issue in case you face any errors, we will get you the right help .Thanks!


implement i64/u64 using BigInt

I tried to implement this feature myself, but I literally am using rust for the first time today.
Serializing the data is simple, by now it's implemented in serde. Sadly I have no idea how to implement the corresponding deserialization of it. Don't want to create a halfhearted PR tho...

BigInt used to be unsupported by wasm-bindgen itself, but searching among their issues, I found https://github.com/rustwasm/wasm-bindgen/pull/2629 that finally implemented it recently.
Among other things, it's added BigInt bindings in js-sys: https://docs.rs/js-sys/0.3.55/js_sys/struct.BigInt.html
Those bindings have conversions from u64/i64 which you can use for deserialization: https://docs.rs/js-sys/0.3.55/js_sys/struct.BigInt.html#impl-From<i64>
I'm actually not sure how to do serialization yet - I don't see relevant APIs - but if you already figured out how to implement it as well, that's great.
PR would be welcome (and welcome to Rust! 😀).

Implemented in #24.

Hi @RReverser,
Refering to https://github.com/rustwasm/wasm-bindgen/issues/2350 so as I a have an issue with u64 may I ask
this works well using  serde-wasm-bindgen = "0.5.0"
        log(&format!(
            "test {:?}",
            serde_wasm_bindgen::to_value(&rand::thread_rng().gen::<u64>().to_string())
        ));

prints
test Ok(JsValue("341827912451372972"))
and this too
        log(&format!(
            "test {:?}",
            serde_wasm_bindgen::to_value(&rand::thread_rng().gen::<f64>())
        ));

however this prints
test Err(Error(JsValue(Error:<PHONE_NUMBER>75377644 can't be represented as a JavaScript number Error:<PHONE_NUMBER>75377644 can't be represented as a JavaScript number
        log(&format!(
            "test {:?}",
            serde_wasm_bindgen::to_value(&rand::thread_rng().gen::<u64>())
        ));

what do I miss here please ? as both wasm-bindgen and serde-wasm-bindgen are supporting u64 to BigInt.
Thanks

@gRoussac See serde-wasm-bindgen docs, in particular:

Serializer configuration options
[...]
.serialize_large_number_types_as_bigints(true): Serialize u64, i64, usize and isize to bigints instead of attempting to fit them into the safe integer number or failing.

@RReverser thank you my eyes did not pass the conversion table, I missed what was below, rtfm fatigue.
So for the curious
from
let json = serde_wasm_bindgen::to_value(&data);

to
let serializer = Serializer::new().serialize_large_number_types_as_bigints(true);
let json = data.serialize(&serializer);

and now I also understand why maps were not objects too =)
thank you very much

and now I also understand why maps were not objects too =)

At that point the variable also shouldn't be called JSON to avoid confusion :)

Yes indeed as far as I understand it is still a JsValue but does not stringify as JSON on javascript side, the "json" var in my use case is actually a complex struct where the u64 is nested into.
Now actually I am wondering if I use the correct approach as the doc says that if you don't need Maps/Sets etc then
#[wasm_bindgen]
pub fn send_example_to_js() -> JsValue {
    let mut field1 = HashMap::new();
    field1.insert(0, String::from("ex"));
    let example = Example {
        field1,
        field2: vec![vec![1., 2.], vec![3., 4.]],
        field3: [1., 2., 3., 4.]
    };

    serde_wasm_bindgen::to_value(&example).unwrap()
}

and
#[wasm_bindgen]
pub fn send_example_to_js() -> JsValue {
    let mut field1 = HashMap::new();
    field1.insert(0, String::from("ex"));
    let example = Example {
        field1,
        field2: vec![vec![1., 2.], vec![3., 4.]],
        field3: [1., 2., 3., 4.]
    };

    JsValue::from_serde(&example).unwrap()
}

are approx equivalent but last is JSON directly so I need to test now in my use case.
Thanks again for your help.

directly

Well "directly" means it uses serde-json on Rust side to serialize to JSON, and then JS parses that string back. It might be sometimes faster and sometimes not, but at least in terms of size it tends to add more overhead. Measuring on your own app is the only way to know for sure.


Constructor in subCollection Error: Type 'typeof foo' is not assignable to type 'new () => IBarInterface'

I have a TypeScript class Game with a subCollection class Team.
// Game.ts
import {Entity, rootCollection, subCollection, ICollection} from 'firebase-firestorm';

@rootCollection({
    name: 'games'
})
export default class Game extends Entity {
    @subCollection({
        name: 'teams',
        entity: Team  // the error refers to this line
    })
    teams: ICollection<Team>;
}

// Team.ts
import {Entity, field} from "firebase-firestorm";

export default class Team extends Entity {
    @field({name: 'name'})
    name: string

    constructor(name: string) {
        super();
        this.name = name;
    }
}

As soon as I add a constructor to the Team class I get the following TypeScript error:
Error:(22, 9) TS2322: Type 'typeof Team' is not assignable to type 'new () => IEntity'.

I don't have this error if I don't have a constructor within the Team class.
Is it not possible to use a constructor within a subCollection class?

Turns out you have to add a default value to the parameter of the constructor as described here to get rid of this error.
So I changed my class Team into this:
export default class Team extends Entity {
    @field({name: 'name'})
    name: string

    constructor(name: string = '') {
        super();
        this.name = name;
    }
}

I don't know why this is needed but it works...
Seems to be an issue with TypeScript and not firebase-firestorm.


⚠ Bofrost.be has degraded performance

In be91ee4, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 882 ms

Resolved: Bofrost.be performance has improved in 6012d46.


Runing NPM

I'm trying to run npm (laravel mix) on my project. I have a "install.php" on my project, for use on localhost, to install dependencies, generate key, etc:
if (file_exists("../.env")) {
    echo "Aplicação já instalada!! Abortando...";
    exit;
} else {
    exec("composer update");
    exec("composer install");
    exec("npm install");
    exec("cp ../.env.example ../.env");
    exec("php artisan key:generate");
    exec("php artisan db:seed");
}

But when he try to install NPM, i'm get the following message:

/bin/sh: npm: not found

I'm connected on the container by VSCode Docker plugin.
Thanks for any help.

npm is available on the node image, not on the php one.
use the node image for installing the npm dependencies.

How i do that with PHP package? only add a new service in and the link in the App?
Thanks for your answer


Extract Genesis types to standalone crate in alloy

Describe the feature
move all genesis types to new crate in alloy https://github.com/alloy-rs/alloy
https://github.com/paradigmxyz/reth/blob/93c0eeac171f20234ca8cae59f8ebdd9a21dd994/crates/primitives/src/genesis.rs
Additional context
No response

I can take this

let's do this in two steps
first copy to alloy-genesis then replace in reth

Note that I already ported it from ethers here: https://github.com/alloy-rs/alloy/blob/ccc6c9340be86cf6c054776747daf70ce18cc4aa/crates/node-bindings/src/genesis.rs#L1
Will need to be updated there too

should be same format, yes


Send out fresh invitations

Is your feature request related to a problem?
As per https://github.blog/changelog/2020-02-05-self-expiring-repository-and-organization-invitations/, invitations will expire after 7 days.
After #39, pending invitations are not automatically refreshed by the automation.
Therefore, we might end up having external collaborators who didn't accept the invitation within the deadline and thus being no longer able to join if we rely purely on the automation. Manual intervention is required instead.
The solution you would like to have available
Make available a manually triggered workflow that forwards again the invitations throughout the selected repos as per the following input:

none: don't send invitations anew (default).
*: send out fresh invitations for external collaborators on all repos.
comma-separated-list of repos (e.g., repo-1, repo-2): send out fresh invitations only for the specified repos.

Alternatives you have considered
No response
Additional context
No response

Implemented in #71.


Added support for nkeys, user jwts, and signing.

Added nkey, jwt, and sig support.
Added auto-upgrade to tls.
Fixed new style request emit unsubscribe event (for new node-req example)
Added simplified connect with second parameter option creators. We can use the nats.cred(filepath) here. e.g. nats.connect(url, opts)
Signed-off-by: Derek Collison<EMAIL_ADDRESS>

Coverage decreased (-7.09%) to 88.527% when pulling c903e98af929d99542b98657956f7a7338a27a71 on ngs into 622c123f0cae2fcbba9f795a6cdc942236d5161e on master.

LGTM - Not sure why we are emitting unsubscribe on a mux subscription for request-reply but sounds like it could be useful.


OpeoNodes.java:90-94: Parse AST from high-level XMIR....

The puzzle 37-bf080a14 from #37 has to be resolved:
https://github.com/objectionary/opeo-maven-plugin/blob/5bd26e5462344d2605c737e501fabff5a08d8ccd/src/main/java/org/eolang/opeo/compilation/OpeoNodes.java#L90-L94
The puzzle was created by @volodya-lombrozo on 11-Jan-24.
Estimate: 90 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0pdd 5 puzzles #74, #75, #76, #77, #78 are still not solved.

@0pdd 6 puzzles #74, #75, #76, #78, #80, #81 are still not solved; solved: #77.

@0pdd 6 puzzles #74, #75, #80, #81, #86, #87 are still not solved; solved: #76, #77, #78.

@0pdd 6 puzzles #74, #75, #80, #81, #87, #89 are still not solved; solved: #76, #77, #78, #86.

@0pdd 5 puzzles #74, #75, #80, #81, #87 are still not solved; solved: #76, #77, #78, #86, #89.

@0pdd 6 puzzles #74, #75, #81, #87, #97, #98 are still not solved; solved: #76, #77, #78, #80, #86, #89.

@0pdd 5 puzzles #74, #75, #81, #87, #97 are still not solved; solved: #76, #77, #78, #80, #86, #89, #98.

@0pdd 5 puzzles #134, #74, #75, #87, #97 are still not solved; solved: #76, #77, #78, #80, #81, #86, #89, #98.

@0pdd 5 puzzles #134, #153, #74, #75, #87 are still not solved; solved: #76, #77, #78, #80, #81, #86, #89, #97, #98.

@0pdd 4 puzzles #134, #74, #75, #87 are still not solved; solved: #153, #76, #77, #78, #80, #81, #86, #89, #97, #98.

@0pdd 3 puzzles #134, #74, #75 are still not solved; solved: #153, #76, #77, #78, #80, #81, #86, #87, #89, #97, #98.

@0pdd 2 puzzles #74, #75 are still not solved; solved: #134, #153, #76, #77, #78, #80, #81, #86, #87, #89, #97, #98.


ESP32 clock runs fast

The ESP32s clock is very inaccurate (eg 10 seconds in 40 minutes in one test) due to the default 150 kHz source. There is an option to use an internal 8.5 MHz source which requires recompiling the Arduino/ESP32 core. This is meant to be more accurate so worth a try.
The simplest fix we can put in is to connect to the internet and do a time sync using NTP or the mobile network every hour to keep this drift in check.
More complicated fixes are to either include a 32 kHz crystal on the board which takes 2 ESP32 pins, or to include a separate RTC and crystal which will need to be communicated with via I2C or SPI.

@dajtxx
The board has an onboard 32 kHz crystal already.
It connected to pins 8 and 9 on the schematic (https://github.com/DPIclimate/enviro-node/blob/master/electronics/motherboard/enviro-node/schematics/microcontroller.pdf).

So it does! Well done :)
Now to recompile the ESP32/Arduino core. Reference:
https://github.com/espressif/esp32-arduino-lib-builder/issues/102

See commit f27495d187aaae2fa22da989a00f72f47fc6b80a
The code to set the RTC source was copied from the Arduino core and modified to only use the external crystal. Given it's only feeding the RTC is should be safe to call from setup().
A clock skew adjustment parameter has also been added to the config class, and some code added to where the time is taken from the modem to try and keep the RTC in line.


breakout_as7262 init fails since moving to common i2c

Prior to moving to common i2c the breakout_as7262 example worked perfectly, but now seems to fail during init call.

Apologies my bad!  When looking at the example again I see you now need to specify the board type in the I2C constructor. The example is set to BREAKOUT_GARDEN, but as I'm using the Pico Explorer I needed to change the value to PICO_EXPLORER which I have now done and it works fine.  So no issue. Thanks.

Has anyone got the pico_pot_explorer example working on the latest version?  Seems to be a conflict between pico_explorer and breakout_potentiometer.  Comment either the pico_explorer or the breakout_potentiometer calls out and they work individually, but not at the same time.

Apologies my bad! When looking at the example again I see you now need to specify the board type in the I2C constructor. The example is set to BREAKOUT_GARDEN, but as I'm using the Pico Explorer I needed to change the value to PICO_EXPLORER which I have now done and it works fine. So no issue. Thanks.

Music to my ears. The common I2C shift was to solve this exact problem because I got caught out over and over again by using the wrong pins on the wrong board.
I wonder if explorer and the potentiometer breakout are fighting over a common pin. Does the example outright crash?

Just ran a quick test and I see the same thing happening. No time to fix it now (or, uh, potentially until the week after next) but there's definitely something awry here!

Yes it just crashes. Prior to the common I2C shift the example worked. It came about as I was trying the BME680 breakout and PICO explorer. I could get each to work individually, but not at the same time so I went back to the examples and found the same issue with the pico_pot_explorer to make sure it wasn't something I was doing.
To add to the mystery the pico_explorer example also fails now. It used to work with no intervention. I have to comment out the msa301.init() call and it starts working.  I suspect the root cause is the same.

Thanks for the update. Glad you were able to recreate it and I wasn't just me going mad.

Been poking this when I can grab a half hour now and then, and finally found and fixed in #165
The Pico Explorer library was trying to use pin 20 for a non-existent backlight connection, which coincided with the SDA pin for Pico Explorer's I2C bus. As you might imagine that breaks I2C pretty handily- forcing it into a perpetually locked state.
Explains the weird pulse I was seeing on the logic analyzer!

Thanks for taking the time to investigate. I've just synced with the latest version and Pico_explorer,  as7262_explorer_bargraph and pico_pot_explorer examples are all fully functioning again :) Thanks again.


Endpoint: Unfavorite Article

DELETE /api/articles/:slug/favorite
Authentication required, returns the Article
No additional parameters required

Feature is done, but there is a potential bug with txn library: https://issues.couchbase.com/browse/TXNN-134
This also affects Favoriting, not just Unfavoriting


Correctly normalize rails paths with optional bound params

Rails supports optional bound parameters.
When using optional params, the generated paths by this gem look something like: /optional_params/(/{id)} instead of the expected /optional_params/(/{id})
If you use tools like widdershins to generate docs from your OpenAPI specification, this can break the parsing.
This PR updates the regex to not include the closing parentheses in the capture group during path normalization.

The official OpenAPI Spec does not support optional path parameters

The official OpenAPI Spec does not support optional path parameters


pkg-config audit pass

We should properly normalize pkg-config .pc files across our generated products.  pkg-config offers many advantages to compiled software projects, but the most important is that it provides an automated method for querying the proper compiler flags for linking against a particular package.  BB so far is mostly ambivalent towards these packages, but we can improve our good-neighborlyness through two steps:

First, if a .pc file is generated by a project, we should ensure it is relocatable.  This is important not just at build time (due to shifting prefix location thanks to platform-specific $destdir locations) but also after installation on a user's machine, in case they want to build something locally and link it against the installed artifacts.  We can ensure relocatability by rewriting .pc files to use ${pcfiledir} as the equivalent of @__DIR__, and rewriting all paths as relative to that directory.  I got this idea from Slide 17 of this presentation, there's other good stuff in there as well.

Second, if a software package contains no .pc files..... we should probably create one that at least declares the proper -L and -I flags.  Something like:

prefix=${pcfiledir}
libdir=${prefix}/lib
includedir=${prefix}/include

Name: $JLL_NAME
Description: $JLL_NAME
Version: $SRC_VERSION
Requires: $dep1 $dep2...
Libs: -L${libdir} -l$lib_product1 -l$lib_product2
Cflags: -I${includedir}

This is very interesting, but I don't understand why the issue is in this repo: audit passes are in BinaryBuilder :grimacing:

This is very interesting, but I don't understand why the issue is in this repo: audit passes are in BinaryBuilder :grimacing:

.....okay now you can show me how to transfer issues.

.....okay now you can show me how to transfer issues.


rover_pos_control: revert sensor_combined purge

Quick test to see if this fixes https://github.com/PX4/Firmware/issues/15628 (in CI at least).

The structure and scheduling of rover_pos_control is a bit weird, I'll review it in detail later and move to new uORB::Subscription and a WQ.


custom partition Exception

Custom partition is a very important feature for me. So I updated ch and  created a table by custom partition. But ch server is down for some reason. When I restart CH.Exception come add told me that I have used invalid partition. I add .  set experimental_merge_tree_allow_custom_partitions=1 before table metadata, it also not work.  Does it has another way to start my ch server ?

Hi @domsia ! First, I must repeat the warning found in the source code: custom partitions are not ready for production yet! The data and metadata formats will change in the future and the future server versions WILL NOT be able to load tables created with the current version.
With that in mind, I would be very grateful if you experiment with the feature and provide feedback on how it fits your use case. To allow creating and attaching tables with custom partitions you should enable experimental_merge_tree_allow_custom_partitions for the default user profile in the users.xml server config file.

Thank you for solving the problem. I have started my ch server by hands. I deleted the table's metadata  SQL and started the server. After that, I set the experimental_merge_tree_allow_custom_partitions and attached table by hands. It works too.


Added support of "long" RTS, making short RTS be "hist_feat_dynamic_real"

Issue #, if available:
Description of changes:
I added support for related time series (RTS) that are accessible in the forecast horizon. I changed the syntax so that 'feat_dynamic_real' now means "long" RTS (having info in the forecast horizon), and 'hist_feat_dynamic_real' refers to "short" RTS (having no info in the forecast horizon). This is a change of syntax from before in the sense that before there was only support for 'feat_dynamic_real', which was assumed to be short, not long.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Please tag this pr with at least one of these labels to make our release process faster: BREAKING, new feature, bug fix, other change, dev setup

Let’s call them past_feat_dynamic_real to be consistent with MQ-CNN.


scipy.optimize._linprog_simplex._apply_pivot relies on asymmetric closeness definition

We are currently looking into using a symmetric closeness definition for numpy.isclose. To judge the impact this move would have on other libraries we ran scipy's test suite against a modified numpy.
While doing this a lot of tests emitted this warning
https://github.com/scipy/scipy/blob/0617428d9fb1d57fe8b0808291818e61b00efdbf/scipy/optimize/_linprog_simplex.py#L219-L229
because it relies on the asymmetric definition.
I've tracked it down to #9081 and specifically https://github.com/scipy/scipy/pull/9081#discussion_r205899909 (cc @Kai-Striega @mdhaber). If I understand the intention correctly it should check if the pivval is significantly greater than the tolerance. If that is the case
if abs(pivval) <= tol * 1e4:
    ...

would be a better choice, because

it communicates the intent better, and
is safer since np.isclose basically checks abs(pivval - tol) <= ... which might lead to unwanted behavior in an edge case.

That's OK if it causes this test to fail. simplex is going to be deprecated anyway.


The ‘balloon-eval’ doesn't work properly on Windows ...

Environment:
Windows 7 64-bit SP1
Command:
gvim.exe -u NONE -U NONE --noplugin -c "set balloondelay=100 balloonexpr=v:beval_text ballooneval" ..\_vimrc
The balloon tip just can show a few times, and then it doesn't work any more ...

can you be a bit more precise? What exactly do you have to do to make the balloon disappear?

I use balloon tip to translate english, but I found it only show only 3~5 times, and then don't show any more.
The command "balloonexpr=v:beval_text" is just a test example ...

I also confirm this issue on Windows 10 64-bit (GVIM version 8.0.69)
You can change to balloonexpr=expand('%'), the function "expand('%')" return current buffer name.
Then move mouse point to any words, see it show up the tip, may be 1~n times will not show again ...
Can anyone also confirm this ? :(

Hi,
I confirm on gvim 64bit compiled issue 8.0.427 on windows 10 with this example:

Maybe fixed by 8.1.0652?

I tested 8.1.0652 and nothing different …… :(

Maybe fixed by 8.1.0652?

I tested 8.1.0652 and nothing different …… :(

Maybe fixed by 8.1.0652?

I tested 8.1.0652 and nothing different …… :(

possibly related: Patch 8.1.1300

possibly related: Patch 8.1.1300

I tested 8.1.1328 and still nothing different …… :(

possibly related: Patch 8.1.1300

YA! I found I can input call balloon_show('') to temporary fix it!

In my case WM_MOUSEMOVE is sent event every +/- 100ms  with the same coordinate thus causing the balloon not to appear because the idle detection is not working. It can be fixed by checking the mouse coordinate in TrackUserActivity in gui_w32.c.
static POINT s_mousemove;

 static void
TrackUserActivity(UINT uMsg)
{
    if (uMsg == WM_MOUSEMOVE) {
	POINT p;
	GetCursorPos(&p);
	if (p.x == s_mousemove.x && p.y == s_mousemove.y) {
	    return;
	}
	s_mousemove.x = p.x;
	s_mousemove.y = p.y;
    }

    if ((uMsg >= WM_MOUSEFIRST && uMsg <= WM_MOUSELAST)
	    || (uMsg >= WM_KEYFIRST && uMsg <= WM_KEYLAST))
	LastActivity = GetTickCount();
}

There is also code for WM_MOUSEMOVE in HandleMouseHide().  Can they be combine?

I don't think so since HandleMouseHide is called in a lot of places, but we can extract the logic to detect mouse movement out and then use it in both HandleMouseHide and TrackUserActivity.


Any way to add a warning when comparing with "==="?

It's very easy to accidentally try to compare two BigNumbers with === instead of .isEqualTo().
Is there any way to throw an error or show a warning when doing this? I understand it might not be possible within the library, but maybe a custom ESLint rule?

Is there any way to throw an error or show a warning when doing this?

I don't think so.

maybe a custom ESLint rule?

Sorry, but I can't help with that.


How to filter one to many relation?

Here is my code for filtering
  const getList = async (
    page,
    perPage,
    location = 'mdk7ej7t9oiqxcy',
    service = ''
  ) => {
    return useAsyncData(async (nuxtApp) => {
      const record = await nuxtApp.$pb
        .collection('profile')
        .getList(page, perPage, {
          sort: 'created',
          expand: 'user, service, operation_areas',
          filter: `service="${service}" || operation_areas="${location}"`,
        })
      return structuredClone(record)
    })
  }

service work as it is one-to-one relation but operation_areas is one-to-many relationship it is not work as its array of ids
SELECT COUNT(DISTINCT [[profile.id]]) FROM `profile` WHERE (([[profile.service]] = '' OR [[profile.service]] IS NULL) OR [[profile.operation_areas]] = 'mdk7ej7t9oiqxcy')

also did not work when I use only   filter: ` operation_areas="${location}"`,

Multi-relation field values are stored as serialized json, so if you want to check only whether one of the relation has the location id, you can do something like:
filter: `service="${service}" || operation_areas.id ?= "${location}"`,

(the ? operator prefix converts the expression to "any/at least one of" condition)

Thanks for quick response. One more thing it did not work when default value is  empty

sure
const getList = async (page, perPage, service = '', location = '') => {
    return useAsyncData(async (nuxtApp) => {
      console.log(service, location)
      const record = await nuxtApp.$pb
        .collection('profile')
        .getList(page, perPage, {
          sort: 'created',
          expand: 'user, service, operation_areas',
          filter: `service="${service}" || operation_areas.id ?="${location}"`,
        })
      return structuredClone(record)
    })
  }

location and service is empty string

Sorry, I'm still not sure that I understand what do you mean.
In case location is empty, the filter will require service="${service}" to be true OR return only records with empty operation_areas.
I've tested it locally with a sample multi-relation field and similar condition and it works fine for me. If you want to disallow returning records with empty operation_areas you can add an extra check, but please note that this is true also for service field if it is not required:
service="${service}" || (operation_areas:length >0 && operation_areas.id ?="${location}")

If you are not experiencing this, please file a new Q&A discussion with more details about your setup - PocketBase and JS SDK versions and ideally a minimal reproducible example with exported collections configuration (you can export your collections from the Admin UI > Settings > Export collections).


Guided Scenario Builder

Guided assistance feature to help inexperienced editors create new scenarios

Likely an option that can be selected when creating a new scenario. Also should be able to turn this feature on or off within the editor interface

BACKLOGGED: Ran out of time to implement; needed to focus on integrating with the backend team to set up new endpoints that worked with the new schema before deploying changes to the server.

This is a quality of life issue. However, the customers have made it clear that this is a top-priority QoL feature.