how to return nothing?

in normal publications, we can this.ready() to prevent query if condition is not satisfied. how to do this in publish-composite?

return an empty cursor, or simply return; from the find section.

thanks


Feature Request: Implement a trolling feature

maybe this will be added in the extended version of nextron coming soon


Updating to angular 17 fails with error: No matching version found for<EMAIL_ADDRESS>

Command
update
Is this a regression?

[x] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
16.2.12
Description
When upgrading our project to angular 17, we get a lot of incompatible peer dependency errors. Even when using --force or --legacy-peer-deps options. The final error message is always:
notarget No matching version found for<EMAIL_ADDRESS>
See the terminal output and the npm log file as attached.
2023-12-06T14_39_04_577Z-debug-0.log
terminal-output.txt
Minimal Reproduction
ng update --force @angular/core@17 @angular/cli@17
Exception or Error
See the attached files

Your Environment
Angular CLI: 16.2.10
Node: 18.19.0
Package Manager: npm 10.2.3
OS: darwin x64

Angular: 16.2.12
... animations, cdk, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1602.10
@angular-devkit/build-angular   16.2.10
@angular-devkit/core            16.2.10
@angular-devkit/schematics      16.2.10
@angular/cli                    16.2.10
@schematics/angular             16.2.10
rxjs                            7.8.1
typescript                      4.9.5
zone.js                         0.13.3

Anything else relevant?
This happens when trying to update our project from angular 16 to angular 17 in VSCode

node-release version 2.0.14 is available on the public NPM repository, see; https://www.npmjs.com/package/node-releases
if you are using a private NPM repository or proxy please make sure that this is syncing correctly.
Closing as this is not an issue with Angular CLI.


Update Commands/SyncCommunity.php - L5.1 + Slack

In gitlab by @A-Lawrence on Dec 30, 2015, 21:56
This will require some factoring also, especially to extract various useful methods.

Try/Catch + Slack
Success messages

Additionally, this should be on hold until the new release of IPB as we can utilise the API.  This avoids any direct database or code involvement of the forum (and we could separate those systems happily).

In gitlab by @A-Lawrence on Dec 30, 2015, 22:00
mentioned in commit 3f167713762e2d71b25594a6f706e18b15cf041f

In gitlab by @A-Lawrence on Jan 4, 2016, 16:47
Removed ~98475 label

In gitlab by @A-Lawrence on Jan 11, 2016, 21:25
Milestone removed

In gitlab by @A-Lawrence on Feb 1, 2016, 22:00
This IPBoard API now exists, although it's scope is limited it does offer the functionality to update basic membership info (just not groups).
Bumped up to 2.2.4 so that it can either be implemented partially, or we hope that they release the next version.


How to read content from attachments

I am not able to read content from attachment. when I checked in debug mode all attachment content showing in message.Body only.
Thanks

Hi,
You need to use the RawMessage property of the Message object, and parse
the Nine Attachement.
Regards!
On Fri, Oct 5, 2018 at 8:04 AM CodeLearner21<EMAIL_ADDRESS>wrote:

I am not able to read content from attachment. when I checked in debug
mode all attachment content showing in message.Body only.
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rfinochi/pop3dotnet/issues/22, or mute the thread
https://github.com/notifications/unsubscribe-auth/AB12Bm2uFgks98U1inFxf3bMGdKmEcQ4ks5uhzy3gaJpZM4XKA4l
.


Planning and architecture

As an initial suggestion, I think we should consider using the wiki to keep track of canonical information as an alternative we could use readme files in the appropriate folder, i.e. project should be described in the root readme, data products in a sub folder. As for new data products I think we should add an issue and a folder in the project to contain the specific timeline. I am sure there are other things to add to this.

In thinking about this more today, I think that keeping track of problem statements, input data, general processing procedures, and output data and formats in a readme in the folder containing the data pipeline may work best.


Implement a block for generating a number within a range

Issue # #99 is preventing this issue from being solved.


Add a Gitter chat badge to README.md

nnattawat/flip now has a Chat Room on Gitter
@JemarJones has just created a chat room. You can visit it here: https://gitter.im/nnattawat/flip.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Oops, was just checking this out didn't mean to create this.


Skill simulator

Do you want to request a feature or report a bug?
feature
What is the expected behavior?
Add skill simulator
Hi @jigsawye ,
A proposal to add skill simulator, and it may be useful for other features in the future to build like damage calculation etc..., how do you think about it?
As below is a simple prototype in the fork repo, if you're interested about it, please have a look

git compare

Hi @ysaaron,
Thank you for the proposal. It looks nice. Looking forward to seeing you finish the prototype.
Please note that:

I didn’t add tests to this project. You have to test all the existed features manually.
You need to make sure that the save/load features are not broken. All the saved
data should be fallback to the default skill data.

You can create a pull request and we will have a further discussion.


Only tests from eth0 interface when two interfaces are being used.

I am using Kali linux and SSH over LAN on a static ip (with no internet on the interface) And using a wifi adapter to connect and speedtest different AP's.
It gives me the following error:
root@kali:~# ./speedtest-cli
Retrieving speedtest.net configuration...
Could not retrieve speedtest.net configuration
If I forward internet to my ethernet interface then it works fine example below but only tests eth0 connection regardless of which AP i am using with wifi interface.
./speedtest-cli
Retrieving speedtest.net configuration...
Retrieving speedtest.net server list...
Testing from TPG Internet (xxxxxxxxxxxxx)...
Selecting best server based on latency...
Hosted by UberGlobal (xxxxxxxx) [7.10 km]: 129.057 ms
Testing download speed........................................
Download: 4.09 Mbit/s
Testing upload speed..................................................
Upload: 0.79 Mbit/s
Why is it unable to speedtest when I turn off internet on eth0 interface and connect to wifi? I dont have a monitor and mouse attached but will test the script on wlan interface without SSH and post my findings. Maybe it is an issue on my side.
UPDATE: With only the wlan interface connected and directly sitting infront of terminal I was able to speedtest successfully on wlan interface.
But the issue remains, It wont work when two interfaces are connected or will give preference to the eth0 interface
thanks

It sounds like you have a OS configuration issue in that your default gateway is incorrectly set when using both interfaces.
However you can use the --source argument to get it to use the proper interface. That option takes an IP address and should be the IP of the interface you want to use when testing.

For others who had this issue, I found the fix after dozens of hours of trial and error.  You have to set the default gateway to one interface or the other, using "sudo route add default gw x.x.x.x".  Speedtest-cli will not be able to use --source to set the interface unless that interface is the default gateway.


chore(localize): updated translation from Crowdin

:tada: This PR is included in version 2.94.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add support for GenericViews in ContainerRowSerde

Description
We recently added support in using the simple function interface for Aggregate functions . This makes it much easier and nicer to write aggregates.
This also allows us to support Generic View types that makes writing these aggregates much simpler as udf authors can now just concentrate on the logic of their function. However one missing piece here is that certain aggregates need to support a way to persist these GenericViews.
For e.g Consider the case in #9498. Here because SingleValueAccumulator which relies on ContainerRowSerde and which only provides an interface which takes a decoded vector and index , the author has had to expose the decodedvector and index from the Genericwriter, which sort of defeats the purpose of the writer.
Hence we should support the GenericView interface in ContainerRowSerde, which doesnt seem like a huge change, as serializeOne logic can be modified to take GenericView (as you can get TypePtr from it and cast it to a concrete view).

cc: @mbasmanova @kagamiori @Real-Chen-Happy

I am interested in working on this problem. May I get assigned? I am on vacation right now so there might be some delay. But I will get back to it once I returned. Thanks!

@kgpai @kagamiori GenericView serialization logic seems trivial for ContainerRowSerde, but deserialization is not straightforward. Currently I try to deserialize it back to GenericView so that GenericWriter can take as an input. However, doing so may require constructing a new GenericView during deserialization which seems too complex. The second approach is that we just pass in GenericWriter in deserialization, but then deserilization will need to access the underline vectors in the GenericWriter, which seems not good either. Do you have any recommended solution? Thanks!

Thanks @Real-Chen-Happy for getting back to this , please give me a few days to refresh myself with the problem and respond.

