text:
Command Error: Unknown Configuration Options and ParseException on ESP32 Platform

Bug description
When running pio run -v, I get the following error:
Warning! Ignore unknown configuration option `rust_lib` in section [env]
Warning! Ignore unknown configuration option `rust_target` in section [env]
Processing debug (build_type: debug; extra_scripts: pre:platformio.git.py, pre:platformio.patch.py, platformio.cargo.py; board: esp32dev; platform: espressif32; framework: espidf; rust_lib: led; rust_target: xtensa-esp32-espidf)
---------------------------------------------------------------------------------------------------------------------------------------------------------------
CONFIGURATION: https://docs.platformio.org/page/boards/espressif32/esp32dev.html
PLATFORM: Espressif 32 (6.1.0) > Espressif ESP32 Dev Module
HARDWARE: ESP32 240MHz, 320KB RAM, 4MB Flash
DEBUG: Current (cmsis-dap) External (cmsis-dap, esp-bridge, esp-prog, iot-bus-jtag, jlink, minimodule, olimex-arm-usb-ocd, olimex-arm-usb-ocd-h, olimex-arm-usb-tiny-h, olimex-jtag-tiny, tumpa)
PACKAGES:
 - framework-espidf @ 3.50001.0 (5.0.1)
 - tool-cmake @ 3.16.4
 - tool-esptoolpy @ 1.40500.0 (4.5.0)
 - tool-ninja @ 1.9.0
 - toolchain-esp32ulp @ 1.23500.220830 (2.35.0)
 - toolchain-xtensa-esp32 @ 11.2.0+2022r1
