Add documentation on exit status codes to man pages

This is step one to start defining useful exit codes for all the commands.

What is the purpose of this change? What does it change?
Add documentation on restic exit status codes to all man pages.
Was the change discussed in an issue or in the forum before?
Yes, it's related to #2364 but doesn't resolve the issue.

Checklist

[X] I have read the Contribution Guidelines
[ ] I have added tests for all changes in this PR
[X] I have added documentation for the changes (in the manual)
[x] There's a new file in changelog/unreleased/ that describes the changes for our users (template here)
[ ] I have run gofmt on the code in all commits
[x] All commit messages are formatted in the same style as the other commits in the repo
[x] I'm done, this Pull Request is ready for review

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@604b18a). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #2461   +/-   ##
=========================================
  Coverage          ?   46.58%
=========================================
  Files             ?      174
  Lines             ?    14480
  Branches          ?        0
=========================================
  Hits              ?     6746
  Misses            ?     6732
  Partials          ?     1002

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 604b18a...cfdb701. Read the comment docs.

@rigtorp Thanks for your contribution, a small but good one!
I have a couple of questions:

Restic can return other exit codes than 0 and 1, so could you reword the text to be something like "Exit status is 0 if the command was successful, and non-zero if there was any error." instead?

Is there any particular reason why the "EXIT STATUS" is quoted, when SEE ALSO is not? Should the quotes be removed, or should the other one have them added?

Since this branch isn't merged, you can force-push amended commits to it if you want.

1. Restic can return other exit codes than 0 and 1, so could you reword the text to be something like "Exit status is 0 if the command was successful, and non-zero if there was any error." instead?

No Restic can only return 0 or 1 currently. It would great to improve that in the future, for example to return a specific error code when backup completed with warnings vs backup failed. See #2364

2. Is there any particular reason why the "EXIT STATUS" is quoted, when SEE ALSO is not? Should the quotes be removed, or should the other one have them added?

I've never thought about it. I've always quoted the section headers. I will remove the quotes and verify that the output still looks correct. Better stick to the existing style.

Restic exit status:
https://github.com/restic/restic/blob/master/cmd/restic/main.go#L107
For reference:

http://www.tldp.org/LDP/abs/html/exitcodes.html
https://github.com/openbsd/src/blob/master/include/sysexits.h
https://www.gnu.org/software/libc/manual/html_node/Exit-Status.html

Actually I found this code where Restic can exit with 130:
https://github.com/restic/restic/blob/master/cmd/restic/cleanup.go#L69

Thanks for elaborating :) I'm certainly not always right, very open to being proven wrong.
Yeah, that one 130, and there's also a 2, 3 and 5 if you enable debug.go I think.
If you don't mind I'd like to change the text, because that will also make it more future-proof, for whenever someone forgets to change the EXIT STATUS text properly down the line.

I'm going to change it to this:
Exit status is 0 if the command was successful, and non-zero if there was any error.

0    Successful program execution.
1    Generic error.
130  Program execution was interrupted with SIGINT.

Later we can use errors.Is() to query for IO, DNS and other errors and return according to /usr/include/sysexits.h

Fair enough - after all, having the various possible exit codes documented is what this is about.

