text:
Issues in "Properties" field

the geolocation info in Properties field turns orange and cannot be change (unless open .md file external and edit it manually).

(there is also a interesting way to change it: in map right click, "new note here(front matter)", copy the field, place it in the original file after deleting the old one.)

right-click-menu: "copy geolocation as front matter" wont work unless the file is blank (or the Properties field ist blank).

for now, the "location" way has issues, but "locations:" way works great.
Regarding the funny way of changing the "location:" filed that mentioned above, I was wondering, if it's a good idea to add one more option in right-click-menu: "change the location property in current file". which I think is a simple way to fix the first issue.
Thanks a lot for making and maintaining this great Plugin!

the same problem after the ob version is updated to V1.4.5, the traditional way of adding loctions in font matter is not available

Same problems here. "Convert to Geolocation (geosearch)" isn't working for me either since updating to 1.4.5 (using Google API), but that could be a separate issue.
This is a great plugin! I'm surprised Obsidian didn't include "Location" (or "Coordinates"?) as a Property type.

Same, I just opened Obsidian and it seems it's auto updated without asking me and this plugin no longer works properly. Existing notes show on the map, but the coordinates of new notes don't change orange and don't show on the map.

Same here.  Until a fix is developed, a workaround is to use inline location URLs instead of frontmatter.  From the documentation:

Another way that the plugin parses location data is through inline location URLs in the format of [link-name](geo:40.68,-74.04), which allow multiple markers in the same note. To prevent the need to scan the full content of all your notes, it requires an empty locations: tag in the note front matter ('locations' and not 'location').

The Obsidian devs guaranteed that an API for plugins to define custom field types are coming, and hopefully this will bring back the ability to use the front matter as we're all used to.
In the meantime, I am unsure why location is not recognized as a YAML list, as it should be...
It sounds like you all are editing the YAML manually, which I never do, which explains why I didn't run into this problem in spite of using "properties" for a very long time (I'm an Obsidian alpha tester, so been running with this even before it was available to insiders). AFAIK the tools for adding geolocations to a note work as usual -- I'm using "add geolocation (front matter)" on an almost-daily basis with no issues.

Do I understand correctly, and is the issue indeed only when editing YAML manually?
If so -- why do you all edit YAML manually? The way I see it, adding locations via the "add geolocation" command (or menu item in a note) is far easier. You can enter a location name or coordinates and it should work out of the box. Same goes for "new note here (front matter)" etc. If it doesn't work, I'd love to hear more details.
I'm not happy about workarounds, but note that you can also use the Obsidian editor's "source view", which will also let you edit the YAML manually.

If so -- why do you all edit YAML manually? The way I see it, adding locations via the "add geolocation" command (or menu item in a note) is far easier. You can enter a location name or coordinates and it should work out of the box. Same goes for "new note here (front matter)" etc. If it doesn't work, I'd love to hear more details.

I have found that many places I add don't show up in the search for "add geolocation", so I've gotten into the habit of just pasting coordinates from Google Maps into front matter directly (I have a place template that includes a blank "location:" in the front matter).  I had not realized that I could paste the coordinates into the "add geolocation" search, so that fixes the problem I had been having.  Thanks for that information!

One of my goals in Map View is to make the flow as automatic as possible, so the more the plugin progressed, I was trying to minimize the occasions on which you would need to edit YAML fields or coordinates. Maybe I should give this more emphasis in the documentation.
That being said, "properties" broke more than just the ability to edit the YAML, and I really hope the Obsidian devs would hold to their promise in exposing an API for field types.

Thanks for the replies!
Actually, the issue is about renew/editing the existed geolocation. It was easy to change a location (just copy and paste). But as all we all realized, the location field is not recognized and cannot be change.
but in my case, I have to change my geolocation (or move) sometimes and a compromised solution for me is

"new note here(front matter)", copy the field, place it in the original file after deleting the old one

(Answer to Q1: editing the YAML manually is just another compromised solution, but it's really not easy/ stupid.)
so regarding this way, I suggest, add one more option in right-click-menu: "change the location property in current file", so that we dont have to create a new file only to change a location. which I think would be a simple solution for now, and it remains a good small option for the guys who need to change their location even if the API comes and things works normal again. : )

Another way to currently renew/edit a geolocation is to remove it (click the icon of the property and click "remove" in the menu) and re-add using the regular Map View command.
That being said, it's not a replacement for better tools of modifying markers, nor instead of the need to have Obsidian recognize a geolocation property.

I think this issue simply stems from Obsidian's YAML formatting requirements in list properties. For instance, using the following list properties to test produces:
location working:
  - "23.57"
  - "23.76"
location:
  - 23.57
  - 23.76
List Test:
  - "23"
  - "2345678"
  - "432.645334"
  - Bob
  - "[[Bob]]"
  - Bob the Builder
  - "#$@##%"

Obsidian wants " " around any numbers in list properties. However, adding quotations that way causes Map View to no longer recognize the numbers as a location. Couldn't the solution just be to make Map View recognize - "23.57" as equivalent to - 23.57 ?

Hi, any news about how to insert coordinates in property field?

I'm not sure what's doable in my jurisdiction to improve direct editing of front matter coordinates, as the Obsidian props editor still hasn't introduced an API or support for geolocations.
Ideas are very welcome.
One thing I do have in mind, is that in the plugin's docs I can hint that editing coordinates manually is probably the least convenient way to add geolocations.

So since by default the location is added as [x,y], why don't have it added by default as "[x,y]"? It will be then recognised as a string and editable.

@Satanniel I agree, this would fix the problem completely.

I have the same issue of not being able to change the location, but I came up with a workaround using a combination of the Commander plugin and the Linter plugin, which I hope may be helpful towards finding a more permanent solution.
Here is what I did:

In Linter plugin settings, turn off every setting except for “Remove the YAML keys specified” in the “YAML” section of the plugin settings.
Still in the Linter plugin settings, in the “YAML Keys to Remove” setting, I set the key I wanted to remove. These two Linter steps effectively make the Linter do only one thing: remove the "location" property.
In the Commander plugin settings, I went to the "Macros" tab and clicked "Add Macro."
In that Macro, I added the following three commands:
a) Linter: Lint the current file (which removes the "location" property of the current file)
b) Delay (for 2050 milliseconds) (this is optional, but I added a small delay to make sure that the location adding command doesn't accidentally happen before the Linter has time to delete the location key)
c) Map View: Add geolocation (front matter) to current note (this now allows you to add the geolocation to the current note, now that the old one has been deleted)
Now just set a keyboard shortcut for the new macro made, and/or add it to the right-click menu, and/or add it to the file's menu (top right three vertical dots). It is now extremely easy to change the location of any note.

The weaknesses of this method I have just described are as follows:

Creates a dependency on two other plugins
Makes it not possible to use Linter plugin for anything else
Still cannot copy the geolocation in a convenient way

Should now be fixed as part of 5.0.0.

authors: Comprehensive-Jason Marco-Mindstone Satanniel Xieleilab architaktus cjt3 esm7 mfloreen mikescott rchaklashiya


text:
fix status spec on Ruby2.3

It fixes #163 spec errors.

thanks!

source: gharchive/pull-request

repo: esminc/adhoq

license: mit

license_type: permissive

license_source: bigquery


text:
Broadcast send for Espnow

Add the implementation for sending broadcast messages with Espnow

Thanks, sorry for opening this.

source: gharchive/issue

repo: esp-rs/esp-wifi

license_source: github-api


text:
The ESP32 cannot ping the router

I used an ESP32 and a router to collect channel status information, set the router as AP and ESP32 as STA, and finally used ESP32 to ping the router, which failed. Come to consult。

Excuse me, I would like to ask you about the difference between ESP32 and Intel5300 in collecting CSI. Could you give me some advice?

@2422092593zhangshuo See https://github.com/espressif/esp-csi/issues/22

Thank you very much. Is the length of CSI  received by ESP32 related to the number of antennas on the router?

The CSI data I collected was carried out under the condition of 20MHz HT,no seconary_channel and stbc. The length of CSI data is 256，how should LLTF and HT-LTF subcarrier indexes be distinguished. Could you give me some help? Thank you

@2422092593zhangshuo
#define CSI_SUBCARRIER_LLTF_LEN                  52 /**< 6 ~ 31 33 ~ 58 ---> 52 */
#define CSI_SUBCARRIER_HTLTF_LEN               56 /**< 66 ~ 122       ---> 56 */
#define CSI_SUBCARRIER_STBC_HTLTF_LEN   56 /**< 123 ~ 191      ---> 56 */

Thanks you for the reply. Calculated in this way, LLTF_LEN is  52,HTLTF_LEN is 56 STBC_HTLTF_LEN is 68.
what's your opinion? Could you give me some advice? Thank you.

Thanks you for the reply. Calculated in this way, LLTF_LEN is  52,HTLTF_LEN is 57 STBC_HTLTF_LEN is 69.
what's your opinion? Could you give me some advice? Thank you.

Thanks you for the reply. Calculated in this way, LLTF_LEN is 52,HTLTF_LEN is 57,STBC_HTLTF_LEN is 69.
what's your opinion? Could you give me some advice? Thank you.

Because some of the sub-carriers have very small amplitude variances, they are filtered

Thank you.The length of CSI data I collected was 256.
#define CSI_SUBCARRIER_LLTF_LEN                  52 /< 6 ~ 31 33 ~ 58 ---> 52 */
#define CSI_SUBCARRIER_HTLTF_LEN               56 /< 66 ~ 122       ---> 56 */
Can I take it this way？

This needs to be judged based on the received Wi-Fi communication protocol, see: https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-guides/wifi.html#wi-fi-channel-state-information

Hello, @2422092593zhangshuo
Closing this issue, feel free to reopen with more updates. Thanks.

repo: espressif/esp-csi


text:
Missing free in ws_echo_server (IDFGH-10244)

https://github.com/espressif/esp-idf/blob/f8d2c0894cf3c12dc231dafebf75d3f1002b312c/examples/protocols/http_server/ws_echo_server/main/ws_echo_server.c#L60
If function fails the memory is lost and will never be freed.

@masterxq Thanks for reporting. We will fix it.

authors: ESP-YJM masterxq

repo: espressif/esp-idf


