fix: StandardCircuitBuilder create_logic_constraint and uint logic_operator

Fixes create_logic_constraint in StandardCircuitBuilder.  Previously it didn't constrain the quads to be computed from the input bits.
Fixes logic_operator in standard uint. Previously it didn't constrain the input values to be equal the new values after the operation.

Benchmark results
No metrics with a significant change found.

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit e58eda80 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,700
180,580
720,100

l1_rollup_calldata_gas
222,844
871,312
3,464,500

l1_rollup_execution_gas
316,035
989,760
3,684,878

l2_block_processing_time_in_ms
1,153
4,330 (-1%)
17,204

note_successful_decrypting_time_in_ms
308
977 (-1%)
3,636 (-2%)

note_trial_decrypting_time_in_ms
18.0 (-8%)
108 (-6%)
140 (-6%)

l2_block_building_time_in_ms
17,959
70,823
283,916

l2_block_rollup_simulation_time_in_ms
13,192
52,153
209,553

l2_block_public_tx_process_time_in_ms
4,740 (+1%)
18,594
74,116

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
13,784 (-1%)
26,224

note_history_successful_decrypting_time_in_ms
2,341 (-1%)
4,587 (-1%)

note_history_trial_decrypting_time_in_ms
86.3
228

node_database_size_in_bytes
17,346,640
33,013,840

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
308
44,528
25,825

private-kernel-ordering
195
44,145
16,225

base-rollup
1,419
128,654
933

root-rollup
82.2
4,192
729

private-kernel-inner
439
71,331
25,825

public-kernel-private-input
241
32,287
25,825

public-kernel-non-first-iteration
237
32,329
25,825

merge-rollup
7.81
2,712
933

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
16,859
40,311

Transaction processing duration by data writes.

Metric
0 new commitments
1 new commitments

tx_pxe_processing_time_ms
554
1,307

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
0.522 (+4%)
576


RNEA not taking external forces into account in Python?

I'm trying out pin.rnea in Python with a list of external forces applied to each joint:

Zero external force on each joint
Except at the left/right wheel joints (robot model is Upkie) where there is a $F_z$ vertical force

Here is a minimal example with $F_z = 100$ N:
import numpy as np
import pinocchio as pin
from robot_descriptions.loaders.pinocchio import load_robot_description

robot = load_robot_description("upkie_description")
model = robot.model
theta = 1.0  # rad
q = np.array([theta, -2 * theta, 0, theta, -2 * theta, 0])
v = np.zeros(model.nv)
a = np.zeros(model.nv)

f_ext = [pin.Force.Zero() for j in range(model.njoints)]
f_ext[model.getJointId("left_wheel")].linear[2] += 100.0  # N
f_ext[model.getJointId("right_wheel")].linear[2] += 100.0  # N

tau = pin.rnea(model, robot.data, q, v, a, f_ext)
print(f"{tau=}")

The issue is that the outcome of this example is the same tau regardless of $F_z$ :thinking: You can try it with 10, 100, 1000, ...

My guess would be that RNEA has some undocumented requirements, for example maybe FK should be run beforehand?

By using:
import numpy as np
import pinocchio as pin
from robot_descriptions.loaders.pinocchio import load_robot_description

robot = load_robot_description("upkie_description")
model = robot.model
theta = 1.0  # rad
q = np.array([theta, -2 * theta, 0, theta, -2 * theta, 0])
v = np.zeros(model.nv)
a = np.zeros(model.nv)

f_ext = [pin.Force.Zero() for j in range(model.njoints)]
f_ext[model.getJointId("left_knee")].linear[:] += 100.0  # N
f_ext[model.getJointId("right_knee")].linear[:] += 100.0  # N

tau = pin.rnea(model, robot.data, q, v, a, f_ext)
print(f"{tau=}")

you will that it works well.
I guess this is due to the current configuration where normal contact forces have no impact on the joint torques

You're right, I forgot that external forces are in local joint frames, so I should have checked the orientation of those frames rather than assuming it would be vertical :sweat_smile:
As it turns out, the z-axis in the local frames is lateral (source: display_joints.py):

When correctly applying $F_z$ on the local y- rather than z-axis it works as expected :+1: Thanks!


bind & loop problem

I try to use @bind & @loop. i share the code https://codesandbox.io/s/quirky-lake-l7ed9w?file=/src/App.js
let self = {
   ...
    a: "Alberto",
    b: {
      name: "John",
      age: 26,
      email<EMAIL_ADDRESS>    },
    c: [
      {
        name: "Albert",
        age: 38,
        email<EMAIL_ADDRESS>      }
    ],
    d: [
      {
        name: "Albert",
        age: 38,
        email<EMAIL_ADDRESS>      },
      {
        name: "Alberto",
        age: 26,
        email<EMAIL_ADDRESS>      },
      {
        name: "Sylvain",
        age: 57,
        email<EMAIL_ADDRESS>      }
    ]
  };

if i use @bind  for self.a or self.d this work , but not just simple object as self.b . I try different test with self.c but same problem.
For @loop, see the code, if i use two time , the first loop appears in second loop.

You can only use the first level properties from the self.
self.b.name <= this is wrong. you need to use only properties directly in the self.

See the loop here : https://codesandbox.io/s/loop-error-7kq48j
What is the wrong code ?

Use different loops. Using the loop, each element of the array gets a DOM element assigned. A DOM Element cannot be in two different locations at the same time. So, you cannot use the same property in two loops.


[InProgress] Virtual machine extensions

Can one of the admins verify this patch?

Adding 'break' label since the getter 'List<> resources()' renamed to 'Map<> extensions'

Runtime changes detected. pull request created. CI running:


Added LastName property to workspace-scratch-def.json file

This is a quick fix to the problem reported in issue 131.

Ah, darn.


vis-academy examples do not seem to work with current version of deck.gl

Steps to reproduce:

git clone vis-academy
cd src/demos/2-scatterplot-overlay
run yarn
run yarn start
observe on localhost:3030 that the scatterplot example is visible
close web serve with cmd+C
in package.json, upgrade the following:

"deck.gl": "^6.1.0",
"luma.gl": "^6.0.1",
"react-vis": "^1.11.1"

these are latest versions
8. now run yarn and then yarn start
Expected
Demo should still work, and render scatterplot
Actual
Scatteprlot does not render.
Error from console:
draw-layers.js:257 Uncaught TypeError: Cannot assign to read only property 'viewport' of object '[object Object]'
    at Function.assign (<anonymous>)
    at drawLayerInViewport (draw-layers.js:257)
    at draw-layers.js:229
    at Array.forEach (<anonymous>)
    at drawLayersInViewport (draw-layers.js:201)
    at draw-layers.js:89
    at Array.forEach (<anonymous>)
    at drawLayers (draw-layers.js:83)
    at draw-layers.js:132
    at withParameters (context-state.js:182)

@octopicorn Please run yarn in the vis-acadmy folder first and then follow the steps you described above.

Your issue might be due to a breaking change introduced in the major release. Have you tried going through the changes before upgrading?

This doesn't seem all too surprising. For me the interesting question is if someone is going to update the vis-academy source code any time soon? Best!

Same issue here. Using React 6.2.3
Got error:
Uncaught TypeError: Cannot assign to read only property 'viewport' of object '[object Object]'     at Function.assign (<anonymous>)     at drawLayerInViewport (draw-layers.js:262)     at draw-layers.js:234     at Array.forEach (<anonymous>)     at drawLayersInViewport (draw-layers.js:206)     at draw-layers.js:94     at Array.forEach (<anonymous>)     at drawLayers (draw-layers.js:88)     at draw-layers.js:137     at withParameters (context-state.js:202)     at drawPickingBuffer (draw-layers.js:131)     at drawAndSamplePickingBuffer (pick-layers.js:218)     at pickObject (pick-layers.js:64)     at LayerManager.pickObject (layer-manager.js:374)     at Deck._pickAndCallback (deck.js:504)     at Deck._onPointerMove (deck.js:733)     at Array.<anonymous> (event-manager.js:360)     at Manager.emit (hammer.js:2525)     at EventManager._onOtherEvent (event-manager.js:412)     at MoveInput.handleEvent (move-input.js:139)

`ERROR in ./src/main.js
Module build failed: SyntaxError: Unexpected token (13:2)
11 |
12 | const Root = () => (

13 |
|   ^
14 |
15 |
16 |     `
Wondering if anyone is every gonna fix the examples here ?

@varunpant The examples run on my end. What steps did you take?


Dockerize the application

@kumaranvram Can you simplify the docker config - this service is being used outside of our projects as well, so would want to expose the simplest possible docker file. The Procfile here https://github.com/sudhirj/notecase/blob/master/Procfile already has the setup command, let's replicate that.
Can remove the config/containers directory - whoever is using it can set up nginx if they want. Just the Dockerfile with the startup command updated to what's in the Procfile makes sense.
Can also ignore .env - this is personal preference, can be set by users.
For the database.yml, let's update it with the Rails 5 template, which is
# PostgreSQL. Versions 9.1 and up are supported.
#
# Install the pg driver:
#   gem install pg
# On OS X with Homebrew:
#   gem install pg -- --with-pg-config=/usr/local/bin/pg_config
# On OS X with MacPorts:
#   gem install pg -- --with-pg-config=/opt/local/lib/postgresql84/bin/pg_config
# On Windows:
#   gem install pg
#       Choose the win32 build.
#       Install PostgreSQL and put its /bin directory on your path.
#
# Configure Using Gemfile
# gem 'pg'
#
default: &default
  adapter: postgresql
  encoding: unicode
  # For details on connection pooling, see rails configuration guide
  # http://guides.rubyonrails.org/configuring.html#database-pooling
  pool: <%= ENV.fetch("RAILS_MAX_THREADS") { 5 } %>
  host: localhost

