Improve five_tuple code coverage

change Protocol to uint8.
FiveTuple.Equal check with Protocol.

Codecov Report

Merging #50 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #50   +/-   ##
=======================================
  Coverage   11.79%   11.79%
=======================================
  Files          16       16
  Lines         670      670
=======================================
  Hits           79       79
  Misses        574      574
  Partials       17       17

Impacted Files
Coverage Δ

internal/allocation/five_tuple.go
100% <100%> (ø)
: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 62a0992...a165058. Read the comment docs.

LGTM


USB suspend-resume process is not properly handled

Describe the bug
The USB subsystem does not properly handle the suspend-resume process.
When a USB device is suspended, all on-going transfers are cancelled:
https://github.com/zephyrproject-rtos/zephyr/blob/0651688dd350851ab5e00265d1170f3f015e77c0/subsys/usb/usb_device.c#L1017-L1025
The CDC ACM device class implementation, for instance, does not restore the cancelled USB transfer upon the resume event and this results in the USB transfer remaining in cancelled state:
https://github.com/zephyrproject-rtos/zephyr/blob/0651688dd350851ab5e00265d1170f3f015e77c0/subsys/usb/class/cdc_acm.c#L437-L440
This causes the USB device to become unresponsive.
To Reproduce

Build and flash samples/subsys/usb/cdc_acm
Open the board USB device serial port and see that everything is echoed.
Close the board USB device serial port
Suspend the board USB device.
Resume the board USB device.
Open the board USB device serial port and see that the device does not echo anything back.

Expected behavior
USB endpoint transfers are in working state after device is resumed.
Impact
USB suspend-resume process does not work, and any attempt by the host to suspend the device will result in the device being unusable until the device is re-enumerated (and configured).
Environment (please complete the following information):

OS: Ubuntu 18.04
Toolchain: Zephyr SDK 0.11.3
Commit SHA: cbb252adef08ba482b14ef2ab562baf879956c93

Additional context
Note that this issue is not specific to the CDC ACM device class; a quick glance over other device class implementations suggest that they will be affected by this issue as long as the USB transfers are cancelled when a suspend event is received.
Also note that the USB_DC_DISCONNECTED event is not implemented or, as far as I can tell, does not work on any platforms.

cc @finikorg @jfischer-phytec-iot

To briefly summarise the kind of mess we are in:

USB drivers do not report USB_DC_DISCONNECTED event because most USB device cores do not report such a thing (and there is a good reason for that), and we must rely on the USB_DC_SUSPEND event occurring when the device is disconnected (and, of course, suspended).
#16193 made the USB subsystem cancel/free all on-going transfer slots upon USB_DC_SUSPEND (in this case, disconnection) in order to make sure that re-connection does not result in USB transfer slot leak.
In case of a genuine suspend condition and in case the device is resumed thereafter, the device is not re-enumerated (and configured), so the USB transfer is not re-initiated.

To Reproduce
1. Build and flash `samples/subsys/usb/cdc_acm`

2. Open the board USB device serial port and see that everything is echoed.

3. Close the board USB device serial port

4. Suspend the board USB device.

5. Resume the board USB device.

6. Open the board USB device serial port and see that the device does not echo anything back.

Can not reproduce, works fine on nRF52840. It works even with #26657, but only few times.

@jfischer-phytec-iot I can reproduce this issue on nRF52840 (nrf52840dk_nrf52840). Please re-check.

The CDC ACM device class implementation, for instance, does not restore the cancelled USB transfer upon the resume event and this results in the USB transfer remaining in cancelled state:

Expected transition is SUSPEND->RESUME, this behavior is correct. Everything after USB_DC_DISCONNECTED will reset the device and reconfigure usb_transfer. If you platform does not support detached or disconnected events suspend can for sure used instead but you must implement resume too.

Expected transition is SUSPEND->RESUME, this behavior is correct. Everything after USB_DC_DISCONNECTED will reset the device and reconfigure usb_transfer. If you platform does not support detached or disconnected events suspend can for sure used instead but you must implement resume too.

@jfischer-phytec-iot I think you are still thinking about the other issue (#26656). This is different.
I am not talking about the disconnect/detach event here, but an actual suspend-resume event; and yes, you need to implement both suspend and resume events to get suspend and resume features working -- that was clear enough.
Try turning on autosuspend on the host and you will see that nRF52840 stops responding on the ACM serial port after the device goes through suspend and resume.

Also I forgot to mention, this issue is seen on all platforms I tested (sam0, sam, kinetis, nrf).
This is a logic issue, not a platform driver issue.

More detailed steps on how to reproduce this:

Turn on autosuspend

echo 'auto' > '/sys/bus/usb/devices/(Zephyr USB Device)/power/control'

Wait for usb_cdc_acm: USB device suspended to be printed on the UART console

Open the board USB ACM serial port (usb_cdc_acm: USB device resumed will be printed on the UART console)

Now you will see that the USB ACM serial port does not echo anything and became completely unresponsive for the reasons described above.

This is a logic issue, not a platform driver issue.

Where is the issue? What you describe is correct behavior:

The CDC ACM device class implementation, for instance, does not restore the cancelled USB transfer upon the resume event and this results in the USB transfer remaining in cancelled state:

Where is the issue? What you describe is correct behavior:

So you are saying that the ACM device class implementation not restoring the USB transfer after resume and being unable to communicate with the host is the correct behaviour?
I don't think it takes an expert to see that is a problem unless, of course, your goal is to maintain the status quo that is an unusable substandard implementation.

So you are saying that the ACM device class implementation not restoring the USB transfer after resume and being unable to communicate with the host is the correct behaviour?

No, I read your sentence too quickly. The transfers are cancelled not disabled. On resume event CDC ACM class sets internal state to dev_data->usb_status = USB_DC_CONFIGURED; and allows to fill the fifo (tx_work_handler -> usb_transfer will pass the data to IN endpoint buffer), callbacks for the OUT endpoint are still configured for the usb_transfer and will be triggered.

I can not reproduce it, I should try an other host. Probably regression or was not really fixed in https://github.com/zephyrproject-rtos/zephyr/issues/17488

The log indicates that cdc_acm_read_cb is not called after resume.
<Suspending>
[00:00:30.384,155] <dbg> usb_transfer.usb_cancel_transfers: Cancel transfer
[00:00:30.384,185] <dbg> usb_cdc_acm.cdc_acm_dev_status_cb: cfg 0x200002d4 status 5
[00:00:30.384,185] <inf> usb_cdc_acm: USB device suspended
[00:00:30.384,216] <dbg> usb_transfer.usb_transfer_work: Transfer cancelled or completed, ep 0x01
[00:00:30.384,216] <dbg> usb_transfer.usb_transfer_work: Transfer done, ep 0x01, status -72, size 0
<Resuming>
[00:00:53.035,644] <dbg> usb_cdc_acm.cdc_acm_dev_status_cb: cfg 0x200002d4 status 6
[00:00:53.035,675] <inf> usb_cdc_acm: USB device resumed
[00:00:53.102,294] <dbg> usb_device.usb_handle_control_transfer: ep 0x00, status 0x00
[00:00:53.102,325] <dbg> usb_device.custom_handler: bRequest 0x00, wIndex 0x0000
[00:00:53.102,325] <dbg> usb_device.usb_handle_std_device_req: REQ_GET_STATUS
[00:00:53.102,416] <dbg> usb_device.usb_handle_control_transfer: ep 0x80, status 0x02
[00:00:53.102,905] <dbg> usb_device.usb_handle_control_transfer: ep 0x00, status 0x00
[00:00:53.102,935] <dbg> usb_device.class_handler: bRequest 0x22, wIndex 0x0000
[00:00:53.102,935] <dbg> usb_cdc_acm.cdc_acm_class_handle_req: CDC_SET_CONTROL_LINE_STATE 0x3
[00:00:53.103,363] <dbg> usb_device.usb_handle_control_transfer: ep 0x00, status 0x00
[00:00:53.103,546] <dbg> usb_device.usb_handle_control_transfer: ep 0x00, status 0x01
[00:00:53.103,576] <dbg> usb_device.class_handler: bRequest 0x20, wIndex 0x0000
[00:00:53.103,607] <dbg> usb_cdc_acm.cdc_acm_class_handle_req: CDC_SET_LINE_CODING 115200 0 0 8
[00:00:53.103,607] <dbg> usb_device.usb_handle_control_transfer: >> usb_data_to_host(2)
<After sending a char on the ttyACM>
[00:01:07.453,948] <dbg> usb_transfer.usb_transfer_work: Transfer cancelled or completed, ep 0x01

I can not reproduce it, I should try an other host. Probably regression or was not really fixed in #17488

Is your host configuring (REQ_SET_CONFIGURATION) the device after resume? Because if it does, this problem will not be reproducible since USB_DC_CONFIGURED calls cdc_acm_read_cb.
It is my understanding that REQ_SET_CONFIGURATION should not be sent after resume and that would indeed be your host's problem.

I can reproduce it on PC, sorry, I think it is a regression as I tested it to fix #17488

At fist glance the fix for cdc acm class could be to re-trigger the transfer for OUT endpoint:
diff --git a/subsys/usb/class/cdc_acm.c b/subsys/usb/class/cdc_acm.c
index 54ebcc6bd9..036a47f7e0 100644
--- a/subsys/usb/class/cdc_acm.c
+++ b/subsys/usb/class/cdc_acm.c
@@ -436,6 +436,8 @@ static void cdc_acm_do_cb(struct cdc_acm_dev_data_t *dev_data,
                break;
        case USB_DC_RESUME:
                dev_data->usb_status = USB_DC_CONFIGURED;
+               cdc_acm_read_cb(cfg->endpoint[ACM_OUT_EP_IDX].ep_addr, 0,
+                               dev_data);
                LOG_INF("USB device resumed");
                break;
        case USB_DC_SOF:

@jfischer-phytec-iot Yes, that works for the most part.
The only problem is that some platforms trigger the USB_DC_RESUME event multiple times when resuming and that may cause multiple transfer slots to be allocated for single endpoint. For instance, on frdm_k64f, I see usb_cdc_acm: USB device resumed happening twice after the device is resumed (cc @MaureenHelm). I suppose we call that a bug with the Kinetis USB driver and address it separately; though, I am also seeing something similar happening on the SAM0 platform (still investigating).
On a side note, I am not an expert on the USB stuff, but does it make sense to cancel all on-going transfers when suspending? I think @anangl raised the same question here too: https://github.com/zephyrproject-rtos/zephyr/issues/17488#issuecomment-516800469
I wonder if it would be better to free all transfer slots (i.e. call usb_cancel_transfers) before configuring the device (i.e. as a first thing in REQ_SET_CONFIGURATION handler) since that is the place where a new set of endpoints are being activated and it makes sense for all existing transfers to be terminated.

@jfischer-phytec-iot Yes, that works for the most part.
The only problem is that some platforms trigger the USB_DC_RESUME event multiple times when resuming and that may cause multiple transfer slots to be allocated for single endpoint. For instance, on frdm_k64f, I see usb_cdc_acm: USB device resumed happening twice after the device is resumed (cc @MaureenHelm). I suppose we can call that a bug with the Kinetis USB driver and address it separately; though, I am also seeing something similar happening on the SAM0 platform (still investigating).

I change the patch to filter spurious resume events which may happen during device detection. I had not time to test on frdm_k64f but will do. Not all the drivers are aligned in the behavior.

On a side note, I am not an expert on the USB stuff, but does it make sense to cancel all on-going transfers when suspending? I think @anangl raised the same question here too: #17488 (comment) #17492 (comment)

We do that on suspend and disconnect events, https://github.com/zephyrproject-rtos/zephyr/blob/master/subsys/usb/usb_device.c#L1017

I wonder if it would be better to free all transfer slots (i.e. call usb_cancel_transfers) before configuring the device (i.e. as a first thing in REQ_SET_CONFIGURATION handler) since that is the place where a new set of endpoints are being activated and it makes sense for all existing transfers to be terminated.

That will be done, if necessary, at the line https://github.com/zephyrproject-rtos/zephyr/blob/master/subsys/usb/usb_device.c#L537


chore(deps): update dependencies

And it looks like I forgot about it again, closing in favor of https://github.com/obartra/picturebook/pull/35


[RFE] Support more SNMP value types for SET command

The net-snmp site shows
 type - one of i, u, t, a, o, s, x, d, n
    i: INTEGER, u: unsigned INTEGER, t: TIMETICKS, a: IPADDRESS
    o: OBJID, s: STRING, x: HEX STRING, d: DECIMAL STRING
    U: unsigned int64, I: signed int64, F: float, D: double

but the help for snmpset.py is missing x: HEX STRING, d: DECIMAL STRING, F: float, and D: double
Management parameters:
   <[mib-module::]object-name|oid type| = value> ...
              mib-module:           MIB name (e.g. SNMPv2-MIB)
              object-name:          MIB symbol (e.g. sysDescr.0) or OID
              type:                 MIB value type
                    i               integer
                    u               unsigned integer
                    s               string
                    n               NULL
                    o               ObjectIdentifier
                    t               TimeTicks
                    a               IP address
              =:                    use MIB for value type lookup
              value:                value to write

For my team we use sets of x type daily like this
$ snmpget -v2c -r1 -c <community string> udp6:2001:558:4033:59:4a1d:70ff:fe65:691d .<IP_ADDRESS>.<IP_ADDRESS>.3.7.0
SNMPv2-SMI::mib-<IP_ADDRESS>.7.0 = Hex-STRING: BA DB AD BA DB AD BA DB AD BA DB AD BA DB AD 00
$ snmpset -v2c -r1 -c <community string> udp6:2001:558:4033:59:4a1d:70ff:fe65:691d .<IP_ADDRESS>.<IP_ADDRESS>.3.7.0 x "FE 80 CD 00 00 00 0C DE 12 57 00 00 21 1E 72 9C"
SNMPv2-SMI::mib-<IP_ADDRESS>.7.0 = Hex-STRING: FE 80 CD 00 00 00 0C DE 12 57 00 00 21 1E 72 9C
$ snmpget -v2c -r1 -c <community string> udp6:2001:558:4033:59:4a1d:70ff:fe65:691d .<IP_ADDRESS>.<IP_ADDRESS>.3.7.0
SNMPv2-SMI::mib-<IP_ADDRESS>.7.0 = Hex-STRING: FE 80 CD 00 00 00 0C DE 12 57 00 00 21 1E 72 9C
$

We use the others rarely but there can be cases for them.
Is there a reason the snmpclitools doesn't support the other types?

@etingof I will work on that, any suggestion would be appreciated.

Would be great! If you share your ideas/problems in form of a PR, I will try to help out with the details if needed.


add more distance in PDF for DEL-tags in new lines (fixes #3205)

Not tested, since I cannot reproduce the issue.
Provide a database where the issue occurs if the problem persists
@emanuelschuetze

@tsiegleauq Thanks. I made some additional changes and rebased it together into your commit. Successfully tested. Looks good now for different cases.

Changelog entry added.


Current generated SSL Certs are unusable in browsers like chrome and safari

Integrate https://github.com/FiloSottile/mkcert to use trustworthy ssl certificates which are accepted by browser

Hi
thank you for this idea
could you give me a short feedback
so the tool installs a local CA, which should be done on the host....
that means that
a) container need a pre installed certificate that its delivered with
b)still needs a manual installation of the CA on the host system
is that correct?
cheers
christian

Hi
thank you for this idea
could you give me a short feedback
so the tool installs a local CA, which should be done on the host....
that means that
a) container need a pre installed certificate that its delivered with
b)still needs a manual installation of the CA on the host system
is that correct?
cheers
christian