Hi @Real-Chen-Happy cant we use the VectorReaders interface (https://github.com/facebookincubator/velox/blob/main/velox/expression/VectorReaders.h#L637). It requires a DecodedVector, so some work might be required there to deal with ContainerRowSerde . Is that your concern ? Feel free to reach out so I can better understand the problem. Thanks !

Hi @kgpai , sorry for taking a long time to reply, as I have been busy wrapping up a couple of deadlines at my work.
I think my concern is that constructing GenericView requires a decoded vector. But creating a decoded vector seems to be non trivial. Here is the idea I came up with:
We will create a new read function in SingleValueAccumulator, which returns a GenericView.
exec::GenericView read(velox::memory::MemoryPool* pool, TypePtr type) const;

pool: we can get pool from HashStringAllocator, which is available in AccumulatorType
type we can get type from GenericWriter which is available in writeIntermediateResult or writeFinalResult

We can create BaseVector in this new read function. BaseVector constructor requires the following args

pool: we can get pool from read function args
type we can get type from  read function args
encoding will be flat
BufferPtr  will be nullptr
length will be 1
Other args will be default

Then, we can pass this BaseVector in ContainerRowSerde for deserialize.
And then, we will create a DecodedVector based on this BaseVector.
Then we can create VectorReader using this DecodedVector.
Then we can get GenericView using VectorReader operator[]
Does this sound good to you? Thanks!

@Real-Chen-Happy I am looking into this - will update you soon.


Deal with sidecar data in Storage

As part of the new Storage design we'd like to be able to handle sidecar data stored outside of consensus. The use case for this is for storing indices of existing JMT keys.
In #1491 we added the following sidecar methods:
fn get_sidecar(&self, key: &[u8]) -> Result<Option<Vec<u8>>>;
fn put_sidecar(&mut self, key: Vec<u8>, value: Vec<u8>);

Since these use byte keys instead of string keys, we can now build indexed binary data (e.g., sorting by an integer by encoding its big-endian bytes into the key). This should be implemented ad-hoc in individual components when we decide we need it.
For this issue, we aim to support range queries in the sidecar. We'll need to build some range-querying functions that interleave data from the overlay and the underlying state.

In State: fn range_sidecar(&self, range_start: &[u8], range_end: &[u8]) ->  Pin<Box<dyn Stream<Item = Result<(Vec<u8>), Self::Error>> + Send + '_>>
This will return a futures::Stream of Vec<u8> items representing every value retrieved from the sidecar storage for keys ranging between range_start and range_end
The unsaved snapshot data should be queried first, falling back to the stored Snapshot data

We currently don't have any components that need to do out-of-consensus (private) indexing, so I think we could punt on filling in the range support for sidecar data until after doing #1532 and replacing the current storage implementation.
Then, we can adapt the API we come up with for the JMT range queries to this use case.

The unsaved snapshot data should be queried first, falling back to the stored Snapshot data

Similar to #1532, we'll need to do interleaving here, I think.

I agree with discussion on Discord that nonconsensus is the most clear name here.

Can we use the RocksDB prefix operations here as in #1532?

As in #1532 see here for more details, I think we should be using (synchronous) Iterators for this interface, not Stream's. We should wrap things in a tokio::spawn_blocking at a layer up beyond this, because using Iterators here allows for sync-to-sync composition with the JMT interface.


"Phoenix catalog" where is it ?

How do I know if I have a support offering that prevents me from purchasing services from the Phoenix catalog?

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

ID: ffca0328-d220-e7bd-9fa0-43b40ab0abcb
Version Independent ID: 554dbcd4-9274-87d2-dd5e-303cd3daa0f3
Content: Microsoft Security Code Analysis documentation FAQ
Content Source: articles/security/develop/security-code-analysis-faq.md
Service: security
GitHub Login: @vharindra
Microsoft Alias: terrylan

@RobertoBorges Thanks for your comment. We are actively investigating and will get back to you shortly.

@RobertoBorges  Kindly contact your Technical Account Managers or you can create a Billing Ticket for the same.


Hello, I reported an error after training. The loss value has always been 0.

 Please double check about the libpng warning.


wav file not include in eval_segment.csv

hi, there is the problem i have met.After unzipping the eval_segments.zip file, i found many wav files are not included in eval_segments.csv. Like YzzbTaK7CZJY.wav. So i do not see the labels of these wavs. Could you provide the csvs you used when training. Thanks!

@zdj97 I have the same problem. How did you solve it? Thanks!

@daisukelab  Thanks! :)

@zdj97 I have the same problem. How did you solve it? Thanks!
I found what caused this. The downloaded wav names like this Y-xxxx.wav, while the eval_segment.csv has wav names like this xxxx.wav. So, drop the Y ahead is Okay.


Connect problem Giiker i3s ?

I cant connect my giiker, i have the new version: http://zcube.hk/Giiker-i3s-Supercube
I can find the cube with my pc bluetooth but it's required a pin never known
If i just use the connect button on your web app (i follow ur instruction for the browser stuff), he just dont see it
( I have no "connecting problem" with the official app, but it sucks :D )
Maybe need an upgrade for i3s ?

My cube name is GiS54511.

On your PC, have you tries 0000 as a PIN? Just curious, as many bluetooth devices use that by default.