development:
  <<: *default
  database: slydes-admin_development

  # The specified database role being used to connect to postgres.
  # To create additional roles in postgres see `$ createuser --help`.
  # When left blank, postgres will use the default role. This is
  # the same name as the operating system user that initialized the database.
  #username: slydes-admin

  # The password associated with the postgres role (username).
  #password:

  # Connect on a TCP socket. Omitted by default since the client uses a
  # domain socket that doesn't need configuration. Windows does not have
  # domain sockets, so uncomment these lines.
  #host: localhost

  # The TCP port the server listens on. Defaults to 5432.
  # If your server runs on a different port number, change accordingly.
  #port: 5432

  # Schema search path. The server defaults to $user,public
  #schema_search_path: myapp,sharedapp,public

  # Minimum log levels, in increasing order:
  #   debug5, debug4, debug3, debug2, debug1,
  #   log, notice, warning, error, fatal, and panic
  # Defaults to warning.
  #min_messages: notice

# Warning: The database defined as "test" will be erased and
# re-generated from your development database when you run "rake".
# Do not set this db to the same as development or production.
test:
  <<: *default
  database: slydes-admin_test

# As with config/secrets.yml, you never want to store sensitive information,
# like your database password, in your source code. If your source code is
# ever seen by anyone, they now have access to your database.
#
# Instead, provide the password as a unix environment variable when you boot
# the app. Read http://guides.rubyonrails.org/configuring.html#configuring-a-database
# for a full rundown on how to provide these environment variables in a
# production deployment.
#
# On Heroku and other platform providers, you may have a full connection URL
# available as an environment variable. For example:
#
#   DATABASE_URL="postgres://myuser:mypass@localhost/somedatabase"
#
# You can use this database configuration with:
#
production:
  url: <%= ENV['DATABASE_URL'] %>
  pool: <%= ENV.fetch("RAILS_MAX_THREADS") { 5 } %>
# #
# production:
#   <<: *default
#   database: slydes-admin_production
#   username: slydes-admin
#   password: <%= ENV['SLYDES-ADMIN_DATABASE_PASSWORD'] %>


Bug 2119932: Address missing VR for a PVC during VRG deletion

There could be cases where a PVC finalizer is present
but, during VRG deletion, post deletion of the VR the update
to remove the finalizer on the PVC failed.
This would on the next reconcile recreate the VR, which stalls
deletion processing as the VR cannot add finalizers to the PVC
and hence cause the deletion to loop forever and fail.
This commit adds checks to ensure VR is missing and if so
complete the deletion process for the PVC, and if not perform
VR deletion orchestration as usual.
Signed-off-by: Shyamsundar Ranganathan<EMAIL_ADDRESS>(cherry picked from commit 01d4e722c466d6a6949f5dd021a20200c648f6d9)

/bugzilla refresh


Start investigating methods for data validation and serialization

In the first step to modernize and cleanup the API backend processing, we are going to investigate the usage of modern libraries for API schema definitions and data serialization. There are two main libraries that should probably be investigated: Marshmallow and Pydantic. Whichever one you decide looks better to implement I'm fine with as both are fast, powerful, highly recommended, and have good community support.
A basic structure has been started from the api-rewrite branch. You only have to worry about files in the api/biomarker directory. Initial thoughts are the general workflow will be as follows (although if you find a better or simpler way to do things by all means):

The basic structure of what is supported for each endpoint is defined in the api/biomarker/config.json file here. This file is read in and stored in the Flask app instance in the __init__.py entry point.
The data models will be built dynamically in the api/biomarker/data_models.py file based on the configuration specifications from the JSON file. The build process will have a generalized function that can automatically walk through the JSON and build a data model. One challenge is figuring out how to handle non-terminal fields (i.e. array or object fields). For example, paginated tables is currently listed as a string type when it really should be list (or array) of objects (or dict). JSON uses array and object as their terms, doesn't matter if you choose the Python or JSON terminology, just need to choose one and stay consistent. Maybe have some sort of nested schema for the config file such as below. This is just an idea, you can find what seems appropriate. You could also decide that the overhead in both maintaining and dynamically building the schemas is unnecessary and we should scrap the JSON approach all together and just build them manually, either approach is valid.

"paginated_tables": {
    "type": "array",
    "items": {
        "table_id": {
            "type": "string",
            "max_len": 200,
            "required": true
        },
        "sort": {
            "type": "string",
            "max_len": 10,
            "required": false
        }
    }
}

Eventually when we get around to actually refactoring the endpoints themselves these data models will be used for automatically serializing and validating the user passed JSON payload bodies.

Once all the above is figured out, the next challenge will be figuring out exactly what each of the GlyGen biomarker endpoints can take as payload bodies to start designing the data models. The endpoints can be found at the GlyGen tst swagger page (you have to be on the VPN to see this page). Places to look to try and get an idea of what can be accepted for an endpoint:
Note: All references to the GlyGen code make sure you are looking at the 2.5 branch.

The config.json file at glygen/conf/config.json. It will probably be much more helpful to stick that in some sort of JSON viewer, the one I use all the time is this one.
The biomarker.py file in the GlyGen API has some data models defined starting on line line 22. However, as we saw with the biomarker/detail/ endpoint these aren't super accurate and actually conflict with the JSON config file at some places.

For the biomarker/detail endpoint here are a couple paginated sections objects that we know works for examples:
{
    "paginated_tables": [
        {
            "table_id": "citation",
            "sort": "authors",
            "order": "asc"
        }
    ]
}

{
    "paginated_tables": [
        {
            "table_id": "citation",
            "limit": 2
        }
    ]
}

ChatGPT gave me these are examples:
{
    "biomarker_id": "BM12345",
    "paginated_tables": [
        {
            "table_id": "glycosylation_reported",
            "sort": "site_category",
            "order": "asc",
            "offset": 1,
            "limit": 10
        },
        {
            "table_id": "expression_tissue",
            "sort": "category",
            "order": "desc",
            "offset": 1,
            "limit": 5
        },
        {
            "table_id": "snv_disease",
            "sort": "gene",
            "order": "asc",
            "offset": 1,
            "limit": 20
        }
    ]
}

@MiguelMazumder And as you make decisions on some of these points, find useful resources, or have questions that come up you can just comment on this ticket and I can respond if its a question. Important to have a thought trail in writing so we can keep track of our decisions, the more documentation, evidence, and resources the better.

For Marshmallow looks like they have a from_dict() function. That is probably what we're looking for. Was just doing a preliminary look at their API Reference docs.

Finished these


Express v4 -> v5 Migration

Copied from my comment in the Release 5.0 PR:
Maybe I missed a v5 planning document somewhere - I'm looking for a few details to help determine how and when to migrate our projects:

When is it planned that<EMAIL_ADDRESS>be published to the npm latest tag?
Is there a plan to publish an Express v4 -> v5 Migration Guide? (or a visible list of breaking changes with short suggestions about what to do about them) The closest thing that I can see is this small breaking note in History.md
What's the current plan for the TypeScript types in DefinitelyTyped @types/express and @types/express-serve-static-core? Does anything need to be changed here? Specifically, things like https://github.com/DefinitelyTyped/DefinitelyTyped/pull/69660#issuecomment-2179358937 cc DT maintainers @borisyankov @samijaber @NatoBoram @bombillazo @AndrewLeedham @gabritto @sandersn

Thanks for creating this issue. I added a reference in the 5.x backlog https://github.com/expressjs/discussions/issues/266 :+1:

When is it planned that<EMAIL_ADDRESS>be published to the npm latest tag?

We have not set a date yet. Largely it depends on how quickly we can get the docs and the rest of the ecosystem dependencies (like DT for example) updated. We also have an LTS proposal we are working on to better schedule this stuff in the future.

Is there a plan to publish an Express v4 -> v5 Migration Guide?

Yes, this is part of the docs we need to write.

What's the current plan for the TypeScript types in DefinitelyTyped

We would love to chat with the DT folks to get better at this in the future. We know it has lacked attention for too long, but we never had a lot of folks signing up and showing up to help us with this. If that is changing now I would love it, but we need more than just drive by help, we need folks who are here to help us get all the ~30 some packages we maintain all doing this consistently and well.

There are existing issues for all of this, I will edit this comment when I get a chance to add them, but wanted to get this written sooner rather than later.

We would love to chat with the DT folks to get better at this in the future. We know it has lacked attention for too long, but we never had a lot of folks signing up and showing up to help us with this. If that is changing now I would love it, but we need more than just drive by help, we need folks who are here to help us get all the ~30 some packages we maintain all doing this consistently and well.

Calling @borisyankov @CMUH @puneetar @dfrankland

I might be able to help with type work

@aredridel That would be awesome! I have been super busy keeping up with the chatter around the release while trying to spare some time for my paying job 😆, so I have not had a chance to pull in all the links to previous conversations around this, but most are in https://github.com/expressjs/discussions

The author of the original PR to DefinitelyTyped is sick.
I started began my own approach here, but it's promising to be massive: dozens of packages have internal references to express' types, declaring * compatibility, having discrepancies in tests, such as returning something where void was expected — that did actually work with void, but no longer works with void | Promise<void> and that is the issue.

OK. I fixed all issues in my PR to types of Express 5 — https://github.com/DefinitelyTyped/DefinitelyTyped/pull/70563
I welcome advises and suggestions on how to make it better and release faster.

@RobinTail perhaps it would be better to coordinate the efforts towards upstreaming the types here in the Express repo? And then subsequently to deprecate the @types/ package.

Perhaps, @jonkoops .
Did you already start doing that?
My job is finished meanwhile — everything is ready and needs only an approval to release the types for Express 5 via the established workflow.

I have not, but I would not mind taking this on. I see also @aredridel volunteered for this, so perhaps they already started working on it. Let me know, I could set up a PR where we can collaborate.

I have not
I would
I could

You know, in my system of values getting things done is more important and cherished rather than

ruminating for 10 years

on how to arrange the seats

for a potential opportunity

to begin negotiations

on a proposed cooperation

in matters of shared responsibility

towards finding a suggested solution

to the elementary problem.

So, I have done it.
If you're having your own solution in mind, which perhaps you believe could be better than mine — don't hesitate and do it @jonkoops

I'd say get the PR merged for the the @types/ package, then we can simply copy them over later :man_shrugging:

Exactly, @jonkoops !
Express always has been pure-JS repo, having no types. So that adding them is a brand new feature, requiring either:

rewriting the code in Typescript with a consequential compilation;
or keeping a manually written declaration in sync with that code.

Anyway, it's a new workflow to establish — it will take time and moreover, as a new feature, it will require a new version — 5.1.0 for example, while already now we have the opportunity to adjust the existing @types/express for supporting the already released version 5.0.0, enabling its availability for the community.

or keeping a manually written declaration in sync with that code.

This is already the case now, with the added complexity (for end users) of it being a separate package. The only downside of moving them in here would be that maintenance needs to happen in this repo, which is totally fine as long as it's taken into account going forward.

This is it

This part wasn't changed, @walmartwarlord . If that augmentation did work for v4, it should work for v5 as well.

Migration guide to Express 5 https://expressjs.com/en/guide/migrating-5.html

Irrelevant to some extend but any news on when v5 will become the default version in npm?
I read this issue: https://github.com/expressjs/express/issues/4920
But the last comment there was stating 2023 and now we are entering 2025. I am not sure if this is normal.

We have had a flurry of events and distractions among the TC members, primarily we have a few deadlines we have been focused on (the STF grant proposal and planning) which will be wrapped up soon. Luckily a lot of folks have been doing great work on the docs and other things we needed to get done, and so as soon as we have time to get back to this it will be the top priority. My guess is that we won't likely push the button on it until the new year to save folks from a lot of noise over the holidays. That said, we have not discussed it directly yet so I would love to hear others opinions.

@wesleytodd apologies for bugging you here, but I was wondering if you could give me a ballpark estimate for when v5 might become the default version. The reason I'm asking is that the NestJS v11 release is pretty much ready to go. However, I'd like to make Express v5 the default as part of the next major release.
If the Express team is thinking that a Q1 release is feasible, I'd consider postponing and waiting. On the other hand, if it's likely to take a few more months, I'll move forward with what we have now and include it in a future release instead.
Thanks in advance!

No problem @kamilmysliwiec, happy to hear from you and your plans to udpate! We just discussed on the TC call today and we think we will be targeting early Jan for promoting to latest. That said, you should absolutely have NestJS depend on ^5.0.0 right now, it is stable and we have had increasing adoption and have all good signals, the only thing delaying us was the docs and lack of community prep after a 10 year stall out, nothing unstable about depending on the release now. The only thing that will change is what users get when they type npm i express, but things like @nestjs/platform-express depending on "express": "^5.0.0" will work just fine today and in the future.

Excellent, thank you @wesleytodd!


Unable to create a new nginx helper that uses rewrite_by_lua_block

Describe the bug
IPv6 nginx helper contains rewrite_by_lua_block which means it can not be re-used in a different helper as it triggers a duplicate error.
To Reproduce
Steps to reproduce the behavior:

Within the nginx container
echo "rewrite_by_lua_block {}" > /usr/local/openresty/nginx/conf/helpers/040_test.conf
nginx -s reload
See error nginx: [emerg] is duplicate in /etc/nginx/helpers/040_test.conf:1

Expected behavior
I should be able to add code to rewrite_by_lua_block within the content that the helpers are being included in app.conf without having to add it to a file name specifically for IPv6.  Given that duplicate blocks are not allowed, suggest that this file is renamed to 030_rewrite_by_lua_block.conf indicating that I can use this file for other code that needs to be executed in this block.
Screenshots
N/A
Additional context
Happy to put up a MR if the above solution is acceptable.

Hi @Feng-Shui this sounds like a good idea - If you can send a MR our way that would be fantastic!

Ok, will do 👍

I'm having a similar problem.. I included two conf file in app.conf file.
And these 2 conf file are having rewrite_by_lua_block and on starting the nginx server, i got the following error-
2021/12/13 19:02:41 [emerg] 29499#29499: is duplicate in /etc/nginx/redirection_url.conf:3
2021/12/13 19:03:46 [emerg] 30423#30423: is duplicate in /etc/nginx/redirection_url.conf:3
2021/12/13 19:10:35 [emerg] 4237#4237: is duplicate in /etc/nginx/redirection_url.conf:3
@Feng-Shui can you help me in this?

@poojas16 if you add another rewrite_by_lua_block anywhere in your custom config files you will get this error.  Any code that needs to be executed with rewrite_by_lua_block must be to be added to the following file:
https://github.com/uselagoon/lagoon-images/blob/main/images/nginx/helpers/030_rewrite_by_lua_block.conf


Fresh clone on Ubuntu shows a bunch of CRLF -> LF changes

...which makes doing basic stuff like checkout out branches impossible. Reproduced on multiple systems with really barebones git configs... what happened?
If I commit the line ending changes to a throwaway branch and then switch to 2.0 I also get problems with a couple files in there.

Herm, seems like no longer able to reproduce outside my VM.
Gonna close this for now.

no no no there's still CRLF files in here


[Issue-5581] Change into transitive dependency(netty.transport.native.unix.common)

Motivation:
Explain why you're making this change and what problem you're trying to solve.
Modifications:
From issue #5581, the class is exposed in the public API.

Class io.netty.channel.unix.DomainSocketAddress is used as public method parameter in core/src/main/java/com/linecorp/armeria/common/util/DomainSocketAddress.java

Result:

Closes #5581. (If this resolves the issue.)
User can use [core] com.linecorp.armeria.common.util.DomainSocketAddress#asNettyAddress without adding dependency

👍 👍 👍 👍


Added subtests to extend UCMP support with Link Bandwidth Community

Pull Request Functional Test Report for #3530 / 0d90e6fa20bb242cec1a3cef47194cce35223871
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

RT-1.52: BGP multipath UCMP support with Link Bandwidth Community
7816a8b5
Log

Cisco 8000E

RT-1.52: BGP multipath UCMP support with Link Bandwidth Community
02c3c706
Log

Cisco XRd

RT-1.52: BGP multipath UCMP support with Link Bandwidth Community
4d33aee1
Log

Juniper ncPTX

RT-1.52: BGP multipath UCMP support with Link Bandwidth Community
8bcc61cd
Log

Nokia SR Linux

RT-1.52: BGP multipath UCMP support with Link Bandwidth Community
e816cdfc
Log

Openconfig Lemming

RT-1.52: BGP multipath UCMP support with Link Bandwidth Community
9fc53137
Log

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

RT-1.52: BGP multipath UCMP support with Link Bandwidth Community

Cisco 8808

RT-1.52: BGP multipath UCMP support with Link Bandwidth Community

Juniper PTX10008

RT-1.52: BGP multipath UCMP support with Link Bandwidth Community

Nokia 7250 IXR-10e

RT-1.52: BGP multipath UCMP support with Link Bandwidth Community

Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

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

Totals

Change from base Build<PHONE_NUMBER>0:
0.0%

Covered Lines:
1983

Relevant Lines:
3588

💛  - Coveralls


Update Composer dependencies (2023-02-10-00-10)

Visual regression test passed!
View the visual regression test report


Output Transform Code

Add ability to define transformation code to run on the output of a block to avoid having too many glue blocks

One thought is to look at how AWS Step functions does this. They have an InputPath and OutputPath that allows transforms both on input and output.

Real question is should we use such JSONPath like syntax or free code? Very similar to your exploration on the model block...


Add optout_type to hook payload

This should inform applications what type of optout has been created for the identity.
Previously the payload sent in the hook when an optout was created only contained the details for the Identity before they opted out. There was no way to determine what sort of optout they had sent.

@hodgestar @imsickofmaps Small review please

👍

:+1:


[3.0.0-beta1] tests fail for debug build of solaris64-sparcv9-cc

Testing on a Solaris 11.3 sun4v sparc machine fails for the --debug build. On the same machine the tests of the --release build are all ok, however. Compiler is a ...
$ cc -V
cc: Studio 12.5 Sun C 5.14 SunOS_sparc 2016/05/31

Configure-dump.txt
Tests are ok until ...
05-test_pbe.t ......................
    # ERROR: (bool) 'PKCS5_PBE_keyivgen(ctx, pbe_password, strlen(pbe_password), algor->parameter, cipher, md, 1) == true' failed @ test/pbetest.c:76
    # false
    # 00000001:error:0308010C:digital envelope routines:(unknown function):unsupported:crypto/evp/evp_fetch.c:346:Global default library context, Algorithm (PBKDF1 : 0), Properties (<null>)
    # OPENSSL_TEST_RAND_ORDER=1625209038
    not ok 1 - test_pkcs5_pbe_rc4_md5
# ------------------------------------------------------------------------------
    # ERROR: (bool) 'PKCS5_PBE_keyivgen(ctx, pbe_password, strlen(pbe_password), algor->parameter, cipher, md, 1) == true' failed @ test/pbetest.c:76
    # false
    # 00000001:error:0308010C:digital envelope routines:(unknown function):unsupported:crypto/evp/evp_fetch.c:346:Global default library context, Algorithm (PBKDF1 : 0), Properties (<null>)
    # OPENSSL_TEST_RAND_ORDER=1625209038
    not ok 2 - test_pkcs5_pbe_des_sha1
# ------------------------------------------------------------------------------
../../util/wrap.pl ../../test/pbetest => 1
not ok 1 - Running PBE test
05-test_pbe.t ...................... 1/? ---------------------------------------
#   Failed test 'Running PBE test'
#   at test/recipes/05-test_pbe.t line 27.
05-test_pbe.t ...................... Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/1 subtests

... then again ok, until ...
20-test_enc.t ...................... 1/?
aes-128-cbc: unable to load provider legacy
Hint: use -provider-path option or OPENSSL_MODULES environment variable.
00000001:error:12800067:DSO support routines:(unknown function):could not load the shared library:crypto/dso/dso_dlfcn.c:118:filename(../../providers/legacy.so): ld.so.1: openssl: fatal: relocation error: file ../../providers/legacy.so: symbol bn_expand2: referenced symbol not found
00000001:error:12800067:DSO support routines:(unknown function):could not load the shared library:crypto/dso/dso_lib.c:162:
00000001:error:078C0105:common libcrypto routines:(unknown function):init fail:crypto/provider_core.c:658:name=legacy
../../util/wrap.pl ../../apps/openssl aes-128-cbc -bufsize 113 -e -k test -provider-path ../../providers -provider default -provider legacy -in ./p -out ./p.aes-128-cbc.cipher => 1
not ok 3 - aes-128-cbc
# ------------------------------------------------------------------------------
#   Failed test 'aes-128-cbc'
#   at test/recipes/20-test_enc.t line 70.

