ENH: Add support for ReadTheDocs popup

This adds some JS that "places" the ReadTheDocs popup in the sidebar so that it looks a bit nicer on page load. It creates a "top" and a "bottom" of our sidebar, so that things stay at the bottom regardless of the content at the top.

FYI: I’m not able to see the injected button on an iPad in landscape mode.

Cross-linking to: https://github.com/pradyunsg/furo/discussions/359 (was hoping @pradyunsg would do it, so then we could steal he's implementation 😅)

For me (macbook, firefox), its only visible after scrolling down

OK the latest commit cleaned things up a bit, but there's still one major UX issue that I think is related to both of the things you reported above:
The height of the sidebar container is 100vh. However, with the announcement bar at the top, it means that the sidebar extends slightly below the screen. This causes the RTD button to be hidden until somebody scrolls down enough to hide the announcement bar.
A better approach would be to somehow get the sidebar container to stretch to fill the remaining vertical space that fits within the screen, however much it is. However, I can't for the life of my figure out how to do it. I've tried align-self: stretch, but that doesn't seem to work, so I'm sort-of at a loss.

@choldgraf, it has actually been implemented in furo now 🎉 , did you look at that implmentation?

If it's in a flex container, flex-grow: 1 will make it fill the parent. I'm not sure if that's what you want tho. :)

yeah I feel like the issue here is some kind of CSS bug that is probably not directly portable to the Furo theme :-/  I can have another look at how the sidebar behavior works though in case that helps illuminate things.
The weird thing is that on the "mobile view" of my browser, the RTD button shows up in the sidebar just fine, but when I try it with my actual phone, it is hidden below the screen. I dunno what's up with that.
@pradyunsg the issue w/ flex-grow is that the flex direction is row, not column, so flex-grow makes the sidebar take up more horizontal space, while I want it to take up all of the available vertical space.

Ah, the problem is the announcements.
I have the same problem on https://furo.readthedocs.io/en/latest/; so if you figure something out, lemme know. :P

OK so at least on mobile I think that I've resolved the issue. Apparently vh units behave differently across mobile devices and it is less reliable in general, so working around that in the latest commit.
At this point the expected behavior is:

On widescreen, the RTD button should be visible by default
On widescreen with an announcement, it will be hidden until you scroll past the announcement
On mobile, it should be visible when you slide out the left drawer

The second bullet is definitely sub-optimal, but hopefully not a huge deal since announcements will likely be used relatively rarely, and if we can resolve #488 , then they can be dismissed on subsequent pageviews.

(If anything here seems to be broken other than the announcement pushing down the RTD button, please let me know!)

