Chapter 6: npm -i bcrypt requires --arch=arm64 flag on Mac

on my Macbook (Sonoma) bcrypt needed to be installed explicitly with --arch=arm64 flag before running the seed command
npm run seed

> seed
> node -r dotenv/config ./scripts/seed.js

node:internal/modules/cjs/loader:1340
  return process.dlopen(module, path.toNamespacedPath(filename));
                 ^

Error: dlopen(/Users/aronberezkin/Desktop/TUTORIALS/nextjs-dashboard/node_modules/bcrypt/lib/binding/napi-v3/bcrypt_lib.node, 0x0001): tried: '/Users/aronberezkin/Desktop/TUTORIALS/nextjs-dashboard/node_modules/bcrypt/lib/binding/napi-v3/bcrypt_lib.node' (mach-o file, but is an incompatible architecture (have 'x86_64', need 'arm64')), '/System/Volumes/Preboot/Cryptexes/OS/Users/aronberezkin/Desktop/TUTORIALS/nextjs-dashboard/node_modules/bcrypt/lib/binding/napi-v3/bcrypt_lib.node' (no such file), '/Users/aronberezkin/Desktop/TUTORIALS/nextjs-dashboard/node_modules/bcrypt/lib/binding/napi-v3/bcrypt_lib.node' (mach-o file, but is an incompatible architecture (have 'x86_64', need 'arm64'))
    at Module._extensions..node (node:internal/modules/cjs/loader:1340:18)
    at Module.load (node:internal/modules/cjs/loader:1119:32)
    at Module._load (node:internal/modules/cjs/loader:960:12)
    at Module.require (node:internal/modules/cjs/loader:1143:19)
    at require (node:internal/modules/cjs/helpers:119:18)
    at Object.<anonymous> (/Users/aronberezkin/Desktop/TUTORIALS/nextjs-dashboard/node_modules/bcrypt/bcrypt.js:6:16)
    at Module._compile (node:internal/modules/cjs/loader:1256:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1310:10)
    at Module.load (node:internal/modules/cjs/loader:1119:32)
    at Module._load (node:internal/modules/cjs/loader:960:12) {
  code: 'ERR_DLOPEN_FAILED'
}

Node.js v18.18.1

My suggestion is that you uninstall bcrypt and install bcryptjs.
The bcrypt package has a lot of, umm, quirks. I don't own a macbook, but are you able to confirm that fixes the issue?
(there will be a few lines of code you'll need to change from bcrypt to bcryptjs)

Related: https://github.com/vercel/next-learn/issues/353
Thank you @Connor-Skudlarek 🙏🏼, I've added a note about using bcryptjs to the chapter.


Handle block count RPC error gracefully

If we fail to fetch the block count, it will just return an error from update_index with a JSON-RPC error. However, any uncommitted blocks will not be committed because it doesn't exit the loop. They will have to be indexed again. This PR will just log the error but continue indexing, so we can keep our cache. It will show a progress greater than 100%, but better than having a JSON-RPC error and uncommitted blocks.

In my testing this call seems to always fail. Not sure why since I'm using only REST and p2p in the entire updater.rs except for this call and the one on line 81 which is the initial one. But probably something to investigate further. I think this should be merged regardless.

The reason it fails is because the socket connection times out because it's not used during sync with REST.

I confirm that https://github.com/casey/ord/commit/50f2ad7aebcca87bd31d2b28fa2d8ab67d7cf7cc on ord 0.5.0 has fixed the time-wasting issue of blocks re-indexing at 775000 every time.
This is awesome. I was able to finish the last 1% of indexing today 🎉
Now i have an ord index compatible with 0.5.0

I second @donnlee . My node succeeded on this as well (4GB raspberry pi)

This PR works!

Hi guys,
I have the version 0.5.0 and I still have the issue of indexing: HTTP response too short: length 0, needed 12
What can I do in this case?
Thank you

@jesuspb2 upgrade to 0.5.1.


job stuck at RUNNING

Environment

Qiskit Terra version: 0.25.0
Python version:  3.11
Operating system: Linux

What is happening?
Stuck at RUNNING:
jobid: cj8sr3c3bgbkhnv844cg
Previous stuck job, not been cancelled due to walltime:
jobid: cj8ihjdtks61ugttpos0
How can we reproduce the issue?
None
What should happen?
Should return results in around 30 mins.
Any suggestions?
No response

This is not an issue with the provider, not sure why this issue was transferred here. For questions regarding why jobs are stuck or not running properly, the qiskit slack would be a better place to ask https://ibm.co/joinqiskitslack


hide overflow for duration section

Status

[x] Ready

Related Issues
Fixes https://github.com/demisto/etc/issues/34218
Related PRs

branch
PR

web PR
link

Description
Hide horizontal overflow for duration section
Screenshots

@yonatan-roth @yanyo87 i dont see a border frame for the widget in the printed report - please make sure you are not missing it

@yonatan-roth @yanyo87 i dont see a border frame for the widget in the printed report - please make sure you are not missing it

@graphicool Please open an issue for that as it's not a regression.


Async code not working , results in exceptions

Context
Details can be found in the logs from the Slack channel from March 28th 2017, but the main idea is that I've not been able to get async invocation working with the latest Zappa codebase published today on master.
[..]
from zappa.async import run

app = Flask(__name__)
error = None

def send_data_test(post_data):
    time.sleep(15)
    archive_url = os.getenv('ARCHIVE_URL', 'https://www.google.com')
    r = requests.post(archive_url, json=post_data, verify=False)
    print(r.status_code)

@app.route('/', methods=['GET', 'POST'])
def index():
    if request.method == 'GET':
        return jsonify({'status': 'ok'})
    elif request.method == 'POST':
        request_body = request.get_json()
        run(send_data_test, args=request_body)
        return jsonify({ 'status': 'ok'})

When POST'ing to the main route , Zappa has unexpected exceptions.
Expected Behavior
I'm expecting that an HTTP Post to the main route to invoke the function send_data_test asynchronously. Zappa should provide the function the serializable arguments found in request_body which is passed into the HTTP Post.  Send_data_test in turn, does some additional work after a sleep.  So I'm expecting my HTTP post to immediately return, and I'm expecting the rest of send_data_test to complete 15 seconds later.
Actual Behavior
The initial HTTP Post results in a return of an HTTP 500 and an exception appears in the zappa tail logs.
[1490743150535] [INFO] 2017-03-28T23:19:10.535Z XXXXXXXX-140c-11e7-ad19-ab40e39d92e8 XXX.XXX.XXX.XXX - - [28/Mar/2017:23:19:10 +0000] "POST / HTTP/1.1" 500 291 "" "curl/7.53.1" 0/112.425
[1490743150574] 'NoneType' object has no attribute 'rsplit': AttributeError
Traceback (most recent call last):
  File "/var/task/handler.py", line 507, in lambda_handler
  return LambdaHandler.lambda_handler(event, context)
  File "/var/task/handler.py", line 241, in lambda_handler
  return handler.handler(event, context)
  File "/var/task/handler.py", line 359, in handler
  result = self.run_function(app_function, event, context)
  File "/var/task/handler.py", line 286, in run_function
  result = app_function(event, context)
  File "/var/task/zappa/async.py", line 177, in route_lambda_task
  func = _import_and_get_task(message['task_path'])
  File "/var/task/zappa/async.py", line 154, in _import_and_get_task
  module, function = task_path.rsplit('.', 1)

Possible Fix
I have a fix already figured out, and I'll issue the pull request as soon as I finish writing this issue up.
Steps to Reproduce
Code is enclosed above.  Additional context can be found in todays Zappa slack log (March 28th 2017).
Your Environment

Zappa version used:  Latest (installed from master today via)

pip install -e git+https://github.com/Miserlou/Zappa.git@master#egg=Zappa --no-cache-dir

Operating System and Python version:  Void Linux , w/ Ubuntu 16 in an LXC container , Python 2.7.12
Link to your project (optional): NA
Your zappa_settings.py:

{
    "uswest1_dev": {
        "app_function": "my_app.app",
        "aws_region": "us-west-1",
        "s3_bucket": "zappa-XXX",
        "environment_variables": {
          "ARCHIVE_URL": "https://XXXX.execute-api.us-east-1.amazonaws.com/useast1_dev/api/ingest"
        }
    }
}

I am using latest version of zappa and having this issue.
from zappa.async import task
ImportError: No module named async

tell me what to do??

@amajeed-dinCloud  - my guess is that they didn't publish the latest version to pypi yet, you can install it directly from the git repo.

Thanks, @mp3il . Strange they have updated the doc and published it before actual feature.
https://github.com/Miserlou/Zappa#asynchronous-task-execution

This should be pushed to pip today. Zappa Github always reflects master until we state otherwise.
I believe the problems mentioned above are fixed.


Update requirements.txt

update pymavlink

https://github.com/dronekit/dronekit-python/pull/728 :-)

Didn't check open pr's... sorry for duplication
Regards,
James

On 11 Jul 2017, at 08:35, Hamish Willee<EMAIL_ADDRESS>wrote:
#728 :-)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

No worries at all. I'd hoped from a review from @peterbarker so this was a timely reminder. Decided we need it so pushed in.


Add cancelreader bsd go1.17 compilation flags

and fix "in switch on ident (mismatched types uint64 and uint32)" error
when building for 32-bit bsd

LGTM!


Colors from tests results are not displayed in ghciwatch output

What happened?
When running a test suite (hspec) with ghciwatch with the following command:
ghciwatch
    --command "cabal repl my-app-test --repl-options=-fdiagnostics-color=always"
    --watch app --watch src --watch test
    --test-ghci "Spec.main"
    --clear

Colors to not appear in the test output.
In addition to passing in -fdiagnostics-color=always thru  a cli option, I have added it to ghc-options in my-app.cabal and project.cabal:
-- my-app.cabal
test-suite my-app-test
     ...
     ghc-options: -Wwarn -fdiagnostics-color=always

-- project.cabal
packages: christmas-gift-drawing.cabal
multi-repl: True
package *
  ghc-options: -fdiagnostics-color=always

What did you expect to happen?
I expected the colored output from the test to be visible in ghciwatch. Here's a screenshot of the output:

Here's what I would expect the it to look like (via ghcid):

Steps to reproduce the issue
I think there's enough info above!
The version of ghciwatch with the bug
ghciwatch 1.0.1

Hmmm, you might need to tweak something in your test suite runner (ghciwatch captures output so it might be seeing that stderr is not a tty or whatever), or this might be a more fixable bug with ANSI escape handling...

I was going to open an issue myself for another problem with colors -- maybe related?

In the screenshot below, there are two tmux panes.

In the upper one is the output of running main from cabal repl.
The lower pane has the output of ghciwatch --command "cabal repl ." --watch . --error-file errors.err --clear --test-ghci main

Since the color blocks are just a space character with colored background, nothing shows up at all in ghciwatch.

Thanks, that's a helpful test case!


P4_14 (v1.0.4): variable length header in add_header

In the specification of add_header primitive action, it is stated that

If the header instance was invalid, all its fields are initialized to 0.

I was wondering if we have a header like this:
header_type ht {
  fields {
    ...
    opt : *;
  }
  length : <length_exp>
  max_length : <max_length>
}

header ht h;

and (while h is invalid) we call
add_header(h);

what should happen in a case where:

eval(<length_exp>) > <max_length>
eval(<length_exp> * 8) < <sum_of_fixed_width_fields>

(As far as I understood) In the parsing phase, p4_pe_header_too_long or p4_pe_header_too_short will be thrown in such cases. But it is not clear to me what should happen in match-action phase. Is it compile time error? runtime error? undefined behavior?

Well, an easy (but probably not preferable) way around this is to just disallow add_header with variable length headers.

I think that the first problem is that we do not know the exact semantics of the length attribute. The wording in the spec really doesn't tell us much about how it is supposed to be used (and I quote):

The length attribute specifies an expression whose evaluation gives the length of the header in bytes for variable length headers.
– It must be present if the header has variable length (some field has width "*").
– A compilerwarning must be generated if it is present for a fixed length header.
– Fields referenced in the length attribute must be located before the variable length field.

In reality, this word "specifies" means nothing. It is not clear whether it is used for parsing, deparsing, processing in the controls or anything else. We can only do our best guess (and P4_16 spec is doing a much better job at this).
Also, the spec is quite silent about which primitives (if any) can be used with the variable length fields.
So, my best guess will be the following:

Variable length fields (as well as the length attribute) are only used for parsing/deparsing
Either there are no primitives that can be used with such fields or, at best, you can use modify_field, probably copying one such field into another
If modify_field is allowed, the copying should always done over the maximum allowed size for that field
The fields that are evaluated in the length attribute must be updated explicitly.

Based on that, disallowing add_header(h) would probably bee too harsh. For example, it should totally be possible to have two headers of the same type, e.g. h1 and h2. Suppose header h1 was extracted and is valid. It should be totally possible to either copy h1 into h2 or (which is the same), add h2 and then copy some fields from h1 there (including h1.op), fill other fields with whatever you need and be done.
Thanks,
Vladimir


support elasticsearch on https ?

Hi,
i have es which is running on ssl. normally we use key and cert to connect from outside.
can cerebro support this ?

@atomwangnus
To support this you have to installed a service in front of cerebro who can take care of this like a really simple apache that redirect to a virtualhost in his config file.
That's the simplest for me.
Or you can add a HAProxy in front, if you configure it, it will behave exactly as apache but it's a bit overkill just to setup this

For heads up:
Cerebro does not work when xpack.security.http.ssl.enabled is set to true.
It gives "403" (forbidden) error.
Cerebro does not give username/password prompt... no matter if CA is set or Cert validation is disabled:
/opt/cerebro/conf/application.conf:
...

play.ws.ssl {
  trustManager = {
    stores = [
      { type = "PEM", path = "/etc/cerebro/elasticsearch-ca.pem" }
    ]
  }
}

# Disabling certificate validation
#play.ws.ssl.loose.acceptAnyCertificate=true

The only way to make this work is to use the list of known hosts with the specified creds:
# A list of known hosts
hosts = [
  {
    host = "https://es-cluster-01:9200"
    name = "ES Cluster"
    auth = {
      username = "elastic"
      password = "REDACTED"
    }
  }
]

好的，我已收到

配置好使，厉害

es version: 8.2.2
cerebro docker tag: 0.9.4
if you have no elastic.pem , you can produce one according crt file;
for example:  openssl x509 -in http_ca.crt -out elastic.pem -outform PEM
then you can config cerebro using application.conf to connect to es over https


Please provide sample step 10,11 of Laravel API Project

I cannot login to Laravel API it error 400 (bad request)
please add detail for step 10,11
10.Add the "Laravel Password Grant Client" id to your .env file under the CLIENT_ID key
11.Add the "Laravel Password Grant Client" secret to your .env file under the CLIENT_SECRET key

i also have a hard time configuring this part please help

just this morning found something that would help. Though its not exactly what you might expect but worked for me. try this code
https://argon-dashboard-laravel.creative-tim.com/docs/getting-started/quick-start.html#html5-doctype

Hello and thanks for using our products.
We updated the installation instructions from the readme and pushed the latest version of code:
Open the projects database and run the following query SELECT id, secret FROM laravel_json_api_pro.oauth_clients WHERE password_client = 1; Add the id from step 10.) to your .env file under the CLIENT_ID key Add the secret from step 10.) to your .env file under the CLIENT_SECRET key
Let us know if this fixed the problem.
Thank you


