Add offboard/set_actuator_control to offboard plugin without specifying group names

Implementation of set_actuator_control method due to https://github.com/mavlink/MAVSDK/pull/771#pullrequestreview-256231538.
I think that this solution most corresponds to mavlink message specification compared first PR.
Also in protobuf3 "0" is the default for numeric types so this would hide the complexity.
The corresponding MAVSDK-Proto PR is here.

Hi @julianoes
I tried to implement your suggestion.
Also I tried to avoid unnecessary load on the CPU by passing the length of the array in MAVSDK API method.
If I misunderstand something I will be happy if you will explain it.
This feature is important for us and we want implement really good solution for community :)
Thank you!

@irsdkv what do you think about this suggestion? https://github.com/mavlink/MAVSDK/pull/771#issuecomment-507982988
So the API would be:
std::array<float, 16> actuators = { 0.1, 0.2, 0.3, 0.4, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0 };

and if the last 8 entries are zero, we only send the message to control group 0, and we don't need to send the message to control group 1.
Would that work?

and if the last 8 entries are zero, we only send the message to control group 0, and we don't need to send the message to control group 1.

Unfortunately, due to px4 uORB subscription subsystem, if we need to fill all controls to zero - we should send all-zero message. If we don't do this then actuator_controls_1 subscribers will not receive zeroed controls and the actual value will still remain the previous received (non-zero) values.

Ok, what about NAN?

Ok, what about NAN?

Hm. I think that this can be a possible solution

It's a pleasure. Thank you too :smile:

CI complains about std::isnan:
/home/user/DronecodeSDK/src/plugins/offboard/offboard.cpp:92:18: error: '__builtin_isnan' is not a member of 'std'
             if ((std::isnan(lhs.groups[i].controls[j]) && std::isnan(rhs.groups[i].controls[j])) ||
                  ^

Try adding #include <cmath> in offboard.cpp.

Looks like the grpc service doesn't quite compile (anymore):
https://travis-ci.org/mavlink/MAVSDK/jobs/556753980#L7412

Looks like the grpc service doesn't quite compile (anymore):
https://travis-ci.org/mavlink/MAVSDK/jobs/556753980#L7412

My bad. Seems like just need to update proto submodule rev

I think now you only need:
./tools/fix_style.sh .

Done

@irsdkv Thanks for all the work! We recently fixed the clang-format issues, that should be fine now. Sorry for the inconvenience there :sweat_smile:.
Can you try fix_style again, from latest develop? Just want to make sure it works for you as well. If not I'll have to investigate.

I think all works ok :)

Hmm somehow it seems like some styling was still wrong. I pushed a fix, let's hope it works this time :-).

Hmm somehow it seems like some styling was still wrong. I pushed a fix, let's hope it works this time :-).

Oh, sorry. It was my changes. I did it because I thought it was more readable and similarly to other functions like this. I apologize for not telling you this.
P.S. Whew. Seems that this pull request has a hard fate :smile:

Oh, you're missing some docs (see Jenkins):
Parsing file /tmp/jenkins/workspace/mavlink_MAVSDK_PR-782/tools/docs/install/include/mavsdk/plugins/camera/came/tmp/jenkins/workspace/mavlink_MAVSDK_PR-782/tools/docs/install/include/mavsdk/plugins/offboard/offboard.h:151: warning: Compound mavsdk::Offboard::ActuatorControl::Group is not documented.

Please check doxygen warnings.

Good news is that this is the last CI test 😆

Argh, somehow having issues with Jenkins. Trying to fix that.


Support appending text when marking tasks

It would be great if one can append text when marking tasks, just like what one can do when completing and cancelling tasks.
Also, it would be great if one can set the appended text to be different from those for completing and cancelling tasks. After all, after completing/cancelling tasks, one may not need to follow up with the task any more. When marking tasks, however, e.g. marked it as [/], one may still need to follow up with such tasks. This way, one can query different types of tasks by looking for different types of appended text.