Actually I think that I might have solve the issue of the footer floating to the bottom in executablebooks/sphinx-book-theme@ddd5d48 (#518) , we should double-check that nothing else unexpectedly changed.

OK the latest push cleans up the CSS a bit so that it looks more in-line with how the other visual elements of the theme look. It is maybe a bit harder to spot, but I think that it looks a lot cleaner, and since it's in the same position as the RTD theme switcher I'd hope it won't be too confusing. I updated the top comment w/ a GIF of how it looks.

ok gonna merge this one in and we can iterate on more improvements if there are issues that pop up!


Ignoring task configuration message pesters the user all the time

Refs: #34953

Currently we show the "Ignoring task configuraiton" message every time a user runs a task
There is absolutely no way for the user to say never show this again

It seems like you are also showing this message whenever a user edits a tasks.json which feels wrong.
Why not use the message service instead, with a never show again action?

This should only be printed once or when you change a tasks.json file.
I got bad feedback about using the message service. This is why all errors from parsing tasks.json files go into the output window. I will make sure that the message is only shown once and that I don't bring the output panel to front
@isidorn would be nice to have a badge on the output panel that indicated unread messages.

@dbaeumer regarding the badge there is already an issue for this https://github.com/Microsoft/vscode/issues/34703

@isidorn thanks

I now show a detailed message in the output window and a message box which has a Don't show again button. The output window doesn't come to front.


The filtering for gguf files in ModelScope is not working

The filtering for gguf files in ModelScope is not working:

UI version: 5dde5ca
Test Result: pass


this is small change

I would prefer Jenkins Zero to Hero so that more people can understand what the repo is about. Thanks alot 🙏


Typo fix for doc

Description
Typo fix for doc: https://xcdat.readthedocs.io/en/latest/generated/xarray.Dataset.temporal.climatology.html, description for reference_period.
('1850-01-01', 1899-12-31') --> ('1850-01-01', '1899-12-31') (missed single quotation mark in front of 1899 added)
Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules

If applicable:

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass with my changes (locally and CI/CD build)
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have noted that this is a breaking change for a major release (fix or feature that would cause existing functionality to not work as expected)

Thanks Jiwoo! Looks good to me.


Fix serialization of transaction_paymentInfo

We serialize to serde_json::Value internally in jsonrpc and without arbitrary_precision feature the u128 (used as Balance) is not support and causes a failure.
Also fixes the format of returned response to be more javascripty.
CC @jacogr

Looks like polkadot.js stuck at connecting websocket :(


Improve error message display

Codecov Report

Merging #108 into release-rc4 will decrease coverage by 0.31%.
The diff coverage is 12.16%.

@@                Coverage Diff                @@
##             release-rc4     #108      +/-   ##
=================================================
- Coverage          32.37%   32.06%   -0.32%
- Complexity           361      363       +2
=================================================
  Files                 54       55       +1
  Lines               4494     4560      +66
  Branches             380      392      +12
=================================================
+ Hits                1455     1462       +7
- Misses              2931     2987      +56
- Partials             108      111       +3

Impacted Files
Coverage Δ
Complexity Δ

...bank/wecross/stub/bcos/contract/RevertMessage.java
11.42% <11.42%> (ø)
2.00 <2.00> (?)

.../java/com/webank/wecross/stub/bcos/BCOSDriver.java
22.10% <12.82%> (-0.46%)
34.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 8766815...0b7985b. Read the comment docs.


Feature/player stats reset/#86c09ubxw

Try Again Sends u to the current level starting place
Stats reset if going to main menu
Save Damage/Health/Speed/Dialog

Task linked: CU-86c09ubxw Reset stats when dying or going to menu


Deploy bot to Azure button

As the functionality to deploy the bot to Azure is not loading the expected form, would it be possible to provide manual instructions detailing how to get to the point of filling in the form to deploy the bot to Azure?
"Now that you've generated your Application Id and Password, it's time to deploy your bot to Azure. Click on the Deploy to Azure button and fill out the form with the necessary information (the Bot Application Id, Bot Application Password and the Moodle Secret are on the Team Settings page, and the Azure information is on the Setup page). Once you've got the form filled out, click the check box to agree to the terms and conditions then click the Purchase button (all Azure resources are deployed to the free tier)."
Thanks
Julia

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

ID: 685f497e-473f-7767-3f4e-5a0d1271a2ec
Version Independent ID: 79dc20db-c743-8150-a660-abd9ffb03b4d
Content: Installing the Moodle integration with Microsoft Teams - Teams
Content Source: msteams-platform/MoodleInstructions.md
Product: msteams
GitHub Login: @o365devx
Microsoft Alias: o365devx

Hi - My apologies for the late reply. I've been unable to reproduce the behavior you seem to be experiencing; can you please provide some more details? The button appears to be working fine in our test environments.

Hi Andrew
Thanks for getting back to me. It seems the issue may be the user account I was using not having access to our institution’s Azure subscription. I will try with the actual admin account, once I can arrange this.
Kind regards
Julia
[https://secure.bhasvic.ac.uk/images/logo.png]  JULIA HOLGATE TURNER
VLE Coordinator
01273552200 Ext: 4819
Email: j.holgateturner<EMAIL_ADDRESS>[https://secure.bhasvic.ac.uk/images/facebook.png]https://www.facebook.com/bhasvic.ac.uk      [https://secure.bhasvic.ac.uk/images/twitter.png] https://twitter.com/bhasvic         BHASVIC (Brighton Hove & Sussex Sixth Form College) 205 Dyke Road, Hove, BN3 6EG |      www.bhasvic.ac.ukhttps://www.bhasvic.ac.uk/home
[https://secure.bhasvic.ac.uk/images/eco.png]   Before printing, think about the environment
The content of this message is confidential. It is forbidden to copy, forward, or in any way reveal the contents of this message to anyone. If you have received it by mistake, please inform us by an email reply and then delete the message. The integrity and security of this email cannot be guaranteed over the Internet. Therefore, the sender will not be held liable for any damage caused by the message. Any views or opinions expressed are solely those of the author and do not necessarily represent those of BHASVIC.
From: Andrew Clear<EMAIL_ADDRESS>Sent: 08 March 2019 15:51
To: MicrosoftDocs/msteams-docs
Cc: Julia Holgate Turner; Author
Subject: Re: [MicrosoftDocs/msteams-docs] Deploy bot to Azure button (#461)
Hi - My apologies for the late reply. I've been unable to reproduce the behavior you seem to be experiencing; can you please provide some more details? The button appears to be working fine in our test environments.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/msteams-docs/issues/461#issuecomment-470975608, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AYpiQophnVA978G0wInLTBiQeRm5_a6Cks5vUob1gaJpZM4bDCHz.


LLVM ERROR: Section too large, can't encode r_address (0x1000008) into 24 bits of scattered relocation entry

Operating system or device - Godot version:
macOS High Sierra 10.13.1, Godot 2.1.4 stable, Xcode 9.1, iOS 11
Issue description (what happened, and what was expected):
After updated Xcode to version 9.1, I recompiled a Godot project which worked just fine with Xcode 8 and found this link error:
LLVM ERROR: Section too large, can't encode r_address (0x1000008) into 24 bits of scattered relocation entry.
clang: error: linker command failed with exit code 1 (use -v to see invocation)
scons: *** [bin/godot.iphone.opt.x86] Error 1
scons: building terminated because of errors.
It only appeared on compilation for simulator, compiling for iDevice is ok.
Steps to reproduce:
Compile Godot with: "Scons platform=iphone tools=no bits=64 target=release ios_sim=yes" for Simulator
Thanks for helping!

That's because you're compiling for x86, iOS 11 doesn't supports 32 bits anymore I believe.

Which means there is no way to compile for iOS simulator ?

I noticed that the 2.1 branch can be compiled for the IOS simulator.
I replaced the ./platform/iphone/detect.py of the 2.1.4 release with the one from the "current" 2.1 branch (and did a "touch" on it). Then the build succeeded:
scons p=iphone target=release_debug ios_sim=yes
As far as I understood, the link time optimization option (-flto) was used in 2.1.4 stable for the "release_debug" target and meanwhile removed (unless forced via paramater).
But I'm an absolute beginner on XCode and iOS..

Thank you wombatstampede. Replacing detect.py with the one from 2.1 branch made the build succeed, but failed while running in the simulator. As you said, I had to download the old version of iOS simulator so that the Godot executable could run on it. I think marcelofg55 is right, iOS 11 doesn't supports 32 bits anymore.


Example of APPNP

❓ Questions & Help
Hi,
I want to test the method proposed in paper "Predict then Propagate: Graph Neural Networks meet Personalized PageRank", and PyG has implemented it as APPNP. But I can not find the example that introduces how to use it.
Thank you.

You can find an implementation in the benchmark suite.

You can find an implementation in the benchmark suite.

Thank you.


Allow specified resource types and names for kube-objects collector

Addresses #187
As well as using the resource type from the <namespace>/<resource-type> values in the DIAGNOSTIC_KUBEOBJECTS_LIST environment variable, this also allows specifying individual resources with an extra /, e.g. <namespace>/<resource-type>/<resource-name> (because the az CLI examples claim that behaviour).
This also adds a number of automated tests to cover the behaviour being fixed.

Codecov Report

Merging #188 (bfb0962) into master (9030f55) will decrease coverage by 0.17%.
The diff coverage is 72.72%.

@@            Coverage Diff             @@
##           master     #188      +/-   ##
==========================================
- Coverage   78.82%   78.64%   -0.18%
==========================================
  Files          13       13
  Lines         713      740      +27
==========================================
+ Hits          562      582      +20
- Misses         92       97       +5
- Partials       59       61       +2

Impacted Files
Coverage Δ

pkg/collector/kubeobjects_collector.go
77.96% <72.72%> (-3.29%)
:arrow_down:

Continue to review full report at Codecov.

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


Error message: Couldn't find status ''

Just asking for the explanation/"how-to" advice:
I tried to create a workflow for closing issues (when an issue status in a project is "Done"). I adopted the example included in the README - in steps I skipped the line open_statuses: ... as I only needed to close issues when status changed to "Done".
After running the workflow I got the error Couldn't find status ''.
Then I added the line open_statuses: To Do, In Progress.... back => now, if I close an issue with a status other than "Done", this issue gets reopened automatically.
How can just close issues with status "Done" whitout any issue reopening?
BTW: great feature!

Hey @Maciej818,
You did everything correctly. For some reason, I receive empty strings when no parameter is provided which causes this error.
I fixed this bug and released a new version. So now skipping the line open_statuses: ... should work again.
Feel free to give it another try and close this issue if it works for you now. :)

Hey @dasmerlon - it worked for me, thank you :)


Exec tachikoma update<PHONE_NUMBER>0209

:hamster::hamster::hamster:   Strategy david
Powered by David Update as a Service - Tachikoma.io Announcement: 2015-06-28 Updated permissions - Tachikoma.io

Coverage remained the same at 85.644% when pulling 9ff18aa75ea1406769ed35b64cee70dee4fd7a0b on pazjacket:tachikoma/update-20160522170209 into e5ed21352e4f5edfb52d407a92301635461c02b7 on festivals-tech:master.


CI: Update jekyll-label-bot.yml

When opening a pull request to submit a new EIP, please use the suggested template: https://github.com/ethereum/EIPs/blob/master/eip-template.md
We have a GitHub bot that automatically merges some PRs. It will merge yours immediately if certain criteria are met:

The PR edits only existing draft PRs.
The build passes.
Your GitHub username or email address is listed in the 'author' header of all affected PRs, inside .
If matching on email address, the email address is the one publicly listed on your GitHub profile.

looks like it still fails? or is that expected?

The old version is still used before the merge, which should be observed after the merge

Thank you

Couldn't get 1 pull request

If is B when together. I find the wrap around lines difficult personally. Fi is B and 1 and 0 is 10 or 01. I do see the n-2 is 3

Fi in 1 . . 0 easy because makes 6


Can we add iceberg table type to either glue create_parquet_table or s3 to_parquet?

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Add a table type arg to either wr.glue.create_parquet_table or wr.s3.to_parquet api calls
Describe alternatives you've considered
Right now boto3 glue client can do IcebergInput in create_table api call
Additional context
It would add functionalities for new iceberg table type
P.S. Please do not attach files as it's considered a security risk. Add code snippets directly in the message body as much as possible.

This was proposed in https://github.com/aws/aws-sdk-pandas/issues/1400 and deemed infeasible because we don't have access to the APIs to implement it in s3.to_parquet. Have you considered wr.athena.to_iceberg?

No I have not. I guess I could use it. Thanks!


[ZEPPELIN-4378] porting Cassandra interpreter to new driver

What is this PR for?
This PR refactors Cassandra interpreter to use new DataStax Java driver (4.x) that has new features & new architecture, that caused significant changes in the code base. But as result, we're getting:

We get supported version of the driver (3.x line will only receive critical bug fixes)
We get support for additional functionality of the DSE (DataStax Enterprise) - security, additional data types, etc.
Improvements in the load balancing, etc.

What type of PR is it?
Refactoring
What is the Jira issue?

ZEPPELIN-4378

How should this be tested?

Travis CI build: https://travis-ci.org/github/alexott/zeppelin/builds/667208139. The failing tests are the same as in the existing master
Was tested manually in addition to the unit tests

Questions:

Does the licenses files need update? Done, minor updates
Is there breaking changes for older versions? There are renames & deletion of the configuration properties, but this doesn't break existing setups
Does this needs documentation? Documentation was updated

@doanduyhai can you help review it?

This is a massive update. I need to check it out and review it locally !

Yep, the new driver requires big refactoring...
I didn’t change much from functional point of view, except removing retry
policy...
On Thu 26. Mar 2020 at 22:34, DuyHai DOAN<EMAIL_ADDRESS>wrote:

This is a massive update. I need to check it out and review it locally !
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/zeppelin/pull/3699#issuecomment-604698981, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAHNBUE5GT54YBHLE54UATRJPC7VANCNFSM4LUFSVTQ
.

--
With best wishes,                    Alex Ott
http://alexott.net/
Twitter: alexott_en (English), alexott (Russian)

Looks good to me.
Please solve the conflics before merging
Thank for the fastastic job of migrating the driver version, it is not an easy task


Create the IConcept interface that extends the IIdentifiableArtefact interface

The IConcept (GSIM: Concept) interface is the interface that extends IIdentifiableArtefact (Identifiable Artefact GSIM)
Properties:

extends IIdentifiableArtefact
Definition (Multilanguage String)
This interface should be found on Presentation.Common.Domain under StructuralMetadata/Interfaces/Gsim/Concept folder

I believe Domain is more suitable to be an abstract class as well. I have changed Domain from an interface to an abstract class and extended Concept from Domain.

Please put IConcept and IUnitType under StructuralMetadat/Interfaces/Gsim/Concept folder

Changes have been made
Commit: 101fe363a8821a370f37bb6985639cc180905ff1 [101fe36]
Parents: a9d1774f00
Author: Veaceslav Chirilov<EMAIL_ADDRESS>Date: Friday, July 2, 2021 12:48:00 PM
Committer: Veaceslav Chirilov

Please remove all unused references from each class


[BIFROST-7955] add_applied_schema and remove_applied_schema nodes

Describe the feature you'd like
Implement UsdPrim::AddAppliedSchema and UsdPrim::RemoveAppliedSchema as Bifrost nodedefs.
This would allow to add or remove schemas names on the apiSchemas metadata of a prim directly from within the Bifrost graph.
For more info about USD Applied schemas: https://graphics.pixar.com/usd/release/api/_usd__page__generating_schemas.html

This is implemented and pushed to dev branch.


JSONDecodeError while loading pre trained model

Python Version: 3.6
Installed DeepPavlov using:
pip install DeepPavlov
Command I ran:
python -m deeppavlov interact ./bot_kvret_infer.json
ERROR:
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/local/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/floyd/home/deeppavlov/__main__.py", line 3, in <module>
    main()
  File "/floyd/home/deeppavlov/deep.py", line 70, in main
    interact_model(pipeline_config_path)
  File "/floyd/home/deeppavlov/core/commands/infer.py", line 93, in interact_model
    config = read_json(config_path)
  File "/floyd/home/deeppavlov/core/common/file.py", line 23, in read_json
    return json.load(fin)
  File "/usr/local/lib/python3.6/json/__init__.py", line 299, in load
    parse_constant=parse_constant, object_pairs_hook=object_pairs_hook, **kw)
  File "/usr/local/lib/python3.6/json/__init__.py", line 354, in loads
    return _default_decoder.decode(s)
  File "/usr/local/lib/python3.6/json/decoder.py", line 339, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/local/lib/python3.6/json/decoder.py", line 357, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 7 column 1 (char 6)

Please help if I am doing something wrong.

JsonDecodeErroroccurs in case of errors in format of json file bot_kvret_infer.json.
Are you using the same bot_kvret_infer.json as in deeppavlov? Or maybe you've edited it and missed a comma?

I am still getting error when running following commands:
python -m deeppavlov interact squad.json
!python -m deeppavlov install squad.json
Note: squad.json is already downloaded
and deeppavlov is installed using pip
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/local/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/usr/local/lib/python3.6/site-packages/deeppavlov/__main__.py", line 3, in <module>
    main()
  File "/usr/local/lib/python3.6/site-packages/deeppavlov/deep.py", line 70, in main
    interact_model(pipeline_config_path)
  File "/usr/local/lib/python3.6/site-packages/deeppavlov/core/commands/infer.py", line 97, in interact_model
    config = read_json(config_path)
  File "/usr/local/lib/python3.6/site-packages/deeppavlov/core/common/file.py", line 21, in read_json
    return json.load(fin)
  File "/usr/local/lib/python3.6/json/__init__.py", line 299, in load
    parse_constant=parse_constant, object_pairs_hook=object_pairs_hook, **kw)
  File "/usr/local/lib/python3.6/json/__init__.py", line 354, in loads
    return _default_decoder.decode(s)
  File "/usr/local/lib/python3.6/json/decoder.py", line 339, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/local/lib/python3.6/json/decoder.py", line 357, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 7 column 1 (char 6)

Please help

@vikmary sorry for the noise.
It was download issue. I provided wrong url when I used wget utitliy.
When I opened up, It was all HTML file.
Now the issue is fixed.


Parsing Headers and Getting Cookies

How do I access the headers and get the cookies from an incoming HTTP:Request? So far I am aware that the following can be done:
for(auto& h : req.base())
{
    std::cerr << "Header: "<< h.name()<< ", Header Name: " << h.name_string() << ", Value: " << std::string(h.value()) << "\n";

But for some headers like:
Content-Type
Content-Type: multipart/form-data; boundary=something

https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Type
There does not seem to be a built in way to access directives like "boundary=something"
It seems that most headers lack support by beast, but I am wondering if I missed something in the docs?

No, it's not supported.


Capella model loading failure due to Duplicate UUID

Hello everyone,
I'm trying to load a Capella model located in a local git repository and I get the following error message:
capellambse.loader.core.CorruptModelError: Duplicate UUID 'XXXX' within fragment My_Model.aird
Can anyone help me fixing the problem, please?
Regards

Hi! That's a somewhat known bug that occasionally happens, and as long as it's only within the *.aird file it's actually safe to ignore (things change a bit when it comes up in a *.capella file. You can still load the model by setting this option:
model = capellambse.MelodyModel(
    ...,
    ignore_duplicate_uuids_and_void_all_warranties=True,
)

(Don't worry. It sounds scarier than it is, because when we first discovered it we weren't sure what would happen ourselves. ;) )

@Luffy-cpu , Capella can mess-up models sometimes but it will not complain about it, it will just take an element and go with it (not sure though if first or last XD. If you use capellambse lib to edit models, a model with such issue is "unsafe" for modifications / can get further messed up, you should clean it up first - locate the duplicate elements in XML and choose which of those you want to keep or delete - but at the end there shall only one element remain with that UUID. However, if you are using library to simply check things / read content / produce some reports - not to modify the model - you can do what @Wuestengecko  suggested above and simply ignore the duplicates. @Wuestengecko , we may want to update the docs so that it is a bit easier to get on track when you face such exception.

Hello again,
Thank you very much @Wuestengecko & @vik378. Both of your answers are very useful for me.

Issue solved :)


this.$router in setTimeout doesn't work

Version
3.0.1
Reproduction link
https://jsfiddle.net/mu6f7zj6/
Steps to reproduce
click Go to Foo , and console said Cannot read property 'push' of undefined.
What is expected?
1s after clicked Go to Foo should redirect to '/' (home), and 'foo' should disapear
What is actually happening?
Cannot read property 'push' of undefined.

There's nothing related to Vue, that's because you're using wrong this context is setTimeout callback.
You should learn more about pure js before using Vue.

solved by add "var self = this, self.$router.push()"

"tonywangcn"  Good Job


Oracle Long, .Net String Property stays empty.

When we use Long as Oracle Type and String as respective .Net type, our Property stays allways empty. How can we solve this problem?
https://stackoverflow.com/questions/18511651/how-to-handle-oracle-long-string-in-vb-net

Okay, I solved this issue by using CLOB instead of LONG, also Oracle indicates that we should migrate from LONG to LOB. https://docs.oracle.com/cd/B28359_01/appdev.111/b28393/adlob_long_lob.htm#i1022660


Add huawei to adopters

Signed-off-by: Poor12<EMAIL_ADDRESS>What type of PR is this?
/kind documentation
What this PR does / why we need it:
Add huawei to adopters
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
None

/cc @GitHubxsy


Add button should be fixed at the end of the task form #230

Add button should be fixed at the end of the task form #230

@all-contributors please add @qibao0722  for code, design


Not passing through properties

Hi
sorry - I wasn't explicit enough in my request to pass through other properties  (like topic (bad example)) - I actually meant all properties - msg.foo msg.bar etc - ie just add yours to the existing incoming message... rather than create a newMsg - just use the incoming msg that triggered the action in the first place, and overwrite the properties you wish (command and payload - and possibly replace topic with the function/mode type).
Thanks

Ah! This makes much more sense. Not sure why I didn’t do this in the first place.  I’ll push out a fix later today.


Update python Werkzeug to 0.10.4

Update python Werkzeug package version to 0.10.4

Looking at travis' logfile the build timed out, not sure how to force it to retry.

Could you run nox-review pr 11209 to test whether dependents still work after this change?

@FRidh on OS X didn't get too far with it, but that is likely the OS X side of things really.
https://gist.github.com/886b88087774ec2db7ba
I'll try when I get home tonight on nixos to see if anything changes.

ping @mitchty

Still building, my nixos laptop is a chromebook, not the speediest of systems. Not sure how to pass along -j or --cores via nox-review but that would speed it up a fair bit as it seems to be rebuilding the world.

@mitchty tnx


Background eviction using dml::executedml::hardware(dml::batch, ..)

This change is

I think you should do rebase because right now this PR contains "Adding libdml to CentOS docker image" commit, but it is already in develop branch.


Detect kernels of other langauges and update the language property of the notebook metadata

This is required to support non-python kernels.
Solution (as discussed some time ago with team)

Get a list of all kernels and find all non-python kernels

Determine the languages of these kernels and map then again VSCode langauages (C# or .NET is not a language, in VSC its csharp, etc)

Modify contentProvider.ts and ensure we update the langauges property to include the other languages (append other languages to this proeprty).

is great , will be greater if it goes true also for remote non-python kernels.


feat: blocking incoming IBC transfers with a dedicated middleware

An initial version of the functionality that allows blocking incoming IBC token transfers for blacklisted addresses. The code is dirty, but it seems to be working.
Implemented as an IBC middleware module blockibc that sits in front of the transfer module and intercepts incoming packets. If a receiver in the packet is in the blacklist (from the tokenfactory module), an error ack is sent back.


NewtonSoft.JSON

HI,
Have you tested it on IOS IPAD/Iphone and Android..
We have had problems usign standard version of Newtonsoft.json on ipad, worked fine on PC/MAC..
We had to use the version of Newtonsoft.JSON located in unity store v. <IP_ADDRESS>.
Thanks

Hi,
You can't use two versions of Newtonsoft and it seems to me that UnityGLTF is using the 8.0 Newtonsoft.
This explains why.
Regards


[Feature Request] Bring back confirmation for training and restart

In 2.4 it would always ask me if I want to train now or later when I saved words, sentences. This is now gone and it trains directly instead. I really liked that feature because I could make changes in multiple places and then train only once, now I have to wait for training to finish for every change I make.
Restarting rhasspy is the same thing, once I click save in the settings tab it restarts. Most of the time that is useful but I actually managed to freeze my whole system by accidentally saving (and therefore restarting) while training was still running. Also I might want to just make a change, look at the advanced settings to see what it does and then undo the change again, or edit it further in the advanced settings, so having to restart two or more times in between is kinda annoying.
On a sidenote, some settings display a small red icon next to them after I change them (I am guessing to restart) but it restarts anyway once i click save, either the icon is not necessary or it wasn't intended to restart with every save.
Daenara

Added this locally. Will be in the next release :+1:


fix: Parse root certificate from either file or directory in tedge cert upload

Proposed changes

Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Improvement (general improvements like code refactoring that doesn't explicitly fix a bug or add any new functionality)
[ ] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue
https://github.com/thin-edge/thin-edge.io/issues/2951
Checklist

[ ] I have read the CONTRIBUTING doc
[ ] I have signed the CLA (in all commits with git commit -s)
[ ] I ran cargo fmt as mentioned in CODING_GUIDELINES
[ ] I used cargo clippy as mentioned in CODING_GUIDELINES
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

I created test case that should cover this PR maybe
*** Settings ***
Resource ../../resources/common.resource
Library ThinEdgeIO
Library String
Library Collections

Test Teardown Get Logs

Test Tags theme:ca_certificates

*** Test Cases ***

Verify Single Certificate File
[Setup] Setup With Self-Signed Certificate
ThinEdgeIO.File Should Exist /etc/tedge/device-certs/tedge-certificate.pem
${output_cert}= Execute Command cat /etc/tedge/device-certs/tedge-certificate.pem
Should Contain ${output_cert} -----BEGIN CERTIFICATE-----

Verify Single Private Key File
[Setup] Setup With Self-Signed Certificate
ThinEdgeIO.File Should Exist /etc/tedge/device-certs/tedge-private-key.pem
${output_key}= Execute Command cat /etc/tedge/device-certs/tedge-private-key.pem
Should Contain ${output_key} -----BEGIN PRIVATE KEY-----

Verify Multiple Certificates in Directory
[Setup] Setup With Self-Signed Certificate
ThinEdgeIO.File Should Exist /etc/tedge/device-certs/tedge-certificate.pem
${dir_contents}= Execute Command ls /etc/tedge/device-certs
Log ${dir_contents}

# List .pem files and check the result
${cert_files}=    Execute Command    ls /etc/tedge/device-certs/*.pem
${cert_files_list}=    Split String    ${cert_files}    \n
@{filtered_cert_files}=    Create List
FOR    ${file}    IN    @{cert_files_list}
    Run Keyword If    '${file}' != ''    Append To List    ${filtered_cert_files}    ${file}
END

${cert_files_length}=    Get Length    ${filtered_cert_files}
Should Be True    ${cert_files_length} > 1
FOR    ${cert_file}    IN    @{filtered_cert_files}
    ${output_cert}=    Execute Command    cat ${cert_file}
    Should Contain Any Line    ${output_cert}    ${cert_file}
END
Verify Invalid Path
[Setup] Setup With Self-Signed Certificate
${result}= Execute Command ls /invalid/path/*.pem ignore_exit_code=True stderr=True stdout=True
${stdout}= Set Variable ${result}[1]
Should Contain ${stdout} No such file or directory

*** Keywords ***

Setup With Self-Signed Certificate
${DEVICE_SN}= Setup skip_bootstrap=${True}
Set Test Variable $DEVICE_SN
Execute Command test -f ./bootstrap.sh && ./bootstrap.sh --cert-method selfsigned

Should Contain Any Line
[Arguments] ${text} ${cert_file}
${cert_present}= Run Keyword And Return Status Should Contain ${text} -----BEGIN CERTIFICATE-----
${key_present}= Run Keyword And Return Status Should Contain ${text} -----BEGIN PRIVATE KEY-----
Run Keyword If ${cert_present} Log Certificate found in ${cert_file}
Run Keyword If ${key_present} Log Private key found in ${cert_file}
Run Keyword If not ${cert_present} and not ${key_present} Fail The file ${cert_file} does not contain a valid certificate or private key.


videogrid does not work.

Code in lib/videogrid is imagegrid

It's difficult to identify the isssue with description you have provided, can you provide code snippet on how you are using video grid component?

@sai6855
As mentioned, the code you put for VideoGRID Component is for ImageComponent. Also, the videogrid example in the readme
file is also not correct for single video,

After testing the ImageGrid to video tag, I understand it now.
You change the component name to MediaGrid (which is for both, image and video) then, if the file['type'].include('image') => render img tag, if not, video tag
https://codesandbox.io/s/react-fb-image-video-grid-forked-b6vl70?file=/src/App.js

Thanks for spotting the issue, i'll update the readme.
Yes you are right currently both ImageGrid and VideoGrid uses same code, we have exported as seperate components because in future if we want to add any props which are media specific or even change some of functionality which are specific to either of components we can do it without breaking the existing code.

closing the issue as readme is updated


Why appear "NaN" on http://battlefield.wikiloop.org/marked

Why appear "NaN" on http://battlefield.wikiloop.org/marked

You are right, this is a bug. I will fix it soon

@xinbenlv I found why is this shows nan.
Time string shows NaN because of this code piece. new Date(this.revision.timestamp).toString()

I'll fix it if I can get an wiki key :)
What's a wiki key? Happy to provide

Okay, I'm deploying now. Btw, I meant I didn't see any data when I run it my locale. I thought this was about MediaWiki consumer key :)

It's not. The reason it has zero data is because a new Heroku Review App
will also create a new MongoDB instance, which is completely empty.
On Tue, Dec 17, 2019 at 11:39 PM Ali GOREN<EMAIL_ADDRESS>wrote:

Okay, I'm deploying now. Btw, I meant I didn't see any data when I run it
my locale. I thought this was about MediaWiki consumer key :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/wikiloop-battlefield/issues/121?email_source=notifications&email_token=AAE4KRPIZLYMNS5ITPOQ4ULQZHHSNA5CNFSM4J2IXHPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHFHA2I#issuecomment-566915177,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE4KRLSPWGGTX74Y3TSVC3QZHHSNANCNFSM4J2IXHPA
.


How to use to expose ports

How does this solve the

Exposing ports from a container while still not letting the container access the WWW

usage listed in the README?
I see traefik used for that in the example instead

Use this pattern.


How to serialize and deserialize transformers in java8

I am looking for java implementation of the below code with all the necessary imports:
Serialization
val pipeline = SparkUtil.createPipelineModel(uid = "pipeline", Array(featureModel, rfModel))

val sbc = SparkBundleContext()
for(bf <- managed(BundleFile("jar:file:/tmp/mnist.model.rf.zip"))) {
        pipeline.writeBundle.save(bf)(sbc).get
      }

val bundle = (for(bundleFile <- managed(BundleFile("jar:file:/tmp/simple-spark-pipeline.zip"))) yield {
  bundleFile.loadMleapBundle().get
}).opt.get

Deserialization
val model = (for(bf <- managed(BundleFile("jar:file:/tmp/simple-spark-pipeline.zip"))) yield {
      bf.loadMleapBundle().get.root
    }).tried.get

@sheikirfan Have a look at https://github.com/combust/mleap/blob/master/mleap-runtime/src/test/scala/ml/combust/mleap/runtime/javadsl/JavaDSLSpec.scala#L81, it should help to get you started.

Closing this issue.

@ancasarb The URL is 404...sad

@SoloBean apologies, we've actually converted that scala class to a java one to make sure that the API works fine in Java.
here's the updated link: https://github.com/combust/mleap/blob/master/mleap-runtime/src/test/scala/ml/combust/mleap/runtime/javadsl/JavaDSLSpec.java


about the show problem in object-c

Hello!
Why my code like this won't show the popup?
`- (void)addImageToAlbum
{
PHPhotoLibrary *lib = [PHPhotoLibrary sharedPhotoLibrary];

[lib performChanges:^{

    PHAssetCollectionChangeRequest *assetCollectionChangeRequest;

    PHAssetCollection *assetCollection = [self getAssetCollection];

    if (assetCollection) {

        assetCollectionChangeRequest = [PHAssetCollectionChangeRequest changeRequestForAssetCollection:assetCollection];
    } else {

        assetCollectionChangeRequest = [PHAssetCollectionChangeRequest creationRequestForAssetCollectionWithTitle:albumTitle];
    }

    PHAssetChangeRequest *assetRequset = [PHAssetCreationRequest creationRequestForAssetFromImage:_image];

    [assetCollectionChangeRequest<EMAIL_ADDRESS>
} completionHandler:^(BOOL success, NSError * _Nullable error) {

    if (success) {
        NSLog(@"%@",[NSThread currentThread]);
        [SVProgressHUD showSuccessWithStatus:@"success"];
    } else {
        [SVProgressHUD showErrorWithStatus:@"fail"];
    }
}];

}
`
But when I modify thread to main thread,it will show the popup.
`
-(void)addImageToAlbum{
PHPhotoLibrary *lib = [PHPhotoLibrary sharedPhotoLibrary];
[lib performChanges:^{
    PHAssetCollectionChangeRequest * assetCollectionChangeRequest;

    PHAssetCollection * assetCollection = [self getAssetCollection];
    if (assetCollection) {

        assetCollectionChangeRequest = [PHAssetCollectionChangeRequest changeRequestForAssetCollection:assetCollection];
    }else{
        assetCollectionChangeRequest = [PHAssetCollectionChangeRequest creationRequestForAssetCollectionWithTitle:albumTitle];
    }

    PHAssetChangeRequest *assetRequest = [PHAssetChangeRequest creationRequestForAssetFromImage:_image];

    [assetCollectionChangeRequest<EMAIL_ADDRESS>
} completionHandler:^(BOOL success, NSError * _Nullable error) {
    dispatch_async(dispatch_get_main_queue(), ^{
        if (success) {
             NSLog(@"%@", [NSThread currentThread]);
            [SVProgressHUD showSuccessWithStatus:@"success"];
        }else{
            [SVProgressHUD showErrorWithStatus:@"fail"];
        };
    });

}];

}`
But I don't think it is the thread problem.
Can you help me?

Hi @caojunhua,
This limitation comes from UIKit, and you should always update the UI on the main thread.
Also note that usually Apple's completion blocks do not call back on the main thread (unless specifically stated like this in the documentation).


BUG: failed to fetch ollama model info

$ gguf-parser --ol-model="llama3.1" --ol-usage
failed to parse GGUF file: open http file: stat: do head request: stat: not support range download

fixed in v0.13.5


H6182 doesn't work with the latest version 10.6.0

What issue do you have? Please be as thorough and explicit as possible.
I can't control H6182 with version 10.6.0 of the plugin, I downgraded to version 10.5.0 of the plugin and it works fine. I am controlling over LAN only (disabled BLE and AWS)
Details of your setup.

Do you use (1) Homebridge UI-X (2) Homebridge CLI or (3) HOOBS?
Homebridge UI-X

Which version of Homebridge/HOOBS do you have?
Homebridge 1.8.0

Which platform do you run Homebridge/HOOBS on (e.g. Raspberry Pi/Windows/HOOBS Box)? Please also mention your version of Node.js/NPM if known.
Ubuntu Focal (20.04.6 LTS)
Node.js v20.12.2

Which version of this plugin (homebridge-govee) do you have? Has the issue started since upgrading from a previous version?
v10.6.0 and yes the issue started after the upgrade from 10.5.0, downgrading to 10.5.0 fixes the issue

Which Govee devices do you have that are causing issues? Please include product models if applicable.
 Code: 143, Signal: null
[4/24/2024, 11:21:59 AM] [Govee] Restarting Process...
[4/24/2024, 11:21:59 AM] [Govee] Launched child bridge with PID 1080777
[4/24/2024, 11:21:59 AM] [Govee] Loaded homebridge-govee v10.6.0 child bridge successfully
[4/24/2024, 11:21:59 AM] Loaded 1 cached accessories from cachedAccessories.0E6A676EB6E5.
[4/24/2024, 11:21:59 AM] [Govee] Initialising plugin v10.6.0 | System linux | Node v20.12.2 | HB v1.8.0 | HAPNodeJS v0.12.0...
[4/24/2024, 11:21:59 AM] [Govee] Plugin initialised. Setting up accessories....
[4/24/2024, 11:21:59 AM] Homebridge v1.8.0 (HAP v0.12.0) (Govee) is running on port 59918.
[4/24/2024, 11:22:01 AM] [Govee] [LAN] client enabled and found 1 device(s).
[4/24/2024, 11:22:02 AM] [Govee] [HTTP] client enabled and found 1 device(s).
[4/24/2024, 11:22:02 AM] [Govee] [AWS] client enabled.
[4/24/2024, 11:22:02 AM] [Govee] [BLE] disabling client as disabled in config.
[4/24/2024, 11:22:02 AM] [Govee] [TV Light] initialising with options {"adaptiveLightingShift":-1,"aws":"enabled","ble":"disabled","brightnessStep":1,"colourSafeMode":false,"lan":"enabled"}.
[4/24/2024, 11:22:02 AM] [Govee] [TV Light] initialised with id [76:3F:A4:C1:38:91:16:42] [H6182].
[4/24/2024, 11:22:03 AM] [Govee] ✓ Setup complete. This plugin has been made with ♥ by bwp91, please consider a ☆ on GitHub if you are finding it useful!
[4/24/2024, 11:22:06 AM] [Govee] [TV Light] current state [off].
[4/24/2024, 11:22:10 AM] [Govee] [TV Light] current state [on].
*** Logs from version 10.5.0 *** WORKING
[4/24/2024, 11:24:18 AM] [Govee] Restarting child bridge...
[4/24/2024, 11:24:18 AM] Got SIGTERM, shutting down child bridge process...
[4/24/2024, 11:24:23 AM] [Govee] Child bridge process ended
[4/24/2024, 11:24:23 AM] [Govee] Process Ended. Code: 143, Signal: null
[4/24/2024, 11:24:30 AM] [Govee] Restarting Process...
[4/24/2024, 11:24:30 AM] [Govee] Launched child bridge with PID 1081073
[4/24/2024, 11:24:30 AM] Registering platform 'homebridge-govee.Govee'
[4/24/2024, 11:24:30 AM] [Govee] Loaded homebridge-govee v10.5.0 child bridge successfully
[4/24/2024, 11:24:30 AM] Loaded 1 cached accessories from cachedAccessories.0E6A676EB6E5.
[4/24/2024, 11:24:30 AM] [Govee] Initialising plugin v10.5.0 | System linux | Node v20.12.2 | HB v1.8.0 | HAPNodeJS v0.12.0...
[4/24/2024, 11:24:30 AM] [Govee] Plugin initialised. Setting up accessories....
[4/24/2024, 11:24:30 AM] Homebridge v1.8.0 (HAP v0.12.0) (Govee) is running on port 59918.
[4/24/2024, 11:24:32 AM] [Govee] [LAN] client enabled and found 1 device(s).
[4/24/2024, 11:24:33 AM] [Govee] [HTTP] client enabled and found 1 device(s).
[4/24/2024, 11:24:33 AM] [Govee] [AWS] client enabled.
[4/24/2024, 11:24:33 AM] [Govee] [BLE] disabling client as disabled in config.
[4/24/2024, 11:24:33 AM] [Govee] [TV Light] initialising with options {"adaptiveLightingShift":-1,"aws":"enabled","ble":"disabled","brightnessStep":1,"colourSafeMode":false,"lan":"unsupported"}.
[4/24/2024, 11:24:33 AM] [Govee] [TV Light] initialised with id [76:3F:A4:C1:38:91:16:42] [H6182].
[4/24/2024, 11:24:34 AM] [Govee] ✓ Setup complete. This plugin has been made with ♥ by bwp91, please consider a ☆ on GitHub if you are finding it useful!
[4/24/2024, 11:25:01 AM] [Govee] [TV Light] current state [off].
[4/24/2024, 11:25:04 AM] [Govee] [TV Light] current state [on].

Looks like the issue is with LAN control, 10.5.0 reports the device as "lan":"unsupported" while 10.6.0 reports it as "lan":"enabled" but will not control over LAN. Disabling LAN in v10.6.0 forces it to control it over AWS and does work fine.


Syncing multiple apps results in YAML anchors

If syncing multiple applications at the same time YAML anchors are used instead of new emtpy objects.
See: https://github.com/baloise-incubator/okd4-apps-root-config/commit/618568edd320118dd90097f43d4099c3a92e8a42

Is is due to dict.fromkeys second argument value is always referencing the same object
https://github.com/baloise/gitopscli/blob/master/gitopscli/commands/sync_apps.py#L86
The code needs to be changed so that always a new object ({}) instead of the same one is used.
Relating stackoverflow question: https://stackoverflow.com/questions/15516413/dict-fromkeys-all-point-to-same-list

:tada: This issue has been resolved in version 3.1.2 :tada:
The release is available on:

v3.1.2
GitHub release

Your semantic-release bot :package::rocket:


Fix Merge System

Fix Merge System
Futures can't be polled without Task.
Spawn creates such Task and provide API to run inner Future
Without this Merge will panic! when it'll try to call Future::poll except if it is trivial custom Future

When I tried the system out it worked, so yeah, maybe my Future was trivial.

It would be really nice to have a test / an example using futures, so we can validate it actually works now.

Shared is a really easy way to generate a panic if this isn't working, as it depends on task::current()

Okay, I'm going to merge this, but I would like to add an example before releasing a new version. We could also port this fix back to 0.9 if you want to.
bors r+


Enable zero-configuration/decentralized networking like 'Cote'

It's a pretty big win if moleculer could also work like this:
http://cote.js.org/#multicast-address
The single point of failture like NATS would be gone.

Thanks, I will check it. I think it can be an other official transporter module.

NATs could be configured as cluster, so there won't be a single point of failure

The problem is, where will we find multicast enabled architecture? Docker for example doesn't support it.

I'm working on a TCP+UDP transporter. It uses UDP broadcast messages to detect other Moleculer nodes on the network. If it find them, it connects to the remote nodes via TCP.

maybe this library can be helpful for the service discovery https://github.com/maxogden/discovery-channel

Or perhaps https://github.com/mafintosh/discovery-swarm that is built on top of discovery-channel could be of use?

Yes, I'm checking discovery-swarm. I'm trying to create a transporter based on it.

Cote itself (as referenced in the original issue) uses https://github.com/wankdanker/node-discover btw.

I'm testing the low-level components (utp-native, peer-network) of discovery-swarm and it seems it is not too stable :confused:
I.e.: if the server shutdown & restart, the clients don't find it again and throw errors.
But I will still play with them.

what about nssocket ?

wow, moleculer is getting better each day, this is going to be an awesome feature.
For the communication, this library can be helpful: https://github.com/nickdesaulniers/node-nanomsg

It is in progress. It will be a "zero configuration" TCP transporter with Gossip internode protocol (similar Cassandra gossip protocol) to detect live/offline nodes & services without every node connects to every other node. So it will able to handle 100+ nodes as well.
Plus a UDP auto-discovery of course.
It will be released in the next 0.12 release in 1-2 weeks.
Branch: https://github.com/ice-services/moleculer/tree/gossip-transporter

Gossip is a peer-to-peer communication protocol in which nodes periodically exchange state information about themselves and about other nodes they know about. The gossip process runs every second and exchanges state messages with up to three other nodes in the cluster. The nodes exchange information about themselves and about the other nodes that they have gossiped about, so all nodes quickly learn about all other nodes in the cluster. A gossip message has a timestamp associated with it so that during a gossip exchange, older information is overwritten with the most current state for a particular node.

That's really awesome @icebob! Are you planning to make this transport as the default one?
I think it can be a good point to start with moleculer and scale your nodes, without having to worry about running a NATS server for example.

@tinchoz49: yes. It is my plan. It will be the default transporter. So no need any transporter config, just start the nodes and everything is working :)
I'm doing performance tests. It will be the fastest transporter with ~80k req/sec on my i7. Others 10k-20k req/sec

Does broadcast messaging work via gossip or does the sending node connect
to all nodes to send the broadcast?
On Sat, Feb 3, 2018 at 8:16 PM, Icebob<EMAIL_ADDRESS>wrote:

@tinchoz49 https://github.com/tinchoz49: yes. It is my plan. It will be
the default transporter. So no need any transporter config, just start the
nodes and everything is working :)
I'm doing performance tests. It will be the fastest transporter with ~80k
req/sec on my i7. Others 10k-20k req/sec
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ice-services/moleculer/issues/46#issuecomment-362850829,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC0_IezSIyxlVjHZWSkBEKT_R9LvpMVQks5tRL6UgaJpZM4N4jhT
.

No broadcast messages. INFO and HEARTBEAT are transferred with gossips. The REQUEST, RESPONSE, EVENT messages are sent to the target node directly.

But of course you can send broadcast events with broker.broadcast. But they are also sent directly.

Gossip protocol in action in TCP transporter: https://www.youtube.com/watch?v=9yJ_Lfw-JCs

!!!!!!!!!!!

??

I couldn’t contain my excitement. This is tremendous work @icebob

Ohh thanks :+1: . This transporter with gossip easily can handle 100+ nodes as well. I hope because I could test only on localhost

niceeee! At the end of my presentation, I said, "the future is decentralized and Moleculer too". Of course a was talking about this amazing feature!

;)

Awesome work!

@icebob you never cease to amaze me :heart:

Released in 0.12 as TCP transporter. More info

I did some tests in this feature... My two cents is:
udpBroadcast should be default at "<IP_ADDRESS>" because multicast enabled network is not that common, and "<IP_ADDRESS>" its a better choice for a default address to avoid manual config on every sample/example to run in some machines (same problem for multicast).
Besides that, my impression is the overall performance for this solution is not good at all (for now). The gossip protocol is interesting but have a unpredictable nature. Almost 28 seconds for 50 nodes is a eternity! I use 5-10s for connection timeout on my microservices for comparison. Compared to Cote, for exemple, i can interconnect 50 nodes in 1.2 seconds (or less).
This gossip implementation is very nice for a TCP communication protocol to enable serverless environments, but as TCP service discovery protocol needs a more direct approach i think.

With "<IP_ADDRESS>" broadcast address has problem with routers. By default, TCP transporter detects all network interfaces and send broadcasts on all interfaces and it works well. But if you need, you can use <IP_ADDRESS> as broadcast address in transporter config:
udpBroadcast: "<IP_ADDRESS>"

TCP transporter has the best performance at service communication thanks to the direct connection. Yes, the first discovery time can be big if you have many nodes. But my question, which is more likely?

you start & stop 50 nodes in every minutes and you have to wait 28 seconds.
you start 50 nodes once a month and your services communicate continuously & quickly.

If you belong to the first group, I suggest that you should use Cote, or any other transporter (NATS, Redis...etc)

Okay, I did not get a chance to look at the code more closely yesterday. (I just run the examples)
Now I believe I understand what is happening.
You actually now (in less than a second) all other nodes. But actually you first add that node as Offline Node, probably to avoid get a cached upd info right?
Usually on UDP discovery you first assume that server is actually running and only put the node on offline state after not receive another information after some time (like the heartbeat), and/or after a broken pipe after try a communication via TCP.
This and other extra care is what's causing slowness.
Can i try a PR to address this?
Also, i found the problem with the default udpBroadcast=true approach -> getBroadcastAddresses()  raises "ipaddr: the address does not have IPv4 CIDR format". That why i need one manually.
But before i make any change, can you explain a little about this (nice) design:

Why udpMulticast as default instead udpBroadcast?
The idea of the Gossip protocol is to overcome the unpredictable nature of the UDP and then make a TCP call to ensure the connection state and make earlier update on target node list?
Gossip is also to overcome multiple networks interfaces?
What do you think about a UDP heartbeat?


[BUG] Redesign the buttons and enhancing hover effects with smoother motions.

Description
The buttons look very basic and the hover effect is also pretty basic.
I would redesign all the buttons and add smother and better hovering effetcs.
Screenshots
https://github.com/user-attachments/assets/444717fe-8129-45e0-a92d-29c3f8fcdb1a
Any additional information?
No response
What browser are you seeing the problem on?
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

@YadavAkhileshh Please assign this issue to me along with level tag

@YadavAkhileshh Can you please tag level to it?


Add docker stats --no-stream show cpu usage

Signed-off-by: Lei Jitang<EMAIL_ADDRESS>The CPU % of docker stats --no-stream is always 0.0
even though the real time cpu usage is not zero of the container.
It's quite weird or misleading when user what use docker stats --no-stream
to get the cpu usage of the container. This PR add docker stats --no-stream to
show the cpu usage.
Before:
[l00284783@localhost ~]$ docker stats --no-stream abc
CONTAINER           CPU %               MEM USAGE/LIMIT     MEM %               NET I/O
abc                 0.00%               196.6 kB/4.143 GB   0.00%               734 B/258 B
After:
[l00284783@localhost ~]$ docker stats --no-stream abc
CONTAINER           CPU %               MEM USAGE/LIMIT     MEM %               NET I/O
abc                 5.07%               200.7 kB/4.143 GB   0.00%               664 B/258 B

ping @cpuguy83

@LK4D4  Update, move get first stats to daemon side. WDYT?

This could use a test

Confirmed this fixes the issue.

Code okay for me. Test would be nice.

I'll add test later

ping @cpuguy83  @LK4D4  test is add

code LGTM. I just wonder if the test can be written without streaming the stats, it might be more easy to understand, but definitely not blocking.

LGTM

ping @jfrazelle for cherry-pick since this is broken as is in 1.7

cherry-picked


Merge v3 into main to restart from latest

Since this is starting as a fork of the snyk action, we're starting fresh. Merging the latest release (v3) into main to start from latest

 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 3 committers have signed the CLA.:white_check_mark: LivingInSyn:x: oslatt:x: olattYou have signed the CLA already but the status is still pending? Let us recheck it.


Add a default value to imageUrl prop in Avatar component and make it optional

Is your feature request related to a problem? Please describe.
Let's face it, when prototyping, trying to find the perfect image takes one down the rabbit hole on a place like Unsplash.
It would be really helpful if the Avatar component comes with useful default images, particularly ones that look like a profile picture, this can help really speed up the prototyping and development process.
Describe the solution you'd like
A good starting point might be to put together some profile picture like images in an Unsplash collection and try to randomise them via the Unsplash API. About 3 images is fine really.
Describe alternatives you've considered
Just using a good 'ol background color is fine too

:tada: This issue has been resolved in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


ENH: update package version

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #27.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #27

@conda-forge-admin, please rerender


Bug: Invalid neo4j configuration supplied; returning default values

Description:
When I am trying to run the container image using the instructions provided in the documentation I get an error
"Invalid neo4j configuration supplied; returning default values".
https://support.bloodhoundenterprise.io/hc/en-us/articles/17468450058267-Install-BloodHound-Community-Edition-with-Docker-Compose
Are you intending to fix this bug? no
Component(s) Affected:

UI
API
Neo4j

Steps to Reproduce:
Follow the instructions by downloading the docker compose, and run the container. (https://support.bloodhoundenterprise.io/hc/en-us/articles/17468450058267-Install-BloodHound-Community-Edition-with-Docker-Compose)
Expected Behavior:
User login should be successful with no error in the docker logs
Actual Behavior:
User login failed and the bloodhound-1 service stops with error "bloodhound-1 exited with code 137"
When looking at the details of the logs I can see message ""message":"Invalid neo4j configuration supplied; returning default values"}"
I tried to change the image version of the bloodhound to 5.14 from latest but it is the same error. Updating the Neo4j configuration file also didn't help.
Screenshots/Code Snippets/Sample Files:
If applicable, add screenshots, relevant code snippets, or sample files that help illustrate the issue.
Environment Information:

BloodHound: latest, 5.14, tried in both version
Collector: N/A, couldn't login to bloodhound
OS: MAC OS 14.5
Browser (if UI related): chrome
Node.js (if UI related: [Node.js version]
Go (if API related): [Go version]
Database (if persistence related): Neo4j 4.4
Docker (if using Docker): Docker version 26.1.0-rd, build cca8e72
Additional Information:
Any additional context or information that might be helpful in understanding and diagnosing the issue.
Potential Solution (optional):
If you have any ideas about what might be causing the issue or how it could be fixed, you can share them here.
Related Issues:
If you've found related issues in the project's issue tracker, mention them here.
Contributor Checklist:

[x] I have searched the issue tracker to ensure this bug hasn't been reported before or is not already being addressed.
[x] I have provided clear steps to reproduce the issue.
[x] I have included relevant environment information details.
[ ] I have attached necessary supporting documents.
[ ] I have checked that any JSON files I am attempting to upload to BloodHound are valid.

this issue has been resolved after updating docker 26 -> 27

Not

The issue was resolved after I provisioned more memory for docker runtime colima
"colima start docker --memory 8"


feat: TestingOnlyStringSliceToTypeList -> StringSliceToTypeList

Avoid error checking all over the place when it can be done safely.

I don't love it on principle but you're right that it can be done relatively safely.

Yeah, weird that Terraform doesn't provide this. I've seen this helper function repeated again and again in a lot of provider codebases.


Support for a C API?

Hi! I maintain Commonmarker, the Ruby bindings to the Commonmark C lib. For a few years now I’ve wanted to move off of the C lib and onto a Rust-generated dylib.
Would you support a PR that exposed a C entry point (eg. crate-type = ["staticlib", "cdylib", "rlib"]) and a header, so that other FFIs can bind to this library?

That seems like a great idea to me!


Fix 2609 - Import packaging by default

Fix https://github.com/huggingface/huggingface_hub/issues/2609.
It's not the first time I see this so let's fix it. It's not really an error, just some IDEs that might yell because of how the import is made.
packaging is a package installed by default with huggingface_hub.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


One OrderedSet test fails

OS: Linux, Ubuntu 16.04
Commands ran:
felix@felix-X550LD ~/D/P/A/commandant> rm -rf .build/
felix@felix-X550LD ~/D/P/A/commandant> git checkout 0.13.0
Note: checking out '0.13.0'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at 7f29606... Release 0.13.0
felix@felix-X550LD ~/D/P/A/commandant> swift test

Failed test output:
Test Case 'XCTestCase.OrderedSet, should preserve the order of the given input' started at 2018-04-09 22:11:17.477
/home/felix/Documents/Programming/AwesomeSystemSwift/commandant/Tests/CommandantTests/OrderedSetSpec.swift:17: error: XCTestCase.OrderedSet, should preserve the order of the given input : failed - expected to not equal <acb>, got <acb>

Test Case 'XCTestCase.OrderedSet, should preserve the order of the given input' failed (0.0 seconds)

What's the output on that line under Mac OS? Maybe it's failing on both ends.

Thanks for filling the issue! This totally makes sense.

No problem ;)


Fix the problem that DateTime.difference reports an error on Flutter Web.

Fix the bug on Flutter Web:

Thank you for your contribution! I left a few comments.

Codecov Report

Merging #6 (549eb76) into main (a1e5860) will decrease coverage by 4.91%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main       #6      +/-   ##
==========================================
- Coverage   94.52%   89.61%   -4.92%
==========================================
  Files           1        1
  Lines          73       77       +4
==========================================
  Hits           69       69
- Misses          4        8       +4

Impacted Files
Coverage Δ

lib/src/const_date_time.dart
89.61% <0.00%> (-4.92%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I added some test code, unfortunately this solution only passes the test in the stable channel, maybe we should commit pr to @dart-lang/sdk?

Filed issue https://github.com/dart-lang/sdk/issues/50750.

I'm happy to land this patch even though it will deliberately break in Dart 2.19. Once Dart 2.19 is released, we can re-evaluate our available options.

Published in version 1.0.1. Thank you for your contribution!


Kubelet state visualization / debugging UI

Need to be able to browse pods and click-thru to info
Need to show more than just Docker's native info (in fact, maybe we should not show that at all and instead abstract it a bit further?)
Need to show our own understanding of the world

I'm interested in this project as part of GSoC 2015. A bit has changed since July 2014. cadvisor#74 got implemented and a simple web-UI is available. I couldn't find any up to date information about this topic.
My understanding of the state of this topic is, that the goal of this project would be to create a web-UI which displays statistics, be able to do what the CLI can do. The data for the statistics is gathered from a database (currently influxdb). Commands are given directly to the kublet server.

@marshauf We'd like the Kubelet web UI to be able to show information about the pods scheduled to the node, and about the node itself -- at least as much information about those objects as available in the apiserver's UI, but ideally going beyond that. CAdvisor's UI could be integrated into it. Links to logs could be made available. Errors from pods executed from local files could be surfaced. Docker status could be reported.
Perhaps @dchen1107 or @vmarmol have additional suggestions.
Scenarios we'd like to support:

Debugging the Kubelet using information not sent to apiserver and/or when apiserver is not available
Debugging applications when apiserver is not available, including when Kubelet is used "standalone" without a cluster

+1 to everything @bgrant0607 said. There is a lot of data in the Kubelet about the pods that we don't send to the apiserver. For users and for debugging it will be very useful to display that data. Displaying pods from non-apiserver sources and their mirror pod equivalents would be of particular value.
Some usage data from cAdvisor can be used and integrated, but some of the things mentioned above are no longer in scope. We'll probably show basic graphs of recent usage.

The UI should help introspect the kubelet itself - provide links to kubelet
logs, track kubelet's uptime, compute resource usage, etc.
On Mon, Mar 23, 2015 at 8:54 AM, Victor Marmol<EMAIL_ADDRESS>wrote:

+1 to everything @bgrant0607 https://github.com/bgrant0607 said. There
is a lot of data in the Kubelet about the pods that we don't send to the
apiserver. For users and for debugging it will be very useful to display
that data. Displaying pods from non-apiserver sources and their mirror pod
equivalents would be of particular value.
Some usage data from cAdvisor can be used and integrated, but some of the
things mentioned above are no longer in scope. We'll probably show basic
graphs of recent usage.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/284#issuecomment-85061577
.

Thank you for the answers.
Which technology could I use for the Kubelet web UI?
cAdvisor uses bootstrap and jquery. The kubernetes web UI uses angularjs.
Would the Kubelet serve the web UI?
What should the proposal for GSoC contain? I couldn't find a template, or a list of requirements.
A project goal, timeline, biography?

Use AngularJS please, we'll be consistent within Kubernetes (if anything we want to re-write cAdvisor in Angular). The UI will indeed be served by the Kubelet.
@bgrant0607 do we have a template?
Project goal is to create a Kubernetes node user interface allowing introspection into the node and it's workloads. Assisting in visualization and debugging of the current state and possible issues.
Requirements:

Show a nodes: state, capacity, links to logs, uptime, any salient errors, and usage overview
Show pods running on the node, their state, configurations, any errors or events associated with them, link to their logs, and usage overview.
What apiserver the Kubelet answers to and the health of that connection.

Timeline would be before the end of the summer (roughly 1.0 timeframe). Feel free to make some stop-gaps in between and let us know if you need help coming up with those :)

I submitted my proposal. Can you review it, please?

Thanks for your proposal! We'll definitely take a look.
Whatever UI templates/assets we have currently are in:
https://github.com/GoogleCloudPlatform/kubernetes/tree/master/www
FYI, a new (cluster-level) UI is being developed:
https://github.com/kubernetes-ui/kubernetes-ui/tree/master/www

Issue seems out of date. Close?

cc/ @adityakali


Can't start the test server on mac

The docker tries to bind to /var/ which throws an error on mac. This isn't mission critical but would be nice to fix going forward

The error is actually in the command pipenv run test-server-up.
The command calls out to scripts/server/start.sh, which uses readlink on line 14. The problem is that mac uses the bsd implementation of readline, which has substantive differences to the linux version, so the command doesn't run. I've attached a screenshot of the start up process here. It looks like readlink fails so it just uses the default paths in /var/.

One solution could be to use a directory in the home folder, but that's messy. We could require that /var is shared with docker from macOS, but that would leave remnants in the future.

But you don't need to use test-server-up anymore right? Local testing is now done through the dev server, so if pipenv run serve is working then you should be good to go.

Since it's now using 8080 and 5000, yes you can just run the main version of the apps. You can't quite run the same steps as in the yaml for testing, but that should be fine


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v5.15.12",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=7a77e22fde6349d276f862a521d190ad932e711d"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "7a77e22fde6349d276f862a521d190ad932e711d",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "mm, hwpoison: fix condition in free hugetlb page path\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.15.12 by commit 7a77e22fde6349d276f862a521d190ad932e711d. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2021-1002840


[SPARK-26454][CORE][Minor] lowering the log level to warn of exceptio…

…n thrown while creating new UDF from same JAR
What changes were proposed in this pull request?
While creating new UDF with JAR,though function is created successfully,  it throws IllegegalArgument Exception,  so instead of giving error log, log level is changed to warning as error log sounds operation failure so giving warning and moving ahead sounds better.

CC @HyukjinKwon please review

Can one of the admins verify this patch?

ok to test

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can we fix the PR title shorter and concise? For instance, [SPARK-26454][CORE][MINOR] Lowering the log level when the same jars are duplicatedly added

Test build #100681 has started for PR 23439 at commit c05b561.

Test build #100681 has finished for PR 23439 at commit c05b561.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100681/
Test PASSed.


docs: add Boba Network Audit Reports

:clipboard:
Overview
Adds Boba Network audit reports
Changes
/
Testing
/

Are there any disclaimers of non-audit we should remove?

aah yes, rem'd the disclaimers!


Add a progress bar or disable the "approve token" button after clicking

We need to add a progress bar or disable the button when the user clicks approve token. Because now the user can do it several times in a row and visually misleads the user
https://user-images.githubusercontent.com/45556620/186705740-6584dc13-c406-433b-b523-81b6a3a29e0e.mp4

checked


change to gplv3 license

Thanks for your contribution! Unfortunately, we are planning on taking the project in a different direction and can't accept this in its current form. Or probably any form.


Refresh new better result

When a new solution with better run time has been submitted, auto refresh by deleting old solution& replacing with new solution

#47


bindings in console

Is there a way to use the same vim bindings in the Jupyter console attached to a notebook?

Thanks for the suggestion! This extension only modifies the notebook currently. It should be possible though since the console is also a codemirror instance. If anyone wants to take a stab at this I would welcome PRs. Thinking out loud, I think you should be able to disable this. I doubt all vim users enjoy vim bindings in command prompts.


Examples in readme.md are missing await operators

Some of the examples need to include a second await operator.
For example
var result = await pdFtk.GetDataFieldsAsync(File.ReadAllBytesAsync("Form.pdf"));
should be
var fileBytes=  await File.ReadAllBytesAsync("Form.pdf")
var result = await pdFtk.GetDataFieldsAsync(fileBytes);`

Examples have been updated 👍  thanks for raising!


🔧 the security result class should not contain the result text

Motivation
At the moment, the security check result class contains a string to describe the error messages. This should not be part of the SDK.
Description
Remove the text description of the security check results.
Progress

[x] Done Task

@aidenkeating @danielpassos @StephenCoady  mind taking a look?

@wei-lee  👀

@aidenkeating mind taking a look again?


fix: force active correctly scope in vue2

This resolves the error scope issue caused on Vue 2 due to the callHook.
closed #1800

See https://github.com/vuejs/pinia/issues/1800#issuecomment-1775774032


Make Features and Special Purposes "notice only" in TCF

Is your feature request related to a specific problem?
Related to https://github.com/ethyca/fides/issues/4137, these TCF entities should effectively be notice-only.
Describe the solution you'd like
We should change the backend so that if the entity is "Feature" or "Special Purpose", then the consent mechanism should be "notice only".
We also need to change the frontend to respect this. If the mechanism is notice only, then most of the FE should fall into place, but there will probably be a few tweaks needed (and tests/fixtures updated)
Describe alternatives you've considered, if any
A description of any alternative solutions or features you've considered.
Additional context
As of https://github.com/ethyca/fides/pull/4139, we do not send any preferences for Features or Special Purposes

Just noting here, Allison already noticed this, https://github.com/ethyca/fides/pull/4201 updates the backend to make the default preferences "acknowledge" for feature and special purpose.  That PR updates the default preferences for many of the TCF sections so I just did these altogether -


Does not support ng-model

I'm trying out your code, currently the animation functionality disappears when you add an ng-model to the input tag.

Quickfix, change this:

templateAttributes.push(attr + '="' + $attrs[attr] + '"');

to this:

if (attr !== 'floatingLabel') {
  templateAttributes.push($attrs.$attr[attr] + '="' + $attrs[attr] + '"');
}

Thanks for the suggestion @jtorbicki! This worked perfectly.
I updated the code and the example (to have an ng-model to prove this fix).
If I find some time I will maybe add some automated tests and publish this to bower and npm
Thanks @jtorbicki and @hyrumwhite


Mark tricks as completed

The MVP of this is fairly simple, a query that let you get your own checklist query { user { checklist } } and a mutation that let you toggle completion of a trick mutation { setTrickCompletion (trickId: ID!, completed: Boolean!): Trick }

fixed by d17c8aa309f14aa226ee8fea66c7c204e18e76dc


Bug: vai is not indenting the last line

Hi, Thanks for this awesome plugin
I am facing one wierd issue, regarding vai and vaI
Below is an excerpt of Vagrant file.
    end

    # starting provisionig only after all nodes are up
    if i == node_count
        config.vm.provision "ansible" do |ansible|
            ansible.playbook = "prereq.yaml"
            ansible.limit = "all"
            ansible.verbose = "v"
            ansible.groups = {
                "dp-yarn-master" => ["master"],
                "dp-yarn-slave" => ["node-[1:#{node_count}]"],
                "[dp-yarn:children]" => ["dp-yarn-master", "dp-yarn-slave"]
                }
            }
        end
    end

if I do vai from line number 15, isn't it supposed to select line number 16 also?
But vaI is working as expected.

vai isn't supposed to select the line below the indent. Only vaI is: https://github.com/michaeljsmith/vim-indent-object#usage


Pull request for issue #1396 supporting srcset property in image tags

this adds an extension for image_tag which enables used of srcset property in image tags, and having the urls for the paths built using asset_url just like it does for your main src. Leaves absolute
urls alone, (having // in the path)
Erb:
<%= image_tage 'pic_1980.jpg', srcset: 'pic_640.jpg 2x, pic_1024.jpg 3x' %>

Also work for markdown rendering, as it is processed through the image_tag helper.
Markdown:*
![image](blank.gif){:srcset="image_2x.jpg 2x"}

This is great. I actually don't see any reason not to put this directly in image_tag and skip the extension portion. What do you think?

Ok sure, so I should just extend image_tag in the default helpers instead of doing the whole
class Middleman::Extensions::ImageSrcsetPaths < ::Middleman::Extension

bit? If so I fix that up.

Exactly. Thanks

ok, did that and squashed.

Lovely, thanks!


Activation email not received

Hi,
I am trying to register to the CoralNet website to collaborate on an ongoing project but I don't receive the activation email.
I tried three consecutive days, with two different emails but it does not work.
Could you help me with this issue?
Best,
Joris

Hi Joris,
I looked through the recent registration attempts; it looks like the .edu email had problems with activation, but the .fr email might have succeeded. Are you able to log in now?


Add Contact Section

Summary
Add contact section and legend for terminology in release notes

/gcbrun

Temporary Url for review: https://release-notes-dev--pr-19-j2vrogzf.web.app


firebase functions:config:set fails with uppercase environment variables

Operating System version: Macos 10.14.4
Firebase SDK version: 6.9.1
Node JS Version: 8.16.0
Firebase Product: functions

Description:
UPPER CASE characters are required for some environment variables from some GOOGLE API's.    The current version of the firebase admin for node prevents the use of uppercase characters.
Setting this configuration:
firebase functions:config:set GOOGLE_APPLICATION_CREDENTIALS="/Users/mickeymouse/functions/promotions-google-services.json"
Results in this error:
Error: Invalid config name GOOGLE_APPLICATION_CREDENTIALS, cannot use upper case

You're in the wrong repo. Function SDK and the associated tooling are in https://github.com/firebase/firebase-functions and https://github.com/firebase/firebase-tools
AFAIK the functions:config:set  is not meant to be used for regular environment variables. It is used to configure parameters that are then accessed from the Functions code. See https://firebase.google.com/docs/functions/config-env

fwiw
Firebase Environment variables docs

Keep in mind that only lowercase characters are accepted in keys; uppercase characters are not allowed.

A bit odd that uppercase isn't allowed, I'm sure there is a good reason for it.


Strawberry Shake - empty result with error, results in error with "Value cannot be null"

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
In Banana Cake Pop I get the following response:

And this looks fine to me.
But in Strawberry Shake I get a client error with “Value cannot be null” and in the Extensions is the GraphQL error:

Steps to reproduce

Consume a empty GraphQL response with raised GraphQLException.

Relevant log output
No response
Additional Context?
No response
Product
Strawberry Shake
Version
12.3.2

This is resolved in 12.5


Model form validation in collections: instance attribute not set

Collections create a copy of the underlying form for validation via HolderMixin.replicate. But replicate does not assign a model instance to model forms, leading to issues with validation.
When creating a new object, this isn't an issue. But when editing an existing object, the form's instance attribute must be set to that object for uniqueness validation to work properly.
To clarify, this only affects validation, more specifically uniqueness validation, because the unique checks take the instance's state and primary key into account: model._perform_unique_checks. Note that including the primary key field in the form is not enough: the form's model instance was instantiated as new object and it will remain so - instance._state.adding remains True until the object was saved.
To set the correct instance, I suggest trying to fetch the existing object from the database when the form is being replicated. I can whip up a PR if you want.

Please retry with this branch: https://github.com/jrief/django-formset/tree/releases/1.0.x
or https://pypi.org/project/django-formset/1.0.dev0/
There the problem with validating unique indices is solved.
Unfortunately it will take a few additional week until I can release an official version, because I have to rewrite parts of the documentation.

Yes, that works for fields with unique=True.
I think I noticed another related issue: collections for related models do not know their parent instance during validation.
Inline formsets are instantiated with the parent instance of the relation, and that instance is passed on to the model forms as data for the InlineForeignKeyField. This way, the parent id is available for validation - most notably unique_together validation.
This does not happen for collections for related models.
You can work around this when editing existing relations of an existing parent: since model_to_dict includes the parent in the initial data of the relation's model form, you just need to include the formfield for the parent in the model form and it validates properly.
However, this doesn't work when you are adding a 'new parent', or a new relation to an existing parent. You need to validate the model form of the parent first, and then you can grab the form's instance and pass it on to the related collections.
Currently when adding a new related object that would violate unique constraints, the validation for  unique_together is skipped when the form of the new object is first validated, because the parent field is missing data. Later, when trying to save the duplicate object, the parent instance is available, and a second validation of the form in construct_instance fails as expected due to the unique constraint. The duplicate object is never saved, but the user doesn't know about this, because they are never presented an error.

Please handle separate issues as such, and do not use an existing one for adding new features, otherwise we get confused.
I'm going to close this. Please open a new issue with the new request.


AddCaseCommand: Save to file (xml) fix

Pull Request Test Coverage Report for Build 107

18 of 97 (18.56%) changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-3.0%) to 78.087%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/investigapptor/model/Investigapptor.java
7
9
77.78%

src/main/java/seedu/investigapptor/storage/XmlSerializableInvestigapptor.java
4
6
66.67%

src/main/java/seedu/investigapptor/model/crimecase/UniqueCrimeCaseList.java
7
9
77.78%

src/main/java/seedu/investigapptor/model/crimecase/Status.java
0
6
0.0%

src/main/java/seedu/investigapptor/storage/XmlAdaptedCrimeCase.java
0
67
0.0%

Totals

Change from base Build 103:
-3.0%

Covered Lines:
1600

Relevant Lines:
2049

💛  - Coveralls


Client-side load-balancing

We have a requirement for doing client-side load-balancing with logspout to avoid the load on the load-balancers. Currently, logspout is routing the logs to logstash servers which are front-ended by a load-balancer. The expectation is to be able to specify a list of server-ips (and port) in the ROUTE_URIS and logspout should randomly (or based on some algorithm) choose one of the server-ip

hi @pradeepcloudstack
This is not currently supported in logspout. I would suggest that you use DNS to accomplish something like this (round-robin DNS record) which perhaps can help spread the load better. I would suggest you also re-evaluate the use of a loadbalancer here, since that would be the best possible solution. Using a load balancer would also detect bad/down logstash servers and ensure that your logging infrastructure is more resilient to failures overall.
Closing this issue for now, as it appears that this enhancement request did not get any attention and is easily fixed with the suggestions above.


Fix Facebook icon

FB icon in Safari:

in Chrome:

@gnibeda  Can we try to exchange the image to another one ?
Doesn't look good on MacOs


bodiesMatch bug fix

Adds an e2e test to catch a bug missed out when improving the bodiesMatch function in #35.

Published as vanilli 4.6.2.


Need to unmount DMG if RecipeRobot mounted it but gives up

Dragging the [OneDrive for Business for Mac Preview DMG](https://download.microsoft.com/download/D/9/8/D98389DB-5946-412C-A0A6-44808C19F99C/OneDriveForBusinessMacPreview.dmg, https://www.microsoft.com/en-us/download/confirmation.aspx?id=45519) to Recipe Robot fails:

                      -----------------------------------
                     |   Welcome to Recipe Robot v1.0.   |
                      -----------------------------------
                                \   _[]_
                                 \  [oo]
                                   d-||-b
                                     ||
                                   _/  \_

Processing /Users/username/Downloads/OneDriveForBusinessMacPreview.dmg ...
Input path looks like a disk image.
    Successfully mounted disk image
Checking whether package is signed...
    Package is signed
Getting developer from pkg signature...
    Developer is: Microsoft Corporation
Getting package signature authority names...
    3 authority names recorded
Expanding package to look for clues...
    Package expanded to: /Users/username/Library/Caches/Recipe Robot/2015-12-04_13-24-42_705123/expanded
Getting information from PackageInfo file...
    Bundle identifier: com.microsoft.pkg.OneDriveBusinessMac
    Install location: /Applications
    Install filename: Applications
Extracting the package payload to see if we can find an app...
    Did not find an app in the package payload
[ERROR] I wasn't able to determine the name of this app, so I can't make any recipes.
[ERROR]
                                    _[]_
                                    [xx]
                                   q-||-p
                                     ||
                                   _/  \_

The problem is that Recipe Robot cleans up its cache but doesn't unmount the DMG (which is not visible in Finder). So, the user can't open the DMG until they umount /Volumes/OneDriveBusinessMacInstaller/.

Under normal circumstances, if the dmg inspection completes successfully, the disk image should be unmounted. (As instructed here.) However, I think the problem you've discovered relates to the fact that Recipe Robot is unable to find an app in the downloaded dmg file. Right now, that's a fatal error, but it doesn't throw the standard LogLevel.ERROR because there's hope that someday we'll detect things other than apps. (A pkg file that installs a QuickLook plugin for example.)
I'll keep this issue open to remind me that when we improve the pkg inspection engine, we should make sure to tie up the loose ends when errors occur.
Thanks!


buienradar==0.7, fix winddirection/azimuth, logging

Description:

Update buienradar external library to version 0.7

better calculation of precipitation forecast
values are now numeric (not '-', but 0 or 0.0)
reversed winddirection/windazimuth (#8173)

weather component

will not load/install required external library
will now correctly initialize

sensor component

added timeframe info into attribution of precipitation forecast
changed unit of measurement of total forecasted precipitation into mm (it represents mm/timeframe, not mm/h)
added debug logging

Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#2911
Example entry for configuration.yaml (if applicable):
No changes to configuration
Checklist:
If user exposed functionality or configuration variables are added/changed:

[X] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[X] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[X] New dependencies have been added to the REQUIREMENTS variable (example).
[X] New dependencies are only imported inside functions that use them (example).
[X] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[X] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Hello @mjj4791,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

e1a3856ace3faa8040e71334e1a2fed066efc633 - This commit has something that looks like an email address (hassdev@nas2). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

Thanks!


chore: Verifying tests for at_lookup

- What I did
Pointed to a local branch of at_lookup for verifying tests
- How I did it
- How to verify it
- Description for the changelog

This PR only verifies the tests after pointing to the at_lookup local branch, Hence closing the PR.


addinvoice --private doesn't add hop hints if no amount is provided

Background
lncli addinvoice --private returns an invoice without hop hints; if an amount is provided the --private option works as expected.
Your environment

lnd version 0.15.0-beta commit=v0.15.0-beta

Expected behaviour
Either provide hop hints. Or if that's not technologically possible, return an error.

capturing root-cause analysis comments
https://github.com/lightningnetwork/lnd/blob/f7b74426808c0305eb594bb0addbf4c1c035d9cd/lnrpc/invoicesrpc/addinvoice.go#L695
Initial assessment:

lnd would use the amt to decide what to place in the hop hint (most MPP), but maybe we had a regression there
[...]
prob a 0 >= 0 case, which causes things to break early
[...]
we check to see if we have enough: the amount is zero and we have zero added in hop hints so it's like ok

e.g.
Due to the amount being 0 sufficientHints prematurely returns true before lnd has assembled enough routing hints


module declares its path as: github.com/Shopify/sarama but was required as: github.com/IBM/sarama

Currently it's not possible to import the migrated module.
go mod tidy
(...)
go: git.my.company/myproject/internal/cmd imports
        github.com/IBM/sarama<EMAIL_ADDRESS>parsing go.mod:
        module declares its path as: github.com/Shopify/sarama
                but was required as: github.com/IBM/sarama

I guess line 1 in go.mod needs to be adjusted, and a new version needs to be published as well.
module github.com/Shopify/sarama

Yes the migration is still work in progress at the moment. You should still be able to pull the module from the gocache at the old import path?

Yes the migration is still work in progress at the moment. You should still be able to pull the module from the gocache at the old import path?

Can module names be lowercased?  github.com/ibm/sarama   ?

Will https://pkg.go.dev/github.com/IBM/sarama also be updated too? As it currently references an old version?

Yes that’s just because pkg.go.dev auto imported all version tags until a go.mod was added I think

Thanks @dnwe and a big thanks to you and IBM for continuing to support this project. I no doubt will start to contribute once things have been finalized your end.

Closing as resolved by v1.40.0 release