Thanks for your contribution! Unfortunately the man pages are automatically generated from the descriptions in the source code, usually before a release (so they are up to date). One example for a man page text is here:
https://github.com/restic/restic/blob/26d1f9f4ba6c106a90e2af9a96df50d8898a9bb5/cmd/restic/cmd_check.go#L23-L27
You can generate the manpages by building restic and then running mkdir manpages && ./restic generate --man manpages. They can be viewed using man like this: man ./manpages/restic-check.1. Inserting a new section works like this (I think it's markdown):
var cmdCheck = &cobra.Command{
	Use:   "check [flags]",
	Short: "Check the repository for errors",
	Long: `
The "check" command tests the repository for errors and reports any errors it
finds. It can also be used to read all data and therefore simulate a restore.

By default, the "check" command will always load all data directly from the
repository and not use a local cache.

EXIT STATUS
===========

Exit status is 0 if command was successful, and 1 if there was any error.
`,
	DisableAutoGenTag: true,

@fd0 Eventually the exit status will diverge between the commands. In particular I want to make the backup command return different status depending on if backup completed without warnings or it failed. Currently it returns 0 when backup completed with warnings.

@rigtorp Maybe it would be a good idea to make that (separate exit codes) a new issue for tracking, and fix this PR to reflect the current state with @fd0's comments above in mind, so we can merge this.

@rawtaz Yes that's what I had in mind.

@fd0 I have addressed all concerns. Specifically I added a note about current behavior of backup command.

@rawtaz Friendly reminder!

@rawtaz @fd0 Can we merge this?

Looked at it earlier today, just waiting for a verification on something before we merge.
But we should remove the changelog file entirely. I'll do that if it's not done by whenever I merge it.

The following needs to be cleaned up, then this can be finally merged:

All changes to files under doc/man/ should be reverted/removed (man pages are generated upon release).
All changes to files under changelog/unreleased/ should be reverted/removed (not needed for doc changes).
Squash commits into one.


Add ruleset for helper classes

We've been discussing the addition of a new ruleset for helper classes, which would allow developers to pick and choose which helpers they want to include in their CSS.  Let's use this issue to track the helpers we want to add.
@thierryk @renatoi

Border helpers.  From issue #76:

For the record: helpers.css (from another repo) has been updated with border classes meant to help when styling boxes with a single pixel border but those rules need to integrate Atomizer somehow.

More details from @thierryk:

border needs 3 values: width style color. We can get style borders the way we want with Atomizer by using those 3 but a common use case is to style border with  a single pixel, so what we added to the helper classes are border declarations mixing both style and width

that way authors can do this: Bd Bdc-000 to get a black border around a box instead of having to go verbose with Atomic Bdc-000 Bds-s Bdw-1px

It would be nice to allow this helpers to be expecified on the configs or to allow 3rd party to create node_modules that extend atomizer, we should make easy for people to create their own helpers.

@3den: Yes, that's the plan!  You will be able to add new rulesets for whatever custom helpers, etc you might want.

Let's make sure we allow namespacing of those too; but with an arbitrary string, not necessarily the same as the one used for "Atomic classes".

All helpers have been added to 3.0.0-alpha.4. Custom rulesets have been supported for a while as well (not sure since when).


Add Hellang.Middleware.SpaFallback.

By default, this added middleware will serve the index.html page.
To not lose the current page URL, I've removed the redirect in the router.
Worked on my machine, flies away now.
Fixes #43.

I will be releasing a new version for this shortly thanks for adding this!

Thanks, take your time though ;)


CTRL clicking item pulls one single item out of the system

A feature from AE that I miss a lot. Like when you normally click it pulls a stack out, if you're holding control it would only pull out one item.

In RS you use middle click

Can ctrl+click move a single item to your inventory then? Like a shift-click but with only one item?

Use shift + ctrl click :p

If you middle click more than once it doesn't give you multiple items FYI

Now that would actually be a very good improvement. It really should give you an item on your cursor every time you do it, in my opinion.

I could do that, yeah.
@RealGrep Btw, real quick, is the issue with the machines dying fixed?

I've got the new version in, but haven't ran any tests yet. I'll remove the chunk loaders temporarily and run some tests later tonight and let you know in the "Solderer dying" PR. Thanks for all the excellent work. :)


SPAWN_WRAP_SHIM_ROOT variable setup requires spawn-wrap update

This is an enhancement request!
Expected Behavior
expected nyc to allow user set to a path where spawn-wrap shim files
Observed Behavior
nyc errors out not able to create spawn-wrap temp directory on virtual machine with read only permissions to $HOME directory.
Error: EACCES: permission denied, mkdir '/home/****'
     at Error (native)
     at Object.fs.mkdirSync (fs.js:922:18)
     at sync (******/node_modules/nyc/node_modules/mkdirp/index.js:71:13)
     at Function.sync (******/node_modules/nyc/node_modules/mkdirp/index.js:77:24)
     at setup (******/node_modules/nyc/node_modules/spawn-wrap/index.js:405:10)
     at wrap (******/node_modules/nyc/node_modules/spawn-wrap/index.js:53:18)
     at Object.<anonymous> (******/node_modules/nyc/bin/nyc.js:57:3)
     at Module._compile (module.js:570:32)
     at Object.Module._extensions..js (module.js:579:10)
     at Module.load (module.js:487:32)

Bonus Points! Code (or Repository) that Reproduces Issue
nyc uses spawn-wrap version 1.3.8 as bundled dependency, but with the latest spawn-wrap @1.4.1 provides an ability to set SPAWN_WRAP_SHIM_ROOT variable to change .node-spawn-wrap-* directory location
Forensic Information
Operating System: Linux.
$ nyc --version
11.3.0
$ node --version
v8.9.0
$ npm --version
5.5.1

unfortunately due to our corporate policy, code or package details are not being reported.

i did update my fork with changes and tested it out, https://github.com/istanbuljs/nyc/compare/master...sridharmallela:master
Will submit the PR, if you are ok with me contributing or my changes look good.

@sridharmallela please take this for a spin:
npm i nyc@next -- you should see 11.4.0 with the updated spawn-wrap; this will go out soon once it's been tested on the next tag for a bit.

@bcoe thanks for getting on to it, seems like there is an issue with spawn-wrap with nyc not able to run in windows or mac os. I have created an issue, may be we might need to delay the release until it is fixed.


Install Agent script

curl -sfL https://get.k3s.io | sh - works great for the controlplane.
It would be very useful if there is something similar for installing k3s (with systemd service) as agent.

@ewoutp Agreed, we will work on the corresponding approach for the agent.  It will probably be something like
curl -sfL https://get.k3s.io | env K3S_SERVER=... K3S_TOKEN=... sh -

Looks good

@ewoutp The updated script for this PR can be found at https://raw.githubusercontent.com/erikwilson/rancher-k3s/install-agent/install.sh for testing.
I have also created a short url for the script: https://goo.gl/NMdUme
For basic use we can install the server without agent like:
curl -sL https://goo.gl/NMdUme \
  | INSTALL_K3S_EXEC="--disable-agent" sh -

And to install the agent without a server:
curl -sL https://goo.gl/NMdUme \
  | K3S_URL=https://server-url:6443 K3S_TOKEN=XXX sh -

The script also has a lot of extra functionality regarding which version to download (or no downloads), the locations of places to put files, and cleaning up processes and networks on uninstall.
Please see the head of the install script for all of the usable environment variables.
Any help with testing to ensure it works as expected is greatly appreciated.

just install k3s-agent can try this
curl -sfL https://get.k3s.io | K3S_TOKEN=... K3S_URL=... sh -


feat: ASH function for pledge allegiance effect for location

Add function that can be used with pledge_allegiance_effect(my_location()) to see if you want to pledge at your current location.
Outputs a string, modifier-like (but with "All Res" instead of spelled out), of the effect you would get. If there's a nicer way of doing this, let me know: I wanted to make it nice for human readers.
Targeted at people like me who use the GCLI a lot. I expect other people will use Guide or a script or whatever.

I think you make a command print text, an ash function should be usable by
scripts
On Sat, 1 Jul 2023, 11:52 Chris Midgley, @.***> wrote:

Add function that can be used with pledge_allegiance_effect(my_location())
to see if you want to pledge at your current location.
Outputs a string, modifier-like (but with "All Res" instead of spelled
out), of the effect you would get. If there's a nicer way of doing this,
let me know: I wanted to make it nice for human readers.
Targeted at people like me who use the GCLI a lot. I expect other people
will use Guide or a script or whatever.
You can view, comment on, or merge this pull request online at:
https://github.com/kolmafia/kolmafia/pull/1797
Commit Summary

c55f93b
https://github.com/kolmafia/kolmafia/pull/1797/commits/c55f93b44bc58339355134acb7d26f7ab1ed3002
feat: ASH function for pledge allegiance effect for location

File Changes
(2 files https://github.com/kolmafia/kolmafia/pull/1797/files)

M src/net/sourceforge/kolmafia/textui/RuntimeLibrary.java
https://github.com/kolmafia/kolmafia/pull/1797/files#diff-a7b6e906c6d61b624d212171a00faf4cc250872dfe45e863e109eee249643d40
(35)
M test/net/sourceforge/kolmafia/textui/RuntimeLibraryTest.java
https://github.com/kolmafia/kolmafia/pull/1797/files#diff-7cc6e1f829c1a3908b19f29139bf987fdb597525ecec7125429363925d789405
(27)

Patch Links:

https://github.com/kolmafia/kolmafia/pull/1797.patch
https://github.com/kolmafia/kolmafia/pull/1797.diff

—
Reply to this email directly, view it on GitHub
https://github.com/kolmafia/kolmafia/pull/1797, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABL4LRUSKBWEMBQNMWVB3DXN76QLANCNFSM6AAAAAAZ2XHWUM
.
You are receiving this because your review was requested.Message ID:
@.***>

It's a function that returns a string. I figured scripts would do their own thing because they're going to want to format it differently... but maybe not?

I think maybe make it return a map of modifier to value like the voting
booth modifiers function?
On Sat, 1 Jul 2023, 15:59 Chris Midgley, @.***> wrote:

It's a function that returns a string. I figured scripts would do their
own thing because they're going to want to format it differently... but
maybe not?
—
Reply to this email directly, view it on GitHub
https://github.com/kolmafia/kolmafia/pull/1797#issuecomment-1615950780,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABL4LQH6RJZ3UF7EBVKATDXOA3N5ANCNFSM6AAAAAAZ2XHWUM
.
You are receiving this because your review was requested.Message ID:
@.***>

I think you make a command print text, an ash function should be usable by scripts

I like this distinction but cannot recall an explicit articulation of that before.  Thank you.
I am somewhat clueless about the new IOTM and what has already been implemented.  But AFAIK pledge_allegiance_effect(my_location())  does nothing but return a string about what would be expected to happen if allegiance was pledged.  Print that string and this feels like a command.
I can see providing similar information via an Adventure proxy record or just reverse engineering the deterministic calculation to generate a list of potential locations for an effect and then checking for location availability.
If the question is - I'm already here shall I pledge? then this works but it needs to be more script friendly if the question is - where should I go or open to get a desired effect?
I have no issues with approving but feel like it is a niche solution that will almost certainly need to be augmented,

If I understand how the new IOTM works:

Once a day, you can Pledge Alliegance to a location.
(I don't know how you do that. Ask your familiar, somehow?)
You get an effect for the rest of the day. It's not an intrinsic, but it may as well be; you can't uneffect it.
(Citizen of a Zone, perhaps?)

Now, once you have gained this effect, I expect we'll give it variable modifiers, as we do for other objects, and it it will show up in modtrace, string_modifiers(effect, "Modifiers"), etc.
This PR proposes a way to speculate on what the modifiers will be for that effect given a particular location.
Returning a string is essentially what the aforementioned string_modifier would do for that effect, assuming you'd already baked in the location.
How about if modifiers.txt had a "Type" -> "Citizen" -> with keys of "0" -"9". I.e., snarfblat modulus.
So, internally, "Citizen:5" is a modifier set.
You could coerce that into a modifier sat - like voting booth initiative.
You can generate a "Modifiers" string - and string_modifier would find it.
You want a string which can be printed?
Sounds like a command could map location to modulus, fetch the "Modifiers" string, and print it.
Either a runtime method or a proxy record field could return a modifier set (like voting booth initiatives).
A command seems good in either case, but I have no idea how a script would want to have the data structured.
Probably in a modifier map.
I dunno.

Yeah. I think I like exposing it to scripts via $location.pledge_alliegance (or something) returning a "Modifiers" string corresponding to the snarfblat modulus of that location.

Yeah. I think I like exposing it to scripts via $location.pledge_allegiance (or something) returning a "Modifiers" string corresponding to the snarfblat modulus of that location.

This sounds very practical. And, honestly, not all that unreadable for GCLI users (of which I am also one!)

Seems reasonable. I did initially think about dumping it into the proxy record, but I thought if we did that for everything it would get rather large. But that doesn't mean it's a bad place for it: it's quite convenient to see it there with everything else.

I've thrown it in. Some comments from this:

I think it might be nice to have a $modifier[] type, for use with the various methods, to confirm what you're passing in is both a valid modifier and passed in to the correct parameter
it's not clear to me how to go from a string list of modifiers (or a map, as voting booth has) to extract a specific modifier value. In Java, we have ModifierDatabase.splitModifiers which gives you a ModifierList from which you can getModifierValue. Looking at garbo, it compares keys as literal strings (which is fine).
ModifierValue takes two strings and doesn't force the first to be a modifier, and it sometimes isn't -- e.g. Voting Booth has Weapon Damage Unarmed which we never added.

I would definitely be in support of $modifier. It'll break some old code
potentially but it's a great idea
On Sun, 2 Jul 2023, 08:55 Chris Midgley, @.***> wrote:

I've thrown it in. Some comments from this:

I think it might be nice to have a $modifier[] type, for use with
the various methods, to confirm what you're passing in is both a valid
modifier and passed in to the correct parameter
it's not clear to me how to go from a string list of modifiers (or a
map, as voting booth has) to extract a specific modifier value. In Java, we
have ModifierDatabase.splitModifiers which gives you a ModifierList
from which you can getModifierValue. Looking at garbo, it compares
keys as literal strings (which is fine).
ModifierValue takes two strings and doesn't force the first to be a
modifier, and it sometimes isn't -- e.g. Voting Booth has Weapon
Damage Unarmed which we never added.

—
Reply to this email directly, view it on GitHub
https://github.com/kolmafia/kolmafia/pull/1797#issuecomment-1616431213,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABL4LR4FEIYD75QVOBDMFLXOESPJANCNFSM6AAAAAAZ2XHWUM
.
You are receiving this because your review was requested.Message ID:
@.***>


server,engine-schema: fix listing service offering for vm scale

Description
Fixes #7389
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

@harikrishna-patnala  cc @sbrueseke this PR fixes the listing of the current custom offering but now it just lists offerings that use the same disk offering when disk_offering_strictness=true. Is this correct behaviour?

yes @shwstppr I think we are enforcing this service layer, it can be while listing the offering as well.
Since disk offering strictness is true then root disk offering cannot be changed, so makes sense

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan test

@shwstppr a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@blueorangutan test

@shwstppr a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-6383)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 46600 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7398-t6383-kvm-centos7.zip
Smoke tests completed. 108 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

@weizhouapache done

it looks like this fix was left out of <IP_ADDRESS>?!?


Hello, I have changed a camera which is DMK 23GP031. Got those error!

Hello, I have changed a camera which is DMK 23GP031. Got those error!
pi@raspberrypi:~ $ gst-launch-1.0 tcambin ! videoconvert ! ximagesink
Setting pipeline to PAUSED ...
Pipeline is live and does not need PREROLL ...
INFO:
Working with src caps: video/x-raw, framerate=(fraction)400000/57327, width=(int)2592, height=(int)1944, format=(string)GRAY16_LE
Setting pipeline to PLAYING ...
New clock: GstSystemClock
ERROR: from element /GstPipeline:pipeline0/GstTcamBin:tcambin0/GstTcamSrc:tcambin-source: Device lost
Additional debug info:
/home/pi/Desktop/tiscamera/src/gstreamer-1.0/gsttcamsrc.cpp(1045): gst_tcam_src_device_lost_callback (): /GstPipeline:pipeline0/GstTcamBin:tcambin0/GstTcamSrc:tcambin-source
Execution ended after 0:00:03.493881237
Setting pipeline to PAUSED ...
Setting pipeline to READY ...
Setting pipeline to NULL ...
Freeing pipeline ...
Originally posted by @StefanHope in https://github.com/TheImagingSource/tiscamera/issues/81#issuecomment-479823077

@StefanHope The error says "device lost" which means that the camera disconnected from the usb bus. This might be due to a faulty USB cable or - especially on the RaspberryPi - due to lack of Power. Are you using a powered USB bus with this camera? The Pi might not provide enough power by itself.
Also keep in mind that the Pi only has USB 2.0 ports. You need to set a way lower frame rate to get the camera working with only a USB 2.0 connection.

DMK 23GP031 is a GigE camera.

By the way, pi 3B+ support Gigabit Network。

My bad. I did not look close enough. In this case you should try setting a lower frame rate and only use the 8 bit GRAY format to lower the datarate. The ethernet port of the Pi is connected via USB 2.0 only and the high datarate will clogg the network connection which causes the software to think the device got lost.

Thanks so much. Could you please give me some instructions for how to set in a lower frame rate?

I am using banana pi and HDF5640 camera. I changed video0/1/2 but only video1 is showing the correct value. so video port is 1.

I am afraid, you are in the wrong issue.


Update bulk send to support versioned txns

/canary-release

/canary-release

/canary-release

Closing because this would require oracle support as well


Chapter 17: Imaging

Fierry137

@Fierry137 will this cover imaging computers, mobile, other, all the above, etc? Just trying to figure out which section this chapter might fall into.

It's probably going to be limited to computers and RAM memory. I think mobile imaging might fit better in one of the mobile chapters due to the various limitations (OS, manufacturer, lock bypassing and physical/logical imaging differences)

@Fierry137 can you make a chapter17.txt file which is what Leanpub parses to generate the chapter?

Sure, I can create it in the coming days (currently on holiday abroad without laptop) and instantly merge any feedback that is generated when the editing is completed.

If you've not already, please consider filling out your bio here before the end of the month: https://github.com/Digital-Forensics-Discord-Server/TheHitchhikersGuidetoDFIRExperiencesFromBeginnersandExperts/blob/main/manuscript/authors.txt

As we approach publishing v1 of this book, please review the most recent files uploaded to this folder and ensure your chapter looks the way you want it to. If not, please make appropriate adjustments and advise me to regenerate the preview book files.
Additionally, if you've not been invited to the book on Leanpub yet, please provide me with an email address so I can invite you on there.
Thank you for completing your chapter! I hope this has been a fulfilling experience for you!

@Fierry137 I added my edits: 27b98f9
It seemed like the chapter ended abruptly? It was good stuff but just wondering if you were done or needed to add a conclusion still? Please advise when you can!

I think a conclusion is in order, I’ll add it in the coming days :)

