multi: query chan update timestamps

In this PR, we start supporting the timestamps query option in the query_channel_range message. If we receive such a message with that bit set then we now populate the timestamps field of the reply_channel_range message.
With this feature we can now ask for the timestamps for the latest channel updates for a channel to be sent along with the SCID. Thus, if we have marked one of the channels in the list as a zombie but we see that the timestamp sent with it make it not a zombie, then we request the channel_update messages for that channel so that we can perhaps awaken the channel.
How to test?
See this comment: https://github.com/lightningnetwork/lnd/pull/8030#issuecomment-1808145257
Fixes https://github.com/lightningnetwork/lnd/issues/7989
Depends on https://github.com/lightningnetwork/lnd/pull/8171

@ellemouton, remember to re-request review from reviewers when ready

@ellemouton, remember to re-request review from reviewers when ready

Still in the process of reviewing, but I think the chunkSize needs to be updated if timestamps are sent because instead of room for ~8000 scid's, there's only room for ~4000 because of the 2 4-byte timestamps

Great catch - thanks! Addressed 👍

Main comment re if we should attempt to sanity check the timestamps we get from the remote party.

👍  for now i've added a place holder that currently only checks that the times are not in the future. Will ponder more over if more can be done & if it is worth doing something more complicated like checking that what they end up sending matches the timestamps they claimed to have.

One way to test this out in the wild would be: create a new node on mainnet, no channels, set the zombie prune horizon to something short like 6 hrs, run this patch, and observe the zombie resurrection happen or not. We might want to add some extra logging statements wehn we realize we need to go fetch a zombie chan due to getting an update.

Good idea 👍  I'll do this next

Did the following manual testing:
Confirmed issue:

Set up a regtest env where there are a bunch of chans but main node, alice, has no chans but learns of all the other channels.
Set alice's ChannelPruneExpiry and GraphPruneInterval to low values so that the node quickly marks all the other channels as zombies. (a quick way to verify this is to do a describegraph and see that there are no edges.
Shutdown alice
now perform some channel updates on the existing channels so that there are some ChannelUpdates that alice misses while the node is offline.
Start alice , connect alice to a node in the network & see that Alice ignores all the SCIDs in the lnwire.ReplyChannelRange message due to them all being marked as zombies.  (describegraph still empty).

PR fixes issue:
Restart alice with this PR and perform the above steps. At step 5, alice will mark the channels as alive and will re-add them to her graph.

...
5. Start alice , connect alice to a node in the network & see that Alice ignores all the SCIDs in the lnwire.ReplyChannelRange message due to them all being marked as zombies.  (describegraph still empty).
PR fixes issue:
Restart alice with this PR and perform the above steps. At step 5, alice will mark the channels as alive and will re-add them to her graph.

Great explanation of the test 🤩
Concept ACK

@roasbeef: review reminder


undefined symbol: _ZN2df12KinectFusionC1ESs on trained model test script

Dear Dr. Xiang @yuxng ,
Thanks for the work. I am working on reproducing your code on my docker image.
Everything is compiled. I can run the test script without kfusion set to True. However, once set it to True, I got the error as follows:
+ set -e
+ export PYTHONUNBUFFERED=True
+ PYTHONUNBUFFERED=True
+ export CUDA_VISIBLE_DEVICES=0
+ CUDA_VISIBLE_DEVICES=0
++ date +%Y-%m-%d_%H-%M-%S
+ LOG=experiments/logs/rgbd_scene_multi_rgbd_test.txt.2018-06-14_23-27-31
+ exec
++ tee -a experiments/logs/rgbd_scene_multi_rgbd_test.txt.2018-06-14_23-27-31
+ echo Logging output to experiments/logs/rgbd_scene_multi_rgbd_test.txt.2018-06-14_23-27-31
Logging output to experiments/logs/rgbd_scene_multi_rgbd_test.txt.2018-06-14_23-27-31
+ '[' -f /home/weizhang/DA-RNN/output/rgbd_scene/rgbd_scene_val/vgg16_fcn_rgbd_multi_frame_rgbd_scene_iter_40000/segmentations.pkl ']'
+ ./tools/test_net.py --gpu 0 --network vgg16 --model data/fcn_models/rgbd_scene/vgg16_fcn_rgbd_multi_frame_rgbd_scene_iter_40000.ckpt --imdb rgbd_scene_val --cfg experiments/cfgs/rgbd_scene_multi_rgbd.yml --rig data/RGBDScene/camera.json --kfusion 1
Traceback (most recent call last):
  File "./tools/test_net.py", line 13, in <module>
    from fcn.test import test_net
  File "/home/weizhang/DA-RNN/tools/../lib/fcn/test.py", line 25, in <module>
    from kinect_fusion import kfusion
ImportError: /home/weizhang/DA-RNN/tools/../lib/kinect_fusion/kfusion.so: undefined symbol: _ZN2df12KinectFusionC1ESs

I am not sure if this is caused by compiling kinect fusion improperly. And comments are very appreciated.
And also @kevinkit , did you successfully run the script with kfusion to 1 in a docker?
Thanks for the help!.

I get the same error, any solution? @Wei2624 @yuxng

@rockkingjy I work it around by adding add_definitions(-D_GLIBCXX_USE_CXX11_ABI=0) to CMakeList.txt in KniectFusion. You can try it.


Fallback Icons not working

Hello Sir,
I am facing an issue that is when a language code provides on the dynamic component if this code is wrong or flag not settled it should return the fallback flag but it does not return that.
<x-dynamic-component component="flag-language-{{ $lang->code }}" />
or
<x-flag-language-zh-HK />

Hi @hellohasan,
Sorry for the delayed response. I think the issue lies with x-dynamic-component itself. If you want the fallback to work correctly, you should use the blade-icons package directives like x-icon or x-svg.
For example:
<x-icon name="flag-language-{{ $lang->code }}" class="w-6 h-6" />


Access child objects' properties