by the way
i just saw the title
its not unusable, we indeed run https on local dev machines, also use apple pay and stuff like that in safari
you can just continue when the warning happens
so its only a warning which is kinda nice to know that you are on the local dev system and not accidentally on a real one
just to let you know that you dont have to wait for a mkcert to use it :)

by the way
i just saw the title
its not unusable, we indeed run https on local dev machines, also use apple pay and stuff like that in safari
you can just continue when the warning happens
so its only a warning which is kinda nice to know that you are on the local dev system and not accidentally on a real one
just to let you know that you dont have to wait for a mkcert to use it :)

i will close this ticket for now
at the moment it will stay as it is
still thank you for this idea


Network resources (the doctor check crashed  Exception: Network resources exceeded maximum allowed duration of 0:04:30.000000

Steps to reproduce
I have no idea
Actual results
I can't create a flutter project, I can't run flutter pub get and also I got this error running flutter doctor
Logs

Logs

<!-- Paste your logs here -->

Flutter Doctor output

Doctor output

C:\Users\jaspe>flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.16.4, on Microsoft Windows [Version 10.0.22000.2538], locale en-US)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[!] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    ! Some Android licenses not accepted. To resolve this, run: flutter doctor --android-licenses
[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.7.6)
[√] Android Studio (version 2023.1)
[√] VS Code (version 1.85.1)
[√] Connected device (3 available)
[☠] Network resources (the doctor check crashed)
    X Due to an error, the doctor check did not complete. If the error message below is not helpful, please let us know
      about this issue at https://github.com/flutter/flutter/issues.
    X Exception: Network resources exceeded maximum allowed duration of 0:04:30.000000

! Doctor found issues in 2 categories.

Thanks for filing this @jasfer223.
This seems similar to https://github.com/flutter/flutter/issues/125488.
Kindly check this out if this describes your issue?

I have the same problem too, it gets stuck when it tries to connect to pub.dev

I fixed mine by running cleaning the cache of pub by running this command:
dart pub cache clean


Update Chinese Traditional Translations

Details of Changes
Add details of changes here.

[Commit 7c76fe2] Translated new strings
[Commit 7e4b08a] Fixed many old strings
[Commit 7a9defb] Synchronize with the latest commit

Validation
How did you test these changes?

[x] Built and ran the app
[x] Tested the changes for accessibility

Why this PR hasn't been approved yet? 🤔

@open3 apologies for the delay. The developers working on Files are doing so on their free time so they can't always review PRs right away. We'll try to get to this PR in a little bit.

I think https://github.com/files-community/Files/pull/6878/commits/7a9defb87ffefbe9468ad26ebd95b14e5f572c25 messed up the privacy policy @open3 Could you revert it?

@d2dyno1 What do you mean? The string exists when I open the file with the multilingual editor.

Oh 😅 my bad. I confused the original string with the new one and thought the old one shouldn't be here. No issues here!

@open3 can you take a look at #7625?


fix: release config

Description

Fix the workflow

Type of change

[x] CI CD

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have checked my code and corrected any misspellings

:tada: This PR is included in version 1.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix(rdbms): check number regex

Regex for checking number is ('^[0-9]+(\.[0-9])*$'), which makes floats with 2+ precision like 0.01 was not treat as a number.

Nice catch -- but then the * needs to be changed to ?, right?

yes nice catch as well 😆 and the pr is updated

Rebased and merged to 2022.9.x and 2023.3.x


chore(main): release 1.5.0

:robot: I have created a release beep boop
1.5.0 (2023-05-15)
Features

