[Remote cluster state] Few custom metadata not getting persisted on storage which dont support async write.

Describe the bug
We rely on writeAsync method of ChecksumBlobStoreFormat to write the global metadata to remote store. writeAsync method internally calls the write method for the cases where async write is not supported. In write method we have hard coded the Snapshot context, so few of the custom metadata will not be persisted in remote store for the store where async write is not supported.
https://github.com/opensearch-project/OpenSearch/blob/e942483e716255b8f801dc1242f909d73019aa70/server/src/main/java/org/opensearch/repositories/blobstore/ChecksumBlobStoreFormat.java#L174
Fix
We will need to expose XContentContext parms in write method as well and pass the context of remote store from writeAsync method to write method.
Meta issue: #10523

Duplicate of : https://github.com/opensearch-project/OpenSearch/issues/10691

Duplicate of : https://github.com/opensearch-project/OpenSearch/issues/10691


Enhancement Request - User Segmentation

Hello!
The idea is to create a dynamic list with the following conditions -

All users created in the last 2 months. OR
All users who have opened emails in the last 3 months.

This will ensure that the pool of users who receive the weekly newsletters remains fresh - the open rates, etc., are always in good shape.
I tried creating this Dynamic list:

The created date as of now can be fixed. Could we have an option like "before y months / before x days"
The email open event can be linked to a specific campaign. Can it be tracked at an overall level - eg. Email opened in last x months.

Request you to please see if this fits into your roadmap.

I commented the need for something like that here: https://github.com/parcelvoy/platform/discussions/514#discussioncomment-10974319

This functionality will be available in the next release, please see #538 for details about the implementation

Hi @pushchris Thank you for working on this. Just curious - when can we expect the new release to be available?


Would like to change publicPath

Out of the box the publicPath is public/webpack/, I tried changing it to public/js, which compiles fine into that location but it does not seem to find it there.
ActionView::Template::Error (Could not load compiled manifest from /app/public/webpack/manifest.json - have you run rake webpack:compile? (original error No such file or directory @ rb_sysopen - /app/public/webpack/manifest.json)):
Is there anything I'm missing? Is it possible to change the output location?

Yeah, the rails configuration should also point to the correct path, (in my case, we use public/compiled)
e.g.
Rails.configuration.webpack.output_dir = 'public/compiled'
Rails.configuration.webpack.public_path = 'compiled'


Gjoranv/split parent3

Second retry for #4304.
New this time: Move all deps using properties from container-dependency-versions up to that pom. There is a comment explaining why in the pom. I've also done the necessary changes in factorylib to account for having two levels of parent poms.

👍


undefined method `default_timeout=' for Watir:Module (NoMethodError)

I get this error:
undefined method `default_timeout=' for Watir:Module (NoMethodError)
For this code:
Watir.default_timeout = FigNewton.watir_default_time
However it only happens when using routes. With this code:
require_rel 'pages'
PageObject::PageFactory.routes = {
....
}
If remove the routes code it works fine but then I can't use routes. If I leave the routes I must then comment out the default_timeout or I get the nomethod error. Seems I can't use both together.

Has this been resolved or is it still an issue?

I am closing this due to lack of activity. If you still have an issue or you have more questions, you can post here or also you can also ask on Stack Overflow using the page-object-gem tag.


[Debugger] Introduces the unsuspended remote debugging capability

Purpose

$subject

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

@SumayyaZiyad / @NipunaRanasinghe Do we need this PR? Looks like it got automatically closed without any attention. If you really don't need this PR anymore, please remove the stale label from the PR. Then it won't get added to the [Stale PR] Ballerina Stale PR Details mail in the future.

@anupama-pathirage Apologies for the late response. Yes we had few concerns on this PR, since there were some debugger design-level changes (i.e. modify breakpoint verification process to work with unsuspended debugging mode) were required to make this work. But we didn't have enough time for that, as sumayya had to commit to the project design tool related work.
Therefore we don't have plans to bring this back anytime soon so I'll remove the stale label for now.


Error if home directory isn't writable

I've recently noticed that pandas_gbq requires that a user's home directory exist and be writeable in order to be used. This means that it cannot be used in restricted environments, where that may not be the case.
I think this is a result of the recent change (#241) to use https://github.com/pydata/pydata-google-auth for auth.
I've raised the underlying issue as https://github.com/pydata/pydata-google-auth/issues/10, however from what I can tell pydata_google_auth is only actually used in the case where the user doesn't provide credentials as part of an operation.
In particular, I'm using a service account (where you're more likely to be in a restricted environment too), and always provide the credentials upfront anyway.
It would therefore be great if pydata-google-auth was a soft requirement rather than a hard one -- both removing it from the requirements of this library and from the checks in _test_google_api_imports, or making those checks aware of whether credentials have been explicitly provided.

I've fixed the issue and released https://pypi.org/project/pydata-google-auth/0.1.2/ in pydata-google-auth.


composer install install wordpress plugins to vendor dir

Trying to install wordpress plugins with composer and wpacakagist
during composer update everything works fine - plugins installed in wp-content/plugins directory
but when i use composer install later - plugins are installed again but into vendor/wpackagist-plugin
Am I forgot something?

Hey @jacobmaster,
You might want to checkout https://roots.io/using-composer-with-wordpress regarding this.


ReadTimeout issue

Hi,
When I try to parse all 10-K for the period 2022-12-15 to 2023-12-14, after about one hour, the following error is shown -
ReadTimeout: The read operation timed out
File c:\Users\mshar\OneDrive - Southern Illinois University\POSITRON_different_files\ParseTenK\Lib\site-packages\httpx\_transports\default.py:69, in map_httpcore_exceptions()
     68 try:
---> 69     yield
     70 except Exception as exc:
Show Traceback

Do you have any idea why this is happening?

Not sure, send me your code.

Hi, I use the following code to extract metadata and ITEMS from 10-K. Thanks
# Loading Necessary packages
import pandas as pd
import numpy as np
from edgar import *
import json
# to install updated version of a package
# pip install sec-parsers -U
from sec_parsers import Filing, download_sec_filing, set_headers