Yeah i tried 0000 1234 1111 everything :(

I have also an is3 GiS51078 - but it doesn't connect... is this cube already supported?

I have a fix, but its for the giiker library that this web app is dependent on. Luckily that app is written by the same guy. I will make a pull request tomorrow.

If you want to use the web app now, you can always rename your cube through the Supercube app. The Giiker Bluetooth wrapper assumes the prefix of the cube name is GiC which the old Giiker i3 started with. The Giiker i3s name starts with GiS.

I have same problem. Renaming cube in the app seem to only change the cube name for this app not globaly. Is it only my problem? I am still not able to connect new Giiker cube to this timer.

Any update? Still unable to connect. Renaming the cube does not help

I have the new i3S version of the cube and I have the same problem. It needs some kind of PIN. That happens on Windows 10. I tested on a Mac and it can connect successfully there, though.

Hello, I have the same problem with connection.. on Android and on Windows.. also i3s.. tried to rename the cube in the official app but without results.  tried Hakatashis and Pdoronilas Forks... Where to start some troubleshooting? Any idea what could be the problem?
Thanks for any ideas in advance!

Hi,
I've the same problem here.
I noticed, my cube's name is Gixxxxxx without "C" or "S" after "i".
@hdodov and @pajinko have you the same start string in your cubes?
Probably this could help.
TheBags

@TheBags Yes, my cube starts also with Gi11xxxx
btw: I found out that this page is working for me: https://avaver.github.io/
but I want this one to be working as it is also supporting Roux.. Anyone capable of comparing with https://github.com/avaver/supercube what the difference is? @pdoronila maybe suggestions from your side?


rename the node

I tried to rename the node,but failed.I need your help very much,thank you！

please show your code


[GHSA-248v-346w-9cwc] Certifi removes GLOBALTRUST root certificate

Updates

Affected products
Description

Comments
On Pypi, versions do not have leading zeros. Tag 2024.07.04 in python-certify becomes 2024.7.4 in Pypi versions. Same goes with 2021.05.30 which becomes 2021.5.30 in Pypi.
The difference in versions numbers opened a vulnerability alert on one of my repos. which ended up being a false positive.

Hi there @alex! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

Hi @alex, thank you for your response. I was not aware of this behaviour from Pypi, I thought it could have been a typo.
However, the current version numbers written in this Security Advisory has triggered a false-positive issue in my repository, as my current version 2024.7.4 was not interpreted as 2024.07.04.
Maybe this would suggest a larger issue in github-advisory's version matching for python ?

If a tool does not correctly normalize pypi versions, it will have many
false positives, not just this one.
On Thu, Jul 11, 2024, 11:48 AM Nathan Malnoury @.***>
wrote:

Hi @alex https://github.com/alex, thank you for your response. I was
not aware of this behaviour from Pypi, I thought it could have been a typo.
However, the current version numbers written in this Security Advisory has
triggered a false-positive issue in my repository, as my current version
2024.7.4 was not interpreted as 2024.07.04.
Maybe this would suggest a larger issue in github-advisory's version
matching for python ?
—
Reply to this email directly, view it on GitHub
https://github.com/github/advisory-database/pull/4597#issuecomment-2223292111,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAAGBAUZKECQHKEHMW6ZMLZL2SL3AVCNFSM6AAAAABKWKYLUKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRTGI4TEMJRGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Nathanmalnoury can you expand on the false positive you're seeing? What version do you have in your manifest? This might be a bug to filter over to the dependabot team.

Hi,
I'll go over everything once again, for now it seems I have mistaken myself (checking my requirements on a non-main branch). I'll repro to confirm that both this GHSA and dependabot are working fine with the leading-zeros.
Apologies for the time wasted and thank you for your responses to you three.

I can confirm, after repro, that having 2024.7.4 in requirements matches with the patched version marked in this GHSA (2024.07.04), even with the leading zeros.

All good. Thanks for the concern and for the double check 👍


Fix some race conditions

Fixes some race conditions that I've encountered in #2207.

This PR should also fix the flaky test that failed the latest run on develop: https://github.com/Open-MSS/MSS/actions/runs/8078528692/job/22071046023


Validate that document ids only contain text characters.

@baldersheim or @vekterli please review

@bratseth FYI

Summary of discussion: Accept only text characters in incoming documents, but allow anything except 0-byte when retrieving existing documents.

@baldersheim and @vekterli please review again. @bratseth FYI


various small semantic, addresses #188

various small semantic changes #188

Will make final changes in a different commit since I can't edit those into this one.


First check in for Migration Dashboard

Database migration status is still work in Progress

This entire dialog will be refreshed.

Pull Request Test Coverage Report for Build 572850155

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at 43.439%

Files with Coverage Reduction
New Missed Lines
%

extensions/import/src/wizard/pages/fileConfigPage.ts
3
72.99%

Totals

Change from base Build 572836368:
0.0%

Covered Lines:
24938

Relevant Lines:
52579

💛  - Coveralls


Stake Pool tab no longer displayed

Version: 3.1.0
Build: #15733

@tdiesler it shows after a while. Daedalus sometimes receives shelley era hardfork info from the Api slower than usual and thus the button shows up with delay (1-2 mins or so). This is a known issue in the cardano-wallet component and the Adrestia team is working on the fix.

@tdiesler it shows after a while. Daedalus sometimes receives shelley era hardfork info from the Api slower than usual and thus the button shows up with delay (1-2 mins or so). This is a known issue in the cardano-wallet component and the Adrestia team is working on the fix.

@tdiesler this issue will be fixed after this PR is merged: https://github.com/input-output-hk/daedalus/pull/2325

The fix for this issue is now available in Daedalus Mainnet 3.3.0.
Please update your version of Daedalus either through the automatic update or manually by downloading the installer from: https://daedaluswallet.io/en/download/
Thank you for your patience!


Feature/optimize mappings

Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #356 (9f91403) into index-operation (e2edd70) will increase coverage by 1.19%.
The diff coverage is n/a.

@@                 Coverage Diff                 @@
##           index-operation     #356      +/-   ##
===================================================
+ Coverage            54.71%   55.90%   +1.19%
===================================================
  Files                  205      209       +4
  Lines                 6494     6713     +219
  Branches              1037     1076      +39
===================================================
+ Hits                  3553     3753     +200
- Misses                2680     2694      +14
- Partials               261      266       +5

Impacted Files
Coverage Δ

.../pages/Indices/containers/IndicesActions/index.tsx
83.33% <0.00%> (-12.50%)
:arrow_down:

...lic/pages/Indices/containers/IndexDetail/index.tsx
80.00% <0.00%> (-6.05%)
:arrow_down:

...onents/FormGenerator/built_in_components/index.tsx
75.00% <0.00%> (-5.00%)
:arrow_down:

...CreateIndex/components/IndexDetail/IndexDetail.tsx
91.04% <0.00%> (-1.15%)
:arrow_down:

...CreateIndex/containers/CreateIndex/CreateIndex.tsx
88.37% <0.00%> (-0.18%)
:arrow_down:

...gement-dashboards-plugin/public/utils/constants.ts
100.00% <0.00%> (ø)

...ublic/pages/Indices/containers/Indices/Indices.tsx
82.60% <0.00%> (ø)

.../Indices/components/ShrinkIndexFlyout/constants.ts
100.00% <0.00%> (ø)

...components/ShrinkIndexFlyout/ShrinkIndexFlyout.tsx
96.38% <0.00%> (ø)

...dices/components/OpenIndexModal/OpenIndexModal.tsx
100.00% <0.00%> (ø)

... and 5 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


exdate in repetitive tasks

I found problems in repetitive tasks. I want to parse an ical from Google Calendar and if event is throughout a day (no hour specified), DTSTART,DTEND and EXDATE are formated YYYYMMDD.
I add this code in process_recurrences function and it seems good.
before:
if ((!isset($anEvent['EXDATE_array'])) || (!in_array($anEvent['DTSTART'], $anEvent['EXDATE_array']))) {}

after:
if ((!isset($anEvent['EXDATE_array'])) || (!in_array($anEvent['DTSTART'], $anEvent['EXDATE_array'])) && (!in_array(substr($anEvent['DTSTART'],0,8), $anEvent['EXDATE_array']))) {}

There is probably a cleaner solution but this one works fine.
Thank you for this lib.

Hi tazzon,
Can you provide a snippet of your iCal file that's causing you this problem so I can reproduce the errors you're seeing? Without this I cannot determine the best course of action.

Hi,
here is a snippet of my ics file.
BEGIN:VEVENT
DTSTART;VALUE=DATE:20150909
DTEND;VALUE=DATE:20150910
RRULE:FREQ=DAILY;COUNT=5
EXDATE;VALUE=DATE:20150911
DTSTAMP:20150831T072047Z
<EMAIL_ADDRESS>CREATED:20150830T073554Z
DESCRIPTION:
LAST-MODIFIED:20150830T073554Z
LOCATION:
SEQUENCE:0
STATUS:CONFIRMED
SUMMARY:bbbbbbb
TRANSP:TRANSPARENT
END:VEVENT

There is 5 events with daily recurring.
When the function process_recurrences creates recurring events, DTSTAST for each are :
[0]
   DTSTART=20150909
   EXDATE=20150911
[1]
   DTSTART=20150910T000000
   EXDATE=20150911
[2]
   DTSTART=20150911T000000
   EXDATE=20150911
[3]
   DTSTART=20150912T000000
   EXDATE=20150911
[4]
   DTSTART=20150913T000000
   EXDATE=20150911

event[0] DTSTART format don't change.
For event [2], it can't find EXDATE_array[20150911T000000], so this event is returned.
I hope this can help you.

@tazzon: see if 868e286 solves things

Hi,
This code works fine, I tested and all my recursive events are good.
Thank you for your work.


Alternative implementation of array_flatten().

A more performant version of util::array_flatten, utilizing the core PHP function array_walk_recursive.
Updated to support UtilPHP's handling of numerical indexes.

Coverage decreased (-0.26%) to 70.16% when pulling 08c6f116f3b50199f571cca739db2f09f79e38f9 on phpexpertsinc:master into 87576da7416766d0ed68b08dc7b01a779e95b39c on brandonwamboldt:master.

Thanks!


Add enums for detector motion

A lot of the movements in DetectorMotion are binary setpoints, using 0/1 to mean open/closed. We should refactor these to be enums to be more descriptive

from ophyd_async.core import StandardReadable
from ophyd_async.epics.motor import Motor

class DetectorMotion(StandardReadable):
    """Physical motion for detector travel"""

    def __init__(self, prefix: str, name: str = "") -> None:
        self.y = Motor(prefix + "Y")  # Vertical
        self.z = Motor(prefix + "Z")  # Beam

        super().__init__(name)

at the moment the file does not have binary setpoints so either it's the wrong file or it has changed substantially since.
@noemifrisina from git log I infer you might have more context here, what happened here?

I'm not sure, I think that the original issue may have been referring to this DetectorMotion device ?
The i24 one is separated because it's quite different and is fine as it is at the moment...

ah I see

   self.shutter_closed_lim = epics_signal_r(
            float, f"{prefix}{device_prefix}CLOSE_LIMIT"
        )  # on limit = 1, off = 0
        self.shutter_open_lim = epics_signal_r(
            float, f"{prefix}{device_prefix}OPEN_LIMIT"
        )  # on limit = 1, off = 0
        self.z_disabled = epics_signal_r(
            float, f"{prefix}{device_prefix}Z:DISABLED"
        )  # robot interlock, 0=ok to move, 1=blocked
        self.crate_power = epics_signal_r(
            float, f"{prefix}{pmac_prefix}CRATE2_HEALTHY"
        )  # returns 0 if no power
        self.in_robot_load_safe_position = epics_signal_r(
            int, f"{prefix}{pmac_prefix}GPIO_INP_BITS.B2"
        )  # returns 1 if safe


Implemented Reject message

Added (very simple) tests, all tests passed, code coverage still at 100% and no new lint issues added. Let me know if there is anything I should change.

Seems I had forgotten to get back to this pull request. Fixed spacing and removed commented out old code. Rebased and should apply cleanly.

Great, thanks for updating!

LGTM

Released in v1.1.0


Night Sky Theme Not working Properly

Describe the bug
The Night Sky Theme in the Tools section on selecting works well, but on refresh it fades all the options on the Navbar.
To Reproduce
Steps to reproduce the behavior:

Go to 'beta.circuitverse.org'
Click on 'simulator'
Apply Night Sky theme and refresh
See error

Expected behavior
The Navbar options should remain as it is like in the other themes.
Screenshots
Default Theme

Night Sky Theme

On refreshing the Page

Device Information [optional]:

OS: [Windows]
Browser [Chrome]

Are you working on this issue? (Yes/No)
No

I think the issue will be here public/src/themer/themes.js, some colors might be missing for some themes.

Hi, @Shivansh2407. Can I work on this Issue?
I'm new to opensource and hope will get some values by contributing to it.

Hi, @Shivansh2407. Can I work on this Issue?
I'm new to opensource and hope will get some values by contributing to it.

Go ahead, this is an easy one. Good luck!

@satu0king, Is the the fix for this issue deployed on Beta Site? Because I am able to reproduce the bug currently in the Beta Site. Please check and Update.


update viewer, add scroll for y overflow

@rgerstenberger fix ui bugs with lists and overflow

LGTM!


(FACT-865) Work around missing stderr output in tests

FACT-865 documents a bug where stderr written by an external fact isn't
reproduced in facter output. Preserve the behavior of an existing
acceptance test that relied on that behavior by writing to a file
instead.

Other than that, :+1:

:+1:


Transfer to conda-forge

@wolfv @traversaro @Tobias-Fischer
This feedstock is ready to transfer to conda-forge

Awesome. Are you happy to create a pull request?

Can I transfer the repo to conda-forge organization and then modify it once there to use the standard build tools?

Oh, nevermind I will create a pull request to staged-recipes

yes, that's the way to go :)

All set: https://github.com/conda-forge/staged-recipes/pull/22741


how can i override layoutPath() ?

I belive I cannot customize/override (in my client code) this func in hbs.js
ExpressHbs.prototype.layoutPath()

coffeescript sample not working for me;
exphbs = require 'express-hbs'
originalLayoutPath = exphbs.layoutPath
exphbs.layoutPath = (filename,layout) ->
  console.log "I never see this output"
  originalLayoutPath filename,layout

# express3 init below

What did I miss there??

I know this is super old, but layoutsDir is the name of the config option and it expects a string :)