... all tests are failing now with similar messages, until ...
#   Failed test 'sm4-ofb'
#   at test/recipes/20-test_enc.t line 70.
sm4-ofb: unable to load provider legacy
Hint: use -provider-path option or OPENSSL_MODULES environment variable.
00000001:error:12800067:DSO support routines:(unknown function):could not load the shared library:crypto/dso/dso_dlfcn.c:118:filename(../../providers/legacy.so): ld.so.1: openssl: fatal: relocation error: file ../../providers/legacy.so: symbol bn_expand2: referenced symbol not found
00000001:error:12800067:DSO support routines:(unknown function):could not load the shared library:crypto/dso/dso_lib.c:162:
00000001:error:078C0105:common libcrypto routines:(unknown function):init fail:crypto/provider_core.c:658:name=legacy
../../util/wrap.pl ../../apps/openssl sm4-ofb -bufsize 113 -a -e -k test -provider-path ../../providers -provider default -provider legacy -in ./p -out ./p.sm4-ofb.cipher => 1
not ok 96 - sm4-ofb base64
# ------------------------------------------------------------------------------
#   Failed test 'sm4-ofb base64'
#   at test/recipes/20-test_enc.t line 70.
20-test_enc.t ...................... Dubious, test returned 94 (wstat 24064, 0x5e00)
Failed 94/96 subtests
20-test_enc_more.t ................. 1/?
enc: unable to load provider legacy
Hint: use -provider-path option or OPENSSL_MODULES environment variable.
00000001:error:12800067:DSO support routines:(unknown function):could not load the shared library:crypto/dso/dso_dlfcn.c:118:filename(../../providers/legacy.so): ld.so.1: openssl: fatal: relocation error: file ../../providers/legacy.so: symbol bn_expand2: referenced symbol not found
00000001:error:12800067:DSO support routines:(unknown function):could not load the shared library:crypto/dso/dso_lib.c:162:
00000001:error:078C0105:common libcrypto routines:(unknown function):init fail:crypto/provider_core.c:658:name=legacy
../../util/wrap.pl ../../apps/openssl enc -aes-128-cbc -k test -provider-path ../../providers -provider default -provider legacy -e -in ./testdatafile -out ./testdatafile.aes-128-cbc.cipher => 1
not ok 3 - aes-128-cbc
# ------------------------------------------------------------------------------

... all following tests fail now with similar messages, until ...
#   Failed test 'sm4-ecb'
#   at test/recipes/20-test_enc_more.t line 60.
enc: unable to load provider legacy
Hint: use -provider-path option or OPENSSL_MODULES environment variable.
00000001:error:12800067:DSO support routines:(unknown function):could not load the shared library:crypto/dso/dso_dlfcn.c:118:filename(../../providers/legacy.so): ld.so.1: openssl: fatal: relocation error: file ../../providers/legacy.so: symbol bn_expand2: referenced symbol not found
00000001:error:12800067:DSO support routines:(unknown function):could not load the shared library:crypto/dso/dso_lib.c:162:
00000001:error:078C0105:common libcrypto routines:(unknown function):init fail:crypto/provider_core.c:658:name=legacy
../../util/wrap.pl ../../apps/openssl enc -sm4-ofb -k test -provider-path ../../providers -provider default -provider legacy -e -in ./testdatafile -out ./testdatafile.sm4-ofb.cipher => 1
not ok 137 - sm4-ofb
# ------------------------------------------------------------------------------
#   Failed test 'sm4-ofb'
#   at test/recipes/20-test_enc_more.t line 60.
20-test_enc_more.t ................. Dubious, test returned 135 (wstat 34560, 0x8700)
Failed 135/137 subtests
20-test_kdf.t ...................... ok

... there are quite a number of further tests failing, all of them with the complain that legacy.so cannot be loaded because of the missing symbol bn_expand2.
The summary:
Test Summary Report
-------------------
05-test_pbe.t                    (Wstat: 256 Tests: 1 Failed: 1)
  Failed test:  1
  Non-zero exit status: 1
20-test_enc.t                    (Wstat: 24064 Tests: 96 Failed: 94)
  Failed tests:  3-96
  Non-zero exit status: 94
20-test_enc_more.t               (Wstat: 34560 Tests: 137 Failed: 135)
  Failed tests:  3-137
  Non-zero exit status: 135
30-test_evp.t                    (Wstat: 16384 Tests: 67 Failed: 64)
  Failed tests:  1-64
  Non-zero exit status: 64
80-test_cms.t                    (Wstat: 256 Tests: 12 Failed: 1)
  Failed test:  4
  Non-zero exit status: 1
80-test_pkcs12.t                 (Wstat: 256 Tests: 7 Failed: 1)
  Failed test:  7
  Non-zero exit status: 1
90-test_store.t                  (Wstat: 512 Tests: 8 Failed: 0)
  Non-zero exit status: 2
  Parse errors: Bad plan.  You planned 416 tests but ran 8.
Files=239, Tests=2812, 1078 wallclock secs (11.94 usr  2.71 sys + 782.03 cusr 144.82 csys = 941.50 CPU)
Result: FAIL

Let me know, if you need further information.

You are correct that the underlying problem here is the inability to load the legacy provider.  I suspect the differences are likely to be environmental.  Is there a difference in how you build the debug version?  Do you install either before testing?

Potential fix in #16019, please test.

Sorry, couldn't make it in time. Have to wait for a window next week.


Fix the validate script: missing uuid module

In this patch we grab all dependencies from ./test/e2e/...
golangci-lint fails otherwise because it's unable to find
the k8s uuid module which we don't vendor.
An alternative would be to start vendoring such module.
Signed-off-by: Miguel Angel Ajo<EMAIL_ADDRESS>

@skitt  ^ what do you think?
An alternative could be just vendoring the missing module (used in the e2e tests)

I think we should vendor the module instead. Since validate is run before the build, any dependencies installed here could be used in the build too, and even if they’re only used in the tests, we won’t know what version we’re getting...

Yeah, I checked we don't do that now (validate before build) but it doesn't mean we would not do it in the future because somebody is not aware of that dept. So let's vendor that one then, I'll fix the patch to vendor it.

Yeah, I checked we don't do that now (validate before build) but it doesn't mean we would not do it in the future because somebody is not aware of that dept. So let's vendor that one then, I'll fix the patch to vendor it.

#88 :-D (not vendoring, running validate before the build).

Yeah, I checked we don't do that now (validate before build) but it doesn't mean we would not do it in the future because somebody is not aware of that dept. So let's vendor that one then, I'll fix the patch to vendor it.

#88 :-D (not vendoring, running validate before the build).

Hmmm, and how does that work?, /me tries :-)

Fixed in the go mod switch


Do we require qmPVC volume to be in PVC?

I'm using IBM MQ with multi-instance for a quite while. I recently notice that /mnt/mqm gets out of space. This is because web service(I think) is generating core dumps in that volume.
I know why it's generating and this is due to my environmental setup I can't avoid.
The question is can we use emptyDir for /mnt/mqm volume instead of a PVC? It looks like stuff under the /mnt/mqm is generate every time when it spins up. There are some error logs stored but emptyDir won't be cleaned up when a container gets restarted with some errors so the error logs should be kept.
What's your thoughts on this or any concerns?

I dont think its permissible as emptyDir is used only for readOnlyRootFileSystem as seen in this template yaml - https://github.com/ibm-messaging/mq-helm/blob/main/charts/ibm-mq/templates/stateful-set.yaml. However I would request @callumpjackson's opinion about this.

@vgavinash Thanks for your response.
I modify the helm chart to have the volume with emptyDir. I have tested it with creating some messages in queues. It looks fine with emptyDir.

Hi - we have not tested this ourselves. I do wonder if it would be a better approach to mount a new file system particularly for the web console logs, instead of generically /mnt/mqm. For instance, if you experienced an issue with the queue manager and needed to submit the logs to support they wouldn't be available in the above approach.

@callumpjackson That is actually a good idea. That will solve our problems as well. I will keep the logs in PV and others in emptyDir or something non-PV.
Thanks.

@chcshinimai if your prob is resolved request you to close this issue.

Assuming resolved and closing.


Component Mounting Twice After React Native 0.39 upgrade with IOS 9.2

Version
Tell us which versions you are using:

react-native-router-flux v3.37.0
react-native v0.39

Expected behaviour
Component Should mount only once
Actual behaviour
Component Mounting twice with IOS 9.2, Works well with IOS 10.2
Steps to reproduce

Basic App with type="reset" Scene
IOS 9.2 emulator
Add logger to ComponentDidMount, logs twice
Works fine with IOS 10.2

Also works well with IOS 10.0

ping @aksonov

Sorry, i'm not using this component at this moment (had to switch to native iOS controllers with react-native-router-native for truly native animations), any PR would be welcome.

Same issue here. Anyone?


validation.py from autoinvent-example has useful classes

validation.py in autoinvent-example has useful classes ('UniqueConstraint' and 'MutationValidation') that should be shared with anyone building an autoinvent backend.

Something is off with the original Auto Invent Example validation.py.
Call signature for UniqueConstraint's call:
def call(self, model, input_dict, instance, **kwargs):
MutationValidation's __validate is not passing in argument for "instance"
def _validate(self, input_dict, **kwargs):
for valid in self.valid_list:
e = valid(self.model, input_dict, **kwargs)

I see what's going on here.
valid goes to self, because it's calling itself. Instead of valid.(args*)
P.S. It worked in the original autoinvent-example, I did some minor refactoring to make it stand alone.