ack: add more ack (#184) (f285c92)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/CCXXXI/thesis/releases/tag/v1.5.0 :sunflower:


clang analyzer reports potential memory leak

The clang analyzer on my machine reports the following potential memory leak:
14345: "Potential leak of memory pointed to by 'constant'"
14524: Assuming 'metadata' is non-null:
14524: Taking true branch
14524: Calling 'yp_parser_metadata':
14320: Assuming 'filepath_size' is 0:
14320: Taking false branch
14331: Assuming 'scope_index' is < 'scopes_count':
14331: Loop condition is true.  Entering loop body
14337: Assuming 'local_index' is < 'locals_count':
14337: Loop condition is true.  Entering loop body
14341: Memory is allocated:
14345: Potential leak of memory pointed to by 'constant'

I am not sure whether this is a false positive.

Just looked into this. There's a lot of indirection, but I'm sure this one is getting free-d because it's part of the constant pool


how to get a easy CLA from presto/presto project?

my account is<EMAIL_ADDRESS>I had send a pdf of CLA to<EMAIL_ADDRESS>but have not receive any email. I can not pass CLA check in my pull request

Hi @mahengyang
I did respond to your ticket in the support desk this morning, but I'll also put my answer here.
You have an ICLA signature on file in our system, so it isn't necessary to also email the pdf to mdolan.
The reason your pull request is not passing the CLA check is because GitHub does not recognize you as the author of your commits. If you notice here https://github.com/prestodb/presto/pull/13936/commits the icon for your commits is blank and there is no github account attached to them.
Github has a help article here https://help.github.com/en/github/committing-changes-to-your-project/why-are-my-commits-linked-to-the-wrong-user about how to fix this issue.
Once this is fixed, you may need to close and reopen that same pr for the cla check to trigger if it doesn't automatically.
Thanks for reaching out,
Heather

@WillsonHG thk! it is my fault, I had change my local email link to github, Easy CLA now is paased!


Use fraction instead of uint32 for percentage in fault injection, following the same change in Envoy

Is your feature request related to a problem? Please describe.
Envoy changed the percentage for fault injection to be a fraction instead of a uint32. The uint32 will be deprecated. I suggest a similar change in Istio.

Additional context
Link to the Envoy issue: https://github.com/envoyproxy/envoy/issues/3904
and PR: https://github.com/envoyproxy/envoy/pull/3978

I'd like to work on this.


refactor to make iteration over ents the primary way that messages are added

https://github.com/senecajs/seneca-github-provider/blob/main/src/github-provider.ts#L36
the for loop should iterate over the ents
there is no need for the switch. do this instead:
for( const ent of ents ) {
  seneca.message(ent.pattern, make_load(ent))
  seneca.message(ent.pattern, make_save(ent))
}

rename prepare_actions to prepare_ents and add the pattern to the ent description in that function
in general, avoid conditions by building the data structure to avoid them

@rjrodger I linked a pr to this issue


Propose reducing update frequency from 3 days to one week

Modified cron job to run once a week (every Saturday)


docs: surface new 1.12.0 features and enhancements

Issue #, if available:
Description of changes:

Surface new functionalities, enhancements, and refresh docs to better surface certain features previously hidden:

Parameters: surface how to change cache TTL
Parameters: add new force_fetch param
Parameters: refresh to use new navigation
Idempotency: add note on reusing a single table for idempotency state
Logger: surface usage of new correlation_id and paths
Dataclasses: remove additional note banners to ease reading
Dataclasses: update AppSync Resolver to emphasize correlation, typos

Checklist

[x] Meet tenets criteria
[ ] Update tests
[x] Update docs
[x] PR title follows conventional commit semantics

Breaking change checklist

RFC issue #:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@heitorlessa
I do have some code example use cases for force_fetch in the PR description:

https://github.com/awslabs/aws-lambda-powertools-python/pull/341

And a code example of the AppSync decorator:

https://github.com/awslabs/aws-lambda-powertools-python/pull/323

Codecov Report

Merging #344 (393405e) into develop (b365fba) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #344   +/-   ##
========================================
  Coverage    99.60%   99.60%
========================================
  Files           95       95
  Lines         3508     3508
  Branches       173      173
========================================
  Hits          3494     3494
  Misses           8        8
  Partials         6        6

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 b365fba...393405e. Read the comment docs.

@heitorlessa
I do have some code example use cases for force_fetch in the PR description:
* #341

And a code example of the AppSync decorator:
* #323

Thanks Michael - I found the first example being better suited for a new Guide section, as we'd want to keep it brief.
The App Resolver decorator, there's a high chance (90%) this will be moved once the RFC #325 and #322 get more details, so let's keep this undocumented for now.

Looks cool.
We might want to remove the New in 1.12.0 in future releases as we might start to clutter the docs. It could be some kind of expiring things :)
:shipit:

Actually there's going to be an even better news that this will no longer be necessary - Our documentation system now supports versioning, meaning we could have that for either per release tag OR per branch (less convoluted)
https://squidfunk.github.io/mkdocs-material/changelog/#700-_-february-22-2021

@heitorlessa
I do have some code example use cases for force_fetch in the PR description:
* #341

And a code example of the AppSync decorator:
* #323

Thanks Michael - I found the first example being better suited for a new Guide section, as we'd want to keep it brief.
The App Resolver decorator, there's a high chance (90%) this will be moved once the RFC #325 and #322 get more details, so let's keep this undocumented for now.

Sounds good. Maybe we have a cookbook guide somewhere that has more detailed use cases (or it could just be blog posts we refer to)


Results?

What kind of results do you get? It would be really cool if you could post some learning curves, or even just the final mean IoU and accuracy on the camvid dataset for reference? :-)

Updated the README with our results. I believe the authors excluded the background class from their global accuracy calculation. If we do the same our results are close, but not matching their result.

The author does exclude the void class if that is what you mean

Is it possible to also include the mean intersection over union metric? I find it to be better for comparisons on these highly skewed datasets.


Cross-chain compatibility for PairUniswapV2, PairStableSwap and PairATokenV2

These pair implementations are cross-chain compatible if we just make them take in Web3 instead of ContractKit.
Need these compatibility changes so that Swappa can be deployed on other EVM compatible networks.
PairAToken should also be easily cross-chained with some more changes but it's not really important even on other chains.

Have you tested this out with other chains? Does generated code work with plain old Web3?

Yeah, tested + deployed to avalanche and polygon, work fine with plain old web3.

For generated code, some of the Celo contract kit stuff were swapped with more generic Web3 stuff, can post those in a follow-up PR, but fairly minimal changes to get swappa working cross-chain


renderer: factorize and move some computation done every frame for every surface in tr_shade to be done only once in tr_shader

Do a refactorization of some renderer thing, move some computation that are done every frame for every surface in tr_shade.cpp to be done only once at map load (or when shader is created) in tr_shader.cpp.
This makes also easier to implement some features, pull request #307 (engine-agnostic normal fomat) relies heavily on this.
Also deletes 150 lines.

Looks OK stylistically. I didn't try to closely review for correctness.

I have tested with many maps from many games without noticing any regression. The changes are not rocket science anyway, were just boring and annoying to do.


[oip] First pass at an incident response

First pass at an incident response OIP

Thanks @jet86. I can't English.

All good - it's much easier to spot typos in someone else's writing than your own. I make typos all the time too :)


Why BigQuery insted of Redshift?

I just read  http://blog.gingerlime.com/2016/a-scalable-analytics-backend-with-google-bigquery-aws-lambda-and-kinesis/ and why  not use Redshift insted of BigQuery?

No particular reason other than that Redshift costs much more than BigQuery as far as I could tell. With Redshift, you need to set up your own cluster and keep it running. Right? BigQuery charges based on queries and storage.

ok thanks, I will take a better look

You're welcome :)


Bump elliptic - v21

Change Description
Bump elliptic version due to security vulnerability
API Changes

[ ] API Changes

Documentation Additions

[ ] Documentation Additions

Checking with product before merging this.

Product approval, so good to go on merging this update in to RC3.


missing example usage

A full example usage is missing so it's difficult to know what to do.  I might piece it together after staring at it for an hour but that's too high a bar for something that is trying to appear simple and easy to use.
Walk me through running this on stampede for example. Just a full typescript of a session showing modules loaded (python), the code you're going to run, env variables you've set, and the batch submit (can do it in idev ? ) would be nice.  Then I can try to follow that.