Classrrom Mode: Setup M2M-based (non-interactive) flows for accessing User Data

Setup M2M-based (non-interactive) flows for Accessing User Data
One of the endpoints that the classroom team needs to return information in the below format:

Click here to explore the schema
{
<EMAIL_ADDRESS>{
        "blocks": {
            "learn-basic-css-by-building-a-cafe-menu": {
                "completedChallenges": [
                    {
                        "id": "5f33071498eb2472b87ddee4",
                        "completedDate":<PHONE_NUMBER>730,
                        "files": []
                    },
                    {
                        "id": "5f3313e74582ad9d063e3a38",
                        "completedDate":<PHONE_NUMBER>322,
                        "files": []
                    }
                ]
            },
            "learn-css-colors-by-building-a-set-of-colored-markers": {
                "completedChallenges": [
                    {
                        "id": "61695197ac34f0407e339882",
                        "completedDate":<PHONE_NUMBER>730,
                        "files": []
                    },
                    {
                        "id": "61695ab9f6ffe951c16d03dd",
                        "completedDate":<PHONE_NUMBER>322,
                        "files": []
                    }
                ]
            }
        }
    },

<EMAIL_ADDRESS>{
        "blocks": {
            "learn-basic-css-by-building-a-cafe-menu": {
                "completedChallenges": [
                    {
                        "id": "5f33071498eb2472b87ddee4",
                        "completedDate":<PHONE_NUMBER>730,
                        "files": []
                    },
                    {
                        "id": "5f3313e74582ad9d063e3a38",
                        "completedDate":<PHONE_NUMBER>322,
                        "files": []
                    }
                ]
            },
            "learn-css-colors-by-building-a-set-of-colored-markers": {
                "completedChallenges": [
                    {
                        "id": "61695197ac34f0407e339882",
                        "completedDate":<PHONE_NUMBER>730,
                        "files": []
                    },
                    {
                        "id": "61695ab9f6ffe951c16d03dd",
                        "completedDate":<PHONE_NUMBER>322,
                        "files": []
                    }
                ]
            }
        }
    }

}

Ref: https://gist.github.com/roberthuskins/04d8c767e1e6a0994440c94c2a00ece8
The implementation should ensure CORS and allow authorized access to the records as needed.

the schema provided is invalid JSON Schema.
I wrote the actual schema required to validate what I presume they are trying to interpret here.
the File schema does not have any constraints and will validation against any type of data because the schema is not well described in the example provided.  If someone can provide that, I can update the schema accordingly.

a valid example of a response based on the schema I provided.
[
    {
<EMAIL_ADDRESS>{
            "blocks": [
                {
                    "learn-basic-css-by-building-a-cafe-menu": {
                        "completedChallenges": [
                            {
                                "id": "5f33071498eb2472b87ddee4",
                                "completedDate":<PHONE_NUMBER>730,
                                "files": []
                            },
                            {
                                "id": "5f3313e74582ad9d063e3a38",
                                "completedDate":<PHONE_NUMBER>322,
                                "files": []
                            }
                        ]
                    }
                },
                {
                    "learn-css-colors-by-building-a-set-of-colored-markers": {
                        "completedChallenges": [
                            {
                                "id": "61695197ac34f0407e339882",
                                "completedDate":<PHONE_NUMBER>730,
                                "files": []
                            },
                            {
                                "id": "61695ab9f6ffe951c16d03dd",
                                "completedDate":<PHONE_NUMBER>322,
                                "files": []
                            }
                        ]
                    }
                }
            ]
        }
    },
    {
<EMAIL_ADDRESS>{
            "blocks": [
                {
                    "learn-basic-css-by-building-a-cafe-menu": {
                        "completedChallenges": [
                            {
                                "id": "5f33071498eb2472b87ddee4",
                                "completedDate":<PHONE_NUMBER>730,
                                "files": []
                            },
                            {
                                "id": "5f3313e74582ad9d063e3a38",
                                "completedDate":<PHONE_NUMBER>322,
                                "files": []
                            }
                        ]
                    }
                },
                {
                    "learn-css-colors-by-building-a-set-of-colored-markers": {
                        "completedChallenges": [
                            {
                                "id": "61695197ac34f0407e339882",
                                "completedDate":<PHONE_NUMBER>730,
                                "files": []
                            },
                            {
                                "id": "61695ab9f6ffe951c16d03dd",
                                "completedDate":<PHONE_NUMBER>322,
                                "files": []
                            }
                        ]
                    }
                }
            ]
        }
    }
]

Hi there, thanks for the feedback. We've revised our proposed JSON schema and this is what we've come up with. We're open to extra feedback with it and look forward to continuing the conversation! (a sample response has been attached at the bottom of this gist)
https://gist.github.com/GuillermoFloresV/537054fbef353966bc11c0f720db2fc9
As for the file schema not having constraints, it currently sits as an array of strings and I believe that I have provided the appropriate validation for it but if not please let me know and we can update it.

I've updated your schema to validate the payload you provided.   I left the comment in the gist.

Hi Jeremy, I've also left a comment to address some of your questions. Let me know if you have further questions.
@raisedadead , if you have a moment, could you take a look as well to make sure we are on the same page regarding the questions that Jeremy asked?

@raisedadead Is this MVP?
I was under the impression MVP was to match 1-1 the current API.

There hasn't been any movement on this for a while and I don't recall the argument for making an exception for this. As such, I've moved it to the post-MVP milestone.


Show full hypothesis space

I often use Clingo + Alan to check the full hypothesis space, with the rather convoluted approach of calling clingo alan.pl bias.pl -n0 | alanp where alanp is a script I have that turns answer sets into Prolog programs.
It might be useful to add this feature to Popper, so the user could simply call something like:
python popper.py folder -hspace 100 (or something similar)

Implemented https://github.com/logic-and-learning-lab/Popper/commit/44aaa13d6bac5b504db8ae3bb91d821625ee1886


How can you tell if there is one instance of the portals variable?

I'm struggling to fully understand how including the library as an external ensure that there is only one instance of the portals variable?
Does this portals variable get server-side rendered into the page source? To then be received by the client and used to remove the statically rendered portals before rehydration?

If you're going to include the library as an external do you not have to include the library manually say through a separate <script> tag? Your package.json references a main script lib/index.js which does not appear to be present in the repository.


Solution for the final optional exercise of Lab 4

My solution to the final exercise of Lab 4 of the IBM Quantum Challenge Fall 2022.
My LinkedIn Post: https://www.linkedin.com/posts/anonymousr007_quantumcomputing-quantum-quantumphysics-activity-7000145891160436736-DTg5?utm_source=share&utm_medium=member_desktop
Lab4: IBM Quantum Challenge Fall 2022
Name: Rishabh Singh
Score: 7268149
Checklist before sharing the notebook to the community

