APT repository signing key changed?

Just noticed on some servers that APT repository signing key has changed. Old key was
pub   rsa4096 2014-05-20 [SC]
      FA0B F24A 0F94 6367 0B88  0EAB 7424 83B7 B222 C443
uid           [ unknown] Gerardo Orellana<EMAIL_ADDRESS>sub   rsa4096 2014-05-20 [E]

and new key (unconfirmed) seems to be
pub   rsa3072 2020-05-17 [SC] [expires: 2022-05-17]
      C03B48887D5E56B046715D3297BD1A0133449C3D
uid           [ unknown] Gerardo Orellana<EMAIL_ADDRESS>sub   rsa3072 2020-05-17 [E] [expires: 2022-05-17]

Can you @allinurl confirm that this is valid change, and would it be possible to

publish they key fingerprint for valid key somewhere in GoAccess.io to make it easier to have at least one additional place to confirm the key validity (not cryptographically secure, but better than having just the key as downloadable file) and
sign the new key with the old one so that it's possible to cryptographically verify that "the person who had access to old key has confirmed that new key is OK".

I'm aware of this note in the download page

but in case of someone breaching website(s) goaccess.io and deb.goaccess.io there is no way currently to ensure that this is legitimate change, and I'm not willing to just pull in "random key" to production machines.

Thanks for posting this. Yes, I can confirm that this is a valid change.

I went ahead and posted the key fingerprint on the downloads page as well.
Unfortunately I won't be able to sign the old key as I had a drive issue with the server containing the key, thus this change.

I apologize about this sudden changes. Though, I hope this helps to confirm these changes :)

Thanks for the fast reply and action, and also for awesome tool :)
Yes, this helps, while not the ideal solution, it's probably best that can be done if old keys got lost (to confirm authenticity via other channels, like GitHub, Twitter, website etc.).
I'm leaving this issue open, as I'm not sure if you want to leave this for others to notice, but feel free to close when it's right time. :)

Great! I'll keep it open for a few weeks. Thank you!

The fingerprint is there, which is great, but the file is now missing, so the instructions for Debian-based distributions are currently invalid.
It also doesn't appear that the public key has been uploaded to any keyservers? You may want to consider doing this (it's usually considered good practice).

@EqUaTe not sure what you mean with the fine is missing. As per the instructions, you can download it here https://deb.goaccess.io/gnugpg.key .

@allinurl Sorry, looks like I fat-fingered it - gnupg.key rather than gnugpg.key (OOI, why is it gnugpg.key? The product is GnuPG :-) )
The other part stands about keyservers though - then the instructions can be something like:
sudo apt-key adv --keyserver keyserver.ubuntu.com --recv-keys 97BD1A0133449C3D

@EqUaTe Yeah not sure where the gnugpg.key came from. For now I'll keep it that way just because there are websites out there with instructions pointing to gnugpg.key.

May I suggest to update the Ubuntu install procedure, to create a GoAccess-specific keyring instead of dumping the key in the default one ?
wget -O - https://deb.goaccess.io/gnugpg.key |
  sudo apt-key --keyring /etc/apt/trusted.gpg.d/goaccess.gpg add -

@dregad Updated. Thanks!


Gh-832: Upgrade JUnit 4 to JUnit 5

Also replace JUnit asserts with AssertJ.
Related Issue

Resolve #832

As with all other PRs at the moment, the build fails due to issues with WARs in Gaffer alpha 2, tests should pass when rerun once alpha 3 has been released.


Fixes #364: Adopt the usage of ASAR in VSCode

Starting with today's insiders, this extension is no longer functioning correctly!
This fixes #364 and adopts the usage of ASAR when loading node modules which are part of the VS Code installation.

