Utilize official Hipchat API

Would be great to switch for using XMPP to Hipchat's v2 API, especially since that seems to be the sactioned path forward for other integrations (and allows access tokens instead of user&pass).
https://www.hipchat.com/docs/apiv2

Note that you can login with a valid access token over XMPP too by using the X-OAUTH or X-HIPCHAT-OAUTH2 SASL mechanisms. These could be supported as well to fix these problems if they're not already.

There is a bit of work in #270 which uses the API for specific functions, but not as a complete replacement...

The v2 API is not a 100% replacement for the capabilities of XMPP, though you are right -- it is the way forward for HipChat integrations and "bots".  Quite a bit of support exists for creating v2 integrations, but porting hubot-hipchat to the v2 API is probably more work than just implementing new-addons with one of the node.js or other SDKs HipChat already supports for that purpose.


Document upgrade guide to 1.0.0

This PR:

documents the upgrade guide to 1.0.0 https://github.com/giantswarm/roadmap/issues/2879

Checklist

[x] Update changelog in CHANGELOG.md.
[ ] Make sure values.yaml and values.schema.json are valid. Not needed

@TheoBrigitte (as you requested changes) and @marieroque (as you tested the bundle breaking change) could you take a look ? I will not merge until you both approved :)

This will merge once @TheoBrigitte approves when he can

Thank you for your reviews :)


AGPL or MIT?

In License file, there's MIT license. On the page you have the AGPL badge. So?...

Hello @CocoaPriest,
The badges in the Readme section of the repository homepage are automatically generated by Shields.io, so I think it's up to them to fix the licence recognition logic. GitHub on the other hand correctly recognise the MIT licence.
GLTableCollectionView follows MIT Licence starting from version 1.61-MIT and GPLv3 for versions before it


Etherwallet page loading and loading....

Hi,
If someone can help me with this problem..
When I unlock my etherwallet I can't see how many ETC I have. The account balance is stuck on 'loading'..
Also the page freezes and keeps on loading..
I use safari web browser with: https://classicetherwallet.com

server had some memory issues. restarted it, pls try again

still not working :(

Op 19 jun. 2017, om 20:43 heeft Elaine Ou<EMAIL_ADDRESS>het volgende geschreven:
server had some memory issues. restarted it, pls try again
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/ethereumproject/etherwallet/issues/45#issuecomment-309529920, or mute the thread https://github.com/notifications/unsubscribe-auth/AcLrZVelOxs6lzjzLBzJYxQEtMe9Yd8wks5sFsFcgaJpZM4N-mbw.


pdr: Missing nvme slot entries in the json

According to the rainier workbook, the rainier1s-4u model has 16 nvme slots 8 in each disk_backplane0 and disk_backplane1.
The ibm,rainier1s-4u json had missing nvme slots entries and were not modelled by PLDM. This commit adds the same.
Fixes: 561341
Change-Id: I0559f1761a6ec223fa62aa14c283885d3aa37e49
Signed-off-by: Pavithra Barithaya<EMAIL_ADDRESS>

Since we are updating the slots, we need to update the power states also (State Set Id 257 : xyz.openbmc_project.State.Decorator.PowerState) for newly added slots

Done


Disabled keys not passing isDisable prop to item

🐛 Bug Report
I've been following the example on the useMenu documentation, but I cannot get the disabledKeys prop to work properly. The keyboard navigation seems to respect the disabledKeys, but the Item itself receives undefined when accessing item.isDisabled.
🤔 Expected Behavior
The menu item should have the correct isDisabled state - either true or false, but not undefined.
😯 Current Behavior
When accessing item.isDisabled, as written at the documentation, it returns undefined.
💻 Code Sample
Here's a Codesandbox to demonstrate the issue:
https://codesandbox.io/s/sweet-bash-ywhx4?file=/src/Menu.js:1041-1067

@lucasmotta I believe this is bug in the docs, based off a time when the Node objects in a collection contained things like isDisabled, isSelected, etc. The docs should probably follow the approach we use for our own MenuItem where we look at the state.disabledKeys to see if a item should be disabled.
Thanks for catching this!

@LFDanLu, seems to be a bug in the docs.
A fix should include:

docs update
or adding support for isDisabled in useMenuItem ?

@LFDanLu, seems to be a bug in the docs.
A fix should include:

docs update
or adding support for isDisabled in useMenuItem ?

sure, i can pick it up.

sure, i can pick it up.

@razvanip Sounds good!

@razvanip Sounds good!


can you pull data similar to TR functions?

does pyrfa allow for similar data retrieval as one would do with the TR functions through excel?

In short, yes it can. You would need know the data feed server IP address
and username. PyRFA also allows retrieving data in other ways that Excel
cannot do such as level-2 data (order book) and timeseries.
--
Wiwat Tharateeraparb
www.devcartel.com |<EMAIL_ADDRESS>+668 9500 9810
On 24 Jun 2015, at 12:40, mstellaris<EMAIL_ADDRESS>wrote:
does pyrfa allow for similar data retrieval as one would do with the TR
functions through excel?
—
Reply to this email directly or view it on GitHub
https://github.com/devcartel/api/issues/10.

hi wiwat,
thanks for the answer. to be more specific, I have provided some examples of TR function used in Excel:
TR("2628.HK","TR.EventStartDate.date","EventType=RES"))
TR.DivExDate(SDate=-3M EDate=1AY DateType=PD)
TR.DivUnadjustedNet(SDate=-3M EDate=1AY DateType=PD)
Could you please comment if it is possible or not to request such dividend data through your library?
Thanks!

Hi, I am not sure if it can get that dividend data really. It looks like it's  timeseries data which I think PyRFA will not be able to retrieve those.


Bugfix - Card style background

Fixes issue https://github.com/react-navigation/react-navigation/issues/5253

@BrendonSled thanks for the PR.
This looks good to me. I can't help but wonder if besides backgroundColorsomething else could/should be applied there too. Unfortunately I am not familiar enough with the project internals.

This could be a little problematic when providing a StyleSheet object to cardStyle as these styles can't be destructured.

@levibuzolic they can be destructed

indeed as of https://github.com/facebook/react-native/commit/a8e3c7f5780516eb0297830632862484ad032c10#diff-cc92f2a0a19634794458cbd431b473e6 StyleSheet objects are just frozen objects. if we land this then react-navigation-stack will depend on react-native >= 0.56

that said, it's still possible for this to be problematic because you could for example pass in [{backgroundColor: 'red'}, {someOtherThing: 'abc'}] so we need to use StyleSheet.flatten here

Oh that's great, thanks for pointing that out, completely missed that change. :clap:


Add files via upload

Updated API link from v1 to v2

Coverage remained the same at 99.138% when pulling bf997166c2e3a4ac4ebbf7fd8f1c06771f914720 on joshuadgreen:joshuadgreen-patch-1 into 1c8b22f0b4c40ac3b4f652404b654c2f1accbf36 on ZoranPandovski:master.

hi @joshuadgreen , did you just change URL version ?


Annotate logs with current connection name

It works by storing the connection name in Thread.current[:multidb_connection_name] and alias-chaining the debug method of ActiveRecord::LogSubscriber to annotate the log message (like the Octopus gem).

Could you make it so that there's just one hash stored in Thread.current? Eg., Thread.current[:multidb_connection] = {connection: conn, name: name}.

May be it is possible to use tagged logs, the good part is that the implementation is much simplier, the bad part is that it will add the current database to any log (not only active record).
Implementation would be:
begin
  Rails.logger.tagged(name) { result = yield }
ensure

See also #22 for a slight tweak on this which plays nicer with lograge.


Combine test suites and create a single xml file

Hi, is it possible to get a single xml file that combines the results of many test suites instead of many xml files?

This is right now not possible, every test creates its own xml file. Why do you need to have one file? It works perfectly in Jenkins. Also join our   for these kind of questions. This is not necessary an issue.