I checked autoinvent-example and it looks good and works as expected.
I ran into issues because I did not do kwargs["instance"] = . If "Instance" is defined in kwargs, then it can be found and used as an argument to _ _call _ _.
Everything looks good.


Doubtful section "DNS configuration"

Sections DNS configuration seems to mix up dns entries. By following this section you will endup in a entry like <AppName>.<ASE name>.scm.appserviceenvironment.net but should be <AppName>.scm.<ASE name>.appserviceenvironment.net like mentioned in section "Publish with an ILB ASE".
Therefore the actions to configure dns would be more like:

create a zone for <ASE name>.appserviceenvironment.net
create an A record in that zone that points * to the ILB IP address
create a zone for scm.<ASE name>.appserviceenvironment.net
create an A record in the scm zone that points * to the ILB IP address

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 2b1afa49-a5b4-92db-68e8-3feeeff52eb3
Version Independent ID: c9191775-e49c-1462-e525-a5ce549ae743
Content: Create internal load balancer with App Service Environment - Azure
Content Source: articles/app-service/environment/create-ilb-ase.md
Service: app-service
GitHub Login: @ccompy
Microsoft Alias: ccompy

Sorry, wrong copy/pase.

create a zone for .appserviceenvironment.net
create an A record in that zone that points * to the ILB IP address
create a zone for scm..appserviceenvironment.net
create an A record in the scm zone that points * to the ILB IP address

@cult-x  Thanks for the feedback! We are currently investigating and will update you shortly.

Thanks for updating the documentation. I did create the 2 zones and A records as stated above. I am using Internal ASE. I created a jumpbox in the same VNET. I am unable to access Kudu console. When I try to access kudu console it get redirected to an invalid URL which results in Server IP address could not be found error.
I will also open a support case to get to the bottom of this issue.

@cult-x Thanks again for the feedback! I have assigned the issue to the content author to review further and update the document as appropriate. @rajcheval Thanks for asking question! Request you to open new issue so that we can track the progress on it. Thanks for your understanding.

Any update on Kudu/SCM access and the dns wildcard entry working as described?  We are looking to cut down on DNS overhead and are hopeful that dns wildcard entry "*" will allow for dns resolution for hosted apps on the ILB-ASE.

@hammer5280  I used fiddler to figure out what was going on. The documentation is indeed missing a required step. It turns for that for kudu console to work I had to create a host entry on my host file for ASE URL.   In addition to what is already mentioned in the documentation I had to point ILB IP to the ASE URL.  Once I did this Kudu console started working.

I succeeded to access kudu on ILB ASE with 3 A records "", ".scm", "@" in ilbasename.appserviceenvironment.net zone, all points ILP private ip address.
Third configuration for @ seems to be missing in this doc.

Sorry, wildcard(asterisk) was removed...
I want to write 3 A records named "", ".scm", "@"

"", ".scm", "@"

Article has been updated with the correct records. #please-close


RPG Step 118 instructions are not clear enough

Describe the Issue
The tests are requiring that the variable monsterHealth be used to set monsterHealthText.innerText but this is not mentioned in the instructions and the hint for this test is vague at best.
Affected Page
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures-v8/learn-basic-javascript-by-building-a-role-playing-game/step-118
Your code
This code is technically correct but not accepted:
function goFight() {
  update(locations[3]);
  monsterHealth = monsters[fighting].health;
  monsterStats.style.display = "block";
  monsterName.innerText = monsters[fighting].name;
  monsterHealthText.innerText = monsters[fighting].health;
}

Changing the last line to use monsterHealth is required to pass:
function goFight() {
  update(locations[3]);
  monsterHealth = monsters[fighting].health;
  monsterStats.style.display = "block";
  monsterName.innerText = monsters[fighting].name;
  monsterHealthText.innerText = monsterHealth;
}

Expected behavior
The hint you get when you use monsterHealthText.innerText = monsters[fighting].health; is:
"You should set the innerText property of monsterHealthText to be the health value of the current monster. Remember that you assigned this value to a monsterHealth variable."
The first sentence in this hint is stating something that is already true but makes it sound like it is false. Using monsters[fighting].health is a valid way to get the health value of the current monster. So the user has fulfilled the requirement of the first sentence but is being told they haven't, which is confusing. Reminding them in the second sentence that they have already saved this value off to the monsterHealth variable might help some people make the connection that the tests want them to use that variable instead of monsters[fighting].health, but I have just been going back and forth with someone in the forums that isn't getting this message, so I don't think it is explicit enough.
If the tests are going to require that the variable monsterHealth be used then I think the instructions need to explicitly state this. Perhaps something like:

Now, set the innerText property of monsterName to be the name property of the current monster. Do the same for monsterHealthText and the health property. Recall that you have already saved off the monster's health in the monsterHealth variable. You must use this variable to set the innerText for the monster's health.

And then the hint can be updated to be more explicit as well.

You must use the monsterHealth variable to set the innerText for the monster's health.

Screenshots
No response
System
N/A
Additional context
No response

Hey @bbsmooth !
I agree with updating the directions here because this step does come up a lot on the forum.
Since this is a small fix, I will go ahead and open this up for contribution for first timers only


How to modify the loading path of a wasm file

Due to certain reasons, I am unable to load directly from the root directory. Is there a way to modify the loading path?
tree-sitter.wasm
tree-sitter-bash.wasm
like:
abc/tree-sitter.wasm
abc/tree-sitter-bash.wasm

There isn't one currently, you'd have to fork the repo.


PFUser become results in 206 in iOS SDK

Hello,
There appears to be a problem when using PFUser.becomeInBackground(sessionToken) from an iOS app.
All writes to the user object return a "[Error]: cannot modify user  (Code: 206, Version: 1.12.0)".
Using the REST API with the session token returned by my Cloud Code works (I attempted to edit a field in the user object, and it worked).
If you are unable to reproduce this issue / need code samples, just say.
Thanks for the attention,
Eduardo Almeida

same here. i get this error when using [user saveInBackgroundWithBlock...] method. And this is the full error:
error : {
    NSLocalizedDescription = "cannot modify user jV2QyE2Hpy";
    code = 206;
    error = "cannot modify user jV2QyE2Hpy";
    temporary = 0;
}

@MegaEduX @aoguzhan  You guys may check #430 that was merged recently and may fix this issue. Try running parse-server pointing to Head instead of the latest release, and check if it solves your problem.

@natanrolnik Downloaded latest release from GitHub, build it... The same error still happens.

We're facing the same issue with users authenticated through -and only- Facebook : #427 .
What about you ? Did you get this error with username/password too ?

@absolutlabs: I am using Digits, using an adapted version for parse-server of this code: https://github.com/felix-dumit/PFUser-Digits/blob/master/user.js
I never tried username/password, as I don't have it setup in my app.

Having a similar issue with the Javascript SDK and Parse Server on Heroku.

It looks like mine was related to using Parse.User.current().fetch() in the Javascript SDK.
Which works, but seems like its messing with the tokens and making me unable to post changes to server.

Could be related.
Just tested with the latest commit, still not working.

Getting the same error with the latest Parse iOS SDK today

This may have been fixed... please update to 2.1.3 and if you're still having a problem please open a new issue and provide as much information as possible.  Thanks.


Removed default setting of any JRuby options

This is to make a start for #296
This PR removes any setting of the ruby compat version or compile mode.
AsciidoctorJ will still run with the defaults and a user can now override the defaults if required.

:+1:
We definitely want to make it very clear in the AsciidoctorJ manual (currently the README) about how to set these flags, especially the compile.mode flag, since it can help improve speed of operation.

I will update the Optimization section and prepare a new PR.

:+1:


Suggestions for updating/enhancing fields