[x] Notebook is in the format Name_Score_Fall22_Lab4.ipynb
[x] Check for import statements

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


Make jerryx_port_handler_print_char truly a port function

In the previous approach jerryx_port_handler_print_char was implemented
in by the jerry-default-port. This implementation however required the
jerry-ext handler header file which created a requirement for the
jerry-default-port for the jerry-ext headers.

@dbatyai I've updated the PR. Now the putchar is used in the default port

lgtm


Object Detection

Hello Slavia, for the object detection module I made the following modifications:

yolov3_weights_to_keras Was modified which was initially constrained to yolov3 architecture on the coco-dataset was modified to allow for yolov3-SPP architecture which utilizes Spatial pyramid pooling alongside some changes to the middle FC blocks. This has been set as the default model but can be modified to the YOLO-v3 by changing the url parameter as well as setting SPP to 0. Secondly, this modification is no longer tied to the coco dataset and can be extended to any dataset for which the labels and weights exist. I included pascal VOC, openimages and coco. These can be passed as parameters. The YOLO layers of the architecture are modified accordingly.

yolo_video: An additional module for applying yolo to video streams uses an additional dependency: opencv-python

detector: The detector can be run in two modes: image and video modes. These are also parameters and the file should change accordingly

workflow: The initial workflow didn't allow parameters to the main be passed to the train and yolo_weights_to_keras modules, but I found a workaround to allow for the mlflow run parameters be passed to dependent modules.
In all it integrates nicely with mlflow.

In the near future, I hope to make certain improvements for even more flexibility in selecting models. Such as allowing for darknet config files to be processed directly and their corresponding weight files. At the moment there's also no provision for training on custom datasets. My modification only allows you use any pretrained weights you have access to provided you include it's label data and it falls into yolov3 or yolov3-spp. Provisions can also be made for yolov3-tiny among others.

@Akeemedes This contribution is amazing! Thanks for taking the time to improve the object detection task.
Please add a README file so which describes ability to detect objects in images, videos and streaming videos. Also, please include usage instructions which describes how to use this model on pascal VOC, openimages and coco datasets. Thanks in advance.

@slavakurilyak You're welcome. I have added a README file and made some minor modifications for ease of use with respect to supplied parameters. The readme file describes with examples some scenarios/use cases with datasets of choice, SPP as well as video/image option

@Akeemedes, thanks once again. your update has been successfully integrated.


Allow graphql.field.parent and code.namespace to be nil

Description of the bug
Source of the problem: https://github.com/open-telemetry/opentelemetry-ruby-contrib/blob/main/instrumentation/graphql/lib/opentelemetry/instrumentation/graphql/tracers/graphql_tracer.rb#L88
After turning on the enable_platform_field option to report execute_field and execute_field_lazy spans in our APM, we can now see some tests raise this error:
invalid span attribute value type NilClass for key 'graphql.field.parent' on span 'graphql.execute_field'

There are cases where we will add nil to graphql.field.parent, for example when the spans are parentless.
OpenTelemetry will validate the span attributes, will ignore invalid attributes and optionally call handle_error when doing so.
Our test helper currently raises an exception when handling such an error, which causes test failures.
We've had to specifically ignore the invalidgraphql.field.parent attribute message. We also do a similar thing for the code.namespace field:
next if message&.start_with?("invalid span attribute value type NilClass for key 'code.namespace'")
next if message&.start_with?("invalid span attribute value type NilClass for key 'graphql.field.parent'")

Share details about your runtime
N/A

Will be fixed by https://github.com/open-telemetry/opentelemetry-ruby-contrib/pull/867


Publish new version to npm

Could you please publish new version of the mercurius-integration-testing package to npm so it allows mercurius 14 as peer dependency?
Thanks :)

@mcollina could you make a release to npm, please? We cannot bump mercurius version because this package depends on an old version.

https://github.com/mercurius-js/mercurius-integration-testing/releases/tag/v8.2.0
published 8.2.0


R installation points to v0.1.7 when requestion v0.1.9

Run the documented installation method for R:
install.packages("duckdb", repos=c("http://download.duckdb.org/alias/v0.1.9/rstats/", "http://cran.rstudio.com"))

repo suggests this is v0.1.9, but installer reports:
trying URL 'http://download.duckdb.org/alias/v0.1.9/rstats/src/contrib/duckdb_0.1.7.tar.gz'

Same issue can be seen in browsing from http://download.duckdb.org/, e.g. 0.1.9 link takes you to:
http://download.duckdb.org/rev/3c353fac49d37719d7592d1819dfeb76ac541736/rstats/
Looks like this may have been flagged with other issues in #711.  I don't experience the build/install issue mentioned there though, only the version issue

Yes, there is some version number confusion here. We are working on getting duckdb on CRAN, so that should be cleared up in that effort as well.

This should now be fixed for now with the release of 0.2.0


Build kafka-go modules to popule go.mod/sum

Merging without waiting for CI, as this relates to a bug fix and doesn't change any code (and also CI is just really unreliable atm, so waiting for CI would delay this substantially)


AwsIotMqttConnectionListener : Connect request failure

I want to publish a message to a topic from an Ec2 instance however I am getting the following error.
My code was working fine and publishing messages but does not when I deployed to Ec2.
I have added a role(AWSIoTFullAccess) to the ec2 instance but still getting this error.
c.a.s.i.c.m.AwsIotMqttConnectionListener : Connect request failure

org.eclipse.paho.client.mqttv3.MqttException: MqttException
	at org.eclipse.paho.client.mqttv3.internal.ExceptionHelper.createMqttException(ExceptionHelper.java:38) ~[org.eclipse.paho.client.mqttv3-1.2.2.jar:na]
	at org.eclipse.paho.client.mqttv3.internal.ClientComms$ConnectBG.run(ClientComms.java:736) ~[org.eclipse.paho.client.mqttv3-1.2.2.jar:na]
	at java.base/java.lang.Thread.run(Thread.java:829) ~[na:na]
Caused by: java.io.IOException: WebSocket Response header: Incorrect upgrade.
	at org.eclipse.paho.client.mqttv3.internal.websocket.WebSocketHandshake.receiveHandshakeResponse(WebSocketHandshake.java:162) ~[org.eclipse.paho.client.mqttv3-1.2.2.jar:na]
	at org.eclipse.paho.client.mqttv3.internal.websocket.WebSocketHandshake.execute(WebSocketHandshake.java:80) ~[org.eclipse.paho.client.mqttv3-1.2.2.jar:na]
	at org.eclipse.paho.client.mqttv3.internal.websocket.WebSocketSecureNetworkModule.start(WebSocketSecureNetworkModule.java:65) ~[org.eclipse.paho.client.mqttv3-1.2.2.jar:na]
	at org.eclipse.paho.client.mqttv3.internal.ClientComms$ConnectBG.run(ClientComms.java:722) ~[org.eclipse.paho.client.mqttv3-1.2.2.jar:na]
	... 1 common frames omitted

com.amazonaws.services.iot.client.mqtt.AwsIotMqttConnectionListener.onFailure Connect request failure
	MqttException (0) - java.io.IOException: WebSocket Response header: Incorrect upgrade.
		at org.eclipse.paho.client.mqttv3.internal.ExceptionHelper.createMqttException(ExceptionHelper.java:38)
		at org.eclipse.paho.client.mqttv3.internal.ClientComms$ConnectBG.run(ClientComms.java:736)
		at java.base/java.lang.Thread.run(Thread.java:829)
	Caused by: java.io.IOException: WebSocket Response header: Incorrect upgrade.
		at org.eclipse.paho.client.mqttv3.internal.websocket.WebSocketHandshake.receiveHandshakeResponse(WebSocketHandshake.java:162)
		at org.eclipse.paho.client.mqttv3.internal.websocket.WebSocketHandshake.execute(WebSocketHandshake.java:80)
		at org.eclipse.paho.client.mqttv3.internal.websocket.WebSocketSecureNetworkModule.start(WebSocketSecureNetworkModule.java:65)
		at org.eclipse.paho.client.mqttv3.internal.ClientComms$ConnectBG.run(ClientComms.java:722)
		... 1 more

How are you sourcing the credentials used to sign the websocket handshake?

@bretambrose

How are you sourcing the credentials used to sign the websocket handshake?

I am using DefaultAWSCredentialsProviderChain to retrieve aws credentials then pass it to AWSIotMqttClient along with client end point, and client id.
DefaultAWSCredentialsProviderChain credentials = new DefaultAWSCredentialsProviderChain();
AWSIotMqttClient client = new AWSIotMqttClient(CLIENT_END_POINT, CLIENT_ID,
credentials.getCredentials().getAWSAccessKeyId(),
credentials.getCredentials().getAWSSecretKey());

I believe IMDS credentials are session based which means they need the session token as well.  They will also expire eventually after which the connection will permanently fail to establish on reconnect.


Hosting

Fix the add payment page

@andersonlebon  hahaha man am so sorry, my bad, just noticed that i submitted irrelevant link


Develop

Cellpose added as an app ( steinbock/_cli/apps.py ).
Cellpose model training added (steinbock/segmentation/_cli/train.py', 'steinbock/segmentation/train.py,  'Steinbock/segmentation/_cli/_cli.py').
Cellpose default parameters changed (steinbock/segmentation/cellpose.py).
Cellpose model zoo added (steinbock/segmentation/_cli/cellpose.py).
List of contributors updated (CONTRIBUTORS.md).
CHANGELOG.md updated.
Segmentation doc updated (docs/cli/segmentation.md).
Utils doc updated (docs/cli/utils.md).
Docker doc updated (docs/install-docker.md).
Opencv-python==<IP_ADDRESS> added to requirements, it was needed to run cellpose training (requirements.txt).
Unifying how panel column datatypes are handled (steinbock/preprocessing/imc.py).

The recent push includes change and fixes to address all the issues raised. I will now move on to test the Docker containers. After this PR is merged I will move on to address other issues via feature branches.

Hi @Jonas, how should we proceed with this PR?

Hi both, sorry for not getting back to you earlier. As commented, I've only reviewed the documentation-related issues so far. Will now take a closer look at the code changes and get back to you asap.

Hi @Milad4849! Apologies for the delay, I completely missed that you re-requested a review. Will do it asap, latest at the end of this week.
CC @SchulzDan

Also, the code still contains #TODO for list-like options


Twotech possibily displaying incorrect data

At one point there was a bug in the contents of Data7, relating to seeds and seed bowls.
An artifact of this remains visible on Twotech where you can still see Twotech showing the ability to put any type of seed into a bowl of gooseberry's, when this is not actually possible.
Need to investigate if Twotech is in fact misrepresenting the data and as to why. Is it possible this data hasn't been regenerated because there is no detected change (doubtful)? Larger bug in data generation causing a lot of things not to be regenerated after an update (likely)? Bug in processing scripts, causing the output of incorrect Data7 data (not so likely)?
I have tested this is not caused by browser cache.
https://twotech.twohoursonelife.com/2743-Bowl-of-Gooseberry-Seeds