Hi Christian, thanks for the answer, I thought you could label this as a question since having a single file is fundamental for junit xray import (https://confluence.xpand-addons.com/display/XRAY/Import+Execution+Results+-+REST#ImportExecutionResults-REST-JUnitXMLresults).
Anyway I'll take a look at the xml creation, maybe I can figure out something on my own, if I find a solution I'll update this issue so that other people can take advantage of it.
Thank you.

@mcelotti feel free to propose a PR that bundles all XML files into one so it works with XRAY, we might can have an option for it so people can choose whether or not to bundle it.

Ok, I'll take a look to make this configurable so we can have both options


[BUG][NOWEB] - Issues encountered in NOWEB 12.6

Failed functionality:

[x] Buttons don't work but trigger WhatsApp Web notifications (no phone notifications)
[x] Webhooks only trigger with engine.event, not with individual events (except session/message updates)
[x] No success/failure response when adding participants during group creation
[x] Media loading times out consistently

API Endpoint issues:

[x] /contacts/about returns 422 "not implemented"
[x] /contacts/profile-picture returns encrypted file link only
[x] /contacts/block and /contacts/unblock return 422 "not implemented"

Hi! What engine do you use?

Is there any docker logs from the start of container and when you start a session may be?

Buttons don't work but trigger WhatsApp Web notifications (no phone notifications)

Buttons are fragile creatures, no guarantee it'll always work :( Looks like Meta got patched our solution recently...
Consider having a fallback logic to regular text in case of such errors.
We'll see what we can do with buttons 🤞

Webhooks only trigger with engine.event, not with individual events (except session/message updates)

Which event exactly do you want to listen?
engine.event is internal engine events, it can be too wordy sometimes, but most of them mapped to WAHA events https://waha.devlike.pro/docs/how-to/webhooks/#events
Just let us know which event doesn't works and we'll double check!
Tested it with labels.* - works fine

No success/failure response when adding participants during group creation

https://github.com/devlikeapro/waha/issues/709

Media loading times out consistently

Could you explain it, when does it happen? What you sending a message or receiving?

/contacts/about returns 422 "not implemented"

Yeah, that's true, NOWEB doesn't support it now :( Consider creating a feature request!

/contacts/profile-picture returns encrypted file link only

that's weird, just tested it - works fine. It could return ".enc" file, but if you open as regular jpeg - it does show the picture

/contacts/block and /contacts/unblock return 422 "not implemented"

Mistake in the doc, it's not supported, yeah :( Feel free to open feature request!

If you could open issue for each real bug / feature request - that'd be awesome! :pray: Hard to keep track of all changes in one big issue :)


Fix #1958 (When PK do not declare at the table head, data misalignmen…

…t occurs after synchronizing data which caused by CDCSOURCE reorders table columns by PK)
Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

dinky/dlink-metadata/dlink-metadata-doris/src/main/java/com/dlink/metadata/driver/DorisDriver.java:[126,17] cannot find symbol Column

Sorry, I've updated this PR just now.


Users added to "Teams" are automatically added to watchers on all repos in organization

Description
When you add new user to Team, this user is automatically subscribed to watch on all repos in the organization.
The planed behavior is to add user WITHOUT watching all repos.
Also, it would be great, if there was a function to manage subscriptions by Admin for managed users, so you can in batch operation subscribe or unsubsribe users to watch repos.
Gitea Version
1.19.0
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
docker
Database
PostgreSQL

Do we know how this works on other Forges like GitHub or GitLab? I'm kind of torn on this one as sometimes these auto-watches are useful, but I do recall complaints about them too.

I'm pretty sure that's not how it works on github. However, for some it may be a feature and for others a bug, so I would parameterize it in the organization's settings.

Yes, per-org configuration would be ideal, and a default in app.ini that applies to new orgs.

I agree with author this issue.
Can you add some option in app.ini to fix problem?

There is a app.ini setting for this but it seems to have 2 purposes:

tell if new repo should be automatically watched by members
tell if new member should automatically watch all repos

https://github.com/go-gitea/gitea/blob/92e07f270aa925d518b13686f78befb63da0a747/custom/conf/app.example.ini#L874-L877
https://github.com/go-gitea/gitea/blob/92e07f270aa925d518b13686f78befb63da0a747/models/org_team.go#L504-L518

per-user would be nice as well. Not all users need or want notifications for everything in the organization

I fully agree with @ddi-acassidy . It would be nice with something similar to the notifications tab in Github: https://github.com/settings/notifications

So I think this issue proposal a per-user level notification setting UI but not a bug.

I forgot to mention that I created https://github.com/go-gitea/gitea/issues/30537, which is a proposal for per-user notification settings.
I don't think this is a duplicate, due @sgabenov's proposal for

[...] a function to manage subscriptions by Admin for managed users, so you can in batch operation subscribe or unsubsribe users to watch repos.

unless we think that https://github.com/go-gitea/gitea/issues/30537 solves that

It goes funny when you set users as hidden:

Create test repository:

Gitea 1.22.0, 1.21.11

Watchers are not team members or organization members.


More informative error types in stream proofs

Codecov Report

Merging #1791 (fb49c79) into main (dc150bd) will increase coverage by 0.03%.
 Consider uploading reports for the commit 15c9b24 to get more accurate results

@@            Coverage Diff             @@
##             main    #1791      +/-   ##
==========================================
+ Coverage   64.52%   64.55%   +0.03%
==========================================
  Files         115      115
  Lines       14059    14070      +11
==========================================
+ Hits         9071     9083      +12
+ Misses       4988     4987       -1

Impacted Files
Coverage Δ

src/irmin-test/store.ml
94.93% <0.00%> (-0.12%)
:arrow_down:

src/irmin/tree.ml
81.23% <80.00%> (+<0.01%)
:arrow_up:

src/irmin/proof.ml
88.79% <100.00%> (+0.56%)
:arrow_up:

src/irmin/store.ml
64.67% <0.00%> (+0.14%)
:arrow_up:

src/irmin-unix/fs.ml
68.38% <0.00%> (+1.29%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


unable to use mysql2 0.4.X

If I upgrade mysql2 to 0.4.X, I get an error:
/home/bewe/.rvm/gems/ruby-2.2.3/gems/activerecord-4.2.0/lib/active_record/connection_adapters/connection_specification.rb:177:in `rescue in spec': Specified 'mysql2' for database adapter, but the gem is not loaded. Add `gem 'mysql2'` to your Gemfile (and ensure its version is at the minimum required by ActiveRecord). (Gem::LoadError)
	from /home/bewe/.rvm/gems/ruby-2.2.3/gems/activerecord-4.2.0/lib/active_record/connection_adapters/connection_specification.rb:174:in `spec'
	from /home/bewe/.rvm/gems/ruby-2.2.3/gems/activerecord-4.2.0/lib/active_record/connection_handling.rb:50:in `establish_connection'
	from /home/bewe/.rvm/gems/ruby-2.2.3/gems/sinatra-activerecord-2.0.9/lib/sinatra/activerecord.rb:45:in `database='
	from /home/bewe/.rvm/gems/ruby-2.2.3/gems/sinatra-1.4.7/lib/sinatra/base.rb:1217:in `set'
	from /home/bewe/.rvm/gems/ruby-2.2.3/gems/sinatra-activerecord-2.0.9/lib/sinatra/activerecord.rb:39:in `database_file='
	from /home/bewe/.rvm/gems/ruby-2.2.3/gems/sinatra-1.4.7/lib/sinatra/base.rb:1217:in `set'
	from /home/bewe/.rvm/gems/ruby-2.2.3/gems/sinatra-activerecord-2.0.9/lib/sinatra/activerecord.rb:22:in `registered'
	from /home/bewe/.rvm/gems/ruby-2.2.3/gems/sinatra-1.4.7/lib/sinatra/base.rb:1400:in `block in register'
	from /home/bewe/.rvm/gems/ruby-2.2.3/gems/sinatra-1.4.7/lib/sinatra/base.rb:1398:in `each'
	from /home/bewe/.rvm/gems/ruby-2.2.3/gems/sinatra-1.4.7/lib/sinatra/base.rb:1398:in `register'
	from /home/bewe/.rvm/gems/ruby-2.2.3/gems/sinatra-1.4.7/lib/sinatra/base.rb:1969:in `register'
	from /home/bewe/.rvm/gems/ruby-2.2.3/gems/sinatra-1.4.7/lib/sinatra/base.rb:2030:in `register'
	from /home/bewe/.rvm/gems/ruby-2.2.3/gems/sinatra-activerecord-2.0.9/lib/sinatra/activerecord.rb:58:in `<module:Sinatra>'
	from /home/bewe/.rvm/gems/ruby-2.2.3/gems/sinatra-activerecord-2.0.9/lib/sinatra/activerecord.rb:10:in `<top (required)>'
	from /home/bewe/.rvm/gems/ruby-2.2.3@global/gems/bundler-1.10.6/lib/bundler/runtime.rb:85:in `require'
	from /home/bewe/.rvm/gems/ruby-2.2.3@global/gems/bundler-1.10.6/lib/bundler/runtime.rb:85:in `rescue in block in require'
	from /home/bewe/.rvm/gems/ruby-2.2.3@global/gems/bundler-1.10.6/lib/bundler/runtime.rb:68:in `block in require'
	from /home/bewe/.rvm/gems/ruby-2.2.3@global/gems/bundler-1.10.6/lib/bundler/runtime.rb:61:in `each'
	from /home/bewe/.rvm/gems/ruby-2.2.3@global/gems/bundler-1.10.6/lib/bundler/runtime.rb:61:in `require'
	from /home/bewe/.rvm/gems/ruby-2.2.3@global/gems/bundler-1.10.6/lib/bundler.rb:134:in `require'
	from /home/bewe/Data-Platform/user-profile/app/core.rb:7:in `<top (required)>'
	from /home/bewe/.rvm/rubies/ruby-2.2.3/lib/ruby/site_ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /home/bewe/.rvm/rubies/ruby-2.2.3/lib/ruby/site_ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from bin/console:3:in `<main>'

Somehow similar to this issue in the Rails world:

http://stackoverflow.com/questions/32457657/rails-4-gemloaderror-specified-mysql2-for-database-adapter-but-the-gem-i

Did you add the mysql2 gem to your Gemfile, like it suggests in the error message? In any way, I know for sure that this is not a bug in sinatra-activerecord.

Yes yes I do have mysql2 in the Gemfile.
But sinatra-activerecord is requiring activerecord 4.2.0.
And these is a bug in activerecord 4.2.0, so that its not allowing to use mysql2 0.4.X (see the above article).
If I manually update activerecord to <IP_ADDRESS> (in Gemfile.lock), it works.

Aha, I see. But I don't think that sinatra-activerecord should exclude a specific ActiveRecord version from the requirement if it has a bug.


Replace the image with no builds for ci if none

Closes https://github.com/ipfs/project-repos/issues/7.

Nice, looks great.


Remove profile link for anonymous user

Fixes MA-2112.
@1zaman @bguertin @BenjiLee plz review

OK, so I guess comments can't be added anonymously, so the isAuthorAnonymous() method should always return false in their case. Otherwise :+1:


UnknownError,UnknownError

Error: UnknownError,UnknownError
at <anonymous> in src/vs/platform/extensionManagement/common/abstractExtensionManagementService.ts:670:10
at Array.reduce (<anonymous>)
at p in src/vs/platform/extensionManagement/common/abstractExtensionManagementService.ts:669:16
at joinErrors in src/vs/platform/extensionManagement/common/abstractExtensionManagementService.ts:342:30
at async Q.z in src/vs/platform/extensionManagement/common/abstractExtensionManagementService.ts:228:5
at async Q.installFromGallery in src/vs/platform/extensionManagement/common/abstractExtensionManagementService.ts:96:11

Go to Errors Site

Fixed by https://github.com/microsoft/vscode/pull/169777


Ng Generate documentation needs examples of using the options

documentation of ng generate doesn't have any examples of how to use the options.
please provide some samples of how to correctly use ng generate options.
example: how is --spec supposed to be used?
do i need to say ng generate --spec=false or is it ng generate --spec false
give the user an idea of how to actually use the options, and give examples to show how to override the defaults.

https://angular.io/cli/generate now contains examples so I think this issue is addressed.


Working Docker CI patch

Description
Fixed issue on docker CI. Fully working CI pipeline integrated to build and push images into docker repo.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

i meant approve 😭


Delete .vscode directory

Could you explain why?

Could you explain why?

I was suggesting deleting .vscode directory because if we somebody wants to have custom configs in .vscode, they would be overidding the folder. Secondly, the .vscode property is something which shouldn't be part of source code as it is not source code.

./web/.vscode can also be deleted.

Sure. I can add that folder as well in the PR
On Fri, 12 Jan, 2024, 5:05 am boojack, @.***> wrote:

./web/.vscode can also be deleted.
—
Reply to this email directly, view it on GitHub
https://github.com/usememos/memos/pull/2693#issuecomment-1888134658, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJBWLL3553GIDBJYB63VZ3YOBZKTAVCNFSM6AAAAABBJFLXFCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBYGEZTINRVHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi @boojack & @Juneezee , I've added the mentioned directories to be deleted.
Kindly give it a look!


chore: do not upload to snap store

Copy paste of https://github.com/ipfs-shipyard/ipfs-desktop/pull/1069
Trying to fix https://gitlab.parity.io/parity/fether/-/jobs/260245

Merging, will see what gitlab says


Change the address model

This should be more close to the current best practice for address forms.

Rename addressee to name.
Rename street to address1.
Rename supplement to address2.
Rename location to city.

These names should be closer to people's expectiations.

Change max_length on these fields to 1024 characters.

Some names can be surpringly long. We should avoid recommending a database
schema that is too restrictive to deal with real-world data.

hmm...
could you please point me onto these so named "best practices" for addresses. An ISO or any kind of other norm would be great.
I tried to use the most descriptive attributes, when creating that class. Something like address2 IMHO is not very descriptive.

I don't think there is a norm for this, at least none from which we could directly derive the fields and their  names.
This is a case where it is actually better to be less descriptive, because it means making less assumptions. For example, an address may not include a street name. So street is not a good name here. Users may need to to input data that is not a street name into this field. Similarly, supplement is often not a good term for the second line of an address (if ever -- I am pretty sure it's the wrong translation for German "Adresszusatz").
I did some research before starting to work on this and found this article:
http://www.uxmatters.com/mt/archives/2008/06/international-address-fields-in-web-forms.php
See in particular the section "Generic formats".
The variance in international address formats is surprisingly huge. I think it makes sense for the default models to use a generic format, as recommended by the uxmatters article. This is also the format in use by amazon.com.
Delivery services like Fedex and UPS also use a format like this, and they should know ;) Having said that, maybe it's worthwhile to include an "Address line 3" field (I took a look at the UPS worldship software, it has "address line 1", "address line 2" and "address line 3").
If integrators want to have a more specific format, for example because they only target customers from a certain country, they can of course still use their own address model.

Here is another source advocating this as a best practice:
https://saasuserexperience.wordpress.com/2008/12/17/once-and-for-all-solved-international-address-fields-in-web-forms/

@rfleschenberg thanks for this long explanation. Yes, the link you provided convinced me to change that. Please go ahead and merge it.

I merged this, but we still need to manually test the example shops. I have that on my radar anyway, for reasons not related to this PR.


[add] webmail flagged, unseen search

Coverage decreased (-0.004%) to 80.55% when pulling 75c434c48e9e64e27e96ac48a816a0d1cffb3cbe on itowtips:dev/gws-webmail-search into 70271eb5b8c747f6b2b03c1886e475e89246e87e on shirasagi:dev/gws.

Coverage increased (+0.2%) to 80.794% when pulling 12825c72d46862915b6ee51bb6b52892f67dbb1c on itowtips:dev/gws-webmail-search into 70271eb5b8c747f6b2b03c1886e475e89246e87e on shirasagi:dev/gws.


Manual restart docker graphql required after syncing db

Summary
When booting my node, it'll start syncing (after installing the snapshot) which will take some time. This results in sockets not being available yet etc. Therefore cardano-graphql will exit.
To get the graphql up and running I have to manually restart it after sync has completed.
How can I make it restart automatically or something like that? I hoped it already did, but after waiting an hour I gave up 😅
Steps to reproduce the bug

Launch new docker(-compose) stack for GraphQL.
Give it time to sync.
Do a test cURL request, and it'll give you an error:
curl   -X POST   -H "Content-Type: application/json"   -d '{"query": "{ cardanoDbMeta { initialized syncPercentage }}"}'   http://localhost:3100/graphql, it'll give you the 'actual result'
docker restasrt cardano-graphql
Try the cURL request again after waiting a few minutes
It'll give you the expected result.

Actual Result
curl: (56) Recv failure: Connection reset by peer

Expected Result
{"data":{"cardanoDbMeta":{"initialized":true,"syncPercentage":100}}}

Environment
cardano-node:1.35.2-1
cardano-graphql:master-testnet
cardano-graphql-hasura:6.2.0
cardano-db-sync:13.0.3
ogmios:v5.5.3-testnet

Platform

[X] Linux (Ubuntu)
[ ] Linux (Other)
[ ] macOS
[ ] Windows

Platform version
20.04
Runtime

[ ] Node.js
[X] Docker

Runtime version
20.10.17, build 100c701

Thanks for the detailed report. I'll see what we can do to handle the initial startup more elegantly

same issue, after restart graphql, the running node index.js does not listen 3100 port at all!
root@a66dfb3debcf:/app/packages/server/dist# netstat -ntulp
Active Internet connections (only servers)
Proto Recv-Q Send-Q Local Address           Foreign Address         State       PID/Program name
tcp        0      0 <IP_ADDRESS>:34583        <IP_ADDRESS>:*               LISTEN      -
udp        0      0 <IP_ADDRESS>:58658        <IP_ADDRESS>:*                           -
root@a66dfb3debcf:/app/packages/server/dist# ps -ef
UID          PID    PPID  C STIME TTY          TIME CMD
root           1       0  0 07:13 ?        00:00:01 node index.js
root         103       0  0 07:16 pts/0    00:00:00 /bin/bash
root         116     103  0 07:18 pts/0    00:00:00 ps -ef


HTTPS Support

Describe the feature you'd like
Pretty simple...support for HTTPs would be nice. If you force the page to load over https (after adding virtualhost 443) the pages are broken because the links in the page reference http...causing them not to load.

Describe the benefits this would bring to existing BookStack users
It's HTTPs...it's pretty standard and businesses would benefit greatly.
Can the goal of this request already be achieved via other means?
Not that I know of :(
Have you searched for an existing open/closed issue?

[X] I have searched for existing issues and none cover my fundemental request

How long have you been using BookStack?
0 to 6 months
Additional context
No response

Hi @Mailstorm-ctrl,
You just likely need to update the APP_URL option in your .env file (Commonly at /var/www/bookstack/.env if installed using the Ubuntu install script).
I've have a video guide on adding HTTP if it helps:
https://youtu.be/wq78komr9rs?t=649

Hi @Mailstorm-ctrl,
You just likely need to update the APP_URL option in your .env file (Commonly at /var/www/bookstack/.env if installed using the Ubuntu install script). I've have a video guide on adding HTTP if it helps: https://youtu.be/wq78komr9rs?t=649

Ah...didn't realize that. Thanks!

Will therefore close this off.


docs(): socket update to 0.8+ recipe

Codecov Report

Merging #135 into master will decrease coverage by 2.55%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #135      +/-   ##
=========================================
- Coverage   37.46%   34.9%   -2.56%
=========================================
  Files         118     118
  Lines        5272    4595     -677
=========================================
- Hits         1975    1604     -371
+ Misses       3297    2991     -306

@mkucharz changes applied


'Pick Block' does not work

Using the 'Pick Block' function of Vanilla Minecraft does not work when this mod is installed on a server.

Hey there, thanks for the issue! I located the problem and have patched it in the newest release.


Prebuilt images

Hi Xavier (cc @ltalirz),
Is it possible to have a few prebuilt images available to use during the install process please? Right now it seems that there's a shortage of N-series VMs on US East, which makes it difficult to continue the install process because we have to build our own images. Also, the image build is rather slow and, as far as I can gather, you get identical results for all builds. Is it a problem with requirements when publishing images, or the fact that they're changing too quickly?
Thanks!
Matt

Yes this is something we are working on. There will be images in the marketplace for infrastructure VMs as well as for compute and viz nodes.
In the meantime, if you don't need viz node at all, you can change the imageid in the config file to be a standard market place image for CentOS 7.9 like OpenLogic:CentOS-HPC:7_9-gen2:latest and skip building the remote desktop image.

Thanks for the heads up! I'll do that.


trailing CR_LF will break binary protocols

Coverage remained the same at 95.256% when pulling a99aa95ac8deb45b91d2bf38c74e7e15ee9f43aa on StarpTech:feature/remove_trailing_data_for_buffers into 93c2cdf0d4e8e2a5467fc1cbe415414bee331aa3 on nats-io:master.

Coverage remained the same at 95.256% when pulling a99aa95ac8deb45b91d2bf38c74e7e15ee9f43aa on StarpTech:feature/remove_trailing_data_for_buffers into 93c2cdf0d4e8e2a5467fc1cbe415414bee331aa3 on nats-io:master.

I think the failing build for node 0.12 is related to this PR. Could be a estimation delta.should.within(10, 25); ?

Coverage remained the same at 95.256% when pulling 0605f10efef36b00061533d0191a981083f6c04b on StarpTech:feature/remove_trailing_data_for_buffers into 93c2cdf0d4e8e2a5467fc1cbe415414bee331aa3 on nats-io:master.

Coverage remained the same at 95.256% when pulling 0605f10efef36b00061533d0191a981083f6c04b on StarpTech:feature/remove_trailing_data_for_buffers into 93c2cdf0d4e8e2a5467fc1cbe415414bee331aa3 on nats-io:master.

Coverage remained the same at 95.256% when pulling 0605f10efef36b00061533d0191a981083f6c04b on StarpTech:feature/remove_trailing_data_for_buffers into 93c2cdf0d4e8e2a5467fc1cbe415414bee331aa3 on nats-io:master.

yep, it was :)

@aricart @derekcollison would be awesome if you could review it.

Coverage remained the same at 95.256% when pulling 62252ce90e44c9ec6748fa2a49f24f05b8312914 on StarpTech:feature/remove_trailing_data_for_buffers into 93c2cdf0d4e8e2a5467fc1cbe415414bee331aa3 on nats-io:master.

Coverage remained the same at 95.256% when pulling 62252ce90e44c9ec6748fa2a49f24f05b8312914 on StarpTech:feature/remove_trailing_data_for_buffers into 93c2cdf0d4e8e2a5467fc1cbe415414bee331aa3 on nats-io:master.

@StarpTech Can you write a negative test?
I augmented the binary.js, to do reuse the logic of the binary test, but also have a specific but taking into account that the payload can be a a buffer and it passes without your change. I did something like this for all tests, let me put a pr up so you can further write some tests
nc.subscribe('invalid2octet', function(msg) {
      msg.length.should.equal(2);
      if(nc.options.preserveBuffers) {
        should.ok(invalid2octet.equals(msg));
      } else {
        msg.should.equal(invalid2octet.toString('binary'));
      }
      finished();
    });

Here's the branch with the passing test.
https://github.com/nats-io/node-nats/tree/binary-tests

@aricart  The reason why it doesn't fail is because they are processed together not in chunks. Apart from this nowhere is CR_LF_LEN appended. You can test it if you do:
Change line 866 to
client.payload.msg = client.inbound.slice(0, client.payload.size + CR_LF_LEN);

Check invalid2octet
    nc.subscribe('invalid2octet', function (msg) {
      console.log((new Buffer("\r\n")).toJSON()); //{ type: 'Buffer', data: [ 13, 10 ] } // binary data of \r\n
      console.log((msg).toJSON()); //{ type: 'Buffer', data: [ 195, 40, 13, 10 ] } see its appended
      msg.length.should.equal(2);
      if(nc.options.preserveBuffers) {
        should.ok(invalid2octet.equals(msg));
      } else {
        msg.should.equal(invalid2octet.toString('binary'));
      }
      finished();
    });

BTW, if you do toJSON() at least on my side I get an error. Doing JSON.stringify() will correctly build. However the error doesn't show unless there's a very large message payload. I updated the test on the branch I have. If you can please incorporate the test into your PR, then I'll merge it.
(https://github.com/nats-io/node-nats/tree/binary-tests)

Coverage increased (+0.2%) to 95.446% when pulling 254372f04f2ac4a2e50d14a0f8c9dbe904751135 on StarpTech:feature/remove_trailing_data_for_buffers into 93c2cdf0d4e8e2a5467fc1cbe415414bee331aa3 on nats-io:master.

Hi @aricart done. I also could reproduce it through test.

Coverage increased (+0.2%) to 95.446% when pulling 8d3ac81ecec4944b8a44d60e66e30e12d06c8ae2 on StarpTech:feature/remove_trailing_data_for_buffers into 93c2cdf0d4e8e2a5467fc1cbe415414bee331aa3 on nats-io:master.

@aricart can you retrigger? Thats not my test.

@StarpTech Awesome - one more thing. Could you please squash all the changes and revert the change on the readme (noop). I'll merge after that goes through.

Coverage increased (+0.2%) to 95.446% when pulling abdde2420a2f6ec149dd28292b9f4933eaa51d6f on StarpTech:feature/remove_trailing_data_for_buffers into 93c2cdf0d4e8e2a5467fc1cbe415414bee331aa3 on nats-io:master.

Coverage increased (+0.2%) to 95.446% when pulling abdde2420a2f6ec149dd28292b9f4933eaa51d6f on StarpTech:feature/remove_trailing_data_for_buffers into 93c2cdf0d4e8e2a5467fc1cbe415414bee331aa3 on nats-io:master.

Coverage increased (+0.2%) to 95.446% when pulling abdde2420a2f6ec149dd28292b9f4933eaa51d6f on StarpTech:feature/remove_trailing_data_for_buffers into 93c2cdf0d4e8e2a5467fc1cbe415414bee331aa3 on nats-io:master.

@aricart sry but I can'´t rebase it. It freeze on my computer. Can you di it for me? Thanks.

@aricart can you publish it today?

working on it.

@StarpTech it is available on npm.


[FEATURE]: add a landing page

Description
The Landing page  is just two buttons, which are the login buttons.
It can't be empty like that, there needs to be information about the project.
Screenshots
No response
Additional information
If you want to work on this, please do a design in figma first.
👀 Have you checked if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read and understood the rules in the Contributing Guidelines

Are you willing to work on this issue ?
None

Hey Dun-sin, I would like to work on this issue.

Hey Dun-sin, I would like to work on this issue.

sure, please provide your proposed design

Hey Dun-sin , I would like to work on this , I have crafted a go-to design, and I invite you to request any potential adjustments or enhancements. https://www.figma.com/file/mSIjiAzJ60Yl8EUwtNIfCT/Whisper-Design?type=design&node-id=0%3A1&mode=design&t=HqlUDk7VtIyAs6CP-1

sure, can you give me permission to access the file on figma

Can you check now

here's my submission: Whischat Landing Page design
https://www.figma.com/file/FrVfWVsQAD9uxQulm4SGsB/whischat-{Landing-Page}?type=design&node-id=0%3A1&mode=design&t=TA0IctRSfBcscYKn-1

here's my submission: Whischat Landing Page design https://www.figma.com/file/FrVfWVsQAD9uxQulm4SGsB/whischat-{Landing-Page}?type=design&node-id=0%3A1&mode=design&t=TA0IctRSfBcscYKn-1

hi @github-harshit is more simple and easy to read, you can have a look at his if you wish and you can request to collab,feel free

here's my submission: Whischat Landing Page design
https://www.figma.com/file/FrVfWVsQAD9uxQulm4SGsB/whischat-{Landing-Page}?type=design&node-id=0%3A1&mode=design&t=TA0IctRSfBcscYKn-1

Is it just me? But this looks like its a company rather than a fun app

here's my submission: Whischat Landing Page design
https://www.figma.com/file/FrVfWVsQAD9uxQulm4SGsB/whischat-{Landing-Page}?type=design&node-id=0%3A1&mode=design&t=TA0IctRSfBcscYKn-1

Is it just me? But this looks like its a company rather than a fun app

yeah, it's too clustered

Hi @Dun-sin I want to work on this.
Here is my proposed design.
Please provide feedback so I can start working on it.
https://www.figma.com/file/POwLhDMKh3gUHK6aC2ZZvc/Whisper?type=design&node-id=0%3A1&mode=design&t=Tf20AqBy6iH02gJU-1

Can you check now

Hi I left some comments, I don't think you got it?🤔

@Dun-sin I would like to work on the landing page request. below is my design for the landing page. I want to design as minimal as possible.

Can you check now

Hi I left some comments, I don't think you got it?🤔

@github-harshit

@Dun-sin review this ? Figma File Link

@Dun-sin I would like to work on the landing page request. below is my design for the landing page. I want to design as minimal as possible.

this isn't good, doesn't follow any good UI/UX principle. kudos to you for trying tho

@Dun-sin review this ? Figma File Link

that's not a landing page, so it doesn't count, good work tho, but I'm afraid you wasted your time, that's why you should always read the issue and its description. Keep that in mind for next time.


added label

Hi
I am trying to set this bot up for the official Joomla repository here https://github.com/joomla/joomla-cms/projects/13
The bot is working because I successfully tested the rule new_issue
However I am unsuccessful in getting the bot to work with the rule added_label
- `added_label` **J3**
Am I missing something obvious?

It looks like that should work. Can you try it without the bold? I just tried adding (& removing) the wontfix label and this Issue correctly moved.
- `added_label` J3

Sadly that didnt work either

I had/have similar issues with this, but it worked on an empty project board. Have you tried that @brianteeman? If you like, you can read about my testing over at https://github.com/philschatz/project-bot/issues/17

My tests were not an empty board in the sense that the board was created with one of the templates. I will try again following your tests and see if that fixes it - thanks

The problem is that the issue has already to be on the board, before the added_label action works for this issue. You can add an issue to an board e.g. via an 'inbox' column with the new_issue action.

Was not the issue in my case, I was experimenting with issues that were already on the board or new issues I created and then added a label.


Be able to find items in cart as left in last session

As a User
I need to find my shopcart prefilled with items filled before
So that I can order items in cart anytime later whenever I am ready
Details and Assumptions

Adding, Deleting, Updating items in shopcart will call the api created for database update for storing the shopcart changes for unique userID

Acceptance Criteria
Given the shopcart `foo` does exist
When I call 'GET /shopcarts/foo'
It returns the products in the shopcart

In other words, we need to save states after operations in the database so that we can retrieve them later.

Potential duplication to #10? @sweta1947 do you think we can close this one?

As per our offline discussion, I'm closing this issue for now. Feel free to reopen it if you have any questions!


[Bug] [1.20.4 Fabric Server] Attempting to run a Minecraft command from Discord results in an exception

Here's the full exception
[19:34:59] [pool-12-thread-1/ERROR]: One of the EventListeners had an uncaught exception
java.lang.NoSuchMethodError: 'int net.minecraft.server.command.CommandManager.executeWithPrefix(net.minecraft.server.command.ServerCommandSource, java.lang.String)'
        at fabric.com.hypherionmc.sdlink.FabricPlatformHelper.executeCommand(FabricPlatformHelper.java:23) ~[SimpleDiscordLink-Combo-1.20-3.0.1.jar:?]
        at fabric.com.hypherionmc.sdlink.server.SDLinkMinecraftBridge.executeMinecraftCommand(SDLinkMinecraftBridge.java:153) ~[SimpleDiscordLink-Combo-1.20-3.0.1.jar:?]
        at fabric.com.hypherionmc.sdlink.core.discord.hooks.MinecraftCommandHook.discordMessageEvent(MinecraftCommandHook.java:55) ~[SimpleDiscordLink-Combo-1.20-3.0.1.jar:?]
        at fabric.com.hypherionmc.sdlink.core.discord.events.DiscordEventHandler.onMessageReceived(DiscordEventHandler.java:63) ~[SimpleDiscordLink-Combo-1.20-3.0.1.jar:?]
        at com.hypherionmc.sdlink.shaded.dv8tion.jda.api.hooks.ListenerAdapter.onEvent(ListenerAdapter.java:446) ~[SimpleDiscordLink-Combo-1.20-3.0.1.jar:?]
        at com.hypherionmc.sdlink.shaded.dv8tion.jda.api.hooks.InterfacedEventManager.handle(InterfacedEventManager.java:96) ~[SimpleDiscordLink-Combo-1.20-3.0.1.jar:?]
        at fabric.com.hypherionmc.sdlink.core.util.ThreadedEventManager.lambda$handle$0(ThreadedEventManager.java:20) ~[SimpleDiscordLink-Combo-1.20-3.0.1.jar:?]
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:539) ~[?:?]
        at java.util.concurrent.FutureTask.run(FutureTask.java:264) ~[?:?]
        at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:304) ~[?:?]
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) ~[?:?]
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) ~[?:?]
        at java.lang.Thread.run(Thread.java:840) ~[?:?]

Oh yeah, this is the one you reported in Discord.
I haven't forgotten about this. I've been working on a new system for CraterLib so our mods can run independently from Minecraft.
Actually showed off the first working implementation of this running Simple RPC to our team over the weekend.
So yes, I am still going to fix this, but with it will also come improvements to prevent this from happening on future versions.

Sounds good, thank you. I just wanted to also have the issue here as I figured having the current list of issues to work on in Github is more convenient than in Discord. Thank you for answering.

I definitely is more convenient, cause I forget stuff reported in Discord. Not even going to lie about it.
Can see you're a developer yourself xD

It definitely is more convenient, cause I forget stuff reported in Discord. Not even going to lie about it.
Can see you're a developer yourself xD

Ye Im a fellow modder xd

where can i download dev version? only found beta

Finally fixed in 3.1.0

Finally fixed in 3.1.0

Awesome, thank you man 👍


MangaSwat URL Change

Source information
"ar.mangaswat 1.4.52"
Source language
"Arabic"
Source new URL
"https://swatapp.me"
Other details
No response
Acknowledgements

[X] I have updated all installed extensions.
[X] I have opened WebView and checked that the source URL has not changed yet.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I will fill out all of the requested information in this form.

For those who are waiting for the extension to update the baseURL in the extension:
You can manually override it in the extension's settings yourself, so you can continue to use the extension until it gets updated.

Go to your Browse Page > Extensions tab > MangaSwat settings, and override the URL with the one mentioned above.
Restart your app after applying the new URL, so the change takes effect.

Not working and can't read anything

They've changed links again, https://healteer.com, that's why when you open the source in WebView, they redirect you to a Google search of their new name. Don't be lazy, and there's a reason why you can override the URL in the extension's settings. Follow the steps above whenever this happen, but put the new link into the override & restart the app.
Works on my end

Ok got it

Keep the issue open, so whoever has this issue can see how to temporarily fix it. Also, whenever a contributor has the time can see the open issue & update the link.
Just edit your request with the new URL


Javascript + hardware (DIY IoT stuff)

@andrewjensen
Dec 17th, 1pm.
What specifically do you want to talk about related to this topic? Is there a project in particular that you've made?

Here are some things we could potentially talk about:
Projects I've worked on

Arcade Commander

CLI tool to trigger terminal commands from physical buttons, more functionality in the future

MIDI Clock Triggers

I threw a dance party at my apartment for precisely 17 minutes. This script automatically pressed play and stop on the party playlist.

MIDI Techno Lights

A collaboration with my dad. A project to control Philips Hue lightbulbs from MIDI messages, to sync with music on stage.

Concepts

HTTP (RESTful APIs)
MQTT
Serial
MIDI?

How to learn

Adafruit, Sparkfun blogs
Instructibles
YouTube channels

Notes and Volts

???

Dev Boards

Famous

Arduino
Raspberry Pi

Not as famous

Teensy
Beaglebone
Tessel
ESP8266

Getting Parts

adafruit
sparkfun
digikey
mouser

Other potential topics

Third-party APIs for IoT

IFTTT
AWS
etc.

IBM's Node-Red project (flow-based programming for IoT)

Thanks, I think we'll have more than enough to talk about.
Here's what I'm thinking about for a basic outline.
What is your background in working with software + hardware?
What is IoT?
What are some things you've built?
What are some good ways to learn or get started with this stuff?


Recive file in ASCII mode

HeLlo!
I own:
iTNC530, NC Software 606420 02 SP14 machine
I have another question. In the client.py > def recive_file :
with local_file.open('wb') as out_file:
            #file_buffer = bytearray()
            if response in self.RESPONSE_S_FL:
                # file_buffer.extend(content)
                if binary_mode:
                    out_file.write(content)
                else:
                    out_file.write(content.replace(b'\x00', b'\r\n'))
                logging.debug(
                    'received first block of file file %s', remote_path)

                while True:
                    response, content = self._llcom.telegram(
                        LSV2.RESPONSE_T_OK, payload=None, buffer_size=self._buffer_size)
                    if response in self.RESPONSE_S_FL:
                        # file_buffer.extend(content)
                        if binary_mode:
                            out_file.write(content)
                        else:
                            out_file.write(content.replace(b'\x00', b''))
                        logging.debug(
                            'received %d more bytes for file', len(content))
                    elif response in self.RESPONSE_T_FD:
                        logging.info('finished loading file')
                        break
                    else:
                        if response in self.RESPONSE_T_ER or response in self.RESPONSE_T_BD:
                            logging.error(
                                'an error occurred while loading the first block of data for file %s : %s', remote_path, self._decode_error(content))
                        else:
                            logging.error(
                                'something went wrong while receiving file data %s', remote_path)
                        return False

in while True: loop it should also be:
 else:
          out_file.write(content.replace(b'\x00', b'\r\n'))

Or am I wrong?
Thank you!

@kekec14 you are correct! Only the bytes in the first block got converted back to a new line.
I will publish a new version with the fix shortly!

Just pushed version 0.6.4 to pypi.org should be available for update shortly :-)


Populate hosts file from inventory

Hi,
I have a group of hosts I'd like configured in each others hosts file.
I think the sanest way will be to make a task which takes the hosts and builds a list to pass to hosts: ; my question is if you think it would be useful in the role or if it should be handled outside.
thanks,
Karl.

Hi, while looking at your question I took the opportunity to move and update the role to weareinteractive.hosts (2.0.0).
I don't believe it's a role's responsibility to collect multiple sources into one variable since this differs from each env and use case so this would be sth to be handled outside.


fix(adapters): Added adapter for broadcasting a tx

:tada: This PR is included in version 3.4.2 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


feat(customization): allow for custom component name registration and class extension

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Addresses: #58
Summary:

introduces the ability to extend the AuroHeader class and register the web component to a custom name (e.g hf-auro-header)

Type of change:
Please delete options that are not relevant.

[x] New capability
[ ] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.= seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


kafka_sink does not work with Python 3

Is this a bug, feature request, or feedback?
This is a bug.
What is the current behavior?
The kafka_sink connector does not work with Python 3. It does not send messages.
What is the expected behavior?
The connector should send messages.
What OS and version of Wallaroo are you using?
Ubuntu, Wallaroo  0.6.0
Steps to reproduce?
Run the kafka_sink with an application that is compatible with it in Python 3. Do something that should cause a message to be sent. The message will not be sent.

The issue is that the Kafka driver expects the key and value to both be bytes, but kafka_sink converts them to strings. In Python 2 this isn't an issue because strings and bytes are the same, but in Python 3 they are different and so it fails to send.


implement MDagPath

This is not ready to be merged yet. I'm making the PR for visibility, and because I had a few questions.
My questions:

Are we aiming to be more consistent with the C++ API or OpenMaya2?
DagPath.extendToShape() returns a reference to self in OM2 but not in the C++ API.
I feel like not returning anything would make more sense since self is already modified by the method
Do we have a standardized format for the error messages?
I've just copied OM2's messages so far.
I was expecting exclusiveMatrix and exclusiveMatrixInverse to fail when called from an invalid DagPath but they return an identity matrix instead.
This seems consistent with OM2 but I'm not sure why they're not raising a RuntimeError where something like childCount does with a similar implementation

Are we aiming to be more consistent with the C++ API or OpenMaya2?

Good question. There are subtle and not-so-subtle differences between the two, some of which I don't understand.
# C++
MTransformationMatrix::addTranslation()
MTransformationMatrix::rotation() -> MQuaternion

# Python
MTransformationMatrix.translateBy()
MTransformationMatrix.rotation() -> MEulerRotation

# But whyyyy?

They've made a few creative choices here, presumably to make it more user-friendly but it's so subjective..
I think we should stick to the C++ documentation as closely as possible and leave creative choices to the user. When that is not possible, such as..
// C++
MDagPath dagPath;
MFnDagNode(mobj).getPath(dagPath);

..where it's modifying a variable from a function rather than return a value (classic C++), we'll need to make some less-creative choices that I expect to become a predictable pattern anywhere this happens in C++.
# Python
dagPath = MFnDagNode(mobj).getPath(dagPath)

In this case, the reason they're doing it the way that they are in C++ is because they choose not to throw exceptions. Instead, they return a MStatus value expecting the user to handle it.
MDagPath dagPath;
MStatus status = MFnDagNode(mobj).getPath(dagPath);
if (status == MS::kSuccess) {
  // It's ok to use dagPath
}

With Python, I expect we'll be able to completely replace MStatus with exceptions.

Do we have a standardized format for the error messages? I've just copied OM2's messages so far.

Let's start there. I think it's hard to know until we have enough of a foundation to actually start using cmdc for actual code and experience what messages make the most sense given the context. At this point, some message is better than no message.

I was expecting exclusiveMatrix and exclusiveMatrixInverse to fail when called from an invalid DagPath but they return an identity matrix instead.

Yes, the API loves this. It's perfectly happy letting you continue working with bad memory and chooses to randomly crash on you whenever it feels like it instead. This is one of the things I'd like cmdc to get rid of. It's a bad habit.
In this case, if the API isn't returning a bad MStatus, but we know for certain the value is bad, we should throw an exception ourselves to prevent the user from experiencing a crash.

Nice, that all sounds good!
It might be worth creating a CONTRIBUTING.md to keep track of these design decisions

It might be worth creating a CONTRIBUTING.md to keep track of these design decisions

Good idea!

I've gotten rid of any MStatus checks here, It looks more consistent between methods that return a relevant status and those that don't.
However, this opens the problem where we might not think of checking all the ways that might make a method fail.
Maybe switching on the status after the manual checks to throw anyway on any unhandled errors would be a good way to go?
Since I'm not using MStatus anymore, I've changed the error message and reworded them to something I felt was more informative.

I've gotten rid of any MStatus checks here

Bah, looks like I commented on an old commit. But anyway, the MStatuses may not always return whether it fails, but it might sometimes. So we should probably still check it, alongside our extra checks. What do you think?

Yeah, that sounds good to me

Not quite done yet but I've implemented most methods now.
For the error handling, I added checks to the returned status for almost everything except apiType because it returns kInvalid when it fails.
A few questions:

The method getDisplayStatus is implemented in OpenMaya 2 but not in the C++ API, I'm guessing it's fine to ignore it?
I couldn't figure out what pathCount is supposed to do, seems to always return 1. Any clue?

The method getDisplayStatus is implemented in OpenMaya 2 but not in the C++ API, I'm guessing it's fine to ignore it?

There's lots of things missing in OM 2. I personally think we should include as much as humanly possible. I run into things missing in OM 2 far too often, many do, and it's one of the motivations behind writing a new binding.
We don't have to implement everything at once though, so feel free to leave it for now if it slows you down.

I couldn't figure out what pathCount is supposed to do, seems to always return 1. Any clue?

I think this has to do with instances. Try making an instance of a transform, and it should return 2.

There's lots of things missing in OM 2. I personally think we should include as much as humanly possible. I run into things missing in OM 2 far too often, many do, and it's one of the motivations behind writing a new binding.

The function is actually there in OM2 but not it the C++ API haha

I think this has to do with instances. Try making an instance of a transform, and it should return 2.

That would make sense, I'll test that

Oh haha, I misread. :D I could have sworn this was in the API because I'm using it all the time, but turns out it's actually called MGeometryUtilities::displayStatus(dagPath).
Maybe they moved that there, because it isn't exposed from it's original place? I haven't used it from Python before.
That's an interesting point actually. I'm hoping we're able to avoid that, and actually expose it where it is in the C++ API rather than move stuff around. E.g. cmdc.GeometryUtilities.displayStatus(dagPath). If not, we might be forced into making some of these creative choices ourselves, and that would be unfortunate. :(

Not 100% sure I like that but how about "implementing" the method but just raising an error saying cmdc.GeometryUtilities.displayStatus(dagPath) should be used instead?
Main pro is that it helps people transition from OM2 easier
But the con is that the existence of the method is misleading and people will be drawn to use it until they realize it just throws an error
Again, not really convinced by this, just throwing the idea out there

I think we should stick to the C++ docs, that will be the beacon of light. Users with experience with the C++ API will be familiar with cmdc, and users of cmdc will become familiar with the C++ API.

A few notes on what I've done and omitted:

I still couldn't figure out pathCount, even instances so I left it unimplemented rather than implemented but untested.
I've added all the missing C++ methods but left them unimplemented
I have left outclassName on purpose because it would have returned MDagPath instead of DagPath and python already has obj.__class__.__name__ anyway

Wasn't sure if I should have squashed all the commits or rebased the branch against master so I haven't done any of that. Let me know if you need me to do more work before merging this :)

Good call.

I have left outclassName on purpose because it would have returned MDagPath instead of DagPath and python already has obj.class.name anyway

Good point!

Wasn't sure if I should have squashed all the commits or rebased the branch against master so I haven't done any of that. Let me know if you need me to do more work before merging this :)

I think this looks good, great work. :)

I've made it so that new tags generate a Maya module for all platforms and all Maya's.
Can give it a try here:

https://github.com/mottosso/cmdc/releases


Workaround bug in macOS 14.4.1 and later where encoding HEIC fails

Closes #66 and #68.

CC: @rlxone

Hi, thanks for the fix, do we know what versions were affected? Is it still an issue?

Can't reproduce on my M1 Mac Mini (macOS 14.6.1 (23G93)) using images from #66.
Do you use macs with intel processors?

Hey @dreampiggy, sorry for bothering you, but seems like you investigated this issue a lot. What is the current state of the issue? Have we reported this bug to Apple?

No. I didn't fired radar for this.
The HEVC encoding seems fails on macOS 15.0 beta, which is not the same issue.
If you found this bug in macOS 14.4.1 and has workaround (using compression quality), maybe it's another case

Actually, if Apple's framework is not trustable, you can provide a fallback workaround
Since HEVC/HEIF is standard, there are also open-source encode/decode lib, like https://github.com/strukturag/libheif
If you don't like the C API, just use my wrapper API for encoding https://github.com/SDWebImage/SDWebImageHEIFCoder

Hello,
@CasualDeveloper here is a build with the relevant changes included.
@rlxone this continues to be a relevant issue. If you're not experiencing this on your M1 Mac Mini, then the issue may be Intel-specific, as my 2019 MacBook Pro running macOS Sonoma 14.6.1 (23G93) exhibits the bug. Unfortunately, I'm not aware of the exact affected version range except for 14.4.1 being a start.

I tried @CasualDeveloper's build, and I can confirm that this is still an issue on my Intel-based Mac. 🙅🏼

Hello,
@CasualDeveloper here is a build with the relevant changes included.
@rlxone this continues to be a relevant issue. If you're not experiencing this on your M1 Mac Mini, then the issue may be Intel-specific, as my 2019 MacBook Pro running macOS Sonoma 14.6.1 (23G93) exhibits the bug. Unfortunately, I'm not aware of the exact affected version range except for 14.4.1 being a start.

I tried @CasualDeveloper build and I can confirm that it works successfully on my Intel-based Mac. ✅

Hello,
@CasualDeveloper here is a build with the relevant changes included.
@rlxone this continues to be a relevant issue. If you're not experiencing this on your M1 Mac Mini, then the issue may be Intel-specific, as my 2019 MacBook Pro running macOS Sonoma 14.6.1 (23G93) exhibits the bug. Unfortunately, I'm not aware of the exact affected version range except for 14.4.1 being a start.

@rlxone @kyleerhabor
I can confirm that that Kyle's build works on my MacBook Pro (2019, Intel i9) running latest macOS Sonoma

FYI I've filed a radar in Feedback Assistant on this issue, and also a copy of it on OpenRadar.

Hey y'all, good news! 😊
The bug doesn't seem to occur on my MacBook Pro (2019, Intel i9) running macOS Sequoia 15.0 Beta (24A5331b) – tested with Equinox v3.0 from App Store.
Creating an Appearance-type wallpaper with two images now complete successfully (I'm assuming with 1.0 quality, since the files sizes seem bigger compared to using 0.9999999 quality)

That's great. I wouldn't mind amending this pull request with checks for the earliest resolved version (either Sequoia 15 or a later version of Sonoma) and an Intel device.

Thank you all for helping with this!
@kyleerhabor feel free to go ahead, i can merge it as soon as you ready with the new additions.

@rlxone I've added the additions.

@kyleerhabor sweet, i'll review tomorrow

@kyleerhabor looks good to me, i'll release a new app store build in a couple of days.
Thanks everyone for helping with his again!


Add ZerosSampler

A dummy sampler which implements cirq.Sampler and returns zeroes for all measurements.
Fixes #2818

Hi Matthew,

ZeroSampler inherits from Sampler, which has default implementation of run_async which  calls run and of run_sweep_async which calls run_sweep. So now async methods of ZerosSampler will work as expected, and I don't need to implement them. Am I right?
I didn't work with gate sets in Cirq before, but here is what I can suggest. We can pass SerializableGateSet object to ZeroSampler constructor. Then in run_sweep we will check for all operations whether it is_supported_operation. And if there is no gate set, don't do this validation. Does it make sense? Do you think it's better to do this in a follow-up PR?
Also, what are "extand qubits"?

1: ok good
2: that sounds entirely reasonable to me
2b, re: follow up PR. up to you. If it's not too drastic of a change, I'd say just put it into this because it will likely involve changing the API of ZerosSampler slightly
2c: re: extant qubits. real devices only have certain qubits and a common failure mode is accidentally using e.g. a LineQubit when you should use GridQubits or using GridQubit(0,0) when the physical numbering scheme starts somewhere else

Hi @mpharrigan,
I added gateset validation, PTAL.
Regarding extand qubits: there is a class cirq.devices.Device, which has method qubit_set. So we could pass the device to a constructor (like I did with gate set) and in run_sweep() get all qubits referenced by a device, and check that they are all in this qubit_set.
However, this can be simplified: we could just pass qubit_set. Or, if we are passing device, we can just call Device.validate_circuit.
What do you prefer?
Also, does Device.validate_circuit validates gateset? If so we don't need SerializableGetSet in ZerosSampler (which I just added).

Question to @mpharrigan : shouldn't the device and gateset validation be handled by setting an appropriate device on the circuit?  What is the primary use case for this class, is it to do this validation?  Then maybe it should be called something slightly different?

Can we merge this?
@mpharrigan @dabacon

Automerge cancelled: A required status check is not present.
Missing statuses: ['cla/google']

Automerge cancelled: A required status check is not present.
Missing statuses: ['cla/google']

@googlebot  please rescan


CC: Drop special handling in recheckApplication

Based on #21445
In recheckApplication we implemented a rule that computed the capture set of an
application f(a) by being the "better" of either the capture set of f union
capture set of a or the capture set of the result type of f(a). This relies
on capture monotonicity, which we try to get away from. Also, it's semantically
dubious if the type of the argument a is a type variable that can be instantiated
to a capturing type as in the following example:
trait Iterator[+A]
  ...
  def flatMap[B](f: A => IterableOnce[B]^): Iterator[B]^{this, f}

Here, we account for every capability in IterableOnce[B]^ already in f. But
those capabilities could also come from A if A is instantiated in the
actual function argument to a capturing type.
There was extensive breakage once the special handling was dropped. One example is
the flatMap definition above. We leave that as a potential soundness hold for now,
but the right way to fix flatMap would be by adding a capture set variable:
def flatMap[B, C^](f: A => IterableOnce[B]^{C^}): Iterator[B]^{this, f, C^}

The problem is that this currently cannot be done in a way that allows flatMap to be called
as before, passing a single type argument for B. We'd have to change the system to either
allow curried type parameters or optional type parameters for capture sets.
Another issue is that now more reach capabilities are registered as used by the enclosing method.
An example is lazylist-exceptions.scala, which has been moved to pending. Here, the use is spurious
because it happens inside an anonymous class creation on the right hand side of the method.
With refined use checking, the use would not propagate to the method, so the reach capability should
not be leaking. But to get there we need to implement refined use checking first.

Close as the changes have been included in #21445.


Doesn't work with --watch

This happens as soon as I change a file and the rebuild finishes. By the way it looks like, a new instance of express is launched at that moment.
Was webpack monitor designed to be used with --watch? If it wasn't, consider this as a feature request.

Hi @dimitarnestorov, you are right, webpack monitor is not ment to be used in a watch mode:
(from #48):

Typically you'd insert webpack monitor as a plugin on your production config and only periodically launch the dashboard.

and the reason being:

the idea being that we are not so concerned about the state of the non-optimized dev builds

That's why on each run a new express instance is started and when the old one blocks the port, an error is trown.
You could set the launch flag to false, or move this to your production webpack config.
maybe webpack monitor will support this one day.
you can also have a look at some similar issues:
EADDRINUSE on compile #48
Support for hot reloading #69
throw error on port 8081 when editing code

@dimitarnestorov , you said supporting a watch mode would be your feature request. What would be your use case, or what behavior would you expect?

with the current default behavior, when the capture flag in on, then each small change would result to a new point in the chart and the stats would be bloated pretty fast.
capture: false would allow a stats server to run and show the captured stats, but local changes would not be visualized in the charts.
another option would be to not capture the changes and save in the stats.json. but to show the current snapshot as the last point in the charts to be compared with captured data.

I am going to use webpack monitor to focus on optimizing the production build.
My issue as that if I don't use watch mode I have to wait 5+ mins between each build in order to see the result.
In order to achieve the result I want right now I will use capture: true, --watch and run the webpack monitor express instance without launch: true.
So the ideal situation is capture and launch both to be true and the webpack watcher not to crash.
And auto reload (but I will do it myself using BrowserSyncPlugin [proxy: 'http://localhost:8081/'])


Traning code

Thanks for the great work！
Will all the training code be released recently？

Thanks for your interest! The training code will not be released very soon (e.g. a month).

@MaureenZOU will be training code still be released?

Yes, I will for sure release. We already have a public draft version support by openseed training pipeline.


snort-<IP_ADDRESS> cannot be downloaded -> Fails nixos-rebuild

I did:
sudo nix-channel --update
sudo nixos-rebuild test

and it gave me:
curl: (22) The requested URL returned error: 404 Not Found
error: cannot download snort-<IP_ADDRESS>.tar.gz from any mirror
builder for ‘/nix/store/sq9jaai089m8p0w1fgpslkkplld37j0i-snort-<IP_ADDRESS>.tar.gz.drv’ failed with exit code 1
curl: (22) The requested URL returned error: 404 Not Found
error: cannot download daq-2.0.4.tar.gz from any mirror
builder for ‘/nix/store/rq38fv4p85xlcydhlvnfiqnclxkfx4vn-daq-2.0.4.tar.gz.drv’ failed with exit code 1

Whereas the second one was already reported in #7947 and should be fixed since, but unfortunately it still shows up. I'm not sure, though, whether the patch has already appeared in the unstable channel.
The first issue is still the version which is in master, at this moment. daq is on 2.0.5 already, which should fix the issue, seems it is not in the channel yet.

The homepage only serves the latest version :-)

That's why I opened a PR to fix this error.

 Anyways, thanks!

I'm not happy with your patch. You download from sourceforge, which is not good. I guess you heard why SF is bad. I'd like to see another solution!

We check the hashes, so I don't really mind using their bandwidth, even if they're bad.


Format stopwatch

issue https://github.com/avinash201199/stopwatch/issues/20#issue-1014275392

hey, you are doing great work :)
please solve the branch conflict


PR for Umbraco 13.0.1 and .net 8 support.

This is a PR for Umbraco 13.0.1 and .net 8 support.
It was missing implementation of members from Umbraco.Cms.Core.Deploy.IDataTypeConfigurationConnector - Umbraco has added IContextCache contextCache.
This implementation simply reroute the calls to the existing methods - not implementing the cache layer.
It also bumps dependency versions to Umbraco 13.0.1 and .net8 due to dependencies - maybe we should use compilerdirectives to keep compatibility with older versions?

Added multitarget framework support for compatibility

Hey @patmoe!
It would be great to get the package compatible with .NET 8 and working on v13, so thank you so much for submitting this.
At first glance it looks great - I'll let you know if I have any questions when I give it a test!

:-) I've tested this together with Lars Erik (LEA) in our project - so I hope it's ok...
btw, just bumped the version to 1.1.4-beta01 to avoid cache issues.

Hey @patmoe - thanks so much for not just realising this didn't work on v13, but for fixing it too!
I've added test sites to the repo for v12 and v13, so was able to duplicate the error, then merge in your PR and confirm the fix worked!
I've now released a new version, and you're listed as a contributor 🎉
https://www.nuget.org/packages/Our.Umbraco.Community.AdminOnlyProperty/1.1.4
Seasons greetings!


Add cluster-admin permissions to the compliance controller for DockerEE

Description

In order to start pods with privileged permissions or host path volumes we need to grant the creating pod cluster-admin privileges in DockerEE (https://docs.mirantis.com/docker-enterprise/v3.0/dockeree-products/ucp/authorization/access-control-model.html#secure-kubernetes-defaults). Our other pods are unaffected because the operator orchestrates their creation, but in the compliance-controller's case, it will start jobs that require host path volumes.
For PR author

[ ] Tests for change.
[ ] If changing pkg/apis/, run make gen-files
[ ] If changing versions, run make gen-versions

For PR reviewers
A note for code reviewers - all pull requests must have the following:

[ ] Milestone set according to targeted release.
[ ] Appropriate labels:

kind/bug if this is a bugfix.
kind/enhancement if this is a a new feature.
enterprise if this PR applies to Calico Enterprise only.

Let's hold off on merging this, until we run out of other options.


Update documentation for top-level classes.

I updated the doxygen documentation for bigtable::Table,
bigtable::TableAdmin, and bigtable::InstanceAdmin. Fixed a lot of
nits, documented which operations are idempotent (and thus normally
retried), documented how to configure the policies in bigtable::Table,
their default values for all 3 classes. And also which operations are
thread safe on these classes.
Incidentally this fixes #801.

This change is

To see the change output look at:
https://coryan.github.io/google-cloud-cpp/0.2596.0/bigtable/index.html
particularly:
https://coryan.github.io/google-cloud-cpp/0.2596.0/bigtable/classgoogle_1_1cloud_1_1bigtable_1_1v0_1_1InstanceAdmin.html
https://coryan.github.io/google-cloud-cpp/0.2596.0/bigtable/classgoogle_1_1cloud_1_1bigtable_1_1v0_1_1TableAdmin.html
https://coryan.github.io/google-cloud-cpp/0.2596.0/bigtable/classgoogle_1_1cloud_1_1bigtable_1_1v0_1_1Table.html


Make sure registration is required.

Fixes: https://github.com/home-assistant/home-assistant-android/issues/348

I was running home assistant companion in lineageos without problems, but this commit breaks it.
The app gets stuck in a loop "unable to register app".

My guess is that this is happening because I am not using google services as I want my data privacy instead of google spying on my home
In version 1.6 I also had the "unable to register app" message, but I restarted the app and it connected properly

Please keep this option open for those who are willing to give up notification in order to avoid using google services.


generic Client() getters lazy loading

Describe changes
I implemented a generic helper client_lazy_loader to be used in any client get method to achieve lazy loading in a pipeline context. It mocks the calls to client method reworked as shown below to build a call_chain.
For example, client.get_artifact_version(...).run_metadata["..."].value will be translated into ClientLazyLoader(method_name="get_artifact_version", call_chain=[(is_call,args=...),(get_attribute="run_metadata"),(get_item="..."),(get_attribute="value")] (I used not actual variable name for more clarity 🙂 ).
Next on step execution, this chain will get evaluated into actual client calls and the user will get metadata value in this case.
Onboarded methods:

get_artifact_version
get_model
get_model_version

Other methods behave as usual.
Client methods modifications needed:
    def get_artifact_version(
        self,
        name_id_or_prefix: Union[str, UUID],
        version: Optional[str] = None,
        hydrate: bool = True,
    ) -> ArtifactVersionResponse:
        if cll := client_lazy_loader(
            method_name="get_artifact_version",
            name_id_or_prefix=name_id_or_prefix,
            version=version,
            hydrate=hydrate,
        ):
            return cll  # type: ignore[return-value]
        return self._get_entity_version_by_id_or_name_or_prefix(
            get_method=self.zen_store.get_artifact_version,
            list_method=self.list_artifact_versions,
            name_id_or_prefix=name_id_or_prefix,
            version=version,
            hydrate=hydrate,
        )

Usage in a pipeline:
save_artifact(
    data="body_preexisting", name="preexisting", version="1.2.3"
)
log_artifact_metadata(
    metadata={"some_meta": "meta_preexisting"},
    artifact_name="preexisting",
    artifact_version="1.2.3",
)

@pipeline(enable_cache=False)
def dummy():
    artifact_existing = clean_client.get_artifact_version(
        name_id_or_prefix="preexisting"
    )
    artifact_metadata_existing = artifact_existing.run_metadata[
        "some_meta"
    ]

    # this artifact and metadata is not existing on pipeline compilation
    artifact_new = clean_client.get_artifact_version(
        name_id_or_prefix="new_one"
    )
    artifact_metadata_new = artifact_new.run_metadata["some_meta"]

    lazy_producer_test_artifact()
    lazy_asserter_test_artifact(
        # load artifact directly
        artifact_existing.load(),
        # pass as run metadata response
        artifact_metadata_existing,
        # pass as artifact response
        artifact_new,
        # read value of metadata directly
        artifact_metadata_new.value,
        after=["lazy_producer_test_artifact"],
    )

On top user can also pass Lazy Loaded calls around to other client methods like:
@pipeline(enable_cache=False)
        def dummy():
            model = clean_client.get_model(model_name_or_id="model_name")
            # load model version by version
            model_version = clean_client.get_model_version(
                # this can be lazy loaders too
                model_name_or_id=model.id,
                model_version_name_or_number_or_id="model_version",
            )
            ...

Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] I have based my new branch on develop and the open PR is targeting develop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.
[ ] If my changes require changes to the dashboard, these changes are communicated/requested.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

@coderabbitai review

I like the approach overall. Tests failing atm, but will you add the other ones discussed on Discord to this PR or make separate tickets/PRs?

Yeah, just did so. Tests shall pass now, I expect.

/fulltest

@coderabbitai review


how to write roi from np array

If i generate a n-d array mask, any idea how to write into roi? Thanks!

Same question here! Is the problem solved now? If not, I think this will be helpful:
https://imagej.nih.gov/ij/developer/source/ij/io/RoiEncoder.java.html
Sadly I am not so familiar with Java.

This package is only for reading ROI files generated by ImageJ. Writing is not implemented but PRs are welcome.


WIP - sync status back to control plane

Adds a spec syncer which will sync spec from upstream to downstream.
This also adds an argument to specify which namespace in the control plane to sync spec from and status to.
The code is not ideal, and I will be submitting a PR to improve it, I just want to get a working POC in, so that MS2 is not blocked.
##Verification
make local-setup MCTC_WORKLOAD_CLUSTERS_COUNT=1
make kind-load-syncer
export KUBECONFIG=./tmp/kubeconfigs/mctc-workload-1.kubeconfig
make deploy-syncer
export KUBECONFIG=./tmp/kubeconfigs/mctc-control-plane.kubeconfig
kubectl create namespace mctc-tenant
kubectl create -f config/samples/gateway.yaml -n mctc-tenant
kubectl annotate gateway example-gateway mctc-sync-agent/all=true -n mctc-tenant
kubectl get gateway example-gateway -n mctc-tenant -o yaml
See that the spec is added as an annotation to the gateway object.

ping @david-martin @R-Lawton @sergioifg94

First pass trying out the PR on its own looks promising.
I'm going to rebase #59 on this branch and see if I can get it integrated before giving the thumbs up.
1 thing I did spot already though is the status annotation name.
mctc-status-syncer-status-kind-mctc-workload-1
For Gateways I want to easily get the statuses of all gateways in all clusters it has been synced to.
The naming convention makes it a little trickier than I'd like i.e. I'll have to know which clusters the gateway was synced to, then construct the annotation names from them to pull out each status.
Alternatively I could loop through all annotations and look for matches on the annotation name prefix.
It's doable, but I'm wondering if it can be made easier?
I don't recall how it was done for kcp, but if its the same pattern, that's ok, we can leave it as is (as I suspect they did it that was for a reason)

First pass trying out the PR on its own looks promising. I'm going to rebase #59 on this branch and see if I can get it integrated before giving the thumbs up.
1 thing I did spot already though is the status annotation name. mctc-status-syncer-status-kind-mctc-workload-1 For Gateways I want to easily get the statuses of all gateways in all clusters it has been synced to. The naming convention makes it a little trickier than I'd like i.e. I'll have to know which clusters the gateway was synced to, then construct the annotation names from them to pull out each status. Alternatively I could loop through all annotations and look for matches on the annotation name prefix. It's doable, but I'm wondering if it can be made easier? I don't recall how it was done for kcp, but if its the same pattern, that's ok, we can leave it as is (as I suspect they did it that was for a reason)

In KCP the labels (we use annotations) used to mark a resource as should-be-synced for a particular agent (i.e. mctc-sync-agent/mctc-workload-1) had a value which was set to a value by the syncer to tell the control plane how the syncing was progressing (e.g. "Synced"). As it happens we didn't use that, as I recall in GLBC we just iterated through all the annotations that had the status prefix.

I'm going to rebase #59 on this branch and see if I can get it integrated before giving the thumbs up.

Done. This helped unearth a couple of issues with the gateway reconcile logic, which If added fixes for to #59.
Approving based on functionality


Criação HTML

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


fix(regex): remove substring caching & add genMatches caching

PR description
Describe your changes in detail here
This removes persistent caching of substrings and adds caching of generated Matches (for approximate regex matching)
Definition Of Done (DoD)
This PR can be squashed / merged if

[x] a developer is assigned
[x] the PR is NOT estimated
[x] the PR is labeled
[x] the PR is assigned to the current sprint
[x] a meaningful title has been set according to https://www.conventionalcommits.org/
[x] the PR is described in detail
[x] the PR links to an issue
[ ] the PR has been reviewed

Add additional conditions here if necessary for this PR
fixes #65

Lgtm

accepting this as reviewed


Unsupported Network

Error when "connect wallet" while the wallet is connected to an unsupported network. Throws Unhandled Runtime Error
To reproduce:

Refresh
Set wallet to network like Polygon
Connect wallet

Deliverables

[ ] Design flow when connected to unsupported network

https://www.figma.com/file/YWU3PMsnqZaqr1gABgAFJK/UI-Refresh?node-id=1959%3A15788


Add support for Windows ARM

Not sure if it is easy or not to make support for a new architecture without a device, but I'm ready to help with testing :-)
https://www.windowscentral.com/hardware/laptops/surface/project-volterra-everything-you-need-to-know

For https://github.com/BashSupport-Pro, I was able to successfully build winpty for Windows ARM64:

I used Visual Studio 2022 Preview, https://visualstudio.microsoft.com/vs/preview/
Changes for Winpty at https://github.com/BashSupport-Pro/winpty/pull/1 . The build was working locally on an ARM device, but may not work on other devices (yet).
The DLL seemed to load fine on the Windows ARM device (on a JBR 17 build for ARM64 with IU 2022.2), but I'm not sure if there are tests somewhere

Related:
For pty4j, building win-helper.exe was also possible.
But I couldn't find a Cygwin build for Windows Arm64 to build the cygwin helper.

VS 2017 Community on x64 host is also enough to cross-build ARM64 target. You'll need some additional (ARM64) components, though.
@bulenkov we're using our fork with some additional stuff JetBrains/winpty/get_current_directory for quite long time already.
So I've added one more branch on top of that — JetBrains/winpty/get_current_directory_arm64, built it and used in pty4j 0.12.10


Camera will not show fullscreen

Here is the issue you asked me to create.
and part of my config that contains the cameras.
pages: [
      {
         title: 'Main page',
         bg: 'images/bg1.jpeg',
         icon: 'mdi-home-outline', // home icon
         head: 'head.html', // clock
         groups: [
            {
               title: '',
               width: 2,
               height: 3,
               items: [
                  {
					 position: [4, 0],
					 id: 'camera.koket',
					 type: TYPES.CAMERA_THUMBNAIL,
					 bgSize: 'cover',
					 width: 2,
					 state: false,
					 fullscreen: {
					 type: TYPES.CAMERA,
					 refresh: 1500, // can be number in milliseconds
					 bgSize: 'contain'
					 },
					 refresh: function () { // can also be a function
					 return 3000 + Math.random() * 1000
					 }
				  },
				  {
					 position: [2, 0],
					 id: 'camera.clara',
					 type: TYPES.CAMERA_THUMBNAIL,
					 bgSize: 'cover',
					 width: 2,
					 state: false,
					 fullscreen: {
					 type: TYPES.CAMERA,
					 refresh: 1500, // can be number in milliseconds
					 bgSize: 'contain'
					 },
					 refresh: function () { // can also be a function
					 return 3000 + Math.random() * 1000
					 }
				 }

               ]
            },

Try to replace type: TYPES.CAMERA, with type: TYPES.CAMERA_THUMBNAIL,

That worked. Thanks Evgeny.


[BUG] Instagram requires login

Instagram's API is now much more aggressive in requiring users to be logged in, which effectively renders Instamancer useless.
Instamancer is designed to be used for gathering publicly available data, and will not add the ability to log in to retrieve data.

@ScriptSmith a while back I saw a fork that had login detection (and supported a credentials file that would be used when that login WAS detected.
I never took it for a spin but needless to say the login detection would not be required for most of what Instamancer does.
I'm not sure if you are discontinuing support due to the fact that beating the login screen is a little more of a grey area, but if you guys are interested there is at least some code to start with.
At some point I will probably give it a go but for anyone who is interested I believe this fork supports login detection and use of a specific set of user credentials: https://github.com/IORoot/instamancer

Try my extension: https://github.com/HOAIAN2/Instagram-Downloader
I use native Instagram API ( I got it from Chrome network tab).
They have 2 API, one for get all user's info (query hash 69ca..) and one for get data form post like photo, video, comment ( query hash 98ff..)
Note: both API still work in Nodejs but they will send back login page later.


Can't use MediaPipe in VisionOS development

OS Platform and Distribution
VisionOS 1.0/1.1
Compiler version
No response
Programming Language and version
Swift
Installed using virtualenv? pip? Conda?(if python)
No response
MediaPipe version
10.12
Bazel version
No response
XCode and Tulsi versions (if iOS)
15.3
Android SDK and NDK versions (if android)
No response
Android AAR (if android)
None
OpenCV version (if running on desktop)
No response
Describe the problem
We want to use MediaPipe on visionOS, we just can't, I've tried with cocoapods, imports do not work if the projects are from different platforms. And I don't know how to build it for visionOS using bazel.
Complete Logs
It just doesn't work.

Hi @boroicamarius,
Thank you for bringing this to our attention. It appears that we have not yet tested this scenario, and its implementation as a new feature will depend on demand. To expedite the process, could you please provide some specific use cases in VisionOS that would be helpful for us to consider when implementing this feature?

We would like to extract facial features from the persona that the vision pro gives us, we only have access to a 2d projection of it and we need accuracy to understand how the face moves. Other than that, if there is no demand but we would like to implement it on the vision pro, could you give us a scheme to understand how the code works and how we could create a new build target using bazel?

I'm also trying to use MediaPipe on VisionOs for one of my projects. It would be very helpful if the model could extract facial features from the Persona.

Hi @boroicamarius, @Razvi103,
Thank you for the thorough explanation. Integrating MediaPipe into VisionPro seems like an interesting feature request. We will mark it as feature request for further consideration and bring it to the attention of our team internally.
Thank you!!


[Docs] inter-cluster-network update

Description
This PR fixes the inter-cluster-network page

/merge


[Sema] Correct lvalue computation in buildStorageRef for synthesizing local accessors.

This is a follow-up to the local property wrapper implementation to address https://github.com/apple/swift/pull/34109#discussion_r496998875
The fix turned out to be pretty simple - the lvalue adjustment for the storage reference base simply needed to be moved earlier and slightly adjusted to account for cases where the base is not self.

@swift-ci please smoke test


mining: always returns .coinbasevalue in getblocktemplate

Fix #96
Although the BIPs specify that coinbasetxn and coinbasevalue are mutually exclusive, both the latest bitcoind (22.0.0) and lbrycrd (0.17.3) returns .coinbasevalue regardless if 'coinbasetxn' is specified in the capabilities.
We'll make lbcd behave the same for compatibility.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
287 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.009%) to 51.171%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

rpcserver.go
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

peer/peer.go
1
76.24%

btcec/signature.go
6
92.02%

rpcserver.go
280
0.26%

Totals

Change from base Build<PHONE_NUMBER>:
0.009%

Covered Lines:
22479

Relevant Lines:
43929

💛  - Coveralls


renovatebot(deps): update spring data to v2.2.0.rc1

This PR contains the following updates:

Package
Update
Change

org.springframework.data:spring-data-commons
patch
2.2.0.M4 -> 2.2.0.RC1

org.springframework.data:spring-data-mongodb
patch
2.2.0.M4 -> 2.2.0.RC1

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR is stale, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about these updates again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.null seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


updated bioavailability_ma_et_al

Updated the templates for this task to improve grammar and that this data is only for the bioavailability via oral pathway.
Note: Oral bioavailability is not really a binary task, it should be a fraction. This paper classified oral bioavailability as “positive” if its bioavailability≥ 20%, otherwise “negative” because they needed it to be binary for their SVM model to do classification.
This means that we might need to think more about this task, some options:

Leave it as it is right now
Use the raw data instead of the SVM binary data: this would change it to a continuous task: http://modem.ucsd.edu/adme/databases/databases_bioavailability.htm
Change the templates to be a bit more vague but stick with the existing data e.g. instead of this SMILES 'is orally bioavailable', we could say this SMILES has 'high oral bioavailability', or 'is likely to show oral bioavailability'.
Happy to discuss what we think is best :)

Wow! That's a detailed analysis 🕵🏽♀
personally, I'd go with the raw data as we could then also do the regression tasks (as you said)

Yeah, I guess the idea from Jack would be the best approach to work with the data from the current PR and add the regression dataset as a new one.
In the templates we should only need to change is {bioavailable#not &NULL}{bioavailable__names__adjective} to has a {bioavailable#low &high }{bioavailable__names__noun}.
I'll check that. :-)

PR https://github.com/OpenBioML/chemnlp/pull/388 adds the proposed changes from this PR.
Issue https://github.com/OpenBioML/chemnlp/issues/389 covers the addition of the raw data in a new tabular dataset.


kernel: fix runtime initialization of k_pipe object

Runtime initialization failed to reset the lock field, causing
problems when the pipe object is located on a stack and passed by
reference to other code.  Lacking an API for initializing a spinlock
by itself use the idiom from _K_PIPE_INITIALIZER().
To simplify maintainability the initialization order is changed
slightly to match the structure field declaration order.
Signed-off-by: Peter Bigot<EMAIL_ADDRESS>

@andyross I'm unaware of in-tree uses.  There is a use in the test provided by #24486, where I encountered it during review posted in the original issue; also the author of that PR asked about the fault on slack which reminded me of the failure and its solution (in that situation, use the existing statically allocated pipe).
I'm not in favor of allocating pipes on a stack, but in restricted situations it might work and be useful, so it seems worthwhile avoiding an uninitialized-memory-reference if it happens.

This fixes some issues I've observed. E.g. if a k_pipe is k_malloc'ed or if it's on the stack as well.


Jesse: Watch domainseotools.com

Jesse requests the watch of the watch_keyword domainseotools\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

domainseotools\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


APY Calc Adjustment for OHM-wstETH Aura

Adjust calc method for OHM-wstETH Aura APY

Looks good to me