If there is a child object it's is not expanded into separate accessible fields:
parent: {
  child: {
    property1: "foo",
    property2: "bar"
}

The parent object can be flattened and the columns can be accessed in the "parent/child/property1"-like style
There is a good example of how it can be implemented (the object is flattened with its name transformed into a full property path):
https://www.convertjson.com/json-to-html-table.htm
Then on getJSON() it can be reassembled back

This turn more complex when you start to create new columns, new rows, etc.
But, that is available on the PRO version.
https://www.jspreadsheet.com/v8/docs/data
We don't have expectations to bring that to the CE version. If you wish, fell free to send a PR.


Document "Provides-Extra"

Provides-Extra" ( in dist-info METADATA files) is currently in "production" use for providing extras support for wheels. Both wheel and setuptools have modifications to support it.
It's an oddball in that it only lived in an earlier version of PEP426, before that was revamped to json.
I feel like it should mentioned publicly somewhere in PEP345 or PEP426, so at least it's searchable in the future to explain how it came to be like this.
In retrospect, I guess the "failure" that led to this oddity was in accepting PEP427 (wheel) w/o realizing that for it to be used in the real world of setuptools, it required a new metadata field.

migrated from https://bitbucket.org/pypa/pypi-metadata-formats/issues/29/document-provides-extra

+1 +1 +1

PEP 426 right before it was jsonified: https://hg.python.org/peps/file/3b67372b39ba/pep-0426.txt
A line in pkg_resources dealing with Provides-Extra: https://bitbucket.org/pypa/setuptools/src/4ce518784af886e6977fa2dbe58359d0fe161d0d/pkg_resources/init.py?at=default&fileviewer=file-view-default#init.py-2854
You might also be interested in just a few lines before that where we take the () off of the version numbers from METADATA!

An addendum to 345 likely makes the most sense - as you say, I missed that 427 actually needed this particular update, unlike the other changes in the earlier drafts of 426.

Given that I only got involved after PEPs 425/426/427 had already been written, I was intrigued to find that @dholth's original idea was actually to add Provides-Extra to PEP 345 as an after-the-fact standardisation of a de facto additional field: https://hg.python.org/peps/rev/4dc72bbf9ed2

In the context of https://github.com/pypa/pypa.io/issues/11, I submitted a PR that mentions Provides-Extra as a qualifier on the reference to PEP 345 as the current metadata specification: https://github.com/pypa/pypa.io/pull/12


Reviewer 3 Comments

Reviewer #3: This manuscript discusses ten simple rules for digital data storage. The ten rules provide simple guidelines for how researchers across a variety of disciplines can store their data for long-term access and ensure that the data are usable by others in the future. Compared with other "ten Simple Rules" papers on scientific data management and curation, this manuscript further addresses another facet of data, longer term storage best practices. The manuscript represents the collaboration among co-authors from a wide variety of academic backgrounds, who are all part of the Software Carpentry project.
The manuscript could benefit from addressing specific needs for the storage of biological and biomedical data to be more in scope with the main themes of the journal.

After some thought I've come down on the side that we shouldn't make any changes to this.  I'm of the opinion that very specific advice might lessen the value of the paper.  What do others think? Do folks have other ideas / thoughts about this?

I agree.  We can respond to the reviewer and point to the last sentence of the Intro.  And something along the lines of:
Since this paper represents the experience and views of a diverse group of authors, we have deliberately chosen to not make very specific recommendations for a given discipline.  Some of are examples are outside of the biological sciences; however, the rules we provide are very applicable to biology as well as other fields.  Further, we feel that very specific advice could lessen the value of the paper.
Or something like that...

I think we could add this to the response to reviewers and also add in a few more things about dealing with sequence data in a few places (see my comment on #138).


How can we exploit forced alignments?

Thank you so much for the work you have done in your tacotron implementation. I have a question if you may.
I have a speech corpus with time alignments. For each audio sample, I have a file that looks like this.
0.471000 121 sil
0.618000 121 Z
0.666000 121 i
0.716750 121 n
0.852974 121 a:
0.910125 121 z
0.987444 121 a
1.070000 121 t
1.130000 121 u
1.182000 121 l
What is the best tacotron implementation that can exploit this information?

I just rewrite this model in keras so you could use my implementation and add a method to create « alignment_target »... Not sure it’s easy but i think it’s easier to add an outpout in keras generator than in tensorflow (except if you code in tensorflow, of course)
My implementation seems correct and working but my model haven’t yet learned attention (i test no more than 10k steps, i finished the implementation a few days ago)


Turn off bluebird warnings for "runaway" promise.

The issue isn't causing any problems so I'm ok with just turning off the warning. This probably won't be a problem in V2 anyway.

Would prefer to fix it instead of turning of the warnings


Fix changing transport type when setting original system

Follow-up to #6715
Closes #6843
To-do: Documentation

@einareiriksson


Add note about ground-truth labels range

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

LGTM


Integration with h5ai (beta)

Not a real issue, just reporting  my experiment.
Today I tried integrating webamp with h5ai to replace the internal audio preview. It works, for the most part, but there are many rough edges (can't move windows, music continues after dismissing preview...). They are likely due to the preview framework of h5ai, maybe it would be better to bypass it entirely.
I just tought you'd like to know. Here's a screenshot:

And here's my test directory (not sure for how long I'll leave it online).
I didn't have to change webamp to get here, but something that could be useful for a better integration are functions to programmatically control playback, such as play(), stop(), enqueueAndPlay(), to be called on a specific instance.
Thanks for webamp and keep up the good work!

Very cool! Thanks for sharing, and sorry for my slow response. Now that you got it working, do you think it's useful? Would it be worth adding more API surface to Webamp to allow you to make the h5ai integration better?
Also, feel free to drop by our Discord channel and say hello! https://discord.gg/fBTDMqR

This was more of an aestethic/nostalgia thing for me, the preview provided by h5ai works fine already. Anyway, the integration could be improved with API to control the player from external code (play/stop/prev/next), so e.g. I can stop the music when the preview is dismissed.
Something that I'd really find useful is the integration with mStream (#398), also with the media library window where I can filter the songs based on arbitrary tag. What happened to that collaboration? There is a link of a "branch of mStream that will server winamp2-js", but it lands on a 404 error page. I could try to help if there is interest in it.

I think these methods have now been added. I'm going to close this issue. Feel free to open a new issue if there are still APIs missing that would enable this integration.


Celsius to Fahrenheit?

I live in the .us and I'd like to convert C to F -- is there a clever way to do this in the python scripts?
thanks!

I don't think there's a conversion function currently built into the library, but it's straightforward to do like this:
fahrenheit = (celsius * 1.8) + 32


Update 'Motion planning for a MoveIt 2 arm' link on Demos page

See https://github.com/osrf/ros2_test_cases/issues/959
The "Motion planning for a MoveIt 2 arm" link is being redirected from http://moveit2_tutorials.picknik.ai/ to https://moveit.picknik.ai/humble/index.html, so update the link.
There is https://moveit.picknik.ai/main/index.html (for Rolling), but there are no versions for other ROS 2 distros, so I just used the Humble version. Perhaps rolling could point to https://moveit.picknik.ai/main/index.html and Iron and older can point to https://moveit.picknik.ai/humble/index.html?

There is https://moveit.picknik.ai/main/index.html (for Rolling), but there are no versions for other ROS 2 distros, so I just used the Humble version. Perhaps rolling could point to https://moveit.picknik.ai/main/index.html and Iron and older can point to https://moveit.picknik.ai/humble/index.html?

That seems reasonable to me, but I'm not entirely sure.  @tylerjw can you comment here on what tutorials we should be pointing to for the different releases?

Thank you for tagging me. I am going to propose a change to that site tomorrow at our wg meeting that will affect this. The tldr is that the main branch of MoveIt supports humble/iron/rolling so we should stop directing people to any version of the tutorial site but the main one. I tried copying the ROS docs site last year with the multiversion stuff and am now regretting it because it just caused confusion.

Sounds good, I changed it to point to /main/.


play 'sorry' when a user passed an invalid command

Purpose

...

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[x] Feature
[x] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone git@github.com:clarkio/ttv-chat-light.git
cd ttv-chat-client
git checkout [branch-name]
npm install

What to Check
Verify that the following are valid

...

Other Information

Everything worked as expected. Thank you for contributing these changes. Let's just get some of that feedback we discussed on stream addressed re: the stlying and we'll be good to go to merge this.

@clarkio I have gone in and reverted the formatting changes as we discussed earlier today. Thanks again for the feedback!

Thanks so much for contributing this @ToeFrog and I appreciate how you separated the work in your PR's 👍


Enable 'RFC5952 IPv6 Formatting' to retain ipv6 compacted version

fixed and verified with https://github.com/open-traffic-generator/ixnetwork/pull/360


Should we change the case order for some of the Coercers - may improve performance?

For the HBase source and sink I am making use of the Coercers...for some of them I think the first case should always be the type you intend to return which is almost like an no-op, e.g. for the LongCoercer it looks like the following:
object LongCoercer extends Coercer[Long] {
  override def coerce(input: Any): Long = input match {
    case b: Byte => b
    case i: Int => i
    case l: Long => l
    case s: Short => s
    case s: String => s.toLong
  }
}

If you are calling this millions of times for large volumes then it might add up? e.g. move case l: Long => l to the top:

object LongCoercer extends Coercer[Long] {
  override def coerce(input: Any): Long = input match {
    case l: Long => l
    case b: Byte => b
    case i: Int => i
    case s: Short => s
    case s: String => s.toLong
  }
}

Also look at IntCoercer, FloatCoercer, DoubleCoercer,

Agreed. Good idea.
On 22 Feb 2017 11:01 a.m., "hannesmiller"<EMAIL_ADDRESS>wrote:

For the HBase source and sink I am making using of the Coercers...for
some of them I think the first case should always be the type you
intend to Coercers which almost like an no-op, e.g. for the
LongCoercer it looks like the following:
object LongCoercer extends Coercer[Long] {
override def coerce(input: Any): Long = input match {
case b: Byte => b
case i: Int => i
case l: Long => l
case s: Short => s
case s: String => s.toLong
}
}

If you are calling this millions of times for large volumes then it
might add up? e.g. move case l: Long => l to the top:

object LongCoercer extends Coercer[Long] {
override def coerce(input: Any): Long = input match {
case l: Long => l
case b: Byte => b
case i: Int => i
case s: Short => s
case s: String => s.toLong
}
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sksamuel/eel-sdk/issues/251, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAtZGnISLPKPstETnTKjB88NkkJX3VP4ks5rfBWDgaJpZM4MIfyu
.

I've adjusted the existing coercers.

That's great


Website down

Hello!
I just noticed that the website with a bunch of the documentation seems to be down - jsph.dev
Amazing project btw!

Moved to github pages.


Add Support for MQTT 5

🚀 Feature Request
Relevant Package [REQUIRED]
MQTT Client
Description [REQUIRED]
Current github.com/eclipse/paho.mqtt.golang module doesn't support MQTT 5
Describe the solution you'd like
The https://github.com/eclipse/paho.golang does and should also support MQTT 3.1
Describe alternatives you've considered
 Have you considered any alternative solutions or workarounds?

@cloudxxx8 , @jumpingliu, I looked at https://github.com/eclipse/paho.golang and it is not plug and play with the 3.x Client. This will require more refactoring of the MQTT implementation than we want to do in the Napa release. Suggest this be moved to the ice box.

@lenny-intel thanks for looking into it.  I agree with you, moving to icebox.

it's currently v0.12, and it contains some breaking changes against v0.11
we want to wait for v1.0, a stable release


[Subtask] Fix method-hidden Pylint rule for client-python

Describe the subtask

According to this doc, the name of attribute in class should not be as same as the method in the class, please try to rename one of them.

Parent issue
#3560

I am working on this issue, the reason is that the method name and attribute name in some classes is the same, e.g. class RenameCatalog in catalog_change.py.
For more details about this pylint checker, please see: https://pylint.readthedocs.io/en/latest/user_guide/messages/error/method-hidden.html


Xoutput freezes with my driver installed

Xoutput works completely fine without my controller driver installed but without it, xoutput wont detect the rumble motors on my controller according to a few replies I saw on the issues page. I installed the driver software which came in a disc and after installing it if I try to open xoutput the program opens, my mouse freezes and then it crashes.
I am attaching all the files in the driver disc below. The disc has an instruction docx, the driver itself and a folder with directx 9 bcs it looks like it needs directx 7 or above to run for some reason and even gives a warning to have it installed before installing the driver.
(the file is temporarily available for a month from the day of posting this.)
https://privfile.com/download.php?fid=6103367c82215-OTIyOA==
In the controller software that the disc gives it has rumble and everything and it works. I would love to know why xoutput is crashing with this installed and how I can fix it.  I could just grab my two dualshock fours but I am curious to see if this can work. I could tell my friends to just get one of these and play rocket league or something I guess. Anyway, thats all I have to say lol.

Try disabling automatic device status detection in settings


Plausible is blocked in Wordpress dashboard

Good day, I have put the shared link to the Wordpress Plausible plugin settings as stated in the instruction, but i receive this ^. What is wrong? Thank you

closing this for now

What do you mean shared link? Do you mean page / post?

It worked, thank you.

thanks for the update @froart!


License

cc @ForNeVeR @Minoru @ttldtor @IngvarJackal @Newlifer
What license should we distribute this game under? I'm ok with MIT.

Btw, I've already put MIT to the pacman.el file :D

But we can change if it's not ok for someone.

I'm okay with MIT for code, but not sure if it's suitable for artistic work. Maybe you should license your sprites under CC-BY?

@Minoru oh, thank you for reminding my about the sprites! I'm ok with CC-BY for them, especially after #11 <.<

You might also blog about the sprites so that others can take advantage of your work. I doubt we're the only people who Googled for freely available Pac-Man sprites.

@Minoru you are so awesome! Thank you! =3


Feat: Implementing Bookmark feature

Describe the feature
Allow users to bookmark their favorite programs for quick and convenient access. This feature enhances user experience by enabling them to save and organize programs they are interested in, ensuring they can easily revisit and engage with their preferred content.
Add ScreenShots
We can improve user experience by adding drawer buttons for accessing features such as "Previous Programs" and "Bookmarks," similar to the ones shown in the video. This  will make it easier for users to navigate and enhance their overall interaction with the app.
Screencast from 2024-05-12 14-50-26.webm
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

i would like to contribute to this issue... kindly assign me i am GSSOC'24 contributor

@Pra-kas Add this Bookmark feature above the About section in the menu option that is already on the homepage

Please assign me sir.

@Pra-kas Add this Bookmark feature above the About section in the menu option that is already on the homepage

ok sir

Sir I have added icons and buttons for bookmark feature. should i need to implement backend logic for bookmark feature.

@andoriyaprashant  Sir i have completed the issue and raised pr.


Detect invalid PIN lengths and ask the user again

We also detect the specific case where the PIN returned by the user is
likely a YubiKey OTP, generated by the user touching it early.
Closes str4d/age-plugin-yubikey#37.

Codecov Report

Merging #46 (68ac190) into main (51910ed) will decrease coverage by 0.60%.
The diff coverage is 0.00%.

@@           Coverage Diff            @@
##            main     #46      +/-   ##
========================================
- Coverage   6.25%   5.64%   -0.61%
========================================
  Files          4       4
  Lines        160     177      +17
========================================
  Hits          10      10
- Misses       150     167      +17

Impacted Files
Coverage Δ

src/key.rs
8.62% <0.00%> (-1.19%)
:arrow_down:

src/util.rs
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Fix UI break on animate from Dec

https://stackoverflow.com/questions/58339739/nestedscrollview-issue-throwing-error-when-switching-between-two-screens


renaming doesnt work with ios

renaming the project still doesnt work for me. i tried with windows, and with a friends computer. still ios directory folders name starts with RNNStarter. what am i missing here.
from the above screen shot. does it mean i have to manually rename folder name's in the ios directory to match the name in the app.json file?
my app.json file.
{
"name": "Aiminaabee",
"displayName": "Aiminaabee"
}

Hi
I'm having the same issue. Any idea?

yes, I have the same issue.

does anyone know how to fix it?

Closing, please check v3


Simplifying the connection switching experience

There are three things that we need to do to simplify the connection switching experience:

Remote the test connection button from the Workspaces window
Extend the duration of the modal dialog box during connection establishment to not return until all secondary sessions (e.g., IntelliSense) are established
Remove the transactional nature of switching connections. We first terminate the existing session (with prompt to user to save state) before establishing a new connection. Yes, this introduces a new state "no connection established" into the process in the case where a new connection cannot be established. This applies to both local-local, local-remote and remote-local switches.

This issue references #2643 as well as #2736 which will be closed when this issue is complete.

#2736 is closed

Remote the test connection button from the Workspaces window

For the last 2 month, there are only 3 recorded issues with test connection: #2747, #2633, and #2742 (not really a test connection issue - it is a bug on UI that was reproduced on many buttons). None of those looks critical. So can you describe what's the problem with the button?

General reduction of ability to interact with the modeless UI. We'd rather not chase hard to repro reports from users. It is generally better not to distribute feature rather than remove it after it is already became public.

Ok, should we cut the whole code path or just hide it from UI?

hiding from UI is probably OK

We actually want to
a). Disconnect first. Also disconnect on Save of edited connection.
b). Connect REPL session only. If fails, remain disconnected, no attempts to reconnect to previous.
c). No test connection functionality.
So I think transaction code can be removed as well as there won't be transaction with rollback anymore.
What's the timeline for this? We should have this done before the public distribution or Dev15 integration.