Is this still a thing? I don't see all the erroneous transitions listed when I look at Bowl of Gooseberry Seeds or Bowl of Gooseberries

Appears to be resolved, thanks!


Slight tweak for combined row/column description

I had never fully understood what the row/column combination on a single element did exactly. Today I clicked on the "Edit in browser" link and found a different explanation in codepen. Key words here: "If you have just a single column, you can save some markup" -- A single column! This shed some light on the concept.
Maybe it's just me, but the current text "If you need a full-width column to use as a container," had never been clear to me which width it was referring to. Also, the "save some markup" part also cleared up the purpose of this technique. So basically I'm just proposing changing the current description to be a little more like the Codepen description.
Before submitting a pull request, make sure it's targeting the right branch:

For documentation fixes, use master.
For bug fixes or new features compatible with the next version, use develop.
For bug fixes incompatible with develop, use the support branch of the latest compatible version, which will be formatted support/6.x.

If you're fixing a JavaScript issue, it would help to create a new test case under the folder test/visual/ that recreates the issue and show's that it's been fixed. Run npm test to compile the testing folder.
Happy coding! :)

I like it! Thanks @adrianpauly


Wasm: Move generic context to local for catch and filter funclets

This enables access to the generic context in catch and filter funclets.  It follows https://github.com/dotnet/corert/pull/8125#issuecomment-622962485 and does the same for wasm - stores the generic context in a local (if exception blocks are present) instead of passing it around.

Thanks!


Requesting Update for pyDARN Package Information

Name: pyDARN
Description:  pyDARN is an open source python library for data visualization of SuperDARN data.
Standards:

community:  Good
documentation: Good
testing: Partially met (no automated testing - can be implemented in near future)
software maturity: Partially met (no conda installation)
python3: Good
license: Good

Conda is a recommendation ("should") not requirement. It looks like pydarn hits everything else on standard 3. Also hits 1, 4, 6, 7, 10, 14. So I'd suggest maturity be Good.

Thanks, that's been updated now!


Fetch app configs and use it for channel type dropdown

client.getAppSettings();

App configs are fetcheable with v1.0.0


Handover custom defaults from files to checkoutAndInitLibrary step

We wanted to pass custom defaults from a file instead from a shared lib.

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

Hi @srinikitha09, thanks for your contribution. We don't work on this repository, I'll send you a link to the new PR I've created.


Better warning for time being out of sync or being put out of sync by Trio

Is your feature request related to a problem? Please describe.
When manually changing the time on a (Medtronic) pump, Trio will move it back to what it was without warning or notification. According to a link shared by Marion, regarding Loop (not Trio), this is desired behaviour. However, the implementation imho is not.
Describe the solution you'd like
Whilst the feature can make sense, the implementation imho needs improvement. Currently, the only thing you see is a yellow clock icon hidden away in pump settings which you'll generally never access for MDT after setup. I suggest showing a yellow clock icon on the home screen or otherwise give a notification that Trio modified the time of the pump.
Describe alternatives you've considered
None. Also, this behaviour and a note that users should not manually change the time on their pump should be documented. It currently isn't.
Additional context

Technical Details

User Impact
Users will properly be warned about the pump time changing/being pushed out of sync (by design) by Trio. If not, the pump will be out of sync until the next DST. It may also save frustration not understanding why the pump keeps changing time for no apparent reason.

If the clock icon is not shown on the main screen for Trio 0.2.2 when there has been a time zone change, this issue should remain open until the next version is released. I know it is fixed there.
Alignment between phone time and pump time (with allowed time-zone differences) is a required feature of using a Medtronic pump. The reconciliation between pump reservoir history and app record for dosing from bolus or temporary bolus requires the time to be accurate.
In the olden days of Loop, there was a button for manually synching the time but as the code progressed, the time sync was modified to be automatic.
The only time difference allowed is when the phone is on a different time zone. In that case, all the therapy settings that have a schedule: Basal Rates, Carb Ratio, Insulin Sensitivity Factor and Target Glucose use the "pump time" for those schedules. The clock icon on the main screen following a time zone change, from travel or season, indicates that the "pump time" does not match the "phone time". When you update the "pump time" in that case, you are changing the hour.  Keeping the pump aligned to the minute/sec is automatic.  (For pod users - this is true across pods as well. Even though a pod doesn't have a date-time clock, the app keeps track of the time-zone for your pod and only changes it when you request the change.)

https://loopkit.github.io/loopdocs/faqs/time-faqs/#time-zones-daylight-savings-time-summer-time


Allow to store all data on s3 (blob and metadata)

Is your feature request related to a problem? Please describe.
Currently only the blobs are stored on s3 with s3ng, the metadata are kept on the local pv. This makes a scaled and distributed installation unnecessary hard because there is a separate distributed / replicated storage needed. Modern s3 stacks like minio replicate the data on their own, so there is normally no need for a complicated distributed storage like ceph, longhorn, glusterfs or others.
Describe the solution you'd like
When specifying the s3 storage driver I would expect that all data (Blobs and Metadata) are stored on s3, so that a additional pv is not needed anymore.
Describe alternatives you've considered
Well, not really one, because the need to have a distributed filesystem does not really exist anymore in our environment. Blob storages replicates themself and databases also. The configs normally reside in configmaps and therefore are replicated by k8s.
Additional context
I would expect that this should be possible, because high data apps like Grafana Loki (Logs)  or Grafana Mimir (Metrics) are already able to handle this.
I know that the old s3 driver stored all data on s3, but not in an efficient way. But I think there should be a solution, because this would really help to reduce the administrative efforts (backup, deployment, ...)

@iFrozenPhoenix Your observation is correct. The S3 storage has a high latency and is not fast enough.
In ownCloud 10 we used the database for storing metadata.
The S3NG driver tries to solve this in a different way. We store the blobs (actual bytes) in the S3 bucket. The high latency is not a big problem here because we only access the data on upload / download.
The metadata and the file tree is stored in the POSIX decomposed storage. That is important because ownCloud needs to have a performant implementation for listing directories and metadata lookup.

Blob storages replicates themself and databases also

I agree for the Blobstore part. But the database part is more difficult. Databases like MySQL are also a big PITA, especially in scaled setups and cloud native environments. So going back to the Database solution is also not something we want to do. It would also not solve the "split brain situations" between files and metadata.
There are very interesting concepts already drafted about tighter and better storage integrations.
We will definately look into CEPH and CEPH Rados in the future as well as Samba.
If you have any ideas how to improve the S3 driver please share them. Like already mentioned, we need a performant way to store and maintain a directory tree. That is unfortunately not possible  by design in an objectstore.

@micbar thank you for your explanations.
Yes, indeed I have a suggestion. I have already mentioned it in issue 2668, second last comment.
I had a few thoughts about it. Once I've read an article about OneDrive and it said that in most cases the users initially load up many files, i.e. many writes, and then the following operations are mainly read. I don't know if this behaviour is also valid for owncloud customer installations but I would expect that it is.
I assume that the Metadata are stored in a boltdb or something similar. This Metadata file is stored on the local disk.
So that read operations (listings, filename requests, ...) are served from the local index but after write operations the index is updated and uploaded to S3. Alternatively I could imagine that the index upload would also be possible for example ever x time interval.
This is also the way grafana handles loki and mimir storage.

still open


1 test failed. utc:utc and local (2337.6)

Client info
Date String   : Wed Jul 22 2015 19:40:09 GMT+0530 (India Standard Time)
Locale String : 22/7/2015 7:40:09 pm
Offset        : -330
User Agent    : Mozilla/5.0 (Windows NT 6.1; WOW64; rv:39.0) Gecko/20100101 Firefox/39.0

====
utc:utc and local (2337.6)
the hours (8) should be correct for local
// Expected 9
// Actual   8
8 === 9

Dup #2389.  Fix is already merged for next release.


ur variable

This issue is to report what appears to be a miscalculation for the ur variable, which is a derived metric based on a combination of US Census Bureau ACS estimates.

This is an upstream issue, I think.


[Feature Request] Add Core Functionalities to the Tree View

Add core functionalities (sort, context expansion, and search) to the linked mentions tree view to maintain consistency with the main interface.

Which functionality in the core plugin do you use heavily? I'm trying to prioritize which core feature to develop first since each will be a significant amount of work

Thanks for asking! It would be great to have an option to uncollapse higher-level backlinks while keeping the deepest levels collapsed for a cleaner, clutter-free view. While not a core feature itself, it feels like a sensible addition to improve functionality.

Love that idea!


erroneous comma in .rs output

using himecc 4.2.0 to generate a parser from a grammar with syntactic actions generates the function
/// Parses the specified string with this parser
pub fn parse_string(input: String) -> ParseResult<'static, 'static, 'static> {
    let text = Text::from_string(input);
    parse_text(text, , &mut NoActions {})
}

with one too many commas in the parse_text call

Good catch, thank you for the feedback.
This has been fixed in 4.2.1 and subsequent versions.


Update sample and documentation on partial view overriding

Fixes #79.
Sample updated to show how to override partials
Documentation updated with text on how to override partials

I've updated the branch based on feedback. Ready to merge unless there's anything else.


bugfix/LIVE-3913 Updated share library (fixes export logs and ops)

📝 Description
During testing of another feature we found out that saving logs was no longer working on Android > 12. Since this library is used to also export operations it's likely it's impacting that feature too. That issue was addressed by the developers of the library some versions ago and simply bumping it solves it.
❓ Context

Impacted projects: ledger-live-mobile
Linked resource(s): https://ledgerhq.atlassian.net/browse/LIVE-3913

✅ Checklist

[ ] Test coverage
[ ] Atomic delivery
[x] No breaking changes

QA ok by @alalmi-ledger merging

Well, after CI tells me I can


Hebrew Vowels Displaying Incorrectly

It looks like Hebrew vowels and chant markers are displaying incorrectly in lagrange:

This is how it should look:

While these markers are not used most of the time for Modern Hebrew, they are necessary for Scriptures and religious texts (the Bible, prayerbooks, etc.), they can be used when vowels cannot be assumed (like with names), and sometimes they are used to help beginners and children learn Hebrew.
I have a test file here: gemini://auragem.space/devlog/test.gmi

Probably the same issue as in #345 and #463: the HarfBuzz script is not being set according to the Unicode code points.

While this is being looked into, it might also be good to make sure Arabic and Farsi's vowel points work as well.
... I took a quick look at arabic and I don't think it's 100% correct either.


Can't build the docker image

Steps to reproduce:
sudo xcaddy build --with github.com/mholt/caddy-l4