text:
Inquiry about Broadcast Interval and Message Sending Limitation in BLE Mesh Testing (IDFGH-12352)

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
idf version:v4.4.6-323-g979a56386d
I want to test the communication success rate, and since BLE mesh uses 1Mbps modulation, I need to create 10% background traffic in the network. Based on the rate of 1Mbps = 125,000 bytes/s, with a message length of 47 bytes including an 11-byte information link layer, I need to send 265 messages per second to achieve this goal.
During testing, I used the Vendor Server Model and called the esp_ble_mesh_server_model_send_msg() function to send undivided messages with a payload length of 11 bytes (including a 3-byte opcode, with actual data length of 8 bytes), and these were unacknowledged messages. I have already disabled the message retransmission mechanism.
I would like to know if there is a broadcast interval when I call this function consecutively twice to send messages. Additionally, during testing, I found that when using a for loop to continuously send messages, I could send a maximum of 50 messages before the sending functionality became inactive, requiring device reboot.
Thank you for your help!

The following image shows capturing a data packet using a Bluetooth sniffer. It can be observed that a data packet can be sent within 1ms. In theory, 1000 data packets should be able to be sent within 1 second.


text:
USB Audio device on ESP32-S3 : Hitting upper limit of open endpoints when I should not; issue in portable/synopsys/dcd_dwc2.c? (IDFGH-12779)

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 am implementing a headset (speaker+mic) on ESP32-S3. I notice that when I pause the playback or record 3 times, playback or record does not happen any more.
I traced the issue to a couple of TU_ASSERT in dcd_edpt_counter_release(). When I pause playback, the endpoint is released and opened again when I start playing.
// Release an endpoint.
static bool dcd_edpt_counter_release(uint8_t rhport, uint8_t dir)
{
  if (dir) {
    TU_ASSERT(dwc2_ep_counters[rhport].active_in == 0);    // Check if number of opened EPs is not zero
    dwc2_ep_counters[rhport].active_in--;                  // Decrease endpoint counter
  } else {
    TU_ASSERT(dwc2_ep_counters[rhport].active_out == 0);
    dwc2_ep_counters[rhport].active_out--;
  }
  return true;
}

Obviously the ASSERT fails when active_in and active_out are >0 and they are not decremented. Do this 3 times and total number of open endpoints are 6, which the limit. Subsequently, dcd_edpt_counter_available() returns false.
When I changed '==' to '>', the problem went away. It makes sense to me. Also the comment implies that it should be '!=' or '>'.
Anybody else faced this issue? Am I missing something or thinking it wrong?

@giri-a,
Thanks for reporting the issue.
The fix was already merged (https://github.com/espressif/tinyusb/pull/25), we will publish the new version soon.
Sorry for inconvenience.

Hi @giri-a,
There is a new version of espressif/tinyusb is available (v0.15.0.9).
If there will be anything else or you will run into any other issue, fell free to reopen this ticket or create a new one.
Thanks for your help.

Perfect. That was quick. Thanks.

authors: giri-a roma-jam


text:
Error in file find while adding component to project. (IDFGH-157)

hi.  I am working on Eclipse and am using esp-idf.  I want to communicate with the cloud using IBM-watson-iot libraries. However, although I struggled a lot, I still could not include this library in the project. The operations I have done to include the library in the project and the errors I get are as follows. What would you suggest me to solve the problem?
I compile the project on windows OS.

I downloaded the Embedded C Client Library from repository
I extracted the files and ran the setup.sh file.
I combined the src file and the lib files.
then I moved this lib file to the components folder in the project file I created in Eclipse.
the new project structure is shown in the picture below.

I created one component.mk file under the lib folder and the contents are as follows:

COMPONENT_ADD_INCLUDEDIRS := mbedtls
CFLAGS += -Wno-error=implicit-function-declaration -Wno-error=format= -DHAVE_CONFIG_H

and when I try to compile the project I get the error "components / lib / mbedtls / iotf_network_tls_wrapper.h: 27: 24: fatal error: sys / select.h: No such file or directory".

The issue may not be related to project structure. ESP-IDF does not provide sys/select.h.
select is currently declared in sys/socket.h (which is the same as lwip/sockets.h).

IDF now provides sys/select.h and select implementation which works both for files and sockets. Related example: https://github.com/espressif/esp-idf/tree/master/examples/system/select.

authors: igrr zafersn


text:
MOSI active on spi_device_transmit with tx_buffer = NULL (IDFGH-1228)

Environment

Development Kit: none
Module or chip used:  [ESP32-WROOM-32D]
IDF version: v3.3-beta1-268-g5c88c5996
Build System: [CMake]
Compiler version: 1.22.0-80-g6c4433a
Operating System:     [Linux]
Power Supply: [USB|Battery]

Problem Description
When i perform an spi_device_transmit with no tx_buffer (spi_transaction as below) the MOSI line shows erratic activity instead of being held low. This can stop spi devices from sending or even change configuration.
   spi_transaction_t transaction;
    transaction.flags = 0;
    transaction.cmd = 0;
    transaction.addr = 0;
    transaction.length = bytesNumber * 8;
    transaction.rxlength = transaction.length;
    transaction.user = NULL;
    transaction.tx_buffer = NULL;
    transaction.rx_buffer = data;
    esp_err_t err = spi_device_transmit(spi_h, &transaction);

Once i assign a tx_buffer that contains actually all zero values the MOSI line is held low as expected.
    spi_transaction_t transaction;
    transaction.flags = 0;
    transaction.cmd = 0;
    transaction.addr = 0;
    transaction.length = bytesNumber * 8;
    transaction.rxlength = transaction.length;
    transaction.user = NULL;
    uint8_t d[32] = {0};
    transaction.tx_buffer = d;
    transaction.rx_buffer = data;
    esp_err_t err = spi_device_transmit(spi_h, &transaction);

This is probably a problem in the spi_master driver that should be addressed or at least included in the documentation.

I think it uses data from the non-DMA buffer, thanks for reporting this

authors: ginkgm itsMeBrice


text:
Is there any way to suspend and resume esp modbus slave task? (IDFGH-8460)

Is your feature request related to a problem?
Hello,
Currently I am working on the non WiFi Modbus implementation on ESP32. I want to discuss regarding the code implementation for my requirement.
My project has two UARTs, one is connected to Modbus Master device and other is connected to the Modbus Slave device.
Master device functionality is to keep poling holding registers from the slave device and display its value on local display. It also write holding register values to the Slave device.
Cloud part will going to add later where slave device's all register values are updated to cloud and any value updated from cloud is write on the slave device.
On ESP I have started with esp-modbus component, ESP act as master for slave device and slave for master device. On ESP I have implemented master task which has polling scheduler for periodic reading of slave device registers and store it into uint16 type array. Slave task is used to send response master device for that same uint16 array is passed to mbc_slave_set_descriptor API. The uint16 array is basically cache values of the slave device.
Currently, all parameters from slave device keep updating in the array as well master device also gets values. So for reading functionality for master device everything is working fine.
But when master device is writing any Parameter, slave task is simply writes value in array and sends response back to master device. Master device then start polling again for reading holding registers. On ESP, write event is detected by checking mbc_slave_check_event API. My requirement is to write values to slave device through master task and pass its response back to master device.
Describe the solution you'd like.
Is there any way whenever write request receive from Master device, I can pause the slave task, write data to salve device through master task, after completing write on slave device send response back to Master device and finally resume the slave task to get read requests from master device.
I check the slave source code, in mbc_serial_slave_start function eMBInit is called which has call backs for pvMBFrameStartCur, pvMBFrameStopCur functions used to resume or suspend the vUartTask for Modbus UART . These callbacks are called using eMBEnable and eMBDisable. I thought of use these API to stop Uart task for slave task and resume as per my requirement. But these are private API and not sure if it is the correct way.
Describe alternatives you've considered.
Alternately, Instead of using slave from esp-modbus I can implement low level UART APIs to receive request and send response back to master device. In this case I need to implement all Modbus parsing and error handling.
Suggest me if any better option I can implement.
Additional context.
No response

Hello
Did you find a solution or workaround for this issue?
Thank you

@alisitsyn : Thanks for your Reply.
I already write my own functions for Modbus slave implementation. In future if required I will get back to you.

@Rbhagwat62,
Thank you for the update. Unfortunately the v1.0.x will not be updated with the new features and I can not change the public interface of it. So, please update it on your side using the example from v2.0.x. Feel free to reopen the issue or ask the questions at any time. Good luck!

authors: Rbhagwat62 alisitsyn mlotorto


text:
ci(github-precommit): Add skipping rules with submodules in pre-commit

DO NOT MERGE ON GITHUB - This is a test PR
Added skipping specific rules in GitHub CI pre-commit action

 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.

authors: CLAassistant tomassebestik


text:
wemos d1 unstable

I bought a bunch of wemos d1 style devices to use however not having much luck.
Running my version of R2 with the previous fixes for RC522 readers but I'm currently trying to use PN532 reader modules as they're smaller.
If I have a PN532 connected the system doesn't appear to broadcast it's wifi SSID
If I go back to a RC522 reader it works but it's slow and card reads are few and far between.
Previously my testing was done with AI-Thinker modules
I guess I'll have to run some diagnostic build to see if there's any errors or what not but I'm just putting this up for now, I'm a little bit frustrated as I need a reliable stable solution for user and machine tracking in the hackspace.
Happy to work on this.
Thanks

Something I recalled from one of the other posted issues was about wifi fallback causing delays.
I was testing with it in AP mode but now that I've configured it to connect to my wifi it does seem a lot more stable

@plex3r we've had similar issues as have others, our in service unit uses a nodemcu and we have test platforms on another nodemcu and on the mark 1 ESPRFID board... all give very poor performance in AP mode and in fallback mode either running really slow or causing reboots from the watchdog. Our normal in use machine works tolerably well (though again not without issues) when running in client mode. In order to keep the wifi for the RFID separate from the normal hackspace user wifi we have a RasPi setup as an AP and MQTT broker that the nodemcu connects to in client mode.
I noted a comment yesterday by @croggero in #615 that refers to #604 that might offer clues as to the problems that might be worth investigating
ooo-xxx

@plex3r @Divyastra I have three devices always on managing a few dozen users every day and they work fine. They reboot every now and then, but I'm not sure if it's due to the power supply or temperature as they are not in temperature controlled spaces.
If you get these problems it might be due to the setup and pins that you are using.
Now I have a various range of ESPs and sensors, if you send me the configuration that you are using, both software and hardware I can try to replicate and see if there's a bug.
Depending on the device you cannot freely use the available pins of the ESP. Some pins have special usages and they will cause issues on esp-rfid if you use them for your sensors.
I'm going to close this for now as I don't have enough information to reproduce, but if you can replicate and provide more info I will have a look. Cheers

repo: esprfid/esp-rfid

license: MIT


text:
[widget_utils] swipe-down Widgest are not rotated when Lefty flip enabled.

Affected hardware version
Bangle 2
Your firmware version
2v17, 2v17.71(Cutting Edge)
The bug
The bug
"Slide down" gesture to show hidden widgets shows wrong when clock is set up in lefty LCD orientation
Installed apps
No response

Hi, thanks - so just to be clear, what actually happens? They respond to a swipe down from the top, but then they slide in from the bottom?
The may be a firmware issue since the overlay is handled in that, and I guess it should probably honour the system rotation setting

Ok, thanks - I've just put a fix in but you'll have to update your Bangle to the latest 'cutting edge' firmware fot it

authors: gfwilliams kamilkrz

repo: espruino/BangleApps


text:
[fuzzyw] Moving translations for Fuzzy Work Clock

As Gordon suggested in #1659 , moved the translations from a custom file into the standard lang files, as per #1311
Removed the settings as they weren't really needed anymore.

Excellent - thanks! It's also great to have text translations for numbers in a bunch of different languages now :)

authors: gfwilliams sir-indy


text:
qcenter - Fastload the clock on swipe

qcenter is essentially a launcher and should be able to go back to the clock like one. Needs #2331 to actually trigger fast loading because else the remove method is not set for setUI. Should do close to nothing without #2331.

Looks great - thanks!

authors: gfwilliams halemmerich


text:
widmsggrid: provides a "message" widget

Looks like 3e5c0538a9776014a7f99d8c3521d1372dde8a90 missed this, I was wondering why the default widget kept getting reinstalled ;-)

