Use OWNER_ALIASES defined teams and add area/* labels

Changes
Using the OWNER_ALIAS defined teams in different (and new) OWNERS files. This also will add some area/* label automatically depending on the modified files.
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[x] Commit messages follow commit message best practices
[ ] Release notes block has been filled in or deleted (only if no user facing changes)

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.

/hold

/kind misc

/retest

/retest
:thinking:

/retest
:eyes:

The following users are mentioned in OWNERS file(s) but are not members of the tektoncd org.
* pipelines-approvers

  * OWNERS

How ?! :thinking: or may we run a really old version of prow ? :thinking:

/test pull-tekton-pipeline-integration-tests

/retest

The CI failure here is down to receiving three events instead of two for one of the reconcile tests:
https://gist.githubusercontent.com/afrittoli/6aef67c52f17919cd143f6fe5faf9917/raw/4dfc8111609b71a8dd6d5df226c35691a479faa6/gistfile1.txt
I'm confused how this could happen - the object involved in the event is exactly the same, same timestamp, same resource version. The event object itself is identical except for the resource version.

/lgtm

Thanks @vdemeester !!
/approve

/retest
(I think that test-unit bug is fixed in master but this PR ain't rebased against it :sweat: )

/retest


Update promo materials static page and files

https://dmptool.org/promote
Update files and revise page as follows:

Update postcard files w/new ones (2 versions - https://github.com/CDLUC3/dmptool/tree/development/docs/postcard)
Revise header: "Postcards" (remove "coming soon")
Create bullets "Postcard 1: Adobe PDF" "Postcard 2: Adobe PDF"
Delete poster section
Update general purpose slides w/new PPT https://github.com/CDLUC3/dmptool/blob/development/docs/genericslides/DMPTool-Generic-Slides.pptx
Revise header: "General purpose slides"

Changes made in static page file in dev branch and new promo materials uploaded to dev

text changes on page look good. i can't test the download links since the new files aren't in master yet, but they appear to go to the correct URLs.


Support embedding native views in macOS shell

We'll need support for embedding NSViews in Flutter views (e.g., for embedding a web view, as on mobile).

Hello, is it possible to embed NSViews view in Flutter view now? I need to embed a WKWebView to display the web, is there a good solution at present?


Contact us is missing from bottom of beta page

Oops!  We don't have the email link anymore

They had me get rid of it in favor of the Provide feedback link (instead of having both).

On Oct 18, 2016, at 9:58 AM, ARWattam<EMAIL_ADDRESS>wrote:
Oops! We don't have the email link anymore
https://cloud.githubusercontent.com/assets/11475316/19480726/77de99d8-9519-11e6-9382-ca539815ad59.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/PATRIC3/patric3_website/issues/1111, or mute the thread https://github.com/notifications/unsubscribe-auth/AALobpo0IM292SIidF-nzXl1e3nWWVLFks5q1NCggaJpZM4KZ0tw.

But I don't even see that!  All that's there is the address, how to cite, and the funding statement

It is in the help menu.

On Oct 18, 2016, at 10:10 AM, ARWattam<EMAIL_ADDRESS>wrote:
But I don't even see that! All that's there is the address, how to cite, and the funding statement
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/PATRIC3/patric3_website/issues/1111#issuecomment-254519125, or mute the thread https://github.com/notifications/unsubscribe-auth/AALobhkh9qt_bl2nTrepVa-cS6ah08T2ks5q1NNBgaJpZM4KZ0tw.

I wonder if we need provide different flavors of the same link in different contexts to make it more intuitive to the users, i.e. report problem, provide feedback, contact us, etc.
-Maulik

@rkenyon what do you think?
Is "Provide Feedback" in the top menu enough or do we need additional link at the bottom of the page and on about patric page?

Now in Help Menu.  Closing ticket.


Multiple distgit new packit api collection object

DO NOT MERGE.
This is just a SPIKE PR to propose multiple solutions on how to refactor the packit_api object to support monorepos.
related to packit#1618

The reason I don't like this approach is that it leaks functionality (breaking up the configuration into individual package configs) related to handling monorepos from PackitAPI to get_packit_api(), and does it in a way which is not reusable.
I would find an implementation, where handling monorepos (including interpreting the configuration) is enclosed in PackitAPI objects (that is: a single API object holds references to the upstream repo and all the downstream repositories) easier to reason about than having an API object for each package. Of course PackitAPI methods (even __init__(), if that's required) should receive an argument to be used to limit their scope to selected packages, if so desired.
I don't see why such an approach wouldn't have the same benefits that @lachmanfrantisek mentioned above (including creating one task per package).
That said, I still can see the approach implemented in this PR to work (although my gut feeling is that it'll require more extensive code changes in Packit Service), but then there should be a mechanism (utility function or method) to reduce a PackageConfig object to include a single package configuration (including the filtering and transformation of the relevant jobs).

For completeness, I've suggested Maja try to provide multiple approaches and write down the cons/pros for each so we can objectively compare those. And also, when trying to work on that, Maja will see what is the easiest one to do. (I have some feelings but I might easily be wrong since I haven't tried to implement that for real.)
Regarding  dist_git_base_url, I don't think it's the best option to start with since it sounds more like a config option of one package than a true monorepo -- e.g. similar to a dist-git branch. But I am not saying we can't implement that after all, just that I would not be too focused on this use case for now.

Today I have worked on this possible version of refactoring https://github.com/packit/packit/pull/1840.
Tomorrow, as I said, I will try to change PackitAPI internally. And then I will let you know which is my favourite implementation and why :D

I close this PR, it was just an example for supporting the SPIKE on how to handle packit refactoring for monorepo.
Related research could be found here: https://github.com/packit/research/tree/main/monorepo-support/refactoring


Bug Fix - onAnchorLinkClick not firing

This PR fixes a bug where onAnchorLinkClick would not fire when stripHash = true and the anchor links to a hash on the same page.

Closed in favor of #32

Closed in favor of #32


Db migrations as separate plugin in configuration

When doing this we can stop basing on the model and use output schema that will be decorated.

@craicoverflow This actually critical bug - when you have now multiple model files db migration will not work and fail with
$ /Users/wtrocki/Projects/sandbox/test-generator/node_modules/.bin/graphback db
There can be only one type named "Query".

There can be only one type named "Mutation".
✨  Done in 1.00s.

$ /Users/wtrocki/Projects/sandbox/test-generator/node_modules/.bin/graphback db
There can be only one type named "Query".

There can be only one type named "Mutation".
✨  Done in 1.00s.

@wtrocki It looks like you have defined multiple Query types across multiple schema files. This will also happen if you define more than one Query type in a single schema file. This particular error occurs during buildSchema and is more down to the design of the data model.

yes.. So we could not prevent this from happening.
Solution is to make our dbmigrations config separate from graphback and then use graphql-config root schema. I would hope to do it before official release as we do not want to break config many times. It should be a very simple thing to do but also might require some testing after.
@cecobask Would that be something that you could help us to do?
Base info - our Graphql-migrations package (in packages folder) accepts schema that comes from model folder which can concatenate multiple different schemas.
We should use schema that is configured in graphqlrc.yml config

Please, provide some feedback of my work in pull request #749. Thank you :)

Fixed by @cecobask


It doesn't work with shaders

Describe your bug here.
I've wanted to put shaders in the game
but apparently they don't work. I tried mods that have shaders but it still doesn't work, and in the last update of the game, they added the function to change the color of the notes, and I think it also works with shaders but it's not working. Here's a photo:

Can someone help me solve this?
¿alguien puede ayudarme?
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No

your system probably isn't compatible, in that case it's not a psych engine issue. Otherise, check the shaders option in settings


Support for python < 3.8

Literal was added to the typing module in python version 3.8, this pull request ensures that pyloggor uses typing-extensions for Literal instead of typing to support lower python versions

coolio, didn't know this!


PutValue is note working

I'm trying to put a value into an existing key to enable automatic windows login but it is not working, and not throwing an error in the callback either.
Here is the code:
var valuesToPut =
                            {
                                'HKLM\\SOFTWARE\\Microsoft\\Windows NT\\CurrentVersion\\Winlogon': {
                                    'AutoAdminLogon': {
                                        value: 0,
                                        type: 'REG_SZ'
                                    }
                                }
                            }
                        regedit.putValue(valuesToPut, function (err) {
                            console.log(err);
 });`
and i have tried to specify the OS Architecture also with no luck
======= INFO =======
OS: Windows 10
OS Arch: x64
Node Version: 6.11.2
Node Arch: ia32
platform: Electron

Can you please also attach the error?

there is no logged error, sorry for the late reply, it acts like everything is fine, but the actual value in the registry doesn't change.1

@anaszgh is this related to #49 ?


TODO: Support CH585/CH584

Where I could find documentation how to connect the CH585 eval board with WCH LinkE?


Custom CloudEvent serialization problem - Data property

Intro
I've stumbled on a problem while publishing events with a custom CloudEvent wrapper.
I'm trying to use custom CloudEvent so that I can specify the type property and use it for routing functionality inside subscribers.
Related to Allow publishing cloud events directly #868
The project is using docker-compose to set up the whole infrastructure (pub/sub services, dapr sidecars, Kafka, etc) for testing.
Demo project based on Dapr and Dapr.AspNetCore ver. 1.8, and .Net 5 (Web Api)
Expected Behavior
Published cloud event JSON schema should look like events published without custom cloud events.
Actual Behavior
Dapr sidecar log shows the following warning/error:
level=warning msg="retriable error returned from app while processing pub/sub event <nil>, topic: RoutingIntegrationEvent, body: {\"type\":\"https://tools.ietf.org/html/rfc7231#section-6.5.13\",\"title\":\"Unsupported Media Type\",\"status\":415,\"traceId\":\"00-9c97aa770091eb243c1f904fbd88c27f-cfe48e0d27c8fc42-01\"}. status code returned: 415" app_id=subscriber instance=b398cfe88dc9 scope=dapr.runtime type=log ver=1.8.0
After comparing published events with and without a custom CE wrapper I've noticed that the "data" property is different between those two types of events.
Specifically, pure event serializes cloud event with "data" property, while custom CE serializes it into uppercase "Data" property.
Steps to Reproduce the Problem

Clone the repository with the fixed and runnable project (under docker-compose). https://github.com/mzj/PubSubRouting
Remove the above-mentioned attribute to reproduce the problem.
Run the solution (tested in VS2022)

Fix
Adding [JsonPropertyName("data")] attribute to CloudEvent.Data property, inside Dapr.Client project.
Release Note
FIX Bug in runtime.

This is still a problem today. The event cannot be used because of the parsing mismatch. Any updates on this soon?


Missing template disambiguator keyword for dependent qualified names

See https://github.com/lattice-land/lala-pc/issues/5
I am re-posting the issue here for visibility.
Thanks in advance,
-Alan

Thanks! Just fixed it.


(ADT-DP) Add diagnostic scopes to all DigialTwins APIs

One question - given that we are creating these scope instances every time the API is called, what would be the performance (memory) impact if I call an API in a loop 100 times? (for disposing - it'll wait until the gc collects it, which isn't always immediate, right?)

well, it won't wait for the gc since as soon as the method call goes out of scope the objects are disposed.

One question - given that we are creating these scope instances every time the API is called, what would be the performance (memory) impact if I call an API in a loop 100 times? (for disposing - it'll wait until the gc collects it, which isn't always immediate, right?)

well, it won't wait for the gc since as soon as the method call goes out of scope the objects are disposed.


[test] Fix CI failure in bad stdlib test

https://github.com/apple/swift/pull/36928 changed the error message while https://github.com/apple/swift/pull/36594 added a test case looking for the old error message. Update the test to look for the non-dynamic substring of the new error message.
CC: @eeckstein @beccadax

@swift-ci Please smoke test


Constrained to implicit message schema

Any schema outside of %{status: <status>, response: %{reason: <reason>}} is ignored.
The sample app at https://github.com/NoRedInk/elm-blogger sends down {"topic":"editor:other","ref":null,"payload":{"status":"ok","response":"hello"},"event":"new_msg"}	 and in Elm this is parsed as { topic = "editor:other", ref = -1, payload = { status = "ok", response = { reason = "NULL" } }, event = "new_msg" }
We should allow a more open schema. We shouldn't have to change the schema Phoenix sends down to use elm-phoenix.

Any suggestions on how this should look?

My opinion is that it probably just recive a Json.Value, and allow users to specify a decoder to use.

I'm playing around with it now. The shape of the payload is generally dependent on the topic and the event.
Either the decoder the user passes in is applied at the root of the message (requiring the decoder to be aware of the fields "topic", "event" and "payload") or we only apply it on the payload and we need a way to allow the user to specify a different decoder for each "topic" and "event" pairing.
I've hand rolled the latter and it works fairly well, but I'm still working out a few kinks.


Add test case for upgrade wrapper

Benchmark-wrapper needs to test the upgrade wrapper similar to other wrappers in the CI to make sure the code changes are not breaking the functionality. The version to upgrade to can be set to the same version as the CI version to avoid upgrading the OpenShift cluster on which CI is running.

I think this is more specific to OpenShift -- that functionality of CI should be outside of benchmark-wrapper.


Update list of supported branches

Q
A

Type
improvement

BC Break
no

Fixed issues

Summary

Yeah, I noticed after creating this PR. I'll create a pull request for the merge then see where we're at.


Add NOTICE to the project

Enhancement
Suggestion
As discussed on the call, we should add NOTICE to all the projects in k8snetworkplumbingwg.

@dougbtv can you please assign this task to me ?


Add support for attachment_url in tweets

If it's possible and I missed it excuse me.
As described in this stackoverflow question:
https://stackoverflow.com/questions/29680965/how-do-i-properly-retweet-with-a-comment-via-twitters-api
TwitterClient.postTweet needs a variant with an attachment_url param to "embed" another tweet while posting a new tweet as explained above without showing the URL in the tweet text itself.
 parameters.put("attachment_url", "TweetToRetweetURL");

^Does it.

Hi,
Did you try to just put in the tweet text the url of the second tweet ?

Hi,
Did you try to just put in the tweet text the url of the second tweet ?

You mean putting the url of the tweet I want to retweet into the text of my new tweet?
That works but it's displayed on Twitter itself differently, you can see that difference in that stackoverflow question above (I just noticed I messed up the link, sorry about that. Works now)
In one case the URL of the retweeted tweet itself is not inside the new tweet's text

Solved, you can now use :
 Tweet tweet = twitterClient.postTweet("test", null, null, "https://twitter.com/TwitterDev/status/1392465174708187137");
Once Twitter team release the V2 endpoint I'll provide a cleaner solution :)


Always got empty items array for GET path array response with unique items

Since merge of PR #12, I've always got an empty array returned from my GET endpoint path with unique items configured:
"/idds/{id}/invoices/{doctorsInvoiceNumber}/documents":
    get:
      summary: List of documents for an IDD specific invoice
      ...
    responses:
        "200":
          description: List of documents
          content:
            application/json:
              schema:
                type: array
                uniqueItems: true
                items:
                  $ref: "#/components/schemas/File"

Find full spec attached

@gschafra thank you. Can you give me a full specification? Or at least definition of #/components/schemas/File? I cannot reproduce an issue.

@strider2038: Find the specification here: REST_API_OAS3.zip. Maybe I just missed something. Many thanks in advanced!

@gschafra the problem is that File has type: object and it is detected as object without properties, not as combined type allOf. I will fix default behaviour of parser to detect combined types first.
File:
      description: File object
      type: object # <---
      allOf:

Issue fixed in v0.1.7


WIP: Python 3 on Linux passes Travis CI tests

Checklist

[ ] npm install && npm test passes
[ ] tests are included
[ ] documentation is changed or added
[x] commit message follows commit guidelines

Description of change

The goal was to make our Travis CI tests pass on Python 3.6 on Linux pass.  -- Success.
Unfortunately, test/test-find-python.js is so fragile that I can not find a way to do this without breaking ALL Python 2 tests.  Hopefully someone else can help to make the Python 2 tests pass again.  #1826
I tried without success:
      if (process.env.NODE_GYP_FORCE_PYTHON === '1') {
        t.strictEqual(stderr, '')
        t.ok(/Python 3/.test(stdout)) // Python 3 writes the version to stdout
      } else {
        t.strictEqual(stdout, '')
        t.ok(/Python 2/.test(stderr)) // Python 2 writes the version to stderr
      }

The change to gyp/pylib/gyp/generator/ninja.py is lifted right out of #1836 and should be uncontroversial.
The changes to .travis.yml shift to:

focus from Python 3.7 to 3.6 because async and await are reserved words in Python >= 3.7.
perform a single test run each on Linux, macOS, and Windows.

Closing in favor of #1843


FCM category issue in IOS (React Native)

In our React Native, we're trying to implement Notifee for creating notification categories, and we pass the category id to FCM to enable action buttons in FCM notifications.
However, we're encountering an issue specific to iOS where tapping on the action buttons in the notifications doesn't launch the app. Instead, the specified function associated with the action button is executed in the background (e.g., redirecting the user to a specific screen).
FYI, We're using Notifee version 7.7.1 and React Native version 0.71.
Are there any fixes or workarounds that can be implemented to address this issue?
Code Snapshot for category creation

@Prafulkumar0512 Have you tried skipping the action buttons and when the user clicks on the notification (which should open the app) use getInitialNotification like below
notifee.getInitialNotification().then((initialNotification) => {   if (initialNotification) {     const { notification, pressAction } = initialNotification     if (notification) {       switch (notification.ios?.categoryId) {         case 'chat':           tabIndex = TABS.CHAT           break         /// ...       }     } else {       // if we are here, a notification did not cause the application to open     }   } })


GitHub Templates

Take advantage of the GitHub Markdown templates for Issues and Pull Requests!
Also, the following:

Moves ruby-style.md to server/README.md to be consistent
with the setup under client/
Removes the Procfile as it isn't used anymore.
Made both README's consistent in styling and updated client's README with new
information

LGTM


Downport to ABAP syntax version v702

Downport to ABAP syntax version v702, to make ABAP2UI5 compatible to a wide range of ABAP stacks.
If there is someone with a v702 system who can test, after the downport, feel free to let me know!
Project dependencies:

Syntax downport to-do:

https://rules.abaplint.org/downport/ can do a lot of it automatically, tried running it, need some bugs ironed out

yes thanks i will go with that! the only problem is that abaplint only checks for syntax and not if sap classes are used which not exist in a certain release. for example: https://github.com/oblomov-dev/ABAP2UI5/issues/46
abaplint with v750 did not warn. life is hard without a system in the sap world 😅

in the current configuration it knows https://github.com/oblomov-dev/ABAP2UI5/blob/main/abaplint.jsonc#L7 which has all the Steampunk objects, you can choose how much or little you want it to know

Adjusted the code, that it is ready to downport with abaplint without rework. All tests are succesfull now.
Thanks for the help, this topic is finished for now and in addition in a very elegant way! 👌


Missing icon for energy

I know, I'm never satisfied~

Oh, haha, yeah.
I think there might also a special primary attack icon for the Firespray with front/back arrows or something if you want to get really specific.

Yeah, that one's not super necessary.  But I want to move away from using the png sprite, and at this point I think it's only the energy icon that's not in the font.


strongly typed scalar .net configuration option

What happens?
currently configuration is hardcoded in the .net integration
What did you expect to happen?
we should have a strongly typed object that is 1 level deep that matches our api reference config object
How can we reproduce the issue?
no config option
Swagger/OpenAPI Example
No response

Done with https://github.com/scalar/scalar/pull/3019

Niiice


[Feature Request] idToken from an AuthenticationResult is not exposed in the Public API

The library exposes function getAccessToken(),
However,  there is not a public api surface to fetch an idToken from a token response.
Once there we add this api, we can add a check to the e2e tests ro verify if returned object contains an IDtoken.
This was the context in which the need for such an API was found : https://github.com/AzureAD/microsoft-authentication-library-for-go/pull/97#discussion_r537883244

IDToken is now public.


Failed to load Camber.dll library

using Civil3D 2022 <IP_ADDRESS> and Dynamo <IP_ADDRESS>, I'm missing most of the Camber nodes.
Dynamo.Exceptions.LibraryLoadFailedException: Failed to load library: C:\Users\USER\AppData\Roaming\Autodesk\C3D 2022\Dynamo\2.10\packages\Camber\bin\Camber.dll
Build error for library: C:\Users\USER\AppData\Roaming\Autodesk\C3D 2022\Dynamo\2.10\packages\Camber\bin\Camber.dll
Could not load type 'ExportableItem' from assembly 'AeccDataShortcutMgd, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.

Please update to 2022.1 at a minimum per the compatibility table in the release notes.


Fix crates.io badge

Hi @atouchet, thank you very much for the fix, I have now merged it!
Have a great day!


fix(protocol): deploy TransparentUpgradeableProxy for bridged token differently

@adaki2004 @davidtaikocha
we should deploy a ProxiedBridgedERC1155 before hand, registered it as "proxied_bridged_erc1155" in AddressManager, then only deploy a TransparentUpgradeableProxy when necessary. The current deployment strategy is very expensive -- each new bridged token will trigger a deployment of the implementation, this is not necessary.
@adaki2004 please finish this PR for the other two vaults, then fix tests.
@davidtaikocha please fix the deployment scripts.

@adaki2004 please finish this PR for the other two vaults, then fix tests.

Done.


S3 - IAM: SignatureDoesNotMatch on bucket-related methods

When trying to call some methods on the s3 client while using moto, I get some unexpected SignatureDoesNotMatch exceptions.
The client can call head_bucket (maybe thanks to #4335 and #4346), but fails to call get_bucket_location or list_objects_v2:
An error occurred (SignatureDoesNotMatch) when calling the GetBucketLocation operation: The request signature we calculated does not match the signature you provided. Check your key and signing method.
According to AWS docs, the only required permission to call list_objects_v2 is s3:ListBucket, and it should be given with s3:*.
The code below works as expected if we pass 8 or more to the @set_initial_no_auth_action_count.
 Check your key and signing method.

somedir/lib/python3.10/site-packages/botocore/client.py:960: ClientError

============================== 1 failed in 0.50s ===============================

Here are the dependencies versions:
boto3==1.26.74
botocore==1.29.74
moto==4.1.2

Hope I gave enough details, please tell me what I did wrong or how to get the s3_client.get_bucket_location to work

Thanks for raising this @Maxi-Mega. It looks like there's a bug in Moto where we do not correctly build the Signature for requests with a querystring, hence the exception. Will raise a PR shortly to fix this.

@bblommers
Hello.
The same bug seems to occur continuously when INITIAL_NO_AUTH_ACTION_COUNT is enabled.
First, create a bucket.
aws --endpoint-url ${AWS_ENDPOINT} s3 mb s3://example

Next, run the bucket listing.
aws --endpoint-url ${AWS_ENDPOINT} s3 ls
2024-02-08 07:57:26 example

It then retrieves the listings in the bucket.
aws --endpoint-url ${AWS_ENDPOINT} s3 ls s3://example/

An error occurred (SignatureDoesNotMatch) when calling the ListObjectsV2 operation: The request signature we calculated does not match the signature you provided. Check your key and signing method.

Hi @nonchan7720, how are you starting the MotoServer?
I cannot reproduce this with the latest Docker image:
> docker run --rm -p 5000:5000 -e 'INITIAL_NO_AUTH_ACTION_COUNT=2' --name moto motoserver/moto:latest &
[1] 9852
> WARNING: This is a development server. Do not use it in a production deployment. Use a production WSGI server instead.
 * Running on all addresses (<IP_ADDRESS>)
 * Running on http://<IP_ADDRESS>:5000
 * Running on http://<IP_ADDRESS>:5000
Press CTRL+C to quit

>
> aws --endpoint-url http://localhost:5000 s3 mb s3://example
<IP_ADDRESS> - - [25/Feb/2024 12:05:52] "PUT /example HTTP/1.1" 200 -
make_bucket: example
> aws --endpoint-url http://localhost:5000 s3 ls
<IP_ADDRESS> - - [25/Feb/2024 12:06:05] "GET / HTTP/1.1" 200 -
2024-02-25 11:05:52 example
> aws --endpoint-url http://localhost:5000 s3 ls s3://example/
<IP_ADDRESS> - - [25/Feb/2024 12:06:20] "GET /example?list-type=2&prefix=&delimiter=/&encoding-type=url HTTP/1.1" 403 -

An error occurred (InvalidAccessKeyId) when calling the ListObjectsV2 operation: The AWS Access Key Id you provided does not exist in our records.
>


Support for 2.13 inch display v2

Summary
Driver supports 2.13 inch display v2 mono
Impact
Function putrun (pointer in struct lcd_planeinfo_s) redraws the screen if argument npixels has value 0. Adding special IOCTL for framebuffes driver can call putrun function in order to refresh e-paper screen.
We don't want to refresh the screen after single line/pixel modification. Driver is not aware if there is next putrun function call or not.
Testing
Tested on lilygo t5v2 with 2.9 inch display and STM Nucleo-wl55JC with 2.13 inch display.

@adamkaliszan  please rebase to the latest code to fix the ci build error.

@adamkaliszan will you fix the rest comment from @pkarashchenko ?


Fix currency input mask localization

The destructuring of the return values from getCurrencyFormat was incorrect, so we always ended up with the default values. There was no unit test for this, so we didn't notice. Added unit tests for two cases, to ensure we use the function correctly. getCurrencyFormat itself is unit tested for all currencies so this should be robust.

Thanks guys!


[AMBARI-23056] [trunk] Fix AMS dependencies that have security issues.

What changes were proposed in this pull request?
Fix AMS dependencies that have security issues.
How was this patch tested?
mvn clean test
Manually tested the newer versions of artifacts introduced.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/767/
Test FAILed.
Test FAILured.

Build failure in Ambari Server module is unrelated.


configuration to get client IP address in ngx.var.remote_addr.

I am trying to use IP-restriction plugin, but ngx.var.remote_addr has random ip address (there is no load balancer before kong) instead of client ip address. I can circumvent this by adding the following in kong.conf,
trusted_ips = <IP_ADDRESS>/0,::/0
and sending X-Real-IP in the request headers, This will replace ngx.var.remote_addr, and i can add this address in IP-restriction blacklist, which blocks that IP. but the client can just change the value of his IP address in X-Real-IP request Header and he will be able to access like a non-blacklisted user. What is the configuration of ngx_http_realip_module, such that we don't need to send X-Real-IP in the request header but ngx.var.remote_addr has correct Client IP address. There is no load balancer before kong, i don't know which ip address ngx.var.remote_addr is picking up, when i am not adding trusted_ips in the kong.conf file.

ngx.var.remote_addr is picking up the ip of the machine that connected to Kong. E.g. the machine holding the IP which connected to Kong. If it is not client, then there must be something in between. E.g. a NAT. With trusted_ips you can tell Kong about the ip addresses that connect to Kong, if they are trusted. If they are trusted, the real_ip module can change the ngx.var.remote_addr to the one that was provided in somewhere else, configured with real_ip_header:
https://docs.konghq.com/0.14.x/configuration/#real_ip_header. The real_ip module will then store the ip of the machine that connected to Kong in ngx.var.realip_remote_addr.
If you trust all addresses then client can easily spoof the ip.

Thanks, I am running kong on docker for mac. I realized running kong on docker for mac may be problematic, since docker for mac itself is inside another virtual machine.


Pex wont save permissions.yml after editing

After editing my pex permissions.yml with notepad++, saving it and uploading it to my server. It wont update after i did /pex reload
Need help please

Sorry, that's a bad server host.  You'll have to talk to them about why their host is not updating the file.


Workshop 2 - Checklist

Checklist:

[ ]    Add a .gitignore file to your project
[ ]    Has a revert commit (don’t edit history: create a new revert commit!).
[ ]    Has a branch implementing a small feature.
[ ]    Has a merge commit (of the branch)
[ ]    Has a branch in conflict with main/master
[ ]    Try to merge and resolve the merge conflict
[ ]    Has working compilation and running instructions in the README.
[ ]    Has issues associated with each of your changes.

This is a checklist for us just so we can keep track of what progress has been done.


Compatibility with Symfony 3

Did you intend to update your bundle for Symfony3?
Maybe the only requirements is to update the composer.json.
I din't test it yet, but you may be more quick for that

Yes, the plan is to make it compatible with Symfony3. Not sure what is required... i will give it a try this we.

First thing is that the alias attribute is now deprecated:
https://github.com/egeloen/IvoryOrderedFormBundle/blob/d573184082cb8dee777718cd23bdc552825bf0b0/Resources/config/services.xml#L20

https://github.com/symfony/symfony/blob/5615b92cd452cd54f1433a3f53de87c096a1107f/UPGRADE-2.8.md#form
I am currently using this extension in 2.8 and will look out for other deprecations.

I have give a try and its more complicated than what I though. The issue is the library behind this bundle only have a deps on the symfony form component which does not ship the Kernel class where the VERSION const is available. So making the code compatible is more complex without this base constant and require to find methods which have been added/removed when the deprecattion has been added and then, check about them to make the code compat...

Maybe the orderFormBundle should ship it's own class and not depend your orderedForm?
Thanks anyway for looking at it

What do you mean by "ship it's own class" ? Removing the lib and put everything in the bundle? It will indeed simplify things but I would prefer keep the bundle/lib decoupled :)

Yes that's what I meant. It would be like having 2 projects doing very similar things.... (chiwh is not best)
Or maybe an other solution might be to have 2 branches on both project.
One for Symfony 2., and an other for 3., and handle it via composer versions requirements.
This way you would know the symfony version and don't rely on Version on Kernel as you mentionned, right?

Thanks for your quick update!


Fix capitalization

Fix the capitalization of the word "if" as it begins a new sentence.

Updated in all 3 meeting types


Generalising enumeration of nibbles to any word size

Codecov Report

Merging #3 into develop will increase coverage by 3.57%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           develop     #3      +/-   ##
=========================================
+ Coverage    96.42%   100%   +3.57%
=========================================
  Files            3      3
  Lines           28     28
=========================================
+ Hits            27     28       +1
+ Misses           1      0       -1

Impacted Files
Coverage Δ

src/nibbles.jl
100% <100%> (ø)
:arrow_up:

src/Twiddle.jl
100% <0%> (+100%)
:arrow_up:

Continue to review full report at Codecov.

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


fix a small annotation mistake in go extension

Commit Message: fix a small annotation mistake in go extension
Additional Description: N/A
Risk Level: low
Testing: N/A
Docs Changes: N/A
Release Notes: N/A

cc @wbpcode


add(helm): Use commonLabels in chart

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


forgejo: 9.0.2 -> 9.0.3, forgejo-lts: 7.0.11 -> 7.0.12

forgejo

https://codeberg.org/forgejo/forgejo/releases/tag/v9.0.3
https://codeberg.org/forgejo/forgejo/src/commit/403a81bdb50db190fb1374396d4768d753e0f3cd/release-notes-published/9.0.3.md
diff: https://codeberg.org/forgejo/forgejo/compare/v9.0.2...v9.0.3

forgejo-lts

https://codeberg.org/forgejo/forgejo/releases/tag/v7.0.12
https://codeberg.org/forgejo/forgejo/src/commit/403a81bdb50db190fb1374396d4768d753e0f3cd/release-notes-published/7.0.12.md
diff: https://codeberg.org/forgejo/forgejo/compare/v7.0.11...v7.0.12

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

tests pass on x86_64-linux
# nix-build -A forgejo.tests -A forgejo-lts.tests
/nix/store/3gkbdf52ar6622lfp5d80xsnjy5wv97j-vm-test-run-forgejo-mysql
/nix/store/wrwvmjnmy8n2cbb06ffy6x0ggqi45sy8-vm-test-run-forgejo-postgres
/nix/store/i9fdaxmfk8i190m78rzpnqdf60gqiv4d-vm-test-run-forgejo-sqlite3
/nix/store/2pfjpzmmx4zmjpqai9b9bwwlia2s8fh1-vm-test-run-forgejo-mysql
/nix/store/xczhbr3qx8kn81i9kvlwqry7s7jkd0i2-vm-test-run-forgejo-postgres
/nix/store/wkwmq37k14w5rr33sam1hagprpcadlj0-vm-test-run-forgejo-sqlite3

we'll use the borg while it exists :)
@ofborg build forgejo.passthru.tests forgejo-lts.passthru.tests

Since this is a security fix we'll want to update the 7.x on 24.05

Since this is a security fix we'll want to update the 7.x on 24.05

Manual backport in #364761


Fix Error un Add-AzurePS capabilities detection

There was an error in Vsts Agent when trying to detect azure capabilities
see :
https://github.com/Microsoft/vsts-agent/issues/353

Get-Module -Name  -ListAvailable isn't working when I tested on PowerShell 4. I've made a different fix: https://github.com/Microsoft/vsts-agent/pull/354

Worked on powershell 5
I dont have v4 to test you're i forgot that


Update execution_handler.py

Remove unused datetime import

I didn't realise the build was considered "failed" if it doesn't pass PEP8! Although, I guess that's pretty useful.

Yes the build was considered "failed" if it doesn't pass PEP8 because fixing flake8 error help to keep a clean code. but PEP8 checking is done after unit tests so you can see in Travis builds if unit tests are passing correctly because I think it's more important to have code that pass unit test than code that pass both...
but to be merged code should pass both.

I find linting to be very helpful in dynamically typed languages - agree RE PEP8 compliance.
@femtotrader is there an easy a way to simulate the travis-CI build locally, before we push code up to branches here?

Agreed. The project will attract more good software developers if we can keep code clean and unit tests passing.

@ryankennedyio simulate the Travis-CI build locally ? I never do this...
Travis rely on Docker containers.
I know that some Python dev use tox http://tox.readthedocs.io/en/latest/ to manage test, deployment...
You can also urge git to run some command lines before commit (run tests, PEP8...)
but I personaly don't do it and rely heavily on Travis.
I also don't use virtualenv (I use conda)


Update & fixes for us/ky/kenton

Changes:

Fixed Kenton geoid.
Switched to address endpoint.

Thanks!  And good catch on the geoid!


FPRS

Key
Value

View
https://database.shabados.com/sources/3/page/8/line/40

Source
Vaaran Bhai Gurdas Ji

Page
8

ID
FPRS

Line
swD AswD, sMgiq Plu pwieAw [6[

@Sarabveer need clarification on this. The vishraam we have is in between. Compare:
AswD, sMgiq in db vs. AswDsMgiq in proof


CommaSeperated with Multiple

I am getting the following error when using lookup editor with multiple =true   during save .Trying to save as integer in Databse .

has anyone found the resolution for issue mentioned in
CommaSeperated with Multiple = True   Issue = #1267

Hi @vinaysalian ,
according to my understanding, this works only if you have a string field as the destination of such a "multiple" value field because the field needs to be able to store something like this: "12,78,34". This cannot be stored within an integer field and therefore cannot not be de-serialized correctly.
With kind regards,
John

ok thanks


Cloud functions fixed for emulator and production

There was a bug where Cloud Functions was not executing on emulated Firebase (FB) servers, although working on production. This was because Cloud Functions requires firebase-admin SDK instead of normal user credentials to gain read and write access with Firestore.
Files changed:
app/functions/src/index.ts :

Initialize app with firebase-admin instead of usual clientCredentials
modified the createUserDocument function to use admin rights

For development and testing using Firebase emulator suite, the serviceAccount.json config file (which contains private key, etc.) needs to be set as an environment variable on your local machine to ensure Functions Emulator works correctly.
More documentation on Confluence.

@IkePureza I just need you to assign the service account json as a GitHub Action secret named "GOOGLE_APPLICATION_CREDENTIALS", just so the CI instance can access the firebase-admin SDK, thanks!


Add a License

Title says it all! Has the existing team discussed what license they would like to use?

Based on the fact the original creators are no longer active, and RPI has ownership of the project, we may want to err on the side of as permissive a license as possible for use, limit liability / warranty, but maintain ownership.
I would say MIT License.

I have opened pull requests for using MIT License as well as an extension there of, the ABSAJ License

I would suggest sticking with the MIT License. I don't believe the ABSAJ License is OSI approved 😛

I am going to leave it there in case anyone wants it. It's not OSI approved but neither is breathing so :man_shrugging:


Make sure dependencies in framework and docker or scratch install do not repeat

Modifies the requirements.txt accordingly


Remove version: Ryujinx.Ryujinx.Ava version 1.1.1151

All InstallerUrls returned 404 Not Found

https://github.com/Ryujinx/release-channel-master/releases/download/1.1.1151/test-ava-ryujinx-1.1.1151-win_x64.zip

Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/Ryujinx/release-channel-master/releases/download/1.1.1151/test-ava-ryujinx-1.1.1151-win_x64.zip
Response status code does not indicate success: 404 (Not Found).
(Automated message - build 896)


Helm on Termux Android

I'm trying to run helm on termux usin the arm64 build and it can't do any name resolution. I can use kubectl to reach the cluster.
$ helm init --client-only
Creating /data/data/com.termux/files/home/.helm/repository/repositories.yaml
Adding stable repo with URL: https://kubernetes-charts.storage.googleapis.com
Error: Looks like "https://kubernetes-charts.storage.googleapis.com" is not a valid chart repository or cannot be reached: Get https://kubernetes-charts.storage.googleapis.com/index.yaml: dial tcp: lookup kubernetes-charts.storage.googleapis.com on [::1]:53: read udp [::1]:60396->[::1]:53: read: connection refused

Output of helm version:
$ helm version
Client: &version.Version{SemVer:"v2.11.0-rc.1", GitCommit:"3a551d01d8e021679fd3d05ef9b97136e99b79e7", GitTreeState:"clean"}
Error: Get https://api.flow.dub.rd2.thingworx.io/api/v1/namespaces/kube-system/pods?labelSelector=app%3Dhelm%2Cname%3Dtiller: dial tcp: lookup api.flow.dub.rd2.thingworx.io on [::1]:53: read udp [::1]:48238->[::1]:53: read: connection refused

Output of kubectl version:
$ kubectl version
Client Version: version.Info{Major:"", Minor:"", GitVersion:"v0.0.0-master+$Format:%h$", GitCommit:"$Format:%H$", GitTreeState:"", BuildDate:"1970-01-01T00:00:00Z", GoVersion:"go1.11", Compiler:"gc", Platform:"android/arm64"}
Server Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.5", GitCommit:"32ac1c9073b132b8ba18aa830f46b77dcceb0723", GitTreeState:"clean", BuildDate:"2018-06-21T11:34:22Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):
Gardener shoot

Hi @luisdavim
Not sure why the issue would be ARM dependent. Can you run another simple ARM container, exec into it and test the name resolution?

I'm going to close this ticket due to inactivity, but please re-open if this still needs to be addressed. Thanks!

/reopen

I am experiencing absolutely the same behavior while the arm64 works fine, but there is no DNS resolution. What I see is that the binary tries to call the DNS server over IPv6 proto.

Here is output from helm list --debug. I am running 3.3.0-rc2 release.
Kubernetes cluster unreachable
helm.sh/helm/v3/pkg/kube.(*Client).IsReachable
        /home/circleci/helm.sh/helm/pkg/kube/client.go:101
helm.sh/helm/v3/pkg/action.(*List).Run
        /home/circleci/helm.sh/helm/pkg/action/list.go:141
main.newListCmd.func1
        /home/circleci/helm.sh/helm/cmd/helm/list.go:79
github.com/spf13/cobra.(*Command).execute
        /go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:842
github.com/spf13/cobra.(*Command).ExecuteC
        /go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:950
github.com/spf13/cobra.(*Command).Execute
        /go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:887
main.main
        /home/circleci/helm.sh/helm/cmd/helm/helm.go:93
runtime.main
        /usr/local/go/src/runtime/proc.go:203
runtime.goexit
        /usr/local/go/src/runtime/asm_arm64.s:1148```


[nrf fromtree] usb: remove Kconfig option USB_UART_CONSOLE

Adjust init level and priority of CDC ACM driver to regular serial drivers.
Limit scope of CONFIG_USB_UART_CONSOLE and finally remove this option.
https://github.com/zephyrproject-rtos/zephyr/pull/40428

Probably we only need the first four commits, WIP.


Added example sudo apt install

Short but useful hints to readers who wish to quickly get building with makefile up and running.

Hi, @Coarist
Thanks for your PR! I've merged this PR into the branch develop-1.8

Hi, @Coarist
Thanks for your PR! I've merged this PR into the branch develop-1.8


DCOS-46040: update rxjs to latest version

Update rxjs to version 6.3.3. This version is a few breaking changes from what we are currently on (5.4.3).
Todo:

[x] Update http-service (see dcos-labs/http-service#18)
[x] Update mesos-client (see dcos-labs/mesos-client#44)
[x] Update reactive-graphql (see mesosphere/reactive-graphql#2)
[x] Update data-service dependency (see dcos-labs/data-service#1)
[x] Fix broken tests

Testing
Be sure to npm install. Things shouldn't break. Navigate around the UI, specifically areas with heavy RxJS usage such as

Jobs pages
Package repositories
SDK plans tab

Trade-offs
N/A
Dependencies
Companion PR: mesosphere/dcos-ui-plugins-private#903

:tada: This PR is included in version 2.51.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


operator: issue a warning if the cluster does not meet minimum requirements

TODO in this:

[ ] Recalculate resources regularly
[ ] Make sure that the helm operations are idempotent
[ ] Add tests

Feel free to review in the meantime.


Tokenize without word details in wakati mode

Result.
Benchmarking Long text/bench-tokenize-str-long-text: Warming up for 3.0000 s
Warning: Unable to complete 20 samples in 5.0s. You may wish to increase target time to 5.4s, enable flat sampling, or reduce sample count to 10.
Long text/bench-tokenize-str-long-text
                        time:   [24.787 ms 25.572 ms 26.581 ms]
                        change: [-67.237% -65.589% -63.798%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 2 outliers among 20 measurements (10.00%)
  1 (5.00%) high mild
  1 (5.00%) high severe

     Running unittests src/lib.rs (target/release/deps/lindera_cc_cedict-5bf6eb9de4a20275)


Added virtual ValueEquals method

This will allow you to easily do custom equality checking on value, like this
public abstract class StringEnumeration<TEnumeration> : Enumeration<TEnumeration, string>
    where TEnumeration : StringEnumeration<TEnumeration>
{
    protected StringEnumeration(string value, string displayName, StringComparison comparison)
        : base(value, displayName)
    {
        Comparison = comparison;
    }

    public StringComparison Comparison { get; private set; }

    protected override bool ValueEquals(string value)
    {
        return Value.Equals(value, Comparison);
    }
}

Notice the StringComparison constructor argument. This will allow you to do culture/casing customization of string value comparison.

Any chance this could make it in?

@khellang can you talk about the use case for this? Are you wanting to have different types of StringEnumeration and they're going to need different StringComparisons? I just don't want to say yes or no without fully understanding the whole picture.

Yes, but not limited to. I just want a way to customize value equality checking. How can I do that today?

Thanks! :+1:


iOS-Build fails with imagepicker plugin 2.4.1

Hi,
I am not able to build our app for the iOS platform anymore with the latest imagepicker-plugin.
Here is my work environment:

Telerik AppBuilder 3.6.7
NativeScript 2.5.0
nativescript-imagepicker 2.4.1
iPhone 4S with iOS 8.4.1

This is the content of build.log:

Project "fanfiktion.tmp.proj" (default targets):
ios Build Tooling revision 2<IP_ADDRESS>r
Build Tooling Hostname administrators-mac-78.local
Warning: Support for Node.js ^4.0.0 is deprecated and will be removed in the next release of NativeScript. Please, upgrade to the latest Node.js LTS version.
Warning: Support for Node.js ^4.0.0 is deprecated and will be removed in the next release of NativeScript. Please, upgrade to the latest Node.js LTS version.
Warning: [xcode-build] ld: directory not found for option '-F/tmp/builds/dUcvLSAoPD7CbENLwb/app/output/NativeScript250/platforms/ios/DerivedData/Build/Intermediates/ArchiveIntermediates/NativeScript250/BuildProductsPath/Debug-iphoneos/AFNetworking'
[xcode-build] ld: directory not found for option '-F/tmp/builds/dUcvLSAoPD7CbENLwb/app/output/NativeScript250/platforms/ios/DerivedData/Build/Intermediates/ArchiveIntermediates/NativeScript250/BuildProductsPath/Debug-iphoneos/GCDWebServer'
[xcode-build] ld: directory not found for option '-F/tmp/builds/dUcvLSAoPD7CbENLwb/app/output/NativeScript250/platforms/ios/DerivedData/Build/Intermediates/ArchiveIntermediates/NativeScript250/BuildProductsPath/Debug-iphoneos/KSCrash'
[xcode-build] ld: directory not found for option '-F/tmp/builds/dUcvLSAoPD7CbENLwb/app/output/NativeScript250/platforms/ios/DerivedData/Build/Intermediates/ArchiveIntermediates/NativeScript250/BuildProductsPath/Debug-iphoneos/AFNetworking'
[xcode-build] ld: directory not found for option '-F/tmp/builds/dUcvLSAoPD7CbENLwb/app/output/NativeScript250/platforms/ios/DerivedData/Build/Intermediates/ArchiveIntermediates/NativeScript250/BuildProductsPath/Debug-iphoneos/GCDWebServer'
[xcode-build] ld: directory not found for option '-F/tmp/builds/dUcvLSAoPD7CbENLwb/app/output/NativeScript250/platforms/ios/DerivedData/Build/Intermediates/ArchiveIntermediates/NativeScript250/BuildProductsPath/Debug-iphoneos/KSCrash'
Error: 2017-02-09 10:46:25.645 xcodebuild[98677:11823113] [MT] IDEDistribution: -[IDEDistributionLogging _createLoggingBundleAtPath:]: Created bundle at path '/var/folders/kn/dw3_mg3d5rv8b69hhphnnv9w0000gp/T/NativeScript250_2017-02-09_10-46-25.644.xcdistributionlogs'.
1.2.840.1136<IP_ADDRESS>
2017-02-09 10:46:37.603 xcodebuild[98677:11823113] [MT] IDEDistribution: Step failed: <IDEDistributionThinningStep: 0x7ff841585140>: Error Domain=IDEDistributionErrorDomain Code=14 "No applicable devices found." UserInfo={NSLocalizedDescription=No applicable devices found.}
error: exportArchive: No applicable devices found.
Error Domain=IDEDistributionErrorDomain Code=14 "No applicable devices found." UserInfo={NSLocalizedDescription=No applicable devices found.}
** EXPORT FAILED **
Error: 'Build failed with error code 2'
Done building project "fanfiktion.tmp.proj" -- FAILED.

I hope this helps you to fix this issue. The build succeeded with the previous version of this plugin: 2.4.0.
Please tell me if you need any other details to find the reason for this issue.
Best regards,
felix

Hi @felix-idf ,
I am still unable to reproduce the exact problem with freezing the screen after selecting an image from the gallery. We will research further this issue and will provide some more info about this behavior.
In the meantime you could update your node to the latest available LTS version 6.9.5 and to build again the app, while using AppBuilder CLI.
This issue could be also related to the way you are getting the image. The image-picker will return an array from imageAssets and getImage() should be used to get the imageSource and to save the image in a temporary file in the app. For example:
asset.getImage().then(function(imagesource){
                    let folder = fs.knownFolders.documents();
                    let path = fs.path.join(folder.path, "Test"+counter+".png");
                    let saved = imagesource.saveToFile(path, "png");

                    if(saved){
                        var task = sendImages("Image"+counter+".png", path);
                        var item = new observable.Observable();
                        item.set("thumb", imagesource);
                        item.set("uri", "Test"+counter+".png");
                        item.set("uploadTask", task);

                        imageItems.push(item);
                    }
                    counter++;
                })

You could also review the full sample project here  and logic, which has been used in the app to get and send the image to the server.

Hi @tsonevn ,
I was on vacation, so sorry for the late response.
Actually, this issue was created, because of my troubles building the app with the latest imagepicker plugin.
According to the problem with freezing the screen after selecting an image, I will change the way I am getting the image as soon as possible.
But I still have trouble building the app for Android and iOS with the imagepicker 2.5.0 and 2.5.1.
Once again my testing environment:

NativeScript 2.5.0
tns-core-modules 2.5.1
imagepicker 2.5.1 (also tested 2.5.0)
Telerik AppBuilder 3.6.7

I noticed that the build works fine for Android when using the local NativeScript CLI (I cannot test for iOS, because we work on Windows operationg system).
If you think that this is absolutely AppBuilder-related and you do not feel responsible for it, then I am going to create an issue in the Telerik AppBuilder Forums with a link to this one here.
Here is the output from build.log for Android:

Project "fanfiktion.tmp.proj" (default targets):
android Build Tooling revision 2<IP_ADDRESS>r
Build Tooling Hostname d1ac1083b85d
Warning: Support for Node.js ^4.0.0 is deprecated and will be removed in the next release of NativeScript. Please, upgrade to the latest Node.js LTS version.
Warning: Support for Node.js ^4.0.0 is deprecated and will be removed in the next release of NativeScript. Please, upgrade to the latest Node.js LTS version.
Warning: FAILURE:
Warning: Build failed with an exception.
What went wrong:
Warning: Execution failed for task ':processF0F1F2F3F4F5F6F7DebugResources'.
Warning: > Error: more than one library with package name 'com.telerik.android.common'
Warning: * Try:
Warning: Run with
Warning: --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.
Warning: Command /tmp/builds/3bReAyVpiHybxpvlFk/app/output/NativeScript250/platforms/android/gradlew failed with exit code 1
Error: Build failed during execution
Error: 'Build failed with error code 127'
Done building project "fanfiktion.tmp.proj" -- FAILED.

And for iOS:

Project "fanfiktion.tmp.proj" (default targets):
ios Build Tooling revision 2<IP_ADDRESS>r
Build Tooling Hostname administrators-mac-14.local
Warning: Support for Node.js ^4.0.0 is deprecated and will be removed in the next release of NativeScript. Please, upgrade to the latest Node.js LTS version.
Warning: Support for Node.js ^4.0.0 is deprecated and will be removed in the next release of NativeScript. Please, upgrade to the latest Node.js LTS version.
Warning: [xcode-build] ld: directory not found for option '-F/tmp/builds/F99wChsXKHbphf7oZd/app/output/NativeScript250/platforms/ios/DerivedData/Build/Intermediates/ArchiveIntermediates/NativeScript250/BuildProductsPath/Debug-iphoneos/AFNetworking'
[xcode-build] ld: directory not found for option '-F/tmp/builds/F99wChsXKHbphf7oZd/app/output/NativeScript250/platforms/ios/DerivedData/Build/Intermediates/ArchiveIntermediates/NativeScript250/BuildProductsPath/Debug-iphoneos/GCDWebServer'
[xcode-build] ld: directory not found for option '-F/tmp/builds/F99wChsXKHbphf7oZd/app/output/NativeScript250/platforms/ios/DerivedData/Build/Intermediates/ArchiveIntermediates/NativeScript250/BuildProductsPath/Debug-iphoneos/IQKeyboardManager'
[xcode-build] ld: directory not found for option '-F/tmp/builds/F99wChsXKHbphf7oZd/app/output/NativeScript250/platforms/ios/DerivedData/Build/Intermediates/ArchiveIntermediates/NativeScript250/BuildProductsPath/Debug-iphoneos/KSCrash'
[xcode-build] ld: directory not found for option '-F/tmp/builds/F99wChsXKHbphf7oZd/app/output/NativeScript250/platforms/ios/DerivedData/Build/Intermediates/ArchiveIntermediates/NativeScript250/BuildProductsPath/Debug-iphoneos/MNFloatingActionButton'
[xcode-build] ld: directory not found for option '-F/tmp/builds/F99wChsXKHbphf7oZd/app/output/NativeScript250/platforms/ios/DerivedData/Build/Intermediates/ArchiveIntermediates/NativeScript250/BuildProductsPath/Debug-iphoneos/SSSnackbar'
[xcode-build] ld: directory not found for option '-F/tmp/builds/F99wChsXKHbphf7oZd/app/output/NativeScript250/platforms/ios/DerivedData/Build/Intermediates/ArchiveIntermediates/NativeScript250/BuildProductsPath/Debug-iphoneos/AFNetworking'
[xcode-build] ld: directory not found for option '-F/tmp/builds/F99wChsXKHbphf7oZd/app/output/NativeScript250/platforms/ios/DerivedData/Build/Intermediates/ArchiveIntermediates/NativeScript250/BuildProductsPath/Debug-iphoneos/GCDWebServer'
[xcode-build] ld: directory not found for option '-F/tmp/builds/F99wChsXKHbphf7oZd/app/output/NativeScript250/platforms/ios/DerivedData/Build/Intermediates/ArchiveIntermediates/NativeScript250/BuildProductsPath/Debug-iphoneos/IQKeyboardManager'
[xcode-build] ld: directory not found for option '-F/tmp/builds/F99wChsXKHbphf7oZd/app/output/NativeScript250/platforms/ios/DerivedData/Build/Intermediates/ArchiveIntermediates/NativeScript250/BuildProductsPath/Debug-iphoneos/KSCrash'
[xcode-build] ld: directory not found for option '-F/tmp/builds/F99wChsXKHbphf7oZd/app/output/NativeScript250/platforms/ios/DerivedData/Build/Intermediates/ArchiveIntermediates/NativeScript250/BuildProductsPath/Debug-iphoneos/MNFloatingActionButton'
[xcode-build] ld: directory not found for option '-F/tmp/builds/F99wChsXKHbphf7oZd/app/output/NativeScript250/platforms/ios/DerivedData/Build/Intermediates/ArchiveIntermediates/NativeScript250/BuildProductsPath/Debug-iphoneos/SSSnackbar'
Error: 2017-02-23 12:51:41.201 xcodebuild[61844:40453880] [MT] IDEDistribution: -[IDEDistributionLogging _createLoggingBundleAtPath:]: Created bundle at path '/var/folders/t9/xhtxg7912rs31d0f53pslcgh0000gp/T/NativeScript250_2017-02-23_12-51-41.199.xcdistributionlogs'.
1.2.840.1136<IP_ADDRESS>
2017-02-23 12:51:57.414 xcodebuild[61844:40453880] [MT] IDEDistribution: Step failed: <IDEDistributionThinningStep: 0x7fbb1d4bc6a0>: Error Domain=IDEDistributionErrorDomain Code=14 "No applicable devices found." UserInfo={NSLocalizedDescription=No applicable devices found.}
error: exportArchive: No applicable devices found.
Error Domain=IDEDistributionErrorDomain Code=14 "No applicable devices found." UserInfo={NSLocalizedDescription=No applicable devices found.}
** EXPORT FAILED **
Error: 'Build failed with error code 2'
Done building project "fanfiktion.tmp.proj" -- FAILED.

Hoe you can figure out what is going wrong here.
Thanks in advance and best regards,
Felix

As I already commented in my original post, the problem with freezing the screen after selecting an image from the gallery is solved. I used a wrong way of getting an image from the picker.
The remaining problem is that the build fails when I try to build the app with the imagepicker plugin 2.5.1 within the AppBuilder CLI. So it seems to be not a general problem with the plugin.

Hi @felix-idf,
In my further research, I found that this behavior could be related due to a problem with installation one of the plugin's dependencies.
nativescript-imagepicker plugin uses nativescirpt-telerik-ui and its RadListView component to display the images from the gallery.
While adding nativescript-imagepicker version 2.5.1, nativescirpt-telerik-ui has not been installed properly, which could cause this issue.
To fix this, you could add nativescript-imagepicker manually with appbuilder plugin add nativescript-telerik-ui to add again the latest available version of nativescript-imagepicker version 2.5.1. then try to rebuild the app.
package.json
{
  "dependencies": {
    "nativescript-imagepicker": "2.5.1",
    "nativescript-telerik-ui": "1.6.1",
    "tns-core-modules": "2.5.0"
  },
  "devDependencies": {}
}

Hope this helps.


Google feedback on TS 4.7-beta

This GitHub issue contains feedback on the TS 4.7-beta release from the team that is responsible for keeping Google's internal software working with the latest version of TypeScript.
Executive summary

Some changes to our TypeScript code are required to make it compile with TS 4.7.
Detail sections below explain the changes to our code we expect to make to unblock the upgrade.

Impact summary

Change description
Announced
Libraries affected

Type Parameter No Longer Assignable to {} in strictNullChecks
yes
0.24%

lib/d.ts changes
yes
0.022%

TS2774 & TS2801 now fire for secondary conditions in if statements
no
0.017%

TS2731 now fires for keyof T
no
0.008%

Unclassified
-
0.014%

The Announced column indicates whether we were able to connect the observed change with a section in the TS4.7-beta announcement.
The following sections give more detailed explanations of the changes listed above.
Changes which were announced
Type Parameter No Longer Assignable to {} in strictNullChecks
We see type errors similar to the ones mentioned in the release announcement.
We're looking into adding extends object to generic type parameters where possible and casts to any to silence the error everywhere else.
lib/d.ts changes
We support the typing improvements. Generally it's clear what's changing.
We observed the following breakdown within this category:

fetch now also accepts URL in addition to RequestInfo for the first parameter

AbortSignal has additional members reason and throwIfAborted

ImageData has additional member colorSpace

HTMLVideoElement has additional members requestVideoFrameCallback and cancelVideoFrameCallback

Object.assign now rejects null/undefined for the first parameter

Object.freeze has an additional overload, which causes some type errors like:
interface Data { prop: Array<'a'|'b'> }
export const DATA: Data = Object.freeze({prop: ['a']});
//           ^^^^ Type 'Readonly<{ prop: string[]; }>' is not assignable to type 'Data'.

We expect to add casts to any to silence the error and apply proper fixes over time.
Changes which were not announced
TS2774 & TS2801 now fire for secondary conditions in if statements
This wasn't specifically mentioned in the release announcement, but looks like a decent improvement. Example:
declare let flag: boolean;
declare function foo(): void;
declare function bar(): Promise<void>;
if (flag && foo) foo();
//          ^^^ This condition will always return true since this function is always defined. Did you mean to call it instead?(2774)
if (flag && bar()) {}
//          ^^^^^ This condition will always return true since this 'Promise<void>' is always defined.(2801)

We expect to add casts to any to silence the error and apply proper fixes over time.
TS2731 now fires for keyof T
This wasn't specifically mentioned in the release announcement, but looks like a decent improvement. Example:
function foo<T>(key: keyof T) {
  console.log(`Key: "${key}"`);
  //                   ^^^ Implicit conversion of a 'symbol' to a 'string' will fail at runtime. Consider wrapping this expression in 'String(...)'.(2731)
}

We expect to add casts to any to silence the error and apply proper fixes over time.

Is there something I should be watching out for w.r.t. the aforementioned fetch change? Making it more permissive shouldn’t be a breaking change…

I think the specific thing I found was an instance of trying to stub/mock out a custom fetch, where the code ended up with issues because it hadn't handled the new case.

I think that you can instead write string & keyof T in some of those examples.

Thanks for the hint. We'll check if this is something we can add automatically to the affected code.

Type Parameter No Longer Assignable to {} in strictNullChecks

This is one of the biggest ones I think we have seen manual intervention with. How are you feeling about these updates right now?

Null check improvements seen generally nice to have to me. We haven't dug deeper into the errors, yet. It's hard to say for me at the moment if this helped catch real errors. I'll try to remember to report back when I know more. Some notes, though:

I appreciate the "This type parameter probably needs an extends object constraint." diagnostic. I hope that makes it possible to automatically add extends object to the relevant locations. Although I suspect adding extends object might cause a ripple effect up the call chain in certain places. I hope it's not too bad, though. 🙂

It would be great to have this diagnostic in more situations. Here's one example I found in our code base, that's fixed by adding extends object, but that doesn't have the special diagnostic:
function extract<T>(obj: {}, other: Partial<T>) {
  (obj as T);
// ^^^^^^^^
// Conversion of type '{}' to type 'T' may be a mistake because neither type sufficiently overlaps with the other. If this was intentional, convert the expression to 'unknown' first.
//     'T' could be instantiated with an arbitrary type which could be unrelated to '{}'.(2352)
}

Is there something I should be watching out for w.r.t. the aforementioned fetch change? Making it more permissive shouldn’t be a breaking change…

Similar to what Daniel said, we see this error in stubs/mocks. E.g.:
window.fetch = async (r: RequestInfo) => {
^^^^^^^^^^^^ TS2322: Type '(r: RequestInfo) => Promise<Response>' is not assignable to type '{ (input: RequestInfo | URL, init?: RequestInit | undefined): Promise<Response>. Types of parameters 'r' and 'input' are incompatible. Type 'RequestInfo | URL' is not assignable to type 'RequestInfo'.
  // do some tests and return mocked response...
};

I think those failures are expected.

That being said, I thought fetch() always accepted an URL? I've never passed anything but a string to it...

You're right that string was always possible. It's now also possible to pass e.g. new URL(...). fetch changed from accepting RequestInfo to RequestInfo | URL. Where type RequestInfo = Request | string;.

Ah, see, I thought URL itself was an alias for string and got really confused.  Not a huge fan of type aliased unions containing primitive types for this reason.  Thanks!

It would be great to have this diagnostic in more situations.

So for the example that you gave
function extract<T>(obj: {}, other: Partial<T>) {
  (obj as T);
// ^^^^^^^^
// Conversion of type '{}' to type 'T' may be a mistake because neither type sufficiently overlaps with the other. If this was intentional, convert the expression to 'unknown' first.
//     'T' could be instantiated with an arbitrary type which could be unrelated to '{}'.(2352)
}

@weswigham's PR at https://github.com/microsoft/TypeScript/pull/48861 should allow casting between {} and T.

Thanks, @DanielRosenwasser. I didn't realize this is unintentional.
I've looked at a handful of the other errors that are probably caused by "Type Parameter No Longer Assignable to {} in strictNullChecks" and don't have the new diagnostic. I think they are all variations of the example I gave and can be resolved by adding extends unknown to the type parameter. This makes me think they will all be fixed by #48861.

We imported 4.7-rc. As expected, we no longer see the breakages caused by "Type Parameter No Longer Assignable to {} in strictNullChecks", since this has been backed out in #48923.
We don't see any other major changes compared to the beta.
Though we did find that the Object.freeze typing addition causes a build failure because we don't include BigInt typings, yet. See #49101.


dns_resconf_search doesn't preserve FQDNs

If a domain is fully-qualified dns_resconf_search should return it and only it, skipping search domain generation.

Fixed in commit 3501e7ef84614b43e8aadbf0b98f3511cadb50f8.


Update finagle-core, finagle-netty4 to 21.11.0

Updates

com.twitter:finagle-core
com.twitter:finagle-netty4

from 21.4.0 to 21.11.0.
Changelog
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.twitter" } ]

labels: library-update, early-semver-minor, semver-spec-minor

Superseded by #276.


getDefaultCalendar throws if no default calendar

It is possible (on Android, at least) for there to be no default calendar. In such cases, getDefaultCalendar throws an error. It would be better if it returned undefined, to discriminate between an actual error and the absence of a default.
To reproduce:

Create a new Android emulator, and do NOT sign in to a google account, or remove your google account from an existing emulator.
Run the capacitor-calendar example app.
Get the default calendar. An error with the message "No default calendar found" is thrown.

Btw, although the Capacitor docs recommend returning undefined instead of null, there is a big debate in the Javascript community over which value better represents "non-existent". In the case of Capacitor, there is one very good reason to return an object with a null value in its result property instead of returning undefined: object destructuring.
Here's how the code would be with null:
const { result: defaultCalendar } = await CapacitorCalendar.getDefaultCalendar()

if (defaultCalendar) {
  console.log(defaultCalendar.title)
}

If getDefaultCalendar returned undefined, the above code would break. We would have to do this:
const result = await CapacitorCalendar.getDefaultCalendar()

if (result) {
  const defaultCalendar = result.result
  console.log(defaultCalendar.title)
}

I hope you agree the first option is much cleaner.

Empty string would also work since a truthy test would work the same.


chore(DTFS2-7439): align some unit test naming conventions

Introduction :pencil2:

Some tests were using an expectedResult convention, when expected should be used.
Some tests were using an matchTest convention, when result should be used.

Resolution :heavy_check_mark:

Replace all instances of expectedResult with expected.
Replace all instances of matchTest with result.

LGTM 👍🏼 However need a ticket number for the PR please.

Have created and updated the PR title, DTFS2-7439


EJECT statements before IDENTIFICATION DIVISION preventing syntax checking

Originally posted by @Morridt in https://github.com/IBM/zopeneditor-about/issues/7#issuecomment-533350782
Our code has program comments above the IDENTIFICATION DIVISION that also includes EJECT statements.  This prevents syntax checking.
EJECT statements after IDENTIFICATION DIVISION work ok and have no affect on syntax checking.
  *--------------------------------------------------------------*
  *--------------------------------------------------------------*
       EJECT
   IDENTIFICATION DIVISION.
   PROGRAM-ID. PGM021.
   AUTHOR. MAINFRAMEWIZARD.
   DATA DIVISION.
   WORKING-STORAGE SECTION.
   01 WS-SUB   PIC 9  VALUE ZEROS.
   PROCEDURE DIVISION.
   10000-MAIN-PARA.
         MOVE 5 TO WS-SUB.
         MVE WS-SUB TO WS-SUB
         STOP RUN.

Just an update. We do have a fix for it. We are just waiting for a release window related to legal approvals required. But it will fixed in our next update.

Thanks!

This issue was fixed in version 0.4.1 now available on the VS Code Marketplace.

Thanks Peter.... Confirmed all good from my side.


onRefresh method is launched whenever i scroll back to the top of the list !!

nice library dude cheers, :)
the onRefresh callback is launched whenever the list is scrolled back to the top elements
an other question is this library tested with a recyclerview ?
thanks :)

The same with  amineMpdam .Had it sovled?


Remove The Big Lebowski reference in the HIG

Look I get it. I've watched the movie and find the (full) quote hilarious. It's great at getting the point across in English. However, I really think this should be removed for the following reasons:

Translations; it's not that practical to really translate or link to IMDB with this string.
It uses cultural specific references that only make sense to North Americans or Europeans which violates the HIG.
The complete quote is a bit NSFW. I'm not a fan of the SJW crowd, but the full quote does violate the community code of conduct.

Yeah, well, you know, that's just, like, your opinion, man.

(I couldn't resist)
However, I'm inclined to agree with you. We can probably find a better way to say that design isn't just the opinion of designers.

I oppose this because I consider it ridiculous.

Linux (and UNIX before) is full of western pop culture references. If we want to follow this consequently, we will have to patch alot applications for no good reason. As developers we're in general heavily influenced by this.
elementary OS is primarily developed by people influenced by western pop culture. I don't believe there is a single contributor on Slack who doesn't know the movie (or at least heard the name and/or the quote in question).
While people unfamiliar with the reference might be a bit puzzled, I'm pretty sure they're intelligent enough to search for it if they feel the need to understand it. We're on the Internet after all.
The quote in question is not only part of western pop culture but also part of "Internet" culture - It's been posted and referenced innumerable times online. Nearly everyone participating in English-speaking communities has encountered it at least once. The assumption developers do not speak English at all is absurd (and aren't able to participate in those communities), nearly all resources for developers are written in English.
Assuming this is offending or unsettling is highly speculative. As long as there is nobody from another cultural sphere actually complaining I don't see why we should change it. It doesn't bother anyone most likely anyway.
Violence and swearing is part of our reality, whether you like that or not. The Internet is not a "safe space". The full quote is not on the page anyway so I'm not sure why the classification of the "full quote" has relevance.

I've elaborated my thoughts below:

My most important concern is that It's tricky to translate. This is both a technical issue, because it consists of a link with words in it that are heavily tied to English, and because it uses some slang. The translated string would also be less effective than the English one depending on the culture and language conventions. If some translators could comment on this however, that would be great.

Yes cultural references are great, but this sentence, despite being in a humorous tone, is meant to communicate an important message, that "design isn't just an opinion". Including Easter eggs is one thing, but relying on pop culture to communicate a message is another thing. It sounds great in English, and I find it hilarious, but what about in other languages? The fact that someone would have to Google it up makes the sentence less effective. The HIG should be as clear as possible in multiple languages, and shouldn't rely on pop culture to say something important; in fact, that's what the HIG says.

The complete linked quote isn't that relevant to the idea it's trying to communicate. If it linked to an informative article, fine, except that it links to a quote where it's just a couple of guys cursing. This something I personally have no problem with, it's just that it's out of context. The quote doesn't expand on the Dude's idea that "Yeah, well, you know, that's just, like, your opinion, man," instead it just shows the pieces of dialog that come before and after that quote which makes no sense to people who haven't watched the movie or that clip.

My last point, which is my most insignificant important point, has nothing to do about "safe spaces", SJWs or that nonsense which I personally despise. Yes violence and swearing is real, but my point is that it's also being slightly hypocritical for elementary to link to that with their current Code of Conduct. Also I consider it to be slightly NSFW, as in Not Safe For Work, in some corporate environments. I personally have no problem with the quote.

I actually just realized that for example the German translation does not contain the reference (nor the link) and just says "design is not just a matter of taste" (Design ist nicht nur Geschmackssache). It looks like the translators already took into account not to translate language-specific references.

If the translators have no issue with it, that part is settled then.
My reason for creating this issue isn't about political correctness, but rather as a  question how effective it is at communicating. Since it looks like that is a non-issue, cool.
However, I still stance on my last two concerns, that linking to the quote is out of context and that it's less accessible in some corporate environments.

After a quick read, (begging your pardonif I forget something), if we want to keep this quote, this is something we should comment in Transifex (e.g "care, quote from The Big Lebowski'')

I do think having pop culture references makes the docs more fun. Docs are boring. If we can make them fun somehow, I'd like to do that. That said, I understand the concerns. I'd recommend that we:

[ ] Remove the IMDB link. Either you get the reference or you don't. Linking the quote doesn't make it any clearer and brings in those language concerns. I don't think we need to link it.

[ ] Make a note for translators that they shouldn't try to translate this word for word and it's okay (maybe even recommended) not to translate the quote. As Martin pointed out, I think good translations should be more about carrying the message across and not try to hard to be exact literal translations. If this doesn't make sense in languages other than English, it shouldn't be translated.

I agree with @danrabbit's suggestions.


Watch method to track config file changes

Add Watch method reloads the config struct every time config file has changed.
This functionality does not work for environment variables as they are kind of designed to stay unchanged for a running process.

Codecov Report
Patch coverage: 63.15% and project coverage change: -1.64% :warning:

Comparison is base (d717df0) 85.22% compared to head (dbb6344) 83.58%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #15      +/-   ##
==========================================
- Coverage   85.22%   83.58%   -1.64%
==========================================
  Files           1        1
  Lines         176      195      +19
==========================================
+ Hits          150      163      +13
- Misses         18       22       +4
- Partials        8       10       +2

Flag
Coverage Δ

unittests
83.58% <63.15%> (-1.64%)
:arrow_down:

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

Files Changed
Coverage Δ

config.go
83.58% <63.15%> (-1.64%)
:arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add datepicker for date fields

This is a nice datepicker: https://eonasdan.github.io/bootstrap-datetimepicker/

I've added a simple html5 date field for orders


implement archetype_container2 removeComponent

Remove a component from an entity
Specialized storage for "empty" entities outside of the three structure and handling of this in the entity ref

Resolved in efaab2829d50cdb4f287a220b601f1a5013ca339


test: Fix typo

In SearchRouteV2R2RSpec, private val writeTestDataFiles = true should be false so the test is actually run.

@SepidehAlassi So you can see the kind of silly mistakes I make. :)

@SepidehAlassi So you can see the kind of silly mistakes I make.

@SepidehAlassi Thanks!


feat: editable cells on hover support

Would like to see some attempts/iteration on a PresentationContext-based approach, changing borderless: true to "something something hover" before this lands. Happy to chat/VC about it/approaches.

+1 to this. Still good work here @0ces! I found I was able to fix a few other issues by updating the PresentationContext through correctly using getTableStyles so we should start re-training muscles to look at that first before adjusting the cell through column config 🙏🏽


gh-628 - Add information to README regarding Maven Central.

Current coverage is 61.19% (diff: 100%)

Merging #637 into develop will increase coverage by 0.05%

@@            develop       #637   diff @@
==========================================
  Files           354        354
  Lines         10995      10995
  Methods           0          0
  Messages          0          0
  Branches       1598       1598
==========================================
+ Hits           6722       6728     +6
+ Misses         3739       3735     -4
+ Partials        534        532     -2

Powered by Codecov. Last update 2f61643...cac656b


Update builder to consider package constraints when skipping binaries (grpc fix)

Update wheel builder to consider package constraints when skipping binaries.
The basic idea is to ensure that --skip-binary can reference packages that are not explicit requirements to always force grpc to be built from source.
Reviewer note: An alternative could be to add grpcio to be an explicit requirement for an integration (e.g. nest/pubsub/google, etc)
For https://github.com/home-assistant/core/issues/56669

@frenck This is required before we do attempt #3

Thanks.

Would one of you mind initiating a wheels release?
Can I bump .github/workflows/wheels.yml in core or do i need to wait for a bot to do it?

Can I bump .github/workflows/wheels.yml in core or do i need to wait for a bot to do it?

You can bump it.

Great, once a new release is made i'll send the PR to update -- i have it ready to go, and ready to exercise one last attempt this week to bump grpcio.

Hi,  Can one of you make a wheels release? My impression is I don't have permission to do that.

We can't as the build are failing as it seems.

Probably related to https://github.com/home-assistant/wheels/commit/2b301602d75b3e51d712722efd419fee5dd95bec

ok.... I've sent https://github.com/home-assistant/wheels/pull/315 to revert?

Yes, discussed it last night with Pascal and others. That is OK to revert, after that we can move forward


Ractive examples

In order to keep Ractive rocking, I think providing examples is a good way to go forward:
https://dagnelies.github.io/ractive-examples/
Contributions welcome, make it even greater. Possibly even linking it in the main site. Sorry if the style is not up to date. Most of it was done pre-theme-change.

Nice!

You can also suggest the next example you want ;)

Hey @fskreuz, can you perhaps add the tag "help wanted"? Perhaps it'll attract some contribution. Thanks

bravo

nice ;-)

This is awesome and thanks for this. I want to see an example doing it the Ractive way where there are two sets of drop-down. One is list of US states and the other one is list of counties. Data for both are available as JavaScript array of objects. Also the requirement is to use select2 library to turn them into better select dropsdowns.
Well this is how I have done it, I wrote a decorator that on it's constructor calls select2 on the nodes.
But to dynamically change the options of county dropsdown I look for select on change and then build an Array depending on the state selected and rebuild a new array finally passing it to ractive set. This sounds and feels clunky because the problem was select2 changes the behavior of drop down and Ractive on- change doesn't fire when drop down value is changed. I am sure I am not adapting it correctly.

@shakeelosmani Sure, sounds simple enough. Can you please provide the data and the way you currently do it now?

@dagnelies sure here you go.
counties.zip
link to select2 library:
https://select2.org/
The states have the keys in id property that is the key for counties for that state.
Thanks
Shakeel

@dagnelies
my implementation of functionally works correctly and here is a step by step images:

When the state of Hawaii is selected then counties shown are only for Hawaii

similarly when state of Alaska is selected the counties of Alaska are shown.
The code I have is as follows:
`

`

Well, I added the basic "dependent selects" example which is fairly straightforward.
The issue you face is because of the "select2" plugin. Making that one data-bound is a bit more work, and specific to that library. A re-usable component widget would be the way to go IMHO, like in the color picker example. However, doing so properly with that library looks like some work.
The one without plugin is here:
https://dagnelies.github.io/ractive-examples/forms/dependent_selects.html
...note that even basic selects respond to keyboard navigation.

Thank you. Looks good I guess I need to brush up on mustache. I will go without select2 I guess.

Late for the dropdown party, but I have a word or two on this topic. According to my experiments, a dropdown is the most complex component to build as it has lots of edge cases. Here is my dropdown component that might provide you another perspective (Note: Pug and Livescript can be compiled to HTML and Javascript respectively).
I would also suggest you use your own simple dropdown with sifter.js for searching functionality. I can not provide a standalone dropdown component (it's bound to scada.js at this point) but here is my implementation:
.ui.segment
    ack-button(on-click="getCities") Get Cities

    dropdown(
        data="{{countries}}"
        selected-key="{{selectedCountry}}"
        async on-select="countrySelected")

    dropdown(
        data="{{cities}}"
        selected-key="{{selectedCity}}")

    .ui.basic.message
        Selected country: #[.ui.green.label {{selectedCountry}}]
        and selected city: #[.ui.orange.label {{selectedCity}}]

# below are Ractive Handlers
'getCities': (ctx) ->
    btn = ctx.component
    btn.state \doing
    err, res <~ request
        .get 'https://restcountries.eu/rest/v2/all'
        .end
    if err => return btn.error err
    @set \countries, [{id: ..alpha2Code, name: ..name} for res.body]
    btn.state \done...

'countrySelected': (ctx, item, progress) ->
    console.log "Selected country is: #{item.id}"
    @set \cities, []
    @set \selectedCity, ''
    progress!
    err, res <~ request
        .get 'https://andruxnet-world-cities-v1.p.mashape.com/'
        .query {query: item.name, searchby: \country}
        .set 'X-Mashape-Key', 'some-random-api-key'
        .set 'Accept', 'application/json'
        .end
    if err => return btn.error err
    console.log "cities are: ", res.body
    @set \cities, [{id: .., name: ..} for (group-by (.state), res.body |> Object.keys)]

...and here is the result:

Hello, I want to do this where I have dependent fields that are databound and they update values dependent on change of certain inputs. I have a fiddle here : https://jsfiddle.net/pqts780m/
On the script side the goal of the code is written in the comments section. Any help or guidance is truly appreciated.
Thanks

@shakeelosmani you were close, but you can't have computed values update other computed values that depend on computed values because there's no definitive stored value to start with as a base. If you make the down payment amount a plain value, then setting the percent can update it and changing the amount will cause the percent to invalidate and update any of its deps. Here's an updated fiddle. Interdependent computeds can get pretty tricky, which is why I typically add a calculate function to the instance, turn off two way binding on the inputs, and trigger the calculate method in on-change.

@evs-chris  Yes you were right we are better off with the event driven model. I had managed to get it working. This is the fiddle: https://jsfiddle.net/rana_tigrina2002/15c4fs7d/
Thanks for all your help, it's cool to know that other way also which you managed on the updated fiddle.

Hi, I was on vacation, so it took me a while :)
I think the whole example/challenge boils down to two input values A and B related by a formula.
They depend on each other because changing either updates the other, while avoiding recursive loops.
Here is the approach I would choose:
https://jsfiddle.net/pqts780m/20/

@dagnelies that is cool and I must agree to the point raised about making it event driven means you lose out on change made to models, although in this particular case it wont cause a problem but I can see how that would affect in other situations. Thank you!

one more demo https://dagnelies.github.io/ractive-examples/misc/tree.html

Anyone have an example of an async combobox(start typing, queries a url with what has been typed and populates a list below a text input to choose from)?  Preferably in plain old javascript/ractive?

@tmilker this uses a slightly convoluted macro to do it, but if it works for you, it's an already consumable plugin available here (or UMD or plain ractive component).
The gist of it is, bind the on-input event of the input element to call a function that replaces a promise for a list in the data. Then use {{#await list then items}}{{#each items}}...{{/each}}{{/await}} to display the list when the promise resolves.

@evs-chris Thanks.  I think I may just have to use your description of how it works and write my own.  I don't want to add pre-processing dependencies to a legacy application which I think your solution requires.  If I'm just not seeing the .min.js I can just include with a

There is a umd here, which should be a drop in for a script tag. It does require a fairly recent version of ractive, though, so it may not work depending on how legacy the app is.

@evs-chris Well, thankfully using ractive is new in this project so I went from 1.3.4 to 1.3.6 but still having issues.  Sorry to be a nuisance about this but do you have a Playground example that uses the UMD because I can't get it to work when I include the script and try to use the component in my template.  I've checked Ractive.components and it doesn't appear to register itself so I tried registering it myself but I get:
Autocomplete.min.js:1 Uncaught (in promise) TypeError: Cannot set property 'RMAutocomplete' of undefined

As long as it gets loaded after ractive, it should work. I believe in the global case, it automatically registers as Ractive.partials.RMAutocomplete, but you can install it normally using Ractive.use(RMAutocomplete.plugin()).

@dagnelies This issue page must be pinned to the repository and change title to "You must check all the Ractive Example before creating a new issue"
Why?

Because all examples are real world business logics that most web applications are using.

What you think?

I think it could even be embedded into the main site ;)


Upgrade dev at dep-local night

@freva please review and merge.

16:26:04 [ERROR] Failures:
16:26:04 [ERROR]   ControllerApiTest.testControllerApi:53 maintenance.json expected:<...intainer"},{"name":"[]EndpointCertificateM...> but was:<...intainer"},{"name":"[DeploymentUpgrader"},{"name":"]EndpointCertificateM...>

PTAL :)


Compatibility with CTHELPER

Hi,
Using wslbridge.exe with the good known cthelper.exe tool I see some errors. For example:

When calling to tmux or screen inside the WSL shell, then if I resize the terminal window some errors appears.

I feel some incompatibility exists at time.
Please, can you provide better compatibilty with cthelper?
Thank you!

Where is cthelper.exe?

Hi @Biswa96 ,
You can found the source it in some repositories. For example:

iPuTTY: https://github.com/iPuTTY/cthelper
KiTTY: https://github.com/svn2github/kitty/tree/master/trunk/cthelper

And you can download the binary from several locations. For example:

http://www.9bis.net/kitty/?page=PuTTYCyg&zone=en

I hope you can improve the compatibility.
Regards.

And how did you use it with wslbridge?

Hi @Biswa96 ,

And how did you use it with wslbridge?

With KiTTY and other PuTTY mods with the CygTerm patch, to run the WSL SHELL.
The current issues are with the WSL Shell, as the Cygwin Shell works perfect.  So if the tandem cthelper-wslbridge is improved then we should use both implementations with the same TERM tools.
I hope you can do something.
Regards.


Add Couter Device Limit in Admin

Added as
https://github.com/wyx2685/v2board/commit/9db84a4a4a38c58e39286f221049a04a256ddd64


openfast Simulink model Time step output

Hello, in the openfast Simulink model, I want to build a reinforcement learning environment to control the pitch angle and obtain maximum power output. During this process, I hope that each time step can output current data, excluding historical data. How can I proceed?

Dear @yinghanguan,
OpenFAST can output the current data at each time step, but I'm not sure what you mean when you refer to "excluding historical data".
Best regards,

When I run two time steps in the way shown in the above figure, the output of out. timeout is shown in the following figure,I hope to obtain the last column, which means that the output for each time step is [1,30]

Dear @yinghanguan,
I'm not sure I really understand your question, but normally the OutData output from the OpenFAST S-Function block is a one-dimensional array containing the values of the OpenFAST outputs at the current time step.
Best regards,

In my simulation results, it is obvious that the output is a two-dimensional array, and the output contains data for both the current and historical time steps, which is different from what I wanted. I only need data for the current time step

It doesn't look like you are using the OutData output from the OpenFAST S-Function block, but rather, you are looking at the output of a bock you added.  What does OutData look like directly?


Question: any chance this will work on native-mobile (cordova) ?

Hi,
any idea if this will work with cordova? anyone tried it?
if no, is there any plan to do so?
Thanks.

@m-abisaad I have not tried to have it compiled to another code-base, in theory it should work.
Please let me know if you start testing this.

it worked,
i had to download conversationaform.min.js and include it in the solution.

@m-abisaad awesome! Please let me know if one can access the implementing at some point.


EFS should be a private service, question 10 in Chapter 5

Hi, I am doing the questions in Chapter 5. I have a small doubt about the given answer to question 10.
Question 10: Is EFS a private service?
I think the answer is Yes, while the answer given by the book is 'No, it is a public service'.
I did some research from the AWS doc, and I found the EFS should be mounted in the VPC, which means it should be a private service.

Hi @lmy931
Here is the feedback shared by the author:
Apologies for the confusion in the book. Question number 10 , Chapter 5.
Amazon EFS is a Private Service, by default it is isolated to the VPC in which it is provisioned into. EFS is part of the AWS Private Zone.
Thank you!


DxValidator with validationRules

Hi,
Before devextreme-angular2 i can automatically configure the adapter for the dxValidator like this
<div data-ng-model="vm.data" dx-text-area="{...}" dx-validator="vm.validationrules">

Now i can use Custom Validation Engine with [isValid] and [validationError] and works as you showed in the example.
The problem is that I already have a lot of validationRules defined in my app ->validationrules = { validationRules: [{ type: "required" }] };
Is there a simple way to associate the validationRules to the editor? [validationRules]="validationrules"?
Is there a way to associate an Editor with a DxValidator?  [validator]="validatorRef/instance/etc"? Transclusion maybe?
If the proper way is to configure the adapter manually, can you provide an example?
Thank You so much.

Having the same issue/question. I tried the following, but that doesn't seem to do anything.
<dx-text-box [(value)]="model.reportNumber" [dx-validator]="{ validationRules: [{ type: 'required', message: 'Report number is required.' }] }"></dx-text-box>

Thank you for your cases. We need to check if we can make dx-validator work as an Angular 2 directive, not a component.
As a workaround, please check if the Angular 2 forms and validators are suitable for you.

Yes, I was able to do what I need to using isValid and validationError bindings. However, I would also like to use the DxValidationSummary component, but it doesn't seem to work, either. I placed it in my HTML like so, but it doesn't display anything:
<dx-validation-summary></dx-validation-summary>