Reading CMake configuration...
LDF: Library Dependency Finder -> https://bit.ly/configure-pio-ldf
LDF Modes: Finder ~ chain, Compatibility ~ soft
Found 0 compatible libraries
Scanning dependencies...
No dependencies
Building in debug mode
"/Users/linus/.platformio/penv/.espidf-5.0.1/bin/python" "/Users/linus/.platformio/packages/framework-espidf/tools/ldgen/ldgen.py" --input /Users/linus/.platformio/packages/framework-espidf/components/esp_system/ld/esp32/sections.ld.in --config "/Users/linus/Code/esp-idf-template/blink/led/sdkconfig.debug" --fragments "/Users/linus/.platformio/packages/framework-espidf/components/app_trace/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/bt/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/driver/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/esp_adc/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/esp_event/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/esp_gdbstub/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/esp_hw_support/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/esp_lcd/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/esp_netif/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/esp_phy/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/esp_pm/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/esp_psram/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/esp_ringbuf/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/esp_rom/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/esp_system/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/esp_wifi/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/espcoredump/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/freertos/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/hal/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/heap/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/log/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/lwip/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/soc/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/spi_flash/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/xtensa/linker.lf" "/Users/linus/.platformio/packages/framework-espidf/components/esp_system/app.lf" "/Users/linus/.platformio/packages/framework-espidf/components/esp_common/common.lf" "/Users/linus/.platformio/packages/framework-espidf/components/esp_common/soc.lf" "/Users/linus/.platformio/packages/framework-espidf/components/newlib/system_libs.lf" "/Users/linus/.platformio/packages/framework-espidf/components/newlib/newlib.lf" --output .pio/build/debug/sections.ld --kconfig "/Users/linus/.platformio/packages/framework-espidf/Kconfig" --env-file "/Users/linus/Code/esp-idf-template/blink/led/.pio/build/debug/config.env" --libraries-file "/Users/linus/Code/esp-idf-template/blink/led/.pio/build/debug/ldgen_libraries_pio" --objdump "/Users/linus/.platformio/packages/toolchain-xtensa-esp32/bin/xtensa-esp32-elf-objdump"
Traceback (most recent call last):
  File "/Users/linus/.platformio/packages/framework-espidf/tools/ldgen/ldgen/entity.py", line 152, in _get_infos_from_file
    results = parser.parseString(info.content, parseAll=True)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/linus/.platformio/penv/.espidf-5.0.1/lib/python3.11/site-packages/pyparsing/util.py", line 256, in _inner
    return fn(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/linus/.platformio/penv/.espidf-5.0.1/lib/python3.11/site-packages/pyparsing/core.py", line 1190, in parse_string
    raise exc.with_traceback(None)
pyparsing.exceptions.ParseException: Expected end of text, found 'default'  (at char 1), (line:2, col:1)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/linus/.platformio/packages/framework-espidf/tools/ldgen/ldgen.py", line 183, in <module>
    main()
  File "/Users/linus/.platformio/packages/framework-espidf/tools/ldgen/ldgen.py", line 159, in main
    mapping_rules = generation_model.generate(sections_infos)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/linus/.platformio/packages/framework-espidf/tools/ldgen/ldgen/generation.py", line 514, in generate
    root_node.insert(entity, sections, target, flags, entities)
  File "/Users/linus/.platformio/packages/framework-espidf/tools/ldgen/ldgen/generation.py", line 241, in insert
    self.child_placement(entity, sections, target, flags, sections_db)
  File "/Users/linus/.platformio/packages/framework-espidf/tools/ldgen/ldgen/generation.py", line 233, in child_placement
    child.insert(entity, sections, target, flags, sections_db)
  File "/Users/linus/.platformio/packages/framework-espidf/tools/ldgen/ldgen/generation.py", line 241, in insert
    self.child_placement(entity, sections, target, flags, sections_db)
  File "/Users/linus/.platformio/packages/framework-espidf/tools/ldgen/ldgen/generation.py", line 233, in child_placement
    child.insert(entity, sections, target, flags, sections_db)
  File "/Users/linus/.platformio/packages/framework-espidf/tools/ldgen/ldgen/generation.py", line 241, in insert
    self.child_placement(entity, sections, target, flags, sections_db)
  File "/Users/linus/.platformio/packages/framework-espidf/tools/ldgen/ldgen/generation.py", line 296, in child_placement
    found_sections = sections_db.get_sections(self.parent.name, self.name)
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/linus/.platformio/packages/framework-espidf/tools/ldgen/ldgen/entity.py", line 197, in get_sections
    obj = self._match_obj(archive, obj)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/linus/.platformio/packages/framework-espidf/tools/ldgen/ldgen/entity.py", line 183, in _match_obj
    objs = self.get_objects(archive)
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/linus/.platformio/packages/framework-espidf/tools/ldgen/ldgen/entity.py", line 176, in get_objects
    self._process_archive(archive)
  File "/Users/linus/.platformio/packages/framework-espidf/tools/ldgen/ldgen/entity.py", line 164, in _process_archive
    parsed = self._get_infos_from_file(stored)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/linus/.platformio/packages/framework-espidf/tools/ldgen/ldgen/entity.py", line 154, in _get_infos_from_file
    raise ParseException('Unable to parse section info file ' + info.filename + '. ' + p.msg)
pyparsing.exceptions.ParseException: Unable to parse section info file /Users/linus/Code/esp-idf-template/blink/led/.pio/build/debug/esp-idf/esp_event/libesp_event.a. Expected end of text  (at char 0), (line:1, col:1)
*** [.pio/build/debug/sections.ld] Error 1
================================================================= [FAILED] Took 4.22 seconds =================================================================

Environment    Status    Duration
-------------  --------  ------------
debug          FAILED    00:00:04.223
release        IGNORED

To Reproduce
Steps to reproduce the behavior:

Follow the guide lines in README.md
cargo binstall espup
Follow the steps in README-pio.md
cargo pio new led --platform espressif32 --frameworks espidf --board esp32dev
It fails on the step pio run

Environment

OS: 14.0 Beta (23A5276g)
rustc 1.70.0 (90c541806 2023-05-31)
PlatformIO Core, version 6.1.8rc2
Python 3.11.4 (main, Jun 20 2023, 17:23:00)
How did you install the environment: espup install && . /Users/linus/export-esp.sh

@oleander All of the errors I see in your trace are in fact within PlatformIO itself.
The version (6.1.8rc2) sounds like something which is not released as well.
Can you try with the latest officially released version of PlatformIO and reply here if pio run completes successfully (or else, what is the output)? Thanks in advance!

authors: ivmarkov oleander

repo: esp-rs/esp-idf-template


text:
Cannot install V1.2.0 "error: -Csplit-debuginfo is unstable on this platform\n\n"

I have tried rustup update and I tried sudo apt-get install libudev-dev
This is the JSON output:
{
  "rustc_fingerprint": 21358032055700506,
  "outputs": {
    "2797684049618456168": {
      "success": false,
      "status": "exit status: 1",
      "code": 1,
      "stdout": "",
      "stderr": "error: `-Csplit-debuginfo` is unstable on this platform\n\n"
    },
    "17598535894874457435": {
      "success": true,
      "status": "",
      "code": 0,
      "stdout": "rustc 1.58.1 (db9d1b20b 2022-01-20)\nbinary: rustc\ncommit-hash: db9d1b20bba1968c1ec1fc49616d4742c1725b4b\ncommit-date: 2022-01-20\nhost: x86_64-unknown-linux-gnu\nrelease: 1.58.1\nLLVM version: 13.0.0\n",
      "stderr": ""
    },
    "931469667778813386": {
      "success": true,
      "status": "",
      "code": 0,
      "stdout": "___\nlib___.rlib\nlib___.so\nlib___.so\nlib___.a\nlib___.so\n/home/newuser/.rustup/toolchains/stable-x86_64-unknown-linux-gnu\ndebug_assertions\nproc_macro\ntarget_arch=\"x86_64\"\ntarget_endian=\"little\"\ntarget_env=\"gnu\"\ntarget_family=\"unix\"\ntarget_feature=\"fxsr\"\ntarget_feature=\"sse\"\ntarget_feature=\"sse2\"\ntarget_os=\"linux\"\ntarget_pointer_width=\"64\"\ntarget_vendor=\"unknown\"\nunix\n",
      "stderr": ""
    }
  },
  "successes": {}
}

Fixed the with: apt install librust-libudev-sys-dev

source: gharchive/issue

repo: esp-rs/espflash

license_type: permissive

license_source: github-api


text:
bump esp-idf-part to v0.4.1

Draft until the new release is cut

Thanks, but I'll be updating all dependencies prior to the release.

source: gharchive/pull-request

authors: MabezDev jessebraham


text:
Add installation guide for developer

This PR adds installation guide for developers of this repository.

Thanks , a couple of thoughts:

We can add torch and espnet versions
S3PRL/Fairseq tested commit ids can be added similar to espnet installation

@karthik19967829
Sorry for the late reply.
Do you mean that we need to specify the version of torch, espnet, and s3prl/fairseq in requirements_export.txt?

@Masao-Someki yes if we specify the tested version that would be helpful for the developers to start with stable version

The detailed developer's guide should include the generic encoder. So with this PR, I will merge it with the additional demonstration.

license: MIT


text:
esp_ota_get_app_description missing (IDFGH-905)

Environment

IDF version (run git describe --tags to find it): v3.1.3
Build System:         Make
Compiler version (run xtensa-esp32-elf-gcc --version to find it): 1.22.0-80-g6c4433a
Operating System:    Linux

Problem Description
I am trying to get the app version using the esp_ota_get_app_description, but the problem is that the function does not exists. It should be defined in components/app_updates, but these files seems to be older and they are not the same version as the one from git. I don't even have all the files from components/app_updates. What should I do?

https://esp32.com/viewtopic.php?f=13&t=9950#p41073

Thanks! The means that we have to wait for V3.3

authors: chegewara tatulea

repo: espressif/esp-idf


text:
Read/Write data to ESP32 from PC over USB (IDFGH-8132)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
I'm trying to build a desktop application that will be able to retrieve details such as the device id, hardware version, firmware version, and model from an ESP32 device over USB communication to the PC. How would I be able to do that?

https://github.com/espressif/esp-idf/tree/master/examples/system/console
https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/system/console.html

@tobyloki I don't have a lot to add to @chegewara's comment. Some basic information (MAC, flash ID, etc.) can be retrieved with esptool.py. Of course, you can implement your own protocol over UART to communicate with an ESP as well, but the linked console application might be more straight-forward.
If you need more support, feel encouraged to post in the ESP Forum as this is meant to discuss programming questions like these. You'll probably get more help there.
Besides the implementation, if you have issues communicating with the ESP-chip please let us know! Otherwise, we would like to close this issue.

@tobyloki we're closing this issue. Feel free to re-open if the aforementioned approaches didn't work for you.

authors: 0xjakob chegewara tobyloki


text:
BT: Check if CCB's fixed channel info is NULL (IDFGH-2874)

A BT device can disconnect while other actions (such as requesting BLE GATT attributes) can be underway. This sets various bits of state to NULL under our feet. When these two states raced, it was possible an ESP32 would crash with a load exception.
When checking for congestion, return TRUE, indicating the channel is congested, so that the caller's error paths can take over.

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

@pvachon Thanks for contribution and sorry for slow turnaround, please help see commit 51a0fcea5. Thanks.

authors: Alvin1Zhang CLAassistant pvachon


text:
PDF doesn't pan nor zoom when opened in browser on mobile devices

pdfrx version: 0.4.54
Steps to reproduce

Open https://espresso3389.github.io/pdfrx/ in a browser on Android or iOS device
Try to scroll or zoom-in using gesture

Actual result
Nothing happens
Expected result
The PDF can scroll and zoom using touch gestures

Text selection affects pan/zooming feature on Flutter Web on mobile devices.
So, just set enableTextSelection: false will enable pan/zooming.
This is just a limitation of the current "experimental" text selection feature (Related: #4).

[CHANGELOG: 0.4.7] already mentioned the issue anyway.


text:
GPS speed does not work on BangleJS 2

Affected hardware version
Bangle 2
Your firmware version
2v16
The bug
GPS works, but GPS speed is always displayed as zero on BangleJS2. If I do
Bangle.on('GPS-raw', print );
reason is very easy to see: GPS generates just $BDGSV, $GNGGA and $GPGSV sentences. Speed (and other useful info) is not part of these. I guess we need to enable additional messages in the hardware, but I don't see how to do that.
Installed apps
No response

Thanks for this - when did you get your Bangle.js?
I just tried on a new Bangle and it's as you say. However I just tried on two different old Bangle.js and those produce BDGSV,GNRMC,GNVTG,GNZDA,GPTXT,GNGGA,GNGLL,GNGSA,GPGSV so at some point the firmware on the Bangle's GPS has been updated and I wasn't told by the manufacturer.
The protocol used for the GPS is called CASIC - there are a few threads on the forum if you search Google for: site:forum.espruino.com AT6558 or CASIC.
But it looks like sending Serial1.println("$PCAS03,1,1,1,1,1,1,0,1*03") is enough to re-enable everything (after GPS is turned on) if you're interested in trying it.
So I guess we need to get Espruino to send that after the GPS is powered now :( Ideally we'd also be able to send some other info like estimated time/location, but it seems basically impossible to get information on how to do that

authors: gfwilliams pavelmachek

repo: espruino/BangleApps


text:
Astrocalc, Sun and Moon info calculator

Astrocalc
View information about the Sun and Moon, such as position in the sky, rise & set times and much more based on your GPS data.
This is quite a beefy project, so apologies if it goes against any community guidelines that I may have missed.
If anyone wants to chip in there are a few outstanding enhancements that could be made:

Language is English only
Doesn't render moon phases for southern hemisphere, this is simple to correct, and doesn't require new images.
It could store your last GPS fix, and use that when it boots as an option, rather than always waiting for a valid GPS lock.
Better Moon images, the current ones are rather uninspiring.
And anything else we can come up with.

Images

Looks great, although do you think you'd be able to do the images a bit differently? The current method will be a bit slow and while I'm sure it works fine, I'd prefer others didn't see it and copy it :)
You could either include the images in your program code (if it's inside a function, it won't use any RAM), or you could do like we do here: https://github.com/espruino/BangleApps/tree/master/apps/animals
So, you export with compression, but as "image string", and then you shove that in a file much like you have done (but with .img extension), but set evaluate:true in apps.json.
Then when you load you literally just have to load the file and put it into drawImage - no messing about at all.

Ah I can't believe I missed the Animal app 🤦♂ Update on way, will test shortly.

Thanks for the image loading tip. It appears to be working as before.

Thanks, that's awesome!

Check my app moonphase. It has some images in there.

authors: Purple-Tentacle gfwilliams paulcockrell


text:
Bigdclock v0.07

Better battery graphic - now has green, yellow and red sections
battery status reflected in the bar across the middle of the screen
better handling of locale

re that last one - not everyone has a locale set up, and when you set a locale up, it seems the date_utils.js disappears or becomes unavailable???
(I've incorporated https://github.com/espruino/BangleApps/commit/5b1b3553c5a028c595845434762147b28de317a6 as best I can).

re that last one - not everyone has a locale set up, and when you set a locale up, it seems the date_utils.js disappears or becomes unavailable???

That shouldn't happen as date_utils gets included in the JS file as it is uploaded. It's just not possible for locale to remove it.
I imagine it just happened because you uploaded your app through the IDE without having the path set in the IDE settings? Info at https://github.com/espruino/BangleApps/blob/master/modules/README.md

not everyone has a locale set up

Absolutely everyone has locale set up. It's included in the firmware, so if you don't have one installed you get the default english locale.
Please could you take out those locale changes? The original var dows = require("date_utils").dows(0,1); will work fine - this gets included all the time by the app loader, and uses locale so will be internationalized. There's really no reason for the try - I think it just stems from you uploading from the IDE without having the Bangle.js libraries in your search path

Yes you were right. Changed implemented. Also I now only check the battery state of charge every 15 minutes instead of every minute, giving me noticeably extra time between having to charge!

Great - thanks! Just merging...

authors: deirdreobyrne gfwilliams


text:
Multiple col-12 in one row

If I have multiple col-12 in one row (see example), the second col-12 got a margin-right which is not needed on col-12.
Example:

         <svg class="footer__icon">
            <use xlink:href="#logo-bear"></use>
         </svg>

XXX

XXX

Copyright © 2015

I think this is expected because in a row of columns, every column except the first one has a margin for the gutter. Typically each row would contain any number of columns that add up to 12, so a col-12 column should in theory be it's own row, with the next col-12 also being in its own row.
However, you should be able to add the class flow-row to your row, like so:

      ...

And that should fix the margin issues you are having.

True, thanks. :+1:

license: mit

license_source: bigquery


text:
TextInput border highlight on focus

As a component, universal ftw.

Low priority.

repo: este/este


text:
Support for door locks

I have a couple deadbolts that currently can't be controlled via the app. Looks like it's basically just a switch entity. Works fine in the webui.

Closing as duplicate of #146


text:
plug device class

Version: 0.5.2
In my Lovelace UI, I have one badge representing an MQTT sensor with device_class: plug. HA Client does not show the associated plug icons (plugged/unplugged), only a black circle within the badge.

Issue was separated. See #336

authors: estevez-dev tdejneka


text:
Add operator survey to repo

This adds the operator survey summary, in MD format, to this repo.  It also creates a WG subdirectory, which seems like a useful place to stash files related to WG activities.

Probably better to put the survey under the existing directory docs (create a subdirectory survey under docs)?

Probably better to put the survey under the existing directory docs (create a subdirectory survey under docs)?

Isn't the Docs folder going to eventually hold the docs for the actual operator?  In which case we don't want survey results there.

Isn't the Docs folder going to eventually hold the docs for the actual operator? In which case we don't want survey results there.

I would make a case for user facing operator docs to go on the etcd website for the most part.

Probably better to put the survey under the existing directory docs (create a subdirectory survey under docs)?

Isn't the Docs folder going to eventually hold the docs for the actual operator? In which case we don't want survey results there.

The goal is to organize all documents under one directory. It's similar to etcd documents, in which there are different kinds of docs.

@jberkus could you update this PR so that we can get it merged?

@hakman @jmhbnz Please approve this PR before I merge it. thx

authors: ahrtr jberkus jmhbnz

repo: etcd-io/etcd-operator


text:
🛑 Regular hhh is down

In 7784274, Regular hhh (https://househunthero.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Regular hhh is back up in e0bbf5a after 14 minutes.


text:
Sitting characters don't appear properly when coming into view

This happens when a player coming into view is sitting - equipment offsets don't take the sit state into account. Refreshing the view fixes it.

This isn't reproducible. It was probably referring to display bugs when a sitting player uses an emote (e.g. afk emote)

repo: ethanmoffat/EndlessClient


text:
Abstract RpcHandler and Remote from tokio-core

jsonrpc-core should not have dependency on tokio-core.
jsonrpc-core::reactor could be moved to separate crate, but RpcHandler and Remote needs to be abstracted.

Transports should probably be generic over FutureSpawner trait.

repo: ethcore/jsonrpc


text:
design doc for 'Always using _disableInitializers()  in constructors'

This design doc proposes always using _disableInitializers() in proxied contracts constructors rather than the prevalent approach of calling the initialize() function from the constructor.
Issue: https://github.com/ethereum-optimism/optimism/issues/12634

Fully support this. It is currently being implemented in #12057 anyways, so it'll get in for isthmus :)

Discussed in #150. Conclusion was to implement it alongside adding the following semgrep rules:

Checks that all initialize functions have external and initializer modifiers.
Checks that all proxied contracts have a constructor and that it calls _disableInitializers()

authors: AmadiMichael maurelian

repo: ethereum-optimism/design-docs


text:
Token chaining API is error prone

Also, this might indicate that we need to do something to fix the cancel token chaining API because honestly, I've had trouble sometimes figuring out what the right order should be. Maybe we need something like token.create_child_token() rather than actually combining two tokens and thus, having to figure out the right ordering.

(From https://github.com/ethereum/py-evm/issues/1234#issuecomment-418223040)
Another thing that is problematic and may need to be redesigned (or at least documented), is the fact that sometimes a class will keep a direct reference to the token passed to its constructor (e.g. https://github.com/ethereum/py-evm/blob/master/trinity/protocol/common/managers.py#L200 and https://github.com/ethereum/py-evm/blob/master/trinity/p2p/handlers.py#L33), which means they will cancel their parents if they ever trigger their tokens. Even though some of those may never actually trigger their tokens (e.g. when they only use the token to detect when their parent has terminated), they could still pass these tokens to other classes that eventually do trigger them

So for the first issue, I propose something like the following.
# legacy
master_token = CancelToken('master')

child_token = CancelToken('child').chain(master_token)

# new
master_token = CancelToken('master')
child_token = master_token.create_child_token('child')

For the issue about things keeping reference to a parent token when they should not, I'm thinking something like this.
# legacy
master_token = CancelToken('master')

# implicit expectation that `Service` class will chain off of
# `master_token` as opposed to using it directly
child_service = Service(master_token)

# new API
child_service = Service(master_token.immutable_reference)

The immutable_reference would support all of the read operations but any calls to things like trigger() or cancellable_wait would throw exceptions.  Then anything that retains a reference to the parent token when it shouldn't will throw errors when it tries to perform operations on it.

A crazy idea:
Split the token into two parts right away: the trigger, and roughly everything else
token_maker = Canceller('master')

parent_trigger, parent_token = token_maker.make_token()

# Only BaseService would keep the parent_trigger in a private location, triggered by cancel()
# The parent_token would be public, though.

###

# Later, something that wants to "chain" can do:

child_maker = parent_token.create_child_canceller('child')

# parent_token.trigger() is NOT a thing
# parent_token.triggered still works

child_trigger, child_token = child_maker.make_token()

These names are... bad. Sorry.

I'd personally prefer the existing APIs (which are simpler) with extra checks to prevent non-owners to trigger tokens

authors: carver gsalgado pipermerriam

repo: ethereum/asyncio-cancel-token


text:
Balance eligibility previous+1 vs current epoch

In get_eligible_validator_indices the previous epoch is used, then "converted" (not strictly the same for genesis epoch) back to current epoch:
previous_epoch + 1 < v.withdrawable_epoch

Now the rewards and penalties, the only place where get_eligible_validator_indices is used, we don't run anything for genesis epoch:
def process_rewards_and_penalties(state: BeaconState) -> None:
    if get_current_epoch(state) == GENESIS_EPOCH:
        return

Does this mean the previous_epoch + 1 could just have been a current_epoch, as there shouldn't be any effective difference?
Context: @terencechain is comparing Prysm and spec/zrnt balances code, upgrading to v0.12, and after discussing the inclusion delay part (which seems to be a subset of eligible validators regardless of the eligibility check, although a tricky/awkward edge case), this difference showed up.
If there is any special reason to use previous_epoch + 1 instead of current_epoch I'd like to know, and we should add a comment as part of the spec function descriptions PR.

Found some previous discussion: https://github.com/ethereum/eth2.0-specs/pull/949#discussion_r277380008
It seems fine to me to use current_epoch.

Closing this issue, previous_epoch is used in the same variable scope already, so it seems ok to just use that, instead of getting the current epoch as well.

authors: hwwhww protolambda


text:
BlockchainTests should only have one failing block at most

The tests should be more restrictive in a way that every yielded test should contain:

At most 1 invalid block
It should always be the last block

For example, on test: https://github.com/ethereum/execution-spec-tests/blob/4de323c5b5e741488dd2b47efde5f63f7cc7f09c/fillers/withdrawals/withdrawals.py#L63-L108
We produce three blocks where the first block is invalid, and then two invalid blocks, but this test could stick to these restrictions by yielding two times: once with only the invalid block, and afterwards with the two valid blocks.
Also, ideally, the filler should catch and restrict this behavior and throw an error when we try to fill a test that does not follow this convention.

why one block only?

authors: marioevz winsvega

repo: ethereum/execution-spec-tests


text:
Get private key

I am trying to use theMnemonic Code Converter found at https://github.com/Coinomi/bip39 to find out the private key of an address. However I can't find the correct combination. My public address is never shown in the list.
Should I go for BIP44 or BIP32? Any particular derivation path if BIP32 for example?
Thanks.

Managed to find it. It's m/44'/60'/0'/0 on BIP32

repo: ethers-io/EthersWallet-ios


text:
fix: display version with commit hash

At @ldeffenb's request, ref

This change is

@significance @ldeffenb thanks for pointing this out, fixed in #2285

authors: mrekucci significance

repo: ethersphere/bee


text:
QKC/OP Devs Meeting #13 Agenda

Meeting Info

Aug 10, 2024, 10:00 UTC+8
Duration: 60 minutes
Meeting link
Anyone can comment on the agenda where they would like this to be discussed.

Agenda

Protocol/App/Research progress

implementing dynamic inbox feature: source
TODO: make batcher aware of dynamic inbox

Try OCWeb and have some discussion with nand

Study Dispute Game V2
Review Bill's code
Submit milestones of the two op-grants

Study Dispute Game V2
Review Bill's code
Submit milestones of the two op-grants

fix the unit test of resolve
The "attack" has been modified to support "da"

Read a lot of articles and docs about parallel EVM.
Writing a proposal for Block-STM Based Parallel EVM on Geth.

authors: Akagi201 JustXxx blockchaindevsh dajuguan qzhodl

repo: ethstorage/pm


text:
AWS S3 unstructured dataset connection support for generate and annotate dataset

To ease the onboarding to Fides, Fidesctl should be able to support unstructured datasets in addition to structured datasets in generating fides annotations. This will allow adopters to be able to create dataset annotations quickly and directly from unstructured datasets.
This issue is a spike for (final scope WIP):

handling the connection to AWS S3 bucket(s)
determining file path(s) in bucket(s)
creating annotation defaults for files in various paths (collection), down to the file-level (field).

closing all tickets older than a year as not planned

authors: Kelsey-Ethyca iamkelllly

repo: ethyca/fides


text:
Fix multiline hash comments

This was my bad-- the multiline hash comments were eating the rest of the file instead of ending properly at the next unescaped newline. Fixed the regex to cover all the cases and added lots more tests.

Codecov Report

Merging #80 into master will increase coverage by 0.05%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #80      +/-   ##
==========================================
+ Coverage   91.52%   91.57%   +0.05%
==========================================
  Files           6        6
  Lines         637      629       -8
==========================================
- Hits          583      576       -7
+ Misses         54       53       -1

Impacted Files
Coverage Δ

apacheconfig/lexer.py
96.61% <100%> (+0.33%)
:arrow_up:

Continue to review full report at Codecov.

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

Thank you! Sorry for it has taken so long!

authors: codecov-io etingof sydneyli

repo: etingof/apacheconfig


text:
Update uuid-creator to 3.7.3

Updates com.github.f4b6a3:uuid-creator from 3.7.2 to 3.7.3.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.f4b6a3", artifactId = "uuid-creator" } ]

labels: library-update, semver-patch

Codecov Report

Merging #132 (a7b0ebf) into main (f55c14d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #132   +/-   ##
=======================================
  Coverage   89.07%   89.07%
=======================================
  Files          86       86
  Lines        2783     2783
  Branches       91       91
=======================================
  Hits         2479     2479
  Misses        304      304

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 f55c14d...a7b0ebf. Read the comment docs.

authors: codecov-commenter scala-steward

repo: etspaceman/kinesis-mock


text:
Add a config option to use consistent hashing.

This helps minimize the unpredictability of issues due to
https://github.com/etsy/phan/wiki/Different-Issue-Sets-On-Different-Numbers-of-CPUs
E.g. if the file list was A,B,C,D,F,G,H,I
Phan would previously assign A,C,F,H to group 1, and B,D,G,I to group 2.
If another file was added, e.g. E:
Phan would assign A,C,E,G,I to group 1, and B,D,F,H to group 2.
Half of the files in each group would have been swapped.
(I don't see where the file name sorting step is in Phan... is there one?)
However, with this way of sorting,
phan would (e.g.) assign A,B,G,I to group 1, and C,D,F to group 2,
and when E was added to a project, E would be added to group 1
without changing groups of other elements.
(Higher variance in files per process, but results are more predictable)
(https://en.wikipedia.org/wiki/Consistent_hashing)
Tested locally with config option enabled/disabled

Love this patch. Thank you so much @TysonAndre!

authors: TysonAndre morria

repo: etsy/phan


text:
Fix ALSA-sw driver not loading

It looks like clang-format added some spacing that completely broke the ALSA driver.
I fixed the spacing and disabled clang-format on the offending line.

interesting
i did some digging and found the same issue present in directsound-sw
this is pretty funny, we didn't notice it for 9 months

repo: etternagame/etterna


text:
Estonian test PID not working

While adding document in EUDI Wallet and choosing National ID, issuing PID from Estonia is not working. In case of choosing Smart-ID or Mobile-ID for authentication, both return an error messages.

Hi @EneliKirme ,
The test PID Provider (https://issuer.eudiw.dev/) is integrated with the Estonian Identity Provider pre-production/test environment. To issue an Estonian test PID, you must use a testing account. For further details, please refer to https://e-gov.github.io/TARA-Doku/Testing#21-quick-reference-of-testing-accounts.
If you're installing the eudi-srv-web-issuing-eudiw-py, you will need to to first submit an application to join the TARA test environment, and configure the "pid_url_oidc" in the eudi-srv-web-issuing-eudiw-py/app/app_config/config_countries.py file.
Thank you for your comment

Comment closed due to inactivity. If further feedback is required, the comment can be reopened.

authors: EneliKirme pinamiranda


text:
Expose REST resource via config

This solves #16695.

retest this please

Thanks @ahmadshahwan I've merged the PR here.

repo: eugenp/tutorials


text:
Update index.d.ts

Changes in PR:
missing types

Thanks for this PR!
I hope it get merged soon, need this too.

Thanks for this PR!
I hope it get merged soon, need this too.

repo: euvl/vue-notification


text:
Install ros bridge

http://wiki.ros.org/rosbridge_suite

https://github.com/ev3dev-lang-java/ev3dev-lang-java/issues/340


text:
Integrate code generation into main package

Merge code generation into evalplus directory
One command to run both generation and evaluation
Package isolation

Implemented by:

a04f3251bcc44b151f42428f21636ea1d65e287b
ca0acd3e6c19c323e7ff5dd5037ce04daca6f7df

repo: evalplus/evalplus


text:
Style improvements

I applied my very poor design skills to the site. This is the result: https://gist.github.com/suhr/a1952cd7d002b48a55f2b5dd907e3b04
Now it looks like this. Still looks poorly, but I hope there's an improvement.
Speaking of improvements, it would be nice to align elements, so they won't look like a mess. Take a look on BeepBox, the settings look nice because they are aligned.

I'm concerned that it will make the site harder to use for people with small monitors.

All right, there's actually a better way to do the layout:
body > div {
    margin: auto;
    padding-left: 1em;
    width: 58em;
}

Together with @media you can get an adaptive layout.

For example, I would rather add space between Title and Tempo, and remove the space above the textbox to indicate that Tempo, Key, and Lowest scale degree affect the textbox.

Tempo affects everything, not only the textbox. That's why there's a visual block, there are global settings.

You  mentioned wanting to align elements (I assume textboxes, buttons, and dropdowns). The problem is that the elements are already  baseline-aligned and styled by the operating system. I don't want to  change that, as I explained here

I'm not talking about anything advanced. There should be a space between a setting name and a corresponding control, and that space should make a vertical edge. You actually could do this even with tables (don't do that).

The margin: auto thing is a good idea. I might use that.
The Tempo slider affects the textbox by adding a line like bpm: 100. Then the tempo stored in the textbox affects everything else.
I tried aligning the settings the way you explained and I don't like it. Here's how it looks.

All right, what about this (the gist is updated)? All Arial, sliders close to the textbox, Title is separated.

Sorry dude, I'm just not interested in doing this.


text:
🛑 www.actionherojs.com is down

In a66a656, www.actionherojs.com (https://www.actionherojs.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: www.actionherojs.com is back up in 55cb936.

repo: evantahler/upptime


text:
Fordpass connection stopped working in evcc

Describe the bug
After a few weeks of no issues evcc can no longer login to fordpass.
my ford account is not locked and is working fine in both fordpass and home assistant.
User and pass are correct
please help
Steps to reproduce
Nothing special… i just start evcc addon in homeassistant
Configuration details
Standard config:
name: Kuga
type: template
template: ford
title: Kuga
user: ***@gmail.com
pass: ****
capacity: 12

Log details
[main ] ERROR 2024/05/18 17:36:15 creating vehicle Kuga failed: cannot create vehicle type 'template': cannot create vehicle type 'ford': login failed: self-assert failed

What type of operating system are you running?
HomeAssistant Add-on
Version
No response

Mit der Core Version 2024.4.1 Supervisior 2024.05.1 geht es bei mir alle Daten vom Ford werden angezeigt

Also die FordPass integration nicht die EVCC

okay, i have not updated to version 2024.05.4 i had problems with automatic integration. The integration with FordPass still works with this  Core Version 2024.4.1

siehe #13949

Already tried the domain part but i had no luck

repo: evcc-io/evcc


text:
Add Zendure

Fix https://github.com/evcc-io/evcc/issues/14797
Note that battery control is not supported. A local api is announced for 2025, we'll revisit then.

This doesn't work if the global server is used which many advanced users even in Europe have chosen since it's supposedly more stable and offers newer devices faster etc.
For the global server the URL to get the credentials is:
https://app.zendure.tech/v2/developer/api/apply
So another optional parameter for the server is needed.
And a Hyper V is mentioned. That's not a Zendure product I know of. The Hyper 2000 is a bi-directional home battery+PV system and the Superbase V is a power station.

Thank you, will update asap.

I'm preparing a pull request for the template with some additional examples etc. right now.

First off all a HUGE THANK YOU ! To y'all for trying your damn hardest to implement the wishes of the users - you all have my deepest respect. (As I had a peek into the code there was no way in hell i could have accomplished this)
I don't want to be a PIA, but the current "Bad Request error" I get while trying to configure is related to the fact, that currently the EU server is used in general and not the global one - correct?

Probably. Waiting for @Raudi1 if he wants to grab it.

Probably. Waiting for @Raudi1 if he wants to grab it.

I already did the pull request. But only for the template. I could probably figure the rest out, but it would probably take me a couple of hours at least since I'd have to set up a dev environment and take a good look at all the code first and then do it wrong a couple of times before I get it right. I don't have the time for that right now since I'm a dentist with my own practice. The last quarter is always the most busy.

I still can't add my Zendure device even with the correct server. When I try to validate it, I get the error
cannot create meter type 'template': cannot create meter type 'zendure': unexpected status: 400 (Bad Request)
Oddly enough I can't find anything in the logs even with the level set to trace.

I can confirm that the global login doesn't work. EU works find. Maybe the device can only be setup in the matching region?

Oddly enough I can't find anything in the logs even with the level set to trace.

Working find for me.

Normally the device should be found on the corresponding server where the Hyper is connected to - either the global or the EU. Hence the different URLS. It should not make any difference - the response from the server should be the same.
I will try to remove the server on the global one and switch over to the European to see if that works.
Oddly enough I still find MQTT entries from my former tries to get the Hyper connected in my logs although it is not configured in the Yaml or the GUI.
[mqtt ] TRACE 2024/11/18 19:19:31 recv DISnXQ5W/Q5mkHULJ/state: '{"sn":"EE1HYMCFM230063","remainOutTime":398}'
I thought maybe that was my problem that entries are double, but I am not able to remove it at the moment.

I can confirm that the global login doesn't work. EU works find. Maybe the device can only be setup in the matching region?

I have the official MQTT and the ioBroker adapter working. If I use the manual method with
curl -i -v --json "{'snNumber': 'HubSeriennummer', 'account': 'Emailadresse'}" https://app.zendure.tech/v2/developer/api/apply
I get the correct response:
{"code":200,"success":true,"data":{"appKey":"xxxxxxx","secret":"xxxxxxxxxxxxxxx","mqttUrl":"mqtt.zen-iot.com","port":1883},"msg":"Operation successful"}* Connection #0 to host app.zendure.tech left intact

Ah, region was missing from the template, fixed!

Fix works - i was able to add the battery with the nightly build.  (I dont use the Hyper as inverter) Thanks @andig!
However @Raudi1 can you confirm that when the Hyper is discharging it is shown as "Batterie laden" instead of "Batterie entladen"?

However @Raudi1 can you confirm that when the Hyper is discharging it is shown as "Batterie laden" instead of "Batterie entladen"?
I will test it next release. Might even add a second Hyper. They just went down a lot. Only around 900€ for the Hyper with 1.92kWh or 2325€ for the Hyper with 7.68kWh and 2 solar panels.

So - I am fairly sure that the values are the wrong way around.
My Hyper is discharging atm with 192W. EVCC tells me battery is charging with 206W.
And another problem from Zendure is that the values are not reset to zero when there is no update to the value. In Homeassistant I had to add an automation to set them to zero when there is no change to the value for 3 Minutes.
I dont know if there is a way for EVCC to do the same (if necessary) - I cant wait for zendure to add the local API...

I think it would be better to use
packInputPower - Discharging of the battery
outputPackPower - Charging of the battery
instead of the used AC values anyway. I had it like that at first and it caused issues because it wouldn't show the battery as charging when it uses PV etc. With the pack values it works as expected for me.

So - I am fairly sure that the values are the wrong way around.

I can't reproduce this. For me, the charge/discharge values are of correct sign. I also don't have an issue with values not dropping back to zero.

authors: Raudi1 andig linuxstony


text:
Autoupdate Supported Brands on Website

This change keeps the list of supported brands on the website up-to date when a new implementation is added to the core repo.

collects brand names from evcc templates
commits list of brands as json to website repository

Future Idea: Currently the push mechanism runs on every master commit. As the project gets more mature we could change this to only update the website (and docs) when a release is made.

Jaha, ich hab in den letzten Tagen mein Go Wissen etwas hochgelevelt. Ich weiss zwar immer noch ob das the Go way ist, was ich da mache. Aber zumindest verstehe ich nu was make, slice und co bedeuten ;)

Und ja, gerne Unicode.

Und ja, gerne Unicode.

Please check

Die Kette bis zum Website Deployment hat funktioniert 🥳
Allerdings scheint die Sortierung in Go das Unicode nicht zu mögen.

Ok, die Sortierung in der Doku (JavaScript) ist genau so gut 😂

@andig Ich dreh bei Skoda mal wieder auf ein normales "S" zurück.

Allerdings scheint die Sortierung in Go das Unicode nicht zu mögen.

Doch, aber falsche Sortierungsfunktion:
"golang.org/x/text/collate"
"golang.org/x/text/language"

collator := collate.New(language.Make("de"))

sort.Slice(types, func(i, j int) bool {
	return collator.CompareString(types[i].Name, types[j].Name) < 0
})

Fehlt da nicht auch noch Mercedes/Benz/Daimler oder wie die heissen?

authors: andig naltatis premultiply


text:
feat: add hold down time and publish while pressing the button

Description

Modified to publish the time the button is pressed as a topic.
Add variable is_publish_while_pressing_button as rosparam.
Change to publish the topic even while the button is pressed when is_publish_while_pressing_button is True.
Modify README to match implementation.

Related Links
eve-autonomy/shutdown_manager#1
eve-autonomy/button_output_selector#1
eve-autonomy/delivery_reservation_lamp_manager#6
eve-autonomy/autoware_state_machine#2
eve-autonomy/autoware_state_machine_msgs#1
eve-autonomy/proj_launch#6
Review Procedure

[ ] Make sure the description is valid.
[ ] Make sure that what is described in the description matches the implementation.

No response is needed in this PR because of the change in specifications.
Move to another PR.
Therefore, this PR is closed.


text:
🛑 /getVideoByID - Get video info by ID is down

In 128b780, /getVideoByID - Get video info by ID (https://tiktok.evelode.com/getVideoByID?video_id=7106855913906081070&cache_timeout=0&license_key=$API_KEY) was down:

HTTP code: 0
Response time: 0 ms

Resolved: /getVideoByID - Get video info by ID is back up in 7b402cf after 32 minutes.

repo: evelode/tiktok-status


text:
建议：改一下代码块的字体/大小以及注释颜色

个人的使用体验：代码块看起来有点费眼睛，不方便向他人展示。
可能原因：字体过于纤细，字体偏小，注释的灰色和背景的黑色对比度不够大
建议：可以换一种稍微粗一点的字体，或适当调大字体，给注释换一种更亮一点的灰色
以下是本主题与 Github 主题代码块的对比，Github 采用的字体在横向上会更舒展，看起来没那么累；而本主题采用的字体在纵向上更细长

@Xiaoven 感谢建议，提上日程。

@Xiaoven 感谢建议，提上日程。

@Xiaoven 你好，首先很抱歉这么久才来回复。其次这一块使用的是 Consolas 字体，Github 也是 Consolas 字体系列。也许是差不多的。


text:
Definition xrefs have disappeared from eco.obo

All definition xrefs appear to have disappeared in the 2017-07-07 release of eco.obo.
For example, this is ECO:0000001 in the 2017-06-02 release:
[Term]
id: ECO:0000001
name: inference from background scientific knowledge
def: "A curator inference that links the current annotation to a different evidence-based annotation via background knowledge of the curator." [ECO:go, ECO:MCC]
is_a: ECO:0000205 ! curator inference
and this is it in the 2017-07-07 release:
[Term]
id: ECO:0000001
name: inference from background scientific knowledge
def: "A curator inference that links the current annotation to a different evidence-based annotation via background knowledge of the curator." []
is_a: ECO:0000205 ! curator inference
property_value: IAO:0000119 ECO:MCC xsd:string
Is this deliberate?

Hi @tonysawfordebi -
With this release, we switched from providing definition sources as annotations on the definition with the database_cross_reference property (the xrefs) to using IAO's 'definition source' annotation property (what you're seeing as IAO:0000119). These were only changed for the definitions, and any xrefs to GO codes were left, e.g. as with 'physical interaction evidence'
[Term]
id: ECO:0000353
name: physical interaction evidence used in manual assertion
def: "A type of physical interaction evidence that is used in a manual assertion." []
synonym: "inferred from physical interaction" EXACT [GOECO:IPI]
synonym: "IPI" EXACT [GOECO:IPI]
xref: GOECO:IPI "inferred from physical interaction"
property_value: IAO:0000117 "mchibucos" xsd:string
property_value: IAO:0000119 ECO:MCC xsd:string

If this is an issue for your work, please let us know so we can get things sorted out.
cc @mchibucos

OK - this means that GO & ECO do things differently now, so I can't use a shared parser (although I'm in the process of switching to consuming the OBOGraph version of GO rather than OWL/OBO, so that's not a major consideration).
I'll modify my code to take account of this change.
Incidentally, was this change announced anywhere before you implented it?
Incidentally (2), do you have any plans to make ECO available in OBOGraph (JSON) form at any point?

@rctauber I've updated our parser so that it now handles the new way of dealing with definition xrefs, but I've noticed a problem with other xrefs, namely that if the id part of the xref itself contains a colon, then the colon is escaped. For example:
[Term]
id: ECO:0000052
...
xref: TAIR:TED:0000063
[Term]
id: ECO:0000005
...
xref: PSI-MI:MI:0415 "enzymatic study"
This is something that seems to have snuck in in this release.

Thanks for bringing this up @tonysawfordebi - I don't see this in the eco-edit.owl file or the eco.owl file, but it is an issue in eco.obo. I'll replace these on the eco.obo file for now and look into it. I'll be doing an unscheduled release in just a bit so that you have a live file to work with.

Thanks, @rctauber

@tonysawfordebi We noticed that we'd like to change the PSI-MI:MI:xxxx refs to simply be MI:xxxx (e.g. MI:0415) to be consistent with other annotations. Would that cause any issues? It would be live with the next release in August.

That's fine with me, @rctauber

Can you switch back to doing the definitions as axiom annotations? This is standard

Yes. We can. would be grateful if you would suggest some documents/refs/etc. about this standard, as I am trying to learn more. @rctauber

On Jul 17, 2017, at 6:32 PM, Chris Mungall notifications@github.com wrote:
Can you switch back to doing the definitions as axiom annotations? This is standard
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@mchibucos / @rctauber Can you give me a heads-up when you're about to make this change, please, as I'll need to change my parser again.
Thanks.

Will do - and again, sorry for the inconvenience.

On Jul 18, 2017, at 2:57 AM, Tony Sawford notifications@github.com wrote:
@mchibucos / @rctauber Can you give me a heads-up when you're about to make this change, please, as I'll need to change my parser again.
Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

No worries.

Thanks, @rctauber - I'll modify my parser accordingly.

@rctauber @mchibucos I've updated my parser to reflect your changes, and I've noticed that the problem of colons in the id part of xrefs being escaped has crept back into eco.obo.
For example:
`[Term]
id: ECO:0000200
...
def: "A type of sequence similarity evidence where a sequence alignment is the basis for making an annotation, but only when the alignment has been manually reviewed; or if the information is in a published paper, the authors have manually reviewed the evidence. Such alignments may be pairwise alignments or multiple alignments." [url:http\://www.geneontology.org/GO.evidence.shtml]
...
[Term]
id: ECO:0000226
...
xref: PSI-MI:MI\:0402 "chromatin immunoprecipitation assay"
`

Hi again @tonysawfordebi -
So sorry about that! I'll take care of that, and we will be doing another release on Monday. Thank you again so much for your patience.

authors: cmungall mchibucos rctauber tonysawfordebi

repo: evidenceontology/evidenceontology


text:
Is there a way to stop images being compressed in the resulting DOCX

Hi, thanks for such an amazing library, it's really great and so easy to use!
I'm looking at using this to create a document with some images, but the resulting DOCX file seems to be fixed to 220 dpi with image compression on.  Is there a way I can change this so the resulting document uses 'High Fidelity' with the image compression switched to off?

It seems Word does compress images but has an option to turn compression off. So I opened Word to do a check and found this under File -> Options -> Advanced

Not sure if there is a way to programatically set that?

@N1kto - Also in the DocumentFormat.OpenXml 2.8.1 documentation I have found this under settings:
doNotAutoCompressPictures (Do Not Automatically Compress Images)
Being used as something like this:
<w:settings>
  <w:defaultTabStop w:val="720" />
  <w:doNotAutoCompressPictures w:val="true"/>
</w:settings>

Don't know if this is helpful at all.

authors: AndyBolam GreggOD

repo: evidenceprime/html-docx-js


text:
filter based on file name

Addresses issue #3

from qiskit.circuit.library.standard_gates.x import CXGate, XGate
basis_gate = qiskit.circuit.Gate(name='cx',num_qubits=2, params=[])

a better way is more simply
basis_gate = CXGate()


text:
Update scalafmt-core to 3.2.0

Updates org.scalameta:scalafmt-core from 2.7.5 to 3.2.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

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

Superseded by #280.

repo: evolution-gaming/kafka-flow


text:
Update sbt-release to 1.0.12

Updates com.github.gseitz:sbt-release from 1.0.11 to 1.0.12.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.gseitz", artifactId = "sbt-release" } ]

labels: semver-patch

Pull Request Test Coverage Report for Build 55

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

Totals

Change from base Build 54:
0.0%

Covered Lines:
117

Relevant Lines:
149

💛  - Coveralls

authors: coveralls scala-steward

repo: evolution-gaming/sequentially


text:
fix: only fire contamination

don't fire confirmation if new functions array is empty
RKT-8396

hotfix-v1.6

repo: evolv-ai/asset-manager


text:
Preserve project list between versions by keeping it in localStorage.

When upgrading Evothings Workbench, the application folder is often first erased, after which the new application folder is put in its place. This deletes the project-list.json file. To preserve the project list, store the project list in localStorage, which physically is stored in the OS:es user folder, which is not deleted when removing the application folder.

An implentation is available in this branch: https://github.com/fredrikeldh/evothings-studio/tree/example-handling

Merged and pushed to branch evothings2.

authors: Sionarch divineprog fredrikeldh

repo: evothings/evothings-studio


text:
[BUG]: Two method are not existing for zarinpal driver, and only work on idpay.

Hello @AmirrezaNasiri  ✌🏻
Describe the bug
Two method are not existing for zarinpal driver,  and only work on idpay.
To Reproduce
Steps to reproduce the behavior:

Set default gateway to zarinpal in config/toman.php
Create Toman request:

 Toman::amount(10000)
            ->orderId('123')
            ->mobile('a valid mobile number')
            ->email('a valid email')
            ->callback('a valid site route')
            ->description('description of this payment')
            ->name('JalalLinuX')
            ->request();

See exception:

Method Evryn\\LaravelToman\\PendingRequest::orderId does not exist.

Now comment ->orderId('123')

 Toman::amount(10000)
            // ->orderId('123')
            ->mobile('a valid mobile number')
            ->email('a valid email')
            ->callback('a valid site route')
            ->description('description of this payment')
            ->name('JalalLinuX')
            ->request();

See exception:

Method Evryn\\LaravelToman\\PendingRequest::name does not exist.

Expected behavior
This package a multi pay driver package and it should have the same usage. but orderId, name method only work for idpay driver.
Screenshots

Exact Versions:

Laravel: 8.54
PHP: 7.4
Package: 2.0.1

Hi @jalallinux :)
Thanks for the issue submission. This is actually a discussable topic; The way we should generalize the package API and I've struggled with the issue when I was working on it.
I chose to follow driver-specific requirements for requests and callbacks and yet generalize some common parts of them; So developers can understand which driver expects what kind of inputs and considerations explicitly. This can for instance prevent depending on the non-existence $request->orderId() in the callback requests when the gateway has switched from IDPay to Zarinpal - because there is no definition of orderId in Zarinpal.
A developer can also use their own strategy design pattern to make gateways interchangeable easily. So my +1 goes for this architecture.
On the other hand, it's about the easiness of usage. A similar package for the study is Laravel Socialite, which generalizes callbacks (same in our case) and puts its related logic up to the developer itself. Though its providers are not completely interchangeable either (because of lack of the common attributes), and we also have the same issue for requesting side, I guess it's a +1 for this architecture too.
To summarize, I'm not sure about either of them. Developers need their own logic for interchangeability for both of them. I'm gonna keep this issue open - This is an interesting topic.

authors: AmirrezaNasiri jalallinux

repo: evryn/laravel-toman


text:
Add EVM benchmarking scripts

According to this issue: https://github.com/ewasm/benchmarking/issues/22
This PR adds the EVM benchmarking scripts and separates the VM's in different Docker files.
@chfast @cdetrio please let me know if you have any comment

Please check the LGTM warnings.

Hey @hugo-dc, the https://github.com/ethereum/go-ethereum/pull/20330 has just been merged, so we can use go-ethereum/master now, and any future release of it.
Changes are looking good, I will review the python code in detail later. Still pay attention to LGTM warnings.

authors: chfast hugo-dc

repo: ewasm/benchmarking


text:
Wish feature request: omp.for_break and omp.for_continue

In writing the parallel t_is_prime demonstration, I find it awkward exiting the application from within the parallel for loop. Folks have suggested the continue statement for OpenMP using C/C++ which I tried. Doing so causes Codon to disable OpenMP somehow. Is that a bug?
@par(schedule='dynamic', chunk_size=1, ordered=True)
for chunk_id in range(num_chunks):
    if not_prime: continue

    low = uint(chunk_size) * uint(chunk_id) + uint(6) - uint(1)
    high = low + uint(chunk_size) - uint(1)
    if high > q: high = q

    if _DEBUG:
        show_chunk_info(chunk_id, low, high)

    while True:
        if low > high:
            break
        if n % low == uint(0) or n % (low + uint(2)) == uint(0):
            not_prime += 1
            break
        low += uint(6)

  # if not_prime:
  #     # OpenMP lacks omp.for_break/omp.for_continue capabilities.
  #     terminate_early(n)  # so, exit the application```

I asked on stackoverflow where to submit a feature request for OpenMP.
As always, thank you for Codon.

Ah, the question was closed on stackoverflow -- so fast.  The OpenMP forum is closed some time ago. I will try sending an email.

repo: exaloop/codon


text:
🛑 My Site is down

In 302f4e8, My Site ($SECRET_SITE) was down:

HTTP code: 0
Response time: 0 ms

Resolved: My Site is back up in 8b60241.

repo: exand/upptime


text:
Template update: dependency update check

Maven plugin versions-maven-plugin included in file pk_generated_parent.pom reports if newer versions are available for some of the projects dependencies. This report is by default displayed for each run of maven package.
The current ticket proposes to remove this report and to replace it by a separate git hub action.

Grooming with @kaklakariada: Closing ticket due to estimated implementation effort appears to outweigh the expected benefit.

authors: ckunki jakobbraun

repo: exasol/project-keeper


text:
update markdown - sybase iq not supported

We discovered that the virtual schema implementation does not support the use of the Sybase IQ DB. This is something we would like to highlight in here for future sybase implementations.

Thanks!

Hello @exa-danilo,
Thanks for the feedback and pull request!
Could you please move it to the README file (after first paragraph)? It would be more visible there than user guide.

Hey there, can we have it in both .md to make sure everybody sees it?

applied the changes

authors: exa-danilo morazow shib-maximalist

repo: exasol/sybase-virtual-schema


text:
Architecture

Overview

https://excalidraw.com/#json=5176497841111040,3fsAWaZz8kfgPh2SzloeHA

Initial open

[ ] Contains a local copy of Excalidraw app
[ ] Loads it and works offline

Open the app

[ ] Loads local version of Exaclidraw app
[ ] Periodically checks in the background for newer versions: version.json
[ ] If newer version is different than the local one then download latest asar packaged from the above URL
[ ] Unpack the contents
[ ] Ask user to reload the app to get the latest features

On Excalidraw.com side

[x] Version can be generated on the build and stored as timestamp: 2020-02-28-21-50-45 in version.json (the actual format is up to debate)

[x] Serve the version: https://excalidraw.com/version.json
{
  "app": "excalidraw-2020-02-28-21-50-45.asar",
  "version": "2020-02-28-21-50-45"
}

[x] Serve downloadable app: https://excalidraw.com/excalidraw-2020-02-28-21-50-45.asar

It sounds good in theory. Practically we will have to investigate whether it's feasible to update the wrapped Excalidraw web app directly on the artifact on the user's machine.
Normally you would bundle update the local version with the artifacts on every release of the desktop app. Not sure whether this will work with auto-updating.
Firstly will we have to see how to integrate the app (as zip or npm) into the electron app.

I created a first version of using a local packed Excalidraw bundle: https://github.com/excalidraw/excalidraw-desktop/commit/9b25168ec1cd3001e468a015311fd699fadb837c
Seems like some files can't be found:

But when I unpack the .asar file locally, the missing files are there. Will have a look what's going on

Seems like they searching for the required file inside the root directory:

authors: lipis pinussilvestrus

repo: excalidraw/excalidraw-desktop


text:
wrong colors handling in dark theme

while using the dark theme the yellow color #ffff00 is showed much more similar to #313100
switching to the standard theme, the yellow color is showed properly instead

hey @Alcarmor , I've fixed issue of yellow color not looks like yellow by adding more brighter yellow color, although keep in mind below things, which will help you understand, why #ffff00 is not looked like yellow in dark mode.
ANS: In the dark mode, the color #FFFF00 (hex code for yellow) may not appear as a vibrant yellow due to the contrast and color adjustments applied by the dark theme. Dark themes typically use darker background colors and adjust the contrast of text and elements to improve readability.

Please fix this, the color adjustments probably shouldn't be applied when explicitly entering in a hex code.

The choice of applying color inverting filters to the whole app is objectively wrong, at least the way it is implemented now. I propose the following:

Rename the current dark mode to "Read mode".
Create a new, proper dark mode, that will only change the colors of the UI elements and the canvas background (as for "performance", just don't apply filters in this mode, or apply filters to the UI elements and just define a new set of colors for the canvas background).
Set the new dark mode as the "default" dark mode, to prevent people coming back to this issue.

I think will cover the best of both worlds. Keep up the good work with Excalidraw.

The fact that manually entered HEX colors get automatically changed to something else is very annoying. Please make it so that manual HEX codes show as expected and that dark theme by default only changes the UI not the manual colors.

repo: excalidraw/excalidraw


text:
Configure online editor

This issue is part of the migration to v3. You can read full details about the various changes here.
In Exercism v3, students can now choose to work on exercises directly from their browser, instead of having to download exercises to their local machine. The track-specific settings for the in-browser editor are defined in the top-level "online_editor" field in the track's config.json file. This field is defined as an object with two fields:

"indent_style": the indent style, either "space" or "tab".
"indent_size": the indent size, which is an integer (e.g. 4).

You can find a full description of these fields in the spec.
Goal
The "online_editor" field should be updated to correspond to the track's best practices regarding indentation.
Example
"online_editor": {
  "indent_style": "space",
  "indent_size": 4
}

Tracking
https://github.com/exercism/v3-launch/issues/2

I think it should remain 4-space indentation.
However the number of spaces is very controversial.

Yea, this is set as 4 spaces for indentation already. I'm closing this as that's not something we should bother to change, and we should try to avoid falling into bikeshedding :)

authors: ErikSchierboom Pablo-No patricksjackson

repo: exercism/cpp


text:
Updated Kotlin to 1.2.61 and enabled parallel test execution.

Parallel execution decreased build-time from 1m30s to 30s.

Hi,
I will make it at the weekend. :)
Gesendet über myMail für Android Donnerstag, 11 Oktober 2018, 09:17vorm. +02:00 von Sam Warner  notifications@github.com :

@sjwarner-bp requested changes on this pull request.
Hi  @lihofm - thanks for suggesting this change. I think that keeping up to date is a sensible idea 🙂
I've one minor concern - it looks like your clock changes have snuck into this PR too. Are you able to remove them?
—
You are receiving this because you were mentioned.
Reply to this email directly,  view it on GitHub , or  mute the thread .

Hello,
I successfully reverted the 2 updates.

Thank you!
I rebased out the extra commits. The final commit can be found here: https://github.com/exercism/kotlin/commit/0e38eadf573aecab7802446f26914a61e8374a59

authors: lihofm mikegehard

repo: exercism/kotlin


text:
Normalize the config file to minimize noise later

We will be scripting a number of changes to the track configuration.
This runs configlet fmt, which ensures that any further changes can be minimized.
See https://github.com/exercism/meta/issues/95

There's a discussion happening in exercism/meta#95 about how this normalization degrades the readability of the configs.
That discussion has some good suggestions for improvements to the serializer.
I'd appreciate it if you would weigh in on that discussion with any suggestions for improving how we handle configs, consistency, and normalization.
Meanwhile, I'm going to close this and wait for that discussion to play out.
I'm sorry about the noise, and any extra work that I've caused you.

repo: exercism/mips


text:
Link to stories, implementations and reference document

Closes #53
It looks like this:

I'm 100% not a designer, so I'll leave that to others to improve :)

Awesome work! :) Let's get this merged and see what it looks like (not reviewed the code :))

Deployed :)

authors: ErikSchierboom iHiD


text:
Add exercise secret-handshake

I speedran the V track this weekend so I figured I'd start adding some exercises. :)
Starting with the ones that were selected for the 12in23 challenge, Secret Handshake is nice for practicing bit manipulation but I also included an enum for the commands because it makes the solution look better.
Also, I think it's a good idea to start populating the practices and prerequisites lists before we end up with a plethora of exercises we'd need to revise. I might work on a PR to add them to all the exercises.

As for your suggestion to add in practices and prerequisites tags, I think that those are only for concept exercises whereas (at least for now) this track only has practice exercises. Once some concept exercises start getting added, I totally agree let's get some tags in here!

So... they are used for practice exercises in the other tracks I contributed to. As I understand it, when a track has a Syllabus, those tags are used to determine which practice exercises should be locked/unlocked depending on which concept exercises the student has solved.
Since we don't have the concepts properly designed yet, I think we could at least create tags based on the concept names that are used in the docs. This will make things easier if we get the chance to create a Syllabus someday. What do you think?

I suppose I should have been clearer, apologies!
When you actually fill in the tags on other tracks, the list inside the tags refer to concept exercise, correct? The point being: in this track we don't have any concept exercises to point to. I think that would be building for something that doesn't exist, and would go against the YAGNI principle.
If it doesn't break anything to add in those tags before we actually have concept exercises, I won't stop ya! However, I think there are more pressing concerns like creating concept exercises, more practice exercises, upgrading the test runner, creating an analyzer, and creating a representer that I'd rather put attention to.

repo: exercism/vlang


text:
strain: Implement canonical-data.json

We want to have a standard set of test inputs and outputs for each exercise to
make it easier to port them to new languages, as well as to help keep the
exercises in sync.
The Strain exercise can be found in the ./exercises/strain/ directory.
The step-by-step instructions for how to compile a canonical-data.json file is
described in this section of the contributing guide.
See http://exercism.io/contribute/canonical-data/strain for the up-to-date list of
language tracks that have an implementation of the Strain exercise.
The tracking issue has more context.

Strain should be folded into list-ops and deprecated.

repo: exercism/x-common


text:
Replace [TestCase] tests with individual tests

We have several places where we use [TestCase] to create a single test with multiple input values. While efficient (no code duplication), this is not the best of user experiences as it makes it harder to run a single test and also doesn't really allow us to mark the various input values with a descriptive text. It also doesn't align with how canonical data for exercises is defined, which only works on single test cases.
Consider the isogram exercises's tests:
[TestCase("duplicates", ExpectedResult = true)]
[TestCase("eleven", ExpectedResult = false, Ignore = "Remove to run test case")]
[TestCase("subdermatoglyphic", ExpectedResult = true, Ignore = "Remove to run test case")]
[TestCase("Alphabet", ExpectedResult = false, Ignore = "Remove to run test case")]
[TestCase("thumbscrew-japingly", ExpectedResult = true, Ignore = "Remove to run test case")]
[TestCase("Hjelmqvist-Gryb-Zock-Pfund-Wax", ExpectedResult = true, Ignore = "Remove to run test case")]
[TestCase("Heizölrückstoßabdämpfung", ExpectedResult = true, Ignore = "Remove to run test case")]
[TestCase("the quick brown fox", ExpectedResult = false, Ignore = "Remove to run test case")]
[TestCase("Emily Jung Schwartzkopf", ExpectedResult = true, Ignore = "Remove to run test case")]
[TestCase("éléphant", ExpectedResult = false, Ignore = "Remove to run test case")]
public bool Isogram_correctly_detects_isograms(string input)
{
    return Isogram.IsIsogram(input);
}

What's the difference between test case one and two? It's not immediately obvious. The canonical-data of this exercise looks much nicer. I think we should convert the [TestCase] tests to individual tests.

In terms of the argument that we're missing the richness of the canonical data I think we could work around that by adding additional comments alongside of the test cases.
However, from a usability standpoint, I'm all for making the tests easier to run if they're difficult to run today. In my setup I'm using Visual Studio which has the ability to run all of the cases for a single test or to run them one by one very easily, but I know different tools don't support the same functionality.
Here are the list of exercises using the TestCase attribute:

[ ] acronym
[ ] atbash-cipher
[ ] beer-song
[ ] clock
[ ] diamond
[ ] grep
[ ] isogram
[ ] luhn
[ ] matrix
[ ] meetup
[ ] nth-prime
[ ] perfect-numbers
[ ] pig-latin
[ ] protein-translation
[ ] pythagorean-triplet
[ ] raindrops
[ ] roman-numerals
[ ] series
[ ] simple-linked-list

Great work gathering this data @robkeim!

I can start working on these. I will always leave a comment here before starting to convert the tests for an exercise - this way you'll know which ones are still up for grabs :)

Sounds great, thanks @balazsbotond.

@balazsbotond Would you mind waiting until I've merged #193? Otherwise we'll have lots of merge conflicts :)

@ErikSchierboom of course!

Great!

@balazsbotond I've just merged #199, which means that this issue is now free to being worked on (if you're still interested).

@ErikSchierboom I am interested in helping with this, if you or anyone is up for a little mentoring along the way.

@morrme Great! We love to receive contributions. As for the mentoring, we'd be happy to.
Let's help you get started. First thing to note is that we are in the process of adding test data generators (#195), which is basically code to automatically generate an exercise's test file. This generation of the code is based on the exercise's canonical data, which can be found in the x-common repository.
I've looked at the remaining exercises for which to replace the [TestCase] with individual tests, and I think your best bet would be to start with the luhn exercise. If you look at it's canonical data, you can see that it is a fairly simple structure. In fact, each test is just a single input string which is passed to a function that returns a boolean. The basic infrastructure for this type of tests is already there, as you can see in the isogram test generator.
The generators can be found in the generators folder. They are built on .NET Core, so you'll need to have .NET Core installed (see this page). Once you have add your custom generator, all you need to do is to run the generators application and the tests file should be updated. You should then check to see if the default stub implementation and the example implementation still compile correctly (which you can do manually or by running the build.ps1 or build.sh command in the root directory).
If you have any specific questions, please post them in our Gitter room. Good luck and enjoy!

Hi @ErikSchierboom,

Yeah sure!

atbash-cipher,  isogram,  nth-prime, perfect-numbers, pig-latin are the one which still needs to be done, right?

@GalaDe eh, not really. If you look at the list in the second comment, you'll find the remaining exercise listed. Those you mentiond have all been checked.

Ops:) @ErikSchierboom ok:)