Checking the fields at http://data.okfn.org/data/okfn/public-bodies, I can think of many useful, practical informations that would be handy to have.
Brazilian public bodies' entries contain many relevant info but they're dumped in the "description" field, for example, such as law of creation (law n. 11.234/1900), keywords regarding the institution's activity ("research", "education", "legal", "indigenous"...). And those informations dumped in the description could be better used if they are spread among specific field ("law of creation", "activities").
Other info could be added: territorial subdivisions of a jurisdiction (as there are many regional and local public bodies, specially in federations like Brazil, Germany, US...); its CNPJ number (the number that identifies in Brazil any entity with legal existence that's not a person), which is used for many purposes, for instance to cross-check its budget incomes and spendings, to sue it in court, etc.; the name of its highest authority ("president" or "director" or "coordinator"...).
Use cases such as searching for all institutions with scope X in countries within criteria Z (eg, all cultural heritage institutions in Latin America, or all public pediatric emergency hospitals in Continental Europe) make me think that each body is allowed to have one and only one classification (category), so Brazilian universities are classified as "fundacao-publica" (public foundation), but "university" (or "higher education facility", or something of the sort) would be pertinent too. And often a university has a hospital in its medical school, or a pro-bono legal clinic in its law school.

Nice observations about the data in the Brazilian data! We should definitely include those as optional in the schema, IMO.
Maybe we could start expanding the fields in the schema by adopting the fields Core Public Organization Vocabulary of the EU. That would also make the data exportable to this standard.

@augusto-herrmann Thanks for linking me here. Yeah it sounds like that this is a better reference mentioned by you in the forum! I believe that one doesn't cancel out the other. It seems that Core Public Organisation Vocabulary has been deprecated and replaced by
e-Government Core Vocabularies.
I'm wondering whether these are developed and available for each country somewhere, i.e. if anyone is already maintaining and publishing this as a data set e.g. for Sweden? Or any other country for that matter.
I believe that this made accessible open source and available as concepts and datasets on data portals such as Swedish Dataportal could increase standardization and use. They could be uploaded or perhaps rather harvested from an external source to be displayed on Swedish dataportal. Sweden currently doesn't have an authority nor collaborative group of organizations which publish and maintain a dataset collecting all authorities according to e-Government Core Vocabularies or any other standard/specification but I think this project could work as a vehicle facilitating the process.

Those are excellent suggestions, @mattiasaxell ! Thanks for looking it up.
Would you like to also take a look at the fields present in the e-Government Core Vocabularies and make a map of what we would need to change in the schema here to make it compatible / compliant?
As for the Swedish data we have here, I believe they are probably very outdated. If there is someone here knowledgeable about Swedish data sources, it may be worth opening another issue to update that dataset.

@augusto-herrmann Thanks! No worries.
I can try and have a go later yes.
Yeah they are very outdated. I know since I added and updated them last time. However I think the issue here it that it does not make a lot of sense to try to keep it up do date in Sweden from volunteer efforts when the original data source is not open data, published nor  coordinated by the owners of different data. I'd rather spend energy on supporting the data owners to open and organize their data here and show how that can put Sweden more on the forefront on the field of open data.

Sure. Let's do what is possible for the moment, then.
If even one of the many data sources is published as open data, or even on a website that could be scraped, it might be worth the effort to automate the data collection like I did recently for Brazilian data, in order to have a partially updated dataset. Additionally, when and if someone has the time and energy to support the data owners in opening up, that would be awesome as well.
Anyway, if we need to further discuss the Swedish data, I would suggest opening a new issue, as this one is for the data schema. :slightly_smiling_face:

@augusto-herrmann Yeah sounds great.
OK, that sounds great. Starting a new issue now.
It might be worth it to write a data specification for a dataset about this? We could use ReSpec https://respec.org/docs/ which is open source: https://github.com/w3c/respec and use https://joinup.ec.europa.eu/collection/semantic-interoperability-community-semic/solution/e-government-core-vocabularies ?

I think we could indeed write a textual specification once it becomes stable enough. For now, to finish this current issue I think we should at least be able to make a proposal for a new set of tables and fields we are going to use, after reviewing the core vocabularies of course.


simSwap API yaml

@DT-DawidWroblewski
Following our exchange on this topic, I was expecting to get a swagger close to this one.
This is absolutely not a final contribution but here we have both flavor: endpoint to accommodate MC implementation and simple endpoint.
Are we still aligned on this specification 'pattern'?

Following Dec13th, 2022 meeting, the team prefer to have 2 distinct specifications defined instead of one. This PR is abandoned.


Does not have dockerfile

Description
Users who consume this template will have to create their own dockerfile, should they wish to leverage the Docker container technology.
Possible Solutions
Create a dockerfile so that anyone who uses the template can instantly run the application, without the need of creating their own.
Create a switch for the template which will optionally exclude the dockerfile when a user uses it to create a new project.

Hi, I can do this!

Amazing! Thank you.
I would recommend taking a look at the dockerfile for DwCheckApi to get you started, if you like. It's a little old, but should still cover the bases.


Move button definitions into ModuleManager-compatible config files.

If we move button definitions, and where they appear, into config files that ModuleManager can modify, it makes this mod significantly more extendable.

Done! I need to document the formats, but that's for another issue.


[MM-19791] Fix issues with enter button deleting filter rows

Summary
This PR fixes some accessibility issues regarding the enter button. Making all of the buttons in the modal require focus in order for the enter button to trigger the button click.
Ticket Link
https://mattermost.atlassian.net/browse/MM-19791

@mickmister I love seeing Accessibility fixes.  Sidenote: My mom was a speech pathologist, and I got my first experience with computers helping her at work with disabled kids who couldn't use a regular keyboard with computer programs.


Add cellhash-only to development branch

This was originally just intended to merge changes from #329 to the development branch, and as such this
Closes #328
While I was at it, I also decided to partially address #334, so here I am adding lines to the stub-metadata.tsv file that include paired- and single-end bulk data, as well as cellhash multiplexed data with and without accompanying bulk data.
This test uncovered another missing file() wrapper, so that is now added in as well (do we want to cherry pick that fix over to main before we cut that release)?

Thought I had requested review for this a while ago, but it was just GitHub suggesting a reviewer. 🤦


CanvasI is not defined

Hi,
Using the 0.0.104 build of Yested and Kotlin 1.0.2. I have a graph on one of the pages in my app. I'm getting an error on line 2113 in Yested.js (return (Kotlin.isType(tmp$0 = this.element, CanvasI) ? tmp$0 : Kotlin.throwCCE()).getContext(id);) that stated CanvasI is not defined. Is the the 104 build not compatible with Kotlin 1.0.2?
Thanks

This was fixed in 1.0.3.


Support events from browser action when the popup closes?

I am tracking a button click within the popup of Chrome extension's browser action. But the button click effectively also closes the popup (window.close). Will the event ALWAYS be send to GA or is there a race condition here?
Quick tests show the event is making it to GA, but is it just luck? Or is there a mechanism in CPA that ensures the XHR request is sent even though the window is closed?
The same applies to tracking a link that opens a new page/tab
Thanks!

There's a race. If you want to ensure the hit is sent, close the window from a callback.
E.g.
tracker.send(allDoneNowHit).addCallback(
function() {
this.closeMe();
}.bind(this));

Alright thanks, the info online on this is very confusing and never specific to Chrome extensions. I was starting to believe in magic...

will the callback be called even if the tracking is disabled (not permitted)?
Would you like me to a add a short section on the wiki regarding callbacks?

The callback is always called "when processing
Sure!


Compiling with -O0 causes the kobject text area to exceed the limit (linker error)

Compiling with -O0 causes the kobject text area to exceed the limit (linker error).
This PR has a failing testcase:
https://github.com/zephyrproject-rtos/zephyr/pull/5225
To reproduce, do
cmake -DBOARD=qemu_x86 -DEXTRA_CFLAGS=-O0 -Hzephyr/tests/kernel/test_build -Bb && make -C b
on commit 967d8b5

Tweaking
#define KOBJECT_TEXT_AREA	128
to be 256
fixed the issue, so perhaps we just need to tweak
#if defined(CONFIG_DEBUG) || defined(CONFIG_STACK_CANARIES)
to fix the issue.


[NeMo W 2023-11-28 20:23:00 transformer_bpe_models:59] Could not import NeMo NLP collection which is required for speech translation model.

after last upgrade to nemo_toolkit[asr]==1.21.0

I'm having same issue. Any updates or suggestions?

roll back to nemo_toolkit[asr]==1.20.0

same issue, @vladgrand2 that worked 👍


Make TargetPlatformVariant.all() the default for widget tests in flutter/flutter

Tracking issue for flutter.dev/go/test-all-platforms

We did not proceed with the linked change because it introduced too much volatility in conjunction with the randomized test execution that recently landed. The shuffled testing should be allowed some time to settle, each day is a new random seed and so it will expose broken tests over time. (example: https://github.com/flutter/flutter/pull/88989)
I am not going to be actively working on this right now, so I am going to un-assign myself. I am going to leave this open though to revisit later. I think this would still be beneficial to the framework. :)


We found that g and b in G_block are the features that I need to change. G and b in your network are generated by Dense(). Is there a way to change the g and b using vectors instead of the network? thank you

We found that g and b in G_block are the features that I need to change. G and b in your network are generated by Dense(). Is there a way to change the g and b using vectors instead of the network? thank you

What do you mean by "using vectors instead of the network" and why?


Day1: python

Hi Gabriel,
great initiative. Here is my solution using python!

Nice! Thank you for participating.


Output SQL console results in react-table

Summary of changes
What's new:

result columns are now "natural width", i.e. the width of the column is now calculated by the browser in the regular HTML <table> element. The previously used ant design Table component draws separate tables for the header, body and footer; hence the need for widths to be fixed for each one to align properly.
the scrollbars that appear when the content is taller or wider than the visible area, are always visible on screen. No more scrolling to reach a scroll bar. :)
results summary and pagination are now on the same row to save space.
results summary and pagination are now always visible regardless of scrolling.
row number now showing in the results table
to save space, result tabs only appear when there are 2 or more result sets.
Created a new CrateTabsShad tabs component from shadcn. If we're happy with this, I will replace the ant version with this one.
various tweaks to existing components.
added yarn check-types to find TypeScript errors easily (tell me if there is a better way to do this)

Some notes @proddata :

Fixing the pagination to the bottom of the page is harder than it should be in a reusable component like this. Adding the ability to put the pagination at the top solves the problem in a much simpler way. I suggest we go with pagination at the top, and if we are unhappy with it, change it in a follow-up ticket. If we do that, it may be easier for the SQL results to have a bespoke table component.
Re: always displaying the scrollbars whether they are used or not,  you can't force the scrollbars to always appear without the bottom (horizontal) scrollbar disappearing off-screen when the height of the table is greater than the height of the viewport. Therefore I have left it like this. Now that the pagination is at the top, I think it is OK (until we think of something better).

Some notes @alexdametto:
I think I hit the limit of TypeScript knowledge with the column types on SQLResultsTable.tsx. Would appreciate your help here, please, for the TypeScript n00b.
Checklist

[x] Link to issue this PR refers to: https://github.com/crate/cloud/issues/2006
[x] Relevant changes are reflected in CHANGES.md.
[ ] Added or changed code is covered by tests.
[ ] Required Grand Central APIs are already merged.

added yarn check-types to find TypeScript errors easily (tell me if there is a better way to do this)

For me, all typescript errors are shown when running the app. I don't know if it's the same for you (it should), but if you find this command useful then happy to have it 😄

Some notes @alexdametto:
I think I hit the limit of TypeScript knowledge with the column types on SQLResultsTable.tsx. Would appreciate your help here, please, for the TypeScript n00b.

I will have a look on these types, they are usually quite complex..


update zh_CN translation