Expected Behaviour:
Build the docker image.
Current Behaviour:
Fails to build.
Host Info:
Debian 12 (Bookworm) server
xcaddy 0.4.0
golang 1.19.8
Error Message:
2024/05/06 21:29:37 [INFO] absolute output file path: /mnt/containers/caddy/caddy_image/caddy
2024/05/06 21:29:37 [INFO] Temporary folder: /tmp/buildenv_2024-05-06-2129.809781120
2024/05/06 21:29:37 [INFO] Writing main module: /tmp/buildenv_2024-05-06-2129.809781120/main.go
package main
import (
caddycmd "github.com/caddyserver/caddy/v2/cmd"
    // plug in Caddy modules here
    _ "github.com/caddyserver/caddy/v2/modules/standard"
    _ "github.com/mholt/caddy-l4"

)
func main() {
caddycmd.Main()
}
2024/05/06 21:29:37 [INFO] Initializing Go module
2024/05/06 21:29:37 [INFO] exec (timeout=0s): /usr/bin/go mod init caddy
go: creating new go.mod: module caddy
go: to add module requirements and sums:
go mod tidy
2024/05/06 21:29:37 [INFO] Pinning versions
2024/05/06 21:29:37 [INFO] exec (timeout=0s): /usr/bin/go get -d -v -tags nobadger github.com/caddyserver/caddy/v2
go: added github.com/beorn7/perks v1.0.1
go: added github.com/caddyserver/caddy/v2 v2.7.6
go: added github.com/caddyserver/certmagic v0.20.0
go: added github.com/cespare/xxhash/v2 v2.2.0
go: added github.com/go-task/slim-sprig v0.0.0-20230315185526-52ccab3ef572
go: added github.com/golang/protobuf v1.5.3
go: added github.com/google/pprof v0.0.0-20210720184732-4bb14d4b1be1
go: added github.com/google/uuid v1.3.1
go: added github.com/klauspost/cpuid/v2 v2.2.5
go: added github.com/libdns/libdns v0.2.1
go: added github.com/matttproud/golang_protobuf_extensions v1.0.4
go: added github.com/mholt/acmez v1.2.0
go: added github.com/miekg/dns v1.1.55
go: added github.com/onsi/ginkgo/v2 v2.9.5
go: added github.com/prometheus/client_golang v1.15.1
go: added github.com/prometheus/client_model v0.4.0
go: added github.com/prometheus/common v0.42.0
go: added github.com/prometheus/procfs v0.9.0
go: added github.com/quic-go/qpack v0.4.0
go: added github.com/quic-go/qtls-go1-20 v0.4.1
go: added github.com/quic-go/quic-go v0.40.0
go: added github.com/zeebo/blake3 v0.2.3
go: added go.uber.org/mock v0.3.0
go: added go.uber.org/multierr v1.11.0
go: added go.uber.org/zap v1.25.0
go: added golang.org/x/crypto v0.14.0
go: added golang.org/x/exp v0.0.0-20230310171629-522b1b587ee0
go: added golang.org/x/mod v0.11.0
go: added golang.org/x/net v0.17.0
go: added golang.org/x/sys v0.14.0
go: added golang.org/x/term v0.13.0
go: added golang.org/x/text v0.13.0
go: added golang.org/x/tools v0.10.0
go: added google.golang.org/protobuf v1.31.0
2024/05/06 21:29:42 [INFO] exec (timeout=0s): /usr/bin/go get -d -v -tags nobadger github.com/mholt/caddy-l4 github.com/caddyserver/caddy/v2
github.com/mholt/caddy-l4 imports
github.com/mholt/caddy-l4/modules/l4tls imports
github.com/caddyserver/caddy/v2/modules/caddytls imports
github.com/smallstep/certificates/authority/provisioner imports
github.com/slackhq/nebula/cert imports
crypto/ecdh: package crypto/ecdh is not in GOROOT (/usr/lib/go-1.19/src/crypto/ecdh) 2024/05/06 21:29:51 [FATAL] exit status 1

golang 1.19.8

Go 1.19 isn't supported. Please use newer Go, preferably latest (1.22).


Question: Where are mixed instances policy variables set in the .tf code (17.24.0)

Description
Quick question; where is the terraform logic that default sets these mixed-instance variables? https://github.com/terraform-aws-modules/terraform-aws-eks/blob/e90c877a741ab3cc4215376a70f7bcc360b6a3d2/locals.tf#L104
I need a bit more help to see exactly where if mixed_instance and launchtemplates: set these variables

[x] ✋ I have searched the open/closed issues and my issue is not listed.

⚠ Note
Before you submit an issue, please perform the following first:

Remove the local .terraform directory (! ONLY if state is stored remotely, which hopefully you are following that best practice!): rm -rf .terraform/
Re-initialize the project root to pull down modules: terraform init
Re-attempt your terraform plan or apply and check if the issue still persists

Versions

Module version [Required]: 17.24.0

Terraform version: n/a

Provider version(s): n/a

v17.x is no longer supported - I recommend updating to v19.x


added array of tocify src file paths to bower.json main property

Added correct JS and CSS filepaths to the bower.json main property. Tools like Wiredep (https://github.com/taptapship/wiredep) use these to auto insert Bower dependencies into HTML.

+1


fix: Update env vars and log filter

Fixes #2505

Failed test is github/golang-samples/media

@codyoss @enocom can we go ahead and merge this?

Sorry, still catching up after being out a couple. This LGTM, force pushing. We will be skipping some tests in a future PR to get things back to green.


Crash when using 'Ice Barrage' on a Target Dummy from "MmmMmmMmmMmm"

Upon using this spell on a target dummy, my game crashes, just as the projectile hits the dummy it seems

What version of the Wizards mod were you using ?
Ice Barrage uses a projectile from Wizards, which name changed after the 1.1.0 release, and it is currently checking for the new one
If you're using a past version of Wizards, I'm guessing the old name doesn't match, so it likely causes that crash
Updating Wizards should fix it!

oop, yep! that was the issue, thanks! it works fine now :D sorry for the false alarm!


Delete redundant "see"

Delete redundant "see"

NOTE: Please check the “Allow edits from maintainers” box (see image below) to
allow reviewers to fix problems on your patch and speed up the review process.
Please delete this note before submitting the pull request.

This change is

/lgtm

@chenopis
PTAL

I'll just go ahead and merge this as it's trivial.


Change to static and class methods

updated ImageFilter to contain static and class methods
added to_json method for frontend
fixed dotted filter (was not working) and refactored it a bit

I have already modified the existing filters with this change.

resolved


Problem in CBLOF when the number of clusters is big and the train set has too many repeated value

Hi
If the train set has many repeated values and a large number of clusters is used, then some clusters will have the same value for the center. So, when defining the equation self.cluster_sizes_=np.bincount(clf.cluster_labels_), the results is an array smaller than the number of cluster, which generate an error and turns impossible to set large and small clusters. This could be avoided by changing self.cluster_sizes_=np.bincount(clf.cluster_labels_) to self.cluster_sizes_=np.bincount(clf.cluster_labels_, minlength=n_clusters). This is an issue that is damaging my code flexibility and I want to know if it is worth getting fixed.
Example of code:
from pyod.utils.data import generate_data
x = [[ 0.30244003],  [0.01218177],[-0.50835109], [-0.36951435],[ 0.97274482], [-0.68325119],
     [0.0], [0.0], [0.08], [0.0], [0.0], [ 0.0],[ 0.0], [ 0.0],[0.09], [0.0],[ 0.0], [0.0],
     [0.0], [ 0.0],[-20.29518778], [0.0],[ 0.0], [0.0],[ 0.0], [ 0.0],
     [0.0], [ 8.38548823], [0.0], [ 0.0]]
test = generate_data(train_only=True)
clf_name = 'CBLOF'
clf = CBLOF(alpha=0.1, n_clusters=15, beta=10, check_estimator=False)
try:
    clf.fit(x)
except Exception as ex:
    print(str(ex))
    print("\n Cluster centers: " + str(clf.cluster_centers_))
    print("\n Cluster sizes: " + str(clf.cluster_sizes_))
    print('\n Supposed to be the cluster size: ' + str(np.bincount(clf.cluster_labels_, minlength=15)))
    print("\n Large clusters: " + str(clf.large_cluster_labels_))
    print("\n Small clusters: " + str(clf.small_cluster_labels_))

Output:
index 11 is out of bounds for axis 0 with size 11

 Cluster centers: [[ 0.00000000e+00]
 [-2.02951878e+01]
 [ 8.38548823e+00]
 [ 9.72744820e-01]
 [-5.08351090e-01]
 [ 3.02440030e-01]
 [-6.83251190e-01]
 [-3.69514350e-01]
 [ 8.00000000e-02]
 [ 1.21817700e-02]
 [ 9.00000000e-02]
 [ 0.00000000e+00]
 [ 0.00000000e+00]
 [ 8.00000000e-02]
 [ 0.00000000e+00]]

 Cluster sizes: [20  1  1  1  1  1  1  1  1  1  1]

 Supposed to be the cluster size: [20  1  1  1  1  1  1  1  1  1  1  0  0  0  0]
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
<ipython-input-39-1b14a2099b96> in <module>()
     18 try:
---> 19     clf.fit(x)
     20 except Exception as ex:

/usr/local/lib/python3.5/dist-packages/pyod/models/cblof.py in fit(self, X, y)
    168         self._set_cluster_centers(X, n_features)
--> 169         self._set_small_large_clusters(n_samples)
    170

/usr/local/lib/python3.5/dist-packages/pyod/models/cblof.py in _set_small_large_clusters(self, n_samples)
    251
--> 252             if size_clusters[sorted_cluster_indices[i]] / size_clusters[
    253                 sorted_cluster_indices[i - 1]] >= self.beta:

IndexError: index 11 is out of bounds for axis 0 with size 11

During handling of the above exception, another exception occurred:

AttributeError                            Traceback (most recent call last)
<ipython-input-39-1b14a2099b96> in <module>()
     23     print("\n Cluster sizes: " + str(clf.cluster_sizes_))
     24     print('\n Supposed to be the cluster size: ' + str(np.bincount(clf.cluster_labels_, minlength=15)))
---> 25     print("\n Large clusters: " + str(clf.large_cluster_labels_))
     26     print("\n Small clusters: " + str(clf.small_cluster_labels_))
     27

AttributeError: 'CBLOF' object has no attribute 'large_cluster_labels_'

Thanks for your help,
Giovanna

Thanks for points this out. Will do some check and let you know if a PR is needed:)

@GiovannaR did some experiments and understood what is going on:)
The error happens when the underlying clustering algorithm could not form the exact number of clusters set by the user. In the case you mentioned, kmeans returns 11 clusters other than 15 clusters.
There are two ways to fix:

Use the actual number of clusters instead of the user-defined one, i.e., use 11 instead of 15 for the calculation. Throw warnings as well.
Raise errors directly to ask for a smaller number of cluster definition.

I feel enforcing np.bincount with minlength==n_clusters is unsafe as empty clusters may result in unpredictable results.
Any thoughts?