Transaction code can't be removed completely because we can't leave the system in the middle of nowhere. If one session is stopped and user click canceled, we need to stop all other sessions. If new RHost is created but connection isn't finished and he clicked Cancel, we need to stop that RHost. And so on.
We had no bugs in RSessionProvider for couple of weeks now. I'd rather not touch anything related to the switch (to avoid introducing any new bugs) and just hide the Test Connection button

I meant we first stop everything (as if package was just loaded) - no cancellation. Then create connection with one session only. Complete disconnect should be part of #2845 I think.

If host is executing some native code, q() may not work and IRSession.StopAsync make take up to 20 seconds for that session. What should we show to the user during that process?
It is actually the same question with #2845.

Can we just abandon the session and create new one or blindly kill the process? There is no particular reason to wait for it to terminate.

I guess reducing to just hiding test connection


Ubuntu 23.04 installation has issueswith installation

Not sure it's thefucks fault, but installation kinda broken on a newer Ubuntu
pip3 install thefuck --user
...
error: externally-managed-environment

× This environment is externally managed
╰─> To install Python packages system-wide, try apt install
    python3-xyz, where xyz is the package you are trying to
    install.

    If you wish to install a non-Debian-packaged Python package,
    create a virtual environment using python3 -m venv path/to/venv.
    Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
    sure you have python3-full installed.

    If you wish to install a non-Debian packaged Python application,
    it may be easiest to use pipx install xyz, which will manage a
    virtual environment for you. Make sure you have pipx installed.

    See /usr/share/doc/python3.11/README.venv for more information.

note: If you believe this is a mistake, please contact your Python installation or OS distribution provider. You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages.
hint: See PEP 668 for the detailed specification.

nvm, new version broke the hell out of python

I have the same problem after upgrading to 23.04, how did you solve this?

Did not really solve, IIUC installation of python packages became hell of a lot harder for security reasons, so I kinda just ditched all packages I installed with pip.


Release 0.1.17

@atramos I haven't contributed much lately, so question for you: is master stable enough for a release?
I hear people really want the faster block download fix.
In case you missed it, we have a discord now, which is mostly troubleshooting. See this announcement.
In case you join, let me know, so I can give you the "maintainer" role.

My production miners are running on the branch from PR #89. I recommend merging that before release. It addresses the exponential growth in failed connections to dead and outbound-only peers.

I approved #89, please merge. I plan to do a release early next week.

I published the release.

I messed up the release, so I made a new one with version 0.1.18.

Also just a quick note, I didn't get around to meeting PR #89, so the soft-ban stuff is not in this release.


Feature/#32

added media query so the loby can be responsive

We should replace the media query with pure css media querys and not javascript


Add issue175&176&177

Pull Request Template
Description
Hatsコントラクト、FractionTokenコントラクト、SplitsCreatorコントラクト用のReact Hooksを追加しました。
そのほか、READMEへの環境変数のメモ書き追加
frontend の .env.example ファイルの更新
pilmco用のReact Hooksへのコメントを追記しています。
確認お願いします！
Fixes
#175
#176
#177
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Screenshots

あと、フロントエンド ビルド用の GitHub Actionsファイルの追加してみました！

以下は、GitHub Copilotで出力された このプルリクエストの要約です。
This pull request includes several updates and new features across multiple files, primarily focused on integrating new smart contract addresses, adding new React hooks, and importing additional dependencies. Below is a summary of the most important changes:
Environment Variable Updates:

Added new environment variables for VITE_HATS_ADDRESS, VITE_FRACTION_TOKEN_ADDRESS, and VITE_SPLITS_CREATOR_ADDRESS in README.md and .env.example files. [1] [2]

React Hooks:

Added new hooks for useFractionToken, useHats, and useSplitsCreator to handle various functionalities related to Fraction Tokens, Hats, and Splits creation. [1] [2] [3]
Updated useBigBang and useSmartWallet hooks to include additional imports and comments for better readability and functionality. [1] [2] [3]

Contract Configurations:

Added new configurations for HATS_ADDRESS, FRACTION_TOKEN_ADDRESS, and SPLITS_CREATOR_ADDRESS in useContracts.ts.

Dependencies:

Added @hatsprotocol/sdk-v1-subgraph to package.json for subgraph functionalities.

Type Definitions:

Added a new type SplitsInfo in types.ts to define the structure for splits information.


[Electron Replace] libffmpeg replacement issue

I tried to install an extension via VSIX file from VS Code marketplace below:

[ ] Extension name: VSC Netease Music
[ ] Extension GitHub or homepage: https://github.com/nondanee/vsc-netease-music

It is a music player plugin like running Spotify in VS Code.
The author said:

[The Electron version used by VS Code does not contain ffmpeg] (https://stackoverflow.com/a/51735036), you need to replace the built-in ffmpeg dynamic link library to play normally (you need to replace it every time you update VS Code).

But I can't find anything about electron, what should I do?

code-server doesn't run with electron, it runs in whatever browser you run it inside of.
I'd file an issue with the extension author. I don't think there's anything we can do here.


[Dubbo-2349][Baiji-7]qos.md translation and correction

What is the purpose of the change
XXXXX
Brief changelog
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Test your code locally by running docsite start, and make sure it works as expected.
[ ] Make sure no files under build directory is added.
[ ] If this contribution is large, please follow the Software Donation Guide.

已修改


Unable to parse Code Signing Certificate for Microsoft

Hi!
First off, many thanks for taking the time to do this useful lib!!
For a side project, I was trying to parse certificates inside exe files -- PE format -- to get the expiration date.
I'll skip the details on how to do it, but it turns out it's using PKCS7 format to store the certificate chain, hence the use of this lib.
When I call Parse() with the payload in DER this is what I get:
structure error: tags don't match (4 vs {class:0 tag:16 length:23 isCompound:true}) {optional:false explicit:false application:false defaultValue:<nil> tag:<nil> stringType:0 timeType:0 set:false omitEmpty:false} unsignedData @2

I had to patch the library and can make a PR if that's necessary: https://github.com/tehmoon/pkcs7/commit/8f1294642f99e46c11e0069a998d74fd28ad7824
As you can see, I de-serialize using RawValue and passing the underlying bytes. I didn't dig much further so I have no idea what's wrong.
I've attached the pkcs7 data in hex string format: data.hex.txt
Even though the patch works well for me, I thought about opening an issue to contribute a little bit.

Not sure if you're still around, but I still get the same error message with the patch.

@atvaccaro yes still around! Could you tell me more about still get the same error message please? If you could share the data that is being parse that would be cool.

Whoops, just went on vacation! I'd been getting the tags don't match error. I ended up just using golang.org/x/crypto/pkcs12 and switching to a PKCS12 file.

OK! let me know if you switch back to pkcs7 for some reason :p


Peddy (NotImplementedError: iLocation based boolean indexing on an integer type is not available)

Version info

bcbio version (bcbio_nextgen.py --version): 1.2.0
OS name and version: Ubuntu 18.04.4 LTS (Bionic Beaver)

To Reproduce

Exact bcbio command you have used: bcbio_nextgen.py ../config/tumor_normal.yaml -n 8
Your sample configuration file: yaml.zip

Observed behavior
2020-04-18 05:58:55 bcbio-virtual-machine peddy.cli[17466] INFO ran in 0.3 seconds
Traceback (most recent call last):
File "/home/bcbio/install/stable/anaconda/envs/python2/bin/peddy", line 10, in
sys.exit(cli())
File "/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/click/core.py", line 764, in call
return self.main(*args, **kwargs)
File "/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/click/core.py", line 717, in main
rv = self.invoke(ctx)
File "/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/click/core.py", line 956, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/click/core.py", line 555, in invoke
return callback(*args, **kwargs)
File "/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/peddy/cli.py", line 226, in peddy
da = df.iloc[df['sample_duplication_error'], 'sample_a']
File "/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/pandas/core/indexing.py", line 1494, in getitem
return self._getitem_tuple(key)
File "/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/pandas/core/indexing.py", line 2143, in _getitem_tuple
self._has_valid_tuple(tup)
File "/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/pandas/core/indexing.py", line 223, in _has_valid_tuple
self._validate_key(k, i)
File "/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/pandas/core/indexing.py", line 2060, in _validate_key
raise NotImplementedError("iLocation based boolean "
NotImplementedError: iLocation based boolean indexing on an integer type is not available
Expected behavior
Peddy should work correctly, but terminated with an error.
Log files
I've attached bcbio-nextgen.log, bcbio-nextgen-commands.log, and bcbio-nextgen-debug.log.
log.zip
Additional context
I've rerun Peddy manually. It still produced the same error below.
bcbio@bcbio-virtual-machine:~/nute/work1/log$ peddy -p 8 --sites hg38 --plot /home/bcbio/nute/work1/qc/pair001/peddy/pair001-effects-annotated-filter-germline.vcf.gz /home/bcbio/nute/work1/qc/pair001/peddy/pair001-effects-annotated-filter-germline.ped
2020-04-18 10:06:05 bcbio-virtual-machine peddy.cli[19171] INFO Running Peddy version 0.4.4
/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/peddy/cli.py:178: FutureWarning: read_table is deprecated, use read_csv instead.
sep="\t")
2020-04-18 10:06:05 bcbio-virtual-machine peddy.cli[19171] INFO ped_check
2020-04-18 10:06:06 bcbio-virtual-machine numexpr.utils[19171] INFO Note: NumExpr detected 48 cores but "NUMEXPR_MAX_THREADS" not set, so enforcing safe limit of 8.
2020-04-18 10:06:06 bcbio-virtual-machine numexpr.utils[19171] INFO NumExpr defaulting to 8 threads.
2020-04-18 10:06:07 bcbio-virtual-machine peddy.peddy[19171] INFO plotting
2020-04-18 10:06:08 bcbio-virtual-machine peddy.cli[19171] INFO ran in 2.6 seconds
2020-04-18 10:06:08 bcbio-virtual-machine peddy.cli[19171] INFO het_check
2020-04-18 10:06:10 bcbio-virtual-machine peddy.pca[19171] INFO loaded and subsetted thousand-genomes genotypes (shape: (2504, 4776)) in 1.2 seconds
/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/sklearn/svm/base.py:196: FutureWarning: The default value of gamma will change from 'auto' to 'scale' in version 0.22 to account better for unscaled features. Set gamma explicitly to 'auto' or 'scale' to avoid this warning.
"avoid this warning.", FutureWarning)
2020-04-18 10:06:11 bcbio-virtual-machine peddy.pca[19171] INFO ran randomized PCA on thousand-genomes samples at 4776 sites in 0.8 seconds
2020-04-18 10:06:11 bcbio-virtual-machine peddy.pca[19171] INFO Projected thousand-genomes genotypes and sample genotypes and predicted ancestry via SVM in 0.1 seconds
/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/peddy/peddy.py:916: FutureWarning: Sorting because non-concatenation axis is not aligned. A future version
of pandas will change to not sort by default.
To accept the future behavior, pass 'sort=False'.
To retain the current behavior and silence the warning, pass 'sort=True'.
df = pd.concat((df, pca_df), axis=1)
2020-04-18 10:06:12 bcbio-virtual-machine peddy.cli[19171] INFO ran in 4.2 seconds
2020-04-18 10:06:12 bcbio-virtual-machine peddy.cli[19171] INFO sex_check
no intervals found for /home/bcbio/nute/work1/qc/pair001/peddy/pair001-effects-annotated-filter-germline.vcf.gz at X:2781480
2020-04-18 10:06:12 bcbio-virtual-machine peddy.peddy[19171] INFO sex-check: 39 skipped / 1411 kept
2020-04-18 10:06:12 bcbio-virtual-machine peddy.cli[19171] INFO ran in 0.3 seconds
Traceback (most recent call last):
File "/home/bcbio/install/stable/anaconda/envs/python2/bin/peddy", line 10, in
sys.exit(cli())
File "/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/click/core.py", line 764, in call
return self.main(*args, **kwargs)
File "/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/click/core.py", line 717, in main
rv = self.invoke(ctx)
File "/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/click/core.py", line 956, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/click/core.py", line 555, in invoke
return callback(*args, **kwargs)
File "/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/peddy/cli.py", line 226, in peddy
da = df.iloc[df['sample_duplication_error'], 'sample_a']
File "/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/pandas/core/indexing.py", line 1494, in getitem
return self._getitem_tuple(key)
File "/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/pandas/core/indexing.py", line 2143, in _getitem_tuple
self._has_valid_tuple(tup)
File "/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/pandas/core/indexing.py", line 223, in _has_valid_tuple
self._validate_key(k, i)
File "/home/bcbio/install/stable/anaconda/envs/python2/lib/python2.7/site-packages/pandas/core/indexing.py", line 2060, in _validate_key
raise NotImplementedError("iLocation based boolean "
NotImplementedError: iLocation based boolean indexing on an integer type is not available

I can fixed this issue by upgrading peddy 0.4.4 to 0.4.6. Next, it leads to another issue about cnvkit. Please take a look at https://github.com/etal/cnvkit/issues/511

Thanks @chatchawit for figuring this out!
peddy is 0.4.6 already in bioconda and switched to python3
https://github.com/bioconda/bioconda-recipes/blob/master/recipes/peddy/meta.yaml
We are not pinning it, on our system it is 0.4.6, but it is was still in python2 env.
I fixed that:
https://github.com/chapmanb/cloudbiolinux/blob/master/contrib/flavor/ngs_pipeline_minimal/packages-conda.yaml
re: cnvkit, try updating it to the latest beta:
https://github.com/bcbio/bcbio-nextgen/issues/3061
Sergey


Add fix for Spotify Autopause

Basically port this over to Replugged please: https://github.com/BluSpring/Spotify-No-Pause/blob/master/index.js
It can get really annoying and in the original Spotify Plugin it also has that implemented.

Will get this done ASAP

I have finished the functionality for disabling auto pause but because of a few unfinished features in my local branch I could not push it upstream today

I just pushed v1.2.7 which should resolve this issue.

Great 👍


Reboot Chromecast Device

It would be nice to be able to force a Chromecast Device to reboot (not by power off/on).
The following command should work, but needs the current ‘cast-local-authorization-token’ header / value.
curl -Lv  -k  -H Content-Type:application/json https://<IP_ADDRESS>:8443/setup/reboot  -d "{""params"":""now""}"
If you have the Auth Token available, would you please add the Reset Device Function?
Thank you!

This would be a cool feature. I don't think we currently have a way to talk to that API, we currently talk to the api on port :8009 (default port). Would be happy to add this if we can work out a way to get the auth token, do you know a good way to retrieve it?

Hello,
Thank you so much for the quick reply. I started down the path to get the auth token, but with my limited tool set- it was a manual process and the token expires.
Here are links to the references to get the token.
https://rithvikvibhu.github.io/gh-web-proto-decode/
https://github.com/rithvikvibhu/GHLocalApi/issues/39#issuecomment-643587959
https://gist.github.com/rithvikvibhu/952f83ea656c6782fbd0f1645059055d
https://github.com/leikoilja/glocaltokens
If you need any additional research I will we glad to help – just say what you need.
Thank you,
John
From: Jonathan Pentecost @.***
Sent: Wed, Mar 16, 2022 1:44 PM
To: vishen/go-chromecast
Cc: testcu; Author
Subject: Re: [vishen/go-chromecast] Reboot Chromecast Device (Issue #131)
This would be a cool feature. I don't think we currently have a way to talk to that API, we currently talk to the api on port :8009 (default port). Would be happy to add this if we can work out a way to get the auth token, do you know a good way to retrieve it?
—
Reply to this email directly, view it on GitHub https://github.com/vishen/go-chromecast/issues/131#issuecomment-1069613030 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AJUPWFSIAQRBE6ICM2FEGHDVAJBXVANCNFSM5Q4VYO5A .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AJUPWFU7VFPBTKM4MHCDQN3VAJBXVA5CNFSM5Q4VYO5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOH7AP7ZQ.gif Message ID: @.***>

I started down the path to get the auth token, but with my limited tool set- it was a manual process and the token expires.

How were you able to get the auth token? I am a little unsure how to get the auth token, and those code-snippets tend to want a username + password, are these a default username + password or is this something different for each chromecast?

I used the User/Password from the device. To verify what it was,  I used the Google Home App.
https://support.google.com/googlenest/answer/7552680?hl=en
Then used the manual process (phython3 https://github.com/leikoilja/glocaltokens) to get the token.

Thanks! I'll have a look when I get a chance :)

It doesn't look as simple as I was hoping. Looks like we need to get an auth token from the Google play store (maybe?) and then use that when making requests. Not sure how easy this will be, but here is some Go code that has done something similar (don't know if they still work):

https://github.com/dainis/go-playstore/blob/a79271e88c44fc168fe9c1ee95e0330a3bd96b65/const.go#L4
https://github.com/neuegram/Ghost/blob/1825110ea4ffbba3c5f2b032e715eff4d80b6889/ghost.go#L541
https://github.com/pogodevorg/pgoapi-go/blob/e8f4e09bf2a3ae15954adf742be95acf1f3cea67/auth/google/provider.go#L79
https://github.com/mjibson/gpsoauth/blob/be41f43784ed68405d473a38b886a4198aaf45b6/gpsoauth.go#L20

And here is some other useful info:

https://github.com/simon-weber/gpsoauth/blob/master/gpsoauth/__init__.py
https://github.com/rithvikvibhu/GHLocalApi/issues/39#issuecomment-644959103
https://rithvikvibhu.github.io/GHLocalApi

I've 'been looking for ways to control my 1st gen Chromecast via command line and just stumbled across your project. Very impressive! You have created the most robust command set I've seen yet in a Chromecast CLI!
Here's my little contribution...
You can reboot a chromecast through a simple http request:
curl -Lv -H Content-Type:application/json http://<IP_ADDRESS>:8008/setup/reboot  -d "{""params"":""now""}"
Don't fell bad. Many times, when you get very involved with a project, you naturally start making more and more complex functions. It's very easy to miss simple solutions once in a while after creating such a large application. That's why it's always a great idea to get a fresh set of eyes on the project once in a while. New blood can bring new perspectives and many times find a completely different way to accomplish something.
Anyway, that's my $0.02 worth of advice. I hope to provide some more assistance as I have time, and will fork the project at some point in the future to help with issues and new functionss.
Thank you for all your hard work and creating such a wonder Chromecast CLI!

This may also come in handy. To do a factory reset use;
curl -Lv -H Content-Type: application/json http://<IP_ADDRESS>:8008/setup/reboot -d ‘{“params”:”fdr”}’ -X POST

@Cyber5imon wrote:

I've 'been looking for ways to control my 1st gen Chromecast via command line and just stumbled across your project. Very impressive! You have created the most robust command set I've seen yet in a Chromecast CLI!
Here's my little contribution... You can reboot a chromecast through a simple http request: curl -Lv -H Content-Type:application/json http://<IP_ADDRESS>:8008/setup/reboot -d "{""params"":""now""}"

Sadly, that only works on 1st gen Chromecast, and doesn't work on newer Chromecast models:
$ curl -Lv -H Content-Type:application/json http://<IP_ADDRESS>:8008/setup/reboot -d "{""params"":""now""}"
*   Trying <IP_ADDRESS>:8008...
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 8008 (#0)
> POST /setup/reboot HTTP/1.1
> Host: <IP_ADDRESS>:8008
> User-Agent: curl/7.81.0
> Accept: */*
> Content-Type:application/json
> Content-Length: 12
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 403 Forbidden
< Content-Length:0
<
* Connection #0 to host <IP_ADDRESS> left intact


Document the className prop

I was hoping that this prop existed, but I had to dig into the source to find it! :)

Sorry! I didn't see this one until now! :( Thanks!


springfox cannot distinguish type List <Byte>

When the parameter tye is  List <Byte> like:
public RestResult getIndexStatistics(List<Byte> bytes) {return null;}
the swagger ui  show the type is "string($byte)"。
While I change the type to "Integer",
the swagger ui  show the type is  "array[integer]",that is the right result.
Can swagger ui distinguish type ”byte“ and convert   List <Byte> to array?

List of bytes are interpreted as string, but you can create a rule to make that not be the case using alternate type rules

List of bytes are interpreted as string, but you can create a rule to make that not be the case using alternate type rules

How Can I do is ?Can you give me an example?

After some research it appears this is a possible enhancement/bug
https://github.com/springfox/springfox/blob/a04d57b6ac14e5aa8703e8947f981ef97a873974/springfox-swagger2/src/main/java/springfox/documentation/swagger2/mappers/ParameterMapper.java#L75-L81


No such column: recurrence

I'm sure this isn't a bug, but it's just me being dumb.
When I try to add recurrence it just says no such column: recurrence
I'm sure there's some way to add it, but I couldn't find anything in the docs.
Thanks!

Hi @sevennotfound,
By default I excluded the recurrence column so it doesn't show up. You can tweak what columns you want in dooit
Check this out!


Fix 1027

I fix 1027...

?

I would like someone to review my new implementation, #1027.

Probably better to keep a comment on that ticket instead


2 Esidras missing with Luke Smith's apocrypha kjv.tsv

Preface: I opened this issue before, it got closed, so I'm reopening it. What you think the number of books in the bible should be has no significance here. The facts are that your software has a bug. I am willing to look into this and give a PR if you dont want to, but please do not close my issue. That is highly disrespectful. Do not try to convert me into protestantism. Allow orthodox christians to have complete bibles if they provide their own tsv without encountering bugs. Just keep this issue opened and do not respond if you don't wish to help.
This happens with any tsv that is 79 or more books. I added extra book to test, behavior is as follows:
78 and lower: untested
79-80 books: 1 book is not available, kjv -l reports a book with the name of " ()" for that book.
81+ books: Same behavior as before, except that any additional books will not display in kjv -l as anything, not even " ()".
By adding the book "Test" to the tsv at line 1, 2 Esidras becomes visible and "Test" becomes the book " ()" which is inaccessible. This is fine as a temporary solution but still makes the kjv -l have that " ()" entry. There is only ever 1 book labeled " ()" and so with 81+ books when more than 1 book will disappear, I cannot just add a book "Test" to my tsv to work around the bug to be the " ()". This is a problem for Ethiopian orthodox because theres going to be quite a few hidden books.

Problem with supplied TSV file, closing this

Preface: I opened this issue before, it got closed, so I'm reopening it. What you think the number of books in the bible should be has no significance here. The facts are that your software has a bug. I am willing to look into this and give a PR if you dont want to, but please do not close my issue. That is highly disrespectful. Do not try to convert me into protestantism. Allow orthodox christians to have complete bibles if they provide their own tsv without encountering bugs. Just keep this issue opened and do not respond if you don't wish to help.
This happens with any tsv that is 79 or more books. I added extra book to test, behavior is as follows:
78 and lower: untested 79-80 books: 1 book is not available, kjv -l reports a book with the name of " ()" for that book. 81+ books: Same behavior as before, except that any additional books will not display in kjv -l as anything, not even " ()".
By adding the book "Test" to the tsv at line 1, 2 Esidras becomes visible and "Test" becomes the book " ()" which is inaccessible. This is fine as a temporary solution but still makes the kjv -l have that " ()" entry. There is only ever 1 book labeled " ()" and so with 81+ books when more than 1 book will disappear, I cannot just add a book "Test" to my tsv to work around the bug to be the " ()". This is a problem for Ethiopian orthodox because theres going to be quite a few hidden books.

Hey,
The project seems to be unmaintained, https://github.com/layeh/kjv/pull/23.
I would like to point to my fork at https://github.com/rofe33/kjv-layeh/tree/kjv-new-options.
I also  created a bunch of repositories for different versions, like:

https://github.com/rofe33/RKJNT
https://github.com/rofe33/BBE
https://github.com/rofe33/DARBY
and so on...

However, I am not that familiar with the C Programing Language.

Preface: I opened this issue before, it got closed, so I'm reopening it. What you think the number of books in the bible should be has no significance here. The facts are that your software has a bug. I am willing to look into this and give a PR if you dont want to, but please do not close my issue. That is highly disrespectful. Do not try to convert me into protestantism. Allow orthodox christians to have complete bibles if they provide their own tsv without encountering bugs. Just keep this issue opened and do not respond if you don't wish to help.
This happens with any tsv that is 79 or more books. I added extra book to test, behavior is as follows:
78 and lower: untested 79-80 books: 1 book is not available, kjv -l reports a book with the name of " ()" for that book. 81+ books: Same behavior as before, except that any additional books will not display in kjv -l as anything, not even " ()".
By adding the book "Test" to the tsv at line 1, 2 Esidras becomes visible and "Test" becomes the book " ()" which is inaccessible. This is fine as a temporary solution but still makes the kjv -l have that " ()" entry. There is only ever 1 book labeled " ()" and so with 81+ books when more than 1 book will disappear, I cannot just add a book "Test" to my tsv to work around the bug to be the " ()". This is a problem for Ethiopian orthodox because theres going to be quite a few hidden books.

Hey,
The project seems to be unmaintained, #23. I would like to point to my fork at https://github.com/rofe33/kjv-layeh/tree/kjv-new-options.
I also created a bunch of repositories for different versions, like:
* https://github.com/rofe33/RKJNT

* https://github.com/rofe33/BBE

* https://github.com/rofe33/DARBY

* and so on...

However, I am not that familiar with the C Programing Language.

Oh, thank you! I can package your versions for gentoo, if you'd like

Oh, thank you! I can package your versions for Gentoo, if you'd like

No, thank you. It is not a big deal, I am planning to remove them and making an update on https://github.com/rofe33/Bibles (I won't remove the PDFs just the terminal stuff), and the terminal bibles.
I don't want to do something that nobody will use, if someone wants a bible version, then I will add it. For now I am just maintaining with what I have for the PDF versions.
Thank you!  I will be glad to add one for you.
I will be updating, my projects and see what is best to do. For now I am not that familiar with C, I am more a python guy.

Just to clarify, I want to delete the terminal stuff, because I am not familiar with C and I can't maintain the code.
I am not sure how long it will stay working and compiling, and I just want more features added!
I may create another script that can do a lot of things using python, and familiar to this kind of project.
God Bless You!

Oh, thank you! I can package your versions for Gentoo, if you'd like

No, thank you. It is not a big deal, I am planning to remove them and making an update on https://github.com/rofe33/Bibles (I won't remove the PDFs just the terminal stuff), and the terminal bibles.
I don't want to do something that nobody will use, if someone wants a bible version, then I will add it. For now I am just maintaining with what I have for the PDF versions.
Thank you!  I will be glad to add one for you.
I will be updating, my projects and see what is best to do. For now I am not that familiar with C, I am more a python guy.

Would you be willing to make NIV bible tsv?

Yes! But from where do I get the data of the NIV bible? For now I am getting the data from https://bible4u.net/en/download#en


Delete routes when endpoint destroyed

Fixes issue #184.

One note here. =)

What we have now is this.
for type in (futils.IPV4, futils.IPV6):
    try:
        if devices.tap_exists(self.tap):
            ips = devices.list_tap_ips(type, self.tap)
            for ip in ips:
                devices.del_route(type, ip, self.tap)
    except:
        if devices.tap_exists(self.tap):
            raise
        break

I think you are proposing this :
if devices.tap_exists(self.tap):
    for type in (futils.IPV4, futils.IPV6):
        try:
            ips = devices.list_tap_ips(type, self.tap)
            for ip in ips:
                devices.del_route(type, ip, self.tap)
        except (futils.WhateverTheExceptionIs):
            if devices.tap_exists(self.tap):
                raise
            break

Is that right? No objection in principle to that restructuring.

@plwhite Exactly right. I'm a big believer in small try...except blocks. =D

Back to you. Note that I did a push -f so that the intermediate version would not appear in the history, but small enough that should not matter much.

Cool, LGTM. :cake:


FIX: save as png doesn't work properly for Dee, Dey, Priya & Ringo. F…

…ixes #39

@sanand0 I did not use Puppeteer based SVG to png converter because svg-png-converer uses FabricJS internally to have the same API for both browser and node.js environments.
This helps us be sure if pngs are converted well for the svgs, then gramener.com/comicgen app also will work fine.

svg-png-converter fails on Windows, @tejesh95 -- this may also mean that the library is not mature enough. Please take a look. I'll merge once this runs on Windows, and am a bit more comfortable with the library's stability.
$ yarn add svg-png-converter
yarn add v1.17.3
...
error Command failed.
Exit code: 128
Command: git
Arguments: clone https://github.com/jankovicsandras/imagetracerjs.git C:\Users\anand\AppData\Local\Yarn\Cache\v4\.tmp\9d1a5040f7226706aef7c3bf70a5551a
Directory: D:\site\gramener.com\viz\comicgen
Output:
Cloning into 'C:\Users\anand\AppData\Local\Yarn\Cache\v4\.tmp\9d1a5040f7226706aef7c3bf70a5551a'...
fatal: Invalid path '/cygdrive/d/site/gramener.com/viz/comicgen/C:\Users\anand\AppData\Local\Yarn\Cache\v4\.tmp\9d1a5040f7226706aef7c3bf70a5551a': No such file or directory
info Visit https://yarnpkg.com/en/docs/cli/add for documentation about this command.

@sanand0 I have removed svg-png-converter dependency.
Added fabricjs library as a dependency. Please review it again?

Works great!
Suggestions:

[ ] Increase facesketch and humaaans PNG resolution -- ensuring height=600
[ ] Use a progress indicator like Python's tqdm for png.js
[ ] Move png.js into build/<something>.js
[ ] Move png.sh into build/<something>.sh
[ ] Update the docs

higher PNG resolution for facesketch and humaans is not easy with this PR.
Opened new PR which uses puppeteer https://github.com/gramener/comicgen/pull/65


Avoid unsafe behaviour iterator

Will work on this

https://users.rust-lang.org/t/implementing-iterator-for-vec/71398/6

Great!

@SimonVervisch any progress on this?

Will say that I'm struggling a lot with this. Mainly making the lifetime of the returned item 'a.
The Iterator trait takes no lifetimes in the "next" function signature. So yeah, I will keep working on this.

Yeah, it's definitely a thing.


Downloading Files Excel or pdf from website results in Webkit error # 102

I'm trying to download a file from a website, specifically an excel file by clicking a download/exportexcel button on the website, but it results in Webkit error # 102 .
I'm using simple LUA construct splash:evaljs('document.querySelector("[Some Selector]").click()')
Screenshot is attached

Is there any update on this issue? I am also facing the same issue while downloading excel file by clicking on button.

There is a workaround I just found out, it may not work in all cases, one can return response as a base64 encoded file, and then later decode into some other client like python and write bytes to a file, I was able to download a pdf with this, splas:http_get or post does not change the current page of splash browser, it simply returns the response
treat = require("treat")
base64 = require("base64")
.
.
local response = splash:http_get("url link to file")
assert(splash:wait(30))
return treat.as_binary(base64.encode(response.body))


rework redis cluster command mapping

closes https://github.com/shotover/shotover-proxy/issues/605
Justification for changes to command mappings:

DBSIZE

Query every master and sum the result. Allows us to pretend to be a single redis instance.

KEYS

Query every master and combine the results.
Running KEYS * in a large production environment could OoM shotover, but I assume in such an environment the instances would be configured to not allow running KEYS, so each redis instance would return an error and then we would return a single error to the client which would be fine.

ACL

Not sure about this one but it seemed like the majority of the ACL commands should be run uniformly against every instance.
Possibly we will need to revisit this in the future.

SAVE + BGSAVE + BGREWRITEAOF

If the user wants to force redis to save to disk it makes sense that they want this to happen to EVERY instance not just the masters

LASTSAVE

The LASTSAVE command is needed to confirm that a previous BGSAVE command has succeed.

In order to maintain this use case we query every instance and return the oldest save time. This way the return value wont change until every instance has completed their BGSAVE.

CLIENT GETNAME/SETNAME

We cant reasonably support these commands in shotover

Connections are pooled so we cant forward it to the redis instance
We cant just implement the functionality shotover side because the connection names are supposed to be viewable from CLIENT LIST

However we dont want to just fail them either as clients like jedis would break so:

We just pretend to accept CLIENT SETNAME returning an "OK" but without hitting any instances
We just pretend to handle CLIENT GETNAME always returning nil without hitting any instances

CONFIG

I believe each instance will need unique configuration for some things so allowing a command to reconfigure every instance with the same config sounded wrong so I disabled it.

SLOWLOG

Instance specific results that cant be combined so I set to Unsupported

INFO

Each instance will return different information and no reasonable way to combine them so I just set Unsupported.

TIME

I just set to Unsupported because of potential differences in time state and I didnt think the functionality was that important?
Maybe we should just pick a random instance though?

ECHO + PING

Stateless commands that return a response, we just need a single instance to handle this for us so shotover can pretend to be a single instance. So we just pick a random instance.

It might be useful to refer to how redis-cluster-proxy handles these commands as well: https://github.com/RedisLabs/redis-cluster-proxy/blob/unstable/COMMANDS.md

oh good point!
I'll do that


Support notifications from 3rd party domains

### Tasks
- [ ] https://github.com/Altinn/altinn-notifications/issues/259
- [ ] https://github.com/Altinn/altinn-notifications/issues/215

pending #577


Speculos api clicking

Implemented dynamic port for binding Speculos API.
Clicking methods will use Speculos API.
Missing:
Update new Docker image and replace DEFAULT_EMU_IMG with new hash. This has to be done before merging.

Awesome work! very happy to see this new API being supported!!


The results of the description do not correspond to the paper

Hello, thank you very much for your work.
When I tested on the msvd-caption-test, I first used the fill_in_video_file you provided to add the video directory. Then, I used the Tarsier-7B and Tarsier-34B models respectively for inference and evaluation. The final CIDEr scores were 56.7 and 58.9, which differ significantly from those reported in the paper. Additionally, I also performed inference and evaluation using MSR-VTT, and the Tarsier-34B result was 31.4.
I used two A800 GPUs, each with 80G of memory, and made no other modifications. Therefore, I would like to ask if there are any other details that I might have overlooked. I look forward to your reply.

Thanks for the reminder! We have just updated the prompts in the metadata to what we used to our latest usage, which is consistent to the test results reported in the paper.


feat: use update hashes

closes #69
unblocks #208

saves the sha256 hash of the channel to a file with file name of the toolchain eg. latest-x86-apple-darwin to a new directory, ~/.fuelup/hashes
we use this hash to avoid re-downloading a toolchain if it already exists.
the hash is saved ONLY if the entire toolchain is installed (skipped if there are partial installations)
the hash is deleted upon uninstallation of a toolchain i.e. fuelup toolchain uninstall latest

This unblocks #208 since we want to check for hashes before trying to update latest and nightly.

Looking good, just clippy nits


typo in help doc of compression options

mediaconch -ha
--LogFile=...
                    Save the output in the specified file
--Compression=Mode
                    Compress report in database using [Mode]
                    [Mode] can be None for no compression
                    [Mode] can be ZLib to use zlib
--cz
                    Same as --Compression=ZLib

gives
mediaconch -mi --cz test5.mkv
Option not known

Should be -cz?

Hi,
yes
But Zlib is the default mode so you should not need it.

https://github.com/MediaArea/MediaConch_SourceCode/pull/322


d

When importing hollow's last hope, which is missing plenty of config, progress switch immediatly to 'Ok'.

Sorry, wrong project page :(


Azure Monitor HTTP Data Collector API has been deprecated

Describe the bug
According to Azure documentation The Azure Monitor HTTP Data Collector API has been deprecated and will no longer be functional as of 9/14/2026. However it is advertised in this repository and in Azure documentation as go-to solution for Sentinel connectors. After 2026-09-14 connectors using shutdown API might stop working
Expected behavior
Logs ingestion API should be used in mentioned in this repository instead deprecated Data Collector API.

Hi @miloszpiglas, Thanks for flagging this issue, we will investigate this issue and get back to you with some updates by 08-12-2023. Thanks!

Hi @miloszpiglas ,We are still working on this issue analysis and will get back to you with an updates by 13/11/2023

Hi @miloszpiglas ,We can use DCR as another option i.e. Logs ingestion api (https://learn.microsoft.com/en-us/azure/azure-monitor/logs/logs-ingestion-api-overview),Please let me know if need more details

Hi @miloszpiglas ,Gentle Reminder, We can use DCR as another option i.e. Logs ingestion api (https://learn.microsoft.com/en-us/azure/azure-monitor/logs/logs-ingestion-api-overview), Please let me know if need more details

@miloszpiglas , Since we have not received a response in the last 5 days, we are closing your issue (https://github.com/Azure/Azure-Sentinel/issues/9529) as per our standard operating procedures. If you still need support for this issue, feel free to re-open at any time. Thank you for your co-operation!


Graceful publishing of stats

Gracefully publish stats for each iteration
If some categories are empty, skip and continue to next iteration
This especially happens with non-PWA websites
On one hand, it may sound like a hard fail is good. But on the other hand, since we have multiple iterations, these cases always happen for one page or the other and it's annoying to have the whole sitespeed build fail just due to one iteration. We users are fine and we even expect one or two iterations to fail, since we have multiple iterations as a fallback
Debated whether we should publish zero score, or skip and continue. If we publish zero score, it would skew the metrics towards zero, hence skipping that one stat and continuing to next iteration
Fixes #40

cc @soulgalore

Yeah I saw it but been busy all day, sorry. Hopefully I can have a look tonight in about 3 hours or so.


Add iso8601 support

Previous support was only for the timestamps including a timezone (RFC3339-like), which is technically incorrect as the OCPP spec expects ISO 8601 timestamps, which can have a variety of different formats.
The PR adds better ISO 8601 support for parsing timestamps in incoming messages. This is achieved leveraging the https://github.com/relvacode/iso8601 lib.

@lorenzodonini may be time for a new release? Lots of good stuff included :)


analytics.binaryfortress.com

from DisplayFusion and possible other apps from https://www.binaryfortress.com/

Thank you @russeg I added this in commit 1ce9ed3.
This will go live in the next release.


Broken link on Page https://sedona.apache.org/latest/tutorial/sql/#__tabbed_3_2

The issue is the files are missing on GitHub.
There is a link on the page as below:
_https://github.com/apache/sedona/tree/master/core/src/test/resources_
I found a copy of the files, which are actually located at: https://raw.githubusercontent.com/apache/sedona/d04a8e0713add59293065f8b9706cbd7828a7a9a/spark/common/src/test/resources

Wrong uid


⚠ Bofrost.lu has degraded performance

In 45f3084, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1357 ms

Resolved: Bofrost.lu performance has improved in d1c33c6 after 7 minutes.


HDFS-11639. PROVIDED Phase 2. Encode the BlockAlias in the client protocol

Closing this PR as this change is no longer required as discussed on JIRA.


Estimate extention

Great, thanks!
I've pulled the first two commits only (Weibull.Estimate) and added a small smoke unit test; please open a new pull request on a dedicated branch for the empirical distribution.

I am new to git hub and I think I screw this up some how. How do I restore my local to the master branch. It is imposseble :) I use vs2015
From: Christoph Ruegg<EMAIL_ADDRESS>Sent: den 9 augusti 2015 19:07
To: mathnet/mathnet-numerics<EMAIL_ADDRESS>Cc: Jon Larborn<EMAIL_ADDRESS>Subject: Re: [mathnet-numerics] Estimate extention (#339)
Great, thanks!
I've pulled the first two commits only (Weibull.Estimate) and added a small smoke unit test; please open a new pull request on a dedicated branch for the empirical distribution.
—
Reply to this email directly or view it on GitHub https://github.com/mathnet/mathnet-numerics/pull/339#issuecomment-129217535 .  https://github.com/notifications/beacon/AKQ1SoaGF3CdcUF88_z3758g_DOSdnexks5ol4BQgaJpZM4Fh7Ux.gif


Jupyterlab extension

Most relevant changes:

Added functionality to the Web UI (pull from GIT repo)
Jupyterlab extension to access the webui

Added the changes discussed on 18/02/2020


changed the bullet train logo on the sign in and sign up pages to ren…

…der the new LEARN logo
Submitting Pull Request
Ticket
ID number:
Branch name:
Description:
Notes
Notes for the reviewer:
Contributors:

Yes, there were. We pulled up each of the original logo's sizing externally and using the LEARN logo as a template, we made three copies mimicing each of the original logos sizing.

Only one logo, specifically the 'logo.png' is being used. The other two LEARN logos are not being used.

we have readded the image 🙌


Fix 'does not validate' wording in readme

Correction for a minor typo in the readme, which implies gjson requires json that doesn't validate.

Much better. Thank you. 👍


Create a REST Service to return talent survey questions

Will be reading all the questions from a excel sheet ( this excel sheet is modifiable and HR/TEP can alter this sheet as per the survey they want to do )


Implement OAuth2 Security for microservices

Since there is a lack of samples on how to secure microservices and this is the most suggested method, it could be a good idea to implement this type of security for communication between microservices and securing them from outside requests

I think that incorporating Spring Cloud Security to our project is a natural follow up of what we already have one here. I am only wondering if securing our demo application wound't make it too complicated. I would like to keep this project as simple as possible, helping people to get started with Spring Cloud. Maybe we should create a separate branch for an OAuth2 version? @arey @dsyer

Agree in keeping the project simple, but would be great to have a branch with the OAuth2 implementation.

Would you like to contribute or to leave implementation to someone else? I think we can go with separate branch first and then decide whether to merge secured version to master.

I would be glad to help as much as I can, but I'm not experienced with oauth2 used in microservices. I wanted to use this as a good base to learn the correct way to implement the service.
I could implement what I know till now and then someone else check for improvements

Hi @ScopeSebastian, any update on this? :)

I'd be interested in contributing to this and other security related items.  In particular what I would like to add (as individual tutorial branches) is:
1) Securing microservices secrets with Hashicorp  Vault
2) OAuth2 authentication
3) Authorization based on both roles and URL in each service
4) Authorization in Angular UI (or maybe migrate to React with auth).
FWIW I have experience in each of these in my day job.  What is the process on this project ?  Just go ahead and implement then submit a pull request ?  Something else ...?

Thank you @pmj0yce for your proposition.
We could have a secure version in a separate branch you'll be in charge to maintain (synchronize with the master branch).
I propose to keep AngularJS. You may go ahead and submit a PR :)

there is any update about Secure version?

We are looking for some contributors. @pmj0yce was interested but I we didn't have any news.

Hi @arey / @dsyer
Can  i take up this issue

@arey / @dsyer
Can you please brief on what is expected out of this task. I am new to OAuth2 and microservices. Can you please brief on what is expected out of this task so that i can code easily. Please. It is an urgent request.

@arey / @dsyer
Can you please advice. Please.

@arey / @dsyer
May i know whether authentication using google or github is needed for this issue. I have started to work on this issue.  Can you please specify the requirements. Please.

I'm not an expert in microservices security but I think we could use the Spring Authorization Server https://www.baeldung.com/spring-security-oauth-auth-server.
I propose to secure  with a JWT token both the API from the Gateway and the API the 3 microservices : vets, visits and customers.
Keep in mind that this code shoud live in a separate code.
If it could help, I'm working on the Spring Cloud 2021.0 and Spring Boot 2.6 migration. See https://github.com/spring-petclinic/spring-petclinic-microservices/issues/188

Thanks @arey may i know the username and password configuration. May i know whether i should create signin and signup page or is it just for a single user.

@arey , the authorization should be for a single user or multiple users with signup and login page. Can you please confirm. Please

@arey / @pmj0yce  Can you please confirm

@arey / @dsyer / @pmj0yce  Can you please confirm

@arey / @dsyer / @pmj0yce Can you please confirm. Should i do in memory authentication or database authentication.

Hi @dsyer / @arey
when i configure spring security for API Gateway, i get the below error message. I removed starter web dependency and i added  javax.xml.bind,javax.servlet dependency still the same error. I am stuck with this from 2 days..Can you please advice.
Caused by: java.lang.IllegalStateException: StandardEngine[Tomcat].StandardHost[localhost].TomcatEmbeddedContext[] failed to start
at org.springframework.boot.web.embedded.tomcat.TomcatWebServer.rethrowDeferredStartupExceptions(TomcatWebServer.java:187)
at org.springframework.boot.web.embedded.tomcat.TomcatWebServer.initialize(TomcatWebServer.java:126)
... 18 common frames omitted

Any news?

A near reference web-app without authentication and authorization. Unbeliveable.

You're right @donhuvy. We have to propose a secured version.
We could use the Spring Authorization Server.
Before starting to work on this issue, we have to clearly define the security architecture.

Thank you for your attention. You can see https://github.com/donhuvy/spring_boot_3_oauth2/ , we base on a video tutorial use Spring Boot 2.x .

Hi @arey
FYI I created a workshop using the whole spring boot / cloud stack w/ spring authorization server
https://github.com/alexandre-touret/rest-apis-versioning-workshop
https://github.com/alexandre-touret/rest-apis-versioning-solution
If you want I can implement it in this project

It's this still available, if yes, can I work on it?

Please do. My preference would be to add a security layer to the gateway, and make the other apps clients of that (so they need a token and a token relay where appropriate). The auth server could be a Spring Auth Server (so that we control the user ids).

Hi
I did that in a workshop where I put both spring cloud gateway and spring authorization server in place.
You can get here


DATA IO LATER

it's scary

use dropdown menu to start

maybe that is enough to just make it not scary? we could also guide them through where we want them to put files maybe? like desktop for the project and code files and then have them use OCSdata to create a data dir within that directory with files to work with


Compilation errors when installing in sl-release-6.6-1.x86_64

Hi,
When following https://github.com/atom/atom/blob/master/docs/build-instructions/linux.md for fedora/centos/Rhel I get a lot of error messages when running script/build. I am told at the end of the message that:
npm ERR! Please include the following file with any support request:
npm ERR!     /home/acampove/Downloads/atom/npm-debug.log
I cant attach the file for some reason, but the lines at the end say:
10864 error node v0.10.40
10865 error npm  v2.13.3
10866 error code ELIFECYCLE
10867 error<EMAIL_ADDRESS>install: node-gyp rebuild
10867 error Exit status 1
10868 error Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
10868 error This is most likely a problem with the scrollbar-style package,
10868 error not with npm itself.
10868 error Tell the author that this fails on your system:
10868 error     node-gyp rebuild
10868 error You can get their info via:
10868 error     npm owner ls scrollbar-style
10868 error There is likely additional logging output above.
10869 verbose exit [ 1, true ]
Any idea what might be the problem?
Thanks.

This issue was moved to atom/scrollbar-style#10


python3Packages.adb-enhanced: 2.5.11 -> 2.5.12

Motivation for this change
Update to latest upstream release 2.5.12
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

2 packages built:

python38Packages.adb-enhanced
python39Packages.adb-enhanced


Refactor plugin to use octo.exe instead of the API to create/deploy releases

Also includes some minor changes to server and step configurations:

Store API Key as a secret instead of plain-text https://github.com/OctopusDeploy/octopus-jenkins-plugin/commit/c087af46b69f32ce8ceeb0bd7a0216f1cb5af577
Add verbose logging to steps https://github.com/OctopusDeploy/octopus-jenkins-plugin/commit/a16592727b34a65510149e466342b961b4d1da4a
Add ignore ssl errors to Octopus connection https://github.com/OctopusDeploy/octopus-jenkins-plugin/commit/2631678e9e1f93c013fb93c1e2b5aff16479bde8

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: hnrkndrssn:x: badrianceYou have signed the CLA already but the status is still pending? Let us recheck it.

Verbose logging looks good to me and works.

ignore ssl looks like it passes the correct parameter to Octo, but Octo is broken on linux.

Secret API key works


refactor: replace time() with Time in Session.php

Description
See #6815
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

Error: CodeIgniter\Session\Session must not depend on CodeIgniter\I18n\Time (Session on I18n)

@kenjis I have no idea how to fix this, can you give some suggestions for me?

--- a/deptrac.yaml
+++ b/deptrac.yaml
@@ -201,6 +201,7 @@ parameters:
       - Cookie
       - HTTP
       - Database
+      - I18n
     Throttle:
       - Cache
     Validation:

See https://qossmic.github.io/deptrac/

Appricated that!
I've been added that.


Write a test suite for gRPC auto API

Added to epic https://github.com/pulumi/pulumi-kubernetes-operator/issues/586


Cache version and is_aurora independently

Caches version and is_aurora independently.
Alternative to https://github.com/DataDog/integrations-core/pull/7471
We should address https://github.com/DataDog/integrations-core/pull/7471#issuecomment-685357152 on a separate PR

When will this be released? We use Datadog through the Heroku build pack, and starting around August 27th we started seeing endless logs from our Postgres instance like the following:
2020-09-04T17:57:14.000000+00:00 app[postgres.26800]: [GREEN] [4482-1]  sql_error_code = 42883 ERROR:  function aurora_version() does not exist at character 8
2020-09-04T17:57:14.000000+00:00 app[postgres.26800]: [GREEN] [4482-2]  sql_error_code = 42883 HINT:  No function matches the given name and argument types. You might need to add explicit type casts.
2020-09-04T17:57:14.000000+00:00 app[postgres.26800]: [GREEN] [4482-3]  sql_error_code = 42883 STATEMENT:  select AURORA_VERSION();
2020-09-04T17:57:20.000000+00:00 app[postgres.26797]: [GREEN] [4483-1]  sql_error_code = 42883 ERROR:  function aurora_version() does not exist at character 8
2020-09-04T17:57:20.000000+00:00 app[postgres.26797]: [GREEN] [4483-2]  sql_error_code = 42883 HINT:  No function matches the given name and argument types. You might need to add explicit type casts.
2020-09-04T17:57:20.000000+00:00 app[postgres.26797]: [GREEN] [4483-3]  sql_error_code = 42883 STATEMENT:  select AURORA_VERSION();
2020-09-04T17:57:21.000000+00:00 app[postgres.26798]: [GREEN] [4483-1]  sql_error_code = 42883 ERROR:  function aurora_version() does not exist at character 8
2020-09-04T17:57:21.000000+00:00 app[postgres.26798]: [GREEN] [4483-2]  sql_error_code = 42883 HINT:  No function matches the given name and argument types. You might need to add explicit type casts.
2020-09-04T17:57:21.000000+00:00 app[postgres.26798]: [GREEN] [4483-3]  sql_error_code = 42883 STATEMENT:  select AURORA_VERSION();

Hi @peterldowns, this change has been released on version 5.0.3 of the check, you can install it with the integration command.


Fixes to scheduled roles

This commit removes roles that show as scheduled when they should
not by:

Only showing active roles in show_scheduled_roles
If repoing encounters a problem, unschedule the role

Coverage decreased (-0.2%) to 58.094% when pulling a15e09953c5988ae7e6060fee8c7001e39e11e03 on mcpeak:fix-show-scheduled into a7db1037063b148b6481ee203e9f3c86f6edc443 on Netflix:master.

looks good


BioImage.IO id resource vs version id

We assign unique ids to all resources on the bioimage.io website. Specifically:
For zenodo contributions:

resource id: <concept doi>
version id: <version doi>

For partner contributions via github:

resource id: <partner id>/<partner internal resource id>
version id: "latest"  # we don't handle actual versions of github resources (yet). At some point htis could be a git hash... but this is a a different discussion.

The full version id of a given resource version is always <resource_id>/<version_id> as version_id is required to be unique only within the context of a resource.
Which id do we want to display?
Currently we show only resource ids, but when sharing the model using the resource id this hinders reproducibility. I'd argue that we should prefer the full version id
I propose showing something like this (please ignore the missing icons)

instead of the current

When using the resource id, e.g. in the links field the most reasonable thing would be to allow both resource and full version ids and resolve a resource_id to the latest version.
As a somewhat central issue, I'd appreciate some input @oeway @akreshuk @constantinpape @esgomezm @fjug

I don't have time to think about this too closely, but as a first comment:
I feel a bit uneasy about using / as  a separator for names that may themselves contain /.

I don't have time to think about this too closely, but as a first comment: I feel a bit uneasy about using / as a separator for names that may themselves contain /.

so true! This bothers me for a quile now...  How about we hexencode the '/' in all IDs? According to https://www.doi.org/syntax.html#:~:text=A series of experiments with,and double quote (").
"Additionally, any other characters may be hex encoded."
This way the shared url also contains the official id, because there the '/' is already hex encoded...

the '/' as separator between resource and version id is somewhat convenient in the backend (not a hard requirement though)

(%20 will be further encoded as %2520).

good point.

from a full doi into something simpler (e.g. 484829)

that sounds error prone, if we ever wanted to use sanbox zenodo again (or any other external source) this wouldn't work
It is highly encouraged not to use certain characters in a doi (control, space, hash mark (#), percent (%), and double quote ("); https://www.doi.org/syntax.html#:~:text=A series of experiments with,and double quote (")), so we should probably forbid those in id's. then if we do hexencode the '/' we don't have to do hex encode a second time and instead can use the id as is in a url.

that sounds error prone, if we ever wanted to use sanbox zenodo again (or any other external source) this wouldn't work

I don't really see why it will cause issue, the version id is internal, we can use anything we want, it can also even randomly generated. We use the first parts (10.5281/zenodo.484829) to locate the resource folder in gh-pages, then use the version id to find the version RDF sub folder or file.

I don't really see why it will cause issue, the version id is internal, we can use anything we want, it can also be a running number, a hash code or even randomly generated. We use the first parts (10.5281/zenodo.484829) to locate the resource folder in gh-pages, then use the version id to find the version RDF sub folder or file.

oh sorry, I didn't read carefully enough. sure we could do that for the version id, but it wouldn't solve the problem for the resource id that would still contain the '/'. Furthermore it is convenient that the version id is a valid doi for external contributions...

I think we should aim to only present a single id to users. Having two separate kind of ids will be confusing. I would vote for only showing the version id on the model card, because this unambiguously identifies which model is used.
I think the only downside of this approach is that a user may come back with the version id to the website and search for it, but don't find it because the model was already updated to a new version. I hope that we could build a technical workaround for this on the website, e.g. linking all cards to their previous versions so that the new version shows up in searches for the old version.
(For other types than models we don't have a real distinction atm in any case, so we just present the version id.)

sure we could do that for the version id, but it wouldn't solve the problem for the resource id that would still contain the '/'. Furthermore it is convenient that the version id is a valid doi for external contributions...

Well, I don't see any issue with the slash in resource id, the format is "partner_id/resource_id" or "partner_id/resource_id/version_id", and it maps well with the folder structure in gh-pages as you already mentioned. The browser will escape the slash in the url correctly. The only thing I would do is to avoid having slash in the version id to simplify the versioned resource id. Plus we have been using it in the website without problem, it's not the best use of our time to make such changes unless we have a strong reason.
In general, I think the resource id along with the version id is not meant to be interpreted outside the scope of bioimageio and the doi won't work for other RDF from partners too. During one of our last meetings and email exchanges with @akreshuk , we have also been discussing about generating id with ajactive+animal.

I think we should aim to only present a single id to users. Having two separate kind of ids will be confusing. I would vote for only showing the version id on the model card, because this unambiguously identifies which model is used.

agreed 👍

I think the only downside of this approach is that a user may come back with the version id to the website and search for it, but don't find it because the model was already updated to a new version

We should make sure that previous versions are still discoverable.

linking all cards to their previous versions so that the new version shows up in searches for the old version.

This requires bidirectional links, right?
Alternatively we can abuse the tags a bit by adding the resource id as tag

generating id with ajactive+animal.

IMHO this would always only be an alias for convenience and should not be the true id

Well, I don't see any issue with the slash in resource id, the format is "partner_id/resource_id" or "partner_id/resource_id/version_id", and it maps well with the folder structure in gh-pages as you already mentioned. The browser will escape the slash in the url correctly

it would be great if every part this composed id would be a valid id itself. Besides removing the tagging of the resource id problem this allows for even higher level grouping by partner.
Not allowing '/' or '%2F' in the id would only make creating urls a tiny bit more invovled as we skip escaping '%2F' and keep everything the same otherwise.

In general, I think the resource id along with the version id is not meant to be interpreted outside the scope of bioimageio

yes, then let's:

forbid "/" in any bioimage.io id
"destroy" the dois  (e.g. substituting "/" with "_")
show the version id for each card
maybe: show overview of versions in a resource card (which is also what you would get if you search for the resource id)

alternatively: make sure you get all versions as search results for a partial verison id (aka a resource id)

forbid "/" in any bioimage.io id

I partially agree, let's disallow it for anything submitted by partners and partner org names. Let's not touch DOIs.

"destroy" the dois  (e.g. substituting "/" with "_")

Why do we have to change anything there. My concern about slashes was due to your proposal of concatenating <resource_id>/<version_id>. As we don't want to do this any more we should leave DOIs as they are to avoid confusion.

show the version id for each card

Yes.

maybe: show overview of versions in a resource card (which is also what you would get if you search for the resource id)
* alternatively: make sure you get all versions as search results for a partial verison id (aka a resource id)

I think the conceptually easiest solution would be if searching for an old version brings up the new version on the website with a message that a new version of the resource is available. The old version can still be accessed through zenodo / bioimageio.core or other libraries. But I would leave this up largely to @oeway's disgression depending on what's feasible / easiest on the webiste. (We just need to make sure that searching for a version that was updated brings up some meaningful result)

My concern about slashes was due to your proposal of concatenating <resource_id>/<version_id>

this is not really a proposal, but rather stating how we define it atm
"The full version id of a given resource version is always <resource_id>/<version_id> as version_id is required to be unique only within the context of a resource."
This confusion is on me; I have not been distinguishing between 'verison id' and 'full version id' sufficiently in this issue.
Another attempt at solving this with minimal change: instead of a single forward slash '/' we forbid double forward slash '//'  and use '//' as separator between resource id and version id. no further changes need to be made. urls as well as file paths seem to ignore them and treat them as single slash, but for us humans it would look distinct. maybe another combination with a '/' like '/' would be suitable as well?

Update from short meeting with @oeway and @FynnBe, we will move ahead with the solution outlined in https://github.com/bioimage-io/bioimage.io/issues/237#issuecomment-1046981929

@oeway the updated rdfs in the collection repo will have the full version id as their id. It is probably the easiest to drop the last part when showing it on the website (esepcially if we also want the advanced full id)... So I'll leave that final id manipulation to you ;-)

I updated the handling of links:
https://github.com/bioimage-io/bioimage.io/blob/4ec005dd1456dd4dfbc581f880f984917b8c9384/src/views/Home.vue#L532-L554