This one is a little tricky.
Annotating the same way (e.g. with a simple date) could be doable with a simple option (and undo/reset could still work, in that case). Annotation with different strings is a different animal, as it then becomes difficult to know what strings to look for or remove if you then change the state to a different value. It isn't impossible, but it is way easier to fubar. ;)
When I do this, I use the task collector to mark the task w/ the special character, but I add the tags or follow-up indicator myself (as it isn't often that consistent.. delegation can be easy enough, but follow-up/check-later would be really difficult to automate consistently, at least as I do things... )

Annotation with different strings is a different animal, as it then becomes difficult to know what strings to look for or remove if you then change the state to a different value.

How about only keep the text corresponding to the latest status of the task then? For example, when the status of the task changes in the following manners, only the text/date correspond to the current status is kept. This also means that whenever the status of a task is changed, the original text would be first removed and then the new text would be appended.
- [ ] Task
- [>] Task 2022-02-07T12:00
- [/] Task 2022-02-08T12:00
- [x] Task 2022-02-09T12:00

Of course, one can manually copy the text corresponding to the previous status of the task and save it as a sub-list item, for example, if one wants.
This way of handing the appended text corresponding to the status of the task also prevents the task description from being too long, if the status changes a few times.

Exactly. Applying the configured append string all of the time is doable (as I already know how to undo that to allow a new value to be set), and yes, could be toggled via config.


"Gateway Timeout" with docker

What version of Traefik are you using (traefik version)?
1.2.0-rc2
What is your environment & configuration (arguments, toml...)?
--web --docker --docker.watch --docker.domain=localhost --logLevel=DEBUG
What did you do?
I have this configuration in docker-compose.yml file for traefik container
version: '2'

services:
  proxy:
    image: traefik:1.2.0-rc2
    command: --web --docker --docker.watch --docker.domain=localhost --logLevel=DEBUG
    networks:
      - webgateway
    ports:
      - "80:80"
      - "8080:8080"
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
      - /dev/null:/traefik.toml

networks:
  webgateway:
    driver: bridge

When I run docker-compose up from my projet I have Gateway Timeout.
Here my docker-compose.yml of my project (simplified)
version: '2'

services:
  nginx:
    container_name: nginx-my_network
    image: nginx:stable
    networks:
      - traefik
    labels:
      - "traefik.backend=nginx"
      - "traefik.frontend.rule=Host:my_project.local"
      - "traefik.port=80"
      - "traefik.frontend.entryPoints=http"
    networks:
      - traefik
      - my_network

  php-fpm:
    container_name: php-my_network
    image: php:7.1-fpm
    command: '-F'
    networks:
      - my_network

  init-my_project:   # _/!\ Important - This service starts at the beginning and then ends when it finishes initialization processing, if i remove this It works all the time._
    container_name: my_project
    build:
      context: ../../..
    networks:
      - my_network

networks:
  my_network:
    driver: bridge
  traefik:
    external:
      name: traefik_webgateway

init-my_project service starts at the beginning and then exit when it finishes initialization processing, if i remove this It works all the time.
What did you expect to see?
I should see my website all the time.
What did you see instead?
It works randomly and rarely.
I can see this warning only
proxy_1  | time="2017-03-08T07:32:48Z" level=warning msg="Error forwarding to http://<IP_ADDRESS>:80, err: dial tcp <IP_ADDRESS>:80: i/o timeout"

I put more logs here: https://gist.github.com/Netmisa/266885e4429a2a09feb1ca95e76c5239
If I go here http://<IP_ADDRESS>:80 it works.
I think the problem comes from the init-my_project, because when I remove this it works.
I tested binary (same version), It works correctly.

Can you create a gist with the debug logs showing the failures?

Hi @dtomcej
I have only this error:
proxy_1  | time="2017-03-08T07:32:48Z" level=warning msg="Error forwarding to http://<IP_ADDRESS>:80, err: dial tcp <IP_ADDRESS>:80: i/o timeout"

I put more logs here: https://gist.github.com/Netmisa/266885e4429a2a09feb1ca95e76c5239
If I go here http://<IP_ADDRESS>:80 it works.
Description updated with this additionals informations.

Hi @dtomcej
I found a concrete exemple:
docker-compose.yml (not work)
version: '2'

services:
  whoami:
    image: emilevauge/whoami
    labels:
      - "traefik.backend=whoami"
      - "traefik.frontend.rule=Host:test.local"
    networks:
      - my_network
      - traefik

  hello_world:
    image: library/hello-world
    networks:
      - my_network

  hello_world_1:
    image: library/hello-world
    networks:
      - my_network

networks:
  my_network:
    driver: bridge
  traefik:
    external:
      name: traefik_webgateway

docker-compose.yml (work)
version: '2'

services:
  whoami:
    image: emilevauge/whoami
    labels:
      - "traefik.backend=whoami"
      - "traefik.frontend.rule=Host:test.local"
    networks:
      - traefik

  hello_world:
    image: library/hello-world

  hello_world_1:
    image: library/hello-world

networks:
  traefik:
    external:
      name: traefik_webgateway

If I remove my_network it will work fine.
I hope this will help you

I'd just like to add that I'm having the same problem as @Netmisa , which was resolved by removing my custom network.

Worked like a charm. We had issues on the same problem and we found that they were always about networking. So finally with this post we decided to remove all the custom networks.
Lucky for us we don't really need them (the custom networks) but what will happen if we do?
Thanks @Netmisa!

Hey. We encounter the same issue. Any workarounds exept of removing overlay network?

@SAprelov If one of your containers leaves, it's doesn't work. So maybe if you keep them alive, it should be work.

Isn't this problem solvable with: - "traefik.docker.network=traefik_default"? I also had this problem and without that flag traefik would randomly choose a network (usually the wrong one).
Example configuration with Odoo:
https://gist.github.com/thomas15v/451acd1eb332f23351a6765901d8f560

Interesting @thomas15v !
I'll try that. Thanks :)

@thomas15v appears to have totally solved my gateway timeout occurrences - thanks for that reminder.

@thomas15v Thanks for your feedback !
You are right it's better (and more readable) to specify traefik.docker.network label
I do this in all my projects after this issue, and I forgot to test this in this situation..
Thanks again
@dtomcej Maybe we can close this issue, if you are ok ?

Make sure all containers are set to the "bridge" network.

@Rich43 that might not be preferable. Not sure but I think it is similar to setting the network_mode to host. Every container that opens ports on this network should open it on the host machine.


v1.20.0 Test Plan - Community help requested

It's testing time again here in the DDEVfactory. We love community help with every item that is interesting or relevant to you, and just using the release candidates is hugely helpful, because you may stumble across something in your regular workflow. Please use DDEV HEAD or later for your testing (or brew unlink ddev && brew install --HEAD --fetch-head drud/ddev/ddev).
I don't think in general you'll have trouble using this in your daily workflow, so would love you do do that. You can switch back and forth , see How can I install a specific version of DDEV in the FAQ.
If you find a bug that you're pretty sure is a bug, then just open an issue for it.
To give feedback or ask questions, please just reply on this issue or discuss in discord or any other support channel.
I am maintaining the provisional release notes for v1.20.0 and would love any comments on those.

install_ddev.sh should work with new pipe-based install.sh. It should brew unlink where necessary.
web_extra_exposed_ports: Try it out with example from docs.
web_extra_daemons: try it out, especially with a daemon that exposes an extra port. Best and most difficult tests involve running something that is in code (in project) and with mutagen enabled.
phpand mariadb default versions: Create a new project, php project is probably best. Defaults should be in it already.
config.*.yaml merging: Create a few config.*.yaml that have extra elements like hooks, environment variables, web_extra_daemons, web_extra_exposed_ports. Should be merged.
ddev craft command - try out on craft project
ddev npm relative directory. Try out ddev npm install in a subdirectory of project. It should run in the same relative directory inside the container.
ddev yarn relative directory. Try out ddev yarn install in a subdirectory of project. It should run in the same relative directory inside the container.
config.yaml composer_version support: Try out all the options in docs
Platformsh provider support: Manually test the platform.sh support, pull and push.

✅ ddev php -v on a php project-type show 8.0.21.  Host system running 7.2

added Laravel project-type and was able to migrate database
php project-type install MariaDB 10.4.25-MariaDB with collation as utf8mb4_general_ci

✅  ddev npm: confirmed.

ddev npm install installed packages listed in package.json in a new project & it created package.lock.

✅  ddev yarn: confirmed.

ddev yarn installed packages listed in package.json in a new project & it created yarn.lock.

✅ install_ddev.sh

curl -fsSL https://raw.githubusercontent.com/drud/ddev/master/scripts/install_ddev.sh | bash works to get v1.19.5 (and did brew unlink for me)
curl -fsSL https://raw.githubusercontent.com/drud/ddev/master/scripts/install_ddev.sh | bash -s v1.19.2 works to get v1.19.2
curl -fsSL https://raw.githubusercontent.com/drud/ddev/master/scripts/install_ddev.sh | bash -s v1.18.0 works to get v1.18.0

✅

Everything I tested was good. Hope it all works out!

Thanks for your testing @tyler36 !


Pug not working in Nuxt 2.3.4

Version
v2.3.4
Reproduction link
https://github.com/nuxt/nuxt.js/tree/dev/examples/pug
Steps to reproduce
Clone https://github.com/nuxt/nuxt.js/tree/dev/examples/pug
npm install
npm run dev
What is expected ?
Build with no error.
What is actually happening?
ERROR  Failed to compile with 1 errors
This relative module was not found:

./about.vue?vue&type=template&id=0a606064&lang=pug& in ./pages/about.vue

Additional comments?
https://github.com/nuxt/nuxt.js/issues/3469

This bug report is available on Nuxt community (#c8526)

It still doesn't work for 2.4.5.


Can't use scoped npm packages

If I try to run docker-compose up with a scoped npm dependency like @google-cloud/language, I get the following error after build...
api_1    | Finished 'build' after 2357ms
api_1    | module.js:538
api_1    |     throw err;
api_1    |     ^
api_1    |
api_1    | Error: Cannot find module '@google-cloud/language'
api_1    |     at Function.Module._resolveFilename (module.js:536:15)

I think maybe this is due to the nested directory structure in scoped package. The package exists at node_modules/@google-cloud/language instead of at the top level of node_modules

This ended up being an issue with @google-cloud/language and isn't related to this project


.string works incorrectly with some parsers

  import cats.parse.*
  val p = (Parser.string("\"\"") *> Parser.pure('"')).rep0
  val str =  "\"\"\"\""
  assert(p.string.parseAll(str) == p.map(c => String(c.toArray)))

I don't see the problem here.
breaking it down:
val p0 = Parser.string("\"\"")
val p1 = Parser.pure('"')
val p2 = p0 *> p1
val p = p2.rep0

okay, so p0 matches exactly two quotes. p1 does not consume any input and returns the character ". p2 matches exactly two quotes and returns 1 character which is ". p will match n = 0, 2, 4, ... quotes and return a list of List[Char] with length n and all entries equal to the character " (because recall pure returns that).
Now, you do p.string.parseAll(str) where str is 4 quotes, since p can match 4 quotes, you do parse everything, but recall what p.string does: it returns the input string that was matched by p, so in this case it will be the string of 4 quotes.
finally, you say:
assert(p.string.parseAll(str) == p.map(c => String(c.toArray)))

but this is ill typed. The left hand side returns an Either[Error, String] and the right hand side returns a Parser0[String] I think. So, I think this assertion should always fail.
Lastly, I want to point out this example is monstrously hard to follow without the aid of a REPL (as I have tried to do here and may have made mistakes) due to your choice of using \". For instance, I would expect if this failed, it would have also failed with the character a which would have been much easier to read and write.
Since the assertion is ill-typed maybe that was a typo on your part. Let's assume the right hand side was supposed to call parseAll as well (p.map(c => String(c.toArray)).parseAll(str)).
Now, by coincidence of the way you wrote this, that assertion should fail I think. Why? Because though p will parse the string of 4 quotes, it will return a List('"', '"') after having done so. When you convert that to an String via String(c.toArray) you will get a string of only two " symbols.
So, I think this assertion should fail for two reasons: 1. lack of parseAll on the right hand side, 2. even if it were there the left would be a string of 4 quote symbols but the right only 2.
I may be wrong, as I said I did this all without the benefit of the repl (I don't have the code checked out on this machine). If you think I'm wrong, can you be a bit more verbose to help me understand and make it clearer where you see an assertion that is failing, and why it should pass, and also possibly printing out the left and right sides of any failed equality?

I don't see the problem here.

You're right. I made to mistakes indeed:

the typo with assert (yes, it was intended to demonstrate that parse results should be equal)
when [not]reading .string documentation, that clearly says that this parser will return the matched substring instead of any value that the underlying parser would have returned

So that's entirely my fault and I'm closing the issue.

it your defense, the name .string isn't great. Maybe p.matchedString would have been a better name for it.

it your defense, the name .string isn't great. Maybe p.matchedString would have been a better name for it.

Probably yes, but scaladoc is there, waiting for somebody to read it :)
Thanks for your patience.
Actually, this is my first encounter with scala-parse  (not monadic parser combinators in general), I was in hurry to parse some text, so I asked chatGPT to generate something, it made this mistake and I didn't check it.
Welcome to the brave new world powered by AI produced mistakes :)
The library seems great anyway.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_b9f4fn1zhv/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_b9f4fn1zhv/clone/recipe/meta.yaml:21: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_b9f4fn1zhv/clone/recipe/meta.yaml:22: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_b9f4fn1zhv/clone/recipe/meta.yaml:40: missing_description: The recipe is missing a description
ERROR: /tmp/abs_b9f4fn1zhv/clone/recipe/meta.yaml:40: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_b9f4fn1zhv/clone/recipe/meta.yaml:40: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_b9f4fn1zhv/clone/recipe/meta.yaml:40: missing_dev_url: The recipe is missing a dev_url
Errors were found


Update kubefed.md

This change is

Docs team, I am not sure if this punctuation is Ok given that there is a continuation - list of shell commands. PTAL.

@madhusudancs The curl commands are referenced by an earlier sentence in the note and are not part of the last sentence, so I think the period there is fine. Thx for looking!

@chenopis Thanks for review. My modification is made because of exactly the same idea as you mentioned above.


[Question] Customizing my algorithem

Required prerequisites

[X] I have read the documentation https://omnisafe.readthedocs.io.
[X] I have searched the Issue Tracker and Discussions that this hasn't already been reported. (+1 or comment there if it has.)
[X] Consider asking first in a Discussion.

Questions
As a beginner in safety RL, I'm very grateful to omnisafe'help for me.
I would like to implement customised algorithm based on algorithms in omnisafe. For example:
`
from omnisafe.algorithms.on_policy.base.trpo import TRPO
class Cust_algo(TRPO):
......
env_id = 'SafetyPointGoal0-v0'
agent = omnisafe.Agent('Cust_algo', env_id)
agent.learn()
`
What should I do to make it works? Or is there any other method making the same target achieved?
Once again, thank you for your efforts, and I look forward to your response. Thank you!

Thank you very much for your recognition. OmniSafe provides support for flexible algorithm customization. You can achieve algorithm customization by simple inheritance and overriding.
Suppose you need to implement a CustomTRPO based on TRPO. You can follow these steps:

Create a new file named custom_trpo.py in the directory of your choice (in this example, we'll choose omnisafe/algorithms/on_policy/base).
Implement the customization for TRPO in the created file.
e.g.

# Copyright 2023 OmniSafe Team. All Rights Reserved.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
#     http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
# ==============================================================================
"""Implementation of the customization of the TRPO algorithm."""

from omnisafe.algorithms import registry
from omnisafe.algorithms.on_policy.base.trpo import TRPO

@registry.register
class CustomTRPO(TRPO):
    """The customization of the TRPO algorithm."""

    # You can customize the algorithm by overriding the methods in TRPO
    # or adding new methods.

Register the custom algorithm in the __init__.py file located in the omnisafe/algorithms/on_policy/base directory.

# Copyright 2023 OmniSafe Team. All Rights Reserved.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
#     http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
# ==============================================================================
"""Basic Reinforcement Learning algorithms."""

from omnisafe.algorithms.on_policy.base.natural_pg import NaturalPG
from omnisafe.algorithms.on_policy.base.policy_gradient import PolicyGradient
from omnisafe.algorithms.on_policy.base.ppo import PPO
from omnisafe.algorithms.on_policy.base.trpo import TRPO
from omnisafe.algorithms.on_policy.base.custom_trpo import CustomTRPO

__all__ = [
    'NaturalPG',
    'PolicyGradient',
    'PPO',
    'TRPO',
    'CustomTRPO'
]

Create a CustomTRPO.yaml file in omnisafe/configs/on-policy directory (note that the file name should match the name of your customized algorithm class). Write appropriate parameters in the YAML file. You can copy the contents from `TRPO.yaml`` and modify them as needed.
Finally, you can run your CustomTRPO algorithm in a similar way to the already implemented algorithms in OmniSafe. For example, you can use the command:

cd examples
python train_policy.py --algo CustomTRPO.

By following these steps, you will be able to customize TRPO and run your CustomTRPO algorithm within OmniSafe.

Thank you for your reply, it really solve my question.

Feel free to open an issue whenever you encounter problem.


Remove .NET Framework 2.0 as a target framework

Remove .NET Framework 2.0 as a target framework from Confuser.Core.Runtime and Confuser.Protections.Runtime.
Addresses #28

The .NET Framework 2.0 targets of the runtime components of ConfuserEx are required. While ConfuserEx is not running itself is not running on .NET Framework 2.0 it supports .NET Framework 2.0 assemblies to be obfuscated. To do this the runtime components need to be compiled for .NET 2.0.


How to set events based on Weekdays

The latest version is currently 7.1.6
How can I add events based on weekdays? My server sends me values such as 1,2,3,4,5,6,7 which corresponds to Mon, Tues, Wed, Thu... If the value is 1 then I want to show a green dot on all Mondays and so on.

just format the data into a format that the lib can read. This lib is a UICollectionView under the hood. Therefore it uses the DataSource/Delegate approach.

Then there is one thing I can do, can you please guide me about how can assign values that I get into array to cellState. day.rawvalue and use conditions appropriately. Any hint?

There are a million ways to do this.
The question that you area asking is not library related. It is specific to you app.
I gave an example of how you can accomplish this in a youtube video. But i already see that you have watched the video.
Are you familiar with how UITableView or how UICollectionView works?

Yes I am aware of how both these elements work but I am getting index out
of range error. Could you please help me out achieving it?
On Sat, Nov 24, 2018, 8:30 PM JTAppleCalendar<EMAIL_ADDRESS>wrote:

There are a million ways to do this.
The question that you area asking is not library related. It is specific
to you app.
I gave an example of how you can accomplish this in a youtube video. But i
already see that you have watched the video.
Are you familiar with how UITableView or how UICollectionView works?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/patchthecode/JTAppleCalendar/issues/940#issuecomment-441373831,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AccITQRI76d7nTo5ytMO_VjH06LClP2Dks5uyV8fgaJpZM4Yw7QX
.

I am not sure how to explain it more.
I gave a clear example here  Look at the video from 4mins-41secs.
In the example i showed you that the data from my server was in the format
array -> [Date:String]

And i showed you how i used that data from the server to create dot events on the screen.
The only difference between your data from the server and my data from the server, is that your data is 1, 2, 3, 4, 5, 6 & 7
Therefore, I am not sure how else I can explain this clearly. The method is exactly the same as in the video.


Add various bug fixes and optimizations for pdb_input_to_molecule_input()

Adds various bug fixes and optimizations for pdb_input_to_molecule_input()
Also sets DEFAULT_NUM_MOLECULE_MODS to 4 to represent mod_protein, mod_rna, mod_dna, and mod_unk. Thoughts?
Also, not sure why all the tests are failing. Is there something wrong with GitHub Actions here?

@amorehead the package i'm using for splitting the tests is broken 😢
i'll fix it!


Settlement method needs to be included when creating an accounting point

Current Behavior

Currently Settlement method is not included upon creation of an accounting point when receiving the ConsumptionMeteringPointCreated integration event.
Expected behavior

The settlement method needs to be included when creating a corresponding accounting for a ConsumptionMeteringPointCreated  integration event. This is relevant for the following move-in rules:

Additional context/screenshots

This can be fixed together with #722

@Korgath Is this included in integration event?

@Christian-Meyer  The Settlement method info is part of the consumption metering point created integration event and the new metering point created integration event.
However Market Roles does not extract that information when creating an Accounting Point

@Korgath Thanks for update


Unable to use power consumption with sonoff POW Elite

Hi,
I am trying to use my sonoff POW Elite (Powr316d) with the API to recover the power consumption from a Raspberry Pi. But it fails, it says: "error: 404, msg: 'No power usage data found'" However all the other options in the API works well, I can eve toggle the device ON state.
This same code works just fine with a sonoff POW R3. And I can see the data correctly in the app and in the web browser. So I think is something related to the device or the API, not my code. Can you help me make this work? Or at least give me an alternative way to archive this?
Greetings!

Thanks a lot, I will check it out and give feedback :)

Dumb question, how I can test this change since there is no new release available?

npm uninstall ewelink-api
npm install https://github.com/baugp/ewelink-api

Thanks, I will test it on the weekend and give feedback.

Thanks :)
I uninstalled the original one and installed the one you told me.
I have been trying over the weekend, but the problem persist. It still says "{ error: 404, msg: 'No power usage data found' }" when using the sonoff pow elite. With the sonoff pow r3 it works just nice.
getDevice() works fine but getDevicePowerUsage('sonoff_id',true) returns error 404.


[bug] Notifications no longer updating for Yale Conexis L1 Door Lock

Version
zwavejs2mqtt: 3.2.0
zwave-js: 7.1.1
home id:<PHONE_NUMBER>
home hex:0xde677210
Checklist:

[ ] I am not using HomeAssistant. Or: a developer has told me to come here.
[x] I have checked the troubleshooting section and my problem is not described there.
[x] I have read the changelog problem was not mentioned there.

Build/Run method

[x] Docker
[ ] PKG
[ ] Snap package
[ ] Manually built (git clone - npm install - npm run build )

zwavejs2mqtt version: 3.2.0
zwave-js version: 7.1.1
Describe the bug
Notifications no longer updating for Yale Conexis L1 Door Lock:

[5-113-0-alarmType] Alarm Type
[5-113-0-alarmLevel] Alarm Level
This used to work and started to fail to refesh from late March, around 22nd

To Reproduce
Steps to reproduce the behavior:

Open Zwave JS Server UI URL
Expand Yale node
Expand the Notification Tab
Nothing changes

Expected behavior
Notifications values should change
Additional context
I do use Home Assistant, and eventually hope/expect that resolution on the JS Server will trickle down to HA, but since the issue is directly seen on the JS Server I'm creating an issue here. I hope this is ok.

Possible dup of   https://github.com/zwave-js/node-zwave-js/issues/2286

I have the same issue, they no longer update, but the entities are available in HA

not sure if helpful, but here are some logs:
2021-04-09 16:29:20.397 INFO ZWAVE: Node 5: has returned from the dead
2021-04-09 16:29:22.126 INFO ZWAVE: Node 5: value updated: 98-0-currentMode 0 => 0
2021-04-09 16:29:22.129 INFO ZWAVE: Node 5: value updated: 98-0-outsideHandlesCanOpenDoor true,false,false,false => true,false,false,false
2021-04-09 16:29:22.130 INFO ZWAVE: Node 5: value updated: 98-0-insideHandlesCanOpenDoor false,false,false,false => false,false,false,false
2021-04-09 16:29:22.132 INFO ZWAVE: Node 5: value updated: 98-0-latchStatus open => open
2021-04-09 16:29:22.133 INFO ZWAVE: Node 5: value updated: 98-0-boltStatus unlocked => unlocked
2021-04-09 16:29:22.134 INFO ZWAVE: Node 5: value updated: 98-0-doorStatus open => open
2021-04-09 16:29:23.061 INFO ZWAVE: Node 5: value updated: 98-0-currentMode 0 => 0
2021-04-09 16:29:23.062 INFO ZWAVE: Node 5 CC Notification {
type: 6,
event: 2,
label: 'Access Control',
eventLabel: 'Manual unlock operation',
parameters: undefined
}
2021-04-09 16:29:29.246 INFO ZWAVE: Node 5: value updated: 98-0-currentMode 0 => 255
2021-04-09 16:29:29.248 INFO ZWAVE: Node 5: value updated: 98-0-outsideHandlesCanOpenDoor true,false,false,false => false,false,false,false
2021-04-09 16:29:29.251 INFO ZWAVE: Node 5: value updated: 98-0-insideHandlesCanOpenDoor false,false,false,false => false,false,false,false
2021-04-09 16:29:29.253 INFO ZWAVE: Node 5: value updated: 98-0-latchStatus open => open
2021-04-09 16:29:29.254 INFO ZWAVE: Node 5: value updated: 98-0-boltStatus unlocked => locked
2021-04-09 16:29:29.256 INFO ZWAVE: Node 5: value updated: 98-0-doorStatus open => closed
2021-04-09 16:29:29.257 INFO ZWAVE: Node 5: value updated: 98-0-lockTimeout 0 => 0
2021-04-09 16:29:29.384 INFO ZWAVE: Node 5: value updated: 98-0-currentMode 255 => 255
2021-04-09 16:29:29.386 INFO ZWAVE: Node 5 CC Notification {
type: 6,
event: 1,
label: 'Access Control',
eventLabel: 'Manual lock operation',
parameters: undefined
}
2021-04-09 16:30:09.348 INFO ZWAVE: Calling api refreshValues with args: [ 5, [length]: 1 ]
2021-04-09 16:30:15.321 INFO ZWAVE: Node 5: value updated: 98-0-operationType 1 => 1
2021-04-09 16:30:15.326 INFO ZWAVE: Node 5: value updated: 98-0-outsideHandlesCanOpenDoorConfiguration false,false,false,false => false,false,false,false
2021-04-09 16:30:15.332 INFO ZWAVE: Node 5: value updated: 98-0-insideHandlesCanOpenDoorConfiguration false,false,false,false => false,false,false,false
2021-04-09 16:30:47.294 INFO ZWAVE: Success zwave api call refreshValues { success: true, message: 'Success zwave api call', result: undefined }

I've noticed the lock showed as dead, but sending an unlock  command from HA worked, I then manually locked it. upon checking my screen again, it was shown as alive...

Duplicate of https://github.com/zwave-js/zwavejs2mqtt/issues/1023

Doing this was the entire point of https://github.com/zwave-js/node-zwave-js/issues/1713 / https://github.com/zwave-js/node-zwave-js/pull/2045 which you raised btw @lolouk44 🤷🏻♂

Sorry @AlCalzone I've been away for a week.
I understand we were trying to map notifications as per https://github.com/zwave-js/node-zwave-js/issues/1713, but if this is done where is that data now available?
Sorry if this is a dumb question, I've not found anywhere where this would be...

Hi,
I own the conexis L1 and recently moved over to z-wavejs the issue I noticed was the alarm level only updated when the value changed (when another key was used)
with the original zwave integration it would update every time regardless is the value was the same.
could this be apart of the above issues and potentially fixed?
currently non of my HA automations will work if I was the last person to unlock the door.

@lolouk44

but if this is done where is that data now available?

node-zwave-js emits a notification event. HA exposes that as the zwave_js_notification https://www.home-assistant.io/integrations/zwave_js/#node-events-notification
@phairplay If the value is mapped, you should get an event every time. Your use case seems like it should be supported for the devices where the mapping is not defined or possible. But that's a HA issue.

Hi,
I own the conexis L1 and recently moved over to z-wavejs the issue I noticed was the alarm level only updated when the value changed (when another key was used)
with the original zwave integration it would update every time regardless if the value was the same.
could this be apart of the above issues and potentially fixed?
currently non of my HA automations will work if I was the last person to unlock the door.

Your statement should be accurate for the zwave_js integration as well. The state trigger will not fire but you should be able to check the last_updated attribute on the entity to see when it was last changed. But that assumes that we are getting updates from zwave-js. My understanding is that either the value gets mapped to events, or you get state updates to the sensors, but not both.

@AlCalzone and @raman325
Thank you for thr response. I wasn't aware of notification, as it was always updated through the exposed sensor.
Thank you

Thanks @AlCalzone and @raman325 for your help.
I can indeed track the event in HA:
  - trigger:
      - platform: event
    event_type: zwave_js_notification

and retrieve data via trigger.event.data.event_label
but I can't seem to be able to create a template trigger as I get this:
Invalid config for [template]: [event_type] is an invalid option for [template]. Check: template->event_type.
(sorry this is turning into an HA thread)
Should this not already be exposed as an attribute instead of re-inventing the wheel?
Where can I raise a request this pls?

Thanks @AlCalzone and @raman325 for your help.
I can indeed track the event in HA:
  - trigger:
      - platform: event
    event_type: zwave_js_notification

and retrieve data via trigger.event.data.event_label
but I can't seem to be able to create a template trigger as I get this:
Invalid config for [template]: [event_type] is an invalid option for [template]. Check: template->event_type.
(sorry this is turning into an HA thread)
Should this not already be exposed as an attribute instead of re-inventing the wheel?
Where can I raise a request this pls?

You have an indentation problem. event_type needs to indented more to be right under platform. See here for examples: https://www.home-assistant.io/docs/automation/trigger/#event-trigger

Thanks @raman325
I had an issue with copy / paste, my sensor is working, the full definition being:
template:
  - trigger:
      - platform: event
        event_type: zwave_js_notification
    sensor:
      - name: "Yale Lock Status"
        state: "{{ trigger.event.data.event_label }}"

My point / question is: why is this not exposed as an attribute?
I'm "happy" creating it but it's not really an "average joe" friendly if you see what I mean
Happy to close this thread and open a new one elsewhere, if someone can point me in the right direction?
Also this will probably not help @phairplay since we can't define who unlock the door since the full parameters are no longer updated...

What I don't understand is why when I use an RF Tag to unlock the door, it works, so I can tell who is unlocking the door I.e. what tag was used, but it doesn't update at any other time e.g. manually locking or unlocking or unlocking via HA...

What I don't understand is why when I use an RF Tag to unlock the door, it works (the alarm level updates), so I can tell who is unlocking the door I.e. what tag was used, but it doesn't update at any other time e.g. manually locking or unlocking or unlocking via HA...

That's probably because "who unlocked the door" is not a standard status so can't be mapped on standard notifications.
if it helps, this is my actual yale sensor:
- platform: template
  sensors:
    porch_door_lock_status:
      value_template: >
        {% if states.sensor.yale_status.last_updated > states.sensor.yale_conexis_l1_alarmtype.last_updated %}
        {{states("sensor.yale_status")}}
        {% else %}
        {% if states("sensor.yale_conexis_l1_alarmtype") == "144" and states("sensor.yale_conexis_l1_alarmlevel") == "2" %}Unlocked by RFID tag (Div)
        {% elif states("sensor.yale_conexis_l1_alarmtype") == "144" and states("sensor.yale_conexis_l1_alarmlevel") == "3" %}Unlocked by RFID tag (Lolo)
        {% elif states("sensor.yale_conexis_l1_alarmtype") == "16"  %}Unlocked by Yale App
        {% else %} {{states("lock.yale_conexis_l1_lock")}} (not caught)
        {% endif %}
        {% endif %}
      friendly_name: Porch Door Lock Status
      icon_template: 'mdi:lock-clock'

sensor.yale_status is picked up from the template above
It still doesn't seem to pick up unlocked by RF as no event appears to be sent when unlocking via HA lock.unlock... Not sure if it's me...
It still means it's not a sensor that's easily set up 😞

@lolouk44

That's probably because "who unlocked the door" is not a standard status so can't be mapped on standard notifications.

That's the question... The specs don't distinguish between keypad unlock and RFID unlock (or rather there's no event for RFID). It seems however that your lock doesn't even have a keypad, so we could map 144 to "keypad unlock" instead. Which would also mean that you would be getting the user ID as part of the event.
To be honest, the original issue got pretty confusing. Do you have a link to a manual that explains the alarm values, so we can update the mapping for the L1?
/cc @blhoward2


iOS 9.2 creates audio issue

After the release of iOS 9.2 the web audio frame work has a bug on this platform related to the sample rate being set incorrectly when the buffer is created.  For more information see this post:
http://www.html5gamedevs.com/topic/17326-ios-9-webaudio-issues/
The symptoms are audio playing to slow, or distorted because the sample rate is set to 48000 by default in iOS.
When you create the buffer you need to be able to set the default sample rate with creteBuffer.  Something like this:
  var buffer = g_WebAudioContext.createBuffer(1, 1, 44100);
  var source = g_WebAudioContext.createBufferSource();
  source.buffer = buffer;

So, could we talk about a patch too pass in the sample rate when you are creating the buffer in the options?

As part of my research (and a more eloquent explanation), this issues has a solution I have found here
https://github.com/Jam3/ios-safe-audio-context
It is the source of the sample code I provided above.

Thanks, i'll have a look at it

I also tried my hand at adding new code
I added a 'desiredSampleRate' to the options object in function WebAudio()
if (options.desiredSampleRate === undefined) options.desiredSampleRate = 44100;
And then tried to re-write the buffer, as in the example code
    req.onload = function() {
      if (req.status !== 200 && audioBuffers[src]) return req.onerror();
      audioCtx.decodeAudioData(req.response, function(buffer) {
        if (desiredSampleRate != buffer.sampleRate && /(iPhone|iPad)/i.test(navigator.userAgent)) {
          var newBuffer = audioCtx.createBuffer(buffer.numberOfChannels, buffer.length, desiredSampleRate);
          audioBuffers[src] = newBuffer;
        }else {
          audioBuffers[src] = buffer;
        }

        // Fire onBuffered event
        var handlers = eventHandlers[src].buffered;
        if (handlers) {
          // We can safely clean up all onBuffered event handlers, as once the
          // src media is cached, the onBuffered event can be fired immediately
          // for any new audio objects that are created henceforth
          eventHandlers[src].buffered = null;

          for (var i = 0; i < handlers.length; i++) {
            deferredApply(handlers[i]);
          }
        }
      });
    };

but with this code, my tracks play but with no audio.
But I thought: Hey, maybe I should share this anyway.

Can't reproduce it on iOS 9.1, will update my device over the weekend to test this

Tried using the test.mp3 -> 32k and 44.1k re-sampled using Audacity on iOS (9.2.1) seems to work fine. Can you share the audio that's causing issues? And have you tried the latest iOS update ?
Also I just pushed an update which should help with iOS playing issues, give that a shot to see if it fixes the media not playing issue.

Reviewing this code, line 124 throws a can't set read only property error:
self.audioSrc.onended = null;

Nice catch, thanks. e446e6235d59e03766764f8d274c606d4ee80765 should fix it

Error: Failed to execute 'stop' on 'AudioBufferSourceNode': cannot call stop without calling start first.
at Error (native)
at WebAudio.stop (http://localhost:8100/lib/ngWebAudio/src/ngWebAudio.js:127:47)

Thanks! Sigh... WebAudio API is just horrible to work with. Pushed a new commit that should address this new bug hopefully fca6b1f12aee348678a3d01624a173205c1bcebe. Thanks for hanging in there!

Thanks, error gone. The only issue I see now is that
onEnd
seems to get fired on "stop" as well as being at the end of the file. Thanks for your help, much appreciated!

That was the original intention of onEnd (e.g the event fires when ever the media "stops"; was a bug that prevent this, which I fixed along with other changes), however might refactor this into two events, onStop and onEnd? Let me know if you think it would be useful.

I think both are valuable. Detecting a finish of play is a unique event in
many ways.
On Tuesday, February 2, 2016, Zhen Wang<EMAIL_ADDRESS>wrote:

That was the original intention of onEnd (e.g the event fires when ever
the media "stops"; was a bug that prevent this, which I fixed along with
other changes), however might refactor this into two events, onStop and
onEnd? Let me know if you think it would be useful.
—
Reply to this email directly or view it on GitHub
https://github.com/nehz/ngWebAudio/issues/2#issuecomment-178954777.

--
Christopher Schaub
http://chris.schaub.com

I've pushed an update for it created a separate issue to keep track of any regressions or unintuitive behaviours #7

Thanks, seems much better. Testing.

Ok, there is still a distortion bug on iOS 9.2. The issue is compounded by the fact that copyToChannel is not available on iOS right now, even with the webkit prefix. This gets at the issue:
https://github.com/Jam3/ios-safe-audio-context/blob/master/index.js

For some reason I can't seem to reproduce the distortion. Is it possible to get a copy your audio (and maybe code if possible), else I could just use the code linked. However I would like to avoid using desiredSampleRate as a workaround as ideally you shouldn't need to worry about it and the API shoud just "work"

The issue is only on iOS 9.2, on a device. You won't hear it on desktop
safari or in chrome, unfortunately.
On Thu, Feb 4, 2016 at 7:07 AM, Zhen Wang<EMAIL_ADDRESS>wrote:

For some reason I can't seem to reproduce the distortion. Is it possible
to get a copy your audio (and maybe code if possible), else I could just
use the code linked. However I would like to avoid using desiredSampleRate
as a workaround as ideally you shouldn't need to worry about it and the API
shoud just "work"
—
Reply to this email directly or view it on GitHub
https://github.com/nehz/ngWebAudio/issues/2#issuecomment-179823673.

--
Christopher Schaub
http://chris.schaub.com

Weird my iPad2 (32bit 9.2.1) does not seem reproduce this, however I can see it on the iOS simulators (9.2). So it could either be 32bit vs 64bit, or iOS version difference. Do you happen to have any devices with 9.2.1 or 9.3 to see if the bug is still there?
I've tried replicating the safe context code both at init time (current implementation) and inside a touchend handler (wrangled all the init code and put it into play just for testing), and it does not seem to fix it on the simulator. I'll keep trying to poke at it. Really wish Apple would stop breaking things haha..

I have a theory. A bunch of the WebAudio webkit functions are not present as of ios 9.2. I wonder if we should be trying just the non prefixed WebAudio api as of 9.2, maybe apple merged them into ios safari?

Ok, tested. If you use change this line:
var AudioContext = window.AudioContext || window.webkitAudioContext;

to
var AudioContext = window.AudioContext;

you will notice distortion is gone.

That's because it will fallback to html5 audio (AudioContext == undefined).
Did the ios-safe-context code work for you?
Might just add it to the code, or allow a way for users to replace the AudioContext (WebAudio.setContext()), so you use the ios-safe-context lib as a dependency.
Wish Apple had a issue tracking this bug ...

Ahh, ok, I havent tried that yet. But would test if you did.
On Friday, February 5, 2016, Zhen Wang<EMAIL_ADDRESS>wrote:

That's because it will fallback to html5 audio (AudioContext == undefined).
Did the ios-safe-context code work for you?
Might just add it to the code, or allow a way for users to replace the
AudioContext (WebAudio.setContext()), so you use the ios-safe-context lib
as a dependency.
Wish Apple had a issue tracking this bug ...
—
Reply to this email directly or view it on GitHub
https://github.com/nehz/ngWebAudio/issues/2#issuecomment-180651018.

--
Christopher Schaub
http://chris.schaub.com

It didn't work for me, but maybe because I only can test the bug with iOS simulators

Yes, you don't see the bug on emulators or simulators. On an actual iPhone 6, iOS 9.2, the audio context sample rate gets set to 48800 -- even though the audio file is clearly 44100! I'm working on something like this ...
 audioCtx.decodeAudioData(req.response, function(buffer) {
        // ios as of 9.2 needs this to fix distortion - maybe just compare rates
        if (/(iPhone|iPad)/i.test(navigator.userAgent)) {
          var newBuffer = audioCtx.createBuffer(buffer.numberOfChannels, 1, desiredSampleRate);
          for (var j = 0; j < buffer.numberOfChannels; j++) {
            newBuffer.getChannelData(j).set(buffer.getChannelData(j), 0);
          }
          audioBuffers[src] = newBuffer;
        } else {
          audioBuffers[src] = buffer;
        }

I can create a new buffer at 44100, but when i copy the data in, the player is still somehow thinking it's 48800.

Sorry, bug ....
audioCtx.decodeAudioData(req.response, function(buffer) {
        // ios as of 9.2 needs this to fix distortion - maybe just compare rates
        if (/(iPhone|iPad)/i.test(navigator.userAgent)) {
          var newBuffer = audioCtx.createBuffer(buffer.numberOfChannels, buffer.length, desiredSampleRate);
          for (var j = 0; j < buffer.numberOfChannels; j++) {
            newBuffer.getChannelData(j).set(buffer.getChannelData(j), 0);
          }
          audioBuffers[src] = newBuffer;
        } else {
          audioBuffers[src] = buffer;
        }

https://github.com/nehz/ngWebAudio/pull/8

decodeAudioData() should automatically resample to the device's sample rate (using audio ctx) (https://developer.mozilla.org/en-US/docs/Web/API/AudioContext/decodeAudioData)
I think the fix will be setting up the audio context correctly


Alias this for classes is deprecated in DMD 2.103

Using alias this for classes has not been clearly specified and the lookup rules in such circumstances are not defined. As a consequence, various failures or crashes may appear when alias this is used in conjunction with classes. Starting with this release, alias this for classes is being deprecated.

https://dlang.org/changelog/2.103.0.html

Closing in favour of #112


Near-All Functionality Broken on Mac

All Brackets functionality is broken on Mac, except for minimize. Brackets must be force quit to close. I've re-downloaded the most recent version of Brackets but the issue persists.
Files can be dragged into Brackets and edited, but not saved.
A screen recording of the issue can be viewed here.
No issues with Brackets prior to this issue.
macOS 10.15.4
Brackets 1.14.2

Hi,
Thanks for reporting us.
We tried to recreate this issue on MacOS 10.15.4, but issue is not reproducible at our end.
Tried on multiple machines.
Will let you know once we encounter the same.
Thanks!

Hi,
Thanks for reporting us.
We tried to recreate this issue on MacOS 10.15.4, but issue is not reproducible at our end.
Tried on multiple machines.
Could you please follow below steps and let us know if issue still persist.

Clear preferences, go to Folder (~/Library/Application Support)
Delete Brackets folder and then relaunch Brackets.
Also try after restarting the machine.

Thanks!

Deleting the Brackets folder within Application Support resolved the issue for me—thank you!


docs: guide on FaissPSQL

@maximilianwerk Yes, good point. I just always forget and it's not enforced by any linter.


Improve each module's code coverage to at least 90%

Update unit tests for download.py, repository_lib.py, repository_tool.py, and updater.py.
Re-generate repository data so that compressed metadata may be tested.

Coverage remained the same at 95.81% when pulling b7cf7e658859a14ae4f12bcf876c456bd72a37a7 on vladimir-v-diaz:code_coverage into 0b62fb59062f62eb6fe67be5bb01e1d0253838e4 on theupdateframework:develop.


feat: add required vars support

closes #29

@Andrew-Morozko any feedback on the approach here, before Anas dives deeper?

#231 is merged!


InputDatePickerFormField should implement textInputAction

Use case
I'm actually creating a Form with multiples inputs. I can easily pass from one to another with the textInputAction: TextInputAction.next,. But the InputDatePickerFormField that is in the middle of the form break the chain because the option doesn't exist.
Proposal
Add a property similar to textInputAction to be able to focus on the next field without hiding the keyboard.

If the issue get proposal label then I would like to work on this issue

Hi @oom-  @yash1200
I see there's an open issue addressing the case you described. https://github.com/flutter/flutter/issues/68057
Please follow up on that issue, I'm closing the current one as a duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you


Modify if statement block to include 'collection_language' and 'collection_id' in extract_base_layer.py

Function currently:
if u == 'copyright_status':
                # looks for attribute value and handles exception if no attribute
                results = bs_object.select(exp)
                try:
                    field_data += results[0]['copyright.status']
                except:
                    field_data += ''

Added 'collection _language'.

Closing as 'collection_id' exception is dependent on resolution of Issue #22.


CORS headers not set for all error responses

For some API error responses e.g. 413 Payload Too Large CORS headers are not set making it impossible for browser clients to properly detect and handle the error.

In order to focus our efforts on issues that are currently creating difficulty for the community we are closing tickets that were created prior to the EOSIO 2.0 release.  If you believe this issue is still relevant please feel free to reopen it or create a new one.  Thank you for your continued support of EOSIO!

I'm unable to re-open but this is still an issue

Please try the latest version (2.1) and let us know if you're experiencing this issue


Add dart support

I wasn't aware of how the test files are used (looked like manual test files so that is what I used).
Let me know if I am missing something!

Thanks!


Problem with root parameter

root option work only for first windows (because in changed manually with cd command).
the second window doesn't ignore change directory option.
Looks like a problem in my tmux. It has -c option, but ignore it (I tried to run command manually, and directory has not changed).
...
$ tmux new-window -c /vagrant/api/pr2 -t pr:1 -n api/pr2
$ tmux new-window --help
usage: new-window [-adkP] [-c start-directory] [-F format] [-n window-name] [-t target-window] [command]

Perhaps we can stop using -c parameter for non-first windows, and change directory manually (with cd command), the same as in a first window?
I have tmuxinator from master branch, ubuntu 14.04, tmux 2.0 (also tried 1.8).
My ~/.tmux.conf empty now
My .tmuxinator.yml looks like:
name: project
root: /vagrant

windows:
  - api/pr1:
      root: /vagrant/api/pr1
      panes:
        - rails s

  - api/pr2:
      root: /vagrant/api/pr2
      pre: nvm use v0.10.25
      panes:
        - node main

My tmuxinator project.render looks like:
#!/bin/zsh

# Clear rbenv variables before starting tmux
unset RBENV_VERSION
unset RBENV_DIR

tmux start-server;

  cd /vagrant

  # Run pre command.

  # Create the session and the first window. Manually switch to root
  # directory if required to support tmux < 1.9
  TMUX= tmux new-session -d -s pr -n api/pr1
  tmux send-keys -t pr:0 cd\ /vagrant/api/pr1 C-m

  # Create other windows.
  tmux new-window -c /vagrant/api/pr2 -t pr:1 -n api/pr2

  # Window "api/pr2"
  tmux send-keys -t pr:0.0 rails\ s C-m

  tmux select-layout -t pr:0 tiled

  tmux select-layout -t pr:0
  tmux select-pane -t pr:0.0

  # Window "api/pr2"
  tmux send-keys -t pr:1.0 nvm\ use\ v0.10.25 C-m
  tmux send-keys -t pr:1.0 node\ main C-m

  tmux select-layout -t pr:1 tiled

  tmux select-layout -t pr:1
  tmux select-pane -t pr:1.0

  tmux select-window -t 0
  tmux select-pane -t 0

  if [ -z "$TMUX" ]; then
    tmux -u attach-session -t pr
  else
    tmux -u switch-client -t pr
  fi

@randaalex I believe this is actually an issue with the whitespace in your YAML.
Try using the following (note the four space indentation before the panel config entries):
# ~/.tmuxinator/wat.yml
name: project
root: /vagrant

windows:
  - api/pr1:
      root: /vagrant/api/pr1
      panes:
        - rails s

  - api/pr2:
      root: /vagrant/api/pr1
      pre: nvm use v6.9.0
      panes:
        - node main

This has come up in the past and I'm not sure what the root cause is - your config is valid YAML - but it's worth investigating further. I will create a separate issue for this. /cc @adamstrickland
Anyways, please close this issue if the above solves your problem. Apologies for the inconvenience.

@ethagnawl I sure YAML config totally valid.
Because I get correct commands from project.render call (I used binding.pry for getting this value):
...
 # Create other windows.
  tmux new-window -c /vagrant/api/pr2 -t pr:1 -n api/pr2

I guess I have problem in my tmux. And my idea: refactor new-window dir changing from -c new/path to tmux send-keys -t pr:0 cd\ /new/path C-m. This let to remove a code for change workflow for old tmux versions (https://github.com/tmuxinator/tmuxinator/blob/master/lib/tmuxinator/assets/template.erb#L15-L25)

@randaalex You're right - it's not a whitespace issue. Though, it appears to be an issue which only affects tmux > 2.0. (I was able to reproduce it using tmux 2.3.)

So, the following setup:
name: test
root: /Users/josefson/
windows:
  - editor:
      layout: main-vertical
      panes:
        - vim
        - top
  - server:
      panes:
          - echo 'server'
          - echo 'server 1'
  - logs:
      panes:
          - echo 'logs'
          - echo 'logs 1'

Outputs me the following and only opens the 1st window
**invalid option: default-path**
can't find pane 2
can't find window 2
can't find window 2
can't find window 2
can't find window 2
can't find window 2
can't find window 2
can't find window 3
can't find window 3
can't find window 3
can't find window 3
can't find window 3
can't find window 3

Although it does cd into the :root folder, it does not create the other windows. If i comment the :root statement however, it opens every window, but i lose the ability to specify a :root folder.
Does anybody knows any work around it?

FWIW I don't have root in any of my windows or panes:
# ~/.tmuxinator/eve.yml
name: eve
root: /Users/bennyp/Documents/elastic/elastic-video-editor

windows:
  - editor:
      layout: main-vertical
      panes:
        - vim
        - yarn start
        - git lg

Any further info on this?

I'm having the same issue, is there a known workaround?

+1


.Net: Azure OpenAI deployment not found for hard-coded chat model in notebook

Describe the bug
API deployed model does not exist.
To Reproduce
Steps to reproduce the behavior:

Go to '/dotnet/notebooks' and set configuration json to use Azure OpenAI.
In your Azure OpenAI account, deploy a custom named embeddings model (not the same name as the model) and a custom named chat model (not the same name as model)
Go to '/dotnet/notebooks/07-DALL-E-2.ipynb'
Run sections through "Generate a random image"
See error

Expected behavior
Section should run without error. Code is using an assumed embeddings and chat deployment models for Azure OpenAI, when user could create own deployment with different name. Solution likely to include deployed model names in the config file, and change code in notebook to reference that model, instead of a hard-coded assumed model.
Screenshots

Additional context
Similar issue encountered in Issue #2642

The issue you're describing appears to involve difficulties with an Azure OpenAI deployment not being found for a hard-coded chat model in a notebook. To address this issue, you can follow these general steps, but note that specific code and configurations will depend on your Azure setup and notebook environment:
Issue Explanation:
The problem is that the Azure OpenAI deployment is not being found for a hard-coded chat model in your notebook. This could be due to various reasons, such as incorrect configurations or permissions.
Solution:
Verify Azure Setup:
Ensure that your Azure environment is correctly configured with the necessary resources and permissions to access the OpenAI deployment.
Check Deployment Name:
Verify that the hard-coded deployment name in your notebook matches the actual name of the deployment in Azure. Ensure there are no typos or discrepancies.
Authentication and Authorization:
Ensure that you have the correct authentication and authorization mechanisms in place to access the Azure resources. This may involve using Azure service principals, tokens, or other authentication methods.
Resource Group and Location:
Confirm that the Azure resources related to your OpenAI deployment are in the correct resource group and location.
Network Configuration:
Check if there are any network or firewall restrictions that might be preventing your notebook from accessing the Azure resources.
Error Handling:
Implement robust error handling in your notebook code to capture and log any errors or exceptions that occur during the deployment lookup process.
Test with Azure SDK:
Consider using the Azure SDK for .NET to programmatically interact with your Azure resources, including the OpenAI deployment. This can help you ensure that the deployment is accessible and correctly configured.

The "Azure Environment" includes Azure configurations, permissions, and network settings.
The "Notebook Environment" represents your notebook's code and configurations, including the hard-coded deployment name, authentication, and error handling.
The "Azure SDK for .NET" can be used to programmatically access Azure resources and validate the deployment's accessibility.

This can be fixed by changing the code in the kernel setup code block to use builder.WithOpenAIChatCompletionService instead of WithOpenAITextCompletionService.

Closing so we can create new docs issues later.


[e2e]: generic workflow_dispatch branch1 default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/branch1
Run: https://github.com/slsa-framework/example-package/actions/runs/8353230883
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.branch1.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.branch1.default.slsa3.yml
Trigger: workflow_dispatch
Branch: branch1
Date: Wed Mar 20 03:06:20 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/8366221135
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.branch1.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.branch1.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Wed Mar 20 21:45:49 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/branch1
Run: https://github.com/slsa-framework/example-package/actions/runs/8369046994
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.branch1.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.branch1.default.slsa3.yml
Trigger: workflow_dispatch
Branch: branch1
Date: Thu Mar 21 03:07:51 UTC 2024
Tests are passing now. Closing this issue.


DX Improvement: Adds Husky for automatic linting/formatting/typechecking on commits

Currently you have to pnpm run pre-commit manually before commits, which can be easy to forget and is one extra step. This PR adds Husky, which is a tool that automatically runs before every git commit (or whatever other git commands you would like it to run before),
In this case, I automated the pnpm run pre-commit step, and removed the comment from the README requiring the user to run that before every commit.

Hey Paul, sorry for the mistake, the doesn't-apply-formats-to-commit issue is now fixed (I was forgetting to re-stage the format changes in husky's pre-commit file).
Regarding failing, it's running the pre-commit script in package.json, so it fails on TS errors (not on warnings -- this can be configured but I thought it's probably too strict) and eslint errors, but not on prettier errors, which it just auto-formats, which makes the most sense to me from an optimal DX perspective (it's probably very rare that a dev would want to see how prettier is formatting and manually run the format script instead of just having it do it for them), but that's just my opinion; happy to change it to pnpm run check-format if you'd prefer.
Also updated the README to reflect the new changes and tell the user how to escape commit prison.
Let me know if there's anything else I need to do before merging!


Fix width of text input in split expense page

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/41766
PROPOSAL: https://github.com/Expensify/App/issues/41766#issuecomment-2107490742
Tests

Go to New Expensify.
Go to FAB > Split expense.
Enter amount > Next.
Select a few users > Next.
Verify that split amount input is fully visible.

[ ] Verify that no errors appear in the JS console

Offline tests

Go to New Expensify.
Go to FAB > Split expense.
Enter amount > Next.
Select a few users > Next.
Verify that split amount input is fully visible.

QA Steps

Go to New Expensify.
Go to FAB > Split expense.
Enter amount > Next.
Select a few users > Next.
Verify that split amount input is fully visible.

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/Expensify/App/assets/57348009/8fc40c60-5aab-4e23-83ff-0e7b1ac47cb3

Android: mWeb Chrome

https://github.com/Expensify/App/assets/57348009/65f97a45-862d-435a-8aa6-eb9efe242277

iOS: Native

https://github.com/Expensify/App/assets/57348009/100dbebe-f3ba-45b9-a2f5-1158d38b00ce

iOS: mWeb Safari

https://github.com/Expensify/App/assets/57348009/e687ad23-8d3a-4405-a55a-a142ed00a907

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/57348009/c5e72621-3046-4765-9d11-384fba899bb5

MacOS: Desktop

https://github.com/Expensify/App/assets/57348009/6e07c093-4fc8-4e60-89cc-9a47e5b94a52

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/chiragsalian in version: 1.4.76-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/Beamanator in version: 1.4.76-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Made the place to insert the element configurable

Added two new options for defining where to the tipsy tooltip should be added in the dom tree.
One option, insertTo, defines which element it should be inserted in relation to and the other new option, insertMethod, defines how to add the tooltip.
Code has been added to adjust the position of the tooltip according to the position of the tooltip elements offsetParent - this makes sure that it's always positioned in the right place no matter where in the dom tree it is inserted.
An advantage of adding the tooltip deeper into the dom tree is that it may move together with that content if eg. the position of a common container was to be changed - which eg. happens on a centered site whenever the browser is resized.

Cleaning up my open PR:s, closing as this one has been open for +12 years


Fix width (at 14 2/9 units) by default

List of Changes

Changed the default fixed-height to fixed-width

Motivation

It appears that the change from 3b1b's fixed-width by default to a fixed-height may have been incidental. To clarify, at a 16:9 aspect ratio, the frame is 14 2/9 by 8. If a square video is made, for example, 3b1b will produce a frame that's 14 2/9 by 14 2/9, whereas Manim Community will produce a frame that's 8 by 8.

Further Comments

I have the point illustrated in issue 275
I didn't find anything indicating that this was done intentionally, hence this pull request, but perhaps someone else can confirm/deny this inference?

Acknowledgement

[x] I have read the Contributing Guidelines

Thanks for flagging the formatting issue, @behackl. I was accustomed to ignoring those, but I see now the issue with Black's formatting complaints is fixed.
I considered documenting the change in the changelog, but I figured since this reverts the feature back to its original behaviour, it wasn't warranted. Thoughts?

Thanks for your contribution! I haven't looked at the code yet, I'll log on later, but let me ask a question real quick: does this change the default frame size? Or does it just fix the width instead of the height?

does this change the default frame size? Or does it just fix the width instead of the height?

It doesn't affect the resolution of the video, just alters the coordinate system. Without specifying a non-16:9 apsect ratio, there's no difference. But with a different aspect ratio, the width is fixed at 14 2/9 (instead of the height being fixed at 8), which matches 3b1b's default behaviour.
I rendered a couple square videos to illustrate the point here. (The one labeled Manim Community was rendered before the change in this PR.)

I don't feel too strongly about this, but to me it is more natural to have the height fixed. Why? Because resolutions are usually specified as 480p, 720p, 1080p, which refer to the height, not the width. These are used regardless of the aspect ratio, so 480p and 4:3 aspect ratio gives a width of 640p etc.
Like I said, I don't know who/when this change was made but we should just pick one and stick with it instead of changing back and forth.

Fair enough. My experience with tools that use a fixed width (specifically Quartz Composer, then Vuo) has obviously contributed to my preference for this behaviour, but I remember finding it really intuitive all those years ago.
My motivation with this PR was to bring this feature back in line with the 3b1b's original behaviour, and with the behaviour that I've come to assume is standard. As I mentioned, I couldn't find anything to indicate that this was an intentional change, so it feels like the right move.

@PhilipEnchin yeah, I'm all for following a standard... it's just that at this point it's unclear whether there is a standard at all ;)
Also, the topic of being in line with 3b1b is a delicate one. The point of this fork is to respond to the community's needs, and that necessarily means that we will depart from 3b1b in some aspects. However, I'm not sure this is one aspect that we want to depart from.... :thinking: :thinking: I'm undecided about this issue. We need the opinion of more members of the community here.

@leotrs Thanks! (As of today, you'll find me on discord. See you there.)

Unfortunately, I think this is not a priority right now and there is no consensus on whether the width or the height should be fixed. I vote we close this for now.

Unfortunately, I think this is not a priority right now and there is no consensus on whether the width or the height should be fixed. I vote we close this for now.

+1. Over the past few weeks, I've also gradually changed my mind when it comes to what seems more natural to me; I'm now more comfortable with having a fixed height.

@PhilipEnchin Sorry this didn't work out. We very much look forward to your future contributions :)


[NativeSql] Bring back ColumnarSort kernel

multiple columns sort supported, changed to use internal row as output.
Now wip to verifying codes with all tpch queries
Signed-off-by: Chendi Xue<EMAIL_ADDRESS>

All tpch quries are runable, while some queries produces incorrect result.
Correct queries are Q1, Q2, Q4, Q6, Q11, Q14, Q15, Q16, Q17, Q19, Q20, Q22
Incorrect queries are Q3, Q5, Q7, Q8, Q9, Q10, Q12, Q13, Q18, Q21
Need to look into this later


Fixed the vue-apollo example

Codecov Report

Merging #1161 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #1161   +/-   ##
=======================================
  Coverage   98.67%   98.67%
=======================================
  Files          12       12
  Lines         679      679
=======================================
  Hits          670      670
  Misses          9        9

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 e00c00d...34f16cf. Read the comment docs.

Hi @vuchl
I worked with Akryum (author of vue-apollo), and the example has been updated and fully working: https://github.com/nuxt/nuxt.js/tree/dev/examples/vue-apollo


ModuleNotFoundError in flush-memcached when installing separate Postgres server

As reported in #production help, PUPPET_CLASSES=zulip::base,zulip::postgres_appdb_tuned zulip-server-3.2/scripts/setup/install fails with
++ dirname /root/zulip-server-3.2/scripts/setup/postgres-init-db
+ /root/zulip-server-3.2/scripts/setup/flush-memcached
Traceback (most recent call last):
  File "/root/zulip-server-3.2/scripts/setup/flush-memcached", line 14, in <module>
    from zproject import settings
  File "/root/zulip-server-3.2/scripts/setup/../../zproject/settings.py", line 19, in <module>
    from .configured_settings import *  # noqa: F401,F403 isort: skip
  File "/root/zulip-server-3.2/scripts/setup/../../zproject/configured_settings.py", line 8, in <module>
    from .default_settings import *  # noqa: F401,F403 isort: skip
  File "/root/zulip-server-3.2/scripts/setup/../../zproject/default_settings.py", line 13, in <module>
    from .prod_settings import EXTERNAL_HOST, ZULIP_ADMINISTRATOR
ModuleNotFoundError: No module named 'zproject.prod_settings'

because the installer only sets up zproject/prod_settings.py for the zulip::app_frontend_base class. I assume this was introduced in #13239.

Additionally, if I comment out the flush-memcached line from postgres-init-db, it fails with
+ su zulip -c '/home/zulip/deployments/current/scripts/setup/initialize-database --quiet'
bash: /home/zulip/deployments/current/scripts/setup/initialize-database: No such file or directory

because /home/zulip/deployments is also only a thing for zulip::app_frontend_base.


[WIP] pynac: switch to python3

Motivation for this change
Getting the sage python3 build ready. Unfortunately switching pynac over to python3 fails:
checking for python3.7... (cached) /nix/store/gpnm7i19lpj8p43mjrdw03d0hjalmskl-python3-3.7.5/bin/python
checking for a version of Python >= '2.1.0'... yes
checking for the distutils Python package... yes
checking for Python include path... -I/nix/store/gpnm7i19lpj8p43mjrdw03d0hjalmskl-python3-3.7.5/include/python3.7m
checking for Python library path... -L/nix/store/gpnm7i19lpj8p43mjrdw03d0hjalmskl-python3-3.7.5/lib -lpython3.7m
checking for Python site-packages path... /nix/store/gpnm7i19lpj8p43mjrdw03d0hjalmskl-python3-3.7.5/lib/python3.7/site-packages
checking python extra libraries... -lpthread -ldl -lcrypt -lncurses -lutil -lm
checking python extra linking flags... -Xlinker -export-dynamic
checking consistency of all components of python development environment... no
configure: error: in `/build/source':
configure: error:
  Could not link test program to Python. Maybe the main Python library has been
  installed in some non-standard library path. If so, pass it to configure,
  via the LDFLAGS environment variable.
  Example: ./configure LDFLAGS="-L/usr/non-standard-path/python/lib"
  ============================================================================
   ERROR!
   You probably have to install the development version of the Python package
   for your distribution.  The exact name of this package varies among them.
  ============================================================================

See `config.log' for more details

@FRidh is this package-specific weirdness or are there any differences between the python2 and python3 packages that I should be aware of?
Archlinux apparently was able to switch this package over without any changes: python2 to python3. I tried export PYTHON=python3 without success.
This issue might be relevant, but setting libdir=${python3}/lib didn't solve the issue.
Here is the relevant m4 file.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @

I have not seen this before with a Python package. Did notice something similar just now on staging-next with SDL_image.
Irrelevant, but note singular is listed twice.

pynac is not really a python package in the traditional sense. It's a C package that links to python.

Irrelevant, but note singular is listed twice.

Thanks, I'll fix that.

I had a look at the config.log. Turns out it actually failed because it was missing ncurses:
configure:4869: checking consistency of all components of python development environment
configure:4895: gcc -o conftest -g -O2  -I/nix/store/zdh16dcvjw99ybam59zd2ijb6bx138j0-python3-3.7.5/include/python3.7m  conftest.c  -L/nix/store/zdh16dcvjw99ybam59zd2ijb6bx138j0-python3-3.7.5/lib -lpython3.7m -Xlinker -export-dynamic -lpthread -ldl -lcrypt -lncurses -lutil -lm >&5
/nix/store/cg0k49h66nkdqx6ccwnqr0i4q0fnfznc-binutils-2.31.1/bin/ld: cannot find -lncurses
collect2: error: ld returned 1 exit status
configure:4895: $? = 1
configure: failed program was:
| /* confdefs.h */
| #define PACKAGE_NAME "pynac"
| #define PACKAGE_TARNAME "pynac"
| #define PACKAGE_VERSION "0.7.26"
| #define PACKAGE_STRING "pynac 0.7.26"
| #define PACKAGE_BUGREPORT<EMAIL_ADDRESS>| #define PACKAGE_URL ""
| #define PACKAGE "pynac"
| #define VERSION "0.7.26"
| #define HAVE_PYTHON "3.7"
| /* end confdefs.h.  */
|
| 		#include <Python.h>
| int
| main ()
| {
| Py_Initialize();
|   ;
|   return 0;
| }
|
configure:4912: result: no
configure:4916: error: in `/run/user/1000/tmp.o3V1EhrbOJ/source':
configure:4918: error:
  Could not link test program to Python. Maybe the main Python library has been
  installed in some non-standard library path. If so, pass it to configure,
  via the LDFLAGS environment variable.
  Example: ./configure LDFLAGS="-L/usr/non-standard-path/python/lib"
  ============================================================================
   ERROR!
   You probably have to install the development version of the Python package
   for your distribution.  The exact name of this package varies among them.
  ============================================================================

Adding it to the buildInputs fixes the issue. I'll include the pynac python3 switch with the sage switch in https://github.com/NixOS/nixpkgs/pull/70474.
Curious that it worked without the ncurses dep in python2 though. Should ncurses maybe be propagated from python3?

Should ncurses maybe be propagated from python3?

That would result in every package needing python to receive ncurses as build-time dependency. That's a bit too much.

Yeah but it somehow must have been included with python2 right? Otherwise I don't know why I didn't need to specify the dependency before. Or maybe it has something to do with
https://github.com/pynac/pynac/blob/85c43c230e8050a1f71898889a021fec28318502/acinclude.m4#L45
(the only mention of ncurses in pynac). I don't really understand what's going on there, since I'm not really fluent in all the autotools stuff.


poco: add apache support flag

Description of changes
This PR adds a toggle, enableApacheSupport, which enables poco's Apache support if toggled.
(Note: This PR doesn't trigger a rebuild because the inputs never technically change.)

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

I've lost motivation to continue work on this. I'll revisit it at a later date.


Identify modal map - zoom to encompass accuracy circles that are larger than default zoom

From: https://forum.inaturalist.org/t/change-identify-modal-mini-map-display-for-large-accuracies/8411
If an observation has a large accuracy circle, there is no indication of it on the Identify modal map, eg:

So if the circle is larger than the default map zoom, can we zoom out to encompass the entire circle?

Not working for this obs: https://www.inaturalist.org/observations/36844705

Possibly relevant: #675
Also @kueda originally changed it from map_scale to 5 over here: d992d86126bb7732638ecee70e95ed06a279d1df


Duplicate comment in interfaces and types

Search terms
tsd-parameters
tsd-comment
duplicate comments
interface comments duplicate
Expected Behavior
Interface should not duplicate comments
Actual Behavior
Duplicates comments in interface
Before 0.25.5

After 0.25.5

Steps to reproduce the bug
Duplicates are appearing only in interfaces and types after typedoc upgrade to 0.25.5
Environment

Typedoc version: 0.25.5
TypeScript version: 5.3.3
Node.js version: 18
OS: Any

Please provide a minimal reproduction. I can't run typedoc on Screenshots.

https://github.com/Kartanovicius/test-rep

Caused by 03426c3ddb2f322aec703a3358795c6e9a1849de, I handled signature checking one level deep, but didn't correctly handle comments attached to a type property on a reflection.
More minimal:
export interface InterceptorMethodAPI {
    /**
     * Duplicated
     */
    callback: () => Promise<any>;
}


Add live script

Fix #2
Fix #3

[ ] Add inputs for nu and theta - set the start, end and step size for both.

@zan8simone  I think this is now ready to merge(?)

Adding the directories to the MATLAB path isn't working as intended on Windows/from the live_scripts directory. Need to think of a more robust solution.

@zan8simone
This should now be fixed.

not indeed.. this error appears now

Hmm, I've not been able to reproduce that error. Could you try once more with the latest code (I've made a couple of extra commits) and a fresh MATLAB session?
I'm using R2023a rather than R2022b, but the tiledlayout function was introduced in 2019b so I doubt that is the problem.

Do you get an error if you execute tiledlayout("horizontal") from the MATLAB command window?

Actually tiledlayout("horizontal") works only from R2023a https://it.mathworks.com/help/matlab/ref/tiledlayout.html

Well spotted!
However, given the current recommendation is to use tiledlayout rather than subplot, and that the demo would run in MATLAB online in the current version, can we go ahead and merge this?


hi,I need you help,thanks

Hi,Can you move the setting button to the side bar.Some node cover it causing that I can't modify the setting.

In that case use Alt+G to open the settings!


Split App component into multiple smaller components

Also changes them to be functional components instead of class components
A few behaviors that I've changed:

When deleting a timetable that's not the active one, it now stays in that timetable instead of going to the first one
Deleting a timetable now shows an alert on success

Things under the hood that have been changed:

ShiftRef was deleted (didn't seem to be doing anything)
Derived state is no longer calculated on state update, but on render (and memoized). This makes it much simpler to update state

Almost done, just need to fix a few things I broke in the process!

I think this is done, and it has feature parity with the production version.
I've updated the top comment with some information.
I have a few more things that I want to change, but I'll open issues instead and do them on another PR, since this is huge already.
Consider squashing this instead of merging.

@all-contributors please add @diogotcorreia for code


Warning: failed loading cafile stream- BaltimoreCyberTrustRoot.crt.pem

On Using the PHP code as mentioned in the Document and enabling the SSL i get the above error  Warning: failed loading cafile stream.
I have an Azure PaaS PHP web app (7.4) trying to connect to Azure PaaS Mysql WebApp (8.x).
I have exactly copy pasted the PHP sample and tried , so i dont think i am doing any mistake at the application code level .
Can you please let me know the possible solution ?

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

ID: 77a94e6f-a799-494f-f2f9-171c1804e5f4
Version Independent ID: 0ca5e2ae-5889-61ef-3ce7-5c447f7519a6
Content: Configure SSL - Azure Database for MySQL
Content Source: articles/mysql/howto-configure-ssl.md
Service: mysql
GitHub Login: @savjani
Microsoft Alias: pariks

Can you share your msqli_ssl_set string in the code? Have you download BaltimoreCyberTrustRoot certificate file in /var/www/html location?

mysqli_ssl_set($conn,NULL,NULL, "/site/wwwroot/Connections/DigiCertGlobalRootG2.crt.pem", NULL, NULL);
My location is different as mentioned above.

Are you connecting to single server or flexible server?

Single server.

The article on which you have commented is for configuring ssl to single server and for that you need to download BaltimoreCyberTrustroot certificate https://docs.microsoft.com/en-us/azure/mysql/howto-configure-ssl

Yes i have tried with BaltimoreCyberTrustroot and it gives me the same error.
Also the doc says that the certificate will expire in Feb 21 and hence i tried with DigiCertGlobalRootG2 and none of these seems to work.

Can u use BaltimoreCyberTrustroot certificate and first connect using MySQL client. Does that work? Would you be able to open a support ticket from Azure portal and one of the support engineer can help assist you.

Yes i will be moving ahead in the direction of opening a support ticket in azure portal thanks.

@savjani Thanks for providing all the necessary information.
@JigneshRawal Please let us know if you have created the support ticket and we can move ahead and close the thread.

Hi @JigneshRawal - Please share the case no with me so I can make sure I follow-up on the ticket while we can close this thread.

Hi
i am a developer and the azure support access is with the Azure Admin team , hence it wont be possible for me to give you the case number.  But they have informed me that they have raised a case with Azure Support.
I think we can close this case but in case if i get a solution then i wish to post it here so that someone who stumbles on the same issue can get the solution here.

thanks @JigneshRawal .. Good thought. I will let @AnuragSharma-MSFT to comment if you can post the solution here but feel free to email us at<EMAIL_ADDRESS>if you are stuck or blocked.

@JigneshRawal Thanks for confirming to close the issue, but please feel free to post the solution once you get things start working again. Its really encouraging and it would help others facing similar issues.


Search: "Recent searches" does not consider different versions

It seems the "Recent searches" shows all the searches performed in the domain. However, in the same domain there could be different versions and even different projects when using subprojects.
We should save the project.slug and version.slug in the recent searches data to know if they match when restoring them from the cache.
How to reproduce:

go to https://test-builds.readthedocs.io/en/latest/
search for "configuration"
click on the result
go to a different version using the flyout
click on the search input from the flyout

At this point, you will see the "Recent search" you perform in a different version. If you click there, you will go to the other version.

Yeah, I did not consider this at all. On it.

I also find the styling of recent searches a bit confusing. I feel like maybe we lost the spinner as well when the search is currently ongoing, so when I do a new search, it's hard to understand that the search is happening and the recent search results aren't the current results. We should likely give them a slight background color or some other UX:

I also find the styling of recent searches a bit confusing. I feel like maybe we lost the spinner as well when the search is currently ongoing, so when I do a new search, it's hard to understand that the search is happening and the recent search results aren't the current results. We should likely give them a slight background color or some other UX:

Hmm, the spinner is indeed not spinning, and I'm not exactly sure why. I just spent 15min on it, and couldn't find anything obvious. Regarding your other comment, should we just hide the recent searches, when the current search starts happening?

Regarding your other comment, should we just hide the recent searches, when the current search starts happening?

That seems like a nice simple solution :+1:

@ericholscher I implemented this in https://github.com/readthedocs/addons/pull/353 . I think it works fine, but it's a bit weird when changing the search query - the results are removed when we change the query, and I'm not sure if recent searches should remain hidden in this case or not. Any preferences?

Not really sure, tbh. Not sure there's a "correct" way, so whatever we do for now is fine, and see if we get any feedback on it.


[SP-5560][PDI-18528] Fix premature closing of xml inputstream on mult…

…ithread executions

:warning: Build finished in 42m 56s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core,engine

:no_entry: Failed Tests
:cloud_with_lightning_and_rain: 2 test(s) failed:
org.pentaho.di.trans.steps.fileinput.text.TextFileInputTest.testErrorHandlerLineNumber (click to expand)
Could not delete "ram:///pdi-2607.txt".

org.pentaho.di.trans.steps.combinationlookup.CombinationLookupTest.testCombiInsert (click to expand)
${result.errorDetails}

Tests run: 4741, Failures: 2, Skipped: 7    Test Results

:information_source:  This is an automatic message

:warning: Build finished in 48m 15s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core,engine

:no_entry: Failed Tests
:cloud_with_lightning_and_rain: 2 test(s) failed:
org.pentaho.di.trans.steps.fileinput.text.TextFileInputTest.testErrorHandlerLineNumber (click to expand)
Could not delete "ram:///pdi-2607.txt".

org.pentaho.di.trans.steps.combinationlookup.CombinationLookupTest.testCombiInsert (click to expand)
${result.errorDetails}

Tests run: 4741, Failures: 2, Skipped: 7    Test Results

:information_source:  This is an automatic message


allproxies.txt

It would be nice to be able to save http, socks4 and socks5 into one file allproxies.txt

Hi.
I don't see any practical point in this.
This file will take up more space than three separate files because a protocol must be specified for each proxy.