I changed the MiniBatchKMeans for the KMeans and the problem ended. When I use MiniBatchKMeans, it finds sometimes repeated cluster centers even if the number of cluster is smaller than the number of unique values. Using KMeans, it only finds repeated cluster centers when the number of cluster is bigger than the number of unique values. I will investigate more the both algorithms.
When comes to the error, I suggest turn it into a warning about changing the number of clusters to use and use the number which is appropriate instead the input.  It is possible to change the for i in range(1, self.n_clusters) in def _set_small_large_clusters(self, n_samples) function to for i in range(1, len(size_clusters), but I don't know if it is the ideal.
Thanks for your help,
Giovanna

refactor default clustering algorithm from MiniBatchKMeans to KMeans.
add a check to calculate the actual number of clusters.
supports multiple-core running with n_jobs.

Thanks for reporting this @GiovannaR :)


Widen scope of threaded column loops

:warning: This issue is a work in progress :warning:
One important ingredient of improving performance is leveraging single processor parallelism via multithreading. ClimaCore supports three ways of multithreading code:

multithreading single broadcast expressions
multithreading a group of column broadcast expressions with bycolumn
multithreading a group of slab broadcast expressions with byslab

Thank you @charleskawczynski I'm taking this one and https://github.com/CliMA/TurbulenceConvection.jl/issues/908 as some first tasks for Nat to start contributing to TC 🚀

Ok, sounds good, also, keep in mind that we have the namelist hierarchy in TC and CA now!

@dennisYatunin, can we make a column-wise version of set_precomputed_quantities!? This is theoretically better for performance (more work per thread)

Another performance regression in 84ba175 cc @dennisYatunin.
Just search for bycolumn in here to see where we've added additional launches.
cc @tapios. This will be a strong scaling GPU issue, not just a CPU multithreading issue. Basically, we need to minimize the number of bycolumn calls, and that's not something we can really enforce at the ClimaCore level, the implementation in ClimaAtmos must be mindful of this level of parallelism.
I'm not sure how to prevent this from happening. @dennisYatunin, can you please fix this or at least not introduce new bycolumn calls? Can @szy21 and or @trontrytel screen for this?

Thanks, @charleskawczynski. Could you please add a section "Best Practice for Developing Performant Code" to the code development best practice doc at https://github.com/CliMA/policies/wiki/Code-Development-Best-Practices-and-Policies, and add the issues you raise here, and others you are aware of, there? Please tag @simonbyrne and me as reviewers.

I'll be on the lookout for additional bycolumns from now on.
Would it be possible to come up with an estimate on how many are needed given our equations and how we split implicit/explicit tendencies?

Would it be possible to come up with an estimate on how many are needed given our equations and how we split implicit/explicit tendencies?

My hope is no more than 2 (one for precomputed quantities and one for summing tendencies)

I'm going to close this issue because this is really only impacting CPU performance now that bycolumn is functionally different on the CPU and GPU, and while widening the scope of the bycolumn loops on the CPU may improve performance, it may not be the most impactful direction since we're targeting the GPU more.


Can't resolve 'encoding' in Next.js

When I am initialising the Notion client in my lib/notion.ts with
const notion = new Client({
    auth: process.env.NOTION_API_KEY,
  });

I get the following error / warning in my debug console:
warn  -<EMAIL_ADDRESS>Module not found: Can't resolve 'encoding' in<EMAIL_ADDRESS>
Import trace for requested module:
<EMAIL_ADDRESS>./node_modules/.pnpm/@notionhq+client<EMAIL_ADDRESS>./node_modules/.pnpm/@notionhq+client<EMAIL_ADDRESS>./lib/notion.ts
./app/(navbar)/blog/page.tsx

I am using Next.js + Typescript with the new app/ library.
My versions are:
Next.js 13.2.4
Node v18.15.0
@notionhq/client v2.2.3
I'd be glad if anyone could help me resolve this issue. 👋

Hey @hecker, this is just a warning and shouldn't stop you from building your project. See #365 for some suggestions regarding this issue. I'm going to close this as it's a duplicate of #365.


refactor(oob)!: merge oob invitation parsing

BREAKING CHANGE: the agent.oob.parseInvitationShortUrl method has been removed and integrated with the agent.oob.parseInvitationUrl. Because of this, the agent.oob.parseInvitationUrl method is now asynchronous.

Codecov Report

Merging #1134 (f39895d) into main (f294129) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1134      +/-   ##
==========================================
- Coverage   88.32%   88.32%   -0.01%
==========================================
  Files         706      706
  Lines       16431    16429       -2
  Branches     2664     2664
==========================================
- Hits        14513    14511       -2
  Misses       1911     1911
  Partials        7        7

Impacted Files
Coverage Δ

packages/core/src/utils/parseInvitation.ts
75.00% <ø> (ø)

packages/core/src/agent/Agent.ts
95.72% <100.00%> (ø)

packages/core/src/modules/oob/OutOfBandApi.ts
84.72% <100.00%> (-0.11%)
:arrow_down:

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


Removed requestingUser from userQueryService and ucjQueryService

Basically ever since migrating to using createdBy where we use the actual context of the user, we do not need this anymore

@tharmaman .. looks like you got a merge conflict you have to resolve.

@tharmaman .. looks like you got a merge conflict you have to resolve.

done


fix(deps): update all

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

GitHub release
v1.1.3

Your semantic-release bot :package::rocket:


infra nw support

Looks like this needs  a fresh rebase? It is showing some other commits as part of this changeset.
Can you please provide a brief description of the functionality you're adding as well as the code changes? That will make it easier to review the changeset.

Changes largely looks good to me.
Please rebase to latest master, godep update ofnet and resubmit the PR.

@jojimt , @shaleman : yes, will rebase and resubmit PR. @jojimt - will add description of changes as well.

Details of changes added:
a) Networks can be of "infra" or "data" type. "data" nw type is for container endpoints. "infra" nw type is for running ceph storage or other such host-node traffic such that it shares the OVS uplink. This is needed in cases where a single 10G port is available at host, and both infra & data traffic need to share the ovs uplink.
b) In case of infra nw, the interface name (or internal OVS port name) is same as network name. Veth pair creation is not done for infra nw.
c) For "infra" nw type, an endpoint is automatically created when per node when the infra nw is created. Infra nw EP creation/deletion is triggered from netplugin on corresponding nw create/delete.

LGTM.


format on commit

If formatting is required on commits, reduce developer load by auto-linting on commit using GtiHub hook.

You mean "auto format" on commit? No auto-lint? :-)


I need full code on how to use this libray.

@monitorjbl could you please give me the full code on how to read a excel file using this library. I am in real need of it now and a Java Beginner.

On the repository README there is a nice example!


[FileUploader]: add shared file uploader component to repo

Add shared file uploader Carbon extension for reuse across the teams
1 - It supports bothbrowse and drag and drop design
2 - The component handles file reading and accepts a callback for when the file data is loaded

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.17.0 :tada:
The release is available on npm package (@release-1.x dist-tag)
Your semantic-release bot :package::rocket:


blank map using router-flux

Version
Tell us which versions you are using:

react-native-router-flux v4.0.0-beta.21 (v3 is not supported)
react-native v0.48.2

hello,
I have the following issue
https://github.com/airbnb/react-native-maps/issues/1653

When I use the Direct MapView component, the map is rendered. But when I use the component in conjunction with the router-flux library I have a blank map.
Only one red line appears on the side of the screen

package.json
"dependencies": { "react": "16.0.0-alpha.12", "react-native": "0.48.2", "react-native-keyboard-aware-scroll-view": "^0.3.0", "react-native-maps": "^0.16.3", "react-native-router-flux": "^4.0.0-beta.21" }
git project
https://gitlab.com/projetosReactNative/enjoyBR

i dont think cause router flux. you should set height and width for style Mapview.

Thanks for the answer @hungdev :D
I have made all the settings that the react-native-maps library shows in the documentation and even then the screen is still blank. I tested in a separate project just the map and it worked ok.

you are resolved. i think you should close the issue.

@hungdev I was able to solve using the command react-native link react-native-maps. The problem was not connected to the RNRF library


Remove notion of initialOps from IConnected

Given ODSP implements get_ops API flow to get ops from PUSH, we no longer need this concept.
It's useful to remove it, given that R11S does not use it.
Change can be done on the client rather quickly (just need to ensure that PUSH changes supporting it are in Prod).
PUSH changes need to be done much later, when 0.45 version of loader / driver is the min version across all clients.
@GaryWilber, @tanviraumi - FYI.

Dup of #7132


From Telos Dev Team - Add Support for ANY EOSIO Chain

Hi Team Greymass -
Great work on this wallet. We have forked it to support Telos, but have since added changes to allow our fork to work with any EOSIO blockchain. We think this could be a valuable addition to your core product and will further spread EOSIO's tentacles across cryptoverse!
Effort primarily in commit https://github.com/greymass/eos-voter/commit/63111b4d92b07333644a752d8d723bc82372a456

Thanks so much - and we completely agree. Apologies for never responding to #314, I really should have, and we ended up talking about it regularly over the last month on how to approach it. We decided that we will open up support and go multi-chain, and the groundwork you laid out is going to be a great guidepost on the areas that need to be touched to make that happen.
The timing is everything though, and is the reason I also forgot to response to your original request. The reason behind this is that since 0.5.1, I've been focused both ledger integration and a complete rebrand of eos-voter (new name, logo, full website), and @dafuga has been on a few supplementary features for a new 0.5.x minor release. I think the plan is for him to shift over to analyzing the work you've done up until this point, and then start applying/adopting it to our develop branch.
My hope is that @dafuga will be able to tackle most of this while I continue on with the rebranding in parallel.
Again - sorry we really didn't respond back, we need to get a bit better about our usage of github for communication. Thanks for what you've done in implementing this and helping enable TELOS while we've been busy. We completely agree that it's combined efforts like this that will help the core EOSIO software expand far beyond what we see today :)

No worries, and thanks for your support! You guys rock! Feel free to ping
me with any questions or if you feel that I could assist in anyway.
On Thu, Sep 13, 2018 at 9:27 PM Aaron Cox<EMAIL_ADDRESS>wrote:

Thanks so much - and we completely agree. Apologies for never responding
to #314 https://github.com/greymass/eos-voter/issues/314, I really
should have, and we ended up talking about it regularly over the last month
on how to approach it. We decided that we will open up support and go
multi-chain, and the groundwork you laid out is going to be a great
guidepost on the areas that need to be touched to make that happen.
The timing is everything though, and is the reason I also forgot to
response to your original request. The reason behind this is that since
0.5.1, I've been focused both ledger integration and a complete rebrand of
eos-voter (new name, logo, full website), and @dafuga
https://github.com/dafuga has been on a few supplementary features for
a new 0.5.x minor release. I think the plan is for him to shift over to
analyzing the work you've done up until this point, and then start
applying/adopting it to our develop branch.
My hope is that @dafuga https://github.com/dafuga will be able to
tackle most of this while I continue on with the rebranding in parallel.
Again - sorry we really didn't respond back, we need to get a bit better
about our usage of github for communication. Thanks for what you've done in
implementing this and helping enable TELOS while we've been busy. We
completely agree that it's combined efforts like this that will help the
core EOSIO software expand far beyond what we see today :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/greymass/eos-voter/pull/348#issuecomment-421201651,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdv0JXUDc9BOuIw_r1iImGjmT3ArT6Dks5uawXngaJpZM4WoXBj
.