A conclusion and some next steps have been added :)

A conclusion and some next steps have been added :)

I just finished reviewing this, check out the edits! Let me know if you have any feedback! Cheers and thank you for finishing your chapter. Will get you added to the ISBN here shortly.


Improvement suggested on OSX installation doc

This is a suggestion to include moving the executable bin file to /usr/bin, as 3rd party libraries will try to execute it directly.

I think we should leave out any opinionated or recommended way to discover the executable. For example, I know a lot of advanced OS X users who don't want to tamper the system-wide directories with third-party executables (putting them in ~/bin).

Sure, it is a rather opinionated way to make it available. But I also think it could help a lot to beginners.
Advanced users can simply ignore the advice and place the bin wherever they want, it just feels to me that those who don't know how to make that bin available system wide could need further instructions.
Maybe a less opinionated way? A message telling to add the bin to their path?

While this is true, how is this specific to PhantomJS anyway? Anytime someone downloads an executable and wants to make it available system wide will follow the same usual steps. There must be some guide on "Using command-line tools on OS X" or something similar that we can recommend.

Just my 2 cents but

the guide really should mention to just use Homebrew, that's the defacto standard to install stuff on Mac OS X nowadays and the formula is well maintained. Maybe mention the --HEAD option for the latest ref if needs be.
+1 for not telling the users to "just sudo mv to /usr/bin", it's adding to the confusion since it's not explaining in any way why there and not for instance in ~/bin or in /usr/local/bin (which wouldn't require you to use sudo)

While Homebrew is nice (and I use it a lot), we try not to mention things we hardly control. In the past I did that and we always end up explaining why the version was not updated etc.

We will leave the installation steps as is for now.


Feature request: gamepad support

Personally I'm specifically interested in the Xbox 360 controller, Steam controller, and Thrustmaster T.Flight HOTAS 4 - but that's because I'm a selfish person and those are the controllers I own. :)
I mentioned this in #5 already but it's worth saying again: your overlay is good and you should feel good for making it. Thank you!

I have added support of gamepad in new release. I tested it with my XBox Series X|S Controller, but it should work fine with 360. It may also work with Steam controller, you can try (but touch pad is not supported).

Did some brief testing with my 360 controllers - it looks like if both the right and left triggers are held down at the same time neither one will light up. They work fine independently, and if I have for example right trigger fully pressed and start to slowly press left trigger, the right trigger highlight will fade to dark.
Seems like maybe they're acting as opposite ends of an axis, so fully depressing both displays as 'neutral' but that's just my guess :)
I seem to have misplaced my Steam controller dongle but if I can find it I'll do a little testing there too.
Is there a way to have two separate overlays for two separate gamepads? This is probably outside the scope of the original request so if it makes more sense to be a separate ticket let me know, but I sometimes play co-op games with my kids and being able to show multiple controllers would be pretty cool. :)

That is what gamepad driver reports, these triggers act like single axis, and driver reports single value. I will investigate if it possible to change "mode" and say driver to report data in a different way. It seems like some games use triggers as independent buttons, and can understand when player hold them both.
I implemented support for multiple gamepads, check https://github.com/Zergatul/Zergatul.Obs.InputOverlay#advanced-usage , paragraph 6. However HDevice will change if you restart PC, and probably if you turn off/on gamepads. Maybe I should find a better way to identify gamepads.
BTW we came up with better design for gamepad overlay, you can try it out (but these changes are not in the main branch yet). Download default-gamepad.html and XBoxSeriesXController.svg from here https://github.com/Zergatul/Zergatul.Obs.InputOverlay/tree/feature/gamepad/src/wwwroot

Oh the triggers really do share an axis? Interesting!

some games use triggers as independent buttons, and can understand when player hold them both
This is what confused me, I assumed they could be recognized separately, no idea how difficult it would be to do that though. Not sure how common it is to be pressing both triggers at once anyway, just seemed odd :)

https://github.com/Zergatul/Zergatul.Obs.InputOverlay#advanced-usage , paragraph 6
Whoops, I must have missed that, thanks! That's probably sufficient for my purposes, although having to reconfigure it every reboot and any time controller plugs are changed is unfortunate :p

better design for gamepad overlay
Wow I just checked it out (had to change the object reference at the bottom of the html) and that's way better, much better use of the space! I feel like the triggers could maybe even be slightly smaller, but since they shrink down when pressed (cool animation btw) it all fits together nicely.

You can check in gamepad settings in Windows, that 2 triggers share single axis, called Z. Open "Bluetooth & other devices", scroll down to "Related Settings", click "Devices and printers", right click on gamepad, select "Game controller settings", select controller and click "Properties". It displays all axes and buttons reported by driver.
Thanks for sharing feedback!

You can check new release. If you have XInput compatible gamepad (like any XBox gamepad), you can use XBoxSeriesX-XInput.html. Also it is much simpler to use multiple gamepads with XInput. I updated docs: https://github.com/Zergatul/Zergatul.Obs.InputOverlay/blob/master/docs/MultipleGamepads.md

I've been meaning to try this with all 4 of my controllers (three 360 controllers and one Steam controller) but I've misplaced the dongle for my Steam controller, and then got sidetracked with other things - sorry for the belated response!
Anyway, at least with 2 Xbox 360 controllers using the XInput configuration method this appears to be working great! The shoulder triggers display individually and simultaneously without issue.
One issues I noticed is that it looks like if I have a gamepad overlay configured (but not active) in OBS but no gamepad plugged in at the moment I run Zergatul.Obs.InputOverlay.exe the overlay crashes.
Steps to reproduce this:

Configure gamepad overlay(s) in OBS
Close InputOverlay.exe, close OBS, unplug all controllers
Launch OBS
Launch InputOverlay.exe

Expected result: InputOverlay should run without crashing; gamepads should display properly once plugged in
Actual result: InputOverlay crashes after displaying the following message:
Unhandled exception. System.NullReferenceException: Object reference not set to an instance of an object.    at Zergatul.Obs.InputOverlay.XInput.XInputHandler.PollingThreadFunc() in C:\Users\Zergatul\source\repos\Zergatul.Obs.InputOverlay\src\XInput\XInputHandler.cs:line 70

Good!
May I ask, when you had problem with 2.2.0, did you have high CPU game running in background along with OBS? Do you use Windows 10 or 11? What is your CPU model?
We tested 2.2.0 version with no gamepads, on 3 different systems many times and this bug never appeared.

Oh interesting - I may have had a couple things running at the time, including an idle game I often have in the background, but CPU usage should have been below 30% I think.
My specs:
Intel(R) Core(TM) i5-7600K CPU @ 3.80GHz   3.79 GHz
16GB RAM
Win10 Home x64
2.2.1 has been working flawlessly so far, but I'm happy to help do any additional troubleshooting on 2.2.0 if that would be helpful, just let me know. :)
If you're on discord feel free to message me @ Nomad#6589 for quicker responses, I don't check my email very often :p

Thanks! No need for additional testing 2.2.0, there was a bug and we fixed it. This is just my curiosity :)


Video shortcode

Inspiration > https://github.com/RoneoOrg/hugo-shortcode-roneo-collection/blob/main/layouts/shortcodes/video.html

[x] native video file embed - html5
[x] youtube video
[ ] vimeo video (in progress) :

{{< vimeo id="727145223" image="/uploads/resizeallimages.png" >}}

resources ^^ : https://hugocodex.org/add-ons/vimeo-shortcode/

This shortcode requires some testing.

Okey !
Gonna  go for

[ ] requires some testing
[ ] and a usage example - ++ doc.  now...

PS: gonna ask the guy - author (french guy) . about licensing... to that we include it, as you suggest

Vimeo is currently not available as a shortcode. This is postponed for now, until we have all current features documented properly.

I have added vimeo as a shortcode.

Yup, Joost...
I see you are Hugobicking... :-)
Let's talk tomorrow, at around 9h ?
good night !
F
Oct 31, 2024 10:55:03 PM Joost van der Schee @.***>:

Closed #30[https://github.com/jhvanderschee/hugobricks/issues/30] as completed.
—
Reply to this email directly, view it on GitHub[https://github.com/jhvanderschee/hugobricks/issues/30#event-15048606281], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AAO775GFZZMWPA7IYWBRA7DZ6KRLLAVCNFSM6AAAAABQ7GIQS2VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJVGA2DQNRQGYZDQMI].
You are receiving this because you commented.
[Tracking image][https://github.com/notifications/beacon/AAO775DWRVWNOI73AU5PP2LZ6KRLLA5CNFSM6AAAAABQ7GIQS2WGG33NNVSW45C7OR4XAZNWJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XKUY3PNVWWK3TUL5UWJTYAAAAAHAHXQJEQ.gif]


Missing URL for Log Analytics Agent

Hi All,
Please can you look at adding the following URL to the list of required ones that are needed for the Log Analytics agent to work as expected.
*.agentsvc.azure-automation.net
When running the TestCloudConnection.exe tool which is part of the Agent (under C:\Program Files\Microsoft Monitoring Agent\Agent) the above URL is mentioned as needing to be opened.
Thanks
TheChad

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

ID: 6b71519d-1b94-c489-ce51-7210eb05b109
Version Independent ID: fb6adce7-aa81-28e9-ad54-8c90dd86afa2
Content: Log Analytics agent overview - Azure Monitor
Content Source: articles/azure-monitor/platform/log-analytics-agent.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @bwren
Microsoft Alias: bwren

@TheChadR Thanks for the feedback.
Assigning content owner @bwren for visibility, review and document enhancement consideration as appropriate.

#label:"CreateAzMonWIFeb2021"

Thank you for your contribution to the Azure Monitor documentation. We have created a workitem for this issue in our internal backlog database. We will prioritize it and make any necessary changes as we are able.
#please-close


Incompatibility with mod Thicker Power Wires

Hello,
Just wanted to let you know that there is an issue with the this mod.
For some reason the colors get changed, see images below:
Without Thicker Power Wires

With Thicker Power Wires (with same colors as above)

With Thicker Power Wires (with differents colors as above)

It's look like this mod keep changing the color but they got messed up by Thicker Power Wires mod.
I don't know if this can be fixed

Thanks for the report, it's now fixed. I'll be uploading to the portal shortly.

Ah and P.S. @lachlanmcdonald I've used your graphics for this compatibility patch, it's just color shifted. You are credited here - please let me know if you would prefer it elsewhere.

This is fine, thanks for the notice @oorzkws.

Thanks both of you


sort BlockTx

sort BlockTx group by sender order by average gas descending

add a config to enable tx sort;  sort timeout make longer.


[website][common]common util for generate docs

Master Issue: #10040
Motivation
Support auto generate HTML page for pulsar client cli tool, for example: https://github.com/apache/pulsar/tree/asf-site/content/tools/pulsar-admin
Modifications
common util for generate docs
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)
Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes
Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)
Documentation
Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