I am following the way here (http://talk.manageiq.org/t/cloudforms-ui-translation/948/5) to submit some of zh_CN updates so far. Hope I can work on it.

@mzazrivec Please review

@chessbyte Reviewed, OK to merge, the PR contains -- as far as I can tell -- 109 new strings
translated to simplified chinese.
@martinliu Thank you for this contribution. Also, please feel free to use translate.zanata.org
as a tool for translations of our project:
https://translate.zanata.org/zanata/project/view/manageiq

I updated simplified chinese translation we have in translate.zanata.org with these new strings.

@mzazrivec I am already in zanata.org for awhile, I will join miq new now. That's will be better then I do resource file.

@mzazrivec It seems I can not copy manageiq project. Is it open for translators? Or  I should request some sort of access right to this project.

@martinliu I'm not quite sure what 'open for translators' means.
I added you to the list of maintainers for ManageIQ in zanata.org (provided that martinliu
is your username in zanata.org).

@mzazrivec I am able to translate miq in zanata.org now. Thanks!


[feature request] make elp eqwalize accept multiple specific modules

Describe the bug
It would be nice if elp eqwalize accepted multiple specific module names as a middle ground between eqwalizing an entire app/project and single modules.  Rationale: I frequently need to eqwalize a set of modules involved in a series of commits, where those modules are either a tiny subset of an application having ~100 modules, or involve multiple apps in a larger project; I want to be able to quickly iterate on the most-relevant small subset before doing elp eqwalize-app or elp eqwalize-all.
To Reproduce
elp eqwalize mod_a mod_b mod_c

Expected behavior
mod_a, mod_b, and mod_c are eqwalized.
Actual behavior
% elp eqwalize mod_a mod_b mod_c
No such command: `mod_b`, did you mean `--as`?

Hi @zerth, thanks for raising this. @VLanvin @alanz does this make sense to you? Contributions are also welcome!

Makes sense, and I think this is a very simple change.


Font size selector not working

Describe the bug
The font size selector in the sidbar does not work anymore.
To Reproduce
There are 5 steps you can choose:
| 1 2 3 4 5 |
Default is 3. Selecting 1, 2, 4, 5 increases the font size by 1(?) step. Selecting 3 sets font size back to normal.
Expected behavior
1 smallest and 5 largest font size
Hugo version:
hugo v0.92.1+extended linux/amd64 BuildDate=2022-01-29T07:56:42Z VendorInfo=debian:0.92.1-1
Theme version:
93a0a2dabb2b75e9b8f4e24b35c6a5138cf939bb (HEAD -> master, origin/master, origin/HEAD) style(docs-catalog): change padding as .5rem

Desktop (please complete the following information):

Firefox

Chromium

blog.natenom.com

Fixed by 433090d


Make intra tasks in IndexingChain.flush parallel execute.

Description
Similar to https://github.com/apache/lucene/pull/13124, https://github.com/apache/lucene/pull/13190.
Can we add a executor to SegmentWriteState to make tasks like writeNorms, writeDocValues, etc. in  IndexingChain.flush parallel execute?

@jpountz Please take a look when you get a chance.

I think you are right @jpountz . Since indexing already use almost all resources in many cases, maybe it is less worth to  add an executor to make intra tasks parallel execute for indexing.
I will close this issue.


allow plug-in analyzers to enable inlining at will

Codecov Report

Merging #212 (5752b49) into master (f2bddd9) will increase coverage by 0.28%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #212      +/-   ##
==========================================
+ Coverage   86.33%   86.62%   +0.28%
==========================================
  Files          12       12
  Lines        1647     1637      -10
==========================================
- Hits         1422     1418       -4
+ Misses        225      219       -6

Impacted Files
Coverage Δ

src/JET.jl
71.04% <ø> (-0.12%)
:arrow_down:

src/analyzer.jl
73.58% <100.00%> (ø)

src/interfaces.jl
89.36% <0.00%> (-0.08%)
:arrow_down:

src/typeinfer.jl
91.73% <0.00%> (-0.07%)
:arrow_down:

src/tfuncs.jl
98.71% <0.00%> (-0.02%)
:arrow_down:

src/abstractinterpretation.jl
93.93% <0.00%> (+2.08%)
:arrow_up:

Continue to review full report at Codecov.

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


Update cron scaler to parse the cron with parser instead of searching for '-'

Signed-off-by: jorturfer<EMAIL_ADDRESS>This PR changes the method to validate the cron. Now the package parser is used to detect invalid expressions instead a custom method.
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[x] Tests have been added
[x] A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
[x] A PR is opened to update the documentation on (repo) (if applicable) https://github.com/kedacore/keda-docs/pull/514
[x] Changelog has been updated

Fixes https://github.com/kedacore/keda/issues/2034

I have just rebased main into this branch to add integrate the changes


Reset tranport on partner auth

This resets the internal state of the transport when partner authentication is being handled. I think this should prevent the auth token errors we're seeing. Testing this on my system now to see if it works.

@jcass77 if you feel like giving this a test it might solve the problem.

So far this seems to be working. I have been able to play and change stations without any token errors


Add support for median filter for LR check depth mode

Other filters are WIP

Included in: https://github.com/luxonis/depthai-core/pull/159


VectorLayer may use outdated hitdetect image

Describe the bug
The hitdetect image may not be updated when the map is moved horizontally and wrapX is enabled.
To Reproduce
Steps to reproduce the behavior:

Go to https://openlayers.org/en/latest/examples/hitdetect-vector.html
Open devtools
enable keyboard input with document.getElementById('map').tabIndex = 0
put focus on map by clicking on it
use left / right keys to  move map
move mouse to test hit detection

Expected behavior
It should hitdetect features under the mousepointer only.

Interesting issue. Do you have an idea what might be causing this, @MoonE ? Scrolling with the keyboard vertically does update the hit detect image.

It is caused by the gutter calculation here:
https://github.com/openlayers/openlayers/blob/e82a86f56337cfa394235df7528da032a5afbf30/src/ol/renderer/canvas/VectorLayer.js#L547-L567
The extent min x is changed to -540 and max x to 540. Because of this the extent check here succeeds.
https://github.com/openlayers/openlayers/blob/e82a86f56337cfa394235df7528da032a5afbf30/src/ol/renderer/canvas/VectorLayer.js#L594-L603
Setting the x extent to the range -540 to 540 here once you leave world 0 will cause  replay groups for the whole world in x direction to be created even when you are at a very high zoom level.
You can see it when you drag the map from left to right. All features are drawn wheres from top to bottom only those that are in the extent at drag start will be drawn.


Windows builds no longer pass "-H windowsgui"

Describe the bug:

Windows binaries seems to no longer pass "-H windowsgui", which means that all binaries (even those built without -console) show a console when opened. This was recently reported to me at https://github.com/Jacalz/rymdport/issues/34.
I suspect that this broke with https://github.com/fyne-io/fyne-cross/pull/42/files#r692376503 but I sadly don't have time to look at it right now.
To Reproduce:
Steps to reproduce the behaviour:

Build a binary for windows without the -console flag.
Run the binary on Windows and notice that console is shown.

Example code:
Any code will do.

Fix available for testing in #99

Will be fixed in the next release.


Attach turn instructions to the GPX result

Use MatchResult.edgeMatches to fill MatchResult.instructions identical to what we do in Path.calcInstructions.
This needs some changes in the core (moving submethod of Path.calcInstructions into public method of e.g. InstructionList.create?).

Improved via #39


NPE Crash with RFTools

Hi again ;)
Another NPE crash for you, this time with RFTools. Crash report is at http://openeye.openmods.info/crashes/0ed3ba654ffc5fe472427796aa717659. Log is at http://pastebin.com/UJ8Njh3Y. I'm reporting here first again but I can report it to McJty if you'd prefer?
To reproduce, create an instance with RFTools (4.10), McJtyLib (1.7.10), Origin (1.7.10-7.1.0), Compression (0.0.4), CoFHLib (1.7.10-1.0.3-175). Start MC, Crash :) Remove either mod and it starts fine.
A simple workaround is to add "mcjty.rftools.items.dimlets.KnownDimlet" to the Item Class Blacklist.
Thanks.

Oops, make that RFTools 4.01

Oops, make that RFTools 4.0.1 and McJtyLib 1.7.0.

Ok. So it looks like I have to do some tinkering with my catch statement, because mod compatibility. Ill keep this open, and I have some ideas, but I cant work on it right this moment.


Please comment to be a Collaborator

Zalando Training Class May 2016
Please comment to be a Collaborator

:shipit:

Hallo

blah

hi-ho

hi

Please add me #notfirstmakesmesad

:-)

hi

:-)

hi!

Comment.

o_O

Happy New Year!

👍

Hey

Morning!

Hallo

:)

Happy Training

Ping

Hello

Please add me as collaborator

.

Hello!!!

addMe();

:)

🚀

a comment :octocat:

Hi folks! Here is the URL to the GitHub for Developers outline again...

hi

push

=

hello -larry

Hi

Hola!

:octocat:

Hi

hi

Also, here is a nice guide to Markdown. It is a lightweight syntax and used i.e. for comments in issues and pull requests...

Hi

:circus_tent:

awesome!

dont you say

Hi

:)

🐾

:octopus: + :cat: = :octocat:

mic check 1 2 1 2

i want to be in too

#1

Hm.....

hi

Adding all new people!

Please add me!

please add me

@sorinpente @kdomoratskyy added!

please add me


[Bug]: check extra764 doesn't handle bucket region properly

What happened?
When running the checks for S3 it picks up the global region instead of the region the bucket is created in
How to reproduce it
Steps to reproduce the behavior:

What command are you running? Check 7.64 [extra764] Check if S3 buckets have secure transport policy enabled
Running in Control tower multi-account environment, running as administrator in specific account
Error - Picks up us-east-1 but our region is not this region

Expected behavior
I would expect it to return the correct region as it has done for the following S3 checks :
[extra73] Ensure there are no S3 buckets open to Everyone or Any AWS use
[extra734] Check if S3 buckets have default encryption (SSE) enabled or use a bucket policy to enforce it
Screenshots or Logs

From where are you running Prowler?

Resource: Workstation
OS: Mac OS Monteray 12.2.1 (21D62)
AWS-CLI Version: aws-cli/2.4.13 Python/3.8.8 Darwin/21.3.0
Prowler Version: Prowler 2.7.0-24January2022
Shell and version:
Others:

Additional context
I picked it up by going to the finding and then going to the S3 instance, only to find it in the correct region and not the US EAST 1 region.

Hi @RoxyR44 , good catch! We have open a Pull Request on behalf of this issue: https://github.com/prowler-cloud/prowler/pull/1077
Please, test it to see if that works for you 😄