--
-Marlon

Thanks @marlonwilliams ! My hope is to get started on this next week. I'll ping you if have questions.

Yes it’s currently under maintenance. You can use our EOS Miami testnet
node: https://api.eos.miami:17441
On Wed, Sep 26, 2018 at 3:09 PM Daniel Fugere<EMAIL_ADDRESS>wrote:

Hey @marlonwilliams https://github.com/marlonwilliams, I started
working on multi chain support this week (my branch can be found here
https://github.com/dafuga/eos-voter/tree/Support_all_eosio_blockchains).
I wanted to start testing my work on the Telos blockchain, but the TestNet
seems to be offline (http://testnet.telosfoundation.io/) ? Do you know
anything about this? Thanks 😄
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/greymass/eos-voter/pull/348#issuecomment-424834446,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdv0FjE2U9Q_sTcLRy1QmWZiTIXxoSAks5ue9ECgaJpZM4WoXBj
.

--
-Marlon

@marlonwilliams I managed to connect to your node, thanks 😄 Any chance you guys can use the history filters plugin?   Here is a link to the plugin: https://github.com/EOSIO/eos/blob/905e7c85714aee4286fa180ce946f15ceb4ce73c/plugins/history_api_plugin/history_api_plugin.cpp

Yes I had it enabled but disabled it during a replay. Sorry! We will do so
and let you know once completed. How do you intend on supporting those
wallet functions now that this plugin is deprecated?
On Wed, Sep 26, 2018 at 7:13 PM Daniel Fugere<EMAIL_ADDRESS>wrote:

@marlonwilliams https://github.com/marlonwilliams I managed to connect
to your node, thanks 😄 Any chance you guys can use the history filters
plugin?  Here is a link to the plugin:
https://github.com/EOSIO/eos/blob/905e7c85714aee4286fa180ce946f15ceb4ce73c/plugins/history_api_plugin/history_api_plugin.cpp
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/greymass/eos-voter/pull/348#issuecomment-424900720,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdv0HPKsq11IvF28n6LQE7gcMI4td6bks5ufAorgaJpZM4WoXBj
.

--
-Marlon

How do you intend on supporting those wallet functions now that this plugin is deprecated?

We intend to continue support of the plugin and/or alternatives that can replicate its behaviour, so that all wallets can continue to use the features. :smile:

And thanks for reenabling the plugin @marlonwilliams ! Looking forward to testing the wallet on the Telos blockchain 😄

Hey @marlonwilliams any ETA on getting that History plugin enabled? 😄

It's been enabled :-) are you experiencing any issues?
-Marlon
On Thu, Oct 4, 2018 at 4:07 PM Daniel Fugere<EMAIL_ADDRESS>wrote:

Hey @marlonwilliams https://github.com/marlonwilliams any ETA on
getting that History plugin enabled? 😄
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/greymass/eos-voter/pull/348#issuecomment-427197443,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdv0FB8slwJPhr61NertrzPwvzsWrmmks5uhpTBgaJpZM4WoXBj
.

@marlonwilliams hmm that is strange. When the wallet calls https://api.eos.miami:17441/v1/history/get_key_accounts the API returns a 500 with this error {code: 500, message: "Internal Service Error",…} code : 500 error : {code: 10, name: "assert_exception", what: "Assert Exception", details: []} message : "Internal Service Error".

Can you send me a message on telegram @qubicles

Hey @marlonwilliams, my bad it was returning a 500 because I was using an address starting with EOS instead of TLOS 🤦♂

Going to close this one out - we have the multichain code implemented in our build now, will be available in a release shortly.


Simplistic fix for metamask loading state

Just to not confuse user more than necessary.
This could be done better.

I did not change much, I will come back to it later.


Fix/blueprint manifest template

Fixes #55 : Blueprint manifest template needs to be updated

Modifying the blueprint manifest template as boshv2 manifest

Modifying the vm and disk type, network type for blueprint manifest

Bumping the blueprint version to 1.7.0

changes are fine merging it


[wip] Toggle SELinux volume label based on OS version

What this PR does / why we need it:
We've been having an issue using the old label on the latest versions.
Source: https://docs.openshift.com/container-platform/3.6/install_config/persistent_storage/pod_security_context.html

The volume will be given a type which is accessible by unprivileged containers. This type is usually container_file_t, which treats volumes as container content. Previously, the label specified was svirt_sandbox_file_t. This label is no longer used due to security concerns.

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

Use the container_file_t volume label on newer os versions.

WIP while I test that operatingsystemrelease does what I expect

I can run the quick verify command locally and it passes.
make -C puppet/modules/tarmak verify

For some reason the variable for the os release looks to not be getting set in the test infra run.

@charlieegan3 I think to reproduce it locally you might need to set this:
export FIXTURES_YML=.fixtures.yml.local

I'm able to see the same failures if I run it with the fixtures variable (thanks Christian)
FIXTURES_YML=.fixtures.yml.local make -C puppet/modules/tarmak verify

Investigating now...

Closed in favour of #211


[Type 3] Publish directly using Datasource/Workbook objects from the Document API

Description
Unless this functionality already exists, it would be helpful to directly use created Datasource/Workbook objects from the Document API as the source when publishing content via TSC. Currently, you have to pass a filepath or a file object, which I haven't found an example of. This would also reduce having to save files to the file system for quick on the fly datasource/workbook creation.
from tableaudocumentapi import Connection, Datasource
import tableauserverclient as tsc

ds = Datasource.from_connections(caption="", connections=[some_connection])

# do login magic, get information for publish
server.datasources.publish(
    datasource_item=tsc.DatasourceItem(project_id, "My Cool Datasource"),
    file=ds, # object? item?
    mode=server.PublishMode.Overwrite,
    connection_credentials=tsc.ConnectionCredentials(username, password),
)

This should already be possible:
import io
from tableaudocumentapi import Connection, Datasource
import tableauserverclient as tsc

ds = Datasource.from_connections(caption="", connections=[some_connection])

# do login magic, get information for publish

# Save as a file-like object in memory
dsblob = io.BytesIO()
ds.save_as(dsblob)

server.datasources.publish(
    datasource_item=tsc.DatasourceItem(project_id, "My Cool Datasource"),
    file=dsblob,
    mode=server.PublishMode.Overwrite,
    connection_credentials=tsc.ConnectionCredentials(username, password),
)

Downloading from Tableau Server to such a file-like object by TSC is not yet supported (see #1172), but publishing should work.


WIP: fix(conn_pool): add watcher to clean stale conn in connection pool

What type of PR is this?
fix: add watcher to clean stale conn in connection pool
What this PR does / why we need it (English/Chinese):
en: Current implementation of long connection pool is not able to clean stale connection in time.
This PR adds mechanism to clean the stale connections.
zh: 在长连接池中添加 watcher 以及时清理过期的空闲连接

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


[BACKLOG-22720] Fix for hidden files/folders

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'plugins/file-open-save/core' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'plugins/file-open-save/core' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  plugins/file-open-save/core/src/main/java/org/pentaho/repo/controller/RepositoryBrowserController.java
  plugins/file-open-save/core/src/main/java/org/pentaho/repo/endpoint/RepositoryBrowserEndpoint.java
  plugins/file-open-save/core/src/main/java/org/pentaho/repo/model/RepositoryDirectory.java
  plugins/file-open-save/core/src/main/java/org/pentaho/repo/model/RepositoryFile.java
  plugins/file-open-save/core/src/main/java/org/pentaho/repo/model/RepositoryObject.java
  plugins/file-open-save/core/src/main/javascript/app/components/files/files.html
  plugins/file-open-save/core/src/main/javascript/app/components/folder/folder.html

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/plugins/file-open-save/core/src/main/java/org/pentaho/repo/model/RepositoryDirectory.java

:heavy_exclamation_mark: ':' should be on a new line. at Line: 63 Column: 118

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'plugins/file-open-save/core' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'plugins/file-open-save/core' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  plugins/file-open-save/core/src/main/java/org/pentaho/repo/controller/RepositoryBrowserController.java
  plugins/file-open-save/core/src/main/java/org/pentaho/repo/endpoint/RepositoryBrowserEndpoint.java
  plugins/file-open-save/core/src/main/java/org/pentaho/repo/model/RepositoryDirectory.java
  plugins/file-open-save/core/src/main/java/org/pentaho/repo/model/RepositoryFile.java
  plugins/file-open-save/core/src/main/java/org/pentaho/repo/model/RepositoryObject.java
  plugins/file-open-save/core/src/main/javascript/app/components/files/files.html
  plugins/file-open-save/core/src/main/javascript/app/components/folder/folder.html


Using AFNetworking to create a Static lib, failed on linker phase!

Hello,
I have a very unpleasant problem. I want to create a Static Lib for the networking layer of my project and add it to my project as a subproject. Every thing is fine on compile phase, but I get Undefined Symbole on linker phase.
"OBJC_CLASS$_AFHTTPSessionManager", referenced from
The lib is built successfully but when I add it as a subproject to my main project it cannot find AFNetworking classes.
Any help would be appreciated.
Thanks

Hello,
I have a very unpleasant problem. I want to create a Static Lib for the networking layer of my project and add it to my project as a subproject. Every thing is fine on compile phase, but I get Undefined Symbole on linker phase.
"OBJC_CLASS$_AFHTTPSessionManager", referenced from
The lib is built successfully but when I add it as a subproject to my main project it cannot find AFNetworking classes.
Any help would be appreciated.
Thanks


seedcracer is not working

Hello i have the seedcrackerX-2.13.1-test instaled but when i start up signleplayer, multiplayer it doesnt work i can see xraied objects but when i come to them nothing happens.

Nothings supposed to happen when you walk to them, the data will be stored (checked using /seedcracker data bits) and then the cracking process will begin after you've found 5 structures.
Read here or watch here


deps(selenium4): upgrade to<EMAIL_ADDRESS>

expect some/all tests to fail because of api changes

is the plan to upgrade to using the alpha, or is this just testing the integration in preparation for the stable 4.0.0 release?

The plan is to get this release in. I think selenium-webdriver is not doing as many new releases since they hit W3C standards. It has been about a year since 4.0.0-alpha.1 was released so we are going to get this in. Progress is here: https://github.com/angular/protractor/milestone/45
I have this upgrade with all the fixes in this: https://github.com/angular/protractor/pull/5095 So I'm closing this one.
I hope to get this done sometime this week.


There is a bug in the new version 1.1.0

I/flutter (21183): type 'Null' is not a subtype of type 'bool' in type cast I/flutter (21183): #0      Emoji.fromJson (package:emoji_picker_flutter/src/emoji.dart:30:40) I/flutter (21183): #1      RecentEmoji.fromJson (package:emoji_picker_flutter/src/recent_emoji.dart:20:13) I/flutter (21183): #2      MappedListIterable.elementAt (dart:_internal/iterable.dart:413:31) I/flutter (21183): #3      ListIterator.moveNext (dart:_internal/iterable.dart:342:26) I/flutter (21183): #4      new _GrowableList._ofEfficientLengthIterable (dart:core-patch/growable_array.dart:188:27) I/flutter (21183): #5      new _GrowableList.of (dart:core-patch/growable_array.dart:150:28) I/flutter (21183): #6      new List.of (dart:core-patch/array_patch.dart:51:28) I/flutter (21183): #7      ListIterable.toList (dart:_internal/iterable.dart:213:44) I/flutter (21183): #8      EmojiPickerInternalUtils.getRecentEmojis (package:emoji_picker_flutter/src/emoji_picker_internal_utils.dart:136:56) I/flutter (21183): <asynchronous suspension> I/flutter (21183): #9      EmojiPickerState._updateEmojis (package:emoji_picker_flutter/src/emoji_picker.dart:214:22) I/flutter (21183): <asynchronous suspension> I/flutter (21183): ----------------------------------------------------

@cnkygmr Thank you for reporting. Can you tell me the steps that led to this error? I assumed it might happen when upgrading from 1.0.8 to 1.1.0 but I can't reproduce it.

Can you let me know if my changes above fixes your issues?
  emoji_picker_flutter:
    git:
      url: https://github.com/Fintasys/emoji_picker_flutter.git
      ref: master

@Fintasys I tried like this, no problem, it worked

Can you let me know if my changes above fixes your issues?
  emoji_picker_flutter:
    git:
      url: https://github.com/Fintasys/emoji_picker_flutter.git
      ref: master

@Fintasys I tried like this, no problem, it worked

Thank you for checking, will release it as 1.1.1 👍


Elasticache: enhance logging for test assertion failures

Description of changes:
Similar to #696, but also logging an AWS describe call and k8s object state where applicable (in addition to controller logs)
Also changing config for github workflow (script is complaining about AWS_PROFILE being unbound)
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Also I still can't add reviewers, I'm a member of the aws-controllers-k8s organization but I believe I also need to be added as a contributor for this repo?

Nice work @echen-98 .
I have added few comments. Please review.

@jaypipes could you take a look when possible?

@echen-98 I presume you're working on the Python-based versions of these tests, yes?

@echen-98 I presume you're working on the Python-based versions of these tests, yes?

Sort of – any new test we write should be in Python. I'm also currently fixing an issue with one of the snapshot tests, and as part of that I'm rewriting it in Python, but we haven't started an all-out effort to convert all of our existing bash tests yet.
I think this PR was mainly us trying to get our tests working on Github Actions for the moment rather than a further commitment to keeping our tests in bash.

@echen-98 I presume you're working on the Python-based versions of these tests, yes?

Sort of – any new test we write should be in Python. I'm also currently fixing an issue with one of the snapshot tests, and as part of that I'm rewriting it in Python, but we haven't started an all-out effort to convert all of our existing bash tests yet.
I think this PR was mainly us trying to get our tests working on Github Actions for the moment rather than a further commitment to keeping our tests in bash.

Sounds good, @echen-98!


TypeScript doesn't report incorrect type when returning AxiosResponse but function declares some different return type

🔎 Search Terms
axios, return, no type check
🕗 Version & Regression Information
The app was using TS 4.9.5, but I updated to 5.3.2 and it seems the same.
⏯ Playground Link
https://codesandbox.io/p/sandbox/axios-ts-bug-gty2pg?file=%2Findex.ts%3A20%2C1
💻 Code
interface SomeBody {
  field1: string;
  field2: string;
}

async function fetchSomething(): Promise<SomeBody > {
  // Buggy code is missing a call to .data on the AxiosResponse
  return await axios.get('http://example.com/some/api'); // TypeScript is fine, but it should not be
}

🙁 Actual behavior
It does not fail type checking, however if you introduce an intermediate variable it does:
async function fetchSomething(): Promise< SomeBody > {
  const response = await axios.get('http://example.com/some/api');
  return response; // TS2322: Type  AxiosResponse<any, any>  is not assignable to type SomeBody
}

🙂 Expected behavior
It fails type checking because the wrong type is returned by axios.get for the return type declared by fetchSomething
Additional information about the issue
No response

Thanks I will report it over there


Test env

hero orbit is needed some updates pls check asap

Merge to main branch without any possible errors...


Requesting Open Source License for DDEV development and automated testing

Is your feature request related to a problem? Please describe.
A lot of DDEV users like Orbstack, I'm sure more will.
DDEV is a FOSS completely noncommercial project
However, you've just turned off the easy license.
Describe the solution you'd like

Provide open-source license for DDEV development
Provide 5 open-source licenses for DDEV automated testing

Describe alternatives you've considered
Not providing explicit support for OrbStack.
Additional context
Add any other context or screenshots about the feature request here.

Please email<EMAIL_ADDRESS>and I can hook you up with free licenses.


Only absolute urls are supported in example. And get empty object istead of file.

In your example

Client logs:

When i'm trying to make same thing on my server and client. I got empty object istead of file :(

Any ideas?

Just looking at the resolver screenshot, it seems you are missing await:
const { stream, filename, mimetype, encoding } = await avatar

Did you end up getting to the bottom of this one?

We can reopen if the plot thickens 🙂


Change log output dir (Log enhancement not included)

Overview
This Pull Request is about changing the log output destination.
Changes

Log Output Destination Change:

The log output destination has been updated to $HOME/.cache/refer/log/refer.log.

Log Enhancement:

Work on improving the logs is still in progress and might take a bit more time to implement.
I'll provide more details on the enhancement in future Pull Requests.

Next Steps
I'll create a separate issue for the log enhancement and submit a PR once it's done. This PR mainly focuses on changing the log output destination, so I'd appreciate your review and merge.
Related Issue

#7

@mtsalt
Thank you so much for your contribution!
Could you just squash your commits into one? Then we should be good to go.
If you would like, you may navigate to the CONTRIBUTING.md for style guide references which might be relevant moving forward.

@Feohr
Thank you for your response; I squashed the commits!


Update UIView Box Implementation

Current dependencies on/for this PR:

trunk

PR #1577   👈

This comment was auto-generated by Graphite.


Issue #102 - HubSpotOAuthApi Authorize and Refresh not returning tokens

Description
Bug fix for Issue #102
Purpose

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

Checklist

[ ] I have added tests to prove that what I have fixed or added does indeed work
[ ] I have added documentation as necessary

Fixed my issue. Thank you @mdbrooks48


example运行不成功啊

Automatically signing iOS for device deployment using specified development team in Xcode project: VP3AM7E5ZL
Running pod install...
Starting Xcode build...
Xcode build done.                                            3.3s
Failed to build iOS app
Error output from Xcode build:
↳
    2018-11-23 13:43:10.667 xcodebuild[47959:716695] +dataWithFirstBytes:1024 ofFile:"/Users/liuyuxiao/Documents/hybrid_stack_manager-master/ios/LICENSE" failed, errno = 2
    ** BUILD FAILED **

Xcode's output:
↳
    === BUILD TARGET path_provider OF PROJECT Pods WITH CONFIGURATION Debug ===
    In file included from /Users/liuyuxiao/Documents/hybrid_stack_manager-master/ios/Classes/FlutterViewWrapperController.m:9:
    /Users/liuyuxiao/Documents/hybrid_stack_manager-master/ios/Classes/FlutterViewWrapperController.h:9:9: fatal error: 'Flutter/Flutter.h' file not found
    #import <Flutter/Flutter.h>
            ^~~~~~~~~~~~~~~~~~~
    1 error generated.

Could not build the precompiled application for the device.

It appears that your application still contains the default signing identifier.
Try replacing 'com.example' with your signing id in Xcode:
  open ios/Runner.xcworkspace

Error launching application on iPhone.

android 也运行不成功
Compiler message:
file:///E:/flutterproject/hybrid_stack_manager-master2/hybrid_stack_manager-master/lib/router.dart:69:49: Error: The getter 'history' isn't defined for the class '#lib1::NavigatorState'.
Try correcting the name to the name of an existing getter, or defining a getter or field named 'history'.
        XMaterialPageRoute pageRoute = navState.history.firstWhere((Route route){
                                                ^^^^^^^
file:///E:/flutterproject/hybrid_stack_manager-master2/hybrid_stack_manager-master/lib/router.dart:86:48: Error: The getter 'history' isn't defined for the class '#lib1::NavigatorState'.
Try correcting the name to the name of an existing getter, or defining a getter or field named 'history'.
    List<Route<dynamic>> navHistory = navState.history;
                                               ^^^^^^^
file:///E:/flutterproject/hybrid_stack_manager-master2/hybrid_stack_manager-master/lib/router.dart:96:48: Error: The getter 'history' isn't defined for the class '#lib1::NavigatorState'.
Try correcting the name to the name of an existing getter, or defining a getter or field named 'history'.
    List<Route<dynamic>> navHistory = navState.history;
                                               ^^^^^^^
file:///E:/flutterproject/hybrid_stack_manager-master2/hybrid_stack_manager-master/lib/router.dart:111:48: Error: The getter 'history' isn't defined for the class '#lib1::NavigatorState'.
Try correcting the name to the name of an existing getter, or defining a getter or field named 'history'.
    List<Route<dynamic>> navHistory = navState.history;
                                               ^^^^^^^
file:///E:/flutterproject/hybrid_stack_manager-master2/hybrid_stack_manager-master/lib/router.dart:146:48: Error: The getter 'history' isn't defined for the class '#lib1::NavigatorState'.
Try correcting the name to the name of an existing getter, or defining a getter or field named 'history'.
    List<Route<dynamic>> navHistory = navState.history;
                                               ^^^^^^^
Compiler failed on E:\flutterproject\hybrid_stack_manager-master2\hybrid_stack_manager-master\example\lib\main.dart
Finished with error: Gradle task assembleDebug failed with exit code 1

我也有同样的问题。

iOS 这需要把 引用到 这个地方#import <Flutter/Flutter.h>，修改成    #import <Flutter/Flutter/Flutter.h>。

16:35:04.746 23 info flutter.tools
/hybrid_stack_manager/lib/router.dart:146:48: Error: The getter 'history' isn't defined for the class '#lib1::NavigatorState'.
16:35:04.746 24 info flutter.tools Try correcting the name to the name of an existing getter, or defining a getter or field named 'history'.
这是什么情况呢？？

好像是版本升级把变量私有化了，希望官方抓紧时间更新一下！

看下README.md

后续我会看下是否能够不用改框架就满足需求。

@moyaying

哦，是这样。我没往改框架代码这一方向想。。。  谢谢你，也更期待优雅解决。

在 2018年12月14日，上午9:53，KyleWong<EMAIL_ADDRESS>写道：
看下README.md
https://user-images.githubusercontent.com/817851/49978563-1ce0df80-ff86-11e8-98d4-a745182026ee.png
后续我会看下是否能够不用改框架就满足需求。
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/alibaba-flutter/hybrid_stack_manager/issues/1#issuecomment-447185876, or mute the thread https://github.com/notifications/unsubscribe-auth/AF0YVPTgrm8pK9rj83Wu-73xqlSiACZRks5u4wSwgaJpZM4YwI4B.