/pulsarbot rerun-failure-checks

@tuteng Some PRs of generate docs rely on this PR, PTAL, thanks


Suggestion: Rename environment variable IDF_PATH (IDFGH-49)

The ESP-IDF framework is using the environment variable IDF_PATH.  I can't help but feel that this is too "generic".  I was watching a colleague look at some of my ESP files today and many of my variables were ESP_XXX except IDF_PATH.  If I were to make a suggestion, it would be ESP_IDF_PATH.  It just feels to me that our ESP32 core prefix is "ESP" and if that were the case, ESP_IDF_PATH would fit in that story just fine.

Yes I'm totally agree ESP_IDF_PATH will be more adapted

Closing this issue. I agree this argument makes some sense, but we're pretty standardized on IDF_ prefix and especially IDF_PATH is now very widely used - so we're unlikely to change it.

Closing this issue. I agree this argument makes some sense, but we're pretty standardized on IDF_ prefix and especially IDF_PATH is now very widely used - so we're unlikely to change it.


Persist energy on Controller break

Still have to make it so it renders the correct texture for the energy level...


Move column test cases from climacore repo

Initial move of ClimaCore test cases.
Co-authored-by: Ben Mackay<EMAIL_ADDRESS>Co-authored-by: kmdeck<EMAIL_ADDRESS>Co-authored-by: Gregory L. Wagner<EMAIL_ADDRESS>Test case 3 still needs to be optimized for a better interface.

@LenkaNovak do we need to do additional work with the ClimaCore examples for this PR?
Looking ahead, the ClimateMachine tests will break when we update the interface. We should discuss if we want to maintain some of them. The ClimaCore cases are currently independent of the CouplerMachine source but should eventually use the CouplerMachine.


Add validation for textarea and datepicker. Update the sample app.

I have made some modification for displaying the error on textarea and datepicker elements with aurelia-validation, which I need in my app. I have also update the sample app with some example for the select, textarea and datepicker elements.
The only problem I get is that the datepicker error doesn't add/remove when changing the value, but only if we click again on the input field or if we trigger the validation manually. But I think that is a problem on the way the value is changed on a datepicker because it doesn't trigger the render function of the ValidationRenderer when changing the value.

Coverage remained the same at 95.154% when pulling 55164a87490145d53d6238bb8717a15fb474a8f2 on Ganbin:master into 03d3487e30b9707771a507bb308181a6e68da8fb on aurelia-ui-toolkits:master.

Thanks. 👍
Yeah, pickadate has some "focus-magic" going on. I think it doesn't blur for some reason..
If you use our commit message conventions your changes will appear in the changelog automatically. I'll edit the log for now.

Hoo yes... I haven't check that I was in the input section. -_-' So I should change the datepicker validation example in the datepicker section too. Sorry for the misunderstood.

I'm not used to contribute to project so for me is like a training and learning so sorry if I do mistake. I will read the commit conventions and try to do it the right way.

No problem at all. 😃
Datepicker validation should go onto datepicker page, yes.

Coverage remained the same at 95.154% when pulling e7b33ae3dc33005854b58d4684072463440394e9 on Ganbin:master into 03d3487e30b9707771a507bb308181a6e68da8fb on aurelia-ui-toolkits:master.

I have made some modification.
Ability to have a showErrortext bindable to the md-datepicker. The same way you have done with the md-select. Then I have change the code in the renderer for the md-datepicker to check the attribute.
I have removed the example on the input section and add a validation example on the datepicker section.

Looks good to me. 👍
The date picker error messages cause an error, though: Uncaught Error: focus is not a function and navigate to the start page.
If you know how to fix it, this would be great but I'm not sure if it is. Maybe create a view model function which opens the date picker.
Also, reset doesn't seem to do anything. I guess it does reset the controller but it doesn't clear the datepicker input. I'll comment on the source in a few minutes (it's more readable that way).

Coverage remained the same at 95.154% when pulling ae500d5ff3b1dced0548db9c8fe6c329654ff1b6 on Ganbin:master into 03d3487e30b9707771a507bb308181a6e68da8fb on aurelia-ui-toolkits:master.

@Ganbin Not sure if you received the notifications in the code comments because they are hidden now. 😃

Ok I see your comments.

Ok here is the fix. I think the rendering is good and I don't see unexpected behaviour if we follow the html and javascript code on the datepicker validation in the sample app.
Thanks a lot for the help to resolve the issues in my pull request.

Merged and released, thank you! 👍
I will upload the catalog app with samples a bit later.


⚠ Site has degraded performance

In 3214ac5, Site (https://fyle.uk) experienced degraded performance:

HTTP code: 503
Response time: 15541 ms

Resolved: Site performance has improved in 378c2d8 after 2 hours, 38 minutes.


Add Dubbo Response Code Mapping

Issues associated with this PR
Dubbo Response Code Mapping is missing.
Solutions
Add and Registry Mapping
UT result
Unit Test is needed if the code is changed, your unit test should cover boundary cases, corner cases, and some exceptional cases. And you need to show the UT result.
Benchmark
If your code involves the processing of every request, you should give the Benchmark Result.
Code Style

Make sure Goimports has run
Show Golint result

Add unit-test.

outdated pull request. if you still need this changed, make a new one

outdated pull request. if you still need this changed, make a new one


CAS logout return link broken

The URL being provided by the CAS client for Cyder is incorrect. It should be https://cyder.nws.oregonstate.edu, but currently looks like: https://cyder.nws.oregonstate.eduhttp//cyder.nws.oregonstate.edu

I can't find the exact reason the next parameter defaults to the url for the site, and trying to override it with CAS_REDIRECT_URL doesn't work (which means the next value is in fact, set).  A hack to make it work for now would be to change the logout URL for the logout button have ?next=/ tacked on to the end.


Error Creating Abstract Class That Extends V In Version 2.2.0

Expected behavior and actual behavior
Expected behavior is to create an abstract class that extends V. I can create a class that is abstract, or a class that extends V, but not both. This is part of a setup script and worked perfectly in version 2.1.x.
Produces the error:
Error: com.orientechnologies.orient.core.sql.OCommandSQLParsingException: Error parsing query:
create CLASS SomeClassV ABSTRACT EXTENDS V
                                       ^
Encountered " <EXTENDS> "EXTENDS "" at line 1, column 33.
Was expecting one of:
    <EOF>
    ";" ...

        Storage URL="plocal:/opt/orientdb-community-2.2.0/databases/my-database"
        Storage URL="plocal:/opt/orientdb-community-2.2.0/databases/my-database"

The error message shows plocal as the connection mode, even though I am using remote.
Steps to reproduce the problem
sudo ./console.sh
CONNECT remote:localhost/my-database database-user database-password
CREATE CLASS SomeClassV ABSTRACT EXTENDS V;
Important Questions
Runninng Mode

[ ] Embedded, using PLOCAL access mode
[ ] Embedded, using MEMORY access mode
[ x] Remote

Misc

[ ] I have a distributed setup with multiple servers. How many?
[ ] I'm using the Enterprise Edition

OrientDB Version

[ ] v2.0.x - Please specify last number:
[ ] v2.1.x - Please specify last number:
[ x] v2.2.x - Please specify last number: 2.2.0

Operating System

[ ] Linux
[x ] MacOSX
[ ] Windows
[ ] Other Unix
[ ] Other, name?

Java Version

[ ] 6
[ ] 7
[ x] 8

Produces the error:
Error: com.orientechnologies.orient.core.sql.OCommandSQLParsingException: Error parsing query:
create CLASS SomeClassV ABSTRACT EXTENDS V
                                       ^
Encountered " <EXTENDS> "EXTENDS "" at line 1, column 33.
Was expecting one of:
    <EOF>
    ";" ...

        Storage URL="plocal:/opt/orientdb-community-2.2.0/databases/my-database"
        Storage URL="plocal:/opt/orientdb-community-2.2.0/databases/my-database"

The error message shows plocal as the connection mode, even though I am using remote.

Hi @decafmainline
In v 2.2 also DDL commands are parsed with the new strict grammar, so you have to respect the order of keywords (see syntax here http://orientdb.com/docs/last/SQL-Create-Class.html)
The correct statement is
CREATE CLASS SomeClassV EXTENDS V ABSTRACT;

Thanks
Luigi


Add polish translation

:)

LGTM! Thanks :)

Just FYI, don't forget to add the language to the po/LINGUAS file as well.
I took care of it for you this time, so no worries: https://github.com/nlogozzo/NickvisionMoney/commit/7b477348045c29cce98dfc080582f828f12fa9b8


Change the integration tests.

The new test specification is now more in line with our other integration tests. We no longer use the shell script either.
Other changes:

Add labels to kubernetes resources to allow us to query them.
Remove unused internal services code.

Nice! Looks awesome!

Thanks Robert.


Implement a 2-point minimum jerk trajectory generator in SO(3)

Disclaimer: this PR includes also the modification implemented in #82. This main introduction of this PR is:  https://github.com/dic-iit/bipedal-locomotion-framework/commit/9c3c9a0ad4e602381fdcfa588d4220fa9fb8fb5d https://github.com/dic-iit/bipedal-locomotion-framework/commit/9271e4354c571eb319464b394f87e5aba4980d41 https://github.com/dic-iit/bipedal-locomotion-framework/commit/dae089d5b2060eaab13cb56e7844dd179de8014a
The PR implements a  2-point minimum jerk trajectory planner for SO(3) elements. The Planner extended the theory explained in Modern Robotics: Mechanics, Planning, and Control (Chapter 9.2), and it introduces the possibility to generate the velocity/acceleration expressed not only in body frame but also in the inertial frame (i.e. left and right trivialization).
cc @xenvre