@GalaDe By the way, have you read this comment? In it, I explain that from now on, we want to use test data generators to generate the test files automatically. There are already several test data generators available. Links to most of the PR's can be found on this page.

@ErikSchierboom, yes, I did. Thank you!

@ErikSchierboom I'm closing this because I feel that it no longer makes sense given that we're moving to test generators (assuming the test generators are creating individual tests and not test cases).
Please feel free to re-open the issue if you disagree.

Agreed!

authors: ErikSchierboom GalaDe balazsbotond morrme robkeim

repo: exercism/xcsharp


text:
Error with token

Hello,
I am trying to set a Symfony 4.1 using PHP 7.2 to send emails but I am having an error:
[2018-08-17 10:17:50] app.ERROR: 401: {"errors":[{"message":"The provided authorization grant is invalid, expired, or revoked","field":null,"help":null}]} [] []
There is no exception being trowed but the emails are not sent and I get this error on the log file.
I could see in Travis CI that the building is being successful (not for PHP 7.2) but if you open the logs you will see the same message.
I already tried to do a cURL with the same token and it's working, so I believe the problem is with this Bundle.
Any workaround or fix preview for this!?
Cheers!

The error is supposed to exists in Travis because we don't provide any api key for automated tests.
I may have an idea, @renatoaraujo can you check your expert_coder_swiftmailer_send_grid.yaml file and check if there is an extra underscore ('_') character  at the end of api_key  line ?

