Websocket connection fails

When i try and connect the front end to hasura I get an error connecting to the websocket URL. I have tried starting the url with http:// and ws://. I looked and it seems hasura is only listening on 8080 with http. The documentation does not provide an example how to have hasura listen via websockets.
[Error] WebSocket connection to 'ws://localhost:8080/v1/graphql' failed: WebSocket is closed before the connection is established.

Websockets are all handled by hasura with no extra setup.

Have you tried using the actual ip address instead of localhost?
Does the subscriptions work on the console?
Are you running both hasura and the ui locally on your own pc?

Okay I was able to reproduce it but can you confirm the web socket really didn't connect after numerous tries?
The error should be from the first try but it connects in the second one:

First:

Second:

I've looked around and it is an issue with the browser but we've set the ws to keep retrying here so it should not be an issue:
const wsLink = new WebSocketLink({
  uri: process.env.NEXT_PUBLIC_GRAPHQL_WS ?? 'wss://localhost:3000',
  options: {
    reconnect: true,
  },
  webSocketImpl: WebSocket,
});

The local setup i tried while following the docs also had no data and it connected.
Since I can't reproduce the error I will provide the steps I took to get it working locally on my local mac env.
PSQL:

Just followed the docs and created a database and tables

HASURA:

Get the docker compose file from the official documentation. Removed the psql service and added the database url (note i'm using host.docker.internal because of the local env being on macOS:

version: '3.6'
services:
  graphql-engine:
    image: hasura/graphql-engine:v2.2.0
    ports:
    - "8080:8080"
    restart: always
    environment:
      HASURA_GRAPHQL_UNAUTHORIZED_ROLE: "anonymous"
      HASURA_GRAPHQL_DATABASE_URL: postgres://<user>@host.docker.internal:5432/<table>
      ## enable the console served by server
      HASURA_GRAPHQL_ENABLE_CONSOLE: "true" # set to "false" to disable console
      ## enable debugging mode. It is recommended to disable this in production
      HASURA_GRAPHQL_DEV_MODE: "true"
      ## uncomment next line to set an admin secret
      HASURA_GRAPHQL_ADMIN_SECRET: <adminsecret>

inside bdjuno/hasura with hasura cli installed run hasura metadata apply --endpoint "http://localhost:8080/" --admin-secret <adminsecret>

UI:

Add the following to .env:

NEXT_PUBLIC_GRAPHQL_URL=http://localhost:8080/v1/graphql
NEXT_PUBLIC_GRAPHQL_WS=ws://localhost:8080/v1/graphql
xNODE_ENV=production
xPORT=3000
NEXT_PUBLIC_URL=<url>
NEXT_PUBLIC_WS_CHAIN_URL=<rpc websocket>
NEXT_PUBLIC_CHAIN_STATUS=mainnet

Start it up and check that it connected inside the console's network tab

Closing due to no activity

hey @ryuash what should we put for NEXT_PUBLIC_URL and NEXT_PUBLIC_WS_CHAIN_URL when running this on localhost:3000 with this setup?
is it
NEXT_PUBLIC_URL=http://localhost:3000
NEXT_PUBLIC_WS_CHAIN_URL=ws://localhost:3000

or something else?
Because I can not get any data showing locally when using starport, I also edited testnet config with proper genesisTime etc.

@skosito
NEXT_PUBLIC_URL=http://localhost:3000
NEXT_PUBLIC_WS_CHAIN_URL=ws://localhost:26657/websocket (or wherever you have the node setup)

NEXT_PUBLIC_URL is app's url for seo purposes (if you deploy outside of local setup)
NEXT_PUBLIC_WS_CHAIN_URL will be renamed NEXT_PUBLIC_RPC_WEBSOCKET in the future releases for clarity

@ryuash thank you very much!


Adding protobuf definitions to types.proto to support fraud proofs

Added protobuf definitions for the fraud proofs and the data types used by them.

When creating a protobuf definition for the StateElement, shall I create a protobuf definition for individual state elements, e.g, Account, Delegation etc.? If so, how can I capture all of them when defining the type of the value within the StateElement? For references, this is where StateElement is defined.

shall I create a protobuf definition for individual state elements, e.g, Account, Delegation etc.?

Yes, otherwise no one can use the higher-level structures.

If so, how can I capture all of them when defining the type of the value within the StateElement?

You can use oneof: https://developers.google.com/protocol-buffers/docs/proto3#oneof

Why was this closed?

https://github.com/celestiaorg/celestia-specs/pull/190#issuecomment-891823621


Add MATCH-LINE macro

This is just a bit of tidying. The easy case in the parser is that it is consuming a single line, dispatching on the first token and consuming a fixed template of successors (some required, some optional). This adds a simple bit of syntax for this.

Also, props to @notmgsk for his "thought leadership".


DeleteIndex doesn't remove indices

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?
[ ] elastic.v2 (for Elasticsearch 1.x)
[ ] elastic.v3 (for Elasticsearch 2.x)
[ ] elastic.v5 (for Elasticsearch 5.x)
[ x] elastic.v6 (for Elasticsearch 6.x)
Please describe the expected behavior
I would expect that if I have two indices in a green state
elasticsearch@b923a6b129ad:/$ curl localhost:9200/_cat/indices
green open prod1_index5af46372 AhPJF5fiReC5pzLYtrrtCw 5 0    0 0 1.1kb 1.1kb
green open prod2_index5af46372 UHNIUBPSQk-IGbP6JiIxnw 5 0 3038 0 2.4mb 2.4mb

And I execute the function
func tearDownIndex(client *elastic.Client, index string) {
        exists, err := client.IndexExists(index).Do(context.Background())
        if err != nil {
                log.Fatalf("%v", err)
        }
        if exists {
                _, err = client.DeleteIndex(index).Do(context.Background())
                if err != nil {
                        log.Fatalf("%v", err)
                }
        }
}

would remove the indices
Please describe the actual behavior
calling the above function removes the documents in prod2_index5af46372 and both indices are in a yellow state instead of being removed
yellow open prod_doc_metadata_index5af46372 AhPJF5fiReC5pzLYtrrtCw 5 1    0 0 1.1kb 1.1kb
yellow open prod_rev_metadata_index5af46372 UHNIUBPSQk-IGbP6JiIxnw 5 1    0 0 1.1kb 1.1kb

If I curl -XDELETE localhost:9200/{index} the indices are removed.
Any steps to reproduce the behavior?

This works like a charm here. Try e.g. this code which creates and deletes an index if it already exists on the 2nd run.
It is correct that in the "expected behavior" you used prod1_index5af46372, while in the actual behavior you have prod_doc_metadata_index5af46372 , which are not the same.

That's a typo. I was renaming the index for sake of simplifing the
write-up. The indices are the same.
On Thu, May 10, 2018, 11:43 AM Oliver Eilhard<EMAIL_ADDRESS>wrote:

This works like a charm here. Try e.g. this code
https://github.com/olivere/elastic/blob/release-branch.v6/recipes/mapping/mapping.go
which creates and deletes an index if it already exists on the 2nd run.
It is correct that in the "expected behavior" you used prod1_index5af46372,
while in the actual behavior you have prod_doc_metadata_index5af46372,
which are not the same.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/olivere/elastic/issues/774#issuecomment-388093624,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABnjTE9M6Y-VhjykaB4WGa7Lyg2wVNgQks5txGAngaJpZM4T6Ewa
.


publish to npm please

workaround
npm install --save https://github.com/Stichoza/font-larisome.git

Done. https://www.npmjs.com/package/font-larisome


[Question] Wonder if there's a way to create build_modes dynamically

Hi there,
I'm using DVSim to run the simulation on VCS and I have a silly question for building the hjson simulation flow with sequential build-options which receiving the parameter from the test cases.
In the design, I have three parameters named PASS_RATE, FAIL_RATE and MAX_ITER, each of them in the range of [10, 100], stride of 10, and my test case may have different value of those parameters. So these parameters can be regarded as both build_opts and run_opts together.
//  The estimated Makefile format may as below
sim:
    vcs -f flist.f +UVM_TESTNAME=$(TEST_NAME) $(PRATE) $(FRATE) $(MAX_ITER) (Other options)

case_1:
    @$(MAKE) sim TEST_NAME=case_1 PRATE="+define+PASS_RATE_10" FRATE="+define+FAIL_RATE_20" MAX_ITER="+define+MAX_ITER_30"

The problem here is, in general way, I need to create the build_modes based on the combinations of the possible values of three parameters, which means that there may be up to 1000 build modes and it is horrible to build them manually.
// The hjson flow format may as below
build_modes:[
            {
                name: mode_1
                build_opts: [
                    "+define+PASS_RATE_10",          // from 10 to 100, stride of 10
                    "+define+FAIL_RATE_20",          // from 10 to 100, stride of 10
                    "+define+MAX_ITER_30"            // from 10 to 100, stride of 10
                ]
                is_sim_mode: 1
            }

           {
              // Up to 1000 build modes depends on the value of the parameters
           }
]

tests: [
        {
            name:           case_1
            uvm_test_seq:   case_1_seq
            uvm_test:       case_1_test
            build_mode:     "mode_1"
            run_opts:      [
                            "+PASS_RATE_10",
                            "+FAIL_RATE_20",
                            "+MAX_ITER_30"
                           ]
        }
]

I wonder if there's an alternative way to create the build_mode dynamically such as using the wildcard in tests to match?
Thank you for all the help!

Before considering options, I am curious why you pass things like PASS_RATE_10 as a +define to the simulator. I would think this is a property of the orchestrator, not of each simulator invocation.

Before considering options, I am curious why you pass things like PASS_RATE_10 as a +define to the simulator. I would think this is a property of the orchestrator, not of each simulator invocation.

Thank you for the reply, the as shown PASS_RATE is the supposed write accuracy limitation and will affect the design behavior because for different PASS_RATE the design has different response method, and usually related to MAX_ITER.

It seems this is only a runtime mode, since a build failure means all bets are off. It seems you only need an iteration count and a pass percentage (the fail rate should be 100 - pass percentage). As for the run modes, you can create various run_modes with different settings of your parameters (say large_iter_80_pct_pass, etc.). And you can assign run modes to tests in the corresponding hjson file, and if needed you can override this in the dvsim invocation, so you could run a regression with your favorite iter_pass_rate mode. However, there is currently no way to assign per test run modes in the dvsim command.


ControllerBase helpers refactored with lambda syntax.

Addresses #6864

@mistakenot,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

I'll get this merged in once the build passes.

Thanks for the PR!


Update the documentation page to include more detail on how to correctly use minecraft:fuel (I'm coding an addon)

I've been working with Addons for Minecraft Bedrock for some time now, and I'm still learning. However, I have been stuck for hours just trying to make an addon that makes items burn in furnaces to smelt other items. However, despite how many times I try, I always seem to screw something up. There is no clear online tutorial or section on the Creator Documentation site that SHOWS people how to use minecraft:fuel, not just states it. All there is for me to work from is this:
"minecraft:fuel":{
    "duration": 3.0
}

I understand the duration part, but I don't know where to add it into my Behaviour pack (I've tried the 'items' folder)
So far, I've added code this to a file that I would like to make a Nether Brick a fuel item to burn in furnaces.

{

  "format_version": "1.16.0",
  "minecraft:item": {
    "description": {
      "identifier": "minecraft:nether_brick"
    },
    "components": {"minecraft:fuel":{
    "duration": 3.0
	}
}

I still can't figure out why this doesn't work.
I have the file saved in this directory:
(My Behaviour Pack)/items/nether_brick.json
To solve this, please could this section be updated on the creator documentation site so that the process of making an item into a fuel is properly explained. (If someone could either just help me out, that would be muchly appreciated!)
Thanks for taking the time to read this, I'm probably just really dumb and missed out on something, but it's worth asking for.
Sorry if I just coded something wrong!
Edit: This is the addon I've created so far:
https://github.com/MicrosoftDocs/minecraft-creator/files/8599691/Craftables.%2B.Smeltables.zip
This is the page in need of an update (Just a suggestion) https://docs.microsoft.com/en-us/minecraft/creator/reference/content/itemreference/examples/itemcomponents/minecraft_fuel

Hello,
It must your incorrect item id that generate an error.
The correct code should be:
{

  "format_version": "1.16.0",
  "minecraft:item": {
      "description": {
          "identifier": "minecraft:netherbrick"
   },
   "components": {
       "minecraft:fuel":{
           "duration": 3.0
	    }
}

In Minecraft:BedrockEdition, the ID of nether brick is netherbrick instead of netherite_brick, actually in JavaEdition netherite_brick is right.
Now that we know what to blame, your code format is not a fault.
Delete the UNDERLINE.

Hi, thank you for your response. I should of double checked the item ID's before creating this!
However, I think I have still made some sort of error as well. Whenever I am in Minecraft, testing my pack, I try to put the Nether Bricks into the furnace however when I click it won't let me use it as a fuel - It just does nothing.
I have all of my experimental features enabled and I can put it in the top half of the furnace but I can't put it in the bottom to smelt items.
Maybe I need some 'tags' next to minecraft:fuel to show it can be used in a furnace, smoker and blast furnace?
I still can't seem to track down the root cause of my issue. Thanks for your help again, I am still new to the world of MC Addons and I'm still learning!

I've carried on trying but I still can't figure this out: It still does not work.....
Any suggestions?

Hello!
It looks like we've made some progress on this. Since it's been a while, I'm going to close this issue, but if you have any additional feedback, please reach out to us!
Thanks!


DataSet, 3D, 1D init()

DataSet_3D
:param data: specify type is dict
Specify is dict of numpy arrays (numerical)
Specify arrays should be 1D
Specify keys should be int() 0, 1 ,2 and optional any
DataSet_1D
:param x: specify type is np.array or dict or np.array with one key(0)
data attr -> specify the same thing

Explain how data works throughly. What are the key? Examples of this. Convention that 'x', 'y' 'z' axes in 3D data (eg. vectors) should be keyed as 0, 1, 2 respectively.
Data is dict of np.array()
Specify extract key exacts a data series, slice extracts a range of samples point across all data series (ie. for a corresponding time range)

Hey @TheMuonNeutrino , can I work on this ?

Hey @kadatatlukishore , I'm happy to take contributions, so go ahead and submit a pull request.
Out of curiosity, what drew your interest to this project? I wasn't expecting anyone to pay attention at this stage.

Hey @TheMuonNeutrino , Just searched for good first issues to contribute. But I liked you work here.
It allows for downloading of satellite data from CDA web, processing of this data and generation of audio it catches all the attention and it is really interesting. So would love to add small contribution to this amazing work of yours.

Thank you for your kind words.
Unfortunately, I'm going to go ahead and close this issue as I've manage to address it while working on another related issue.


KAFKA-4372: Kafka Connect REST API does not handle DELETE of connector with slashes in their names

Kafka Connect REST API does not handle in many places connectors with slashes in their names because it expects PathParams, this PR intends to :

Reject as bad requests API calls trying to create connectors with slashes in their names
Add support for connector with slashes in their names in the DELETE part of the API to allow users to cleanup their connectors without dropping everything.

This PR adds as well the Unit Test needed for the creation part and was tested manually for the DELETE part.

The test failures do not seem to be related to the PR

Aside from open Q about whether or not to disallow and when, LGTM
cc @ewencp or @hachikuji for thoughts & review

Fixed code styling in tests too

LGTM, thanks!

thx


Extend new application tutorial to advertise using git grep to find things

git grep is one of the single most useful tools for Zulip development, and we don't presently advertise it anywhere prominent (it's in the Git guide somewhere).  I think we should cover it in the "new application tutorial", since that's something we point new contributors at pretty often to learn about the codebase, and I think is a natural setting for explaining how git grep is the magic tool to let you find the implementation of any feature in the UI.  I think the best version would be showing the example of using git grep starting from a string you see in the UI to find the implementation for a setting?  Not sure.
@laurynmm FYI.

@zulipbot claim

Unassigning myself from this issue to work on others. I've already created a PR fixing this, pending review.


Suggestion: Add missing translations

1. Evaluate -> Result Window :

Graph
List

2. Misc -> Stash navigation :

Disabled
Normal
Inverse

3. Dialog window:

"amount"  when mouse is over mini graph

Amount, Seller, Age in dialog values list view

bars when mouse over

_ Hours ago
_ Minutes ago
few second ago

when mouse over Orb "Click' = In-Game Browser, 'CTRL + Click' Default Browser"

4. Corner online indicator
also is it suppose to be changable to other time amount ?

5. In panel "Command", separate key for element "command" above shortkey

6. Can menu in tray also be translated ?

1. Evaluate -> Result Window :

Graph
List

✔

2. Misc -> Stash navigation :

Disabled
Normal
Inverse

✔

3. Dialog window:

"amount"  when mouse is over mini graph

✔

Amount, Seller, Age in dialog values list view

✔

bars when mouse over

_ Hours ago
_ Minutes ago
few second ago

✔ Automatically translated by momentjs.

when mouse over Orb "Click' = In-Game Browser, 'CTRL + Click' Default Browser"

✔

4. Corner online indicator
also is it suppose to be changable to other time amount ?

❌ No, not for now. Yes, it's changeable via scrolling.

5. In panel "Command", separate key for element "command" above shortkey

✔

6. Can menu in tray also be translated ?

❌ No, not for now. Unfortunately this requires quite some effort.

The ammount of daily effort you put into this project amazes me. Thank you very much!
You mentioned you used BabelEdit for translations, I looked into it because of curiosity. How did you generated JSON files out of it ? I've seen only excel,cvs export format.

The export format can be select during the creation of a project. See attached screenshot. I'm using the ngx-translate variance.


rspec-puppet fails on nested data type aliases

first type alias:
type Core_stdlib::Port = Integer[0,1024]
second type alias:
type Globals::Uri = Struct[{   'fqdn' => String,   'port' => Core_stdlib::Port, }]
rspec-puppet fails with error message:
Found value has wrong type, references an unresolved type 'Core_stdlib::Port'
Puppet agent is working as expected. rspec-puppet throws error.

@tuxmea I'm not able to reproduce this locally. Is the core_stdlib module available in the spec/fixtures directory?

This could also be due to forgetting to add core_stdlib to dependencies list in metadata.json file.

I can check in two weeks (when I am back at customer). Maybe it is related to the missing dependency in metadata.json (which is a nightmare my 5cent)

could not reproduce. Everything is working as it should. Closing issue.


Use Hamcrest assertions instead of JUnit in  config/encryption - backport 2.x (#1749)

Part of #1749
Original PR

/trigger


[Feature request]: Notification content only

Preflight Checklist

[X] I have read the Contributing Guidelines for this project.
[X] I agree to follow the Code of Conduct that this project adheres to.
[X] I have searched the issue tracker for an issue that matches the one I want to file, without success.

Request type
Request for enhancement of a component
It affects the following packages
Components
Functionality
Request for a content-only variant of the component notification. It is important to consider whether it should be a variant of the Lyne Notification or whether it should be created as a separate project component.
Link to design proposal file
https://www.figma.com/file/9Cdo8bGTUtBSR2am6O4fHl/Notification-content-only?type=design&node-id=9201%3A375735&mode=design&t=6UgGCYvd0fp8Xcxj-1

Discussion with @mcilurzo -> different component, something like "sbb-hint". to be challenged with @florianstuerzinger

proposal in figma: https://www.figma.com/file/9r6xSfNmEfCFxl1yFYedrj/Lyne-Components?type=design&node-id=26122%3A501619&mode=design&t=qhk605TgsdvxvovG-1

Will be continued in #1932


Make functions in integration-test.sh script POSIX compliant.

Remove BASH function definition syntax in integration-tests.sh to be able to run it in a system with a different default shell (e.g. ubuntu).

@irozzo-1A thanks for this PR. LGTM. so merging now.


rocoto breaks with the upcoming Guice 4.0 release (currently in beta)

Guice 4.0 is currently in beta - https://code.google.com/p/google-guice/wiki/Guice40. As stated in the wiki page:

Many things inside com.google.inject.internal changed and/or moved. This is especially true for repackaged Guava (formerly Google Collections)

This impacts rocoto because it will not find Guice-internal Preconditions:

java.lang.NoClassDefFoundError: com/google/inject/internal/util/$Preconditions

I am wondering if it makes sense to include a direct dependency on Guava. Reworking imports will break compatibility with at least Guice 3.0.

Yes, this should be fixed. I upgraded to Guice 4.0 today and my project broke. It's a horrible idea to depend on internal Guice classes. See https://github.com/99soft/rocoto/pull/8

I think this should be fixed with https://github.com/99soft/rocoto/commit/17ae11665cc4e42d9b17c33493d043c7452d69c8


can't change style format

component page
import React  from 'react';
import Clock from 'react-live-clock';

export default class Mytime extends React.Component {
    render() {
        return(
        <Clock format={'HH:mm A'} ticking={true} timezone={'Asia/Bangkok'}  />
        )
    }
}

main page
var des2 ={
  color: 'white',
  marginLeft: '10px'

}
<Mytime style={des2}  />

Hi @s4wet
I am preparing a new release and I'll include your suggestion into it.
It will be ready on weekend.
Thank you.

Hi @s4wet  it is fixed in the new version (3.0.0)


Minor hidpp20 driver work

Removes useless setting of the dpi index when updating resolutions.
Removes weird empty line.

Also added the commit that replaces the gotos (was previously in #870).


Add a way to change modal background

currently, you can change the inner element background, but not the overlay

https://github.com/bodyflex/react-native-simple-modal/pull/11 fixes this

#11 has now been merged!


[Docs] Update README with New Library Functionalities

Description
The current README file does not reflect the recent additions and changes to the library functionalities. It is essential to update the README to include the new functions and provide clear usage examples for users.
Tasks

Add New Function Descriptions:

Register URL
Register S3
Register Kafka
Search Resource

Update Existing Function Descriptions:

Ensure Register Organizations and Search Organizations are correctly documented.

Provide Usage Examples:

Include examples for registering URL, S3, and Kafka resources.
Include examples for searching resources using the new search functionality.

Update Installation and Configuration Sections:

Verify that installation instructions are up-to-date.
Ensure configuration examples include any necessary details for the new functionalities.

Review and Update Testing Section:

Ensure the testing section aligns with the new functions and their respective tests.

Additional Context
Updating the README is crucial to ensure that users can effectively utilize the new features and understand how to interact with the updated library.
References
Refer to the recent commits and the following files for details on the new functionalities:

register_url.py
register_s3.py
register_kafka.py
search_resource.py

Done.


Impossible to use JetStream

Please update Nats.Client version, because JetSteam has been added in 0.14
Without JetStream impossible to use acknowledgement

Sorry for the delayed response @DFSko I noticed you created a PR and closed it. Please re-submit the PR so I can merge it ^^


Non touch navigation issue

Thank you from italy your work made a lot of us very happy. I would like to ask you if you can fix the player because in my case i dont have touch screen so, i have to scroll with the joystick (mercedes a class w176). Many time i Cant navigate because it doesnt let me choice the link.

Which player ? Youtube or local ?

Both my friend, it is very difficult for us that we dont have touch screen to use the Youtube and navigate on the web

I see. I am not able to exactly figure out the difficulty since my car doesnt have hardware controls. Would it be possible to explain in a little more detail ? What is the issue in scrolling with joystick ?

I can send you a video maybe it is easier for me to explain what do you Think?

Yes, send me

Hello. Thank you for the guide how to use plex, youtube and many other application with android auto.
I have the same problem, no touchscreen (Audi Q5 2019 with MMI Plus). I have a bmultidirectional wheel (up,down,left,right, click) and a touchpad for writing inkl. click function.
I can e.g. open plex and see the page for choosing the user (I have 3 there) can also switch the items bi clicking left oder right but when I click on the wheel nothing happens. I'm not able to open the Overview of my user profile.
Same wehn I try netflix, can switch between de films/series but when I click nothing is happening.
Youtube worked but could not search for video only the videos on the main page where selectable.
Any suggestion who to handle that? Would be nice to use these apps in the car.
Regards

same issue here with 2018 MB E-Klass. No touchscreen, I am able to control the UI somewhat with the car controls, but the selected elements are not highlighted so I cant see what I would be selecting.
Would be awesome if this could be fixed, I offer to do some testing.

good night is the problem resolved?  I have the same problem...thanks

Same here... Mazda3 2023. I can stream and scroll up and down but nothing else. Will this be fixed or fixed already? Any info greatly appreciated.

Same here. On alfa romeo stelvio non touch screen. Any updates on this? Anyone had any luck?

Seems that who have only hardware controls cannot use carstream...
Anyone know if is possible to connect a Bluetooth controller like a mouse to the multimedia and control with the pointer?


Scroll dev 0201

Open PR to track changes by @noel2004

We have merge most udates into main in a9214c so there should not be so much pending commits left.

Got it. It is harder to track changes without PRs because commits do not have as much visible information.
We still need the remaining changes though, right? Let me know if I can help.


dex does not use our standard ciphersuite

Describe the bug
The majority of our services use a limited set of TLS ciphers that are recommended by various scanning tools.  The dex component does not currently use this cipher suite.
To fix this, we'll need to submit a patch upstream to change the ciphers in use or make them configurable.

We've added exceptions to our own TLS scanning here https://github.com/chef/automate/blob/master/integration/helpers/ssl_filters.jq#L35-L39 that highlights the various issues.

I've opened a pull request upstream to start addressing this: https://github.com/dexidp/dex/pull/1540


Add missing helper functions

Description
Add remaining helper functions in spec that are missing from the repo.

Please double check you're not duplicating effort already available in https://github.com/PegaSysEng/artemis/pull/326. 😄


Add page describing tooltip module

Sorry this took longer to merge, I found a bug in one of the docfx plugins. I have reformatted the page into a sample and added metadata etc.
Any issues please ping me or comment here.

And of course, thank you 👏 for taking time to contribute - this looks like an awesome module.


Circumvent Broken llama.cpp Pre-Tokenizer

Fixes https://github.com/outlines-dev/outlines/issues/820
Problem:
llama.cpp's pre-tokenizer doesn't handle unicode properly (draft PR: https://github.com/ggerganov/llama.cpp/pull/5613). This results in tokens which are incompatible with Outlines byte-wise FSM and causes #820's error.
Solution:

If models.llamacpp() specifies a LlamaHFTokenizer, populate the vocabulary used in index construction with tokenizer.get_vocab(). This takes advantage of huggingface's working pre-tokenizer.

Warn users that they should pass a LlamaHFTokenizer:

>>> from outlines import models, generate
>>> model = models.llamacpp("Qwen/Qwen1.5-0.5B-Chat-GGUF", "*q8*.gguf")
/opt/conda/lib/python3.10/site-packages/outlines/models/llamacpp.py:294: UserWarning: llama.cpp pre-tokenizer is broken. You may recieve an Outlines error during Regex index construction.
To avoid this error when using `models.llamacpp` you may pass `tokenizer=llama_cpp.llama_tokenizer.LlamaHFTokenizer.from_pretrained(<hf_repo_id>)` to `models.llamacpp()`
  warnings.warn(

Debug Notes / Observations:

The problematic token in Qwen1.5 is 29333 (b' \xef\xbf\xbd')
The issue can be reproduced with models.llamacpp, but not models.transformers
AutoTokenizer's get_vocab() is inconsistent with its encode / decode output.

get_vocab()[ b'\xc4\xa0\xc3\xaf\xc2\xbf\xc2\xbd'.decode()] = 29333
get_vocab()[ b' \xef\xbf\xbd'.decode()] -> KeyError
encode( b'\xc4\xa0\xc3\xaf\xc2\xbf\xc2\xbd'.decode()) = [144242,  37572,  30182,  26062]
encode(b' \xef\xbf\xbd'.decode()) = [29333]

tokenizer.get_vocab() has a distinct mapping from tokenizer.decode due to the pre-tokenizer:
>>> tokenizer = transformers.AutoTokenizer.from_pretrained("Qwen/Qwen1.5-0.5B-Chat")
>>> tokenizer.backend_tokenizer.pre_tokenizer.pre_tokenize_str(b' \xef\xbf\xbd'.decode())[0][0].encode()
b'\xc4\xa0\xc3\xaf\xc2\xbf\xc2\xbd'

Looks good, thank you for the fix. I think it will be valuable to many people in the community :pray:


Parsing error on HN feed

I added the feed from https://news.ycombinator.com/rss and I'm getting this error message in the console:
XML Parsing Error: mismatched tag. Expected: </link>.
Location: https://news.ycombinator.com/rss
Line Number 6, Column 3: rss:6:3
Error: 503: Service Unavailable feed-parser.js:14:18

The extension then retries to fetch and parse the feed and fails again.
This issue might be related to #26 and #7.
I'm getting continuous requests for http://www.tagesschau.de/xml/rss2, too, but it was not logged.

When you say continuous request, you mean not every 5 minutes (the default update interval), but more often? I can reproduce the HN problem, by refreshing our preview page quickly, it seems to be some kind of rate limiting, so that would make sense if we request very often.  Error: 503: Service Unavailable Otherwise we can parse the HN RSS feed just fine.


assign Response Format

GMT should be changed to UTC (GMT is referring to Greenwich in the United Kingdom, UTC is borderless. Because we are a DAO, localizing the time to the United Kingdom doesn't make sense.)
The updated format should look as follows:

@0xcodercrane the time limit for this bounty is on Wed, 12 Apr 2023 03:10:59 UTC
Your payment address is 0x00868BB3BA2B36316c2fc42E4aFB6D4246b77E46
Please use /wallet <WALLET_ADDRESS | ENS_NAME> to register a new address.

          @0xcodercrane The time limit for this bounty is on Wed, 12 Apr 2023 03:10:59 GMT

Your currently set address is:
0x00868BB3BA2B36316c2fc42E4aFB6D4246b77E46
please use /wallet 0x4FDE...BA18 if you want to update it.
Originally posted by @ubiquibot[bot] in https://github.com/ubiquity/bounty-bot/issues/240#issuecomment-1502626858

/help

It would be nice to design some type of standardized ascii art "user interface" in the bot comments.

Both pull requests don't fulfill the specification. Please review the updated message format carefully.

ok lets revert it. I think @omraval18 is the original PR owner. would you be assigning yourself?

/assign


Binding credentials should include FIPS endpoints

By default, the bind operation returns a non-FIPS GovCloud SES domain. SES supports FIPS endpoints and binding should return those endpoints instead.

SES endpoint (scroll down in list for FIPS): https://docs.aws.amazon.com/general/latest/gr/ses.html
FIPS Endpoints CLI instructions: https://docs.aws.amazon.com/general/latest/gr/rande.html#FIPS-endpoints

Resolved in linked commit.


[BUG] Attempting to Purchase a node with a Credit Card using Transak, nothign happens

user attempt to purchase node on mobile with Transak sees the image prompting for wallet, but nothing happens

@taariq It looks like a Buymore board page, but it's not a purchase page. e tried using a credit card, but why is he on the Buymore board page? User don't need to connect wallet when purchase with credit card on purchase page.

Closing as won't fix.


Trying to access array offset on value of type bool [L7]

Laravel Version: 7.29.3
PHP Version: 7.4
Database Driver & Version: Mysql 5.7

Description:
I get the following error: Trying to access array offset on value of type bool when using $this->throttle.
The error is here:
public function acquire()
    {
        $results = $this->redis->eval(
            $this->luaScript(), 1, $this->name, microtime(true), time(), $this->decay, $this->maxLocks
        );

        $this->decaysAt = $results[1];

        $this->remaining = max(0, $results[2]);

        return (bool) $results[0];
    }

If $results is false, it throws an error that is hard to debug.
Steps To Reproduce:

When $this->throttle fails, it doesn't fail gracefully.

Please re-submit with a small reproducable example that we can paste into a fresh Laravel application.

Laravel Version: 9.8.1
PHP Version: 8.1.4
Redis Version：4.0
redis occasionally returns false


Review the implementation of IDisposable.

We used to dispose just the 'hash' but we create other disposable objects.

We need to review if there are classes that need to implement IDisposable. For example, SecurityKey is one such class.

We need to implement `IDisposable

We need to implement IDisposable if a class

is creating unmanaged resources
is creating objects that implements IDisposable. It's important to note that class is responsible for disposing only the objects it OWNS. It should not dispose objects that it did not create.

We do not create/use any unmanged resources in our code. So, we only considered Point 2 to determined whether a class should implement IDisposable or not. We found that we do not need to implement IDisposable in SecurityKey class, since these classes do not create any disposable objects. Also investigated all other classes in our library and made sure we are not missing anything. The only place where a change is needed was HttpDocumentRetriever class where we create a new HttpClient object. That has been fixed in the related PR.


[WIP] Fix doc generator

Proposed changes
This PR addressed errors raised when running scripts/generate_api_docs.py
Types of changes
What types of changes does your code introduce to agents-aea?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist
Put an x in the boxes that apply.

[ ] I have read the CONTRIBUTING doc
[ ] I am making a pull request against the develop branch (left side). Also you should start your branch off our develop.
[ ] Lint and unit tests pass locally with my changes and CI passes too
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that code coverage does not decrease.
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Turns out most of the errors were caused by badly formatted doc strings, I've fixed them but there's formatting issues that still remains which is
https://github.com/valory-xyz/open-aea/blob/f2c207fe357e463039550c52014fa9ed7433fd8d/aea/protocols/dialogue/base.py#L244
here, I reformatted it but it still causes error.

So I tried fixing this but It seems the parser formats the code before rendering the markdown so we'll have to keep this as it is

I locally still get failures:
[WARNING - pydoc_markdown.contrib.processors.crossref]: 1 cross-reference(s) could not be resolved:
  aea.configurations.data_types.Dependencies: name

...
Processing aea/protocols/dialogue/__init__.py... Skipping, it's in ignore patterns
Traceback (most recent call last):
  File "/Users/davidminarsch/.local/share/virtualenvs/open-aea-WmhCcIRs/lib/python3.10/site-packages/yapf/yapflib/pytree_utils.py", line 119, in ParseCodeToTree
    tree = parser_driver.parse_string(code, debug=False)
  File "/Users/davidminarsch/.pyenv/versions/3.10.4/lib/python3.10/lib2to3/pgen2/driver.py", line 103, in parse_string
    return self.parse_tokens(tokens, debug)
  File "/Users/davidminarsch/.pyenv/versions/3.10.4/lib/python3.10/lib2to3/pgen2/driver.py", line 71, in parse_tokens
    if p.addtoken(type, value, (prefix, start)):

Also, why do we not throw a proper exception? Now I need to manually parse through the log output...

The local failure is caused by this https://github.com/valory-xyz/open-aea/pull/315#pullrequestreview-1107834403
We don't throw proper exception because we run the pydoc in a subprocess and don't check for the exit code also stderr is also not set the it'll write the error to console and since we don't check for exit code we can't raise an exception if the process fails


Customize YAML dumping order

It would be nice to dump YAML keys in field order, rather than alphabetic order. Perhaps this could be done via the Config class.

I stumbled upon this issue today after switching from pyyaml to ruamel.yaml. Would be a much appreciated feature!

I actually implemented this a week ago, but didn't have time to fully validate it locally to release it. I'm unsure whether it's best to change the default order to dump it by keys. Probably yes, will just need to set a new minor version for a breaking change.


Flavor Names, part 1

Database-side FN support. lookups to come

Merged into https://github.com/PennyDreadfulMTG/Penny-Dreadful-Tools/pull/11633


modulateGlobal parameter?

Howdy-
Is there a parameter/function for setting the global 'opacity' (Alpha value) of an overlayed volume?  That is, currently if I overlay one unthresholded volume B over an underlay A, I can't see anything of A.  The handy Niivue.prototype.setModulationImage(...) function allows one to modulate each voxel's Alpha based on a reference volume (great!), but I was wondering if there were a simpler bit of functionality:  putting a max on each voxel's Alpha.  This could be done by:

defining a new property in nvimage.js like:  this.maxAlpha = 1;
defining a function in niivue.js like:

Niivue.prototype.setModulationMax = function (
  idTarget,
  maxAlpha = 1
) {
  let idxTarget = this.getVolumeIndexByID(idTarget);
  this.volumes[idxTarget].maxAlpha = maxAlpha;
  this.updateGLVolume();
};

and (after any modulateAlpha considerations have been applied) then checking to see if a volume's maxAlpha < 1.  If so, scale the current Alpha value by maxAlpha.  In this way, the global maxAlpha scaling can be applied on top of the modulateAlpha, which is how AFNI applies things, and that has been useful.

Is that reasonable/existing/possible?  I saw this function in niivue.js:
Niivue.prototype.overlayRGBA = function (volume) {...}
... but the comments said this was slated for removal.
I would try adding that functionality, but my JS is not so elegant.
thanks,
pt

NB: initially I thought modulateGlobal might be a good name for the described parameter, but then later thought maxAlpha would be (hence the seemingly unrelated Issue title).

@mrneont I am not sure I understand the question. You can use the setOpacity() function to explicitly set the opacity of an overlay. Here is a live demo. Maybe you can be more specific about your use case. Do you think that the volume opacity should be considered when computing the alpha modulation of one image to the other?

Oh, yes, that's it exactly. I guess I had focused on searching for Alpha instead of the obvious...  I'll go get another cup of coffee and hope to be more observant next time.
Thanks!

@mrneont the challenge with multiplying (modulating) a fractional alpha with the already modulated alpha is with generating the outlines. I have created a demo with a possible solution that also shows the limitations. In this example, the opacity slider adjusts both the overlay opacity, but also the outline boundary threshold. The issue is that on the GPU we represent the alpha as an 8-bit integer, and we get some slight rounding errors. You can see this in the demo by adjusting the opacity slider - note that the outline location wavers a little bit. I think we could add a bit of logic to the alpha modulation formula to fix this, or we could decide the errors are so small that we can ignore them. This approach also reduces the dynamic range of the alpha modulation. Internally, the code introduces a new shader uniform named overlayOutlineAlpha. I would consider this feature experimental for now, it seems to have a lot of compromises. There are a couple of ways to achieve this, but they all seem to impact the tidiness and maintainability of the algorithms.
git clone --branch drawing git@github.com:niivue/niivue.git
cd niivue
npm install
npm run demo

And then open the Modulation of AFNI statistics demo web page.

OK, thanks for looking more into this.
In the demo on the drawing branch, I only see the Modulation slider, not the additional Opacity one.  But I take the point about the rounding issues here.  Would introducing a flooring() or ceiling() function into the modulation make the truncation errors "consistent", though slightly biased?


Fix filtered json logic

Description
In JS filteredRow !== {} does not have any sense, cause it will be always true. One of the way to check if the object is empty is usage of its keys length: Object.keys(filteredRow).length

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hello @mmarkelov, thanks for making this pull request.  As per the Contributing guideline, please make your pull request against the mainline-dev branch.


Assembly definition files in Unity package

It would be nice if Unity package version would come with assembly definitions.

I believe since the unity package version uses dlls, they are already split up into different definitions. Not 100% sure about this one though.

The Unity package version will come with assembly definitions like all Unity packages. Closing this, if you would like to get more/different assembly definition files than on develop, please re-open

Just want to note that I was talking about the .unitypackage version that contains the MLAPI installer, not the devel branch version.


Pass RSOP variables to lookup

The initial configurations lookup is explicit about passing its
parameters down to Resolve-NodeProperty, but the following lookups
are not. This produces unexpected results if there is no global
variable with the default name set in Resolve-NodeProperty. While
$Node is specified here and will work, may as well be explicit.

Thanks Liam!


Android demo: unable to build with Bazel, could not read RELEASE.TXT

Environment info
Operating System: Ubuntu 14.04 LTS 64-bit
Installed version of CUDA and cuDNN: none (not using GPU)
Steps to reproduce
Note: TensorFlow was installed previously.

Install Bazel as instructed here: http://www.bazel.io/docs/install.html#install-on-ubuntu
Install Android Studio (which includes the SDK).
Install Android NDK through the Android Studio SDK Manager.
Download and unzip the TensorFlow graph as instructed here: https://github.com/tensorflow/tensorflow/blob/master/tensorflow/examples/android/README.md
Uncomment the Android entries in the WORKSPACE file and add in paths to the SDK and NDK (in my case, these were /home/me/Android/Sdk and /home/me/android-studio/android-studio/plugins/android-ndk)
Run $ bazel build //tensorflow/examples/android:tensorflow_demo

What have you tried?

I've looked around, and my understanding is that the RELEASE.TXT file is not included in the most recent version of the Android NDK. Since the NDK installed via Android Studio is a .jar file, I wasn't sure what to do with that, so I went to the path indicated by the terminal log and created a blank RELEASE.TXT file. This made no difference.
According to #1468, it can be resolved by downgrading to an earlier version of the NDK which contains RELEASE.TXT. I downloaded the version of Bazel (for Linux) from the links given, but the downloaded file is a .bin, which is unusable to me. As such, I found this solution to be a dead end.
Commenting out the NDK entry is said to resolve the issue, but I haven't tried this yet, since I don't know if it'll cause more complications down the road.

Logs or other output that would be helpful
ERROR: no such package '@androidndk//': Could not read RELEASE.TXT in Android NDK: /home/me/.cache/bazel/_bazel_me/f3471be34d1e62bf21975aa777cedaa3/external/androidndk/ndk/RELEASE.TXT (No such file or directory).
ERROR: no such package '@androidndk//': Could not read RELEASE.TXT in Android NDK: /home/me/.cache/bazel/_bazel_me/f3471be34d1e62bf21975aa777cedaa3/external/androidndk/ndk/RELEASE.TXT (No such file or directory).

Is there another way to resolve this issue without downgrading or commenting out the NDK entry? If not, how can I install a previous version of Android NDK? Thanks in advance.

This seems to be a duplicate of https://github.com/tensorflow/tensorflow/issues/1468
Unfortunately #1468 was closed by the submitter after people suggested he download an old version of the NDK.
The workaround helps the person that submitted #1468, but the issue with the TensorFlow build system remains ...

@cardshuffle Which version of Bazel did you install? 0.3.0 is required for NDK 11 support. Otherwise you'll need to use Android NDK r10e (links in #1468) as @lukesleeman suggests.

Automatically closing due to lack of recent activity. Please reopen when further information becomes available.

@lukesleeman #1468 was the issue I linked to, and I already discussed how I tried the solutions suggested there.
@andrewharp I installed version 0.3.0 as confirmed by bazel version. This is the log output:
Build label: 0.3.0
Build target: bazel-out/local-fastbuild/bin/src/main/java/com/google/devtools/build/lib/bazel /BazelServer_deploy.jar
Build time: Fri Jun 10 11:38:23 2016<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>

Also, a correction: I mistakenly said that #1468 suggested downgrading Bazel where it was actually the Android NDK, and I just fixed it.

@cardshuffle
Can you please clarify what issue "it" was that you fixed by downgrading your ndk, what version you downgraded to, and what issue still remains?
I was just able to clone the TF repo and build the tensorflow demo using NDK r11c and Bazel 0.3.0 on Ubuntu. The RELEASE.TXT issues should no longer occur with this version combination.
It's possible that Android Studio has downloaded a different variant for you, though -- Bazel does not officially support r12 at the moment. You could try manually downloading r11c here: http://dl.google.com/android/repository/android-ndk-r11c-linux-x86_64.zip and see if that works.
If that doesn't work, can you please paste the NDK entry from your WORKSPACE?

@andrewharp I didn't downgrade the NDK (at the time of that post). What I "fixed" was my initial post in this thread, when I mistakenly said Bazel instead of NDK. I also stated in my initial post that I was unable to downgrade to NDK using the links provided in #1468 (I tried to downgrade to r10e). Sorry if my wording was confusing.
I downloaded the NDK r11c using the link you provided, unzipped it to the Documents directory, and modified the WORKSPACE file:
android_ndk_repository(
    name="androidndk",
    path="/home/me/Documents/android-ndk-r11c",
    api_level=21)

Then I ran the Bazel build command again, and I got this error:
ERROR: /home/me/tf/tensorflow/tensorflow/examples/android/BUILD:12:1: error loading package 'tensorflow/core': Extension file not found. Unable to load package for '//google/protobuf:protobuf.bzl': BUILD file not found on package path and referenced by '//tensorflow/examples/android:libtensorflow_demo.so'.
ERROR: Analysis of target '//tensorflow/examples/android:tensorflow_demo' failed; build aborted.
INFO: Elapsed time: 0.491s

Did you git clone with the --recursive option?

@andrewharp When I initially installed TensorFlow and cloned the repository, I didn't use --recursive, but I just re-cloned it in a new folder using --recursive, and redid all the steps (excluding installation of the SDK/NDK/Bazel), up to the Bazel build. However, I got this output:
WARNING: /home/me/.cache/bazel/_bazel_me/95be766f297ed076701dc5f5e00e0a6b/external/protobuf/WORKSPACE:1: Workspace name in /home/me/.cache/bazel/_bazel_me/95be766f297ed076701dc5f5e00e0a6b/external/protobuf/WORKSPACE (@__main__) does not match the name given in the repository's definition (@protobuf); this will cause a build error in future versions.
WARNING: /home/me/.cache/bazel/_bazel_me/95be766f297ed076701dc5f5e00e0a6b/external/highwayhash/WORKSPACE:1: Workspace name in /home/me/.cache/bazel/_bazel_me/95be766f297ed076701dc5f5e00e0a6b/external/highwayhash/WORKSPACE (@__main__) does not match the name given in the repository's definition (@highwayhash); this will cause a build error in future versions.
WARNING: /home/me/.cache/bazel/_bazel_me/95be766f297ed076701dc5f5e00e0a6b/external/re2/WORKSPACE:1: Workspace name in /home/me/.cache/bazel/_bazel_me/95be766f297ed076701dc5f5e00e0a6b/external/re2/WORKSPACE (@__main__) does not match the name given in the repository's definition (@re2); this will cause a build error in future versions.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:avgpooling_op.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:bounds_check.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:eigen_activations.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:eigen_attention.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:eigen_backward_cuboid_convolutions.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:eigen_backward_spatial_convolutions.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:eigen_cuboid_convolution.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:eigen_patch_3d.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:eigen_pooling.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:eigen_softmax.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:eigen_spatial_convolutions.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:maxpooling_op.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:ops_util.cc' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:ops_util.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:pooling_ops_common.cc' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:pooling_ops_common.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/util/ctc:ctc_beam_entry.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/util/ctc:ctc_beam_scorer.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/util/ctc:ctc_beam_search.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/util/ctc:ctc_decoder.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/me/tf_m/tensorflow/tensorflow/core/BUILD:619:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/util/ctc:ctc_loss_util.h' directly. You should either move the file to this package or depend on an appropriate rule there.
INFO: Found 1 target...
ERROR: missing input file '@androidsdk//:platforms/android-23/android.jar'.
ERROR: /home/me/tf_m/tensorflow/tensorflow/examples/android/BUILD:47:1: //tensorflow/examples/android:tensorflow_demo: missing input file '@androidsdk//:platforms/android-23/android.jar'.
Target //tensorflow/examples/android:tensorflow_demo failed to build
Use --verbose_failures to see the command lines of failed build steps.
ERROR: /home/me/tf_m/tensorflow/tensorflow/examples/android/BUILD:47:1 1 input file(s) do not exist.
INFO: Elapsed time: 11.984s, Critical Path: 0.27s

Does platforms/android-23/android.jar exist in your sdk directory? Can you paste the WORKSPACE entry for your sdk and ndk please?

No, but the path platforms/android-24/android.jar does. This is my WORKSPACE entry:
android_sdk_repository(
    name = "androidsdk",
    api_level = 23,
    build_tools_version = "23.0.1",
    # Replace with path to Android SDK on your system
    path = "/home/me/Android/Sdk",
)

android_ndk_repository(
    name="androidndk",
    path="/home/me/Documents/android-ndk-r11c",
    api_level=21)

I suspect that the error is because the file says that the api is version 23 whereas I think I have version 24 installed. Is this the case? If so, then will I have to change the version of the build tools too?

Sounds like it. You can use /home/me/Android/Sdk/tools/android to confirm.
There's no problem having multiple Android api levels installed in your SDK, so you can use the android tool to install 23, or just change WORKSPACE to match what you have.

This is the error now:
ERROR: /home/me/.cache/bazel/_bazel_me/95be766f297ed076701dc5f5e00e0a6b/external/gif_archive/BUILD:14:1: C++ compilation of rule '@gif_archive//:gif' failed: namespace-sandbox failed: error executing command /home/me/.cache/bazel/_bazel_me/95be766f297ed076701dc5f5e00e0a6b/execroot/tensorflow/_bin/namespace-sandbox ... (remaining 46 argument(s) skipped).
external/gif_archive/giflib-5.1.4/lib/gif_err.c:10:29: fatal error: gif_lib_private.h: No such file or directory
 #include "gif_lib_private.h"
                             ^
compilation terminated.
Target //tensorflow/examples/android:tensorflow_demo failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 6.687s, Critical Path: 3.33s

And this is my SDK entry for the WORKSPACE file:
android_sdk_repository(
    name = "androidsdk",
    api_level = 24,
    build_tools_version = "24.0.0",
    # Replace with path to Android SDK on your system
    path = "/home/me/Android/Sdk",
)

Now you've run into an outstanding TF issue :)
https://github.com/tensorflow/tensorflow/issues/3374
I have a fix incoming, until then just run ./configure for a simple workaround.

Thanks for all your help! ./configure worked and the Bazel build was successful. One final question: is there a way to open up a project built with Bazel in Android Studio? I find it easier to view and edit code that way.

glad you got it working!
Android Studio should be able to let you edit the Android/Java portion of the demo, and maybe the native code too. See http://stackoverflow.com/questions/27570516/how-to-open-an-existing-project-in-android-studio
However, you're probably better off using bazel directly to actually build it for deployment.

this is a noob question but can i know how to find the location of bazel in my system

@swarathesh try which bazel

thanks @andrewharp

while ./configure im getting this error Can't find swig.  Ensure swig is in $PATH or set $SWIG_PATH.
but i have already installed brew install swig , what to do now @andrewharp


How to import Bible Notes into JWLMerge

Hi,
I noticed that when we export Bible Notes, it gets converted to an excel file. However, when we want to import the Bible Notes, it requires a txt file instead.
Could it be possible to allow exporting Bible Notes in an excel format?
It can help users who are comfortable on PCs to edit and update their notes and then using JWLMerge, create a proper JWL file which they can import into their JW library app

See also #75

There are several other proposals for importing/exporting which we should probably look at together.

The latest pre-release allows you to export to text in the same format as that required for import of Bible notes. This means you can export notes, modify outside JWL and then re-import them.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/772 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target.git

Docs Build status updates of commit 8d15c38:
Pending

Docs Build status updates of commit 8d15c38:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_master_2021-02-02-09-09-39 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


fix(@anu-vue/nuxt): Auto Import and Enable Icons

Fixes

Instead of doing scripts to get composables, Instead from the anu-vue package export Composables as const to then add them within the nuxt 3 module to then add dynamically.

To make this a easier process it would be then to create another export within package.json to have a path specific to the composables.

Enable Uno Preset Icon Preset by default and set the anu-preset icon props if not defined.
Fixed an issue where the icon object wouldn't be able to get passed within testing.

Why use pascal case for composables? Is it conflicting? I have same query for AnuComposables 🤔

@jd-solanki the reason behind this is get away from build scripts and to clarify that it’s exporting just composables. If we do it this way I can directly just import composables on demand and automatically add then to the library without any extra deps (globby) or extra scripts.
The way I was thinking is to have a import like so ‘import * as composables from “anu-vue/composables”’ this would make it to where I can just filter out composables. In this case we would have to refactor how vite gets bundled so short term is to add pascal case to tell the user that this is all the composables.

I guess this will be fine for nuxt but for users who don't use nuxt, they have to do something like the below:
import { Composables } from 'anu-vue'

const { useGroupModel } = Composables

Instead, we should provide:
import { useGroupModel } from 'anu-vue'

If we want clean way to auto import composables in nuxt, how about doing this:
// index.ts

export * as composables from './composables'
export * from './composables' // for vue

and later in nuxt module:
// module.ts

import { composables, presetAnu, presetIconExtraProperties } from 'anu-vue'

// ...
    // Add Auto Completions for Anu Composables
    Object.keys(composables)
      .filter(key => key.includes('use') && key !== 'useProp')
      .forEach(name => {
        addImports({
          name,
          from: 'anu-vue',
        })
      })

How about this? This preserves the casing convention (in JS we use PascalCase for Classes) and will allow vue users to import composable directly from anu-vue instead of importing composable object that contains composables.
Please let me know your thoughts on this.
Regards.

Furthermore, we can improve below line:
.filter(key => key.includes('use') && key !== 'useProp')

To something like:
composablesToExclude = ['useProp']

.filter(key => key.includes('use') && !composablesToExclude.includes(key))

It has two benefits:

Easy to understand (due to the name of array variable)
Makes it easier if we add more composables that shouldn't be used.

@jd-solanki That sounds good! I agree that would be better.
I will work on this in the morning! and have a fix up to discuss the changes proposed!


JFX Components not working

only some of the components are working when import into scene builder is there a reason for this

To get around such issues with Scene Builder I create and add such problem components in code via the initialize() method. Unfortunately it seems this lib is in trouble, quite a few compatibility issues with later versions of Java. Hiding the com.sun packages in Java 9 and the module system really screwed a lot of things up. :(


Build features

This is a draft so far, it builds on top of my other PR's. Eventually this should make the library configuration much simpler. I would appreciate any help in this area since I am primary only interested in Linux can cant test elsewhere.

Works now. this adds some more build dependencies to run a build.rs script now and a intermezzo on myself created https://docs.rs/conf_test/latest//conf_test/index.html for configuration checks (missed that functionality anyway rust/cargo). Initial build time got a bit up now, but that's only a a once only overhead.
OS Features and capabilities are now checked from the code snippets in conf_tests/*.rs and enabled automatically. Would be nice when people could test this on other (non linux/non macos) operating systems because it may happen that the tests are not 'on the spot' yet. For example that older OSX supports O_DIRECTORY but fcntl() won't return that flag left me initially puzzled. Newer OSX does not seem to have this behavior.
Anyway when problems are known its pretty simple now to to factor a 'feature' of of that and add a test enabling that feature.


Land detector (MC): Increase ground detection robustness

I would like to figure out if this change adds any value to the community.
Description
The ground contact state in the multicopter land detector, as the name suggest, tries to capture the instant when the vehicle touches the ground during a landing. It uses the following conditions for the detection:

There is a spread between down velocity and down velocity setpoint -> hit_ground. This assumes that either the pilot or the mission is demanding a large enough down velocity setpoint during landing. This is also the reason why you should keep pulling the throttle stick down when landing.
The vehicle is in a low_thrust condition which basically means that the current throttle is below a predefined threshold.

In the current implementation only one condition is necessary to trigger the ground_contact state.
This implementation requires both condition to be true at the same time.
However, if the vehicle already has landed or maybe_landed state active, then the old logic is used.
Advantages

reduces changes of false ground_contact detection. A false positive will lead to the controller pulling the throttle to 0 instantly which will result in the vehicle free-falling for a split second before the free-fall detection kicks in. Some vehicles might not recover from such a situation and crash.
This is the primary reason why this logic was implemented.

Disadvantages

landing detection might be slower for setups with bad land detector parameters.

I can confirm that this logic works well if the land detector is setup properly.
The main question is if this deteriorates the land detection performance for vehicles in general (because the land detection params are not setup properly.)
Looking forward to hear some opinions and to get some test results.
@PX4/testflights FYI.

@RomanBapst, I think that there logical error in the PR
On my case, I had cases of false ground detection, never because of low trust. (but because of altitude not locked) So basically this logic make the ground contact to be detected only when _maybe_landed_hysteresis or _landed_hysteresis is applied.
maybe_landed depend on ground_contact:
https://github.com/PX4/Firmware/blob/380cae18d199d1c237517f04d181468a7cad1420/src/modules/land_detector/MulticopterLandDetector.cpp#L215
and landed, depends on maybe_landed:
https://github.com/PX4/Firmware/blob/380cae18d199d1c237517f04d181468a7cad1420/src/modules/land_detector/MulticopterLandDetector.cpp#L238
so ground_contat will never will apply.

Thrust during landing is close to hover thrust which is not low, then when the drone touches the ground it would be low once the controller lowers it down to compensate for the vertical velocity (ideally being 0) differing from the setpoint (landing velocity setting). So in the end it entirely depends on the vertical velocity estimate, as does the "vertical movement" condition. But now the ground contact would be triggered once both conditions are met, which is more time than having one of them.
So I am thinking, what is the difference between this change and just increasing the ground contact hysteresis time?

Tested on PixRacer V4:
Modes Tested
Position Mode: Good.
Altitude Mode: Good.
Stabilized Mode: Good.
Mission Plan Mode (Automated): Good.
RTL:
- Procedure
Arm and Take off in position mode, after flying for approximately one minute, switched to altitude then stabilized mode proceed to switch to mission plan mode then make sure that vehicle follows all waypoints as shown in QGC, once completed all waypoin activate RTL and see landing behaviour.
Notes:
When the vehicle was landing, and I touched the ground, it began to make small leaps, but did not land completely and not disarm after a few seconds, it was necessary to land manually and disarm.
Log: https://review.px4.io/plot_app?log=7191421c-baa1-41f7-807e-6799a5044378

Tested on Pixhawk4 mini V5
Modes Tested
Position Mode: Good.
Altitude Mode: Good.
Stabilized Mode: Good.
Mission Plan Mode (Automated): Good.
RTL:

Procedure
Arm and Take off in position mode, after flying for approximately one minute, switched to altitude then stabilized mode proceed to switch to mission plan mode then make sure that vehicle follows all waypoints as shown in QGC, once completed all waypoin activate RTL and see landing behaviour.

Notes:
When the vehicle was landing, and I touched the ground, it began to make small leaps, but did not land completely and not disarm after a few seconds, it was necessary to land manually and disarm.
Log:
https://review.px4.io/plot_app?log=efe2f639-c69c-4d1d-8057-663b8cbbb8ef
rtl test
log:
https://review.px4.io/plot_app?log=465c0bd4-fdfb-4e19-bb7c-4ca9dc912d85

@RomanBapst What do we do with that?

@bresch I'm going to close this.


ENH : Clustering of carpet plot rows

This PR implements the reordering of the rows such that voxels with similar BOLD dynamics are placed close to one another.
One can choose between two clustering methods using the flag plot_carpet(ward=False/True): Ward’s hierarchical clustering or hierarchical average linkage clustering. As Nilearn recommanded ward's algorithm for clustering, the latter is set as default.
The clustering is done on euclidean distances between pairs of z-scored time-series within each regional block (crown, cortical GM, subcortical GM ...). This reordering follows the method developed by Aquino et al. 2020 to reveal non-global structures present in the fMRI signal.
IMPORTANT : This code builds on the PR #651 and as such, depends on the change in the fMRIprep repository commited under the pull request nipreps/fmriprep#2621 and on the change in the MRIQC repository under the pull request nipreps/mriqc#968

Hello @celprov, thank you for submitting the Pull Request!

In the file niworkflows/viz/plots.py:

Line 199:9: F841 local variable 'crown_surf' is assigned to but never used
Line 199:22: F821 undefined name 'get_crown_cifti'
Line 215:32: E203 whitespace before ':'
Line 303:29: E203 whitespace before ':'
Line 1275:47: E203 whitespace before ':'

To test for issues locally, pip install flake8 and then run flake8 niworkflows.

IMPORTANT : This code builds on the PR https://github.com/nipreps/niworkflows/pull/651 and as such, depends on the change in the fMRIprep repository commited under the pull request https://github.com/nipreps/fmriprep/pull/2621 and on the change in the MRIQC repository under the pull request https://github.com/nipreps/mriqc/pull/968

Why is such a long dependency chain necessary here? We should actually leverage that this ordering could be done independently of whether you have the crown mask or not. That would make this PR easier to check, and should generalize naturally to the crown when both PRs have been merged.

Please continue in the other PR, which is based off of master.


CT Core still has ties to non-existing "examples" folder for cppad

I just updated to v3.0.2 and had an error compiling my catkin workspace. The CT core still tries to include files from the "cppad/examples/" folder you guys had a while back. Lines 21 and 22 in this file point to files that do not exist: core.h
For the time being, version 3.0.1 works for me. When you can, please look into this.
FYI: On my system, I am running Ubuntu 16.04 with ROS Kinetic, and I installed cppad separately.

Dear @tsender, thanks a lot for reporting your issue, this is valuable information to us.
In order to reproduce the problem, would you mind letting us know

what version of CppAD you used in your custom install
how (and possibly where) you installed it?

Generally, in any valid recent CppAD release and installation, the example folder https://github.com/coin-or/CppAD/tree/master/include/cppad/example should be present and the code should build -- in that sense I am very curious to learn what version you installed and how you are using it. Once we have narrowed down the problem, I'm happy to provide a fix!

I do not know how to find the exact version of CppAD that I have, but I can tell you it is either 20190200.3 or .4 (based on the dates of each release and when I likely installed it).
I used the regular cmake process (git clone, cd build, cmake, sudo make install) and it is installed in /usr/include/cppad/ on my computer.
I ran catkin build again and here is the situation:

ct_core and ct_optcon build successfully on their own
In one of my catkin packages, one header file contains the line: #include <ct/optcon/optcon.h>

This include in my code is what is triggering the error. I also checked that my installed version of cppad contains the cppad/example/eigen_mat_inv.hpp" file, as this file exists in: /usr/include/cppad/example/atomic_two/eigen_mat_inv.hpp
I even tried to add the #Include <cppad/example/eigen_mat_inv.hpp> in one of my own files, and again, the compiler says it can't find the file. Any ideas why it wouldn't be able to find that one file? Thanks.

Thanks for the detailed report. I can reproduce the issue. A patch will follow as soon as my time allows!

@tsender please check out the above mentioned branch and test if this works for you. If yes, I will merge.

@markusgft I checkout out the bugfix branch, ran a catkin clean on my system and everything compiled successfully (aside from warnings about a few dependencies I don't have). I also tested the LQR solver in my gazebo simulation for verification and it ran without errors. So, you should be good to merge the change.
Thank you very much for the quick turn around!
FYI: I have been using 2019 CppAD releases in my own repo without ever having to include "cppad/example/..." in any header, and I have never had a problem. Removing those includes should not cause any harm.


Unable to load WASM Dlls in chrome extension .NET7

Hi Team ,
I have created an browser extension with Blazor webassembly (mingyaulee) .I am unable to load WASM assembly in chrome extensions and getting error as "Cannot read properties of undefined (reading 'BrowserExtension')" and unable to load coreinternalJs files .

Hi there, can you share a screenshot of the error or anything helpful from the extension error logs?

Hi there, can you share a screenshot of the error or anything helpful from the extension error logs?

I suspect that something is missing from the manifest. Are you loading any of the template extension from this project? If not, can you share the manifest you are using here?

manifest.zip

I suspect that something is missing from the manifest. Are you loading any of the template extension from this project? If not, can you share the manifest you are using here?

@mingyaulee ..I have attached the Manifest and other required file ..can u pls verify that and lemme know what is the exact issue ..Thanks in Advance ..!

@geetShukla I have tried with the files you have uploaded with the integration test project assemblies, I removed all the icons from manifest and inserted a new div in the html with the element ID expected from the Blazor app, and it was loaded without any issue for me. So it is something else that is causing the error that you are getting (browser, network, etc.).

@mingyaulee  Can you please share the files which u tried in integration test project .

I used the project from the main repo to try out, but you can use any of the samples in this repo as well. Just copy your files to the compiled output browserextension directory and load it from the browser.

I used the project from the main repo to try out, but you can use any of the samples in this repo as well. Just copy your files to the compiled output browserextension directory and load it from the browser.

@mingyaulee  if  I am using the published code available in samples  then I am able to load the extension properly .However when I am rebuilding the solution in VS2022 and publishing the code then the published code is not working and its giving error

as attached . I have attached the rebuild setup and error screenshots .

browserextension.zip

Based on the screenshots provided I am assuming you are not loading from the browserextension directory in the output. You can try to load it from there instead because _framework is not a valid folder name for a browser extension.

@mingyaulee  can u pls check the attached setup here ..same I am using to load the extension in chrome and there I donot have any folder with _framework
browserextension.zip
or _Content

@mingyaulee can u pls check the attached setup here ..same I am using to load the extension in chrome and there I donot have any folder with _framework browserextension.zip or _Content

@mingyaulee  any update on this issue ..I have attached the setup in previous reply ..Can u pls check it and let me know what is wrong in that ..

@geetShukla sorry I have been caught up in work recently. It seems like the publishing is causing issue when trimming and compression is enabled. A workaround for now is to disable both of them with
<PublishTrimmed>false</PublishTrimmed>
<BlazorEnableCompression>false</BlazorEnableCompression>

I am still figuring out what is causing it to break by testing with a standard Blazor app.

@geetShukla I have looked into the extension you have uploaded, it seems like the publishing job was not done properly by this package (the contents of the JS files were not replaced). Maybe you can try again with the latest version of the dotnet SDK and use dotnet publish to publish the extension again.

Closing due to inactivity


Cache get always unlinks cache file

Issue summary
On getting a cache item time() >= $expires_at always is true.
System informations

Information
Description

Pdp version
5.6.0

PHP version
7.2.15

OS Platform
Linux version 4.19.76-linuxkit (root@d203b39a3d78) (gcc version 8.3.0 (Alpine 8.3.0)) #1 SMP Thu Oct 17 19:31:58 UTC 2019

Standalone code, or other way to reproduce the problem
 $this->cache->set('foo','bar');
//Cache file is created
 $this->cache->get('foo');
//Cache file is deleted

Expected result
'bar'
Actual result
null

This is the problem I'm having in #259
Running CentOS 7 on Vagrant with a Windows 7 Host.
Once a day the cache file is deleted so I have to manually run composer from windows to update the cache files again.
I tried increasing the cache duration to 14 days with $manager = new Manager(new Cache(), new CurlHttpClient(), '14 DAY');, but it made no difference.
It's not a problem on remote, but still quite frustrating.


Rewrite Module Packaging Logic

Is your feature request related to a problem? Please describe.
Hello! I'm back for business.

The current logic with packaging modules is very complicated. I would know as I wrote most of it 😪 Having looked through some of the CDK esbuild logic I like what they've done and instead of trying to determine what modules to package based on npm ls or yarn list they instead simply create a fake package.json and run the equivalent install command. This should make supporting newer and future bundlers simpler.

I would like to add per function external and exclude logic. I think excluding deps because we can't detect them being used in a function is error prone so we should instead encourage users to use per function excludes if they choose to package individually.

Add support for pnpm.

Describe the solution you'd like
A clear and concise description of what you want to happen.
Described as above
Describe alternatives you've considered
N/A
Additional context
Add any other context or screenshots about the feature request here.

I'd also like to improve lifecycle hooks, add new hooks like before/after bundle, before/after package, etc.

it could be a valuable step toward the serverless-esbuild v2

I'll look to raise some PRs just shortly after the new years, in case someone is wondering.

Any progress on this?

Any progress on this?

Hey sorry,
In all honesty I've stopped using this as my company has either moved on to Serverless 4 who have built out their own esbuilf support or CDK so I have found little or no time outside of work or my other open source projects to build this out. I will likely have to give up on doing this.


docs.rs Failing to build

See build logs: https://docs.rs/crate/rust-criu/0.2.0/builds/579956
I believe this is because build.rs is unconditionally generating protobufs and the output is being written to a read-only path.
If that is the problem, would it make sense to have a separate create feature for generating protos?

Thanks. We will look into it. There is even a PR open for that (#13) from someone wanting the same.
I guess our background is too strong in C where we always generated protobuf during compile and never checked it in. But we have already seen it in the Go bindings that pre-generating is expected.

Should be fixed now with version 0.4.0. Thanks for letting us know.

@adrianreber Thanks a lot :bow: :heart_eyes:


[MicroPaymentQueue]: Review request denied.

Hello, I have noticed that whenever I am triggering the InAppReview flow, the device console is logging:
Process: itunesstored - Message: [MicroPaymentQueue]: Review request denied.

However, that seems like is not actually failing, since I am using a try-catch, and the catch clause is not executed either.
try {
	if (!InAppReview.isAvailable()) {
		throw new Error('InAppReview is not available');
	}
	const result = await InAppReview.RequestInAppReview();
	if (!result) {
		throw new Error('InAppReview request failed');
	}
} catch (error) {
	// Do something else.

Realised this is expected.


Runtime device

Added runtimedevice and schema description

Can I suggest we rather pass through only runtimeClassName into the spec dictionary?
The reasons being:

Potential for clashes with existing keys
Not inventing new properties in the chart (please point to existing schemas so we get the correct validation + dont have to maintain them)

My suggestion is
    spec:
      {{- if .Values.runtimeClassName }}
      runtimeClassName: {{ .Values.runtimeClassName }}
      {{- end }}

@marcelldls I asked Richard to use the approach you used for hostMounts etc.
I agree that this opens up clashes as you can override the whole podSpec, but I also did not want to release a new ec-helm-charts each time we needed a new podSpec item.
In my head the model is that we provide opinionated templating for the fields that most IOCs will make use of, but leave customisation capabilities reasonably open.
I'm not strongly attached to this model and @marcelldls helm skills are much more fresh that mine so I'll go with what you vote for.
@MaxHerbs to do the schema as Marcell requests, you want to refer out to the K8S manifest schema that you are duplicating e.g. see "resources": just above your addition to the template.

I've made the changed but haven't tested it. The beamline is in use today so follow up when I've verified it.

OK, lets go with that then. @MaxHerbs please can you switch to @marcelldls' suggestion above and update the schema to reflect that change.
(I'm not sure that calling out to the K8S schema would work for a single field - but if it does I guess you will get the correct K8S description of the field and formatting restrictions for the string)

@MaxHerbs The schema would look as follows:
        "runtimeClassName": {
          "$ref": "https://kubernetesjsonschema.dev/v1.18.1/_definitions.json#/definitions/io.k8s.api.core.v1.PodSpec/properties/runtimeClassName"
        },

Thanks! LGTM

Thanks both, merging ...

@MaxHerbs you can do the honours and release this as 4.1.3.
Do it from this page https://github.com/epics-containers/ec-helm-charts/releases

Draft new Release
Choose Tag and type in 4.1.3
Generate Release Notes
Publish Release


[Feature][DataQuality] Add common entity that are used in data quality task

What is the purpose of the pull request
This pull request adds enum、rule relevant entity and task parameter that are used in data quality task

Codecov Report

Merging #4883 (6928646) into data_quality_design (14d49bc) will decrease coverage by 0.61%.
The diff coverage is 23.46%.

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


connection issues

No matter where I try this from I get this error.. If I go to the IP listed via https I get this msg. I do an nslookup for getcomics.info and I get that ip and 10.112, but those still don't work in my browser.. Any idea what's going on here? the url works fine. I'm trying from 3 different networks. I'm on the newest version.
ERR_SSL_VERSION_OR_CIPHER_MISMATCH
(07-25-19 22:07:19) - Searching for https://getcomics.info/page/1/?s= failed.
(07-25-19 22:07:19) - System.Net.WebException: Unable to connect to the remote server ---> System.Net.Sockets.SocketException: A connection attempt failed because the connected party did not properly respond after a period of time or established connection failed because connected host has failed to respond <IP_ADDRESS>:443
at System.Net.Sockets.Socket.DoConnect(EndPoint endPointSnapshot, SocketAddress socketAddress)
at System.Net.ServicePoint.ConnectSocketInternal(Boolean connectFailure, Socket s4, Socket s6, Socket& socket, IPAddress& address, ConnectSocketState state, IAsyncResult asyncResult, Exception& exception)
--- End of inner exception stack trace ---
at System.Net.HttpWebRequest.GetResponse()
at Omnibus.Form1.searchComics(String function)

I redownloaded <IP_ADDRESS> and get this.
(07-26-19 14:37:51) - Searching for https://getcomics.info/page/1/?s= failed.
(07-26-19 14:37:51) - System.Net.WebException: Unable to connect to the remote server ---> System.Net.Sockets.SocketException: A connection attempt failed because the connected party did not properly respond after a period of time or established connection failed because connected host has failed to respond <IP_ADDRESS>:443
at System.Net.Sockets.Socket.DoConnect(EndPoint endPointSnapshot, SocketAddress socketAddress)
at System.Net.ServicePoint.ConnectSocketInternal(Boolean connectFailure, Socket s4, Socket s6, Socket& socket, IPAddress& address, ConnectSocketState state, IAsyncResult asyncResult, Exception& exception)
--- End of inner exception stack trace ---
at System.Net.HttpWebRequest.GetResponse()
at Omnibus.Form1.searchComics(String function)
(07-26-19 14:38:37) - Searching for https://getcomics.info/page/1/?s= failed.
(07-26-19 14:38:37) - System.Net.WebException: Unable to connect to the remote server ---> System.Net.Sockets.SocketException: A connection attempt failed because the connected party did not properly respond after a period of time or established connection failed because connected host has failed to respond <IP_ADDRESS>:443
at System.Net.Sockets.Socket.DoConnect(EndPoint endPointSnapshot, SocketAddress socketAddress)
at System.Net.ServicePoint.ConnectSocketInternal(Boolean connectFailure, Socket s4, Socket s6, Socket& socket, IPAddress& address, ConnectSocketState state, IAsyncResult asyncResult, Exception& exception)
--- End of inner exception stack trace ---
at System.Net.HttpWebRequest.GetResponse()
at Omnibus.Form1.searchComics(String function)


change from url to uri and added field url_type + videos

As explained here: https://github.com/CruxConnect/crux-api/issues/751
The api docs need to be updated in sync with the release ^.

Moving to backlog ... backend changes are not complete

FYI - these changes will be released for dev'ing against on Thursday.


[DO NOT MERGE] debug: Maximise GH runner space action failure

to debug issue https://github.com/canonical/bundle-kubeflow/issues/813 and #224

debugging effort completed, issue #224 is closed by #225


Added Back to home in Privacy Policy page

@YadavAkhileshh please review and merge it
before

after
https://github.com/user-attachments/assets/10576ffe-6f46-4645-bd06-3bf51bad9492

@YadavAkhileshh review this too

Merged this one too


SaveDscDependency throws error on Linux

Steps to reproduce
Run with version 1.18.0 using SaveDscDependency switch on an arbitrary ps1 file (doesn't have to be a configuration) on Linux (I'm running Ubuntu 18.04 on WSL).

It works fine on Windows when using PowerShell 6.2.

Invoke-ScriptAnalyzer -Path '/mnt/c/justascript.ps1' -SaveDscDependency

Expected behavior
Just run without an issue
Actual behavior
Invoke-ScriptAnalyzer : Value cannot be null.
Parameter name: path1
At line:1 char:1
+ Invoke-ScriptAnalyzer -Path '/mnt/c/Projects/GO-M-Automation/Src/Prod ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : NotSpecified: (:) [Invoke-ScriptAnalyzer], ArgumentNullException
+ FullyQualifiedErrorId : System.ArgumentNullException,Microsoft.Windows.PowerShell.ScriptAnalyzer.Commands.InvokeScriptAnalyzerCommand

PSMessageDetails      :
Exception             : System.ArgumentNullException: Value cannot be null.
                        Parameter name: path1
                           at System.IO.Path.Combine(String path1, String path2)
                           at
                        Microsoft.Windows.PowerShell.ScriptAnalyzer.Generic.ModuleDependencyHandler..ctor(Runspace
                        runspace, String moduleRepository, String tempPath, String localAppDataPath)
                           at Microsoft.Windows.PowerShell.ScriptAnalyzer.Commands.InvokeScriptAnalyzerCommand.ProcessR
                        ecord()
                           at System.Management.Automation.Cmdlet.DoProcessRecord()
                           at System.Management.Automation.CommandProcessor.ProcessRecord()
TargetObject          :
CategoryInfo          : NotSpecified: (:) [Invoke-ScriptAnalyzer], ArgumentNullException
FullyQualifiedErrorId : System.ArgumentNullException,Microsoft.Windows.PowerShell.ScriptAnalyzer.Commands.InvokeScriptA
                        nalyzerCommand
ErrorDetails          :
InvocationInfo        : System.Management.Automation.InvocationInfo
ScriptStackTrace      : at <ScriptBlock>, <No file>: line 1
PipelineIterationInfo : {}

Environment data

> $PSVersionTable
Name                           Value
----                           -----
PSVersion                      6.2.0
PSEdition                      Core
GitCommitId                    6.2.0
OS                             Linux 4.4.0-43-Microsoft #1-Microsoft Wed Dec 31 14:42:53 PST 2014
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

> (Get-Module -ListAvailable PSScriptAnalyzer).Version | ForEach-Object { $_.ToString() }
1.18.0
1.17.1

Agreed with Ben

Sorry, I've been busy lately, I plan to have a look at this possibly by next week

Thanks @bergmeister . Looking forward to it.

I can repro this and have a local fix ready. Will open a PR in the next days. The problem is this line here that is Windows specific as Linux does not have an APPDATA environment variable. The fix will be to replace it with something like ? Environment.GetEnvironmentVariable(RuntimeInformation.IsOSPlatform(OSPlatform.Windows) ? "LOCALAPPDATA" : "HOME") but I need to write a failing test now first and see if downloading an actual DSC module works end to end as well in Linux.

Thanks for all the great work @bergmeister!
To answer your question, It is not that we are using DSC on Linux. It's that we want our CI QA tests to pass on all ps files in the repo (some of them being DSC configurations) and off course script analyzer is a major part of the QA. QA tests are targeted to run both on Windows and Linux in our Azure DevOps environment. This is also done to assure every dev in our project can validate the base QA tests on their own system wether it's mac, win or linux.
That being said, we could opt for a more granular approach where we check every file individually to see if it's a DSC configuration and don't test when running on Linux. Right now we go in by storm and have a generic set of parameters for Invoke-ScriptAnalyzer for every file we check.

You're welcome @bgelens
Just that you are aware: PSSA is not 100% static and only 90% of the functionality is available on all PS versions/platforms:

PSSA makes e.g. Get-Command calls for it's analysis, therefore service will e.g be flagged up on Windows as an implicit alias to Get-Service (a PowerShell easter egg: if a command cannot be found by its name, PS will try to pre-pend Get- and execute that command if it can be found) but not on Linux where service is a binary.
The mentioned difference in feature set is mainly due to either PS not providing certain features on certain versions (e.g. DSC was only added in v5) or the underling ,Net Framework. For example, .Net Core does not have a Pluralization API and therefore the PSUseSingularNouns rule is not available in PowerShell Core.
The general guidance is to run PSSA on a PS version and platform where the scripts are likely to be run and if possible use at least PS version 5/6 (PS v3 and PS v4 are technically supported by PSSA v 1.x but have a reduced feature set due to PS not providing certain APIs and v3 might even have some bugs as we do not have CI tests run on it; we are thinking of dropping support for PS v3/4 in PSSA 2.0 btw)


WIP: centosci-tasks-dnxhq: Check for NPM dependency updates

Check for any outstanding update of point version of any outstanding NPM package dependency

[ ] npm-update

npm-update in progress on centosci-tasks-dnxhq.
Log: https://<IP_ADDRESS>:8493/logs/npm-update-593-20190313-131010/