Failing because of https://github.com/artivis/manif/issues/162

Let me know if the modifications I did are fine with you. If yes I merge the PR

@artivis

Hey, sorry I didn't mean to barge in your work, but since you are working on trajectory planning I thought you might be interested in having a look to the couple algorithms implemented in manif if you haven't already. Especially, Bezier, Decasteljau and the smooth interpolation. I myself implemented a mobile-base motion planner based on the later.

Thank you very much!! I will take a look for sure!

Accordingly to https://github.com/dic-iit/bipedal-locomotion-framework/pull/84#discussion_r487765095 and discussing with @prashanthr05 we decided to merge the PR as it is.
Merging!


Bun error: Node-API function "napi_type_tag_object" is not implemented yet

Not sure if this is a bun or a libsql issue, but in earlier versions 0.3.10 this issue did not appear. A workaround is to lock the libsql version to 0.3.10 in your package.json.

I just checked and it seems like this issue starts occuring on version 0.3.13, when it didn't occur on 0.3.12. Diff: https://github.com/tursodatabase/libsql-js/compare/v0.3.12...v0.3.13
My theory is that the Neon update might be the culprit here, but I am not 100% certain on that.

Follow-up on my comment: Building the libsql-js cargo project with the following:
[dependencies.neon]
version = "1.0.0"
default-features = false
features = ["napi-6"]

Seems to make it work again in bun. I am not in a position where I can easily open a PR right now, but if this change doesn't harm any other part it should be an easy fix.