# to see all installed packages and their version
# pip list
# Tell the SEC who you are
set_identity("My Name<EMAIL_ADDRESS># edgartools
set_headers("My<EMAIL_ADDRESS># sec-parsers

# Fetch filings
filings = get_filings(form="10-K", filing_date="2022-12-15:2023-12-14", amendments=False)

# Convert filings to a DataFrame
filings_df = filings.to_pandas()

# Create a list to store the Item 1c text
item1a_texts = []

# Iterate over each filing
for n, filing in enumerate(filings):
    url = filing.document.url
    cik = filing.cik
    filing_date = filing.filing_date,
    reporting_date = filing.header.period_of_report,
    comn = filing.company

    # Download and parse the filing
    html = download_sec_filing(url)
    sec_filing = Filing(html)
    sec_filing.set_filing_type('10-K')
    sec_filing.parse()

    # Extract the text for Item 1c
    item1a = sec_filing.find_section_from_title('Item 1A')
    item7 = sec_filing.find_section_from_title('Item 7')

    if item1a is not None:
        item1a_text = sec_filing.get_text_from_section(item1a, include_title=True)
        subsections_item1a = sec_filing.get_nested_subsections_from_section(item1a)
        subsections_item1a_title = [item.attrib['title'] for item in subsections_item1a]

    else:
        item1a_text = None
        subsections_item1a_title = None

    if item7 is not None:
        item7_text = sec_filing.get_text_from_section(item7, include_title=True)
        subsections_item7 = sec_filing.get_nested_subsections_from_section(item7)
        subsections_item7_title = [item.attrib['title'] for item in subsections_item7]

    else:
        item7_text = None
        subsections_item7_title = None

    # Bypass None
    #try:
        #item1c_text = sec_filing.get_text_from_section(item1c, include_title=True)
        #subsections_title = print([item.attrib['title'] for item in subsections])
    #except:
        #item1c_text = None
        #subsections_title = None

    item1a_texts.append({
        'CIK': cik,
        'Filing Date': str(filing_date),
        'Item 1a Text': item1a_text,
        'subsections_item1a_title':subsections_item1a_title,
        'Item 7 Text': item7_text,
        'subsections_item7_title':subsections_item7_title,
        'url': url,
        'comn': comn,
        'reporting_date': str(reporting_date)
    })

# Create a DataFrame from the Item 1c text data
item1a_df = pd.DataFrame(item1a_texts)

Hmm, that's definitely not related to SEC-parsers. I'm not sure what's going on, do you have a longer bug report?

Hi, I don't have the longer bug report as I restart the kernel. When I will run it again, I can share the longer bug report with you. Thanks


🐛 Fix missed update of df property when resetting the dataframe

This would only update the hdf5 file in the repository but not the self._df attribute which would be used when accessing the df property on the unstored node.
Also simplified and tested behaviour for using a non-default filename of the dataframe

Codecov Report

Merging #5 (710ecdc) into main (981842f) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main       #5      +/-   ##
==========================================
+ Coverage   99.03%   99.08%   +0.04%
==========================================
  Files           4        3       -1
  Lines         104      109       +5
==========================================
+ Hits          103      108       +5
  Misses          1        1

Impacted Files
Coverage Δ

aiida_dataframe/data/dataframe.py
100.00% <100.00%> (ø)

aiida_dataframe/__init__.py

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


Update scalajs-react:core, ... to 1.7.0

Updates

com.github.japgolly.scalajs-react:core
com.github.japgolly.scalajs-react:extra
com.github.japgolly.scalajs-react:test

from 1.6.0 to 1.7.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.japgolly.scalajs-react" } ]

labels: library-update, semver-minor

Included in #129.


Enhancement: Improve codeviewer example:  No protrude splittable both side .

In the "vertical splitter" example currently used in codeviewer, the left side does not protrude due to the minimum size specification, but the right side has no limit and will protrude outside the window.
I'd like to improve this, but I'm wondering if you have any good examples.
Sorry Im poor English end use machine translation sorry.

If this is not appropriate here, I would be grateful if you could guide me to the right place.

How to get parent size in composable function? If exist, coerceAtMost() is work it.

You can look at this:
https://github.com/JetBrains/compose-jb/commit/9e67cfc916f696220d630ceb6e5b67e9570777a3
But it is not finished. If we resize main window then we will see a small lag (size of panel will not be immediately changed)

Getting size composable itself to use onSizeChanged() and size-state is workaround? Allow it normally?

Getting size of composable itself to use onSizeChanged() and size-state is workaround? Allow to use it normally?

Probably the lag when we resize the main window caused by onSizeChanged (because the panel will be resized only on the second frame).
Probably we need to constrain sizes of panels not inside onSizeChanged, but inside VerticalSplittable function

Oh, sorry. Apart from this issue, I wanted to ask if this usage is normal. Probably because there are going to be many situations where you want to know the size of itself.

I wanted to ask if this usage is normal

Probably in most cases we should use Layout or WithConstraints
Because we will know the size of the container immediatly (in the current frame).
But in the case it is not importand to know the size in the current frame, we can use onSizeChanged and size-state.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Wordcounter

So, i have a new simple idea - add word, symbols and lines counter a the right-bottom of interface of redacting.

sounds good 😄


Add NEW_VERSION option

Changes made:

Update repo and version (v missing) in the README for the action example
Use --new-version arg to specify your own version instead of letting bumpversion determine that for you. If unspecified the action will work as before
Added .hadolint.yml to ignore pin pip / pin apk packages warning.

The reason for adding NEW_VERSION is I'd like to use this to action update all the files that need a version update only, but the bumping of the version I'll be using something different which will always bump version from the latest git tag.

Nice! Thanks for this


Fix get_tr docstring: returns transfers, not bequests

@MaxGhenis Thanks for fixing this typo.  It looks like a C/S tests is failing due to the version bump for Tax-Calculator, which now uses ParamTools.  Do you want to try to fix that here or shall I open another PR to fix?

I'm not too familiar with ParamTools so might be better to fix that in a separate PR.

Hey @MaxGhenis and @jdebacker, @Peter-Metz and have started going through the process of updating some of the cs-config code for Tax-Cruncher here: https://github.com/PSLmodels/Tax-Cruncher/pull/38.
Tax-Cruncher doesn't have the CPS/PUF distinction so it's a little more straightforward. I'm planning to do the update to account for some parameters only being compatible with one of the files tomorrow or Wednesday.

Thanks for the pointer, @hdoupe.

Hi all, I've started working on converting Tax-Brain's cs-config package to use ParamTools: https://github.com/PSLmodels/Tax-Brain/pull/128
I moved most of the inputs processing code into the cs_config/inputs.py file:

convert_policy_defaults: Checks the section_1 header and data source selection to determine what defaults should be returned to C/S.
convert_policy_adjustment: Converts parameters from some_param_checkbox to some_param-indexed.


Displaying an useable accordion with expansion panels generated
out of dynamic data fetched by a method in a *ngFor-loop
What is the current behavior?
fires method infinitely until process ran out of memory and freezes
What are the steps to reproduce?
https://stackblitz.com/edit/angular-material2-issue-esus4p
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular CLI: 7.1.2
Node: 10.10.0
OS: win32 x64
Angular: 7.1.2
... animations, cli, common, compiler, compiler-cli, core, forms
... http, platform-browser, platform-browser-dynamic, router
... service-worker
Package                           Version
@angular-devkit/architect         0.11.2
@angular-devkit/build-angular     0.11.2
@angular-devkit/build-optimizer   0.11.2
@angular-devkit/build-webpack     0.11.2
@angular-devkit/core              7.1.2
@angular-devkit/schematics        7.1.2
@angular/cdk                      7.1.1
@angular/material                 7.1.1
@ngtools/webpack                  7.1.2
@schematics/angular               7.1.2
@schematics/update                0.11.2
rxjs                              6.3.3
typescript                        3.1.6
webpack                           4.23.1
Is there anything else we should know?

This is how change detection works and it doesn't have anything to do with Material. Angular has to re-run the getPanels function on each change detection cycle in order to know whether something has changed.


Add git  and github documentation in the course

The course does not have the clear explaination of the git and github topic , and the link is also not redirecting to the page , please add the documentation on git and github for the beginners to understand the concepts of the git and github . A clear and concise explaination on git and github is required from the beggining to the advance concepts

Add Comprehensive Documentation on Git and GitHub done

Can I please be assigned to this issue?

Hi, @AdityaSrivastava185 can i also collaborate on resolving this issue?

Hi, @AdityaSrivastava185 can i also collaborate on resolving this issue?

Sure @yashksaini-coder  you also can collaborate on this issue , just implement the changes and make a PR

Can I please be assigned to this issue?

Hi @favourachara07 you can improve the issue and then make a PR , for now it is open and collaborative to everyone

Hey @yashksaini-coder
I would like to work on the topic, git if that's okay with you

Hey @yashksaini-coder I would like to work on the topic, git if that's okay with you

That's absolutely alright 🙂 @favourachara07

Sorry for asking this but I’m a new to open source
Should i create a branch separate from @yashksaini-coder or we will share a single branch for the issue

Sorry for asking this but I’m new to open source Should i create a branch separate from @yashksaini-coder or we will share a single branch for the issue

Yes , You should create a new branch then make changes and make a PR , i suggest to go through Contributing-Guidelines before making changes and making a PR

Thank you

Thank you

Happy to help you , Happy Contribution ✨

Okh I will create a new branch that will contain the beginners level guide of GitHub. It will be a complete different section. For which I will be raising an entire new issue. That way @favourachara07  and I both can tackle the problems very easily while also following the contribution guidelines.
Is this okay for you... ?
@favourachara07
@AdityaSrivastava185

Okh I will create a new branch that will contain the beginners level guide of GitHub. It will be a complete different section. For which I will be raising an entire new issue. That way @favourachara07 and I both can tackle the problems very easily while also following the contribution guidelines.
Is this okay for you... ? @favourachara07 @AdityaSrivastava185

Yes 👍

Good then. We will proceed this way.

Great Work @favourachara07 , Merged this issue but still there are lot of changes to improve this , So keep contributing

Please could you tell me where i can make changes  so that i can know and make improvements
Thank you

@favourachara07  , currently you can add multiple images , also some git and github working concepts to  , multiple github programs and perks , make the documentataion clean and easy to learn step by step for those people who wants to learn on the way and implement it

okay
so i should make these changes in my branch
then i will make a pr like before

Yes

Here is a message you can use to ask the repository owner to assign the issue to you:

Hi @AdityaSrivastava185,
Could you please assign this issue to me? I would like to work on it.
Thank you!
Best regards,
@yadavmangesh07


chore(main): release pixels 0.8.1

:robot: I have created a release beep boop
0.8.1 (2024-06-04)
Bug Fixes

deps: update nextui monorepo (98b9382)
pixels: accordion required props (026aa17)
pixels: allow rendering accordion items as react children (8251313)
pixels: use storybook useArgs in Modal story (19c656e)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/fuf-stack/uniform/releases/tag/pixels-v0.8.1 :sunflower:


Added js test

Added a test task which currently only holds a js-test task. This subtask uses the following plugins:

gulp-jshint
gulp-jscs

Both plugins use an external configuration file placed in the root of the project. These files can also be used by an IDE to provide live feedback when editing javascript files.

Cleanup tabs and space across the entire project to match the preferred indention of 4 spaces.

Thanks! JSCS didn't work with ES6 syntax yet, so I needed to fix that as well in d79d2d8.


install secure version of urllib3

This LGTM, can we merge it?

CI might be failing for real reasons here.

Whoops, I thought I'd deleted this. I actually abandoned this PR for a few reasons:

I don't think it's that important. It just fixes the error:

/usr/local/lib/python2.7/dist-packages/pip/_vendor/requests/packages/urllib3/util/ssl_.py:122: InsecurePlatformWarning: A true SSLContext object is not available. This prevents urllib3 from configuring SSL appropriately and may cause certain SSL connections to fail. You can upgrade to a newer version of Python to solve this. For more information, see https://urllib3.readthedocs.io/en/latest/security.html#insecureplatformwarning.

I don't actually see how to fix it given the constraints of CI. AFAICT, to upgrade urllib inside of travis, I'd need to pip install urllib, which has cryptography as a dependency. Installing cryptography (which has C/C++ components) in turn requires an apt-get update/upgrade which doesn't finish inside the runtime constraint for Travis tests

Even if I could figure out how to make this work, I think it would add a lot to our CI runtime (because all of these dependencies would have to be installed every time we ran CI), so I'm not sure it's worth it. I'm just going to delete the PR


Multiple optical files

Added support for multiple optical files in a zip folder.
Checked support on the pulse3D dashboard = supported.

Depending on if Jason wants this released or not, you might need to bump the version and changelog since it tries to publish each merge!


Sending hangouts message to a uProxy user kills the freedom-social-xmpp module

To replicate, have Alice sign into uProxy, then have Bob (who is already a contact of Alice) send a hangouts message to Alice that is not a JSON encoding string (e.g. "hello").  You will see that the freedom-social-xmpp module fails (uncaught exception trying to do JSON.parse).
Quick fix:

Add a try/catch around JSON.parse.  If JSON.parse fails, we should probably just emit 1 onMessage event with the incoming (non-parsed) message

Additional cleanup:

We should only batch messages in JSON format when sending to an "ONLINE" (i.e. ONLINE with uProxy, not ONLINE_WITH_OTHER_APP) user.  This will allow ONLINE users to communicate to ONLINE_WITH_OTHER_APP and remain human readable

https://github.com/freedomjs/freedom-social-xmpp/issues/97

@ryscheng - Currently, when sending with message type 'chat', we are batching the messages but separating with a '\n'.  Do you think this is fine to keep?  I will still be adding the try/catch so we don't fail when receiving non-JSON messages.

That seems fine to me

Fixed in https://github.com/freedomjs/freedom-social-xmpp/issues/99


SUPRAMATIC HT3

Hi,
I have Supramatic HT3 anc cannot reach the communication...
I am using ESP32-dev board with this connection:

there are no incoming / outcoming messages... how to check if connection between BUS and RS485 is good?
because I am not sure about the pinout of the line... just checked, that I have already 24V and Ground

I found, that for my supramatic is different connection :

so finaly it works and I get this logs:


Type safety feature requests collection

Version 0.1 of the type safety setting was completed and can be found in pull request #59.
However, the feature is not done yet, some things we would still like to add:

[ ] Return type checking
[ ] Generic functions e.g. fun(in: T[]): T
[ ] Typed code completion
[ ] Field assignment type checks

Additionally, there are currently some workarounds in the code to negate some missing features of the original plugin functionality.
This is a list of features we would like to see to continue developing the type safety sytem:

[X] Parse 'any' in docs as TyUnknown (issue #58)
[X] Parse primitives in docs as proper primitive types
[X] Parse 'nil' in code as TyNil type instead of TyUnknown
[ ] Add --- @override annotation that copies the doc of the function from the closest superclass.
[ ] Parse negative numbers as type number, not as type any.
[ ] Allow defining operator overrides, ie --- @param o T --- @return T function T:__add(o) to specify the result of a:T + b:T is of type T.

Additionally there are some more complex features we would like to see eventually, but are not really a priority right now:

[ ] Allow for generic function specification in docs, something like --- @generic T, --- @param x T, --- @returns T[]. I want to be able to implement function map<T,U>(T[], fun(x: T): U): U[]
[ ] Some method to append to the docs of already specified functions (for example in the standard library). I am not sure how this would look exactly.

More issues might arise later, but for now having the top list would help us out greatly.

Figured out how to do parsing of nil and primitives into their proper class types, updated top comment.

I haven't looked in detail yet, but are userdata and thread (coroutines) types handled by that?
Anyway, good job, it looks nice :)

I did not take userdata and thread into account, I don't have any experience or testcases with those in lua. I looked them up in the lua docs and they are official types, so I think they should also be considered primitives, would you agree?

Yep, if possible :)

Will add them later today!

No, I think userdata is just a runtime type, it also can be a defined doc-type, not same as primitives

I suppose that as long as it works with type annotations, it's fine for me.
Technically at "compile"-time (I mean, we know the type before execution, anyway), a userdata can be obtained with newproxy(). But indeed it's not something like "foobar" that's directly of type string.
Same goes for threads when creating coroutines.

According to https://www.lua.org/pil/2.html userdata and thread are proper lua types, therefore I would say implement them the same we I have implemented table and function primitives, that serve as a kind of 'any' type for tables and functions.
I.e. fun(a: number): string is a subtype of 'function' (TyPrimitive(PrimitiveKind.Function))
The system allows more specific types to be defined with these as superclass.

After thinking about this for a day I have to side with @tangzx on this, I will try to explain my reasoning:
While it is true that thread and userdata are considered as their own type in lua, there are (as far as I know) no special ways to declare them, i.e. function() end for type function, {} for type table. Every instance of these types will therefore originate from some form of API call. Therefore considering these two types as just regular classes will suffice in this case, assuming the methods spawning them are properly annotated (spoiler: the standard library is NOT properly annotated for this).
Therefore I think that for now thread and userdata will be considered just instances of the class type, unless a good reason is found not to do so.

A good feature would be prioritizing variables of required type in autocomplete when typing an argument


[WIP] Configuration Overhaul

Fixes #47
Related PR: #68

Pull Request Test Coverage Report for Build 1662

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 62.793%

Totals

Change from base Build 1655:
0.0%

Covered Lines:
562

Relevant Lines:
895

💛  - Coveralls

This Pull Request was merged without enough reviews
Please go here to review and +1


New TAC OSL member

Add Chris Kulla as the TAC representative for Open Shading Language

Thanks JF.  This includes the TAC notes as well, so I'll merge this PR and discard the other.


18668 - External influence section on add SI component.

*Issue:18668 *https://github.com/bcgov/entity/issues/18668
Description of changes:

Added section for "external influence" on add individual page
added info on summary table

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

Temporary Url for review: https://bcregistry-btr-dev--pr-110-7rvwfzek.web.app

/gcbrun

Temporary Url for review: https://bcregistry-btr-dev--pr-110-7rvwfzek.web.app

/gcbrun

Temporary Url for review: https://bcregistry-btr-dev--pr-110-7rvwfzek.web.app


Temporarily disable reindexing.

Coverage remained the same at 89.037% when pulling 2b2c14046bce300b84ced26e9bf7c44a3191a6f8 on akatsoulas:disable-reindexing into a748fd7cb2e533fe6b92d7a41f6e065e65e7aa14 on mozilla:master.


[K32W] Use a custom Persistent Data Manager

This commit add support for a custom PDM (Persistent Data Manager) which is also used inside Project CHIP.
Signed-off-by: Doru Gucea<EMAIL_ADDRESS>

Hi, @jwhui, @librasungirl
There are OT NS tests which are failing. However, the files touched by this pull-request are platform-related so I don't see why those tests (which should be stack-related) are failing. I also checked the OT NS logs from here (https://github.com/openthread/openthread/pull/5496/checks?check_run_id=1077504647) and the message error is "could not find factory to build UDP payload".
Thanks,
Doru


M3U parser ignores EXTGRP directive

Check https://github.com/zikwall/gom3u-content-parser for implementation

Closed with https://github.com/SCP002/xTeVe/compare/4769c35...3704f9a


Release 0.3.3 (or higher)

Not much commits since the last release but I'd like to see #4 and #5 included in a deb. Thank you.

Who is the maintainer of this repository? Also how is it released, since it doesn't have a package.xml?

According to its ros wiki this package is maintained by Ioan and @jacquelinekay (see also the last release request https://github.com/ros/rosdistro/pull/10864).

0.3.3 will be in the next indigo, jade, and kinetic syncs, so I am closing this issue. Feel free to reopen.


Networking mappings

I mostly focused on mapping around the new Configuration feature; this isn't a complete mapping of the network packages, but it gets us pretty close.
These mappings will help with the QSL networking/reg sync port

This is ready for review now.

javadoc error:
/net/minecraft/client/network/ServerInfo.java:275: error: reference not found
    * net.minecraft.network.packet.s2c.play.ResourcePackS2CPacket server

love that it doesn't show up from gradlew build

love that it doesn't show up from gradlew build

gradle build javadocJar baybee

@TheGlitch76 can I polish this up for you, or would you like to come back to it?

Please do!


Adding correct Project files so that eclipse PDE plugins can work

I don't know a ton about this but an example is:
https://github.com/eclipse/lsp4mp/tree/master/microprofile.jdt/org.eclipse.lsp4mp.jdt.tp
we may also need:
https://github.com/eclipse/eclipse.jdt.ls/blob/master/javaConfig.json

fixed by #8


Test robustness

Hey :)
Thanks for this great lib. 😀two questions :

See here

This test is problematic. Among "production = dynamic runtime config" being too much opiniated, there is also a critical trouble with heroku. Let me explain.

The main purpose of your mecanism is to load a "runTime config" over "compileTime" environment variables. This is necessary, on Heroku, in 2 use cases :

When "promoting". The promoted slug embed the compileTime environement variables, which are stales, (because it does not re-run build scripts which usualy hard-copy env variables to source-code inside frontend builds)
But also when dynamicaly changing environement variable without rebuilding the app (through Heroku CLI / user interface inside settings)

This last use case is only covered in production mode with your mecanism, because of that aforementionned test. I personnaly use NODE_ENV=staging on my staging pipeline step, and I'm not willing to change it to production for now
I'd change it to
if (!runTime.matches(/REACT_APP_VARS_AS_JSON_*/))

To have a quick-win. But as suggested here, replacing directly to a literal JS object (not a JSON string) would allow us to only go for :
// It was already replaced
if (typeof runTime !== 'string')

What do you think ?

Please fork this repo and modify it as you desire. This project is not something I’m investing time into anymore.

Roger that, good point


Correct DOMParser parsing of HTML entities in XML

To be consistent with browser implementations, parseFromString should not parse HTML character entities in XML documents.
node sax already provides a strictEntities option. This commit enables this option, ensuring only predefined XML character entities (&, ', >, <, and ") are parsed.
See: https://github.com/isaacs/sax-js/tree/master#arguments
See: https://www.w3.org/TR/REC-xml/#sec-predefined-ent

Chrome 61:

Great catch, thanks. Can you add this to a separate file instead of the dont-upstream.html? That one is supposed to be just a copy of the existing one from upstream but with tests we don't pass commented out.

@domenic Sure. Should that test file also live in the test/web-platform-tests/to-upstream/domparsing/ directory?

Yep!

New tests should pass lint, not be ignored :)

@domenic sorry, I didn't realise that ES5 syntax was ok!

@domenic awesome! thanks


update to yarn 3.6.3

closes #102
update to Yarn 3.6.3 now that Dependabot supports Yarn v2/v3: https://github.com/dependabot/dependabot-core/issues/1297#issuecomment-1285998196
For migration steps see https://yarnpkg.com/getting-started/migration
Also use corepack similar to https://github.com/docker/actions-toolkit/pull/323 to download yarn on-fly.
$ corepack enable
# https://yarnpkg.com/getting-started/install#updating-to-the-latest-versions
$ yarn set version stable
$ yarn --version
3.6.3

Also install plugin-interactive-tools yarn pkg: https://yarnpkg.com/api/modules/plugin_interactive_tools.html
$ yarn plugin import interactive-tools

Why is yarn itself committed to the repository? We don't even store the modules we use in the repo.

Yarn Modern (v2/v3) has some breaking changes with old v1 and therefore requires Yarn releases to be kept versioned: https://yarnpkg.com/getting-started/qa#which-files-should-be-gitignored. This prevents potential issues if, say, two engineers use different Yarn versions with different features.

@tonistiigi Setting yarn version in the Dockerfile works: https://github.com/docker/bake-action/compare/master...crazy-max:docker-bake-action:yarn-update-2#diff-3ed65642ec1610de6b0f162c77dc8499662ebeb7057d3f6c89108952d1cb33deR10-R14 but Dependabot will still use Yarn v1 and corrupt the lock file :disappointed:.
Unfortunately we can't define yarn version in the yarnrc config, just the yarnPath: https://yarnpkg.com/configuration/yarnrc#yarnPath

The yarnPath setting is currently the preferred way to install Yarn within a project, as it ensures that your whole team will use the exact same Yarn version, without having to individually keep it up-to-date.

@tonistiigi Switching back to npm looks to be the only alternative we have if you still don't want yarn installed within the repo: https://github.com/docker/bake-action/pull/102

@tonistiigi Now using corepack to avoid having yarn in the repo.


HotChocolate Subscription + UseFiltering publishes nulls

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
We're using the [UseProjection], [UseFiltering] and [UseSingleOrDefault] middlewares to allow clients to subscribe to notifications by ID.

Current behavior: When a TopicEvent is sent, all subscriptions for IDs that do NOT match will be triggered, receiving a null object.

Expected behavior: When a TopicEvent is sent, all subscriptions for IDs that do NOT match are not triggered.
Steps to reproduce

Set up a HotChocolate endpoint with subscriptions.
Apply the UseProjection, UseFiltering, and UseSingleOrDefault attributes.
Use Banana Cake Pop (or other client) to subscribe to a topic, applying a filter.
Trigger the subscription topic with an entity that does not pass the applied filter.
(Unexpected behavior) Subscription topic is triggered, null object is received.
Expected behavior: Subscription topic is not triggered.

Relevant log output
No response
Additional Context?
No response
Product
Hot Chocolate
Version
12.6.2

Filtering will not work in this way. We can have a look at throwing an exception if filtering is applied on a root field of subscription.

Closing this issue since it does not work this way by design.

You could however create something like this on your own by handling the stream yourself.


HystrixCommand annotation ignored

Thanks for creating this library, it is exactly what we were looking for.
I created a very simple project to test this library.
I trying to text the HystricCommand annotation, but it looks like it's being ignored.
The relevant code chunks are shown below at the end of this message.
I'm trying to use the main method now, but as you can see I set up the code to work also as a servlet for future tests.
When I call the main method out put is:
starting main method
Jan 15, 2016 11:40:53 AM org.springframework.context.annotation.AnnotationConfigApplicationContext prepareRefresh
INFO: Refreshing org.springframework.context.annotation.AnnotationConfigApplicationContext@69d0a921: startup date [Fri Jan 15 11:40:53 EST 2016]; root of context hierarchy
hystrix bean: com.netflix.hystrix.contrib.javanica.aop.aspectj.HystrixCommandAspect@4cc8eb05
starting hystrix method
ending hystrix method
ending main method
So I can see that the spring context has been successfully initialize. When the call the method protected by hystrix I don't see anything happening. It should time out but it doesn't. I'm not a deep spring expert, but it looks like the hystrixApect is not picking up the hystrixCommand annotation.
Any suggestion on how to setup my little proof of concept?
@WebServlet("/greeting")
@Configuration
@ComponentScan
@EnableAspectJAutoProxy
public class MyServlet extends HttpServlet {
@Bean
public HystrixCommandAspect hystrixAspect() {
return new HystrixCommandAspect();
}
public void init(ServletConfig conf){
ApplicationContext context = new AnnotationConfigApplicationContext(this.getClass());
System.out.println("hystrix bean: "+context.getBean("hystrixAspect"));
}
@HystrixCommand(commandProperties = {
@HystrixProperty(name = "execution.isolation.thread.timeoutInMilliseconds", value = "500")
})
public void myHystrixMethod() throws InterruptedException{
System.out.println("starting hystrix method");
Thread.sleep(1000);
System.out.println("ending hystrix method");
}

@dmgcodevil @spencergibb Any ideas on what's happening here?

Where does myHystrixMethod get called?

we can close this. My problem was due to lack of knowledge on the spring aop framework.


Terminal text visually overlaps, not going to next line

reaching the end of the line with characters, terminal just stays on the same line and looks like it's overwritting the existing text but it's just visually overwritting

What it looks like

This happens to me, too, and nothing I've tried has fixed it! Is this possibly because of the use of carriage return escape sequences (\r) in the terminal rather than newlines (\n)? I've been seeing them in terminal.coffee and it made me think there might be a connection.
I've encountered this kind of issue before (in plain ol' xterm) as the result of a poorly-formatted $PS1, but no amount of re-writing of the prompt string in .bash_profile (or any of the other usual fixes) relieves the problem here. That makes me think it might just be an issue with the implementation in terminal.coffee?

Likely another "fix" to be gained with #200

this happens by default with xterm. there might be a way around it, but I would want to avoid any custom hack-ery. just need to dig through the docs and see if there is a nice way of accomplishing this.
Actually—it's probably not xterm that is doing this, but the fact that what is written to the terminal is explicitly what comes from the remote shell. So, there is no line break sent, because why would it be, and so the xterm cursor doesn't know what to do except start over? Maybe? Maybe not. Just need to dig around with it.

see https://github.com/sourcelair/xterm.js/issues/179
Looks like the way to resolve is likely on the server side


Create a new endpoint for the API that returns all organization aliases

Overview
We need to create a new endpoint for the API that returns all organization aliases to be used in Search projects page.
Action Items

[ ] Create a new endpoint that generates a list of all aliases and organization tags that can be used to cross reference which tags go under Organizations/Affiliations and removes them from the topic tags.

Resources/Instructions

Progress:
Blocks:
Availability:
ETA:  update By Sunday

I looked at what @emecas already built. As far as I can tell, this already exists.
https://test-civictechindexadmin.herokuapp.com/api/aliases/
https://test-civictechindexadmin.herokuapp.com/api/aliases/hackforla/
Could someone please link to a figma or other design document for how this is to be used so I can tell if this API fulfills the need or if there might be a better way to present the data.
I would also like to know how we will be building the aliases data. Perhaps the existing endpoints were overlooked because we do not have any alias data (or didn't until I added a couple of entries for Hack For LA)

As @cnk mentioned the endpoint returning all the alias for an specific tag / organization already exists.
I'm wondering if here we are referring to a more specific requirement about all the tags and  `alias' used for one specific organization or project

all tags can be supplied by a GitHub API call
returning all tags and aliases for an organization or project would need a functionality performing the join and filtering between GitHub's tags to aliases

@Olivia-Chiong will follow up @giosce, @cnk and @emecas for solutioning.

Following discussion on 3/14/2021 we have decided to pull from GitHub API and filter through CTI database. Process will be mapped out in #407 Closing this issue now.


refactor(CopyableText): Use clipboard API

Last stop on the update-to-new-api train. Updating scoobie should flow down to everything else that's left (as per a search for CopyableText)

I didn't think it necessarily needed a release, since it doesn't really change any of the functionality. Pretty happy for this to be bundled up with whatever the next actual change is.


Appstudio purge rhtap-demo-component-hqbn

Pipelines as Code configuration removal

Pipelines as Code CI has failed.There was an error creating the PipelineRun: rhtap-demo-component-hqbn-on-pull-request-
creating pipelinerun rhtap-demo-component-hqbn-on-pull-request- in namespace rhtap-demo-xdds-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


Class 'Symfony\Component\DomCrawler\Crawler' not found in OC #424

After Update OctoberCMS to build #424, have this error.

Thanks for the heads-up. This is fixed in version 1.0.2.


Create unlogged database tables in test environment

Context
This theoretically speeds up database oerations.
Not to be used in stable environments like production.

PostgreSQL allows the creation of “unlogged” tables, which do not
record data in the PostgreSQL Write-Ahead Log.

Changes proposed in this pull request
Create unlogged tables in test environment to speed up the test suite.
Guidance to review

Link to Trello card

Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[ ] Required environment variables have been updated added to the Azure KeyVault

I'm not seeing significant improvements here. I will close this PR as I don't feel adding this extra configuration is worth the potential cost of maintenance and debugging in case of an error.


Managed to fix the top bar theme issue

I may still need to work on the switch botton cuz it's bothering me lmaooo


Add grpc proxying to enable bring-your-own-proto

Closes https://github.com/dapr/dapr/issues/3231.
Closes https://github.com/dapr/dapr/issues/3096.
Closes https://github.com/dapr/dapr/issues/3065.
This PR enables users to work with their existing gRPC protos for service invocation, while still utilizing Dapr's service invocation features like mTLS, API auth, telemetry, access lists etc.
Notes for the reviewer
High level changes made in this PR

Access lists functionality moved from pkg/configuration to pkg/acl. This is for the better and ACLs now live in a dedicated and reusable package.
Adding Stream trace middleware for gRPC under grpc_diagnostics.go to support users with gRPC streams.
Added the use of a gRPC transparent handler that listens to requests for services that Dapr doesn't know about (user services). This is done through the user of this package which I've reviewed. Once the review process is done, I'll make another commit to migrate the relevant code to Dapr with appropriate attributions. The reason is to remove the dependency ownership.
Marked this feature as experimental, meaning it's opt-in until it graduates to a stable level.
Added unit tests

Two phase review
In order to keep the list of files changed small for the actual code change, I'm waiting with adding the e2e tests. Once the initial review is done, the following will be added:

E2E test
Code for the transparent proxy handler

Could you share more about why choose this grpc proxy and what changes have made?

Could you share more about why choose this grpc proxy and what changes have made?

I chose this library as it's complete functionality wise, very well tested, and has no dependencies other than Go stdlib and grpc libraries.

LGTM. Please, address the comments above.

Ok, I'll now move into the second phase which is E2E and the porting of the proxy library.


Strip unknown regexp option line from error format

What was the end-user problem that led to this PR?
On Gemfile syntax error like gem 'foo', :path => /unquoted/string/syntax/error, Ruby 2.7 will show strange error message like:
[!] There was an error parsing `Gemfile`: unknown regexp options - trg - ...foo', :path => /unquoted/string/syntax/error
       ...                        ^~~~~~~. Bundler cannot continue.

due to recent change in Ruby trunk https://github.com/ruby/ruby/commit/b6468b01f7288789ef2e9bf548d81cdadb8ef053. This caused Bundler's CI failure on ruby/ruby repository.
Bundler's Travis ruby-head has not failed yet because Travis ruby-head is still a little old. It will fail later and this is fixing the future failure.
What was your diagnosis of the problem?
SyntaxError's message was changed to have an additional message indicating a position of an unknown regexp option like:
foo /bar/baz
         ^~~

or
...oooooooooooooooooooooo /bar/baz
...                            ^~~

What is your fix for the problem, implemented in this PR?
My fix tries to strip the new message part because the error line is already shown in another part of DSLError#to_s.
Why did you choose this fix out of the possible options?
To keep the best compatibility with older Bundler versions.

This will need a test to make sure we don't introduce a regression in the future.

I intentionally left it as is because having a test that fails by ruby's SyntaxError behavior change is the most important here and the failed test is doing the role.
That said, which did you mean by the reply:

You need to write a unit test that fails in older rubies rather than testing against actual Ruby behavior
You need to increase the number of test cases like the failed one to verify Bundler's behavior against the running Ruby better

I tend to agree with @k0kubun here in the sense that fixing the ruby-head test failure while keeping the rest of the specs green should be enough here? Ruby changed in a way that affects bundler and our specs caught that. Fixing the failure that caught this should be enough?
Without having looked into this, only concern is @k0kubun's comment in the ruby repo: "Possible fix would be bundler/bundler#7038, but Ruby script ending with ^~~ could be wrongly stripped by that?". Maybe it's worth adding a test that checks just that.

@bundlerbot r+

We will find the regression about new feature of Ruby head on https://dev.azure.com/rubylang/ruby/_build?definitionId=3.


Python 3.12 compatibility: distutils is no longer available

Maybe it's time to remove support for dmtx version < 0.75 and remove the if statements.
distutils (and distutils.LooseVersion) are no longer available on Python 3.12:

from pylibdmtx import pylibdmtx
Traceback (most recent call last):
File "", line 1, in
File "....venv\Lib\site-packages\pylibdmtx\pylibdmtx.py", line 10, in
from .wrapper import (
File "....venv\Lib\site-packages\pylibdmtx\wrapper.py", line 8, in
from distutils.version import LooseVersion
ModuleNotFoundError: No module named 'distutils'

I suppose to remove the if statements including the version check and to support newer dmtx versions only.

You might want to get libdmtx0a or libdmtx0b


Sync with Jord or Rowdy - UI and Features

Jord and/or Rowdy - UI + Features
- Walkthrough Storybook
- High-level design decisions, philosophy of the architecture
- How we are approaching the component specs: modularity, composability, flexibility - How "legos" informs our features

Link Storybook into the UI section [https://storybook.daohaus.fun/]

Here is the link to the notes: https://hackmd.io/bGpwOXyWQ2-01EnanFbPfQ?view

https://github.com/HausDAO/daohaus-monorepo/issues/43

https://github.com/HausDAO/sdk-docs/pull/44


Cassandra 17563

Talking to @ekaterinadimitrova2 to lower the risk of breaking generate.sh I reverted that logic and used the new scripts to generate the diff

Could we get more detail on the help output for these scripts? It's not clear what they're supposed to do, how they work, how to test, etc w/out opening them up. Nothing in readme.md and the help output is:
usage: circle-ci-config.py [-h] [--stdout] target
circle-ci-config.py: error: the following arguments are required: target

Also, having "HIGHER" as a target when we have LOWRES, MIDRES, HIGHRES seems pretty confusing for me:
51 targets = {
52         'MIDRES': {
...
89         'HIGHER': {


Add .NET instructions under documentation/integrations

What happens?
Currently, the documentation/integrations directory doesn't contain docs for the .NET integration. We should copy them over here.
What did you expect to happen?
I would expect to see a dotnet.md file with instructions on how to use Scalar's Scalar.AspNetCore NuGet package.
OpenAPI Document
No response

This sounds like the perfect task for me to tackle over the weekend! 👀 I'll write some nice docs with some examples.

@xC0dex Sounds good! Happy to chime in with a review once you get a PR out if you'd like.


LGTM.com - false positive

The detector sees a memory allocation of strlen(foo) bytes followed by a function call using both the string pointer and the allocated buffer pointer and assumes the allocated buffer won't have room for the trailing nul. However, in this case the function that is called is doing a Base64 decode of the original string, so the allocated buffer is actually 1/3 larger than necessary and will always have enough room for a trailing nul.
https://lgtm.com/projects/g/apple/cups/snapshot/9f5e54627ad9177bbc854bc3f6c901a06e363cf6/files/scheduler/auth.c?sort=name&dir=ASC&mode=heatmap#x1efd92d7955f17cb:1

This is indeed a false positive.
The query is https://github.com/Semmle/ql/blob/master/cpp/ql/src/Security/CWE/CWE-131/NoSpaceForZeroTerminator.ql. It simply looks for cases where the size argument to malloc comes directly from a call to strlen. An alert is then generated unless  the returned buffer is filled in by a known memcpy-like call (which would be good evidence that the programmer didn't intend it to be zero-terminated).
Thus, we alert without trying very hard to understand the code that fills in the buffer after it is allocated. We could look explicitly for strcpy and other standard-library functions that explicitly write a zero terminator, but that still leaves a large grey area of cases where the buffer is filled in by a hand-written loop. We'll want to alert there too, because a strlen result is very rarely the right length for a buffer that is filled in that way. In the query's defense, that's exactly what happens in this case -- the buffer is filled by a byte-by-byte loop, and strlen is indeed not the correct length for the buffer. Only the error happens to be strictly in the safe direction here.
One might imagine doing a deeper analysis to find hand-written loops that try to write zero terminators (for example look for a taint flow from the malloc to the LHS of an explicit '\0' assignment, though that won't find K&R-style while(*t++ = *s++); copy loops). But we would have to weigh the risk of missing some actual bugs against the annoyance of false positives.
(The alert would disappear if any sort of calculation happens on the way from strlen to malloc -- even explicitly adding 0 would probably be enough to convince LGTM you've thought about what you're doing. Or you could say len = len - len/4; and a save a tiny bit of space for the effort.)

OK, for now I will be adding 0 to suppress this since the Base64-decoded size isn't exactly 3/4, the memory savings could well be less than the alignment size, and the buffer gets freed almost immediately...
Thank you for looking at this and explaining!

Thank you for the report. We've just merged #2402, which should hopefully make this false positive result go away when deployed to lgtm.com in about two weeks.

An improvement related to this (https://github.com/Semmle/ql/pull/2402) has been merged, and will reach LGTM within about two weeks.  I expect this false positive will no longer be reported after that.


New version of python throwing this error

Upgrading pip to the latest version

error: externally-managed-environment
× This environment is externally managed
╰─> To install Python packages system-wide, try apt install
python3-xyz, where xyz is the package you are trying to
install.
If you wish to install a non-Debian-packaged Python package,
create a virtual environment using python3 -m venv path/to/venv.
Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
sure you have python3-full installed.

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

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

note: If you believe this is a mistake, please contact your Python installation or OS distribution provider. You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages.
hint: See PEP 668 for the detailed specification.
Error while running last command
dpkg: error processing package howdy (--configure):
installed howdy package post-installation script subprocess returned error exit status 1
Processing triggers for man-db (2.11.2-1) ...
Errors were encountered while processing:
howdy
E: Sub-process /usr/bin/dpkg returned an error code (1)

I added these lines to ~/.config/pip/pip.conf:
[global]
break-system-packages = true
This is a bad solution but it can work. Pip will through a warning every time, because its a bad idea to mix dependencies from different package managers in a non-virtual environment.

I'm closing this as it is a duplicate of #807, feel free to reopen it if you don't agree.

I tried the solution proposed my @Garret-Martin, however I still get error: externally-managed-environment after running apt install howdy
Error while running last command dpkg: error processing package howdy (--configure):  installed howdy package post-installation script subprocess returned error exit  status 1 Processing triggers for man-db (2.11.2-1) ... Errors were encountered while processing:  howdy E: Sub-process /usr/bin/dpkg returned an error code (1)


Duplicate events received when containerd is restarted with running containers.

Description

When restarting containerd with running containers, upon starting containerd back up and attempting actions on a container we get duplicate events.
DEBU[0018] event published                               module="containerd/events" ns=moby topic="/tasks/exit" type=containerd.events.TaskExit
DEBU[0018] event published                               module="containerd/io.containerd.runtime.v1.linux" ns=moby topic="/tasks/exit" type=containerd.events.TaskExit
DEBU[0018] event published                               module="containerd/io.containerd.runtime.v1.linux" ns=moby topic="/tasks/delete" type=containerd.events.TaskDelete
DEBU[0018] event published                               module="containerd/tasks" ns=moby topic="/tasks/delete" type=containerd.events.TaskDelete

Steps to reproduce the issue:

Start containerd
Start some containers and leave running
Stop containerd (even gracefully)
Stop a task
Watch event stream

Describe the results you received:
Duplicate events for the task exit and delete are sent.
Describe the results you expected:
Only one event should be posted as is normal when containerd has not been restarted.
Output of containerd --version:
containerd github.com/containerd/containerd v1.0.3 773c489c9c1b21a6d78b5c538cd395416ec50f88

OK on 1.0.3 I was able to reproduce:
2018-04-27 19:48:41.292772272 +0000 UTC default /tasks/exit {"container_id":"nginx","id":"nginx","pid":4562,"exit_status":137,"exited_at":"2018-04-27T19:48:41.292746572Z"}
2018-04-27 19:48:41.29290327 +0000 UTC default /tasks/delete {"container_id":"nginx","pid":4562,"exit_status":137,"exited_at":"2018-04-27T19:48:41.292746572Z"}
2018-04-27 19:48:41.293858855 +0000 UTC default /tasks/delete {"container_id":"nginx","pid":4562,"exited_at":"2018-04-27T19:48:28.281798784Z"}

On master it appears that it's fixed:
2018-04-27 19:50:53.595071351 +0000 UTC default /tasks/exit {"container_id":"nginx","id":"nginx","pid":6999,"exited_at":"2018-04-27T19:50:53.576701238Z"}
2018-04-27 19:50:57.821996952 +0000 UTC default /tasks/delete {"container_id":"nginx","pid":6999,"exited_at":"2018-04-27T19:50:53.576701238Z"}

@cpuguy83 can you try master (or 1.1.0) to see if you can still reproduce?
Thanks!

On 1.1 I'm having some other issues where ctr events exiting when it receives an event.

@cpuguy83 that should have been fixed in https://github.com/containerd/containerd/pull/2311


Relaxer is not picked up in subsequent runs

Description

Given a Relaxer is defined the processor does not picks it if the tests run without changes multiple times.

Done with 0.3.0-rc02


chore: [Hub Bot] Refresh metadata 2023-05-25

Updates Plugin Definitions

Closing as the only thing that needs changing is handled https://github.com/meltano/hub/pull/1351.


Document repo-specific prompt customization

What problem or use case are you trying to solve?
I believe that OpenHands supports two specific types of repo-specific prompt customization:

We have the .openhands_instructions file that is used by the github resolver
We have the .openhands directory, which allows for the creation of repo-specific micro-agents.

However, I believe these are not documented in the documentation site.
Describe the UX of the solution you'd like
It would be good to have these documented as a sub-folder under "Prompting" (after "Micro-Agents"):

Do you have thoughts on the technical implementation?
To implement this, it would be good to look through the entire OpenHands directory to references to the ".openhands_instructions" file and ".openhands" directory, understand exactly how they work.
Once we know the mechanisms for these two, they can be documented.

OpenHands started fixing the issue! You can monitor the progress here.

A potential fix has been generated and a draft PR #5804 has been created. Please review the changes.


Slightly incorrect guidance on maximum email address length

Description of the issue
The email address pattern page states:

Make sure the field can accommodate up to 256 characters, which is the longest an email address can be.

However, according to RFC 3696 the actual maximum length is 320 characters, 64 in the local part and 255 in the domain part:

In addition to restrictions on syntax, there is a length limit on
email addresses.  That limit is a maximum of 64 characters (octets)
in the "local part" (before the "@") and a maximum of 255 characters
(octets) in the domain part (after the "@") for a total length of 320
characters.  Systems that handle email should be prepared to process
addresses which are that long, even though they are rarely
encountered.

@lfdebrux suggested I raise this as a bug after a discussion on Slack 👋🏽

@peteryates Thanks for raising this issue!
We've put in a fix to update the content. After talking to some users and digging online, there seemed to be a consensus forming around 254 characters.
For example, in the RFC Errata:

In addition to restrictions on syntax, there is a length limit on email addresses.  That limit is a maximum of 64 characters (octets) in the "local part" (before the "@") and a maximum of 255 characters (octets) in the domain part (after the "@") for a total length of 320 characters. However, there is a restriction in RFC 2821 on the length of an address in MAIL and RCPT commands of 254 characters. Since addresses that do not fit in those fields are not normally useful, the upper limit on address lengths should normally be considered to be 254.

Some other links that informed our decision:

(https://stackoverflow.com/questions/386294/what-is-the-maximum-length-of-a-valid-email-address)
Lifewire - Character Limitations in an Email Addresshttps://stackoverflow.com/questions/386294/what-is-the-maximum-length-of-a-valid-email-address)Moonmail Blog - What is the maximum length of a valid email address?[Stack Overflow - What is the maximum length of a valid email address?]


[SPARK-20566] ColumnVector should support appendFloats for array

What changes were proposed in this pull request?
This PR aims to add a missing appendFloats API for array into ColumnVector class. For double type, there is appendDoubles for array here.
How was this patch tested?
N/A (This was a missing API.)

Test build #76394 has started for PR 17836 at commit a03f927.

Hi, @nongli .
Could you review this PR?

Test build #76394 has finished for PR 17836 at commit a03f927.

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

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

Merged build finished. Test PASSed.

Test build #76404 has started for PR 17836 at commit d979d0f.

LGTM
cc: @sameeragarwal @cloud-fan

Thank you for review, @kiszk

Test build #76404 has finished for PR 17836 at commit d979d0f.

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

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

Merged build finished. Test PASSed.

cc @michal-databricks

LGTM

Thank you for review, @hvanhovell and @cloud-fan .

Not sure if intentional, but the added test ("Float APIs") does not cover the new function (is uses put API mostly). Actually as far as I can see the "appends" APIs are not used at all in the code. The singular versions are also not covered by the tests but is used by toBatch.

Thank you, @michal-databricks and @viirya .
It was my mistake. I thought I did all. :(
I'll add that correctly, too.

Test build #76411 has started for PR 17836 at commit 6cec5da.

I updated the test cases to include appendFloat/appendFloats/appendDouble/appendDoubles.

Good catch. Is is better to apply these changes to other types (e.g. null, boolean, byte, short, int, and long)?

Test build #76411 has finished for PR 17836 at commit 6cec5da.

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

Merged build finished. Test PASSed.

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

Sure, @kiszk . I'll update the other test suite to improve coverage.

Test build #76438 has started for PR 17836 at commit 2b58c9f.

Test build #76438 has finished for PR 17836 at commit 2b58c9f.

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

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

Merged build finished. Test PASSed.

Thanks, LGTM

thanks, merging to master!

Thank you for review and merging again, @kiszk and @cloud-fan .

Hi, @cloud-fan .
Apache Spark 2.2 seems to be passed now.
Do you think it is possible to backport this into branch-2.2 for 2.2.1?

It's cherry-pickable into branch-2.2.

to be safe, can we create a new PR for 2.2? thanks

Sure!


Refactor syncUpload method to utilise new ResultProcessor Interface

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #2103
Description
Clear and concise code change description.
Encapsulate the processing logic in syncUpload into a new interface called ResultProcessor. The new structure should enable future enhancements and custom processing logic without requiring modifications to the core syncUpload method.
Alternative(s) considered
Have you considered any alternatives? And if so, why have you chosen the approach in this PR?
Type
Choose one: (Bug fix | Feature | Documentation | Testing | Code health | Builds | Releases | Other)
Screenshots (if applicable)
Checklist

[x] I have read and acknowledged the Code of conduct.
[x] I have read the Contributing page.
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA.
[x] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach.
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project.
[x] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally.
[x] I have built and run the demo app(s) to verify my change fixes the issue and/or does not break the demo app(s).

Closing in favour of PR #2137


Update to k8s 1.11 dependencies

everything seems to compile and test fine locally

/lgtm

lol, the review bot is like "only Nail can handle this" :-)
obviously I need to run the linters more often...

I think I'll have to destroy the etcd health checker fail test.
Now if etcd fails, there's a glog.Fatalf in flow, so it kills the process.

I think we should move the test to the scripted integration test https://github.com/kubernetes-incubator/service-catalog/blob/master/contrib/hack/test-apiserver.sh

couldn't get it to work, but wrote down a flavor of the idea, with some ideas in #2165

travis is timed out after completely finishing the actual cross build https://travis-ci.org/kubernetes-incubator/service-catalog/jobs/397870719
This is effectively GREEN.

just checking, but I think the bot is set up so one can self
/approve

rebased and cleaned up

/retest

/retest
doesn't seem to work for me

worked that time

not sure why travis can't resolve github, will try again later.

/retest

I agree more & more with 'make a build image', tho it only pushes the failure from pulling some binary off of github to pulling the image.

seems to pick up the command only some of the time
/retest

rebased

/retest

/retest

@kibbles-n-bytes any idea what part of this change could break the jenkins? I'm seeing this
[Build & Unit/Integration Tests] curl: (6) Could not resolve host: github.com

when it tries to call curl for dep.

taking off the go change, in case that was a problem

something about golang 1.10 docker images seems to be busted on jenkins.

well, I'm tired of the timeouts, and everything builds, but just not fast enough.
LGTM
/lgtm
/approve


Test for enum membership

An enum type in Postgres generates the following Go code:
CREATE TYPE company_role AS ENUM ('member', 'admin', 'guest');

type CompanyRole string

const (
	CompanyRoleMember CompanyRole = "member"
	CompanyRoleAdmin  CompanyRole = "admin"
	CompanyRoleGuest  CompanyRole = "guest"
)

Given a random string, I would like to test whether it is a member of the enum or not without going to the database. I can create a function, but s == CompanyRoleMember || s == CompanyRoleAdmin || s == CompanyRoleGuest breaks without warning as soon as a new field was added to the enum.
Is there a way to provide a map, or func ValidCompanyRole(s string) bool or func (c CompanyRole) Valid() bool, or anything I could call to automatically check for enum membership?

Is there a way to provide a map, or func ValidCompanyRole(s string) bool or func (c CompanyRole) Valid() bool,

sqlc could generate this function, but so far I've erred on the side of generating less code rather than more code.
The closest example I could find is the generated enum code for protocol buffers, which generates two static maps for conversion between strings and ints. However, I think they only do that because enums in protocol buffer land are mapped to int32.
So I think I'm happy with the code we generate today, since the function you're looking for isn't too difficult to write by hand.


add decimals field to graph

This allows changing the Grafana auto setting for the number of decimal places to something fixed - which I often tend to do.

Awesome! Merging


K8 Edge App 2022 Hackathon: 'Ad free news application'

Project Name
CNews
✍ Project Description
There are several news sites, but they all have one shortcoming, the news sites are either paid or have tonnes of ads on the news page. The goal of this project was to create a news site that is free and has no ads. With the freely available News API, it is possible to retrieve news from several news sites, making it possible to have a variety of news from different news sites.
Project URLs
Cnews
GitHub Repo
Project Video
Project Video
Project Tech Stack
Due to limited time developing and deploying the project, I decided to go with Python and Django frameworks because of their simplicity and ease of use. As much as I have worked with Python and Django before, this came as a challenge since my core experience is mobile application development.
After going through the available APIS, I decided to go with the News API. Although Parsing the JSON response was a challenge, I could parse the data and pass it to the Django template.
KEI Deployment Process
I used the Section KEI platform to deploy the project. Unfortunately, the deployment process was a bit complicated due to the abstraction of some Kubernetes components such as Ingress and Loadbalancer. On the other hand, the documentation was of great help by not having an in-depth explanation of some of the concepts. The section documentation assumes some of the concepts, making it hard for developers with little Kubernetes knowledge to deploy the applications on the section platform. Furthermore, the documentation does not only mention that Kubernetes Service types like Loadbalancer, Nodeport and Ingress are not supported. Mentioning the supported service types and the unsupported service types would significantly be important.
Although it was my first time configuring a domain, I could get the domain to work. This was a great learning experience after messing up with the domain DNS records 😊. I later learned that using sections hosted nameservers was the easiest way to get the domain working instead of going the CNAME and A records way.
Author
Name: Paul Odhiambo
Email<EMAIL_ADDRESS>Author Profile: Odhiambo Paul

👏

@paulodhiambo
Any extra info you could share on the deployment would be extremely helpful!
Any extra info on how easy/difficult the process was. Or if it was difficult, or were any parts confusing? What could we improve?
What did you like about the process? Any parts you got stuck on?

👏


How to remove top or bottom padding

Hi,
Thanks for such a great library.
I have been using this library to create custom dialogs but the problem is that i am getting top and bottom margins. Could you please suggest how can i mould DialogLayout.kt, so that i can pass padding from outside if needed.

The title layout and buttons layout are hardcoded in and not intended to be changed - only the content in between are

Could you please tell me where can I change this? Don't have enough time to walk through the whole code and i really need my custom view to be stuck to the top of dialog


Fix mobile landscape

Very nice theme!
On mobile when having it in landscape, it won't show the menu buttons.
To test go in Chrome into the responsive mode -> Select Iphone 12 Pro -> Rotate it to landscape
However I found the problem, you are specifying the menubar in relative-percent (Im not really sure why you specified it this way). But this way the menu is too wide on such screens and the translate when opening up the menu is not enough any more to fully show it. My change fixes that

(I cut the picture for privacy, but the tablet was in landscape)

Hey, thanks for the PR. I believe I may have copied that part from another theme. I am unsure now. I made a Frankenstein theme for me and then decided to share it, so I am not sure anymore what came from where, and what I added. But in my mind having a % would make it not break on different screen resolutions... clearly I thought wrong...
Regarding the PR, can you update the value from 320px to 315px instead? At least when I checked 320px is still slightly visible both on my phone and on the iPhone 12 Pro.
And I was not aware you could test it on different screens like that.  Thanks for letting me know! I noticed a lot is broken in landscape (I never use it at that orientation), and will give it a go at fixing this when I have some time.

But in my mind having a % would make it not break on different screen resolutions... clearly I thought wrong...

Yeah for responsive/mobile designs % is usually bad, given that a size of 20% will be very small on mobile but still large on desktop. In that case however it breaks because jellyfin shows the menu by translating it to the left by 320px (effectively hiding it) and % then does not fit to that absolute translation
I updated it to 315px (probably the border was visible)

Awesome! Merged.


Artifact modal shows both Aether and Lumine names

Modal when you click an artifact in a generated build

@nguyentvan7 Is this fixed already? Tried to reproduce


document the rule to list all platform variables in README

variables under var/ are used to set platform-dependent variables. they are used in defaults/mail.yml to handle differences in platforms. README is the primary reference for users. it is not sufficient to list all defaults, which often reference to platform variables.
README should list not only default variables but all platform variables, including version-specific variables. An example:
https://github.com/reallyenglish/ansible-role-java
this should be documented.

this was already fixed in https://github.com/reallyenglish/ansible-role-example/commit/68056dab3769fd0a95b8a69d512f15f5f7cc32f4#diff-d25bdbab75a7d3048db636b29a3c12a9R205


Generate decks at local execution

TL;DR
this PR breaks it, because it moves _outpu_deck to entrypoint.py . we use entrypoint.py only when we run the workflow remotely.
Type

[x] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
^^^
Tracking Issue
https://unionai.slack.com/archives/C02CMUNT4PQ/p1687268923664029
Follow-up issue
NA

@pingsutw, some checks are failing. Can you take a look at them?

No, The deck files will be written to different locations.


feat: add sl1pm4t/k2tf

sl1pm4t/k2tf: Kubernetes YAML to Terraform HCL converter
$ aqua g -i sl1pm4t/k2tf

CI failed, but maybe this is due to CI scripts.
I'll take a look.

Thank you for your contribution!

CI failed, but maybe this is due to CI scripts.
I'll take a look.

Thank you!!

ERRO[0000] install the package                           aqua_version=1.19.0 env=linux/amd64 error="package isn't found in the registry" package_name=sl1pm4t/k2tf package_version=v0.7.0 program=aqua registry=standard registry_ref=v3.22.0
FATA[0000] aqua failed                                   aqua_version=1.19.0 env=linux/amd64 error="it failed to install some packages" program=aqua

---
# aqua - Declarative CLI Version Manager
# https://aquaproj.github.io/
registries:
  - type: standard
    ref: v3.22.0 # renovate: depName=aquaproj/aqua-registry
  - name: local
    type: local
    path: ../registry.yaml
packages:
  - name<EMAIL_ADDRESS>  - import: ../pkgs/sl1pm4t/k2tf/pkg.yaml

Run aqua i --test
  aqua i --test
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
  env:
    AQUA_LOG_COLOR: always
    AQUA_CONFIG: aqua/test.yaml
    CI_INFO_IS_PR: true
    CI_INFO_HAS_ASSOCIATED_PR: true
    CI_INFO_PR_NUMBER: 5330
    CI_INFO_BASE_REF: main
    CI_INFO_HEAD_REF: feat/sl1pm4t/k2tf
    CI_INFO_PR_AUTHOR: ponkio-o
    CI_INFO_PR_MERGED: false
    CI_INFO_TEMP_DIR: /tmp/ci-info464[82](https://github.com/aquaproj/aqua-registry/runs/7713170218?check_suite_focus=true#step:4:86)4282
    CI_INFO_REPO_OWNER: aquaproj
    CI_INFO_REPO_NAME: aqua-registry
    AQUA_GOOS:
    AQUA_GOARCH:

https://github.com/aquaproj/aqua-registry/pull/5331

Released. https://github.com/aquaproj/aqua-registry/releases/tag/v3.23.0


E/UIFirst (19112): failed to open /proc/19112/stuck_info, No such file or directory

chatty  (19112): uid=12175() identical 2 lines
Unable make api calls

hey, I too face the same issue.. Can anyone help?

Cannot reproduce


"Find in Files" functionality

Do you have any nearby plans to include a "Find in Files" functionality similar to the GDScript editor? That would be useful in large codebases to quickly find, for example, dialogs featuring particular character, without resorting to greping the whole project.
If you don't, I won't mind porting it, for example, from https://github.com/nathanhoad/godot_dialogue_manager :)

Hello! Thanks for the suggestion. The idea is to improve the editor, but to be honest this was not on my list. It's definitely a useful missing feature. I was under the impression it could be worked around by using the one in the script editor, but it doesn't look like that.
If you already have an action plan or want to contribute, feel free to  push a PR with the changes. Otherwise, I can look into it sometime in the coming weeks. Cheers.


Question: Make Title Banner Smaller?

I want to make the large text banner slightly higher on the screen to match the Apple native version. What should be changed to achieve this? Thanks.

Hey.
You should be able to use the headerTitleStyle prop.

Thanks. Where should I put this prop and what are its potential values?

@zaptrem Just on the main component and it's a regular style prop, so fontSize.


changed help flag usage

changed help flag usage
Checklist

[x] Code compiles correctly
[ ] Relevant tests were added (unit / contract / integration)
[ ] Relevant logs were added
[x] Formating and linting run locally successfully
[x] All tests passing
[x] UA review
[ ] Design is documented
[ ] Extended the README / documentation, if necessary
[ ] Open source is approved

The code is not formatted.


train/dev/test split

Why your dev triples are included in training data?
code/data/preprocessing_scripts/nell.py:
out_file.write(e1+'\t'+r+'\t'+e2+'\n')
if np.random.normal() > 0.2:
----dev.write(e1+'\t'+r+'\t'+e2+'\n')
Theoretically you are supposed to split it into 2 datasets (train/test) or 3 (train/dev/test) without overlaps. Please explain the reason behind this. Thank you.

The NELL dataset consisted of a train/test split. The dev was created for hyperparameter tuning. The preprocessing script is not complete. There was another script used to remove out the duplicates and inverse duplicates. You can use the dev set we created (https://github.com/shehzaadzd/MINERVA/blob/master/datasets/data_preprocessed/nell/dev.txt).

Thank you for your response. I understand you split NELL train data into train and dev sets, Would you please let me know what was the proportion of the train/dev split you used in your paper? Because I am trying to reproduce the experimental results on your paper. I notice you didn't mention it on your paper. Thank you.


Witch's are unable to cast ritual spells

[10:53 PM] BjoernTheFish: Hey, sorry if this has already been adressed somewhere, but are others also having the issue where Witches can't cast any spells as rituals? Neither mod spells or vanilla spells. I don't use any other mods

I am able to ritual cast. Pending more information from the user before closing this.

The user that reported this said on a preview dev build it is working. It seems something got messed up during their initial character creation.


Require navigation-prop from one to many

Thank you for this library.
I have a question regarding the configuration of the entities involved in a SoftDelete scenario.
In a One-To-Many relationship, Client-ApplicationUser, the Client has a navigation-property CreatedByUser.

If I have a ICollection on the ApplicationUser entity the soft delete works as expected.
If I do not have the explicit ICollection on the ApplicatonUser entity, the softDelete fails with the error: You cannot soft delete a one-to-one relationship. It causes problems if you try to create a new version..

I've created a small demo project for this scenario: https://github.com/danmusk/SoftDeleteServiceTest (https://github.com/danmusk/SoftDeleteServiceTest/blob/master/SoftDeleteOneToOne/Models/ApplicationUser.cs)
In this project I use the LocalDb-SQLServer.
Is it required to explicit define navigationProps on the One-side back to the Many-side?

Hi @danmusk,
The relationship you defined is a one-to-many, but without the collection. I have updated the code to detect this and not throw an exception. See version 1.1.3 on NuGet.
Thanks for the pull requests - I didn't use them because I wanted to define specific unit tests, but they were useful to see how you were defining your relationship.

Updated to version 1.1.3 of the nuget in our project, removed the ICollection<>'s  we don't need, and everything now work as expected. Thanks for the quick response and fix.


making it write to the search layer... again?

Current coverage is 97.53%

Merging #23 into develop will not affect coverage as of dd9eada

@@            develop     #23   diff @@
=======================================
  Files            43      43
  Stmts          1216    1219     +3
  Branches          0       0
  Methods           0       0
=======================================
+ Hit            1186    1189     +3
  Partial           0       0
  Missed           30      30

Review entire Coverage Diff as of dd9eada

Powered by Codecov. Updated on successful CI builds.


KTOR-7239 Add tutorial "First steps with Kotlin RPC"

KTOR-7239
Preview on Staging

modified article content to include procedures
added project examples
added screenshots for light and dark theme
added meta, card and link descriptions
added new topic to the Welcome page

Hi Viktoriya! Thank you very much for preparing this tutorial. A few comments, which I think can improve the experience and help us get conversions.

Do you think we can remind to download IntelliJ IDEA at the beginning of the "Hello, kotlinx.rpc" section for those who immediately skip there after the intro? As an example it can be formulated as in this tutorial: https://kotlinlang.org/docs/jvm-get-started.html

For now, I suggest we remove references to future articles: "Future articles will drill into low-level details and real-world considerations." and "Future articles will show how it would work with KMP-based desktop and mobile clients." as we don't know how soon and who can create them.

In the current structure, I think that RPC vs REST and Kotlin RPC vs Google RPC chapters can be moved to the end (probably before the Next Steps chapter, though, WDYT?) as they are more general, and Extend the example still feels like the continuation of the tutorial.

Let's provide options where people can share their feedback at the very end of the article: "so we encourage you to explore it and share your feedback with us." We can mention that bugs and feature requests can be submitted on YouTrack while general discussions take place on Slack. Here is the link to the channel for those who are already on Slack https://kotlinlang.slack.com/archives/C072YJ3Q91V , and the link for newcomers to get their invite: https://surveys.jetbrains.com/s3/kotlin-slack-sign-up

Thank you!

Thanks for the feedback @meilalina!
Regarding 1, since we have a 'Prerequisites' section, I don't think it is necessary to add this in the 'Hello, kotlinx.rpc' intro, but I added the same download link to the first step:

Launch IntelliJ IDEA.

Another question arises though if we should provide an alternative approach to generating the projects as intelliJ IDEA is only a recommendation or should we mention the community edition as a minimum requirement?
2,3 and 4 - done ✅


Release azure-cli-sql 0.1.1b1

Reviewers, please verify the following for this PR:

[ ] The PR title has format Release <component-name> <version>.
[ ] setup.py has been modified with the same version as in the PR title.
[ ] HISTORY.rst has been modified with appropriate release notes.
[ ] If releasing azure-cli or azure-cli-core, __version__ defined in __init__.py should also be modified to match.

Release of 'azure-cli-sql' with version '0.1.1b1' successful. View at https://pypi.python.org/pypi/azure-cli-sql/0.1.1b1.

GitHub release created. https://github.com/Azure/azure-cli/releases.


Remove unused state var in clockSync3.qnt

Remove the unused state variable adj from clockSync3.qnt.
This corresponds to unimplemented functionality in the TLA+ template where this was translated from.
The variable was declared and initialized, but overlooked in the other actions.
@bugarela Shouldn't the effects checker notice a missing assignment?
It was only flagged in Apalache's transition finder.

There must be something wrong in the spec. If we are not using adjustments, the clock sync algorithm does not make a lot of sense. We should compare it with the TLA+ version

There must be something wrong in the spec. If we are not using adjustments, the clock sync algorithm does not make a lot of sense. We should compare it with the TLA+ version

Ah, this is clockSync3, I assume the full spec is clockSync7.
Shall we file an issue to write the entire spec in Quint?

Ah, this is clockSync3, I assume the full spec is clockSync7.
Shall we file an issue to write the entire spec in Quint?

Oh yes, that would be great. I thought we would use it for a demo today, but apparently not :D

@bugarela Shouldn't the effects checker notice a missing assignment?
It was only flagged in Apalache's transition finder.

The effect checker doesn't know that init and step should be used together. They are evaluated as any other action, so init updates adj, step doesn't, and since they are not ever combined in the spec, there is no issue. You'd probably see an issue if you declare something like run a = init.then(step).
I've raised this before but I can't find any issue on this. My proposal is that init and step are required to have their effect updating all state variables. Currently there is no check for unused state variables as well.

The effect checker doesn't know that init and step should be used together. They are evaluated as any other action, so init updates adj, step doesn't, and since they are not ever combined in the spec, there is no issue. You'd probably see an issue if you declare something like run a = init.then(step).

Aah good point, that makes sense!

I've raised this before but I can't find any issue on this. My proposal is that init and step are required to have their effect updating all state variables. Currently there is no check for unused state variables as well.

This came up in a workshop too. Rather than having special treatment of operator named init or step, it might be more flexible and consistent with our design to have a keyword modifier like:
complete action foo = ...

Or
transition foo ...

or something, that acts as an effect annotation stating all regions are meant to be updated by the action.


dummy update cat

Docs Build status updates of commit 15976d1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

ds1/cat.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


fix: Do recreate runner pod earlier on registration token update

Apparently, we've been missed taking an updated registration token into account when generating the pod template hash which is used to detect if the runner pod needs to be recreated.
This shouldn't have been the end of the world since the runner pod is recreated on the next reconciliation loop anyway, but this change will make the pod recreation happen one reconciliation loop earlier so that you're less likely to get runner pods with outdated refresh tokens.
Ref https://github.com/actions-runner-controller/actions-runner-controller/pull/1085#issuecomment-1027433365

is it going to trigger a restart in the middle of a pod running? If so, we just have another potential active runner deletion.

@jbkc85 Yes, you're absolutely right!
To be clear, as ARC has no way to atomically delete the runner if and only if the runner isn't busy, actions/runner needs to be able to gracefully stop at any time. (I.e. polling "list runners" API to detect if the runner is busy or not and deleting the runner only after that, isn't a solution. it's still subject to race condition)
This fixes the issue that runner takes too much time to restart even after the token is already expired.
This doesn't fix the issue that actions/runner is unable to stop gracefully at any time.

This fixes the issue that runner takes too much time to restart even after the token is already expired.

Got it!  That makes perfect sense.

@jbkc85 Good catch! Yeah we'd definitely better to take IsPhemeral into account to prevent potentially unnecessary restarts.

@jbkc85 Ah sorry I'm pretty confused.
From my observation, the "runner" container for an ephemeral runner does exit with code 0 when it finishes running a job.
We should definitely recreate the pod in that case. Otherwise, the pod gets stuck forever and you'll end up with zero ephemeral runner pods actually running the runner containers, right?

ARC has no way to atomically delete the runner if and only if the runner isn't busy

To be clear, this turned out to be my mistake. RemoveRunner seems like the API to atomically delete the runner. See https://github.com/actions-runner-controller/actions-runner-controller/pull/1125#issuecomment-1046003049


Autoload not working after conversion to 4.0

Godot version
4.0 beta1
System information
Windows 10
Issue description
Hi guys,
I converted my game to 4.0. I have several autoloads. The engine doesn't recognize the autoload scripts.

I removed and re-added the autoload script, then it worked and showed me the next error of another autoload. I just saved it and reloaded the project because I want to approach this conversion step by step. After It reloads, I receive all the errors again like it didn't save.
Steps to reproduce
Just a project conversion from 3.5 to 4.0 with several autoload scripts.
Minimal reproduction project
No response

Did you fix all script errors from your autoloads?

Not all. I have so many, I wanted to fix a few autoload script errors, save, reload, and continue.

I think autoloads won't work properly until you fix all of them.

i'll give it a try

Now, it's switching back to errors. After I autoload the next script, it throws back the previous autoload error.
https://user-images.githubusercontent.com/37254287/191492559-03157343-005f-468e-97fe-58ec0412bd77.mp4

Weird 🤔
It would be helpful if you could provide some minimal reproduction project. I use autoloads in my project and they work fine, so it might be something more specific.

Reproduction Project:

open godot 4.0beta1 with this project
open test.tscn scene and attached script
see the error, now load singleton "leveldetails" from the singleton folder as "leveldetails"(no caps)
enter a new line to process the errors
the error for "leveldetails" disappears, now load the globals singleton, enter new line to process the error
now the previous error appears again.

err_autoloads godot4.0beta1.zip

Fixed by fixing errors in both autoloads:

Although for whatever reason it didn't work immediately, I had to modify the script twice to make it reparse properly.

I'm gonna have issues fixing the complete script.


Please add a sample code /tutorial that shows how to use this package.

Please add more details about this package like how to use this , how to integrate with a android and ios .

There are examples here :
Android: https://github.com/wix/react-native-open-file/blob/master/example/RNOpenFileExample/index.android.js
iOs: https://github.com/wix/react-native-open-file/blob/master/example/RNOpenFileExample/index.ios.js

There are examples here :
Android: https://github.com/wix/react-native-open-file/blob/master/example/RNOpenFileExample/index.android.js
iOs: https://github.com/wix/react-native-open-file/blob/master/example/RNOpenFileExample/index.ios.js

404 Not Found

The example of https://github.com/wix/react-native-open-file/blob/master/example/RNOpenFileExample/App.js does not make sense needs more details and update the read me I find this https://github.com/wootwoot1234/react-native-open-file/blob/master/example/RNOpenFileExample/index.ios.js but I still not know how to use


[SPARK-5990] [MLLIB] Model import/export for IsotonicRegression

Model import/export for IsotonicRegression

Test build #29410 has started for   PR 5270 at commit 2b2f5a1.

Test build #29410 has finished for   PR 5270 at commit 2b2f5a1.

This patch passes all tests.

This patch merges cleanly.

This patch adds the following public classes (experimental):

    case class Data(boundaries: Array[Double], predictions: Array[Double], isotonic: Boolean)

This patch does not change any dependencies.

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

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

Test build #29953 has started for   PR 5270 at commit 49600cc.

Test build #29953 has finished for   PR 5270 at commit 49600cc.

This patch fails Spark unit tests.

This patch merges cleanly.

This patch adds the following public classes (experimental):

    case class Data(boundary: Double, prediction: Double)

This patch does not change any dependencies.

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

test this please

Test build #29960 has started for   PR 5270 at commit 49600cc.

Test build #29960 has finished for   PR 5270 at commit 49600cc.

This patch fails Spark unit tests.

This patch merges cleanly.

This patch adds the following public classes (experimental):

    case class Data(boundary: Double, prediction: Double)

This patch does not change any dependencies.

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

Test build #30593 has started for   PR 5270 at commit f80ec1b.

Test build #30593 has finished for   PR 5270 at commit f80ec1b.

This patch passes all tests.

This patch merges cleanly.

This patch adds the following public classes (experimental):

    case class Data(boundary: Double, prediction: Double)

This patch does not change any dependencies.

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

LGTM except minor inline comments.

Test build #30635 has started for   PR 5270 at commit 872028d.

Test build #30635 has finished for   PR 5270 at commit 872028d.

This patch passes all tests.

This patch merges cleanly.

This patch adds the following public classes (experimental):

    case class Data(boundary: Double, prediction: Double)

This patch does not change any dependencies.

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

Merged into master. Thanks!