Thanks for the contribution, but @PrashanthCorp already had a PR out to fix this (#368). It was just pending on the answer to my question in the original issue


[$500] Log in - Unable to log in again after entering the incorrect magic code

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to to the login page.
Enter any email/phone number >> click 'Continue.'
Enter an INCORRECT magic code (e.g., 123456) >>  an error message will appear.
Edit each digit field one-by-one, from the 1st to the 6th field, using the CORRECT magic code.
Observation: Despite entering the correct magic code, the user is unable to log in.

Expected Result:
User should be able to login when using correct magic code.
Actual Result:
Despite entering the CORRECT magic code, the user is UNABLE to log in
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/547066cd-24fe-40f6-9f1a-b84b9b6b1d77
https://github.com/Expensify/App/assets/93399543/c4441990-4142-4119-a04e-6b314438823b
Expensify/Expensify Issue URL:
Issue reported by: @tranvantoan-qn
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1695233198179649
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~019165a8d87ce7d34b
Upwork Job ID:<PHONE_NUMBER>946873344
Last Price Increase: 2023-09-22

unable to reproduce

Maybe try it on ANDORID/IOS (as showing in attached video here ) may help reproduce it easier. I'm sure it will happens

Hi @ZhenjaHorbach can you please guide me how can I get the setup of Expensify app up and running?

Proposal
Please re-state the problem that we are trying to solve in this issue.
The user enter an incorrect magic code and then modify individual digits, the user unable to log in even when entering the correct code.
What is the root cause of that problem?
After an incorrect OTP attempt, if a user modifies individual digits without clearing out all fields, the length of the OTP value remains six.
src/CONST.ts:    MAGIC_CODE_LENGTH: 6,
src/components/MagicCodeInput.js:    maxLength: CONST.MAGIC_CODE_LENGTH,
const validateAndSubmit = () => {
...
if (!props.shouldSubmitOnComplete || _.filter(numbers, (n) => ValidationUtils.isNumeric(n)).length !== props.maxLength || props.network.isOffline) {
            return;
        }
...

This results in the system believing that a complete OTP has been entered, which in turn triggers the API call for validation.
What changes do you think we should make in order to solve the problem?
Clear the OTP fields upon a failed verification, so It forces users to re-enter the entire OTP. By clearing the code after a failed verification, the length of the OTP value gets reset. This ensures the condition for making an API call (all fields being filled) isn't met until all six fields are populated again.
What alternative solutions did you explore? (Optional)
NA

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01cca145e98bf4d1e2

Can be considered dupe of #28019 as the final solution arrived at that issue should also resolve this issue.


Crash when some directory has no permission to access

Directories which have no permission at all like these could throw an exception:
% ll ~/.rstudio/viewer_history
total 88K
-rw-r--r-- 1 myuser myuser    2 Nov  2 00:19 current_index
-rw-r--r-- 1 myuser myuser  638 Nov  2 00:19 history_entries
d--------- 2 myuser myuser 4.0K Nov  2 00:19 viewhtml28ac27802dcf
d--------- 2 myuser myuser 4.0K Nov  2 00:19 viewhtml28ac2d385cdf
d--------- 2 myuser myuser 4.0K Nov  2 00:19 viewhtml28ac321a80ce
d--------- 2 myuser myuser 4.0K Nov  2 00:19 viewhtml51947ee84612

Of course I cannot even cd to those directories.
% cd viewhtml28ac27802dcf
cd: permission denied: viewhtml28ac27802dcf

And in vim, LeaderF complained when trying accessing those folders:
Error detected while processing function leaderf#File#startExpl[2]..leaderf#LfPy:
line    1:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/myuser/.vim/bundle/LeaderF/autoload/leaderf/python/leaderf/manager.py", line 766, in startExplorer
    self.input(content)
  File "/home/myuser/.vim/bundle/LeaderF/autoload/leaderf/python/leaderf/manager.py", line 47, in deco
    func(self, *args, **kwargs)
  File "/home/myuser/.vim/bundle/LeaderF/autoload/leaderf/python/leaderf/manager.py", line 816, in input
    for cmd in self._cli.input(partial(self.setContent, content)):
  File "/home/myuser/.vim/bundle/LeaderF/autoload/leaderf/python/leaderf/cli.py", line 26, in deco
    for i in func(*args, **kwargs):
  File "/home/myuser/.vim/bundle/LeaderF/autoload/leaderf/python/leaderf/cli.py", line 326, in input
    if callback() == False:
  File "/home/myuser/.vim/bundle/LeaderF/autoload/leaderf/python/leaderf/manager.py", line 778, in setContent
    for i, line in enumerate(itertools.islice(content, 10)):
  File "/home/myuser/.vim/bundle/LeaderF/autoload/leaderf/python/leaderf/asyncExecutor.py", line 80, in read
    raise Exception(lfEncode(lfBytes2Str(err)))
Exception: find: ‘./.rstudio/viewer_history/viewhtml28ac32dfc4ea’: Permission denied
find: ‘./.rstudio/viewer_history/viewhtml28acb279c7c’: Permission denied
find: ‘./.rstudio/viewer_history/viewhtml28ac526efee2’: Permission denied
find: ‘./.rstudio/viewer_history/viewhtml28ac321a80ce’: Permission denied
find: ‘./.rstudio/viewer_history/viewhtml28ac46929574’: Permission denied
find: ‘./.rstudio/viewer_history/viewhtml28ac44ef1337’: Permission denied
find: ‘./.rstudio/viewer_history/viewhtml38d72082d523’: Permission denied
find: ‘./.rstudio/viewer_history/viewhtml28ac3c57cc59’: Permission denied
find: ‘./.rstudio/viewer_history/viewhtml28ac27802dcf’: Permission denied
find: ‘./.rstudio/viewer_history/viewhtml28ac45b5d7b8’: Permission denied
find: ‘./.rstudio/viewer_history/viewhtml28ac3d62d80a’: Permission denied
find: ‘./.rstudio/viewer_history/viewhtml519449e39c31’: Permission denied
find: ‘./.rstudio/viewer_history/viewhtml28ac5570cd28’: Permission denied
find: ‘./.rstudio/viewer_history/viewhtml28ac4f454bbb’: Permission denied
find: ‘./.rstudio/viewer_history/viewhtml28ac2d385cdf’: Permission denied
find: ‘./.rstudio/viewer_history/viewhtml28ac633cf89’: Permission denied
find: ‘./.rstudio/viewer_history/viewhtml28ac74a625a1’: Permission denied
find: ‘./.rstudio/viewer_history/viewhtml28ac59c15b29’: Permission denied
find: ‘./.rstudio/viewer_history/viewhtml51947ee84612’: Permission denied
Error detected while processing function leaderf#File#startExpl:
line    2:
E171: Missing :endif

All settings about LeaderF is by default.

This works as designed, I'm think of that is it necessary to suppress the error message.

@Yggdroot Maybe this trace-back would turn into error info at the bottom of vim when detecting those files in the new version. 😃


The npy file can not download

Thanks for you great job, it really help me.
But the net to download the pretrained GoogleNet npy file can not open. I don't know if it only happen on me. Anyway could you provide it for me?
Thanks so much.

Did you put the correct path of the pre-trained npy file in 'setup_env.py'? 'PARA_DIR' is the path for the npy file.

@SilencerChen  Sorry the link does not work now. I have updated the link for downloading.

In the backpropagation process, how is the error of the last hidden layer connected by the Inception model calculated? Is the sum of the errors of the four branches?


Adds a programmatic node api

Fixes #4
Changes in this PR

Moves all cli-specific code (commander configuration) to cli.js and exports used functions from index.js

Looks good to me, thanks for the contribution!


Param plotting fix

@Bobfrat @birdage @FBRTMaka @oceanzus Let's take a good look at this, even after the passing/failing of the Travis CI build.  A lot of funkiness happened on this.

@Bobfrat @birdage Passed... give it a fast review??

:+1:

@birdage @Bobfrat Can one of you merge?

@DanielJMaher you might wanna test it.


(Perf) Improve Eth Dataflow API: Allow caller to get channel addresses offsets directly

To avoid recomputing offsets every time we want to check the status of a channel (these offset calculations end up adding up), we should cleanup the API to let a user cache a direct pointer to the channel struct and pass that to the various eth dataflow api functions.
For now I've resorted to storing a local pointer direct to each pointer field and essentially reimplementing these functions in the EDM directly. This is not the safest as any changes to the API under the hood will need to be explicitly duplicated/propagated to EDM as well.

FYI @aliuTT, I tentatively co-assigned to you, as the owner of that code, but feel free to unassign yourself if you'd like me to own this instead.


Missing META-INF

Looks like the build.xml file is trying to copy ${jps.dir}/META-INF to ${jps.out}/META-INF, and it seems like ${jps.dir} is actually ./salesforce-jps-plugin and there is no META-INF in there. Was it supposed to be generated?

Maybe it's meant to be salesforce-jps-plugin/src/META-INF?

I tried it to build in IDEA, but no luck, as i'm not experienced in building plugins.
I would be grateful is someone build it and upload somewhere.


Update BWC zip links

Signed-off-by: Bharathwaj G<EMAIL_ADDRESS>Description
Removes opendistro versions and replaces with opensearch 1.1 plugin zip
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

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

@@           Coverage Diff           @@
##             main     #162   +/-   ##
=======================================
  Coverage        ?   83.65%
  Complexity      ?      558
=======================================
  Files           ?       61
  Lines           ?     2050
  Branches        ?      146
=======================================
  Hits            ?     1715
  Misses          ?      230
  Partials        ?      105

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


Get bus error when running

Hello,
I think I have everything set up, however, when I actually access the page from my computer I get:
'pid' bus error  python visualpru.py
from the beaglebone. Not really sure what the deal is. I'm going to investigate.

@pmpbar sounds like you might not have the PRU hardware enabled. Try typing cat /sys/devices/bone_capemgr.*/slots and make sure that you have an overlay loaded which enables the PRU hardware. I highly recommend that you use the universal overlay, since it makes life much easier. Alternative is to take a look in /lib/firmware and export your PRU overlay.

Thanks.


Statement list is not in sync with the backend

The data in the list doesn't seem to be in sync with the backend.

Can you give some more details? Is the statement list available in the backend when accessing with curl? If the same problem happens with curl, it is likely a caching issue (I'll check).

Hi @wastl, see for instance Q354529, dataset FBK-strephit-soccer.
Statement list:

Front end:

API call:
https://tools.wmflabs.org/wikidata-primary-sources/entities/Q354529?dataset=FBK-strephit-soccer&state=approved
{"activities":[{"state":"approved","timestamp":"+2015-09-08T18:15:28Z/14","user":"CristianCantoro"}],"dataset":"","format":"v1","id":19594295,"state":"approved","statement":"Q354529\tP54\tQ6767\tS854\t\"http://www.treccani.it/enciclopedia/massimiliano-allegri/\"","upload":48974752},
...
{"activities":[{"state":"approved","timestamp":"+2015-09-10T12:16:44Z/14","user":"Aubrey"}],"dataset":"","format":"v1","id":18385145,"state":"approved","statement":"Q354529\tP54\tQ6767\tS854\t\"http://biografieonline.it/biografia.htm?BioID=2404&biografia=Massimiliano+Allegri\"","upload":48974752}

This can be closed for the same reason as https://github.com/google/primarysources/issues/71: the source biografieonline.it was (on purpose or not, separate off-issue discussion started by @marfox) blacklisted.


msglist: Throttle fetchOlder retries

This approach is different from how a BackoffMachine is typically used,
because the message list doesn't send and retry requests in a loop; its
caller retries rapidly on scroll changes, and we want to ignore the
excessive requests.
The test drops irrelevant requests with connection.takeRequests
without checking, as we are only interested in verifying that no request
was sent.
Fixes: #945

(Rerunning CI.)

(Working on another update to address a potential race; don't review yet)

The PR has been updated with the proposed changes, and additionally, a check for this.generation == generation to avoid potential races after adding _updateEndMarkers and notifyListeners to the backoff callback.
I have also moved the backoff machine variable, renamed to _fetchOlderCooldownBackoffMachine with Cooldown as a noun, to _MessageSequence right next to fetchOlderCoolingDown, because they are relevant to each other. The backoff machine does not have a public getter and we keep it private.
The loading indicator change might be out of scope for this PR, and we should work on that as a follow-up.

Looks like CI is failing, could you take a look?

Updated the PR. Thanks!

Thanks for the review! This has been updated.

Marking for Greg's review; Greg, please see in particular a question above: https://github.com/zulip/zulip-flutter/pull/1050#discussion_r1877185303

Then, either here or as a followup, we could give the UI an "error"/"problem" state that's distinct from the loading state. If the request has failed, and especially if it's failed several times, that's an important sign that maybe the user should stop expecting it to succeed.

(Filed #1126 for this.)

Updated the PR. Thanks for the review!

Thanks! Looks good — rebased, and will watch CI.


stable/prometheus-operator Back off limit exceeded

Describe the bug
When installing fresh the admission jobs fail with backoff limit exceeded
Version of Helm and Kubernetes:
Helm
Client: &version.Version{SemVer:"v2.15.1", GitCommit:"cf1de4f8ba70eded310918a8af3a96bfe8e7683b", GitTreeState:"clean"

Kubectl
Client Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.2", GitCommit:"c97fe5036ef3df2967d086711e6c0c405941e14b", GitTreeState:"clean", BuildDate:"2019-10-16T17:33:45Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.8", GitCommit:"211047e9a1922595eaa3a1127ed365e9299a6c23", GitTreeState:"clean", BuildDate:"2019-10-15T12:02:12Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"linux/amd64"

Which chart:
prometheus-operator
What happened:
Error: Job failed: BackoffLimitExceeded

What you expected to happen:
For the admission webhooks to install
How to reproduce it (as minimally and precisely as possible):
helm install --version=7.0.0 --name prometheus stable/prometheus

Anything else we need to know:
Disabling the webhooks then installing installs the chart however the operator fails to run.
Then running upgrade will install the admission webhook then the chart will fail with
Error: Job failed: BackoffLimitExceeded
Error: UPGRADE FAILED: Job failed: BackoffLimitExceeded

Deleting the jobCreateSecret.yaml from the chart and then re-running the helm upgrade will finally succeed however reverting the jobCreateSecret.yaml will cause the issue to occur again.
The admission-created job is completed
Pod
ca-prometheus-prometheus-o-admission-create-tft4j                 0/1     Completed   0          104s

Job
ca-prometheus-prometheus-o-admission-create   1/1           2s         77s

However it also not being removed

I've confirmed in a separate cluster that this doesn't occur all details are the same.
Possibly race condition in the admission create?

Could you please post the logs from job pods?

I've been unable to replicate it again. I'll close this off and if it occurs again I'll ensure I have all the appropriate logs.
Thank you


TransactionOutput deserialisation issues

I recently (approx. 17 days ago) launched Carp on a server and had it sync against the blockchain/Oura.
I want to deserialise the payload in the TransactionOutput table to the TransactionOutput class in Typescript/Javascript using the library https://www.npmjs.com/package/@dcspark/cardano-multiplatform-lib-nodejs.
However, I keep getting an error Variant not found.
Interestingly, I can successfully deserialise an object like TransactionMetadata from it's payload in the Carp db.
I was wondering if there's been a change to the way that TransactionOutput is being serialized? (that ends up throwing off deserialisation)?
Also,if the answer to the first question is 'Yes', what version of the npm package should I use that is compatible with the latest serialisation format of TransactionOutput?

If you're using the latest version of multiplatform-lib, it should work
Keep in mind that Byron (very old outputs) are a different type than everything after Shelley so the type is different for that one

Ooh, I see. Didn't know that. Trying out with outputs from more recent epochs/blocks.
Will let you know.
Thanks!

Btw, where is the TransactionOutput serialisation code located? Is that automatically handled by serde's serialisation lib? Any specific advice if one wants to deserialise it using another library? (custom one, maybe another language). Is it cbor?

I just tried deserialising this, and it still failed:
82584c82d818584283581cceb6369411755dcd25a28c08b11d2b1c90a1503e4cbc12ac324d0c76a101581e581c2b0b011ba3683d6c65c3ef2a7a3ff08e39a526fae99d163c4b038331001a2a3ac8661a7a308480

@omefire what is the transaction hash this output belongs to?

Transaciton payload: \x84a500828258200147c4657d86232ee93777dad5c6008b4f541a05aa6c2cebd9099f81e2496970018258209d0dde8c785534db15bc58a9764fe2d11c01de7c8a4e1da19642e2a1d7060dfe01018283581d714a59ebd93ea53d1bbf7f82232c7b012700a0cf4bb78d879dabb1a20a821a001cd277a1581ca5bb0e5bb275a573d744a021f9b3bff73595468e002755b447e01559a156484f534b594361736847726162303030303237383730015820577e10ec0580b6056c50cbdc9efbf8a64414549d1341520170139f494c4d790582583901568df7b66d34c998c1106241d502d905a550314f8a9b11397f8444862bf217ee86f7975e7a11b0df4c07155ef19a7a25881632bb383d5e62821a05b3d5bfa2581c0d27ee4630ee8a9e62664ccd8e61fc43a62699ae28f3718b4a66a36ba14e50657273697374656e636539343801581ce30726d8b91be8e8f3860db0eaccfa6b7c038b2a13bfd8e3df18b9bba14b5061636b436f696e32323401021a00033da7031a0329f6f90758207f7b854a91e18fd09eee3ad063d28944a58147da483a7e484abf5e80aecee0f4a100818258206e8596ba9abdd0f7306d6de68613a076b18594d98a92d7e63ec44a0d1f56a3ac584004bfac722ba3fbe8437461614039a9d52990e6e580c106666293d5fa2c320386233b0b5229f0bbc220636733beff3884adca410672c5b6612c6d8cbdb8185609f5a800783835363864663762363664333463393938633131303632343164353032643930356135353033313466386139623131333937663834343438360167383030303030300278386634346638373531663033643736376261353163346665393838656432383962376365643463376538333232323364343462333162636365036231300678406164647231713974676d61616b643536766e7878707a7033797234677a6d797a3632357033663739666b79666530377a796670337437677437617068686a613007782738357964736d6178717739323737786438356676677a6365746b77706174653371646d777532370878406164647231713836796c7036333771376876376139723338376e7a3864397a6468656d32763036706a796737356676636d656e337267387434713366383072350978273670393378717a686375703077376535686571376c6e61796a7a716175336466733779726c7335

Transaction hash:
\x52f98f3862f73057a3d7049b6f97d71c5cc2da3d3450583316a1ca0cf7e84cba

Are you sure you posted the right bytes for the TransactionOutput?
I see the full transaction payload you posted does give the transaction hash you provided, but the two outputs for this transaction have a different payload than what you posted
Instead, I get
83581d714a59ebd93ea53d1bbf7f82232c7b012700a0cf4bb78d879dabb1a20a821a001cd277a1581ca5bb0e5bb275a573d744a021f9b3bff73595468e002755b447e01559a156484f534b594361736847726162303030303237383730015820577e10ec0580b6056c50cbdc9efbf8a64414549d1341520170139f494c4d7905

and
82583901568df7b66d34c998c1106241d502d905a550314f8a9b11397f8444862bf217ee86f7975e7a11b0df4c07155ef19a7a25881632bb383d5e62821a05b3d5bfa2581c0d27ee4630ee8a9e62664ccd8e61fc43a62699ae28f3718b4a66a36ba14e50657273697374656e636539343801581ce30726d8b91be8e8f3860db0eaccfa6b7c038b2a13bfd8e3df18b9bba14b5061636b436f696e32323401

I did change to a different TransactionOutput when looking for one that is more recent.
The TransactionOutput I posted belongs to a Transaction that happened more recently, and it still fails to deserialise.
In addition, the two outputs you posted also fail to deserialise.

It seems to deserialize fine for me
TransactionOutput::from_bytes(
    hex::decode(
        "83581d714a59ebd93ea53d1bbf7f82232c7b012700a0cf4bb78d879dabb1a20a821a001cd277a1581ca5bb0e5bb275a573d744a021f9b3bff73595468e002755b447e01559a156484f534b594361736847726162303030303237383730015820577e10ec0580b6056c50cbdc9efbf8a64414549d1341520170139f494c4d7905"
    ).unwrap()
).unwrap();

I used the following code (Typescript):

let txOutStr: string = "83581d714a59ebd93ea53d1bbf7f82232c7b012700a0cf4bb78d879dabb1a20a821a001cd277a1581ca5bb0e5bb275a573d744a021f9b3bff73595468e002755b447e01559a156484f534b594361736847726162303030303237383730015820577e10ec0580b6056c50cbdc9efbf8a64414549d1341520170139f494c4d7905";

let txOut = wasm.TransactionOutput.from_bytes(Buffer.from(txOutStr));

wasm.TransactionOutput.from_bytes(Buffer.from(txOutStr, 'hex'));

If you don't specify hex, it will try and parse the string as utf8

Haha, thanks very much!

Strange that it did work for other types like Metadata, hmmm


File listed in output are not correct

terrascan version: 1.6.0
Operating System: MacOS catalina (10.15.7)

Description
When running terrascan scan, violations that occur in modules do not provide useful file reference in output. Example output:
Description    :	Ensure S3 buckets have access logging enabled.
	File           :	../../../../../var/folders/vl/1zz0rcwn471_qlq4dnmr2h2dfc7zw3/T/dbhhh7/main.tf
	Plan Root      :	environments/nonprod-shared
	Line           :	105
	Severity       :	MEDIUM

What I Did
terrascan scan -d environments/nonprod-shared/

Output:
	-----------------------------------------------------------------------

	Description    :	Ensure that S3 Buckets have server side encryption at rest enabled with KMS key to protect sensitive data.
	File           :	../../../../../../../var/folders/vl/1zz0rcwn471_qlq4dnmr2h2dfc7zw3/T/bvnp4o/main.tf
	Plan Root      :	./
	Line           :	5
	Severity       :	HIGH
	-----------------------------------------------------------------------

	Description    :	Ensure S3 buckets have access logging enabled.
	File           :	../../../../../../../var/folders/vl/1zz0rcwn471_qlq4dnmr2h2dfc7zw3/T/l0oepu/main.tf
	Plan Root      :	./
	Line           :	105
	Severity       :	MEDIUM
	-----------------------------------------------------------------------

	Description    :	Ensure S3 buckets have access logging enabled.
	File           :	../../../../../../../var/folders/vl/1zz0rcwn471_qlq4dnmr2h2dfc7zw3/T/we3rxq/main.tf
	Plan Root      :	./
	Line           :	5
	Severity       :	MEDIUM
	-----------------------------------------------------------------------

	Description    :	Ensure S3 buckets have access logging enabled.
	File           :	../../../../../../../var/folders/vl/1zz0rcwn471_qlq4dnmr2h2dfc7zw3/T/bvnp4o/main.tf
	Plan Root      :	./
	Line           :	5
	Severity       :	MEDIUM

None of the above folders actually exist. I would expect that the file should reference the location in the consuming module.

Hi @jhancock93 ,
Thank you for reporting this issue. I have a few queries:

Which IaC are you using, is it terraform?
Is your IaC referencing a remote module? May be a remote terraform registry or a git repository?
Would it be possible to run terrascan with -l debug option and upload the output in a file with us for better debugging this issue?

Hi @jhancock93 ,
Thank you for reporting this issue. I have a few queries:

Which IaC are you using, is it terraform?
Is your IaC referencing a remote module? May be a remote terraform registry or a git repository?
Would it be possible to run terrascan with -l debug option and upload the output in a file with us for better debugging this issue?

I am using terraform and my IaC is referencing both local and remote modules, both terraform registry and git repositories.
I probably can't send you the output of the scan for the aforementioned scan, but let me see if I can reproduce the issue with a stripped down example.

Hey @jhancock93 ,
Thank you for your quick response.
I have a quick follow up question:
Do you see the above incorrect file paths for all the resource violations or only for resources in remote modules?

Hey @jhancock93 ,
Thank you for your quick response.
I have a quick follow up question:
Do you see the above incorrect file paths for all the resource violations or only for resources in remote modules?

It looks like local modules report the file path correctly, so it is an issue with resources in remote modules.

So @jhancock93 , the problem here is that terrascan downloads the remote modules in the tmp dir and scans it from there. That's the reason we are seeing those tmp directory file paths. Terrascan deletes the downloaded modules from tmp dir once the scanning is complete.
We have opened #841 with a bit of technical details and would try to fix this issue. Thank you!

So @jhancock93 , the problem here is that terrascan downloads the remote modules in the tmp dir and scans it from there. That's the reason we are seeing those tmp directory file paths. Terrascan deletes the downloaded modules from tmp dir once the scanning is complete.
We have opened #841 with a bit of technical details and would try to fix this issue. Thank you!

Would it also be possible to include the file location that is instantiating the remote modules? Ideally, the report would include both the line number in the local codebase where the module is instantiated. The user may not be able to change the remote module, but ideally they would be able to include a ts:skip directive, at least as a temporary measure.


Add setting to choose display unit

Currently the extension always shows BG in mg/dL. I would like to see my values in mmol/L instead.
Either as a setting in the plugin, or by requesting the setting from nightscout would work.

the unit is now automatically pulled from the user's nightscout site main profile.

although you can't currently manually choose between mg/dL and mmol/L, it's now always the same as on the user's nightscout site.


Generate Fake User for Seeding

Title: Generate Fake User for Seeding
Description:
This GitHub issue aims to create a function named generateFakeUser for generating a fake user that can be used for seeding purposes. The function should utilize various libraries and utilities to generate realistic-looking user data.
Tasks:

Implement generateFakeUser Function:

[ ] Create a function named generateFakeUser in the seeder/utils.ts file.
[ ] Generate a fake user with the following properties:

Email: Generate using the faker library.
Password: Generate using the faker library.
Username: Generate using the faker library.
User ID (UUID): Generate using the faker library.
Full Name: Generate using the faker library.

[ ] Return the generated user data as an object.

Acceptance Criteria:

The generateFakeUser function is successfully implemented.
The function generates a fake user with the specified properties.
The generated user data is returned as an object.

Additional Information:

This function is intended for seeding purposes and will generate a fake user with realistic-looking data.
The issue should be considered resolved once the function is successfully implemented and tested.

@prasenjeet-symon Review my pull request #151

@prasenjeet-symon Review my pull request #151

Merged successfully


Retry AttachNetwork when it fails to find network

When trying to attach to swarm scope network for an unmanaged container
sometimes even if attaching to network succeeds, we may not find the
network because some other container which was using the network went
down and removed the network. So if it is not found, try to detach and
reattach to re-download the network from the manager.
Fixes #26588
Signed-off-by: Jana Radhakrishnan<EMAIL_ADDRESS>

ping @mavenugo

LGTM

@thaJeztah Added a retry limit.

SGTM (not a maintainer)

ping @thaJeztah


Port binding/specs: external port min empty string handling is invalid

The fix for #31 copied behavior from docker-compose port spec parsing: https://github.com/docker/docker-py/blob/master/docker/utils/ports.py#L32, but forgot about the handling of empty string min external ports (e.g. :8000-8001) the task is to review whether the behavior is consistent with docker in case of port specs like :8000-8001.

Behavior is consistent


Form submission element is absent from event.submitter

When I submit a , I'd like to know which element submitted the form.
In native HTML, I have access to the element that submitted the form via event.submitter
Read more about the event.submitter spec here
Example
// JS
document
.querySelector("form")
.addEventListener("submit", (event) => {
let submitter = event.submitter;
});
// HTML

First
Second

Expected:
event.submitter should be a reference to the  DOM node
Received:
event.submitter is null

I've also noticed that the submit button/element does not appear in formData object on submission events.
If a button is used to submit a form and also includes a name/value pair, it should be in FormData.
Read more here
Example
// JS
document
    .querySelector("form")
    .addEventListener("submit", (event) => {
        let formData = new FormData(event.currentTarget);
        formData.get('FirstButton') // should be reference to <button/> but is undefined
    });

// HTML
<form>
    <button name='FirstButton' value='FirstValue'>First</button> // Click
    <button name='SecondButton' value='SecondValue'>Second</button>
</form>

I think this is a good feature request. I think that the submit() function could be updated to accept an optional submitter param, and also be updated to conditionally use form.requestSubmit(submitter) if it is available?
Regarding the request about FormData, the submitter is the second arg to new FormData(form, submitter) so when creating a FormData, you can add the submitter.
However, I think maybe the formValues() helper could also be updated to accept an optional submitter param so that param can be passed to the new FormData() construction that happens in formValues()

I think the names should be similar for the buttons only their values different. check https://www.youtube.com/watch?v=w2i-9cYxSdc&pp=ygUZUmVtaXgganMgc2luZ2xlIG11bHRpZm9ybQ%3D%3D


Update worlds.yml

Update the list of city spawn points to remove inactive cities and to remove duplicate spawn points

We have no reason to remove these, as we have no official metric for inactivity we could use to fairly remove cities selectively, there are still builds in these locations, and these spawns will only be used if there is a player in range


Octopus Deploy: Failed status is treated as "Running"

Describe the bug
A failed Octopus Deploy task continues to show as "Running".
To Reproduce
Steps to reproduce the behavior:

Configure a Duck instance with an Octopus Deploy collector.
Change the status of the Octopus task shown in Duck to "Failed". This can be done on the deployment details page in Octopus.
View the Duck dashboard.
See that the task is running, not failed.

Expected behavior
A failed Octopus deployment should appear as failed in Duck.
Additional context
It looks like the cause is here: https://github.com/duckhq/duck/blob/master/src/providers/collectors/octopus/mod.rs#L174
"Failed" is in the list of statuses that map to "Running". The fix should be removing "Failed" from that list and letting the wildcard catch it and map it to failed.
I can use this issue as an opportunity to get my dev environment set up for Duck and submit a PR.

@ryanrousseau Great catch! Funny thing is that we've been using this in prod for quite a while and for some reason I've never seen this before. Probably because Octopus does such an awesome job and we almost have any failed deployments.
Looking forward to the pull request. Reach out if you run into any problems!

Luckily my home lab is still in the early stages so I run into failures a bit more often than usual :D
Local dev setup was very smooth. The only trouble I ran into was that the duck-ui container didn't start up when running docker-compose up the first time, but I think that was just a fluke. It's worked fine since then. PR incoming 👍


Scheide related issue:  references sometimes displays gibberish

Some digital objects are displaying several lines of coding with the label "references", rather critically at the moment this includes our Gutenberg Bible.
https://dpul.princeton.edu/scheide/catalog/7d278t10z#?c=0&m=0&s=0&cv=0&xywh=-9882%2C-401%2C24996%2C8000

Actually, I haven't seen any record with data attached to this label which is of any benefit.  Is the solution here just to suppress it from display?
The label "references" was making me think it would/could hold something useful.

I think we should probably just suppress this from bibdata's JSON-LD view so it never gets imported.

Resolved by https://github.com/pulibrary/marc_liberation/pull/405


Small change to QA email view

Context
As a content designer, I can't see the reference name of the email so I have to jump back and forth. Also, without the name, if you're looking at a number of different emails it gets confusing.
Changes proposed in this pull request
Add the mailer and email reference names to the email preview header
Before

After

Guidance to review

commit by commit
the first commit is a straight copy from the Rails repo
do we need tests here?
this change isn't scoped by environment. Is that a problem?

Link to Trello card
https://trello.com/c/5Lf2AxrW/3033-not-a-small-change-to-qa-email-view
Things to check

[x] This code does not rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] New environment variables have been added to the Azure config

@malcolmbaig the render_preview_wrapper method in the mail-notify gem is private. Should we be able to do this? Does this surprise you?

Looks good to me, thanks Raam!

Suggested a few tweaks, after which the header will look more like this:

@paulrobertlloyd Cheers for the styling updates, it looks much neater now

@malcolmbaig the render_preview_wrapper method in the mail-notify gem is private. Should we be able to do this? Does this surprise you?

Ideally the gem would expose some public mechanism for us to declare which template is used for the preview wrapper. In the absence of that, I think overriding the private method is fine - especially as the worst that can happen is that previews start breaking after a mail-notify version update.
An alternative option with a smaller footprint—and worth considering applying here—is the following:

Add prepend_view_path('spec/mailers/previews') just before this line https://github.com/DFE-Digital/apply-for-teacher-training/commit/e901fbad662db5b08faa18652b03e582c4b7e5f4#diff-e530c57fe094b50f394218dfd0b938dda863d461c8a1d61301dece5925367048R4
Rename our custom template to spec/mailers/previews/rails/mailers/email.html.erb
Remove the render_preview_wrapper override.

The render_preview_wrapper in mail-notify will then look for the email template in our directory before searching the default view paths.

@malcolmbaig thanks again for the help with this. I've made those changes now


Added a new code in graph section which is asked in some coding contests.

Description of Change

Checklist

[ ] Added description of change
[ ] Added file name matches File name guidelines
[ ] Added tests and example, test must pass
[ ] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[ ] Relevant documentation/comments is changed or added
[ ] PR title follows semantic commit guidelines
[ ] Search previous suggestions before making a new one, as yours may be a duplicate.
[ ] I acknowledge that all my contributions will be made under the project's license.

Notes:

@MuskanTripathi  If you review the contributions guidelines and the readme file for the repo according to the provided check-list in the pull-request description, you will be able to fix the errors and improve the code to the repo standards.


Check argument types in contract calls (RFC)

Just a first progress PR. I don't intend to merge this, but I welcome comments on the approach.

I think the approach looks sound :-)

I'm dropping this in favor of the rebased PR https://github.com/aeternity/epoch/pull/1696


Alternative for S3MockApplication.getPort()

Hi there,
version 2.4.x has deprecated the S3MockApplication.getPort() method. Which alternative that isn't deprecated do you recommend to determine the port of the S3 Endpoint URI from within JUnit 5? So far we used URI.create("https://localhost:%s".formatted(s3MockApplication.getPort())).
Thanks,
Thilo

Thilo,
sorry, I should have added the deprecation notice description to the JavaDoc, and will do so soon. Thanks for pointing that out!
In general, using the S3Mock directly through it's Java API is now discouraged.

Rationale:
Starting and using the S3Mock directly (in a JUnit test) is and always will be problematic because the dependencies and the framework we use (Spring Boot) directly interferes with the classes / dependencies / framework you are using in your own application.
Unfortunately, any public class and method, and even the Java baseline is (unintentionally) part of the public API.
Any customer could be using any number of methods, and I could be breaking any number of integrations with a simple refactoring.
I have heavily refactored the code base in the last couple of months and I'm far from done.
For example:
The controller classes are too large by far, I'd like to split those up.
With the current way we persist binaries and metadata, it is hard to add additional features like ACLs or versioning, which people have asked about. I'll probably end up rewriting the whole FileStore.
Spring Framework 6 and Spring Boot 3 will be released later this year, raising their Java baseline to 17. Once I update, this will update the Java baseline of the plugin as well (which will almost definitely break a lot of customer projects, as any customer directly using the S3Mock would have to update to Java 17+ as well).
I am also toying with the idea of refactoring the whole thing into Kotlin (again, breaking a lot of direct integrations, although the binary code produced should be compatible to Java 8+).

The best way to insulate both the application under test and the S3Mock is by running two separate JVMs. This can easily be done by running the S3Mock in a Docker container, either by starting up the Docker container (e.g. through Maven or Gradle) before running the tests on your application, or by using TestContainers
Cheers,
Arne


[qa] Added commit message style check #64

Closes #64

Pull Request Test Coverage Report for Build 223

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

Totals

Change from base Build 222:
0.0%

Covered Lines:
465

Relevant Lines:
468

💛  - Coveralls

Thanks 😄 @nemesisdesign


[SPARK-28891][BUILD][2.3] backport do-release-docker.sh to branch-2.3

What changes were proposed in this pull request?
This PR re-enables do-release-docker.sh for branch-2.3.
According to the release manager of Spark 2.3.3 @maropu, do-release-docker.sh in the master branch. After applying #23098, the script does not work for branch-2.3.
Why are the changes needed?
This PR simplifies the release process in branch-2.3 simple.
While Spark 2.3.x will not be released further, as @dongjoon-hyun suggested, it would be good to put this change for

to reproduce this release by others
to make the future urgent release simple

Does this PR introduce any user-facing change?
No
How was this patch tested?
No test is added.
This PR is used to create Spark 2.3.4-rc1

cc @srowen @dongjoon-hyun

I asked @maropu who is the release manager of Spark 2.3.3. He said that he used do-release-docker.sh in the master branch for releasing Spark 2.3.3.
As I wrote in the description, I used these scripts to release Spark 2.3.4-rc1.

@srowen thank you for your comment. got it.

Since the vote passed, I'll merge this to branch-2.3 as a last commit on branch-2.3.

Thank you, @kiszk and @srowen .


Sort FoldScopeLocations to ensure predictable output order

for multiple folds-- was causing a Python2 test to fail earlier but not Python3, oddly enough.

Python 2 dicts don't have a defined iteration order, but Python 3 dicts are ordered by default.


cardano-api: IsPlutusScriptLanguage is not exported

This disallows constructing HasTextEnvelope (PlutusScript lang) dictionaries for dynamic lang.

https://github.com/input-output-hk/cardano-node/pull/4554


has the tabs overview disappeared in the latest nightly version?

Usually when I use tabs, these are displayed on a top bar of the editor, together with the name of the split file under focus, and the total number of splits. This is very convenient. For example:
With:
~/Downloads$ ./Onivim2-x86_64-stable.AppImage --version
Onivim 2 (0.5.4)

I get:

But with the latest nightly build:
$ oni2 --version
Onivim 2 (0.5.9-nightly)

I get by contrast, even with several tabs open:

Are you able to reproduce / is this a bug?
If you are not able to reproduce, any idea what the problem is? I am running both on the same computer, would expect the same packages and config apply.

Hi @jerabaul29 ,
Thanks for the screenshots! Very helpful.

Are you able to reproduce / is this a bug?

At first I wasn't able to reproduce - but was able to do with this:

oni2 test.c
:tabnew test.h

On 0.5.4 - switches out of zen mode and shows tabs. On 0.5.9 - stays  in zen mode, tabs stay hidden.
Indeed it is a bug - possibly a regression from https://github.com/onivim/oni2/pull/3160

Thanks @jerabaul29!
It looks like this changed in https://github.com/onivim/oni2/pull/3160 - the behavior is controlled by a configuration setting, "editor.zenMode.hideTabs": true - looks like the default was tweaked in that PR. If you prefer the previous behavior, you can try setting this to false instead :+1:


Variable prefix?

Hello, thanks for role (and promtail one)
Wanted to ask if it's possible to change variable prefix to loki_canary_*, which will be also more consistent with role name (loki-canary).

Thank you for the feedback - is there any specific problem you are currently encountering with the variable names?
As the variable names are already quite long, I would then rather prefer to use the "ansible-openshift" convention of using the first two letters of the role name for the variable:
https://github.com/openshift/openshift-ansible/blob/master/docs/style_guide.adoc#ansible-role-variables
This would result in lc_* variables - however to me this seems a bit more confusing and might more easily overlap with other roles
Do you have another community document where best-practices regarding variable names are documented?

Sorry for late reply

is there any specific problem you are currently encountering with the variable names?

I happen to have canary role myself, internal app. I know, my problems, but still.

This would result in lc_* variables - however to me this seems a bit more confusing and might more easily overlap with other roles

Better than canary_ for sure, but probably can be confused with internal linux LC_* (locale) variables.
I do not know about any guidelines for variable names.


I  have a  big  bugs ! help!

Why  i get  http://httpbin.org/ip
result is
{
"origin": "<IP_ADDRESS>, <IP_ADDRESS>"
}
origin hava  2 ip ?

Duplicate of #535.


Enabling Grafana-wide variables to be used in cell links

Enabling Grafana-wide variables to be used in cell links

this should address #9


Version parsing from namespace bug fix

When some controller namespace contains segment starting with v, and the next segment contains actual version, then VersionByNamespaceConvention skips second segment due to dividing dot capture.
For example version parsing fails with namespace VersioningSample.V1.Controllers.
This fix uses regex lookahead on trailing dot.

I've actually gone ahead and fixed the issue separately in #377, but referenced your PR. The final pattern change just needed to be a set rather than a non-capture group. I added your example to the test suite to prevent future regressions. Thanks for not only reporting the issue, but taking the time to try and fix it. It's appreciated.


actions testing

Noting some change details here for future reference.

The main driver for this work was the removal of the enterprisedb.com package underpinning the EmbeddedPostgres.scala module. This caused multiple test suites to abort, presenting errors on de.flapdoodle, ru.yandex, and eventually 403s on enterprisedb.com requests if you looked deep enough

2024-07-09T05:42:09.5724215Z ^[[0m[^[[0m^[[0minfo^[[0m] ^[[0m^[[0m^[[31m  at de.flapdoodle.embed.process.store.Downloader.downloadInputStream(Downloader.java:131)^[[0m^[[0m
2024-07-09T05:42:09.5726866Z ^[[0m[^[[0m^[[0minfo^[[0m] ^[[0m^[[0m^[[31m  at de.flapdoodle.embed.process.store.Downloader.download(Downloader.java:69)^[[0m^[[0m
2024-07-09T05:42:09.5729125Z ^[[0m[^[[0m^[[0minfo^[[0m] ^[[0m^[[0m^[[31m  at de.flapdoodle.embed.process.store.PostgresArtifactStore.checkDistribution(PostgresArtifactStore.java:66)^[[0m^[[0m
2024-07-09T05:42:09.5731341Z ^[[0m[^[[0m^[[0minfo^[[0m] ^[[0m^[[0m^[[31m  at de.flapdoodle.embed.process.runtime.Starter.prepare(Starter.java:52)^[[0m^[[0m
2024-07-09T05:42:09.5733508Z ^[[0m[^[[0m^[[0minfo^[[0m] ^[[0m^[[0m^[[31m  at de.flapdoodle.embed.process.runtime.Starter.prepare(Starter.java:45)^[[0m^[[0m
2024-07-09T05:42:09.5735678Z ^[[0m[^[[0m^[[0minfo^[[0m] ^[[0m^[[0m^[[31m  at ru.yandex.qatools.embed.postgresql.EmbeddedPostgres.start(EmbeddedPostgres.java:193)^[[0m^[[0m
2024-07-09T05:42:09.5737964Z ^[[0m[^[[0m^[[0minfo^[[0m] ^[[0m^[[0m^[[31m  at ru.yandex.qatools.embed.postgresql.EmbeddedPostgres.start(EmbeddedPostgres.java:165)^[[0m^[[0m
2024-07-09T05:42:09.5740613Z ^[[0m[^[[0m^[[0minfo^[[0m] ^[[0m^[[0m^[[31m  at ru.yandex.qatools.embed.postgresql.EmbeddedPostgres.start(EmbeddedPostgres.java:139)^[[0m^[[0m
2024-07-09T05:42:09.5742686Z ^[[0m[^[[0m^[[0minfo^[[0m] ^[[0m^[[0m^[[31m  at hmda.utils.EmbeddedPostgres.beforeAll(EmbeddedPostgres.scala:51)^[[0m^[[0m
2024-07-09T05:42:09.5744608Z ^[[0m[^[[0m^[[0minfo^[[0m] ^[[0m^[[0m^[[31m  at hmda.utils.EmbeddedPostgres.beforeAll$(EmbeddedPostgres.scala:50)^[[0m^[[0m
2024-07-09T05:42:09.5745848Z ^[[0m[^[[0m^[[0minfo^[[0m] ^[[0m^[[0m^[[31m  ...^[[0m^[[0m
2024-07-09T05:42:09.5747969Z ^[[0m[^[[0m^[[0minfo^[[0m] ^[[0m^[[0m^[[31m  Cause: java.io.IOException: Server returned HTTP response code: 403 for URL: http://get.enterprisedb.com/postgresql/postgresql-11.1-1-linux-x64-binaries.tar.gz^[[0m^[[0m

The decision was taken to migrate to testcontainers-1.19.8, the current latest version. It seems to be working well as a drop in replacement at this time.
A fix was put in to prevent empty lists (List()) from being csv-printed and re-initialized as single-empty-string lists (List("")). This primarily affected the "Institution CSV Parser must parse values into CSV" test.
An additional suite abort problem was resolved by increasing slick task queue sizes to 100 where they were previously 10. (The default is 1000.) This originally presented with RejectedExecutionExceptions.


copy and paste is not working in Safari

I documented this problem in the MacOs section for now. But it seems Guacamole needs to change how it talks to the clipboard?


Previous Files Opening projects with Cyrillic characters in path

Error in hou module.
Traceback (most recent call last):
  File "C:/Users/MarkWilson/Documents/Hammer-Tools/python2.7libs\hammer_tools\previous_files.py", line 176, in __call__
    self.logEvent(hou.hipFile.path(), SessionWatcher.EventType.Save)
  File "D:/HOUDIN~1.556/houdini/python2.7libs\hou.py", line 40187, in path
    return _hou.hipFile_path(*args)
UnicodeDecodeError: 'utf8' codec can't decode byte 0xf0 in position 3: invalid continuation byte

From Vladyslav:
Houdini: 18.0.597
OS: Windows 10

Recent files can open this file.


CLI freezer

Hi Simon,
First of all, thanks for the package. It works beautifully.
Having said that, I think it would be nicer if it comes with a command line interface that generates static files without worrying about custom URL generators. So I'm proposing a command with the following interface:
$ flask-freezer [OPTIONS] APP_MODULE space-separated-urls < additional-new-line-separated-urls

where:

[OPTIONS] are the same as current frozen_flask options. (i.e. destination, destination-ignore, etc)
APP_MODULE is of the pattern $(MODULE_NAME):$(FLASK_VARIABLE_NAME).
space-separated-urls is a list of urls to be used in the generator

Additionally, the command can accept list of urls via stdin.
Internally, it would a create  Freezer instance and with an URL generator that iterates over the URLs.
Thoughts?

I’m not convinced this is that useful (you’re already writing Python code for your app anyway), and I’m slightly worried that this command-line interface will grow (through requests from various people) until it exposes (and duplicates) the entire Python API.


Feature request: add lighten and darken util

Question or Suggestion
I'm in the middle of reworking the neogit theme for cyberdream and need to use a darker color of red/green in git diff bg. I'm trying to avoid hard coding a specific color, so I remembered that tokyonight has a util where we can lighten/darken any of the theme colors. His util for that can be found here: https://github.com/folke/tokyonight.nvim/blob/main/lua/tokyonight/util.lua
This would allow a little more of flexibility without having to add additional colors to theme itself..
Which you can call it anywhere you need it with:
local util = require("tokyonight.util")
highlight = { fg = util.lighten(colors.red1, 0.3) } -- number between 0 and 1. 0 results in white, 1 results in red1

I thought maybe it would be a good idea to implement something similar to cyberdream.
Also a quick question, is it ok to set global variables in the extension, like so:
local base = {
        txt1 = t.fg,
        txt2 = t.bgHighlight,
        bg1 = t.bg,
        bg2 = t.bgAlt,
        bg3 = t.bgHighlight,
}
local accent = {
        p1 = t.orange,
        p2 = t.green,
        p3 = t.blue,
        p4 = t.red,
        s1 = t.cyan,
        s2 = t.pink,
        s3 = t.magenta,
        s4 = t.yellow,
        s5 = t.purple,
}

So I can reference it in the actual highlights?
 local highlights = {
        NeogitNormal = { bg = base.bg1 },
        NeogitStatusHEAD = { bg = base.bg1 },
        NeogitFold = { fg = base.txt1 },
        NeogitRemote = { fg = accent.s1 },
        NeogitBranch = { fg = accent.s2, bold = true },
        NeogitBranchHead = { fg = accent.s1, bold = true },
        NeogitWinSeparator = { fg = base.bg3, bg = t.bg1 },
        NeogitChangeDeleted = { fg = accent.p4, bold = true, italic = true },
        NeogitChangeModified = { fg = accent.s1, bold = true, italic = true },
        NeogitSectionHeader = { fg = accent.p1, bold = true },
        NeogitCommitViewHeader = { fg = accent.p1, bg = base.bg3 },
}

Thanks!

Hey @gldtn!
Thanks for the feedback 🙂 I think most of what you're trying to do can be achieved without too much effort.

util where we can lighten/darken any of the theme colors

I already do something very similar with blend which takes in two colours and a weight in order to generate new colours. You can blend an existing colour with fg or bg to lighten or darken it.
One of the main reasons I prefer this approach is because the same highlight will darken/lighten dynamically based on whether the user is using the light or dark variant.
For example, lightening a foreground colour in dark mode won't have much impact on it's readability. In light mode, you risk lowering contrast to the point where text becomes difficult to read.
For diffs, I recommend linking to the already defined Diff... groups that already use the blend function. You can see an example of this here in gitsigns.lua.

is it ok to set global variables in the extension

Yes, since these are actually locals not globals 😉
However, the example you provided doesn't make a whole lot of sense to me. What benefit does this provide aside from a slightly shorter name for some colours? It makes it harder to tell what colour is being used when looking at the highlights.

For example, lightening a foreground colour in dark mode won't have much impact on it's readability. In light mode, you risk lowering contrast to the point where text becomes difficult to read.

That makes sense.

For diffs, I recommend linking to the already defined Diff... groups that already use the blend function. You can see an example of this here in gitsigns.lua.

The problem is that Neogit don't use a separate highlight for text color, so I would need to  use nvim_get_hl to set fg/bg as far as I know.
So I ended up just using the blend util, something like:
NeogitDiffAdd = {
            fg = util.blend(t.bg_solid, t.green, 0.3),
            bg = util.blend(t.bg_solid, t.green, 0.8),
        },

Unless you can think of a more clever way..

Yes, since these are actually locals not globals 😉

Lol I meant locals, thanks!

However, the example you provided doesn't make a whole lot of sense to me. What benefit does this provide aside from a slightly shorter name for some colours? It makes it harder to tell what colour is being used when looking at the highlights.

While you are right it's currently acting more of a schema, especially since I'm still testing out to see what colors works best. This way If I want to test everything with a different background color I don't have to alter each individual highlight, I can just change the bg variable above.
Another question, how do you reload highlights without exiting nvim? I tried a few things, but no luck. Do you use autocmds or anything to help you out?

Ah, right. Understood!
There is .lazy.lua which adds some QoL features for development like colour previews and hot reloading. You'll need to be using lazy.nvim as your package manager for it to work properly.
You would have seen a prompt like this after cloning the repo and opening it for the first time:

You can get it to reappear by changing something in the file, like adding a comment etc.

Awesome, thanks for all your help.


(DON'T MERGE YET) UG edits and resolving add and find issues

Resolves the following issues:

#102
#138
#144
#157
#162

Also some minor standardization of UG and command messages

Codecov Report

Merging #164 (c1a7979) into master (e5ad85a) will increase coverage by 0.05%.
The diff coverage is 45.45%.

@@             Coverage Diff              @@
##             master     #164      +/-   ##
============================================
+ Coverage     68.38%   68.44%   +0.05%
  Complexity      689      689
============================================
  Files           108      108
  Lines          2293     2291       -2
  Branches        234      233       -1
============================================
  Hits           1568     1568
+ Misses          617      616       -1
+ Partials        108      107       -1

Impacted Files
Coverage Δ

...letubbies/logic/commands/ConfirmExportCommand.java
56.00% <ø> (ø)

...java/teletubbies/logic/commands/DeleteCommand.java
93.33% <ø> (ø)

...n/java/teletubbies/logic/commands/EditCommand.java
97.18% <ø> (ø)

...n/java/teletubbies/logic/commands/HelpCommand.java
100.00% <ø> (ø)

...java/teletubbies/logic/commands/RemarkCommand.java
100.00% <ø> (ø)

...n/java/teletubbies/logic/commands/UndoCommand.java
81.81% <ø> (ø)

...in/java/teletubbies/model/util/SampleDataUtil.java
25.00% <0.00%> (ø)

...ain/java/teletubbies/commons/core/UserProfile.java
80.76% <100.00%> (ø)

...in/java/teletubbies/logic/commands/AddCommand.java
100.00% <100.00%> (ø)

...n/java/teletubbies/logic/commands/DoneCommand.java
96.55% <100.00%> (ø)

... and 2 more

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 e5ad85a...c1a7979. Read the comment docs.


feat: add support for read-only connections

This PR implements a read-only mode, where a user would not be allowed to make changes to a given room.

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

Thank you very much, sorry for the delay!


Discussions

Hello, Michael
I'd like to chat, somewhere.  For now, this.
How do you run the hexe's on Windows for testing?  Am I missing something obvious?
And thanks.  :-)

Hi Brian,
Here's an invite to the Hopper Discord server (very quiet right now but a better platform for chat / discussion):
https://discord.gg/KK3wpfCh
hexe's run under the Hopper runtime. The current Hopper runtime on Windows is the HopperNET.exe in the distribution / release.
hexe's that are built for Windows will run in that runtime (like the shell, the editor, the compiler, etc).
Some features are platform specific though (like GPIO pins, SPI, I2C, etc) so those hexe's will only run properly on their target runtime, MCU's in this case.


Error! Failed to initialize.

When i click the install button in the WLED application, i get the following response:
Failed to initialize. Try resetting your device or holding the BOOT button while selecting your serial port.
I have a ESP8266 chip and i also tried ESP home and Tasmota, both gave the same issue.

What device?
Does it have a BOOT button?
Did you hold it while pressing install?

Hi,
I use the AZDelivery D1 Mini NodeMcu met ESP8266-12F and unfortunatly it doesn't have a BOOT button.
If i try to install something different, it works. I also tried with a brand new device, and get the same error message.

Does the serial port show up in the task manager? And is it still present when the failure appeared?
I've seen a few of those that had really flaky USB connectors.
You may also try a different USB cable and/or connect it directly to an USB port on the PC instead of via a hub.

I can see the device in my device manager with a COM port (currently COM 4).
I've tried a couple of USB cables and i can see the device in the device manager, so that should be good.
I've attached the specific error on the webpage.
The funny thing is that i've tried on a different computer and i get the same error.
Should i check the COM settings, like BAUT rate? if so, do you know what data i have to use?
Thanks in advance ;)

.

Hi, I probably applied too much heat to the board during soldering, so I've ordered some new ones and try it out on those. As soon as i know more, i'll let you know 😎

Closing this issue as this is based on an old version of ESP Web Tools.

Can you reopen the issue, it is still happening, on current version for me, with same model esp.

Please open a new issue and include logs + model and steps you did to reproduce.

Will do, no problem.
On Thu, Mar 17, 2022 at 12:59 AM Paulus Schoutsen @.***>
wrote:

Please open a new issue and include logs + model and steps you did to
reproduce.
—
Reply to this email directly, view it on GitHub
https://github.com/esphome/esp-web-tools/issues/118#issuecomment-1070327715,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4EKQUCVM7W2YCYA43LORTVAK347ANCNFSM5H7IFGQQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

Where did u opened it? I'm facing the same described above with the same hardware.

hallo ik heb een esp32 en ik probeer wled er op te downloaden maar elke keer als ik dat probeer geeft die aan dat ik de boot knop moet in drukken maar als ik dat doe verandert er niks. wat moet ik doen???

In my case it was lack of privileges for /dev/ttyUSB0.
Granting access to all ( sudo chmod 777 /dev/ttyUSB0) proves it's privileges issue.
Relevant devtools log on failure:
:673  DOMException: Failed to execute 'open' on 'SerialPort': Failed to open serial port. _handleConnectSerialSubmit @ wi

Thank you @MichalKrasowski this solved the issue for me.

Granting access to other (sudo chmod o+rw /dev/ttyUSB0) temporary solve the privileges issue. The 'x' permission is not required.
To permanently solve this, you must add yourself in the group associated to this /dev/ttyUSB0 file.
This can be done with the following command followed by a restart of you session: sudo usermod -a -G $(stat -c %G /dev/ttyUSB0) $(whoami)


Validate the efficiency for inflows and outflows for conversion assets

Description
As a result of discussion #432, we want to validate the efficiencies for inflows and outflows for the conversion assets so that only one efficiency takes effect (i.e., the other efficiency should be 1), otherwise the model throws an error.
Related Issues
#436

@suvayu feel free to add your thoughts

The implementation of the primary validations will be covered in #461. So, closing this issue 😉


Add MariaDB sequence support to the 3.10.x branch

This is basically a port of PR 1024, but for the 3.10.x branch.  Since MariaDB is effectively broken for MariaDB 10.3 and up, I don't think this should wait for 4.0.
I did use slightly different SQL than the original PR to account for permission problems in the innodb tables.

name: Add MariaDB 10.3 sequence suport
about: This fix will allow MariaDB users on 10.3 and up to use Liquibase.
title: ''
labels: Status:Discovery
assignees: ''

Environment
Liquibase Version: 3.10.x
Liquibase Integration & Version:
Liquibase Extension(s) & Version:
Database Vendor & Version: MariaDB 10.3 and up
Operating System Type & Version:
Pull Request Type

[x] Bug fix (non-breaking change which fixes an issue.)
[ ] Enhancement/New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
MariaDB added support for sequences, Liquibase knows about the support (MariaDBDatabase.supportsSequences returns true), but the SequenceSnapshotGenerator doesn't know how to get them, causing Liquibase to fail completely in that environment - even if there are no actual sequences in the database.
Steps To Reproduce
List the steps to reproduce the behavior.

Create a MariaDB 10.3 database.  Nothing needs to be in it, it just needs to exist.
Run the dropAll action against it.
(optional) Create a changelog with a createSequence change.
Run the update action against it.

Actual Behavior
Liquibase throws an exception at step 2.
Expected/Desired Behavior
Liquibase succeeds.  DropAll won't do anything, but that is fine.  Step 4 will result in a sequence being created in the database.
Screenshots (if appropriate)
If applicable, add screenshots to help explain your problem.
Additional Context
No new tests were added, but it doesn't look like any existing tests target MariaDB databases.  This change is being discussed in a sister PR 1024.
Fast Track PR Acceptance Checklist:

[x] Build is successful and all new and existing tests pass
[ ] Added Unit Test(s)
[ ] Added Integration Test(s)
[ ] Documentation Updated

Need Help?
Come chat with us on our discord channel
┆Issue is synchronized with this Jira Story by Unito
┆Fix Versions: Liquibase 3.10.2

Well, given the nice things MariaDB has said about us publicly, I think it would be just plain rude to not put this one at the front of the line, @nvoxland .
@stevesaliman, does this issue exist in SkySQL, too? Have you checked that out? Thanks!

Agree with @r2liquibase , pushing this as a low risk to 3.10.x

I merged this to a local LB-287 branch for easier internal testing


PromoterDB Header should be scrollable

Right now half of the UI in promoter db is locked on the header and there is less than half screen for the content.
The header should scroll with the rest, when scrolling the search should scroll as well up to the top where the search should be stick if the user continue to scroll.

Keeping the header visible, search box would be scrollable and be sticky under the header, image would hide. @ddelpiano


[Feature] Maintaining dynamic bar width based on the series data

What problem does this feature solve?
Is it possible to maintain the bar width dynamically based on series data availability?
What does the proposed API look like?
NA

An example of "series data availability" ?

Any example of "series data availability" ?

Example 1 -
series: [
{
name: 'Max',
type: 'bar',
data: [40,10,15,67,50,34,23,28]
}
]
Series array has 8 number of data. So, the bar count increase, gap between two bars adjusted as per this, bar width also maintained automatically.

Now if I remove some of the data from the series data array. As example below,
Example 2 -
series: [
{
name: 'Max',
type: 'bar',
data: [40,10,15]
}
]

Bar width increased. gap between bar also maintained proportionally. I want to keep the bar width maintained same in all the cases where ever data increase/decrease inside the series array. and the gap between bars can maintain a such way so that bars can be spread throughout the chart body all the time. Not only the left/right/center aligned.

There is no problem if Xaxis data remains the same - Demo Code.

There is no problem if Xaxis data remains the same - Demo Code.

Yes, We can do that like all bar is shifted to the left. The right side there is a blank spaces. So, I wanted to spread the bars through out the chart area. The spaces in between bars will increase/decrease. Example demo screenshot below,

But the width of bar is increasing that need to keep same.

use barWidth - Demo Code

But the width of bar is increasing that need to keep same.

use barWidth - Demo Code

Yes, to keep bar width same we can use barwidth property. But it will aligned in left. So, I was seeking any way out to spread the bars in total chart area and increase gap in between each bar.

How about some sample code, so I can stop guessing ?

There is no problem if Xaxis data remains the same - Demo Code.

is it possible to get this without setting extra labels in xAxis? I have added barWidth to reduce with of bar but i don't want big space between bars. it should be all aligned to left.

It is possible to control both barWidth and xAxis depending on series data.
But usually just updating the axis is enough, leaving barWidth to auto-resize.
Demo Code with dynamic data.


I see the problem now. Thanks for your patience. I should have remembered this one sooner.

I see the problem now. Thanks for your patience. I should have remembered this one sooner.
These are warnings generated by the Protobuf CocoaPod when it is built as a static library because of clang bug with file name casing - see https://github.com/google/protobuf/issues/3218.
It's an error for you because your Podfile is treating all warnings as errors.
Workarounds include allowing warnings for the Protobuf library build or adding use_frameworks! to the Podfile.
Originally posted by @paulb777 in https://github.com/firebase/firebase-ios-sdk/issues/1419#issuecomment-399455175
it will give the warning with the import.
Solution=> Fix the warning by replacing  " &lt Firebase &gt"  with  'Firebase'

It's not clear what the issue is here since it quotes an old resolved issue.


build: use environment compilers and flags if defined

do not clobber compilers and CFLAGS if present in the environment
ensure LDFLAGS always include the required flags for libdisk

Actually the use of ?= doesn't work as for example Gnumake has a default value for CC. I'm going to revert that bit and tool overrides should be specified on the make command line: for example make CC=$(CC). Flag defines can have initial values in the environment which get appended to in the Makefile.
The LDFLAGS changes in the main Makefile stay as they're a good fix.


Error When Trying To Generate Secrets With Terraform

Hi I keep getting this error when I try to create new secrets with terraform
Error: Repo not found (server.repo_not_found)
I've been following the guide that is in the secrethub docs. Was wondering if someone could help me figure out what's going on?
Here is my terraform
provider "aws" {
region = var.region
}
provider "secrethub" {
credential = file("~/.secrethub/credential")
}
terraform {
backend "s3" {
bucket = "poc-terraformstate"
key = "poc/tirgus-infrastructure/"
region = "us-east-2"
}
}
data "aws_caller_identity" "current" {}
locals {
secrethub_dir = "tirgus-api/poc"
}
resource "secrethub_secret" "db_password" {
path = "${local.secrethub_dir}/db/password"
generate {
length      = 22
}
}
resource "secrethub_secret" "db_user" {
path  = "${local.secrethub_dir}/db/user"
value = "schema_user"
}
resource "aws_db_instance" "postgres_instance" {
allocated_storage = 20
storage_type = "gp2"
engine = "postgres"
engine_version = "9.6.3"
instance_class = "db.t2.micro"
name = "tirgusApiDb"
identifier = "tirgus-api-db"
username = secrethub_secret.db_user.value
password =  secrethub_secret.db_password.value
publicly_accessible = true
final_snapshot_identifier = "tirgus-api-db-final-SNAPSHOT"
parameter_group_name = "default.postgres9.6"
}
secrethub version 0.37.0, build 22adaf4
Terraform version 0.12.23
Sorry if this isn't the correct place to ask this question. I wasn't sure where I should ask though.
Thanks,
Ethan

Hi Ethan,
the secret resource creates the secret in the path is refers to, in your example tirgus-api/poc/db/password and tirgus-api/poc/db/user. It relies on the tirgus-api/poc repo and tirgus-api/poc/db directory to already exist.
We'll be adding resources for repositories and directories as well, so that you can manage those in Terraform. In the meantime, you can create them using the CLI with repo init and mkdir.
Hope this helps :smile:
You can ask questions in our community Discord and via<EMAIL_ADDRESS>, but also feel free to ask more questions here :)


Docs: Animate and colorize footer buttons

Current State:
The buttons in footer.html of the Meshery documentation are not colorized or animated, but remain static when users hover over them.

Related issue - https://github.com/layer5io/layer5/issues/255
Desired State:
Buttons in the documentation footer should behave like those used on layer5.io.

Contributor Resources

Meshery documentation site
Meshery documentation source
Instructions for contributing to documentation

Hi @ashiscs are you having trouble with this issue? I was looking at it and was thinking I could take it.


Redis: 'instance' samples don't include required field 'name'

Code snippets in client reference docs for update_instance (possibly create_instance) don't work properly because required string for instance name is not passed.
Expect test code like:
from google.cloud import redis_v1

client = redis_v1.CloudRedisClient()
paths_element = 'display_name'
paths = [paths_element]
update_mask = {'paths': paths}
display_name = 'UpdatedDisplayName'
instance = {'display_name': display_name, 'name': 'projects/<project-name>/locations/us-east1/instances/test-66'}
response = client.update_instance(update_mask, instance)

The resulting instance:
alternativeLocationId: us-east1-c
authorizedNetwork: projects/<project-name>/global/networks/default
createTime: '2019-03-08T01:24:52.103494599Z'
currentLocationId: us-east1-d
displayName: **UpdatedDisplayName**
host: x.x.x.x
locationId: us-east1-d
memorySizeGb: 2
name: projects/<project-name>/locations/us-east1/instances/test-66
port: xxxx
redisVersion: REDIS_3_2
reservedIpRange: x.x.x.x/yyy
state: READY
tier: STANDARD_HA

Recognizing that this code is autogenerated is there a way to update the snippets such that the update persists other client changes?

Docs link
Proto description

cc/ @rickyye14

I have submitted a change to update the sample configuration. A future synth should bring this change.

https://github.com/googleapis/python-redis/pull/5


add optional shorthand to full refresh command

resolves #5878
Description
Adds an optional shorthand for the --full-refresh flag!
dbt build -fr for an incremental model passes:

and acts as expected:
16:11:54.082634 [debug] [Thread-7  ]: Began running node model.davebt.incremental
16:11:54.083352 [info ] [Thread-7  ]: 4 of 5 START sql incremental model dbt_dconnors.incremental .................... [RUN]
16:11:54.085262 [debug] [Thread-7  ]: Acquiring new snowflake connection "model.davebt.incremental"
16:11:54.085788 [debug] [Thread-7  ]: Began compiling node model.davebt.incremental
16:11:54.086130 [debug] [Thread-7  ]: Compiling model.davebt.incremental
16:11:54.104372 [debug] [Thread-7  ]: Writing injected SQL for node "model.davebt.incremental"
16:11:54.105039 [debug] [Thread-7  ]: finished collecting timing info
16:11:54.105302 [debug] [Thread-7  ]: Began executing node model.davebt.incremental
16:11:54.153305 [debug] [Thread-7  ]: Writing runtime sql for node "model.davebt.incremental"
16:11:54.154408 [debug] [Thread-7  ]: Using snowflake connection "model.davebt.incremental"
16:11:54.154609 [debug] [Thread-7  ]: On model.davebt.incremental: /* {"app": "dbt", "dbt_version": "1.3.0b2", "profile_name": "dbt_metrics_integration_tests", "target_name": "snowflake", "node_id": "model.davebt.incremental"} */
create or replace transient table development.dbt_dconnors.incremental  as
        (

with

final as (

    select * from development.dbt_dconnors.mock_incrementing_source

)

select * from final
        );

Opening as a draft for now, assuming I need some testing here, but didn't find a test that quite looked like the right place at first glance!
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[x] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[ ] I have run changie new to create a changelog entry

Personally, I like -fr ("for real"), though I could also see an argument for -f ("force," a la git push)

I would be happy with just updating one of the existing tests/functional tests that do "--full-refresh" with the new shortcut, such as tests/functional/materializations/test_runtime_materialization.py


Operation Cancelled on CheckConnection

Hello,
So I'm using a remote mongoDB Atlas service to host my mongo cluster, and the connection string is in this format:
mongodb+srv://user:pass@cluster-id.mongodb.net/database?retryWrites=true&w=majority.
I have tried using multiple variations of it within the UseMongoClient method, and it does goes fine up to the CheckConnection method, where it throws a ThrowOperationCancelledException, when reaching this piece of code on the MongoStorage class:
               try                 {                     _dbContext.Database.RunCommand((Command<BsonDocument>)"{ping:1}", cancellationToken: cts.Token);                 }                 catch (Exception e)                 {                     throw new MongoConnectException(_dbContext, CreateObscuredConnectionString(), e);                 }
The exception message is not very descriptive, just says:

The operation was canceled.

Here is the exception stack trace:
   at System.Threading.CancellationToken.ThrowOperationCanceledException()    at MongoDB.Driver.Core.Clusters.Cluster.WaitForDescriptionChangedHelper.HandleCompletedTask(Task completedTask)    at MongoDB.Driver.Core.Clusters.Cluster.WaitForDescriptionChanged(IServerSelector selector, ClusterDescription description, Task descriptionChangedTask, TimeSpan timeout, CancellationToken cancellationToken)    at MongoDB.Driver.Core.Clusters.Cluster.SelectServer(IServerSelector selector, CancellationToken cancellationToken)    at MongoDB.Driver.MongoClient.AreSessionsSupportedAfterServerSelection(CancellationToken cancellationToken)    at MongoDB.Driver.MongoClient.AreSessionsSupported(CancellationToken cancellationToken)    at MongoDB.Driver.OperationExecutor.StartImplicitSession(CancellationToken cancellationToken)    at MongoDB.Driver.MongoDatabaseImpl.UsingImplicitSession[TResult](Func2 func, CancellationToken cancellationToken)
at MongoDB.Driver.MongoDatabaseImpl.RunCommand[TResult](Command1 command, ReadPreference readPreference, CancellationToken cancellationToken)    at Hangfire.Mongo.MongoStorage.CheckConnection() in D:\Users\Felipe\Documents\Source\Repos\Hangfire.Mongo\src\Hangfire.Mongo\MongoStorage.cs:line 79
It'd be awesome if somebody could help shed a light on this. Thank you!

We are having the same exact issue as described above, unfortunately also I cannot get more information to share, as the exception doesn't tell much

Hi,
I'm currently on holiday. Did you try to bypass the connection check?
On Mon, 15 Jul 2019, 12:23 Matteo<EMAIL_ADDRESS>wrote:

We are having the same exact issue as described above, unfortunately also
I cannot get more information to share, as the exception doesn't tell much
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sergeyzwezdin/Hangfire.Mongo/issues/195?email_source=notifications&email_token=ADC6URLFCM3EANZQJXFR7PTP7RF2NA5CNFSM4IBVN35KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZ5I7PA#issuecomment-511348668,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADC6URNZPTE6R42DWY27I6DP7RF2NANCNFSM4IBVN35A
.

hello @gottscj,
by bypassing the check the connection is then ok. I cloned the repo and I'll try to see  what it is about
have good holidays!

Ok, it's just the cancellationToken that is too short, as it's set to 1 second. MongoAtlas in some situations can reply to a ping after 1 second, hence the command gets cancelled, throwing the exception
MongoConnectException(_dbContext, CreateObscuredConnectionString(), e);
On MongoStorage.cs:75 maybe the timeout for the cancellationToken should be configurable?

@uublive Did increasing the timeout solved it for you? For me it didn't matter if it was 1 or 5 or 10, the command still got cancelled in less about 1 second. If I connected to atlas with MongoDB Compass, it connected just fine.

Yes, as a test i increased it to 5 seconds and it went through. I'm still keeping the check disabled though.
I think you can safely disabled the check, as it's just a { ping } command anyway
On 16/07/2019 16:10:37, Felipe Amaral<EMAIL_ADDRESS>wrote:
@uublive [https://github.com/uublive] Did increasing the timeout solved it for you? For me it didn't matter if it was 1 or 5 or 10, the command still got cancelled in less about 1 second. If I connected to atlas with MongoDB Compass, it connected just fine.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub [https://github.com/sergeyzwezdin/Hangfire.Mongo/issues/195?email_source=notifications&email_token=AAOCW7LLWQWXU7V3FVBQNNTP7XJF3A5CNFSM4IBVN35KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2A7HAQ#issuecomment-511832962], or mute the thread [https://github.com/notifications/unsubscribe-auth/AAOCW7JTYGJZ436DEVK4TFDP7XJF3ANCNFSM4IBVN35A].

Same issue here. I have disabled the ping, for the time being.

@F-Amaral,
I increased the timeout to 5 seconds and made the timeout configurable.
Please let me konw if this sovles your issue. :)

closing


Create a Facade for the search.

There are Facades for Vimeo and Youtube.
These load a clickable image, so instead of the entire video loading, it loads an image with a play button, once clicked the remaining javascript loads. This allow the initial page load to complete much quicker.
The same concept can probably be applied to the search. (Dont actually load the search index or search javascript until user clicks into the search box)
vimeo facade: https://github.com/slightlyoff/lite-vimeo
youtube facade: https://github.com/justinribeiro/lite-youtube
I have also not implemented the facade for vimeo or youtube in Abridge yet, I test them locally and they do work, but my local implementation was violating the CSP. (will revisit this eventually)

I have this half completed, the search related javascript does not get loaded until the user clicks into the search box. After clicking in the search box you can see that the javascript for the search is then downloaded. You can also start typing and see the search suggestions popup and even click to one of the results.
The half that is not completed is the results page, if you type something for your search and hit enter or click the search icon, then the results page does not load.
This code is currently on this branch: https://github.com/Jieiku/abridge/tree/search_facade
Here is a gif showing it in action:

I just need to figure out what is stopping the search results page, I am sure it is something simple but I have not yet figured it out.

Figured it out, and it was something simple. I would previously use the window.onload event to bind the search function to the form submit event, when using a facade this event has already fired, the solution was to not wait for that event to bind the search function to the form id, instead I just add it as the last line in the javascript file.
Just need to do a bit more testing, also I don't think this will be usable for everyone because Keats (zola creator) has no desire to add the ability to bundle javascript files to Zola, essentially what this means is to make use of this facade you will need to issue a zola build followed by uglifyjs, and then another zola build. If there is a clever way of getting around this I would love to hear it.
After a bit of testing on this I will work to implement tinysearch next, maybe it will eliminate some of these shortcomings since we would no longer be using elasticlunr.

This is now Completed, and you can see it live here: https://abridge.netlify.app/
The end result is that with the search related javascript removed the site became substantially lighter, only 2kb of JS on page load!
There is however a catch, the facade uses a bundle that has all js related to the search in a single js file, this ensures that the files are in the proper order and that once the facade downloads the file the search is ready to use.
to generate this file you have to do this:
zola build
uglifyjs public/search_index.en.js public/elasticlunr.min.js public/search.js -o static/search_bundle.min.js -c -m

If this is not something you want to have to do then you can not use  the facade by commenting out the facade line in the config.toml:
#js_search_facade = "search_facade.js" # Do not load search until user interaction with search. [comment out to disable]
```shell

I will update the readme next and put emphasis on this, as by default I have the facade enabled so that it can be showcased on the demo site.

If Zola ever adds the ability to bundle js files then this will become much easier.

Found several pitfalls on this one. I think I have most of them addressed, also updated the readme.


Error:(169) Attribute "listPopupWindowStyle" has already been defined

添加依赖后发生这个错误
Error:(169) Attribute "listPopupWindowStyle" has already been defined

I'm Having the same issue here

just found the solution
add the library to your gradle as follow
compile 'com.zhihu.android:matisse:0.5.0-alpha4'
and make sure you are using an up to-date build tools version
i have moved my build tools version from
23.0.1 --> 25.0.2

@molhamstein  thanks,i chose the other libraries now


Breaking changes between versions

Hi Luke,
Thanks for the amazing job you are doing for Hero.
Since yesterday I started to use Hero for one of my apps and I'm having problem with a transition, also it performs differently between versions.
What I would like to do is: scale a view to full screen and having some cards sliding from the bottom.
I tried several options:
One is having all the view available from the source view: so no modifiers involved.
With 0.1.8 it works fine and this is the result:
https://cl.ly/1z2i2r1K3Y1O
With 0.1.9 and 0.2.0 there is a little flash in one of the white cards:
https://cl.ly/0o1A3V3O0e3m
With 0.3.0 took from master this is what happens (same as before + hiding on the way back) and there is also an alpha transition of the source view on the way back:
https://cl.ly/3e2S3n1k0V15
Then I tried removing the white cards from the source view and using the translate modifier and the transition of the white cards disappear:
https://cl.ly/1B160n182p0H
If i use the zPosition I can see the transition only on the way back:
https://cl.ly/36401t1C0u0Q
Ideally I was hoping to use, the latest version with the modifier. not having to duplicate the view inside the source and without "the alpha animation" introduced in that version, but looks like the only way I could get it work it's with the 0.1.8 and having the duplicate views.
I really would like to help and to collaborate to the project, but I still don't have enough knowledge of Hero. If you could hint me on what I'm doing wrong or try something else, or if you think it's a bug where I should investigate more, maybe I can send a PR for that :)
Anyway thanks a lot, for taking the time for reading this.

I could make it work with the latest version (0.3.0) doing a series of changes:
The background view was a separated view, now it's the view of the destination controller, that solved the alpha transition noticed on the dismiss.
I removed the copied views from the source and replaced with the translate modifier. That gave an animation for both way (presenting and dismissing) but the line was not straight. So I tried to use the useGlobalCoordinateSpace, that fixed the straight line on presenting, but on the dismissing the white cards just disappear, to fixed that I had to add zPosition modifier when dismissing.
Maybe when using useGlobalCoordinateSpace, something wrong happens with the zPosition and I have to set on dismissing?

First of all. Thanks for such a detailed feedback! Glad that some one is testing this out. Hero is still in beta under heavy development, there are two more features I would like to add to the library before making a stable version 1.0. Currently, a good stable version to fallback to would be 0.1.7 if you so desire. But I would love to get your guys' feedbacks on newer versions, I will try to respond quickly to solve any bugs you guys are experiencing.
Let me explain the effects you are seeing and why they work/not work. I will just target 0.3.0 here.

The alpha fade when your background view isn't the root view. This effect is the default animation provided Hero. If your destination/source root view doesn't have animation specified, Hero will perform its default animation on source/destination root views. You can disable this behavior by calling

Hero.shared.setDefaultAnimationForNextTransition(.none)

Or
Hero.shared.disableDefaultAnimationForNextTransition()

Or just specify some custom animations to either root views, like what you did here changing the background view to the root view.

You shouldn't need the copied views in your initial VC. Though they should work. The glitch you are seeing with the white views can be eliminated by setting the isOpaque property to false on your white views. Currently, Hero reads the isOpaque property to determine if it need to do a cross fade. If both source/destination view's isOpaque property is set to true, then it will just overlay the destination view on top of the source view during animation. Hero does this because it assumes the destination view can fully cover the source view. But in your case, it doesn't, the some white views are transparent and cannot cover the source views. In the next version, I will have Hero check the alpha properties, too.

The zPosition modifier shouldn't be needed in your case with useGlobalCoordinateSpace. might still be a bug. Would love to get my hands on your project if that is possible. Let me try see whats wrong.

Thanks for clarifying about the default animation 👍
I was researching a bit more about the bug. It looks like the view hierarchy is wrong while animating with global coordinates.
Look here:
The selected view is the background, while the three views before are the white cards

Instead without the .global I get the white cards views inside the background view.

I'll try to recreate this case if you think it can help

Wait, I just saw your latest commit :) let me try that

Please check if the latest master fixes this problem. Help me create
another issue if it doesn't.
On Sat, Feb 11, 2017 at 1:39 PM Alfredo Delli Bovi<EMAIL_ADDRESS>wrote:

Thanks for clarifying about the default animation 👍
I was researching a bit more about the bug. It looks like the view
hierarchy is wrong while animating with global coordinates.
Look here:
The selected view is the background, while the three views before are the
white cards
[image: screen shot 2017-02-11 at 19 30 57]
https://cloud.githubusercontent.com/assets/67916/22856423/661dc646-f091-11e6-807d-73230dcf728f.png
Instead without the .global I get the white cards views inside the
background view.
[image: screen shot 2017-02-11 at 19 32 56]
https://cloud.githubusercontent.com/assets/67916/22856422/65fb0b7e-f091-11e6-8a34-2f9a875e5f33.png
I'll try to recreate this case if you think it can help
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/lkzhao/Hero/issues/84#issuecomment-279166433, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADNEahNbv9o8JjTrk02ka3WRQeEC_89Bks5rbgBngaJpZM4L-JF_
.

Just tried out, the bug persist. Do you want me to close this issue and create a new one?

Yea sure!
On Sat, Feb 11, 2017 at 1:47 PM Alfredo Delli Bovi<EMAIL_ADDRESS>wrote:

Just tried out, the bug persist. Do you want me to close this issue and
create a new one?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/lkzhao/Hero/issues/84#issuecomment-279166966, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADNEaqcSWvkwBjQNX5hE7Px9ue8CkrVVks5rbgImgaJpZM4L-JF_
.

Ok thanks!


Initialize project structure

We need to pick the tech stack and initialize the repo with the project structure corresponding to that tech stack.
Describe the solution you'd like
The repo should be organized into directories, using the approach appropriate for the chosen tech stack. I think React JS client with a Flask (python) server side is a reasonable solution. @AbhilashKotha @loganwyas @Sai9797 What do you think on this?

Although I don't know React, I am very interested in learning it and believe that this will work for the project.

Resolved with PR #11


Avoid escaping quotes in quotemarks rule

Mostly enforce single quotes, but if you're using single quotes in the string, like in 'I\'m green, da ba dee', you should instead use double quotes: "I'm green, da ba dee".

👍


Installation Error:  Building wheel for scikit-learn (pyproject.toml) did not run successfully.

Dear all,
I am trying to install ojd-daps-skills via the command
pip install ojd-daps-skills
however, the installation stopped due to errors in building wheels for scikit-learn.  The error message is rather long, see here:
error_log.txt
I extracted some as follows:
...
Building wheels for collected packages: scikit-learn
Building wheel for scikit-learn (pyproject.toml) ...
error: subprocess-exited-with-error
× Building wheel for scikit-learn (pyproject.toml) did not run successfully.
...
I have tried upgrading pip, cython, setuptools but the error still persisted.
The environment where the error occured:
Python 3.9.13,
Windows 10,
Cython version 3.0.0,
pip 23.2.1
I also tried installing it in Google Colab and the problem persists.
Most grateful for any helps.  Thanks !

Hi @SC-TO - thanks for opening this issue!
It looks like this issue might have been alerted to sklearn recently too - see this
Looks like using an older version of cython might help things?
Do let us know if you found a solution that worked from that sklearn issue thread and we can update the docs. Thanks!

Is there a reason to restrict numpy, scipy, and scikit-learn for our ojd package, while packages depending on them (notably scikit-learn for transformers/ sentence-transformers packages) don't restrict them?
On my M1 Mac with Python 3.8, the minimum version for scikit-learn that doesn't run into this issue is 1.2.2, and not restricting scikit-learn results in installing 1.3.0 .
I wonder if expanding on integration gh-action workflows would be helpful in the future, since there isn't tests for other environments besides from ubuntu and our local development (osx-arm64).
It also looks like the only available archives for scikit-learn 0.23.2 are via conda-forge (for Python 3.8 and 3.9).
Note, awscli==1.27.25 also has a cython related issue on compilation due to it's PyYAML dependency. It could be better practice to not install awscli via pip, since it often has multiple conflicting dependencies.

Got the same issue on M1 even trying the open PR of @lizgzil