I just verified with my codebase, bun now works again on local databases starting with 0.3.17 (after ensuring it's up to date and the dependency is not stale, i recommend clearing ~/.bun, deleting node_modules and bun.lockb), i guess this issue can be closed now

Fixed by https://github.com/tursodatabase/libsql-js/pull/105, right @rphsoftware?

Confirmed, this can be closed now! Thanks everybody


Add autorequire on parent ACL

I can't determine that this behaves any differently from what's already in master. What is really needed (and what I haven't figured out how to do yet) is to make an Acl for a directory require Files that are children of the directory. So for example, given theseFiles:
file { "/tmp/testdir":
    ensure => directory,
    owner => blah,
    group => blah,
}

file { "/tmp/testdir/b":
    ensure => file,
    owner => blah,
    group => blah,
}

and this Acl:
acl { "/tmp/testdir":
    action     => set,
    permission => [
                   "user::rwx",
                   "group::---",
                   "mask::r-x",
                   "other::---",
                   "default:user::rwx",
                   "default:group::r-x",
                   "default:mask::rwx",
                   "default:other::---",
                   ],
    provider   => posixacl,
    recursive  => true,
    recursemode => deep,
}

the Acl should require both of the File resources, since /tmp/testdir/b will (probably) be created after the Acl is applied to /tmp/testdir, and so /tmp/testdir/b will have the ACLs that are calculated at file create (mask::rw-, user::rw-) instead of the ACLs that are intended to be applied (mask::rwx, user::rwx).
I'm not sure how to do this.

@dobbymoodge done

This doesn't work because it looks for the additional requirements in the filesystem instead of in the catalog. Also, I haven't been able to get it to work (under ruby 1.8.7).

what about this

@roidelapluie I'll have to test it again tomorrow, but at first glance it isn't handling the test case outlined above. I'm out of time today, unfortunately.

@roidelapluie At least with the test.pp file I listed above, this still requires 2 runs before the final calculated ACLs are applied to /tmp/testdir/a. I wonder if we're trying to solve the same problem? I would really love to see the Acl resources applied only after all the File resources which are interdependent with the Acl are applied.
Maybe it's possible to force the Acl resources to only get applied after every File resource is applied? Or are the Acl autorequires unsatisfied because the interdependent File paths don't exist at the time the catalog is compiled? If that's the case, then maybe the only way to avoid multiple runs for nested ACLs is to implement https://github.com/dobbymoodge/puppet-acl/issues/16

I know the problem now. Quickly taking a look.

now it should be fine

it is not good yet

Now it should be ok


User and Stack Routes Structured

#106

@tharindupr Please have a look at the PR

Fix this warning in the PR as well WARNING: The useMongoClient option is no longer necessary in mongoose 5.x, please remove it.

Not related to this issue but the API endpoint localhost:8080/api/post/create validator is not working properly. The reason is due to validator checking for an attribute called link_issue. The attribute should be linkIssue

Sure @psnmissaka

@psnmissaka  @tharindupr  changes made, kindly review it. #106


Update package name to @digitalbazaar/vc-status-list.

Codecov Report

Merging #2 (128e59c) into main (aff50ab) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage   98.11%   98.11%
=======================================
  Files           3        3
  Lines         106      106
=======================================
  Hits          104      104
  Misses          2        2

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 aff50ab...128e59c. Read the comment docs.


Improved table handling

Summary
I found several issues in data-table handling in worksheets regarding existing tables:

loading existing Tables from workbook errors and alters table
hidden header rows in Tables corrupt file on save
totals row toggles on file save -> corrupt file on save

in addition to that I missed following features:

define / create a table over existing data in Worksheet
delete a table in Worksheets

Updates

fixed all errors reading existing Tables
added ability to define Table over existing Data
added Documentation for deleting a Table (function existed)
added Documentation for defining a Table

Test plan
I added a test to create a table, save it, read it back and comparing them to some extend
However, valid tests would also include reading generated Table in Excel and visually comparing them and creating Tables in Excel and reading them with the library
Other thoughts

I am not sure if current table data handling (duplicating worksheet data) is useful
I am not sure if current requirement to commit table data changes is necessary useful

let me know what you think

Hello! I am currently having an issue in which I cannot have a column header with a name with a tab or newline (e.g. Foo\tBar or Foo\nBar), it says Excel found unreadable content and asks if I want to recover. I saw your PR, I was wondering, does your PR fix this? If not, do you have any idea on how to fix this?

Hello! I am currently having an issue in which I cannot have a column header with a name with a tab or newline (e.g. Foo\tBar or Foo\nBar), it says Excel found unreadable content and asks if I want to recover. I saw your PR, I was wondering, does your PR fix this? If not, do you have any idea on how to fix this?
Thank you 😬

Probably not. My fixes concern Data Tables in Excel.
.xlsx files are .zip files. Looking at a string generated by Excel, I see that a CR/LF is required for a new line.

Hey exceljs, any feedback on my pullrequest? And how can I fix the npm error?

Hi @delcon, I am not part of the exceljs team but I have some suggestions:

Fix the conflict in README.md
There are three tests failing with your PR, you can see the first here and just scroll down to see the other two. Apparently your PR caused the totalsRowShown and ref fields to disappear in the generated XML.

@delcon any progress on this? I'm having issues with loading existing tables from worksheets and I think this may fix it

@delcon any progress on this? I'm having issues with loading existing tables from worksheets and I think this may fix it

There doesn't seem to be much interst for my pull request. I stopped working on it. Your can clone my repo and see if it works out for you. Let me know If it works.

@delcon It's working fine for me, thanks for doing this. I might help out with it if I have the time but I hate doing npm packages that aren't in TypeScript...


Video Link to Channel 9 does not work.

Video Link to Channel 9 does not work.

@HelderN  Thanks for the feedback. What page is the broken link on? There's no information with this issue beyond your text, so I don't know where the problematic link is located.

Refused to display 'https://channel9.msdn.com/Shows/Mechanics/Introduction-to-Shielded-Virtual-Machines-in-Windows-Server-2016?nocookie=true' in a frame because it set 'X-Frame-Options' to 'sameorigin'.

@HelderN  Thank you for this. It looks like the video is working and the link is right (https://channel9.msdn.com/Shows/Mechanics/Introduction-to-Shielded-Virtual-Machines-in-Windows-Server-2016), but that it's not displaying properly from GitHub. I will have to look into this and see if we can figure out why it's not showing properly.

@HelderN  Figured out the issue, the full link needs to be: https://channel9.msdn.com/Shows/Mechanics/Introduction-to-Shielded-Virtual-Machines-in-Windows-Server-2016/player. This will be fixed with our publish at the end of the week.


Replace input/output geometry type stubs

When the point-in-polygon type was implemented (#7031) the definitions of the input/output geometry types of analysis nodes were hardcoded in the local camshaft reference wrapper (per prior agreement offline w/ rochoa since he was going on vacation at the time). The valid-input/output geometry types should be defined in the camshaft reference instead.
TBD:

[ ] add definitions in reference; need to define format/requirements?
[ ] add operationparam for point-in-polygon to mandate wanted output geometry of point-in-polygon?
[ ] enough to use the (current) simplified geometry type or need the original one? (e.g. point vs ST_Point/ST_MultiPoint`/…?
[ ] replace hardcoded stub implementation

ping @rochoa @javierarce @xavijam

:+1:

:ok_hand:

About

[ ] add definitions in reference; need to define format/requirements?

Here is my initial proposal: https://github.com/CartoDB/camshaft/pull/5/files

About

[ ] add operation param for point-in-polygon to mandate wanted output geometry of point-in-polygon?

I've just realised it's not that easy to make that to the reference as it is something that has to be evaluated, probably the best option is to have two different node types in those cases so it's easier to serialise.
I did a proof of concept for the library itself, as per https://github.com/CartoDB/camshaft/commit/8af31804e6418967c566c82584f10733b774abf9, but that takes advantage of having full access to node context. The reference cannot access the library but only a JSON file.

Closing this, since finally we're getting geometry type from given query (data, once available) 👉  #7374


Get rid of AppConstants.java

Pull Request
Description
Removed AppConstants.java and used Value annotation to import constants to EmailService.java
Related Issue
#16
Checklist
Please review and check all applicable items:

[x] I have tested my changes thoroughly
[x] My code follows the project's coding standards and guidelines
[x] I have updated relevant tests
[x] I have reviewed my changes to ensure there are no unnecessary changes
[x] My pull request has a descriptive title and includes a detailed description of the changes

Additional Notes
I've tested the changes and it seems to work just fine
Screenshots (if applicable)

Yaaay! Thanks!!!


Patch 31 titles

Type

[ ] Bug Fix
[ ] New Feature
[ ] Sample
[x] Documentation

Related Issues?
n/a
What is in this Pull Request ?

Updated titles and online versions

Thank you @PowershellScripts , awesome improvements , much appreciated 🔥🙏😊


Remove spaces from ruleId and make lowercase

Allows ignoring messages with remark-message-control when the phrase has spaces.
Right now, if I want to ignore specific simplify messages with remark-message-control, for example with something like;
<!--quality-docs disable filler overall in addition -->

It fails for phrases that contain spaces. This change converts any ruleId with spaces into dashes, and makes the string lowercase for consistency. Now I can do this to ignore a rule;
<!--quality-docs disable filler overall in-addition -->

Current coverage is 100% (diff: 100%)

Merging #4 into master will not change coverage

@@           master    #4   diff @@
===================================
  Files           2     2
  Lines          39    39
  Methods         6     6
  Messages        0     0
  Branches        3     3
===================================
  Hits           39    39
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 9847dd3...0ab1e3d

👍 Will be released as 3.0.0 in a minute! Travis took a while testing my refactor :/


[feature] thumbs options - add option to control multiple slide activation

In this image we use slidesPerView of 1.5 for the main images, the connected thumbnail slider uses this value to activate multiple thumbnails. In our usecase we only want the first (real) slide to get activated. CSS workarounds would be ugly or impossible.
That's why I suggest to add an option for it. I've set the default value to the current behaviour. Additionally i rounded down the thumbsToActivate value in case someone is using it (as we did) with a fractional value (a small logic fix).

Thanks, merged!
P.S. I have renamed the option a bit :)

Thanks! Sure, I wasn't so clear about that option name :D


Week1

The week1 folder has the https://github.com/broadinstitute/warp/blob/develop/tasks/broad/Qc.wdl file and the manual translation of it using cwl. Also includes the errors I got while running the two files. The md5 folder has the md5 checksum workflow containing both wdl and cwl files and their respective outputs.

Commands:
WDL:
miniwdl check Qc.wdl --no-shellcheck  : to check for errors in .wdl file
miniwdl run Qc.wdl  input_bam=simple.bam  metrics_filename="metrics_file"  preemptible_tries=2 --verbose : run the workflow
CWL:
cwltool --validate Qc.cwl
cwl-runner Qc.cwl Qc.yml
Errors:
The errors are in the cwl_error.txt and wdl_error.txt files inside the week1 folder. If you go in to the 20210527_195245_CollectQualityYieldMetrics/ folder, error.json gives more details on the error (for the wdl run)

Were you able to run CollectQualityYieldMetrics manually, without using WDL or CWL?

Did you mean using the tool? http://broadinstitute.github.io/picard/ this one?

I have added the setup.py file and the modified python code(create_cwl.py) to create the cwl file (result.cwl)

WdlV1_1ParserVisitor.py has been modified to get the inputs, outputs, and the relevant fields of a task required for the creation of the equivalent .cwl file.

Converted “CollectQualityYieldMetrics” task to CWL using cwl-utils after parsing the wdl file

I have fixed the errors as discussed and moved everything out of the week1 folder.


[DUPLO-14500] - TF: Not able create OpenSearch with (warm enable =false and cold storage=false)

…
[DUPLO-14500] - TF: Not able create OpenSearch with (warm enable =false and cold storage=false)
Change description

fixed the bug when warm enabled and cold storage options are set to false

Type of change

[ ] fixed the bug when warm enabled and cold storage options are set to false

Related issues

Fix #1

Checklists
Development

[ ] Pull requests may not be submitted to the master branch (use develop instead) - or they will be closed.
[ ] Lint rules pass locally
[ ] Application changes have been tested thoroughly
[ ] Automated tests covering modified code pass

Security

[ ] Security impact of change has been considered
[ ] Code follows company security practices and guidelines

Code review

[ ] Pull request has a descriptive title and context useful to a reviewer.
[ ] "Ready for review" label attached and reviewers assigned
[ ] Changes have been reviewed by at least one other contributor
[ ] Pull request linked to task tracker where applicable

Task linked: DUPLO-14500 TF: Not able create OpenSearch with (warm enable =false and cold storage=false)


Support DS18B20 and AM2302/DHT22 on Master and Slave

Currently the master is limited to the DS18B20 in OneWire mode and the slave is hardcoded to the AM2302

The current implementation of the slave uses node-dht-sensor which depends on bcm2835 which can only be installed directly on a raspberry pi and i can not find any way around this issue to generically implement this package into the docker image

https://stackoverflow.com/questions/30059784/docker-access-to-raspberry-pi-gpio-pins

@thelittlebug this is already used for the DS18B20, but the DHT22 npm package uses the bcm2835 library which i don't know how to install on my development machine. The only possibility i can think of is to develop directly on a Raspberry Pi...


Button name/value not included in submitted form

Fixed by adding following before form.submit() call:

            var hidden = document.createElement('input');
            hidden.type = 'hidden';
            hidden.name = self.attr('name');
            hidden.value = self.attr('value');
            form.append(hidden);

Hi kaetemi !
It is Interesting as the submit button is not submit, however i think we have to check if there are attr name/value present on the button.
Would you PR your fix ?
Thanks :)

WIth dd3a278 i created 9f5c503609a1898418925f069f4bef06c2758f54. I fixed this.


Attribution?

There's no attribution to the original source for the code.

Certainly! Just saw that I got the original code from your SO post! I will add this in asap
https://stackoverflow.com/questions/6491463/accessing-nested-javascript-objects-with-string-key

@raybellis I've added to the README.md. Is that good?  Also, sorry I didnt have your attribution before.


Please have the positional circle show only on melee jobs.

With one of the most recent updates the circle shows even for casters and ranged.  It's kinda pointless and an unneeded visual distraction.  Is it possible to revert it to only be melee again or at least give us the option to only enable for melee?

You already have the option to enable it only on melee. You can set specific profiles for each class.


poll: winit-0.20.0-alpha3: Combining event loops?

Winit has a particular way it's implementing an event loop.  Any examples of using the NEW 0.20.0 event loop with mio?  Any examples that run mio in a thread of it's own?

I haven't used winit before, but taking a quick look at the docs it might work if the create thread that polls Mio and sends the (relevant) events to the Winit event loop using EventLoopProxy::send_event.
I don't know if the EventLoop type of Winit is backed by a file descriptor on Unix, if it is you could try and register that with Mio using EventedFd poll Mio.

Closing this as it seems there is nothing to do.


Fix the incorrect parse result for output from ps aux

A fix for this https://github.com/neekey/ps/issues/13
Fix the problem by only handing un-empty domains

: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:


[dv,top_level] Add test for sensor_ctrl deep sleep wake up

This is a sim_dv test that forces sensor_ctrl alert inputs while the chip is in deep sleep. It will fail unless the sensor_ctrl synchronizes incoming alerts with the aon clock. Can be used to verify the fix for #20614.

The CI failure is from
//sw/device/silicon_creator/rom/e2e/jtag_inject:openocd_debug_test_otp_test_unlocked0_fpga_cw310_rom_with_fake_keys
This could not be due to this PR, so merging it.


location header shouldn't be set when the DirectResponse returns 200

We are unconditionally setting the Location:  header when returning direct responses:
https://github.com/envoyproxy/envoy/blob/master/source/common/router/router.cc#L395
Whereas the header should only be set when the response code is either 201 or 3xx per:
https://tools.ietf.org/html/rfc7231#section-7.1.2
cc: @sc0ttbeardsley @mattklein123 @alyssawilk

(I can send a PR to address this in a bit, assuming we agree it needs fixing).

Also: https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Location.

We ran into this while returning non-3xx direct responses.


add support for optional opts argument to container.stats()

In addition to adding support for the opts to container.stats(), the repeated task of dealing with optional opts and the tailing callback arguments has been factored out into its own helper function processArgs() in order to remove some redundancy.

Thanks! :+1:
Will ping as soon as it is published.

I will actually refactor this.
processArgs shouldn't be a local function in container, instead it will be moved into util and used across all dockerode and not only container.

It was heavily refactored and published in v2.2.0


Typo in GUI Sketches

Instead of "Temperature", it says "Temerature" in the GUI Sketches.

Seen. will be changed in 2 hours aprox. Changing Temperature and other names to SensorType1-3 to have a picture that is more independent of the later actual content.

Done


Better Way Of Defining Linux System Constants

So the Linux constants may vary from one architecture to another. Why not get the actual values from the Linux system itself? Incorporate a script like this into your build process for Linux, by including its output in the generated source file:
#/usr/bin/python3
#+
# Better way of defining constants for this
# <https://github.com/apple/swift-system/blob/main/Sources/System/LinuxPlatformConstants.swift>
# source file.
#-

import sys
import os
import subprocess
import tempfile
import shutil

consts_to_define = \
    (
        {
            "symbols" :
                [
                    "EPERM", "ENOENT", "ESRCH", "EINTR", "EIO", "ENXIO",
                    "E2BIG", "ENOEXEC", "EBADF", "ECHILD", "EAGAIN", "ENOMEM",
                    "EACCES", "EFAULT", "ENOTBLK", "EBUSY", "EEXIST", "EXDEV",
                    "ENODEV", "ENOTDIR", "EISDIR", "EINVAL", "ENFILE", "EMFILE",
                    "ENOTTY", "ETXTBSY", "EFBIG", "ENOSPC", "ESPIPE", "EROFS",
                    "EMLINK", "EPIPE", "EDOM", "ERANGE", "EDEADLK", "ENAMETOOLONG",
                    "ENOLCK", "ENOSYS", "ENOTEMPTY", "ELOOP", "EWOULDBLOCK", "ENOMSG",
                    "EIDRM", "ECHRNG", "EL2NSYNC", "EL3HLT", "EL3RST", "ELNRNG",
                    "EUNATCH", "ENOCSI", "EL2HLT", "EBADE", "EBADR", "EXFULL",
                    "ENOANO", "EBADRQC", "EBADSLT", "EDEADLOCK", "EBFONT", "ENOSTR",
                    "ENODATA", "ETIME", "ENOSR", "ENONET", "ENOPKG", "EREMOTE",
                    "ENOLINK", "EADV", "ESRMNT", "ECOMM", "EPROTO", "EMULTIHOP",
                    "EDOTDOT", "EBADMSG", "EOVERFLOW", "ENOTUNIQ", "EBADFD", "EREMCHG",
                    "ELIBACC", "ELIBBAD", "ELIBSCN", "ELIBMAX", "ELIBEXEC", "EILSEQ",
                    "ERESTART", "ESTRPIPE", "EUSERS", "ENOTSOCK", "EDESTADDRREQ", "EMSGSIZE",
                    "EPROTOTYPE", "ENOPROTOOPT", "EPROTONOSUPPORT", "ESOCKTNOSUPPORT",
                    "EOPNOTSUPP", "ENOTSUP", "EPFNOSUPPORT", "EAFNOSUPPORT", "EADDRINUSE",
                    "EADDRNOTAVAIL", "ENETDOWN", "ENETUNREACH", "ENETRESET", "ECONNABORTED",
                    "ECONNRESET", "ENOBUFS", "EISCONN", "ENOTCONN", "ESHUTDOWN", "ETOOMANYREFS",
                    "ETIMEDOUT", "ECONNREFUSED", "EHOSTDOWN", "EHOSTUNREACH", "EALREADY",
                    "EINPROGRESS", "ESTALE", "EUCLEAN", "ENOTNAM", "ENAVAIL", "EISNAM",
                    "EREMOTEIO", "EDQUOT", "ENOMEDIUM", "EMEDIUMTYPE", "ECANCELED",
                    "ENOKEY", "EKEYEXPIRED", "EKEYREVOKED", "EKEYREJECTED", "EOWNERDEAD",
                    "ENOTRECOVERABLE", "ERFKILL", "EHWPOISON",
                ],
            "defines" : [],
            "includes" : ["errno.h"],
            "mark" : "errno",
        },
        {
            "symbols" :
                [
                    "O_ACCMODE", "O_RDONLY", "O_WRONLY", "O_RDWR",
                    "O_CREAT", "O_EXCL", "O_NOCTTY", "O_TRUNC",
                    "O_APPEND", "O_NONBLOCK", "O_DSYNC", "FASYNC",
                    "O_DIRECT", "O_LARGEFILE", "O_DIRECTORY",
                    "O_NOFOLLOW", "O_NOATIME", "O_CLOEXEC",
                    "SEEK_SET", "SEEK_CUR", "SEEK_END",
                ],
            "defines" : ["_GNU_SOURCE", "_LARGEFILE64_SOURCE"],
            "includes" : ["sys/types.h", "sys/stat.h", "fcntl.h"],
            "mark" : "File Operations",
        },
    )

sys.stdout.write("#if os(Linux)\n")
for consts_entry in consts_to_define :
    tempdir = tempfile.mkdtemp(prefix = "swift-consts-")
    srcfilename = "defconsts.c"
    exefilename = "defconsts"
    srcfile = open(os.path.join(tempdir, srcfilename), "wt")
    for define in consts_entry["defines"] :
        srcfile.write("#define %s\n" % define)
    #end for
    for include in consts_entry["includes"] :
        srcfile.write("#include <%s>\n" % include)
    #end for
    srcfile.write \
      (
        "#include <stdio.h>\n"
        "\n"
        "int main(void)\n"
        "  {\n"
      )
    srcfile.write \
      (
            "    fputs(\"\\n// MARK: %s\\n\", stdout);\n"
        %
            consts_entry["mark"]
      )
    for symbol in consts_entry["symbols"] :
        srcfile.write("    fputs(\"\\n@_alwaysEmitIntoClient\\n\", stdout);\n")
        srcfile.write \
          (
                "    fprintf(stdout, \"internal var _%(symbol)s: CInt { %%d }\\n\", %(symbol)s);\n"
            %
                {"symbol" : symbol}
          )
    #end for
    srcfile.write \
      (
        "    return 0;\n"
        "  } /*main*/\n"
      )
    srcfile.close()
    subprocess.check_call \
      (
        args = ("gcc", "-o", exefilename, srcfilename),
        cwd = tempdir
      )
    sys.stdout.write \
      (
        subprocess.check_output
          (
            args = ("./" + exefilename,),
            cwd = tempdir,
            text = True
          )
      )
    shutil.rmtree(tempdir, ignore_errors = True)
#end for
sys.stdout.write("\n#endif\n")

I’m sure you can see how to extend it to add other groups of constants as needed.

Thanks for posting this. We're definitely in a tough due to language/compiler limitations around emitting these constants, hence why we have the hard-coded values. Ideally, we'd get language support to just forward the constants from the headers, but in the meantime we may have to resort to code-gen for this.

https://github.com/apple/swift-system/pull/22 is also aiming at addressing this. It would be ideal if that could be a self-hosted script whose artifacts we can check in, giving us bootstrapping and automatic regression checks.

Not a fan of putting automatically-generated stuff into version control. Especially not stuff that is crucially dependent on the current system setup.

Good news, this is no longer required: https://github.com/apple/swift-system/pull/29


Setup prow CI

Tested in https://github.com/kargakis/acs-engine/pull/115
TODO:

[ ] setup dns and update ingress resource
[ ] create hmac and oauth secrets

TLS can be fixed in a follow-up

Tested in https://github.com/kargakis/acs-engine/pull/115
TODO:

[ ] setup dns and update ingress resource

TLS can be fixed in a follow-up

/lgtm

/lgtm

/lgtm

/lgtm

/lgtm

/approve

/label DO-NOT-MERGE

Waiting on https://github.com/kubernetes/test-infra/pull/8075

Because of the issue in the ingress controller, the link is not working
properly atm.
On Wed, May 23, 2018, 01:18 Cecile Robert-Michon<EMAIL_ADDRESS>wrote:

@CecileRobertMichon commented on this pull request.
In .prowci/config.yaml
https://github.com/Azure/acs-engine/pull/2923#discussion_r190082144:

@@ -0,0 +1,18 @@
+prowjob_namespace: default
+log_level: debug

+tide:

target_url: http://ci-bot-aks-ingress.eastus.cloudapp.azure.com/tide.html

why is this commented out?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Azure/acs-engine/pull/2923#pullrequestreview-122386868,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADuFf6iBgdZW6qCHU5ttfb6KicBt0ewZks5t1JywgaJpZM4T7OTG
.

@CecileRobertMichon can you post logs from hook? I expected it to pick up this merge and update its config. You will need to do a make prow-config-update from .prowci and sync what we merged here with what's in the cluster.


Complex key patch

Issue:
Existing Behat tests conflicts with new complex_key feature. There are many key names containing 's (example: Order's account).
Solution:
Just changed the sign from ' to ` (backtick).

Paging @manuerumx to review the back-end work in this pull request!
Paging @smijo149 to review the testing work in this pull request!

Also, all the tests who stores/retrieves a file are failing in stdcheck
Even with this patch.
behat features/web/manage/patients/messages/viewAttachment.feature

Looks like a wrong issue is attached to this PR. Could you please check.

@smijo149 Yes it is not directly releated. But this blocking my main issue thats why I attached.

Agree with other reviews. I will wait until the next change to review.

I have changed the implementation as suggested. Updated the solution above


run the bot in Android

Hi @ohld
do you have experience in running the bot in an Android device?
Would it be possible to install it in Android?
thank you

The simplest solution would be for you to build a web GUI for it with Flask or Django and then access it through your mobile browser.

@simongomez95 but then it would be something like VPN connection to my computer just with a GUI, no?

No, then it would be just access to a website you set up. I may be wrong, but I feel like you are rather unexperienced in these matters. Do you have programming skills?

@onkaraman  you are right. I have 0 experience :(

This is not just a program you can simply use. You need to have some programming skills in Python to build a program around this project. Sorry.


Zapier integration unable to load Content Type data in v10

Umbraco v10.2.0
Umbraco.Cms.Integrations.Automation.Zapier v1.0.3
When adding Zapier integration to a fresh install hosted on Azure App Service the Zapier trigger fails to load the 'Content Type' data.
Steps to reproduce

Create a fresh Umbraco v10.2.0 install
Install Umbraco.Cms.Integrations.Automation.Zapier v1.0.3 nuget package
Publish/Deploy to an azure web app (so that we get a base URL)
Add Zapier integration for Umbraco (Either login credentials or API Key)
Attempt to set up New Content Published trigger
Should see an error when choosing content type "Unable to load choices
We're having trouble loading 'Content Type' data. maximum redirect reached at: {baseURL}/Account/AccessDenied?ReturnUrl=%2F"

It appears that the Zapier app is failing to pass authentication possibly.
Running a 'test' on the Umbraco connection passes successfully so not sure if this is something specific to the call for Content Types

Hi Luke,
Thank you for bringing this up to our attention, I will try to replicate the issue and I will get back with more details.

Thank you for the prompt respone @acoumb, If I can provide any more info let me know (likewise if you discover that it works and I'm doing something stupid please do let me know 😄)

Hello again @lukehook ,
I replicated your use case using an Azure App Service, and I was able to query the CMS data from my Zap.
The URL of my Umbraco 10.2.0 site is https://zapierazureapp20220927110307.azurewebsites.net and I was using this API key: Y8uB84PMv1x%C%rw0u&i. This video demonstrates my test case https://drive.google.com/file/d/1IaqrfDmXbBrfNwEyBRmnTDVYuadtbcSY/view?usp=sharing.
Could you test a Zap against my test site and see if it works?
Then, would you be able to share with me the URL of your site, with a test API key so I can have a look?
Thank you

@lukehook that sounds great. Regarding your request, I will add it to the backlog and include it soon with a new version of the package.

@lukehook could you tell me what published property you are using on your document type?

Sorry @acoumb I completely missed this reply in my emails. Is this for the Image mentioned above do you mean? If so I was using Umbraco.MediaPicker3


Question: where is the cFS Deployment Guide?

https://github.com/nasa/cFE/blob/main/cmake/README.md says
"For a more in-depth guide and specific examples, see Appendix A of the cFS Deployment Guide."   I have searched for the cFS Deployment Guide and cannot find it.   Has the name changed?   Where is it now?

That reference should be removed, the deployment guide is outside the scope of what we've been able to maintain from the cFS Bundle.  The most current/best resource for configuration and build within this bundle is the sample in https://github.com/nasa/cFE/tree/main/cmake/sample_defs, along with all the *.cmake files in the lab apps, modules, etc.  For assembling a distribution (main purpose of the cFS Deployment Guide), see https://github.com/nasa/CFS-101 or https://github.com/OpenSatKit/OpenSatKit.


CMake: allow parent project to disable unit tests for streamvbyte

While integrating into another cmake project via add_subdirectory()
the target check might already be defined by a parent directory.

Does STREAMVBYTE_ENALBE_TESTS include a typo? ENABLE vs. ENALBE?

@lemire Yeah! good catch. I fixed the typo.

Merging.


Force arrows to show (even in a disabled state)

When the width of the track can fit all the slides, the arrows disappear. I'd like to be able to force the arrows to be visible always, whether disabled or not.
Here's a demo that has one slide and the arrows are missing:
https://codesandbox.io/s/zljkw3x09x
If I add more slides, the arrows show up. My goal is to always display the arrows (they would be disabled if they can't navigate left and/or right). How can I do that?

Hi,
It's actually pretty easy. You need to override css for "slick-disabled" class.
Or you can define custom arrows with your own CSS (https://react-slick.neostack.com/docs/example/custom-arrows) :wink:

This solution is not working

The arrows are not even rendered, so can't use slick-disabled.


task 2.

Who is doing task 2? I need to ask something?.

Nobody. ask another group

Ok thank you


Server scans blockchain

To deal with the overhead in tests, cache initial state

Requires re-running pip install -r requirements.txt

This change introduces too much instability in the test system. #15 needs to happen first.


Relative path routing in Production

Locally, my relative paths and on-click instructions work fine however, in production they do not work. I generally get a 404 Not Found. I have been trying to resolve this now for some time.

<< BING_CHAT
It seems like you are facing an issue with your React app. When you run the app with a dev server, the link <a href="/login" className="nav-link" onClick={this.logOut}> works fine, but when you build the app and host it on a web server, it gives a 404 Not Found error.
This issue is likely caused by the fact that the web server is not configured to handle client-side routing. When you use client-side routing, the URL is changed locally only, but no request to the server is made ¹. Therefore, when you refresh the page or enter the URL manually, the server does not know how to handle the request and returns a 404 error.
To fix this issue, you need to configure your web server to handle client-side routing. One way to do this is to set up a fallback route that redirects all requests to the root URL of your app ³. Here is an example of how to do this with Nginx:
location / {
    try_files $uri /index.html;
}

This configuration tells Nginx to try serving the requested URL first. If the file is not found, it will serve the index.html file instead ³. This way, your React app will be loaded and the client-side routing will work as expected.
I hope this helps! Let me know if you have any other questions.
Source: Conversation with Bing, 12/5/2023
(1) React-router URLs don't work when refreshing or writing manually. https://stackoverflow.com/questions/27928372/react-router-urls-dont-work-when-refreshing-or-writing-manually.
(2) Nginx not showing custom 'Page Not Found' page in React app. https://stackoverflow.com/questions/67629969/nginx-not-showing-custom-page-not-found-page-in-react-app.
(3) Warning Prop href did not match. using react server-side-rendering. https://stackoverflow.com/questions/47336589/warning-prop-href-did-not-match-using-react-server-side-rendering.
(4) undefined. http://example.com/about.
BING_CHAT

This seems to be fixed now. Closed.


EULA filelist incomplete

The Reactive Extensions EULA has a link to the files included to be distributed in object code form. This list is currently:

System.Reactive.dll
System.Reactive.Providers.dll
System.Reactive.Windows.Forms.dll
System.Reactive.Windows.Threading.dll

However, the nuget packages

Rx-Core.2.2.5
Rx-Interfaces.2.2.5
Rx-Linq.2.2.5
Rx-PlatformServices.2.2.5
Rx-Xaml.2.2.5

all refer to this Reactive Extensions Eula. These packages build to different files.
This issue prevents us from including these libraries into our enterprise software product. We are an ISV.

Does the license.txt in the repo address this? All of the code is released under Apache 2.0.

The packages for 3.0 will be updated to point to the Apache license here: https://github.com/Reactive-Extensions/Rx.NET/blob/master/LICENSE
No further EULA should be required.


[Serialization] Emit an error on deserializing an invalid declaration

When running in the allow errors mode
(-experimental-allow-module-with-compiler-errors), modules may contain
invalid declarations. The rest of the compiler pipeline, however,
expects to have valid declarations unless diagnostics have emitted an
error. Emit an error while deserializing to maintain this assumption.
Note that these errors will not have a useful location, unless there's a
corresponding .swiftsourceinfo. This isn't a problem for the intended
use case in IDEs, where diagnostics outside the current file would be
ignored anyway.
Since reading declarations is lazy, SILGen (and thus SIL diagnostics)
can still run as long as any invalid declarations weren't referenced in
the compiling module.
Resolves rdar://74325388

@swit-ci please test

@swift-ci please test

@xedin I've added a fatal diagnostic for the case when an invalid declaration was read in a module without allow errors enabled.
Also fixed up the test so it actually crashes in the case the radar mentions before the change (would still have failed, but not with the crash).
@swift-ci please test

Looks good, thank you!


Fix unit tests while running on a machine that has FIPS enabled.

Description
This is a first pass at fixing a couple failing unit tests when running them on a system that has FIPS enabled. I think for the purposes of these tests, it's fine to assume FIPS is disabled, but if folks disagree, I could just pass through the actual value of a File::read of /proc/sys/crypto/fips_enabled.
Issues Resolved
Fixes #5533

I think we need to get to a point where running these tests on a FIPSified system actually ensures FIPSiness, so let's pass through the read. I want to get a FIPS windows node and a FIPS linux node in to the test matrix soon.

Fixed by #7207


fix(inlinenotification): use lowContrast inline notification

Closes #
Summary

add lowContrast to InlineNotification used in PageWizard

Change List (commits, features, bugs, etc)

PageWizard

Acceptance Test (how to verify the PR)

PageWizard

Is using the low contrast version of inline notification a change from design?
cc @lukefirth @bryancboyd

Yes, it came up while fixing this issue: https://github.ibm.com/wiotp/monitoring-dashboard/issues/533

Where does the yarn.lock change come from? If we have no changes to our package list there shouldn't be a need to update yarn.lock

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


marp-cli: init at 3.2.0

Description of changes
Add the latest release of Marp, a tool to transform markdown into slideshows (HTML, PDF, ...)
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
23.11 Release Notes (or backporting 23.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

[x] Fits CONTRIBUTING.md.

cc @lilyinstarlight


Task/Update log format on dev and prod

🌲 LOGGING!! 🌲
We love it!
Some pretty big changes here, or maybe small changes, depending on your perspective.
Long & short:

All logs are stdout or stderr
All logs are JSON
No logs in cypress
Removed winston logger
You can delete your /logs directory
Stopped logging all CSS, JS, img files, etc.

//
Basically, after thinking about it for a while, I really had only a couple of constraints.

All logs should go to stdout
All logs should be JSON

Everything stdout
According to Docker, "By default, docker logs or docker service logs shows the command’s output just as it would appear if you ran the command interactively in a terminal." So, we don't want to be creating big log files using winston, which is what we were doing. So I got rid of logging to files, which means you can delete your /logs directory (should be in the project root).
Everything JSON
The decision to do JSON is based on past experience trying to search through logs for things. JSON logs are generally easier because you can find keys and add new ones whenever you want without having to do weird string parsing because of some unexpected formatting change.
winston can handle JSON, yes, but morgan seemed simpler and faster to work with. So basically, to get morgan to use JSON (which it doesn't ordinarily do) I used this gist: https://gist.github.com/cgmartin/913bb12097ff07132597
Once I got morgan the way I wanted it, I removed winston. We can add it back later if we need something it does but let's not overcomplicate things at this point.
What about the format?
No strong opinions. We want to have useful prod logs but not overwhelming dev logs. Check the morgan.config.js file for formatting. Happy to change it if needed.
What about errors?
Errors were a bit weird. I wanted to keep all the error stuff in there, but push them into our JSON logs because I don't want them to get lost if we end up having to debug weird prod issues.
 Merge away!


[Bug]:OSC lock user failed

ODC version
odc433
OB version

What happened?

What did you expect to happen?
Lock table successfully changed without lock structure
How can we reproduce it (as minimally and precisely as possible)?

Create lock free structure, change settings, lock users
Add data using locked users
Data insertion failed while switching tables

Anything else we need to know?
No response

pass


Add more ignore options to readonly rules

The rule is too strict and doesn't allow adding readonly by little steps.
Some of missing:

ignore-interface: Doesn't check for readonly in interfaces
ignore-class: Doesn't check for readonly in classes

You can have an interface or class with only some properties readonly by using the ignore-prefix option. Below is an example with interface but it should work with class too.
interface Person {
  readonly name: string,
  fooAge: number // This is OK with ignore-prefix = "foo"
};

But of course this requires you to prefix your mutable properties with something. I'm open for a PR for both of your suggestions.

Released in 4.5.0.


Fix or allow all clippy warnings

Fixes https://github.com/knurling-rs/defmt/issues/360

bors r+

bors r+


More information on publisher

Please provide more information about the aaptapps publisher. I can observe "aaptapps" in one of the screenshots in this document, but I cannot find more information on this publisher mentioned anywhere else beyond the empty GitHub profile. Organization owners are understandably reluctant to approve access when it appears to be no official information on the publisher.

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

ID: 7129a93f-f0eb-fe26-fd1d-a91e4654fd98
Version Independent ID: fe9e378d-25e7-f1ce-36ab-2b2ab91b109d
Content: Access, monitor, and manage your GitHub repo - Azure Logic Apps
Content Source: articles/connectors/connectors-create-api-github.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan

@Hjaf Thank you for your feedback! We will review and update as appropriate.

Hi @Hjaf, "apptapps" is an alias owned by our internal Connectors team. I've asked them to change the alias to something more recognizable. Meanwhile, I've updated the doc to clarify the publisher's identity.


Japanese Chinese and Philippine translations

Added Japanese Chinese and Philippine translations

@alejandrocordon I was testing your changes. Overall they look and work well but I'm still seeing some missing translations. I didn't check all localizations files but I found some non translated string at least in the dashboard module string file (which are easily visible in the dashboard view of the app where the tabs for example are not translated). Could you please go over all string files and check for missing translations? Once that's done we can merge and process the pmt. Thanks in advance

@tomastiminskas you are right. I updated it.


Add ESLint to CI

Is your feature request related to a problem? Please describe.
Currently CI only formats codes by prettier but there is no linter.
Describe the solution you'd like
Add eslint and rules to match with vue, typescript and prettier
Describe alternatives you've considered
Is there any?
Additional context
I tried to implement ESlint once while adding GitHub Actions. But error occurs on run eslint, some kind of error inside dependencies.

can I take a chance on this one!

Hi @averrois ,
Thank you for your volunteer!
The requirements from me are:

Install eslint and plugins for typescript and vue3.
Add eslint config file and rules to apply plugins.

However, please be careful not to conflict with prettier.

add eslint to GHA workflow
Fix or ts-ignore codes causing error on lint

But if you have any better solution, feel free to enlighten me! 😄
I will assign you to this issue now.

alright...

is there an reason way u are using and older version of @typescript-eslint/eslint-plugin !

That's because some runtime error occurs on while running eslint on latest eslint and plugins.
Seemed like some package inside eslint but not quite sure.
I was trying to find the difference between older version but haven't find the reason yet...

ooh I see, I may self I am getting warning about ts-eslint, for  not supporting the current version of TS, when I look it up most solution that I find is downgrade version of TS, but since I am knw to this  lol, I am not quit  sure if this a good approach !

@averrois
You're right. We supposed not to keep using the older version to accept the vulnerability ...
In my local environment, it seems like an error on eslint built-in default formatter stylish.js.
I don't know why this causes an error in typescript env it never happened in other projects I have.

If I change formatter to other options suitable for command line such as compact or visualstudio, it works well.

Maybe we can simply use this.
Please checkout the demo branch I pushed on #18
This could fix the problem for now... 🤔

it's nice to knw that u fix it...,

I tried it, and it works , as you said it might solve the issue for the moment...