Interactive walk through in process in tests/ (see  #23 and #26). Script acts as a tutorial/example.

Lucas,
Can you send me a transcript of the script being run.  I'm unable to get it
to work properly, and frequently get:
Please do so again: you did not enter an appropiate y or n. Please do so
again: you did not enter an appropiate y or n. Please do so again: you did
not enter an appropiate y or n. Please do so again: you did not enter an
appropiate y or n. Please do so again: you did not enter an appropiate y or
n. Please do so again: you did not enter an appropiate y or n. Please do so
again: you did not enter an appropiate y or n. Please do so again: you did
not enter an appropiate y or n. Please do so again: you did not enter an
appropiate y or n. Please do so again: you did not enter an appropiate y or
n. Please do so again: you did not enter an appropiate y or n. Please do so
again: you did not enter an appropiate y or n. Please do so again: you did
not enter an appropiate y or n. Please do so again: you did not enter an
appropiate y or n. Please do so again: you did not enter an appropiate y or
n. Please do so again: you did not enter an appropiate y or n. Please do so
again: you did not enter an appropiate y or n. Please do so again: you did
not enter an appropiate y or n. Please do so again: you did not
On Tue, Jul 25, 2017 at 9:33 AM, Lucas A. Wilson<EMAIL_ADDRESS>wrote:

Interactive walk through in process in tests/ (see #23
https://github.com/TACC/launcher/pull/23 and #26
https://github.com/TACC/launcher/pull/26). Script acts as a
tutorial/example.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TACC/launcher/issues/19#issuecomment-317756898, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AK5iKxmCXpjkEmG-ffsnFnEi47iL6IR8ks5sRfyggaJpZM4N0cui
.

I've put a new script quicktest in the /tests directory which will allow you to do a sanity test. No inputs required on your end, just run:
cd tests; ./quicktest
I'm going to have @marshalllerner debug the interactive walkthrough.

cool, looks good now
On Tue, Jul 25, 2017 at 2:58 PM, Lucas A. Wilson<EMAIL_ADDRESS>wrote:

I've put a new script quicktest in the /tests directory which will allow
you to do a sanity test. No inputs required on your end, just run:
cd tests; ./quicktest
I'm going to have @marshalllerner https://github.com/marshalllerner
debug the interactive walkthrough.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TACC/launcher/issues/19#issuecomment-317854238, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AK5iK-B9BYQTCF_UMTxL1tVtk17OXbB2ks5sRkjogaJpZM4N0cui
.

for some reason when I correct something and then save it it adds the /r carriage returns because I'm using Windows OS. I'll correct that now.
On Jul 25, 2017, 3:04 PM -0500, Galen Arnold<EMAIL_ADDRESS>wrote:

cool, looks good now
On Tue, Jul 25, 2017 at 2:58 PM, Lucas A. Wilson<EMAIL_ADDRESS>wrote:

I've put a new script quicktest in the /tests directory which will allow
you to do a sanity test. No inputs required on your end, just run:
cd tests; ./quicktest
I'm going to have @marshalllerner https://github.com/marshalllerner
debug the interactive walkthrough.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TACC/launcher/issues/19#issuecomment-317854238, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AK5iK-B9BYQTCF_UMTxL1tVtk17OXbB2ks5sRkjogaJpZM4N0cui
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

#31 adds a section to the README.md about verifying Launcher installation using the quicktest script.


Update office-store-checklist.md

Docs Build status updates of commit c527aec:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/publishing/office-store-checklist.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


ESP32 WIFI LORA (IDFGH-4413)

Hi, I have this message showing up.
I use an ESP32 WIFI LORA with a GY-GPS6MV2 module
I use the IDE and when compiling it asks to press the RST
when I open the serial monitor, this message appears and stabilizes, it does nothing else.
rst:0x1 (POWERON_RESET),boot:0x17 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:2
load:0x3fff0018,len:4
load:0x3fff001c,len:1216
ho 0 tail 12 room 4
load:0x40078000,len:9720
ho 0 tail 12 room 4
load:0x40080400,len:6364
entry 0x400806b8

Hi @JenniferGladysPereiraCavalcante, ESP32 WIFI LORA can only use with another ESP32, It won't work with others type of chips

Thanks for reporting.


Can you provide the api source code on the git hub ,like how did you communicate with the Azure notification hub from app center portal , It would help any one from not an API background like me to understand and implement the same ,just to continue the process of Push uninterrupted , as i am using it now on production app.

[Enter feedback here]

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

ID: 849cb3f8-82b9-0657-7b21-ab81214d8fbd
Version Independent ID: 4a79c4db-6586-6770-c470-c8abfae29861
Content: Push Migration Guide - Visual Studio App Center
Content Source: docs/migration/push/index.md
Service: vs-appcenter
Sub-service: app-center-general
GitHub Login: @jwargo
Microsoft Alias: jowargo

@rockclane You cannot communicate with ANH from the App Center Portal. They're completely different systems (although App Center uses ANH under the covers). You'll have to switch to using both the ANH SDK (to register devices) and the ANH backend REST APIs to send notifications.


Adjust the grid component so that the input is not misaligned when the error message appears

References: #51
What?

Remove the "align-items: 'center'" prop from the Grid component

Why?

The "align-tems" property was causing the input to be misaligned when the error message appeared

Hello, @RenanRSilva thanks for your PR! Can you please show us the entire form? Just to check how it looks like with this fix


Better tools for working with logprobs

results.select('kid_raw_model_response')[0]['raw_model_response.kid_raw_model_response'][0]['choices']

Questions now have a "simple_ask" method that helps here. The problem is that our more advanced way of doing it kind of breaks things because the probabilities are over JSON formatting elements (blech)


Bump Deps

Once rust-lightning updates bitcoin it can be updated here.
https://github.com/lightningdevkit/rust-lightning/pull/3063

I think once on the new version of bitcoin we can also use their re-export of base64 and remove it as a direct dep.
https://github.com/cashubtc/cdk/blob/267e5f044d8e99a698dd6662b54483a65d8648b9/crates/cdk/Cargo.toml#L21


Added functional Galveston College scraper

@jodizzle I was able to refactor it to simplify it even further because of how from_response handles things.  How does it look?

This looks very clean now.


Unexpected value 'Ng2CompleterModule' imported by the module 'AppModule'

I'm trying to use your ng2-completer component inside of an ionic 2 project and I'm getting the following error.
`ngc error: Error: Unexpected value 'Ng2CompleterModule' imported by the module 'AppModule'
at C:\Users\Hyrum\Documents\ProjectDev\test\ionic-conference-app\node_modules@angular\compiler\bundles\compiler.umd.js:14109:37
at Array.forEach (native)
at CompileMetadataResolver.getNgModuleMetadata (C:\Users\Hyrum\Documents\ProjectDev\test\ionic-conference-app\node_modules@angular\compiler\bundles\compiler.umd.js:14094:46)
at C:\Users\Hyrum\Documents\ProjectDev\test\ionic-conference-app\node_modules@angular\compiler\bundles\compiler.umd.js:12936:58
at Array.forEach (native)
at OfflineCompiler.analyzeModules (C:\Users\Hyrum\Documents\ProjectDev\test\ionic-conference-app\node_modules@angular\compiler\bundles\compiler.umd.js:12935:21)
at CodeGenerator.codegen (C:\Users\Hyrum\Documents\ProjectDev\test\ionic-conference-app\node_modules@angular\compiler-cli\src\codegen.js:105:47)
at codegen (C:\Users\Hyrum\Documents\ProjectDev\test\ionic-conference-app\node_modules@angular\compiler-cli\src\main.js:7:81)
at Object.main (C:\Users\Hyrum\Documents\ProjectDev\test\ionic-conference-app\node_modules@angular\tsc-wrapped\src\main.js:30:16)
at Object. (C:\Users\Hyrum\Documents\ProjectDev\test\ionic-conference-app\node_modules@angular\compiler-cli\src\main.js:14:9)
[14:26:22]  ngc error: Compilation failed
[14:26:22]
[14:26:22]  bundle prod started ...
[14:26:22]  Error: Could not resolve entry (.tmp/app/main.prod.js)
at C:\Users\Hyrum\Documents\ProjectDev\test\ionic-conference-app\node_modules\rollup\dist\rollup.js:8602:28
at process._tickCallback (node.js:369:9)
[14:26:22]  sass started ...
[14:26:24]  sass finished in 1.65 s
[14:26:24]  minify started ...
[14:26:24]  cleancss started ...
[14:26:24]  uglifyjs started ...
[14:26:24]  Error: ENOENT: no such file or directory, open 'C:\Users\Hyrum\Documents\ProjectDev\test\ionic-conference-app\www\build\main.js'
at Error (native)
at Object.fs.openSync (fs.js:549:18)
at Object.fs.readFileSync (fs.js:397:15)
at addFile (C:\Users\Hyrum\Documents\ProjectDev\test\ionic-conference-app\node_modules\uglify-js\tools\node.js:68:22)
at C:\Users\Hyrum\Documents\ProjectDev\test\ionic-conference-app\node_modules\uglify-js\tools\node.js:79:17
at Array.forEach (native)
at Object.exports.minify (C:\Users\Hyrum\Documents\ProjectDev\test\ionic-conference-app\node_modules\uglify-js\tools\node.js:77:26)
at runUglifyInternal (C:\Users\Hyrum\Documents\ProjectDev\test\ionic-conference-app\node_modules@ionic\app-scripts\dist\uglifyjs.js:34:19)
at runUglify (C:\Users\Hyrum\Documents\ProjectDev\test\ionic-conference-app\node_modules@ionic\app-scripts\dist\uglifyjs.js:23:28)
at Object.uglifyjs (C:\Users\Hyrum\Documents\ProjectDev\test\ionic-conference-app\node_modules@ionic\app-scripts\dist\uglifyjs.js:9:12)
[14:26:25]  cleancss finished in 1.07 s
[14:26:25]  minify finished in 1.07 s
[14:26:25]  build prod finished in 6.66 s
`
I've been unsuccessful in getting it to work in my application so I cloned this sample ionic project and built it according to the instructions given on the front page.
run "npm install" from the project root.
Install the ionic CLI with "npm install -g ionic"
run the project in a browser with "ionic serve"
The project runs fine and everything in it works.
I then add the ng2-completer module using the instructions given here and when I try to run the project again I get the above error.
I've looked for quite a bit for a solution and found this similar issue.
Any help would be greatly appreciated

@jjacobs33589 I'm not familiar with Ionic but I can see that it uses ngc so it might be better to import the source code and not the bundled package. try to:
import { Ng2CompleterModule } from "ng2-completer/src/ng2-completer.module";

I did try that and it yielded a dozen more issues. In looking into it more I came across this and wondered if it might be possible to include the metadata.json files with the ng2-completer component.

@oferh importing the source module brings another error from ngc
Error: Error encountered resolving symbol values statically. Calling function 'require', function calls are not supported. Consider replacing the function or lambda with a reference to an exported function, resolving symbol CompleterListCmp in ...

Probably ngc doesn't work with require() in CompleterListCmp ? https://angular.io/docs/ts/latest/cookbook/aot-compiler.html#!#tree-shaking (notes in Rollup section)

Followup from my last comment, I commented out all require() in all components and use HTML and CSS paths in @Component decorator, ngc produces another error
Error: Error encountered resolving symbol values statically. Expression form not supported (position 10:54 in the original .ts file), resolving symbol CompleterService

which are these 2 lines
@Inject(LocalData) private localDataFactory: () => LocalData,
@Inject(RemoteData) private remoteDataFactory: () => RemoteData

I changed to this
-@Inject(LocalData) private localDataFactory: () => LocalData,
+@Inject(LocalData) private localDataFactory: LocalData,
-@Inject(RemoteData) private remoteDataFactory: () => RemoteData
+@Inject(RemoteData) private remoteDataFactory: RemoteData

Which then the errors from ngc was gone but I'm not sure what I was doing. Totally not familiar with using @Inject() decorator 😅

@armno it's not Inject that caused the issue it's about ngc and  the need for the code to statically analyzed

@oferh any guide on how to transform those 2 lines into something that can be statically analyzed by ngc? for example, use exported functions like what ng2-translate did here https://github.com/ocombe/ng2-translate#usage

related to #60

@oferh there were some errors because at some places templates are using variables which were set as private in ts. so aot cant export that properly. I'll write those variables when i get home.

@oferh
[12:15:53]  Error: Error at
            /Projects/SL/.tmp/node_modules/ng2-completer/src/components/ng2-completer/completer-list-item-cmp.ngfactory.ts:124:40
[12:15:53]  Property 'parts' is private and only accessible within class 'CompleterListItemCmp'.
[12:15:53]  Error at
            /Projects/SL/.tmp/components/auto-completer/completer-cmp-md.ngfactory.ts:216:18
[12:15:53]  Property 'completer' is private and only accessible within class 'CompleterCmpMd'.
[12:15:53]  ngc failed
[12:15:53]  ionic-app-script task: "build"
[12:15:53]  Error: Error

Hi guys, I guess you already know about this issue but just to let you know that I tried installing via jspm and it also fails :
Error: (SystemJS) Unexpected value 'Ng2CompleterModule' imported by the module 'PublicModule'
Error: Unexpected value 'Ng2CompleterModule' imported by the module 'PublicModule'
at eval (https://<IP_ADDRESS>:3000/jspm_packages/npm/@angular/compiler@2.0.2/bundles/compiler.umd.js:13523:25)
at Array.forEach (native)
at CompileMetadataResolver.getNgModuleMetadata (https://<IP_ADDRESS>:3000/jspm_packages/npm/@angular/compiler@2.0.2/bundles/compiler.umd.js:13509:40)
at RuntimeCompiler._compileComponents (https://<IP_ADDRESS>:3000/jspm_packages/npm/@angular/compiler@2.0.2/bundles/compiler.umd.js:15936:47)
at RuntimeCompiler.compileModuleAndComponents (https://<IP_ADDRESS>:3000/jspm_packages/npm/@angular/compiler@2.0.2/bundles/compiler.umd.js:15878:37)
at RuntimeCompiler.compileModuleAsync (https://<IP_ADDRESS>:3000/jspm_packages/npm/@angular/compiler@2.0.2/bundles/compiler.umd.js:15869:21)
at PlatformRef.bootstrapModuleWithZone (https://<IP_ADDRESS>:3000/jspm_packages/npm/@angular/core@2.0.2/bundles/core.umd.js:4898:25)
at PlatformRef.bootstrapModule (https://<IP_ADDRESS>:3000/jspm_packages/npm/@angular/core@2.0.2/bundles/core.umd.js:4881:21)
at execute (https://<IP_ADDRESS>:3000/app/main.ts!transpiled:31:65)
Error loading https://<IP_ADDRESS>:3000/app/main.ts
at eval (https://<IP_ADDRESS>:3000/jspm_packages/npm/@angular/compiler@2.0.2/bundles/compiler.umd.js:13523:25)
at Array.forEach (native)
at CompileMetadataResolver.getNgModuleMetadata (https://<IP_ADDRESS>:3000/jspm_packages/npm/@angular/compiler@2.0.2/bundles/compiler.umd.js:13509:40)
at RuntimeCompiler._compileComponents (https://<IP_ADDRESS>:3000/jspm_packages/npm/@angular/compiler@2.0.2/bundles/compiler.umd.js:15936:47)
at RuntimeCompiler.compileModuleAndComponents (https://<IP_ADDRESS>:3000/jspm_packages/npm/@angular/compiler@2.0.2/bundles/compiler.umd.js:15878:37)
at RuntimeCompiler.compileModuleAsync (https://<IP_ADDRESS>:3000/jspm_packages/npm/@angular/compiler@2.0.2/bundles/compiler.umd.js:15869:21)
at PlatformRef.bootstrapModuleWithZone (https://<IP_ADDRESS>:3000/jspm_packages/npm/@angular/core@2.0.2/bundles/core.umd.js:4898:25)
at PlatformRef.bootstrapModule (https://<IP_ADDRESS>:3000/jspm_packages/npm/@angular/core@2.0.2/bundles/core.umd.js:4881:21)
at execute (https://<IP_ADDRESS>:3000/app/main.ts!transpiled:31:65)
Error loading https://<IP_ADDRESS>:3000/app/main.ts

ngc is supported in 0.3.1

Thank you! I added this into my project yesterday and it appears to be working just as it should. No build issues


Goodbye Dependabot, Hello Batch Dependency Updates

swyxkit depends on alpha software, primarily sveltekit, but also has a dozen more dependencies that move on a fairly frequent basis. The problem of keeping dependencies up to date is a pressing one.
The usual answer to this is Dependabot, which is available as a one click setting inside of GitHub. However, it generates a new PR per dependency update, which usually gets fairly annoying.
Fred Schott from Astro recently tweeted about how they do nightly lockfile updates, which seems like a much smarter solution: https://twitter.com/FredKSchott/status/1489287560387956736
I adapted it to make it weekly, and figured I'd share the process.
Step 1 - Add a new GitHub Action.
Pretty much just create a file like this one, GH actions are so easy to make. I modified the cron syntax to only run once a week to limit the amount of updates going on.
name: 'Nightly'

on:
  schedule:
    # Runs at 12:00 UTC on Fri.
    - cron:  '0 12 * * 5'
  workflow_dispatch:

jobs:
  lockfile:
    if: github.repository_owner == 'sw-yx'
    runs-on: ubuntu-latest
    steps:

      - name: Check out code using Git
        uses: actions/checkout@v2

      - name: Set Node version to 16
        uses: actions/setup-node@v2
        with:
          node-version: 16
          cache: 'npm'

      - name: Clear lockfile
        run: rm -rf package-lock.json node_modules

      - name: Install dependencies
        run: npm install --ignore-engines --ignore-scripts

      - name: Create Pull Request
        id: createpr
        uses: peter-evans/create-pull-request@v3
        with:
          token: ${{ secrets.NIGHTLY_PERSONAL_GITHUB_TOKEN }}
          commit-message: '[ci] update lockfile'
          title: '[ci] update lockfile'
          body: >
            This PR is auto-generated by a nightly GitHub action.
            It should automatically be merged if tests pass.

      - name: Mark Pull Request for Auto-Merge
        if: steps.createpr.outputs.pull-request-operation == 'created'
        uses: peter-evans/enable-pull-request-automerge@v1
        with:
          token: ${{ secrets.NIGHTLY_PERSONAL_GITHUB_TOKEN }}
          pull-request-number: ${{ steps.createpr.outputs.pull-request-number }}
          merge-method: squash

Step 2 - create your token
Notice that the script depends on a NIGHTLY_PERSONAL_GITHUB_TOKEN variable. You can create it here: https://github.com/settings/tokens

and enter it here in your project

Is there an approach that would allow you to require checks and tests to pass before auto-merging? This doesn't bother me as-is for something like a blog, but seems a bit dangerous to add to my company's product.

i think that's a function of you setting up your CI correctly rather than anything to do with this setup - if your checks and tests dont pass, github would refuse to automerge


fix: postgresql model validation errors

Remove empty body to fix RESPONSE_SCHEMA_NOT_IN_SPEC errors

Automation for azure-sdk-for-go

Automation for azure-sdk-for-python

Can one of the admins verify this patch?


I want to participate in this project

I want to be involved in this project, where do I start? Aren't there any good first issues?

Hi @junbeomlee, and welcome!
As noted in the README:

We are developing in the open, but not ready to accept or review significant contributions. Keep checking back!

There are a handful of issues tagged "good first issue", but we don't yet have the documentation or mentorship time to nurture third party contributions yet. Things are moving along and that may change soon.

I understood well. I will look for it and contribute if there is something I can contribute.
And I wonder if the project is hiring full-time employees?

Our job openings are listed here: https://blueskyweb.xyz/join
At the moment none of them are backend/golang positions, but keep an eye out!


Reusing existing Linked Data vocabularies

I’ve noticed the work on a JSON API for GTFS data here: https://github.com/FIWARE/dataModels/blob/master/specs/UrbanMobility/GtfsTrip/doc/spec.md
It links to Linked GTFS, but as far as I can see, the Linked GTFS terms are not mentioned in the @context. Furthermore, a specific way of creating non HTTP identifiers is put forward, contrary to existing mapping scripts such as

https://github.com/OpenTransport/gtfs-csv2rdf,
https://github.com/linkedconnections/gtfs2lc or
https://github.com/oeg-upm/morph-csv/blob/master/evaluation/transport-performance/mappings/gtfs.yml

Does NGSI-LD always need its own NGSI-specific mappings?

yes, we need to Cross Reference Linked GTFS in the @context

regarding ids, please could you put an example of proper ids?

you can use any URI for identifying your Entities in NGSI-LD, so fine.
Existing Linked Open Data vocabularies can be used by just defining the proper mappings in the @context. Our @context generation tool maps terms by default to a FIWARE URI but exceptions can be defined in the following file
https://github.com/FIWARE/dataModels/blob/master/tools/ldcontext_mappings.json
So you can make a PR to define exceptions for GTFS terms and map them to the URIs already defined by the LinkedGTFS project
@pietercolpaert would you be in a position to make such a PR?
Thanks!

@jmcanterafonseca I tried to work on this, but immediately came in conflict with the data model here. Simple example: you use 0, 1 or 2 as values for wheelchair accessibility. Back in 2014 we chose to map this to URIs instead of literal values.
Are these data models still able to change?

sure, they are not written in stone. please propose alternative values for
that and we can fix it.
El mar., 24 sept. 2019 a las 9:26, Pieter Colpaert (<
<EMAIL_ADDRESS>escribió:

@jmcanterafonseca https://github.com/jmcanterafonseca I tried to work
on this, but immediately came in conflict with the data model here. Simple
example: you use 0, 1 or 2 as values for wheelchair accessibility. Back in
2014 we chose to map this to URIs instead of literal values.
Are these data models still able to change?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/FIWARE/dataModels/issues/536?email_source=notifications&email_token=AAFJT7QTCFREZMUTJHU5CG3QLG6I7A5CNFSM4IUHCWK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7NLWYI#issuecomment-534428513,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFJT7T6VREAE5LQTDMUTB3QLG6I7ANCNFSM4IUHCWKQ
.

@jmcanterafonseca So, we had to wrap our heads fully around the spec to understand how we could do that. Sorry it took a while to respond.
We created a small example below. Can you validate this looks like something you would use?
We propose not to use the expanded form because with this example:
{
  "id":"http://example.org/trips/Spain:Malaga:1",
  "type":"Trip",
  "direction":{
    "type":"Property",
    "value":false
  },
  "headsign":{
    "type":"Property",
    "value":"San Andr\u00e9s"
  },
  "hasRoute":{
    "type":"Relationship",
    "object":"http://example.org/routes/Spain:Malaga:1"
  },
  "hasService":{
    "type":"Relationship",
    "object":"http://example.org/services/Malaga_LAB"
  },
  "hasShape":{
    "type":"Relationship",
    "object":"http://example.org/shapes/Shape01"
  },
  "@context":[
    {
      "gtfs":"http://vocab.gtfs.org/terms#",
      "Trip":"gtfs:Trip",
      "direction":"gtfs:direction",
      "headsign":"gtfs:headsign",
      "hasRoute":"gtfs:route",
      "hasService":"gtfs:service",
      "hasShape":"gtfs:shape"
    },
    "https://cors-anywhere.herokuapp.com/https://uri.etsi.org/ngsi-ld/v1/ngsi-ld-core-context.jsonld"
  ]
}

The range of  e.g., gtfs:headsign is not any longer an xsd:string. This break compliance with the vocabulary.
In the simplified form it does work however:
{
  "id":"http://example.org/trips/Spain:Malaga:1",
  "type":"Trip",
  "direction":false,
  "headsign":"San Andr\u00e9s",
  "hasRoute":"http://example.org/routes/Spain:Malaga:1",
  "hasService":"http://example.org/services/Malaga_LAB",
  "hasShape":"http://example.org/shapes/Shape01",
  "@context":[
	{
  	"gtfs":"http://vocab.gtfs.org/terms#",
  	"Trip":"gtfs:Trip",
  	"direction":"gtfs:direction",
  	"headsign":"gtfs:headsign",
  	"hasRoute":"gtfs:route",
  	"hasService":"gtfs:service",
  	"hasShape":"gtfs:shape"
	},
	"https://cors-anywhere.herokuapp.com/https://uri.etsi.org/ngsi-ld/v1/ngsi-ld-core-context.jsonld"
  ]
}

that can work but instead of putting the GTFS mappings inline in the
@context you can add those mappings to
https://github.com/FIWARE/data-models/blob/master/tools/ldcontext_mappings.json
so that when the FIWARE Data Models @context is automatically generated
mappings for GTFS are conveniently added. That way you could still be using
https://schema.lab.fiware.org/ld/context as your main @context
El mar., 15 oct. 2019 a las 18:05, Pieter Colpaert (<
<EMAIL_ADDRESS>escribió:

@jmcanterafonseca https://github.com/jmcanterafonseca So, we had to
wrap our heads fully around the spec to understand how we could do that.
Sorry it took a while to respond.
We created a small example below. Can you validate this looks like
something you would use?
We propose not to use the expanded form because with this example:
{
"id":"http://example.org/trips/Spain:Malaga:1",
"type":"Trip",
"direction":{
"type":"Property",
"value":false
},
"headsign":{
"type":"Property",
"value":"San Andr\u00e9s"
},
"hasRoute":{
"type":"Relationship",
"object":"http://example.org/routes/Spain:Malaga:1"
},
"hasService":{
"type":"Relationship",
"object":"http://example.org/services/Malaga_LAB"
},
"hasShape":{
"type":"Relationship",
"object":"http://example.org/shapes/Shape01"
},
"@context":[
{
"gtfs":"http://vocab.gtfs.org/terms#",
"Trip":"gtfs:Trip",
"direction":"gtfs:direction",
"headsign":"gtfs:headsign",
"hasRoute":"gtfs:route",
"hasService":"gtfs:service",
"hasShape":"gtfs:shape"
},
"https://cors-anywhere.herokuapp.com/https://uri.etsi.org/ngsi-ld/v1/ngsi-ld-core-context.jsonld"
]
}
The range of e.g., gtfs:headsign is not any longer an xsd:string. This
break compliance with the vocabulary.
In the simplified form it does work however:
{
"id":"http://example.org/trips/Spain:Malaga:1",
"type":"Trip",
"direction":false,
"headsign":"San Andr\u00e9s",
"hasRoute":"http://example.org/routes/Spain:Malaga:1",
"hasService":"http://example.org/services/Malaga_LAB",
"hasShape":"http://example.org/shapes/Shape01",
"@context":[
{
"gtfs":"http://vocab.gtfs.org/terms#",
"Trip":"gtfs:Trip",
"direction":"gtfs:direction",
"headsign":"gtfs:headsign",
"hasRoute":"gtfs:route",
"hasService":"gtfs:service",
"hasShape":"gtfs:shape"
},
"https://cors-anywhere.herokuapp.com/https://uri.etsi.org/ngsi-ld/v1/ngsi-ld-core-context.jsonld"
]
}
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/FIWARE/data-models/issues/536?email_source=notifications&email_token=AAFJT7UCYYOYH54F637NXODQOXS3XA5CNFSM4IUHCWK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBJKMZI#issuecomment-542287461,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFJT7T53DEZA6GJLC4V4GTQOXS3XANCNFSM4IUHCWKQ
.

Hi Pieter it's me Alberto Abella.
could you move this issue to here
https://github.com/smart-data-models/dataModel.UrbanMobility
because these repos are frozen since February and evolution would be derived to there


More accurate episode air time from TV maze

Air time on TMDB are not very accurate, for example, for Euphoria S02E05 its
6 February 2022, but on Trakt it's February 7, 2022 2:00 AM on HBO, just like on TV maze.
Just like Trakt air time should be pulled from TV maze

This would be a great enhancement!


Add npm run build:js:watch to watch for js changes

Add a new command which automatically rebuilds lib.js when changes occur
in the JavaScript files.

Not sure if you would be interested, but I find this useful when working on the JavaScript code


Bug: IP_STATUS_FILE and PORT_FORWARDING_STATUS_FILE not updated after restart command on the HTTP control server

TLDR: *IP_STATUS_FILE and PORT_FORWARDING_STATUS_FILE do not get updated after restart command issued through the http control server *

Is this urgent?

[x] Yes
[ ] No

What VPN service provider are you using?

[x] PIA
[ ] Mullvad
[ ] Windscribe
[ ] Surfshark

What's the version of the program?
Running docker image with tag latest

What are you using to run the container?

[ ] Docker run
[x] Docker Compose
[ ] Kubernetes
[ ] Docker stack
[ ] Docker swarm
[ ] Podman
[ ] Other:

Extra information

Configuration file:
version: '2'

services:
  gluetun:
    image: qmcgaw/private-internet-access
    container_name: gluetun
    cap_add:
      - NET_ADMIN
    network_mode: bridge
    ports:
      - 8888:8000/tcp

    environment:
      - VPNSP=private internet access
      - PROTOCOL=udp
      - OPENVPN_VERBOSITY=1
      - OPENVPN_ROOT=no
      - TZ=xxxxxxxxxxx

      # PIA
      - REGION=xxxxxxxxxxx
      - USER=xxxxxxxxxxx
      - PASSWORD=xxxxxxxxxxx
      - PIA_ENCRYPTION=strong
      - PORT_FORWARDING=on
      - PORT_FORWARDING_STATUS_FILE=/forwarded_port

      # Others
      - IP_STATUS_FILE=/ip
      - EXTRA_SUBNETS=<IP_ADDRESS>/24

      - SHADOWSOCKS=off
      - TINYPROXY=off

    volumes:
      - /home/xxxxxxxxxxx/xxxxxxxxxxx/gluetun/ip:/ip
      - /home/xxxxxxxxxxx/xxxxxxxxxxx/gluetun/forwarded_port:/forwarded_port

    restart: always

Host OS: Debian

Indeed, thanks! I'll fix it tonight.

Awesome, thanks :)

It should now redo the initial procedure (except dns over tls) whenever an Initialization sequence completed is encountered, instead of just on the first occurrence. I haven't got the time to test it thoroughly, but it should do the job.
Feel free to comment if it doesn't work. Enjoy!


Update custom-script-windows.md -need to update a FAQ item for Windows

adding FAQ item

@GabstaMSFT

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Vector MBTiles

It's not clear from the documentation whether TileStache supports vector MBTiles.  I'd be grateful for clarification of this.

I also had the same question. I looked into the code and it seems it isn't supported. In MBtiles.py if format of mbtiles isn't png,jpg or json exception is raised. My vector MBtiles have pbf as format. And PR #264 which added support for PBF format seems to only change files for Vector provider not MBtiles.

I looked into the code and tried to add support for vector MBtiles since it seems it should be easy. Since only the response type is different. PBF instead of PNG/JPG/json. I get correct PBF which I can read with protocol buffer but when trying to load it with mapbox-gl I can't see anything on a map even though there are no HTTP or JS errors. Only sometimes there is python error when writing to response socket. But this can be seen in browser as error 500.
Maybe the problem is too slow response. Mapbox response times are around 360 ms. My local tilestache is 1-3 seconds.

Hi @buma , did you merge your work on vector mbtiles back to the master branch? It should be great for you to do that.

Hi @buma ,
we are also stuck with serving mbtiles as pbf on mapbox.
Tested with your changes and it is working fine!
So It would be really great help to community if you merge your changes to master.

I looked into the master and it seems that this commit: https://github.com/TileStache/TileStache/commit/91b23656 added support for PBF MBtiles. It has same changes as my branches except adding GZIP header on PBF responses. Are you sure that master isn't working with PBF tiles?

Yes, we want those GZIP header changes also. Can you please merge them also?

@buma Why have you added headers.add_header('Content-Encoding', 'gzip') for MBTiles? because it works while serving mbtiles but doesn't work while serving vector tiles from postgis.

I haven't been able to get this to work. What I have are .mbtile files with vector/pbf data. Here is the select * from metadata output, truncated...
sqlite3 2017-07-03_asia_iraq.mbtiles 'select * from metadata'
attribution|© OpenMapTiles © OpenStreetMap contributors
center|43.842805,33.22115,14
description|Extract from https://openmaptiles.org
maxzoom|14
minzoom|0
name|OpenMapTiles
pixel_scale|256
mtime|1499626373833
format|pbf
id|openmaptiles
version|3.6.1
maskLevel|5
bounds|38.784,29.05167,48.90161,37.39063
planettime|1499040000000
json|{"vector_layers":[{"maxzoom":14,"fields":{"class":"String"},"minzoom":0,"id":"water","description":""},...
Any clue how I can go from something like this to something that will be viewable in TileStache?


feature request: add time picker to library

It's would be great if your library has time picker.

@SaeedZhiany You can create a time picker with Modal easily.


Fix a typo in README

@@ -197,7 +197,7 @@ spec:
       image: gcr.io/google-samples/hello-app:1.0
       ports:
         - containerPort: 8080
-      livenessProbe:
+      readinessProbe:
         httpGet:
           path: /ready
           port: 8080

Oh, Thanks for reporting.

You are welcome ;-) ... and thank You for sharing of great knowledge

Now there are two typos ;-)

Wow, revised!


Cannot find module './config'

You need to rename config.default.js to config.js, located at mongo-express root folder.

@miguelbaldi  is right - try this command
cp config.default.js config.js


Ability to init with raw command instead of flags

I have a monorepo where I use ts project references. I want to start the tsc from the nvim cwd (repo root dir), by executing "tsc --build /home/user/my-repo/tsconfig.json --watch".
I investigated the code, and I could not figure out a way to do this (specifically to define the path of tsconfig).
Project references build command cannot be invoked with --project.
I tried to trick it with bin_path option, but you check whether is executable.
Via the flags you prepend double-dash.
Maybe you can provide a way to directly specify the command instead of the various options-flags?
I think this may be the only way, because project references --build and rest args, should be placed in a specific order. (--build first, then tsconfig, then rest args).
Thanks!
Nice plugin btw :)

If the plugin respected the order of the flags in the config would that resolve this?

I do not think so because you prepend -- at flags and in the command for ts project references, the arg for project tsconfig has none.
I find it difficult to abstract away flags-options and args to cover all cases. That's why I proposed a raw command

Is the main issue that you want to run the TSC usrcmd and specify the path to the dir to use?
If so would adding an option to specify a dir when running the TSC command work for you? Just thinking providing an option to hard code it in the package config would mean editing your nvim config whenever you change what you are working on.
The other option would maybe be the ability to provide a function which resolves to a string which can be run to set the location of the config when TSC is called?

Yes I think this will resolve it!
This will contradict the project flag.
Setting project = false so no default flag is used, will be needed.


Help with pushing userId to datalayer

I am using redux-beacon with react-redux and GTM. Since GA has support for getting logged in user ID from datalayer variable, is there a way to push the userId to datalayer on login and keep in the datalayer till the user logs out?

Hi @vivekdwivedi.

is there a way to push the userId to datalayer on login and keep in the datalayer till the user logs out.

Yes. You would need a Redux action to fire whenever the user logs in (e.g. USER_LOGGED_IN).
Then you would create an event definition and map it to that event like so:
// assumes action.payload contains the userId
const userLogIn = action => ({
  event: 'authentication',
  userId: action.payload,
});

const eventsMap = {
  ...
  USER_LOGGED_IN: userLogIn,
  ...
};

This will push the userId to the dataLayer when the user logs in. The userId will remain there in the dataLayer until you change its value to something else. So if you want to have a generic userId for users that aren't logged in, you could fire another event to replace the userId.
const userLogOut = () => ({
  event: 'authentication',
  userId: 'unregistered-user',
});

const eventsMap = {
  ...
  USER_LOGGED_OUT: userLogOut,
  ...
};


parser does not parse EventID 3

when doing a log search with the following data show up
Event
| where Source contains "sysmon"
| where EventID == 3

When using the parser no data shows up
Sysmon | where EventID == 3

when using a different EventID (22) data is being parsed.

fyi, got MS on it, there is a similar issue with event id 7 and the hashes here
https://github.com/Azure/Azure-Sentinel/issues/387


Support arrow keys for vscroll

This PR adds support for using the up/down arrow keys to scroll the viewport (and shift + arrow keys for "fine" scroll).
The current values for row_scroll_delta were chosen fairly arbitrarily, any input re: actual user preferences is welcome.
fixes: #33
cc @elliottslaughter @lightsighter

I'll defer to Elliott on the code. Happy to have scrolling in all directions. 😁

When we have multiple windows, what's the expected behavior? Right now it appears to only vertically scroll the first window.

When we have multiple windows, what's the expected behavior? Right now it appears to only vertically scroll the first window.

I'm not actually sure how to run the profiler with multiple windows, are there instructions/docs somewhere?

If you use the --serve mode, you can create two (or more) servers and then use --attach with two URLs. See:
https://legion.stanford.edu/profiling/#remotely-viewing-profiles-serve-and-attach-modes

When we have multiple windows, what's the expected behavior? Right now it appears to only vertically scroll the first window.

I tested this with --attach and currently the vertical scroll affects each window independently (each window can be vertically scrolled, and vertically scrolling a window does not affect any other windows).
Without knowing anything else, this is the behavior I would assume is correct. These are intended to be independent views of the same underlying data, correct? There is no guarantee the windows are even the same shape or size, and, AFAIK even in a "shared google doc" type scenario, my scrolling a window does not scroll other user's windows.

Ok, I tried this out and it does work. I'm confused, but I guess there's some magic in egui that tracks the cursor position and sends the scroll event to the right place.


Audio device selector: iOS review scenarios

With BT device connected before join meeting:

?

With BT device disconnected before join meeting, and enabling it inside meeting:

?

With BT device connected in meeting, and disables it BT option:

?

With BT device connected before join meeting:

Routes automatically to BT device ✅
Shows the BT device when open modal for the first time ✅
Device selector works perfectly in this case ✅

With BT device disconnected before join meeting, and enabling it inside meeting:

Routes automatically to BT device ✅
Does not show the BT device when open modal for the first time ❌
The second time it shows up, and when try to change to Speaker, it routes perfectly ✅
Device selector works perfectly in this case ✅

With BT device connected in meeting, and disables it BT option:

If connected to speaker: routes automatically to SPEAKER ✅
If connected to BT devices: routes automatically to SPEAKER ✅
Shows the BT device when open modal for the first time, even if the BT is disabled ❌
Second time flush the devices, and only shows the SPEAKER ✅


Experiment w/ using Lerna publish if available

https://github.com/lerna/lerna

Log warning if version of Lerna doesn't match what we test with


libroadrunner forward simulation is different than pyciemss

libroadrunner forward simulation is different than pyciemss

pyciemss

libroadrunner

related https://github.com/pnnl-predictive-phenomics/concerto/issues/7

issue was resolved after setting n_samples=1 for pyciemss.


AutoHeight feature not updating on slide change

I've attempted using this awesome new feature. However, I can't get it working all the way.
The active slide is giving the wrapper and all slides the same height but it's not updating when I go to the next slide.

It seems this was a css issue, sorry about that, closing now.


Builds failing due to issue with "rand" crate

See Travis: https://travis-ci.org/dalek-cryptography/x25519-dalek/jobs/482814498
It appears as though the rand crate is causing a build failure using Rust nightly.

This should be fixed with 0.6.


Brothers page django tests

Starting to build the tests
Known problems:
Need fixture data with filled out userinfo for the test to be meaningful. Base fixture data ('dev_data') does not have userinfo by default. Can't have multiple tests using different fixture data either to my knowledge, or if they do this fixture data has to be carefully arranged to not conflict with each other.

Also looks like it hasn't been updated for a bit, so has a conflict with master


Make calculation of integrals via ECPIntegral thread-safe

This PR essentially marks the functions compute_shell_pair and related functions for derivatives as const. As a result, no members of the involved classes change during these function calls thus making these functions thread-safe.
This enables codes using libecpint to instantiate ECPIntegral only once and use it in all threads, whereas before one instance per thread was required.
Quite a number of changes were required to achieve the above. A small number of these changes may in principle affect performance, so it should be tested that performance is not degraded by the contents of this PR.

Need to add
#include <cassert>

in gaussquad.cpp to fix build failure. You can either do this locally and update this pull request (preferred to keep merges granular), or I can include it in my next update - currently working on issue #21 has delayed things slightly

Need to add
#include <cassert>

in gaussquad.cpp to fix build failure. You can either do this locally and update this pull request (preferred to keep merges granular), or I can include it in my next update - currently working on issue #21 has delayed things slightly

Done. Sorry about that.


SceneObjectBase: Call self activation handlers before child data, time range and variable handlers

I noticed a bug in scenes dashboards with AnnotationDataLayer not working with data source variables, resulting in error AnnotationsDataLayer.js:95 AnnotationsDataLayer error {message: 'Datasource ${ds} was not found'}
I tracked it down to an issue with scene $data, $timeRange and $variables being activated before the local (own) level activation handlers. Resulting in data layers on the top dashboard level being activated before the dashboard scene itself (that sets the window.__grafanaSceneContext without this the variable system is not working (because scopedVars with sceneObject was not passed in AnnotationsDataLayer)
Changes

Call own local level activation handlers before calling $data, $timeRange, $variables and $behaviors activation handlers
Pass scopedVars with context when getting data source in AnnotationsDataLayer

Release notes
Activation handlers are for a scene object is now called before any direct child activation handlers. Before this release the activation handlers of direction $data, $timeRange, $variables and $behaviors was called before the SceneObjects own activation handlers.

@dprokop had not pushed it :)


Try adding Snyk CI support for vulnerability detection

Issue Summary
Ghost currently has a number of node package vulnerabilities, I only noticed because I run Ghost in a Docker container and my container scanner noticed them.
I've opened a few PRs to fix some of the vulnerabilities:
https://github.com/TryGhost/gscan/pull/21
https://github.com/TryGhost/gscan/pull/20
There are services like Snyk that can provide CI for security vulnerabilities in packages, making it easy to update them.
Snyk seems the best one to do this, as it checks dependancies down the chain for vulnerabilities, whereas https://nodesecurity.io/ only checks the dependancies listed in the package.json, when they're normally two or three packages downstream.
It's free for open-source projects 👍
Example
https://snyk.io/test/npm/ghost/0.10.0
Snyk can also be run as a command line job, so could be added to .travis.yml if full API access is problematic.

@petems Thanks for the note & for the PRs to gscan 👍  We already have several different services like this enabled including Snyk and bithound, sending us regular notifications. When it matters, we take notice & act accordingly. The vulnerabilities deep in the dependencies of gscan were not exposed to Ghost - only to the gscan website.
We take security extremely seriously. I have commit access to many of our dependencies just so I am able to do these sorts of updates when necessary 😉


MSID: 90046 Version: 1 DOI: 2<IP_ADDRESS>9563

"msas": "Neuroscience"
"msid": "90046"
"preprintDoi": "10.1101/2<IP_ADDRESS>9563"
"Reviewed Preprint posted": "2023-10-27"
"Sent for peer review": "2023-06-30"
"Posted to bioRxiv": "2023-07-19" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>9563v1/")
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=90046)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/90046/v1)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[x] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>9563)
[x] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[ ] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/90046/v1/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/90046)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/90046

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/90046

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/90046/v1/90046-v1.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/90046)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

elocations missing from the references.


fix: drupal10.1 now complains about trusted_hosts_patterns even with general regex

The Issue
Drupal 10.1 is whining about the trusted_host_patterns even though we have always had one.
How This PR Solves The Issue
Add more to the patterns.
Manual Testing Instructions
Automated Testing Overview

Related Issue Link(s)
Release/Deployment Notes

I'm not seeing a warning on Status page with Drupal 10.1.0 & DDEV 1.22.1.
It seems happy with the settings:

If I use 4db441a, the site has more "trusted hosts" but the same number of warnings & errors.

Thanks for checking this so carefully @tyler36 - I'm sure I had the warnings in 10.1.1. But I definitely don't have them in 10.1.2. I'll close this and move on for now.


button submits form

Timepicker sends form if I push button pm/am. attribute type="button" should be added to all <button ... >... tags

Closing due to inactivity. No extra info provided.


Simplified Concat Operation Produces Incorrect Numerics

What happened?
In this gist, there are two very similar linalg IR reproducers. Compiling and running on CPU, they unexpectedly generate mismatching results.
The only difference between these two IR is the very last sequence:
    %169 = linalg.generic {indexing_maps = [#map10, #map19, #map4], iterator_types = ["parallel", "parallel", "parallel"]} ins(%cst_13, %168 : tensor<128xf32>, tensor<1x1x128xf32>) outs(%166 : tensor<1x1x128xf32>) {
    ^bb0(%in: f32, %in_87: f32, %out: f32):
      %171 = arith.addf %in, %in_87 : f32
      linalg.yield %171 : f32
    } -> tensor<1x1x128xf32>
    %cast_83 = tensor.cast %169 : tensor<1x1x128xf32> to tensor<?x1x128xf32>
    %collapsed_84 = tensor.collapse_shape %161 [[0], [1, 2], [3, 4]] : tensor<1x64x2x14x14xf32> into tensor<1x128x196xf32>
    %170 = tensor.empty() : tensor<1x196x128xf32>
    %transposed_85 = linalg.transpose ins(%collapsed_84 : tensor<1x128x196xf32>) outs(%170 : tensor<1x196x128xf32>) permutation = [0, 2, 1]
    %concat_86 = tensor.concat dim(1) %cast_83, %transposed_85 : (tensor<?x1x128xf32>, tensor<1x196x128xf32>) -> tensor<1x197x128xf32>
    return %concat_86 : tensor<1x197x128xf32>
  }
}

Notice the cast operation %cast_83, somewhat stupidly makes the first dim dynamic before passing to %concat_86. However, this IR produces correct numerics.
Alternatively, the simplified version below will give wildly incorrect numerics:
    %169 = linalg.generic {indexing_maps = [#map10, #map19, #map4], iterator_types = ["parallel", "parallel", "parallel"]} ins(%cst_13, %168 : tensor<128xf32>, tensor<1x1x128xf32>) outs(%166 : tensor<1x1x128xf32>) {
    ^bb0(%in: f32, %in_87: f32, %out: f32):
      %171 = arith.addf %in, %in_87 : f32
      linalg.yield %171 : f32
    } -> tensor<1x1x128xf32>
    %collapsed_84 = tensor.collapse_shape %161 [[0], [1, 2], [3, 4]] : tensor<1x64x2x14x14xf32> into tensor<1x128x196xf32>
    %170 = tensor.empty() : tensor<1x196x128xf32>
    %transposed_85 = linalg.transpose ins(%collapsed_84 : tensor<1x128x196xf32>) outs(%170 : tensor<1x196x128xf32>) permutation = [0, 2, 1]
    %concat_86 = tensor.concat dim(1) %169, %transposed_85 : (tensor<1x1x128xf32>, tensor<1x196x128xf32>) -> tensor<1x197x128xf32>
    return %concat_86 : tensor<1x197x128xf32>
  }
}

Steps to reproduce your issue

Download the two reproducers from this gist.
run iree-compile --iree-hal-target-backends=llvm-cpu correct_numerics.mlir -o correct_numerics.vmfb
run iree-compile --iree-hal-target-backends=llvm-cpu wrong_numerics.mlir -o wrong_numerics.vmfb
generate inputs with something like:

import numpy
import struct

rng = numpy.random.default_rng(19)
a = rng.random((1,3,224,224)).astype(numpy.float32)
with open("input.0.bin", "wb") as f:
    mylist = a.flatten().tolist()
    bytearr = struct.pack("%sf" % len(mylist), *mylist)
    f.write(bytearr)
    f.close()

run iree-run-module --module="correct_numerics.vmfb"<EMAIL_ADDRESS>run iree-run-module --module="wrong_numerics.vmfb"<EMAIL_ADDRESS>compare the first few values of the output.

What component(s) does this issue relate to?
Compiler, Runtime
Version information
local build at commit ae6e5d323ecc63e421c79768087f63dc42490cd2
Additional context
Reproducer generated from the model pit_ti_224 in SharkTestSuite.
Comparing with the output of the add node %169, it is clear that there is some issue with the compilation of the simplified IR in wrong_numerics.mlir.
I tried to get a smaller reproducer for this issue but was unsuccessful. E.g.,
module {
  ml_program.global private mutable @global_seed(dense<0> : tensor<i64>) : tensor<i64>
  func.func @torch_jit(%arg0: tensor<1x3x2xf32>, %arg1: tensor<1x1x2xf32>) -> tensor<1x4x2xf32> {
    %cast_83 = tensor.cast %arg1 : tensor<1x1x2xf32> to tensor<?x1x2xf32>
    %concat_86 = tensor.concat dim(1) %cast_83, %arg0 : (tensor<?x1x2xf32>, tensor<1x3x2xf32>) -> tensor<1x4x2xf32>
    return %concat_86 : tensor<1x4x2xf32>
  }
}

vs. removing the cast operation does not reproduce this issue.
I have not tried reproducing this issue on other backends/devices.

Adding this to compiler bugs until proven otherwise.

The fact that a smaller repro didnt cause an issue seeems like could be a stream allocation issue, i will start with looking there, also @zjgarvey how do you know which of these two numerics are incorrect? with the inpout generator (thanks for providing that) both outputs are just random numbers..

I was originally using the test suite and comparing with onnxruntime CPU result.
You can verify that the simplified IR is incorrect from IREE alone:

Modify either IR to instead return the result of node %169 (the add node). Call this "add_result.mlir".
Compile "add_result.mlir" to "add_result.vmfb".
Run "add_result.vmfb" on the same input "input.0.bin" and inspect the outputs.

The first few values of this add result should match the first few values of the correct concat operation. By inspection, the unsimplified IR result matches the add result, whereas the simplified IR generates an output with completely different values.

This issue simply comes down to the fact that in the simplified IR we are able to fuse the add into the linalg.batch_mmt4d op which we couldnt fuse in the unsimplified case, and the below dispatch is just giving wrong numerics
      #map11 = affine_map<(d0, d1, d2, d3, d4) -> (d0, d1, d2, d3, d4)>
      func.func @torch_jit_dispatch_56_batch_mmt4d_1x1x32x64x1x4x1_f32(%4: tensor<1x1x64x1x1xf32>, %5: tensor<1x32x64x4x1xf32>, %6: tensor<1x1x32x1x4xf32>) -> tensor<1x1x32x1x4xf32>{
        %c0 = arith.constant 0 : index
        %cst = arith.constant 0.000000e+00 : f32
        %7 = tensor.empty() : tensor<1x1x32x1x4xf32>
        %8 = linalg.fill ins(%cst : f32) outs(%7 : tensor<1x1x32x1x4xf32>) -> tensor<1x1x32x1x4xf32>
        %9 = linalg.batch_mmt4d ins(%4, %5 : tensor<1x1x64x1x1xf32>, tensor<1x32x64x4x1xf32>) outs(%8 : tensor<1x1x32x1x4xf32>) -> tensor<1x1x32x1x4xf32>
        %10 = linalg.generic {indexing_maps = [#map11, #map11, #map11], iterator_types = ["parallel", "parallel", "parallel", "parallel", "parallel"]} ins(%6, %9 : tensor<1x1x32x1x4xf32>, tensor<1x1x32x1x4xf32>) outs(%7 : tensor<1x1x32x1x4xf32>) {
        ^bb0(%in: f32, %in_0: f32, %out: f32):
          %11 = arith.addf %in, %in_0 : f32
          linalg.yield %11 : f32
        } -> tensor<1x1x32x1x4xf32>
        return %10 : tensor<1x1x32x1x4xf32>
      }

Without the elementwise it gives the correct numerics. Investigating why that would be now.


Rewrite _break_dm_spin_symm

Rewrite _break_dm_spin_symm to form a doublet state density instead of zeroing out interatomic blocks of density matrix.
Closes #1839

Changing the initial guess may break some existing programs. It is better to have a parameter to control whether to use the new initial guess. The default one can be the new initial guess.


Fix repeatedly invoke build_detection_model error.

Bug
When I repeatedly invoke build_detection_model function to build the model, I will get the error: AttributeError: 'ResNet' object has no attribute 'layer1'.
To Reproduce
model = build_detection_model(cfg)
model = build_detection_model(cfg)

The reason
The variable ResNet50StagesTo4 is a global generator expression, so when we build the model secondly, the code executes to Line 82 for stage_spec in stage_specs: and stage_specs will return empty leading to do not add any stage. Finally, self._freeze_backbone try to freeze the backbone by executing m = getattr(self, "layer" + str(stage_index)) firstly. At the moment, it will throw the AttributeError AttributeError: 'ResNet' object has no attribute 'layer1'.
I guess you want to define ResNet50StagesTo4 as the tuple, so I try to fix by add tuple type qualifier.
The solution
Add the tuple type to ResNet50StagesTo5, ResNet50StagesTo4, ResNet50FPNStagesTo5, ResNet101FPNStagesTo5.
I do not know whether there are similar bugs existing, so you need to review my solution. Thank you!

Thanks for the answer. Replacing resnet.py code to this worked for mw
ResNet50StagesTo5 = tuple(
StageSpec(index=i, block_count=c, return_features=r)
for (i, c, r) in ((1, 3, False), (2, 4, False), (3, 6, False), (4, 3, True))
)
ResNet-50 up to stage 4 (excludes stage 5)
ResNet50StagesTo4 = tuple(
StageSpec(index=i, block_count=c, return_features=r)
for (i, c, r) in ((1, 3, False), (2, 4, False), (3, 6, True))
)
ResNet-50-FPN (including all stages)
ResNet50FPNStagesTo5 = tuple(
StageSpec(index=i, block_count=c, return_features=r)
for (i, c, r) in ((1, 3, True), (2, 4, True), (3, 6, True), (4, 3, True))
)
ResNet-101-FPN (including all stages)
ResNet101FPNStagesTo5 = tuple(
StageSpec(index=i, block_count=c, return_features=r)
for (i, c, r) in ((1, 3, True), (2, 4, True), (3, 23, True), (4, 3, True))
)