Yes! It was exactly this! Now it's working.
So, another question that I have now that I took some time on the code.
The Exception that is raised in this example shouldn't be thrown instead of just logging and returning 0? Just asking because I still can do an "if" instead of try/catch.
Many thanks for your feedback.
Cheers!

As you can see here https://github.com/expertcoder/SwiftmailerSendGridBundle/blob/0880f2a71ed72298c4b0f7d8ed0b01a03ad7bf90/src/Services/SendGridTransport.php#L184 this was an original choice.
I've not planned to throw the error because it could break other people code, maybe for 3.x version some day.

repo: expertcoder/SwiftmailerSendGridBundle


text:
Remove duplicated react-native-social-share

Duplicated module from lines https://github.com/expo/react-native-libraries/blob/master/react-native-libraries.json#L247-L251

@giautm thank you so much! LGTM.

authors: giautm jimmylee

repo: expo/react-native-libraries


text:
Allow React 18 for peer deps

What:
Previously when running npm install you got an error saying that React is only supported up until version 17. You could get arround this by adding the following to project using this package:
  "overrides": {
    "react": "^18.1.0",
    "react-dom": "^18.1.0"
  },

But this PR is adding React 18.x as well so it's not needed anymore.
React 18 is really not that different from React 17. The project itself is still using React 16.
I tried this with the change, building it to a local package, installing it in another project and it worked just fine.
Why:
More projects is using React 18 every day and we don't want them to be forced to override dependencies if possible.
How:
Updated the peer dependencies for react and react-dom
Checklist:

[x] Documentation added/updated
[x] Typescript definitions updated
[x] Tests added and passing
[x] Ready to be merged

@xskipy @smac89 any chance to get this included?

@GoetzGoerisch I'm not a regular contributor to this project, but the changes look fine. @Conor-Hinchee

Ping @Conor-Hinchee

@Conor-Hinchee ?

Any update to this?

Ping @Conor-Hinchee

Ping @mrbinky3000 could you take a look please?

Thank you @Conor-Hinchee! Could your release a new version please?

Even though the semantic release does not require a new version. Could you please release a new version manually, @Conor-Hinchee?
Thank you.

authors: GoetzGoerisch Orkin jonathangraf rahimimo smac89 tonygustafsson

repo: express-labs/pure-react-carousel


text:
add pull request template + compliance check action

Based on an X conversation - it seems there's at least one coding course that instructs students to spam express. Hopefully this can help cut it down.
https://x.com/mmkalmmkal/status/1828811200802816185
I'm not 100% sure the compliance check will actually work with the default GitHub token. If not there may be a free GitHub app that can do something similar.

I'm not 100% sure the compliance check will actually work with the default GitHub token. If not there may be a free GitHub app that can do something similar.

I also think it won't work. Because GITHUB_TOKEN generated by a PR author who has no write access to this repository won't get write scope, so they won't be able to update or create comment in the PR.
We have a similar situation in https://github.com/expressjs/expressjs.com/issues/1553