This is now fixed from PR https://github.com/prowler-cloud/prowler/pull/1077. Available in master branch now. It will be part of the next fix release in a week. Thanks!


[FLOC-1535] Update cluster local state before supplying it to calculate_necessary_state_changes

Fixes https://clusterhq.atlassian.net/browse/FLOC-1535
We'd also planned to remove the local_state argument of calculate_necessary_state_changes, thinking that the cluster_state would have all the required information. But actually, the NodeState attributes running and not_running and used_ports are not available from cluster_state. It is a Deployment and  only has Node records.
There should probably be a followup issue to create a DeploymentState record which has NodeStates rather than Nodes and a corresponding update to the control service and AMP protocol to broadcast DeploymentState records.

Ah, there are some tests to fix. :frowning:

Ok. So some points for review / discussion:

@itamarst and @exarkun discussed the case of a convergence agents that is not node specific. In this case, what is the local_state?

Perhaps an example of such an agent might be an flocker-openstack-agent which carries out only the attach and detach of openstack block devices leaving the mounting (and if required formatting) of the volume (inside the node VM) to flocker-dataset-agent.
But it seems to me that it'd still discover_local_state and that would still need merging with the cluster state received from the control service....so perhaps that's not a good example.

It looks like this is going to require a ton of changes to the existing tests, which supply dummy local and cluster state objects instead of real Deployment and NodeState objects. Which prevents the ConvergenceLoop doing anything with those arguments.

I need some ideas about how to refactor those tests, if we choose to go ahead with this.

Fix failing tests and address comments, resubmit.

Made suggested changes and also moved ControllableAction to flocker.node.testtools.

Looks good, address one or two minor comments then merge.


Clean up build-site --help

$ ./bin_files/build-site --help
Possible options are: no-status, no-preserve-state, no-refresh, recompile, full-render, without-processing, without-report, without-completion, before, after, collection-info, dump-at, debug-when, verbose-when, with-only
Options passed were: help
  in sub collect at /home/builder/MoarVM/install/share/perl6/site/sources/F92650F2CEB9118638B7566E195B3CA15F04EBB9 (Collection) line 13
  in sub MAIN at ./bin_files/build-site line 5
  in block <unit> at ./bin_files/build-site line 3

Can we display (some of these) from config.raku?

completed with Collection:ver<0.14.2>


Use crosshair for color selection

It works but the problem is that the cursor is most time a little bit of the position of those two bars.

I'm not merging this because that cursor looks weird imo. That pallete cursor looks ok as standard arrow.


Windows Support

Pull Request Status: Almost Ready
Important Changes

Improved CMakeLists.txt.
Improved .travis.yml.
Added appveyor.yml.
Separated the tests.
Separated the examples.
Created cross-platform versions of _isatty(...) and _fileno(...).

Linux / macOS Build

All tests compile.
All tests pass.
All examples compile.

Windows Build

test_struct.cpp fails.
All examples compile.

Notes

By the time I forked this repository it seems that you created new 1 commit. This probably caused a merge conflict, but I think it will be easy to resolve! If you face any problems resolving it just text me here and we will find a solution (I will probably have to copy & paste the code segments you added)!
For some reason AppVeyor does not stop when a command fails to execute. I am sure you can change this behavior through settings.

I will also rewrite the my commit history when I get the first successful windows build.

@sharkdp
I go get some sleep now :rofl:
Let me know what you think!

Thank you for working on this!

* By the time I forked this repository it seems that you created new 1 commit. This probably caused a merge conflict, but I think it will be easy to resolve! If you face any problems resolving it just text me here and we will find a solution (I will probably have to copy & paste the code segments you added)!

It would be great if you could rebase your branch or merge in origin/master, so that we can really see all the changes in the PR here (and test the merged state).

It may take me sometime to resolve all the issues today. I go eat :yum:

It may take me some time to resolve all the issues today. I go eat yum

By all means, take your time :smile:! There is absolutely no hurry. I will be very busy the upcoming days, so this might take some time to finally review & merge anyway.

I need some help with resolving the merge conflicts. Please provide any usefull learning material.

I will open a new PR without merge conflicts.


Urgent help required in using boxplot

This is my current code, I am unable to use the BoxPlot and I'm experiencing "Uncaught Error: Highcharts error #17". Is there something I did not import or something wrong with my syntax for BoxPlotSeries?
Really need help urgently for a project due in 1 week's time, thank you in advance!
import React, {Component} from 'react';
import { withHighcharts, HighchartsChart, Chart, XAxis, YAxis, Title, Subtitle, Legend, LineSeries, BoxPlotSeries } from 'react-jsx-highcharts';
import Highcharts from 'highcharts';
render() {
if (this.state.mount === true) {
return(

        <Title>Performance over Past 20 Levels</Title>

        <Subtitle>Source: App</Subtitle>

        <XAxis>
            <XAxis.Title>Level</XAxis.Title>
        </XAxis>

        <YAxis id="number">
          <YAxis.Title>Percentile</YAxis.Title>
          <BoxPlotSeries id="1" name="L1" data={[this.state.performanceArr[0]['minTiming'],this.state.performanceArr[0]['25thPercentile'],this.state.performanceArr[0]['medianTiming'],this.state.performanceArr[0]['75thPercentile'],this.state.performanceArr[0]['maxTiming']]} />
        </YAxis>
      </HighchartsChart>
    </div>
    )} else {
      return null;
    }
  }
}

export default withHighcharts(StudentBoxPlot, Highcharts); // Injecting the Highcharts object

Hi, Box Plot is not included in the standard series set, you need to extend Highcharts with the "more" module they provide.
import Highcharts from 'highcharts';
import addHighchartsMore from 'highcharts/highcharts-more';

addHighchartsMore(Highcharts);

Thank you for your reply! The error has disappeared, but the boxplot is still not showing, may I know if you could help me with the syntax for BoxPlot? I can't seem to find any documentation on how the syntax is supposed to be in the render method. Thank you! Also, this is an amazing package, really love it!!


SyntaxError: positional argument follows keyword argument

 base_model = tf.keras.applications.MobileNetV2(input_shape=( 160, 160, 3),
                                                   include_top=False, # <== Important!!!!
                                                   weights='imagenet') # From imageNet

    # freeze the base model by making it non trainable
    base_model.trainable = False

    # create the input layer (Same as the imageNetv2 input size)
    inputs = tf.keras.Input(shape=160,160,3)

    # apply data augmentation to the inputs
    x = data_augmentation(inputs)

    # data preprocessing using the same weights the model was trained on
    x = preprocess_input('imagenet')

    # set training to False to avoid keeping track of statistics in the batch norm layer
    x = base_model(x, training=False)

    # add the new Binary classification layers
    # use global avg pooling to summarize the info in each channel
    x = keras.layers.GlobalAveragePooling2D()(x)
    x = keras.layers.Dropout(0.2)(x)
    # include dropout with probability of 0.2 to avoid overfitting

    # use a prediction layer with one neuron (as a binary classifier only needs one)
    outputs = keras.layers.Dense(1)(x)

This is from documentation:

Is this is a bug ??
Thanks!

Hi @starboyvarun !
We see that the issue template has not been filled, could you please do so as it helps us analyse the issue [tf version, steps followed before you ran into this error or stand alone code/colab gist to reproduce the issue faced].Thanks!

@mohantym Thank you. I have solved this particular issue.


fix: issue a warning if CRD couldn't be applied

fix: unwrap optional
fix: issue a warning if CRD couldn't be applied

I'm actually not sure of what's going on there, I think I will re-open the PR in a cleaner way as I originally had it targeting the wrong branch.


Add a Gitter chat link to readme.txt

neutrinolabs/xrdp now has a Chat Room on Gitter
@speidy has just created a chat room. You can visit it here: https://gitter.im/neutrinolabs/xrdp.
This pull-request adds a link to your readme.txt.
If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

what do you think about a telegram group ?

Closing automatically raised pull request.


Fulfillment answers don't reach Telegram, Facebook and Slack integration

I created a simple bot to test some new resources, but I'm facing an issue with fulfillment responses on some clients. I'm testing with many integrations.

✅ The fulfillment work fine with Web Demo and Dialogflow Messenger
🔥 The fulfillment fails with Facebook Messenger, Slack and Telegram

I wrote two intents to test:

"Hello" - Text Response Intent
"Hi" -  Fulfillment Intent answered by node using actions-on-google package.

What happened:

"Hello" - intent works on all integrations, like expected.
"Hi" - intent works like expected with Dialogflow native integrations (✅ )
"Hi" - intent doesn't work with other integrations (🔥). It shows "Typing..." while my node backend creates the answer, but the text response is never showed.

In debug mode, I see that the request is received, the intent match, and the Response object are like expected.

Node code:
const { dialogflow } = require('actions-on-google');
const app = dialogflow({ debug: true });
...

app.intent('Hi', conv => {
  console.log('>> Hi Fulfillment reached');
  conv.ask(`Hi! I'm a fulfillment response on node!`);
});

Node response (console output):
>> Hi Fulfillment reached
Response {
  "status": 200,
  "headers": {
    "content-type": "application/json;charset=utf-8"
  },
  "body": {
    "payload": {
      "google": {
        "expectUserResponse": false,
        "richResponse": {
          "items": [
            {
              "simpleResponse": {
                "textToSpeech": "Hi! I'm a fulfillment response on node!"
              }
            }
          ]
        }
      }
    }
  }
}
POST /api/chatBot/ 200 156 - 1419.611 ms

I recorded my screen to show better that behavior.

I also posted this as a question on StackOverflow

This library is not designed to work with any platform other than Actions on Google. If you are looking at hosting your application on other platforms, you would do better to use dialogflow-fulfillment which should be more generalized.

Ok. Got it. Thank you @Fleker !


Update dependency: grpc-httpjson-transcoding

Signed-off-by: Wayne Zhang<EMAIL_ADDRESS>Update to the latest changes
Risk Level: None
Testing:  Unit-tested
Docs Changes: None
Release Notes: None

/lgtm deps