Great - thanks!

authors: gfwilliams rigrig


text:
Added a script for doing RASH+HTML checking.

This script enables RASH users to check their documents simultaneously both against the specific requirements in the RASH grammar and also against the full set of HTML checks that the W3C Nu HTML Checker (validator.nu backend, aka HTML5 validator) does for all HTML documents (to check all requirements given in the HTML spec). This ensures that RASH users get alerted to more potential mistakes in their docs so that they can fix them (not only to RASH-specific problems). Among other things above just using the RASH grammar only, this adds relatively sophisticated checking of the datatype microsyntaxes of attribute values.
It’d be fairly easy to rewrite this script in Python, to have something for Windows users to run.

Dear Michael,
very thanks for this, it's just amazing. I was wondering if you would like to became an "official" contributor of the main RASH project (github.com/essepuntato/rash) - actually, would be an honour to have you among the developers. What do you think?

Sure yeah I’d be very glad to help. I like the ambition of this project, and the idea of a Web-first authoring format for writing academic papers that's pure text/html HTML and that's doesn't need to be well-formed-XML XHTML. And your Web-based display engine is great. So I think you’re really headed in the right direction with this and I’d be very glad to help make it successful, so yeah, please sign me up. 😄

authors: essepuntato sideshowbarker

repo: essepuntato/rash

license: isc


text:
NaN years ago

This app appears to be abandoned. Retracting.

@rex4539 this is not abandoned but upgraded to v2: https://github.com/esteemapp/esteem-mobile
We will soon archive this repo after officially depreciation of v1 from app stores.

repo: esteemapp/esteem


text:
ngTable with  promise tracker

ngTable support busy indicator on http requests. can use promise tracker and intercept http request. why not having in build directive for busy indicator.

?????????????????????

repo: esvit/ng-table


text:
VCPKG build error under windows

The latest release of the library doesn't compile with VCPKG on windows.
Please make a new release with a working vcpkg/etcd-cpp-apiv3 set of configuration files since REF 1b68f9c79dede1264a0977a239e497358760b0ac is not compiling.

I have just tag a v0.2.8 release. May I know the error message in your build failure?
Thanks!

install-x64-windows-dbg-out.log
Here the log with all the errors. Derived from missing "#define NOMINMAX" which is not part of the tag 1b68f9c79dede1264a0977a239e497358760b0ac present in the portfile.cmake.

Will bump up the revision in the vcpkg script once I have access to my Windows laptop.

authors: sdeastis sighingnow


text:
Add a print sanity function

It would be quite useful for debugging sanity expressions to have deferrable version of print that would print its arguments and then return them, so that you insert it anywhere in a sanity expression:
self.sanity_patterns = sn.assert_eq(sn.count(sn.extract_all(...)), 10)

# print the extracted elements
self.sanity_patterns = sn.assert_eq(sn.count(sn.print(sn.extract_all(...))), 10)

I have this ready, I am just waiting for #1050 to be merged. @teojgo Can you approve and merge it?

@vkarak I merged it.

authors: teojgo vkarak

repo: eth-cscs/reframe


text:
[feat] Allow Syslog's address to be set from an environment variable

Also rename RFM_GRAYLOG_SERVER to RFM_GRAYLOG_ADDRESS for symmetry. RFM_GRAYLOG_SERVER is now deprecated.

Fixes #1056.

Codecov Report

Merging #1427 into master will decrease coverage by 0.00%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master    #1427      +/-   ##
==========================================
- Coverage   91.83%   91.83%   -0.01%
==========================================
  Files          82       82
  Lines       12786    12798      +12
==========================================
+ Hits        11742    11753      +11
- Misses       1044     1045       +1

Impacted Files
Coverage Δ

reframe/frontend/cli.py
78.52% <50.00%> (-0.36%)
:arrow_down:

unittests/test_logging.py
98.26% <0.00%> (+0.03%)
:arrow_up:

reframe/core/logging.py
83.38% <0.00%> (+0.47%)
: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 f6111d7...070a903. Read the comment docs.

authors: codecov-commenter vkarak


text:
HTTP Error 503: Service Unavailable

首先感谢开发这么实用的插件，我之前还用的好好，今天突然不能用了debug提示错误：
[2017-05-17 21:13:07][ERROR: input.scriptfilter] JSON error: JSON text did not start with array or object and option to allow fragments not set. in JSON:
HTTP Error 503: Service Unavailable
<items />

你好，这个可能是 proxy 的问题，你看看是不是设置的问题

@ethan-funny 感谢，现在好了，之前应该是双重代理了。

authors: ethan-funny xinnyu


text:
Remove backup scanner

Using the provider without watch functionality will end up with an Error throw by scanner. This has to be handlede by top layers (UI, CLI)

Pull Request Test Coverage Report for Build 1209

2 of 3 (66.67%)  changed or added relevant lines in 1 file are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 83.147%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Scanner/Scanner.ts
2
3
66.67%

Files with Coverage Reduction
New Missed Lines
%

src/Actions/Pending.ts
4
37.84%

Totals

Change from base Build 1203:
-0.2%

Covered Lines:
538

Relevant Lines:
624

💛  - Coveralls

repo: ethereum-alarm-clock/timenode-core


text:
Improve Docs of KZGSettings

Minor improvements to the documentation of KZGSettings + Remove domain_size in these docs.

LGTM! Thanks!


text:
Add "design goals" to the README?

There is much chat about the "design goals" of ETH2.0.
Is it worthwhile to add a section to the README that lists the design goals for this project?
I think this could be helpful in terms of communicating to the broader community about the context around ETH2.0.

I think that makes sense. I plan on adding links and other info to the readme and will do so with that.
Also thinking about adding an FAQ in there as well

some quick ones that come to mind:

live through major network paritions (WWIII)
all components are either quantum secure or can be easily swapped out for quantum secure counterparts when available
utilize crypto and design techniques that allow for large participation of validators in total and per unit time
a typical laptop with O(C) resources can process/validate O(1) shards (including any system level validation such as the beacon chain)

live through major network paritions (WWIII)

I'd clarify this as "live through major network partitions and very large portions of nodes going offline". WWIII is more of a metaphor for big disaster in general; something like the Shanghai DoS attacks is a very real possibility that already happened.
I'd add a very big one, "minimize complexity, even at the cost of some losses in efficiency". IMO this spec is already at the upper limit of the level of complexity that is acceptable for this kind of system.

authors: djrtwo ralexstokes vbuterin


text:
ssz: proofread + uintN notation

Proofread work
uint: 8/16/24/32/64/256 -> uintN

@vbuterin check with you, CHUNK_SIZE is set to 128 in SSZ spec while another CHUNK_SIZE constant is set to 256 in phase 1 spec, is that correct?

@vbuterin check with you, CHUNK_SIZE is set to 128 bytes in SSZ spec while another CHUNK_SIZE constant is set to 256 bytes in phase 1 spec, is that correct?

Yes, maybe rename one to SSZ_CHUNK_SIZE. Though honestly we should consider setting both to the same value; I'd be fine with 128.

authors: hwwhww vbuterin


text:
fix committee assignment bugs

2 bugs:

We were modifying current_shuffling_epoch and then calling get_current_epoch_committee_count expecting that it was still getting the committee count from the previous value of current_shuffling_epoch. Switch operations to prevent this error
when getting next epoch committee assignments, we were sometimes modifying committees_per_epoch and shuffling_epoch when these things should have remained constant.

At this point get_crosslink_committees_at_slot is black magic to me. I wish it could somehow be massively simplified, similar to how we massively simplified the validator state machine (which had a bunch of subtle bugs, and was brittle).

These fixes are pretty critical. Let's get it merged and try to simplify after

At this point get_crosslink_committees_at_slot is black magic to me. I wish it could somehow be massively simplified, similar to how we massively simplified the validator state machine (which had a bunch of subtle bugs, and was brittle).

Can confirm, whenever there's a validator run time failure we immediately look at this function then begin to scratch our head 😆

Let's get it merged

Approved!

authors: JustinDrake djrtwo terenc3t


text:
chore: add tests in table-test format for NewOverlayFromEthereumAddress

added 4 test cases under TestNewOverlayFromEthereumAddress  to test NewOverlayFromEthereumAddress of crypto library
Checklist

[x] I have read the coding guide.
[ ] My change requires a documentation update, and I have done it.
[x] I have added tests to cover my changes.
[ ] I have filled out the description and linked the related issues.

Description

Open API Spec Version Changes (if applicable)

Motivation and Context (Optional)

Related Issue (Optional)

Screenshots (if appropriate):

@notanatol I have reopened the PR with the suggested updates.

@notanatol I have reopened the PR with the suggested updates.

thanks, but you don't have to re-open the PR, just push the fix commits to this branch, they are squashed anyway as we merge to master.

authors: Aviksaikat notanatol

repo: ethersphere/bee


text:
Handle Bee status in Desktop to avoid HTTP API startup issues

Related: https://github.com/ethersphere/bee-dashboard/issues/499 and https://github.com/ethersphere/bee-dashboard/issues/558
Upon Bee launch, the HTTP API fires up and reports healthy and ready statuses. Contrary to this, some endpoints do not work for a short brief during startup period. A good example is the chequebook API, which wrongly reports that the chequebook is not deployed. This causes the issue above.
The proper fix for this is in Bee, which we have already discussed with the core team during the API feedback round. As a temporary measure, we could implement a system to track the real Bee status (stopped, starting up, running) which we expose in the Desktop, and rely on this in the Dashboard.