I also think it won't work. Because GITHUB_TOKEN generated by a PR author who has no write access to this repository won't get write scope, so they won't be able to update or create comment in the PR.

I am not confident of it, but GitHub might allow GITHUB_TOKEN to modify the PR that triggered it in the first place. The problem in that referenced issue might come from the workflow_dispatch trigger, maybe?
I'll try it out on a test repo and report back here.

A few of us have suggested things in this same vein, but we've opted to not go this route.
I'd rather not optimize for the misguided folks and add these gates. It is a good thought, and ty for thinking of us. We close and lock at least a few of these a week, which is the easiest part of the work here 😉

Fair enough!

Yeah, this has been ongoing at a low simmer for a long time now. The one thing I really like from this PR is the disclaimer, which I think if we just did that maybe I would be onboard with.
# STOP! Read this before submitting your pull request
## ⚠ Warning ⚠
This repository is not for educational exercises or trivial changes. Pull requests that do not provide substantial value will be closed immediately.
-->```

Maybe we could just land this alone?

authors: IamLizu jonchurch mmkal wesleytodd

repo: expressjs/express


text:
Question - Stream to stdout or stderr based on status code

Is it possible to stream all the status code above 400 to process.stderr and below 400 to process.stdout?
Basically logging to two different files depending on the status code.

Yes, just put two morgan instances: One with the stdout stream and the skip set to what you don't want there and the other with stderr stream and the inverse skip.

Oh right makes sense. Thanks!

No problem! Let me know if you want an example; I didn't right now because I wanted to reply at first from a phone :)

Its cool, I understood. I''ll paste the code I'm using successfully -
app.use(morgan('combined', {
    skip: function (req, res) {
        return res.statusCode < 400
    }, stream: process.stderr
}));
app.use(morgan('combined', {
    skip: function (req, res) {
        return res.statusCode >= 400
    }, stream: process.stdout
}));

repo: expressjs/morgan


text:
is it support extism 0.3

make fail, error message is:
bigdata@Work-for-NGY:~/gitrepo/extism-sqlite3$ make
cc -shared -fPIC -o libextism_sqlite3.so extism_sqlite3.c -lextism -L.
extism_sqlite3.c: In function ‘ext_load’:
extism_sqlite3.c:60:25: error: too few arguments to function ‘extism_plugin_new’
   60 |   ExtismPlugin plugin = extism_plugin_new(context, wasm, len, false);
      |                         ^~~~~~~~~~~~~~~~~
In file included from extism_sqlite3.c:5:
/usr/local/include/extism.h:167:14: note: declared here
  167 | ExtismPlugin extism_plugin_new(ExtismContext *ctx,
      |              ^~~~~~~~~~~~~~~~~
make: *** [Makefile:2: lib] Error 1

bigdata@Work-for-NGY:~/gitrepo/extism-sqlite3$ extism info
Prefix  /usr/local
Version v0.3.0

Hi @taoistwar - I think you're right, we need to update this example to work with the latest version. Thanks for bringing this up!

@taoistwar - this should be fixed now, but please let us know if that's not the case. Thanks!

authors: nilslice taoistwar

license: ISC


text:
chore: rename package to exclude zig as its implied

@usdogu - not sure if this will compile (still setting up zig locally), but I would like the package name to be consistent with other PDKs, whereby it excludes the name of the language being used. IMO this is implied since the PDK is specific to a language / ecosystem. We tend to only use the language name in the repo URL (github.com/extism/zig-pdk), and something like extism-pdk for the actual package/library itself.

LGTM

authors: nilslice usdogu

repo: extism/zig-pdk


text:
File ICO on MacOS is not shown as good as Windows

The file ICO on MacOS is not shown as good as Windows. It is filled with a white background instead of being transparent.

Looks like a duplicate of #4 - the applications definitely need some love 😄


text:
format key might not right, so need check if the key is exist

							/*
								func (c *NumberCol) String(wb *WorkBook) []string {
									if fNo := wb.Xfs[c.Index].formatNo(); fNo != 0 {
										t := timeFromExcelTime(c.Float, wb.dateMode == 1)
										if value,ok:=wb.Formats[fNo];ok{
											return []string{yymmdd.Format(t, value.str)}
										}else {
											return []string{strconv.FormatFloat(c.Float, 'f', -1, 64)}
										}
									}
									return []string{strconv.FormatFloat(c.Float, 'f', -1, 64)}
								}
							*/
							col := row.Col(j) //has something wrong, need change above function in file col.go

repo: extrame/xls


text:
🛑 Extratone is down

In 1003bd0, Extratone (https://www.extratone.com) was down:

HTTP code: 502
Response time: 8438 ms

Resolved: Extratone is back up in d4c5b6d.

repo: extratone/up


text:
1.8版本下载后添加服务器节点。pc端可以，手机端会导致整个网络访问不了包括国内的网站

你正在使用哪个版本？
1.8
使用最新版本是否还遇到同样的问题？
你的手机型号和手机操作系统版本是多少？
荣耀V10 安卓9.0
你遇到了什么问题？
上不了网
你遇到的问题可以用固定步骤重现吗？

你正使用的配置
{
"dns": {},
"log": {
"loglevel": "none"
},
"outbounds": [    {
"protocol": "vmess",
"settings": {
"vnext": [          {
"address": "www.xxxxx.xyz",
"port": 443,
"users": [              {
"alterId": 1086,
"id": "72e08bef-3d8c-4f8d-99be-679a2795ae3c",
"security": "auto"
}
]
}
]
},
"streamSettings": {
"network": "ws",
"security": "tls",
"tlsSettings": {
"allowInsecure": true
},
"wsSettings": {
"headers": {
"Host": "www.xxxxx.xyz"
},
"path": "/v2raypath"
}
},
"tag": "proxy"
},
{
"protocol": "freedom",
"settings": {
"domainStrategy": "AsIs"
},
"tag": "direct"
},
{
"protocol": "blackhole",
"settings": {},
"tag": "block"
}
],
"policy": {
"levels": {
"0": {
"bufferSize": 4096,
"connIdle": 300,
"downlinkOnly": 0,
"handshake": 4,
"uplinkOnly": 0
}
}
},
"routing": {
"domainStrategy": "AsIs",
"rules": [      {
"network": "tcp,udp",
"outboundTag": "proxy",
"type": "field"
}
]
}
}

Is this working ?

How can i get it on my kitsunebi app??

Thank you. The problem has been solved!
y9861215ai@163.com
发件人： Joe
发送时间： 2019-12-13 23:48
收件人： eycorsican/kitsunebi-android
抄送： 深海藏蓝; Author
主题： Re: [eycorsican/kitsunebi-android] 1.8版本下载后添加服务器节点。pc端可以，手机端会导致整个网络访问不了包括国内的网站 (#125)
How can i get it on my kitsunebi app??
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

hello please help  github is complex I don't know where to start from

authors: JAIR1314 heisiam unclejoesh

repo: eycorsican/kitsunebi-android


text:
add extra include file , to avoid build error in VS2019

In fact, there is no build issue in VS2017.
But may be some header file change in VS2019, it fails.

Thanks for your PR.

authors: mythay orca-zhang


text:
unSupported: ltm monitor postgresql /Common/int-postgres-monitor

Environment

Application Services Version: ACC 1.15
BIG-IP Version: 15.1

Summary
While trying to convert a virtual server using a custom monitor for postgresql, I am getting an unsupported object error from ACC. However, I am not sure what is exactly unsupported. Is it the entire monitor? or a specific attribute?
Steps To Reproduce
Steps to reproduce the behavior:

Convert following config:

ltm virtual /Common/dc1_dev_corp_int_web_app_backend_vs {
    creation-time 2021-08-23:05:43:55
    description "F5-34532 - DEV"
    destination /Common/10.1.10.16:8080
    ip-protocol tcp
    last-modified-time 2021-09-16:15:06:00
    mask 255.255.255.255
    pool /Common/dc1_dev_corp_int_app_pool_backend
    profiles {
        /Common/tcp { }
    }
    serverssl-use-sni disabled
    source 0.0.0.0/0
    source-address-translation {
        type automap
    }
    translate-address enabled
    translate-port enabled
}
ltm pool /Common/dc1_dev_corp_int_app_pool_backend {
    members {
        /Common/int_apps.example.com:5432 {
            address 10.1.20.10
            description F5-12346
            session user-disabled
        }
    }
    monitor /Common/int-postgres-monitor
}
ltm monitor postgresql /Common/int-postgres-monitor {
    count 0
    database pgdb_db
    debug no
    defaults-from /Common/postgresql
    destination *:*
    interval 5
    recv f
    send "select pg_is_in_recovery()"
    time-until-up 0
    timeout 10
}

Observe the following error message:

unSupported: {
    'ltm monitor postgresql /Common/int-postgres-monitor': {
      count: '0',
      database: 'pgdb_db',
      debug: 'no',
      destination: '*:*',
      interval: '5',
      recv: 'f',
      send: '"select pg_is_in_recovery()"',
      'time-until-up': '0',
      timeout: '10'
    }
  },

Expected Behavior
Convert my postgresql monitor to AS3 successfully.
Actual Behavior
See error above returned by ACC.

Thank you this is being tracked internally as CHARON-472


text:
[BUG] Syntax Error:(/config/bigip.conf at line: 7346) ";" unknown property

Describe the bug
Opening this bug on behalf of Chris. Tried in his lab to migrate a BIG-IP from vCMP to VELOS but failed restoring the modified UCS with the following error message when trying to do a tmsh load sys config after platform-migrate command completed.
Syntax Error:(/config/bigip.conf at line: 7346) ";" unknown property
We also noticed the deployment never ends fails or succeeds in Journeys.
To Reproduce
Steps to reproduce the behavior:

Get UCS from SR 1-6993580048\INCOMING\jun_10\lab02.ucs
Restore the UCS on a vCMP guest 14.1.4.6
Use Journeys 3.3.2 to migrate the config to a VELOS tenant 14.1.4.6 after resolving all the errors in the tool

Error
[root@bipmn013lab02:/S2-red-P::Offline:Disconnected (Sync Only)] config # tmsh load sys config
Loading system configuration...
/defaults/asm_base.conf
/defaults/config_base.conf
/defaults/ipfix_ie_base.conf
/defaults/ipfix_ie_f5base.conf
/defaults/low_profile_base.conf
/defaults/low_security_base.conf
/defaults/policy_base.conf
/defaults/wam_base.conf
/defaults/analytics_base.conf
/defaults/apm_base.conf
/defaults/apm_oauth_base.conf
/defaults/apm_saml_base.conf
/defaults/app_template_base.conf
/defaults/classification_base.conf
2022 Jun 10 18:55:57 slot3 logger[4145]: Re-starting lind
/var/libdata/dpi/conf/classification_update.conf
/defaults/ips_base.conf
2022 Jun 10 18:55:58 slot1 logger[2881]: Re-starting lind
/var/libdata/ips/ips_update.conf
/defaults/daemon.conf
/defaults/pem_base.conf
/defaults/profile_base.conf
/defaults/sandbox_base.conf
/defaults/security_base.conf
2022 Jun 10 18:55:58 slot3 logger[4413]: Re-starting mcpd
/defaults/urldb_base.conf
/usr/share/monitors/base_monitors.conf
/defaults/cipher.conf
/defaults/ilx_base.conf
Loading configuration...
/config/bigip_base.conf
/config/bigip_user.conf
/config/bigip.conf
Syntax Error:(/config/bigip.conf at line: 7346) ";" unknown property
Context

Please specify, app version: 3.3.2
And Platform version of the system you want to migrate: vCMP guest to VELOS tenant 14.1.4.6

I encountered this recently, as well.  In my original UCS archive, this was the config stanza being processed:
security dos bot-signature "/Common/Citrix ICA Client" {
    category "/Common/Service Agent"
    risk na
    rule "headercontent:\"\\Citrix\\ICA Client\\\"; useragentonly; nocase;"
    user-defined false
}

This is what Journeys generated.  The line that reports a syntax error about is the rule line.  Note that Journeys has malformed the escaping:
security bot-defense signature "/Common/Citrix ICA Client" {
    category "/Common/Service Agent"
    risk na
    rule headercontent:"\\Citrix\\ICA Client\\"; "useragentonly;" "nocase;"
    user-defined false
}

The failure is evident just trying to load that config stanza onto a running v15.1.5.1 system:
# cat /var/tmp/journeys-66
security bot-defense signature "/Common/Citrix ICA Client" {
    category "/Common/Service Agent"
    risk na
    rule headercontent:"\\Citrix\\ICA Client\\"; "useragentonly;" "nocase;"
    user-defined false
}
# tmsh load sys config merge verify file /var/tmp/journeys-66
Validating configuration...
  /var/tmp/journeys-66
Syntax Error:(/var/tmp/journeys-66 at line: 4) ";" unknown property
#

Is this a v14 problem?  The same stanza in v15 has a 'factory-rule' defined that is encoded:
security bot-defense signature "/Common/Citrix ICA Client" {
category "/Common/Service Agent"
factory-rule $M$wA$pbtF+8Up02HOaZ7awz6+CmNmu8+dnR0Em5qvNceInZGW5FqQ56z0EWq/AwotbC/W6e5bbSUJ5hba004YgrkR4A==
risk na
user-defined false
}

Is this a v14 problem? The same stanza in v15 has a 'factory-rule' defined that is encoded:

Sort-of.
It may only occur when the source configuration file originated in v14.1.x or earlier.
In my case, Journeys was migrating a configuration from v15.1.x to v15.1.x.  I no longer have the UCS archive around, but I suspect the source device was originally on v14.1.x and was upgraded to v15.1.x at some point in the past.
In other words: it's entirely possible to have systems on v15.1.x or above that still have that particular stanza in bigip.conf.
I haven't tested, but I assume it's also possible for a user to configure a similar signature rule that trips over the incorrect escaping.

The bug was fixed in release 3.3.5


text:
[RFE] - ASM XML in Device Discovery

When connecting to a device or uploading QKView, UCS, etc, it'd be nice to discover ASM/AWAF policies in XML format.  This can lend itself to WAF as Code and position conversion from XML to JSON.  Maybe using the docker tool to convert XML to JSON from the NGINX team.
https://docs.nginx.com/nginx-app-protect/admin-guide/install/#converter-tool-docker-image

would you happen to know where in the ucs/qkview these files are located?  we can look at adding them to corkscrew


text:
Fix typo in argument handling

Set function args for the cip30Function using array.splice, not array.slice
remove debug logs

I'm closing this and creating a new pr that will only contain the needed changes and follow the contribution guidelines.

repo: fabianbormann/cardano-peer-connect


text:
How to provide mocked named dependency?

We are using DaggerMock rule because of convenience of declaration field, which is injected then into dependency graph. With named dependencies, I would expect next api possible
public class MainServiceTest {

    @Rule public final DaggerMockRule<MyComponent> rule = new DaggerMockRule<>(MyComponent.class, new MyModule());

    @Mock RestService restService;

    @Mock @Named("mykey") MyPrinter myPrinter;

    @InjectFromComponent MainService mainService;

    @Test
    public void testDoSomething() {
        when(restService.getSomething()).thenReturn("abc");

        mainService.doSomething();

        verify(myPrinter).print("ABC");
    }
}

But it doesn't work after upgrading from 0.6.3 to 0.6.4. @Named fields are not replaced at the graph.
Would you consider such api as valid feature request?

It depends how you define MyPrinter in the module, if you define it using the same Named annotation it should work. Here you can find a test that verify this behavior.

Thank you for quick reply.
I checked the sample and it worked. Then I get back to my project and turned out issue was not with DaggerMock :man_facepalming: Sorry for taking you time.

authors: fabioCollini plastiv

repo: fabioCollini/DaggerMock