Allow basemap to be installed using pip, virtualenv and a requirements.txt file

I ran into problems trying to install basemap using pip, virtualenv and a requirements.txt file. On installation, it tries to use setup.py to check the egg_info, and when it does this it throws an ImportError because numpy has not yet been installed (it is also in the requirements.txt file).
Another project, netcdf4-python, had this problem and solved it by checking to see which command was being used with setup.py and only installing numpy if it was during installation: https://github.com/Unidata/netcdf4-python/commit/18f21a2ae2c21531f7d3bf8610cbc27cf2df2381. I copied the idea, with the main change being that setup.py still uses setup and Extensions from numpy during installation, as I wasn't sure if this was important and didn't want to break things for anyone.
The fix can be checked using a virtualenv. Create a virtualenv:
virtualenv basemap_env
cd basemap_env
source bin/activate
pip install -r requirements.txt

requirements.txt (works OK):
numpy
matplotlib
git+https://github.com/markmuetz/basemap.git@pip_install#egg=basemap

requirements.txt (doesn't work)
numpy
matplotlib
git+https://github.com/markmuetz/basemap.git@master#egg=basemap
# or can just be:
# basemap

:+1: Seems fine to me.
@jswhit Should I feel free to merge changes like this?


Shorter type names in recursive types

Description:
This is a workaround for trait bounds generated for recursive types in scale-codec.(see paritytech/parity-scale-codec#603, fixes paritytech/subxt#1603).
To generate short Paths in recursive types we will be temporarily storing parent_path inside the settings struct that's passed around when generating TokenStreams.
So instead of passing just the alloc_crate_path to the TypePath::to_syn_type we will pass all of the settings to make use of parent_path.  So when we encounter a Path with a param and it will be equal to the parent_path we will only leave the last segment inside of it.
Examples:
  #[allow(unused)]
    #[derive(TypeInfo)]
    #[scale_info(skip_type_params(A))]
    struct Example<A, B> {
        id: A,
        rest: B,
        inner: Vec<Example<A, B>>,
    }
    ...
    // Registered:
    let for_codegen = Testgen::new()
        .with::<Example<String, u32>>()
        .with::<Example<u8, u32>>()
        .with::<Example<u16, u32>>()
        .try_gen_tests_mod(Default::default(), true)
        .unwrap();
    ...
    /// Generated
        let expected = quote! {
        pub mod tests {
            use super::types;
            pub struct Example1<_1> {
                pub id: ::std::string::String,
                pub rest: _1,
                // Note the short path for `Example<_>`
                pub inner: ::std::vec::Vec<Example1<_1> >,
            }
            pub struct Example2<_1> {
                pub id: ::core::primitive::u8,
                pub rest: _1,
                pub inner: ::std::vec::Vec<Example2<_1> >,
            }
            pub struct Example3<_1> {
                pub id: ::core::primitive::u16,
                pub rest: _1,
                pub inner: ::std::vec::Vec<Example3<_1> >,
            }
        }
    };

Do we need this or can we add the #[codec(dumb_trait_bound)] attr by default to the codegen?

Do we need this or can we add the #[codec(dumb_trait_bound)] attr by default to the codegen?

We can, but i'm not sure if we will break something wrt trait bounds, because we will add this attribute to every enum/struct that there is

tested with subxt-codegen with just indiscriminately adding dumb_trait_bound to every struct/enum and it compiles. (tested only for polkadot-full metadata)

That's probably a good sign then :)

closing in favor of paritytech/subxt#1630


chore: remove extra license header in swagger-ui

subtask of #1880
we're looking forward to a better way to import the swagger-ui instead of including the whole front-end dir (in dist)
currently, remove the extra license first

@javeme useless jar?

@javeme useless jar?

if need it, move it to doc repo & download it
And another binary file:

@zyxxoo consider handle the 4 files in another PR


Exposes workspace functions to clear popups on an individual editor or all editors

Closes QP-6001

I was trying to think of any other methods to expose, but I couldn't think of any yet.
I was thinking it might be nice to have a "show completions" or similar methods, to programmatically show those items, but those seemed tricky to add in a meaningful way (since we'd have to extract the token information manually). Maybe we can add that later.


adding "artifactName" field to package.json doesn't change the output file

Hello,
I've added "artifactName" field to package.json, but the output file stays the same.
There are two package.json files, one in the root dir, and one inside /app.
The root level package.json:
"build": {
    "artifactName": "${name}-${version}-${os}.${ext}",
    "productName": "VueElectron",
    "appId": "com.pepo.vue-electron",
    "category": "public.app-category.tools",
    "dmg": {
      "contents": [
        {
          "x": 410,
          "y": 150,
          "type": "link",
          "path": "/Applications"
        },
        {
          "x": 130,
          "y": 150,
          "type": "file"
        }
      ]
    },
    "files": [
      "dist/",
      "node_modules/",
      "package.json"
    ],
    "mac": {
      "icon": "app/icons/icon.icns"
    },
    "win": {
      "icon": "app/icons/icon.ico"
    },
    "linux": {
      "target": [
        "AppImage"
      ]
    }
  },

running "npm run build" produces "VueElectron Setup 0.0.0.exe" (disregarding the artifactName).
I've also noticed that changing the "version" in the main package.json doesn't affect the output file, but changing the version in /app/package.json DOES change the name of the output file.
Thanks in advance!
#########

Node version: 6.11.0
NPM version: 3.10.10
vue-cli version: (if necessary)
Operating System: Windows

@syonip
The version will come from app/package.json because the use of the 2 package.json structure. You also typically want to change the version for every build you make, as I'm not sure if electron-builer will overwrite previous builds. I'm not entirely sure what happened based on what you described, but this is definitely not an issue with electron-vue. Hope this helps.


adservice add adServiceHighCpu featureflag to flagd files

Add adServiceHighCpu featureflag to flagd file. See upstream PR here: https://github.com/open-telemetry/opentelemetry-demo/pull/1510

This PR will need to wait for the demo PR to be merged and released.

@EislM0203 please run make generate-examples CHARTS-opentelemetry-demo


DynamicParameters is called anytime resolving a component even if its lifestyle is singleton

I need to inject a depenendy at runtime in the constructor of a singleton component, I have tried to use DynamicParameters() and DependOn(), but they works fine except for that they are called each time I request an instance of the component to the container even if the component is Singleton.
I have returned a lambda for the dispose of the dependency injected but it is called only for the first object created, the only one used in the constructor of the singleton component. The others instances created seem to create memory leaks.
I have a service MyService that implements IService interface and a component MyLogger that implement ILogger interface.
I need to register only MyLogger component as Singleton and then passing an instance on MyService at runtime, I have tried DynamicParameters and DependOn with the same result
_container = new WindsorContainer();
        _container.Register(Component.For<ILogger>()
            .ImplementedBy<MyLogger>()
            .DynamicParameters((k, d) =>
            {
                var service = new MyService();
                d[typeof(IService)] = service;

                return r => service.Dispose();
            })
            //.DependsOn((k, d) =>
            //{
            //    var service = new MyService();
            //    d[typeof(IService)] = service;

            //    return r => service.Dispose();
            //})
            .LifestyleSingleton());

If I call the resolve of the ILoggerComponent 2 times, I have the same instance for MyLogger, but the code inside DynamicParameters or DependOn are colled twice, so 2 instance of MyService are created.
        var logger1 = _container.Resolve<ILogger>();
        logger1.Log("first event");

        var logger2 = _container.Resolve<ILogger>();
        logger2.Log("second event");

so when I call the Dispose of the container,
_container.Dispose();
the component MyLogger is Disposed, and only the first instance of MyService is Disposed, the one used in the constructor of the singleton component MyLogger, the second instance of MyService is not disposed.
I think that the DynamicParameters code or the DependOn code must be called only the first time for a component of tipe Singleton.
In the end the classes that I have used.
public interface IService
{
    void DoSomething();
}

public class MyService : IService, IDisposable
{
    private static int counter = 0;
    private readonly int _counter = 0;

    public MyService()
    {
        counter++;
        _counter = counter;
    }

    public void DoSomething()
    {

    }

    public void Dispose()
    {
        Console.WriteLine("MyService Disposed Id=" + _counter);
    }
}

public interface ILogger
{
    void Log(string message);
}

public class MyLogger : ILogger, IDisposable
{
    public MyLogger(IService service)
    {

    }

    public void Log(string message)
    {
        Console.WriteLine(message);
    }

    public void Dispose()
    {
        Console.WriteLine("MyLogger Disposed");
    }
}

I've been able to verify what you're saying in a quick LinqPad script.  It does indeed seem wrong to have the DynamicParameters / DependsOn callbacks execute for each resolution of the logger.
It is indeed acting as a singleton because you get an identical object instance back for both logger1 & logger2.
Seems like a bug, but perhaps there's method in the madness?


fix(Tag): Change filled and outline to primary and secondary

part of #954

I would say this is maybe more a feat then a fix since we aren't really fixing anything here? 😅


Provide ability to override application name when spin deploy

is there a way to override the app name (configured in spin.toml) using command line flag?
my use case here is that I am working on a POC Github action to deploy preview of apps, and right now I am doing a text replacement in the file to generate a unique name of app.
it would be nice to be able to do spin deploy --name my-unique-name

+1 to be able to control the deployed apps name. I unintentionally created an application with the same name as something already running in the cloud, and once pushing, I overwrote the existing app in Cloud.

This would mark the first time that deploy is overriding something in the spin manifest. Keeping the manifest as the source of truth seems ideal. Are there other scenarios we can think of, as this is not a simple change?

The name in the app manifest is not used for anything user-facing in Spin. I think the flow from source (code + manifest) to a package (OCI) to an instance running somewhere (app in Fermyon Cloud, or Pod in k8s) should have the ability to use different names.
I'm not sure what the current logic is for OCI packaging, but having a default to use app manifest name plus version as tag seems reasonable, but still having an option to provide both name and tag should be provided. This I believe is similar to creating a deployment to the cloud.
To your question @kate-goldenring, I wouldn't consider this overwriting something in the manifest, more a fallback if a name for the app instance in the cloud was not provided.
As the cloud might evolve with features for revisions or OCI pull, there's probably more info needed about an instance, which can maybe be derived from the manifest (e.g. version), but I'm not sure that's the best option.


Type checking issues with worker receive with check

Description:
$title. Type checking does not happen for the error relevant to check and the enclosing invokable's return type.
Applicable for sync send too.
Steps to reproduce:
import ballerina/io;

const R = "error reason";

type E error<R>;

public function main() {
    worker w1 returns error? {
        if (true) {
            return error("generic error");
        }

        5 -> w2;
    }

    worker w2 returns E? {
        int j = check <- w1; // the check-ed error is of type `error`
    }

    E? res = wait w2;

    if (res is error) {
        io:println("res reason: ", res.reason());
    }
}

This results in the following at runtime.
$ ballerina run foo.bal
error: generic error
        at foo:$lambda$0(foo.bal:10)

Affected Versions:
$ ballerina -v
Ballerina 1.0.0
Language specification 2019R3

Related Issues:
https://github.com/ballerina-platform/ballerina-lang/issues/18162

This seems to be a generic issue with check.
See https://github.com/ballerina-platform/ballerina-lang/issues/19299.
Need to check once https://github.com/ballerina-platform/ballerina-lang/issues/19299 is fixed.

Resolved with #19374


Road to V11: CLI and toolchain

Want to get a bit of a discussion going here.
@strues @carsonperrotti @enten @bkniffler
After the major cleanup parse and release of v10 I definitely want to move into a CLI and separate toolchain model.
In regards to the toolchain, @enten has been doing some work privately and has come up with a very interesting project called tension which would support the requirements for react-universally.  I haven't been involved in it's architecture so I can't speak in depth about it, but it should be taken under consideration when moving into the next evolution of this project.
@bkniffler has been the heading up the CLI movement.  I think it will be super helpful and allow us to bring together the branching strategy we have been evolving.  We could display a list of technology options to the user and based on their selection generate an appropriate boilerplate template.
Have you guys used @gaearon's create-react-app?  It is pretty amazing how he has set the project up.  I have been going through it and have been thinking of adopting some of his ideas.  It wouldn't be a verbatim implementation as I want to crank up the configurability somewhat but his structure is pretty amazing.  I am not sure if this would touch on some sort of licensing issue though.  I wouldn't be using his exact code, but the intellectual ideas would have definitely stemmed from his work.  Do you guys think this would be an issue?
What do you think about his approach? Do you think it would be appropriate for what we would like to achieve?

I really like the sound of where this is headed. Taking steps toward a CLI and toolchain method offers up a ton of options for the end user. Of course, that leaves lots of work to be done. I think it would be in our best interest to lay out and prioritize different aspects of the options we'd like to provide.
Stylesheets:

[ ] Standard CSS
[ ] PostCSS
[ ] Sass/SCSS

Test Frameworks:

[ ] Jest
[ ] Enzyme / Chai / Mocha

Server:

[ ] Express
[ ] Koa

Additionally, I would imagine we'd want to get some form of website and demo online. Hosting it somewhere other than a Heroku free plan, in my opinion, is best in terms of speed. I don't mind offering up resources on one of my servers if there's a need.

I added glamor to the styles. :)

Added hapi to the servers list.

Looking at that list as it stands I would be happy.  It provides a fairly good blend of very popular technologies.
Agreed that we will need a website to better represent the product.  We'll need to source a designer if and when. :)

The first iteration of this looks like a lot and it might be unwise to offer up more than preconfigured implementations. As opposed to a fully functional almost Yeoman-esque generator. Maybe in the second release we can consider adding functionality generators based on user input. Thoughts?

To be honest I think you are right.  Let's phase this out, start off with a base implementation for v11 and then look towards tech stack options for a future release.  I guess the key will be in allowing for user configuration and extension, which is way more important.
With that in mind lets agree on the base tech stack, my vote is for:

express
css (standard)
jest
react-router v4

Another idea...
After the first implementation we can create seperate example repo's showing off how to create a project using redux/mobx/apollo etc

We should then consider deprecating the existing branches.

What is the purposes of the CLI? It's important to determine its scope.
If is it used to bootstrap a new application: a yeoman generator is more suitable.

Have you guys used @gaearon's create-react-app?

Once time if I remember correctly. I didn't like the lack about SSR support. I thought it wasn't a tool adapted for universal applications. And then I got there :smile: .

You should add Gitter integration to this repo or we should get in a Slack/Discord some time. Can be easier to hash some of these brainstorming ideas out in real time instead of a massive comment trail

Gitter room created.
@carsonperrotti  - it struggled to send you an invite.  You may need to open gitter and check it out manually.

@enten it also appears to have struggled to send you an invite.

https://gitter.im/react-universally/Lobby

Hey guys, @ctrlplusb, why think about this as a CLI? We could try and package everything into dependencies, that wrap the whole webpack, babel and bootstrapping (app and server) into sane packages (react-universally, react-universally-sass, react-universally-mobx), so creating a new application doesn't need boilerplate or generators or else, only installing the dependencies (which would be easy to update as well!).
react-universally-sass for example would hook into webpack config and add the appropriate loader, react-universally-mobx would contain the basic app creator function and maybe some helpers
// Browser
import createApp from 'react-universally-apollo';
const app = createApp({ uri: 'http://my-api.graphql.com' });
ReactDOM.render(app, root);

// Server
import createServer from 'react-universally-apollo/server'
const app = createServer(config);
app.listen();

The biggest challenge here would be to not sacrifice too much flexibility. Then again, everyone could create their own react-universally modules if the default ones don't suite their needs.
There is so many generators, CLI tools, boilerplates, but in the end, I'd just like to abstract away all the complexity of webpack and boilerplate into somewhere I don't need to look at constantly, and at the same time be able to implement new features without having to change all my projects.
What do you think?

I suggest meddleware (from the great krakenjs) to manage express middlewares in a better way. It's not a requirement but only a suggestion.
@bkniffler
These approachs (cli or runtime packages configuration) will create a huge abstraction layer. And @ctrlplusb  said:

Webpack already scares people enough as it is
I am scared to abstract an abstraction
Because it becomes hard for users to imagine their webpack configuration when it no longer looks similar to the webpack docs and that their webpack config is split across modules/projects.

@bkniffler @enten some solid ideas there.  There is obviously a ton to consider here.  I am going through a round of doing research on all of this.  Hopefully come back soon with some more ideas.

@enten i don't see the benefit of meddleware. its configuration based instead of imperative - thats it?
@ctrlplusb tech stack looks good. i'd rather have glamor than css, but thats just a matter of taste. going with express, react-router v4 and jest lgtm. other than glamor, i'm mostly interested in redux and apollo.

@bkniffler I think your ideas have a lot of merit.  It opens itself nicely towards providing some extensibility points too.  I still think it could work well in combination with a CLI though (or yeoman generator).
Playing with your idea a bit below...
// src/client/index.js

import { createApp } from 'react-universally';

// The following could be a boilerplate created example with some basic
// routes implemented as an example.
import Routes from '../shared/components/Routes';

// Return back an object so that it opens up potential to return other items
// in the future? e.g. redux store / meta object etc
const { App } = createApp(config);

ReactDOM.render(<App><Routes /></App>, root);

The above would assume react-router v4 with my code-split-module etc all configured.
To be honest as we are going down this approach I am slightly apprehensive in addingreact-router v4 and my code-split-module as prebaked items.  It gives user's less power, however, the server rendering options for router stacks are vast and complex so it may be best for us to commit to using certain technologies.
An alternative approach is to move specific technologies into separate packages (or sub folders) as you suggest and then user's can import and pass them as configuration.
e.g.
// src/client/index.js
import { createApp } from 'react-universally';
import reactRouterV4 from 'react-universally/routers/rr4';
import Routes from '../shared/components/Routes';
const { App } = createApp({ router: reactRouterV4 });
ReactDOM.render(<App><Routes /></App>, root);

@code-punkt
Yes is it. But there are some cool additions like app events and mostly the flow control (to run middlewares in parallel for example). And we can varying the server configuration depending to the environment easily (when it's associated with a config store like confippet, config, confit, etc).
The main advantage come if there exists a default meddleware configuration: so the developer will be able to easily enable/disable or change a middleware configuration (or a group of middlewares) through a simple configuration file.
It's not a necessity but only a suggestion.

@ctrlplusb
There are already some good "bootstrapers". Are you sure that you want to turn react-universally (= react-universally) into another one of them?
These people make a fantastic work with a different approach. But they are often restrictive or too abstract to hack them.
I made the choice of RU because you propose a boilerplate 100% hackable, comprehensive (by your relevant comments) and ready to use (and branches like redux are very useful).
I can accept to kickstart a project with a yeoman generator because I know that I will keep the control of the stack. But if I need to learn how to deal with another abstraction layer: I'm more reluctant.
Decoupling the toolchain is important to help developer which want to benefit of its latest updates (without deals with irrelevant merging conflicts). But we must do it without create another API layer: we must reuse and marrying API's tools which we used.
Create a CLI to help with npm's scripts relatives to the development and building process: it's okay. But a CLI to manage a plugged system scares me.
Definitely, create an plugged ecosystem is a big amount of work. And there already exists some. And this moves away from RU's initial purposes.
I'm afraid that the real strengths of RU (the awesome toolchain and universal application configuration) disappear (drowned in the middle of a redundant plugged system).
Please read this blog post: Why frameworks are evil.
Okay RU isn't a framework yet. I feel that it takes the way. If it's your wishes, I will respect that (but I will be sad to lost a really effective codebase for an universal react project).
The illustration below summarizes my thoughts about why I'm scared by pseudo-framework:

@enten I see your points, and yes, what I propose would effectively make react-universally a framework. But then again, it will really help structuring the code in a way that makes it easily extendable and decoupled from the application itself. There is nothing holding people off (and indeed, we could have a CLI for that) to copy the react-universally code into a project folder universally and have full control over it, we could even encourage people to do so. Having an update mechanism wouldn't be to hard neither, the CLI would just overwrite the files and you could use git to make sure your own changes to files will remain and cherry-pick changes. You're also free to never update.
I agree @ctrlplusb that react-router and codesplit can be a part of the core package, and we could have different packages for app-boilerplate (mobx, apollo, redux), server-boilerplate (express, hapi, koa) and webpack config (e.g. styles via less/scss/glamour).
In my opinion this approach wouldn't be too complicated to maintain and it will give anyone what they need.

@enten, you are right about that we shouldn't try to abstract everything away in a new API. Thats not what I imagine neither. But extending the webpack config through some kind of middleware shouldn't be a huge issue here. And the API surface to bootstrap your app doesn't need to be hugely complex, just some kind of createApp (which calls the /src/browser/index with some component) and a createServer (which calls the /src/server/index) that returns e.g. the express server with all the nifty react-universally features, and you could then add some middleware and start listening.

@bkniffler
I keep thinking that yeoman generators are more suitable.
If a developer need to change a technology in its stack: it will always has more stuff to do that only call a cli command (which will just make "basic" hidden setup). For the good at all developers, they need to see how we used libraries in the stack. So, when they open the documentation of a library (that we have chosen to use): they easily understand our intentions.
RU isn't a simple boilerplate only: it's a source of learning for developers. And learning how to use libraries in a good way will be harder if we wrap the concrete implementation in several packages. Otherwise, we will be better to collaborate on existing similar tools.
I should want to avoid that we lost a lot of time on issues already solved or work by other projects and keep RU's initial purposes unbroken.

I've got a few projects that could easily be managed in such a way. But different people have different requirements and different philosophies for solving problems, and thats okay.
As I said, I'm fine with this repo being a starting point for people who look for the state-of-the-art stack, but in the end I imagine it'll mostly be @ctrlplusb dedicating time to this project, and as he made clear at some point, this project is born out and living of his own need.
I think you've made some excellent points and I've also given some arguments and thoughts, @ctrlplusb will be experimenting with the ideas and I'm sure react-universally will remain a great repository that I'll use as a base, whatever he decides the future of it looks like.

Some amazing points gents. I am loving this discussion.  Thank you. x

I just added https://github.com/styled-components/styled-components to the list!

FYI, my current thoughts are not to create an CLI or rip out the toolchain as it creates an abstraction that goes against the goals of this being a starter kit.
But we have been working in some of the amazing ideas from @enten, @carsonperrotti, @strues and @bkniffler into the current tooling configuration.
Hopefully we can achieve a happy medium where the current project is structured in such a manner to provide just enough configuration abstraction and extensibility via the config folder, allowing for much easier merge/upgrade paths.
My other hope is the a seperate CLI tool (in another repo) can be created which target this repo and create packages that can be used in a similar way as described by @bkniffler in his personal projects.  This main repo wouldn't need to be changed at all I think, which again is great for keeping this repo in alignment with it's goals.  We may perhaps need to consider strict semantic versioning strategies though, which shouldn't be too bad to manage as the project is definitely heading towards a bit more stability/maturity in my opinion.

In my opinion there are need Browsersync support in the project.

@Kimel1 - do you mind elaborating on that?

Closing - archived.


fix: dot-repeat special symbols handling

Due to the way dot-repeat currently works, typing (or pasting) text that contains special strings like  or , or < in the Insert mode and then exiting the Insert mode will result in unexpected behavior.
Here are a couple of examples where extension behaves unexpectedly:
"<BS>" example

Type (or paste) in insert mode ""
Press Enter (adds new line)
Press Escape (switches to Normal mode)
Press . (dot)

Expected output:
<BS><BS>
<BS

Actual output:
<BS><BS>
<BS

Demo
https://user-images.githubusercontent.com/16840190/222908112-c52e2f3e-fa7d-49c2-a0bd-5c7043e67400.mp4
<LT><cmd>call VSCodeNotify('workbench.action.files.openFile') command injection example (also works with other commands, as well as with "<<Cmd>" instead of "<LT><Cmd>")

Type (or paste) in insert mode "<LT><cmd>call VSCodeNotify('workbench.action.files.openFile')"
Press Enter (adds new line character)
Press Escape (switches to Normal mode)

Expected output:
\<LT\>\<cmd\>call VSCodeNotify('workbench.action.files.openFile')

Actual output is the same as the expected output, but the workbench.action.files.openFile command gets called
Demo
https://user-images.githubusercontent.com/16840190/222908135-2c967581-7964-4c19-8c18-5be43d17e28b.mp4

Confirmed, fixes https://github.com/vscode-neovim/vscode-neovim/issues/1185.

Confirmed, fixes https://github.com/vscode-neovim/vscode-neovim/issues/1185 on a Mac.

Confirmed, fixes #1185 on a Mac.

Thanks. Need a Windows user to help out with the test failure.

@theol0403, I fixed the test.


Funnel/Chute first inventory slot selected

Describe the Bug
Chute/Funnels sending items into containers don't go to existing stacks, which is annoying and is inconsistent with forge, who have items going to existing stacks

Reproduction Steps
Create a chest
Place items like image 1
Place a funnel in image 2
Throw a dirt into the funnel

Expected Result
Items should go into existing stacks
Screenshots and Videos
No response
Crash Report or Log
N/A
Operating System
Windows 11
Mod Version
0.5.1d
Minecraft Version
1.19.2
Other Mods
Many, N/A
Additional Context
No response

It matches the behavior of hoppers. I think it makes more sense to call the forge behavior a bug.


wezterm: fix build for darwin

Description of changes
This PR applies this fix (kitty) to the wezterm package, unbreaking the build on macOS.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 180311 run on x86_64-darwin 1

Result of nixpkgs-review pr 180311 run on aarch64-darwin 1

1 package built:

wezterm


Add nim playground to playground list

What does this PR do?
Add resource(s)
For resources
Description
Adds the nim playground which is a website when you can run and share nim code
Why is this valuable (or not)?
It is valuable since it allows easy code sharing and quick testing
How do we know it's really free?
There are no signups or anything else required to use the website
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines
[x] Search for duplicates.
[ ] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction)

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

TIP :rocket:: Thanks for taking your time on fill Pull Request fields in your first contribution :heart:
It's always a good practice which it use to increase the mantainers productivity.


[mdButton] using interpolation within class break the button style

Bug, feature request, or proposal:
Bug
What is the expected behavior?
using interpolation within class should not break the button style
What is the current behavior?
The md-button style not applied
What are the steps to reproduce?
<button class="{{ 'test' }}" md-raised-button>Raised button</button>

Demo: https://stackblitz.com/edit/angular-qtykwp?embed=1&file=app/app.component.html
What is the use-case or motivation for changing an existing behavior?
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Is there anything else we should know?

This isn't the correct way to bind a class. You can use ngClass instead:
<button [ngClass]="'test'" md-raised-button>Raised button</button>


Update several development dependencies

In detail:
@babel/core               ^7.23.2  →   ^7.23.3
@babel/preset-env         ^7.23.2  →   ^7.23.3
@babel/preset-typescript  ^7.23.2  →   ^7.23.3
@types/jest               ^29.5.5  →  ^29.5.10
eslint                    ^8.51.0  →   ^8.54.0
release-it                ^16.2.1  →   ^17.0.0
typedoc                   ^0.25.2  →   ^0.25.4
typescript                 ^5.2.2  →    ^5.3.2
vite                      ^4.4.11  →    ^5.0.2

This needs more work, the vite changes need to be handled... this is WIP for the time being

@dnlkoch you can look again, if you want.

Thanks for the review.


IPv*Interface objects in the same network but different addresses hash to the same value and compare as equal

What steps will reproduce the problem?
Using python 2.7 with the 3144 branch of the ipaddr module on Linux:

>>> import ipaddr as ipaddress
>>> addr1 = ipaddress.ip_interface('<IP_ADDRESS>/24')
>>> addr2 = ipaddress.ip_interface('<IP_ADDRESS>/24')
>>> addr1 == addr2
True                                         <==== should be False
>>> hash(addr1)
4127129088
>>> hash(addr2)
4127129088                                   <==== should be different than
hash(addr1)

What is the expected output? What do you see instead?
addr1 and addr2 should not hash to the same value because they represent
different addresses.  The implementation for IPv4Interface.__hash__ and
IPv6Interface.__hash__ in python 3.3 found at
http://hg.python.org/cpython/file/default/Lib/ipaddress.py includes the IP in
the calculation:

    def __hash__(self):
        return self._ip ^ self._prefixlen ^ int(self.network.network_address)

This would cause <IP_ADDRESS>/24 and <IP_ADDRESS>/24 to hash to different values.

The __eq__ method also incorporates the IP in the calculation:

    def __eq__(self, other):
        address_equal = IPv4Address.__eq__(self, other)
        if not address_equal or address_equal is NotImplemented:
            return address_equal
        try:
            return self.network == other.network
        except AttributeError:
            # An interface with an associated network is NOT the
            # same as an unassociated address. That's why the hash
            # takes the extra info into account.
            return False

The same goes for __lt__.

Using python 3.3.1:
>>> import ipaddress
>>> addr1 = ipaddress.ip_interface('<IP_ADDRESS>/24')
>>> addr2 = ipaddress.ip_interface('<IP_ADDRESS>/24')
>>> addr1 == addr2
False
>>> hash(addr1)
25
>>> hash(addr2)
26

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 13 May 2013 at 8:03

The problem is not solved yet, I have the same issue when I use IPv4Network and try to use a dictionary in Python2.


How to use params_example_preproc_event.json to processing data include event

Hi, Cristina,
I noticed that in the second version of the release you posted, there is a params_example_preproc_event.json file，I wonder  to know how to use params_example_preproc_event.json to processing data include event.
Thanks!

Hi,
The readme was updated with the preprocessing parameters for event-relate preprocessing. You can have a look there.
Best,
Cristina


Download Edge version via Docker Hub

Signed-off-by: Stefan Scherer<EMAIL_ADDRESS>Proposed changes
Update the link to download the latest Docker Desktop Edge version via Docker Hub. The previous link downloads <IP_ADDRESS> and we want users to get the current Edge version.

The page on Docker Hub on its hand points back to download.docker.com (https://download.docker.com/win/edge/Docker Desktop Installer.exe) wondering if we need the "Docker Hub" link in between, or could just point to that URL?

@thaJeztah Thanks for chiming in. We discussed this with @nebuk89 and agreed to point users to https://hub.docker.com/editions/community/docker-ce-desktop-windows for download URLs.


test build asks to enter "new" passcode when turning passcode off

Should say "current" like Blackphone 2.1.4 (and I assume Blue and Orange 2.1.4 do - Blue is having server issues at the moment so I can't verify).

Good catch - fixed in 17233e2e8d40a577cbd39c783332362528969c66 and 6957b4df00560a410dc4d6fe6f05efba5f8b4bf7 (yeah, two commits for two edits to the same string - oy).

Still in the latest build I got

Actually, only in Spanish. It's corrected in English, both Australia and USA

The disable-passcode message has been fixed, along the way, and reads in the current version: "Ingresa tu código de acceso de 4 dígitos" ~= "Enter your 4-digit access code".


Add New Package/Tool | 404 Support Page as npm Package

I want to add my custom npm package which provides a custom 404 page to the user in a single command installation.
https://github.com/user-attachments/assets/d536d88c-f6aa-481e-9add-b31b7d792bb8

Describe the feature broadly, it's not clear, what you're conveying to add.

I want to add my custom npm package which provides a custom 404 page to the user

I want to add my custom npm package which provides a custom 404 page to the user

not needed for our usecase


Unable to write to device

Hiya,
I recently started use this library and It is amazing, thank you very much.
I am trying to get this working in the Unity Editor for faster debugging. After connecting to my BLE device I am able to receive data just fine but the device does not receive any data I try to send.
I know the device works because I can send data through our webpage.
I am not too familiar with Bluetooth development so I was wondering if anyone had run into this issue. I was also wondering what the best way I can debug this issue is?
Thank you.

For investigation, you can download and run the Microsoft's Bluetooth Low Energy sample and see whether writing the characteristic works there.
Can you see any error message in the GUI?
You could also try out changing GattWriteOption::WriteWithoutResponse to GattWriteOption::WriteWithResponse at https://github.com/adabru/BleWinrtDll/blob/main/BleWinrtDll/BleWinrtDll.cpp#L536 . That would make the call wait for the BLE device to send a "I received" confirmation. In BLE terms that would be "request"+"response" instead of a "command" without a response.

Hello,
I managed to compile the code and changed the line 536. I customized the code and I'm trying to write a letter "C" or "F" but when I press the Write Button I get:
"BleWinrtDll.cpp:545 SendDataAsync catch: The attribute cannot be written."
I also seem to subscribe to the first characteristic of my BLE peripheral despite clicking on the second characteristic.
This bug only happens from within unity, it doesn't happen with either the BLE scanner app from my android phone nor the same PC BLE scanner app. Those apps seem to work fine.

if I have only one characteristic the write button works fine

Thank you. This hint helped me solve this issue. I logged a bunch of things and found out that changing the characteristics dropdown was not correctly setting the selected characteristic in the script. It was always trying to subscribe to the first one since the script selects the first item by default.
This issue is that the dropdown was never hooked up to the SelectCharacteristic function in the Demo script.
Simple fix:

select the characteristics dropdown in unity
in the inspector, find "On Value Changed (Int32)" in the "Dropdown" component
add Demo.SelectCharacteristic and pass "DropdownCharacteristic" as the argument

@ehauner would you mind creating a pull request :) ? Currently I don't have the setup to test this.

Hi,
we started to use the BlewinrtDll we need it for our Unity app and the solution seems great.
we can connect to the BLE device and can receive data but the function "SendData" always return false
we have try to use the precompiled dll that e find in the unity example non threaded
the device can recieve correctly from/with other libraries for ios and Mac
we have tried also this
You could also try out changing GattWriteOption::WriteWithoutResponse to GattWriteOption::WriteWithResponse at https://github.com/adabru/BleWinrtDll/blob/main/BleWinrtDll/BleWinrtDll.cpp#L536 . That would make the call wait for the BLE device to send a "I received" confirmation. In BLE terms that would be "request"+"response" instead of a "command" without a response.
but the returnis still false.
what can we do about it?
Thank you in advance.

Hi @tommasoFabbri :),
please open a new issue and write which error message you are getting. Please also try the Microsoft provided sample linked in the Readme to checkout if sending data works there.


Create Dockerfile

Closes #32.
Adds Dockerfile to create a Parrot Docker image.
Slim version of Rust image was chosen because the alpine one has musl instead of glibc, and cross-compiling Opus gets somewhat complicated.

This is a really good idea, @rafaeldamasceno. I've been testing it and everything is working fine. 👏


libz-sys is incompatible with Visual Studio 2022 (unsupported or unknown VisualStudio version: 17.0)

error: failed to run custom build command for `libz-sys v1.1.3`

Caused by:
  process didn't exit successfully: `C:\Users\kscot\OpenGMK\target\release\build\libz-sys-27377adaf726fbab\build-script-build` (exit code: 101)
  --- stdout
  cargo:rerun-if-env-changed=LIBZ_SYS_STATIC
  cargo:rerun-if-changed=build.rs

  --- stderr
  thread 'main' panicked at '

  unsupported or unknown VisualStudio version: 17.0
  if another version is installed consider running the appropriate vcvars script before building this crate
  ', C:\Users\kscot\.cargo\registry\src\github.com-1ecc6299db9ec823\cmake-0.1.45\src\lib.rs:797:25
  stack backtrace:
     0:     0x7ff6137010cf - <unknown>
     1:     0x7ff613719d8a - <unknown>
     2:     0x7ff6136fac78 - <unknown>
     3:     0x7ff613703a36 - <unknown>
     4:     0x7ff61370351c - <unknown>
     5:     0x7ff613704095 - <unknown>
     6:     0x7ff613625c80 - <unknown>
     7:     0x7ff61362ec4f - <unknown>
     8:     0x7ff613625b8c - <unknown>
     9:     0x7ff61362bbb6 - <unknown>
    10:     0x7ff61362825b - <unknown>
    11:     0x7ff6136227c8 - <unknown>
    12:     0x7ff613621198 - <unknown>
    13:     0x7ff613623db6 - <unknown>
    14:     0x7ff613623787 - <unknown>
    15:     0x7ff61362578c - <unknown>
    16:     0x7ff613700a17 - <unknown>
    17:     0x7ff613625770 - <unknown>
    18:     0x7ff613623296 - <unknown>
    19:     0x7ff61371e0e4 - <unknown>
    20:     0x7ffc458e54e0 - BaseThreadInitThunk
    21:     0x7ffc468a485b - RtlUserThreadStart```

With #87 merged, this should be fixed. Please feel free to open an issue if you encounter a bug with the new version.


*: Fix truncated thread name

What problem does this PR solve?
Problem Summary:
The max length of thread name is 16 limited by kernel. The length of thread name of unified read pool and storage read pool exceeds 16. Kernel truncates the thread name so that we can't group by the thread name in metrics conveniently. See

What is changed and how it works?
What's Changed:

Change thread name of unified-read-pool to unified-read
Change thread name of store-read-low/normal/high to store-read-lo/nl/hi

Check List

No code

Release note

None

Could the previous way using unified-read-po* cause any problem?

I suggest to maintain a map between thread id and thread name instead. You can see a previous implementation at #5593.

Could the previous way using unified-read-po* cause any problem?

can't see CPU usage for each thread


XEN-Implant

137760 GLAUCOMA, PRIMARY OPEN ANGLE; POAG
https://www.ncbi.nlm.nih.gov/pmc/articles/PMC7962186/
Poelman HJ, Pals J, Rostamzad P, Bramer WM, Wolfs RCW, Ramdas WD. Efficacy of the XEN-Implant in Glaucoma and a Meta-Analysis of the Literature. J Clin Med. 2021 Mar 7;10(5):1118. doi: 10.3390/jcm10051118. PMID: 33800112; PMCID: PMC7962186.
XEN is a minimally invasive surgical procedure to treat glaucoma, a condition that can damage the optic nerve and lead to loss of vision. The XEN procedure involves implanting a small, flexible, biocompatible tube (XEN gel stent) into the eye to improve the drainage of aqueous humour, the fluid that circulates in the eye, and lower intraocular pressure. The stent is designed to create a permanent connection between the anterior chamber of the eye and the subconjunctival space. This filtering action allows for improved drainage and lower intraocular pressure.

added


Consider alternative for missing upstream compile output besides purging from local repository

The problem identified in https://github.com/OpenLiberty/ci.maven/issues/1201 was fixed in
https://github.com/OpenLiberty/ci.maven/pulls/1207.
As noted in the #1561 discussion (here and in the following comments), the #1207 solution has some negatives.
Liberty dev mode doesn't otherwise do anything with installing artifacts to a local repo.  It would be natural for a user to think they could decide when to install or not to install artifacts completely independently from running dev mode.
It is a complicated process by which Maven resolves the artifacts in a scenario like the #1201 scenario, but I think I have a prototype which might solve this problem in a better way.

I was looking at your pull request and found some issues with it:

the call purgeLocalRepositoryArtifact() is still being done
project.getBuild().getOutputDirectory() already ends with classes, no need to append that
When I change this and run the steps from https://github.com/OpenLiberty/ci.maven/issues/1201 output contains the following warnings:

[INFO] --- liberty-maven-plugin:3.6.2-SNAPSHOT:dev (default-cli) @ guide-maven-multimodules-war ---
[INFO] CWWKM2102I: Using artifact based assembly archive : io.openliberty:openliberty-kernel:null:<IP_ADDRESS>:zip.
[INFO] CWWKM2102I: Using installDirectory : C:\Ontwikkel\workspaces\overig\guide-maven-multimodules\finish\war\target\liberty\wlp.
[INFO] CWWKM2102I: Using serverName : defaultServer.
[INFO] CWWKM2102I: Using serverDirectory : C:\Ontwikkel\workspaces\overig\guide-maven-multimodules\finish\war\target\liberty\wlp\usr\servers\defaultServer.
[INFO] Running maven-resources-plugin:resources
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory C:\Ontwikkel\workspaces\overig\guide-maven-multimodules\finish\war\src\main\resources
[INFO] Running maven-compiler-plugin:compile on C:\Ontwikkel\workspaces\overig\guide-maven-multimodules\finish\war\pom.xml
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 1 source file to C:\Ontwikkel\workspaces\overig\guide-maven-multimodules\finish\war\target\classes
[WARNING] /C:/Ontwikkel/workspaces/overig/guide-maven-multimodules/finish/war/src/main/java/io/openliberty/guides/multimodules/web/HeightsBean.java:[50,59] package io.openliberty.guides.multimodules.lib does not exist
[WARNING] /C:/Ontwikkel/workspaces/overig/guide-maven-multimodules/finish/war/src/main/java/io/openliberty/guides/multimodules/web/HeightsBean.java:[61,61] package io.openliberty.guides.multimodules.lib does not exist

These are warnings and not errors because the maven compile mojo is called with failOnError explicitly set to false. Why this would be necessary is not clear to me.
All in all I think you've got an interesting alternative here

@maroschutte thanks for taking a look.  My apologies since I messed up my rebase before pushing (as well as didn't get around to fixing the tests so couldn't rely on that to catch it).  I did fix this if you take a look at the latest; it addresses both of those two bullets.
As far as using the warning vs. failing, the dev mode loop is designed to stay active and give you a chance to fix any compile errors rather than crashing, so that's the idea behind that.


Provider for Windows 386 - 0.60.0 and above

0.60.0
The provider version you are using.
1.2.4
The version of Terraform you were using when the bug was encountered.
Provider not found for windows_386
Since version 0.60.0, terraform complains that there's no package available for windows_386.
I have checked here and couldn't find a matching package.
Is that intentional?
Unfortunately, some of the devs in our project are stuck in Windows.

I am also facing this issue. Error is as below .
╷
│ Error: Incompatible provider version
│
│ Provider registry.terraform.io/snowflake-labs/snowflake v0.63.0 does not
│ have a package available for your current platform, windows_386.
│
│ Provider releases are separate from Terraform CLI releases, so not all
│ providers are available for all platforms. Other versions of this provider
│ may have different platforms supported.

I am facing the same issue.  Is there a solution or work around?

I downloaded the AMD64 version of terraform.exe from  https://developer.hashicorp.com/terraform/downloads and it seems to work with version 0.64.0 of Snowflake provider.  I noticed there was a package for Windows AMD, terraform-provider-snowflake_0.64.0_windows_amd64.zip.  Not sure if this will work for others depending on their CPU.

Hey @hariendemel @tcsumaadm @georgeb-accelins.
386 builds were added back in #2038 (released in v0.70.1). Please upgrade (at least) to this version.

Closing due to long inactivity, please use the proposed version (or preferably the latest one) for windows_386 support.


metadata file "_coursera" not found

I am trying anyint assignment, solver.py ran perfectly. submit.py gives this error:
metadata file "_coursera" not found
_coursera file is in the same directory as submit.py but this error is strange. any idea?

never mind, i figured it out.

glad to hear it.


remove guava dependency

quite often there are CVE that prevent jgitver to be deployed due to guava.
The lib is mainly used for directory/file manipulation in tests. It should be quite easy to replace with pure JRE stuff.

the guava dependency is "test" and therefor the library is not used when using jgitver jar.  should this be closed therefor as not necessary any more?


Feature Request(ImageCacheIt): Support Multiple Filter attributes

It seems {N} does not support filter css atribute. It is possible to parse multiple filter for example.
filter="sepia(.5) contrast(1.25) brightness(1.15) saturate(.9) hue-rotate(-2deg)"

Inspiration: intagram.css filter mixture

Have you tried using it before ? like


inotify-tools: <IP_ADDRESS> -> <IP_ADDRESS>

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for inotify-tools is: ""
meta.homepage for inotify-tools is: "https://github.com/inotify-tools/inotify-tools/wiki"
meta.changelog for inotify-tools is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/fcss6k0nm8xdy2vpdbxhlnv0fqmg520d-inotify-tools-<IP_ADDRESS>/bin/inotifywait had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/fcss6k0nm8xdy2vpdbxhlnv0fqmg520d-inotify-tools-<IP_ADDRESS>/bin/inotifywatch had a zero exit code or showed the expected version
0 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found <IP_ADDRESS> with grep in /nix/store/fcss6k0nm8xdy2vpdbxhlnv0fqmg520d-inotify-tools-<IP_ADDRESS>
found <IP_ADDRESS> in filename of file in /nix/store/fcss6k0nm8xdy2vpdbxhlnv0fqmg520d-inotify-tools-<IP_ADDRESS>

Rebuild report (if merged into master) (click to expand)

19 total rebuild path(s)

17 package rebuild(s)

17 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
CuboCore.coretoppings
appimagekit
dovecot
dovecot_fts_xapian
dovecot_pigeonhole
fragments
gerbera
git-sync
inotify-tools
libtransmission
llpp
nextcloud-client
sshlatex
torrential
transmission
transmission-gtk
transmission-qt

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fcss6k0nm8xdy2vpdbxhlnv0fqmg520d-inotify-tools-<IP_ADDRESS> \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A inotify-tools https://github.com/r-ryantm/nixpkgs/archive/2d8df8cab2cff06eb4bb698ae62b559c2aecec4d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fcss6k0nm8xdy2vpdbxhlnv0fqmg520d-inotify-tools-<IP_ADDRESS>
ls -la /nix/store/fcss6k0nm8xdy2vpdbxhlnv0fqmg520d-inotify-tools-<IP_ADDRESS>/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

17 packages built:

CuboCore.coretoppings
appimagekit
dovecot
dovecot_fts_xapian
dovecot_pigeonhole
fragments
gerbera
git-sync
inotify-tools
libtransmission
llpp
nextcloud-client
sshlatex
torrential
transmission
transmission-gtk
transmission-qt

Maintainer pings
cc @marcweber @pSub @SCOTT-HAMILTON for testing.

@GrahamcOfBorg eval