Related issue in Bee: https://github.com/ethersphere/bee/pull/3248
Seems they have fixed it, we should recheck

repo: ethersphere/swarm-desktop


text:
EthStorage Devs Meeting #20 Agenda

Meeting Info

Feb 21, 2023, 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
TBA

Application

W3URL

JS SDK for Web3URL

Test for web app and nodejs - done
Tutorial documentation  - todo

Website update

Subscribe button - in progress
Error handling for dApps - in progress

pixel-war dapp&slides - done
on-chain music player

Research

OP Stack & Bedrock - 30%

Discussion on the Grant program
Submit a grant for paralleled EVM
Prepare a VDF grant

Application

Columbus net setup

ChainId apply 3335: pr (https://github.com/QuarkChain/chains/pull/1)
Test doc

EthStorage mining

Reading related code
implementation (40%)

Sharded storage with P2P support

Design doc (Updated)

Efficient p2p discovery protocol (implementating )

Application

MindReading Part I: ready to review https://github.com/QuarkChain/go-ethereum/pull/101
MindReading Part II: read to review https://github.com/QuarkChain/go-ethereum/pull/121
Research
Plonk Summary [50%]
Aztec [30%]

Due to the issue with the Tencent meeting account, the meeting was not recorded.

authors: cyl19970726 ping-ke qizhou qzhodl

repo: ethstorage/pm


text:
Remove access control notification banner

Is your feature request related to a specific problem?
In a future release, we should remove the warning about access controls being migrated.
Describe the solution you'd like
It's possible the NotificationBanner may be useful in the future, so it may be good to keep around but remove the calls to it.
Describe alternatives you've considered, if any
A description of any alternative solutions or features you've considered.
Additional context
Originally introduced by https://github.com/ethyca/fides/pull/2842

Oh darn, I missed this on the HomeLayout component. will fix shortly

moving to done

authors: Roger-Ethyca allisonking

repo: ethyca/fides


text:
[Backend] Fix Validation Error where Current Privacy Preferences might be None

Closes https://github.com/ethyca/fidesplus/issues/979
Description Of Changes
Under higher loads, saving Privacy Preferences could return Pydantic Validation Errors because the CurrentPrivacyPreferenceSchema is not supposed to be None.
2023-06-27 15:53:55.491 [ERROR] (logger:_log_exception:34): 1 validation error for CurrentPrivacyPreferenceSchema
response -> 0
  none is not an allowed value (type=type_error.none.not_allowed)

The root cause is that if a single user saves their preferences in rapid succession, the older PrivacyPreferenceHistory history record will no longer have a relationship with the CurrentPrivacyPreference record because a newer PrivacyPreferenceHistory had superseded it.
Code Changes

[x] Add a new method to PrivacyPreferenceHistory that returns both the created  PrivacyPreferenceHistory and upserted CurrentPrivacyPreference record instead of relying on the relationship between the PrivacyPreferenceHistory and the CurrentPrivacyPreference to fetch the latter after the fact before returning a response.   The response should return the CurrentPrivacyPreference upserted at that moment, even if the FK has since been replaced.

Steps to Confirm

[x] I would just validate that saving preferences still works - PATCH {{host}}/privacy-preferences

Example request:
{
   "browser_identity": {
       "fides_user_device_id": "75495a74-fc09-4e21-8250-0d53b854877d"
   },
   "preferences": [{
       "privacy_notice_history_id": "pri_d6bbc55d-bc06-4b7c-858b-016ed280e668",
       "preference": "opt_out"
   }],
   "user_geography": "us_ca",
   "privacy_experience_id": "pri_475641b6-75e4-4e77-8a30-d9640c3616e7",
   "method": "button"
}

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation:

[ ] documentation complete, PR opened in fidesdocs
[ ] documentation issue created in fidesdocs

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md
[ ] For API changes, the Postman collection has been updated

@adamsachs hoping you could review this small change - it was the source of the first error we were running into here, creating higher than expected failure rates.

@adamsachs yes! you're able to hit this error locally if you send requests at a high enough rate, and I did verify that this particular error has gone away.
Thanks for reviewing this!

Failing test not related and ticketed here: https://github.com/ethyca/fides/issues/3736


text:
Fidesops 78 mssql support

Purpose
Adds SQL Server support
Changes

Adds connection support for mssql
Adds connection integration tests for mssql
To manually test, in Postman PUT {{host}}/connection/{{mssql_key}}/secret, use the following url: "mssql+pyodbc://sa:Mssql_pw1@mssql_example:1433/master?driver=ODBC+Driver+17+for+SQL+Server", alternatively you can break each part out into separate components, refer to connection_secrets_mssql.py.

Checklist

[x] Applicable documentation updated (guides, quickstart, postman collections, tutorial, fidesdemo, database diagram
[x] Good unit test/integration test coverage

Ticket
Fixes #78

Ok, @pattisdr , over to you again! I cleaned up the PR based on your suggestions, and was able to get logs to print for make integration-env using your suggestion: @docker-compose -f docker-compose.yml -f docker-compose.integration-test.yml logs -f -t.
I also reused the @docker exec -it fidesops python tests/integration_tests/mssql_setup.py step for spinning up the test db for integration tests.

great will take a look!

authors: eastandwestwind pattisdr

repo: ethyca/fidesops


text:
Loop Closure Message

General
This PR adds a new message type for publishing LCs. Generally, the type is independent of the LC's source and defines all needed params in the msg.
Changelog

Added LC message definition

@cynosure4sure At the moment the message contains the pose delta as PoseWithCovariance and there is an additional field for the quality/confidence.  I don't think we need both, wdyt? Would you rather have a 6Dof cov or confidence between [0,1]?

@LBern thanks for the PR. 6DOF covariance is very helpful and in that case quality is not necessary. However, it is my understanding that the LC will be only verified and published on maplab side hence on the receiving end no checks will be performed. If this is the case then quality is not necessary at all especially with 6DOF covariance.

Thank you @LBern
Lgtm


text:
Prepare to release ng components v9.0.0

Changes proposed in this Pull Request:

Prepare to release v9.0.0

To Test:

yarn initialize && cd demos/showcase && git checkout feature/migrate-to-ng15 && cd ../..
yarn && yarn start
Review the theme in the context of the showcase demo.
Be sure to check both light and dark themes.

Am I crazy or is the showcase demo not initializing properly? I see neither a ./showcase (as implied by initializeSubmodule.sh) folder nor a ./demo/showcase folder, as implied in the PR instruction

Am I crazy or is the showcase demo not initializing properly? I see neither a ./showcase (as implied by initializeSubmodule.sh) folder nor a ./demo/showcase folder, as implied in the PR instruction

Let me check on this.

authors: daileytj huayunh

repo: etn-ccis/blui-angular-component-library


text:
Check the PHP version before re-using a cached extension

Fixes #102

Merged it to local branch, this one fixed my travis, so :+1:

authors: cweagans scaytrase

repo: etsy/phan


text:
🛑 nginx is down

In d66400d, nginx (https://nginx.eugene.ng) was down:

HTTP code: 0
Response time: 0 ms

Resolved: nginx is back up in b7ecca1.

repo: eugeneyng/upptime


text:
Running hot-patch examples

Hello, I'm having trouble understanding and running the hot patch examples.
They make use of a cli which doesn't seem to exist any more (bpftime-cli), which I think is just now bpftime. However that definitely does not have the same functionality as expected in the examples like here.
Both examples prepare a patch (this is unexplained), and take a generated JSON file (no examples exist I can find, and unexplained how it's made) and give it to the CLI to patch something.
What is the correct way to get the hot-patching to work?

The README is too old, for now you may use bpftime load redis-patch and bpftime attach <pid to redis server> instead

authors: Officeyutong aFuerst

repo: eunomia-bpf/bpftime


text:
Refactor pytest_collection_modify hook

Refactor the doctest collection stage a bit.
The key simplifications are

for dev installs, only run pytest on the build-install folder. Its contents matches the installed version, so it should be identical to $ pytest --pyargs scipy for an installed scipy (I think?). This obviates the need for the PY_IGNORE_IMPORTMISMATCH=1 env var --- almost. The remaining need seems to be due to some weirdness in scipy.signal, which indeed has a matching .py and .so modules:

E   _pytest.pathlib.ImportPathMismatchError: ('scipy.signal._spectral', '/home/br/repos/scipy/scipy/build-install/lib/python3.10/site-packages/scipy/signal/_spectral.cpython-310-x86_64-linux-gnu.so', PosixPath('/home/br/repos/scipy/scipy/build-install/lib/python3.10/site-packages/scipy/signal/_spectral.py'))

This probably needs to be looked at on the scipy side.

The logfile generation can be done with pytest --collect-only ... and some light postprocessing (below the fold)

The full stanza to run this, including pytest ignores, is currently (I know)
$ python dev.py shell
$ cd ~/repos/scipy/scipy
$ PY_IGNORE_IMPORTMISMATCH=1 pytest build-install/lib/python3.10/site-packages/scipy/ --doctest-modules --ignore=build-install/lib/python3.10/site-packages/scipy/interpolate/_interpnd_info.py --ignore=build-install/lib/python3.10/site-packages/scipy/_lib --collect-only

Log posprocessing scripts are below the fold:

$ cat modify_log.py

# convert the result of
# $ pytest ... --doctest-modules --collect-only
# to simplify comparing to the doctest_.log from doctest-based runs

PACKAGE = "scipy."

if __name__ == "__main__":
    import sys
    if len(sys.argv) != 2:
        print("Usage: modify_log.py LOGFILE")

    fname = sys.argv[1]
    with open(fname, 'r') as inf:
        in_lines = inf.readlines()

    out_lines = []
    for line in in_lines:
        line_ = line.strip()

        if line_.startswith("<DoctestItem"):
            out_lines.append(line_[13:-1])

    print("\n".join(sorted(out_lines)))

and
$ cat modify_doctest_log.py
# strip module names from the doctest.log
# to simplify comparing to the doctest_.log from doctest-based runs

PACKAGE = "scipy."

if __name__ == "__main__":
    import sys
    if len(sys.argv) != 2:
        print("Usage: modify_log.py LOGFILE")

    fname = sys.argv[1]
    with open(fname, 'r') as inf:
        in_lines = inf.readlines()

    out_lines = []
    for j, line in enumerate(in_lines):
        line_ = line.strip()

        if line_.startswith("="):
            continue

        if j < len(in_lines) -1 and in_lines[j+1].startswith("="):
            continue

        out_lines.append(line_)

    print("\n".join(sorted(out_lines)))

With these, the logs agree for the following combination: https://github.com/ev-br/scipy/tree/doctest_plugin + this PR, and https://github.com/scipy/scipy/pull/16391, modulo https://github.com/ev-br/scpdt/issues/106.

@Sheila-nk @melissawm any chance you could review this PR? Would be much appreciated.

https://github.com/scipy/scipy/pull/20112 should take care of IGNORE_IMPORTSMISMATCH

Merging now to keep the ball rolling as I work on test errors after collection is successful. A post-merge review would still be most welcome.

repo: ev-br/scpdt


text:
Restore license headers to external files

I've just noticed that you have taken a few files out of libgba and just removed the license header that credits the authors and specifies the conditions to use the files:
https://github.com/evanbowman/blind-jump-portable/blob/76caaa9ee72121cbae535db7639c76f1264a14b0/source/platform/gba/interrupt_dispatch.s
https://github.com/evanbowman/blind-jump-portable/blob/76caaa9ee72121cbae535db7639c76f1264a14b0/source/platform/gba/bios_math.s
https://github.com/evanbowman/blind-jump-portable/blob/76caaa9ee72121cbae535db7639c76f1264a14b0/source/platform/gba/reset.s
Original files:
https://github.com/devkitPro/libgba/blob/3751d012eb5a9d60b172d1ec2a9b581136d389c5/src/InterruptDispatcher.s
https://github.com/devkitPro/libgba/blob/3751d012eb5a9d60b172d1ec2a9b581136d389c5/src/Div.s
https://github.com/devkitPro/libgba/blob/3751d012eb5a9d60b172d1ec2a9b581136d389c5/src/Reset.s
Also, you've got code from libtonc:
https://github.com/evanbowman/blind-jump-portable/blob/76caaa9ee72121cbae535db7639c76f1264a14b0/source/platform/gba/memcpy.s
Original file:
https://github.com/devkitPro/libtonc/blob/ccc03fa321e56f51aed5e2ee1d6e3df3d1cbc803/asm/tonc_memcpy.s
Please, restore the headers, and do the same to other files if you have removed their headers. And, if you modify them, specify so below the original header.

It's not exactly that I copied the files and removed the licenses, you'll notice that some of the files don't even have the same names. I didn't want to have to link to the libgba library just for these files, and I copied the assembly instructions into these files, and probably didn't bother to copy the comment block at the top of the file. Now that I look at this licensing, does this mean that I need to convert this project to gpl, or just make the project open source, as I'm already doing?
P.s. Tonc is specifically credited in the special thanks section of the in-game credits, I'm not trying to steal credit for other people's work.

Yeah, the files don't have the same names, but the content is the same. I understand not wanting to link with libgba just for 2 or 3 files, to be honest, but the fix is really easy in this case.
libgba is LGPL. I'm not sure how it works when you link it statically, to be fair. The thing is, your game is MIT, which is less restrictive than LGPL, so all you have to do is keep it as it is, open source, and restore the headers.
Giving credit is nice, but if the files don't have the right copyright, whoever copies them from your game doesn't know where they came from or the license they have, and they may assume they are MIT-licensed.

Ah I see, just a minute, I'll fix the files.

Actually, I don't even think that I'm using any of the code from bios_math.s, at first I thought that these math routines from the bios were actually faster than the emulated division from devkitarm compiler, but turns out that there isn't really any significant difference, and in some cases, the compiler can optimize the emulated division better, because it has no concept really of what the assembly instructions in the bios looks like.

I'll add the licences everywhere for now, and remove unused files later.

Ok, I added back the licenses. For memcpy.s, which never had a license attached to the file in the first place, I inserted the licence from the PR that you linked at the top of the file. Let me know if this is acceptable.
https://github.com/evanbowman/blind-jump-portable/blob/master/source/platform/gba/memcpy.s

Works for me.

Cool, thanks for your suggestions.

authors: AntonioND evanbowman

repo: evanbowman/blind-jump-portable


text:
Error thrown on non-other-player-related death

Here's the error: https://i.imgur.com/JoOmak3.png
Thrown if a player dies in any other way than being killed by someone else.

I’m aware of this issue.  I plan on fixing this in the next 1-2 days when I return home from break.

authors: DarkKingKaanMC evancolewright

repo: evancolewright/HeadDrops


text:
Not really a pull request yet

Hi Evan,
Two main additions here, mainly:

add a code to rearrange ImageNet validation data (based on the earlier issue I opened)
create a sub folder in srcEt accessed through entryEt.py. What it does: to run the minimal example (slightly modified).

Bear in mind that I try to make as little change as possible to your existing code, just to be safe. Please review the following before taking action on this pull request, your help is very much appreciated.
I would like to ask your opinion about change number 2 which is the main issue here. Normally, I understand that the minimal example will access src/milannotations/datasets.py. In line ~158, each layer will be processed. In my case, that is too slow and resource consuming (my process keeps getting killed) as the code is run in my local machine for debugging. So I made the mini version that is run using entryEt.py and accesses srcEt/milannotations/datasetsMini.py.
The difference between datasetsMini.py and datasets.py: rather than computing every layer for every run, datasetsMini.py only computes the selected layer for lookup (only small change in the code, see pull request). Is this okay? It seems like all other layers are not really interfering with the looked up layer.
Btw the following raises "TypeError: 'module' object is not callable" error (I have seen the relevant commit you made too)
outputs = milan(sample.images[None], masks=sample.masks[None])
so I used the following instead
outputs = decoder(sample.images[None], masks=sample.masks[None])
In any case, I keep getting 'Edges' as the output from the following
print(outputs.captions[0])
even if I changed the layers and units. Furthermore, sample.images[None] appear as black images across all "channels".
Can you give some hints on what might be wrong?
I'm running it on the ubuntu >20 btw.

Thanks Evan, will look forward to it

repo: evandez/neuron-descriptions


text:
Handling item selection 2-way binding on Spinners

Hi,
Is there a way to implement something like "bind:selection" on an AppCompatSpinner to achieve 2-way binding? I have tried several routes involving BindingAdapters and InverseBindingAdapters, but I can't get anything to work.
Thanks,
Adrian

Hi, it should be possible with something like this: android:selectedItemPosition="@={viewModel.selectedItem}", where selectedItem is ObservableInt inside you viewModel/whatever.
I'm using this in my app.

authors: Trellian janbina

repo: evant/binding-collection-adapter


text:
🛑 www.actionherojs.com is down

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

HTTP code: 0
Response time: 0 ms

Resolved: www.actionherojs.com is back up in 06816e6.

repo: evantahler/upptime


text:
🛑 www.actionherojs.com is down

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

HTTP code: 0
Response time: 0 ms

Resolved: www.actionherojs.com is back up in adceeb9.


text:
Support for Hardy Barth cPH1

Is your feature request related to a problem? Please describe.
I use a Hardy Barth cPH1 Wallbox which is currently not supported
Describe the solution you'd like
Add support for Hardy Barth cPH1 Wallbox
Describe alternatives you've considered
use Hardy Barth cPH1 Wallbox internal features, which are currently very limited

Siehe https://github.com/evcc-io/evcc/issues/2229

repo: evcc-io/evcc


text:
Heidelberg: Add async watchdog timer

Was ist denn als interval in evcc eingestellt?
Wenn mehr als 15 Sekunden ist klar dass die Boxen ständig die Verbindung verlieren.
Originally posted by @premultiply in https://github.com/evcc-io/evcc/discussions/3596#discussioncomment-6667429

Wenn das Default interval nicht mehr 10 Sekunden ist passt dies nicht mehr zur Standardkonfiguration der Box mit 15 Sekunden.
Hier ist daher ein asynchroner Watchdog Handler erforderlich.

Wenn sowas im Charger reicht würde ich das in nen PR machen.
	go func() {
		ticker := time.NewTicker(5 * time.Second)
		for {
			select {
			case <-ticker.C:
				wb.Status()
			}
		}
	}()

for time.Tick() ....

bitte analog der anderen heartbeats- nix neues erfinden ;)

Habe zumindest noch diese Schreibweise entdeckt
for ; true; <-time.Tick(time.Hour)
In der Doku steht man solle aufpassen wegen garbage collection da der Ticker drunter nicht abschaltbar ist. Werden also bei den Tick Aufruf im for nicht dauernd neue Ticker ins Nichts gestartet? In der Doku wird der Channel vor dem for initialisiert.
https://pkg.go.dev/time#Tick

Nein…

Jetzt gibts einen weniger: https://github.com/evcc-io/evcc/commit/7a314c79ee6d493049606a8debd992c023567e2f

authors: andig panzerdev premultiply


text:
slimmelezer-v2 correction phase powers

regarding https://github.com/evcc-io/evcc/discussions/16542

no idea, why test fails


text:
Allow alt="" or role="presentation"

As we discussed on #6, it is okay to use alt="" without
role="presentation". It is possible that we want to enforce no alt
prop if role="presentation" but I'll leave that for another time and
possibly a different rule.

Coverage increased (+0.03%) to 98.182% when pulling 483eff9e5efb392bf2b1532a1d8b33a86a6ebf6c on lencioni:img-uses-alt-role into 43a3c435f03f431656fced3ae095fcb79001eaed on evcohen:master.

Coverage increased (+0.03%) to 98.182% when pulling f1d3fdf5919f3c71386cc24bee595c645f995dc4 on lencioni:img-uses-alt-role into 43a3c435f03f431656fced3ae095fcb79001eaed on evcohen:master.

Awesome, thanks! I think it makes more sense to return the actual attribute value when it's a literal, since "" isn't a truthy value, anyway.
@lencioni Can you merge latest into your branch?

Rebased!

Coverage increased (+0.03%) to 98.182% when pulling d2340ff955acc40b0a1da2810ba800b9fcbdefe4 on lencioni:img-uses-alt-role into dbc9773d3ddaa989ea782aabad68882c303283c5 on evcohen:master.

I'm going to merge and patch - nicer to hammer down rule details and not continually bump minors before v1.0.0 since npm doesn't play well with ^0.y.z

authors: coveralls evcohen lencioni


text:
Speech Demo

Created a demo using speech recognition and speech synthesis called "Adventure Blockly." Uses custom blocks to listen to user input, generate speech, and change images and text on the screen. Acts as a sandbox area to create new projects, and comes with several prebuilt demos. Uses the JS Interpreter to asynchronously block until a block is done executing (has listened for user input, generated speech, displayed an image, etc.). Currently only Chrome fully supports speech recognition and speech synthesis (Firefox coming soon).

This change is

+@vicng

Review status: 0 of 23 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


text:
feat(module.d.ts, package.json): npm module declaration file to help with ts

i wanted to use evervault with ts but couldnt due to missing types. so, i made a declaration file. i dont know if it works when you try to import the module upon installation from npm but i run a few quick tests locally by importing it directly from the fs and it went smoothly
Why
I wanted to use Evervault with ts
How
I made a simple declaration file, and used the docs to type the file
Checklist

[ ] Commit messages are formatted as per the convention, and breaking changes are marked

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: evervault/evervault-node


text:
How to change rating by tapping, not by dragging?

I am trying to make my star rating get the rate by tapping, not by dragging for user experience.
But I cannot find any code for this.
Is there anyone who can help me?
Thank you in advance!

to make it work in tableview, simply add this in cell
let gestureRecognizer = UITapGestureRecognizer(target: self, action: #selector(dummy))
        gestureRecognizer.cancelsTouchesInView = false
        addGestureRecognizer(gestureRecognizer)

repo: evgenyneu/Cosmos


text:
Proposed definitions for ECO terms w/out definitions

A recent survey of ECO found 10 terms that lacked definitions.  A search of issues revealed that definitions for these terms have not been previously discussed.
Tentative definitions proposals follow with the intention of generating review and discussion.
James
spatial pattern of transcript expression evidence ECO:0000047
A type of transcript expression evidence that accounts for the position of transcription ranging in scale from the position of genes on the chromosome to differing tissues to regions of an anatomical structure.
Note: really not too sure about this one as I read several papers and the spatial degree that was analyzed varied considerably.
biological target-disease association via drug evidence ECO:0000360
mchibucos, 2014-9-26
Becky noted: I think it has something to do with drug-target interactions? https://bmcbioinformatics.biomedcentral.com/articles/10.1186/s12859-018-2199-x
A type of experimental evidence resulting from the prediction of drug-target interactions by computational means.
protein separation followed by direct sequencing evidence ECO:0000158
A type of protein separation evidence that is followed by direct protein sequencing whereby a protein's amino acid sequence is determined experimentally by Edman degradation or by mass spectrometry.
protein separation followed by fragment identification evidence ECO:0000160
Becky noted: These might help
https://pubs.acs.org/doi/10.1021/bi3005126 and https://www.ncbi.nlm.nih.gov/pmc/articles/PMC5632530/
A type of protein separation evidence which then utilizes the screening and identification of small-molecules that bind to proteins.
peptide affinity enrichment evidence ECO:0001110
mchibucos, 2014-3-16, SIB:PG
A type of affinity evidence resulting from the appending of affinity tags on proteins so that they may be purified from their biological source using an affinity technique.
blocking polyclonal antibody evidence ECO:0001015
mchibucos, 2014-3-16, SIB:PG
A type of immunological assay evidence resulting from the inhibitory effect of polyclonal antibodies (antibodies secreted by different B cell lineages), which while not reacting with a specific antigen, serve to prevent other antibodies from doing so.
Note: 'blocking monoclonal antibody evidence' def. should be reviewed as it states, "A type of immunological assay evidence resulting from inhibitory effect of monoclonal antibody combining with an antigen over other antibodies." but blocking antibodies DO NOT combine / react with antigens.
macroscopy evidence ECO:0001094
mchibucos, 2014-3-16, SIB:PG
A type of experimental evidence that is visible to the naked eye cf. microscopy evidence which requires the aid of a microscope.
spatial pattern of protein expression evidence ECO:0000045
A type of protein expression evidence that accounts for the position of RNA translation to a protein product ranging in scale from differing tissues to regions of an anatomical structure.
Note: what is protein expression?  There is gene expression and protein production.
Note: really not too sure about this one as I read several papers and the spatial degree that was analyzed varied considerably.
single exon transcript confirmation via alignment evidence ECO:0000345
mchibucos, 2013-4-3
Becky noted: Seems like some sort of RNA-seq stuff, this might be a starting point but it might also not be useful: https://www.ncbi.nlm.nih.gov/pmc/articles/PMC4897596/
A type of sequence alignment evidence resulting from the comparison of a single exon transcript to a collection / database of known single exon genes.
transient rescue experiment evidence ECO:0000038
A type of functional complementation evidence resulting from the introduction of nucleic acids, which are not permanently incorporated into the genome, to temporarily prevent, or "rescue" an organism from a condition.

These were defined and released prior to paper publication. Will revisit each of these one by one to assess.

repo: evidenceontology/evidenceontology


text:
Set up your Kanban board

https://github.com/evillaz/react-redux-group-project/projects/1
We are two people in the group.

STATUS: CHANGES REQUIRED ♻ ♻
Hi @evillaz and @Atril33,
Good job so far!
✔ Complete Kanban columns
However, there are some issues that you still need to work on to go to the next project but you are almost there!
Required Changes ♻

[ ] You have 24 tasks including tasks marked as "Team 3". Kindly remove all Team 3 tasks. This should bring the total number of tasks to 17 for two people.

[ ] You have not assigned your tasks. Kindly ensure you properly assigned your tasks. This is what a task should look like if assigned.

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the Issue thread if something is not 100% clear. And please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Issue for re-reviews. You should use the same Issue submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Fixed and implemented the required changes

Hi Team,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights

Kanban Setup ✔
Equal distribution ✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

authors: Olamarx TedLivist evillaz

repo: evillaz/react-redux-group-project


text:
Add Async crock to Documentation

Yet Another

This PR adds the Async README to the documentation. That is all.

Coverage remained the same at 100.0% when pulling 851f625a58f7004da0c4c94df7e3cbe35a21da4c on move-async-docs into 85d549ae92d85bfa2309fea6071fb994efdee61c on master.

Coverage remained the same at 100.0% when pulling 851f625a58f7004da0c4c94df7e3cbe35a21da4c on move-async-docs into 85d549ae92d85bfa2309fea6071fb994efdee61c on master.

Coverage remained the same at 100.0% when pulling 851f625a58f7004da0c4c94df7e3cbe35a21da4c on move-async-docs into 85d549ae92d85bfa2309fea6071fb994efdee61c on master.

authors: coveralls evilsoft

repo: evilsoft/crocks

license: ISC


text:
:construction_worker: Fix: Fix rome.json not ignoring nx

Description
Not ignoring nx file caused it to lint a huge file
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #708

PR #711

PR #718

PR #719

PR #720   👈

PR #709

PR #710

PR #654

This stack of pull requests is managed by Graphite.

Current dependencies on/for this PR:

main

PR #708

PR #711

PR #718

PR #719

PR #720   👈

PR #709

PR #710

PR #654

This stack of pull requests is managed by Graphite.

repo: evmts/evmts-monorepo


text:
Replaced Feedback button with Examples button

Replaced Feedback button with Examples button that opens the examples documentation page.

Code review pass.

authors: Sionarch divineprog

repo: evothings/evothings-studio


text:
style(esl-event-listener): regroup event listener code (#1449)

Closes: #1449

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: exadel-inc/esl


text:
🛑 Portfolio is down

In 699766a, Portfolio ($SECRET_P_SITE) was down:

HTTP code: 404
Response time: 818 ms

Resolved: Portfolio is back up in b8780fb.

repo: exand/upptime


text:
Replace Language Container Stuff with PEC and SLC pytest plugin

PEC provides the flavor definition for Extension SLC and LanguageContainerBuilder

AAF doesn't need the directory language_container anymore
Replace nox-session implementation with LanguageContainerBuilder from PEC
Replace local LanguageContainerDeployer by PEC LanguageContainerDeployer implementation, see transformer-extension

Use SLC pytest plugin instead of  https://github.com/exasol/advanced-analytics-framework/blob/2b59f9b91a4a75eb4588a4e53bf7f8c54d0f036f/tests/integration_tests/with_db/fixtures/build_language_container_fixture.py#L21

repo: exasol/advanced-analytics-framework


text:
Add docstrings

Background

Parts of the code don't contain docstrings

Acceptance Criteria

All relevant classes and functions container docstrings

Planned for next cleanup event.

authors: ckunki tkilias

repo: exasol/data-science-utils-python


text:
メニューグループ用パネル画像作成機能

「メニューグループ管理」メニューにパネル画像作成用の機能を追加する。

要ドキュメント修正

資材の払戻完了
ドキュメント修正は未着手

保存、読み込み関連の文言について意見を伺いたいです。

authors: katokeita sinatommy ta-matsuo

repo: exastro-suite/it-automation


text:
production build broken in Vite

Unclear what Vite does on production build, but it breaks Excalidraw:
index.22c8f9a6.js:32 Error: Minified React error #130; visit https://reactjs.org/docs/error-decoder.html?invariant=130&args[]=object&args[]= for the full message or use the non-minified dev environment for full errors and additional helpful warnings.
    at $0 (index.22c8f9a6.js:32:49942)
    at G (index.22c8f9a6.js:30:17215)
    at Ur (index.22c8f9a6.js:32:10779)
    at i3 (index.22c8f9a6.js:32:46238)
    at n3 (index.22c8f9a6.js:32:40175)
    at L8 (index.22c8f9a6.js:32:40103)
    at ym (index.22c8f9a6.js:32:39956)
    at gv (index.22c8f9a6.js:32:36292)
    at J4 (index.22c8f9a6.js:32:35256)
    at fe (index.22c8f9a6.js:17:1678)

Repro:
https://stackblitz.com/edit/vitejs-vite-qa5tsy?file=package.json&terminal=serve

Was looking into this and this looks like an issue with vite js using roll-up plugin and hasn't been fixed yet, the recommended solution to fix is to use the default export until it gets fixed in rollup plugin / vite itself.
const ExcalidrawComp = Excalidraw.default ? Excalidraw.default : Excalidraw;

and use  in rendering.
https://github.com/vitejs/vite/issues/2139
https://github.com/rollup/plugins/issues/481
Sadly this affects a lot of popular libs as well like react-select, react-switch , react-notification and a lot more
Also here it like the issue is only with just default export,  as rollup is not able to identify the default export hence all libs using default export tend to fail so not using default export should fix this as well

it also wrong.
src/nodes/ExcalidrawNode/ExcalidrawModal.tsx:183:34 - error TS2339: Property 'default' does not exist on type 'typeof import("F:/code/skill/know/node_modules/.pnpm/@excalidraw+excalidraw@0.14.1_biqbaboplfbrettd7655fr4n2y/node_modules/@excalidraw/excalidraw/types/packages/excalidraw/index")'.

183   const _Excalidraw = Excalidraw.default ? Excalidraw.default : Excalidraw;

authors: ad1992 dwelle kulame

repo: excalidraw/excalidraw


text:
build: add a flag VITE_APP_ENABLE_PWA for enabling pwa in dev environment

[x] add a flag VITE_APP_ENABLE_PWA for enabling pwa in dev environment
[x] Add id which was missing from the manifest

how about adding VITE_APP_ENABLE_PWA=false in .env.development to make it discoverable?
yep done


text:
[Q] Cell comment width calculation

I have been trying to use cell Comment from exceljs but it is of a fixed width . Can't we introduce a feature for cell comment where we could define the width and height of each cell comment based on the content of it ?
I guess it would be very helpful if this is implemented .

Hi Team,
Do we have any updates yet ?

Yes . They both are the same .

any update on this issue please? This is an important feature!

yes it's an important feature!

yes it's an important feature! For now the only solution I've found is to add VBA script, however it's not a good solution.

Could you share that VBA code snippet ?

yes it's an important feature! For now the only solution I've found is to add VBA script, however it's not a good solution.

Could you share that VBA code snippet ?

https://www.extendoffice.com/documents/excel/1572-excel-autosize-comments.html
Excel will prevent macro code from executing in the excel downloaded from Internet, so it doesn't match our requirement, but I hope it can help you.

yes it's an important feature! For now the only solution I've found is to add VBA script, however it's not a good solution.

Could you share that VBA code snippet ?

https://www.extendoffice.com/documents/excel/1572-excel-autosize-comments.html
Excel will prevent macro code from executing in the excel downloaded from Internet, so it doesn't match our requirement, but I hope it can help you.

Okay thanks . Will try this until they provide a solution 😀

authors: SyedSalmanNizami csgka1 eminenceit

repo: exceljs/exceljs


text:
[BUG]  workbook.xlsx.writeBuffer() returns the buffer two times if application uses React.StrictMode

🐛 Bug Report
Steps To Reproduce
demo.js
import * as Excel from 'exceljs/dist/exceljs.min.js';

const onSave = (workbook) => {
  workbook.xlsx.writeBuffer().then((buffer) => {
    console.log('write file')
  });
};

export default () => {
const workbook = new Excel.Workbook();
onSave(workbook);
  return null;
};

index.js
import React from 'react';
import ReactDOM from 'react-dom';
import Demo from "./demo";

ReactDOM.render(
  <React.StrictMode>
    <Demo />
  </React.StrictMode>,
  document.getElementById('root')
);

The expected behaviour:
writeBuffer() should not returns the buffer two times
Possible solution (optional, but very helpful):
Remove the React.StrictMode and all works well.

@Krijovnick this is known behavior for React 18
https://github.com/facebook/react/issues/24502
it's not related with exceljs

authors: Elecweb Krijovnick


text:
[BUG] WorksheetReader did not read rowBreaks section

🐛 Bug Report
Lib version:  exceljs@4.3.0
Steps To Reproduce

Prepare an xlsx file such that the first sheet contains a rowBreaks(page break).
Watch rowBreaks right after loading the file.

const reader1 = new FileReader();
reader1.readAsArrayBuffer("theRowBreaked1.xlsx")
    .then(xlsxArrayBuffer1 => {
        const workbook1 = new ExcelJS.Workbook();
        return workbook1.xlsx.load(xlsxArrayBuffer1 as ArrayBuffer);
   }).then(workbook2 => {
          console.log( workbook2.worksheets[0].rowBreaks);  // empty !!
});

The expected behaviour:
I expected workbook1.worksheets[0].rowBreaks to expect page break positions in the xlsx file.
But workbook1.worksheets[0].rowBreaks was empty.

@master-maintenance1-peer-connect Can you provide your excel file?

Submitted a PR to resolve this issue https://github.com/exceljs/exceljs/pull/2602


text:
Make use of start action for statistics

It would make active sessions count easier.

Actually not...

authors: BluePyth slandelle

repo: excilys/gatling


text:
Errors in Golang track not glanceable

Just dry-running https://github.com/exercism/blog/blob/main/posts/exercism-is-the-official-go-mentoring-platform.md and noticed that the errors output seem to be the literal test function...
Fine for me, because I've been coding go for a few years. But for a person unfamiliar... I think YMMV
For a start a learner needs to be able to reconstruct the function call from the table tests (I love table tests, and that you're using them, perhaps these could be structured in a way with an args struct to disambiguate test state from function arguments?
Once this is done They need to spot the error message. If they make a syntax boo-boo, the output makes a lot of sense. But for failing test cases, it's actually easier to not code in the browser, run with verbose flags and watch go make useful noise.

The specific exercise that bit me was me rushing and being lazy, but https://exercism.org/tracks/go/concepts/slices
Locally I got the following error (because I'd forgot to bounds-check)

--- FAIL: TestRemoveItem (0.00s)
--- FAIL: TestRemoveItem/Remove_out_of_bounds_index (0.00s)
panic: runtime error: slice bounds out of range [:7] with capacity 4 [recovered]
panic: runtime error: slice bounds out of range [:7] with capacity 4

I didn't get this in browser when I omit lines to check for preExisting element at index
func RemoveItem(slice []int, index int) []int {
	_, preExisting := GetItem(slice, index)
	if preExisting {
		return append(slice[:index], slice[index+1:]...)
	}
	return slice
}

vs
func RemoveItem(slice []int, index int) []int {
	return append(slice[:index], slice[index+1:]...)
}

Hi, thanks for the input.

I didn't get this in browser

Could you please paste what you saw in the browser instead?

https://user-images.githubusercontent.com/2605791/135803504-b3aaf158-9b2c-4ccb-960a-6fab40e34517.mp4

Textual output

FAILED
Test 1
TestRemoveItem

CODE RUN
func TestRemoveItem(t *testing.T) {
	type args struct {
		slice	[]int
		index	int
	}
	tests := []struct {
		name	string
		args	args
		want	[]int
	}{
		{
			name:	"Remove an item",
			args: args{
				slice:	[]int{3, 4, 5, 6},
				index:	1,
			},
			want:	[]int{3, 5, 6},
		},
		{
			name:	"Remove the first item",
			args: args{
				slice:	[]int{3, 4, 5, 6},
				index:	0,
			},
			want:	[]int{4, 5, 6},
		},
		{
			name:	"Remove the last item",
			args: args{
				slice:	[]int{3, 4, 5, 6},
				index:	3,
			},
			want:	[]int{3, 4, 5},
		},
		{
			name:	"Remove out of bounds index",
			args: args{
				slice:	[]int{3, 4, 5, 6},
				index:	7,
			},
			want:	[]int{3, 4, 5, 6},
		},
		{
			name:	"Remove negative index",
			args: args{
				slice:	[]int{3, 4, 5, 6},
				index:	-7,
			},
			want:	[]int{3, 4, 5, 6},
		},
	}
	for _, tt := range tests {
		t.Run(tt.name, func(t *testing.T) {
			if got := RemoveItem(copySlice(tt.args.slice), tt.args.index); !slicesEqual(got, tt.want) {
				t.Errorf("RemoveItem(slice:%v, index:%v) = %v, want %v", tt.args.slice, tt.args.index, got, tt.want)
			}
		})
	}
}
TEST FAILURE

=== RUN   TestRemoveItem

--- FAIL: TestRemoveItem (0.00s)

Interestingly. I Think it might be in that collapsed block, which leads me to question why the first test is not outputting helpful error messages. In any case I solved it because I can read go. I was checking out the new guided feature from the exercism blog, just to see how it would be for learners.

Thanks for the video. The issue you showed there is already reported here https://github.com/exercism/go-test-runner/issues/44 and the root cause for this is https://github.com/exercism/go-test-runner/issues/8.

repo: exercism/go


text:
config.json: add topics & difficulty for diamond

See #345 & #731

Updated @leenipper @tleen @ferhatelmas, thanks :+1:


text:
simple-cipher: Fix typo

Remove :  which seems extraneous?

You're absolutely right. It should actually be updated according to https://github.com/exercism/problem-specifications/blob/master/exercises/simple-cipher/canonical-data.json#L81.
Would you be able to do that? (And update the other test-cases as well)? You can then also make the package.json version be 2.0.0, as long as all the tests match ones from that problem-description repo.

I can do that. I compared the problem specification and the test file, and there are test cases in the test file that are not in the specification file (generates keys at random and has a key that is at least 100 characters long, and the whole section about throwing errors when the key is incorrect). I guess I do not have to change the specification, only make sure the actual test cases have the same descriptions?

, and the whole section about throwing errors when the key is incorrect

yes, they were removed!

I guess I do not have to change the specification, only make sure the actual test cases have the same descriptions?

Correct! Sync OUR tests with THAT canonical data 🗡

Alright! I'll remove the extra tests that do not appear in the problem spec then!

The generator that I'm still working on will by default copy paste whatever is in there.
In this case, I actually don't feel that the ie. jdjdjdjdjd stuff adds much value. I would suggest you keep the lowercase (for now) and if you want add the ie. jdjdjdjdjd as a comment above/inside the test :)

Done. I think I've implemented all your comments now :slightly_smiling_face: Let me know if there's anything else!

You're the best! @pawamoy

authors: SleeplessByte pawamoy

repo: exercism/javascript


text:
Fix bin/test-all

This accounts for the changes from https://github.com/exercism/lua/pull/287

I just wondered why the test was failing. I guess this can be merged now?

I just wondered why the test was failing. I guess this can be merged now?

I can't merge without a review... and unfortunately I'm the only active maintainer for the Lua track. Would you like to become a maintainer? You've been very helpful and it would let you review and merge in addition to your contributions.

Yes, sure :)
Since this track and your mentoring taught me a lot while learning a bit of programming with this wonderful language, I'm happy if I can give something back by contributing here and help you out.

Thanks! All you need to do is submit a PR adding yourself to https://github.com/exercism/lua/blob/main/config/maintainers.json. Once I've merged your PR, the Exercism infrastructure should give you commit rights to this repo.

authors: imolein ryanplusplus

repo: exercism/lua


text:
The master branch will be renamed to main

In line with our new org-wide policy, the master branch of this repo will be renamed to main. All open PRs will be automatically repointed.
GitHub will show you a notification about this when you look at this repo after renaming:

In case it doesn't, this is the command it suggests:
git branch -m master main
git fetch origin
git branch -u origin/main main

You may like to update the primary branch on your forks too, which you can do under Settings->Branches and clicking the pencil icon on the right-hand-side under Default Branch:

We will post a comment below when this is done. We expect it to happen within the next 12 hours.

As of today, the default branch of this repository has been changed to main.

As of today, the default branch of this repository has been changed to main.

Should this be closed?

authors: ErikSchierboom kotp

repo: exercism/request-new-language-track


text:
Added a section on rebasing

This is my first try at explaining rebasing.

This is a good addition. Should we open by saying something like "You'll often be asked to rebase your branch before we merge a pull request." so that people have an idea why we're explaining this?

Yes, that sounds good.

This looks great, would you mind squashing the commits, please?

Thanks, it's my first time at contributing to something.

Thank you so much! (and congrats on your first open source commit :tada:)

authors: heatherlim kytrinyx

repo: exercism/x-common


text:
Investigate track health and status of the track

I've used Sarah Sharp's FOSS Heartbeat project to generate stats for each of the language track repositories, as well as the x-common repository.
The Exercism heartbeat data is published here: https://exercism.github.io/heartbeat/
When looking at the data, please disregard any activity from me (kytrinyx), as I would like to get the language tracks to a point where they are entirely maintained by the community.
Please take a look at the heartbeat data for this track, and answer the following questions:

To what degree is the track maintained?
Who (if anyone) is merging pull requests?
Who (if anyone) is reviewing pull requests?
Is there someone who is not merging pull requests, but who comments on issues and pull requests, has thoughtful feedback, and is generally helpful? If so, maybe we can invite them to be a maintainer on the track.

I've made up the following scale:

ORPHANED - Nobody (other than me) has merged anything in the past year.
ENDANGERED - Somewhere between ORPHANED and AT RISK.
AT RISK - Two people (other than me) are actively discussing issues and reviewing and merging pull requests.
MAINTAINED - Three or more people (other than me) are actively discussing issues and reviewing and merging pull requests.

It would also be useful to know if there a lot of activity on the track, or just the occasional issue or comment.
Please report the current status of the track, including your best guess on the above scale, back to the top-level issue in the discussions repository: https://github.com/exercism/discussions/issues/97

No data available for this track. It is inactive.

Hello. I have taken the Ceylon track under my wing and have done some work on it the past few days, out of my interest in the language.
My primary concern for myself is that I am very much a beginner in Ceylon, so the danger is that I will write unidiomatic code or create unusual project structures. I have tried to document my decisions so that if a more experienced Ceylon user comes along later, the rationale behind my decisions will be understandable and that informed decisions can be made on whether to change them.
I can't promise "I will definitely get this track launched", but I will definitely work to satisfy my curiosity on Ceylon, and I hope the work can become useful to someone.

I've invited @chochos to the team. He's a member of the Ceylon core team, and is willing to provide feedback on the pull requests.

@petertseng if you have any doubts you can contact the team directly at our gitter page https://gitter.im/ceylon/user

authors: chochos kytrinyx petertseng

repo: exercism/xceylon


text:
RFC: Mitigating DST troubles

For some reason, I've lately been thinking a lot about the troubles we had earlier this year with the gigasecond exercise, and I've come up with some ideas about detecting and preventing future issues with that or perhaps with future exercises (todo: make list of other datetime exercises).
Now that we have exercise/example testing setup, the first is simply to schedule a TravisCI build for the DST switch dates in the US. If the tests fail for some reason, the scheduled build should let us know almost as soon as can be known. I poked around the TravisCI settings and didn't see anything like this, so I thought about setting up an IFTTT solution (I'm pretty sure I have an account, but I don't remember the password, as I don't think I've ever used it). Any other suggestions welcome.
The second is that it's seems possible that perhaps only a few CL implementations could be affected by the DST switch due to a bug in time handling. It would be nice to figure out how to conditionally allow failures on a per-test-case basis, so that even if there's a problem with the DST affected exercise tests on that implementation, we can still get feedback on the other exercises until the issue is resolved (or until the switch back).
If more serious problems turn up, we could also look for a date-time library for CL that's well maintained and, if that works out, recommend it to users.
Lastly, I think we should also consider a ---well maybe "policy" is too strong a word, but something between a recommendation and a policy, where we prefer time-dependent test implementations to work for standard time, if for some reason we can't make them work for both DST and ST.
I don't suggest that any of these be permanent or immediately acted upon, I mainly wanted to get my thoughts on the topic out there and collect some ideas going forward.
http://www.nist.gov/pml/div688/dst.cfm
Keep the faith, someday, sometime, the tyranny will end.

On Mon, 22 Jun 2015 22:59:23 -0700, William Clifford notifications@github.com said:

wobh> [...] to schedule a TravisCI build for the DST switch dates in
wobh> the US. If the tests fail for some reason, the scheduled build
wobh> should let us know almost as soon as can be known.

I like this idea of setting it up so that the Travis job fires at the
DST change times. It appears that you are not the first to ask how to do
this
http://stackoverflow.com/questions/20395624/travis-ci-builds-on-schedule.
wobh> The second is that it's seems possible that perhaps only a few
wobh> CL implementations could be affected by the DST switch due to
wobh> a bug in time handling.

I had not thought about this possibility. I don't remember asking the
people submitting the reports of broken tests (is there a name for a
person who does Exercism exercises?) what implementations they were
using - but maybe one of us did and we have some data?
wobh> It would be nice to figure out how to conditionally allow
wobh> failures on a per-test-case basis, so that even if there's a
wobh> problem with the DST affected exercise tests on that
wobh> implementation, we can still get feedback on the other
wobh> exercises until the issue is resolved (or until the switch
wobh> back).

It seems to me (without deep review / thought) that the existing test
harness runs all tests and then reports pass/fail. So if that is correct
then even if a time related test failed due to DST/TZ issues we'd still
get feedback from the other tests. Or are you asking about allowing such
a test to fail - but allow the entire run for that implementation to
pass?
This sort of feature sounds like something we could build into the test
harness. But it also feels like the sort of thing that we should not add
unless we are sure we need it. Allowing some tests to sometimes fail and
not fail the test run sounds like a good way to not notice a problem.
wobh> Lastly, I think we should also consider a ---well maybe
wobh> "policy" is too strong a word, but something between a
wobh> recommendation and a policy, where we prefer time-dependent
wobh> test implementations to work for standard time, if for some
wobh> reason we can't make them work for both DST and ST.

I think a better policy is to not have time related exercises in
Exercism.IO. While time issues always seem easy, they never are. It
seems to me that complicated topics such as time or threading should be
kept out of these exercises.
wobh> Keep the faith, someday, sometime, the tyranny will end.

The tyranny of time? Down with clocks!
Suddenly I just thought of Metric Time - maybe that is what you mean by
'standard time'?

Good points, especially the last. I think we're agreed that we should at least set up some automatic test runs starting on 2015-11-01T02:00:00 (maybe a little later) so from here on out it's just a question of means and procedures.
TronCI seems like a good find, but I find myself a bit uncomfortable turning over my github credentials for this. I suppose it's a baseless worry though.
I think we should setup the test runs to use Eastern time per http://stackoverflow.com/a/23374438 and I think it might be a good idea to setup up the run time for a couple minutes after the change over. I wonder if we could set the datetime on the Travis VM and run some tests in advance?

Perhaps another way to approach this problem is to change the gigasecond interface to include a timezone parameter?

Hey, guess what came and went without any fuss?

Happy winter, everyone.

authors: kytrinyx verdammelt wobh

repo: exercism/xlisp


text:
#49 Add generics to Anagram exercise

Closes issue #49.
@masters3d Let me know what you think of the style. This is the most common formatting I've seen.

Looks good. That's what I was thinking.

repo: exercism/xobjective-c


text:
Deprecated cops, deprecated file patterns

Made corrections for deprecated file patterns and style cops

This is not ready, gem version for Rubocop needs to be specified.

@kytrinyx Where should we set the version requirement for Rubocop?

Is it not in the .travis.yml file? We could add a gemfile, too.

It is indeed in the travis file.  What version should we use?  The current version is 0.36.0 and unfortunately, my local check script does not check travis.yml file.
The real problem becomes:  The local check in bin does not enforce the rubocop running version based on the travis config setting, which is the real cause of this confusion.

Yeah, let's add a Gemfile -- then we can use bundle exec locally.
If the most recent version of rubocop works with what we have, then we may as well upgrade, I think.

This means that the travis file should use Gemfile to keep things up to date.  One place to do the configuration.
Then it will not be 'bundle exec locally' bundle exec no matter where.
I have the following code example for the Gemfile.
However, perhaps the travis config file should be to do the needful, with the bundle exec command instead, and then instead of the gem file getting gem information from Travis, travis configuration does its commands from there.
source "https://rubygems.org"
require 'yaml'

# We read the travis.yml file and pull information from there
config = YAML.load(IO.read('./.travis.yml')).to_hash

config['install'].each do |g|
  gem_detail = g.split
  # Example lines from travis, ignoring the yaml array directive "- "
  # gem install rubocop -v 0.36.0 --no-document
  # gem install minitest --no-document
  # These will both be appropriate as the 2nd and 4th elements will work with
  # the gem command as parameters appropriately.
  gem gem_detail[2], gem_detail[4]
end

I like the code better that will be coming in shortly though. :)

This means that the travis file should use Gemfile to keep things up to date. One place to do the configuration.
Then it will not be 'bundle exec locally' but bundle exec no matter where.

Agreed. Also agreed that travis should just bundle and use that.
This all looks good, thanks!

authors: kotp kytrinyx

repo: exercism/xruby


text:
Placed sieve between pangram and roman numerals.

Hello! this is in reference to #258. I have changed config.json to place the sieve exercise between pangram and roman numerals.

Looks good.

authors: kotp mostlybadfly


text:
Unable to share map since updating to 1.2.3

Trying to share a corporation map with a second corp, when you enter the corp name in the input box it just says "No Results Found" no matter what is entered.  This has only happened since the update to 1.2.3, it worked fine on 1.2.2.

I can confirm that not much corps show up, but some actually do. Checking my sql database I found out, that only the names from the corp table which have the shared attribute set to 1, although I have no clue what that setting is actually doing.

1 member of the corp must set share Corp maps to allow from the Corp you are attempting to add. You must also set this to allow. if you dont and no member of the corp  you are attempting to add has it set to allow the list will not populate

Menu(Top Left) -> Account -> Share -> Corp Maps - Enable

Awesome that fixed it, thanks heaps!

This is actually in the documentation.  Click Map -> Manual and the scroll down to 'Sharing maps'.

Yeah this is some kind of "spam protection" on public servers. Corps/Allies have a "MAX_COUNT" for maps (pathfinder.ini).
It could be annoying if a hostile group send "share invites". Ship locations will be shared as well...

maybe remove corp share option from all users and add it to the new admin page so only those with corp director roles can enable. to prevent Spy activity. as most people won't notice an additional map.


text:
Typo in results key -> context_ relevancy

results.keys() return
dict_keys(['answer_relevancy', 'context_ relevancy', 'faithfulness', 'ragas_score'])
where there is a whitespace in context_ relevancy
Due to https://github.com/explodinggradients/ragas/blob/9f54d016fd0743c4e6abac585cba76d36e3b1436/src/ragas/metrics/context_relevance.py#L110

Thanks a lot for pointing this out @ogencoglu, I've fixed it and will be out in the next release
Apologies for the delay from my end but do let us know your feedback on using ragas :)

authors: jjmachan ogencoglu

repo: explodinggradients/ragas